Timestamp string conversion is not deterministic

Description
Critical areas of the code like https://github.com/hyperledger/fabric/blob/c24c9b57439cfb1a9c2332d953286946ebfc3600/orderer/common/cluster/util.go#L812-L820
are using timestamp's String() method in order to calculate it's string representation.
The string method uses:
https://github.com/hyperledger/fabric/blob/c24c9b57439cfb1a9c2332d953286946ebfc3600/vendor/google.golang.org/protobuf/types/known/timestamppb/timestamp.pb.go#L271-L273
https://github.com/hyperledger/fabric/blob/c24c9b57439cfb1a9c2332d953286946ebfc3600/vendor/google.golang.org/protobuf/internal/impl/api_export.go#L173-L177
https://github.com/hyperledger/fabric/blob/c24c9b57439cfb1a9c2332d953286946ebfc3600/vendor/google.golang.org/protobuf/encoding/prototext/encode.go#L85-L98
Which clearly states "Do not depend on the output being stable. It may change over time across different versions of the program.".
Evidence to that is the way the timestamp seconds:1693225332; nanos:471402000 is converted to string:

seconds:1693225332  nanos:471402000
seconds:1693225332 nanos:471402000
note the double vs single spacing.

This string is then converted to bytes array and it's hash is used to calculated different security components.
Steps to reproduce
No response

@semil i like to work on this issues

@semil can i work on this issues . i have more reliable approach is to serialize the timestame. we can use  ISO 8601 Format for formating the timestamp

I think we just forgot to close this issue when the fix https://github.com/hyperledger/fabric/pull/4407 was merged. Should we just close this issue @semil or do you want to consider different approaches?


typo in a2?

https://github.com/umd-cmsc701/s2024/blob/754cac0184c9d3aa6c924358f40686bc4f134e34/_assignments/bloom_filter.md?plain=1#L40
I presume this line is copypasted from a1, just bitvector right?
also, might wanna note if we should floor or ceil the m and k values when printing in part 1. could check ofc when submitting to gradescope but would be nice to have in the readme :)

Good points; I've updated the spec on the website.


fix(safari): re-auth after account access is removed

If you remove account access via http://getpocket.com/connected_applications.php, the extension won’t save and throws an error. Instead, it should attempt to re-auth. https://www.dropbox.com/s/bj7lwepy1wju6et/Screenshot 2019-09-20 17.01.59.png?dl=0
The only way I see to clear this error is to log out of Pocket and log in again.

@jrochell Do you have any insight into why we add our extension to this list in the first place?  This makes sense for 3rd party apps that you would want to deny access to your Pocket account.  Since this is a first party app, it seems odd that it is included.

That's a good question. I can't say I know definitively why we include 1st party apps and extensions, but I think it's related to account security and something that has been helpful for users within support interactions.
For example, I seem to recall one instance where someone gifted an old device than happened to still be logged into Pocket, and by removing access to the app, they could make sure that the new owner of the device could not access their Pocket account.
For extensions, it's a bit less clear why you'd need to remove access like that, but I can imagine that it might be helpful to log out of an extension remotely if you didn't want to be logged in on that computer anymore.
If you want more technical details about why we want 1st party apps there, I think Matt or maybe Brian would be good people to ask.


Upgrade JavaFX

ToDo
JavaFX 18 and a new version of DRM-library have been released.
There are two remaining issues

mouse pointer is not visible in DRM
audio is not supported in JavaFX 17 build 17

In close collaboration with Gluon: upgrade to proper JavaFX (maybe JavaFX 18 ea)
Done

[ ] JavaFX 18 is used and all examples are running
[x] dropSound and background music of HelloFX are working
[ ] mouse pointer is visible in HelloFX in DRM and X11-Mode
[ ] mouse pointer can be disabled in DRM

Seems the current JavaFX version is 20.0.2. Can this ticket be closed?

Yes, can be closed.
The main reason is that Gluon stopped working on DRM, though.
Maybe we can open another issue for "remove all references to DRM"


Implement BSD sendmsg()

Create implementation and unit tests for BSD socket sendmsg() API.
Also as an example, add SO_TXTIME socket option that uses the sendmsg() ancillary data to pass data from user space to network device driver.

Fixed merge conflicts.


Displaying invalid email incorrectly

Describe the bug
Displaying invalid email on entering a valid email address.

Would you like to work on the issue?

open for all

Can I work on this?

@liveHarshit this is happening when using https://api.eventyay.com/v1/ and not on https://open-event-api-dev.herokuapp.com/v1/ I believe 1st api isn't fully working. Correct me if I am wrong. Moreover, it's a server side bug because server must be returning HTTP 422
404 Not Found
404 Not Found

@iamareebjamal Is there any issue with the server?

@CosmicCoder96 Can you please check?


Feature: Recall last Export folder

When exporting a file to a location, have it capture the folder and file name last used.
Next time, rather than defaulting to the folder the file is located in, default it to the last folder and file name you used. If the folder no longer exists, default to the original file location folder.
This is handy since I tend to be exporting multiple results and it is painful to have to navigate back to the intended output folder each time.

I have looked at this, but I don't think the vscode api supports this, but I'll investigate more. I could maybe make the default location configurable

I can save the last export folder within one "session". A session means the duration of an opened editor. When we close the editor, we'll lose that information.

One session would still be a big upgrade

I'll prioritize this.

I can save the last export folder within one "session". A session means the duration of an opened editor. When we close the editor, we'll lose that information.

Is it possible to save the last export folder to a variable that is reloaded next time you open VSCode? The post mentions state mementos:
https://stackoverflow.com/questions/51821924/how-to-persist-information-for-a-vscode-extension

Thank you, that can help. I can't help but wonder how to store the state for each opened parquet file?
I'll store it as default location for now. But we can also use vscode configuration API for that.

I suppose another useful one would be for it to remember your last selected Page Size (as long as the array hasn't changed where your last selection is no longer available in the dropdown)

I've merged the recall in memory for now. Can you create a new issue where you describe above? where it persists between vscode reloads.

I've merged the recall in memory for now. Can you create a new issue where you describe above? where it persists between vscode reloads?

Sure thing -- done! It looks like your latest change works where it remembers the path in current session, but I don't think the Open folder feature got added to this one.

Correct, the PR is still open and under development :)


Unable to install, no available version compatible with VS Code '1.22.2`

Hi there, @budparr. Trying something crazy by switching to VSCode after years of ST3, but I'm getting the following error when trying to install:
Unable to install 'budparr.language-hugo-vscode'; there is no available version compatible with VS Code '1.22.2'.

Thanks!

Hey @rdwatters Have you tried installing it with an updated version of VSCode? It's at 1.25.1 right now, and that's the version I made it with, so perhaps it assumes that's the min.

You're absolutely right. Closing issue. And thanks for making this awesome plugin. Looking forward to using it on my next project!


Update mapnik version, upgrade to node 8

Coverage remained the same at 82.564% when pulling 29e42545113a503e469f6a18739923c57db19f33 on upgrade-mapnik-version into dae723bbc5136b5ab849f459f0421728ea19aa35 on master.

nvm


RichTextFX highlight two languages in one CodeArea?

Hello everyone, I'd like to know how to highlight PHP and HTML in the same CodeArea. I've tried a lot with if (newValue.startsWith("") and so on. How can I do that? I've tried a lot...

This is something related to how you parse the source code. RichTextFX does not tell you how to parse your source code. The flip side is that it does not provide you with any support for parsing. All it cares about are the style ranges computed from the result of your parser. The demos provided with RichTextFX use regular expressions to parse Java tokens. For more sophisticated parsing, you might want to use something else, like ANTLR or Parboiled, just to name a couple. Or you can keep hacking with regular expressions, but their power is limited.
So I suggest:

Choose your parsing method.
Find out how to parse HTML + PHP using that method. (RichTextFX might not be the best place to get help on that.)
We can then help you with how to use the parse result to highlight text in RichTextFX.


Run Test Bug in jQuery courses

so i was following the courses about jQuery with title "Target the same element with multiple jQuery Selectors" link but there is a bug preventing me from running the test and passing to the next course
when you click on run test or ctrl + enter the task bar fade out and i got stuck there
so i decided to let you now about this bug i tried in google chrome and firefox

@ZiedChekir thanks for the issue. Can you post your code you have tried for this challenge? It'll help diagnose your problem as a bug with the site or an issue with your code.
Note: in the future, if you get stuck with a challenge, please first ask questions to the Help Room.

sure :)

jQuery Playground

#left-well

#target1
#target2
#target3

#right-well

#target4
#target5
#target6

yea sure but how can i post my code because github reads the code and prints the result of the code

$("button").addclass("animated "); -> extra space after animated
$(".btn").addclass("shale");            -> shake spelled incorrectly
addclass()                                      -> should be addClass()

@ZiedChekir as @HassHz has pointed out, it looks like you have a typo. If that fixes your issue, please close this issue. If not, please report back your correct code and any screenshots and errors you got.

ah sorry, it was a test i wanted to see how the run test behaves when i have a problem. besides when it   s correct (with shake*) the run test button doesnt do anything ...

في الجمعة، ١ يوليو، ٢٠١٦, ZiedChekir<EMAIL_ADDRESS>كتب:

ah sorry, it was a test i wanted to see how the run test behaves when i
have a problem. besides when it s correct (with shake*) the run test button
doesnt do anything ...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/9470#issuecomment-229826556,
or mute the thread
https://github.com/notifications/unsubscribe/AP1TBUpPdWCUNAYrwt_Fz6Ss-9GMzb8nks5qRGJggaJpZM4JCYvI
.

--
null

@ZiedChekir the Run Test button should create an animation to show that the tests were run and if you don't get a modal pop up, that means your code is incorrect.
So does this mean there's no bug? I'm confused on your issue now.

@ZiedChekir Did you change addclass to addClass? That's what was causing the expected test results at the bottom of the Run Tests button to disappear for me.

I'm having the same bug with the code, I've checked multiple times if i had a typo but it seems alright, i can't keep going with the lessons
This is the code if there's anything wrong please point it out
$(document).ready(function(){
$("button").addClass(animated)
$(".btn").addClass(shake)
$("#target1").addClass(btn-primary)
});

@JetroLopez if you get stuck with a challenge, please first ask questions to the Help Room. Hint: check for semi-colons and correct use of the .addClass() method; are you passing in a correctly formatted class name? Happy coding!


[Map] DM-swJumpPad

Hash: 3fd06e303d28759944c81227b062f7a7802031ea
Current name: DM-swJumpPad
This is a Test/Demo map for the swJumpPad and not a DM map.

Fixed


Unble to edit Attribute name

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

Steps to Reproduce (for bugs)

Create Project
Make label that has couple attributes
Annotate
Edit attribute on project page

Expected Behaviour
It should be possible to change attribute name
Current Behaviour
Unable to change attirubte name after i created
Possible Solution
can add new attribute however, it's not possible to delete attribute either
Context
when doing project, customers want to change attribute name on the middle of or at the end of the project.
it's pain to annotate all again just for the change of attribute name
Is there any reason why cvat disabled attribute edit box?
thanks so much for your help!

Hi, there is no such option yet. As a workaround, you can export a backup, rename the attribute in the downloaded files, then recreate the project from the backup.

hi , i would like to contribute please assign me .

So currently we don't have the endpoint for deleting an attribute?

Hi, I think, currently the attribute changes require all the attributes to be sent in the request. You can find tests here. Skeleton labels can also only be sent as a full list, specific sublabels cannot be changed separately. There are 2 ways to update labels now, one it changing a specific label via /api/labels, and another one is by sending the new list of labels in /api/{projects|tasks}/<id> endpoints. The implementation for these endpoints is located here.

Yes, currently all the attributes are sent in the request. Currently updating an attribute name is creating a new one and not updating the previous one. That might have something to do with the serialization. Thanks for providing the reference.

@ritikraj26 ,

the attribute name is being used as the primary key

Could you please explain why you think so? Please post a link on the code in the GitHub repo.

@nmanovic ,
Let's say an attribute's name is updated in the front-end and the updated list of attributes is being passed in the request. Now to locate the same attribute in the database we will require the previous name as it is the primary key. However, we will receive the updated name which is not present in the database, and hence a new attribute will be created instead of updating the previous one.
Please find the reference here

@nmanovic
Raised the PR here


fix: handle topline in the viewport-changed event

Fixes #2194
Problem:
Our highlight manager needs the correct offset of the current viewport. When scrolling with the mouse during incsearch, the offset sometimes becomes outdated. This happens because the viewport-changed event isn't triggered, and the 'win_viewport' event occurs after the grid-line event. As a result, this.lineCells becomes incorrect. This is problematic because we rely on accurate this.lineCells data when computing line highlights.
https://github.com/vscode-neovim/vscode-neovim/blob/160d7aa03cd72c59ab1a1cfa6e40a591298a549a/src/highlights/grid_line_handler.ts#L87
https://github.com/vscode-neovim/vscode-neovim/blob/160d7aa03cd72c59ab1a1cfa6e40a591298a549a/src/highlight_manager.ts#L105
Solution:
Trigger viewport-changed to get a up-to-date viewport

Have you actually debugged the win_viewport event? In my memory, the topline has always been accurate.

@xiyaowong Yes, I realized it's more of a timing issue. When topline is updated in 'on_win' event, the 'grid_line' handler is still using the old value to draw lines. Our 'win_viewport' handler is called after 'grid_line' with the correct topline value. I'm not sure if this is a good fix, but it seems work for me.

@gjf7 Please use the following debug statement to check the output.
diff --git a/src/highlight_manager.ts b/src/highlight_manager.ts
index e2eff48..7c744c2 100644
--- a/src/highlight_manager.ts
+++ b/src/highlight_manager.ts
@@ -103,6 +103,7 @@ export class HighlightManager implements Disposable {

     private handleGridLine(gridId: number, row: number, col: number, cells: VimCell[]): void {
         const gridOffset = this.main.viewportManager.getGridOffset(gridId);
+        console.log(`gridOffset: ${gridId} => ${gridOffset.line}`);
         const drawLine = gridOffset.line + row;
         // Offset for the statuscolumn
         const startCol = col + gridOffset.character;
diff --git a/src/viewport_manager.ts b/src/viewport_manager.ts
index 2e8437c..cce1326 100644
--- a/src/viewport_manager.ts
+++ b/src/viewport_manager.ts
@@ -116,6 +116,7 @@ export class ViewportManager implements Disposable {
         switch (name) {
             case "win_viewport": {
                 for (const [grid, , topline, botline, curline, curcol] of args) {
+                    console.log(`win_viewport: ${grid} => ${topline}`);
                     const view = this.getViewport(grid);
                     const { line, col } = view;
                     view.topline = topline;

I also print topline from on_win.
diff --git a/runtime/lua/vscode/viewport.lua b/runtime/lua/vscode/viewport.lua
index a8693cb..1ec0989 100644
--- a/runtime/lua/vscode/viewport.lua
+++ b/runtime/lua/vscode/viewport.lua
@@ -1,6 +1,7 @@
 local M = {}

 local api, fn = vim.api, vim.fn
+local vscode = require("vscode.api")

 M.event_group = api.nvim_create_augroup("vscode.viewport", { clear = true })
 M.viewport_changed_ns = api.nvim_create_namespace("vscode.viewport.changed")
@@ -36,22 +37,26 @@ local function setup_viewport_changed()
       view.botline = botline
       view.lnum = view.lnum - 1

+      local data = { value = "topline from on_win: " .. topline  }
+      fn.VSCodeExtensionNotify("print", data)
+
       local cache = view_cache[view.winid]
       if cache and vim.deep_equal(view, cache) then
         return
       end
       view_cache[view.winid] = view

+
       --#region XXX: Temporary fix for #2165
       -- Avoid unnecessary notifications
       -- For highlighting #1976 #2194
       local leftcol_changed = cache and cache.leftcol ~= view.leftcol
-      local topline_changed = cache and cache.topline ~= view.topline
+      -- local topline_changed = cache and cache.topline ~= view.topline
       -- For cursor position #1971
       local cursor_changed = cache
         and (cache.lnum ~= view.lnum or cache.col ~= cache.col)
         and api.nvim_get_mode().mode == "c"
-      if not leftcol_changed and not cursor_changed and not topline_changed then
+      if not leftcol_changed and not cursor_changed then
         return
       end
       --#endregion
diff --git a/src/highlight_manager.ts b/src/highlight_manager.ts
index e2eff48..5d0148d 100644
--- a/src/highlight_manager.ts
+++ b/src/highlight_manager.ts
@@ -25,9 +25,16 @@ export class HighlightManager implements Disposable {
             this.groupStore,
             eventBus.on("redraw", this.handleRedraw, this),
             eventBus.on("flush-redraw", this.handleRedrawFlush, this),
+            // @ts-ignore
+            eventBus.on("print", this.print, this),
         );
     }

+    // @ts-ignore
+    private print(data: Record<any, any>) {
+        console.log(data[0].value);
+    }
+
     // Get or create a HighlightGrid for the given gridId
     private getGrid(gridId: number): HighlightGrid {
         if (!this.highlightGrids.has(gridId)) {
@@ -103,6 +110,7 @@ export class HighlightManager implements Disposable {

     private handleGridLine(gridId: number, row: number, col: number, cells: VimCell[]): void {
         const gridOffset = this.main.viewportManager.getGridOffset(gridId);
+        console.log(`gridOffset: ${gridId} => ${gridOffset.line}`);
         const drawLine = gridOffset.line + row;
         // Offset for the statuscolumn
         const startCol = col + gridOffset.character;
diff --git a/src/viewport_manager.ts b/src/viewport_manager.ts
index 2e8437c..db007f0 100644
--- a/src/viewport_manager.ts
+++ b/src/viewport_manager.ts
@@ -119,6 +119,7 @@ export class ViewportManager implements Disposable {
                     const view = this.getViewport(grid);
                     const { line, col } = view;
                     view.topline = topline;
+                    console.log(`win_viewport: ${grid} => ${topline}`);
                     view.botline = botline;
                     view.line = curline;
                     view.col = curcol;

open a file with this content.

Copyright (c) <year> <owner>.

Redistribution and use in source and binary forms, with or without modification, are permitted provided that the following conditions are met:

1. Redistributions of source code must retain the above copyright notice, this list of conditions and the following disclaimer.

2. Redistributions in binary form must reproduce the above copyright notice, this list of conditions and the following disclaimer in the documentation and/or other materials provided with the distribution.

3. Neither the name of the copyright holder nor the names of its contributors may be used to endorse or promote products derived from this software without specific prior written permission.

 IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.

search in and scroll using mouse, until incorrect highlights are showing up

You can see the offset still old value 0, and topline from on_win is 1. win_viewport is triggered after grid_line event.

on_win is executed before redraw, so it definitely has the latest data. The problem now is why the win_viewport data is incorrect, and also, I can't reproduce this issue.
Please print and check the sequence of nvim events here. There might be a bug in our eventBus. 🤔
https://github.com/vscode-neovim/vscode-neovim/blob/160d7aa03cd72c59ab1a1cfa6e40a591298a549a/src/main_controller.ts#L302

I checked the event order. Neovim sends 'win-viewport' after 'grid-line'. If we only use 'topline' from 'win_viewport' ('viewport_changed' doesn't fire due to filtering condition) This causes the wrong 'topline' value.
As a temporary fix, I've tested it on my Mac, and it works.


Disable TestSieveNoDecision until its stability can be improved

Description
Skips the unstable TestSieveNoDecision test.
Motivation and Context
This is breaking CI, and Sieve is currently experimental, with known stability problems which are not currently being investigated.
How Has This Been Tested?
No functional code changes, so no tests needed.
Checklist:

[X] I have added a Signed-off-by
[X] Either no new documentation is required by this change, OR I added new documentation
[X] Either no new tests are required by this change, OR I added new tests
[X] I have run goimports, go vet, and golint. I have cleaned up all valid errors and warnings in code I have added or modified. These tools may generate false positives.  The goal is clean, consistent, and readable code.

Signed-off-by: Jason Yellick<EMAIL_ADDRESS>

LGTM

Since Sieve is a prototype, we will put all its tests in a different bucket.


Reloading of proxy seems to result in HAProxy zombies over time

roel      9979  0.0  0.0  14196  1016 pts/0    S+   20:51   0:00 grep --color=auto haproxy
root     20846  0.0  0.0      0     0 ?        Zs   20:38   0:00 [haproxy] <defunct>
root     20972  0.0  0.0  14164   808 ?        Ss   20:38   0:00 haproxy -f /cfg/haproxy.cfg -D -p /var/run/haproxy.pid -sf 11```

I experienced this problem as well. Both on docker for mac in swarm mode and on an ubuntu cluster.  A while back this was a bug in consul-template as well. It originates from the usage of the -sf instead of -st option from haproxy and reaping processes in docker. I got it working with hashicorps reap lib. Here is some more detail about reaping https://github.com/hashicorp/consul-template/issues/428

tini might be worth a look too: https://github.com/krallin/tini
Been using it for quite some time.

I did not manage to add tini to the project. I gave up to fast so that does not mean that it is not doable. @morph027 would you like to give it a try with a PR?

Sure, just need some time to reproduce the issue and then see if it's solved afterwards :+1:

The issue is fixed and available in the release 17.09.14-15.
Thanks @carlhals for the PR #328.


Fix ObjectUnsubscribedError

Prevent calling unsubscribe function on RxJs Subject which causes ObjectUnsubscribedError

Great idea to catch this case!
Is this possible without importing rxjs as a dependency? It would be awesome to avoid increasing the bundle size for downstream consumers who do not use rxjs...

@marcelriegr, can you give me an example for this use case, please?

See https://stackblitz.com/edit/angular-route-navigate-acifsw
The problem is that sometimes (and this is intermittent!) the componentDestroyed.next() in About component ngOnDestroy is invoked first and before ngx-auto-unsubscribe starts unsubscribing therefore when it unsubscribes from Subject there will be an error ObjectUnsubscribedError.


Reading and writing FileGDB (filegdb driver) with Fiona (Geopandas) not working

Hello,
I am trying to use geopandas 0.4.0. to read and write with FileGDB. I already reported in the geopandas Github that I can't get it working, but no solution yet. I tried a lot of things already and searched a lot of blogs to find a solution but nothing helped yet. I see there are some people who get it working, so suppose it should be possible.  Because Fiona library is used to read and write and errors are fiona specific I wanted to try my luck here.
Steps to reproduce the problem.
What I currently did is running these codes in the pyhon window of Arcgis to find out what is going wrong:
`

c = fiona.open(r'C:\Users\broekaar\Documents.........gdb')
c.driver
u'ESRI Shapefile'
fiona.supported_drivers
{'PCIDSK': 'r', 'OpenFileGDB': 'r', 'SUA': 'r', 'GML': 'raw', 'ARCGEN': 'r', 'GeoJSON': 'rw', 'GPKG': 'rw', 'S57': 'r', 'Idrisi': 'r', 'GPX': 'raw', 'ESRI Shapefile': 'raw', 'SEGY': 'r', 'BNA': 'raw', 'CSV': 'raw', 'FileGDB': 'raw', 'AeronavFAA': 'r', 'GPSTrackMaker': 'raw', 'DGN': 'raw', 'MapInfo File': 'raw', 'DXF': 'raw'}
fiona.open(r'C:\Users\broekaar\Documents.............gdb', driver='FileGDB')
Runtime error
Traceback (most recent call last):
File "", line 1, in
File "C:\Python27\ArcGIS10.6\lib\site-packages\fiona\env.py", line 405, in wrapper
return f(*args, **kwargs)
File "C:\Python27\ArcGIS10.6\lib\site-packages\fiona_init_.py", line 257, in open
layer=layer, enabled_drivers=enabled_drivers, **kwargs)
File "C:\Python27\ArcGIS10.6\lib\site-packages\fiona\collection.py", line 162, in init
self.session.start(self, **kwargs)
File "fiona\ogrext.pyx", line 467, in fiona.ogrext.Session.start
File "fiona_shim.pyx", line 73, in fiona._shim.gdal_open_vector
DriverError: C:\Users\broekaar\Documents...............gdb: Permission denied
`

Operating system
Windows 10.2018.3
64-bit Opereating system, x64-based processor
Python 2.7 32-bit
Fiona and GDAL version and provenance
-the Fiona-1.8.3-cp27-cp27m-win32.whl installed from Gohlke using pip version 18.1.
-the GDAL-2.2.4-cp27-cp27m-win32 installed from Gohlke using pip version 18.1.
-geopandas 0.4.0
-Visual studios Community 2017
-the FileGDB_API_1_5_1-VS2017 downloaded. And FileGDBAPI.dll added to C:\Python27\ArcGIS10.6\Lib\site-packages\osgeo

created GDAL_Driver_path for: C:\Python27\ArcGIS10.6\Lib\site-packages\osgeo\gdalplugins

If more info is need please let me know

@Rbroekie as far as I can see, you've followed the complicated directions for setting up the plugin correctly. You're fortunate that Gohlke includes osgeo/gdalplugins/ogr_FileGDB.dll in the wheel because it is tricky to build plugins.
When I saw "permission denied" in the traceback (thanks for including it!) I did some googling and I think advice at https://community.esri.com/thread/17026 may apply. I'm going to close this issue as OGR plugins are out of scope for Fiona and because the project doesn't have any special solutions for them.

Thanks @sgillies for the quick response. I know I followed the complicated direction but because I found some documentation on that I hoped to get it working. I also tried the 'easy' way via anaconda but got stuck in all kind of compiling issues too. In general it is a nightmare to get the FileGDB driver working.
Considering your link to the Esri thread, I had a look at it but as far as I can see it is not really a general permission issue, at least it looks like I have all the permissions. And I can open the gdb with fiona (Esri_shapefile driver, which seems weird to me) but also the OpenFileGDB driver. Maybe a FileGDB driver permission issue? Well, I have no idea what the issue exactly is but because the OGR plugins are out of Fiona scope, I will try to see if I can find the solution elsewhere.


Fix [sic] in docs

This was causing a link breakage warning in the docs. Thanks!

tbh this should probably link to https://en.wikipedia.org/wiki/Sic because not everyone will know what it means even though it is a good use of it.


After save image field with save_as => array, it's not possible to change image

Issue description:
After save image field with save_as => array, it's not possible to change image
Version used:
3.0.25 and dev branch
Using theme_mods or options?
theme_mods
Code to reproduce the issue (config + field(s))
		 Kirki::add_field('image_setting_array', array( 			'type'        => 'image', 			'settings'    => 'image_setting_array', 			'label'       => esc_attr__( 'Image Control (array)', 'kirki' ), 			'description' => esc_attr__( 'Description Here.', 'kirki' ), 			'section'     => 'colors', 			'default'     => '', 			'choices'     => array( 				'save_as' => 'array', 				), 			) 		);

I test it with Kirki 3.0.34, problem the problem still exists

Hello and thank you for reporting this issue.
We are closing most open GitHub issues with the release of Kirki 4 to start fresh with this major release.
If you are still running into issues after you have tested your theme with Kirki 4, please let us know so we can reopen the issue.
Please note that Kirki 3 is no longer actively supported. Moving forward, we will only provide support the very latest version of Kirki.
Thank you for your understanding.
Best,
David


Support very large test results

It looks like I used and reported this issue on a friendly fork of this project. Since the code is still almost identical, reporting here.
My project produces millions of test executions over a 2 hour build. I was hoping to use this action for a summary of the results, which would be a lightweight as just the counts. Unfortunately the action failed with an out of memory error on a 2gb heap. Can the results be processed in a streaming fashion rather than read fully into memory, which avoids having a limit?

In my case, I think the simplest is to preprocess the xml to remove the individual test cases
find . -type f -name '*.xml' -exec sh -c 'grep testsuite {} > {}.out && mv {}.out {}' \;


Adding HDFS 3 sink full example  [skip secret scan]

Adding HDFS 3 sink full example showing how to overcome no name for user issues derived from the libraries used with the connector.

Feel free to merge in when you are done


add all adapters

Include all currently supported adapters as part of the package dependencies.

@mstemm

According to https://github.com/juttle/juttle/issues/305, opentsdb is 0.1.2 and sqlite is 0.3.0.

The opentsdb dependency is fine as-is since the only change from 0.1.1 to 0.1.2 was how we run juttle in travis. For sqlite it looks like 0.3.1 contains a bugfix so there's no reason to not pull that one in as well.
Still we should be consistent -- @VladVega can you update the versions in juttle/juttle#305?

@demmer changed sqlite to 0.3.1 in the issue.

OK I'm merging this now.


issue 492: Create distribution that returns <: Union{Int,BigInt}

This draft PR is as per #494 except that I've sharpened the rand return value from Real to Union{Int,BigInt}

Codecov Report
Attention: Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 89.59%. Comparing base (21fb991) to head (986ef7e).
Report is 6 commits behind head on main.

Files with missing lines
Patch %
Lines

EpiAware/src/EpiAwareUtils/SafePoisson.jl
50.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #497      +/-   ##
==========================================
- Coverage   90.02%   89.59%   -0.44%
==========================================
  Files          57       52       -5
  Lines         742      740       -2
==========================================
- Hits          668      663       -5
- Misses         74       77       +3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

This is ready for review. Things to note:

I am interpreting the benchmarking positively. The most impacted by this change: ["EpiObsModels", "NegativeBinomialError", "gradient", "ADTypes.AutoReverseDiff(compile=true)", "linked"] has has a decent improvement.
This branch has not had #491 merged, but has also resolved #490

Agree this looks like it improves average performance which is great. Should we revert #490 here to show it fixes the problem and because that was only ever a hotfix?

This is great but its kind of mad we have to do it! As I said I think we should revert the test hotfix here before merging.

Lets do it.
Even more context: the test branch with Real can run all the prior predictive checks but fails various parameter recovery tests but this branch still needs restrained priors (e.g. no sampling for 1e67 infections).

I caught a typo and a non-use of the SafeInt alias

I see a test failure now (also noting we have a noisy sampling step again in our tests)

I see a test failure now (also noting we have a noisy sampling step again in our tests)

Yes just saw. Hmmm

Hmmm seems to be a Ubuntu problem. Not sure why.

Here right:
LatentDelay parameter recovery with mix of IGP + latent processes: Negative binomial errors + EpiProblem interface                                                                                |   29     1     30  40m09.0s
  Check true parameters are within 99% central post. prob.:                                                                                |   29     1     30  40m04.2s

(and is it really taking 40 minutes to run!)

If yes it is the section of code that had the hotfix: https://github.com/CDCgov/Rt-without-renewal/blob/dfe658a0260241768f1259958e814f0e54a6d264/EpiAware/test/EpiObsModels/modifiers/LatentDelay.jl#L177
So perhaps revert that and see?

Here right:
LatentDelay parameter recovery with mix of IGP + latent processes: Negative binomial errors + EpiProblem interface                                                                                |   29     1     30  40m09.0s
  Check true parameters are within 99% central post. prob.:                                                                                |   29     1     30  40m04.2s

(and is it really taking 40 minutes to run!)

Yeah its a matrix of full inference checks, its been about 40 mins since we committed it.

where are we with this? Did you try adding back in the test hot fix so we can merge this?

where are we with this? Did you try adding back in the test hot fix so we can merge this?

Yes in another branch but that failed. I'm investigating.

Plan is to check it again with the hotfix readded and explore that in its own PR

I think still waiting for the hotfix to go back in

I think still waiting for the hotfix to go back in

We are currently tweaking this on #503 .

🥳


Method Result Not Required To Be Tokenize

This PR fixes generating code for solidity functions that don't return anything.
Test Plan
dex-services

@fleupold It's so we can downgrade a MethodBuilder to a ViewMethodBuilder in a type-safe way. It is used in MethodBuilder::call and MethodBuilder::view and the type needs to be carried around by the MethodBuilder to be used those places.


nixos/pgbouncer: rework RFC42 integration

Commit bfb9d1825d545d96278db84d43e0b2a529775089 added RFC42 support which is a good thing in general, but this implementation has the following flaws:

services.pgbouncer.logFile was not renamed to [...].log_file, but to [...].logfile. Also the use of mkRenamedOptionModule is inappropriate here because the two options are not equivalent: the old option took a path relative to the home directory, the new an absolute path.

Using mkRenamedOptionModule with options that don't exist (but are keys in a freeform attr-set or an attrsOf X), you get the following error when referencing an option you didn't declare:
error: evaluation aborted with the following error message: 'Renaming error: option `services.pgbouncer.settings.pgbouncer.listen_port' does not exist.'
This error is pretty bad because it's not actionable for an end-user of the module. A possible use-case is doing
networking.firewall.allowedTCPPorts = [ config.services.pgbouncer.listenPort ];
without specifying a custom listen port. This is an example of why you want to keep options, they already contain defaults and you can re-use those defaults in other parts of your system configuration.
I decided to re-add a bunch of options where I figured that it's either useful to be able to address those in the NixOS configuration or having documentation directly in the options' reference in the NixOS manual.
I didn't add all options, I'll leave that to the maintainers of pgbouncer.

cc @SuperSandro2000  @1000101
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] 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 rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.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.

Add a :+1: reaction to pull requests you find important.

@SuperSandro2000 anything else to add from your side?
Otherwise I'd merge soonish.

Didn't catch anything immediately and didn't want to nitpick things. Fine for me if you tested it.


chore: added TcUnit package in config.json

see https://github.com/TcHaxx/rplc/issues/1
I added TcUnit, because the committed variant is UNITTEST

The build will fail in the PR, because GitHub doesn’t share secrets with PR by externals, see https://securitylab.github.com/research/github-actions-preventing-pwn-requests/ for details. It will only work after merging unfortunately


implement jinja templating to handle linux installer updates

experiment with using jinja to template linux spec files
I've started with alpine as it's the simplest to template

@gdams, oddly,  I've been looking at something similar today, just some bash scripting at the moment...

@gdams see : https://github.com/steelhead31/installer/tree/sfrautomate/linux/sfr.dev


DISCUSSION: Community in Sidebar?

Just throwing this out there for discussion, since I saw it on 11ty's doc site when I was running audits. What do we think of something like this?

Hi, sidebar expert here.
I love the idea of a community section, maybe we can even change the placement of some other stuff? We just have to be sure of what we would want to be linked in this section.
I have been working on this in a local branch, what do you guys think if the placement was something like this?

Looks fantastic @Yan-Thomas! I wonder if having More above Community would make sense as that keeps all the page-specific stuff together?

I wonder if having More above Community would make sense as that keeps all the page-specific stuff together?
Hi @delucis!

It would bug me that "More" is not the final section, like in a sentence, but maybe that's just me? Since "More" in my print only has "Edit/translate this page", maybe we should rename it to "Contribute"?

Yeah, in this scenario "Contribute" is fantastic here.


main: only print reports if there are entries

Without this change, two newlines are emitted on stderr, which offsets
the output of ct when run in a terminal.

needs rebase


Podman rmi output differs from Docker

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description
When removing images, Docker is much more explicit as to what is going on than Podman.
Podman only prints a bare ID. Docker prints Deleted: sha256:$ID for all image SHAs, and then extra Untagged: lines for any names that are untagged.
Steps to reproduce the issue:

docker rmi `docker images -q`

2.podman rmi -a
Describe the results you received:
Podman does not match Docker's output
Describe the results you expected:
Podman prints the additional information that Docker does
Additional information you deem important (e.g. issue happens only occasionally):
Output of podman version:
Current master built on F29

Noticed this while cooking up a fix for #2877

@mheon i think we are OK with what we have.  do you still want this changed?

Still no Untagged and Deleted notices - we're definitely still different than Docker, and showing a lot less information on what's going on.

Ok, lets hash this out.  I don't believe we need to provide more information as the output we are providing is exactly accurate.  I personally think the docker output for rmi is too verbose.  But for the sake of argument, we do differ in our output.  I attribute this to differences in our image management.  Looking at where we differ, I think it boils down to three basic areas:

We do prepend "Deleted" to the image ID being deleted.  That said, the command is "remove image", so if you run a command called remove image, it seems to me we are doing the right thing.

For removing images, Docker always does an untag and remove and reports it as a Untagged and Deleted.  If the case is that our image has only one repotag name, we do not report a tag because no tag actually occurs.

For removing images in podman, if an image has multiple images and the user requests to rmi one those repotag names, we respond that we untagged the image.

My vote is that we are doing enough here and because our image library differs from them, we are justified in doing as such.  Unless you are feeling strongly about this, I would like to close this issue and have a user report at some point why we need to make this kind of change?

The Docker 'Untagged: ...' syntax does make it clear that image IDs are not the only things being removed - names are also becoming inaccessible.
Example:
mcs@bellerophon code/libpod (fix_memory_leak) » sudo docker pull busybox
Using default tag: latest
Trying to pull repository docker.io/library/busybox ...
sha256:9f1003c480699be56815db0f8146ad2e22efea85129b5b5983d0e0fb52d9ab70: Pulling from docker.io/library/busybox
ee153a04d683: Pull complete
Digest: sha256:9f1003c480699be56815db0f8146ad2e22efea85129b5b5983d0e0fb52d9ab70
Status: Downloaded newer image for docker.io/busybox:latest
mcs@bellerophon code/libpod (fix_memory_leak) » sudo docker images --no-trunc | grep busybox
docker.io/busybox   latest              sha256:db8ee88ad75f6bdc74663f4992a185e2722fa29573abcc1a19186cc5ec09dceb   2 weeks ago         1.22 MB
mcs@bellerophon code/libpod (fix_memory_leak) » sudo docker rmi db8ee88ad75f6bdc74663f4992a185e2722fa29573abcc1a19186cc5ec09dceb
Untagged: docker.io/busybox:latest
Untagged: docker.io/busybox@sha256:9f1003c480699be56815db0f8146ad2e22efea85129b5b5983d0e0fb52d9ab70
Deleted: sha256:db8ee88ad75f6bdc74663f4992a185e2722fa29573abcc1a19186cc5ec09dceb
Deleted: sha256:0d315111b4847e8cd50514ca19657d1e8d827f4e128d172ce8b2f76a04f3faea

vs
mcs@bellerophon code/libpod (fix_memory_leak) » sudo podman pull busybox                                                                                1 ↵
Trying to pull docker.io/library/busybox...Getting image source signatures
Copying blob ee153a04d683 done
Copying config db8ee88ad7 done
Writing manifest to image destination
Storing signatures
db8ee88ad75f6bdc74663f4992a185e2722fa29573abcc1a19186cc5ec09dceb
mcs@bellerophon code/libpod (fix_memory_leak) » sudo podman images --no-trunc | grep busybox
docker.io/library/busybox       latest            sha256:db8ee88ad75f6bdc74663f4992a185e2722fa29573abcc1a19186cc5ec09dceb   2 weeks ago     1.44 MB
mcs@bellerophon code/libpod (fix_memory_leak) » sudo podman rmi db8ee88ad75f6bdc74663f4992a185e2722fa29573abcc1a19186cc5ec09dceb
db8ee88ad75f6bdc74663f4992a185e2722fa29573abcc1a19186cc5ec09dceb

Docker makes it clear that two layers (not just the top layer, as Podman reports) and two tags were removed. We don't do any of that right now.

at the libpod level, we don't know about the later (layers).  and again, im not sure the users care about layer levels when they have asked to delete an image.  what do you think

I usually lean towards matching Docker functionality.  But I will leave this to you guys to figure out.

this should now be doing what you wanted @mheon


df.subtract along year axis

I tested the .subtract() function passing axis='year' but I guess that was my misunderstanding and it is not supposed to work as the result needs to be assigned to a 'year'.
e.g.
variable = 'Emissions|CO2'
df.filter(variable=variable).subtract(a='2100', b='2020', name='Emissions|CO2 2100-2020', axis='year')

@gidden suggested to add at least an error warning advising that the operation along the 'year' axis is not supported.
Error message:
ValueError                                Traceback (most recent call last)
pandas\_libs\lib.pyx in pandas._libs.lib.maybe_convert_numeric()

ValueError: Unable to parse string "net change in Land Cover"

During handling of the above exception, another exception occurred:

ValueError                                Traceback (most recent call last)
~\AppData\Local\Temp/ipykernel_176/2230843147.py in <module>
      5 name = f'net change in Land Cover'# Affor-Refor ({lastyear}-{baseyear} {unit})'
      6 vari = 'Land Cover|Forest|Afforestation and Reforestation'
----> 7 dfar6.filter(variable=vari).subtract(a=2100, b=2020, name=name, axis='year', ignore_units=True)

c:\github\pyam\pyam\core.py in subtract(self, a, b, name, axis, fillna, ignore_units, append)
   1928             self.append(_value, inplace=True)
   1929         else:
-> 1930             return IamDataFrame(_value, meta=self.meta)
   1931
   1932     def multiply(

c:\github\pyam\pyam\core.py in __init__(self, data, meta, index, **kwargs)
    143                 setattr(self, attr, value)
    144         else:
--> 145             self._init(data, meta, index=index, **kwargs)
    146
    147     def _init(self, data, meta=None, index=DEFAULT_META_INDEX, **kwargs):

c:\github\pyam\pyam\core.py in _init(self, data, meta, index, **kwargs)
    159         # cast data from pandas
    160         if isinstance(data, pd.DataFrame) or isinstance(data, pd.Series):
--> 161             _data = format_data(data.copy(), index=index, **kwargs)
    162         # read data from ixmp Platform instance
    163         elif has_ix and isinstance(data, ixmp.TimeSeries):

c:\github\pyam\pyam\utils.py in format_data(df, index, **kwargs)
    336
    337     # format the time-column
--> 338     df = format_time_col(df, time_col)
    339
    340     # cast to pd.Series, check for duplicates

c:\github\pyam\pyam\utils.py in format_time_col(data, time_col)
    357     """Format time_col to int (year) or datetime"""
    358     if time_col == "year":
--> 359         data["year"] = to_int(pd.to_numeric(data["year"]))
    360     elif time_col == "time":
    361         data["time"] = pd.to_datetime(data["time"])

~\Anaconda3\envs\py38\lib\site-packages\pandas\core\tools\numeric.py in to_numeric(arg, errors, downcast)
    152         coerce_numeric = errors not in ("ignore", "raise")
    153         try:
--> 154             values = lib.maybe_convert_numeric(
    155                 values, set(), coerce_numeric=coerce_numeric
    156             )

pandas\_libs\lib.pyx in pandas._libs.lib.maybe_convert_numeric()

ValueError: Unable to parse string "net change in Land Cover" at position 0

Thanks for reporting this issue, @byersiiasa!
Played around for a few minutes, I think there are three issues:

The year-column has integer values, but you passed arguments a and b as strings
The name-argument for the computed data should be again an integer, but this indeed runs into a bug - it works in my test case, however, if you pass an integer-castable string, i.e., name="2006"
The larger question is whether an operation likes "value[2100] - value[2005]" makes sense to be reported as a time domain. The "meta" attribute sounds like a more reasonable usage...

A suggested alternative using the subtract method and then adding the result to meta:
df.set_meta(
    meta=df.filter(variable="Emissions|CO2").subtract(a=2100, b=2020, name="0", axis="year", append=False)._data,
    name="Emissions|CO2 2100-2020",
)

Indeed - at the moment the numerical operations assume explicitly that they will not be applied to the time axis (e.g., year). The result of such a calculation would have a time dimensionality of 0, and thus be considered metadata in our current data model.
I think it's an open question as to whether we would want to support this directly in the operation interface (e.g., if axis is year, then add to meta) or have a separate interface that supports that.
The implementation you provide @danielhuppmann I think is great for that, now it's just a question of what we want to support in a 'first class' manner.

For clarification, the numerical operations work fine on the "year" dimension, i.e.
df.subtract(a=1, b=2, name="3", axis="year")

except for the "bug" that the returned index-value on the time domain has to be specified as an integer-castable string instead of an integer.
However, in many cases, this will not really make sense as a "timeseries"-type data. (Usually, diff() might be more useful here.)
So imho, the actual question is: should we explicitly raise an error when doing data operations along the time domain?
I would not overload the data-ops methods with some optionality to add to meta instead of data - instead, the method set_meta_from_data would be a better prototype...


Compatybility with Embedded Videos

Now when i have aded videos from Youtube with Embedded Videos extension, in slideshow i can see only thumbnail, is possible to show code to play video from Youtube?

No, sorry. Video in PhotoSwipe is really tricky, and I won't extend it beyond playing server-local html5 compatible video files. Not at this time.

Ok, thanks for answer.


Improve Codespaces experience

I'd suggest turning on prebuilds, so that the Codespace will boot faster for new users.


Append useId to the filter id

Motivation
If a website contains multiple marble avatars and the first avatar is hidden, all other avatars' filters are also hidden. The issue is similar to the mask id https://github.com/boringdesigners/boring-avatars/issues/64 that has been fixed in https://github.com/boringdesigners/boring-avatars/pull/66.
Solution
Append useId to the filter id.

Hey @Robert-Steiner thanks for flagging this one. I just updated the Marble component here https://github.com/boringdesigners/boring-avatars/commit/51e724e59bc5934577552a9b591f187f5abbe56b, that should fix the issue using the latest 1.10.2 version 👋

Awesome! Thank you 🙏


1.0.2

@ArduinoBot

@ArduinoBot

@ArduinoBot

@RASPIAUDIO
❗❗❗❗❗❗❗❗❗❗❗❗❗❗❗❗❗❗❗❗
❗⚠⚠⚠⚠⚠⚠⚠⚠⚠⚠⚠⚠⚠⚠⚠⚠⚠⚠❗
❗⚠ PLEASE READ THIS IMPORTANT INFORMATION ⚠❗
❗⚠⚠⚠⚠⚠⚠⚠⚠⚠⚠⚠⚠⚠⚠⚠⚠⚠⚠❗
❗❗❗❗❗❗❗❗❗❗❗❗❗❗❗❗❗❗❗❗

@per1234
Thank you for your feedback. I’d like to clarify the sequence of events regarding our pull requests:

The first PR on November 29, 2024, was submitted to add our URL.
The second PR was accidental, as it unintentionally modified one character in the README.md file, which triggered a manual review. My apologies for this oversight.
The third PR was an attempt to correct the README.md to prevent rejection. If there’s a better approach I should have taken, I’d be grateful for your guidance.

I sincerely apologize for any inconvenience or frustration caused. As we are all learning and striving to contribute constructively, I assure you our intentions are genuine and in good faith.
That said, I believe maintaining professionalism in our communication is essential. Referring to someone as "unprofessional" or "incompetent" may not foster the constructive environment we all value in the open-source community.
Thank you for your time and consideration. I’ll make sure to be more thorough in the future.

@ArduinoBot

Hello @RASPIAUDIO. The automated system here only operates on open pull requests. Since the pull request is closed, the bot won't pay any more attention to it.
But this human is happy to provide assistance. Was there something you needed from the bot?

Hello @RASPIAUDIO. The automated system here only operates on open pull requests. Since the pull request is closed, the bot won't pay any more attention to it.
But this human is happy to provide assistance. Was there something you needed from the bot?

Thanks for your help @per1234 , version 1.0.2 of our library was published in the Arduino library manager but I need to make a version 1.0.3, I have pushed my modification to my referenced repository but the library mananger doesn't seem to pubish the changes after 3 days.
Am I missing something?
thanks
https://github.com/RASPIAUDIO/Muse_library

Hello @RASPIAUDIO. The automated system here only operates on open pull requests. Since the pull request is closed, the bot won't pay any more attention to it.
But this human is happy to provide assistance. Was there something you needed from the bot?

Thank you for your help, @per1234.
Our library was indeed accepted in PR #5492, but it seems that the latest version (1.0.3), committed to our library repository, is not being updated automatically.
I was under the impression that the crawler would automatically fetch new versions. Could you kindly clarify what might be causing this issue?

Hi @RASPIAUDIO. You can check on the indexing of a library by the automated system by looking at the dedicated logs web page for the library. The logs page for this library is here:
https://downloads.arduino.cc/libraries/logs/github.com/RASPIAUDIO/Muse_library/
2024/12/08 14:40:13 Checking out tag: v1.0.2
2024/12/08 14:40:13 Release :1.0.2 has wrong library name, should be Muse_library
2024/12/08 14:40:13 Checking out tag: v1.0.3
2024/12/08 14:40:14 Release :1.0.3 has wrong library name, should be Muse_library

The indexer rejects any release that does not meet the requirements. The 1.0.2 and 1.0.3 releases of the library do not meet the requirements, so they were rejected by the indexer.
At the time you registered the library, the metadata was valid:
https://github.com/RASPIAUDIO/Muse_library/blob/v1.0.0/library.properties#L1
name=Muse_library

However, a corruption was later introduced:
https://github.com/RASPIAUDIO/Muse_library/commit/e8c84d2f0ca0a9585101a17dc483570494da13fd
--- a/library.properties
+++ b/library.properties
@@ -1,4 +1,4 @@
-name=Muse_library
+sp32-sh1106-oledname=Muse_library
 version=1.0.1
 author=Raspiaudio
 maintainer=Raspiaudio<EMAIL_ADDRESS>
You changed the property name from name to sp32-sh1106-oledname. This effectively removed the definition of the library name from the metadata.
To solve this, you should do the following:

Change the name of the sp32-sh1106-oledname property in library.properties back to name.
Update the value of the version property to the number you want to use for the new release (e.g., 1.0.4)
Make a new release or tag in the library repository.

After that, you only need to wait for the indexer job to run, and then for the updated index to propagate to you and the library users. After a successful indexing, it does take a bit of time before you will see a new release in your Library Manager due to caching, but it will get there eventually. We give a conservative estimate of availability within one day's time.


Add support to force recreation of packs virtualenvs

This PR adds support for --register-setup-virtualenvs-force-recreation to st2ctl. Use-case is, like we have it now with the upgrade to st2 v3.4, an upgrade of the Python version inside the virtual environments for example.
I'll run some tests now but wanted to provide the PR already so that others who ran into this issue can give it a try, too.

Can we rename options to --setup-venvs or --setup-venvs-reset?  Does venvs need registering, is that why we have to prefix it with register?

The context is that a new flag is a nice to have addition that just mimicks the following construction, required for 3.4 upgrade only:
sudo rm -rf /opt/stackstorm/virtualenvs/*
sudo st2ctl reload --register-setup-virtualenvs

@m4dcoder I think at this point we're just adding a new flag instead of refactoring existing params which would be a breaking change and undesired as a last-minute release enhancement.

The context is that a new flag is a nice to have addition that just mimicks the following construction, required for 3.4 upgrade only:
sudo rm -rf /opt/stackstorm/virtualenvs/*
sudo st2ctl reload --register-setup-virtualenvs

@m4dcoder I think at this point refactoring existing params is undesired as a last-minute release enhancement and would be an overkill breaking change.

@armab we don't change any defaults so I don't see how this is could be a breaking change. Do I miss something?

@winem I think @armab was referring to @m4dcoder's request to rename the options:

Can we rename options to --setup-venvs (replaces --register-setup-virtualenvs) and --reset-venvs (replaces --register-setup-virtualenvs-force-recreation)? Does venvs need registering, is that why we have to prefix it with register?

Renaming the options would be a breaking change. Merely adding a new option as this PR does, is not a breaking change, IMO.

Yeah, I agree with @cognifloyd.
In the future, I maybe be OK with renaming it (if we can work around oslo.config limitations), but for this last minute change I'm for --register-recreate-virtualenvs since it's the shortest and consistent with the current naming scheme.

Try pinning mail-parser to >=3.9.1,<3.10.0 in contrib/core/requirements-tests.txt and see if that fixes it.
That package just had a release like 4 hours ago:
https://pypi.org/project/mail-parser/#history
And they might have broken something we were relying on, or fixed a bug that we were relying on.

Oh saw that you just pinned it. So let's see  if that fixes the issue.
@cognifloyd thanks for clarifying it. Now it makes sense.

Signed off my review comments, I've not had chance to try on an upgrade to check command works - but didn't want my comments to be a blocker for merging.

Travis needs payment. I'll be out for an hour and fix it after that.

@amanda11 your input was actually very helpful and is always appreciated. And I added an integration test. :)

Paid off Travis-CI, adding a commit to kick CI again.

:tada:


Login Failed.

Hi ,
I have configured my django cms using ADFS, nginx as web server to and gunicorn to run the app.
I am observing different behaviour after integrating with ADFS.

if a user is logged in and if he logs out and tries to login again , he gets login failed.
Sometimes a new user gets created but sometimes it fails to create new user , again with the same error login failed.
if a user is logged into one browser window and tries to login the cms console in another browser then again login fails.
And in above process , user permission also gets lost . for example if a user is having staff and superuser access , he just looses both of the access.

Now below is the configuration part :

entire installation is running on a container , and django cms is accessible over https .
nginx is used to redirect http traffic to https.
ADFS is installed using

pip install django-auth-adfs

Some part of settings.py

SECURE_PROXY_SSL_HEADER = ('HTTP_X_FORWARDED_PROTO', 'https')
SECURE_SSL_REDIRECT = True
SESSION_COOKIE_SECURE = True
CSRF_COOKIE_SECURE = True

AUTH_ADFS = {
    "SERVER": "xxxxxxxxxxxxxxxxx",
    "CLIENT_ID": "xxxxxxxxxxxxxxxxxx",
    "RELYING_PARTY_ID": "xxxxxxxxxxxxxxxxxxx",
    "AUDIENCE": "xxxxxxxxxxxxxxxxxxx",
    "CLAIM_MAPPING": {"first_name": "given_name",
                      "last_name": "family_name",
                      "email": "email"},
    "USERNAME_CLAIM": "winaccountname",
    "GROUP_CLAIM": "group",
    "BOOLEAN_CLAIM_MAPPING": {"is_staff": True,
                              "is_superuser": True},

}

INSTALLED_APPS = [
    'django_auth_adfs',
....
]
LOGIN_URL = "django_auth_adfs:login"
LOGIN_REDIRECT_URL = "/en/?edit"

AUTHENTICATION_BACKENDS = [
    'django_auth_adfs.backend.AdfsAuthCodeBackend',
]

nginx conf
upstream cms {
    server localhost:5000;
}

server {

    listen 443 ssl;
    if ($http_x_forwarded_proto != 'https') {
      return 301 https://$host$request_uri;
     }
    add_header Strict-Transport-Security "max-age=31536000; includeSubDomains" always;
    server_tokens off;
    ssl_certificate /etc/ssl/cms.crt;
    ssl_certificate_key /etc/ssl/cms.key;
    access_log /var/log/nginx-access.log;
    error_log /var/log/nginx-error.log;
    charset     utf-8;
    client_max_body_size 4096M;

    location /media  {
        alias /apps/static_media_files/media;
        client_max_body_size 4096M;
    }

    # your Django project's static files - amend as required
    location /static {
        alias /apps/static_media_files/static;
    }

    location / {
        proxy_pass http://cms;
        proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_set_header Host $host;
        proxy_redirect off;
        add_header Strict-Transport-Security "max-age=31536000; includeSubDomains; preload" always;
    }

}

Issue is sometimes user is able to login , but sometimes they get login failed error. even new user creation doesn't happen automatically and user gets login failed but sometimes it allows new user creation as well .
in logs I can see -
ADFS server returned an error: MSIS9612: The authorization code received in 'code' parameter is invalid.

Hi,
Please enable debug logging as described in the documentation and provide a full output of the stack trace and logs.

Hi,
Please enable debug logging as described in the documentation and provide a full output of the stack trace and logs.

@JonasKs I have added below code in settings.py
LOGGING = {
    'version': 1,
    'disable_existing_loggers': False,
    'handlers': {
        'file': {
            'level': 'DEBUG',
            'class': 'logging.FileHandler',
            'filename': '../tmp/debug.log',
        },
    },
    'loggers': {
        'django': {
            'handlers': ['file'],
            'level': 'DEBUG',
            'propagate': True,
        },
    },
}

attaching debug.log for your reference.
debug.log

@JonasKs I have added below code in settings.py
LOGGING = {
    'version': 1,
    'disable_existing_loggers': False,
    'handlers': {
        'file': {
            'level': 'DEBUG',
            'class': 'logging.FileHandler',
            'filename': '../tmp/debug.log',
        },
    },
    'loggers': {
        'django': {
            'handlers': ['file'],
            'level': 'DEBUG',
            'propagate': True,
        },
    },
}

attaching debug.log for your reference.
debug.log

Do you still need help with this @gauravdutt13? If so, feel free to reopen 👍


Refactoring To-do list

[ ] #206
[ ] #166
[ ] move PAD-US analysis inside repo instead of storing results as a flat file. The only reason it is arranged as such is that we didn't have a cloud data cache set up at the time and the input data was too large for git.
[ ] move Energy Communities analysis inside repo, if the dependencies can be reasonably resolved. I suspect this project has been abandoned by RMI, so I don't expect to lose access to future updates.
[ ] cache pudl MCOE calculation. It takes over half the total run time and never changes.
[ ] update db writes to use COPY instead of shockingly slow INSERT. In combination with the above, this would drop run time from ~4 minutes to tens of seconds. This is a qualitative step change from "run it and get a drink" to an interactive process. Interactivity allows for more experimentation and error checking due to the reduced costs of experiments.


🐛 [BUG] : When message is read, the chat scrolls up

To Reproduce
Steps to reproduce the behaviors.

Send message
Stay in chat
When message is read, the chat scrolls up

Expected behavior
has to be stay at bottom.
Screenshots
If applicable, add screenshots to help explain your problem.
Smartphone (please complete the following information):

Device: ios androis web

Contributor: @xbloodyknight from Discord
If you want speed up process of this issue, please send 🚀 as a reaction.

https://github.com/languageXchange/languageXchange/assets/3280475/2155782c-d49f-4111-a828-8f92fef40ce9

Solved w/ #657


KAFKA-16252: Fix the documentation and adjust the format

Jira ticket: https://issues.apache.org/jira/browse/KAFKA-16252
Background & Modification
Currently, there are few document files generated automatically like the task genConnectMetricsDocs
However, the unwanted log information also added into it.
And the format is not aligned with other which has Mbean located of the third column.
I modified the code logic so the format could follow other section in ops.html
Also close the log since we take everything from the std as a documentation
Testing Strategy
Regen the docs and compare the format
origin generated file
Top few lines from connect_metrics.html after executed the ./gradlew genConnectMetricsDocs
[2024-03-05 18:11:45,945] INFO Metrics scheduler closed (org.apache.kafka.common.metrics.Metrics:694)
[2024-03-05 18:11:45,948] INFO Metrics reporters closed (org.apache.kafka.common.metrics.Metrics:704)

kafka.connect:type=connect-worker-metrics

Attribute name
Description

	connector-count	The number of connectors run in this worker.

	connector-startup-attempts-total	The total number of connector startups that this worker has attempted.

	connector-startup-failure-percentage	The average percentage of this worker's connectors starts that failed.

	connector-startup-failure-total	The total number of connector starts that failed.

fixed generated file

Metric/Attribute name
Description
Mbean name

connector-count
The number of connectors run in this worker.
kafka.connect:type=connect-worker-metrics

connector-startup-attempts-total
The total number of connector startups that this worker has attempted.
kafka.connect:type=connect-worker-metrics

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Could you run the website with the change and paste the screenshot? Thanks.

Sure!
Previous Version

Now


Surprising Mechanical Pipe and TE's Portable Tank Integration

I just report it because it seems a bit weird to me, no idea if that's a bug or not.
The setup is a Thermal portable tank and a mechanical pipe.
When the tank is in output mode (push fluids) and pipe in normal mode, water flows to the pipe (at the tank's rate).
When the tank is input mode and the pipe is in extraction mode, water flows (at the pipe's rate).
But now, when the tank is in output mode and the pipe in extraction mode, nothing happens.
That's the weird thing, the mechanical pipe can only extract fluids from input mode TE tanks.

This is technically correct behavior, as extract mode does not accept any fluids from the connected source but instead attempts to manually pull it out.  Apparently TE's tanks have the exact same functionality, but swapped.


Portable window example error

When I run the portable window example and try to drag the window, it start to twitching and flay away from under cursor.
I would by happy to add a gif record, but I can't.

Manjaro Linux XFCE.

Manjaro Linux XFCE.

I'm afraid this issue could be related to XFCE desktop environment. There is not much I can do.

I noticed the same issue in Pop_OS! Gnome. So it it not only related to xfce.

I noticed the same issue with Debian 11, running with XFCE. I'll check at the code

@Lecrapouille If the problem is on mouse, why does is work as expected on Windows?

@raysan5 I should try on Windows, but I'm noob with Windows. I do not know why on Windows it works, it should not work. When I tried to make my 1st example, I got issue with scrolling the list widget (I forget the name) it was the single widget I add: the scroll was also erratic (while with your example full of widget the scroll worked).

Hi @raysan5 , I meet the same issue.
After some investigation, I found that when the window position is updated in raylib, the mouse position does not get updated accordingly.
I made this change and it works good on my computer with glfw:
static void WindowPosCallback(GLFWwindow* window, int x, int y)
{
    // Set current window position
    CORE.Window.position.x = x;
    CORE.Window.position.y = y;
    double mouseX, mouseY;
    glfwGetCursorPos(window, &mouseX, &mouseY);
    MouseCursorPosCallback(window, mouseX, mouseY);
}

Is this the intended behavior? Perhaps we need a mechanism to ensure the mouse position is updated.


The app should take the user back to the profile page when the user clicks the back button in the web browser

Details

When viewing the profile photo, the app should take the user back to the profile page when the user clicks the back button in the web browser
Fixed Issues

$ https://github.com/Expensify/App/issues/27856
PROPOSAL: https://github.com/Expensify/App/issues/27856#issuecomment-1727633670
Tests

Go to settings > profile
Click on avatar and add a photo if you don't have any
Click on the avatar again
Click on View photo
Click on the system/browser back button
Verify that the app takes the user back to the profile page

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

Go to settings > profile
Click on avatar and add a photo if you don't have any
Click on the avatar again
Click on View photo
Click on the system/browser back button
Verify that the app takes the user back to the profile page

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/129500732/91e9b3b0-f17b-4d64-913c-0ecf1f6d113d

Mobile Web - Chrome
https://github.com/Expensify/App/assets/129500732/20ee0799-8aa2-4a6b-bbba-c1c2451018cc

Mobile Web - Safari
https://github.com/Expensify/App/assets/129500732/f018e1b3-82b1-44d9-9e37-c1f26bc6e5a1

Desktop
https://github.com/Expensify/App/assets/129500732/152b39ac-dece-4ffc-a10a-baf21b15622f

iOS
https://github.com/Expensify/App/assets/129500732/e90a5db0-96c8-417c-90b8-f8f6c063a13d

Android
https://github.com/Expensify/App/assets/129500732/ecf5c95b-1f89-4e7c-b5f2-de62bbbc6cb5

Create a unique modal route for all image modals. This could apply to any new image modal coming in the future
Please let me know which options that you prefer

Can you elaborate on what these new routes would look like?

Yeah separate routes seems fine for now 👍

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to production by https://github.com/thienlnam in version: 1.4.31-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Sidebar won't be colored

Hi there!
I've followed the ReadMe Tutorial and installed the custom css snippet.
I then added those Prefixes in front of my Notes which sadly didn't change anything.

Im having the same issue any luck figuring out ?

Im having the same issue any luck figuring out ?

I'm not sure if both of you have figured it out yet, but just in case, if I'm looking at that image correctly, you're trying to color a note, these colors will only affect folders however.
Create a new folder and name it with the correct prefix to get the color you want.

My folders are not being coloured

No matter what i do can i get it to work !

Ah I think I see what your problem is.
What you did is inside your vault you created a folder called 'Vault' and then inside THAT folder is where you're trying to color the folders, that won't work, I just tested it.
The color needs to be applied to the TOP most folder, only then will it work.
Try making a new folder outside of that vault folder or renaming the vault folder itself to one of the colors (I repeat, the vault folder, NOT the vault itself.)
Then it'll be whichever color it was selected for it and all subsequent folders inside it.
As far as I know you can't color sub folders, only top ones, at least with this snippet.

Ah I think I see what your problem is. What you did is inside your vault you created a folder called 'Vault' and then inside THAT folder is where you're trying to color the folders, that won't work, I just tested it.
The color needs to be applied to the TOP most folder, only then will it work. Try making a new folder outside of that vault folder or renaming the vault folder itself to one of the colors (I repeat, the vault folder, NOT the vault itself.) Then it'll be whichever color it was selected for it and all subsequent folders inside it. As far as I know you can't color sub folders, only top ones, at least with this snippet.

I think it would be great to have sub folders colored as well, i take notes of things i work on in my current company and i don't want to make a new vault for every company.
my folder structure is as follows:
01 - companyname ltd.
    02 - overview
        01 - about
        02 - provided items
        03 - tasks
   03 - selfdocumentation
       2024
          2024-06
             01 - KW 23
             02 - KW 24
             03 - KW 25
             04 - KW 26

KW stands for Kalenderwoche (German) aka. Calender Week

Yeah, if you want different colors for different companies you'd need different folders, not different vaults necessarily, but for coloring subfolders, other than trying to find or create a snippet that does just that, you can only try to ask Cyan for that as a feature.
It'd be a neat thing to have indeed.

I had a crack at solving the OP's 3rd September request and made a PR. It works with 01, 02, etc. subfolders. I had a go at making it work with generically named subfolders as suggested by @Rex9988 , but didn't quite succeed.
As I was working with a 1. 2. 3. type folder structure previously for things I'd decided the order on, to be honest I was happy having the control that only folders starting with 01 to 07 got their own special colours... It was getting a bit rainbow'y in there and kinda distracting!
Hope this helps someone in any case!

Cheers for help  i dropped messing to time consuming