Sacrificial Pact takes screenshot

I was going a bit nuts but eventually found out.
if you are playing a death knight in the left tree you can select Sacrificial Pact, which kills your pet for some healing, this makes the addon take a screenshot because it's a PARTY_KILL event

for pvp maybe it's better just to take screenshot of the final boards of arena/battlegrounds

Hello @masterkain how are you? I hope you are doing great.
For me, your bug report it's very special, It's my first open source project and you are my very first issue reporter. I really appreciate it and I hope to fix that soon.
There is other bugs that I'm fixing this days I'm this one I was not track of. Really thank you. Probably it must happen too with other spec that have some kind of similar behavior in killing their pets. I don't play with warlock, but I guess that it has some interaction with this mechanic too. I'll test it more.

Another thing that I'll like to point it out is that I've created this project just with a fun purpose and to try Lua for the first time in my life. I'm really lost, mainly because I'm really blind with the documentation and all of the events that I can capture in the game, other issue that I'm having is debugging. For this print, it looks like you have some knowledge about the events and parameters to capture. How can I do that? Do you know any better documentation that the wowpedia? Any class that I can learn more about this environment?
Thank you

hey it's no problem, just testing things out
keep in mind my knowledge is limited too, I can point some links in the hope they can be useful

https://www.townlong-yak.com/framexml/beta (generated lua api)
https://github.com/Ketho/vscode-wow-api (wow api addon for vscode)
https://warcraft.wiki.gg/wiki/Patch_11.0.0/API_changes (for addon upgrades)
https://www.curseforge.com/wow/addons/devtool (in-game event monitor)
https://gist.githubusercontent.com/masterkain/3886b00ef401d83e0caf2213bc4fc2f1/raw/512bff164ece9dcaf5fa842d7aa9e9cec9116827/gistfile1.txt (old memoria addon from which we can pick some useful events)

Thank you my dude. Every help it's a move in the right direction. BTW, if you agree, on the next version I'll create a tab with some acknowledgments. If you are interested, I'll be pleasured in add your name, battletag or character name (and realm) if you want to. Just comment here and I'll close this issue on the PR with all this features and fixes in the next days.

Fixed on new version.

Fixed on new version.


Cannot load Bible reference at the moment

Describe the bug
The following message is being displayed on popup of Scriptures: "Cannot load Bible reference at the moment" for all versions available of the Bible. This bug is occurring on all devices and in all browsers.
Getting a 403 Forbidden error in console for request to https://api.scripture.api.bible/...

Thanks for the heads up @kimberlyas. I released v1.0.3 to address this
I also spotted a bug affecting the wiki popup style and this is also fixed in v1.0.4 (the latest version)
Please let me know if the latest version solved the issue for you

Yes, thank you for resolving the issue @Bukunmikuti. The new update is working.

I'm having the same issue with message "Cannot load Bible reference at this time."  Wordpress does only has v1.01 available. How do I upload v1.04?

Hello @Smoses00814
The WordPress plugin will be updated to the latest very soon, before the weekend. Your issue might be a problem with cache. Please try with a different browser and see if it solves it. Please let me know if it works or not.

Thank you. I was able to clear the cache and references are loading now. I look forward to the new version being available on WordPress.


Enforce correct AVA and XO versions when targetting Node <= 0.12

Currently it always tries to enforce the unicorn * version.

Added in a5bf8260486a947cb100862b3173c044f171a41d and db5ff8be2bd0a9abfe8fcbd74425ca3d56f58a66


Support for schema 4.2

Do you plan on supporting schema 4.2 any time soon?

Probably not, sorry. The schema change was so substantial that the update would require significant time that I'm unable to spare.

That’s also my issue. Apart from me not being a fan of the implementation of the new feature. When it becomes more applicable I’ll dig into it.

Got it, Thanks guys


Update release-notes-5.1.0.md

https://internal.almoctane.com/ui/entity-navigation?p=131002/6001&entityType=work_item&id=694004

A developer build has not yet been created for this branch. Click here to go ahead and create the build...


Support pattern matching for Lint/OutOfRangeRegexpRef cop

Closes #10976.
Note: This is the first time I familiarized myself with the pattern matching in ruby, so be a little more careful when reviewing this PR.

Looks good to me.


chore: remove bad simps, and add test

!bench

Two files got faster by at least 10⁹ instructions (and none got slower by that amount):
Files that got faster:
  Mathlib.CategoryTheory.Sums.Associator:              -2.6996 * 10⁹ (-9.97%)
  Mathlib.CategoryTheory.Sums.Basic:                   -1.5400 * 10⁹ (-5.13%)

I have restored the old lemma (without the simp attribute) and changed the name of the new one. LGTM, thanks!
bors d+

bors merge


writeStatus ignored if called after writeHeader

It seems like calls to writeStatus are ignored if called after writeHeader. For example, this results in the HTTP status 200 OK:
res.writeHeader('Content-Type', 'application/health+json');
res.writeStatus('503')
res.end(JSON.stringify({ status: 'fail' }))

Ah guess this is standard HTTP message semantics (see https://developer.mozilla.org/en-US/docs/Web/HTTP/Messages).
As this library is somewhat lower-level than other HTTP frameworks Node devs are likely used to we should perhaps call this out clearly in the docs. I'm not really familiar with the C++ side of things, could we perhaps throw an error into JS if these methods are called out of order?

I have the same issue. I have to store all my writeHeaders and then write them after I set the status in my route.

If this is how you see things then clearly you're not the intended user base

I would be more respectful with people trying my work. I could use it in my company to get more performance, that's why
i'm trying it out.

Respect has nothing to do with any of this. I'm just stating neutral facts. If you see things this way, you're not seeing things the intended user would see things.

@lusatiro you are replying to someone from Feb 2019 to make random statement so I think you are just confusing people. And you just posted noobish question in other issue about Options method so sounds like you don't know what you are doing, maybe other people at your company know what they are doing

You are just arguing that everyone jumps directly to mid or senior
positions. Everyone has to start from some point. I work in the industry
for some quite time now. Yes, I never had to config an options cors method
protocol myself, all this stuff generally comes out of the box. I replied
'cause I need the solutions and there's little to no information about this
lib out there, thus the issues are the only place I could find some
solution. I could stick with express, but I preferred to learn new stuff.
Em sex., 16 de out. de 2020 às 11:42, hst-m<EMAIL_ADDRESS>escreveu:

@lusatiro https://github.com/lusatiro you are replying to someone from
Feb 2019 to make random statement so I think you are just confusing people.
And you just posted noobish question in other issue about Options method so
sounds like you don't know what you are doing, maybe other people at your
company know what they are doing
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uNetworking/uWebSockets.js/issues/74#issuecomment-710088920,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AME4HQ3A6JGB2CJI6OOCMWDSLBLURANCNFSM4GZZATDA
.

Stop trying to be Linux, you're far away from this.
Em sex., 16 de out. de 2020 às 13:20, Lucas Sátiro<EMAIL_ADDRESS>escreveu:

You are just arguing that everyone jumps directly to mid or senior
positions. Everyone has to start from some point. I work in the industry
for some quite time now. Yes, I never had to config an options cors method
protocol myself, all this stuff generally comes out of the box. I replied
'cause I need the solutions and there's little to no information about this
lib out there, thus the issues are the only place I could find some
solution. I could stick with express, but I preferred to learn new stuff.
Em sex., 16 de out. de 2020 às 11:42, hst-m<EMAIL_ADDRESS>escreveu:

@lusatiro https://github.com/lusatiro you are replying to someone from
Feb 2019 to make random statement so I think you are just confusing people.
And you just posted noobish question in other issue about Options method so
sounds like you don't know what you are doing, maybe other people at your
company know what they are doing
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uNetworking/uWebSockets.js/issues/74#issuecomment-710088920,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AME4HQ3A6JGB2CJI6OOCMWDSLBLURANCNFSM4GZZATDA
.

I am saying you should stop spamming this repo with noob http questions and use Google instead to do some basic learning, come back when you are ready


Security: Autoscaled EC2 instances currently need to be accessible through the internet

Moved from gitlab, original message from @sanderegg:
In the current implementation, it is not possible to start an EC2 instance for autoscaling without making it available through the internet.
This is security wise a problem, as anyone with the IP and ssh may be able to connect on the machine.
Currently it is necessary for the machine to be Internet accessible due to the following facts:
- need to ssh on the machine for debugging purposes
- the machine needs to access Dockerhub to pull images

Please devise what should be done there. [@kaiser](https://github.com/kaiser) talked about having some kind of router.
Take your time to find out, but I do think it would be desirable before we unleash sim4life full

### Tasks
- [ ] https://github.com/ITISFoundation/osparc-simcore/pull/5882#pullrequestreview-2082140345
- [ ] https://github.com/ITISFoundation/osparc-ops-environments/pull/682
- [x] ops: change master infra
- [x] ops: change staging infra [STAG Release]
- [ ] ops: change prod infra [PROD Release]
- [ ] https://github.com/ITISFoundation/osparc-simcore/pull/5883
- [ ] https://github.com/ITISFoundation/osparc-ops-environments/pull/683
- [ ] https://github.com/ITISFoundation/osparc-simcore/pull/5886

[ ] master (PR)
[ ] stag
[ ] prod

Waiting for PROD release

This PR https://github.com/ITISFoundation/osparc-ops-environments/issues/195 tracks this issue


Environment Requirements

Hi,  I have some problem with the environments, the version of some packages doesn't match. Could you please provide some details about your environment requirements? Thank you very much.

Hello, I can upload the requirements.txt file but there may be other packages which are not required for this repo. They would be unnecessarily downloaded. If the issue is still not resolved, I will upload it by EOD

Thank you so much!


Is it possible to use the same random seelction multiple times in a prompt?

I have a file of random "careers" for characters.  I'm hoping to reuse the random choice  to indicated both the career and the uniform the character is wearing but I am not having much success.
I've tried:
A __career__ wearing a __career__ outfit but what I end up with is something like:
A teacher wearing a fishmonger's outfit  [NOTE: not that there's anything wrong with that in real life!] when I wanted a teacher in a teacher's outfit
I thought I'd be clever and see if using "$1" or "\1" as a regex group holder would work, but that failed miserably :)

I've also had this interest -- it would be nice to have some kind of operator which might 'lock' a particular wildcard to be consistent.
A Career(1) wearing a Career(1) outfit while at a Career(2) worksite, speaking to a Career person.

I like that idea - I'll give it some thought. In your example, it would be:
A teacher wearing a teacher outfit
and not A teacher wearing a teacher's outfit.

I like that idea - I'll give it some thought. In your example, it would be:
A teacher wearing a teacher outfit and not A teacher wearing a teacher's outfit.

Well, if it was a __career__ wearing a __career__'s outfit then you'd get "a teacher wearing a teacher's outfit", which is fine.  As long as the string is constant (if the user wants it what way). so you do not get "a nurse wearing a welder's outfit" :)

Can you test out the new Jinja2 templates feature? You can read about it here: https://github.com/adieyal/sd-dynamic-prompts#jinja2-templates
In your example, you can do something like:
{% for career in ['teacher', 'doctor', 'lawyer'] %}
    {% prompt %}A {{ career }} wearing a {{ career}}'s outfit{% endprompt %}
{% endfor %}

Can you test out the new Jinja2 templates feature? You can read about it here: https://github.com/adieyal/sd-dynamic-prompts#jinja2-templates
In your example, you can do something like:
{% for career in ['teacher', 'doctor', 'lawyer'] %}
    {% prompt %}A {{ career }} wearing a {{ career}}'s outfit{% endprompt %}
{% endfor %}

Hmmm... I usd the automatic update in AUTO1111 and it says my version was behind, did an update but still loaded 0.13.7

Could you give me an example of a prompt that didn't work?

Could you give me an example of a prompt that didn't work?

Sure, this is on the latest (couldn't download via AUTO update, downloaded manually and threw it in the extensions folder)
PROMPT: professional digital airbrush art of {% for career in ['teacher', 'doctor', 'lawyer'] %}     {% prompt %}A {{ career }} wearing a {{ career}}'s outfit{% endprompt %} {% endfor %}, masterpiece
OUTPUT:  A lawyer wearing a lawyer's outfit
Steps: 35, Sampler: DPM++ 2S a, CFG scale: 12, Seed:<PHONE_NUMBER>, Size: 512x512, Model hash: 81761151, Model: Stable1.5, Eta: 0.77, aesthetic_score: 5.5
And the same for the other two careers.

The entire prompt needs to be in the {% prompt %}...{% endprompt %} block, e.g.
{% for career in ['teacher', 'doctor', 'lawyer'] %}
    {% prompt %}professional digital airbrush art of A {{ career }} wearing a {{ career}}'s outfit, masterpiece{% endprompt %}
{% endfor %}

Also, why couldn't you update via auto update? Did you get an error message?

The entire prompt needs to be in the {% prompt %}...{% endprompt %} block, e.g.
{% for career in ['teacher', 'doctor', 'lawyer'] %}
    {% prompt %}professional digital airbrush art of A {{ career }} wearing a {{ career}}'s outfit, masterpiece{% endprompt %}
{% endfor %}

OH!  OK, that worked!

Also, why couldn't you update via auto update? Did you get an error message?

I did get an error message, can't grab it now because the update tab is now telling me Dynamic Prompts are up to date.  But before that, it was something like this message for the prompt travel script (same messagr for your script):
Error pulling updates for stable-diffusion-webui-prompt-travel:
Traceback (most recent call last):
File "E:\StableDiffusion\modules\ui_extensions.py", line 39, in apply_and_restart
ext.pull()
File "E:\StableDiffusion\modules\extensions.py", line 70, in pull
repo.remotes.origin.pull()
File "E:\StableDiffusion\venv\lib\site-packages\git\remote.py", line 910, in pull
res = self._get_fetch_info_from_stderr(proc, progress,
File "E:\StableDiffusion\venv\lib\site-packages\git\remote.py", line 750, in _get_fetch_info_from_stderr
proc.wait(stderr=stderr_text)
File "E:\StableDiffusion\venv\lib\site-packages\git\cmd.py", line 502, in wait
raise GitCommandError(remove_password_if_present(self.args), status, errstr)
git.exc.GitCommandError: Cmd('git') failed due to: exit code(128)
cmdline: git pull -v origin
stderr: 'fatal: unable to auto-detect email address (got '[usernameredacted]@[computernameredacted].(none)')'
Also, when it restarts the Detection Detaile script throw some errors and I am told PIP is out of date.  Since the erors started after installing a few scripts, i figure they are not playing nice with each other, so it must be a config issue on my side, or witin AUTO.  I'm going to update PIP to see if that helps

If you come across another installation issue that you think might be related to this extension, please create a new issue.


chosen/select2 abstraction

It would be great to use this library with chosen or select2.
(I believed select2 was based on chosen but I didn't find any official confirmation).
I know there's already a angular-ui ui-select2 but I prefer this one :

$translate 2 support (translations with promises)
ngOptions support
...

We have ngOptions Support!
For translate, you're able to change 3 messages from Chosen Library.

no-results-text="'No results match'"
placeholder-text-multiple="'Select Some Options''"
placeholder-text-single="'Select an Option''"

Here is an example in Portuguese language: http://plnkr.co/edit/P3bWW2?p=preview


Revert Eclipse classpath separation

As discussed on Slack, this feature needs some rework before it goes in, so let's revert for 4.3 and do it in 4.4 instead.

Done by https://github.com/gradle/gradle/commit/0f7ac32fbfd8cf7181e0e47ee059496bb77aa4cb

This issue is listed in the release notes as fixed. Maybe we want to remove it from there. Should we just remove the milestone tag?

We probably shouldn't list "chores" in general. The milestone is useful to know when we are ready to release (no more stories with that milestone).


Glitches when toggling Dark Mode

I received the following feedback:

When first selecting dark mode, the [dark mode] button stays highlighted with a light background after the theme switches. Subsequent toggles fix this instantly, but thought it was worth mentioning to you.

UI and theme will change in new apps


[Icon Request] Support for Biml (Business Intelligence Markup Language)

[x] I'm sure this issue is not a duplicate.

[x] I'm submitting an icon request:

Type: languageId
Icon Name: Biml
Sample original Icon:
Language ids: biml

Language Extensions: https://marketplace.visualstudio.com/items?itemName=zbecknell.biml-support

More info: https://www.varigence.com/Biml

I received a request to add biml support, which is very similar to my existing T4 template syntax extension.

@zbecknell You own extension at marketplace? Nice! 👍

Yes, I added the extension for Biml support.  Thanks so much for the great work on vscode icons!


FEAT: Add GlacierCTF Writeups

yeah gotta add some more writeups but closed too early cuz I linked it

I don t have any for this one


Require schema

Replaces #32 with just the required commits. From that PR:
Check that there's a schema file of the expected name (type.[jinja|py].schema).
Add newlines to types list.
Fixes #22.
In particular:
requiring a schema file in the version directory (e.g., types/redis/v1/redis.jinja.schema)

I signed it.


Format modal client deprecation warning better

We have some code that does rich formatting of deprecation warnings that was malfunctioning for client deprecations:

The main problem is that we assume that modal.exception.DeprecationError warnings are issued from modal.exception.deprecation_warning, which prefixes the message with the date of deprecation. But when we issue the client deprecation warning we don't include a date.
We also have a flag in deprecation_warning to suppress the source of the error in cases where the surrounding context is not relevant for the warning; I copied that approach here to clean up the message. Result:

I don't love how the alarm emoji look here, but they're helpful in the app logs. Could maybe omit them when modal.is_local.

cc @erikbern is there somewhere else we're handling the client deprecation warnings now that we're trying to get rid of ClientHello?


Update the documentation to import a subset of k8s logs

Updated the doc to import a subset of k8s logs
Details:
Updated documentation for Log collection for a subset of pods or containers
File modified:
charts/opentelemetry-collector/README.md

@TylerHelmuth, Can you review & approve the PR: https://github.com/open-telemetry/opentelemetry-helm-charts/pull/1276


Não tocar música

:bug: Describe the Bug

A clear and concise description of what the bug is.

:pencil2: Steps to Reproduce

Go to '...'
Type '...'
See error...

:confused: Expected Behavior

A description of what you expected to get (if applicable).

:scroll: Log
Paste a relevant portion of the console log here.

:camera: Screenshots

Paste a screenshot of the bug (if applicable).

:question: Other Information

Node Version: Version of the node runtime.
Ñ toca música

I'm assuming you're the person in the screenshot because of the same name.

Your problem is that you don't have ffmpeg installed. You can install it with
npm install --save-dev ffmpeg-static

(By the way, your issue is not very descriptive, you should've uploaded the screenshot you sent me here.)


Extract HTTP::Client#build_request method

It would sometimes be handy to have a public method that returns a request object so that we can modify it and/or pass around and then use perform on it.
Please take a look and if you find the idea is OK I will add specs etc.

I think we will completely refactor this bit in v4.0.0; I think that after all #request must return Request instance, that will respond to #perform. In any case I'm fine to merge it to v3.x branch and to the master (for sake of keeping things in sync).
@tarcieri any objections?

So, is it going to be merged?

Yeah. I think it's good. Thank you!

@ixti Can we backport it to version 3 by the way?

@tycooon Sure. It does not brings any breaking changes - so totally fine - will handle this.

@tycooon REleased as 3.3.0

What's the way to perform the built request?

What I now is:
request = HTTP.build_request(...)
client = HTTP::Client.new
client.perform(request, client.default_options)


Mounts shared pvc to shared folder.

reolves https://github.com/canonical/data-science-stack/issues/102
based on the comment we will mount pvc to shared instead of the home folder.

thanks @misohu !
As a first iteration this is ok, but I don't believe this PR resolves #102 entirely.
We still need to keep track of the piece to have a separate PVC for each notebook, so I suggest before you merge, either:

change the description and keep the issue open
or file a new issue for the separate PVCs.


BUG: migrate the login lifecycle specification to the wiki

Source: https://people.redhat.com/sgrubb/audit/user-login-lifecycle.txt

Migrated to https://github.com/linux-audit/audit-documentation/wiki/SPEC-User-Login-Lifecycle-Events


[Snyk] Upgrade @hashgraph/sdk from 2.46.0 to 2.47.0

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to upgrade @hashgraph/sdk from 2.46.0 to 2.47.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.

The recommended version was released on 25 days ago.

Release notes

Package name: @hashgraph/sdk

2.47.0 - 2024-06-14What's Changed

revert: mirror node queries changes by @ svetoslav-nikol0v in #2328
feature:  change or remove existing keys from a token [HIP-540] by @ svetoslav-nikol0v in #2299
proto release: v2.15.0-beta.1 by @ svetoslav-nikol0v in #2351
release: v2.47.0 by @ svetoslav-nikol0v in #2352

Full Changelog: v2.46.0...v2.47.0

2.46.0 - 2024-06-06What's Changed

ci: updated for best practices by @ rbarkerSL in #2260
chore(deps): bump step-security/harden-runner from 2.7.0 to 2.8.0 by @ dependabot in #2295
fix: typo in SDK query file by @ svetoslav-nikol0v in #2293
fix: enhanced Logger to accept a log file location by @ jeromy-cannon in #2298
Bump actions/deploy-pages from 4.0.4 to 4.0.5 by @ dependabot in #2212
Bump actions/setup-java from 3.13.0 to 4.2.1 by @ dependabot in #2202
chore(deps): bump renovatebot/github-action from 40.1.0 to 40.1.12 by @ dependabot in #2323
chore(deps): bump pnpm/action-setup from 3.0.0 to 4.0.0 by @ dependabot in #2286
fix: naming convention by @ svetoslav-nikol0v in #2303
feat: solution to query the mirror node for the account balance, account info, and contract info data by @ svetoslav-nikol0v in #2289
release: stable version v2.46.0 by @ svetoslav-nikol0v in #2325
update: release workflow by @ svetoslav-nikol0v in #2326

New Contributors

@ rbarkerSL made their first contribution in #2260
@ jeromy-cannon made their first contribution in #2298

Full Changelog: v2.45.0...v2.46.0

    from @hashgraph/sdk GitHub release notes

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

merged via dependabot PR


RecursionError

I am trying to run yolox with the coco dataset, when I call the train_detector I get an error as shown below:
RecursionError: maximum recursion depth exceeded in comparison
Does anyone, have an idea how we can resolve it?

Hi @Anurich ,
Could you follow the issue template to provide more details? For example, which mmdet version do you use?

anny progess on this

I also had this problem, did anyone come up with a fix? I got this error trying to install mmdet 2.21 after running command "mim install mmdet==2.21"


Add very simple installer; show dialog to choose vanilla folder and copy resources

Summary
Warning: it is experimental feature in early stage!
If all resources already exists, it does nothing. If not, Elona foobar shows a dialog to choose vanilla folder and copy the required resources from graphic and sound folders of vanilla to foobar folder. To check whether the selected elona is v1.22, foobar checkes graphic/deco_blend.bmp, which were not included in v1.16.

Closed the PR because it is hard to merge it into the current develop.


DP Creator Demo Dataverse: Display warning not to upload private data

On the Dataverse, including a warning not to upload private data. This would be in a similar place as the note here: https://demo.dataverse.org/

Instructions added under deploy/DATAVERSE_NOTES.md


Add changes to expose metric for installed knative version

JIRA: https://jira.coreos.com/browse/SRVKS-187
Description:
This PR include changes to expose installed knative version information to metric and servicemonitor.

All Review comments addressed from this PR https://github.com/openshift-knative/knative-serving-operator/pull/49


Feature request: Integrate DevStyle

DevStyle is a huge improvement over the native dark theme. Is there any possibility to bundle it by default?

You can install it manually using this instruction.

Yes, this would be quite handy to include it. However there are a few concerns:

DevStyle plugins aren't open source and it's devlopment can be stopped or it my become commercial.
It is not well-doumented and thu is so not easy to customize DevStyle for a particular product.

Hoever I'll mark this as FR, perhaps we'll find a way to include DevStyle, maybe with some simplified installation process.

DevStyle p2 repository config was added in both CE and EE so it can be installed in two clicks.
I think we won't make any deeper integration.


test: test for lua rules erroring out - v2

Test for correct behvaior when a Lua rule crashes.
On crash, a Lua script should not result in a match when negation is
applied. Also test for:

error stat
one time logging per thread

Ticket: https://redmine.openinfosecfoundation.org/issues/6940
Expected to fail on current branches.
New version pushed to break some caching.

Merged in #1835, thanks!


Context refactor

Consolidates state into a single context
Adds support for stepping through Appear steps with left/right arrow keys and space bar - fixes #68
Reorganizes modules
Adds tests for updater functions
Simplifies Appear component
Adds metadata state for notes, steps, etc.
Exports withDeck HOC for context consumption

cc @pomber: Want to be sure you're aware of some of these internal changes. Hopefully this will make the code-surfer integration simpler, but also might run into some breaking changes. I can help out/clarify the changes over the next few days to make sure nothing breaks

I like it. Much simpler.
CodeSurfer component is similar to Appear so I think I won't have problems adapting it.

@pomber if you're cool with the proposed changes here, I might go ahead and roll this out, but if it's helpful for me to publish a prerelease for testing first I could do that. Also happy to help with any missing docs or updates that might be helpful for CodeSurfer

A prerelease would be nice :). I'll try to test the new integration tonight.

Cool, it's published as mdx-deck@alpha (v1.7.0-0) – I'm guessing the Appear component will serve as a reference, but feel free to ask any questions or let me know how it goes

It mostly worked.
I'm having some minor issues with deck.step when going backwards from one slide to the previous one.
I'm implementing sCU like this:
    shouldComponentUpdate() {
      return this.props.deck.active;
    }

but I'm not sure if it's the correct way to do it because I'm missing some updates when the slide changes.
Anyway, if you want to release 1.7 go ahead, I'll publish a new verion of CodeSurfer afterwards.
Thanks!


metrics: add nydus daemon version

Add nydus daemon version to prometheus format metrics.
# HELP nydusd_version_string The version of nydus daemon.
# TYPE nydusd_version_string counter
nydusd_version_string{version="v2.1.0-rc.3.1-473-gf4dd9e1c"} 1

Moreover, remove some metrics that have been deleted in nydus and improve descriptions for daemon FS metrics.
Signed-off-by: Bin Tang<EMAIL_ADDRESS>

Codecov Report
Base: 31.81% // Head: 31.72% // Decreases project coverage by -0.09% :warning:

Coverage data is based on head (237a839) compared to base (ef09f76).
Patch coverage: 0.00% of modified lines in pull request are covered.

:exclamation: Current head 237a839 differs from pull request most recent head ed3a82b. Consider uploading reports for the commit ed3a82b to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #302      +/-   ##
==========================================
- Coverage   31.81%   31.72%   -0.10%
==========================================
  Files          34       34
  Lines        3564     3575      +11
==========================================
  Hits         1134     1134
- Misses       2314     2325      +11
  Partials      116      116

Impacted Files
Coverage Δ

pkg/daemon/daemon.go
0.00% <0.00%> (ø)

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

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

/retest


merge is not a function

Since webpack merge package has been updated to use destructure
Doc example here https://webpack.js.org/guides/production/#setup
was giving the following error

Solution was simply to edit and add destructure to merge package imports like this

Hope it Helps!

Where @EugeneHlushko

Do you mean file i've changed where be specific

@evilebottnawi Do you mean file i've changed where be specific?

See your diff, you've changed __ to ** everywhere, revert those changes

Fixed Thank you for you time it was my prettier setup but i have fixed it I genuely appreciate all of you

@makuzaverite can you sign the CLA please?

Thanks!


Update CryptoKey documentation to mention ECC.

The documentation makes it seem like CryptoKey can only hold an RSA key.  This is compounded by the fact that Cypto only has a function generate an RSA based key.  Godot however is perfectly happy loading and using ECC based keys.

Example showing signing and verifying with an secp256k1 key.
var key = CryptoKey.new()
key.load_from_string("""-----BEGIN EC PRIVATE KEY-----
MHQCAQEEIGIumMFpusqAcqCkwN15oFS/QXNIf9LsmTCYMNwp9/G8oAcGBSuBBAAK
oUQDQgAEigp/7pvDMxn0mNDDuJuepWgWeXTw71jNk/aSo8LqyTSHXULaUbZh3G57
G3P3LqXcwRgm2Wz7J+9JoGzDokr1Pw==
-----END EC PRIVATE KEY-----""")

print(key.save_to_string())
print(key.save_to_string(true))
var pub = CryptoKey.new()
pub.load_from_string(key.save_to_string(true), true)

var ctx = HashingContext.new()
ctx.start(HashingContext.HASH_SHA256)
ctx.update("Hi!".to_ascii_buffer())
var hash = ctx.finish()

var sig = Crypto.new().sign(HashingContext.HASH_SHA256, hash, key)
print("SIG", sig)
print("Ok?", Crypto.new().verify(HashingContext.HASH_SHA256, hash, sig, pub))

As we're just passing this through to mbedtls, I'd expect both RSA and ECC to work, yes. Adding a link to the Crypto class for actually generating keys would also make sense. We should also make a note to add ECC key generation there, I think.

We should also add ECC key generation there.

I was looking into this, but held off for now.  It needs a bit of API design on how the user specifies which curve they want to use and also how to enumerate the available curves along with their bit size.

Could you amend the commit message to be more explicit, like the PR title?

Could you amend the commit message to be more explicit, like the PR title?
Edit: Let me know if you're not familiar with how to do this with Git, I can also do this update myself.

Done :)

Thanks!


Integrate knex.js seed

It would be really nice to have a functionality to insert test or default data, after running database migrations. There is already an CLI as in knex.js docs outlined.

Seed files allow you to populate your database with test or seed data independent of your migration files.

Are there any plans to integrate that feature? What do you think about it?

check out trello its planned there


[DOC] Add links to Best Practices for Security and Getting Started in Security

What do you want to do?

[x] Request a change to existing documentation
[ ] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
A 'best practices for security in OpenSearch' was recently added, but it seems a bit hidden. Should we move this up, and/or link to it from other topics, for instance, https://opensearch.org/docs/latest/security/?
We also have a new 'Getting Started in security' topic that should also be linked to other topics, for instance, https://opensearch.org/docs/latest/security/.
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.
https://opensearch.org/docs/latest/security/configuration/best-practices/
https://opensearch.org/docs/latest/getting-started/security/

PR created.


[Task] Refactoring of Google Sheets plugin for better UX

[x] Differentiate between Authorize and Re-authorize
[x] Swap the text to say Authorize and then Save
[ ] Move the Create API button to the bottom of the form, add logic to only enable this button after authorization
[ ] Also make the Re-authorize button of secondary type
[ ] Attach templates for all requests that need a body

Dropping a few notes from @aakashDesign here:

[ ] The google sheet icon on New API page isn't the right size, and a little skewed. We can try 48x48px
[ ] Can we jump to new datasource form if there are no gsheet datasources.
[ ] Hide api button and reauthorise button design.
[x] We should select read/write by default here.

@hetunandu @ApekshaBhosale could one of you please help with these?

@nidhi-nair would you please explain the second point? //Can we jump to new datasource form if there are no gsheet datasources.//

@rohan-arthur I think that one's been fixed as part of a redesign project some time last year. The issue was that when there were no GSheets datasources configured for the app, the new datasource page used to still go to the list of available datasources and have an option to create a new datasource as another click from there.

Thanks Nidhi. Closing the issue then


FormFutura EasyFil ePLA (Cardboard Spool)

Printing temp: 200-220C
Bed temp: 0-60C
Empty spool weight: 165g
Advertised filament weight: 1000g - 1kg
Outside Diameter: 200mm
Height: 52mm
Hole Diameter: 56mm

the empty spool is 10g heavier than FormFutura ReForm - rPLA (Cardboard Spool)

Thanks, I will add. It seems the weight of the cardboard spools generally vary a lot

But actually -- we will need picture of the empty spool placed on the scale

i didn't take any photo and i trashed it afterwards. i will also not buy this filament again as its pretty much trash.
the only reason i measured it because i used the 155g value from the website to calculate my print but because it was wrong the print didnt finish. i wanted to help others avoid this 👍🏻


UG: Add Get Started section

closing #17

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

Coverage data is based on head (47c4dac) compared to base (20556b4).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #60   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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

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


[YouTube8M] How to extract the pb file by python

Since I ran code in https://github.com/tensorflow/models/tree/master/research/audioset/vggish and got a different result from this code, I want to check the following pb file to see if the params are the same.
curl -O http://data.yt8m.org/pca_matrix_data/inception3_mean_matrix_data.pb
curl -O http://data.yt8m.org/pca_matrix_data/inception3_projection_matrix_data.pb
curl -O http://data.yt8m.org/pca_matrix_data/vggish_mean_matrix_data.pb
curl -O http://data.yt8m.org/pca_matrix_data/vggish_projection_matrix_data.pb
The decoding code is written by C++, so I wonder if there are some ways to extract thest pb file by python simply.
Could you plz release a example py file to do this?
C++ is too hard to read for me :(

Can anyone answer this question plz?

Hi @gaoruiyang, Could you please respond whether you are still looking for the above query. And share us more your observation with the MediaPipe Youtube8M solution.


All lights turned off on boot

I'm using this plugin to control a custom server and whenever I boot home bridge all lights are sent a power off message after fetching their status, regardless of their status.
I can't find if this is related to this plugin or homebridge.
Thoughts?

I noticed that this only happens when switchHandling is set to realtime, if I set it to yes this doesn't happen.

Same thing.

Consider this a +1


ifvisible is not defined

Please confirm that ifvisible.js was loaded before you using it. Maby you pass it via the closure:

(function (ifvisible) {

if (typeof ifvisible === 'object') {
}

})(this.ifvisible);

Good idea!  Do you have any recommendations on what to do if ifvisible is not defined?  I could include a console.log() message, but I don't know if people typically want log messages (even during exceptional cases like this) to be written to the console.  Perhaps I should throw an exception instead?

I suggest to write an internal method ifVisible for the external library like:

(function (ifvisible) {
..
..
..
getVisible: function() {
    if (typeof ifvisible === 'object') {
      //return the ifvisible object
    }
    else {
      throw 'error, warning or whatever you think';
    }
}
..
..
})(this.ifvisible);

Later on I suggest to use this internal method / wrapper inside a listen method that is made for the event handling... ifvisible.on(...)

listen: function () {
..
TimeMe.getVisible.on('...')
..
}

Finally you let the user start the listen method or add it to your init method.

I put together a pull request: https://github.com/jasonzissman/TimeMe.js/pull/11.  Care to take a look at it? Thanks as always for the help.

Looks good

Please create a new tag like 1.1.0

Done!


BNodeGenerator.generate function is wrong

generate func if expression condition is incorrect.

Use the 13-set-up-cucumber branch
Please remove the fisrt (value == null) condition.
Then commit and push

The modifications are done as requested.


why status stopped

For the XMR miners that always means that either the pool is not valid or it has stopped for some reason, you can check back in a few minutes to see if it's stilled "Stooped". Though why do you have two XMR miners on the same computer?

botnet are executed 3.1.1 and 3.4.0 so i will pused one 3.1.1

do you can make any fake av simiulator for me i will pay you ?
like avast or any just dummy fake antivirus program

or do u know any github project exiting ?

i want to set av status always "you are protected"
no need extra futures
please tell me bro ?

I don't do any paid projects, though I wouldn't have enough free time to make that either way. I don't think there are any existing projects exactly like that since there's no reason really to make one. There are fake antiviruses on GitHub but many of them are malware.

but it's very helpfull for spreding bypassing av
anyway thank's bro....
i have one quation are you never sleeping ?
you always active how is possible ?

Haha, it's just that I'm at the computer almost all the time so it might seem that way.


PyTorch or TensorFlow

Can a new example be built on the TensorFlow framework?

More specifically I have TensorFlow based deep neural network model. How should I proceed if I want to upload it as an example?

Hi, we currently do not support TensorFlow even though it is in our future plan. Currently, you need to implement your model in PyTorch and create an example.


window.location is undefined..

I'm submitting a...

[ ] Bug report
[ ] Feature request
[ ] Documentation issue or request

In package

[ ] @ng-toolkit/init
[ ] @ng-toolkit/serverless
[ ] @ng-toolkit/universal
[ ] @ng-toolkit/pwa
[ ] @ng-toolkit/firebug

Current behavior

Added "@ng-toolkit/universal": "^1.1.41-beta.110" module.
Injected Window Dependency in the service.

import { WINDOW} from '@ng-toolkit/universal';
constructor(
@Inject(WINDOW) private window: Window
) {
console.log('Window' + JSON.stringify(this.window));
console.log('location' + JSON.stringify(this.window.location));
this.preview =  this.window.location.pathname.includes('private') || this.window.location.origin.includes('localhost'); }
}
Output:
"this.window" gives me [Window{"navigator":{"userAgent":"fakeAgent"}}]
"this.window.location" gives me "undefined"
I'm unable to access any property inside the window object.
Expected behavior

Should be able to access the window object and its properties in the code as the window dependency has been injected.
Minimal reproduction of the problem with instructions

Example repository

What is the motivation / use case for changing the behavior?

Environment

Angular version: X.Y.Z

- Node version: XX
- Platform:

Others:

⚠ Closed as invalid issue. Please, fill the issue template when opening new issues in order to help us track the bug.
Cheers!


Take out "debugger switch"

In a recent commit, I added the ability to switch between the old debugger (JadeDebugger) and the new debugger @jgfoster is working on. (RowanDebugger).
Before we release a version of Oscar to the customer, and when the new debugger is "good enough", take this out and default to the new debugger.

[x] Remove transcript menu item
[x] Default to RowanDebugger

Removed switch, always use new debugger.
Done so we can ship to customer soon. Fixed in 3.0.16

And the pressure is on!

I think this is not closed and ought to be on the to-do list, since, in fact, it represents a task that is not done.  I suppose it could be deferred.

The switch isn't in 3.0.38 so calling this fixed.


jQAenhencement alters database

Steps to reproduce:

Generate a RD visualization from a database created with jQA
Run the generation process a second time

Expected results:
In both cases the same visualization should be generated
result:
The second visualization has an additional package ring. Probably, the jQAEnhancement creates an additional node or edge that causes this.

fixed on development branch


Check the status of operation when creating or deleting workspace.

…r deleting workspace

@jerrychenhf Hi haifeng. I've supported to check global and regional operation status when creating or deleting workspace on GCP.

Looks good to me. Thanks you!


Move documentation to BioSCape-io organization

Just to keep all our resources together, can we move this to https://github.com/BioSCape-io
@EvanDLang

Done!


Android Lint Failure - DataExtractionRules

Describe the bug
Both Core and Sign SDKs cause a lint failure on API 33 when added as dependencies due to use of deprecated xml attributes.
SDK Version

Client: Kotlin
Version 1.3.0

To Reproduce

Add Core or Sign SDKs as dependencies to your app.
Run ./gradlew :app:lintRelease

Expected behavior
No lint failures.
Screenshots
AndroidManifest.xml:9: Error: Missing <full-backup-content> resource [DataExtractionRules]
    <application android:fullBackupContent="@xml/backup_rules" />
                                            ~~~~~~~~~~~~~~~~~
AndroidManifest.xml:9: Error: The attribute android:fullBackupContent is deprecated from Android 12 and higher and may be removed in future versions. Consider adding the attribute android:dataExtractionRules specifying an @xml resource which configures cloud backups and device transfers on Android 12 and higher. [DataExtractionRules]
    <application android:fullBackupContent="@xml/backup_rules" />
                                            ~~~~~~~~~~~~~~~~~

   Explanation for issues of type "DataExtractionRules":
   Before Android 12, the attributes android:allowBackup and
   android:fullBackupContent were used to configure all forms of backup,
   including cloud backups, device-to-device transfers and adb backup.

   In Android 12 and higher, these attributes have been deprecated and will
   only apply to cloud backups. You should instead use the attribute
   android:dataExtractionRules, specifying an @xml resource that configures
   which files to back up, for cloud backups and for device-to-device
   transfers, separately. If your minSdkVersion supports older versions,
   you'll still want to specify an android:fullBackupContent resource if the
   default behavior is not right for your app.

   https://developer.android.com/about/versions/12/backup-restore#xml-changes

@prashanOS We've updated the backup rules for API 33 in Core 1.4.1 and Sign 2.7.1


Fix code scanning alert - Dereferencing NULL pointer 'history'.

Tracking issue for:

[x] https://github.com/nyankyu/AI-reversi/security/code-scanning/3

It was referring before the error check of malloc() was done. It was corrected so that it might refer after the error check was done.


Adding roadmap R2

Description
Adding roadmap of release2
Status
READY
Type of change

Documentation (fix or adds documentation)

Checklist:

[x] I have made corresponding changes to the documentation (if applicable)
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

It's Ok


Data Platform: Fix Shared VPC members

Hey Lore, we're normalizing exchanges between stages and are almost done (see #549). I'll get on this right after we merge that PR, and do the changes to align myself if you don't mind.

ah sorry, this is for the example :facepalm:


Merge pull request #29 from auchenberg/greenkeeper-tap-5.0.0

Update tap to version 5.0.0 🚀

@sky1818 what's the intent of this PR? It doesn't change anything


Error when var is used in fragment only

Error happens, when var is used in a fragment, but not on the top level
Steps to reproduce:

initialize with Anilist public graph API elm-gql init https://graphql.anilist.co
create the following query

query ($search: String!) {
  Page {
    ...foo
  }
}

fragment foo on Page {
  characters(search: $search) {
    name {
      full
    }
  }
}

run elm-gql run ./api/Api/schema.json
observe the following error in the log

--ELM GQL------------------------------------------------------src/myQuery.gql--

I found the following variables:

    $search

But I ran into an issue:

    $search is unused.

Here's what I think the variables should be:

note that same query works in GraphQL explorer https://studio.apollographql.com/sandbox/explorer?endpoint=https%3A%2F%2Fgraphql.anilist.co

I've had the same issue but the error thrown was a bit misleading. May be worth looking into as well.
I had a query like this:
query ($topVar: String!, $fieldVar: String!) {
  topLevel(topVar: $topVar) {
    ...Record
  }
}

fragment Record on Record {
  fragmentQuery(fragmentVar: $fragmentVar) {
    fieldWithVar(fieldVar: $fieldVar) {
      result
    }
    subRecord {
      ...SubRecord
    }
  }
}

The error thrown was "can't find fragment on type SubRecord".
If I re-run elm-gql the error was ignored but the file was unchanged.
Sometimes it complained about other (valid) fragments but most times it complained about the type nested alongside the fragment that used a variable.
To find the actual error, I inlined the SubRecord fragment into Record - then I got the unused variable error.
query ($topVar: String!, $fieldVar: String!) {
  topLevel(topVar: $topVar) {
    ...Record
  }
}

fragment Record on Record {
  fragmentQuery(fragmentVar: $fragmentVar) {
    fieldWithVar(fieldVar: $fieldVar) {
      result
    }
    subRecord {
      fieldA
      fieldB
    }
  }
}

Finally, I inlined Record so the variable is used in the same scope and re-inserted SubRecord as a fragment and everything worked.
query ($topVar: String!, $fieldVar: String!) {
  topLevel(topVar: $topVar) {
    fieldWithVar(fieldVar: $fieldVar) {
      result
    }
    subRecord {
      ...SubRecord
    }
  }
}

fragment SubRecord on SubRecord {
  fieldA
  fieldB
}


Add Wagtail prefix to non-taggit settings

Is your proposal related to a problem?
All of our settings are prefixed with WAGTAIL_ except for two Wagtail specific tagging settings.

TAG_SPACES_ALLOWED
TAG_LIMIT

Describe the solution you'd like
Adding an alias for the other non-Taggit settings, to have a name spaced WAGTAIL_ prefix, while still supporting the current naming but triggering a warning.

TAG_SPACES_ALLOWED -> WAGTAIL_TAGS_SPACES_ALLOWED
TAG_LIMIT -> WAGTAIL_TAGS_LIMIT

See how a similar rename was done a while ago https://github.com/wagtail/wagtail/pull/11525 (including warnings RemovedInWagtail70Warning).
This way we have Wagtail prefixed (namespaced) settings once the next major release bump happens.
Additional context
The PR #12564 introduced a hard limit on number of tags returned to 10 to avoid fetching all items possible which was causing performance issues for large databases - see https://github.com/wagtail/wagtail/issues/12415 .
There may be a future desire to make this hard limit configurable, for now though we will not cover that on this issue.
Working on this

PR WIP already https://github.com/wagtail/wagtail/pull/12639
Anyone can contribute to this. View our contributing guidelines, add a comment to the issue once you’re ready to start.

@lb- is there a particular direction you had in mind?

@aayushman-singh  I have updated the description, something like that should be good I think, including a clean up of the existing settings.
We probably should use delay not debounce for the setting name, we may also update this on the controller also.
What do you think?

Makes sense, thanks for the improved description. I'll get started after the .js PR, let me know if we want to use delay instead of debouce for the controller.

I'd question whether we really need a configuration setting for this. What kind of situation do you envisage where a user wouldn't find the current hard-coded limit or debounce delay acceptable? They might have a personal preference for some other value, but there are any number of other equally minor details about the Wagtail admin UI that aren't currently customisable (like, say, the ordering and layout of forms in the Settings area). At some point, it's more sensible (and arguably a better developer experience) for Wagtail to just make appropriate choices and stick with them, rather than presenting endless options for customisation that make it harder for developers to find the specific settings that they are interested in.

You make a valid point. This issue originally arose because someone found the lack of control over this configuration frustrating, which led to the hardcoding of smaller values. It might be better to have this flexibility available, even if it's not always needed. However, we can change this approach to whatever suits Wagtail dev experience better.

I may have missed some of the earlier discussion, but did their frustration come from the lack of control over the setting, or more that Wagtail originally had badly-chosen values for these limits (or rather: didn't set a limit at all), leading to performance issues and poor user experience? If it's the latter, then the current solution of setting better values is enough.
One reason I don't like introducing new config settings is that it introduces "action at a distance" - you end up with one part of the code (namely, the settings file) having control over many aspects of the application that are very far removed from that location. If we feel that configurability is important here, I'd like to look for approaches that are "closer" to where the feature is being used. For example, maybe it could be a parameter on the form widget, so that when you're defining a page model with a tag field on it, you would write:
FieldPanel("tags", widget=TagWidget(limit=20))

But that will introduce more "moving parts", and I think we should only go ahead with it if there's a demonstrated need for it.

Makes sense, we can update the issue to simply adding an alias for the other non-Taggit settings, if that is preferable.

Thanks @gasman  - good call on this.
@aayushman-singh  are you able to update your PR to just cover the renaming of the current Wagtail tag settings only?
I will update this issue accordingly but add some notes for future, if there's a demand/need for more control over these it can be raised as a separate issue.

@lb- , @gasman thanks for the guidance, I've updated the PR accordingly.

can i work on this?

I've got it covered, thanks.


🌐 Add Azerbaijani translation for docs/az/docs/index.md

index.md translated to Azerbaijani

@tiangolo please tag lang-all and lang-az

Awesome! Could you please remove the .DS_Store files?

Thank you check, all removed

📝 Docs preview for commit ad4a2489b78b9a0f9b368a283dac3e46eb7a39b8 at: https://649a2c7805f6af1e7bd6ced6--fastapi.netlify.app

As this PR had requested changes to be applied but has been inactive for a while, it's now going to be closed. But if there's anyone interested, feel free to create a new PR.


Using Raspberry Pi Stream with h264-live-player server-tcp

Hi there,
Thanks for this amazing repository, it save my time to understand how to decode h264 stream on browser. The serve-rpi.js worked good on raspberry Pi, so I try to run use server-tcp.js with standalone raspivid stream service. Somehow, I got problem on doing this, would you like to help me?
Raspberry Pi(stream video on :3030):
$raspivid -n -w 200 -h 200 -t 0 -l -o tcp://<IP_ADDRESS>:3030 -pf baseline -fps 12 -b 950000

Server TCP:
const feed    = new RemoteTCPFeedRelay(server, {
  feed_ip   : "<IP_ADDRESS>",
  feed_port : 3030,
  width: 200,
  height: 200
});

I can receive the data on browser, but the video looked terrible.
Picture:

Thanks!

Fix this when I set the width and height to 320 and 180.
How could I change the width:height for decoder?

I am also meet the same problem，my vedio is record from Android device.
Only when vedio size in 720x1280 is ok. Others vedio size like 616x1280, 1080x1920 all wrong.
How could I change the width:height for decoder?

Sorry, no time to look at it right now..

Update:
I found the related issue in broadway decoder, they also require 16:9 video.
https://github.com/mbebenita/Broadway/issues/137

I am also meet the same problem，my vedio is record from Android device.
Only when vedio size in 720x1280 is ok. Others vedio size like 616x1280, 1080x1920 all wrong.
How could I change the width:height for decoder?

Have you solved this problem on Android devices? i alse meet the same problem，720x1280 is ok, other size failed。


avoid downloading again already existing chunks

Since we know where chunks are expected to be written before starting the download
and
Since chunks are written after they have be fully downloaded (and stored into memory)

Then it should be possible to avoid downloading chunks already present on-disk
The only possible issue with this are chunk partially written on-disk (eg: insufficient disk-space at the time of writing)
To avoid corrupted chunk being stored on-disk, a callback() could remove the partially written file (== if fs.writeFile() returns an error)

Thanks for opening this issue. I will start working on it as soon as I can allocate some bandwidth for this.


Small fixes for 0.9.4 release

This PR implements a number of small fixes for issues tagged for resolution in release 0.9.4:

Updated installation documentation to recommend conda installation first, then pip (resolves #850)
Added more documentation about Nyquist plot scaling (resolves #830)
Deprecated issys() (resolves #876)
Changed short package name for control.optimal from opt to obc consistently throughout documentation.
Fixed sphinx processing of version to allow proper links to source code (resolves #837).
Added numpy 1.24.2 to list of exceptions for flatness test on linux using Generic BLAS (resolves #866).
Additional updates to docstrings and rst files as I went through the above items.

Coverage: 94.867% (+0.001%) from 94.866% when pulling ee173f21df9370622e1365ca7e5245260bff4b62 on murrayrm:smallfixes-26Mar2023 into 2c5c2f01302372e9743e4a95a51194a144fa5737 on python-control:main.


Investigate switching dart cli to use an AOT snapshot

AOT has significant faster startup performance than JIT, and it's getting very comparable perf to even warmed up JIT. For example, here is the current perf with the benchmark provided by the dart_style CLI:
$ dart run benchmark/benchmark.dart
Run  #1: 10.87ms ======================================================
...
Run #91: 5.90ms =============================
Best   : 5.90ms =============================

$ dart compile exe benchmark/benchmark.dart
Generated: /Users/mit/tmp/dart_style/benchmark/benchmark.exe

$ ./benchmark/benchmark.exe
Run  #1: 7.00ms ===================================
...
Run #25: 6.87ms ==================================
Best   : 6.87ms ==================================

I would like to see us investigate if we can swift the Dart CLI to use AOT.

Postponing for now; this would require major architectural changes in the VM.


New version: KNITRO v0.9.3

Registering package: KNITRO
Repository: https://github.com/jump-dev/KNITRO.jl
Created by: @frapac
Version: v0.9.3
Commit: 6fbee14d2e2417e81da93323abff92339a4cb7dd
Reviewed by: @frapac
Reference: https://github.com/jump-dev/KNITRO.jl/commit/6fbee14d2e2417e81da93323abff92339a4cb7dd#commitcomment-44963143

[noblock]
CI will fail because this package has a commercial third-party dependency.

I'm going to merge this, but it'd be really helpful if you could follow this recommendation in order to make AutoMerge work.  This will save you three days of delay :wink:

Thanks for the recommendation @giordano , I was not aware of this discussion. I will update accordingly Knitro.jl :+1:


Change Logo

I am trying to change logo but getting error
found removed line: - "logoURI": "https://github.com/testsystemca/blcksgn/blob/main/Logo.png",

please link a PR

https://github.com/solana-labs/token-list/pull/11631

https://github.com/solana-labs/token-list/pull/11587/commits/e96c501efe5a6b463648dd05d225855d24e933e7

that's a commit, not a PR

https://github.com/solana-labs/token-list/pull/11652/checks?check_run_id=4670469916

https://github.com/solana-labs/token-list/pull/11652/checks?check_run_id=4670469916

Here is your Pr link
#11652

https://github.com/solana-labs/token-list/pull/11652/checks?check_run_id=4670469916

Here is your Pr link
#11652

Thanks
Any idea  how to fix it?

will be manually merged

https://github.com/solana-labs/token-list/pull/11652/checks?check_run_id=4670469916

Here is your Pr link
#11652

Thanks
Any idea how to fix it?

will be manually merged

will be manually merged

https://github.com/solana-labs/token-list/pull/11652/checks?check_run_id=4670469916

Here is your Pr link
#11652

Thanks
Any idea how to fix it?

will be manually merged

Could you please send me user guide if you have for  that?

merged

merged

Thanks


GTFS feed not working from a variety of Bay area agencies

santa cruz metro transit is not coming up in the trip options.

GTFS Feeds they have are not updated in the long time. I cannot find current GTFS feed for those agencies.
21:50:40.425 WARN (Graph.java:541) Agency Santa Cruz Metro has no calendar dates which are after today; no trips will be plannable on this agency
21:50:40.426 WARN (Graph.java:541) Agency AC Transit has no calendar dates which are after today; no trips will be plannable on this agency
21:50:40.426 WARN (Graph.java:541) Agency SFMTA has no calendar dates which are after today; no trips will be plannable on this agency

seems like GTFS exchange is shutting down.
Transitfeeds and transitland is picking up the slack
http://transitfeeds.com/l/67-california-usa  - This has santa cruz metro, sfmta, actransit
this site looks nice as it breaks it down into static schedule and real time updates and vehicles.
https://transit.land/feed-registry/  has ac transit, sfmta but not santa cruz metro

I'll revise the GTFS sources.
Caltrain has invalid GTFS file, it has subfolder in it. I guess I have to write a script to repackage it into a valid GTFS file.

hmmmmm, that's weird.  I just checked and both feeds are the same txt files.  transitland seems easier, is that what you are going with.
looks like both are inside 2016APR_GTFS folder.
not sure if we need to subscribe to developer changes http://www.caltrain.com/developer/developer-notifications.html

The invalid Santa Cruz feed:
http://transitfeeds.com/p/santa-cruz-metro/343


ghostdriver didn't seem to follow redirect

Try to run a simple test, but ghostdriver didn't seem to follow redirect. According to wireshark (packet capture), ghostdriver sent the HTTP request to http://www.python.org and got 301 redirect, but it didn't try to connect to it.  Any ideas?  Thanks!
from selenium import webdriver
from selenium.webdriver.common.keys import Keys
from selenium.webdriver.common.desired_capabilities import DesiredCapabilities

#driver = webdriver.Firefox()
driver = webdriver.Remote(
   command_executor='http://<IP_ADDRESS>:4444/wd/hub',
   desired_capabilities=DesiredCapabilities.CHROME)
driver.get("http://www.python.org")
assert "Python" in driver.title
elem = driver.find_element_by_name("q")
elem.send_keys("pycon")
elem.send_keys(Keys.RETURN)
assert "No results found." not in driver.page_source
driver.close()

Same problem, using 2.0.0


Prepare the 1.6.3 release.

Work towards #665.

This depends on #680 being merged.

Thanks John! I know we don’t follow SemVer, but we might want to call extra attention to #678 in the release announcement as it’s a significant API change without any prior warning.

Will do.


fix git clone error update auto_size_text

update auto_size_text to https and latest master

This change fixed it for me! 🙌 Would be great if it's merged into the default codebase as well since without this change flutter pub get can't run at all.


Support any URL for RPC

Currently you have to set host and port, which may not work for public RPC endpoints (for example Gastracker). We need to support plain URL
For example https://api.gastracker.io/web3 (also notice that it's under https)

Emerald's RPC right now does not need to talk with an Ethereum RPC endpoint. It's used standalone and doesn't require any network communication. So I think we should be fine here?

Ah, right. I'm just using an old version for development


Fine tuning of two seperate layers in a model

Hello, I read your blog post about fine-tuning. very good explanation. I have an issue with a model, which could be interesting to investigate. Please have a look:
https://github.com/nicolov/segmentation_keras/blob/master/model.py
I want to fine-tune this with a different number of classes (for example 2).
By the way I should modify these two layers and change 21 to 2:
model.add(Convolution2D(21, 1, 1, activation='linear', name='fc-final'))
and
model.add(Convolution2D(21, 1, 1, name='ct_final'))
Also, freeze other layers and make them trainable = False
Now my questions:

I don't know how can I determine how many layers model have to freeze until one of these layers.

I should freeze except than two separate layers. if the correct method of freezing in such situations is to do separately, I should know what is the number of these layers in the model. How can I know that?

Glad to hear that my blog post helps :)
To answer your question, I think it's really more of an art than science to determine the the number of layers to freeze. Since the first few layers encapsulate low level concepts such as lines and curves that can be re-used in other problems, we normally freeze those layers. Freezing layers can also speed up training. So it's a hyper-parameter you have to determine.
To know the number of layers in the model, you can do len(model.layers)

Thank you very much. Do you have any suggestion about fine-tuning models such as this one? two layers makes it tricky for me and I found no example for this in the internet. This could be an interesting topic for the blog either.

The same principle for fine-tuning one layer can be applied to fine-tuning multiple layers. You replace the two layers with your own layers (from 21 classes to 2 classes) and run fit(). I don't see how having 2 layers would affect the fine-tuning procedure.

I should not freeze the weights before these layers as you did? This makes it tricky for me.

You can just freeze the weights for the first few convolutional layers for VGG16, keep other layers trainable


Automatically assign categories to journals

From https://inspirehep.zulipchat.com/#narrow/stream/195298-experts/topic/QIS/near/258521117
Some of the journals we receive exclusively contain papers related to a single or a couple of INSPIRE categories, so we could automatically assign them to all the papers in that journal.
For new records, after journal title normalization, we should add an extra step in the workflow that adds to the record in the workflow inspire_categories.term from the journal record.

TECH NOTES:

If it's a new record on normilize journals get the inspire_categories and save them to extra_data.journal_inspire_categories.
After the branch get the extra_data.journal_inspire_categories and add them to inspire_categories


Adding Security Features (UEFI SecureBoot and BootGuard) in NFD

Discovery of security features (UEFI SecureBoot and BootGuard)
Advertisement of node labels node.alpha.kubernetes-incubator.io/nfd-security-UEFISecureBoot=true and node.alpha.kubernetes-incubator.io/nfd-security-BootGuard=true corresponding to UEFI SecureBoot and BootGuard respectively

@swatisehgal What is the status of this PR? Is it ready for a review?

Any updates on this @swatisehgal?

Yes, its ready for a review. It needs a rebase but I think that can done while I make updates based on reviews? Let me know if you would like me to rebase first.


Right Click Menus: Close on Blur

The context menu renders an invisible mask that covers the entire screen (behind the menu). When a user clicks on this mask, the menu will close.
Implement a way to close the menu without needing to render the mask. This may be done with a blur event handler or the focusin and focusout events.

Note: React 17+ uses focusout rather than blur events for the onBlur event handler, so using onBlur may be exactly what is necessary. We want to only close the menu when focus leaves it or anything inside of it
https://blog.saeloun.com/2021/05/14/react-17-uses-browse-focusin-focusout-for-onfocus-onblur

@csossi this touches the right click menus that are due on Friday. If you could test the following for it (the rest if blocked by a feature flag that doesn't need testing yet) it would be a massive unblocker for our team.
This associated PR #8719 updates the context menus used in the dashboard. Can you verify the following:

Should be able to open the menu by clicking or with the keyboard in the My Stories view
Should be able to navigate the menu with keyboard.
Tabbing away from the menu should close it.
Arrow keys should not close the menu.
When navigating with keyboard you should see the focus border.
All actions in the menu should work as before.

Thank you!

Verified in QA (according to scope specified above)


Update testrunner-toolkit.md

Wrong word.

Thank you for contributing to the sauce-docs project!
A well described pull request helps maintainers quickly review and merge your change
Before submitting your PR, please check our contributing
guidelines, applied for this repository. Avoid large PRs, help reviewers by making them as simple
and short as possible.

Description

Motivation and Context

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Documentation fix (typos, incorrect content, missing content, etc.)

Checklist

[ ] I have read the contributing document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Thank you for catching this, @wesgrant! I am going to close this PR because the build tests are failing due to the missing Algolia search keys, but I have fixed it in another branch and will merge the fix shortly. Thanks again! --Nancy


[Event Request] Codeunit 7324 "Whse.-Activity-Post" – New Event “OnBeforeTransferHeaderModify”

[Event Request] Codeunit 7324 "Whse.-Activity-Post" – New Event “OnBeforeTransferHeaderModify”
Could we please have an new publisher OnBeforeTransferHeaderModify with the following parameters:

WhseActivHeader
TransHeader
ModifyHeader

It would be great if this event existed before the following line in the function "InitSourceDocument":
if ModifyHeader then TransHeader.Modify();

Please always put the full detailed signature of the requested events along with the fragments of code that could define location of the event call unambiguously.
The process of implementation will be faster if you put AL code copied from VSCode instead of screenshots.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Expose the Links header

Make it so the Links header is accessible from JavaScript when accessing a paginated endpoint.
Fixes #380

Thank you 👍

You should probably update the plural.js GET /:resource?_page= test.

Yes, going to push the update in a few minutes.


Current Blinky web page on IoT contains proof reading error

https://ms-iot.github.io/content/en-US/win10/samples/Blinky.htm
Still references 'None' in the text box in item 3 although the picture alongside it is correctly showing the new choices under u1 of Universal (Unencrypted Protocol).

Thanks for the feedback!  We will correct this ASAP.

Pull Request to fix this: https://github.com/ms-iot/content/pull/747

That should do it


Add qt6 support

Description
Add bv_qt6 module.  It only downloads and compiles the modules needed by VisIt (base, tools and svg).
Qt 6 can be requested by '--qt6'.  If vtk is also being built, version 9 will be forced.
Updated the FindVisItQt module to support Qt 6.
Fixed code within VisIt to do a Qt version check for code removed or deprecated in Qt6 but not deprecated in previous versions.
Since newer VS compiler was needed to build Qt on Windows, all TP libraries were rebuilt with MSVC2022, and Windows config-site updated to support.  Only Qt 6 and VTK 9 are available with MSVC2022.
Added Qt 6 support to toss 4 systems on CZ, and updated poodle18 config-site. To use QT 6, set  USE_QT6 CMake variable to true for your build.  This is still considered experimental since it requires VTK 9 and we are still working out some issues with VTK 9 and Mesa.
Type of change

* [ ] Bug fix

[X] New feature~~
* [ ] Documentation update
* [ ] Other

How Has This Been Tested?
Compile cleanly on pascal with 'USE_QT6' unset (meaning qt5 is used), and with it set to true.
Compile cleanly on Windows with both MSVC2017 (qt5) and MSC2022 (qt6).
Checklist:

[X] I have commented my code where applicable.~~
[X] I have updated the release notes.~~
- [ ] I have made corresponding changes to the documentation.
- [ ] I have added debugging support to my changes.
- [ ] I have added tests that prove my fix is effective or that my feature works.
[X] I have confirmed new and existing unit tests pass locally with my changes.~~
- [ ] I have added new baselines for any new tests to the repo.
- [ ] I have NOT made any changes to protocol or public interfaces in an RC branch.

I checked out this branch and I ran build visit with this invocation:
./build_visit --makeflags -j24 --mesagl --mpich --qt6 --vtk --required --optional --no-gdal

I saw this worrying message from the terminal partway through building the TPLs:
Building Qt (~60 minutes)
Unzipping/Untarring qt-everywhere-src-5.14.2.tar.xz . . .
Patching qt . . .
Patching qt 5.14.2 for numeric-limits
Patching qt 5.14.2 for qjp2handler cmath include
Patching qt 5.14.2 for Linux and Mesa-as-GL
Configuring Qt5:
Building Qt5 . . . (~60 minutes)
Installing Qt5 . . .
Done with Qt5
Done building Qt

Shouldn't we not be building qt5? Is it building qt5 because I turned on --optional? If qt6 is turned on should we disable the qt5 option?
Then this happens:
library qwt depends on qt6
library qt6 depends on mesagl glu
mesagl is already installed, skipping
glu is already installed, skipping
Building QT6 base (~10 minutes)
Unzipping/Untarring qtbase-everywhere-src-6.4.2.tar.xz . . .
Configuring Qt6 base: . . .
Building Qt6 base . . .
Qt6 base build failed. Giving up
Unable to build or install QT6. Bailing out.
Error in build process. See build_visit_log for more information. If the error is unclear, please include build_visit_log and contact the VisIt project via https://visit-help.llnl.gov. You may need to compress the build_visit_log using a program like gzip so it will fit within the size limits for attachments.
Log file full path:  /usr/workspace/justin/visit_builds/qt6-develop/build_visit_log

This is what the log says:
CMake Error at CMakeLists.txt:40 (message):
  The path
  "/usr/workspace/justin/visit_builds/qt6-develop/qtbase-everywhere-src-6.4.2-build"
  contains symlinks.  This is not supported.  Possible solutions:

              - map directories using a transparent mechanism such as mount --bind
              - pass the real path of the build directory to CMake, e.g. using             cd $(realpath <path>) before invoking cmake <source_dir>.
Call Stack (most recent call first):
  CMakeLists.txt:47 (qt_internal_check_if_path_has_symlinks)

'/usr/WS2/justin/visit_builds/qt6-develop/third_party/cmake/3.24.3/linux-x86_64_gcc-7.3/bin/cmake' '-DOPENGL_INCLUDE_DIR:PATH=/usr/workspace/justin/visit_builds/qt6-develop/third_party/mesagl/17.3.9/linux-x86_64_gcc-7.3/include' '-DOPENGL_gl_LIBRARY:STRING=/usr/workspace/justin/visit_builds/qt6-develop/third_party/mesagl/17.3.9/linux-x86_64_gcc-7.3/lib/libGL.so' '-DOPENGL_opengl_LIBRARY:STRING=' '-DOPENGL_glu_LIBRARY:STRING=/usr/workspace/justin/visit_builds/qt6-develop/third_party/mesagl/17.3.9/linux-x86_64_gcc-7.3/lib/libGLU.so' '-DOPENGL_GL_PREFERENCE:STRING=LEGACY' '-DBUILD_SHARED_LIBS=ON' '-DCMAKE_INSTALL_PREFIX=/usr/workspace/justin/visit_builds/qt6-develop/third_party/qt/6.4.2/linux-x86_64_gcc-7.3' '-DQT_BUILD_EXAMPLES=FALSE' '-DQT_BUILD_TESTS=FALSE' '-DCMAKE_BUILD_TYPE=Release' '-DINPUT_dbus=no' '-DINPUT_egl=no' '-DINPUT_eglfs=no' '-DINPUT_sql_db2=no' '-DINPUT_sql_ibase=no' '-DINPUT_sql_mysql=no' '-DINPUT_sql_oci=no' '-DINPUT_sql_odbc=no' '-DINPUT_sql_psql=no' '-DINPUT_sql_sqlite=no' '-DINPUT_libjpeg=no' '-DINPUT_libpng=qt' '-DINPUT_system_zlib=no' '-G' 'Unix Makefiles' '/usr/workspace/justin/visit_builds/qt6-develop/qtbase-everywhere-src-6.4.2'
-- Configuring incomplete, errors occurred!
CMake Error at /usr/workspace/justin/visit_builds/qt6-develop/qtbase-everywhere-src-6.4.2/cmake/QtProcessConfigureArgs.cmake:965 (message):
  CMake exited with code 1.

+ set +x
make: *** No targets specified and no makefile found.  Stop.

I guess it is mad about symlinks in the qt build dir? Strange. Did I set something up incorrectly?

@JustinPrivitera
Building qt5 came about from the --required.
I suppose using --qt6 should disable qt 5, good catch. I generally don't use the --required/--optional options, so I missed this scenario completely.
Re: qt6 build failure, what OS are you running on? I haven't seen that problem.
Any pertinent messages from the downloading of QT 6 ?

@biagas
I am running on rztopaz which is still on toss3.
I see this from the download messages:
Checking for qt-5.14.2
Downloading qt-everywhere-src-5.14.2.tar.xz . . .
verifying SHA256 checksum c6fcd53c744df89e7d3223c02838a33309bd1c291fcb6f9341505fe99f7f19fa for qt-everywhere-src-5.14.2.tar.xz . . .
verified
Download succeeded: https://github.com/visit-dav/third-party/releases/download/v3.3.0/qt-everywhere-src-5.14.2.tar.xz
Checking for qwt-6.3.0
Downloading qwt-git-d3706f6e7f0351d278be2d989a4caaf92b399bbd.tar.xz . . .
verifying SHA256 checksum 39839f3aa83f41d09109296d41659e04bb234d9e41ab551af9f4e9b4fceed251 for qwt-git-d3706f6e7f0351d278be2d989a4caaf92b399bbd.tar.xz . . .
shasum -a 256 failed: looking for 39839f3aa83f41d09109296d41659e04bb234d9e41ab551af9f4e9b4fceed251 got
Download attempt failed: https://github.com/visit-dav/third-party/releases/download/v3.3.0/qwt-git-d3706f6e7f0351d278be2d989a4caaf92b399bbd.tar.xz
verifying SHA256 checksum 39839f3aa83f41d09109296d41659e04bb234d9e41ab551af9f4e9b4fceed251 for qwt-git-d3706f6e7f0351d278be2d989a4caaf92b399bbd.tar.xz . . .
verified

Later we have this:
Checking for qt-6.4.2
Downloading qtbase-everywhere-src-6.4.2.tar.xz . . .
verifying SHA256 checksum a88bc6cedbb34878a49a622baa79cace78cfbad4f95fdbd3656ddb21c705525d for qtbase-everywhere-src-6.4.2.tar.xz . . .
shasum: qtbase-everywhere-src-6.4.2.tar.xz:
shasum -a 256 failed: looking for a88bc6cedbb34878a49a622baa79cace78cfbad4f95fdbd3656ddb21c705525d got
Download attempt failed: https://github.com/visit-dav/third-party/releases/download/v3.3.0/qtbase-everywhere-src-6.4.2.tar.xz
verifying SHA256 checksum a88bc6cedbb34878a49a622baa79cace78cfbad4f95fdbd3656ddb21c705525d for qtbase-everywhere-src-6.4.2.tar.xz . . .
verified

@JustinPrivitera thanks for the info, I will investigate.

@JustinPrivitera which compiler were you using? Qt6 requires gcc9/10, so I module loaded gcc/10.2.1 on rztopaz and didn't have any problems building qt6.
I will be pushing some changes soon that fix the problem with qt5 building  when --qt6 specified.

@biagas I was using gcc 7.3. Is there a way to make it quit and complain about the compiler version?

@biagas I was using gcc 7.3. Is there a way to make it quit and complain about the compiler version?

Yes, bv_main.sh has a compiler version check. I can change the checks based on whether or not qt6 is being built, but I'm not sure what to do with different linux flavors (and versions of flavors) having different requirements (eg OpenSuse vs SuseEnterprise vs RH8 vs RH9 vs Ubuntu).
https://doc-snapshots.qt.io/qt6-6.5/supported-platforms.html#desktop-platforms

@biagas wow that looks like a real mess. Maybe the best thing to do is just to document the compiler requirements for the different systems and point users to that page you linked when they run into trouble.
I guess alternatively we could require the highest GCC version required on any system.. so I guess GCC11? Not sure if that is the best solution.

@JustinPrivitera Yes, the QT supported platforms docs create quite a mess if I wanted to try comile-checking for them.  I'm not 100% certain that those levels of compilers are strictly necessary for building Qt from source. Might be the requirement for pre-built versions of Qt, which makes more sense.
I don't think requiring the top g++ version (11) is a good idea, so I am going with the smallest I know will get qt to build, and adding information to the docs about possibly needing newer versions if the qt6 build fails.
FYI, I tried gcc-7.3 on rztopaz again, got a compiler error directly related to c++17 functionality, so 7.3 with qt6 is definitely a no-go.

@JustinPrivitera I modified the minimum g++ version (if building with qt6)  in the compiler checks. This along with @markcmiller86's  fix to the version check in build_visit now has the script failing at the beginning if the compiler version is too old.  I  also added a brief statement regarding compiler versions to the Basic Usage page of the Building Visit docs. Let me know what you think.

@JustinPrivitera I am not sure what is going wrong, there shouldn't be any symlinks in qtbase-everywhere-src-6.4.2-build, as that subdirectory gets filled in by qmake/cmake.  Do you have environment variables related to qt? Or is there a Qt in your path that perhaps is causing problems?
Did you blow away the old qtbase-everywhere-src-6.4.2-build before you tried again?
To just test the building of Qt 6, I would invoke build_visit  with '--third-party --no-visit --cmake --qt6'
I notice you didn't have --cmake,  is the build using a system cmake, then? Do you know which version?

@biagas

Do you have environment variables related to qt? Or is there a Qt in your path that perhaps is causing problems?

No and no.

Did you blow away the old qtbase-everywhere-src-6.4.2-build before you tried again?

Yes; I started with a fresh directory.

I notice you didn't have --cmake, is the build using a system cmake, then? Do you know which version?

cmake must be included under either --required or --optional, since it gets built as a result of my build invocation. It ends up building and using cmake-3.24.3.

okay, what are the symlinks in that -build dir then? Perhaps knowing those would help to isolate the problem.

I am getting that same error each time with the symlinks, but there do not appear to be symlinks in that directory...
bash-4.4$ ls -lR qtbase-everywhere-src-6.4.2-build/
qtbase-everywhere-src-6.4.2-build/:
total 20
-rw------- 1 justin justin 4499 Jun  6 14:38 CMakeCache.txt
drwx--S--- 3 justin justin 4096 Jun  6 14:38 CMakeFiles
-rw------- 1 justin justin  327 Jun  6 14:38 config.opt
-rw------- 1 justin justin  741 Jun  6 14:38 qt.config.out

qtbase-everywhere-src-6.4.2-build/CMakeFiles:
total 8
-rw------- 1 justin justin   85 Jun  6 14:38 cmake.check_cache
drwx--S--- 2 justin justin 4096 Jun  6 14:38 pkgRedirects

qtbase-everywhere-src-6.4.2-build/CMakeFiles/pkgRedirects:
total 0
bash-4.4$ ls -lR qtbase-everywhere-src-6.4.2-build/ | grep ^l
bash-4.4$

Odd.

Well, the only thing I can think of now, is that '/usr/workspace/justin' is a symlink to '/usr/WS2/justin'.
Somehow the build system is using the symlink instead of the resolved path.
Not sure why that doesn't happen to me when I build on rztopaz. Gonna try an experiment.

Well, the only thing I can think of now, is that '/usr/workspace/justin' is a symlink to '/usr/WS2/justin'. Somehow the build system is using the symlink instead of the resolved path. Not sure why that doesn't happen to me when I build on rztopaz. Gonna try an experiment.

@JustinPrivitera  Yes, the problem is /usr/workspace/justin.  I tried this:
cd  /usr/workspace/kbonnell/mybuild
and ran the same build_visit invocation as you, and I get the symlink error,
if I instead
cd  /usr/WS1/kbonnell/mybuild and run build_visit, I don't get the symlink error.
I usually use the full path when I navigate the filesystem and not the symlink, so that's why I didn't run across this before.
I will check bv_qt6 and see if I can find a way to mitigate this problem.

Wow that is sneaky. I don't think I would have thought of that.

@JustinPrivitera   Qt actually gave us a clue:
 pass the real path of the build directory to CMake, e.g. using             cd $(realpath <path>)
I modified bv_qt6.sh to use cd -P ${QT6_BASE_BUILD_DIR} instead of just cd ${QT6_BASE_BUILD_DIR}.
I was concerned that 'realpath' might not be available everywhere.
At any rate, the change resolved the issue for me, can you give it a whirl?

@JustinPrivitera I fixed issues with the host-profile creation: qt5 was added to host profile even though --qt6 was requested, and --qt6 wasn't adding the 'SETUP_APPP_VERSION' call which tells the find module which version of Qt to look for.
Building of VisIt will still fail using the build_visit script, because it will check out a copy of develop, but you need my branch in order to have the QT6 stuff in it.  If you still want to check the build, grab my changes to build_visit, and re-run. Let the build of visit fail, then cd to the visit dir and checkout branch associated with this PR: task/biagas/add_qt6_suppport, Then run build_visit again.

Just fixed that!

Ok I am running now; it seems to be getting farther than before. Made it past configuration.

I don't think we currently have a way to tell build_visit to checkout a particular branch.

What I've been doing is cloning the visit repo, changing to my favorite branch, and then creating build visit from that branch and running. I don't think build visit changes the branch back to develop, so I think in that way I've been able to build my branches with TPLs with little hassle. That's just doing what you suggested but in a different order, right?

What I've been doing is cloning the visit repo, changing to my favorite branch, and then creating build visit from that branch and running. I don't think build visit changes the branch back to develop, so I think in that way I've been able to build my branches with TPLs with little hassle. That's just doing what you suggested but in a different order, right?

Sounds right.  Definitely different than my workflow, but makes more sense, especially if you want build_visit to build visit itself (which I don't often do).

Ok, so it builds successfully, but the main window is broken:

No matter what I do, it stays black.

Yep, that's a VTK-9 with Mesa issue, and Qt 6 requires VTK 9. That issue is still being worked on.

So in that case, if I can build successfully, then everything is probably good?

That issue is still being worked on.

For now, if a user runs build visit with --qt6 will this happen no matter what?

I think things are fine with a good enough system GL, but if you have to use Mesa, then that's when things go awry.
If you use qt5 but ask for --vtk9 and --mesagl you will have the same problem. (So it wasn't introduced by this PR).
Since this is on develop, and the kinks should be worked out before we put it in a release, I'm not sure if any mitigating steps should be taken, like a warning in build_visit if --mesagl and ( --vtk9 or --qt6) are requested. What do you think?

If I run build visit with --osmesa instead it should work properly?

I don't think so.

Sorry. Is there a combination of build arguments that will give me visit with qt6 and vtk9 that will not have the black screen?

Sorry. Is there a combination of build arguments that will give me visit with qt6 and vtk9 that will not have the black screen?

On a system with sufficient graphics drivers and opengl dev (3.2)?, (like my windows machine, or a desktop linux)  just use the same args without --mesagl.

@JustinPrivitera Thank you very much for you diligence in this review.

@biagas glad I could assist :)

When I built without --mesagl I get a visit that looks like this:

Not wrong, just different.

@JustinPrivitera thanks for the screenshot.  Was this a full rebuild of TP libs, or just Qt , or just VisIt?

@biagas started from scratch: full rebuild of TPLs and visit with this build visit invocation:
./build_visit --makeflags -j24 --mpich --qt6 --vtk --required --optional --no-gdal

@biagas  do you think this problem warrants me opening a ticket about it? It seems pretty minor...

@biagas do you think this problem warrants me opening a ticket about it? It seems pretty minor...

It is probably a good idea to keep track of issues like this.   In your notes include the system, your build_visit args, and just flag the fact that it is VTK9 and Qt6 combined. Don't know which library is the source of the problem.


"Fatal: hdf5 files missing" when doing compile on h5utils

(Context: Followed the Build from Source code on the MEEP documentation site.)
Despite downloading HDF5 from the HDF libraries site into /usr/Downloads, compiling, and then doing make && sudo make install , the HDF files I had compiled in the extracted HDF library folder under Downloads did not translate to the compilation under
cd ~/install
cd h5utils
sh autogen.sh CC=mpicc LDFLAGS="${MY_LDFLAGS}" CPPFLAGS="${MY_CPPFLAGS}"
This resulted in fatal error during make.
I tried downloading HDF files through Synaptic but to no avail.

I went to packages.ubuntu.com and did exact search for libhdf5 packages on the search option.
credit to @pandya for the idea under https://askubuntu.com/questions/378558/unable-to-locate-package-while-trying-to-install-packages-with-apt.
After downloading the deb files (used .th Asia mirror) for all the libhdf5 packages under bionic 18.04 (the version of Ubuntu I'm currently using), I went back to the h5utils dir in ~/install and did compile. This time compile worked with no error. Make also resulted in no error and now finally the log code showed that hdf5.h was present, etc.


monitor issues

taskmanager has no prometheus api, so don't add it in prometheus config

/ or /status print a html page, not good for prometheus, maybe openmldb-exporter can handle it, but info is simple.

what means openmldb_connected_seconds_total? https://github.com/4paradigm/openmldb-exporter/blob/53792409f9a53cceaacf0091f658ca124692d356/openmldb_exporter/collector/collectors.py#L80
current time - connect time in show component incorrect: when openmldb-exporter is alive, stop clear start all, it can't reconnect to zk, and fail to execute show components or table status, won't update counters, so the openmldb_connected_seconds_total won't change. only zk restart can be handled, reset(start from small zxid) can't, check if needs improve
why grafana var role doesn't have tm? fix grafana dashboard template

Table Partition Number show deleted tables partitions, we know it's deleted in graph, but the big number make me confused.

TODO all metrics in grafana, and how to read them when do perf profiling

About doc
It's better to give a simple and clear steps for monitor deployment, current doc is hard to read.

Grafana Update

[ ] put metrics of tablets: we can know the insertion performance


Import suggestion, chunking?

Importing 800k information to the engine, memory struggles a lot when you do it in one shot:

But when you do it in chunks like, sending 50k json array in multiple times, it stays low.

It tries to index 50k by 50k when you upload that way.
I think it should be done on the engine side as it handles all the information it needs to index. Thanks for this brilliant engine!

Hey @Duoquote,
You are right. The current indexing system is not that good for big datasets like yours. We are working on a new indexing system combined with a faster, better, and stronger search engine! It is currently able to index more than 115 million little documents without much memory usage.
About the high memory usage that is "garbage collected" on reboot, we are using Rust, and there is no such thing as garbage collection. I would say that the amount of memory that you see is related to the memory mapping that LMDB does under the hood. This is the operating system that decides that there is enough RAM to memory map the whole file and no other program forces any memory reclaiming. Therefore the high memory usage you can see. This is normal and OK.
Thank you for the report!


feat(utils): add merge and isPlainObject

PR Type
enhancement, tests

Description

Introduced merge function in shared.js to merge objects with deep clone capability.
Added isPlainObject function in assertion.js to check if a value is a plain object.
Enhanced test coverage by adding tests for merge and isPlainObject.
Refined existing tests for isEmptyArray, isEmptyObject, ariaAttr, and dataAttr.
Updated documentation to reflect new utility functions.

Changes walkthrough 📝
Relevant filesEnhancement

assertion.jsAdd `isPlainObject` function to assertion utilities

packages/utils/src/assertion.js
Added isPlainObject function to check if a value is a plain object.

+10/-0

shared.jsAdd `merge` function with deep clone capability

packages/utils/src/shared.js
Added merge function to merge objects.  Implemented deep clone functionality for objects.

+37/-0

Tests

assertion.test.jsAdd tests for `isPlainObject` and refine existing tests

packages/utils/src/tests/assertion.test.js
Added tests for isPlainObject.  Removed redundant test cases for isEmptyArray and isEmptyObject.

+50/-15

shared.test.jsAdd tests for `merge` function and refine attribute tests

packages/utils/src/tests/shared.test.js
Added tests for merge function.  Separated tests for ariaAttr and dataAttr.

+126/-10

index.test.jsUpdate export tests to include new utilities

packages/utils/tests/index.test.js

Updated expected exports to include isPlainObject and merge.

+2/-0

Documentation

tonic-ui-945-utils.mdDocument new utility functions in changeset

.changeset/tonic-ui-945-utils.md

Documented the addition of merge and isPlainObject functions.

+5/-0

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

/improve

/review

/describe

/review

/improve

/review

/improve

/ask enhance test coverae for the merge function with arrays


Documentation: Missing information in README.md "Key diversification"

Hi Jon,
the readme.md misses an information in the "Key diversification" section.
byte[] masterKey = new byte[]{ /* ... */ };
KeyInfo keyInfo = new KeyInfo();
...

Please add the line
keyInfo.diversifyKeys = true;
to activate the diversification, thanks.
Greetings
Michael

Sweet - thanks.


🐛 [firestore-bigquery-export] Import existing firestore documents into big query not available in the options

Describe your configuration

Extension name@version: <EMAIL_ADDRESS>
Describe the problem
The option that I expect to be in my configuration is not available and I cannot find it in the config process. Isn't it available anymore?
I have already my Firestore collection full of documents and I want them to be imported. Otherwise, I cannot use the extension.
screenshot.pdf

Hi there, this feature was included in a recent version of the extension and then rolled back, as there were a few breaking changes/issues that needed to be addressed. For now you can use the npx import script to import existing documents.
Closing for now as we are already tracking the re-release of the lifecycle import


Where is the ChexNet here, man?

You have implemented a normal Densenet.

CheXnet is Densenet with 14 outputs at last layer fine tuned on NIH ChestXRay dataset. This implementation follows the same. Please refer :https://arxiv.org/pdf/1711.05225.pdf