Update splunk-otel-javaagent to latest

Update splunk-otel-javaagent to v2.5.0
Update libsplunk.so to support OTEL_EXPORTER_OTLP_PROTOCOL and OTEL_METRICS_EXPORTER
Update linux installer script:

Change default for --otlp-endpoint to empty and sets OTEL_EXPORTER_OTLP_ENDPOINT only if non-empty
Add --otlp-endpoint-protocol option (default empty) and sets OTEL_EXPORTER_OTLP_PROTOCOL only if non-empty and --otlp-endpoint is also specified
Add --metrics-exporter option (default empty) and sets OTEL_METRICS_EXPORTER only if non-empty

v2 of the java agent changed the default otlp protocol from grpc to http/protobuf (see https://github.com/signalfx/splunk-otel-java/releases/tag/v2.0.0-alpha).  This will break our installer script and deployment modules since they configure the otlp endpoint as http://<IP_ADDRESS>:4317 (grpc) by default.  We'll need to add support for the OTEL_EXPORTER_OTLP_PROTOCOL env var to the installer script, deployment modules, and libsplunk.so to align with the configured endpoint.

Basically, the installer script will set OTEL_EXPORTER_OTLP_ENDPOINT, OTEL_EXPORTER_OTLP_PROTOCOL, and OTEL_METRICS_EXPORTER only if the respective command-line options are explicitly specified, otherwise we'll leave them unset and defer to the default values of each activated SDK.

Quick question: I was unable to find the OTEL_METRICS_EXPORTER exporter configuration for .NET instrumentation here https://docs.splunk.com/observability/en/gdi/get-data-in/application/otel-dotnet/configuration/advanced-dotnet-configuration.html. In this case, would the variable be set to none? I don't expect that to be an issue but want to confirm cc @pjanotti.

Quick question: I was unable to find the OTEL_METRICS_EXPORTER exporter configuration for .NET instrumentation here https://docs.splunk.com/observability/en/gdi/get-data-in/application/otel-dotnet/configuration/advanced-dotnet-configuration.html. In this case, would the variable be set to none? I don't expect that to be an issue but want to confirm cc @pjanotti.

OTEL_METRICS_EXPORTER will only be set to none if the user explicitly specifies the --metrics-exporter none installer script option.

Quick question: I was unable to find the OTEL_METRICS_EXPORTER exporter configuration for .NET instrumentation here https://docs.splunk.com/observability/en/gdi/get-data-in/application/otel-dotnet/configuration/advanced-dotnet-configuration.html. In this case, would the variable be set to none? I don't expect that to be an issue but want to confirm cc @pjanotti.

According to the upstream doc, the default for OTEL_METRICS_EXPORTER  is otlp. I assume that this is carried over to our distro?

Discussed internally, this is good to go.


Specify Hornet net config supplied by .env

Add the ability to specify Hornet net config JSON file that is supplied by the .env.
This is to avoid direct editing the file docker\docker-compose.yml.
So, in the .env, a new param HORNET_CONFIG_PATH is introduced.

This commit is still unsigned 😆


2D corner collision issue

Godot version:
3.1
OS/device including version:
Windows 10
Issue description:
If you create a wall composed of multiples shapes, with tilemap for example, and then you hit the intersection between 2 shapes with a ball, the physics engine will considere that the ball should bump in a corner, even if the corner is filled by the adjacent shape.
This basically makes some 2D ball based games impossible to create.
Steps to reproduce:
Just run the project below with collision shapes visible
Minimal reproduction project:
https://github.com/sollidsnake/godot-corner-collision-issue

Related to #32182

Related to #32182 (probably)


response preview doesn’t pretty print JSON anymore

With the recent switch from ScalarCodeBlock to CodeMirror we might have lost the pretty print for JSON responses.

They may release a quick fix soon; however, if you want to resolve this issue, you can revert to version 1.25.66. It uses the latest release by default, which can sometimes break the styling
// specify the version. I use NextJS
ApiReference({
    cdn: 'https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>    spec: { content: json },
    hideDownloadButton: true,
    darkMode: true
})


Handle network issues

Resolved in #9 and #https://github.com/flacial/nyaasi-bot-tg/pull/11

What about closing the issue when it's resolved? :)

What about closing the issue when it's resolved? :)

@fiestaLake It's left open because I didn't resolve all the network issues. It kind of works as a remainder.


feat(dialog-service): refactor consolidate open methods in one

Starting point for #224, #223
Will work on the tests after we clear the new open API.

@EisenbergEffect @PWKad What do you think about the .open method API? Is returning a structure like { openResult: Promise<OpenDialogResult>, closeResult: Promise<CloseDialogResult> } too unfriendly? Do you prefer an API where you pass a setting to control the output:
open(settings: DialogSettings & { yieldController: false | undefined }): Promise<CloseDialogResult>;
open(settings: DialogSettings & { yieldController: true }): Promise<OpenDialogResult>;

Or you have something else?

I was trying to find where I had mentioned this before but combing back through issues and gitter I couldn't.  I prefer the second that you had brought up before.
So the usage would be something like -
export class MyViewModel {
  submit(){
    this.dialogService.open({ viewModel: Prompt, model: 'Good or Bad?'}).then(response => {
      if (!response.wasCancelled) { console.log('good'); }
    });
  }
}

And with the second flag
export class MyViewModel {
  submit(){
    this.dialogService.open({ viewModel: Prompt, model: 'Good or Bad?', yieldController: true }).then(controller => {
      controller.result.then(response => {
        if (!response.wasCancelled) { console.log('good'); }
      }
    });
  }
}

Right?

@PWKad On gitter, but I was left with the impression that you preferred the other solution, good thing I asked again 😅 For this signature though we will need #217 to properly type it, at least I don't see how to do the overloads with the current generator.
I'll write back more questions after pushing the changes.

@PWKad Updated the .open() API as discussed + tests. A couple of questions:

naming
new interfaces: OpenDialogResult, CloseDialogResult - comments?
OpenDialogResult#closeResult or just result(too much result 😕 ). Will it be confusing/misleading if we name it output also(DialogResult/CloseDialogResult)?
can you comment on #223 because I've designed OpenDialogResult(#wasCancelled) so that .canActivate() => false is considered a cancellation.

@StrahilKazlachev anything we need to answer before merging this?

@PWKad Will add some more tests, update readme and sample/example.

Going to merge this first thing in the morning, please let me know if there is an order to merge in.

@PWKad no order for this one.


Remove unused class weekday-schedules-height-span from src/css/weekday.styl

Commit 267a5504d26434 removes the usage of CSS class weekday-schedules-height-span from src/js/view/template/week/weekdayInWeek.hbs .  The class is still generated by css/weekday.styl.

CSS class grid-date-title is also generated by stylus, but not used.

src/js/view/template/month/more.hbs adds class month-more-allday  to the generated HTML, but this class is not defined  or used anyhow.  Adding the class serves no purpose.

js/view/template/popup/scheduleDetailPopup.hbs adds class popup-detail-item to the generated HMTL.  This class is not defined by the stylus templates and not used anyhow by JavaScript. I recommend removing popup-detail-item from js/view/template/popup/scheduleDetailPopup.hbs .

src/js/view/template/month/weekdayInMonth.hb adds class weekday-grid-header to the generated HMTL, but this class is not used anyhow.

@dilyanpalauzov
Thanks for the report. I'll check it with the next update.

@dilyanpalauzov
I've checked it, and I think I should remove unused CSS styles in stylus files but not classes in the hbs files.
Those classes have semantic annotations that help to understand what the div(or else) tag means. And there might be a chance some users are using it for their custom styles.


Support for exist and list in FileSystem

@majetideepak  can you plz take another look?

@majetideepak thanks again! Addressed your comments


"Dumbrăvița", "Cărbunari", "Chechiș", "Rus", "Sindrești" and "Unguraș" are listed in the drop-down menu

Description:
"Dumbrăvița", "Cărbunari", "Chechiș", "Rus", "Sindrești" and "Unguraș" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Dumbrăvița" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Dumbrăvița, MARAMUREȘ (Dumbrăvița)" and "Comuna Dumbrăvița, MARAMUREȘ".

Step 3
Type "Cărbunari" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Cărbunari, MARAMUREȘ (Dumbrăvița)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Type "Rus" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Rus, MARAMUREȘ (Dumbrăvița)".

Step 8
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 9
Type "Sindrești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Sindrești, MARAMUREȘ (Dumbrăvița)".

Step 10
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 11
Type "Unguraș" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Unguraș, MARAMUREȘ (Dumbrăvița)".


ISSUE-11035: add ZH ElasticSearch index mapping

Describe your changes:
Fixes 11035

changes:

add zh index mapping json files under openmetadata-service/src/main/resources/elasticsearch
add ZH enum value to openmetadata-spec/src/main/resources/json/schema/configuration/elasticSearchConfiguration.json

test:

mvn clean install --DskipTests successfully
change env ELASTICSEARCH_INDEX_MAPPING_LANG to ${ELASTICSEARCH_INDEX_MAPPING_LANG:-ZH} in docker/development/docker-compose.yml. change docker image of elasticseach to elasticsearch-ik:7.11.2 with ik analyzer pre-integrated
docker compose -f docker/development/docker-compose.yml up --build -d
Re index all  with ZH language on page http://localhost:8585/settings/openMetadata/search/on_demand
search chinese words. and search results is as expected.

Type of change:

[ ] Improvement

Checklist:

[ x] I have read the CONTRIBUTING document.

[ x] My PR title is Fixes <issue-number>: <short explanation>

[ ] I have commented on my code, particularly in hard-to-understand areas.

[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

[ ] I have added tests around the new logic.

[ ] For connector/ingestion changes: I updated the documentation.

@heroWang Thank you for the patch. Can you please run  mvn googleformatter:format@reformat-sources to format the code and check-in those changes

mvn googleformatter:format@reformat-sources
Nothing changed after running this command

@harshach Nothing changed after running  mvn googleformatter:format@reformat-sources

@harshach Nothing changed after running  mvn googleformatter:format@reformat-sources

@harshach Nothing changed after running  mvn googleformatter:format@reformat-sources


Update event loop to latest Timer changes (fixes #56)

Unfortunately, failing on v0.4 because of recent changes that broke DataFrames.

https://travis-ci.org/JuliaStats/RCall.jl/jobs/67940138#L453-L472

Thanks, sorry it took so long


feat(react-native-payments): Expo plugins

This pull request includes several changes to the react-native-payments package, focusing on adding support for Expo and improving the payment plugin integration. The most important changes include adding new dependencies, updating the package configuration, and creating new plugins for Apple Pay and Google Pay.
Expo Integration and Plugin Support:

packages/react-native-payments/app.plugin.js: Added a module export to include the with-payments plugin.
packages/react-native-payments/readme.md: Updated the documentation to include instructions for integrating with Expo custom builds.

Package Configuration Updates:

packages/react-native-payments/package.json: Added new keywords, updated exports, included new files in the package, added new dependencies (@expo/config-plugins, fs, path), and updated peer dependencies to include expo. [1] [2] [3] [4] [5]

Plugin Implementations:

packages/react-native-payments/plugins/with-apple-pay.js: Created a new plugin to add Apple Pay support by modifying the iOS project files.
packages/react-native-payments/plugins/with-google-pay.js: Created a new plugin to add Google Pay support by modifying the Android project files.
packages/react-native-payments/plugins/with-payments.js: Combined the Apple Pay and Google Pay plugins into a single with-payments plugin.

Minor Changes:

packages/react-native-payments/src/index.ts: Added an empty line for better readability.

tested locally on my pc and it works ;)


[MINOR][DNM] Validate 0.14.0 RC3 bundles

Change Logs
As above.
Impact
Testing only.  Should not be merged.
Risk level
none
Documentation Update
N/A
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

Closing this testing-only PR.


Fixes the behaviour of gcontainer's __destruct_at_end.

The function __destruct_at_end shouldn't update __begin/__end otherwise an incorrect size() will be pass to deallocate() where some allocator may fail to free memory as expected.

Codecov Report

Merging #141 (8f30a46) into master (b60ce0c) will decrease coverage by 0.03%.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #141      +/-   ##
==========================================
- Coverage   69.92%   69.89%   -0.03%
==========================================
  Files          64       64
  Lines        5768     5767       -1
==========================================
- Hits         4033     4031       -2
- Misses       1735     1736       +1

Flag
Coverage Δ

app_tests
69.89% <ø> (-0.03%)
:arrow_down:

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

Impacted Files
Coverage Δ

grape/utils/gcontainer.h
85.81% <ø> (-0.10%)
:arrow_down:

... and 1 file with indirect coverage changes


Spelling/ Grammer on Front page.

Hey,
Wanted to point out that the front page elements have several spelling and grammar issues including "Avoid risk and grap your chancse" and "Enjoy playing EVE fly safe".
I figured it would be a quick fix and since it was pointed out by one my users while I was introducing it to the corp and fixing it will help to reinforce Pathfinder as a real and authentic mapping program :)
Sorry for the spelling hits, I know you have a lot more important things to work on. :)
Thanks.

No problem, can you post string replacements here ? It will help exodus4d fix the issue(s) quickly.
Thanks

I wasn't able to find that string in the code base, then realized that as
the words are not intractable I believe it is a static image. However I'm
having trouble figuring out which one it is as it is not treated as an
image by chrome.
Thanks.
On Sep 4, 2016 5:26 AM, "dkrotil"<EMAIL_ADDRESS>wrote:

No problem, can you post string replacements here ? It will help exodus4d
fix the issue(s) quickly.
Thanks
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/exodus4d/pathfinder/issues/308#issuecomment-244600722,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABr8UqbOM_O-mNvILpjrmf4Afc6HLee2ks5qmrkDgaJpZM4J0gXu
.

Sure, with strings I mean , write what is wrong and should be there like
"Avoid risk and grap your chancse" -> "Avoid risk and grab your chance"
Exodus4d  is from Germany, I´m from Czech Republic, English is not our native  language.

Alright, all the love to all of my euro friends. :) I will proofread the
full page later today, there is a lot of material and I would prefer to
provide an accurate set of corrections that only has to be applied once.
On Sep 4, 2016 12:28 PM, "dkrotil"<EMAIL_ADDRESS>wrote:

Sure, with strings I mean , write what is wrong and should be there like
"Avoid risk and grap your chancse" -> "Avoid risk and grab your chance
"
Exodus4d is from Germany, I´m from Czech Republic, English is not our
native language.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/exodus4d/pathfinder/issues/308#issuecomment-244623692,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABr8Uo0vRdd9_kB5631cmcG1ShDebed-ks5qmxvzgaJpZM4J0gXu
.

@cadetmcgregor Are you still planning to help with this Issue? - If not I´ll close it because I can´t solve this issue without help from a native English speaker.

Sorry,  this totally slipped my mind. Will take a look at it later today.

Hey @exodus4d possible to re-open this issue? I can provide some spelling/grammar fixes here if so.


[WIP] Add LibreJS support

https://github.com/gogits/gogs/issues/4084 I've finished the librejs template file, but I need some help with routes.
I'm not sure how do templates routes work and I don't want to make something wrong. I'm trying to make gogsbaseurl.com/librejs to contain the LibreJS page, but can make it work and I'm afraid to break something. Once that is done, I'll have no problem adding a link to the footer template. Could one active/experienced programmer in this project help me?
Sorry for my ignorance, I never programmed in Go and I'm having a hard time in understanding how the code works.

Does the URL has to be gogsbaseurl.com/librejs?

No, it could be in gobaseurl.com/otherfolder/librejs or in another place, but it should always be in the gobaseurl.com website.

Hmm, if it's not a fixed path, how does LiberJS know where to find this page?

In the footer.tmpl template, you must create a link pointing to that URL and also use data-jslicense="1". I didn't do that yet, because first I want to make the URL to work.
JavaScript licenses

I see. So for the HTML page you're trying to put, I think it is purely static and only for the purpose of LiberJS?
Then we could put it under public/assets/liberjs/liberjs.html, and make it a self-contained HTML page. To access that page by gobaseurl.com/assets/liberjs/liberjs.html. Sounds good?

@jorgesumle I love this PR: would you mind sending a similar one for https://github.com/go-gitea/gitea ?

Done. Thanks for your help @Unknwon. I'll watch this repo and update the LibreJS table every time a new JavaScript library is updated or added.
Apart from the changes you suggested, I changed the link of the source file to the tar.gz file because I was told that that was the correct approach in the LibreJS mailing list: http://lists.gnu.org/archive/html/help-librejs/2017-02/msg00003.html.

Looks good!

I'll watch this repo and update the LibreJS table every time a new JavaScript library is updated or added.

You might have some work soon regarding this PR: https://github.com/gogits/gogs/pull/4070 It's going to be merged with two new JS libs.

@jorgesumle I love this PR: would you mind sending a similar one for https://github.com/go-gitea/gitea ?
I don't think I have time right now. I would have to check its JavaScript files, install Gitea and test it. I made this contribution because I was a NotABug user and I use Gogs every day, but it's very tedious to make a pull request like this.
You might have some work soon regarding this PR: #4070 It's going to be merged with two new JS libs.
No problem with that. :)

FYI, the #4070 is merged, it's good to merge that PR before this one so you only need to verify your work once at this point.

OK, I added /plugins/marked-0.3.6/marked.min.js and /plugins/notebookjs-0.2.6/notebook.min.js from #4070 to the table of LibreJS. I used git push -f so that there is only one clean commit.

I used git push -f so that there is only one clean commit.

Actually we don't need to do that anymore because GitHub allows squash commits while merging.
Another question, since we're now link the license page for each JS lib, do we need to keep the license file (e.g. https://github.com/gogits/gogs/tree/master/public/plugins/marked-0.3.6) in our codebase anymore?

No. Here is an example from LibreJS documentation:

      all.min.js

          GPL-3.0+

          Apache-2.0

          gpl-script.js

          apache-script.js

As you can see the license links are external. However, I chose to link to the license file directly because it's inside Gogs (/plugins/marked-0.3.6/LICENSE). There is an example of that also in the LibreJS documentation:

More information on JavaScript Web Labels is detailed here: https://www.gnu.org/software/librejs/free-your-javascript.html#step3 and here: https://www.gnu.org/licenses/javascript-labels.html.

Thanks for the explanation!
In fact I would like to delete license files of JS libs from Gogs codebase and point to its license in liberjs.html file.
I can do that after merge the PR, or if you would like to help a bit and do it within the PR.

Can you please do that?

OK, I'll do it. Thanks again for the PR!

Just realized I keep typing liberjs where should be librejs... Anyway, it's now online on https://try.gogs.io/assets/librejs/librejs.html

Looks like a few scripts have been updated now but the Javascript licenses page no longer reflects that. I wonder if there's a way to automate this process during build? Gogs is broken in LibreJS.

Would love to see another PR to fix this... :trollface:


Move NetworkManager.UpdateScene code into OnSceneLoaded

I don't use scenes very much. tests are appreciated.

works fine with @MrGadget1024 's lobby example

Is this good to go?

@MrGadget1024 can you try this in your scene switching projects? you know about common errors by now :)

I think I'd like to follow up on the co-routines idea with Paul on this as part of other work in scene management.  Leaving this open for now but it may be superseded by another PR.


catch 301/302 in check-links. Close # 574

Deploy preview for landscape ready!
Built with commit 555e8954330fa5fa9fe700ced21e17cf9436bfe8
https://deploy-preview-577--landscape.netlify.com


Parser crash on "3*-"

Input:
3*-

See #65 for general information.

Also fixed by 2e878d257ac63e7ea60ec74dd67e6f0cf863a5a6


Add Typescript typings definition file (.d.ts) to npm

Would be great if you added a typings file to npm for your extension. Thx

Would you be willing to make a pull request for that?


Help with autostart

[x] I have read the README.md file and visited the Documentation to see if the information there helps.

Action description
I am trying to get mqtt-io to autostart.
II can run the application successfully with:
python3 -m mqtt_io /home/pi/Documents/mqtt-gpio-bridge/pi-mqtt-gpio.yml

However I can not get this service to start:
/etc/systemd/system/gpiomqtt.service
[Unit]
Description=MQTT GPIO service
After=multi-user.target

[Service]
Type=simple
User=pi
WorkingDirectory=/home/pi
ExecStart=/usr/bin/python3 /home/pi/.local/lib/python3.7/site-packages/mqtt_io/server.py /home/pi/Documents/mqtt-gpio-bridge/pi-mqtt-gpio.yml
SyslogIdentifier=gpiomqtt
StandardOutput=syslog
StandardError=syslog
Type=idle

[Install]
WantedBy=multi-user.target

This is the result of sudo systemctl status gpiomqtt
● gpiomqtt.service - MQTT GPIO service
   Loaded: loaded (/etc/systemd/system/gpiomqtt.service; enabled; vendor preset: enabled)
   Active: failed (Result: exit-code) since Mon 2022-01-24 21:24:32 AEDT; 4min 10s ago
  Process: 2879 ExecStart=/usr/bin/python3 /home/pi/.local/lib/python3.7/site-packages/mqtt_io/server.py /home/pi/Documents/mqtt-gpio-bridge/pi-mqtt-gpio.yml (co
 Main PID: 2879 (code=exited, status=1/FAILURE)

Jan 24 21:24:32 gpiomqttbridge1 gpiomqtt[2879]:     import enum
Jan 24 21:24:32 gpiomqttbridge1 gpiomqtt[2879]:   File "/usr/lib/python3.7/enum.py", line 2, in <module>
Jan 24 21:24:32 gpiomqttbridge1 gpiomqtt[2879]:     from types import MappingProxyType, DynamicClassAttribute
Jan 24 21:24:32 gpiomqttbridge1 gpiomqtt[2879]:   File "/home/pi/.local/lib/python3.7/site-packages/mqtt_io/types.py", line 5, in <module>
Jan 24 21:24:32 gpiomqttbridge1 gpiomqtt[2879]:     from typing import TYPE_CHECKING, Any, Dict, Union
Jan 24 21:24:32 gpiomqttbridge1 gpiomqtt[2879]:   File "/usr/lib/python3.7/typing.py", line 31, in <module>
Jan 24 21:24:32 gpiomqttbridge1 gpiomqtt[2879]:     from types import WrapperDescriptorType, MethodWrapperType, MethodDescriptorType
Jan 24 21:24:32 gpiomqttbridge1 gpiomqtt[2879]: ImportError: cannot import name 'WrapperDescriptorType' from 'types' (/home/pi/.local/lib/python3.7/site-packages
Jan 24 21:24:32 gpiomqttbridge1 systemd[1]: gpiomqtt.service: Main process exited, code=exited, status=1/FAILURE
Jan 24 21:24:32 gpiomqttbridge1 systemd[1]: gpiomqtt.service: Failed with result 'exit-code'.

I'm pretty much flying in the dark here, copying bits from other help requests like this: https://github.com/flyte/mqtt-io/issues/74#issuecomment-529181805
Unfortunately using ExecStart=/usr/bin/python3 /usr/local/lib/python3.5/dist-packages/pi_mqtt_gpio/server.py /home/pi/Documents/mqtt-gpio-bridge/pi-mqtt-gpio.yml did not work either:
pi@gpiomqttbridge1:~/system_sensors/src $ sudo systemctl status gpiomqtt
sudo: unable to resolve host gpiomqttbridge1: Name or service not known
● gpiomqtt.service - MQTT GPIO service
   Loaded: loaded (/etc/systemd/system/gpiomqtt.service; enabled; vendor preset: enabled)
   Active: failed (Result: exit-code) since Mon 2022-01-24 21:50:16 AEDT; 5s ago
  Process: 32465 ExecStart=/usr/bin/python3 /home/pi/.local/lib/python3.7/site-packages/mqtt_io/server.py /home/pi/Documents/mqtt-gpio-bridge/pi-mqtt-gpio.yml (c
 Main PID: 32465 (code=exited, status=1/FAILURE)

Jan 24 21:50:16 gpiomqttbridge1 gpiomqtt[32465]:     import enum
Jan 24 21:50:16 gpiomqttbridge1 gpiomqtt[32465]:   File "/usr/lib/python3.7/enum.py", line 2, in <module>
Jan 24 21:50:16 gpiomqttbridge1 gpiomqtt[32465]:     from types import MappingProxyType, DynamicClassAttribute
Jan 24 21:50:16 gpiomqttbridge1 gpiomqtt[32465]:   File "/home/pi/.local/lib/python3.7/site-packages/mqtt_io/types.py", line 5, in <module>
Jan 24 21:50:16 gpiomqttbridge1 gpiomqtt[32465]:     from typing import TYPE_CHECKING, Any, Dict, Union
Jan 24 21:50:16 gpiomqttbridge1 gpiomqtt[32465]:   File "/usr/lib/python3.7/typing.py", line 31, in <module>
Jan 24 21:50:16 gpiomqttbridge1 gpiomqtt[32465]:     from types import WrapperDescriptorType, MethodWrapperType, MethodDescriptorType
Jan 24 21:50:16 gpiomqttbridge1 gpiomqtt[32465]: ImportError: cannot import name 'WrapperDescriptorType' from 'types' (/home/pi/.local/lib/python3.7/site-package
Jan 24 21:50:16 gpiomqttbridge1 systemd[1]: gpiomqtt.service: Main process exited, code=exited, status=1/FAILURE
Jan 24 21:50:16 gpiomqttbridge1 systemd[1]: gpiomqtt.service: Failed with result 'exit-code'.

Can anyone shed any light on what I am doing wrong?
Hardware

Platform: pi3b
Connected hardware:  nil

System:

OS: Buster
Python version: [output of python --version]
User you're running as: pi
Using a virtualenv?: no

Additional context
Add any other context about the problem here.

This fails to start with the same error:
WorkingDirectory=/home/pi/Documents/mqtt-gpio-bridge
ExecStart=/usr/bin/python3 /home/pi/.local/lib/python3.7/site-packages/mqtt_io/server.py pi-mqtt-gpio.yml

Though that's where it appears to be:
pi@gpiomqttbridge1:~/.local/lib/python3.7/site-packages/mqtt_io $ ls
config              constants.py  exceptions.py      __init__.py  modules  __pycache__  tests     utils.py
config.sockets.yml  events.py     home_assistant.py  __main__.py  mqtt     server.py    types.py

This (the obvious that I should have tried first) works!
[Unit]
Description=MQTT GPIO service
After=multi-user.target

[Service]
Type=simple
User=pi
ExecStart=/usr/bin/python3 -m mqtt_io /home/pi/Documents/mqtt-gpio-bridge/pi-mqtt-gpio.yml
SyslogIdentifier=gpiomqtt
StandardOutput=syslog
StandardError=syslog
Type=idle

[Install]
WantedBy=multi-user.target


[Feature request] Sync decode method needed!

I use glide to download and cache the svga file.
When show svga, I use glide extent module to decode the svga file to SVGAVideoEntity, and the decode call is already in the background thread, so it's better to provide a sync metho to decode the svga file and return the SVGAVideoEntity as a result or throw an exception.

We will provide glide support later.

Sounds Good！
But i think there still need the sync decode method.
In my project, I  remove the anim code to SvgaDrawable, and control the anim start/stop also repeat part frames int the drawable. So I may need the method even you provide the glide integration.
By the way, when I write the Glide integration for svga, I find it is hard to get the svga source url when decode the svga InputStream, hope you have a better way.

If there is an unique id in the svga file property like width/height, then even if there is only inputstream, the parse method will know the path to uzip the cache filke.
Just like the issue i open #130 .
Hope to here your idea.

Any updates ?


Push triggered DNSConnections into an atomic queue to prevent DNSConnection lost.

The DNSConnection is put into NetHandler to wait for READ event.
NetHandler will call DNSConnection::triggered() to put it into DNSHandler::triggered queue when it is readable.
There are some issues if CONFIG proxy.config.dns.dedicated_thread INT 1:

The DNSHandler::triggered queue is not atomic queue
The DNSConnection::triggered() calls triggered.enqueue() directly without mutex locked.
The DNSHandler::recv_dns() calls triggered.dequeue() directly without mutex locked.

otherwise ET_NET == ET_DNS and only the NetHandler inside the [ET_NET 0] would get the DNSConnection event.

sorry, this not a bug.
ATS has create NetHandler for dedicated DNS thread.


Melissa to make Anna Ho's changes to tex

Melissa needs to do this by COB Tuesday March 1.

I have done this and DHW has the marked up tex. Some things outstanding from Anna's email that should be addressed (either now or at respond to ref e.g. for point 2):

Did not touch the Figure order (Figures are not referred to in order).
Units on plots - inconsistent in that sometimes there, sometimes not
Anna suggests stating we do not create ground truth we only propagate the training set (see below for more details).

I won't close this issue because we have the three things @mkness lists as still open issues.

no I am going to close  and open a new one.


How to handle types defined outside header without include

I'm trying out this project and am trying to find a way to generate bindings from GLFW.
GLFW does have 2 headers. glfw3.h that contains general type definitions etc. and glfw3native.h that contains some platform specific functions that expect as parameters some opaque types from glfw3.h. Unfortunately glfw3.h is not included in glfw3native.h. You are meant to include first glfw3.h and than glfw3native.h. How can I generate bindings for glfw3native.h? As it throws an error: error: unknown type name 'GLFWmonitor' what is logical as it is defined in glfw3.h but not included.

Nvm found it, I mis understood the traverse option, found the solution in https://github.com/dotnet/ClangSharp/issues/432

Nvm found it, I mis understood the traverse option, found the solution in https://github.com/dotnet/ClangSharp/issues/432


MCP

MCP does not work. I have the modified LawinServerV2 and nomcp off. Please help

its workin for me

I dunno what i am doing wrong


refactor(brave-search): rewrite

🔧 What does this fix? 🔧
Closes #713. Seems like a bunch of things changed and the vars weren't right anymore, started changing and decided to start fresh.
🗒 Checklist 🗒

[x] I have read and followed Catppuccin's contributing guidelines.
[x] I have updated the version appropriately in the ==UserStyle== header of the catppuccin.user.css file.

@ndsboy feel free to take over this PR seeing as you are the actual maintainer :D

Hi @uncenter, sorry for the late reply, GitHub did not send me a notification for this one...
Thank you very much for your rewrite, the base looks great, however I felt like changing the container color to base and adding a more prominent border aligns more with the default brave design and with the other designs.
I've also changed up the icon code a bit, so now the icon and tab color changes depending on your accent color (like reddit or youtube).
Please let me know what you think of these changes :)

Hi @uncenter, sorry for the late reply, GitHub did not send me a notification for this one... Thank you very much for your rewrite, the base looks great, however I felt like changing the container color to base and adding a more prominent border aligns more with the default brave design and with the other designs.
I've also changed up the icon code a bit, so now the icon and tab color changes depending on your accent color (like reddit or youtube).
Please let me know what you think of these changes :)

No worries! It is still marked as a draft pull request which is why you didn't get a notification for this. I totally agree with your changes (and this is your userstyle anyway xD)!


[ENH]: Switching to official Ollama python client

Description of changes
Summarize the changes made by this PR.

Improvements & Bug fixes

Ollama EF now uses the official ollama python client
Improved tests
Updated docs
The EF now uses Chroma's default EF (all-minilm-l6-v2) as a default

Test plan
How are these changes tested?

[x] Tests pass locally with pytest for python, yarn test for js, cargo test for rust

Documentation Changes
Python integration example updated

#2946  👈
main

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

Join @tazarov and the rest of your teammates on  Graphite


Ignore NuGet Packages

Reference Issue

What does this implement/fix? Explain your changes.

Ignore restored NuGet packages.
Any other comments?

The .gitignore rules are taken from default .gitignore for Visual Studio repositories:
# NuGet Packages
*.nupkg
# The packages folder can be ignored because of Package Restore
**/packages/*
# except build/, which is used as an MSBuild target.
!**/packages/build/
# Uncomment if necessary however generally it will be regenerated when needed
#!**/packages/repositories.config
# NuGet v3's project.json files produces more ignorable files
*.nuget.props
*.nuget.targets

A better way is to use VisualStudio.gitignore from github/gitignore.

Yes for sure.

A better way is to use VisualStudio.gitignore from github/gitignore.

Update PR with VisualStudio.gitignore


Data Box Archive Tier

It's unclear from documentation here https://docs.microsoft.com/en-us/azure/databox/data-box-how-to-set-data-tier what would happen if importing data via data box and copying it to the databox using azcopy utility with the archive flag. Would this be possible? OR are we required to copy data without any tier flags into a Hot tier container and then either use a lifecycle policy or run a script to change tier for every blob imported?
Azure documentation issue guidance
Thanks for opening an issue in the Azure technical documentation repository.
We use GitHub issues as the primary channel for customer and community feedback about the Azure documentation.
Creating an issue
We prefer that you create documentation feedback issues using the Feedback link on the published article - the feedback control on the doc page creates an issue that contains all the article details so you can focus on the feedback part.
You can also create a feedback issue here in the repo. If you do this, please make sure your issue lists:

[ ] The relevant Azure service or technology.
[ ] A link to the published documentation article that you have feedback about.
[ ] Clear, specific feedback that the author can act on.

Pull requests and article contributions
If you know the change that is needed in an article, we encourage you to submit the changes directly using a pull request. If the change is large, or if you want to contribute an entire article, follow these guidelines:

[ ] Don't surprise us with a big pull request or a pull request with a new article! Submit an issue that describes the details of the proposed large change or new article.
[ ] Include the service or technology area.

We'll route the issue to the appropriate content team for review and discussion.
Tech support and product feedback
If you would like to contact Microsoft about other things, such as product feedback or tech support, please review these guidelines:

If you need technical support using Azure, the paid and free support options are described here: https://azure.microsoft.com/support/options/.

Each article in the Azure technical documentation contains a product feedback button - it's best to submit product feedback directly from a relevant article. Otherwise, you can submit product feedback for most Azure products in the following product feedback forum: https://feedback.azure.com/d365community/forum/79b1327d-d925-ec11-b6e6-000d3a4f06a4.

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

ID: 86dd7fe5-a4df-a207-37c0-a30571388a51
Version Independent ID: 83316ef9-7986-f163-bcc1-8178e7014019
Content: Send data to Hot, Cold, Archive blob tier via Azure Data Box/Azure Data Box Heavy
Content Source: articles/databox/data-box-how-to-set-data-tier.md
Service: databox
Sub-service: pod
GitHub Login: @alkohli
Microsoft Alias: alkohli

@eugene-lipsky Thank you for your request here. We will investigate and update the thread .

@eugene-lipsky  Data Box currently does not support direct ingestion into the Archive tier. Data is copied to whatever is the default tier for the account. You can choose to use a lifecycle policy to spill to Archive.
Please note direct ingestion into Archive via Data Box is something we plan to support in an upcoming release but I do not have a specific date to share just yet.
If you wish to leave your feedback/UserVoice here. All the feedback you share in these forums will be monitored and reviewed by the Microsoft engineering teams responsible for building Azure.
https://feedback.azure.com/d365community/
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Updated about page (updated submission format and re-organized it a bit)

Updated the "About" page, please review.

restructured according to the new call page
updated and shortened the submission instructions

excellent, thanks! Merged with minor grammatical tweaks

BTW I had to tweak some things, but the call page is now online: https://iclr-blogposts.github.io/2024/call

the about page (https://iclr-blogposts.github.io/2024/about) is not yet online, I'm trying to debug this

clearing the cache fixed the issue.

Thanks a lot for the fixes!


[v2 regression] nerdctl start fails after restarting the host (msg="failed to lock state dir: unable to unmarshall lifecycle data: unexpected end of JSON input")

Description
nerdctl start fails after restarting the host
Steps to reproduce the issue

sudo nerdctl run -d --name foo busybox sleep infinity
Reboot the host

$ sudo nerdctl start foo
FATA[0000] 1 errors:
failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error running createRuntime hook #0: exit status 1, stdout: , stderr: time="2024-08-23T14:07:24+09:00" level=fatal msg="failed to lock state dir: unable to unmarshall lifecycle data: unexpected end of JSON input": unknown

Describe the results you received and expected
Received: error
Expected: starts
What version of nerdctl are you using?
$ sudo nerdctl version
Client:
 Version:	v2.0.0-rc.1
 OS/Arch:	linux/amd64
 Git commit:	778975fcaa57e365ee44dbad4a9d8d63180ae320
 buildctl:
  Version:	v0.0.0+unknown

Server:
 containerd:
  Version:	v2.0.0-rc.3
  GitCommit:	27de5fea738a38345aa1ac7569032261a6b1e562
 runc:
  Version:	1.2.0-rc.2
  GitCommit:	v1.2.0-rc.2-0-gf2d2ee5e

Are you using a variant of nerdctl? (e.g., Rancher Desktop)
None
Host information
$ sudo nerdctl info
Client:
 Namespace:	default
 Debug Mode:	false

Server:
 Server Version: v2.0.0-rc.3
 Storage Driver: overlayfs
 Logging Driver: json-file
  Cgroup Driver:  : systemd
  Cgroup Version: : 2
 Plugins:
  Log:     fluentd journald json-file syslog
  Storage: native overlayfs
 Security Options:
  apparmor
  seccomp
   Profile:	builtin
  cgroupns
 Kernel Version:   6.8.0-40-generic
 Operating System: Ubuntu 24.04 LTS
 OSType:           linux
 Architecture:     x86_64
 CPUs:             4
 Total Memory:     15.57GiB
 Name:             suda-ws01
 ID:               5097992c-2934-44bd-a1cb-5ac20cf2013f

I am getting different errors after reboot (lima VM):
$ sudo nerdctl start foo
FATA[0000] 1 errors:
failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error running hook #0: error running hook: exit status 1, stdout: , stderr: time="2024-08-22T23:29:54-07:00" level=error msg="failed re-acquiring name - see https://github.com/containerd/nerdctl/issues/2992" error="name \"foo\" is already used by ID \"f58b1233941da3daabdff9ce4befde5ce0fea208109dcb421b1985c311bb3109\""
time="2024-08-22T23:29:54-07:00" level=fatal msg="failed to call cni.Setup: plugin type=\"bridge\" failed (add): failed to allocate for range 0: <IP_ADDRESS> has been allocated to default-f58b1233941da3daabdff9ce4befde5ce0fea208109dcb421b1985c311bb3109, duplicate allocation is not allowed": unknown

Just repeating the command then succeed:
$ sudo nerdctl start foo
foo

You may repro the issue with https://github.com/lima-vm/lima/pull/2178/commits/fe7c317fd8741722949ed4885ba890f2f626aff0

Will try tomorrow with update lima + containerd v2rc.
Note:
Issue seems to have been here for some time somehow.
This is what I get with nerdctl 1.7 + ctd 1.7 on reboot:
sudo ./nerdctl start foo
FATA[0000] 1 errors:
failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error running hook #0: error running hook: exit status 1, stdout: , stderr: time="2024-08-22T23:40:46-07:00" level=fatal msg="failed to call cni.Setup: plugin type=\"bridge\" failed (add): failed to allocate for range 0: <IP_ADDRESS> has been allocated to default-ac9d84dbfc0b4873564385f72cde9efd872bc2608d185fb6414cd7f8aaee7de4, duplicate allocation is not allowed"
Failed to write to log, write /var/lib/nerdctl/1935db59/containers/default/ac9d84dbfc0b4873564385f72cde9efd872bc2608d185fb6414cd7f8aaee7de4/oci-hook.createRuntime.log: file already closed: unknown

sudo ./nerdctl version
Client:
 Version:	v1.7.6
 OS/Arch:	linux/arm64
 Git commit:	845e989f69d25b420ae325fedc8e70186243fd93
 buildctl:
  Version:	v0.12.5
  GitCommit:	bac3f2b673f3f9d33e79046008e7a38e856b3dc6

Server:
 containerd:
  Version:	v1.7.16
  GitCommit:	83031836b2cf55637d7abf847b17134c51b38e53
 runc:
  Version:	1.7.19
  GitCommit:	v1.1.13-0-g58aa920

Same thing here, on the second time you invoke start, it works.

The original error you saw ("failed to lock state dir") clearly points to:  https://github.com/containerd/nerdctl/blob/607d5600fb08bb74a79cd3efdcd13762005808a6/pkg/ocihook/state/state.go#L59
Although there is clearly more to it (as prior versions seems to fail as well), this one is suggesting that there is something wrong going on with filesystem locks / reboot.

Seems solved in the current main branch


Black screen

I get a black screen at the login prompt when i use the POST EFI. Any help would be appreciated. I have your exact PC configuration. Thanks

@daphyta was using the FHD EFI instead of UHD.
I'm closing this issue.

@daphyta was using the FHD EFI instead of UHD.
I'm closing this issue.


☂ Remove various _filename records.config variables.

[ ] proxy.config.cache.storage_filename
[ ] proxy.config.cache.control.filename
[ ] proxy.config.cache.ip_allow.filename
[ ] proxy.config.cache.hosting_filename
[ ] proxy.config.cache.volume_filename
[ ] proxy.config.dns.splitdns.filename
[ ] proxy.config.hostdb.filename
[ ] proxy.config.log.config.filename
[ ] proxy.config.url_remap.filename
[ ] proxy.config.ssl.server.multicert.filename
[ ] proxy.config.ssl.servername.filename

The argument is that

This doesn't fly well with the move to a unified YAML namespace
This functionality is superseded with the traffic layout / runroot features.

Blocks #5457

Moving out to 10.0.0, unless someone wants to work on this.

@zwoop I could.
Do you have any ideas here around the config file to config subsystem signaling here?


Debug console hangs when many lines are printed in Python code

Issue Type: Bug
When running the following piece of Python code, the debug console hangs after showing certain amount of lines in the debug console.
In the current example, it always tends to freeze around 14000.
for i in range(20000):     print(i)
VS Code version: Code 1.29.1 (bc24f98b5f70467bc689abf41cc5550ca637088e, 2018-11-15T19:13:36.375Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-4690 CPU @ 3.50GHz (4 x 3492)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
31.89GB (16.62GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (42)

Extension
Author (truncated)
Version

better-comments
aar
2.0.3

insertnumbers
Asu
0.9.1

multiline-find-and-replace
axl
0.0.1

file-properties-viewer
bru
0.2.0

insert-unicode
bru
0.2.1

bracket-pair-colorizer-2
Coe
0.0.25

dart-code
Dar
2.21.1

flutter
Dar
2.21.1

githistory
don
0.4.4

jupyter
don
1.1.4

gitlens
eam
9.0.3

toggle-excluded-files
eam
1.6.0

LogFileHighlighter
emi
2.4.2

json-tools
eri
1.0.2

path-autocomplete
ion
1.13.1

pubspec-assist
jer
0.3.1

nbpreviewer
jit
1.1.0

asciidoctor-vscode
joa
2.1.0

svg
joc
0.1.4

vscode-simple-icons
Lau
1.9.2

vs-color-picker
lih
0.7.0

code-fragments
mar
1.2.0

rainbow-csv
mec
0.8.0

vscode-todo-parser
min
1.9.1

ecdc
mit
0.12.0

python
ms-
2018.11.0

azure-account
ms-
0.6.2

cpptools
ms-
0.20.1

csharp
ms-
1.17.1

PowerShell
ms-
1.10.0

autodocstring
njp
0.2.3

vscode-code-outline
pat
0.2.1

vscode-docker
Pet
0.4.0

material-icon-theme
PKi
3.6.1

polacode
pnp
0.2.2

vscode-yaml
red
0.1.0

text-marker
ryu
1.9.0

vscode-smoothtype
spi
1.2.0

sort-lines
Tyr
1.7.0

reopenclosedtab
uyi
1.1.0

change-case
wma
1.0.0

highlight-trailing-white-spaces
yba
0.0.2

(2 theme extensions excluded)

Should be fixed in latest vscode insiders, we are using a new console.
Please try with https://code.visualstudio.com/insiders/ and if it is not fixed let us know and we will reopen

Hi @isidorn, I noticed that the debug console would cause plenty of hangs when it was opened (visible) after it passes a certain number of lines as well.
I installed the insiders version, and while the hangs seem to be much shorter than in v1.30, they're still a few seconds. It constantly took ~3s when switching from Debug Console to, for example, Output.

@dubeg please open a new issue with clear reproducable steps for the hang. Thank you


key error when using monoscene to visualize pkl

File "monoscene/scripts/visualization/kitti_vis_pred.py", line 169, in main
T_velo_2_cam = b["T_velo_2_cam"]
KeyError: 'T_velo_2_cam'

Thank you for your interest in our project. I apologize for the inconvenience caused by this issue.
In brief, you can fix the key error on lines 169-172 of the original MonoScene visualization script by modifying it as follows:
T_velo_2_cam = b['cam_pose']  # instead of b['T_velo_2_cam']
vox_origin = np.array([0, -25.6, -2])

y_pred = b['pred']  # instead of b['y_pred']

I'd like to mention that we haven't released the complete visualization scripts yet, as we are currently working on unifying the interface for visualizing various datasets. This is still an ongoing process, and we will update the visualization file as soon as it is completed.
Please let me know if you have any other questions! Once again, thank you for your interest in our project and for bringing this issue to our attention. Your feedback is valuable in helping us improve the project.


Automated Package Upgrade

:tada: This PR is included in version 7.7.356 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


cadl, refactor for multiple server

TODO: may see if I can move some function/class out of code-model-builder. The file already too long.


integrade IPPool docs

Signed-off-by: Icarus9913<EMAIL_ADDRESS>

@Jeanine-tw  update the EN doc


Support custom property mapping

Allow Zotero fields to be mapped to Notion properties with any desired name. Even better, pull the properties from Notion and let the user choose them instead of having to type the names.
Resolves #257

This would be wonderful!!

Notero could not modify the database property name now, which means that you can only use the default property name. That's not cool.

阮乐
@.***

------------------ 原始邮件 ------------------
发件人: "Zheng @.>;
发送时间: 2024年11月19日(星期二) 晚上9:17
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [dvanoni/notero] Support custom property mapping (Issue #355)
what are you guys talking about??? TTTTT
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


neba-139 MVC / Web Infrastructure: Refactor MvcServlet, MvcContext an…

#139

Coverage decreased (-0.002%) to 91.142% when pulling 88928b128e52beab03b588366e5511490b0b4e90 on neba-139 into 8d6ca806348000af9b21fbc9c0e6ae805190231d on develop.

Coverage increased (+0.09%) to 91.239% when pulling 5cea9342da8623524618480d723a319ca821f3c9 on neba-139 into 8d6ca806348000af9b21fbc9c0e6ae805190231d on develop.


FileNotFoundError when using CTRL + V from Clipboard feature on Load Image node in ComfyUI

Steps to reproduce :

Add Load Image node
Connect it's IMAGE output to JoyTag node
Instead of choosing an image from the file explorer, CTRL + C an image from the internet and CTRL + V on it while having the Load Image node selected.
Queue

Wait nvm, it works now, what the


feat: package.json overrides to enable Node16

implements #23

add support for differentation of output by outDir
(does not detect if tsconfig.json outDir matches an override)
add option to override top level properties in package.json
"type": "module" or "commonjs" controls default .js interpretation and tsc mode

Have since built own tsc wrapper with hooks to handle package.json reading.


set Indexes

How can I set Indexes for tables?

@gnuhel no support for that yet. Ill make this a feature request and plan to add it to a near-future release.


Unexpected token < - missing file lib/angular2-modal/plugins/bootstrap.js

I am facing some issues in loading angular2-modal popup. Getting this exception "Unexpected token < - missing file lib/angular2-modal/plugins/bootstrap.js" After compiling when I try to launch the application, it is trying to evaluate the file in the path
"Evaluating http://localhost:49928/lib/angular2-modal/plugins/bootstrap.js"
But in the specified location no such file exists and hence the application is getting crashed. Under the lib/angular2-modal/plugins/ folder, bootstrap.js file doesn't exists. But why is it looking for this file here?
Error: SyntaxError: Unexpected token <
        at eval (<anonymous>)
        at Object.eval (http://localhost:49928/app/AppModule.js:13:19)
        at eval (http://localhost:49928/app/AppModule.js:110:4)
        at eval (http://localhost:49928/app/AppModule.js:111:3)
    Evaluating http://localhost:49928/lib/angular2-modal/plugins/bootstrap.js
    Evaluating http://localhost:49928/app/AppModule.js
    Evaluating http://localhost:49928/app/main.js
    Error loading http://localhost:49928/app/main.js

SystemJS.Config.js file is as below
(function (global) {
    System.config({
        paths: {
            // paths serve as alias
            //'npm:': 'node_modules/'
            'npm:': 'lib/'
        },
        // map tells the System loader where to look for things
        map: {
            // our app is within the app folder
            app: 'app',
            // angular bundles
            '@angular/core': <EMAIL_ADDRESS>            '@angular/common': <EMAIL_ADDRESS>            '@angular/compiler': <EMAIL_ADDRESS>            '@angular/platform-browser': <EMAIL_ADDRESS>            '@angular/platform-browser-dynamic': <EMAIL_ADDRESS>            '@angular/http': <EMAIL_ADDRESS>            '@angular/router': <EMAIL_ADDRESS>            '@angular/forms': <EMAIL_ADDRESS>            // other libraries
            'rxjs': 'npm:rxjs',
            'angular2-in-memory-web-api': 'npm:angular2-in-memory-web-api',

            'angular2-modal': 'npm:angular2-modal'
         },
        // packages tells the System loader how to load when no filename and/or no extension
        packages: {
            app: {
                main: './main.js',
                defaultExtension: 'js'
            },
            rxjs: {
                defaultExtension: 'js'
            },
            'angular2-in-memory-web-api': {
                main: './index.js',
                defaultExtension: 'js'
            },
            'angular2-modal': {
                main: 'bundles/angular2-modal.umd',
                defaultExtension: 'js',
            }
        }
    });
})(this);

My package.json
 "dependencies": {
    "@angular/common": "~4.1.1",
    "@angular/compiler": "~4.1.1",
    "@angular/core": "~4.1.1",
    "@angular/forms": "~4.1.1",
    "@angular/http": "~4.1.1",
    "@angular/platform-browser": "~4.1.1",
    "@angular/platform-browser-dynamic": "~4.1.1",
    "@angular/router": "~4.1.1",
    "@ng-bootstrap/ng-bootstrap": "^1.0.0-alpha.6",
    "angular-in-memory-web-api": "~0.3.0",
    "systemjs": "0.19.40",
    "bootstrap": "3.3.7",
    "core-js": "^2.4.1",
    "rxjs": "5.0.1",
    "zone.js": "^0.8.4",
    "angular2-modal": "2.0.3"
  },
  "devDependencies": {
    "concurrently": "^3.2.0",
    "lite-server": "^2.2.2",
    "typescript": "~2.1.0",

    "canonical-path": "0.0.2",
    "tslint": "^3.15.1",
    "lodash": "^4.16.4",
    "jasmine-core": "~2.4.1",
    "karma": "^1.3.0",
    "karma-chrome-launcher": "^2.0.0",
    "karma-cli": "^1.0.1",
    "karma-jasmine": "^1.0.2",
    "karma-jasmine-html-reporter": "^0.2.2",
    "protractor": "~4.0.14",
    "rimraf": "^2.5.4",

    "@types/node": "^6.0.46",
    "@types/jasmine": "2.5.36",

    "gulp": "^3.9.1",
    "gulp-conventional-changelog": "^1.1.0",
    "gulp-git": "^1.12.0",
    "gulp-header": "^1.8.8",
    "gulp-shell": "^0.5.2",
    "gulp-sourcemaps": "^2.2.0"
  }

AppModule.ts
import { NgModule, ErrorHandler }         from '@angular/core';
import { BrowserModule }    from '@angular/platform-browser';
import { FormsModule }      from '@angular/forms';
import { HttpModule }       from '@angular/http';
import { NgbModule }        from '@ng-bootstrap/ng-bootstrap';

import { ModalModule } from 'angular2-modal';
import { BootstrapModalModule } from 'angular2-modal/plugins/bootstrap';
import { Overlay, overlayConfigFactory } from 'angular2-modal';
import { Modal } from 'angular2-modal/plugins/bootstrap';
import { DialogRef, ModalComponent, CloseGuard } from 'angular2-modal';
import { BSModalContext } from 'angular2-modal/plugins/bootstrap';
...

@NgModule({
    imports:
    [
        BrowserModule,
        FormsModule,
        HttpModule,
        ModalModule.forRoot(),
        BootstrapModalModule,
        LOGIN_ROUTER_PROVIDERS
    ],
    declarations:
    [
        AppLoader,
        Login,
        Home,
        ..
    ],
    providers:
    [
        ....
        {
            provide: ErrorHandler, useClass: CustomErrorHandler
        }
    ],
    entryComponents:
    [
        ....
        StoryBook, AddStoryBook
    ],
    bootstrap: [AppLoader]
})
export class AppModule { }

Any ideas why I am getting this error? Why it is looking for bootstrap.js file under angular2-modal folder?

Try version 3.0.0, use the UMD bundle directly not the module files.

Thanks for your reply.
use the UMD bundle directly not the module files
Sorry I am not understanding this, Can you please elaborate? what is the actual changes I need to do?

This is a SysteJS configuration issue.
Personally I suggest moving to webpack...
For SystemJS configuration you can compare your setup to this plunker setup and see what's wrong.
As you can see, plunker is working so it is a configuration issue at your end... closing.


Modify the source code to compile and run tests on mac with VS2017

TCK tests are not supported and need to be excluded from the project

Included in #310


How to handle dependencies that are only used in test?

I'm not sure how to handle test/support/.... files.
In my test/support I sometimes add things to the same module structure for example:
lib/foo:
module Foo do
end

test/support/factory.ex:
module Foo.Factory do
end

Now Boundaries complains that Foo.Factory is not allowed from an other file in test/support (for example a helper file to setup some domain data).
Ideally I think I want to be able to say to ignore some files, or be able to change the boundaries for tests.

I believe that changes proposed in #5 (in particular, the section "Unclassified modules") should address this. Let me know what you think.

I added the first draft for supporting some ignore mechanism. You can take a look at the idea here. Do you think something like this would suffice for this scenario?


[Idea]: Merge Two Sorted List

Idea Title
Merge Two Sorted List | Linked List
Idea Description
Merge the two lists into one sorted list. The list should be made by splicing together the nodes of the first two lists.
We are supposed to return the head of the merged linked list.
Input: list1 = [1,2,4], list2 = [1,3,4]
Output: [1,1,2,3,4,4]
Participation: gssoc-ext
Potential Benefits

[ ] Improves user experience
[ ] Enhances learning
[X] Adds a new feature
[ ] Improves code quality
[ ] Other

Implementation Suggestions (Optional)
No response

Hello
Do i have to create a new file e.g. mergeLL.md and add the code for merging two LL


nan during the first forward pass

I'm fine tuning the pretrained ECO Full architecture on my own dataset. The inference (deploy.protoxt) runs with no issues. The only difference in the two files is the last layer and the data layer.
I noticed in the first forward pass, res3b_bn blows up (inf). I'm wondering if this is an issue someone has seen before. I've attached the log file.
log.txt

Reinstalling caffe with the correct version of cudnn solved this problem.


Handle single CPU issue requiring detachment before casting to numpy

Was running into an error when trying this with only a single CPU. I don't fully understand the ramifications of this change, but it seems to work in a pinch.

No worries, happy to be the first person to try training this without a GPU


Allow styled mode (css) for highcharts

From my understanding, right now, react-jsx-highcharts imports the top level 'highcharts'. But as of version 5 there is a styled mode using css for separation of concerns purposes. And as per the highcharts documentation for Style by CSS, you need to load highcharts/js/highcharts.js instead of the top highcharts as you would usually do for the svg styled mode.
Now have I missed something in the configuration or is it actually something that doesn't exist yet?
I don't mind trying to open a PR and try to fix it from my end if you're okay with it.

Interesting, I assume that means we would lose the automatic theming?
My only concern would be is if 'highcharts/js/highcharts' plays nicely with Webpack aliasing, that we use for the highstock package.
Would love it if you could make a PR so I could give it a test.

Cheers for the swift answer.
It could also be done in a more explicit way by creating a separate component.
For instance instead of using HighchartsChart, we'd have HighchartsChartCSS (there might be a better name for it.
It wouldn't be that bad of a design choice if we consider that it actually is a different Highcharts script being loaded anyway.
What do you think?
In any case I'm gonna try to make it work in a fork and keep you up to date. I'll also have a look at Webpack aliasing.

HighchartsCSSChart would be a better name

@whawker Hi! So I found it easier to tweak the config rather than providing a CSS component, since other components were importing highcharts.
I guess that means a new module could be produced from this - react-jsx-highcharts-styled?

This might also just be considered a proof of concept and be implemented totally differently, I'm really not sure what would be best tbh.

@whawker on my end I have to use it in a hurry, so I'm gonna create an npm package with an explicit name to show that it is a temporary package, and unpublish it when this will be sorted.
Let me know if it is a problem for you, I don't want to make it awkward and will respect whatever you say about that.
Cheers.

Fair enough, I've been mulling it over too. Wondering if we should inject the Highcharts global as a prop to the root component (with a shouldComponentUpdate() => false). Then we could provide the Highcharts global via context to the sub components that require it.
That way people can inject whatever Highcharts global they want, whether it be Highcharts/Highcharts style by CSS/Highstock/etc, or even a modified version of Highcharts, as appropriate for their use case.
Needs more thought though I think.

I think being able to inject Highcharts would save a lot of unnecessary pain indeed

So I've been exploring options, I think this is my favourite. Any thoughts?
Basically using a HOC withHighcharts to inject the global. Kinda of like how withRouter/injectIntl works with React Router/React Intl.
import React, { Component } from 'react';
import Highcharts from 'highcharts';
import {
  withHighcharts, HighchartsChart, Chart, XAxis, YAxis, Title, LineSeries
} from 'react-jsx-highcharts';

class SimpleChart extends Component {
  render() {
    return (
      <HighchartsChart>
        <Chart />

        <Title>My Title</Title>

        <XAxis>
          <XAxis.Title>Time</XAxis.Title>
        </XAxis>

        <YAxis id="frequency">
          <YAxis.Title>Frequency</YAxis.Title>
          <LineSeries id="my-series" data={[43934, 52503, 57177, 69658, 97031, 119931, 137133, 154175]} />
        </YAxis>
      </HighchartsChart>
    );
  }
}

export default withHighcharts(SimpleChart, Highcharts);

+1 Simple and solves the problem.

I know I've been quiet on this for quite a while now, but just wanted to give you an update. It took me a while to get the tests working.
Pretty sure everything is working with this new pattern (see the v2 branch), if you want to take a look, let me know what you think.

I tried v2 branch using withHighcharts. Works nicely with styled mode.
Thank you for making this library, it is extremely nice and saves a lot of time working with Highcharts.

It's working but I struggled for a long time before understing that YAxis is now supposed to be a sibling of LineSeries and not a parentNode anymore.. or did I miss something?
Anyways, brilliant work man can't wait for the npm package to be updated.

That wasn't intentional if that is the case! 😟 I'll look into it.
I haven't changed any of the examples other than the addition of withHighcharts.
Would it help if I published a beta package, rather than requiring you to build from source?

Thanks! Yes I had a look at the examples as well, but just the source didn't actually tried to run them...
Also that would be great, yes! It would allow me to push my changes on the project using it!

Ok, I've just published

<EMAIL_ADDRESS><EMAIL_ADDRESS>
Could you let me know if you see the same issue?

Yes it takes more time than I would have thought, so just to let you know - I'm working on it :)

It's working great, no breaking changes from what I see, I don't know what I did... but all's well.

That's a relief!
Just so you know, I'm toying around with being able to pass extra options as an optional third argument to withHighcharts() as I think that will help with Highcharts 6.0 support. But hopefully should publish 2.0.0 as stable within the next few days.

Just publish version 2! So this is now officially supported!
See the example

Brilliant!


Configure lints in Cargo.toml

This requires the unstable lints feature, so to run lints you have to have a nightly toolchain installed and run:
cargo +nightly clippy --all-features --all-targets -Zlints

This also:

Enables a variety of lints I’m standardizing on.
Enables lints in tests, examples, etc.

As expected:
❯ cargo test --all-features
warning: /Users/daniel/personal/projects/matchgen/Cargo.toml: unused manifest key: lints
warning: /Users/daniel/personal/projects/matchgen/Cargo.toml: unused manifest key: workspace.lints
warning: /Users/daniel/personal/projects/matchgen/matchgen_tests/Cargo.toml: unused manifest key: lints
   Compiling matchgen v0.2.0 (/Users/daniel/personal/projects/matchgen)
   Compiling matchgen_tests v0.2.0 (/Users/daniel/personal/projects/matchgen/matchgen_tests)
. . .

I think cargo-lints is the way to go: #32.


Patch extract check based on presence of {{unarchive_dir}}/{{patchid}}/README.txt doesn't work for combo patches

Unarchive module for extracting patches uses creates: attribute to check if patches have to be extracted or not. This is based on the existence of the file unarchive_dir/patchid/README.txt. In Oracle's combo patches, though, there's no such file under patchid directory, just a README.html. Hence combo patches will always be extracted, regardless of they already were before.
To make it working a bit more generic, it should be safe to omit the file check and just check for existence of unarchive_dir/patchid directory, which always exists in combos, one-offs, RUs etc.

Would be fixed by https://github.com/oravirt/ansible-oracle/pull/483

Hi Thorsten,
Yes, you're basically right. The issue with combo patches could easily be worked around by setting the creates: attribute. Thanks for pointing this out. Anyway, removing the README.txt file from the check and only check for the presence of the extract directory would introduce a better default, imho. So, well, it's not really a bug and thus not really to be fixed. I'd leave it up to you to merge or reject.
Best regards,
Thilo.


Storage v11 integration

Integrate build for Storage v11.
Integrate Live Tests for Storage v11.
Update Live test credentials with Service Principal.
Update license Headers

Can one of the admins verify this patch?

@weshaggard Please can you merge this into the repo retaining revision history. Thanks!


"Push" no longer lets you insert items into machines

Issue Description:
When setting a machine side to "Push" the machine will no longer accept items from that side. For example when you set up an Applied Energistics Interface to auto-craft with an EnderIO machine pushing the product back into that interface. This no longer works.
What happens:
Machine does not accept items.
What you expected to happen:
Machine should accept items.
Steps to reproduce:

Place an Alloy Smelter
Set a side to "Push"
Place Item Conduit pulling from a chest to that side
...

Not really, but until recently it was the other block's responsibility to honour those restrictions. This changed with item capabilities. We have been using a Forge-provided compatibility layer to support item capabilities until now. With this version we provide out own handler. It is much faster and really good in respecting our restrictions.

I see, well I'm just worried that one day I'll have a case of wanting to put items inside a machine without having it pull items from the nearby inventory, but with it pushing items into that inventory.
Which is to say I feel like the way it was acting was better.


Sporadic 'Connection reset' issues

Bug description
In <1% of requests, some users of our cluster face sporadic java.net.SocketException: Connection reset errors, when calling an endpoint.
As we were not able to reproduce this issue, the only error information we have are the user's stacktraces:
Caused by: java.net.SocketException: Connection reset at java.net.SocketInputStream.read(SocketInputStream.java:210) at java.net.SocketInputStream.read(SocketInputStream.java:141) at sun.security.ssl.SSLSocketInputRecord.read(SSLSocketInputRecord.java:466) ...
To trace the requests, we enabled request logging on istio side and checked incoming requests. For the timestamp the user mentioned an issue for, we saw 8 requests for a given IP on NLB, but only 5 of them were logged in istio ingress gateway.
Might be related to #29995
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
Expected behavior
No 'Connection reset' errors occur.
Steps to reproduce the bug
N/A, as we were not able to reproduce it ourselves.
Version
`istioctl version --remote

client version: 1.9.0
control plane version: 1.8.3
data plane version: 1.8.3 (66 proxies)
kubectl version --short
Client Version: v1.17.6
Server Version: v1.17.12-eks-7684af
helm version --short
v3.3.4+ga61ce56`

How was Istio installed?
Via istio-operator 1.8.3.
Environment where the bug was observed (cloud vendor, OS, etc)
AWS EKS
Istio config information:

6 Ingress Gateways
5711 VirtualServices
14620 K8s Services
761 Namespaces

I could not attach an Istio bug-report, as it runs since 2 hours now and currently is at namespace 4/761. It it does not timeout, I will add it later.

Hello @martin-dodo-hartmann, we have a similar issue in AKS. Did you find any solution for this?

Unfortunately not. We are currently updating to istio 1.9 to see if this helps us somehow. But with 1.8.3, our users still face these issues.

Just for the records: We finally found the root cause for our issues.
In our infrastructure we have the following request flow:
NLB -> IstioIngress -> Tomcat(3rd party app)
The tomcat has configured a connectionKeepAlive of 5 seconds, whereas envoy doesn't have such a timeout. This lead to the case, where 2 requests cam in within an (exact) 5 second window. While tomcat was closing the connection for idleness, envoy just sent another request at the exact same time.
Our solution was, to add a keepAlive configuration on the affected VirtualServices of 4seconds (slightly below what tomcat has), so that always envoy is the one, terminating connections.

@martin-dodo-hartmann could you provide exact istio settings that you did?

Just for the records: We finally found the root cause for our issues. In our infrastructure we have the following request flow: NLB -> IstioIngress -> Tomcat(3rd party app) The tomcat has configured a connectionKeepAlive of 5 seconds, whereas envoy doesn't have such a timeout. This lead to the case, where 2 requests cam in within an (exact) 5 second window. While tomcat was closing the connection for idleness, envoy just sent another request at the exact same time. Our solution was, to add a keepAlive configuration on the affected VirtualServices of 4seconds (slightly below what tomcat has), so that always envoy is the one, terminating connections.

@martin-dodo-hartmann could you provide exact istio settings that you did?

@Yorkerrr : Did the issue got fixed in AKS?

@martin-dodo-hartmann , Could you please share the config implemented on virtual service?
We are also experiencing same issue but in our case AKS NLB has 4min of timeout and there is no proof from the client side that request took more than 4min nor it was idle.


Switch most dependabot updates back to daily.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?
[ ] Have you successfully run brew man locally and committed any changes?

Except for sorbet, which is updated pretty much on each commit, and non-critical updates for generating the /docs website.

Thanks for this @reitermarkus ❤


WORKSPACE snippet should be io_bazel_rules_scala

As far as I've been able to tell, the WORKSPACE snippet for 5.0.0 should be:
http_archive(
    name = "io_bazel_rules_scala",
    sha256 = "141a3919b37c80a846796f792dcf6ea7cd6e7b7ca4297603ca961cd22750c951",
    strip_prefix = "rules_scala-5.0.0",
    url = "https://github.com/bazelbuild/rules_scala/archive/refs/tags/v5.0.0.tar.gz",
)

(I as "as far as I've been able to tell" as I'm getting the cryptic Analysis of target '.... failed; build aborted: when trying to use scala_proto_library, but I'm working under the hypothesis I've messed that up. You can't configure the toolchains if you use rules_scala rather than io_bazel_rules_scala.)

Thanks for reporting, @sacsar! Updated existing release with the correct name.


Silent notification displayed in Android

In Android, when the app is active a push notification with contents, dat, and android_sound is not displayed, but when the push notification does not have any of those fields (= it should be silent regardless if the app is active or not) the notification is displayed in the Notification Shade. I have set inFocusDisplaying to None.
OneSignal.inFocusDisplaying(0);
"react-native-onesignal": "3.0.4"
React Native: 0.46.2
Device: Moto G 4th gen, Android 7.0

Is this a bug in react-native-onesignal or in the One Signal Android SDK? Easily reproducible with curl:
curl --include \
     --request POST \
     --header "Content-Type: application/json" \
     --header "Authorization: Basic 7b8ayb890yaep89vby8adpb89adyb89HIOH99IHiohhihihih" \
     --data-binary "{\"content_available\": \"true\", \"app_id\": \"000000000-0000-0000-0000-000000000000\", \"priority\": \"10\", \"android_sound\": \"nil\", \"android_visibility\": \"-1\", \"android_group\": \"mygroup\", \"filters\": [{\"field\": \"tag\", \"key\": \"abc\", \"relation\": \"=\", \"value\": \"true\"}, {\"field\": \"tag\", \"key\": \"efg\", \"relation\": \"=\", \"value\": \"true\"}]}" \
     https://onesignal.com/api/v1/notifications

@tapz inFocusDisplaying only refers to the notification behavior when the application is in the foreground. I'm unable to produce a case where inFocusDisplaying is set to NONE and the notification appears in the shade while the app is foregrounded. I used the same payload parameters as your curl call
Are you trying to prevent the notification from appearing in the shade even when it's in the background?

@tapz upon further investigation, it looks like this issue should be resolved as it was addressed in the 3.6.2 release - since the react project takes the latest 3.x OneSignal build, these fixes should be already incorporated.


Demangler Microsoft errors

Describe the bug
I imported a version of msdia140.dll and got some demangling errors.
To Reproduce
Steps to reproduce the behavior:

Load this file: will expire in 30 days, hash and info
Get PDB from MS symbol server.
See errors:

Demangler Microsoft> Failed to apply mangled symbol at 18011182c; name:  ?inf_lowercase@?2???$parse_floating_point_possible_infinity@DV?$input_adapter_character_source@V?$string_input_adapter@D@__crt_stdio_input@@@__crt_strtox@@_K@__crt_strtox@@YA?AW4floating_point_parse_result@1@AEADAEAV?$input_adapter_character_source@V?$string_input_adapter@D@__crt_stdio_input@@@1@_K@Z@4QBDB (Demangler Microsoft/ghidra.app.util.demangler.DemangledVariable)
Demangler Microsoft> Failed to apply mangled symbol at 180111844; name:  ?lowercase@?2???$parse_floating_point_possible_nan@DV?$input_adapter_character_source@V?$string_input_adapter@D@__crt_stdio_input@@@__crt_strtox@@_K@__crt_strtox@@YA?AW4floating_point_parse_result@1@AEADAEAV?$input_adapter_character_source@V?$string_input_adapter@D@__crt_stdio_input@@@1@_K@Z@4QBDB (Demangler Microsoft/ghidra.app.util.demangler.DemangledVariable)
Demangler Microsoft> Failed to apply mangled symbol at 18011185c; name:  ?lowercase@?1???$parse_floating_point_possible_nan_is_ind@DV?$input_adapter_character_source@V?$string_input_adapter@D@__crt_stdio_input@@@__crt_strtox@@@__crt_strtox@@YA_NAEADAEAV?$input_adapter_character_source@V?$string_input_adapter@D@__crt_stdio_input@@@1@@Z@4QBDB (Demangler Microsoft/ghidra.app.util.demangler.DemangledVariable)
Demangler Microsoft> Failed to apply mangled symbol at 18014ff50; name:  ?_OptionsStorage@?1??__local_stdio_scanf_options@@9@4_KA (Demangler Microsoft/ghidra.app.util.demangler.DemangledVariable)

Expected behavior
No errors.
Screenshots
https://i.imgur.com/Qy6YOu9.png
Attachments
If applicable, please attach any files that caused problems or log files generated by the software.
Environment (please complete the following information):

OS: Win10
Java Version: [e.g. 11.0]
Ghidra Version: 9.2

@justanotheranonymoususer
First, the error message could be improved to provide more information.  I'll try to rectify that soon.
The real problem is that, while the Microsoft's mangling scheme provides a lot of information, including whether we have a char * or a char[], Microsoft seem to have chosen to create "pointer" symbols for addresses of global "arrays." This could be just as a convenience, as when someone is debugging a program, the program contains the type information, and when linking a program, all that matters is the uniqueness of the names.
This is a known problem that we haven't come to grips with how we should handle. Since the type is ambiguous, it might be better if we do nothing. We have thoughts of having the demangler lay down program properties at addresses instead when there is ambiguity, and having another analyzer follow behind to make better decisions that might include type propagation.
What is going on in this program example is that at address 180111828 (4 bytes lower in address than 18011182c), there is an array of 4 bytes that seems to be a null terminated string.  However the mangled symbol at that address says char * so we have been laying down a pointer (8 bytes for the 64-bit program), but this collides with the data and symbol at address 18011182c.  The demangler seems to be doing its job of demangling the symbol, but it is failing to lay down the data type at that address as it would overlay the pointer that is being incorrectly laid down 4 bytes lower in address space.  Due to this failure, the demangled string is also not shown in a block comment.
What would help you do your job better? We could lay nothing down, or continue to flip a coin (always deferring to the "type" indicated by the mangling scheme).  Maybe a demangler option to control this? In the longer term, we would still like to do a "smarter" approach.

What would help you do your job better?

Thanks for asking for my opinion. It doesn't affect me all that much, at least not that I specifically noticed. In general, like most other users, I'm interested in as much high quality analysis as possible. Perhaps in this case we can consider the mangling info only if the pointer is valid and points inside the module, otherwise discard the mangling info (in which case it will probably deduced as a string anyway?).

Thanks for your input on this, and I appreciate the desire for quality.  I plan to get back to this, but not sure how soon at this time.


feat: add /progress/ endpoint to show scan progress

This patch adds a new endpoint to kubei's web-app at /progress/. It prints scanner.ScanProgress() as json.
Example:
GET /progress/
{
  "ImagesToScan": 15,
  "ImagesStartedToScan": 13,
  "ImagesCompletedToScan": 2
}

This can be useful if you have a script that's waiting for results. That way it can poll this endpoint until all scans completed

Hi @Scrumplex, most of kubei's logic was recreated, including the UI, API, backend, etc.
The progress API is now available in https://github.com/cisco-open/kubei/blob/main/api/swagger.yaml:
/runtime/scan/progress
You are welcome to try out the new features :)
Thanks!

Looks cool. Thanks for the heads up!


import { useForm } from "react-hook-form" not present in Form Guide

For someone who's absolutely copying and pasting form the guide, this would give an error, as useForm is not imported in the guide.
Can simple fix it by adding that in the documentation.

If you want, I can fix this

I can fix this, but maybe good first issue label would do the justice here.

Hi, if installed via CLI, zod & react-hook-form are automatically included in the project along with the form component. For manual installation, follow the instructions in the installation section.

Thanks for offering this. I noticed this problem the other day too. I just opened the PR that fixed this issue.
https://github.com/shadcn-ui/ui/pull/1486


Supporting WDL 1.0 "struct"

dxCompiler 2.4.1 is not supporting "struct" properly, and gives error message during compiling. dxWDL 1.50 is on the other hand compiles successfully.
==============START OF ERROR MESSAGE================
[error] Error translating test-struct.wdl to IR
java.lang.Exception: input <sample_name, TString> to call  is missing from the environment. We don't have sampleStruct.sample_name in the environment.
...
==============END OF ERROR MESSAGE================
===============START OF WDL FILE =====================
version 1.0
task test {
input {
String sample_name
Array[File] read_fastq_list
Array[String] flowcell_id_list
}
command <<<
set -ex
mkdir fastq_dir
while read flowcell_id fastq_file
  do mkdir ${flowcell_id}
  mv ${fastq_file} $flowcell
done < <(paste ~{write_lines(flowcell_id_list)} ~{write_lines(read_fastq_list)})
echo $(ls -R )

output {
String out = read_string(stdout())
}
runtime {
docker: "docker.io/scicomppublic/centos:7"
}
}
struct SampleStruct {
String sample_name
Array[File] read_fastq_list
Array[String] flowcell_id_list
}
workflow test_workflow {
input {
SampleStruct sampleStruct
}
call test_map_serialization {
input:
sample_name = sampleStruct.sample_name,
read_fastq_list = sampleStruct.read_fastq_list,
flowcell_id_list = sampleStruct.flowcell_id_list,
}
output {
String out = test.out
}
}
===============END OF FILE =====================

Thanks, we'll take a look at this.

this issue is fixed and will be in the next release


Erijpkema made example work

Some changes I found usefull when using cacafonisk for our internal project. Each commit message hopefully explains my reasons.

Comments ended up in: https://github.com/VoIPGRID/cacofonisk/pull/6

Am a bit dissapointed in my vim !sort. Will fix that ;-)

We might have to review our review process for github :). I may have lost sight of the woods in between the trees.
If you could squash the commits, I'm pretty sure I'd gladly merge this and do a new release.


Potential circular import problem

As documented here:
https://github.com/Xilinx/finn-base/pull/2#issuecomment-709327273

This is addressed by https://github.com/Xilinx/finn-base/pull/6 and should no longer an issue. I've also run pycycle on the project to ensure there were no further circular imports.


Improve Conditions handling for API objects

Is there an existing issue for this?

[X] I have searched the existing issues

Problem Statement
There is  a limit of 8 entries per resource set in the API definition. Currently, when the condition needs to be updated, the code updates all the conditions that matches. In example:
func (r *ControlPlaneReconciler) ensureControlPlaneIsMarkedProvisioned(
	ctx context.Context,
	controlplane *operatorv1alpha1.ControlPlane,
) error {
	updatedConditions := make([]metav1.Condition, 0)
	for _, condition := range controlplane.Status.Conditions {
		if condition.Type == string(ControlPlaneConditionTypeProvisioned) {
			condition.Status = metav1.ConditionTrue
			condition.Reason = ControlPlaneConditionReasonPodsReady
			condition.Message = "pods for all Deployments are ready"
			condition.ObservedGeneration = controlplane.Generation
			condition.LastTransitionTime = metav1.Now()
		}
		updatedConditions = append(updatedConditions, condition)
	}

	controlplane.Status.Conditions = updatedConditions
	return r.Status().Update(ctx, controlplane)
}

The Gateway implementation currently prunes the oldest conditions when a new one needs to be added. That's fine, as there is only single occurrence of given condition. But that's not really readable way to update single condition or safe, as if by any change we duplicate a conditions, we might reach the limit of 8.
Then we would need to decide which condition(s) to prune.
Proposed Solution

[ ] Ensure functions that set conditions cannot be misused by adding duplicate condition.
[ ] Work out how to prune conditions, if limit of 8 is reached.

Additional information
Add more robust logic for how this is handled and have a decision tree for which conditions need to be present and which don't or equivalent solution.
Acceptance Criteria

[ ] Ensure functions that set conditions cannot be misused by adding duplicate condition.
[ ] Work out how to prune conditions, if limit of 8 is reached.

@shaneutt Did I messed anything in the description?

Nope looks good.

Pasting suggestions from Ruben Romero from the Slack chanal.

I've been looking into the gateway-operator code and I would like to make some comments with my personal experience. It is mainly about the Status Condition management.
Conditions as a snapshot: I'd suggest to treat the conditions as a snapshot of the current state, instead of a event log. This way you can read the current state of the Scheduled condition without having to sort by date. This helps with custom columns in the state, for example.
Reason only when status=false. This is a common practice, if the condition is fine, no need to add a reason why it's fine.
Ready. It's very common to always have a Ready condition when all the other conditions are true. This simplifies objects observability.
Suggested conditions for the Gateway. Scheduled, Ready, ControlPlaneReady, DataPlaneReady. The last ones are new
What do you think? I don't mind working on a PR to make these changes if you find them useful.

this doc helped me a lot in the past
https://maelvls.dev/kubernetes-conditions/


Add Lacnic rr connector

Hi @dsdenes,
Thanks for this contribution.
I created #3, which is the way I would implement your proposal.
Please, let me know what you think about it and if it fits your purpose. Feel free to edit it.
If everything is fine, I will release on npm.
Thanks

Hi @dsdenes,
Thanks for this contribution.
I created #3, which is the way I would implement your proposal.
Please, let me know what you think about it and if it fits your purpose. Feel free to edit it.
If everything is fine, I will release on npm.
Thanks

LACNIC irr is merged, see https://github.com/massimocandela/bulk-whois-parser/pull/3.
I'm going to close this. Thanks for your contribution.

LACNIC irr is merged, see https://github.com/massimocandela/bulk-whois-parser/pull/3.
I'm going to close this. Thanks for your contribution.


[FEATURE] Add targeting to iOS version as the same on Android

On Android version, if your app is to children directed and the ads are not target to this, your app could be banner, on iOS this doesnt happen, but is good to have it because iOS some day can add it, or we just dont want to our users to see improper ads.
Guide: https://developers.google.com/admob/ios/targeting

Fixed


Add scripts with server

Pull Request (PR) description
This change allows packaging scripts with the server instance.
Example usage:
openvpn::server { 'test-site':
    ....
    up => '/etc/openvpn/test-site/scripts/add-tap-to-bridge.sh',
    scripts => {
        "add-tap-to-bridge.sh" => {content => 'a'},
    },
}

Anything that can go in a regular Puppet file resource can be used in the hash including template, source, and content.
up, down, client-connect and client-disconnect scripts will be checked for / and be considered to be in the scripts folder otherwise.
Tests pass locally.

Build failure looks like an infrastructure issue, not test issue

Build failure looks like an infrastructure issue, not test issue

I restarted the job.

@yakatz please have a look at the inline comments I made. Can you also document this feature in the README.md?

Ping @yakatz

For some reason I didn't get notifications from GitHub about this. I will work on it over the next few days.

Looks like a test timeout again. Is this something I am doing wrong or is it just a test system issue?

Once again, test failures look to me to be infrastructure issues. There is nothing changed in the last commit that would change the outcome of these tests.

Once again, test failures look to me to be infrastructure issues. There is nothing changed in the last commit that would change the outcome of these tests.

On failing configuration, the vpnserver system under test does not see the vpn client.
On working configuration:
https://travis-ci.org/voxpupuli/puppet-openvpn/jobs/604083503#L2210
On failing configuration
https://travis-ci.org/voxpupuli/puppet-openvpn/jobs/604083491#L2134
Right now i am unable to say more.

@yakatz the code looks good to me. Are you able to squash the commits down to a few?

@bastelfreak Squashed to 2

@bastelfreak Any thought on when this could make it into a release?


Unable to Load Addon

Made rotation for Amrs Warrior on Prepatch PTR
Worked fine untill i aquired dual spec and made some macros.
Now i cannot open settings. I get the following LUA error when I try:
Message: Interface\AddOns\RotationMaster\main.lua:403: attempt to index field 'optionsFrames' (a nil value)
Time: Sun Aug 14 16:48:13 2022
Count: 4
Stack: Interface\AddOns\RotationMaster\main.lua:403: attempt to index field 'optionsFrames' (a nil value)
[string "@Interface\AddOns\RotationMaster\main.lua"]:403: in function `OnClick'
[string "@Interface\AddOns\AtlasLootClassic\Libs\LibDBIcon-1.0\LibDBIcon-1.0.lua"]:144: in function <...tlasLootClassic\Libs\LibDBIcon-1.0\LibDBIcon-1.0.lua:142>
Locals: _ = LibDBIcon10_RotationMaster {
0 =
fadeOut =  {
}
dataObject =  {
}
isMouseDown = false
showOnMouseover = true
Show =  defined =[C]:-1
SetParent =  defined =[C]:-1
db =  {
}
icon =  {
}
Hide =  defined =[C]:-1
}
button = "LeftButton"
frame = RotationMasterLDBFrame {
0 =
}
dropdownFrame = RotationMasterLDBDropdownFrame {
0 =
Text = RotationMasterLDBDropdownFrameText {
}
Icon = RotationMasterLDBDropdownFrameIcon {
}
Button = RotationMasterLDBDropdownFrameButton {
}
Right = RotationMasterLDBDropdownFrameRight {
}
Middle = RotationMasterLDBDropdownFrameMiddle {
}
Left = RotationMasterLDBDropdownFrameLeft {
}
}
(*temporary) =  defined @Interface\FrameXML\InterfaceOptionsFrame.lua:423
(*temporary) = nil
(*temporary) = RotationMasterLDBFrame {
0 =
}
(*temporary) = "UIDropDownMenuTemplate"
(*temporary) = RotationMasterLDBDropdownFrame {
0 =
Text = RotationMasterLDBDropdownFrameText {
}
Icon = RotationMasterLDBDropdownFrameIcon {
}
Button = RotationMasterLDBDropdownFrameButton {
}
Right = RotationMasterLDBDropdownFrameRight {
}
Middle = RotationMasterLDBDropdownFrameMiddle {
}
Left = RotationMasterLDBDropdownFrameLeft {
}
}
(*temporary) = nil
(*temporary) =  {
UpdateOnBarHighlightMarksBySpell =  defined =[C]:-1
ERR_OUT_OF_CHI = "Not enough chi"
DH_HAVOC_CORE_ABILITY_2 = "Strong melee attack that consumes Fury. If it critical strikes, some Fury is refunded."
MultiCastActionButton6Cooldown = MultiCastActionButton6Cooldown {
}
MerchantItem9ItemButtonStock = MerchantItem9ItemButtonStock {
}
GetTrainerServiceTypeFilter =  defined =[C]:-1
UNIT_NAMES_COMBATLOG_TOOLTIP = "Color unit names."
SetTrainerServiceTypeFilter =  defined =[C]:-1
LE_GAME_ERR_CHAT_RAID_RESTRICTED_TRIAL = 788
BT4Button46Shine5 = BT4Button46Shine5 {
}
SPELL_FAILED_CUSTOM_ERROR_71 = "This partygoer wants to dance with you."
LE_GAME_ERR_PET_SPELL_TARGETS_DEAD = 406
ERROR_CLUB_TICKET_COUNT_AT_MAX_COMMUNITY = "Can't create any more invite links for this group."
CompactUnitFrameProfilesGeneralOptionsFrameHealthTextDropdownButtonNormalTexture = CompactUnitFrameProfilesGeneralOptionsFrameHealthTextDropdownButtonNormalTexture {
}
MultiCastActionButton2Cooldown = MultiCastActionButton2Cooldown {
}
ERR_TRADE_EQUIPPED_BAG = "You can't trade equipped bags."
PVP_RANK_6_1 = "Corporal"
MultiBarLeftButton7 = MultiBarLeftButton7 {
}
InterfaceOptionsNamesPanelUnitNameplatesShowAll = InterfaceOptionsNamesPanelUnitNameplatesShowAll {
}
VideoOptionsFrameDefaults = VideoOptionsFrameDefaults {
}
MerchantItem1AltCurrencyFrameItem1Text = MerchantItem1AltCurrencyFrameItem1Text {
}
OPTION_TOOLTIP_ACTION_BUTTON_USE_KEY_DOWN = "Action button keybinds will respond on key down, rather than on key up."
BINDING_NAME_NAMEPLATES = "Show Enemy Name Plates"
INSTANCE_UNAVAILABLE_OTHER_TEMPORARILY_DISABLED = "%s cannot enter. This instance is temporarily disabled."
LFGBrowseGroupDataDisplayRoleCount_Update =  defined @Interface\AddOns\Blizzard_LookingForGroupUI\Blizzard_LFGBrowse.lua:699
MultiBarBottomRightButton8Shine5 = MultiBarBottomRightButton8Shine5 {
}
IsReferAFriendLinked =  defined =[C]:-1
MAIL_LETTER_TOOLTIP = "Click to make a permanent
copy of this letter."
UnitFrameManaBar_UnregisterDefaultEvents =  defined @Interface\FrameXML\UnitFr.
The rotation itself still works if i do /rm toggle. just cant oppen settins to change anything, also cant use it on second talent spec

Can you test if this is still an issue in 1.4.1 Beta?

Hello,
Issue was still present so i had to reset the addon.
It seems there was something in the rotation that caused it to bug at some point (even though initially it worked fine). I was unable to open it ingame to modify.
This is the rotation i was using, if it helps:
AzReBAAuSdA2Es2URM0mdVMS+UZoJ6i9aLfEfgh+SkrDFPxWaiiEajCghosclCTHB
M4ptFBSLQVgsaDlED2UBJ6qloE8VzRREpoq1WRR0ykXAXQVSl2muk6wkYTNUGyHkt
NCj6H6LEl9h1BGQshhotGSYhxJfwG5BXRNSuchPZNimpLbAM2AnUdukxZ3YSccsPP
CdKAiAZ0hb2hyfibXmpe4xV0IQ8NS+Zh5OYp5QZgRHPIawTVCLagG3oJNKEd8EFXp
vnh8lxwt7jLZeSypBWcnHLDzWFILzs4bmpyvYysU/EyuNV+XZtoRgF+zBeOFEo1PW
qKOpPGGJufh2k4jDLHzhibn69gKBT+Ta83Xtc8ZSzUC8MYgAFeK1TXGCyUan7REHU
kj3LDTaLyhyflaADQ+xylD7ZkhjTDEzgvTgYoEdIa8ocn5mVxrXuIWdyEMtdCg8jH
zS8ytqGIasFHXCK42LzCUDQb0EBqGQVzKRs08vkuOFneZ7DyW1IKz84Ym5ysCrGMr
FPxMrjUPwEvmhRKNN4msdqPxhibn4dpOoxFu2HlhOlCRgM6QNzQ5P5NrzUvc4i6Ex
UZdfMx8g8lhzqMI5gASwdUMIjiGMgCXOLqHqqqKOIbHd8nR4i6TnTc0OV/HVij4mV
xrvstpjgCM2UgE4LreO/NCicYnLVFH0miEFzsvTAcqEtAZ8oYm52dyrXmJW9yFMNR
CjZKPmRdzIELKjIPqoGzBhWMGcFSuAMSXDFwGOWwgkcd3ydHiLpOdx5JacHbCATtv
aHK+jq1Q8DO51vyFnL0gAFd0aTW8UKLxSUvUkTTCEDuPJbXghIcKId7Q5DwigFWVO
yLEK+j8To6UtpF8ZYB/R1QF80KJxptEVrvRCDfsNBpvaHK+FgCNRrhl54XpQEUCOk
Pe0OT8TK41L3GLuxCGWmhBfMx8C6lSLjUAVTVowYXjGQwHXyAmjy1mqVAoqjiNsJK
dF6YDFQnoDHlDk/Mzuda8xC3uQrgZGmjxMfMD1GSMUgzAHKwRXIEIgWOrFEFX4gsK
AtFVseXxXXK0uMtk54USjU9aoo7kDU3OV6qCq2g0Gj8Ri4bXsNNoJA==
If i import it, i get the same error as previously.
I noticed that some conflicts in the rotations tend to bug out the whole addon (on all chars) instead of just the one rotation.
I had a similar issue when i selected a mana condition for the warrior, when i started using the addon. Had to reset it to make it work again.
/rm toggle still worked so maybe you should consider a full reset command until this sort of issue happens less often.
Also, v 1.41 beta seems to be missing proc based buffs like "Taste for blood" or "Sudden Death" on the buff present conditton.

re: buffs - I don't list buffs or debuffs.  I just use what Blizzard sets.
When you say missing, do you mean you can't choose it? or conditions with it aren't firing.  If the latter, check they aren't actually debuffs (there was a mage proc buff that Blizzard actually identified as a debuff).


Import Numpy in Pose docs

Description of issue (what needs changing)
import numpy as np is missing in the Pose documentation code. np is used without importing.
Clear description
Just a documentation fix to import the missing module.
Correct links
No response
Parameters defined
No response
Returns defined
No response
Raises listed and defined
No response
Usage example
No response
Request visuals, if applicable
No response
Submit a pull request?
Yes, already submitted.

Hi @schmidt-sebastian,
Could you please look into this pull request https://github.com/google/mediapipe/pull/4145? Thank you!

Closing this as the issue has been merged commit d6fb7c3


Add options.ctol to prima_is_success

This was discussed in the Python binding PR but not implemented. Something that would need to change to accommodate this would be setting the default options.ctol in C to sqrt(eps), which means we could no longer pass it to Fortran as an optionally parameter.
Alternatively, in prima_is_success we could check if option.ctol is NaN and use sqrt(eps) if so, but I seem to recall there were some issues with NaN checks with certain compilers and options?

I see the issue ...


youtube-dlc: Add version 2020.09.13

Community fork of youtube-dl

I wasn't sure if this belongs in the Main bucket or here in the Extras bucket, seeing that the original youtube-dl is in the Main bucket.

Move this PR into the versions bucket. There is no reason to have development fork, which is not approved by the original authors here in Extras.


Investigate failed Sonarqube builds

Description
Sonarcube seems to be intermittently failing to build in CI jobs. See examples here and here
Steps to reproduce

Open a PR
Sonar will try building but fail with the following error code

> Your project contains .java files, please provide compiled classes with sonar.java.binaries property, or exclude them from the analysis with sonar.exclusions property.

Kicking the job seems to fix it somehow, so wondering if it's just an issue on first creation?
Expected behavior
Sonar building on the first try when unrelated code is put up for review

@shanice-skylight  I seem to be able to repro this pretty consistently when I'm manually rerunning the sonar job. See this PR as an example

Notes on why this error is triggered:

Compiled .class files are required for java projects with more than one java file. If not provided properly, analysis will fail with the message:

Your project contains .java files, please provide compiled classes with sonar.java.binaries property, or exclude them from the analysis with sonar.exclusions property.
Reference used
I added the sonar.java.binaries property, now I'm working through fixing a message that says Could not calculate hash for class