feat: add aspect ratio for ocr predictor

This PR adds the option to preserve aspect ratio of the pages in the whole OCRPredictor pipeline, which leads to much better results on the detection task.
Any feedback is welcome!

Yes @fg-mindee it doesn't hurt performances for now, and I will retrain all detection models preserving the aspect ratio to have a coherent pipeline, because it is way better not to deform writtings on documents for the model convergence!


next: generic lifetime

Removed cut/scene slots
Removed preemption deactivation

Checklist

[x] npm test passes
[x] tests and/or benchmarks are included
[ ] documentation is changed or added

Great works 👍


Add aliases for js files

1⃣ Is this something that is wanted/needed? Did you create a feature-request issue first?
This PR is related to #63
Before
Before this PR, you need to import your files like this:
import Message from '../message'
import { debounce, walk } from '../util'
import morphdom from '../dom/morphdom'
import DOM from '../dom/dom'
import DOMElement from '../dom/dom_element'
import nodeInitializer from "../node_initializer";
import store from '../store'

After
import Message from '@message'
import { debounce, walk } from '@util'
import morphdom from '@dom/morphdom'
import DOM from '@dom/dom'
import DOMElement from '@dom/dom_element'
import nodeInitializer from "@node_initializer";
import store from '@store'

How?
The module aliases are defined within the webpack.min.js:
mix.extend('aliasConfig', new class {
    webpackConfig(webpackConfig) {
        webpackConfig.resolve.extensions.push('.js', '.json', '.vue');
        webpackConfig.resolve.alias = {
            '@': __dirname + '/src/js',
            '@action': __dirname + '/src/js/action',
            '@component': __dirname + '/src/js/Component',
            '@connection': __dirname + '/src/js/connection',
            '@dom': __dirname + '/src/js/dom',
            '@drivers': __dirname + '/src/js/connection/drivers',
            '@message': __dirname + '/src/js/Message',
            '@morphdom': __dirname + '/src/js/dom/morphdom',
            '@node_initializer': __dirname + '/src/js/node_initializer',
            '@store': __dirname + '/src/js/Store',
            '@util': __dirname + '/src/js/util',
        };
    }
});

Thanks for the contribution @mateusjunges!
I'm open to adding an alias for the root: @, but I don't think I want to use aliases for all the directories - too much to maintain and too rigid.
If you'd like to scope the PR down to fixing the bad imports and adding a root alias, I'll merge it - thanks!

Yes, i can do it. I'll close it and open a new PR, think it's better than lose time resolving merge conficts.

@calebporzio check #107


Undefined array key 1 {"file":"Message.php","line":174}

Hello
It looks it fails on headers like this one:
ARC-Seal: i=1; a=rsa-sha256; t=1731423677; cv=none;
d=google.com; s=arc-20240605;
b=HWyeknBTKrwZQ4qfvrr/EOGaEDVxwAyAT9bNP2FRlQOXW/pkf8cy0wsRwswG0LzJWX
pGPOZfBLUGr2XFwuvg5/k9uxamiYe5hIHGtcSrUpq6gatYyBBqMh2eLikBf1mWLK1EPU
7YbBYQ7Gxcigx6hEZxnEcpXTvhEfNskuF81UxoMfTfoqWRMXKaAb8r5SNOs50E5G4ZIQ
lUqqxjvfaJS/C4UdnxOzlSs02DTaFiruj7ZPxnPe35G6JoeT7KW+Z1VofLf3ZBFoY2f9
cIOZoYSzKuquOnqChKRDyW+29iD2QyO6htwn7iPwRvvJnn7CGhb/l6yEPtyINlgX/OP5
8b+g==
I think line 175 in Message.php is totally wrong:
$headerInProgress = str_ends_with($this->headers[$headerInProgress], ';');
It sets "1" to $headerInProgress variable (the result of str_ends_with function). Maybe this header is not fully compatible with RFC, but the library should be immune to wrong headers because the rest of them may be 100% correct. I tested this library on 4 random EML files and it failed on 2 of them.
Best regards
Marcin

@marcin-lawrowski hey Marcin, thanks for the report! Would you mind sharing the full failing EML file here? If it's test data, or anonymised. I'll the case to the test suite and find a solution.

Erpix Karcher Center2024-11-12 1601.zip

Here is EML file I tested this library with. I anonymised few things, but it should behave the same. Thank you.

Hey @marcin-lawrowski , this should now be fixed in v0.2.0. Be sure to update the package with composer update opcodesio/mail-parser
Let me know if you're still experiencing issues!


Django upgrade rebased on current development

this is the entirety of the django upgrade branch rebased on the current development branch.
it includes all PRs previously merged as well as the new changes to upgrade to django 4.2
do not squash this PR! each commit is specifically crafted to group specific sets of changes based on their function and complexity.
docker is not working yet.
docker builds. running it needs to be tested with a full setup: https://github.com/fossasia/eventyay-tickets/blob/master/doc/admin/installation/docker_smallscale.rst

I run the command docker build . and it built successfully

Sounds good, will squash merge to keep history clean and for easier rollbacks. Let's test on the dev server and see.


Filter to return only low calorie dishes

Current code returns high calorie dishes.

thanks

thanks

You are welcome! :)


LMPOP doesn't work

Description
None of the following work
client.LMPOP(3, 'lmlist1', 'lmlist2', 'lmlist3', 'LEFT', 'COUNT', '2'})
client.LMPOP(3, 'lmlist1', 'lmlist2', 'lmlist3', 'LEFT', {COUNT: 2})
client.LMPOP(['lmlist1', 'lmlist2', 'lmlist3'], 'LEFT', {COUNT: 2})
client.sendCommand(['LMPOP', '3', 'lmlist1', 'lmlist2', 'lmlist3', 'LEFT', 'COUNT', '2'])

Errors:
ERR unknown command `LMPOP`, with args beginning with: `3`, `lmlist1`, `lmlist2`, `lmlist3`, `LEFT`, `COUNT`, `2`,

or

TypeError: Invalid argument type

While against the same server using redis-cli
lmpop 3 lmlist1 lmlist2 lmlist3 LEFT COUNT 2

works fine.
Same thing for BLMPOP
Node.js Version
18
Redis Server Version
7
Node Redis Version
<EMAIL_ADDRESS>Platform
Linux
Logs
No response

I just tried
client.LMPOP(['lmlist1', 'lmlist2', 'lmlist3'], 'LEFT', {COUNT: 2});

and it works for me.. am I missing something?

It definitely does not work for me.  Are you running<EMAIL_ADDRESS>?

It definitely does not work for me.  Are you running 4.6.10?

It definitely does not work for me.  Are you using 4.6.10?


Add property to active timers JSON that is the exact time a timer will expire

Feature request:
Add property to active timers JSON that is the exact time a timer will expire.
Background
I need the exact time each timer will expire (not just the next timer).   It seems like the raw JSON data should be enough to get the answer, but I'm always off by a few seconds when I try to calculate it myself.  I would be nice if the raw JSON has a property (e.g., expireTime) that would be the exact time the timer was to expire.  This would be the same value that is getting returned for the entity itself but add this value to the raw JSON of each timer so that its available for not just the "next timer" but for all active timers.
Example of Issue
Below is the output of a template (run via the Developer Tools-->Template page).  Note that the calculated values is 2 seconds different then the value returned from the entity.  14:23:36 vs 14:23:38
 From raw JSON data:
key: 230ac9db-fe6d-37f4-acac-5b553c2e738f
expireTime: 2020-09-30 14:23:36
---
From the next timer value:
States: 2020-09-30T14:23:38-06:00

Here is the template:
{% set timers = state_attr('sensor.front_room_dot_next_timer', 'sorted_active') | from_json %}
{% for timerTokenData in timers %}
 {% set timerData = timerTokenData[1] %}
From raw JSON data:
key: {{ timerTokenData[0] }}
expireTime: {{ ((timerData.createdDate + timerData.remainingTime )/1000) | timestamp_local   }}
{% endfor %}
---
From the next timer value:
States: {{ states('sensor.front_room_dot_next_timer') }}

All of the raw JSON data we have is already provided to the user.  Timers need to be calculated as they are not absolute in the API. Please note pausing doesn't change the createdDate last I checked so I'm not aware of any absolute value we can grab. There is also some lag in when the request is received by HA (I have no idea why it's so variable as we're all using aiohttp).  If this has changed, then please let me know.
Regardless, feel free to submit a PR if you can figure out a way to accurately pull and calculate it.

I see what you mean that the Alexa API does not return an absolute value.  Bad Amazon design.
Well it seems that the value for the "next_timer" (for the sensor) is within 1 second (and often dead on) from what is expected.  However when I try to calculate the value via the RAW json, I can get 10+ seconds off.  I believe what is happening is the following code is executed quickly after the API call is made and hence has a very high accuracy.
https://github.com/custom-components/alexa_media_player/blob/dev/custom_components/alexa_media/sensor.py#L447
However when I try to do the same calculation from the RAW JSON within an HA template, HA can take awhile to go through all the state changes before it then interpretes my template and finally does the calculation.  By this time several seconds (sometimes as much as 10) has passed and since remainingTime is frozen in time, and I don't know how much time has passed, the result is inaccurate.
So I'm wondering if we can add the calculated value from here into the raw JSON values(s) -- e.g., an expireTime into each timer that is in the sorted_active list.  This way the value is as close in real-time as we can get to "snapshotting" when each timer will expire.   Just an idea.
If you don't like the idea of adding a synthetic property to the sorted_active JSON, we could create another attribute of the sensor that is just an map of timer id and calculated expire time.
Note the new expireTimes attribute in below example:
recurrence: Never Repeat
prior_value: 2020-10-01T07:59:23-06:00
expireTimes: ['8734d10a-16ef-34e4-bb21-d6e775d59705':<PHONE_NUMBER>376,
   '50cdbd6a-4686-31d7-a8a6-a17c344c4714':<PHONE_NUMBER>950]
total_active: 2
total_all: 12
sorted_active:  ....

Have you tried using the last_changed or last_updated attribute instead of now()? That won't have any drift due to the system due to the local system.
As a note, what is your actual goal? I am about to introduce event firing when a notification goes off as I found a way to calculate it that seems accurate enough.

So if I use last_updated to calculate the value and compare that to the NextTimer, I can get a variance.  Sometimes its in-sync but sometimes it's off by many seconds.   So the timestamp used to calculate NextTimer internally is not the same timestamp returned by last_updated.  This is the reason it would be helpful to either include the internally calculated time (which is accurate) in the attributes, or expose the timestamp used from here to calculate the remainingTime as an attribute so that a template can calculate all timers accurately.
Example:
From raw JSON data:
CalcNext:  2020-10-01T16:37:47-0600<PHONE_NUMBER>.509802
NextTimer: 2020-10-01T16:37:44-06:00<PHONE_NUMBER>.0
Diff: 3.5098021030426025

Test Template Script that produced the above output:
{% set timers = state_attr('sensor.tv_room_next_timer', 'sorted_active') | from_json %}
{% for timerTokenData in timers %}
 {% set timerData = timerTokenData[1] %}
 {% set expireTime =  (timerData.remainingTime/1000 + as_timestamp(states.sensor.tv_room_next_timer.last_updated)) %}
From raw JSON data:
key: {{ timerTokenData[0] }}
CalcNext:  {{ expireTime | timestamp_custom("%Y-%m-%dT%H:%M:%S%z")  }}  {{ expireTime }}
NextTimer: {{ states('sensor.tv_room_next_timer') }} {{ as_timestamp(strptime(states('sensor.tv_room_next_timer'), "%H:%M, %Y-%m-%d")) }}
Diff: {{ expireTime - as_timestamp(strptime(states('sensor.tv_room_next_timer'), "%H:%M, %Y-%m-%d")) }}
{% endfor %}

Actual Goal
To you question of the actual goal:  I need the exact expiration time (as accurately as possible given the limitations of the Alexa API) for ALL active timers.
The use case for me is a remote LED matrix display that shows all alexa timers.  Currently its controlled via a Raspberry PI with a bluetooth connection to a echo dot via the Alexa Gadgets API.  This works well and is very accurate as the timer counts down to zero on the display, the echo dot plays its alert tone.   The problem is that it only shows timers from the 1 echo dot that the bluetooth is paired with.   What I want is to display all timers (from all echo devices).   To achieve this I'm looking to abandon the bluetooth method and switch to having HA with this addon to send the timer expire information to the raspberry pi.   To make this work well, the timer expire needs to be within a second or so otherwise the LED display countdown is not in sync with the echo playing the alert tone.

What I need is access to this timestamp:
https://github.com/custom-components/alexa_media_player/blob/dev/custom_components/alexa_media/sensor.py#L358
Where does this timestamp come from?   Is it exposed anywhere that can be access via a template?

No, it's not exposed. If that's all you want, just subtract out the 'timeRemaining' from the sensor value and you'll have the value.
It's generated when we contact the API.

When you say "from the sensor value", how to I get this value (timestamp) from a template?

FYI, I just hacked the code to insert a timestamp_base into the attributes and it's the exact time I need to perform all the calculations.   If this timestamp is already available via some other way, please let me know.  If not, can I request that it be added?
This was my quick and dirty change:  (added 'timestamp_base')

        attr = {
            "recurrence": self.recurrence,
            "prior_value": self._process_state(self._prior_value),

            "timestamp_base":
                dt.as_local(
                        datetime.datetime.fromtimestamp(
                            self._timestamp.timestamp()
                        )
                ).isoformat()
                if self._timestamp
                else STATE_UNAVAILABLE,

            "total_active": len(self._active),
            "total_all": len(self._all),
            "sorted_active": json.dumps(self._active, default=str),
            "sorted_all": json.dumps(self._all, default=str),
        }

Looking at the code you referenced, a better name for the new attribute would have been process_timestamp

I don't do templating so I don't know. You can ask the forums. Feel free to submit a PR if you want. Please make sure it doesn't break alarm and reminders though.

Created PR https://github.com/custom-components/alexa_media_player/pull/966.  This PR adds a new attribute to notification sensors called process_timestamp.  The value of this attribute provides access to HA templates allowing very accurate calculations of all future timers.
I verified alarms, notifications, timers to ensure nothing broke.

Thanks merged it in. It's in dev. Probably release it this weekend.
With that I assume this issue is closed. Let me know if you think otherwise.


update attendance tool icon

FA icon to associate with it. Please use check-square (checkmark)

this is already done


treewide: Replace bzero with memset

bzero has been deprecated by POSIX 2008. It recommends the use of memset
instead.

It may be deprecated by POSIX, but this is platform specific code and zero is not deprecated on macOS. Will consider it...

OTOH that Darwin file already uses mender.


Feature request: Supply initial state

Hi,
I am using the dynamic-dimmer node for some of my more complicated light setups. Since I am only doing some controls through the dimmers and others directly, I am missing an option to supply an initial state to the node.
For instance, I want the temperature of my lights to change slowly to a warmer tone in the evening. If they were off before I can just set the temperature and then slow increase the brightness. However, if they were on before through another routine, they already have a specific tone set - so I would need the dimmer just to change the value from 0.7 to 0.3 for example.
However, when supplying the target through a payload the dimmer always starts at its minium value from the node configuration - which would be zero.
As far as I understand the documentation there is currently no way to supply an inital starting value...

@koshisan Have you found another solution for

I'm currently facing the same problem. @koshisan Have you found another solution already?

I wouldn't go so far and call it a solution. I basically decoupled the dimmer from the actual item I want to control and instead use it only to generate a series of numbers. I use this series of numbers in a function node which substracts/adds them to the actual starting value and then writes that value to the target item...


change inputs back to string from list(string)

what

Change the optional inputs of the module to be string rather than list(string)

/test bats

/test readme terratest

@mcalhoun While these changes do avoid the  "count" value depends on resource attributes that cannot be determined until apply... error, it looks like they do that by exploiting a bug in Terraform. See https://github.com/hashicorp/terraform/issues/29973
I updated the test to surface the error. The first apply fails, although a second apply will succeed. I don't want to recreate https://github.com/cloudposse/terraform-aws-elasticache-redis/issues/121

/test all

@mcalhoun said:

I see the problem and the issue you opened. I guess we can't use this method until the underlying bug is fixed. Should we ask Hashcorp on that issue what a workaround is instead of turning string into list(string)?

Turning string into list(string) is Hashicorp's solution to the problem of detecting the presence of optional inputs when the values are unknown.

@mcalhoun Indeed, as I expected, https://github.com/hashicorp/terraform/issues/29973 has been fixed by ensuring "unknown values upgraded to a collection via a splat expression result in a DynamicVal (unknown value of unknown type), because the resulting type may be a tuple with 0 or 1 elements."


Using ClientCertificate with Android

Description
Hi,
As a requirement for our App, we need to do two way SSL pinning.
We can validate our APIs certificate using "options.RemoteCertificateValidationCallback".
But when we try to add a Certificate to our requests on Android. We get hit by a null pointer. I searched for the issue and found one from the old xamarin.android: https://github.com/dotnet/android/issues/7274 . Which is the exact issue we are facing.
I added a repo, we use RestSharp, but it seems to happen even if we use the default HttpClient.
Steps to Reproduce
git clone https://github.com/Pmr-precure/MediaElementDispose.git
Run project on Android.
I init a RestClient in onappearing on the homepage, and it will crash shorting after, with a null pointer.
Link to public reproduction project repository
https://github.com/Pmr-precure/MediaElementDispose/tree/client_certificate
Version with bug
9.0.12 SR1.2
Is this a regression from previous behavior?
Not sure, did not test other versions
Last version that worked well
Unknown/Other
Affected platforms
Android
Affected platform versions
I have only seen it on Android.
Did you find any workaround?
No
Relevant log output
[mono-rt] [ERROR] FATAL UNHANDLED EXCEPTION: System.NullReferenceException: Object reference not set to an instance of an object.
[mono-rt]    at RestSharp.RestClient.ConfigureHttpMessageHandler(HttpClientHandler handler, RestClientOptions options)
[mono-rt]    at RestSharp.RestClient.<>c__DisplayClass29_0.<.ctor>g__GetClient|0()
[mono-rt]    at RestSharp.RestClient..ctor(RestClientOptions options, ConfigureHeaders configureDefaultHeaders, ConfigureSerialization configureSerialization, Boolean useClientFactory)
[mono-rt]    at TestProject.MainPage.InitClient() in C:\Users\PMR\source\repos\TestProject\TestProject\MainPage.xaml.cs:line 84
[mono-rt]    at TestProject.MainPage.OnAppearing() in C:\Users\PMR\source\repos\TestProject\TestProject\MainPage.xaml.cs:line 25
[mono-rt]    at System.Threading.Tasks.Task.<>c.<ThrowAsync>b__128_0(Object state)
[mono-rt]    at Android.App.SyncContext.<>c__DisplayClass2_0.<Post>b__0() in /Users/runner/work/1/s/xamarin-android/src/Mono.Android/Android.App/SyncContext.cs:line 36
[mono-rt]    at Java.Lang.Thread.RunnableImplementor.Run() in /Users/runner/work/1/s/xamarin-android/src/Mono.Android/Java.Lang/Thread.cs:line 37
[mono-rt]    at Java.Lang.IRunnableInvoker.n_Run(IntPtr jnienv, IntPtr native__this) in /Users/runner/work/1/s/xamarin-android/src/Mono.Android/obj/Release/net8.0/android-34/mcw/Java.Lang.IRunnable.cs:line 84
[mono-rt]    at Android.Runtime.JNINativeWrapper.Wrap_JniMarshal_PP_V(_JniMarshal_PP_V callback, IntPtr jnienv, IntPtr klazz) in /Users/runner/work/1/s/xamarin-android/src/Mono.Android/Android.Runtime/JNINativeWrapper.g.cs:line 22

Based on the issue you linked being fixed on Jul 10, it is likely only in .NET 9.
Try targeting your sample to net9.0-android and see if that fixes it.

Based on the issue you linked being fixed on Jul 10, it is likely only in .NET 9.
Try targeting your sample to net9.0-android and see if that fixes it.

I will try that tomorrow. Ill let you know

Based on the issue you linked being fixed on Jul 10, it is likely only in .NET 9.
Try targeting your sample to net9.0-android and see if that fixes it.

I can confirm that its fixed in .NET 9 :) Thank you


Fix bugs: Carousel

Q
A

Bug fix?
[x]

New feature?
[ ]

New sample?
[ ]

What's in this Pull Request?
Fix Carousel control:

Missing 'Carousel.ts' file in the src folder - to export the control.
Remove unused code.
Fix wrong condition to disable next button for fixed elements array.

Codecov Report

Merging #337 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #337   +/-   ##
=======================================
  Coverage   72.88%   72.88%
=======================================
  Files          17       17
  Lines         767      767
  Branches      149      159   +10
=======================================
  Hits          559      559
  Misses        161      161
  Partials       47       47

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 9d399f9...7ad4d73. Read the comment docs.

Thanks @siata13


Support downloading boost on Mac OS

MacOS doesn’t support wget natively, so use curl
instead
The source forge link doesn’t seem to work to just
download the boost archive anymore, replace it with
the official archive from boostorg

Why not use curl for everything?

@drubinstein Thats a good point, I will make that fix


"Model governance" vs. "Publishing models"

Contributions

[X] I have read the contribution docs, and understand what's expected of me.

What page(s) or areas on docs.getdbt.com are affected?
When searching for "model versions", why does this say "Publishing models" instead of "Model governance"? Can we change it?

Possibly related (or not): the following redirect didn't work for me this weekend, but it does now 🤷
According to the _redirects file, this page should redirect to here.
It wasn't redirecting this weekend for me for some unknown reason.
What changes are you suggesting?
It should say "Model governance" instead of "Publishing models".
Additional information
No response

I think this is no longer an issue, because

the redirects have had enough time to work their magic
the new links have taken priority in Algolia's index
some secret third thing

Thank you for confirming @jtcohen6 👍


Player selling wands using pheromones gives lots of money

So, host selling wands works okay (though the player doesn't get any money for some reason), as host gets reasonable cash. But, when player sells wands, it gets both the player and the host like 600 or 800 gold for ANY wand sold, and it's stats doesn't even matter.

I've found that this issue can easily stack up with this issue here, leading to the wand not disappearing on one side and being turned into gold on the other side
me and my friend got through a few cycles while laughing his head off at how powerful this was before he(host) crashed and lost the save

i think this is fixed, gold may apear on both sides still depending on how you sell it but i dont get extra gold


Update test_fakeData.py

Codecov Report

Merging #33 into develop will increase coverage by 22.98%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           develop      #33       +/-   ##
============================================
+ Coverage     0.00%   22.98%   +22.98%
============================================
  Files            4        5        +1
  Lines           26      248      +222
  Branches         0       33       +33
============================================
+ Hits             0       57       +57
- Misses          26      191      +165

Impacted Files
Coverage Δ

tests/conftest.py

setup.py

tests/test_farm.py

src/cafe/__init__.py
100.00% <0.00%> (ø)

src/cafe/farm.py
8.39% <0.00%> (ø)

src/cafe/importData.py
0.00% <0.00%> (ø)

src/cafe/simulateCoOp.py
0.00% <0.00%> (ø)

src/cafe/fakeData.py
100.00% <0.00%> (ø)

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 a185eed...c3adcc2. Read the comment docs.


⚠ Quirk Club Argentina (API endpoint) has degraded performance

In c5d5d60, Quirk Club Argentina (API endpoint) (https://us-central1-quirkclub-dev.cloudfunctions.net/api/check/api) experienced degraded performance:

HTTP code: 200
Response time: 6942 ms

Resolved: Quirk Club Argentina (API endpoint) performance has improved in 4a3b0d4.


Renamed or deleted images do not render in diffs

Gitea Version
1.16.1
Git Version
git version 2.30.2
Operating System
Debian Bullseye
How are you running Gitea?
We build our own Docker images using Debian Bullseye. This approach was adapted from the official upstream images that use Alpine. The Dockerfile used to build Gitea 1.16.1 is visible here: https://opendev.org/opendev/system-config/src/commit/f3b767cf60d58be979ba6bb61112907032912fb5/docker/gitea/Dockerfile. Note this is a Gitea 1.15.11 instance separate from the 1.16.1 instance that we are testing.
Database
MySQL
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Description
When viewing diffs for files that have been renamed viewing the raw data for the old file name is a 404. When viewing diffs for files that have been deleted the raw data for the file is a 404. In both cases this seems to trip the diff rendering of the commit for that file because one side of the diff 404s. In these situations we get a spinning circle rather than the diff as expected. Older gitea (1.15.11 for example) seems to handle this appropriately. In the case of deletions showing only the old side of the data and in the cases of renames finding both versions of the file.
If you click on view file for the renamed files the file loads fine directly outside of the diff view. The server logs indicate:
...s/context/context.go:290:PlainTextBytes() [E] PlainTextBytes: Not found.

I think the issue is that when the diff viewer is unable to find both sides of the diff it fails now. Previously it was able to find the old and new file content for renamed files and it ignored missing data for deleted files. It only showed the before side and not the after side for deleted files.
I believe that https://try.gitea.io/fx6904/gitea/commit/7cf23399a0b97cb8211e8e7b00020e9b1a5fdac1 shows this on the demo site for public/img/favicon.ico. This file appears to have been deleted so the new side of the diff is a 404 and the view just spins.
It is possible that this is also a problem for text files, but I haven't looked into that yet.
This issue https://github.com/go-gitea/gitea/issues/18811 is related in that it describes a new method of handling deleted files in diffs. I think showing that the file has been deleted instead of trying to diff it would be fine. However, that doesn't handle the renamed file case.
Screenshots
Here is Gitea 1.16.1 attempting to diff renamed and deleted files for us:

And this shows the old Gitea 1.15.11 behavior:

I've checked and renamed/deleted text files do not suffer from this issue. It seems related to more complex file types like the SVG and PNGs in my example.

Is fixed with #18799


[C++] Add setChunkOptions method

Public-Facing Changes

[C++] McapWriter.setChunkOptions() for closing the current chunk and starting a new chunk with new options (chunk size and compression settings)

Description
The C++ library gives fine-grained control over closing the current chunk and starting a new chunk, but the compression and chunk size options cannot be modified after opening a file for writing. This new method allows a new chunk to be started with new chunk size and compression options. This is practically useful when writing channels to separate parts of an MCAP file, such as writing small compressible topics first with 1MB size + ZSTD, followed by large incompressible video/image topics with 4MB size + NONE.

Seems like some potential improvements were identified but this PR hasn't been revisited recently – any interest in continuing to pursue it?

Closing this for inactivity. If you do come back to this and want to address the comments we can revisit.


Weird issue relating cat and pids

root@debian:~# caddy start
cat: /var/run/caddy/caddy.pid: No such file or directory

  Starting Caddy Web Server...  DONE

root@debian:~# caddy

  Caddy Web Server is NOT running

  Welcome!
  run caddy help  for help

I found this command in the source-code to get Caddy's pid's I guess, here's the output:
root@debian:~# ps aux | grep /opt/caddyserver/caddy | grep -v grep | awk '{ print $2 }' | sort -
20014
20017

The script doesn't detect Caddy is running, but the pidfile exists and that command returns two pids. And btw, shouldn't the cat error message be silenced/omitted?
I'm running Debian 8 (Jessie). Anyone with the same problem?
root@debian:~# uname -a
Linux debian 2.6.32-042stab113.17 #1 SMP Wed Feb 10 18:31:00 MSK 2016 x86_64 GNU/Linux

Should be fixed now!
run this wget -q https://raw.githubusercontent.com/sayem314/Caddy-Web-Server-Installer/master/caddy.sh -O - -o /dev/null|bash

Thanks @sayem314, it's fixed :)
The cat error still shows up, but it's not critical:
cat: /var/run/caddy/caddy.pid: No such file or directory


Segfault during (Re)importing assets of converted project

Godot version
4.0 alpha 14; custom build 58f8f3a; 4.0 Beta 1
System information
Arch Linux
Issue description
I tried to convert my Godot 3 project to Godot 4 and edit it.  After conversion, it got to "(Re)importing assets" and crashed with a segfault during this process.  Opening the same project from the list afterwards yields the same result.
I'm not expecting the project to convert cleanly, but simpler projects have imported without crashing and I could fix the differences in nodes and scripts afterwards.
Console errors before the crash, and the backtrace:
backtrace.txt
The script errors aren't surprising, but the !importer.is_valid() sounded like it might be relevant.
Steps to reproduce

git clone https://github.com/RTForge/DawnOfTheDuck
Using Godot 4, import the project in the duck subdirectory from the editor, or edit the project from the command line

Minimal reproduction project
See above

This is difficult to reproduce. It happens only 5%-10% of the time for me. It indeed still happens in RC2, even though many importer related issues were already solved.
TL;DR (see below): This seems to be a bug with the dynamic font importer.
Reproduction steps:
clone the attached project (yes, it's made for Godot 3.x. that's fine.)
Edit project.godot and remove run/main_scene from it. That makes it easier to exit out of the editor without having errors pop up.
Note: I was unable to reproduce the issue after porting the project to Godot 4.x. Here is my diff if you're interested:
duck_to_4.diff.txt
Then keep running this repeatedly, closing the editor each time it loads successfully.
rm -rf .godot; ~/Downloads/Godot_v4.0-rc2_win64.exe -w -e

when it crashes, it's usually very early on.
$ Vulkan API 1.3.224 - Forward+ - Using Vulkan Device #0: NVIDIA - NVIDIA GeForce RTX 3090

ERROR: Cannot open file 'res://tp/new_theme.tres'.
   at: (scene/resources/resource_format_text.cpp:1649)
ERROR: Failed loading resource: res://tp/new_theme.tres. Make sure resources have been imported by opening the project in the editor at least once.
   at: (core/io/resource_loader.cpp:221)
ERROR: Cannot open file 'res://.godot/imported/Raleway-Medium.ttf-bf3a59b6b84ee7d9b4ef514fb82fdb8e.fontdata'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/SigmarOne-Regular.ttf-c54f2e16d2b46652f2ad680fe092e574.fontdata'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/boat2.glb-fe26d62f6ca55c071a4b687d6a0f4ecc.scn'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/duck.glb-af6623336133712d01a6bfdefd9d6344.scn'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/gun.glb-05f42a236277268338275ecffb48440f.scn'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/muzzle_flash.obj-8f77222eab2a9102e995fbb8d9e55f50.mesh'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/tub.glb-b050456f185c0941f22da0a05b67312d.scn'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/duck_intro.ogg-75ff4c73f791e148447ced02042586f3.oggvorbisstr'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/engine.wav-5941c30789ec1b20ddeb5177648accc9.sample'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/faucet.wav-c64c418ac8b13b1dfbca9ee15a387a1f.sample'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/fireball.wav-d12de690fb1bc5635b3139e19432a5d6.sample'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/good_bad_ugly.wav-a1967205b565992397d00404d9fbd6b1.sample'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/gun1.wav-53ac4776464b0aabe9fd2135e1575e8f.sample'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/gun2.wav-a8181bf7dbe45fabe700db54c3f71de3.sample'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/gun3.wav-c8cddd93351d54c5fdfd782185777204.sample'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/gun4.wav-4df729fde5f1488da84543ae9e59c5f8.sample'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/metal1.wav-ee30351527ac3efe7f953556aeb67979.sample'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/metal2.wav-676d64d49af5dd9f0799238bfc25941c.sample'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/metal3.wav-0b8640b9535c0fa11254c2d6b8b7912d.sample'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/metal4.wav-2792b5a566923c1990b717071c307bc6.sample'.
   at: (core/io/resource_format_binary.cpp:1249)
ERROR: Cannot open file 'res://.godot/imported/splash.wav-ff0f95428d6fe31dc6a062bb450bc559.sample'.
   at: (core/io/resource_format_binary.cpp:1249)
Unicode parsing error, some characters were rep
[1]+  Segmentation fault      ~/Downloads/Godot_v4.0-rc1_win64.exe -w -e

I was able to reproduce the crash in both rc1 and also rc2.
The Unicode parsing error is not always displayed, but it does give a clue that perhaps it is related to fonts.
I was able to reproduce a crash from Visual Studio.
Here are the relevant all-thread stacktraces from two different crashing runs in a custom build on 4848877, close to RC2. I used Debug.ListCallStack /ShowLineOffset /AllThreads to display stacktraces from all threads in Visual Studio's Command Window.
Crashing Run 1: all_threads_stacktraces_fontdata_import.txt
Crashing Run 2: all_threads_stacktraces_fontdata_import2.txt
In the first one, two threads were importing fonts at the same time (ResourceImporterDynamicFont::import)
In the second stacktrace, only one was importing at that moment, but the other import thread was actively cleaning up from a .ttf import - this time I was able to print out the filenames in the debugger:
Thread 42: res://fonts/Raleway-Medium.ttf.import
Thread 43: res://.godot/imported/SigmarOne-Regular.ttf-c54f2e16d2b46652f2ad680fe092e574
Pinging @bruvzg since you might know something about font import

For now, as a workaround, disable multithreaded import in project settings, or retry multiple times until all of the problematic files get imported and Godot moves on. I think at least one file gets successfully imported each time this happens.

I gave my original reproduction steps a try with the official 4.0.3 and 4.1-beta 3 release packages.
4.0.3:  No crash during import, dependency error (without crash) when editing the project afterwards.  Closing and re-opening the project opens the converted project in the expected form.
4.1-beta3: No crash during import, and editing the project opens it in the expected form without the error from 4.0.3
It seems to me like the original segfault is no longer happening, and the intermediate dependency error in 4.0.3 isn't worth worrying about.

Closing based on the above comment. Thanks for testing!


Remove macos tests

Remove failing macos tests.
These tests are failing because of OS config issues. They'll be added back once I figure out why they suddenly started failing with "abort trap 6" errors

Tests will be added back in #26


Improved scheduling for the PipelinedExecutor

As discussed before , an easy improvements on
the round-robin scheduling implemented before, is sending tasks always to the process with the shortest request queue. This is implemented in these changes.
This should improve GPU performance, too, although I have not benchmarked this yet. With these changes, it should also be quite easy to preferentially send tasks to GPU workers when they are idle. Implementing proper resource support should now also be possible.
In addition, this also speeds up the tests by a lot; on the system I tested a full tests_pipelined run goes from ~108s to ~28s - by using less worker processes, and by catching errors on startup faster. Also marked the "default spec" case as slow, as that will always try to start ~many workers on large machines.
This sits on top of #1443 and I'll rebase once that is merged.
TODO

[ ] Rebase once #1443 is merged
[ ] Update documentation

Contributor Checklist:

[ ] I have added or updated my entry in the creators.json file
[ ] I have added a changelog entry for my contribution
[ ] I have added/updated documentation for all user-facing changes
[ ] I have added/updated test cases
[ ] I have included the rebuilt production build of the client (only if changes were made to the GUI)

Reviewer Checklist:

[ ] /azp run libertem.libertem-data passed
[ ] No import of GPL code from MIT code

To keep this PR short (once it is rebased), I'm putting the work to support task resources into its own PR

Oof, looks like Mac OS doesn't support getting the queue size, so there needs to be a fallback to the previous behavior.

/azp run libertem.libertem-data

Thanks for the review!

Looks good to me. I noted there isn't really a good way to test the scheduling implementation -one way could be to play some games with time.sleep() in the first task and os.getpid() to figure out where each task executed. Not sure it's worth it, though!

Yeah - that has the potential to be a flaky test too, as the queue size method this is based on is "not reliable" in an unspecified way. In #1453 there are a few more tests, including test_device_classes_limit_to_cpus where we do check if at least each CPU worker did get work at all. It's also complicated by the live case using the queue for more than just one message per task - meaning the scheduling should in theory work better when used with LiberTEM-live, which is hard to check from tests in LiberTEM. So I'm fine with not having an explicit test for scheduling behavior.
I'm going to merge this now; if we are inspired, we can improve the testing further in #1453


feat: Recent NFT mints in drop page

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[x] Feature
[ ] Refactoring

Context

[x] Closes #7009
[ ] Requires deployment <snek/rubick/worker>

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[x] My fix has changed something on UI;

free drop

dot drop

Copilot Summary

🤖 Generated by Copilot at ca68c0c
This pull request adds a new carousel component to display the latest NFT mints for a collection on the drop and unlockable pages. It also modifies the existing carousel component and the Subsquid queries to support this feature. The new component uses a slot to customize the card info for each NFT.

🤖 Generated by Copilot at ca68c0c

We're fetching the latest NFT mints from the Subsquid API
We're using slots to customize the card info, aye aye
We're heaving on the CarouselTypeLatestMints component
One, two, three, and pull, we'll make the drop page look magnificent

pay 50 usd


Terget is set to esnext in tsconfig.mjs.json

Hi,
The target is set to esnext in tsconfig.mjs.json, is it on purpose?
Why not the same target as in tsconfig.json?
The problem there is a Nullish coalescing operator, which is passed by esnext, but converted by es5.

Hello @david-ichilov-evinced,
Good point, thank you.
Fixed in v3.0.1.


[Bug]: [CosmosDB] replaceDocument function fails with a document that contains a JSON-unfriendly string

Connector Name
module/cosmosdb (Cosmos DB)
What happened?
This is reported via Discord https://discord.com/channels/957996897782616114/1108861656316002335/1108861656316002335
Relevant log output
error: Illegal unquoted character ((CTRL-CHAR, code 10))

The issue with JSON-unfriendly string can be resolved by escaping the special characters such as \n.

Optionally, we can Base64 encode the specific data and insert it in the document or if the JSON-unfriendly string is large we can go to an option like storing the string in a separate attachment and refer it in the document.

We cannot replace the id as per the current implementation of the connector.

The option to replace the _etag is also not supported as per the current implementation of the connector. We have to check if this is possible to do with our native implementation. This option is supported in CosmosDB SDKs

We can send an _etag of our choice in the document body but, CosmosDB replaces this with an auto generated _etag of it's own. If we try to do this via their data explorer the result is same. So this requirement is not possible at the moment


Use exrcise package to automatically set up your exercises and keys

https://github.com/jhudsl/intro_to_R_class/issues/71

I think this is done

@avahoffman this PR did this right? https://github.com/jhudsl/intro_to_r/pull/548

i used a different approach, but yes :)


Unclear that Bonfire:StandInLine/Queue won't pass unless Display Code is removed

Challenge Bonfire: Stand in Line has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.106 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
// Setup
var myArr = [1,2,3,4,5];

function queue(arr, item) {
  arr.push(item);
  return arr.shift();  // Change this line
}

The above version passes, since the three initially-supplied calls at the bottom to Console.log have been removed.  Without the removal, submitting the code yields a stack overflow error.
In other Bonfires and Waypoints that I've done (a lot of them), there was no instance where the testing code (usually, just an illustrative function call) needed to be removed for a submittal to pass.  Thus, this Bonfire does not conform to the normal pattern.
IMO, providing this testing code at the bottom for this Bonfire is overkill.  By this time, if campers want to test in their own environment, they have figured out how to do so.  Also, the calls to JSON.stringify are unnecessary - console.log can handle an array of integers without it.

@dcc1079
Posting your exact error will help. Is that error message like the one posted in this #5560 ?

@dcc1079
Posting your exact error will help. Is that error message like the one posted in this #5560 ?

The console.logs work fine for me.  Regarding the stack overflow error, I've had that happen to me when I hadn't yet finished my code and I hit the run tests button.  Trying to run the incomplete code (empty function calls, missing semi-colon, parentheses, bracket, etc.) caused this to happen until I refreshed the page and made sure my code was in order before hitting the run tests button.
Refreshing your page usually clears the error up though.

The console.logs work fine for me.  Regarding the stack overflow error, I've had that happen to me when I hadn't yet finished my code and I hit the run tests button.  Trying to run the incomplete code (empty function calls, missing semi-colon, parentheses, bracket, etc.) caused this to happen until I refreshed the page and made sure my code was in order before hitting the run tests button.
Refreshing your page usually clears the error up though.

Hi Raja,
Thanks for the reply.  5560 is the same error.
I tried to again just now, on Opera (not logged in) and on Chrome (in an
incognitio window so that my changes are not loaded and instead I get a
fresh page), and I can't reproduce it.  Perhaps you've found and fixed the
problem(?).
Thanks and regards,
Don Cameron
On Fri, Jan 1, 2016 at 11:30 AM, Raja Gopal<EMAIL_ADDRESS>wrote:

@dcc1079 https://github.com/dcc1079
Posting your exact error will help. Is that error message like the one
posted in this #5560
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/5560 ?
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/5715#issuecomment-168327118
.

Hi Raja,
Thanks for the reply.  5560 is the same error.
I tried to again just now, on Opera (not logged in) and on Chrome (in an
incognitio window so that my changes are not loaded and instead I get a
fresh page), and I can't reproduce it.  Perhaps you've found and fixed the
problem(?).
Thanks and regards,
Don Cameron
On Fri, Jan 1, 2016 at 11:30 AM, Raja Gopal<EMAIL_ADDRESS>wrote:

@dcc1079 https://github.com/dcc1079
Posting your exact error will help. Is that error message like the one
posted in this #5560
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/5560 ?
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/5715#issuecomment-168327118
.

I've seen the same issue with this challenge.

I've seen the same issue with this challenge.

I've seen the "stack overflow" issue in numerous challenges. As @ShaggyTech mentioned, I've noticed that when I submit a wrong solution/bad code, correct it and resubmit it the stack overflow error comes up. Refreshing the page clears it up and the same code works.

Well I was getting "shift is not a function" error on external javascript compiler, I was able to solve that by re- assigning the array from the parameter into a new local array variable, but I'm still getting the
"Range Error: Maximum call stack size exceeded" error message.
var myArr = [1,2,3,4,5];

function queue(arr, item) {
  // Your code here
  var myArray = arr;
  myArray.push(item);
  var shifted = myArray.shift();
  return shifted;// Change this line
}
console.log("Before: " + JSON.stringify(myArr));
console.log(queue(myArr, 6)); // Modify this line to test
console.log("After: " + JSON.stringify(myArr));

This code works externally but not in the checkpoint

A stack error gets trown, when a function is called that does not exist within the console.log wrapper object.
This code will allways trow a stack error:
function queue(arr, item) {
  arr.push(item);
  return arr.shift();
}
console.log(queue([5], 6)); // shall return 5
console.log(trowStack()); // shall return 5

I couldn't think of any possible solution to fix it. I tried figuring out where the stack gets trown. But i couldn't get closer than this. Its to complicated for my expierence. Hopefully someone more expierenced can take it from here.

@SaintPeter or @BerkeleyTrue, can you close this? It looks like the issue is fixed.

@erictleung Thanks!


Update-xDscResource Command Ignores Next-to-Last Line of Resource File

The Update-xDscResource command seems to ignore the next-to-last line of the Resource .psm1 file that it's updating. For example, just as a demonstration, change the "*" in the Export-ModuleMember line to an "X", so the line reads "Export-ModuleMember -Function X-TargetResource". If there is a single blank line following that line, the Update-xDscResource command still works fine (although you would expect it to fail). However, if you add a second blank line at the end of the file, it fails as expected. This is when using version 1.13 of xDSCResourceDesigner.
Issue created by request from Johan Ljunggren.

@Rob-S Thanks for submitting the issue to here.


[Bug] Copyright year missing in the footer section of contributors page.

Description
No response
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[X] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

@Rakesh9100 Please assign this issue to me under the SWOC label.

Proceed @gauravsingh1281


Update cats-mtl to 1.4.0

About this PR
📦 Updates org.typelevel:cats-mtl from 1.2.1 to 1.4.0
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel", artifactId = "cats-mtl" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.typelevel", artifactId = "cats-mtl" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, version-scheme:early-semver, commit-count:1

Superseded by #579.


entryDetail: validate that Amount is non-negative

NACHA states that amounts cannot be negative.

Codecov Report

Merging #561 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #561      +/-   ##
==========================================
+ Coverage   90.46%   90.46%   +<.01%
==========================================
  Files          64       64
  Lines        5084     5086       +2
==========================================
+ Hits         4599     4601       +2
  Misses        351      351
  Partials      134      134


feat: added public service endpint url for the cluster

Description
Add cluster URL output to Red Hat OpenShift Container Platform on VPC landing zone
Release required?

[ ] No release
[ ] Patch release (x.x.X)
[x] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

Some testing is still pending - please do not review.

Output
workload_cluster_id = "cr3gsrqd0krk13m3mud0"
workload_cluster_ingress_hostname = "rajat-roksqs-workload-clu-3b5bf5f75003778663c521c8c35ad277-0000.us-south.containers.appdomain.cloud"
workload_cluster_private_service_endpoint_url = "https://c100.private.us-south.containers.cloud.ibm.com:32110"
workload_cluster_public_cluster_url = "https://console-openshift-console.rajat-roksqs-workload-clu-3b5bf5f75003778663c521c8c35ad277-0000.us-south.containers.appdomain.cloud"
workload_cluster_public_service_endpoint_url = "https://c100-e.us-south.containers.cloud.ibm.com:32110"

Updated output -
workload_cluster_console_url = "https://console-openshift-console.rajat-roksqs-workload-clu-3b5bf5f75003778663c521c8c35ad277-0000.us-south.containers.appdomain.cloud"
workload_cluster_id = "cr3gsrqd0krk13m3mud0"
workload_cluster_ingress_hostname = "rajat-roksqs-workload-clu-3b5bf5f75003778663c521c8c35ad277-0000.us-south.containers.appdomain.cloud"
workload_cluster_private_service_endpoint_url = "https://c100.private.us-south.containers.cloud.ibm.com:32110"
workload_cluster_public_service_endpoint_url = "https://c100-e.us-south.containers.cloud.ibm.com:32110"

/run pipeline

/run pipeline

Getting this error as public_service_endpoint_url is returning   an "" empty string rather than null value. Updated the code to handle the scenario
2024-08-26T10:11:02.4167399Z TestRunUpgradeRoksPattern
2024-08-26T10:11:02Z retry.go:99: Returning due to fatal error: FatalError{Underlying: error while running command: exit status 1;
2024-08-26T10:11:02.4170135Z Error: Error in function call
2024-08-26T10:11:02.4174135Z   on ../../outputs.tf line 99, in output "cluster_data":
2024-08-26T10:11:02.4175604Z   99:       public_service_endpoint_url = coalesce(cluster.public_service_endpoint_url, null)
2024-08-26T10:11:02.4177261Z       │ cluster.public_service_endpoint_url is ""
2024-08-26T10:11:02.4190151Z Call to function "coalesce" failed: no non-null, non-empty-string arguments.
2024-08-26T10:11:02.4196012Z Error: Error in function call
2024-08-26T10:11:02.4197467Z   on ../../outputs.tf line 99, in output "cluster_data":
2024-08-26T10:11:02.4198443Z   99:       public_service_endpoint_url = coalesce(cluster.public_service_endpoint_url, null)
2024-08-26T10:11:02.4199995Z       │ cluster.public_service_endpoint_url is ""
2024-08-26T10:11:02.4201232Z Call to function "coalesce" failed: no non-null, non-empty-string arguments.
}

/run pipeline

/run pipeline

/run pipeline

/run pipeline

:tada: This issue has been resolved in version 5.31.0 :tada:
The release is available on:

GitHub release
v5.31.0

Your semantic-release bot :package::rocket:


Fix release HAL crate workflow

Summary
Installing lazy_static no longer works as a workaround to force a refresh of the Cargo registry. Replace with cargo update instead.

I sneaked in a small fix to the pygamer nightly builds.


[FEATURE] Support for readonly flag in opensearch_snapshot_repository resource

Is your feature request related to a problem?
During the migration of an OpenSearch cluster, preventing write operations on the destination OpenSearch can avoid accidents.
What solution would you like?
I hope readonly flag in opensearch_snapshot_repository resource like below.
# Create a snapshot repository
resource "opensearch_snapshot_repository" "repo" {
  name = "es-index-backups"
  type = "s3"
  settings = {
    bucket   = "es-index-backups"
    region   = "us-east-1"
    role_arn = "arn:aws:iam::123456789012:role/MyRole"
    readonly = true
  }
}

What alternatives have you considered?
A clear and concise description of any alternative solutions or features you've considered.
Do you have any additional context?

API docs: https://opensearch.org/docs/1.0/opensearch/snapshot-restore/

[Triage]
Hey @shmokmt I assume you are looking for the following when readonly = true is added
  "index_settings": {
    "index.blocks.read_only": false
  },

This snapshot feature was added from PR https://github.com/opensearch-project/terraform-provider-opensearch/pull/125, can you please take a look and see if you can contribute ?
Thanks
Adding @duboisph @rblcoder @bbarani

This request is regarding the opensearch_snapshot_repository registering, not Snapshot Management feature added in #125.
That said, looking at the opensearch_snapshot_repository code, this should already work out of the box. The settings map is passed completely to the API.
You can use all settings as allowed by the API, docs: https://opensearch.org/docs/latest/api-reference/snapshots/create-repository/
@shmokmt Have you tried this and isn't it working?

I'm sorry. It seems I was mistaken.
It worked fine.


Updating staging and prod registries

Closes #91
Signed-off-by: Chris Hein<EMAIL_ADDRESS>

/assign @vincepri


kubric dataset & real world dataset

Hi! Thank you for your excellent work on this project. I've noticed the generated Kubric-based dataset featured on your project website, and I was wondering if you could please release this dataset or the generation script for Kubric? Additionally, could you please update the code for the real-world dataset?
Thank you for your time and assistance.

Hi authors, just wondering if you have any plans in releasing the kubric dataset and the real-world dataset presented in the paper. Cheers.


move health API to a separate class

currently health API is part of the ServiceRegisterApi class.

[ ] Move it to another class called AdminServicesApi class.
[ ] Rename the method from health to healthCheck

I will take this.


envoy_reloadable_features_validate_upstream_headers deprecation

Your change commit 31e1cb08409edfe6c8db5daf51ce302a92e9b322 (http: add support for validating upstream header k) introduced a runtime guarded feature. It has been 6 months since the new code has been exercised by default, so it's time to remove the old code path. This issue tracks source code cleanup so we don't forget.

ping!


Migrated Objective C library to C batch API

This closes #1345. A later pull request should modify the higher-level interface to make better use of batches.

I think I addressed all of the comments so far

\o/

I changed the startBatch wrapper API to match the most recent comments.

This is a prerequisite for #1397 (Making this comment so that you can see the status of this PR from that one).

I've noticed that some of the comments I add on the diff view don't show up as part of this "conversation view". And then when a new commit is pushed they vanish from the diff view :(

There are still a couple of comments in the diff view that don't show up here. And one in the conversation that's collapsed right between commits fe2c0c6 and 33655f9 in this view.

OK, my most recent changes should have resolved the outstanding comments. Including the one I found in the diff that I didn't see in this view.

There's also the pending comment to change ivar names of the GRPCOp classes from _lower_underscore to _lowerCamel.

Resolved pending comments.

It all LGTM now except for two minor things:

Add yourself to the list of authors of the GRPCClient library in gRPC.podspec :)
I cloned the branch to run the tests, but in the XCode project the tests classes appear but don't seem to be recognized as tests (??). Does that happen to you?

All done. And the tests appear just fine for me.

No need to wait for Travis, as we don't have tests there yet and this only touches ObjC. Merging.


[docs]Updates npwg-demo-1 files to track latest changes

Almost files in npwg-demo-1 are fixed but I found several files
still use old definitons. This changes fix the stuff.

Pull Request Test Coverage Report for Build 356

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 45.0%

Totals

Change from base Build 354:
0.0%

Covered Lines:
351

Relevant Lines:
780

💛  - Coveralls


Allow RHTAP admins to view/edit appliedclusterresourcequota

Also move existing clusterresourcequotas under the right api group.

/lgtm


Support loading string from ResX

Could we support some one that can be used for i18n?
Such as System.ComponentModel.DataAnnotations.DisplayAttribute.ResourceType
When I make an i18n app using ResX, I got an error CS0182.
[CliCommand(Description = Resource.CommandDescription)]
internal sealed class Command
{
    // ...
}

FYI, this feature is now ready in v1.6.9:

Added support for localizing commands, options and arguments.
You can now specify a nameof operator expression with a resource property (generated by resx) in the attribute's argument (for string types only)
and the source generator will smartly use the resource property accessor as the value of the argument so that it can localize at runtime.
If the property in the nameof operator expression does not point to a resource property, then the name of that property will be used as usual.

[CliCommand(Description = nameof(TestResources.CommandDescription))]
internal class LocalizedCliCommand
{
    [CliOption(Description = nameof(TestResources.OptionDescription))]
    public string Option1 { get; set; } = "DefaultForOption1";

    [CliArgument(Description = nameof(TestResources.ArgumentDescription))]
    public string Argument1 { get; set; }

    public void Run()
    {
        Console.WriteLine($@"Handler for '{GetType().FullName}' is run:");
        Console.WriteLine($@"Value for {nameof(Option1)} property is '{Option1}'");
        Console.WriteLine($@"Value for {nameof(Argument1)} property is '{Argument1}'");
        Console.WriteLine();
    }
}


num_req_outstanding might overflow in tlul_socket_1n

Suppose a host has one outstanding request to some peripheral. In that case, the host's 1:N socket will have its num_req_outstanding set to "1" and dev_select_outstanding pointing to the appropriate device. The device responds by raising the d_valid signal, which is considered accepted if the host's d_ready signal is also HIGH. If the response is accepted, num_req_outstanding decrements to "0", and assuming the host is not making any new requests, dev_select_outstanding retains its previous value.
If the peripheral continues holding the d_valid signal HIGH, a new "response" might be registered by the socket if the d_ready signal is still HIGH (as is always the case with e.g. the main core). This is because the response routing logic only takes the dev_select_outstanding signal into account:
  always_comb begin
    tl_t_p = tl_u_i[N];
    for (int idx = 0 ; idx < N ; idx++) begin
      if (dev_select_outstanding == NWD'(idx)) tl_t_p = tl_u_i[idx];
    end
  end

This extra response will then get registered as accepted and gets forwarded to the host. More importantly, the num_req_outstanding gets decremented again resulting in an overflow, which then raises the hold_all_requests signal, essentially blocking the host from making any further requests to the crossbar.
I have managed to create this scenario by adding a corrupted peripheral to the system for the purposes of my research:

Even though this behavior is not anticipated from any of the peripheral devices during normal operation, I think a small change of the response routing logic code would add to the security of the system, like for example:
  always_comb begin
    tl_t_p = tl_u_i[N];
    for (int idx = 0 ; idx < N ; idx++) begin
      if (dev_select_outstanding == NWD'(idx) && num_req_outstanding != '0) tl_t_p = tl_u_i[idx];
    end
  end

that's a reasonable change although I'm not quite sure what security this would add to the system. Are you worried about a denial of service..?

The attack scenario, even with the changes, cannot be prevented in my opinion. As d_valid somehow was corrupted already, the transactions are not paired anymore.
The list goes on and on to completely block this attack. I think it would be simpler to rely on Watchdog timer in this case as the crossbar hangs.

yeah i tend to agree with @eunchan. The denial of service of type attacks are not something we really cover.
Because someone could attack the clock, the processor, the memory etc and the results would all be the same.
Still I think this particular enhancement is not cumbersome, i could just absorb it.

@msfschaffner

This is not something we are going to address for M3 since it is a guard against something that is not really in scope.
The thread is acknowledged, but we will probably want to find a more holistic ways to tackle issues like these.

Triaged for tlul. Keeping https://github.com/lowRISC/opentitan/labels/Type%3AFutureRelease as per the last comment.

I am closing this issue as not planned as it adds marginal defense against something that is not in scope anyway (denial of service on bus).


Add PrivacyInfo.xcprivacy

This PR adds a privacy manfiest (PrivacyInfo.xcprivacy), Apple has not been mentioned here, but this SDK is using the User Defaults API
Additionally, since the swift-tools-version in Package.swift is set to 5.0, it was not possible to add resources to the target. Functions that include resources available from version 5.3, so I have updated it to 5.3. (swift-package-manager github)
resolve #299 #300

@dp221125 Thank you for your pull request! Since NSPrivacyCollectedDataTypes is not included in PrivacyInfo, will there be no error when generating a Privacy Report?

@dp221125 Thank you for your pull request! Since NSPrivacyCollectedDataTypes is not included in PrivacyInfo, will there be no error when generating a Privacy Report?

I'm sorry for the late reply. I have fixed it. 4b59143

@sunshinejr could you review PR, please?

@sunshinejr请您查看一下 PR 好吗？

@sunshinejr Please merge pr


boards: m5stack: cores3: Add TF-Card slot configuration

Adding TF-Card slot configuration.
Adding SPI CS configurations. Reserving id-0 for TFT.

@sylvioalves @marekmatej
Thank you for advance your cooperation with this as well.


chore: sha

Current dependencies on/for this PR:

main

PR #127   👈

This comment was auto-generated by Graphite.


Official Web-site crashed

@ganelson https://inform7.com/ is 404...

I think this was a certificate problem, and has now been fixed.


Chart 3.12.5 is not published.

❯ helm search repo atlantis
NAME                	CHART VERSION	APP VERSION	DESCRIPTION
runatlantis/atlantis	3.12.4       	v0.14.0    	A Helm chart for Atlantis https://www.runatlant...

Please publish this.  I'd like to upgrade our workflow to use Terraform version 0.13.5, since 0.14.0 already went GA.

It looks like a GitHub Actions pipeline failure.
See https://github.com/runatlantis/helm-charts/runs/1311869164?check_suite_focus=true

could please fix the 404
output from the tf apply by using the helm chart provider
`Error: chart "atlantis" version "3.12.5" not found in https://runatlantis.github.io/helm-charts repository
on ../../modules/atlantis/main.tf line 39, in resource "helm_release" "atlantis":
39: resource "helm_release" "atlantis" {`

This is still not fixed.
runatlantis/atlantis                              	3.12.4       	v0.14.0                     	A Helm chart for Atlantis https://www.runatlant...

This is still not fixed.
runatlantis/atlantis                              	3.12.4       	v0.14.0                     	A Helm chart for Atlantis https://www.runatlant...

You can work around it by downloading the release and replacing
runatlantis/atlantis with the local path of the tgz archive, in helm
commands.
But I am still hoping we’ll get a fix soon.
On Thu, Jan 7, 2021 at 8:31 AM Justin Seiser<EMAIL_ADDRESS>wrote:

This is still not fixed.
runatlantis/atlantis 3.12.4 v0.14.0 A Helm chart for Atlantis
https://www.runatlant...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/runatlantis/helm-charts/issues/17#issuecomment-756225593,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHFQ2J4WDHGYP6IIHNKXGDSYXOVZANCNFSM4UK5T4FQ
.

--
—
Gabor

You can work around it by downloading the release and replacing
runatlantis/atlantis with the local path of the tgz archive, in helm
commands.
But I am still hoping we’ll get a fix soon.
On Thu, Jan 7, 2021 at 8:31 AM Justin Seiser<EMAIL_ADDRESS>wrote:

This is still not fixed.
runatlantis/atlantis 3.12.4 v0.14.0 A Helm chart for Atlantis
https://www.runatlant...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/runatlantis/helm-charts/issues/17#issuecomment-756225593,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHFQ2J4WDHGYP6IIHNKXGDSYXOVZANCNFSM4UK5T4FQ
.

--
—
Gabor

This has been fixed! Sorry for the delay. The publish action looks to be working now.

This has been fixed! Sorry for the delay. The publish action looks to be working now.


Fix Race Conditions

Problem: If user types to fast in the search, the app just crashes.
The "fetch as you type" is the UX I want, but there needs some delay added, since the hooks are setup to fetch with each keystroke. This is apparently a common problem, and I found some blog posts:

https://sebastienlorber.com/handling-api-request-race-conditions-in-react
https://dev.to/nas5w/avoiding-race-conditions-when-fetching-data-with-react-hooks-4pi9

This has been fixed with https://github.com/gregrickaby/nextjs-reddit/commit/9d68603b14527e58d68ba6364dbe1d46f04844e1
To help simplify things (for me, I'm still learning React Hooks) I dumped the useForm and useFetch. I read some blog posts, and found this https://usehooks.com/useDebounce/ which led me to https://codesandbox.io/s/711r1zmq50?file=/src/index.js:1135-1163


[BUG] prometheus servicemonitor selector mismatch with service labels

Describe the bug
Current servicemonitor selector from botkube helm chart does not match with labels of created botkube kubernetes service:
servicemonitor selector block:
  selector:
    matchLabels:
      app.kubernetes.io/name: {{ include "botkube.name" . }}
      helm.sh/chart: {{ include "botkube.chart" . }}
      app.kubernetes.io/instance: {{ .Release.Name }}
      app.kubernetes.io/managed-by: {{ .Release.Service }}
      component: controller
      app: botkube

service labels block:
  labels:
    app.kubernetes.io/name: {{ include "botkube.name" . }}
    helm.sh/chart: {{ include "botkube.chart" . }}
    app.kubernetes.io/instance: {{ .Release.Name }}
    app.kubernetes.io/managed-by: {{ .Release.Service }}
    app: botkube

As you see, extra label component: controller makes service unmatchable for servicemonitor
To Reproduce
Steps to reproduce the behavior:

install botkube chart v0.11.0 from https://infracloudio.github.io/charts with .Values.serviceMonitor.enabled: true
go to your prometheus/targets
see, that there are 0/0 up for botkube targets

Expected behavior
With correct servicemonitor selector there should be actual count of botkube pods

Thanks @asurahack, I stumbled upon this embarrassing bug recently, so I prepared a fix in #616. It will be merged soon 👍


Fix Halo2 verification

Depends on #1053
Fixes the bug reproduced in #1051 (which wasn't a soundness bug after all).

test failing


Unchecked cast warning

Hi Jawher,
Here is my modest contribution. Mostly to remove unchecked cast warning.
(flat)map FTW !
Cheers,
Lucien

Thanks a lot Lucien for your contribution.


Studio creates a file called String which is of 190+ MB

Description:

Once you run few simulations etc. on the editor, you see a new file gets created under wso2/editor/String which is of 190+ MB and not getting cleaned up.
du -hs wso2/editor/String
190M	wso2/editor/String
Suggested Labels:

Suggested Assignees:

Affected Product Version:
beta5
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

What kind of simulation are you running here ?

Random data, feed simulation.

This could not be reproduced. Reducing the severity because editor does not go for production . will check regarding this.

Cannot be re-produced in SP-4.3.0 either. Hence, suggesting to close this issue.


New cell error "Cannot read property 'charAt' of undefined at"

Describe the bug
When I click the plus sign to create a new cell in a workbook I get an error. The new cell isn't created and I see this error message in a larger than usual font.
The error message survives a restart of the kernel.
The error is gone after I restart VSCode.
The error is also gone after I save / close the file and re-open it.
Error Text follows:
TypeError: Cannot read property 'charAt' of undefined at
in t
in div
in t
in div
in t
in div
in t
in div
in div
in div
in div
in div
in div
in t
in h
in t
in div
in div
in div
in t
in main
in div
in t
in h
in f

Please complete the following:
Which version of .NET Interactive are you using? There are a few ways to find this out:

In a notebook, run the #!about magic command.
Version: 1.0.142601+586ab2f29818edea7c19c383874977fe5d0391fc
Build date: 2020-09-09T14:18:07.6932204Z

OS

[X ] Windows 10

Browser

[X - default ] Chrome
[x] Edge
[x] Firefox
[ ] Safari

Frontend

[ ] Jupyter Notebook
[ ] Jupyter Lab
[ ] nteract
[X ] Visual Studio Code
Version: 1.49.0 (user setup)
Commit: e790b931385d72cf5669fcefc51cdf65990efa5d
Date: 2020-09-10T13:22:08.892Z
Electron: 9.2.1
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.18362
[ ] Other (please specify)

Screenshots
If applicable, add screenshots to help explain your problem.

This is coming from the Python extension, not .NET Interactive. If you have both extensions installed,  you can open the notebook in .NET Interactive using the command palette command .NET Interactive: Open notebook.


Confusing error message when trying to use evaluated on an input key

Given a build.sbt of
version := "0.0.1"

org.scalastyle.sbt.ScalastylePlugin.Settings

val t = taskKey[Unit]("task")

t := {
  //  org.scalastyle.sbt.PluginKeys.scalastyle is an input key
  val ignored = org.scalastyle.sbt.PluginKeys.scalastyle.evaluated
  // should be
  // val ignored = org.scalastyle.sbt.PluginKeys.scalastyle.toTask("").value
}

and project/plugins.sbt of
resolvers += "sonatype-releases" at "https://oss.sonatype.org/content/repositories/releases/"

addSbtPlugin("org.scalastyle" %% "scalastyle-sbt-plugin" % "0.4.0")

sbt.version = 0.13.1
When reloading sbt, the following message is given:
C:\temp\ff\build.sbt:9: error: `value` can only be called on an input task
within a task definition macro, such as := or Def.inputTask.
  val ignored = org.scalastyle.sbt.PluginKeys.scalastyle.evaluated
                                                         ^
[error] Type error in expression
Project loading failed: (r)etry, (q)uit, (l)ast, or (i)gnore?

This is confusing, because it refers to value, not evaluated. What would be nice would be to say something along the lines of 'should you be using toTask()'

Has anyone figured any workaround?

Still an issue in sbt 1.x.x.

Confirmed it still happens in SBT 0.7.1


Ubuntu 22.04 - Python SDK raises ModuleNotFoundError: No module named 'h2'

Issue
When running a nitric python application on Ubuntu 22.04 with Python 3.11 using nitric start a ModuleNotFoundError is raised.

This is an issue with the version of Python that gets installed from the standard repositories on Ubuntu 22.04. When installing python with sudo apt install python3.11 a release candidate version is installed 3.11.0rc1. Installing a stable release resolves the error. Some details on this issue are available here
Solution
Install a stable version of Python 3.11+, there are several ways to do this, such as from alternate repositories:
sudo add-apt-repository ppa:deadsnakes/ppa
sudo apt install python3.11

When installing the project's dependencies you will need to ensure a compatible version of python is used. The default python version on Ubuntu 22.04 is 3.10, which isn't compatible with the nitric sdk. You can specify a version during the depedency install step like so:
pipenv install --dev --python 3.11

Resolution steps included above.


onnx model precision

We transferred the model to onnx and found that the accuracy was not consistent with the original model

Hi we also encountered some problems when transferring to onnx.
We found it a dynamic model according to the code, which means the input is a DYNAMIC dimension  and leads to the wrong format of onnx.
I would like to ask how you solved this problem ?


multi repo publishing from gh-pages has wrong base url

Site is here :  https://dbjdbj.github.io/experimental/ ...
pressing F12 shows that obviously base url is wrong
this is wrong --> https://dbjdbj.github.io/
Am I doing anything wrong? In my config.toml I have:
    baseURL = "https://dbjdbj.github.io/experimental/"

my "build command" is :
      hugo --cleanDestinationDir --destination "d:/devl/public" --environment PRODUCTION --gc

Please advise ...

changing the base URL to "/" .. (again) obviously solved the issue ...


WIP python27Packages.cherrypy: fix build

Motivation for this change
Due to the recent update of python 2.7 from 2.7.16 to 2.7.17 (see 05dfc5063928030483ad5f5e468d3b9ec8632a1d) the StaticTest::test_null_bytes test case started failing on 2.7 too.
This is because additional validation has been backported to httplib in PR 13315 in https://github.com/python/cpython to to address a security issue.
(cherry picked from commit 1295a8e9feb3f7f99f53c93234a680c18eca8dda)
Note: This is currently not building due to some other issue involving pytest-black.
Backport: https://github.com/NixOS/nixpkgs/pull/73920

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

Seems like maybe the world has just moved on...
black requires python3.6 (brought in via portend), which is fixable by only including pytest-black for newer versions, but also webtest fails due to pastedeploy not working for 2.7 anymore.
@FRidh: I think it might be worth considering to just ditch cherrypy for 2.7 alltogether, as 2.7 will be EOL by the time of next release.

@d-goldin that's up to the maintainer of the expression. I'm all for it.

@FRidh: Alright, then I'll make a PR for removal and let's see what the maintainer says. In any case, we can look at the fix for 19.09 in isolation then for the time being.

Closing in favour of deprecation PR.


feat: Upgraded to use apline base images

Description:
There are many vulnerabilities present with the debain images and currently the latest debain image with these fixes doesn't have any target date. Also alpine images are very light weight and have regular updates to vulnerability fixes.
I raised a ticket for the same - https://github.com/IzakMarais/reporter/issues/284
This PR is having the changes i did to rebuild reporter with alpine image. Also did run the scan and all the high vulnerabilities are no more present.
Please let me know if you have any suggestions in this PR, we can work together to get this fixed.

you were about 2h faster :)

I don't think the image on docker hub is not yet updated. Would you please do that? @IzakMarais


IE 11 raises error in jBone.contains function

jBone.contains = function(container, contained) {
var result;
container.reverse().some(function(el) {
if (el.contains(contained)) {      //<------ error: Object doesn't support property or method 'contains'
return result = el;
}
});
return result;
};

sorry it's my bug.
Please delete the issue

Hi @budarin, ok, thanks.

Hello, actually I have the same issue in IE11.
What was the solution to fix it?
version jBone v1.0.28

Sorry, I don't remember the context but it was not the library's problem

Hi @alexandrbig, could you show example of usage .contains.

http://screencast.com/t/qS1SDLUa
It seems I've added $(document).on(...) and it calls dispatch method.

It seems that my issue is not the same, but similar
I've got error in line 399:

(el !== e.target && el.contains(e.target))

Hello, the problem is in absense of contains method for document in IE but document.body has this method.
So $(document).on(...) may not work in IE but $(document.body).on(...) will do.


Create vault-batch-token.py

added test

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


KeyError: 'llama' in /recover_model_weights.py

[continuation of #69 ]
When trying recover_model_weights.py on llama-7b-hf as cloned from https://huggingface.co/decapoda-research/llama-7b-hf (following advice by lxuechen to look for copies of llama-7b on huggingface spaces), we (still) get the error
Downloading sft10k
Traceback (most recent call last):
  File "/p/projects/ou/labs/gane/rlhf/git/alpaca_farm-collective/pretrained_models/recover_model_weights.py", line 112, in <module>
    model_tuned, tokenizer_tuned = load_weight_diff(hf_hub_name, is_reward_model, args.device, args.path_to_sft10k)
  File "/p/projects/ou/labs/gane/rlhf/git/alpaca_farm-collective/pretrained_models/recover_model_weights.py", line 48, in load_weight_diff
    model_tuned = transformers.AutoModelForCausalLM.from_pretrained(
  File "/p/projects/ou/labs/gane/rlhf/envs/alpaca-env/lib/python3.10/site-packages/transformers/models/auto/auto_factory.py", line 434, in from_pretrained
    config, kwargs = AutoConfig.from_pretrained(
  File "/p/projects/ou/labs/gane/rlhf/envs/alpaca-env/lib/python3.10/site-packages/transformers/models/auto/configuration_auto.py", line 873, in from_pretrained
    config_class = CONFIG_MAPPING[config_dict["model_type"]]
  File "/p/projects/ou/labs/gane/rlhf/envs/alpaca-env/lib/python3.10/site-packages/transformers/models/auto/configuration_auto.py", line 579, in __getitem__
    raise KeyError(key)
KeyError: 'llama'

Can you point us to any version of llama that should work, so that we can rule out that it is a problem with the model version?

Which version of transformers was used when running the weights conversion script? We converted the raw llama weights into huggingface format using transformers==4.29.2 (as documented in the README)

4.26.0, as required per requirements.txt. If I upgrade to 4.32.0, I get an even faster error:
Traceback (most recent call last):
  File "/p/projects/ou/labs/gane/rlhf/git/alpaca_farm-collective/pretrained_models/recover_model_weights.py", line 11, in <module>
    from alpaca_farm.models.reward_model import RewardConfig, RewardModel
  File "/p/projects/ou/labs/gane/rlhf/envs/alpaca-env/lib/python3.10/site-packages/alpaca_farm/models/reward_model.py", line 20, in <module>
    from .. import common
  File "/p/projects/ou/labs/gane/rlhf/envs/alpaca-env/lib/python3.10/site-packages/alpaca_farm/common.py", line 32, in <module>
    from transformers.trainer import WEIGHTS_NAME, is_deepspeed_zero3_enabled
ImportError: cannot import name 'is_deepspeed_zero3_enabled' from 'transformers.trainer' (/p/projects/ou/labs/gane/rlhf/envs/alpaca-env/lib/python3.10/site-packages/transformers/trainer.py)


Show user guidance to configuration file if file not exists

Closes #17

Thank you for your contribution! I'm going to look at this weekend :)

Thank you for your contribution! I'm going to look at this weekend :)


Review the not implemented tests in test_protocol_tables.py --> TestNetworkInstancesProtocolTables class

Test not implemented

ni
protocol_tables
import-policy

Is better if we divide this in 2 rpcs

Fixed up. @j-suarezp

Fixed in #94


About creat_gridpoint.m

Hi,Euan
I encountered a problem about "unrecognized function or variable 'dread‘ "，I found it’s not a matlab built-in functions, and in your scripts, I couldn't found a self-definition function.
My goal is that I have a surface from your guideline now, I only want to use your script to compute normals, so I only run your creat_gridpoint.m , but I got this error.
Can you help me ? thank you very much!

Hi, dread is a Dynamo function, it sounds like Dynamo is not loaded correctly. Usually running a dynamo_activate.m script will load Dynamo for you. If it still doesn't work it will be a Dynamo installation issue

Yes, you're right! I'll try it later.


[feat] Add Dolphin mode

Is your feature request related to a problem? Please describe.
No.
Describe the solution you'd like
A dolphin mod that allows the player to just consistently "swim like a dolphin" across the top of the water
Describe alternatives you've considered
none
Additional context
none

Added in commit 90aec07a


Adding CI for conda

One problem, we only have an osx package for symudev. Maybe its better to wait for the packages for all plateforms for symudev before merging this.

One problem, we only have an osx package for symudev. Maybe its better to wait for the packages for all plateforms for symudev before merging this.


DPL-098 - As Scott I need to do a Lysate plate consolidation for the Combined LCM DNA / RNA-seq pipeline [ToR 12]

User story
As Scott I need to do a Lysate plate consolidation for the Combined LCM DNA / RNA-seq pipeline
Who are the primary contacts for this story
Scott / Steve
Acceptance criteria
To be considered successful the solution must allow:

[ ] Scan up to 4 source plates
[ ] Transfer all the samples from sources into 2 plates (DNA and RNA) as described in https://docs.google.com/document/d/1s5AvMllBHZzMpx85simlCivBSmnfctJ5cwl_AXJFF5I/edit
[ ] Validate wireframe with users

Dependencies
This story is blocked by the following dependencies:

#<issue_no.>
sanger/#<issue_no.>

References
This story has a non-blocking relationship with:

#<issue_no.>
sanger/#<issue_no.>

Additional context
Add any other context or screenshots about the feature request here.

Plates coming into the process are of laser-capture microdissected tissue samples in lysis buffer in 96-well PCR plates. The nature of this step means that at most 24 wells of the plate will be filled due to the length of time it takes to capture a tissue sample in each well; more samples than this may lead to samples degrading. We are enforcing the same sample layout on all plates going into the process; samples will be present in wells A1-H3. So the very first thing that needs to happen before anything else is a consolidation of up to four of these partially filled plates into a single 96 well plate.

Combined.LCM.DNA-.RNA-seq.pdf

Details in the google doc:
https://docs.google.com/document/d/1s5AvMllBHZzMpx85simlCivBSmnfctJ5cwl_AXJFF5I/edit

Unfortunately the 'simpler' behaviour means we can't just re-use the existing behaviour. Might be worth double checking if the fixed layout stamping is a requirement, or an attempt to simplify things for our benefit. Mostly I'm concerned about what the correct behaviour is in the event of plates with <24 wells. (Either due to failures or insufficient material)
If they are happy with the existing consolidation behaviour, then this is just configuration.

Background:
Existing consolidation plate config:
# This configuration file is for the Bespoke Chromium Aggregation pipeline.
# LBC Stock plates are submitted to the Aggregation pipeline to be combined
# through LBC Aggregate plates into LBC Cherrypick plates.
# LBC Cherrypick plates can be submitted to various Chromium 10X automated
# pipelines for library preparation.
---
LBC Aggregate:
  :asset_type: plate
  :creator_class: LabwareCreators::TenStamp
  :file_links: []
LBC Cherrypick:
  :asset_type: plate
  :creator_class: LabwareCreators::TenStamp
  :state_changer_class: StateChangers::AutomaticPlateStateChanger
  :work_completion_request_type: 'limber_bespoke_aggregation'
  :file_links:
  - name: 'Download Hamilton Aggregate Cherrypick CSV'
    id: 'hamilton_aggregate_cherrypick'
  - name: Download Concentration (ng/ul) CSV
    id: concentrations_ngul

LabwareCreators::TenStamp is a subclass of LabwareCreators::MultiStamp The key new behaviour seems to be the self.transfers_layout = 'sequential' configuration, which determines how source plats map to the destination. We'll need a new creator class with a different transfers_layout setting, and a new layout configured in app/javascript/shared/transfersLayouts.js

The volume behaviour of the LabwareCreators::TenStamp class probably isn't strictly needed here, as we aren't generating CSVs, but it shouldn't harm.

Also, see: https://github.com/sanger/limber/issues/787 As there's the possibility we'll be needing to register two shildren in this plate creator.

Communication chain:

Hello Scott and SSRs (Steve CCed),
GPL-098 and GPL-099 as part of delivering the LBM DNA/RNA pipelines have been prioritised, but I’d like to clarify the requirements slightly:
The stories are:
DPL-098 - As Scott I need to do a Lysate plate consolidation for the Combined LCM DNA / RNA-seq pipeline
https://github.com/sanger/limber/issues/786
DPL-099 As Scott after Lysate Plate Consolidation I need to split of the total volume of lysate in each well (50ul) into two equal 25ul fractions on different plates
https://github.com/sanger/limber/issues/787
From the process map it appears that the consolidation onto the ‘Cherrypick?’ plate, and the subsequent stamp out onto the RNAseq cherrypick and DNAseq cherrypick plates happens on the bravo as a single step.

 Scott: Is there a desire to track the intermediate Cherrypick plate, or could this remain untracked? Generally if a plate never leaves the robot, and doesn’t need to be stored, there isn’t a huge amount of value in tracking it.

a.      Tracked workflow
Scan in a source plate > Click ‘Create Cherrypick? Plate’ > Scan in additional source plates > Click ‘Create  RNAseq cherrypick plate’ > Click ‘RDAseq cherrypick’ > Robot verify with all 7 plates
b.      Untracked workflow
Scan in a source plate > Click ‘Create RNAseq/DNAseqPlates’ > Scan in additional source plates > Robot verify with 6 plates

 SSRs: As there are effectively two different libraries being made, we need a request for both.

Currently this would require two separate submissions for the untracked workflow, one for each library. Would this be acceptable?
For the tracked pipeline we run into more significant issues upfront, but it will probably be worth having a meeting to discuss that further if we find we need to go that route. Which SSRs are best placed to be involved in this discussion.
Best wishes,
James

SSR response:

HI James,
From an SSR point of view I don’t think this will be a problem – will this be able to work like GnT sequencing where we are able to do both submissions within the same document just for a different library type ?
As an SSR contact point I’m more than happy to help as I am sure CASM will be a big user of this line , so it would make sense to help the development.
Please let me know if I can be any further help
Best Wishes
Maisie

Scott response:

Hi James,
Yes, as you correctly determined the ‘Cherrypick?’ plate never leaves the robot, so if it’s easier to create the untracked workflow you mention and generate the DNAseq & RNAseq plates directly that’s absolutely fine with me.
Thanks,
Scott G.

The main advantages to not tracking the 'Cherrypick?' plate is it makes it much easier to handle the two library requests. It places the branch right at the beginning of the pipeline, meaning that there is no need for an aggregation request, and also that we will not need to change the submission builder to handle branching submissions. The main reason for proposing the aggregation request, is otherwise the aliquots on the cherrypick? plate would be associated with two requests. The alternative, a migration to a many-many relationship, would be a challenge, and would have performance considerations due to the additional joins and insertions.
The untracked approach does mean that you'd probably want to spit out both the RNA and DNA plates at this stage, otherwise there is repetition when scanning in the layouts, and it would be easy to make a mistake and not use the same layout each time.

** Tasks **

[ ] Create wireframe about the process with SS and Limber about the consolidation + 2 plates creation (DNA and RNA)
[ ] Create transfer layout as described in https://docs.google.com/document/d/1s5AvMllBHZzMpx85simlCivBSmnfctJ5cwl_AXJFF5I/edit
[ ] Create plate creator to handle creation of both destination plates (DNA and RNA) in one click

Possibly it could require:

2 Library types (DNA and RNA)
1 Submission Template (they will create 2 submissions in each source, one for DNA and one for RNA)
1 Request Type

Reply to wireframe from Scott G.
-          What is the name for the new 2 library types required (for DNA and RNA)
Combined LCM DNA, Combined LCM RNA
-          What is the name for the submission template? (we defined temporarily Limber-LCM DNA / RNA-seq)]
Limber-Combined LCM DNAseq/RNAseq
-          What is the product catalogue?
Not sure what is meant by this (sorry)
-          What is the product line?
o   Options : Illumina A, IlluminaB, IlluminaC, Illumina http, Bespoke
Should be able to go down the same line as the current LCM pipeline?
-          What is the name of the plate purposes? We defined this:
o   Lysate LCA
o   Lysate DNAseq cherrypick
o   Lysate RNAseq cherrypick
I don’t know if there are any particular rules regarding plate names, but the following makes more sense to me:
Lysate LCM
Consolidated lysate DNAseq
Consolidated lysate RNAseq

One final thing I might add based on looking at the wireframe is juts to clarify that the lysate consolidation is happening on a Bravo, not a Hamilton as shown in the wireframe.


dashing-js trying to install old<EMAIL_ADDRESS>

gyp ERR! build error
gyp ERR! stack Error: `C:\Program Files (x86)\MSBuild\14.0\bin\msbuild.exe` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onExit (C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\lib\build.js:258:23)
gyp ERR! stack     at emitTwo (events.js:126:13)
gyp ERR! stack     at ChildProcess.emit (events.js:214:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:198:12)
gyp ERR! System Windows_NT 10.0.15063
gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\node_modules\\node-gyp\\bin\\node-gyp.js" "rebuild"
gyp ERR! cwd C:\Users\sunka04\AppData\Roaming\npm\node_modules\dashing-js\node_modules\node-sass
gyp ERR! node -v v8.9.4
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok
Build failed
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node build.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\sunka04\AppData\Roaming\npm-cache\_logs\2018-02-17T19_18_14_829Z-debug.log

PS C:\Users\sunka04\Documents\Source> npm --version
5.6.0
PS C:\Users\sunka04\Documents\Source> node --version
v8.9.4

Facing the same issue, is anyone looking into this?

Same thing happens for me. Any advice would be appreciated

Same issue here - seems like a blocker to actually this library at the moment.

Thanks a lot, simbro! You saved me big time!
Edit:
simbro deleted his fork

Do you have a fork? I am facing this issue


Automated cherry pick of #5782

Cherry pick of #5782 on release-5.25.

#5782: allow pasting tables into edit post, keep

/cc  @Willyfrog

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: 5f60eff6d6d51ae1989d29dde8ba68df699b7b80


Request: Docker image - Text editor

Is your feature request related to a problem? Please describe.
Not a problem but to simplify editing text files inside the container a minimal text editor could be included.
The existing options of text manipulation is limited to juggling various posix/unix-like utilities, namely: sed tr awk cat echo nl sort cut head tail grep find more while there's more my experience is already stretched.
Describe the solution you'd like
Including a minimal text editor in the default docker image.
I'd suggest vi/vim or nano/pico as it's common for headless systems.
Describe alternatives you've considered
I've considered alternatives but most require learning curves or are niche, and not included in common repos so I'm appealing broadly.
Additional context
(This is merely a suggestion, and may be closed)

The base image used is ubuntu:20.04, so a text editor should already be included. Are you sure you are attaching to the container properly?

By attaching do you mean with docker exec?
I'm using docker exec -it arm-ripper /bin/sh with the prebuilt dockerhub image fb19fb2e0514.
I did a fresh setup with podman run using the template.
Inside the container apt list --installed isn't showing nano/vi/vim, and I didn't see either binaries inside /usr/bin/ or /usr/sbin/ as root or arm users.
(I'm using Fedora as my host system with dnf so there's no confusion about package management.)

Gotcha. I guess that image doesn't come with the full tooling. I'll make a PR to add an install for nano to the Dockerfile

Thanks!
For others who need editing currently without much experience, manually copying the file locally, and back from the host is another option.
Copy from container to host's pwd
docker cp arm-rippers:/location/of/file.conf ./file.conf
Copy from host to container's path
docker cp ./file.conf arm-rippers:/location/of/file.conf

Don't close this issue just yet! Standard practice is to close it when the PR is merged.

My mistake!

Closing as this change is now merged.
Ill need to find a github action to push the new changes to dockerhub
Thanks for your work on this!