Stateless parser API

Pull Request Description
Stateless (static) parser interface. Buffer-reuse optimization is now hidden within Parser implementation. Fixes #11121 and prevents similar bugs.
Important Notes

Also simplify EnsoParser API, exposing only a higher-level interface.

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[ ] The documentation has been updated, if necessary.
[ ] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[ ] All code follows the
Scala,
Java,
TypeScript,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
[ ] Unit tests have been written where possible.

I moved the thread-local buffer-recycling logic to the Java bindings, to keep the Rust code threading-agnostic in case we would like to compile it to WASM in the future.

What happens when there is a Rust code working on a buffer and some other thread drops it?

What happens when there is a Rust code working on a buffer and some other thread drops it?

The parsing thread "checks out" the state for the duration of the operation:
https://github.com/enso-org/enso/pull/11147/files#diff-e237fc34931809c65628b8bcde58e32add8959b8e66940c46a5f63f557530302R169-R181
So that particular parsing thread's buffer won't be dropped by the freeAll call, but will eventually dropped if the parser becomes unreferenced.


patchutils: add 0.4.2 variant + remove meta.executable + enable tests + fix inter-dependencies by wrapping

Motivation for this change
Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

the packages which failed to build are pre-existing


Empty resources block on sync

Hi,
I'm struggling with something right now, when I try to sync with the gist resources block is always empty whereas my collection has requests :
{
  "_type": "export",
  "__export_format": 4,
  "__export_date": "2021-11-10T11:48:47.309Z",
  "__export_source": "insomnia.desktop.app:v2021.6.0",
  "resources": []
}

Did I miss something on the configuration ?
Thank you

I have same problem.
{   "_type": "export",   "__export_format": 4,   "__export_date": "2021-11-19T09:08:53.709Z",   "__export_source": "insomnia.desktop.app:v2021.6.0",   "resources": [] }

Hi. I noticed the same behavior. I've investigated about that, it's a bug on Insomnia plugin API, it started in recent versions. So there is nothing to do for now. We need to wait for them. You can downgrade to a old version of Insomnia.


Replace ForgeryPy with fake-factory

These changes were code reviewed over in this pr for flask base: https://github.com/hack4impact/flask-base/pull/20

For now on, feel free to push flask-base changes right to master.
On Tue, Oct 27, 2015 at 1:46 AM Alex Harelick<EMAIL_ADDRESS>wrote:

Merged #5 https://github.com/hack4impact/women-veterans-rock/pull/5.
—
Reply to this email directly or view it on GitHub
https://github.com/hack4impact/women-veterans-rock/pull/5#event-446394559
.


⚠ Bofrost.com has degraded performance

In 09b2436, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 423 ms

Resolved: Bofrost.com performance has improved in 4dd7baa after 6 minutes.


Zotero is not responding

Debug log ID
D
What happened?
After I downloaded the 6.7.154 version and installed it in my zotero and clicked restart now, my zotero just went blank in the main window (that contains the list of all the papers) and I cannot click on anything. After a while, it just said zotero not responding...can anyone help me with it? I have stored so many stuffs in zotero. I am so afraid this is going to ruin my library

Your library is safe. I can look at it in an hour, please restart zotero and after start, choose "send to file.io" from the help menu,and post the ID you get. Then you can disable BBT until I figure out what's wrong.

Thank you! I actually cannot click on anything after i started zotero, it completely freezed. I learned to safe open it (by using Shift tab when open the zotero) so that all addons been blocked. I removed BBT and zotero works now for me. But I still wish to install BBT.
Yesterday i tried the latest version of BBT, it installed fine but cannot be detected by Obsidian zotero integration, that's why I decided to try a previous version of BBT today. And Then it went like this.
Will send the io file now. Hope it can be solved.
获取 Outlook for iOShttps://aka.ms/o0ukef

发件人: Emiliano Heyns @.>
发送时间: Monday, February 5, 2024 1:50:10 PM
收件人: retorquere/zotero-better-bibtex @.>
抄送: zoe667 @.>; Mention @.>
主题: Re: [retorquere/zotero-better-bibtex] Help - Zotero is not responding (Issue #2783)
Your library is safe. I can look at it in an hour, please restart zotero and after start, choose "send to file.io" from the help menu,and post the ID you get. Then you can disable BBT until I figure out what's wrong.
―
Reply to this email directly, view it on GitHubhttps://github.com/retorquere/zotero-better-bibtex/issues/2783#issuecomment-1927801610, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BCQBY6JWHDFJMIGXL67PCJTYSESWFAVCNFSM6AAAAABC2SS336VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRXHAYDCNRRGA.
You are receiving this because you were mentioned.Message ID: @.***>

219153784

Can't read those, and I don't support older versions of BBT. What operating system do you use?

64-bit Windows
获取 Outlook for iOShttps://aka.ms/o0ukef

发件人: Emiliano Heyns @.>
发送时间: Monday, February 5, 2024 2:04:30 PM
收件人: retorquere/zotero-better-bibtex @.>
抄送: zoe667 @.>; Mention @.>
主题: Re: [retorquere/zotero-better-bibtex] Help - Zotero is not responding (Issue #2783)
Can't read those, and I don't support older versions of BBT. What operating system do you use?
―
Reply to this email directly, view it on GitHubhttps://github.com/retorquere/zotero-better-bibtex/issues/2783#issuecomment-1927847122, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BCQBY6MPYSYNCUMOJTGWAVLYSEUL5AVCNFSM6AAAAABC2SS336VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRXHA2DOMJSGI.
You are receiving this because you were mentioned.Message ID: @.***>

Try to capture a log to a text file from the command line using https://www.zotero.org/support/debug_output

Also, this happened again when I tried to install it again.
See below screenshot - i cannot access to debug log:
[cid:b65a063a-fc8c-463e-9c7a-a0f18861fc10]

From: Ziying Song @.>
Sent: Monday, February 5, 2024 2:43 PM
To: retorquere/zotero-better-bibtex @.>
Subject: Re: [retorquere/zotero-better-bibtex] Help - Zotero is not responding (Issue #2783)
Got it, thanks. The screenshot is like below:
[cid:d861d48a-15a2-4986-8f64-f4b1a6e2824d]

From: Emiliano Heyns @.>
Sent: Monday, February 5, 2024 2:17 PM
To: retorquere/zotero-better-bibtex @.>
Cc: zoe667 @.>; Mention @.>
Subject: Re: [retorquere/zotero-better-bibtex] Help - Zotero is not responding (Issue #2783)
Try to capture a log to a text file from the command line using https://www.zotero.org/support/debug_output
—
Reply to this email directly, view it on GitHubhttps://github.com/retorquere/zotero-better-bibtex/issues/2783#issuecomment-1927867886, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BCQBY6IXAXJCX7OPFJ4VA53YSEV6HAVCNFSM6AAAAABC2SS336VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRXHA3DOOBYGY.
You are receiving this because you were mentioned.Message ID: @.***>

I think you are emailing something -- attachments don't show up on github if you reply by email.
But screenshots don't help. Close zotero, open a terminal (cmd) window, type
C:\Program Files (x86)\Zotero\zotero.exe" -ZoteroDebugText > zotero.txt

and attach zotero.txt here after zotero starts and hangs

D905231851
[JavaScript Error: "Upload request 1/EFX9VWMZ failed"]
[JavaScript Error: "You have reached your Zotero File Storage quota. Some files were not uploaded. Other Zotero data will continue to sync to the server.
See your zotero.org account settings for additional storage options.
Pereira et al. - 2017 - Generation expansion planning with high share of r.pdf (519 KB)"]
[JavaScript Error: "Upload request 1/65F774S6 failed"]
[JavaScript Error: "You have reached your Zotero File Storage quota. Some files were not uploaded. Other Zotero data will continue to sync to the server.
See your zotero.org account settings for additional storage options.
Levin et al. - 2023 - Energy storage solutions to decarbonize electricit.pdf (1268 KB)"]
version => 6.0.30, platform => Win32, oscpu => Windows NT 10.0; WOW64, locale => en-US, appName => Zotero, appVersion => 6.0.30, extensions => Zotero PDF Translate (1.0.24, extension), Zotero LibreOffice Integration (6.0.4.SA.6.0.30, extension), Zotero Word for Windows Integration (6.0.3.SA.6.0.30, extension), Notero (0.5.14, extension), ZotFile (5.1.2, extension), Zotero Better Notes (1.0.4, extension), Zotero Reference (0.5.8, extension), ZotCard (2.7.6, extension)
(3)(+0009960): Scheduling next feed update
(4)(+0000000): SELECT ( CASE WHEN lastCheck IS NULL THEN 0 ELSE strftime('%s', lastCheck) + refreshInterval * 60 - strftime('%s', 'now') END ) AS nextCheck FROM feeds WHERE refreshInterval IS NOT NULL ORDER BY nextCheck ASC LIMIT 1
(3)(+0000002): No feeds with auto-update
(4)(+0000002): SELECT libraryID AS id FROM feeds WHERE refreshInterval IS NOT NULL AND ( lastCheck IS NULL OR (julianday(lastCheck, 'utc') + (refreshInterval/1440.0) - julianday('now', 'utc')) <= 0 )
(3)(+0000003): Running update for feeds:
(3)(+0000000): All feed updates done
(3)(+0000001): Scheduling next feed update
(4)(+0000000): SELECT ( CASE WHEN lastCheck IS NULL THEN 0 ELSE strftime('%s', lastCheck) + refreshInterval * 60 - strftime('%s', 'now') END ) AS nextCheck FROM feeds WHERE refreshInterval IS NOT NULL ORDER BY nextCheck ASC LIMIT 1

D905231851

These debug log IDS D<number> are no help -- they go to the Zotero team, I have no access.

partial log

there must be more than this. Try
C:\Program Files (x86)\Zotero\zotero.exe" -ZoteroDebugText > zotero.txt 2>&1

all output should go to a file called zotero.txt. Please attach it here, don't paste its content into the issue.

And no screenshots please. They don't help usually.

after I input the code in terminal, where should I find the zotero.txt? I searched and seemed find nothing...

In the directory where you typed the command

do you mean terminal? Where is the directory for that...I just use "window + x" to call the terminal out and have no idea where its directory is

Type
cd %UserProfile%\Desktop

before you type the zotero command and the file will appear on the desktop

It showed:
cd : Cannot find path 'C:\Users\zyson%UserProfile%\Desktop' because it does not exist.
At line:1 char:1

cd %UserProfile%\Desktop

  + CategoryInfo          : ObjectNotFound: (C:\Users\zyson\%UserProfile%\Desktop:String) [Set-Location], ItemNotFou
 ndException
  + FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.SetLocationCommand

I have redirected my repository to desktop, but after I paste and execute zotero code, there is no new file appear on my desktop...

In what way have you redirected your repository to the desktop? I hope you didn't move your Zotero directory there?

I meant
type
cd %UserProfile%\Desktop

press enter, and then enter the earlier command

Does
cd C:\Users\zyson\Desktop

work?

Yes I used this way below to redirect to desktop
cd C:\Users\zyson\Desktop
获取 Outlook for iOShttps://aka.ms/o0ukef

发件人: Emiliano Heyns @.>
发送时间: Monday, February 5, 2024 3:48:00 PM
收件人: retorquere/zotero-better-bibtex @.>
抄送: zoe667 @.>; Mention @.>
主题: Re: [retorquere/zotero-better-bibtex] Zotero is not responding (Issue #2783)
Does
cd C:\Users\zyson\Desktop
work?
―
Reply to this email directly, view it on GitHubhttps://github.com/retorquere/zotero-better-bibtex/issues/2783#issuecomment-1928067591, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BCQBY6PBOSBCARZU3QBBJ63YSFAQBAVCNFSM6AAAAABC2SS336VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRYGA3DONJZGE.
You are receiving this because you were mentioned.Message ID: @.***>

Ah OK, and no zotero.txt? If you type
echo zotero > zotero.txt

make a zotero.txt file appear?

Let's try something different:
Press the Windows key, type PowerShell into the search bar, and press Enter. This will open a new PowerShell window.
In the PowerShell window, type the following command and press Enter to start output capture:
Start-Transcript -Path "$HOME\Desktop\transcript.txt"

This will create a new text file named transcript.txt on your desktop and start logging all console output to this file.
Now, you can start Zotero by typing the following command and pressing Enter:
& "C:\Program Files (x86)\Zotero\zotero.exe" -ZoteroDebugText

Let zotero start and freeze, then quit Zotero. Logging to the powershell window will stop
Stop the transcription by typing the following command and pressing Enter:
Stop-Transcript

Then attach (not paste) transcript.txt that is on your desktop here

Hi! I'm back - I have tried the above/latest way you shared to acquire the debug file, please see attached (note that I have changed the file path because I put Zotero in D disc). I am not sure if it provided the right information?
transcript.txt

Sorry, I had my powershell commands wrong, please delete transcript.txt from your desktop, and enter
Start-Transcript -Path "$HOME\Desktop\transcript.txt"
Start-Process -FilePath "C:\Program Files (x86)\Zotero\zotero.exe" -ArgumentList "-ZoteroDebugText"

wait for Zotero to start and freeze, then
Stop-Transcript

and attach the file here again.

note that I have changed the file path because I put Zotero in D disc

That is fine

I also encountered the freezing error with Better BibTex 6.7.159
Start-Transcript -Path "$HOME\Desktop\transcript.txt"
Start-Process -FilePath "C:\Program Files (x86)\Zotero\zotero.exe" -ArgumentList "-console -ZoteroDebug"

When I ran it the first time, there is no useful output to the transcript. Whatever debugging information (which I could see) is frozen within the Zotero and cannot be piped back to the PowerShell. When I ran it again, it actually works and there is no error or freezing.

I think I have a way to capture the output. It's convoluted on windows, but there appears to be no way around that. Start cmd, and type
"D:\Program Files (x86)\Zotero\zotero.exe" -console -ZoteroDebugText

a second terminal window will pop up. In the top-left corner, click the icon, and from the menu, choose Edit -> Select All, click the icon again, choose Edit -> Copy, open notepad (or any text editor), paste, and attach that here.

Nah...it showed me below error:
PS C:\Users\zyson> "D:\Programs_D\Zotero\zotero.exe" -console -ZoteroDebugText
At line:1 char:35

"D:\Programs_D\Zotero\zotero.exe" -console -ZoteroDebugText

                              ~~~~~~~~

Unexpected token '-console' in expression or statement.
At line:1 char:44

"D:\Programs_D\Zotero\zotero.exe" -console -ZoteroDebugText

                                       ~~~~~~~~~~~~~~~~

Unexpected token '-ZoteroDebugText' in expression or statement.
+ CategoryInfo          : ParserError: (:) [], ParentContainsErrorRecordException
+ FullyQualifiedErrorId : UnexpectedToken

I also encountered the freezing error with Better BibTex 6.7.159
Start-Transcript -Path "$HOME\Desktop\transcript.txt"
Start-Process -FilePath "C:\Program Files (x86)\Zotero\zotero.exe" -ArgumentList "-console -ZoteroDebug"

When I ran it the first time, there is no useful output to the transcript. Whatever debugging information (which I could see) is frozen within the Zotero and cannot be piped back to the PowerShell. When I ran it again, it actually works and there is no error or freezing.

I also encountered the freezing error with Better BibTex 6.7.159
Start-Transcript -Path "$HOME\Desktop\transcript.txt"
Start-Process -FilePath "C:\Program Files (x86)\Zotero\zotero.exe" -ArgumentList "-console -ZoteroDebug"

When I ran it the first time, there is no useful output to the transcript. Whatever debugging information (which I could see) is frozen within the Zotero and cannot be piped back to the PowerShell. When I ran it again, it actually works and there is no error or freezing.

Magical...I tried your code and it actually started to work! Thanks a ton! I almost give up.

Nah...it showed me below error:
PS C:\Users\zyson> "D:\Programs_D\Zotero\zotero.exe" -console -ZoteroDebugText

This is because you started powershell, not cmd.

Magical...I tried your code and it actually started to work! Thanks a ton! I almost give up.

That bit of powershell code had nothing to do with it, it just starts Zotero with logging on; you can do that from the Help menu inside Zotero. I'd still like to find out what's the cause of this, and a log may tell me.

  [1]

zotero(3)(+0000007): PluginInstaller: fetching addon info

zotero(3)(+0000002): PluginInstaller: addon info fetched

zotero(3)(+0000002): {better-bibtex} +674 Loading Better BibTeX: starting...

better-bibtex orchestrator: [FAE2CTCM.start                                    ] startup    started         at 2024-03-05T01:27:06.086Z running [start]
zotero(3)(+0000002): {better-bibtex} +2 progress: 0 zotero

better-bibtex orchestrator: task startup.start starting
zotero(3)(+0000063): Loaded primaryData in My Library in 77 ms

zotero(3)(+0000001): Loading childCollections for collections in My Library

zotero(4)(+0000001): SELECT C1.collectionID, C2.collectionID AS childCollectionID FROM collections C1 LEFT JOIN collections C2 ON (C1.collectionID=C2.parentCollectionID) WHERE C1.libraryID=? [1]

zotero(3)(+0000004): Loaded childCollections in My Library in 5 ms

zotero(3)(+0000000): Loading childItems for collections in My Library

zotero(4)(+0000001): SELECT collectionID, itemID FROM collections LEFT JOIN collectionItems USING (collectionID) WHERE libraryID=? [1]

zotero(3)(+0000012): Loaded childItems in My Library in 13 ms

zotero(3)(+0000001): Loading relations for collections in My Library

zotero(4)(+0000001): SELECT collectionID, predicate, object FROM collections LEFT JOIN collectionRelations USING (collectionID) LEFT JOIN relationPredicates USING (predicateID) WHERE libraryID=? [1]

zotero(3)(+0000003): Loaded relations in My Library in 4 ms

zotero(3)(+0000001): Loaded collections in My Library in 102 ms

zotero(3)(+0000000): Loading all searches in My Library

zotero(3)(+0000000): Loading primaryData for searches in My Library

zotero(4)(+0000001): SELECT O.savedSearchID, O.savedSearchName AS name, O.libraryID, O.key, O.version, O.synced, DS.savedSearchID IS NOT NULL AS deleted FROM savedSearches O LEFT JOIN deletedSearches DS ON (O.savedSearchID=DS.savedSearchID) WHERE 1 AND O.libraryID=? [1]

zotero(3)(+0000002): Loaded primaryData in My Library in 3 ms

zotero(3)(+0000000): Loading conditions for searches in My Library

zotero(4)(+0000001): SELECT savedSearchID, searchConditionID, condition, operator, value, required FROM savedSearches LEFT JOIN savedSearchConditions USING (savedSearchID) WHERE libraryID=?ORDER BY savedSearchID, searchConditionID [1]

zotero(3)(+0000001): Loaded conditions in My Library in 2 ms

zotero(3)(+0000000): Loaded searches in My Library in 5 ms

zotero(3)(+0000001): Initializing Quick Copy

zotero(3)(+0000001): Running initialization callbacks

zotero(3)(+0000001): Triggering "zotero-loaded" event

zotero(3)(+0000001): Running unlock callbacks

zotero(3)(+0000001): Initializing Zotero pane

zotero(3)(+0000043): Initializing React CollectionTree

zotero(3)(+0000009): Refreshing collections pane

zotero(4)(+0000001): SELECT savedSearchID FROM savedSearches WHERE libraryID=? [1]

zotero(3)(+0000027): Rendering tag selector

zotero(3)(+0000002): Rendering tag list

zotero(3)(+0000005): LibreOfficePlugin: Communication server listening on <IP_ADDRESS>:23116

zotero(3)(+0000001): LibreOfficePlugin: Registering shutdown listener

zotero(3)(+0000000): React CollectionTree initialized

zotero(3)(+0000001): Initializing React ItemTree main

zotero(3)(+0000003): itemTree.render(). Displaying Item Pane Message

zotero(3)(+0000001): React ItemTree main initialized

zotero(3)(+0000001): User interface ready in 3645 ms

zotero(3)(+0000016): Connecting to streaming server at wss://stream.zotero.org/

zotero(3)(+0000002): itemTree.render(). Displaying Item Pane Message

zotero(4)(+0000030): SELECT itemID FROM items JOIN deletedItems USING (itemID) WHERE libraryID=? [1]

zotero(3)(+0000027): CollectionTree.selectWait(): selectEventsSuppressed.  [1]

better-bibtex orchestrator: task startup.start finished
better-bibtex orchestrator: [FAE2CTCM.start                                    ] startup    finished        at 2024-03-05T01:27:06.376Z running []
zotero(3)(+0000014): {better-bibtex startup: [start]} +290 progress: 8.333333333333334 zotero

better-bibtex orchestrator: [FAE2CTCM.TeXstudio                                ] startup    started         at 2024-03-05T01:27:06.377Z running [TeXstudio]
zotero(3)(+0000001): {better-bibtex} +0 progress: 8.333333333333334 TeXstudio support

better-bibtex orchestrator: task startup.TeXstudio starting

It is stuck there for minutes and I closed it.

Dear Retorquere,
I have the same problem on Windows 11 with Better BibTex (6.7.165) not initiating and Zotero (6.0.35) freezing on the main window at the start.
Here is the debug log :
Debug_log_BetterBibtex.txt
Thanks a lot for your help and your brillant work !

Can you try build 5913 above?

Hi Orange Bowl,
I ran into this issue again when my Zotero automatically updated to 6.0.35 version early this week, so I blocked BBT and then it worked fine. This morning, Zotero asked me for an update, which is now 6.0.36, and BBT adds on works - no freezing. Maybe you can try this.
Best

From: orange-bowl @.>
Sent: Wednesday, March 13, 2024 8:26 PM
To: retorquere/zotero-better-bibtex @.>
Cc: zoe667 @.>; State change @.>
Subject: Re: [retorquere/zotero-better-bibtex] Zotero is not responding (Issue #2783)
Dear Retorquere,
I have the same problem on Windows 11 with Better BibTex (6.7.165) not initiating and Zotero (6.0.35) freezing on the main window at the start.
Here is the debug log :
Debug_log_BetterBibtex.txthttps://github.com/retorquere/zotero-better-bibtex/files/14595105/Debug_log_BetterBibtex.txt
Thanks a lot for your help and your brillant work !
—
Reply to this email directly, view it on GitHubhttps://github.com/retorquere/zotero-better-bibtex/issues/2783#issuecomment-1996180493, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BCQBY6JTWC6LSYZ7WLYML7TYYDVEXAVCNFSM6AAAAABC2SS336VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOJWGE4DANBZGM.
You are receiving this because you modified the open/close state.Message ID: @.***>

Thanks a lot !
I uninstalled/reinstalled Zotero following your advice, and everything is now working :)

If noone is going to try build 5913 I'm going to close the issue. Discussion of the matter can then continue in the discussion boards. Issues are for stuff that requires action from me.

Okay！I don't know what is 5913 actually. But thank you so much for your help!
获取 Outlook for iOShttps://aka.ms/o0ukef

发件人: Emiliano Heyns @.>
发送时间: Thursday, March 14, 2024 11:03:08 AM
收件人: retorquere/zotero-better-bibtex @.>
抄送: zoe667 @.>; State change @.>
主题: Re: [retorquere/zotero-better-bibtex] Zotero is not responding (Issue #2783)
If noone is going to try build 5913 I'm going to close the issue. Discussion of the matter can then continue in the discussion boards. Issues are for stuff that requires action from me.
―
Reply to this email directly, view it on GitHubhttps://github.com/retorquere/zotero-better-bibtex/issues/2783#issuecomment-1997666823, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BCQBY6NMUBJUSS4OBX2IMDDYYG32ZAVCNFSM6AAAAABC2SS336VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOJXGY3DMOBSGM.
You are receiving this because you modified the open/close state.Message ID: @.***>

I am very sorry @retorquere, I just encountered the same freezing problem this morning when I restarted my computer and Zotero. I thought it was definitely fixed with Zotero updating, but unfortunately it is not.
As you mentioned, I tried the testbuild 5913 and 6.7.165 BetterBibTex versions with Zotero (6.0.36). Here is the debug log :
Debug_log_BetterBibtex_testbuild.txt
I think this problem is similar with this issue (https://github.com/retorquere/zotero-better-bibtex/issues/2104) and this one on the Zotero forum https://forums.zotero.org/discussion/comment/455093#Comment_455093. I tried to delete the extensions.json file in Zotero Profile Diretory, but it doesn't work.
It seems like BetterBibTex initialization uses a lot of memory :

My only way to use Zotero is in safe mode, or with the BetterBibTex plugin disabled.
Thanks a lot for your help !

This adds more logging, please try to replicate and attache a new log.

Thank you so much for your reactivity !
The new testbuild 5931 didn't fix Zotero starting issue yet, here is the new log :
Debug_log_testbuild_5931.txt
Best

Many thanks @retorquere for the new testbuild !
When Zotero starts, the debug log is endlessly working, here is an extract of the new debug log :
Debug_log_testbuild_5933.txt
The message displayed in the main menu has evolved, and get stuck at this point :

Best

I tried this new testbuild (<IP_ADDRESS>83.5935) and everything works perfectly from now !
Well done @retorquere !! Many thanks for your quality work and availability :)


File loggers should log their verbosity

I look through a lot of MSBuild logs, and when they come from other people I don't always know what verbosity they're logged at--so it's not always clear whether a log message I'm looking for is missing because that build event didn't happen, or because the verbosity isn't where I expect it.
I'd like to see a line at the top of the log that mentions what verbosity is configured.

Sounds great!

It's time to implement this - can I do it?

@CodeTherapist absolutely! Please let us know if you run into any trouble.

@rainersigwald Thank you - very appreciated.
I have implemented this change - but got a small question that I can't guess from your specification:
Do we expect this verbosity info also when the logger option warningsonly or/and errorsonly is set?
The test EmptyErrorLogUsingWarningsErrorsOnly fails as expected - it is expecting an empty log file.

Ah, I didn't think of that! I definitely think we should emit this message as "a normal message", so it shouldn't be emitted in errors-only and warnings-only cases, or if the verbosity is set to minimal. That way it provides information in situations where there was already a lot of output, but doesn't trip people's "detect errors by seeing if the .err file is non-empty" scripting.

@rainersigwald I'm not certain that I implemented it as it should be 😟, because MSBuild is not that simple.

@CodeTherapist That's certainly true! We'll take a look at your PR and let you know what we think.


Fix broken links + Clarify custom Spoofax

Clarify the usage of a custom Spoofax version. It is not clear that this can not be downloaded from the original Spoofax website.
Fix broken links due to "assignments" being renamed to "project".

Thanks :+1:


debian/ubuntu kernel headers: properly use available headers

We have no guarantee on what packages are or will be available on Debian package repositories (or Ubuntu's), instead of trying to solve the package version based on the running kernel this patch uses the meta packages: linux-headers-cloud-amd64 for debian and linux-headers-gcp for ubuntu.
With that the solution becomes more resilient over what is available in the repositories always working with whatever is present.

Thanks for the improvement! :)


dialogue script only working for one week

Describe your bug here.
I put the dialogue script in every song in the data folder, and its only working for one week, every time I restart the game and click a week that weeks dialogue works and the others don't, not even the week 6 dialogue works.
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Mac
Did you edit anything in this build? If so, mention or summarize your changes.
no

even of putting in data put it in scripts floder so u can put it in every song

even of putting in data put it in scripts floder so u can put it in every song
I tried it and that doesn't work either

even of putting in data put it in scripts floder so u can put it in every song
I tried it and that doesn't work either

what version of pysch engine are u using?
is it in source code is it (no source code)
and send that lua


question about train model

hi, after I got the train model, I cannot use this model to detect. I try to convert the model format to .npy, but it didn't work. Do you have any ideas?

Could you provide the error message you got?


Outdated error message in 'addAttributesToSVGElement' plugin

In this error the provided example configuration must be updated for v2.

Fixed in https://github.com/svg/svgo/releases/tag/v2.0.3


kafka-eagle 1.4.6配置ssl后报错

kafka-eagle 1.4.6
kafka version 2.12
配置信息

日志如下

使用java的kafka生产者和消费者都可连接到kafka，用kafka-eagle连接不到ssl

Fixed by v1.4.7, If you want to use it in advance, you can run ./build.sh to compile the master branch.

@zlandyberg
Fixed by v1.4.7, If you want to use it in advance, you can run ./build.sh to compile the master branch.

怎么配置zookeeper的ssl呢 我看到issue#238 是v1.4.0支持zookeeper ssl但没找到如何配置zookeeper的ssl

@zlandyberg
Fixed by v1.4.7, If you want to use it in advance, you can run ./build.sh to compile the master branch.

使用1.4.7后报错如下

Use the latest released v1.4.7, and SSL configure.


(PUP-10844) Add puppet facts show on 6.x

This is mostly a cherry-pick of the following commits
that added puppet facts show on main:

https://github.com/puppetlabs/puppet/commit/88fbc32576bcb036cef451d2db22538f7e3052c4
https://github.com/puppetlabs/puppet/commit/eafff42154eec6929f27823d30fc610fa8481885
https://github.com/puppetlabs/puppet/commit/c9dfabb6a395ca96c216b52d6686678e0b0751ee
https://github.com/puppetlabs/puppet/commit/cac467756b3eb70fe0c2b0a07ed108714b383ff1
https://github.com/puppetlabs/puppet/commit/fdbcce0212bab85c41f11ff8736c520b041b57ca
https://github.com/puppetlabs/puppet/commit/3a8626fdb17db2a27fd0004418c1aa196d222800
https://github.com/puppetlabs/puppet/commit/0de8cfc0e34083e3c3479b0d64099c322044989d

Seems like Ruby 2.3/2.4 to_yaml output differs(wraps the value in '') from Ruby newer Rubies:

❯ ruby  -e "puts RUBY_DESCRIPTION; require 'psych'; puts({ 'macaddress' => '64:52:11:22:03:25' }.to_yaml)"
ruby 2.6.3p62 (2019-04-16 revision 67580) [x86_64-darwin18]
---
macaddress: 64:52:11:22:03:25

❯ ruby  -e "puts RUBY_DESCRIPTION; require 'psych'; puts({ 'macaddress' => '64:52:11:22:03:25' }.to_yaml)"
ruby 2.4.10p364 (2020-03-31 revision 67879) [x86_64-darwin19]
---
macaddress: '64:52:11:22:03:25'


Update demonstration website

As title, it seems that the http://localhost:3000/projects/browser-php-detecting-a-users-browser-from-php/ demonstration site should be updated.
It should have a global website on the Internet to be visible for everyone.
I can support this domain and demonstration website if @cbschuld wants to do this :).

@peter279k - thanks, I updated the location
I am going to updating the site more in the coming days as well
Thanks, Chris


Update league event

The EventMiddleware uses a feature that is in the latest development branch. Waiting for a tagged release. /cc @frankdejonge

@sagikazarmark 2.1.0 has been released!

Awesome, thanks.


Turkish Translation

You can mention me for new strings I can contribute more if I have time.

This does not seem to compile :
values-tr.xml:200:36: Failed to flatten XML for resource 'welcome_message' with error: Invalid unicode escape sequence in string


readerMap when element exists on multiple levels

I am currently working on processing a XML file of a third party and am stuck on handling node with an identical name, but on multiple levels. For example:
<products>
  <product>
    <name>Test</name>
    <deeplinks>
      <product>http://link-to-product</product>
      <documentation>http://link-to-documentation</documentation>
      <warranty>http://link-to-warranty</documentation>
    </deeplinks>
  </product>
</products>

The result I am looking for, looks like this:
$products = [
  [0] => Product object {
      'name' => 'Test',
      'deeplinks' => [
        'product' => 'http://link-to-product'
        ...
      ]
  ...
  }
]

But I cannot figure out how to fix this with sabre-xml. The problem is that the Product-node exists on multiple levels, once for products->product and once for products->product->deeplinks->product.
'{}Product' => function(Reader $reader) { }

works on both different types Product-nodes. Is there a way to distinguish between the two different types of Product-nodes within the sabre-xml library?

Both elements are defined within the same namespace?
If so, I dont think this is valid xml...?

Hi! It definitely is valid XML. Elements' definition can depend on context.
There's a built in feature to deal with this exact thing. I'm on mobile, so I'll be a bit short.
Inside your deseializer function you can call:
$reader->pushContext()

At that point you can override things in
$reader->elementMap

And when you hit the bottom of your reader function call
$reader->popContext

A few reader functions also just support passing the elementMap which becomes the context for the entire subtree.
Let me know if you have more questions. Can go deeper once I'm at a laptop again.

@evert Now I'm suffering from the "deer in the headlight" syndrome. I've been looking into the documentation/code the last hour but I have literally no idea where to start. Are there any examples of usage of this function? I could not find any. I would be happy with bringing pie to Fruux HQ in exchange for help though.

Hi @niektenhoopen ,
Here's an example. I don't know for certain if this makes the most sense for you. My assumption is that any element in deepLinks is just a single key->value element with urls and things don't appear more than once.
This is how I would define the deserializer for deeplinks and I'm assuming a http://example.org/ namespace.
$server->elementMap['{http://example.org/}deeplinks'] = function(Sabre\Xml\Reader $reader) {
   // Create a new parsing context.
   $reader->pushContext();

   // Override the elementMap for this sub-tree. In this case we don't want _any_ custom parsers.
   $reader->elementMap = [];

   // Parse sub-elements
   $result = \Sabre\Xml\Deserializer\keyValue($reader, 'http://example.org');

   // Restore original parsing context.
   $reader->popContext();

   return $result;
};

Hope this makes sense!

Yes, I should be able to make that work! Thank you, @evert!


Feat: relay-101

added relay endpoint to sfu.proto
ran ./build_protos.sh
added server implementation to relay
added examples for relay in examples
Description
Reference issue
https://github.com/pion/ion-sfu/issues/101
Fixes #...

shoud restart after simulcast stable?


[FEAT] Reorganize the demo section in the HTML home page

Is your feature request related to a problem? Please describe.
In the HTML home page, the demos are not stored in a specific order. They are added as they are added, following the existing list.
Demos provided by this repository are mixed with demos provided by external repositories.
Some demos are dedicated to a use case, others have been done for events and presentations and cover several topics.
https://cdn.statically.io/gh/process-analytics/bpmn-visualization-examples/v0.39.0/examples/index.html#demos

Describe the solution you'd like
I suggest we better sort the demos to make this clearer for users.
We may introduce a dedicated sections for the demo done for an event.

### Tasks
- [ ] Decide of a new layout (this proposition can be totally rejected and the issue closed) with @csouchet
- [ ] Update the page and add a new section if required
- [ ] Update the README to reflect the new organization

Some examples of propositions

I realized that the "external" bubble in my screenshot was confusing.
I added it not as an example of something that I had in mind but only to show where the external demos are currently placed 😢
My proposal was to move them in a dedicated section or move them all to the end of the list.
That's being said, your proposals @csouchet are also great. I prefer the 2nd one as it is more discrete. I think we don't need to highlight a lot that the demo and the code are hosted externally, just pass the information.
We could hide an info icon on the right of the text to explain in a tooltip what it means.

I just made some tests.
Your proposition is ok for me too (even if I didn't understand it at the first time :slightly_smiling_face:).
We should discuss in person the next week ^^


How to remember authorization decisions?

Got this setup and everything is working great except for one "snag".  Everytime a user goes through a new authorization request they have to re "allow" the access.
I've been scouring the web to figure out what the standard is here.  It seems like there is some differences of opinion on whether a user must authorize access every time or only the first time a client request access.
My ideal goal is to skip the authorization process completely, as if the user always clicked "allow".  Failing that, only require to click "allow" once for each client.
Currently I'm just auto-submitting the allow button on the decision page, but am hoping there might be a hook I can use to force "allow" in all cases.
Is this even possible? And if so, what's the proper place to splice this in?

you, my friend, are a hero! @leejefon

This is really very helpful information! Thank you @leejefon!

@leejefon You sir, just made my week! This needs to be better documented!

@leejefon thanks!


Update index.md

Please confirm this pull request meets the following requirements:

[ ] I followed the contributing guidelines: https://github.com/github/explore/blob/main/CONTRIBUTING.md.
[ ] I am not the sole author or employee of a company who created the topic or collection I am changing.

Which change are you proposing?

[X] Suggesting edits to an existing topic or collection
[ ] Curating a new topic or collection
[ ] Something that does not neatly fit into the binary options above

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

[ ] Image (and my file is *.png, square, dimensions 288x288, size <= 75 kB)
[ ] Content (and my changes are in index.md)

Please replace this line with an explanation of why you think these changes should be made.

Curating a new topic or collection

[ ] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[ ] My folder contains a *.png image (if applicable) and index.md
[ ] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/main/docs

Please replace this line with an explanation of why you think this topic or collection should be curated.

Something that does not neatly fit into the binary options above

[ ] My suggested edits are not about an existing topic or collection, or at least not a single one
[ ] My suggested edits are not about curating a new topic or collection, or at least not a single one
[ ] My suggested edits conform to the Style Guide and API docs: https://github.com/github/explore/tree/main/docs

Please replace this line with an explanation of your proposed changes.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Hi,
Thanks for starting this pull request. In order to review it for acceptance, we need to make sure that all of the prerequisites are satisfied. I see that you didn't check this:

I am not the sole author or employee of a company who created the topic or collection I am changing.

This is a requirement so that we can maintain a high level of independence in this project.
Thanks!

Hi, Thanks for starting this pull request. In order to review it for acceptance, we need to make sure that all of the prerequisites are satisfied. I see that you didn't check this:

I am not the sole author or employee of a company who created the topic or collection I am changing.

This is a requirement so that we can maintain a high level of independence in this project.
Thanks!

Hello thanks
i change this and pull again


minor: Add databricks dialect to dialect_from_str

Databricks dialect is missing in dialect_from_str function

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.001%) to 89.248%

Totals

Change from base Build<PHONE_NUMBER>7:
0.001%

Covered Lines:
28927

Relevant Lines:
32412

💛  - Coveralls

Databricks dialect is missing in dialect_from_str function

Is there anything that you plan implementing for Databricks dialect? I think databricks mostly should be able to use ansidialect.

I'm not sure I'll be able to implement anything in short term, I've tried to use it from datafusion to realise it's not there.
the edge case I've found is CREATE TABLE USING https://docs.databricks.com/en/sql/language-manual/sql-ref-syntax-ddl-create-table-using.html
please ignore PR if it should not be expose

also, as a follow up, once sqlparser has been upgraded, error messages have to handle new dialect as well

https://github.com/apache/datafusion/blob/6e790cf1f363652c3f2ed64286da75a1d5e88cb9/datafusion-cli/src/exec.rs#L218
https://github.com/apache/datafusion/blob/6e790cf1f363652c3f2ed64286da75a1d5e88cb9/datafusion/core/src/execution/session_state.rs#L491

maybe generic approach should be taken like in https://github.com/apache/datafusion/blob/6e790cf1f363652c3f2ed64286da75a1d5e88cb9/datafusion-cli/src/helper.rs#L75

maybe generic approach should be taken like in https://github.com/apache/datafusion/blob/6e790cf1f363652c3f2ed64286da75a1d5e88cb9/datafusion-cli/src/helper.rs#L75

I generic approach that lists all the dialects programatically makes a lot of sense to me.

I merged this PR up from main and added a small test


Revert "nixos/desktop-managers/xterm: Disable by default"

This reverts commit f140dfb161804871048c00f2c51485dd55b50efa.
This reverts commit cf56cefd95eb6f0ead3856ed69299b8ae7153712.
This reverts commit 456c42c3e8787d83d577526af90263de9b3d512d.
This has caused more issues than originally intended. It looks like many configurations rely on xterm being set to work correctly. This includes when you are starting a WM without going through a DE. We shouldn't break these setups (if stateVersion is unset). To make matters worse, most users don't actually know they are relying on xterm. LightDM seems to be the main one effected for now:

#68360
#70142

I expect more issues to come up as users of 19.03 upgrade to 19.09.

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @

Plus, the state version is there to help with the migration.

I don't like hacking around upstream bugs like this, despite having been impacted by this.

I disagree that this is an upstream bug. The problem is we need some placeholder WM to be available when nothing else has been manually specified. For a long time on NixOS that has been xterm, so might as well keep it available for new users. And these are just defaults, so you can always disable xterm manually if you really don't like it for some reason.

It looks like both issues were with lightdm? I think it is an upstream bug in that lightdm should give a better error message when there is no selected session. It's a classic "forgot about base case" (0) bug.

Yeah, I honestly don't understand why anyone was ever abusing the xterm session for this purpose.
As for having a placeholder WM there's https://github.com/NixOS/nixpkgs/pull/68371
Which makes much more sense for a placeholder, as a placeholder shouldn't really be much of anything. Though I'm not sure what's being supported. It feels like supporting users not configuring their systems, why is a placeholder session needed? Why can't these things be coordinated with home-manager to have a placeholder WM so they can start their .xsession?

It looks like both issues were with lightdm? I think it is an upstream bug in that lightdm should give a better error message when there is no selected session. It's a classic "forgot about base case" (0) bug.

It seems that lightdm notoriously gives no feedback in multiple cases of a session not starting.
And in particular here, where no session exists at all. Not sure if that code is greeter specific though.

Having xterm set as default caused a lot of headache on my side when I tried to setup autologin with any display manager. I failed to set desktopManager.default = "none"; because I did not read the documentation of nix options properly.. So essentially I experienced the same issue but in reverse since my window manager (xmonad) is just a little different.

Hmm, it appears at least in lightdm this failure feedback is greeter side

https://github.com/elementary/greeter/pull/335

I guess lightdm-gtk-greeter lacks code like that ^ @Ericson2314.


feat(TFD-2027): Restore avpath in filter row column name.

What is the current behavior? (You can also link to an open issue here)
FilterRow can only access top-level columns by column name.
What is the new behavior?
TFD-2027: avpath can be used to select hierarchical structures, and test against multiple values.
Change default mapType for FilterRow to FLAT_MAP
Please check if the PR fulfills these requirements

[X] The commit message follows Talend standard
[X] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[ ] Bugfix
[X] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[X] No

Pull request test result: success :star:
Sonar results

Pull request test result: failure :x:

Pull request test result: success :star:
Sonar results

Pull request test result: failure :x:

Pull request test result: success :star:
Sonar results

Pull request test result: success :star:
Sonar results


translate content_zh/docs/setup/kubernetes/upgrading-istio into chinese

original code original page
@rootsongjc please review, thanks.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: xiaosuiba
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: gyliu513
Assign the PR to them by writing /assign @gyliu513 in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content_zh/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @xiaosuiba. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gyliu513, xiaosuiba
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content_zh/OWNERS [gyliu513]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@xiaosuiba CI  failed, to fix this you need to rebase the master.
- ./public/docs/tasks/telemetry/fluentd/index.html
  *  213:53: ERROR: Unexpected end tag : p (line 213)
htmlproofer 3.9.1 | Error:  HTML-Proofer found 1 failure!

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gyliu513, rootsongjc, xiaosuiba
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content_zh/OWNERS [gyliu513,rootsongjc]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Add v03 clinvar script.

Writes to a v03 prefix so that the existing stuff doesn't change.
Only keeps the fields needed upstream.  I think we want to do this (we get a 2/3 size reduction on gcs)?
Some fixes to the ftp download to make it work locally.

Resolves https://github.com/broadinstitute/seqr-pipeline-airflow/issues/129

Also just a general question for v03, we flip between camel case and snake case(did this in v02 too)

I think the idea is in v3 to be consistent and converge on camel case as it reduces the number of transformation seqr will have to do. The one exception is we seem to have adopted the convention of camelCase_id for fields where we have mapped a string to an integer id, but if folks don't like that we can have a different naming convention. Otherwise for v3 though, we should be consitant and change variable name as needed


remove login marker cookie on logout

I was disappointed in past when a small fix was dragging for too long because of the release process not being accommodating in this way, so I'd be ok with getting this (and any safe minor bugfix for the matter) in, either as soon as possible, or indicate when such as "next patch right after the release", so that the contributor doesn't feel like we don't appreciate their effort.

@maxime-rainville this is trying to restore the functionality that was once working in SS3, see https://github.com/silverstripe/silverstripe-framework/blob/3/security/Member.php#L679
How would I write a test for this? login_marker_cookie has never had a test before, not in SS3 nor SS4. Thank for your help.

You can find more information about how to write unit test in the Silverstripe Doc:

Read Write test for an overview
Read How to write a SapphireTest for a detailed example.
Read How to Write a FunctionalTest for a detailed example on writing a functional

While I fully appreciate that this is a lot to take in, those are useful skills to learn that you can apply to our own projects.
There's already some test coverage for this class in SessionAuthenticationHandlerTest.phph. If you look at the existing tests, you should have a good starting point for your own tests.
You can run those test locally with executing this command from your project root: vendor/bin/phpunit vendor/silverstripe/framework/tests/php/Security/MemberAuthenticator/SessionAuthenticationHandlerTest.php  "" flush=all

Thanks, @maxime-rainville. I have added a test to make sure the cookie is set and removed correctly. But the build fails for I believe an unrelated issue? not sure?

@maxime-rainville, I have fixed the tests, squashed the commits and rebased the PR onto 4.9. But the travis build for some reason picked branch 4.8 as a base and failed?

I think travis is not smart enough to realise when your PR targeted branch has changed. If you force push another commit on your branch, that should unstuck it.

@maxime-rainville thanks, this worked.

@maxime-rainville any updates on this? Cheers

@maxime-rainville as this is a bugfix, could this still go into 4.10?


Scrollbar Broken

About this discussion
The scrollbar in the app is not functioning as it should. The letter is not updating and is just staying as #. In addition it goes off the bottom of the page underneath the toolbar.
https://github.com/user-attachments/assets/56de99b2-2808-4c92-a695-5987e815506c

The scrollbar label is intentionally non-functional for this release, as fixing it requires additional changes (and Lawnicons is overdue by 1 month in terms of the release schedule).
I plan on fixing this as well soon.

Re-opening this issue with label, until we release Lawnicons 2.12.0.


AliasLanguages should report error when users define same extension twice

⏯ Playground Link

Playground link with relevant code
💻 Code

Code:
ruleDirs:
  - rules
utilDirs:
  - utils
aliasLanguages:
  vue:
    aliasOf: html
    extensions: [vue]
  vueTsx:
    aliasOf: tsx
    extensions: [vue]

🙁 Actual behavior
The second alias language will be ignored.

🙂 Expected behavior
AliasLanguages should report error when users define same extension twice

You can define two config to resolved it:

sgconfig.yml for html.

ruleDirs:
  - rules
utilDirs:
  - utils
aliasLanguages:
  vue:
    aliasOf: html
    extensions: [vue]
  vueTsx:
    aliasOf: tsx
    extensions: []

sgconfig-tsx.yml for tsx.

ruleDirs:
  - rules
utilDirs:
  - utils
aliasLanguages:
  vue:
    aliasOf: html
    extensions: []
  vueTsx:
    aliasOf: tsx
    extensions: [vue]

This is solved by not using aliasLang!


Missing mention of dependency in readme

I was unable to install this module unless I had libssl-dev installed.
Could be useful if you added it to the readme. :)

thank you for your advisement.:)  This issue has been fixed.


MESSAGE_UPDATE goes boom for webhooks that have embeds

https://gist.github.com/queer/d7a7dd8fb8eb8e8f56bb277c70954c4e

Couldn't reproduce.


Move to pragma once, clang modules support, turn inlining off by defa…

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Do we have profiling data that the inline static declarations significantly improve performance? Maybe we should rip off the bandaid and just use the normal C calling style.


Code Highlighting doesn't work with Golang

If you try to execute this script in Ranna and select golang as the Language, it doesn't highlight any word.
package main

import "fmt"

func main() {
    fmt.Println("Hello World.")
}

wrong repo


[WIP] Document api

This is the first draft of the "Device API". Note that it is a first draft, and requires cleanup, notably formatting, reducing duplication, style and others, specifically the API.md document.
It includes the following:

Write an initial draft of the actual Device API in api/API.md
Update api/README.md to reflect the proper relationship between and locations of protobufs, generated code and consumers
Write the correct README.md in sdk/README.md and sdk/go/README.md
Write docs/REGISTRATION.md which defines the EVE-specific boot process and how it interacts with a Controller via the Device API

@aw-zededa can you take a look here please? yetus is complaining about tabs instead of whitespace in code-quoted inside markdown. That looks like a bug to me.
The error report is:
./sdk/go/README.md:9:	"github.com/zededa/eve/sdk/zconfig"
./sdk/go/README.md:10:	"github.com/zededa/api/zmet"

The markdown is:

To use:

```go
import (
        "github.com/zededa/eve/sdk/zconfig"
        "github.com/zededa/api/zmet"
)
```

That should be tabs, as go fmt insists.

@aw-zededa can you take a look here please? yetus is complaining about tabs instead of whitespace in code-quoted inside markdown. That looks like a bug to me.

The whitespace test plugin is super, super simple.  The only context it takes into consideration is the filename.  It may be worthwhile to set it to be 'advisory' by changing the Yetus configuration by adding a --test-filter=whitespace to the command line.

It may be worthwhile to set it to be 'advisory' by changing the Yetus configuration by adding a --tests-filter=whitespace to the command line

@aw-zededa are you suggesting to do that in .circleci/config.yml by adding:
--tests-filter=whitespace

or by adding markdown files to the list of excludes here by changing it to:
--whitespace-tabs-ignore-list='.*Makefile.*','.*\.go','.*\.dts','.*\.md'

@aw-zededa are you suggesting to do that in .circleci/config.yml by adding:

Either should work. Yetus gives you choices. :)  It's really a project decision.  The project can decide tabs are ok in markdown files by setting the ignore flag or it can decide that Yetus should just warn about tabs in markdown but not fail the PR if they show up. (The other choice is to just turn off the whitespace plugin altogether, but then you lose the end of line support part.  Probably should separate those out at some point.)

it can decide that Yetus should just warn about tabs in markdown but not fail the PR if they show up.

I am going to go for the "warn about tabs but not fail the PR if they show up in markdown." If it becomes an issue again later, we can revisit. Minimal change for now.
That having been said, I am glad it ignores *.go files. Those should be passed through go fmt and go lint.
Separate PR coming shortly...

See #59

Merged in #59 to fix the yetus failing (thanks @aw-zededa), will address @eriknordmark 's extensive comments shortly.

Hey @deitch & @nordmark  do you guys think we can merge it in its current form and keep evolving?
Especially with the reshuffling of APIs in #64 we may need to update the docs a bit more based on things like one API folder per REST end-point, etc.

First of all, the style and structure of what I wrote here is, shall we say, somewhat embarrassing. It needs a few hours of work to make it not so. I still wanted to get the mess out so we could agree on the API. Let's do that first.
More importantly, changing an API, especially its endpoints and requirements, can be pretty radical. I would be happier giving us into the first or second day of the week to get this to a reasonably agreed state.

Makes sense @deitch -- let us know when you're ready to merge it. My only point is that we have to be ready for 5/10 cut-over to Linux Foundation.

@rvs : Erik and I are working out the spec/protocol issues. Once it is agreed, we can merge it in. I can do the proper style/formatting/etc. cleanup in another PR. We will keep you appraised here.

@eriknordmark : updated based on our video call. Covered the ping and register endpoints, flows, auth+enc, etc. Take a look and see.
Back at you....

Thanks for those comments @eriknordmark . Updated and repushed.

Just waiting for CI to pass, and then will merge. I know it is only documentation, but I prefer to be a stickler for the process.
Thanks @eriknordmark !


arrow-cpp: add GCS feature flag

Motivation for this change
Add GCS support for arrow-cpp.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Well, that's fun. It looks like we're hitting this https://github.com/boostorg/process/issues/55. In theory this should build if we use boost170 on darwin. I'll add an assert so that people are aware of the need to upgrade to boost170 on darwin when s3 is enabled :grimacing:


feat: [provenance] upgrade provenance to v1.14.1 and proposals parsing to correctly parse v0.46.x gov proposals

Description
Closes: #XXXX

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[x] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[x] targeted the correct branch
[x] provided a link to the relevant issue or specification
[x] added a changelog entry to CHANGELOG.md
[x] included comments for documenting Go code
[x] updated the relevant documentation or specification
[x] reviewed "Files changed" and left comments if necessary
[x] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed ! in the type prefix if API or client breaking change
[ ] confirmed all author checklist items have been addressed
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

Codecov Report

:exclamation: No coverage uploaded for pull request base (chains/provenance/mainnet@51ff03e). Click here to learn what that means.
The diff coverage is n/a.

@@                     Coverage Diff                      @@
##             chains/provenance/mainnet     #534   +/-   ##
============================================================
  Coverage                             ?   48.87%
============================================================
  Files                                ?       21
  Lines                                ?     1731
  Branches                             ?        0
============================================================
  Hits                                 ?      846
  Misses                               ?      778
  Partials                             ?      107


Swift Package Manager support

I added Swift Package Manager support, but unfortunately I can't compile the Example Project.
/Users/marius/Developer/iOS/Hero/Examples/HeroExamples/Examples/ImageGallery/ImageViewController.swift:67:38: Binary operator '+' cannot be applied to two 'CGPoint' operands
maybe you can have a look @lkzhao

It won't compile because the example project is using the CG+Hero.swift file. Basically, that file is added to both the Hero framework target and the HeroExamples app target. Since you remove the Hero framework target and move it outside. You cannot add CG+Hero.swift into the example project's target anymore.

Removing the target will break Carthage support and won't display warnings for changes we make inside the Hero framework.
Can SPM work without removing the target?

I also thought SPM don't support iOS projects yet. Is that fixed now?

SPM works with iOS but only when you are doing things manually at the moment. But IMO SPM will be heading in the right way and to support this right out of the box is a good addition. Especially if SPM works easier with iOS together

now it should be everything ok :)

Awesome! Thank you! Can you tell me how to install this via the SPM? I have no experience using it. I can put this into the usage guide.

I will update it tomorrow directly in the README.md 🍻

@mRs- Sure. Please add it here: https://github.com/lkzhao/Hero/wiki/Usage-Guide

I'm done with adding it to the usage guide.

There are some conflicts. Can you resolve them? The files you moved are changed in the master branch. Guess you will have to move them again. Thanks!

Done


memory leak when the network update function is used

I'm seeing a memory leak in recent versions of tev when sending images from pbrt running another process. (I've been seeing this for a while now, maybe a few months, but only made a repro case now--apologies.)
The attached repro program (extracted from pbrt, so not minimal!) takes a single command-line argument and connects to a running tev server at that address. It then spams it continuously with image updates. I am seeing two things (this is on OSX, FWIW):

tev's memory use (as reported by top) grows by about 1 MB every second
interestingly, if I hide tev (command-H), or make another window active and then click on tev to make it active again, memory use jumps up by about 50MB within a few seconds than another 40MB or so again a few seconds later.

So, between those two, it can add up to gigabytes after a while, especially with a long render running.

I did a little archaeology and the leak seems to go back to at least bb2b088, so it doesn't seem to have been introduced recent.y

Inspecting the code, I'm fairly certain I know what it is, thanks for letting me know!
It seems like old color histograms are never cleared -- even when an image that generated them is closed or mutated. I guess back when I implemented the histograms (before image updates over the network were a thing), I thought there would never be more than a couple thousand unique images per tev session. Whoops. :P
(Fix incoming tomorrow. :))

To shed a bit more light on why histograms aren't directly associated with their "images" in the first place:
there is actually a unique histogram for each combination of image, reference image, error metric, and selected channels -- too many combinations to exhaustively enumerate.
Thus, histograms live in a separate data structure that also handles their lazy computation in the background (to keep things snappy). This makes deletion without data races every so slightly non-trivial, which is (I presume) why I didn't add it back in the day. Really should have, retrospectively... typical example of "should have done things properly from the start"!

If you have a C++20 compiler handy, I would appreciate if you could try the above PR to see if it gets rid of the problem with your test program.

I just tried it (by checking out the cpp20 branch). Unfortunately it still seems to be leaking.

Okay, I'll dig some more. Could you try attaching the repro program again? I don't think it got correctly bundled with your original post.

Oops. Look like I forgot to attach it--apologies. Here it is:
tevspew.cpp.txt

I have done some more digging and have some more information. Shortly after starting to send the image, there have been 6 allocations of a total of 150MB with this call stack (out of a total of about 250MB of memory used.)
      6 (150M) CONTENT:  VM: IOSurface
      + 6 (150M) IOSurfaceClientCreate  (in IOSurface) + 0  [0x18d42b4d4]
      +   6 (150M) -[IOSurface initWithProperties:]  (in IOSurface) + 80  [0x18d42b490]
      +     6 (150M) CA::SurfaceUtil::CAIOSurfaceCreate(unsigned int, unsigned int, unsigned int, unsigned int, unsigned int, unsigned int, unsigned long long, CA::SurfaceUtil::SurfaceAlignment, __CFString const*)  (in QuartzCore) + 12264  [0x18bc18ca0]
      +       6 (150M) CA::Render::create_iosurface_with_pixel_format(unsigned int, unsigned int, unsigned int, unsigned int, unsigned long long, __CFString const*)  (in QuartzCore) + 288  [0x18bb61700]
      +         6 (150M) get_unused_drawable(_CAMetalLayerPrivate*, bool, bool)  (in QuartzCore) + 412  [0x18bb61404]
      +           5 (125M) -[CAMetalLayer nextDrawable]  (in QuartzCore) + 1296  [0x18bb60fd4]
      +           ! 4 (100M) nanogui::metal_window_next_drawable(void*)  (in tev) + 88  [0x1043d3f94]  darwin.mm:178
      +           ! : 4 (100M) nanogui::Screen::draw_setup()  (in tev) + 52  [0x1043dafd8]  screen.cpp:573
      +           ! :   4 (100M) nanogui::Screen::draw_all()  (in tev) + 40  [0x1043db0a4]  screen.cpp:618
      +           ! :     2 (50.0M) nanogui::mainloop(float)  (in tev) + 520  [0x1043d43f0]  common.cpp:185
      +           ! :     | 2 (50.0M) tev::mainFunc(std::__1::vector<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > > const&)  (in tev) + 7108  [0x104311ed4]  main.cpp:449
      +           ! :     |   2 (50.0M) main  (in tev) + 820  [0x104314bd8]  main.cpp:494
      +           ! :     |     2 (50.0M) start  (in libdyld.dylib) + 4  [0x1857d1430]
      +           ! :     1 (25.0M) nanogui::Screen::resize_event(nanogui::Array<int, 2ul> const&)  (in tev) + 76  [0x1043db614]  screen.cpp:728
      +           ! :     | 1 (25.0M) nanogui::Screen::resize_callback_event(int, int)  (in tev) + 152  [0x1043dc380]  screen.cpp:906
      +           ! :     |   1 (25.0M) -[GLFWWindowDelegate windowDidResize:]  (in tev) + 236  [0x104390800]  cocoa_window.m:267
      +           ! :     |     1 (25.0M) __CFNOTIFICATIONCENTER_IS_CALLING_OUT_TO_AN_OBSERVER__  (in CoreFoundation) + 28  [0x1858a5f08]
      +           ! :     |       1 (25.0M) ___CFXRegistrationPost_block_invoke  (in CoreFoundation) + 52  [0x185947e5c]
      +           ! :     |         1 (25.0M) _CFXRegistrationPost  (in CoreFoundation) + 456  [0x185947dc8]
      +           ! :     |           1 (25.0M) _CFXNotificationPost  (in CoreFoundation) + 720  [0x1858749d8]
      +           ! :     |             1 (25.0M) -[NSNotificationCenter postNotificationName:object:userInfo:]  (in Foundation) + 64  [0x1866003c0]
      +           ! :     |               1 (25.0M) -[NSWindow _setFrameCommon:display:fromServer:]  (in AppKit) + 3156  [0x1880f6ac0]
      +           ! :     |                 1 (25.0M) _glfwPlatformSetWindowSize  (in tev) + 292  [0x104392b60]  cocoa_window.m:1055
      +           ! :     |                   1 (25.0M) tev::ImageViewer::ImageViewer(std::__1::shared_ptr<tev::BackgroundImagesLoader> const&, bool, bool, bool)  (in tev) + 13736  [0x1042ecf1c]  ImageViewer.cpp:455
      +           ! :     |                     1 (25.0M) tev::mainFunc(std::__1::vector<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > > const&)  (in tev) + 6768  [0x104311d80]  main.cpp:433
      +           ! :     |                       1 (25.0M) main  (in tev) + 820  [0x104314bd8]  main.cpp:494
      +           ! :     |                         1 (25.0M) start  (in libdyld.dylib) + 4  [0x1857d1430]
      +           ! :     1 (25.0M) tev::mainFunc(std::__1::vector<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > > const&)  (in tev) + 6792  [0x104311d98]  main.cpp:435
      +           ! :       1 (25.0M) main  (in tev) + 820  [0x104314bd8]  main.cpp:494
      +           ! :         1 (25.0M) start  (in libdyld.dylib) + 4  [0x1857d1430]
      +           ! 1 (25.0M) -[MNVGcontext renderFlush]  (in tev) + 568  [0x1043cae64]  nanovg_mtl.m:1405
      +           !   1 (25.0M) nvgEndFrame  (in tev) + 40  [0x1043a5148]  nanovg.c:396
      +           !     1 (25.0M) nanogui::Screen::initialize(GLFWwindow*, bool)  (in tev) + 624  [0x1043dac44]  screen.cpp:505
      +           !       1 (25.0M) nanogui::Screen::Screen(nanogui::Array<int, 2ul> const&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, bool, bool, bool, bool, bool, bool, unsigned int, unsigned int)  (in tev) + 976  [0x1043da83c]  screen.cpp:406
      +           !         1 (25.0M) tev::ImageViewer::ImageViewer(std::__1::shared_ptr<tev::BackgroundImagesLoader> const&, bool, bool, bool)  (in tev) + 144  [0x1042e9a04]  ImageViewer.cpp:33
      +           !           1 (25.0M) tev::mainFunc(std::__1::vector<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > > const&)  (in tev) + 6768  [0x104311d80]  main.cpp:433
      +           !             1 (25.0M) main  (in tev) + 820  [0x104314bd8]  main.cpp:494
      +           !               1 (25.0M) start  (in libdyld.dylib) + 4  [0x1857d1430]
      +           1 (25.0M) -[CAMetalLayer nextDrawable]  (in QuartzCore) + 640  [0x18bb60d44]
      +             1 (25.0M) nanogui::metal_window_next_drawable(void*)  (in tev) + 88  [0x1043d3f94]  darwin.mm:178
      +               1 (25.0M) nanogui::Screen::draw_setup()  (in tev) + 52  [0x1043dafd8]  screen.cpp:573
      +                 1 (25.0M) nanogui::Screen::draw_all()  (in tev) + 40  [0x1043db0a4]  screen.cpp:618
      +                   1 (25.0M) nanogui::mainloop(float)  (in tev) + 520  [0x1043d43f0]  common.cpp:185
      +                     1 (25.0M) tev::mainFunc(std::__1::vector<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > > const&)  (in tev) + 7108  [0x104311ed4]  main.cpp:449
      +                       1 (25.0M) main  (in tev) + 820  [0x104314bd8]  main.cpp:494
      +                         1 (25.0M) start  (in libdyld.dylib) + 4  [0x1857d1430]

After a few hide/reveal cycles, total memory use is up to 397MB and there have been 5 more allocations along that call stack, now accounting for 275MB.
      11 (275M) CONTENT:  VM: IOSurface
      + 11 (275M) IOSurfaceClientCreate  (in IOSurface) + 0  [0x18d42b4d4]
      +   11 (275M) -[IOSurface initWithProperties:]  (in IOSurface) + 80  [0x18d42b490]
      +     11 (275M) CA::SurfaceUtil::CAIOSurfaceCreate(unsigned int, unsigned int, unsigned int, unsigned int, unsigned int, unsigned int, unsigned long long, CA::SurfaceUtil::SurfaceAlignment, __CFString const*)  (in QuartzCore) + 12264  [0x18bc18ca0]
      +       11 (275M) CA::Render::create_iosurface_with_pixel_format(unsigned int, unsigned int, unsigned int, unsigned int, unsigned long long, __CFString const*)  (in QuartzCore) + 288  [0x18bb61700]
      +         11 (275M) get_unused_drawable(_CAMetalLayerPrivate*, bool, bool)  (in QuartzCore) + 412  [0x18bb61404]
      +           10 (250M) -[CAMetalLayer nextDrawable]  (in QuartzCore) + 1296  [0x18bb60fd4]
      +           ! 9 (225M) nanogui::metal_window_next_drawable(void*)  (in tev) + 88  [0x1043d3f94]  darwin.mm:178
      +           ! : 9 (225M) nanogui::Screen::draw_setup()  (in tev) + 52  [0x1043dafd8]  screen.cpp:573
      +           ! :   9 (225M) nanogui::Screen::draw_all()  (in tev) + 40  [0x1043db0a4]  screen.cpp:618
      +           ! :     7 (175M) nanogui::mainloop(float)  (in tev) + 520  [0x1043d43f0]  common.cpp:185
      +           ! :     | 7 (175M) tev::mainFunc(std::__1::vector<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > > const&)  (in tev) + 7108  [0x104311ed4]  main.cpp:449
      +           ! :     |   7 (175M) main  (in tev) + 820  [0x104314bd8]  main.cpp:494
      +           ! :     |     7 (175M) start  (in libdyld.dylib) + 4  [0x1857d1430]
      +           ! :     1 (25.0M) nanogui::Screen::resize_event(nanogui::Array<int, 2ul> const&)  (in tev) + 76  [0x1043db614]  screen.cpp:728
      +           ! :     | 1 (25.0M) nanogui::Screen::resize_callback_event(int, int)  (in tev) + 152  [0x1043dc380]  screen.cpp:906
      +           ! :     |   1 (25.0M) -[GLFWWindowDelegate windowDidResize:]  (in tev) + 236  [0x104390800]  cocoa_window.m:267
      +           ! :     |     1 (25.0M) __CFNOTIFICATIONCENTER_IS_CALLING_OUT_TO_AN_OBSERVER__  (in CoreFoundation) + 28  [0x1858a5f08]
      +           ! :     |       1 (25.0M) ___CFXRegistrationPost_block_invoke  (in CoreFoundation) + 52  [0x185947e5c]
      +           ! :     |         1 (25.0M) _CFXRegistrationPost  (in CoreFoundation) + 456  [0x185947dc8]
      +           ! :     |           1 (25.0M) _CFXNotificationPost  (in CoreFoundation) + 720  [0x1858749d8]
      +           ! :     |             1 (25.0M) -[NSNotificationCenter postNotificationName:object:userInfo:]  (in Foundation) + 64  [0x1866003c0]
      +           ! :     |               1 (25.0M) -[NSWindow _setFrameCommon:display:fromServer:]  (in AppKit) + 3156  [0x1880f6ac0]
      +           ! :     |                 1 (25.0M) _glfwPlatformSetWindowSize  (in tev) + 292  [0x104392b60]  cocoa_window.m:1055
      +           ! :     |                   1 (25.0M) tev::ImageViewer::ImageViewer(std::__1::shared_ptr<tev::BackgroundImagesLoader> const&, bool, bool, bool)  (in tev) + 13736  [0x1042ecf1c]  ImageViewer.cpp:455
      +           ! :     |                     1 (25.0M) tev::mainFunc(std::__1::vector<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > > const&)  (in tev) + 6768  [0x104311d80]  main.cpp:433
      +           ! :     |                       1 (25.0M) main  (in tev) + 820  [0x104314bd8]  main.cpp:494
      +           ! :     |                         1 (25.0M) start  (in libdyld.dylib) + 4  [0x1857d1430]
      +           ! :     1 (25.0M) tev::mainFunc(std::__1::vector<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > > const&)  (in tev) + 6792  [0x104311d98]  main.cpp:435
      +           ! :       1 (25.0M) main  (in tev) + 820  [0x104314bd8]  main.cpp:494
      +           ! :         1 (25.0M) start  (in libdyld.dylib) + 4  [0x1857d1430]
      +           ! 1 (25.0M) -[MNVGcontext renderFlush]  (in tev) + 568  [0x1043cae64]  nanovg_mtl.m:1405
      +           !   1 (25.0M) nvgEndFrame  (in tev) + 40  [0x1043a5148]  nanovg.c:396
      +           !     1 (25.0M) nanogui::Screen::initialize(GLFWwindow*, bool)  (in tev) + 624  [0x1043dac44]  screen.cpp:505
      +           !       1 (25.0M) nanogui::Screen::Screen(nanogui::Array<int, 2ul> const&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, bool, bool, bool, bool, bool, bool, unsigned int, unsigned int)  (in tev) + 976  [0x1043da83c]  screen.cpp:406
      +           !         1 (25.0M) tev::ImageViewer::ImageViewer(std::__1::shared_ptr<tev::BackgroundImagesLoader> const&, bool, bool, bool)  (in tev) + 144  [0x1042e9a04]  ImageViewer.cpp:33
      +           !           1 (25.0M) tev::mainFunc(std::__1::vector<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > > const&)  (in tev) + 6768  [0x104311d80]  main.cpp:433
      +           !             1 (25.0M) main  (in tev) + 820  [0x104314bd8]  main.cpp:494
      +           !               1 (25.0M) start  (in libdyld.dylib) + 4  [0x1857d1430]
      +           1 (25.0M) -[CAMetalLayer nextDrawable]  (in QuartzCore) + 640  [0x18bb60d44]
      +             1 (25.0M) nanogui::metal_window_next_drawable(void*)  (in tev) + 88  [0x1043d3f94]  darwin.mm:178
      +               1 (25.0M) nanogui::Screen::draw_setup()  (in tev) + 52  [0x1043dafd8]  screen.cpp:573
      +                 1 (25.0M) nanogui::Screen::draw_all()  (in tev) + 40  [0x1043db0a4]  screen.cpp:618
      +                   1 (25.0M) nanogui::mainloop(float)  (in tev) + 520  [0x1043d43f0]  common.cpp:185
      +                     1 (25.0M) tev::mainFunc(std::__1::vector<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > > const&)  (in tev) + 7108  [0x104311ed4]  main.cpp:449
      +                       1 (25.0M) main  (in tev) + 820  [0x104314bd8]  main.cpp:494
      +                         1 (25.0M) start  (in libdyld.dylib) + 4  [0x1857d1430]

Whether it's a nanogui bug or a bug in tev's usage of nanogui, I have no idea!
(For posterity, this was measured by running tev with the MallocStackLogging env variable set and then running malloc_history [TEV-PID] -callTree -invert -showContent to dump out stack traces of all of the allocations.)

(tev doesn't seem to ever call metal_present_and_release_drawable(); I'm not familiar with nanogui usage, but it seems fishy that it repeatedly acquires them but doesn't release them.)

Actually now I see that it isn't tev calling metal_window_next_drawable but that it's a nanogui class, so nevermind, perhaps.

After some additional digging, it seems that merely refreshing the screen is causing the leak. I can make memory usage climb by just opening tev and rapidly moving the mouse.
Image updates seem to be an innocent bystander that just happens to cause extra screen refreshes.
This seems to be related to nanogui's usage of the Metal backend (perhaps even nanovg_metal). When compiling with OpenGL, the leak disappears, but so does EDR support. :(
I've given the nanogui/nanovg Metal code a cursory glance and think I narrowed it down somewhat (see https://github.com/mitsuba-renderer/nanogui/issues/93 ), but this is starting to leave my bubble of expertise. Perhaps Wenzel has an idea.

nanogui is fixed now -- I can no longer reproduce the memory leak. Closing. :)

Confirmed fixed here as well!


Window is sometimes not redrawn with tiling window manager on Linux

I use a tiling window manager such as XMonad on Linux and LightTable sometimes does not redraw the window. Attached is an image of the problem (scaled down for convenience).

Anyone – comment whether this is still unresolved, particularly with a build of the latest source (on the master branch), and we can reopen it.

This is fixed.

@sstewartgallus Excellent


Defer encryption if host is upgrading Vespa

@hakonhall

Yes, the same mechanism is used for OS upgrades.


WIP: irqwatch command

TBD
Signed-off-by: Francesco Romani<EMAIL_ADDRESS>

@Tal-or has a point here. Let's first fix ReadStats. So spun off https://github.com/openshift-kni/debug-tools/pull/25/ to handle it.

/lgtm


Host showing up as hosting itself

If you look at OCNZ's page, it shows OCNZ under the contributions section, with the label hosted since... which implies it's a hosted Collective.

If you switch to the hosted Collectives view, it doesn't show up, but it's confusing to have a card there representing and linking to the page you're already on.

This is still happening. It looks really weird for hosts with only one or a few Collectives, because the card representing itself is very obvious.

If the heading is going to say "We are fiscally hosting this Collective" then things which are not hosted Collectives cannot be allowed to appear here. Also, it messes up the grammar because that sentence refers to a singular thing, but two cards show below.

This is still happening. It looks really weird for hosts with only one or a few Collectives, because the card representing itself is very obvious.

If the heading is going to say "We are fiscally hosting this Collective" then things which are not hosted Collectives cannot be allowed to appear here. Also, it messes up the grammar because that sentence refers to a singular thing, but two cards show below.


ConfirmMenuItem style different from MenuItem

Here is a video demo of the ConfirmMenuItem component. The Supprimer ConfirmMenuItem is different from the other MenuItem.
https://user-images.githubusercontent.com/3920615/121869467-5d11ac00-cd02-11eb-8c3d-7f9b2c69b50f.mp4

Can I take this one !

Ok @yoannfleurydev, thanks!

Any news ?

Hey, the issue fixed, watch this :

https://user-images.githubusercontent.com/71929996/146028002-b7de7cef-66e4-45e2-8aa1-3633be192bd2.mov

https://user-images.githubusercontent.com/71929996/146028023-a7a221c1-cd0f-40f0-a248-46f83ed38c3e.mov

Hi @ImbuPro, thanks for your time. This seems to be fixed, probably by a Chakra UI update as we didn't change anything for this. I close this issue.


Child webviews

Is your feature request related to a problem? Please describe.
I'm trying to create a browser. Tauri fits all my needs except that it hasn't support for webviews in windows. See for example Electron's BrowserView.
Describe the solution you'd like
Webview support.

Duplicate of #2709.

Oh! Sorry. Forgot to check for duplicates.


Zong Han - Update DG and PPP

Changes
This PR updates the DG to include implementation details of specific sections in the appointments subsystem and the implementation details of the UI display list switching. It also includes updates to the PPP for iteration 1.2. A preview is available here.
Related Issues

Closes #182

Codecov Report

Merging #200 (b978c9a) into master (51999fa) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #200   +/-   ##
=========================================
  Coverage     72.63%   72.63%
  Complexity      997      997
=========================================
  Files           137      137
  Lines          3062     3062
  Branches        350      350
=========================================
  Hits           2224     2224
  Misses          775      775
  Partials         63       63

Continue to review full report at Codecov.

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


Improve kos-registry dependency

See also: https://github.com/gbv/kos-registry/issues/9
Currently, if you already had kos-registry as a subfolder, npm run kos-registry will not refresh that. Instead, manually pulling and running npm run all in kos-registry is necessary.

Fixed by download


Question on token storage

Excuse me, does oauth_flutter automatically save and load tokens? If not, how can I save and load the token? Thank you for your patience in answering my question.

Yes it handles token storage automatically

Also for questions like this I would much rather communicate on Discord if that's something you're comfortable with. I have a server link in my GitHub profile.

Yes it handles token storage automatically

I restarted the application, and it asked me to click authorize button on oauth page. Is this the correct behavior?
Thank you.

Also for questions like this I would much rather communicate on Discord if that's something you're comfortable with. I have a server link in my GitHub profile.

I'm worried that instant messaging may affect your work and rest.


docs(readme): consider using merge-chance badge

Description
Some more statistics and a badge that comes with it that you can put into your readme:
https://merge-chance.info/target?repo=hyperledger/cactus
cc: @sfuji822 @takeutak @hartm @jonathan-m-hamilton

The merge-chance product no longer exists (domain redirects to a registrar parking page or similar spam site)


Remove prophet-specific testing

With the resolution of #3908, prophet is no longer a "special" dependency that can only be installed on some machines and needs explicit testing. We should remove our prophet-specific make functions, install methods, and tests since they are no longer needed.

Note: this should resolve #3190 as well


Console application does not successfully deploy to the App Service

Does this occur consistently? Yes
Repro steps:
1º Right-Click the code folder
2º Choose Deploy to Web App...
I only managed to deploy my code by uploading it in a zip file in the portal.
The failure only occurs when trying to do it from VSCode, but the error message doesn't help at all
Action: appService.Deploy
Error type: 5
Error Message: Task to execute is undefined
Version: 0.16.5
OS: win32
OS Release: 10.0.17134
Product: Visual Studio Code
Product Version: 1.42.1
Language: en

@FlavioDiasPs what type of app is this? Dotnet? Do you have "appService.preDeployTask" set in ".vscode/settings.json" - you might want to check.
That being said - the error message is a little gross, so we still want to improve that if you can give us more detailed repro steps


Add region specific tables

Several parameters in Temoa require users to specify a region. This information should be stored in the Technology class as a dictionary. E.g.
self.input_comm = {'ILLINOIS':steam,
                               'DELAWARE':hydrogen,
                               'OREGON':electricity
                              }

The following "tables" should be implemented

[x] efficiency
[x] LifetimeTech
[ ] LifetimeProcess
[ ] LifetimeLoanTech
[ ] Capactiy2Activity
[x] CapacityFactor
[x] CostFixed
[x] CostInvest
[x] CostVariable

This issue has taken much longer than originally planned because virtually every table in Temoa has a region component.
Further tables that need to be implemented

[ ] ramp up
[ ] ramp down
[ ] planning reserve margin
[ ] emission activity
[ ] emission constraint


Remove I18n from controllers, views and helpers

Review app deployed to https://teaching-vacancies-review-pr-2527.london.cloudapps.digital


Remove another instance of flyway baseline

For consistency with 681d234e1b7bddd03f356ddab80b367de3626702 - we
should configure flyway the same way everywhere.

👍

very nice


files: fixes for reading and decoding files

Codecov Report

Merging #76 into master will decrease coverage by 0.37%.
The diff coverage is 63.26%.

@@            Coverage Diff             @@
##           master      #76      +/-   ##
==========================================
- Coverage   91.99%   91.61%   -0.38%
==========================================
  Files          35       35
  Lines        4261     4307      +46
==========================================
+ Hits         3920     3946      +26
- Misses        229      240      +11
- Partials      112      121       +9


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-25-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3559

Questions ?
Please refer to the Backport tool documentation


fix(DEVOPS-3524): align simple rest docker image tagging to all components one

What is the current behavior? (You can also link to an open issue here)
The simple rest imaged is tagged with the docker version
What is the new behavior?
Get the same type of tagging as the "all components" image with the project version and timestamp
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: success :star:
Sonar results

the docker build fails
mvn clean install -DskipTests -Pdocker
I have this error
[ERROR] Failed to execute goal io.fabric8:docker-maven-plugin:0.20.1:build (start) on project components-api-service-rest: Execution start of goal io.fabric8:docker-maven-plugin:0.20.1:build failed: Given Docker name 'registry.datapwn.com/talend/tcomp-components-api-service-rest-local:0.21.0-SNAPSHOT-2017-12-13T17:20:13Z' is invalid:
[ERROR]    * tag part '0.21.0-SNAPSHOT-2017-12-13T17:20:13Z' doesn't match allowed pattern '^[\w][\w.-]{0,127}$'

The mvn clean install -DskipTests -Pdocker is succesful on my side and the images are properly created.
It seems that on your env, the maven.build.timestamp gives a different results than on my laptop and on the CI, but it should have been already the case for the all-components build

Pull request test result: success :star:
Sonar results

this works fine now, thanks


How change the package name of tensorflow android camera demo

I have built two Android demo with two different custom models. However in order to install both apps on my device I need to change the name of one of these. Is there an easy way to change the name of the apk file?

You can change the Android package in AndroidManifest.xml to avoid the conflict.


Menu/Navigation - Changes

The goal of this ticket is to make the menu more concise and user-friendly.
In the menu,

Bold "Bills" and change to "Browse Bills"
By default, collapse "Learn" so that the sub-page titles show once "Learn" is expanded
By default, collapse "About" so that "Our Mission & Goals" and "Our Team" show once "About" is expanded. Also, add a new subpage in About, titled "How to Support MAPLE" (copy to come in another ticket)
Create a new parent menu item, titled "Why use MAPLE", and put the "for legislators", "for organizations" and "for testifiers" as, by default collapsed, only showing once "Why Use MAPLE" is clicked on. Change "For Testifiers" to "For Individuals"

@mvictor55
Do you want the "for testifiers" language changed to "for individuals" across the site? If so, I can create another issue and work in updating that.

Nope - just that page! We’re differentiating between the types of
testifiers on these pages
On Tue, Mar 28, 2023 at 10:34 PM Tim Blais @.***> wrote:

@mvictor55 https://github.com/mvictor55
Do you want the "for testifiers" language changed to "for individuals"
across the site? If so, I can create another issue and work in updating
that.
—
Reply to this email directly, view it on GitHub
https://github.com/codeforboston/maple/issues/1012#issuecomment-1487863175,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARANGSAGYQXWTBMHV6YKE6LW6ONUXANCNFSM6AAAAAAWIQ6YOI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Please change header and url! Good catch.
Copy for the new page will be out later today
On Wed, Mar 29, 2023 at 8:19 AM Tim Blais @.***> wrote:

@mvictor55 https://github.com/mvictor55
I think I worded that in a confusing way. To clarify, do you want the
header and url updated on the actual page, or just in the navbar?
[image: fortestifiers]
https://user-images.githubusercontent.com/97904016/228532318-00e908a2-b838-4f68-9657-0a7e349cc5b5.PNG
I'll be able to put some time in tonight after work. If you have the copy
for the "How to Support MAPLE" page, let me know and I can work on putting
that together.
—
Reply to this email directly, view it on GitHub
https://github.com/codeforboston/maple/issues/1012#issuecomment-1488499341,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARANGSBHXLEGKFOC6XSOITLW6QSFPANCNFSM6AAAAAAWIQ6YOI
.
You are receiving this because you were mentioned.Message ID:
@.***>

PR #1018 closes the initial portions of the issue
awaiting for review:

[ ] copy for How to Support MAPLE
[ ] implementation of page for about/how-to-support-maple route

Should I be seeing the new menu in the dev site? I don’t think I see it
On Wed, Mar 29, 2023 at 9:16 AM mertbagt @.***> wrote:

PR #1018 https://github.com/codeforboston/maple/pull/1018 closes the
initial portions of the issue
awaiting for review:

copy for How to Support MAPLE
implementation of page for about/how-to-support-maple route

—
Reply to this email directly, view it on GitHub
https://github.com/codeforboston/maple/issues/1012#issuecomment-1488593327,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARANGSFBDLHC7LTWQJSFZE3W6QY3TANCNFSM6AAAAAAWIQ6YOI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Here's the text for the "How to Support MAPLE page": https://docs.google.com/document/d/1fcIckkjpFJRNaYsHgyvPsQnYVLxdefF4I-RTL4GNHGQ/edit
If you're comfortable applying the basic MAPLE formatting for this, please do so. I've also asked our design team to draw it up, but we don't need to wait on that (and will implement later).

Hi @mvictor55,
Thanks, I'll get that put together tonight with the standard formatting.

Fantastic. Thanks!
On Wed, Mar 29, 2023 at 7:37 PM Tim Blais @.***> wrote:

Hi @mvictor55 https://github.com/mvictor55,
Thanks, I'll get that put together tonight with the standard formatting.
—
Reply to this email directly, view it on GitHub
https://github.com/codeforboston/maple/issues/1012#issuecomment-1489474233,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARANGSFFMDSLQX5RLTAYRLDW6TBSBANCNFSM6AAAAAAWIQ6YOI
.
You are receiving this because you were mentioned.Message ID:
@.***>


CAS-319: Prevent user from deleting strategy used by active proposal

Ticket: CAS-319
Description

Adds check to ensure that strategy to be deleted is not in use by a pending/active proposal
Adds /communities/:communityId/strategies endpoint that returns unique strategies on any pending/active proposals
Error message show to user when attempting to delete a strategy that is in use.

@jbluks just need to clean up the merge conflict here after refactor merge, the I think it's good to go