ca 1 [t] command: unexpected behavior when Video Split is nonzero

If the Video Split interval is set (non-zero) and you issue (for example) ca 1 20 to FIFO11, the system records 20 seconds worth of video as expected, but then starts recording again and runs (as far as I can tell) forever. Log:
{2019/10/13 18:53:16} Capturing 20 seconds {2019/10/13 18:53:16} Executing macro /var/www/html/macros/start_vid.sh > /var/www/html/macros/start_vid.sh.out 2>&1 "/var/www/html/media/vi_00020_20191013_185316.mp4" & {2019/10/13 18:53:16} Capturing started {2019/10/13 18:53:36} Stopping video from timer {2019/10/13 18:53:37} Capturing stopped {2019/10/13 18:53:37} Add /var/www/html/media/vi_00020_20191013_185316.mp4 to Box Queue at pos 22 {2019/10/13 18:53:37} DEBUG 3 {2019/10/13 18:53:37} Restarting next split of 300 seconds {2019/10/13 18:53:37} Executing macro /var/www/html/macros/start_vid.sh > /var/www/html/macros/start_vid.sh.out 2>&1 "/var/www/html/media/vi_00021_20191013_185337.mp4" & {2019/10/13 18:53:37} Capturing started {2019/10/13 18:53:38} Start boxing /var/www/html/media/vi_00020_20191013_185316.h264 to /var/www/html/media/vi_00020_20191013_185316.mp4 Queue pos 22 {2019/10/13 18:53:45} Finished boxing /var/www/html/media/vi_00020_20191013_185316.mp4 from Box Queue at pos 22 {2019/10/13 18:53:45} Executing macro /var/www/html/macros/end_box.sh > /var/www/html/macros/end_box.sh.out 2>&1 "/var/www/html/media/vi_00020_20191013_185316.mp4" & {2019/10/13 18:53:45} Removed item from Box Queue {2019/10/13 18:58:37} Stopping video from timer {2019/10/13 18:58:37} Capturing stopped {2019/10/13 18:58:37} Add /var/www/html/media/vi_00021_20191013_185337.mp4 to Box Queue at pos 23 {2019/10/13 18:58:37} DEBUG 3 {2019/10/13 18:58:37} Restarting next split of 300 seconds {2019/10/13 18:58:37} Executing macro /var/www/html/macros/start_vid.sh > /var/www/html/macros/start_vid.sh.out 2>&1 "/var/www/html/media/vi_00022_20191013_185837.mp4" & {2019/10/13 18:58:37} Capturing started {2019/10/13 18:58:38} Start boxing /var/www/html/media/vi_00021_20191013_185337.h264 to /var/www/html/media/vi_00021_20191013_185337.mp4 Queue pos 23
etc.
If you zero out the Video Split setting, it behaves as expected.

Yes. The video split was introduced for users who were doing continuous recording and wanted the video set into shorter segments.
Currently if it is set then the fixed time recording set by the optional second second parameter of ca is overridden and continuous recording is performed.
I'll  check into changing this behaviour.

Latest version (6.6.6) now allows time limited capture (ca 1 n) with video split and will terminate after n.

Thanks! Closing.


Implement STT

Plan and Implement STT engine.
Ideas:

Should be able to handle different input and out audio formats reliable by converting automatically as needed. E.g. Discord -> MS or Website -> OpenAI
Low latency
Allow modular design with different API like in TTS and fallback.
Some way to handle interrupting the bot when it is speaking and handling disfluencies when the user is talking. We don't want to interrupt the user? Or do we? Maybe let the bot decide that? Maybe use a mix of canceling the response of the bot entirely and canceling the response and telling the bot where it got cut off, and letting the bot decide when the user is struggling to form a thought and interrupting the user to help him?

Here is what chatgpt says about this:
It sounds like you're considering an important aspect of natural conversation! Here are a few additional thoughts:

Layered Listening: You could create a dual-mode listening system, where the bot continues to listen even when it's speaking. This would allow for graceful interruptions, where the bot could detect user input and either pause or stop speaking in response to the interruption.

Adaptive Interruption Handling: The bot could have contextual awareness of when it’s appropriate to interrupt, based on factors like the user’s speaking speed, emotional tone, or specific phrases (e.g., “Wait,” “Hold on”). Using sentiment analysis or natural language understanding (NLU), it could dynamically decide if it should interrupt or wait.

Cut-off and Recovery: When the bot is interrupted, it could have a system where it remembers the cut-off point and resumes the thought if it's still relevant, or adapt the conversation flow based on the user’s input. You might want to train it to recover gracefully without sounding awkward, e.g., “I was just about to mention…” or “Right, you were saying…”

Disfluency Handling: For user disfluencies (e.g., stuttering, filler words), the bot could detect them and offer suggestions or help complete thoughts, but this might need to be highly personalized. Some users might appreciate this, while others might find it intrusive. A potential solution could be to allow the bot to learn user preferences over time regarding interruptions and disfluency handling.

User-Defined Control: Offering users some control over how the bot interrupts could also help. For example, a setting where the user decides if they want the bot to interrupt when they struggle to form thoughts, or if they'd prefer the bot to always wait.

Timed Pauses: If the user seems to be pausing for too long, the bot could have a “wait threshold” where it steps in with a helpful prompt or suggestion after a few seconds of silence to maintain the flow of conversation.

This blend of adaptive interaction with contextual listening should make for a much smoother conversational experience.


Is needed or not needed

Hi, right at the start it says:
"POA enables scheduling for non-Azure Hosted clusters." and then says: POA is not required for non-Azure hosted clusters.
Should that read "POA is not required for Azure hosted clusters"? Or am I just misunderstanding?

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

ID: a3a16900-c967-ccf9-2d87-1c994013bf0e
Version Independent ID: 474287be-47ca-50b3-1240-8cca745eca9e
Content: Azure Service Fabric patch orchestration application
Content Source: articles/service-fabric/service-fabric-patch-orchestration-application.md
Service: service-fabric
GitHub Login: @khandelwalbrijeshiitr
Microsoft Alias: brkhande

Thanks for the question! We are investigating and will update you shortly.

@TommyQuality the statement looks correct to me.
Azure virtual machine scale set automatic OS image upgrades is the best practice for keeping your operating systems patched in Azure, and the Patch Orchestration Application (POA) is a wrapper around Service Fabrics RepairManager Systems service that enables configuration based OS patch scheduling for non-Azure hosted clusters. POA is not required for non-Azure hosted clusters, but scheduling patch installation by Upgrade Domains, is required to patch Service Fabric clusters hosts without downtime.
So although you can use it for OS Patch Scheduling does not mean you have to.

@TommyQuality I will close for now but if you have additional feedback just let me know and we can always reopen and continue the discussion.


Detect when nodes are banned from certain sites

People have shoved enough stuff through ArchiveBot that we're now running into a problem where certain high-usage fetch nodes are being banned from large website providers.  One of them appears to be banned from all Squarespace sites.
There's a similar problem with nodes that are started on heavily filtered networks.  There's some protection against this with the CheckIP task, but we can't cover all the bases there.  For example, we had a node in Singapore that would have been unable to grab anything that fell under the censorship list of the Singapore Media Development Authority.
It would be nice to identify when it looks like a node cannot complete a job due to these conditions and send out an alert.  With suspend/resume it would then be possible to move that job to a different node.

One way to do this might be to start jobs on multiple pipelines on (e.g.) different ASes, but the current job dequeuing setup doesn't allow that.
Perhaps some sort of prefetch check task might work: if the first few responses are 4xx or network timeouts, the job is failed and put back into the queue.


Use Endb as your primary database.

What is the title of your proposal?
Use Endb as your primary database.
Is it a suggestion for the website or for the bot?

[x] Bot
[ ] Website

Explain your proposal more accurately (It's best to give as much information as possible so that we can implement the proposal better)
There are tons of advantages of using Endb. It has great features such as:

Object-oriented & Persistent Storage
Configurable (you can create multiple SQLite tables, and configure almost everything.)
Performant, it is 100x faster than normal SQLite3 database wrapper.
User and beginner-friendly, with the key-value feature like Enmap, it can be pretty easy to switch and it is a great choice.

Why should we add this feature?
Since Enmap is not receiving any features for quite a while, it uses a pool instead of ORIGINAL better-sqlite3 package, it is so simplified that you can do anything with simplicity.

Currently we are rewriting the bot to use sharding, multiprocess and mongodb as our database. Therefore we have no need of using another SQLite Driver. Also SQLite has a significant problem we came across. It doesn't support multiprocess usage as it's locking the file.

@maltesermailo, can I see the development progress?

The development progress is available at the branch named recode.

@maltesermailo thanks for the information, cheers! 🥂


merge with dbadmin library

Hi, I propose to merge record-macros and dbadmin ( https://github.com/ncannasse/dbadmin  ).
both libraries are tightly coupled and the code style is the same, so the fixes we'll make to record-macros can benefit to dbadmin ( and vice-versa )
Nicolas hasn't made any update since 2015, so I guess it will be easier for us if everything is at the same place, in a single repo.
technically I just copied the files from the original repo and isolated dbadmin in sys.db.admin.* package ( it was in sys.db.* )
-François

Does anybody maintain this repo currently? @jonasmalacofilho ?
If nobody is willing to step forward I can fix CI and merge this PR.

@RealyUniqueName
I am, even if I haven't been terribly active lately.
It's not immediately obvious to me what's what's causing that PHP error.  Do you know?
On the subject of merging dbadmin: I'm not familiar with that codebase and thus not the best person to evaluate this ideas.
But it's a lot of code coming to a repo that already lacks interest/contributors, and the purpose of each projects is different.  Record-macros is a dependency, dbadmin is (AFAIK) an application.  I also see that it depends on hscript, which means that's it's likely not working on Haxe 4 right now.
P.S. I was going to ask Simon, but I just noticed I can enable recurrent rebuilds myself.  This will help me find out about new errors sooner.

CI issues is probably caused by mbstring php extension not enabled. I'll try to fix it.
Regarding this PR I tend to agree with your arguments.

CI has been fixed

Thanks @RealyUniqueName.
Regarding the adoption of dbadmin, I'll try using it for a bit before commenting anything else here.

Hi, thanks for the feedback.
dbadmin has mainly 2 purposes :
1/ act like a super simple phpMyAdmin : view , search , edit, delete records.  It understands some specificities of record-macros like SEnum, SFlags, foreign keys (@:relation)
2/ make db migrations : sync record-macros objects definitions to MySQL : update field types, field names, add/remove indexes ...etc
Part 1 can be easily replaced by external tools like phpMyAdmin, adminer or MySQL Workbench ... I could stop using it and let that part sink in the deep sea of forgotten librairies :-)
But part 2 is super usefull, it generates the required SQL request to update my database shema , each time I modify my models. Like in this screenshot https://imgur.com/a/UsOWR9z
We could imagine a solution where we give up part 1, and we could move the code of part 2 to record-macros. Not the interface, just the feature.  Then the developer is free to create the interface he wants for this : admin panel, CLI command ...
What do you think about this ?

That sounds much better than merging dbadmin.

Ok, I'll work on this and submit a new PR


[FLINK-33210] Introduce lineage graph relevant interfaces

What is the purpose of the change
This PR aims to add lineage graph relevant interfaces such as LineageGraph, LineageVertex, SourceLineageVertex and LineageEdge
Brief change log

Add lineage graph relevant interfaces
Add default lineage graph implementation

Verifying this change
This change added tests and can be verified as follows:

Added test LineageGraphTest

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no) no
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no) yes
The serializers: (yes / no / don't know) no
The runtime per-record code paths (performance sensitive): (yes / no / don't know) no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know) no
The S3 file system connector: (yes / no / don't know) no

Documentation

Does this pull request introduce a new feature? (yes / no) yes
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented) not applicable

@flinkbot run azure

Thanks @JingGe , please help to review this PR again

@HuangZhenQiu @mobuchowski I have add dataset and facet interfaces in lineage vertex, please help to review this PR cc @X-czh

@FangYongs Thanks for the PR. The changes LGTM except for a nit on the UT.


Request to enable tests for windows

This package should enable tests for windows using http://www.appveyor.com/ a sample appveyor.yml is here.
$ cat appveyor.yml
# Operating system (build VM template)
os: Visual Studio 2015

platform: x64

clone_folder: c:\gopath\src\github.com\klauspost\reedsolomon

# environment variables
environment:
  GOPATH: c:\gopath

# scripts that run after cloning repository
install:
  - '"C:\Program Files\Microsoft SDKs\Windows\v7.1\Bin\SetEnv.cmd" /x64'
  - set PATH=%GOPATH%\bin;c:\go\bin;%PATH%
  - rd C:\Go /s /q
  - appveyor DownloadFile https://storage.googleapis.com/golang/go1.6.windows-amd64.zip
  - 7z x go1.6.windows-amd64.zip -oC:\ >nul
  - go version
  - go env
  - cd %GOPATH%\src\github.com\klauspost\reedsolomon

# to run your custom scripts instead of automatic MSBuild
build_script:
  - go test .
  - go test -race .

# to disable automatic tests
test: off

# to disable deployment
deploy: off

I am pretty sure that Appveyor has Go installed already. Maybe if you based it on this, it could be simplified a little:
https://blog.klauspost.com/travisappveyor-ci-script-for-go/

I am pretty sure that Appveyor has Go installed already. Maybe if you based it on this, it could be simplified a little:
https://blog.klauspost.com/travisappveyor-ci-script-for-go/

Ah i see you wrote a nice blog,  the go installation is just to make sure we have latest version - they usually lag behind a bit.  Should i send a PR?

I would prefer the pre-installed,  even if it is a bit old. That way we don't have to do manual updates. Travis should test the different Go versions anyway.


Windows: some Firefox web tests are timing out randomly

https://dev.azure.com/monacotools/Monaco/_build/results?buildId=178791&view=logs&j=b599f925-57da-5b26-3abb-63aa7efd9fc6&t=8cfaeab5-d16b-5ade-99ec-032efde9358d

This seems random, from https://monacotools.visualstudio.com/DefaultCollection/Monaco/_build/results?buildId=179032&view=logs&j=306b2268-2a17-5e97-ab97-f41a26dc5206&t=ad0cf992-b66d-54d3-7b97-28b0a2304eda&l=16977

Not sure where the stacktraces have gone but looking at the full log at least reveal that they fail on FF

I think I figured out when this happens: if timeout is reached of 5000ms!

We can probably show a better message then, I am not sure what the Mocha message is not shown then...

Same, same but different? https://dev.azure.com/monacotools/Monaco/_build/results?buildId=179066&view=logs&j=b599f925-57da-5b26-3abb-63aa7efd9fc6&t=8cfaeab5-d16b-5ade-99ec-032efde9358d

There is now a run with evidence: https://dev.azure.com/monacotools/Monaco/_build/results?buildId=179145&view=logs&j=b599f925-57da-5b26-3abb-63aa7efd9fc6&t=8cfaeab5-d16b-5ade-99ec-032efde9358d&l=17175

 - Files - BrowserTextFileService i/o readStream - large UTF16 BE (reason: Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - Files - BrowserTextFileService i/o readStream - UTF16 LE (no BOM) (reason: Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - Files - BrowserTextFileService i/o readStream - UTF16 BE (no BOM) (reason: Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - Files - BrowserTextFileService i/o readStream - autoguessEncoding (reason: Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - Files - BrowserTextFileService i/o readStream - FILE_IS_BINARY (reason: Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - QuickAccess provider (reason: Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - QuickAccess quick pick access - show() (reason: Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - StorageService (browser) "after each" hook for "Get Data, Integer, Boolean (application)" (reason: Timeout of 20000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - IndexDBStorageDatabase (browser) Basics (reason: Timeout of 20000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - IndexDBStorageDatabase (browser) Clear (reason: Timeout of 20000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - IndexDBStorageDatabase (browser) Inserts and Deletes at the same time (reason: Timeout of 20000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - HistoryService go to last edit location (reason: Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - HistoryService open next/previous recently is reset when other input opens (reason: Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - EditorsObserver observer closes editors when limit reached (across all groups) (reason: Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - EditorsObserver observer closes editors when limit reached (in group) (reason: Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)

Pushed a change to bring playwright up to date with latest 🤞

I think that did not help and the timeouts are really quite random. From https://dev.azure.com/monacotools/Monaco/_build/results?buildId=180211&view=logs&j=b599f925-57da-5b26-3abb-63aa7efd9fc6&t=8cfaeab5-d16b-5ade-99ec-032efde9358d&l=16953:

Pushing a change to increase timeout to see if its helping for Firefox or if its a fundamental issue.

In https://dev.azure.com/monacotools/Monaco/_build/results?buildId=180536&view=logs&j=b599f925-57da-5b26-3abb-63aa7efd9fc6&t=c64bf68a-a40a-513c-efca-4af37a9b3e94&l=17009 there are timeouts again, even with 30s suggesting that something more fundamental seems to be broken running Firefox tests on Windows.
The followings tests are failing:
 - ViewContainerModel add event is triggered only once when view is set visible while it is set active (reason: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - ViewContainerModel add event is not triggered only when view is set hidden while it is set active (reason: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - HistoryService go to last edit location (reason: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - HistoryService open next/previous recently is reset when other input opens (reason: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - EditorService active editor change / visible editor change events (reason: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - EditorService file delete closes editor (reason: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - QuickAccess provider (reason: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - QuickAccess quick pick access - show() (reason: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - Debug - AbstractDebugAdapter event ordering inserts task boundary before response (reason: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - Debug - AbstractDebugAdapter event ordering inserts task boundary after response (reason: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - Debug - AbstractDebugAdapter event ordering does not insert boundaries between events (reason: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - Debug - REPL repl ordering (reason: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - Notebook Find Update find matches basics (reason: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - Notebook Find Update find matches basics 2 (reason: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - Notebook Find Update find matches basics 3 (reason: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - Notebook Find Update find matches, #112748 (reason: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)
 - Notebook Find Reset when match not found, #127198 (reason: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.)

Tried moving the tests to our macOS machine and hit https://github.com/microsoft/playwright/issues/16259. Found https://github.com/microsoft/playwright/issues/16141 which maybe is related.
I think we still have good coverage from the integration tests that still hit Firefox on Windows, but for unit tests, needs investigation why random tests timeout.

No firefox unit tests until Playwright gets more stable, https://github.com/microsoft/playwright/issues/16259#issuecomment-1206158650 sounds promising.


chore(deps): update module github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper to v1.30.1

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.30.0 -> v1.30.1

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.30.1
Compare Source
Bug Fixes

deps: update gomod (#782) (84ee505)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

:tada: This PR is included in version 1.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


How can a CAS client determine a user's authentication status without redirecting the browser to the CAS login page?

This is kind of a high-level, conceptual question, since I couldn't find an answer on the mailing list. Apologies if this is not the right place to ask.
From what I understand, the CAS client (webapp) needs to maintain a session with the user. What happens if the user logs out of CAS in a new browser tab, but remains on the webapp in another tab? Is there any way for the CAS client to simply check on the user's authentication status without redirecting through CAS?
Our scenario is that we don't have "normal" session management (we use encrypted cookies with a limited lifespan) and now have the requirement to find out wether a user is still authenticated with CAS without doing the normal redirect flow.
I was told that it should be possible to use proxy granting tickets, saved in the client's browser cookies, to accomplish that. But I can't really wrap my head around the concept.
From my understanding of CAS, it is simply not possible to find out wether a user is still authenticated without following the normal workflow which includes redirects. Is this correct?

Your understanding is correct, so as long as you are excluding gateway=true (which does involve a redirect). There is no heartbeat mechanism, yet.

This question has been asked several times by the way, and I wonder if we could perhaps provide a protocol extension to support a heartbeat without involving the browser and gateway. Do post on the mailing list and see if there are any takers.


fix(*): add timeout flag to all commands

@maksadbek, please change the PR type to feat since you are altering the expected behavior. This is hardly a bug fix.


Avoid creating empty folders when no new posts exist

This is another low priority issue. When doing a crawl on a blog, if there have been zero posts since the last crawl, and empty folder will be created for the blog, and of course no content will be added to it. Would it be possible to not create this folder, or delete it upon the end of the crawl or closing the program or something?

Well in case you could do this clean yourself, because in my case i want the folder to be there even empty...
Else it could be an option, but at this time i think it's better to focus on functionality and testing before asking new features.

I like the idea, but I'm against this suggestion. I like the folders as a reference to get back all blogs in case the indexing system is reset (i.e. could happen with new versions of the app).


GPU memory usage

The GPU memory usage reported in your paper is about 10G, but the GPU memory usage on my machine is about 18G when I train the model. Is there some different setting in the  repo with your paper?

The validation memory usage is about 7G and the SECOND is not loaded during validation.

Could you try run distributed training using only 1 gpu? The reason might be load the model in a single GPU multiple times.

Make sure you run the code using the script given in README

Make sure you run the code using the script given in README

Thanks for your replay. I have try training with only 1 GPU by the command 'CUDA_VISIBLE_DEVICES='1' ./scripts/dist_train.sh 1 dev configs/stereo/kitti_models/liga.3d-and-bev.yaml', the GPU memory usage is still same. And here is the
log_train.txt
log.

If you train on multiple GPUs, are GPU memory usage roughly the same for every GPU? My model is trained on TiTAN X, and its memory is only 12 GB. Maybe you can print out real GPU memory assumption using pytorch APIs, sometimes pytorch will allocate more GPU than needed.

If you train on multiple GPUs, are GPU memory usage roughly the same for every GPU? My model is trained on TiTAN X, and its memory is only 12 GB. Maybe you can print out real GPU memory assumption using pytorch APIs, sometimes pytorch will allocate more GPU than needed.

Actually the memory allocated is about 10G, but I don't know why the GPU memory usage is about 18G.

If you train on multiple GPUs, are GPU memory usage roughly the same for every GPU? My model is trained on TiTAN X, and its memory is only 12 GB. Maybe you can print out real GPU memory assumption using pytorch APIs, sometimes pytorch will allocate more GPU than needed.

Actually the memory allocated is about 10G, but I don't know why the GPU memory usage is about 18G.

When training on multi-gpus, the gpu memory usage is same for every GPU.

Maybe pytorch will pre-allocate GPU memory for future usage, which will not be freed automatically. Potential solutions include explicitly limiting GPU memory usage or torch.cuda.empty_cache() to free the cache.

empty_cache

Thanks for help. I tried torch.cuda.empty_cache() but not working. I am looking for another solution.

你好，能问一下，gpu oom的问题解决了吗


sourcegen: BleepVersion.scala:8: BleepVersion is already defined as case class BleepVersion [8:1]

So source generation generates code, which the compiler somehow meets twice. Subsequent compiles typically work fine.
I have seen this particular error intermittently in Scala 2.x going back a full decade. I have no idea how to diagnose or solve it, but I have stopped using sbt-buildinfo in at least three different professional projects over the years because of this issue. Most of the time it has been the case that compilation with intellij has triggered it while sbt has been unaffected, and now I see it often through bleep, which goes through BSP.
I'm opening an issue to track it and see if there are solutions to be found

Seems to be fixed by #302


Not time_range filters create empty select query

With the current master, filter boxes create an empty select if no ordering criteria are selected.
Expected results
The distinct values of the configured filter.  If I configure the filter with sorting criteria the right SQL query is generated:
SELECT EventTypeASEventType,         COUNT(*) AS count FROM. WHERE ReferenceHour >= CAST('2020-11-02T00:00:00.000000' AS DATETIME)    AND ReferenceHour CAST('2021-02-02T00:00:00.000000' AS DATETIME)  GROUP BYEventType  LIMIT 50000
Actual results
Error message

400 Syntax error: SELECT list must not be empty at [2:1] (job ID: 3d46b551-77bd-4276-b6ac-5d9e3b45858b) -----Query Job SQL Follows----- | . | . | . | . | . | . | . | . | 1:SELECT 2:FROM ***.*** 3:WHERE ReferenceHour >= CAST('2020-11-02T00:00:00.000000' AS DATETIME) 4: AND ReferenceHour < CAST('2021-02-02T00:00:00.000000' AS DATETIME) 5:LIMIT 50000 | . | . | . | . | . | . | . | . |
This may be triggered by:
Issue 1002 - The database returned an unexpected error.
Please reach out to the Chart Owner for assistance.

Generated query
SELECT FROM . WHERE ReferenceHour >= CAST('2020-11-02T00:00:00.000000' AS DATETIME)   AND ReferenceHour < CAST('2021-02-02T00:00:00.000000' AS DATETIME) LIMIT 50000
Screenshots

How to reproduce the bug

Create a new filter on a not temporal value
Do not add sorting criteria on the filter configuration
Run the query
See the error

Environment

superset version: master
python version: Python 3.7.9
node.js version: N/A
database: BigQuery

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[X] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[X] I have reproduced the issue with at least the latest released version of superset.
[X] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
I am using the master version because I would l was testing #12836

Can you check if this fixes it? https://github.com/apache/superset/pull/12890

Hi @villebro. I can confirm that the #12890 fixes the issue.
Thanks!


File name too long when installing Python 3.8 on MacOS 15.2

Describe the bug
Trying to install Python 3.8.12 on MacOS 15.1:
Looking in links: /var/folders/d6/y0bbdlcx5h5dgqgkr33d1sww0000gn/T/tmptu14_667
Processing /private/var/folders/d6/y0bbdlcx5h5dgqgkr33d1sww0000gn/T/tmptu14_667/setuptools-56.0.0-py3-none-any.whl
ERROR: Could not install packages due to an OSError: [Errno 63] File name too long: '/private/var/folders/d6/y0bbdlcx5h5dgqgkr33d1sww0000gn/T/pip-req-tracker-lq8prlo_/c0091313ab27c45bd27e9c7fc5bf4bf4ae46b5793f3b9ec3a169cce60000000000791602010000003003926f010000005876a80001000000040000000000000000003b5cae20095d380bc600010000003088c900010000004835c2010100000000791602010000008003926f010000003884a300010000007003926f010000000c75a800010000008003926f0100000024f7a6000000000001000000000000004835c2010100000000791602010000000000000000000000507ae029010000009004926f01000000d003926f010000007c91af000100000000000000000000005035c2010100000010c57302010000007041070301000000a10000000000000012c5730201000000c033c2010100000000c57302010000004005926f01000000445faf00010000002004926f01000000640db500010000002004926f01000000700fb500010000000047590201000000b0671a0301000000d0f3b60101000000406d73020100000000000000ffffffff2835c2010100000000000000000000004035c20101000000ffffffff0000000000a2b60101000000000000000000000000c5730201000000507ae02901000000c869a700010000005823c50001000000406d730201000000c004926f01000000640db500010000005035c201010000000079160201000000809c5b000100000007000000000000005823c50001000000406d730201000000f004926f01000000700fb5000100000000003b5cae20095d4030a7010100000000000000000000000100000000000000507ae029010000003088c90001000000d8633e0301000000e8633e030100000002000000000000003088c90001000000f0633e03010000000000000000000000c033c20101000000507ae029010000008005926f0100000078f1a2000100000002000000000000'

Traceback (most recent call last):
  File "/private/var/folders/d6/y0bbdlcx5h5dgqgkr33d1sww0000gn/T/python-build.20241227233620.60583/Python-3.8.12/Lib/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/private/var/folders/d6/y0bbdlcx5h5dgqgkr33d1sww0000gn/T/python-build.20241227233620.60583/Python-3.8.12/Lib/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/private/var/folders/d6/y0bbdlcx5h5dgqgkr33d1sww0000gn/T/python-build.20241227233620.60583/Python-3.8.12/Lib/ensurepip/__main__.py", line 5, in <module>
    sys.exit(ensurepip._main())
  File "/private/var/folders/d6/y0bbdlcx5h5dgqgkr33d1sww0000gn/T/python-build.20241227233620.60583/Python-3.8.12/Lib/ensurepip/__init__.py", line 206, in _main
    return _bootstrap(
  File "/private/var/folders/d6/y0bbdlcx5h5dgqgkr33d1sww0000gn/T/python-build.20241227233620.60583/Python-3.8.12/Lib/ensurepip/__init__.py", line 125, in _bootstrap
    return _run_pip(args + [p[0] for p in _PROJECTS], additional_paths)
  File "/private/var/folders/d6/y0bbdlcx5h5dgqgkr33d1sww0000gn/T/python-build.20241227233620.60583/Python-3.8.12/Lib/ensurepip/__init__.py", line 34, in _run_pip
    return subprocess.run([sys.executable, "-c", code], check=True).returncode
  File "/private/var/folders/d6/y0bbdlcx5h5dgqgkr33d1sww0000gn/T/python-build.20241227233620.60583/Python-3.8.12/Lib/subprocess.py", line 516, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['/private/var/folders/d6/y0bbdlcx5h5dgqgkr33d1sww0000gn/T/python-build.20241227233620.60583/Python-3.8.12/python.exe', '-c', '\nimport runpy\nimport sys\nsys.path = [\'/var/folders/d6/y0bbdlcx5h5dgqgkr33d1sww0000gn/T/tmptu14_667/setuptools-56.0.0-py3-none-any.whl\', \'/var/folders/d6/y0bbdlcx5h5dgqgkr33d1sww0000gn/T/tmptu14_667/pip-21.1.1-py3-none-any.whl\'] + sys.path\nsys.argv[1:] = [\'install\', \'--no-cache-dir\', \'--no-index\', \'--find-links\', \'/var/folders/d6/y0bbdlcx5h5dgqgkr33d1sww0000gn/T/tmptu14_667\', \'--root\', \'/\', \'--upgrade\', \'setuptools\', \'pip\']\nrunpy.run_module("pip", run_name="__main__", alter_sys=True)\n']' returned non-zero exit status 1.
make: *** [Makefile:1198: install] Error 1

On Debian 12, it seems to work:
INFO<EMAIL_ADDRESS>  PYTHONPATH=/home/josh/.local/share/mise/installs/python/3.8.12/lib/python3.8 LD_LIBRARY_PATH=/tmp/python-build.20241227234207.1281177/Python-3.8.12 \

.mise.toml:
[tools]
# Python 3.8.12
# https://www.sublimetext.com/docs/api_environments.html#selecting-the-python-version
python = {version='3.8.12', virtualenv='.venv'}

To Reproduce

MacOS 15.2 ARM64 (m2) (24C101)
mise --version 2024.12.20 macos-arm64 (2024-12-25)

Expected behavior
Python 3.8.12 compiles on MacOS.
mise doctor output
version: 2024.12.20 macos-arm64 (2024-12-25)
activated: yes
shims_on_path: no

build_info:
  Target: aarch64-apple-darwin
  Features: DEFAULT, NATIVE_TLS
  Built: Wed, 25 Dec 2024 20:45:03 +0000
  Rust Version: rustc 1.83.0 (90b35a623 2024-11-26) (Homebrew)
  Profile: release

shell:
  /opt/homebrew/bin/zsh
  zsh <IP_ADDRESS>-dev (arm-apple-darwin24.1.0)

dirs:
  cache: ~/.cache/mise
  config: ~/.config/mise
  data: ~/.local/share/mise
  shims: ~/.local/share/mise/shims
  state: ~/.local/state/mise

config_files:
  ~/.config/mise/config.toml
  ~/dev/sublime_toggle_case_plugin/.mise.toml

ignored_config_files: (none)

backends:
  aqua
  asdf
  cargo
  core
  dotnet
  gem
  go
  npm
  pipx
  spm
  ubi
  vfox

plugins:
  poetry  https://github.com/mise-plugins/mise-poetry.git#daaaab6

toolset:
<EMAIL_ADDRESS>(missing)

path:
  ~/.bin
  ~/.cargo/bin
  /opt/homebrew/opt/coreutils/libexec/gnubin
  /opt/homebrew/opt/ed/libexec/gnubin
  /opt/homebrew/opt/findutils/libexec/gnubin
  /opt/homebrew/opt/gawk/libexec/gnubin
  /opt/homebrew/opt/gnu-indent/libexec/gnubin
  /opt/homebrew/opt/gnu-sed/libexec/gnubin
  /opt/homebrew/opt/gnu-tar/libexec/gnubin
  /opt/homebrew/opt/gnu-which/libexec/gnubin
  /opt/homebrew/opt/grep/libexec/gnubin
  /opt/homebrew/opt/gsed/libexec/gnubin
  /opt/homebrew/opt/make/libexec/gnubin
  /opt/homebrew/opt/sqlite/bin
  ~/.config/jetbrains
  /opt/homebrew/opt/postgresql@16/bin
  /opt/homebrew/bin
  /opt/homebrew/sbin
  ~/.nix-profile/bin
  /nix/var/nix/profiles/default/bin
  /usr/local/bin
  /System/Cryptexes/App/usr/bin
  /usr/bin
  /bin
  /usr/sbin
  /sbin
  /var/run/com.apple.security.cryptexd/codex.system/bootstrap/usr/local/bin
  /var/run/com.apple.security.cryptexd/codex.system/bootstrap/usr/bin
  /var/run/com.apple.security.cryptexd/codex.system/bootstrap/usr/appleinternal/bin
  /Library/Apple/usr/bin

env_vars:
  MISE_SHELL=zsh

settings:
  disable_hints         ["python_multi"]              ~/.config/mise/config.toml
  experimental          true                          ~/.config/mise/config.toml
  jobs                  1                             ~/.config/mise/config.toml
  python.uv_venv_auto   true                          ~/.config/mise/config.toml
  python_compile        true                          ~/.config/mise/config.toml
  status.missing_tools  "if_other_versions_installed" ~/.config/mise/config.toml

No problems found

Additional context
Seems like mise might need to check the max length for paths for tmp installs, I've ran into similar problems with my SSHControlPath on MacOS being too long without much effort.

this doesn't seem to be related to mise


Update floating-action-button-code.txt

Fix prematurely closed tag.

@shaurya947 This pull request now fixes the closings of two tags.  One should have been self-closing, the other should have included the child and was not supposed to be self-closing.

Thanks @ngbrown !


[Low Fantasy Gaming] Adding default values to TF & GS fields?

The attribute/skill rolls on the sheet don't function if the TF & GS fields are left empty, and for the everyday users it's not obvious this is the problem. Forum thread about one user who don't know what's the problem
The Readme nor the free LFG pdf mentions what TF or GS could stand for, so people who haven't seen the Deluxe edition of the rulebook have no idea what's wrong with their sheet. I haven't played the game, but after first looking at the readme, the free rulebook pdf and the sheet code, I didn't manage to understand waht they where for, and only after looking at the git history of the sheet did I find it adressed in the PR by dedurtett.
So I'd suggest to give then default values of 1 & 20 if nothing else, and include some documentation of what the numbers represent. It's a minor thing, but would make using the sheet smoother if you don't have to fill all those fields manually with every new sheet you create. Relating to that, it might be a good idea to give the attributes& attribute max values some default values as well but likely isn't as important as they would need to be filled by the users anyway with whatever stats they've generated for themselves.
I tagged @dedurrett who made the changes, and is likely best suited to address this issue.

Thanks for bringing this to my attention. Default values have been added!

Great!
Will you mention them in the documentation or add comments to the code so people have easier to figure lut what they represent? There will probably be more people who doesn't immidielty realize what the acronyms stand for, at least they weren't common enpugh to find on the pdf or on a web search related to the rpg.

Default values added as well as clarifications for the stats added to Readme


Modified lines doesn't get annotated if you pull while in annotation mode

VS Code Version: 1.41.1
Commit: 26076a4de974ead31f97692a0d32f90d735645c0
Date: 2019-12-18T14:57:51.166Z
Electron: 6.1.5
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Darwin x64 18.7.0
Steps to Reproduce:

Open a file with Annotations turned on (from the top menu button)
Go to a section of the code where you know the code was changed in master
Do a git pull to get latest code from master branch
Notice that the new modified line isn't annotated
After un-annotating and re-annotating it works as expected

Result:

Does this issue occur when all extensions are disabled?: Yes

What are Annotations? Is this an extension? A screen recording would help.


Decouple gnu compiler/flags from cmake infrastructure, update readme,…

… remove unreachable break statements from -Wall, workaround from #atomic capture += 0 (immediate) with some compilers, minor qol adjustments

you broke the build

What issue(s) are you seeing? If there are any CI/CD tools coupled to the repo, I can't seem to locate them/the output.
The only change of note here was to remove the gcc specific flags in favor of having the user supply them for their chosen compiler via -DCT_CFLAGS, -DCT_CXXFLAGS, and -DCT_LINKER_FLAGS.
The motivation for the above was that I needed to test the OpenMP offloading and OpenACC backends for Michael/Pedro, but none of the systems I have access to have offloading integrated into their gnu compilers. Others compilers did support it. Since users may run into similar issues or want to use something other than gcc it seemed like a better option to make things more generalizable.
Having said that, if you have some local script testing the builds, it will likely affect things since -fopenmp/etc. are no longer added by default.

cmake -DENABLE_OMP=ON ../ on Ubuntu 20.04 reveals the following.  The same thing errors out on OSX, Ubuntu 18.04, Ubuntu 16.06, Centos 7/8, etc.
[ 20%] Building CXX object src/CircusTent/Impl/CT_OMP/CMakeFiles/CT_OMP_OBJS.dir/CT_OMP.cpp.o
[ 40%] Building C object src/CircusTent/Impl/CT_OMP/CMakeFiles/CT_OMP_OBJS.dir/CT_OMP_IMPL.c.o
[ 40%] Built target CT_OMP_OBJS
Scanning dependencies of target circustent
[ 60%] Building CXX object src/CircusTent/CMakeFiles/circustent.dir/CT_Main.cpp.o
[ 80%] Building CXX object src/CircusTent/CMakeFiles/circustent.dir/CTOpts.cpp.o
[100%] Linking CXX executable circustent
/usr/bin/ld: Impl/CT_OMP/CMakeFiles/CT_OMP_OBJS.dir/CT_OMP.cpp.o: in function `CT_OMP::AllocateData(unsigned long, unsigned long, unsigned long, unsigned long)':
CT_OMP.cpp:(.text+0x1d1): undefined reference to `omp_set_num_threads'
/usr/bin/ld: CT_OMP.cpp:(.text+0x1ee): undefined reference to `omp_get_num_threads'
/usr/bin/ld: Impl/CT_OMP/CMakeFiles/CT_OMP_OBJS.dir/CT_OMP_IMPL.c.o: in function `RAND_ADD':
CT_OMP_IMPL.c:(.text+0x12): undefined reference to `omp_get_thread_num'
/usr/bin/ld: Impl/CT_OMP/CMakeFiles/CT_OMP_OBJS.dir/CT_OMP_IMPL.c.o: in function `RAND_CAS':
CT_OMP_IMPL.c:(.text+0x62): undefined reference to `omp_get_thread_num'
/usr/bin/ld: Impl/CT_OMP/CMakeFiles/CT_OMP_OBJS.dir/CT_OMP_IMPL.c.o: in function `STRIDE1_ADD':
CT_OMP_IMPL.c:(.text+0xc1): undefined reference to `omp_get_thread_num'
/usr/bin/ld: Impl/CT_OMP/CMakeFiles/CT_OMP_OBJS.dir/CT_OMP_IMPL.c.o: in function `STRIDE1_CAS':
CT_OMP_IMPL.c:(.text+0x111): undefined reference to `omp_get_thread_num'
/usr/bin/ld: Impl/CT_OMP/CMakeFiles/CT_OMP_OBJS.dir/CT_OMP_IMPL.c.o: in function `STRIDEN_ADD':
CT_OMP_IMPL.c:(.text+0x162): undefined reference to `omp_get_thread_num'
/usr/bin/ld: Impl/CT_OMP/CMakeFiles/CT_OMP_OBJS.dir/CT_OMP_IMPL.c.o:CT_OMP_IMPL.c:(.text+0x1b2): more undefined references to `omp_get_thread_num' follow
collect2: error: ld returned 1 exit status
make[2]: *** [src/CircusTent/CMakeFiles/circustent.dir/build.make:103: src/CircusTent/circustent] Error 1
make[1]: *** [CMakeFiles/Makefile2:378: src/CircusTent/CMakeFiles/circustent.dir/all] Error 2
make: *** [Makefile:141: all] Error 2
Build step 'Execute shell' marked build as failure
Sending e-mails to<EMAIL_ADDRESS>[Set GitHub commit status (universal)] PENDING on repos [] (sha:5f4f8b8) with context:CircusTent/label_exp=ubuntu2004
Finished: FAILURE

That makes sense, but, assuming CC=gcc CXX=g++,  does it work with "cmake -DENABLE_OMP=ON -DCT_LINKER_FLAGS="-fopenmp" ../" for example? I updated the readme step 3 to reflect this approach. Admittedly, the added versatility necessitates a bit more in terms of verboseness.
[brodyw@tessa003 build]$ cmake -DENABLE_OMP=ON -DCT_LINKER_FLAGS="-fopenmp" ../
-- The C compiler identification is GNU 8.5.0
-- The CXX compiler identification is GNU 8.5.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- I could not find the latex command.
-- I could not find the pdflatex command.
-- I could not find the htlatex command.
-- I could not find the bibtex command.
-- I could not find the biber command.
-- I could not find the makeindex command.
-- I could not find the dvips command.
-- I could not find the ps2pdf command.
-- I could not find the pdftops command.
-- USING MACHINE-SPECIFIC LINKER_FLAGS: -fopenmp
-- ENABLING OpenMP Implementation
-- DISABLING OpenMP Target Implementation
-- DISABLING OpenSHMEM Implementation
-- DISABLING MPI Implementation
-- DISABLING XBGAS Implementation
-- DISABLING Pthreads Implementation
-- DISABLING OpenACC Implementation
-- DISABLING OpenCL Implementation
-- DISABLING C++ Standard Atomics Implementation
-- CT_INCLUDE_PATH set to /global/home/users/brodyw/circustent/include
-- CT_SRC_PATH set to /global/home/users/brodyw/circustent/src/CircusTent
-- Configuring done
-- Generating done
-- Build files have been written to: /global/home/users/brodyw/circustent/build
[brodyw@tessa003 build]$ make
[ 20%] Building CXX object src/CircusTent/Impl/CT_OMP/CMakeFiles/CT_OMP_OBJS.dir/CT_OMP.cpp.o
[ 40%] Building C object src/CircusTent/Impl/CT_OMP/CMakeFiles/CT_OMP_OBJS.dir/CT_OMP_IMPL.c.o
[ 40%] Built target CT_OMP_OBJS
[ 60%] Building CXX object src/CircusTent/CMakeFiles/circustent.dir/CT_Main.cpp.o
[ 80%] Building CXX object src/CircusTent/CMakeFiles/circustent.dir/CTOpts.cpp.o
[100%] Linking CXX executable circustent
[100%] Built target circustent
[brodyw@tessa003 build]$

why doesn't the cmake link do this automatically?
On Fri, Oct 21, 2022 at 8:51 PM Brody Williams @.***>
wrote:

That makes sense, but, assuming CC=gcc CXX=g++, does it work with "cmake
-DENABLE_OMP=ON -DCT_LINKER_FLAGS="-fopenmp" ../" for example? I updated
the readme step 3 to reflect this approach. Admittedly, the added
versatility necessitates a bit more in terms of verboseness.
@.*** build]$ cmake -DENABLE_OMP=ON
-DCT_LINKER_FLAGS="-fopenmp" ../
-- The C compiler identification is GNU 8.5.0
-- The CXX compiler identification is GNU 8.5.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- I could not find the latex command.
-- I could not find the pdflatex command.
-- I could not find the htlatex command.
-- I could not find the bibtex command.
-- I could not find the biber command.
-- I could not find the makeindex command.
-- I could not find the dvips command.
-- I could not find the ps2pdf command.
-- I could not find the pdftops command.
-- USING MACHINE-SPECIFIC LINKER_FLAGS: -fopenmp
-- ENABLING OpenMP Implementation
-- DISABLING OpenMP Target Implementation
-- DISABLING OpenSHMEM Implementation
-- DISABLING MPI Implementation
-- DISABLING XBGAS Implementation
-- DISABLING Pthreads Implementation
-- DISABLING OpenACC Implementation
-- DISABLING OpenCL Implementation
-- DISABLING C++ Standard Atomics Implementation
-- CT_INCLUDE_PATH set to /global/home/users/brodyw/circustent/include
-- CT_SRC_PATH set to /global/home/users/brodyw/circustent/src/CircusTent
-- Configuring done
-- Generating done
-- Build files have been written to:
/global/home/users/brodyw/circustent/build
@.*** build]$ make
[ 20%] Building CXX object
src/CircusTent/Impl/CT_OMP/CMakeFiles/CT_OMP_OBJS.dir/CT_OMP.cpp.o
[ 40%] Building C object
src/CircusTent/Impl/CT_OMP/CMakeFiles/CT_OMP_OBJS.dir/CT_OMP_IMPL.c.o
[ 40%] Built target CT_OMP_OBJS
[ 60%] Building CXX object
src/CircusTent/CMakeFiles/circustent.dir/CT_Main.cpp.o
[ 80%] Building CXX object
src/CircusTent/CMakeFiles/circustent.dir/CTOpts.cpp.o
[100%] Linking CXX executable circustent
[100%] Built target circustent
@.*** build]$
—
Reply to this email directly, view it on GitHub
https://github.com/tactcomplabs/circustent/pull/24#issuecomment-1287577317,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACOPI2HQDS6FGI76VR6FMODWENCATANCNFSM6AAAAAARLSCQTU
.
You are receiving this because you commented.Message ID:
@.***>

Previously, it would do so because we added “-fopenmp” to the flags by default in the top-level CMakeLists.txt file whenever -DENABLE_OMP was specified. The downside of course was the user would have to edit CMakeLists if they wanted to use something like the intel compiler.
Originally, I also had similar hard-wired flags for the offloading backends which caused issues when I was testing with icx/icpx and nvc/nvc++.
On Oct 21, 2022, at 8:54 PM, jleidel @.***> wrote:

why doesn't the cmake link do this automatically?
On Fri, Oct 21, 2022 at 8:51 PM Brody Williams @.***>
wrote:

That makes sense, but, assuming CC=gcc CXX=g++, does it work with "cmake
-DENABLE_OMP=ON -DCT_LINKER_FLAGS="-fopenmp" ../" for example? I updated
the readme step 3 to reflect this approach. Admittedly, the added
versatility necessitates a bit more in terms of verboseness.
@.*** build]$ cmake -DENABLE_OMP=ON
-DCT_LINKER_FLAGS="-fopenmp" ../
-- The C compiler identification is GNU 8.5.0
-- The CXX compiler identification is GNU 8.5.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- I could not find the latex command.
-- I could not find the pdflatex command.
-- I could not find the htlatex command.
-- I could not find the bibtex command.
-- I could not find the biber command.
-- I could not find the makeindex command.
-- I could not find the dvips command.
-- I could not find the ps2pdf command.
-- I could not find the pdftops command.
-- USING MACHINE-SPECIFIC LINKER_FLAGS: -fopenmp
-- ENABLING OpenMP Implementation
-- DISABLING OpenMP Target Implementation
-- DISABLING OpenSHMEM Implementation
-- DISABLING MPI Implementation
-- DISABLING XBGAS Implementation
-- DISABLING Pthreads Implementation
-- DISABLING OpenACC Implementation
-- DISABLING OpenCL Implementation
-- DISABLING C++ Standard Atomics Implementation
-- CT_INCLUDE_PATH set to /global/home/users/brodyw/circustent/include
-- CT_SRC_PATH set to /global/home/users/brodyw/circustent/src/CircusTent
-- Configuring done
-- Generating done
-- Build files have been written to:
/global/home/users/brodyw/circustent/build
@.*** build]$ make
[ 20%] Building CXX object
src/CircusTent/Impl/CT_OMP/CMakeFiles/CT_OMP_OBJS.dir/CT_OMP.cpp.o
[ 40%] Building C object
src/CircusTent/Impl/CT_OMP/CMakeFiles/CT_OMP_OBJS.dir/CT_OMP_IMPL.c.o
[ 40%] Built target CT_OMP_OBJS
[ 60%] Building CXX object
src/CircusTent/CMakeFiles/circustent.dir/CT_Main.cpp.o
[ 80%] Building CXX object
src/CircusTent/CMakeFiles/circustent.dir/CTOpts.cpp.o
[100%] Linking CXX executable circustent
[100%] Built target circustent
@.*** build]$
—
Reply to this email directly, view it on GitHub
https://github.com/tactcomplabs/circustent/pull/24#issuecomment-1287577317,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACOPI2HQDS6FGI76VR6FMODWENCATANCNFSM6AAAAAARLSCQTU
.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHubhttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftactcomplabs%2Fcircustent%2Fpull%2F24%23issuecomment-1287577867&data=05|01|brodyw%40nvidia.com|4a17c082208347c8eb6e08dab3d05fce|43083d15727340c1b7db39efd9ccc17a|0|0|638020004789477881|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=89z6ySZMG8bn%2FzuSn%2Fc9%2FjeNfxaGzRvt4Cpxjy9XFLs%3D&reserved=0, or unsubscribehttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAIS7YM2Z57YOCLBULV2DMSDWENCNZANCNFSM6AAAAAARLSCQTU&data=05|01|brodyw%40nvidia.com|4a17c082208347c8eb6e08dab3d05fce|43083d15727340c1b7db39efd9ccc17a|0|0|638020004789477881|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=s1UMDTsPGrxfh%2FhuEQkuFgINrYsU3iKoP7siZvAFnQs%3D&reserved=0.
You are receiving this because you were assigned.Message ID: @.***>


Copy nibabel header when creating output prediction

Checklist
GitHub

[x] I've given this PR a concise, self-descriptive, and meaningful title
[x] I've linked relevant issues in the PR body
[x] I've applied the relevant labels to this PR
[ ] I've assigned a reviewer

PR contents

[x] I've consulted ivadomed's internal developer documentation to ensure my contribution is in line with any relevant design decisions
[ ] I've added relevant tests for my contribution
[x] I've updated the relevant documentation for my changes, including argparse descriptions, docstrings, and ReadTheDocs tutorial pages

Description
Before, the output prediction was created by instantiating a new nibabel object, via the copy of the original affine matrix. However, doing so does not ensure the same qform_code and sform_code are preserved. This PR fixes the issue.
More context: https://nipy.org/nibabel/nifti_images.html#default-sform-and-qform-codes
Testing for the proposed change will be implemented in a subsequent PR: https://github.com/ivadomed/ivadomed/issues/713
This PR also includes minor fixes to the unit testing documentation.
Linked issues
Fixes #711
Related to https://github.com/neuropoly/spinalcordtoolbox/issues/3284

Pull Request Test Coverage Report for Build 652238474

2 of 3 (66.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.004%) to 67.081%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

ivadomed/inference.py
2
3
66.67%

Totals

Change from base Build 640885547:
-0.004%

Covered Lines:
5457

Relevant Lines:
8135

💛  - Coveralls

Mmmm thank you for solving this... very specific issue!!
Less critical, but we may also want to apply this change to other instances in ivadomed?

uncertainty
preprocessing
postprocessing
visualize
loader/adaptive
evaluation

Excellent point-- issue opened here https://github.com/ivadomed/ivadomed/issues/715

What's the reasoning for having the tests in a separate pull request?

What's the reasoning for having the tests in a separate pull request?

because i don't have the time to do it 😊   but if pls feel free to chip in1

What's the reasoning for having the tests in a separate pull request?

because i don't have the time to do it 😊 but if pls feel free to chip in1

Haha okay, I'll look into adding some tests then! :)

@jcohenadad the easiest patch I can think of is doing something like this in the segment_volume function (here), instead of changing the code in pred_to_nib where the prediction will never have the same dimensions as the original image.
                pred_list = split_classes(pred_nib)

                def change_header(pred_list, fname_ref):
                    """Change header to keep qform and sform codes
                    Args:
                        pred_list (list): List of predictions (with shape H,W,D)
                        fname_ref (str): Path to the reference image used to extract the header.

                    Return:
                        list: List of prediction with updated header.

                    """
                    nib_ref = nib.load(fname_ref)
                    pred_updated_list = []
                    print("CHANGE HEADER")
                    for pred in pred_list:
                        nib_pred = nib.Nifti1Image(pred.get_fdata(), None, nib_ref.header.copy())
                        pred_updated_list.append(nib_pred)
                    return pred_updated_list

                pred_list = change_header(pred_list, fname_images[0])
                target_list = context['loader_parameters']['target_suffix']

If you agree, with this change I can update the code in the branch. I tested it locally and it fixes the issue of the qform and sform codes.

@jcohenadad the easiest patch I can think of is doing something like this in the segment_volume function (here), instead of changing the code in pred_to_nib where the prediction will never have the same dimensions as the original image.
                pred_list = split_classes(pred_nib)

                def change_header(pred_list, fname_ref):
                    """Change header to keep qform and sform codes
                    Args:
                        pred_list (list): List of predictions (with shape H,W,D)
                        fname_ref (str): Path to the reference image used to extract the header.

                    Return:
                        list: List of prediction with updated header.

                    """
                    nib_ref = nib.load(fname_ref)
                    pred_updated_list = []
                    print("CHANGE HEADER")
                    for pred in pred_list:
                        nib_pred = nib.Nifti1Image(pred.get_fdata(), None, nib_ref.header.copy())
                        pred_updated_list.append(nib_pred)
                    return pred_updated_list

                pred_list = change_header(pred_list, fname_images[0])
                target_list = context['loader_parameters']['target_suffix']

If you agree, with this change I can update the code in the branch. I tested it locally and it fixes the issue of the qform and sform codes.

Ah 3290f85 this does the same as I what I suggested. I think only the code in split_classes should be change. I don't think we need to change the code in pred_to_nib.

I think only the code in split_classes should be changed.

split_classes takes as input nib_prediction:
https://github.com/ivadomed/ivadomed/blob/0a4b7bb3fad8092b31b347332b16f5308be09a54/ivadomed/inference.py#L306
Which means, that if the header of nib_prediction is wrong, then the wrong header will be propagated inside split_classes as well, even if we introduce the change in https://github.com/ivadomed/ivadomed/commit/3290f85b0f417af451fbfd3d8f07152ec3ea44f2.
I think we need to change only one line in pred_to_nib:
If you are suggesting to only change this line:
https://github.com/ivadomed/ivadomed/blob/0a4b7bb3fad8092b31b347332b16f5308be09a54/ivadomed/inference.py#L89
into this:

nib_pred = nib.Nifti1Image( dataobj=arr_pred_ref_space, affine=None, header=nib_ref.header.copy() )

we will end up with the situation i am describing in https://github.com/ivadomed/ivadomed/pull/714/files#r594479677 (hence the hack).
I feel we are going in circles-- maybe a quick chat would be more efficient


Most recent commit with long first line causes repo view to overflow

Description
Hello.
The changes in https://github.com/go-gitea/gitea/pull/21124, specifically here cause the repo file browser to overflow off the page when the most recent commit message has a long first line, which happens a lot when using the Gitea UI to merge two branches with median-length first lines, by default.
By having display: inline, the .commit-list .message-wrapper in the repository file browser doesn't respect the max-width to prevent overflow.
Changing display: inline to display: inline-block fixes the issue on the repository page, but breaks the changes in the above PR, otherwise I would have submitted a PR for it.
I cannot reproduce on the Gitea demo site because it won't allow me to create an account (500 error).
Screenshots
(Sorry for the obfuscation, I couldn't create an account on the demo site, otherwise I would have used that.)

Gitea Version
1.18.0
Can you reproduce the bug on the Gitea demo site?
No
Operating System
Linux
Browser Version
Firefox 108.0, Google Chrome 108.0.5359.124

I was able to fix it on my end without effecting the other page with a header template file containing:
<style>
#repo-files-table .commit-list .message-wrapper {
    display: inline-block;
}
</style>

Fix suggestion in https://github.com/go-gitea/gitea/issues/22413#issuecomment-1381745041.


Flashlight interferes with system toggle

Reported by one user on Android 12, Sony - they didn't say the exact version, but specified that it was happening before they updated to version 5.0.0, so I imagine it's been around for a while and only impacts a subset of devices (maybe just Sony?).
Disabling the toggle with system setting worked for them.
I'm thinking the service is either blocked from running or there's an error causing it to crash - therefore turning off the flashlight on destroy.

May be beneficial to disable that integration with system by default

Possible solution: Keep track of who turned on the flashlight and how it was turned off.
If it was turned on via Trail Sense, then always turn off the flashlight when the service stops.
If it was turned on via the system (i.e. not Trail Sense), then only turn off the flashlight when the user stops it via Trail Sense, otherwise just stop the service when the flashlight turns off.

After some thought, the solution that makes the most sense is to not start the flashlight service when the app is closed. The description has been updated with tasks.


problem with numbers in file names

This bug reproduced if I have images with index in filename more then 10 ((
For example, if icons count = 12 with names from icon1.png to icon12.png
>> Normal sprite has inconsistent size with retina sprite. "icon1" is 81x81 while "icon11@2x" is 112x90.
>> Normal sprite has inconsistent size with retina sprite. "icon12" is 56x45 while "icon1@2x" is 162x162.

The @ signs are messing with how grunt is reading in the order of the files. We have a few options:
Manually specify file order over using globs
We can tell grunt to load files in a given order by not using a glob:
src: ['icon1.png', 'icon2.png', ...]

Move to zero-padded numbers
Our sort is being messed up due to @ being considered lower than 1/2. If we introduce zero padding to our file names then, we will avoid that issue (e.g. 0 will compare to 1 and 1 will compare to 1 for 01 and 11).
mv icon1.png icon01.png
mv icon2.png icon02.png
mv icon3.png icon03.png
mv icon4.png icon04.png
mv icon5.png icon05.png
mv icon6.png icon06.png
mv icon7.png icon07.png
mv icon8.png icon08.png
mv icon9.png icon09.png

Hello! maybe you can include some nat. sort module to fix this bug?
For example: https://www.npmjs.com/package/natural-compare-lite

If we were to move to natural sorting, then sorting would inevitably break for someone else. It's best to keep it consistent/predictable with the rest of JS

Howe can i fix it?
Normal sprite has inconsistent size with retina sprite.
"./img/sprite/menu-greendark-close.png" is 40x38 while
<EMAIL_ADDRESS>is 40x38.

@toptalo The retina image (i.e. @2x.png) is expected to have twice as large dimensions (i.e. 80x76) but they are the same size.

@twolfson, yes, but look to file names it different.
There a two files:
menu-greendark-close.png and menu-greendark-close-small.png
and two retina variants:
<EMAIL_ADDRESS>and<EMAIL_ADDRESS>I think it will be better to serch just %fileneme%@2x file for retina, then just take next in order

Ah, I missed that part. Can you file a separate issue? It's related but not the same as the original one on this thread.

opened Issue #163
Please help =)


Tooltip appears in the wrong position

Issue description

components: tooltip (maybe also others that use popper)
reactstrap version #5.0.0
import method not sure
react version #16.3.0
bootstrap version #4.0.0

What is happening?
The tooltip does not appear in the correct position. Some of the css properties does not appear to be set, namely the transform and top/left properties seem to be missing.

What should be happening?
Tooltip should appear in the correct position

Steps to reproduce issue

Create a new app with create-react-app
add bootstrap and reactstrap
Add a tooltip
Tooltip does not appear in correct position

Code
I could for some reason not get it to show the error in the codepen when I tried earlier. Not sure if it's a version mismatch or something of the sort.
But you can see the error here: https://reactstrap-error-tooltip.firebaseapp.com/
And the git repo is here: https://github.com/Maistho/reactstrap-tooltip-error
I've spent way too long trying to find exactly where the issue is, and I'm not sure if the issue is in reactstrap or in popper or in react-popper. I need some help 😢

fixed upstream. If anyone else runs into this, removing node_modules and doing a fresh install should grab a version of Popper.js (within the server range) which doesn't have this issue

Can someone explain how this issue is resolved? How do i make sure the Popper.js i am using is the correct version? My popover always appear at the top left of the screen. @TheSharpieOne do u mind if u can help me on this?

You can run npm ls popper.js to see what version of popper.js you have. You need at least v1.14.3 (or just not 1.14.1). Since everything is semvar reference, it should get getting the latest. If you have a package-json or yarn.lock file which is pinning the version, you may want to delete them or force them to update.

Thanks for the reply @TheSharpieOne, I ran npm ls popper.js in my project dir, but the result is (empty) am i doing something wrong?

erm, when i checked my package-lock.json, i see it states the latest version of 1.14.3, and i have also deleted node_modules and do a npm install. Man, it kinda confuses me, because seem no one else facing the same problem. I have been bugged by this for months without a real solution T-T


Support ClickHouse deployment with Persistent Volume

This commit supports deploying ClickHouse with Persistent Volume. It provides examples using Local PV, NFS PV or other customized StorageClass.
Signed-off-by: Yanjun Zhou<EMAIL_ADDRESS>

Codecov Report

Merging #3608 (9252fe4) into main (8ef6808) will decrease coverage by 21.45%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #3608       +/-   ##
===========================================
- Coverage   63.12%   41.67%   -21.46%
===========================================
  Files         278      324       +46
  Lines       27829    40590    +12761
===========================================
- Hits        17568    16914      -654
- Misses       8388    22101    +13713
+ Partials     1873     1575      -298

Flag
Coverage Δ

integration-tests
38.72% <ø> (?)

kind-e2e-tests
?

unit-tests
43.44% <ø> (-0.08%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/ovs/openflow/default.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/util/runtime/runtime.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/openflow/groups.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/nodeportlocal/k8s/annotations.go
0.00% <0.00%> (-93.55%)
:arrow_down:

pkg/ovs/openflow/ofctrl_pipeline.go
0.00% <0.00%> (-93.34%)
:arrow_down:

pkg/ovs/openflow/logs.go
9.52% <0.00%> (-90.48%)
:arrow_down:

pkg/apis/controlplane/register.go
0.00% <0.00%> (-90.48%)
:arrow_down:

pkg/agent/controller/networkpolicy/reject.go
0.00% <0.00%> (-85.08%)
:arrow_down:

pkg/flowaggregator/certificate.go
0.00% <0.00%> (-77.48%)
:arrow_down:

pkg/agent/client.go
0.00% <0.00%> (-77.42%)
:arrow_down:

... and 308 more

Hi @antoninbas Could you take another look at this PR?

@yanjunz97 I approved but looks like you have a merge conflict

@yanjunz97 I approved but looks like you have a merge conflict

Thanks! I have rebased and resolved the conflicts.

/test-e2e
/test-conformance
/test-networkpolicy

/test-e2e
/test-conformance
/test-networkpolicy
/test-integration

/test-e2e

Hi @antoninbas This PR can be merged if no other comment.


input Event is missing

bootstrap-select is missing 'input' event which is provided by jquery.
See this fiddle: https://jsfiddle.net/bababalcksheep/0safnvmL/9/
input event is triggered when you change select and it is # NOT  a select-picker
See this fiddle: https://jsfiddle.net/bababalcksheep/0safnvmL/8/
input event is not triggered when you change select and it is a select-picker

also changed.bs.select is triggered twice when a select-picker is change
$('.picker').on('changed.bs.select', function (e) {
 console.log('changed.bs.select deteced');
});

Ok now it triggers once
$('select.picker').on('changed.bs.select', function (e) {
 console.log('changed.bs.select deteced');
});

to fix this, I had to do this but this should be part of this awesome library
$('select.picker').on('changed.bs.select', function (e) {
  $(this).trigger('input');
});

Please 👍 and follow #1305.


Missing CSS

Web console reports these files as missing:
http://pi/_assets/jQuery-File-Upload-9.22.0/css/jquery.fileupload.css
http://pi/_assets/jQuery-File-Upload-9.22.0/css/jquery.fileupload-ui.css
http://pi/_assets/MaterialDesign-Webfont-master/css/materialdesignicons.min.css.map

@ComicSans
thanks! Added it to the develop branch which I am merging to master - after my call for testing in the subfolder thread provoked no response :)

@MiczFlor What subfolder thread? Is there a better place to report error messages like this?


16500 - Fixed Restoration People Section Bug

Issue #: /bcgov/entity#16500
Description of changes:

Conversion to Full Restoration or Limited Restoration Extension page, when we edited the Applicant Person's information, click "Save and Resume", then go back - the "Undo" function doesn't work anymore
Fixed this bug by removing the re-assign officer ID.
Fixed the bug that removed and added person are not updated, previously in ticket
https://app.zenhub.com/workspaces/entities---olga-65af15f59e89f5043c2911f7/issues/gh/bcgov/entity/23277
(verify this ticket)
Fixed the bug when the person is a business/corporation, the file and pay fails (fixed the party schema)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

/gcbrun

Temporary Url for review: https://business-edit-dev--pr-588-3ztuvlhp.web.app

Temporary Url for review: https://business-edit-dev--pr-588-3ztuvlhp.web.app

Try: https://business-edit-dev--pr-588-3ztuvlhp.web.app/BC0883552/limitedRestorationExtension?restoration-id=150942&accountid=3040

/gcbrun

Temporary Url for review: https://business-edit-dev--pr-588-3ztuvlhp.web.app

Try: https://business-edit-dev--pr-588-3ztuvlhp.web.app/BC0883552/limitedRestorationExtension?restoration-id=150995&accountid=3040

/gcbrun

Temporary Url for review: https://business-edit-dev--pr-588-3ztuvlhp.web.app

Test the undo editing person info
https://business-edit-dev--pr-588-3ztuvlhp.web.app/BC0883552/limitedRestorationExtension?restoration-id=150996&accountid=3040

Test the remove and add person
previously in ticket
https://app.zenhub.com/workspaces/entities---olga-65af15f59e89f5043c2911f7/issues/gh/bcgov/entity/23277
https://business-edit-dev--pr-588-3ztuvlhp.web.app/BC0883554/limitedRestorationExtension?restoration-id=151060&accountid=3040

Test when the person is a business/corporation, the file and pay goes through now:
https://business-edit-dev--pr-588-3ztuvlhp.web.app/BC1218838/limitedRestorationExtension?restoration-id=151054&accountid=3040

/gcbrun

Temporary Url for review: https://business-edit-dev--pr-588-3ztuvlhp.web.app


Move invoking contracts with Stellar transactions

Preview is available here:http://soroban-docs-pr488.previews.kube001.services.stellar-ops.com/


Core: Permissions

Store permissions in a user's protected meta data.

Specific permissions could be implemented as meta links.
For added safety, hashes could be stored in private data (of both), and must match for "hashed permissions" to be valid.

Metadata and Links are stored at different permission levels now. Currently, users only have permission to edit specific public metadata. For security reasons this should probably stay the case. Other metadata and links should be managed by other API calls... ?

I need to review this, but this might be done.


Also allow unnecessary_fallible_conversions lint

This fixes the clippy lint firing on macOS on the conversion which needed for portability. For some reason, the logic in https://github.com/rust-lang/rust-clippy/pull/11669 to avoid an overlap is not working.

@arpad-m could you please rebase the branch to trigger a workflow on macOS (rerunning job doesn't pick up new labels)


[Bug]Modifications for MobileNetV3

Hi,
This pr mainly fixes the following bugs:

Move activation function after the SE layer. Please refer to Figure 4 in original paper.
Set parameters for Hsigmoid activation function. The original ones use the default parameters, which are incorrect.

Other modifications mainly include:

Create layers in the manner of MobileNetV2.
Modify unit-test and some assertion in models according to the new layers.
Add some configs for training.

This pr can use the pretrained weight converted from here(https://github.com/kuan-wang/pytorch-mobilenet-v3.git) to do inference on ImageNet and get 67.37% accuracy.
There’re still works to do with training. I could only train the MobileNetV3 model to around 67%.
If there’s any question, please let me know.
Best regards
Yidi

Replaced by #293. Close this PR now.


HTML syntax errors

What happened:

Many external links contain target="_" instead of target="_blank"
Missing rel="noopener noreferrer" attribute in all the external links
Inside html body tag there are currently many random occurrence of head tag and html tag
Missing alt attribute in the image tag
Incorrect closing h4 tag
Incorrect closing a tag

What you expected to happen:
No HTML syntax error
How to reproduce it (as minimally and precisely as possible):
Validate from HTML syntax from https://validator.w3.org/

Hello, Can I take up this issue?  I'm new to the project, so I would be really glad if you could point to me what files to look into to resolve this issue


Can the functions support win_length in stft and melspectrogram?

It is really a great job. However, could it been possible that support "win_length" args?
Here are the references:

https://librosa.org/doc/0.9.1/generated/librosa.feature.melspectrogram.html
https://librosa.org/doc/0.9.1/generated/librosa.stft.html

we may required to pass this variable to stft, and change get window getter

I will add this feature when I have time recently


Carousel accessibility

Proposed changes
Enhanced accessibility performance of Materialize's Carousel component by adding proper ARIA attributes and better support to keyboard navigation.
Following W3C's accessible carousel example, some new accessibility features have been implemented:

Indicators are now focusable by default, since they are no longer simple list items, but buttons instead.
Indicator text label (only detected by assistive technologies): It is possible to change the default label by setting a proper function in "indicatorLabelFunc" option;
"Aria-controls" attribute to indicators/buttons;
Slides are now automatically focused when its corresponding indicator has been selected;
"Hidden" slides are no longer focused and are hidden to assistive technologies (full width carousel only).
Every carousel must have a proper id, so, indicators are able to reference . Therefore a new id will be assigned to the carousel element on initialization if it does not have one yet.

NB: There are other aspects of an "accessible carousel" covered by W3C example which were not implemented here due to lack of escope (e.g.: our current implementation does not provide "autoplay" functionality (Slider should be used, instead)).
Screenshots (if appropriate) or codepen:

Types of changes

[x] 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).

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My commit messages follows the conventional commit format
[x] My change requires a change to the documentation, and updated it accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Since Materialize is skipping to v2.0, this PR makes no sense anymore (I'd also need to convert it to TS, which would require a new PR).

Hey thanks for the PR! You are right, the codebase was refactored in the meantime. It would be cool if you could provide further improvements for accessibility. What are your plans according to creating a new PR?


Performance: Monomorphic AST Nodes

The problem
JavaScript engines commonly use optimizing compilers that rely on objects having stable "shapes".  Meaning access sites for object properties should hopefully only observe objects that all have the same properties, and each of those properties has a consistent internal type, e.g. an integer/string/boolean.

Monomorphic means that only a single shape is observed at the site of a dynamic operation.
Polymorphic means that 2-4 shapes are observed
Megamorphic means that > 4 shapes are observed

Currently accessing common properties of Node such as kind is megamorphic. This means that constructs such as switch (node.kind) and type node type guards (is* functions which test the kind) are much slower than they need to be and this slows down the entire compiler.
The same issue is also present in Type
Proposed solution
Make node monomorphic by having a single object shape with common properties (such as kind, pos, end) and move all other properties in a separate data object in the node.
To preserve compatibility with existing code, we can add accessors for known properties of all nodes:
export class NodeImpl {
    pos: number;
    end: number;
    kind: number;
    id: number;
    data: any;

    …
    get resolvedSymbol() {
    }
    set resolvedSymbol(value: any) {
        this.data.resolvedSymbol = value;
    }

    get modifiers() {
        return this.data?.modifiers;
    }
    set modifiers(value: any) {
        this.data.modifiers = value;
    }

    //// All other properties of all nodes

}

The same approach can also be used for Type.
Results
The proposed solution was implemented in #58928 for both Node,  Type and signature. The performance results from the PR on Node are very promising:

a total time win of 16.0% (ranging 10.5% to 23.7%)
a checker time win of 20.8% (ranging 16.6% to 26.8%)
a memory cost of 2.7% (ranging 1.9% to 4.4%)

This is a memory for performance exchange rate of 5.8 on total time, and 7.6 on checker time.
A breakdown of the contribution of making each type individually monomorphic is also available:
AST Nodes #59190 :
- total time win: 3.3% - 9%
- Check Time win: 3.7% - 11%
- Mem cost: ~0-1%
Types #59191:
- total time win: 4.1% - 20%
- Check Time win: 5.3% - 22.5%
- Mem cost: 0.75%-1.2%
Signatures #59192:
- total time win: 0.45% - 1.72%
- Check Time win: 0.56% - 2.04%
- Mem cost: ~0%
Acknowledgements
This work is the result of a collaboration between me and Ashley Claymore (@acutmore) . Ashley first noticed that all is* guard functions were megamorphic and switching to a switch statement can already improve performance considerably. We both hypothesized that speeding up access to kind in some way would speed up the compiler significantly.
This work builds on the work of Ron Buckton (@rbuckton). Both his work in Node-less megamorphic (without this work the current proposal would not be feasible) and also his creation of Deopt Explorer which was used to drive the experimentation.
Potential issues with the approach
For API clients that dynamically inspect what the properties of a Node are, this will be a breaking change since the properties of a node are no longer directly visible on the Node instance
This new approach allocates more memory, resulting in an increase in memory usage of between 1.9-4.4%. While this cost is not insignificant, the performance wins outweigh the cost.
This new approach is optimized for V8 - other runtimes might see different results both in terms of performance gain and memory usage.
Potential future improvements
Since some property names are common between multiple node kinds, the accessors for them will be megamorphic. While even with this extra cost we still get a significant amount of performance improvement, switching to using data directly might yield even better results.
Alternatively maybe v8 could improve access to base class properties, and this change might no longer be needed in a future version of v8. Ron has raised this issue with v8.

V8 team have responded to Ron's issue.


np.floor_divide does not match numpy for divide by zero with floats

When numpy divides a positive value by zero as floating point, it returns nan.  numba returns inf.  Observe:
Python 3.7.7 (default, Mar 26 2020, 10:32:53)
[Clang 4.0.1 (tags/RELEASE_401/final)] :: Anaconda, Inc. on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import numpy as np
>>> np.__version__
'1.18.1'

>>> import numba
>>> numba.__version__
'0.49.0'

>>> @numba.njit
... def floordiv(x, y):
...     return np.floor_divide(x, y)
...
>>> floordiv(1.0, 0.0)
inf

>>> np.floor_divide(1.0, 0.0)
nan

>>> a = np.ones(2, dtype='float64')
>>> floordiv(a, 0*a)
array([inf, inf])

>>> np.floor_divide(a, 0*a)
array([nan, nan])

I don't know why numpy behaves this way.

why doesn't floor_divide follow divide?!

I'll say that's numpy's problem.
>>> np.floor(np.divide(1., 0.))
inf
>>> np.floor_divide(1., 0.)
nan

Numpy doc say:

y = floor(x1/x2) This is a scalar if both x1 and x2 are scalars.

(https://docs.scipy.org/doc/numpy/reference/generated/numpy.floor_divide.html)

Related Numpy issue: https://github.com/numpy/numpy/issues/7709

Closing this issue as it seems to be resolved. If this is not the case please re-open with a comment about any item that appears to be unresolved. Many thanks.

Closing this issue as it seems to be resolved. If this is not the case please re-open with a comment about any item that appears to be unresolved. Many thanks.


Unable to load symbols. Error initializing the symbol locator. Unknown Error: 0x80131534 on windows 7

Environment data
dotnet --info output:
.NET Core SDK (reflecting any global.json):
Version:   2.2.101
Commit:    236713b0b7
Runtime Environment:
OS Name:     Windows
OS Version:  6.1.7601
OS Platform: Windows
RID:         win7-x64
Base Path:   C:\Program Files\dotnet\sdk\2.2.101\
Host (useful for support):
Version: 2.2.0
Commit:  1249f08fed
.NET Core SDKs installed:
2.2.101 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.2.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.2.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.2.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
C# Extension version: Copied the already installed version in another system to this system as internet was not working.
Name: C#
Id: ms-vscode.csharp
Description: C# for Visual Studio Code (powered by OmniSharp).
Version: 1.17.1
Publisher: Microsoft
VS Marketplace Link: https://marketplace.visualstudio.com/items?itemName=ms-vscode.csharp
Steps to reproduce
Open a c# project and start debugging.
Expected  behavior
Debugging should work.
Actual behavior
Getting error Unable to load symbols. Error initializing the symbol locator. Unknown Error: 0x80131534

Can anyone please help with this issue. Why are we getting this. Any workaround to resolve it.

Unfortunately error code 0x80131534 is not very helpful - that indicates that a type failed to load.
A new version of the C# extension came out today. Are you still seeing this issue with it?
If so, can try enabling logging and see if anything interesting comes out?

Hi Gregg. Thanks for the reply.
Can the issue be because the system is not connected to internet and we
just copied the. Vscode extension folder from internet connected machine to
this machine..
On Wed, Mar 27, 2019, 6:16 AM Gregg Miskelly<EMAIL_ADDRESS>wrote:

Unfortunately error code 0x80131534 is not very helpful - that indicates
that a type failed to load.
A new version of the C# extension came out today. Are you still seeing
this issue with it?
If so, can try enabling logging
https://github.com/OmniSharp/omnisharp-vscode/wiki/Enabling-C%23-debugger-logging
and see if anything interesting comes out?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OmniSharp/omnisharp-vscode/issues/2939#issuecomment-476913990,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aitguow4lFlHPeftWJ6uRuM_Dc9kFexSks5var8CgaJpZM4cG6uW
.

Having no internet connection should not be a problem. My guess is that you didn't quite get something quite right with the copy though. You can use these steps to build a platform-specific version of the extension with all runtime dependencies built in.


ERROR: Certain metadata types, like Audience are not currently supported by the CLI.

ERROR running force:source:retrieve:  We can’t retrieve the specified metadata object: Audience. Certain metadata types, like Audience are not currently supported by the CLI.

This issue has already been reported and discussed here: https://github.com/forcedotcom/cli/issues/470


Restrict scalar indexing to Numbers

This lets us exploit Julia's new indexing utilities.
For the linspace thing to work, it will require a to-be-submitted julia PR, to which I'll link shortly.

https://github.com/JuliaLang/julia/pull/12486

LGTM! Thanks for taking the time! :)
Referencing #48 here too, for completeness.

we might want to add a couple of sanity tests for this, too.

Ah, right; I should have waited to hit merge. I'll file an issue to remind myself.


[dunfell][gatesgarth][hardknott][honister] {galactic} Upgrade to 2021-09-20 release

https://discourse.ros.org/t/new-packages-for-galactic-geochelone-2021-09-20/22374

Merged to other branches as well:
eaff96c50..2b8de21dc  dunfell -> dunfell-next
7af2b6d36..111f698f6  gatesgarth -> gatesgarth
7af2b6d36..111f698f6  gatesgarth -> gatesgarth-next
386a653a0..c0d8db985  hardknott -> hardknott
386a653a0..c0d8db985  hardknott -> hardknott-next
029ba2f34..69131eb01  honister -> honister
029ba2f34..69131eb01  honister -> honister-next
029ba2f34..69131eb01  master -> master
029ba2f34..69131eb01  master -> master-next


Error when calling deliver!

When using the deliver! method (with a bang), The mail gem throws a "You have a nil object when you didn't expect it! You might have expected an instance of Array. The error occurred while evaluating nil.[]"
This is because the delivery_method used by action mailer (This is a AWS::SES::Base instance) tries to do this:
    def deliver!
      response = delivery_method.deliver!(self)
      inform_observers
      delivery_method.settings[:return_response] ? response : self
    end

And because the delivery_method.settings is nil, it returns the error when trying to access the array. Maybe the AWS::SES::Base settings accessor should be initialized with an empty array.

+1 for this new pull request, please! It won't work in Rails 3.1 without it.


Blue dots not changing colour

Sometimes when tests fail the dots stay blue instead of turning red and after that it seems they don't go green even after fixing the tests until I restart vscode and re-run them.

Tks do you have sample that I can clone or a specific scenario to reproduce coz I can't reproduce it  in my tests

I get the same behaviour as @spacem described. Its hard to provide samples as the behaviour is not consistent.

@spacem @ThatBrianDude do any of you have any Mocha hooks like before, beforeEach, afterEach, etc?

@rosesonfire in my test I'm using hooks and never saw this problem do you have any consistent way to reproduce it ?

@maty21 I am not sure if this is the same problem, but sounds quite similar. This is the issue I am talking about: #15. It has the sample code that produces a similar bug in my environment.

@spacem  please try to reproduce it in version 0.12.0
tks a lot

@spacem I'm closing the issue feel free to reopen it again if it not fixed for you

Sorry for the delayed response. I don't have hooks and I don't have a sample I can share right now. Just updated to 0.13.2, restarted vscode, ran a single test and the blue circle didn't turn green - but I can see the mocha output window. I'll try sometime to make a little test repository that demonstrates the issue.
In case it helps here is the ouput shown in the mocha tab:
`Running Mocha with Node.js at C:\Program Files\nodejs\node.EXE
Applying Mocha options:
{
"compilers": {
"ts": "ts-node/register"
}
}
Test file(s):
c:\work\project1\project1\criticality\test\criticality-export-route.spec.ts
Grep pattern:
^criticality export exports with children$
criticality export
√ exports with children
1 passing (11ms)
###end
finish failuare amount:0
debug: found children 2
{
"passed": [
{
"name": "exports with children",
"fullName": "criticality export exports with children",
"suitePath": [
"",
"criticality export"
],
"file": "c:\work\project1\project1\criticality\test\criticality-export-route.spec.ts"
}
],
"failed": []
}`


migrate react 16.4

challenges

Thank you @yoution.

https://github.com/appirio-tech/client-app-layer/pull/4
https://github.com/appirio-tech/webpack-config/pull/39
the two project also need merge


visual studio 2015 update3 crash when edited asp.net core 1.0.0  project project.json

i edited a  project project.json file  cause  circular dependency, there is no any tips but visual studio 2015 update3  directly  go crash  ! ! !

This issue was moved to aspnet/Tooling#818

@anuo I moved this to the Tooling repo so you can track this issue there. Thanks!


Particles renders incorrect on Samsung M31

Description
Particles renders incorrect on:
Samsung M31
Android 10.0
CPU - Samsung Exynos 9611
Should be this:

On Samsung M31:

Project: https://playcanvas.com/editor/project/823129

Does it support WebGL2?

Yes

I've been to the office and unfortunately all the Samsung devices that we have are Snapdragon CPUs and don't showcase the issue :( (S10, S10+ and S7)

I've been to the office and unfortunately all the Samsung devices that we have are Snapdragon CPUs and don't showcase the issue :( (S10, S10+ and S7)

@yaustar thanks for checking at the office. What's your proposed solution for the given problem then? Maybe I can send you a device - its overseas, but possible.

@dexterdeluxe88 Trying to see if our remote device lab can source a device despite being in the US. I'm also going to have a look if I can find an exact model number and source one in the UK

Just to get an idea of urgency, is this a blocking issue?

Testing project: https://playcanv.as/e/p/RWgZ7YlT/ on a range of Exynos devices I've found
Not reproducible on:
SM-A730F - Exynos 7885 (Android 7.1.1)
SM-G935F - Exynos 8890 (Android 7)
SM-J260M - Exynos 7570
SM-A520W - Exynos 7880
GT-I9500 - Exynos 5410

The S10 has a Exynos 9820 and the M31 has a Exynos 9611 so maybe this is super specific to the 9 series CPU?

Not reproducible on:
Galaxy S9 with Exynos 9810

I can reproduce the issue on this project, as well, on Samsung Galaxy S10 with Exynos 9820, running on WebGL2.
Switching to WebGL1 resolves the issue.

Reproduced on Samsung Galaxy A51 SM-A515F. Trying to get hold of one now

Device has been add to device lab: https://deviceremote.sc-corp.net/#!/devices
As Samsung SM-A515F
Use the following guide to remote debug: https://docs.google.com/document/d/1yinoGlv1Qi5IwmsC91RqEl3Jyhc3f5VAoZpLQIv76yk/edit#heading=h.bj1ncrj0ki7r
Or use ngrok to use a locally hosted project. https://ngrok.com/

We have manage to get hold of one and connected to a remote device lab that we can reproduce the issue on

As we think this is a hardware driver issue, we've created a workaround that forces using the CPU on ARM based Samsung devices: https://github.com/playcanvas/engine/pull/4147

Closing this as this is no longer reproducible. See PR https://github.com/playcanvas/engine/pull/5255/


web-ghc fails to compile Haskell programs that use non-ANSI characters

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
web-ghc endpoint returns 500 error when provided with a Haskell program that uses non-ANSI characters
Steps to reproduce the behavior

Open https://playground.plutus.iohkdev.io/
Load Starter example
Modify the first comment to have a non-ANSI character.

-- Thís is a starter contract, based on the Game contract,

Note the accent in Thís
4. Click compile button
Actual Result
FailureResponse (Request {requestPath = (BaseUrl {baseUrlScheme = Http, baseUrlHost = "web-ghc-plutus-apps-prod.plutus.aws.iohkdev.io", baseUrlPort = 80, baseUrlPath = ""},"/runghc"), requestQueryString = fromList [], requestBody = Just ((),application/json;charset=utf-8), requestAccept = fromList [application/json;charset=utf-8,application/json], requestHeaders = fromList []), requestHttpVersion = HTTP/1.1, requestMethod = "POST"} (Response {responseStatusCode = Status {statusCode = 500, statusMessage = "Internal Server Error"}, responseHeaders = fromList [("Content-Type","text/plain; charset=utf-8"),("Date","Thu, 09 Dec 2021 16:11:09 GMT"),("Server","Warp/3.3.17"),("Transfer-Encoding","chunked")], responseHttpVersion = HTTP/1.1, responseBody = "Something went wrong"})

Expected Result
Compilation successful

What have you tried to resolve this issue?
The same exact code without the accent in the i compiles fine
Checked our docs?

[X] Yes, I have looked in the the readme, plutus docs, and technical report for help on this issue.

Checked Stack Exchange?

[X] Yes, I have searched Stack Exchange for this issue and it doesn't exist.
[X] I have considered raising this issue in Stack Exchange.

System info
OS: Arch Linux (5.15.5-arch1-1 - x86_64)
Browser: Chrome Version 95.0.4638.69 (Official Build) (64-bit)
Screenshots and attachments

Hey @palas thanks for the issue, we will investigate this and I will let you know what transpires.

@palas while we continue to investigate the reason the live site isn't handling this case, this seems to not be an issue in the current branch, as an interim fix I suggest running the playground locally using the steps in the README here.
I will then let you know when a fix goes live.


kohya-ss/sd-scripts example didn't seem to work. Updated README and added a Dataset config to streamline running train_network.py

When using the provided examples with sd-scripts, the 'train_data_dir' was always ignored and images weren't found. Added dataset config to rectify the issue and updated the command in the README

Thx for providing the informations!
BTW, will the "resolution" settings in dataset_config affect the ARB settings in training_config?

BTW, can you open a new PR to dev branch?

BTW, can you open a new PR to dev branch?

Sure! I'll do that tomorrow. I'll double check the dataset resolution setting as well.

Changed to dev. Looking into resolution today

I think not matter what "resolution" is, this PR still ok though.
Since if user want to get more detail they should just goto kohya-ss/sd-scripts

Thanks for merging! Was away on holiday!


Added logging for DatasetNotFound error

Motivation and context

This PR adds logs for further invistigation of DatasetNotFound error which frequently appears in case of importing datasets in various formats
How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[ ] I have created a changelog fragment
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (see GitHub docs)
[ ] I have increased versions of npm packages if it is necessary
(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

@coderabbitai help

@coderabbitai review