deliveryservices/id/capacity gives Internal Server Error

I'm submitting a ...

[X] bug report
[ ] new feature / enhancement request
[ ] improvement request (usability, performance, tech debt, etc.)
[ ] other

Traffic Control components affected ...

[ ] CDN in a Box
[ ] Documentation
[ ] Grove
[ ] Traffic Control Client
[ ] Traffic Monitor
[X] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] unknown

Current behavior:

When making a GET request to /deliveryservices/{{id}}/capacity where the deliverservice id pertains to a delivery service that was recently created and has no server assignments, the response returned is:
HTTP/1.1 500 Internal Server Error
{
    "alerts": [
        {
            "text": "Internal Server Error",
            "level": "error"
        }
    ]
}

Expected / new behavior:

If there is no capacity that can be obtained it should return:
HTTP/1.1 200 OK
{
    "response": {
        "availablePercent": 0,
        "unavailablePercent": 0,
        "utilizedPercent": 0,
        "maintenancePercent": 0
    }
}

Minimal reproduction of the problem with instructions:

Anything else:

error.log shows: getting delivery service capacity: capacity was zero!

Is this a regression? How did Perl handle that situation?

Looks like it is a regression. The Perl returned:
HTTP/1.1 200 OK
{
    "response": {
        "availablePercent": 0,
        "unavailablePercent": 0,
        "utilizedPercent": 0,
        "maintenancePercent": 0
    }
}

@ocket8888 @mitchell852 Same issue was reported and resolved.
Issue Reported: https://github.com/apache/trafficcontrol/issues/4848
Solution: https://github.com/apache/trafficcontrol/pull/4892
This bug has been fixed and needs to be closed.

No, that was a different endpoint. That was /cdns/capacity, this is a similar bug in /deliveryservices/{{ID}}/capacity. I just looked at the code and it would seem the issue still exists.

@rimashah25 - remove all servers from ds then snapshot and see what the result of /deliveryservices/{{ID}}/capacity is...

Thanks @mitchell852.


Throws errors when binary file is in template folder

When a binary file such as a image file is located within the templates folder and addMany is used, it will throw a very cryptic error message as such:
[FAILED] addMany Lexical error on line 1. Unrecognized text.
  ☺ ☻ ►►      h♦  &
^

A code example can be found here:
https://github.com/crutchcorn/generate-me/tree/14370feb32d05f913387929c7ac1359c52ed3b1a
By running node index.js "Test" n y, you can see the behavior

addMany is for adding files that are processed as handlebars templates. Maybe we could use something like isBinaryFile to present a better error.

I think there's also room for a built-in copy action that would copy files without processing them as templateFiles.

Certainly a better error should be implamented. However, I'm wondering if we should even ignore binary files in the built-in copy action. I'd argue that having binary files copied without errors is the most logical step given the rest of the libraries' behavior but you would have a much better understanding of the expected behavior as it's your project.
If you say one way or another, I'll start on a PR that will fix this issue

released with plop v2.1.0


Balanced ChemVend Stock

About the PR
I rebalanced what comes in the ChemVend and the ChemVend refills to be closer to actual usage levels by the average chemist
Why / Balance
You do not need as much Radium as you do carbon. It's silly to get the same amounts of every chem when some are used for almost everything, and some have basically no use.
I'm not a Chemist main or anything so if some of these numbers look off to you, feel free to tweak them. But this is based on several shifts of my chem experience
Technical details
Just changed the numbers in a .yml, nothing fancy at all
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes
None, it's just a quantity swap.
Changelog
Rebalanced ChemVend stock for more accurate usage levels

iirc sugar is used a lot

My initial reaction is maintaining this will probably be annoying as new reagent/reactions get added.
Another thought is that this also means that the restock box will give quantities in line with this change iirc. which might be undesirable but meh.

iirc sugar is used a lot
I agree, +1'd it

My initial reaction is maintaining this will probably be annoying as new reagent/reactions get added.
Another thought is that this also means that the restock box will give quantities in line with this change iirc. which might be undesirable but meh.

Haven't seen new reagents in a long while, and this balance is definitely needed, should be easy to tweak if needed once we've got things close.

That is, roughly speaking, there are more of some substances, and less of some now?  There’s no point about radium, yes, chemists don’t need it in part, but botanists need it for mutagen and phalangimine.  I have explained what radium is for now because it is indicated in the example, I have not seen the numbers yet.


Filter Order was disrupted when user application extends the SPI of org.apache.dubbo.rpc.Filter

Environment

Dubbo version: 2.7.11
Operating System version: win10
Java version: oracle jdk 1.8.0_281

Steps to reproduce this issue

Define the SPI extension point of org.apache.dubbo.rpc.Filter

write the filter and dubbo service
(1)filter

(2)dubbo service

start the dubbo provider application

Expected Result
ExecuteLimitFilter -> MonitorFilter
(the picture below is sorted Filters with only dubbo Native Filler)

Actual Result
MonitorFilter -> ExecuteLimitFilter
Causes the concurring count to be inaccurate !

What actually happens?
the Filters were sorted in the TreeMap , using ActivateComparator. When the order of Flter is not set,  the traversal order will affect the sort results.

I think this should not affect it. If the order is not set, it means that the user actually doesn't care about the execution order of this filter.

I think this should not affect it. If the order is not set, it means that the user actually doesn't care about the execution order of this filter.

Of course,  if user's  extension filter  does't set order,  we  can  think of it  as the user doesn't care about the order of user filter's execution.
However, the default order of dubbo's own  filter，for example, ExecuteLimitFilter and MonitorFilter ,the order between them, is disrupted.
MonitorFilter should record  the number of concurres that actually execute the method, when the order is disrupted, it turned out to record the request concurres.  The number of concurrency in the monitoring data will be greater than the threshold. This will cause confusion to users, whether the concurrency settings of the service take effect.

Of course, if user's extension filter does't set order, we can think of it as the user doesn't care about the order of user filter's execution.
However, the default order of dubbo's own filter，for example, ExecuteLimitFilter and MonitorFilter ,the order between them, is disrupted.
MonitorFilter should record the number of concurres that actually execute the method, when the order is disrupted, it turned out to record the request concurres. The number of concurrency in the monitoring data will be greater than the threshold. This will cause confusion to users, whether the concurrency settings of the service take effect.

This may be a problem. That is, whether the filter that comes with the system is equivalent to the user-defined filter.

If they are equivalent, then since MonitorFilter and ExecuteLimitFilter do not define an order, they do not care about the order of execution.

@PromiseChan if the user-defined filter is inserted into the system-defined filter, will this affect the result?
My understanding is that the filter is in order to do some pre-filtering.
If the filter has a strong consistent relationship, it should be specified during configuration. If not, there is no need for the order of relationships, do you think so?

@PromiseChan if the user-defined filter is inserted into the system-defined filter, will this affect the result?
My understanding is that the filter is in order to do some pre-filtering.
If the filter has a strong consistent relationship, it should be specified during configuration. If not, there is no need for the order of relationships, do you think so?
Yes, i do. If users want an order between  filters, they should make configuration on filers. By the way, the result was not affected  with unordered pre-filtering filters. Just the meaning of monitoring data has changed.

ok

@PromiseChan if the user-defined filter is inserted into the system-defined filter, will this affect the result?
My understanding is that the filter is in order to do some pre-filtering.
If the filter has a strong consistent relationship, it should be specified during configuration. If not, there is no need for the order of relationships, do you think so?

Yes, i do. If users want an order between filters, they should make configuration on filers. By the way, the result was not affected with unordered pre-filtering filters. Just the meaning of monitoring data has changed.

The monitoring time you mentioned by monitor data has changed, right?

ok

@PromiseChan if the user-defined filter is inserted into the system-defined filter, will this affect the result?
My understanding is that the filter is in order to do some pre-filtering.
If the filter has a strong consistent relationship, it should be specified during configuration. If not, there is no need for the order of relationships, do you think so?

Yes, i do. If users want an order between filters, they should make configuration on filers. By the way, the result was not affected with unordered pre-filtering filters. Just the meaning of monitoring data has changed.

The monitoring time you mentioned by monitor data has changed, right?

monitoring time?  I  mean the method's concurrent number collected by  MonitorFilter is incorrect when the filter order is monitorFiler -> executeLimitFilter

ok

@PromiseChan if the user-defined filter is inserted into the system-defined filter, will this affect the result?
My understanding is that the filter is in order to do some pre-filtering.
If the filter has a strong consistent relationship, it should be specified during configuration. If not, there is no need for the order of relationships, do you think so?

Yes, i do. If users want an order between filters, they should make configuration on filers. By the way, the result was not affected with unordered pre-filtering filters. Just the meaning of monitoring data has changed.

The monitoring time you mentioned by monitor data has changed, right?

monitoring time? I mean the method's concurrent number collected by MonitorFilter is incorrect when the filter order is monitorFiler -> executeLimitFilter

First, I set the executes=10 , only system-defined filters, the recorded max method concurrent number is 10 .
Second, I set the executes=10  with some user-defined filters , the recorded max method concurrent number is 17, which is  exceeded the threshold 10.

If this is the case, it indicates that the user needs to set the number of concurrency to 17, so I think this is ok.

If this is the case, it indicates that the user needs to set the number of concurrency to 17, so I think this is ok.

Maybe I didn't make it clear...
Case 1 : run 20 threads to invoke dubbo service A (only system-defined filters) , set executes=10 on service A
Case 2:  run 20 threads to invoke dubbo service A (system-defined filters + user-defined filters) , set executes=10 on service A
We use dubbo-monitor-simple project  to monitor services.
we find that even if  both cases have set executes=10, the dubbo-monitor displays the concurrent number  is 17 in case 2. In case 1, the concurrent number  is 10.
At first we think the settings , "executes=10" ,  didn't work in case 2.
Finally we find that the order between ExecuteLimitFilter and MonitorFiler affects the count of concurrent number

In case two, did the user-defined filter tamper with the value of executes?

In case two, did the user-defined filter tamper with the value of executes?

no , the value of executes in ExecuteLimitFiler is still 10. Just  in dubbo-monitor-simple project, the number  displays as 17.

In case two, did the user-defined filter tamper with the value of executes?

no , the value of executes in ExecuteLimitFiler is still 10. Just in dubbo-monitor-simple project, the number displays as 17.

Which indicator are you focusing on in the monitoring center? concurrent this?

In case two, did the user-defined filter tamper with the value of executes?

no , the value of executes in ExecuteLimitFiler is still 10. Just in dubbo-monitor-simple project, the number displays as 17.

Which indicator are you focusing on in the monitoring center? concurrent this?

yes, the concurrent

I probably understand the problem you described, let me think about it.


Server Functions Middleware

Is it possible to add custom error handling logic or logging to all server functions (actions) at once, without the need to manually wrap each server function in a middleware function, similar to the example provided here: https://github.com/solidjs/solid-start/issues/1434#issuecomment-2097769617 ?

Does vinxi even know about server functions from solid-start?

Here is how the server functions work:
https://github.com/nksaraf/vinxi/blob/dbf6e8661f0a03f4161abce4afc8c9f22b747183/packages/vinxi-server-functions/server-handler.js#L14-L35
I think we need a way to adjust the action result, maybe via something like:
- const result = action.apply(null, json);
+ const result = middleware ? middleware(action, json) : action.apply(null, json);

This will allow us to handle server action errors, for example:
async function middleware(action, data) {
  try {
    return await action(data)
  } catch (error) {
    return "modified server function response here"
  }
}


Add Dirac as Optional Profile

Dirac goes beyond cljs-devtools with a full REPL and debugging etc.
Should be an optional profile, not required, as it is not a beginner tool.

thheller/shadow-cljs#636

Sadly, Dirac is a dead project.


NaN hue values

Clearly related to this issue/question: https://github.com/LeaVerou/color.js/issues/56
But IMO this is a bug as it relates to the numeric values in the hsl, hwb, lch, and oklch properties of Color, and I wanted to express it in a more concise form than issue 56.  See this codepen for an illustration: https://codepen.io/sidewayss/pen/VwqWoGx
For monochrome rgb values, the conversion to hue results in NaN instead of zero in the numeric arrays represented by the properties mentioned above.  I see that in the string properties for these color spaces NaN is replaced by the keyword none, which is equivalent to zero, but for the numeric properties NaN is awkward and essentially incorrect.

For monochrome rgb values, the conversion to hue results in NaN instead of zero in the numeric arrays represented by the properties mentioned above. I see that in the string properties for these color spaces NaN is replaced by the keyword none, which is equivalent to zero, but for the numeric properties NaN is awkward and essentially incorrect.

This is actually quite appropriate. A monochromatic color has no hue; therefore, one cannot be provided on convert. It should be noted that while CSS may treat NaN or none as 0 when it feels it absolutely needs a number, an undefined value is not equivalent to 0, it is simply the resolution that has been chosen.

It's only inappropriate if you're requesting an array of 3 numbers as your converted value.  That's my use case.  You can't scale or adjust NaN in any way for animation, while zero is a viable number.  Yes, I can and now do convert NaN to zero, but it's awkward and seems wrong to me.

It may be appropriate to expose some way to grab coordinates with undefined values normalized.

It may be appropriate to expose some way to grab coordinates with undefined values normalized.

I'm new to color spaces and new to colorjs, but it seems to me that the place to do it is these existing properties that expose an array of numbers.

I see here that the NaN is initial value for HSL/HWB hue, and here and here that lch and oklch set it explicitly. I can see how you are talking about another set of properties, especially if these are used internally.  Oh well, I'm probably barking up the wrong tree here, as no one else has noticed or cared enough to mention it.


Question: can I add this feature

Hello. @Akryum, I apologize if it's not correct place for questions, but I wasn't able to get answer in discord chat and unable to find you there.
I want to add option for writing graphql queries in Vue SFC block, using reworked version of existing loader:  https://github.com/zephraph/vue-graphql-loader
Everything I want to do is just to add automatic configuration of this loader to vue-cli-plugin-apollo, and update vue-apollo documentation to mention this. But before I start working on this, I would like to know your feedback.
I think it would be nice to have this option, because it's standard approach for vue to structure application. It really worth to be a part of vue-apollo.
So, can I try to create PR for this?

I guess this should go to the vue-cli-plugin-apollo repository then :smile_cat:

@Akryum apology, fixed it: https://github.com/Akryum/vue-cli-plugin-apollo/issues/221


Support for modulo (%)

Hi, we use your lib in Chamilo LMS.
Do you have any intention of supporting the modulo (%) operation?
For example: 3%2 = 1, 7%4=3, etc.
I was a bit surprised to have + - * / ^ and _ but not %... (more than anything because it seems pretty easy to add).

Hi!
As title states this repository is just a Composer Version of EvalMath by Miles Kaufman lib. However I don't mind if you're going to add new operators support. If you find it easy, please implement this, test for backward compatibility, open pull request than I'll create a new version.

Miles Kaufman website is kind of anti-social (just a logo), so I couldn't contact him. I'll try to send a PR (or have someone send it) soon.

Hi, have any method to do give error when input like this cos(a)+sin(c)*tan(f)--sqrt(d)/pi(o) equation? -- decrement operator should not accepted by class but give error. what should be done for it? have any method to reject this type of equation?

@ywarnier I've added support for modulo operator (see https://github.com/dbojdo/eval-math/blob/master/tests/EvalMathTest.php)

Great, thanks! Sorry I couldn't do it faster. Been busy with our own free software :+1:

Your Welcome, I faced this issue in practical so i asked to you this
question. Thanks to you also that you have accept my view.
On Mon, Feb 15, 2016 at 7:04 PM, Yannick Warnier<EMAIL_ADDRESS>wrote:

Great, thanks! Sorry I couldn't do it faster. Been busy with our own free
software [image: :+1:]
—
Reply to this email directly or view it on GitHub
https://github.com/dbojdo/eval-math/issues/3#issuecomment-184210785.

@dbojdo We have integrated the new version to Chamilo LMS https://github.com/chamilo/chamilo-lms/commit/da8bce63019338ab1261a6b1be75adbf2f99d340
We should be releasing a new version of Chamilo in a few weeks time, so this will be used by a few people, probably. Modulo was missing because we use it in a feature called "Calculated answer" where we generate a set of possible answers for an equation-type question: "Judy has [x] apples. How many apples are left after she distributes them equally between her [y] brothers?". Obviously, in this case you need to use the modulo operation in order to get the right number of apples left.
You can try it out on https://unstable.chamilo.org/main/exercice/overview.php?cidReq=OPEN&exerciseId=4 if you want to see how it looks
It was not a high priority on our list, but it still very nice to have it. Thanks.


[Issue]:  Write samle base on SemanticKernel with content copy from Example04_Dynamic_GroupChat_Coding_Task, can't work

Describe the issue
[Issue]:  Write samle base on SemanticKernel with content copy from Example04_Dynamic_GroupChat_Coding_Task, can't work. I don't know what's wrong happend.
This is my code : https://github.com/feynmanloo/AutoGen.BasicSamples/blob/master/Example04_Dynamic_GroupChat_Coding_Task.cs
Steps to reproduce
No response
Screenshots and logs
"C:\Users\Feynman Loo\AppData\Local\Programs\Rider\plugins\dpa\DotFiles\JetBrains.DPA.Runner.exe" --handle=23160 --backend-pid=25060 --etw-collect-flags=67108622 --detach-event-name=dpa.detach.23160 "C:/Users/Feynman Loo/Documents/Workspaces/AutoGen.BasicSamples/bin/Debug/net8.0/AutoGen.BasicSamples.exe"
Hello, World!
from: admin
{
    "to": "coder",
    "task": "Write a C# program to calculate the 39th Fibonacci number and save the result in result.txt",
    "context": ""
}

Unhandled exception. System.ArgumentException: Invalid message type
at AutoGen.SemanticKernel.SemanticKernelAgent.<>c.b__11_0(IMessage m)
at System.Linq.Enumerable.SelectEnumerableIterator2.MoveNext()    at System.Linq.Enumerable.Any[TSource](IEnumerable1 source, Func2 predicate)    at AutoGen.SemanticKernel.SemanticKernelAgent.BuildChatHistory(IEnumerable1 messages)
at AutoGen.SemanticKernel.SemanticKernelAgent.GenerateReplyAsync(IEnumerable1 messages, GenerateReplyOptions options, CancellationToken cancellationToken)    at AutoGen.Core.GroupChat.SelectNextSpeakerAsync(IAgent currentSpeaker, IEnumerable1 conversationHistory)
at AutoGen.Core.GroupChat.CallAsync(IEnumerable1 conversationWithName, Int32 maxRound, CancellationToken ct)    at AutoGen.Core.AgentExtension.SendMessageToGroupAsync(IAgent _, IGroupChat groupChat, IEnumerable1 chatHistory, Int32 maxRound, CancellationToken ct)
at AutoGen.Core.AgentExtension.SendAsync(IAgent agent, IAgent receiver, IEnumerable`1 chatHistory, Int32 maxRound, CancellationToken ct)
at AutoGen.Core.AgentExtension.InitiateChatAsync(IAgent agent, IAgent receiver, String message, Int32 maxRound, CancellationToken ct)
at AutoGen.BasicSamples.Example04_Dynamic_GroupChat_Coding_Task.RunAsync(IKernelBuilder kernelBuilder, OpenAIPromptExecutionSettings settings) in C:\Users\Feynman Loo\Documents\Workspaces\AutoGen.BasicSamples\Example04_Dynamic_GroupChat_Coding_Task.cs:line 235
at Program.$(String[] args) in C:\Users\Feynman Loo\Documents\Workspaces\AutoGen.BasicSamples\Program.cs:line 20
at Program.(String[] args)
Process finished with exit code -532,462,766.
Additional Information
No response

@LittleLittleCloud plz

Seems that you forget to register message connector for the helper agent?
https://github.com/feynmanloo/AutoGen.BasicSamples/blob/07d03885cb3a7b83cdd217f45699da49aa1af355/Example04_Dynamic_GroupChat_Coding_Task.cs#L29

Seems that you forget to register message connector for the helper agent? ( and maybe admin agent as well)
https://github.com/feynmanloo/AutoGen.BasicSamples/blob/07d03885cb3a7b83cdd217f45699da49aa1af355/Example04_Dynamic_GroupChat_Coding_Task.cs#L29

@LittleLittleCloud Any document about register messsage connector and other api ?
After I register message connector for the helper agent and admin agent, I got another exceptions,below here:
System.InvalidOperationException: Sequence contains no matching element
at System.Linq.ThrowHelper.ThrowNoMatchException()
at System.Linq.Enumerable.First[TSource](IEnumerable1 source, Func2 predicate)
at AutoGen.Core.GroupChat.SelectNextSpeakerAsync(IAgent currentSpeaker, IEnumerable1 conversationHistory)    at AutoGen.Core.GroupChat.CallAsync(IEnumerable1 conversationWithName, Int32 maxRound, CancellationToken ct)
at AutoGen.Core.AgentExtension.SendMessageToGroupAsync(IAgent _, IGroupChat groupChat, IEnumerable1 chatHistory, Int32 maxRound, CancellationToken ct)    at AutoGen.Core.AgentExtension.SendAsync(IAgent agent, IAgent receiver, IEnumerable1 chatHistory, Int32 maxRound, CancellationToken ct)
at AutoGen.Core.AgentExtension.SendAsync(IAgent agent, IAgent receiver, String message, IEnumerable`1 chatHistory, Int32 maxRound, CancellationToken ct)
at AutoGen.BasicSamples.Example04_Dynamic_GroupChat_Coding_Task.RunAsync(IKernelBuilder kernelBuilder, OpenAIPromptExecutionSettings settings) in C:\Users\Feynman Loo\Documents\Workspaces\AutoGen.BasicSamples\Example04_Dynamic_GroupChat_Coding_Task.cs:line 238

@feynmanloo I took a look at your code. The issue is caused by missing stop words in admin's prompt setting. You might want to pass [":"] to stop sequence when creating admin. When I add the right stop sequence to admin, the error was gone.

However, the task still fail because of group admin fails to generate the next speaker in the right format even after I add stop sequence. I note that you are using llama3:7b as the backend model for all agents, including group admin. Yet the prompt for speaker selection used by group admin is based on gpt-3.5/4. So that might be why the admin fails to follow the prompt here because different LLM behaves differently even using the same prompt. And next speaker selection prompt for GPT-series might not work well with llama-series.
So here is the suggestion that you may try

use gpt-3.5/4 for admin, and llama3 for others.
If you still want to use llama3 for admin, adding a fallback logic using middleware to admin so it always returns a legitimate next speaker. For example, in the middleware, if the reply from inner agent is not From xxx, return a hard-code value.
manually orchestrate agents, for example, having a seperate agent generate step and assign that step to specific agent. In this way you no longer need group chat and can achieve the full flexibility in constructing agentic workflow.


[BUG] Some code problems

When I try to run the evaluate command of “Translation-train SUMBT for cross-lingual DST”, I foud some code problems.

convlab2/dst/sumbt/multiwoz_zh/sumbt.py , line 33
DOWNLOAD_DIRECTORY = os.path.join(SUMBT_PATH, 'pre-trianed')
Maybe 'pre-trianed' should be changed to 'pre-trained'
convlab2/dst/sumbt/multiwoz_zh/convert_to_glue_format.py , line 126,
value = value_trans.get(value, value)
Maybe  value_trans.get(value, value) should be trans_value(value)
convlab2/dst/sumbt/crosswoz_en/sumbt.py , line 36 (same as no.1)
DOWNLOAD_DIRECTORY = os.path.join(SUMBT_PATH, 'pre-trianed')
Maybe 'pre-trianed' should be changed to 'pre-trained'

Thanks! We've fixed the bugs


Add link to Feather M0 Express

and note other boards are upcoming.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


Is django-perf-rec working with sqlite?

I've tried to make django perf-rec work with a test environment that uses only sqlite database (it's easier to set-up) and I had a strange result.
Here's my test:
from django_perf_rec import TestCaseMixin as PerfTestCaseMixin
from app.models import MyModel

class TestCase(PerfTestCaseMixin, TestCase):
    pass

class PerfTest(TestCase):

    def test_foo(self):
        with self.record_performance():
            self.assertEqual(MyModel.objects.count(), 0)

The resulting YAML is:
PerfTest.test_foo:
- db: QUERY = u# - PARAMS = ()

Where do you think this might come from? should I edit some settings to have a usable output?

Django used to freeze sqlite queries as QUERY = ... PARAMS = ... because the sqlite driver didn't keep a history of queries, but this was updated in some more recent version. I think it was 1.8 or before. This project tests on 1.8 with sqlite iirc.

in the said test, I'm using Django==1.8.17 :/

ah! I see this: https://github.com/YPlan/django-perf-rec/blob/master/tests/settings.py#L16-L19 ; do you think it might be a clue?

all right: using this database backend, the output is way more informative.
What could eventually been done is:

to include this "test db backend" in the distribution (I had to copy the django-perf-rec/tests/django18_sqlite3_backend directory instead of simply import it),
at least document this special case and give a hint on how to run tests using Django 1.8.x + sqlite.

I'd welcome a PR to expose that backend... but unless your prod deployment is using sqlite my strong recommendation is to use the database you use in production for tests.


r-theta plot

Our current stratagey for radial plots works alright for donuts/pies/radial charts, but there is a constellation of additional chart types we might want. I propose that we add a notion of R-Theta Plots (to match the XY Plots) for controlling charts the express themselves in polar coordinates. This will allow us to make partition graphs, radial scatterplots, polar rose diagrams.

For anyone watching this thread, my current plan of attack to address this feature is as follows

Introduce huge amount of tests
Add a concept of accessors to series, so that users can provide things like

xAccessor={row => row.get('my-x-col-name')}

This in turn will open us up to immutable.js, (which is a super cool feature by itself but will require some refactoring of data processing logic, again the important of the first step being tests)

Introduce a new object called RThetaPlot that wraps all of it's children in a coordinate transform moving polar to XY

After a lot of thinking, I think this is the wrong tree to be barking up. As seen in arc-series it is perfectly possible to get all the functionality we need from the XYPlot! I still think introducing accessors is a good project, but I think it deserves it's own ticket. Marking as invalid.


CacheLib Performance Parity Benchmarking (w/ background threads)

The goal of this experiment is to so the progress towards single tier DRAM performance parity. We also investigate some initial background data movement parameters.
Background Movement
The branch to use is here: https://github.com/byrnedj/CacheLib/tree/mt-background-movers
The cache configuration parameters to test are the following:
-backgroundEvictorIntervalMilSec: 0, 10
-evictorThreads: 4 , 12
-evictionHotnessThreshold: 40, 200
-lowEvictionAcWatermark: 91, 98
-highEvictionAcWatermark: 88, 95
Since highEvictionAcWatermark must be less than lowEvictionAcWaterMark, then the combinations are the following (all with background evictor milSec = 10 and DRAM:PMEM ratio 1:4):
evictorThreads: 4, evictionHotnessThreshold: 40, lowEvictionAcWatermark: 91, highEvictionAcWatermark: 88
evictorThreads: 4, evictionHotnessThreshold: 40, lowEvictionAcWatermark: 98, highEvictionAcWatermark: 88
evictorThreads: 4, evictionHotnessThreshold: 40, lowEvictionAcWatermark: 98, highEvictionAcWatermark: 95
evictorThreads: 4, evictionHotnessThreshold: 200, lowEvictionAcWatermark: 91, highEvictionAcWatermark: 88
evictorThreads: 4, evictionHotnessThreshold: 200, lowEvictionAcWatermark: 98, highEvictionAcWatermark: 88
evictorThreads: 4, evictionHotnessThreshold: 200, lowEvictionAcWatermark: 98, highEvictionAcWatermark: 95
evictorThreads: 12, evictionHotnessThreshold: 40, lowEvictionAcWatermark: 91, highEvictionAcWatermark: 88
evictorThreads: 12, evictionHotnessThreshold: 40, lowEvictionAcWatermark: 98, highEvictionAcWatermark: 88
evictorThreads: 12, evictionHotnessThreshold: 40, lowEvictionAcWatermark: 98, highEvictionAcWatermark: 95
evictorThreads: 12, evictionHotnessThreshold: 200, lowEvictionAcWatermark: 91, highEvictionAcWatermark: 88
evictorThreads: 12, evictionHotnessThreshold: 200, lowEvictionAcWatermark: 98, highEvictionAcWatermark: 88
evictorThreads: 12, evictionHotnessThreshold: 200, lowEvictionAcWatermark: 98, highEvictionAcWatermark: 95
Then we have two baselines runs:

backgroundEvictorIntervalMilSec: 0, with DRAM:PMEM set to 1:4
backgroundEvictorIntervalMilSec: 0, evictorThreads: 4, evictionHotnessThreshold: 40, lowEvictionAcWatermark: 98, highEvictionAcWatermark: 95 with DRAM:PMEM set to 1:1

For a total of 14 experiments per workload configuration.
The workload configuration parameters are:
-opRatePerSec: 500000, none
-htBucketPower 28 (for both workloads)
-48 threads and 5M ops/thread
-workload: leader_obj, follower_obj
There are 4 different workload configurations, so the total number of background evictor experiments is 56.
Additional baselines
For comparison we need to test the additional baseline systems for each workload configuration:
-https://github.com/igchor/CacheLib-1/tree/issue75_rebased using the DRAM:PMEM 1:1
-https://github.com/igchor/CacheLib-1/tree/issue75_rebased using the DRAM:PMEM 1:4
-https://github.com/igchor/CacheLib-1/tree/issue75_rebased using PMEM only
-https://github.com/igchor/CacheLib-1/tree/issue75_rebased using DRAM only
There are 4 different workload configurations, so the total number of additional baseline system experiments is 16.
In total there are 72 experiments - each should last around 5 minutes so total execution time is estimated to be 6 hours.

How about running some benchmarks for smaller evictorThreads number? When delayOps/ratePerSec is used we should not need to much threads.
Also, it would be great if we could print the percent of allocated slabs and classes/pools during the benchmark (I did not implement this yet). This would allow us to see if we can keep up with eviction and maintain the specified watermarks. Based on this info, we can later adjust number of threads (we should probably use the lowest possible amount which can keep up).

Also, I would like to wait with performing those benchmarks until we actually merge evition/promotion to develop branch. We could use some additional review for that.


use clerk-rs client lib

Why?

simplify integration by using clerk-rs instead of direct API calls
add error logging for all interactions with clerk-rs

I think there is an issue w/ the clerk_organization_memberships implementation as it is in this PR, but I think clerk_rs should still work for that use case?

I think issue w/ clerk_organization_memberships is resolved now.

@Jayko001 this is ready for review.


chore(main): release docker 0.1.1

:robot: I have created a release beep boop
0.1.1 (2024-09-22)
Bug Fixes

molecule tests (#18) (37770c6)

This PR was generated with Release Please. See documentation.

🤖 Created releases:

docker-v0.1.1

:sunflower:


Remove random.seed from templates?

Given Zeek's deterministic option:
$ zeek --help 2>&1 | grep -- -D
    -D|--deterministic              | initialize random seeds to zero

Does it make sense to remove the random.seed file from the templates and use -D instead? How can the option be passed automatically? It wasn't clear to me how the RANDOM_SEED var is used.

Nice one, Yacin... the reason we use random.seed in Zeek's testsuite is that by setting ZEEK_SEED_FILE btest.cfg we can fix it once for all tests, unless they unset the variable. That's a bit cleaner than invoking zeek -D everywhere, or aliasing the invocation. But it's a great question for the template, and much of this is simply historic.
I'm not aware of a way to default-set -D (which'd be in the script layer, presumably). Flags and their script-layer correspondence are a bit of an inconsistent area — for example, we have -C to ignore checksums, and there's ignore_checksums in the script layer, but running with -C doesn't set ignore_checksums.
Fwiw ... I'm tempted to revamp the template's approach to testing quite a bit. The pattern so far is that people don't go back to editing the auto-created tests at all, so they're cruft from the get-go. I'm tempted to remove the default tests completely and make them a feature instead. You'd have to ask for tests to be in the resulting package specifically, and we could make the testing setup a lot more configurable. (There are certain scenarios settings where default test make a lot of sense, like when a test verifies that a plugin builds & loads — I wouldn't change that.)
So given all this ... does it still bug you? I'd leave it in there for now, and revisit this with that feature revamp.

Credit is due to @JustinAzoff for letting me know about -D. Came up in code review.
It doesn't bother me much but I was a bit curious how easy it would be to change it, i.e., add default Zeek flags to all tests that invoke Zeek. But given the flexibility of btest I can see that might not always be the case and would probably be clunky to implement.
Agreed on removing the default tests, but make them a feature instead. If I may, a common class of test case for me is "make test X that runs X.pcap and diffs against log Y". Being able to generate that skeleton would be nice, however, I usually find the pcaps as I'm writing the package so it would be nifty to have zkg template work on an existing repo to generate the full test when I have the pcap(s) in hand. I'm typically copying/pasting/editing or cat test1 | sed ... > test2 once I have the pcaps and a hand-written test anyway, so this would be a nice convenience.
I'll close this guy for now. Thanks @ckreibich !


Issue #1. Critical Push Notifications.

Description
Added an event listener to request permissions to the user once the app is opened, no matter if it was opened by the user or by a tapped push notification.
This will be helpful since we don't need to wait the execution of the "init" method of this plugin to ask access to the user.
Related Issue

Critical Push Notifications
Motivation and Context

I added this change because the plugin should request the permissions to the user as soon as the user opens the application without having to wait for the business logic of the application. Also, this change was requested time ago by T4M and it has been used that way ever since because we already know the app will use normal and critical notifications.
How Has This Been Tested?

Changes tested in a mobile device with the following specs:

16.5.1 iOS version.
iPhone XR model.

Screenshots (if appropriate):
https://github.com/TransformativeMed/cordova-plugin-push/assets/12654157/7f2f30cc-7f64-41d2-925b-308779c035b9
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Closing this PR since the PR #10 will move this changes into the master branch.


Multiple generic thermostats don't trigger heaters

Home Assistant release (hass --version): 0.35.3
Python release (python3 --version): 3.4
Component/platform: generic_thermostat
Description of problem:
Hi,
I have 2 heaters (two command_line switches) so 2 generic_thermostat that I control from a unique input_slider which adjust each target_temp of my 2 generic thermostats in the same time. Since I upgraded Home Assistant from a Rpi2b to a Rpi3 there is no more lag so when the temperature is below the target_temp the 2  generic_thermostat trigger exactly in the same time. And here is the problem : the 2 switches of my heaters are indeed correctly switched on in Home Assistant but in the reality my heaters don't heat.
A picture of my HA group :

I found a temporary solution to use my unique input_slider (but it still doesn't work when it is the temperature variation which trigger the 2 generic thermostat) : when I adjust my input_slider it also triggers in the same time the 2 thermostats so I just put a delay between each thermostat target temperature adjustement and it works.
Generic thermostats:
- platform: generic_thermostat
  name: Salon
  heater: switch.salon_radiator
  target_sensor: sensor.inside_temperature
  min_temp: 0
  max_temp: 22
  target_temp: 0
  tolerance: 0.3

- platform: generic_thermostat
  name: Cuisine
  heater: switch.cuisine_radiator
  target_sensor: sensor.inside_temperature
  min_temp: 0
  max_temp: 22
  target_temp: 0
  tolerance: 0.3
#  min_cycle_duration:
#    seconds: 1

I tried to set only one thermostat with 1 second of min_cycle_duration but it doesn't work too.
Input_slider automation:
- alias: 'Slider temperature to climate si en marche'
  hide_entity: True
  trigger:
    platform: state
    entity_id: input_slider.temperature
  condition:
    condition: and
    conditions:
      - condition: state
        entity_id: input_select.climate_state
        state: "En marche"
  action:
    - service: climate.set_temperature
      data_template:
        entity_id: climate.salon
        temperature: '{{ trigger.to_state.state | float }}'
    - delay:
        milliseconds: 500
    - service: climate.set_temperature
      data_template:
        entity_id: climate.cuisine
        temperature: '{{ trigger.to_state.state | float }}'

Heaters swhitches :
- platform: command_line
  switches:
    salon_radiator:
      command_off: "/var/www/hcc/radioEmission 0 XXXXXXXX 0 on"
      command_on: "/var/www/hcc/radioEmission 0 XXXXXXXX 0 off"
    cuisine_radiator:
      command_off: "/var/www/hcc/radioEmission 0 XXXXXXXX 1 on"
      command_on: "/var/www/hcc/radioEmission 0 XXXXXXXX 1 off"

Expected:
I made a lot of tests to determine the problem (with different scripts or automations to adjust the target temperature or the temperature) and it seems that it happens when it is triggering in the same time.
There is also nothing written in HA logs.
Something which could be good too : the possibility to set multiple heaters for an only generic_thermostat. It could be another solution to use only one thermostat (but not for people using multiple thermostat indeed...)
I didn't really find other people reporting this problem, maybe here, and finally I hope it is only a problem from my configuration.
PS: sorry it is a mix of french and english in my code

Finally it was a problem with the script which triggers my 433mhz transmitter: it doesn't work when it is launched several times in the same time.


[Bug]: WARNING: Falling back to CPU-only rendering. Failed to initialize WebGL context.

Package Version
5.2.2
Flutter Version
Flutter 3.26.0-0.1.pre • channel beta
Platforms
Web
How to reproduce?
bugs: WARNING: Falling back to CPU-only rendering. Failed to initialize WebGL context.
bytes data: image_data: []
I found the bug onImageEditingComplete: (bytes) async {}.
here, no byte data I'm getting. how can I solve this or this is a package issue?
this is happening only in web. android is okay.
Logs (optional)
No response
Example code (optional)
No response
Device Model (optional)
No response

Could you share your current code with the image editor so we can try to reproduce the issue? It would also be helpful to know which web browser you're experiencing this problem on. Have you already tried using the example code? If so, do you encounter the same issue when you copy and paste sections of it into your project?

I got the issue. It was because of  imageGenerationConfigs: ImageGenerationConfigs() configuration. I've set ratio very large that's why it was happening.


Debug why translated Japanese files fail to render

Found by npm run check-pages-render -- --translations on Jan 24:
translations/ja/build/bit-ordering.mdx
translations/ja/build/introduction-to-qasm.mdx
translations/ja/build/specify-observables-pauli.mdx
translations/ja/build/unitary-synthesis.mdx
translations/ja/run/circuit-execution.mdx
translations/ja/run/configure-runtime-compilation.mdx
translations/ja/run/instances.mdx
translations/ja/run/processor-types.mdx
translations/ja/run/reserve-system-time.mdx
translations/ja/run/retired-systems.mdx
translations/ja/run/system-information.mdx
translations/ja/start/install.mdx
translations/ja/transpile/defaults-and-configuration-options.mdx
translations/ja/api/migration-guides/qiskit-algorithms-module.mdx
translations/ja/api/migration-guides/qiskit-quantum-instance.mdx

To check that they render, run ./start locally then go to e.g. localhost:3000/ja/build/bit-ordering.

@ilyaspiridonov (ilyaspiridonov) sent the following message over slack

Hi all,
upon looking at those errors with our dev team, here are the issues we have identified:

The JA folder seems to contain English files that we did not translate. I don’t believe they are supposed to be there. Here’s an example https://github.com/Qiskit/documentation/blob/main/translations/ja/api/migration-guides/qiskit-algorithms-module.mdx
Those English files contain certain issues that were likely introduced after copying (do you have some sort of processor?). Example from the above file:
**[Legacy]\:**
while in the En source this string looks like this
**[Legacy] Using QuantumInstance:**
The files we have translated seem to have common issues introduced by Crowdin, which is escaping that makes the the formula parser go crazy:
\\
instead of:
\
Another issue presumably introduced by crowdin is the extra Tab characters before the closing tags like here:

<Admonition type="tip" title="Recommendations">

- Learn how to generate OpenQASM code in the [Explore gates and circuits with the Quantum Composer](https://learning.quantum.ibm.com/tutorial/explore-gates-and-circuits-with-the-quantum-composer) tutorial.
- Review the [OpenQASM 3 feature table](qasm-feature-table).
- Read the [OpenQASM 3 Qiskit API](/api/qiskit/qasm3) reference.
- Read the [OpenQASM 2 Qiskit API](/api/qiskit/qasm2) reference.
- Visit the [OpenQASM Live Specification](https://openqasm.com/).
	</Admonition>

Perhaps you also downloaded the untranslated files from Crowdin (English)? Which may explain why there are English files in the JA folder, and why they are also broken.
We’ve fixed those issues in the files we translated, here’s the package: fixed-translations.zip
You may want to remove the other English files from the JA directory and talk to Crowdin about fixging those issues (escaping and closing html tags being not on the same level as the open tags (extra tabulation)


Add find feature

Add a feature that enables a user to find a word in the text.

I want to work on this issue. Please assign me this issue.

I want to work on this issue. Please assign me this issue.

Ok @ayushrastogi689 I am assigning you this issue. Make sure you complete it in 3-4 days.

Sure.

Thank you @ayushrastogi689


Cross-Session Persistence of User Settings

Small issue but would be nice to save map settings.  Everytime I run your the app I have to toggle off navigational data

I see what you mean. Up to this point, I have only implement client-side session-persistence, i.e. you have to configure these things once per flight, and then it stays this way for this particular flight. This should already be working properly in 1.1.0. Do you encounter issues with this "session persistence"? Or are you requesting persistence across multiple flights?

Yeah...I'm requesting persistence across multiple flights?

This will be fixed with upcoming v1.6!


WIP: Chain Class

In this PR we will integrate the Chain class in cpp.
The Chain class represents an open kinematic chain
and helps model the Wrench exerted on the body
using joint torques and the Jacobian.

Good progress! Please request a re-review when you want me to look again.

@varunagrawal @dellaert
Thanks for the reviews.
merging.

Great work @danbarla! Looking forward to using this in my state estimator 😄


feat: add tekton resources

Adds Tetkton resources to base reconciler (introduced in https://github.com/3scale-ops/saas-operator/pull/268)
Fix a bug in resource deletion

/kind feature
/kind bug
/priority important-soon
/assign

/lgtm

/approve


feat (datasets): allowing control over certification datasets

SUMMARY
We are interested in controlling who can certify datasets. We have a central analytics engineering team who are the single-source of truth regarding datasets. We would like them to be the only team that can certify their datasets.
This introduces the ability to create a new type of role ('certifier') that we can grant to members of that team. There is an additional feature flag that would enable this feature.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[x] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[x] Introduces new feature or API
[ ] Removes existing feature or API

Will try a different approach


[chore](routine load)  make error msg clear if routine load name illegal (#40037)

pick (#40037)
Make error msg clear if routine load name illegal(such as job name is too long).

run buildall

run buildall


Docs: Add information about padding and margin utilities

Adds a description of padding and margin utilities which are exported from @styled-system/space.
This week I had a situation when I had to provide only margins in one of my components. I couldn't find any information in the documentation. I had to read the sources to find that space package also exports padding and margin subsets of the space utility. Later I found a mention about them in the Packages section, but it was not an obvious place for me to search.
As a result, I decided to add a small section in the Spacing section with an example and a paragraph to the Reference Table section.
Let me know what do you think.

Awesome, thanks!


Error with remove connection picker

Hi Norm
We are getting this error on multiple apps:
err TypeError: Cannot read properties of null (reading 'connectionPicker')
at KTL.js:7474:35
at Array.forEach ()
at removeConnectionPicker (KTL.js:7472:18)
at ktlProcessKeywords (KTL.js:6736:17)
at HTMLDocument. (KTL.js:6481:21)
at HTMLDocument.dispatch (k_0c7f01687ca49a56d119a7de6c60bb37a9b6e3e5.js:2:199750)
at HTMLDocument.s (k_0c7f01687ca49a56d119a7de6c60bb37a9b6e3e5.js:2:195785)
at Object.trigger (k_0c7f01687ca49a56d119a7de6c60bb37a9b6e3e5.js:2:198319)
at HTMLDocument. (k_0c7f01687ca49a56d119a7de6c60bb37a9b6e3e5.js:2:206094)
at Function.each (k_0c7f01687ca49a56d119a7de6c60bb37a9b6e3e5.js:2:176060)
Craig

I can't reproduce it in any page.
Can you?  What view type?

That's weird we had the error in 2 apps but now we can't replicate it.  I'll close for now and keep an eye on it.
Craig

Hi Norm
I have this error again:
KTL.js:6747 err TypeError: Cannot read properties of null (reading 'connectionPicker')
at KTL.js:7474:35
at Array.forEach ()
at removeConnectionPicker (KTL.js:7472:18)
at ktlProcessKeywords (KTL.js:6736:17)
at HTMLDocument. (KTL.js:6481:21)
at HTMLDocument.dispatch (k_40283f29cfbec3c607fd76000daff689ed00f78a.js:2:199750)
at HTMLDocument.s (k_40283f29cfbec3c607fd76000daff689ed00f78a.js:2:195785)
at Object.trigger (k_40283f29cfbec3c607fd76000daff689ed00f78a.js:2:198319)
at HTMLDocument. (k_40283f29cfbec3c607fd76000daff689ed00f78a.js:2:206094)
at Function.each (k_40283f29cfbec3c607fd76000daff689ed00f78a.js:2:176060)
And I can replicate it.
Craig

In your case, when the bug appears, why view type is it?

A Details View
On Fri, 13 Sept 2024, 16:01 Cortex R&D Inc., @.***>
wrote:

In your case, when the bug appears, why view type is it?
—
Reply to this email directly, view it on GitHub
https://github.com/cortexrd/Knack-Toolkit-Library/issues/363#issuecomment-2349165176,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACQQXA2ZGTLXH5EVGUBQRV3ZWL44XAVCNFSM6AAAAABN6RQUFKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNBZGE3DKMJXGY
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Got it now.

I've pushed the fix in Dev branch.
Can you test it please?

Great Thanks


Update README.md

Closes #264
👨💻 Changes proposed

The link to the website has been added in the title along with the logo/favicon of the website in the title of the README.md
Demo has been moved above the Features list because i feel it would be amazing if people could try and then come back to see how it works/ was built
Table for tools and languages has been replaced with badges of languages used in this repo ( i put up the sae languages which were listed in the table
Minor changes in Features list ( for Grammatical and good Readability)

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

I know that this issue was just to add the borer generator in readme but since it was already listed i looked through the README.md and changed very minor things from almost perfect .md
Any request for changes are highly accepted

📷 Screenshots

When contributing there's a mind your business rule. Please follow that and do what your issue said to do.
And fix the PR compliance error

Ok sorry for the trouble....i didn't know much coz I'm first timer for open source and stuff

Ok sorry for the trouble....i didn't know much coz I'm first timer for open source and stuff

It's fine, just do the following:
— Edit your PR title and the correct thing
— delete every line of code that doesn't have to do with your Issue


Option code optimizer

Imagine there is a higly loaded application making heavy usage of the Option types, and there is a monadic code with all those FlatMaps and Maps, or even worse - linq syntax expression, invoked in a loop somewhere. That brings pressure on the GC as closures and anonymous objects are continiously created.
What if
(a, b, c) =>
    from aprime in a
    from bprime in b
    from cprime in c
    where Predicate(aprime, cprime)
    select Combine(aprime, bprime, cprime)

transformed automagically into something like
(a, b, c) =>
{
    if (!a.HasValue) return default;
    var aprime = a.GetValue();
....
    if (!Predicate(aprime, cprime) return default;
    return Combine(aprime, bprime, cprime);
}

when passing it as an expression to a special function?
I understand this might be not a frequent case and one would say "use low-level things in this case", but anyway, should an optimized Option code generation be considered as an appropriate feature for this library?
I have my own small library (which nobody really knows about) with maybe, either, CPS, collection extensions and so on, and there's a code there for doing what I've just described (not published anywhere yet), I decided it would be more productive to contribute here.

Hi!
I am certainly not against performance improvements, insofar we can keep them hidden from the public API.
Could you elaborate a bit on how you would introduce such optimizations, so I can better evaluate the pros/cons?
Thanks!
/ Nils


Website Font & Layout

Is your feature request related to a problem? Please describe.
Website font and layout are not aesthetically pleasing.
Describe the solution you'd like
Header and paragraphs to be more proportional.
Better website design / layout.

We would need more info on what specifically needs to be updated along with suggestions ideally.


Loosen bundler version dependency

Bundler 2.0 is out, and Travis is being ornery with bundling any gem using private_gem:
Bundler could not find compatible versions for gem "bundler":
  In rails4.2.gemfile:
    private_gem was resolved to 1.1.3, which depends on
      bundler (~> 1.7)
  Current Bundler version:
    bundler (2.0.1)

The tests seem green with bundler 2.0, so lets open up the restriction.
cc @staugaard @grosser @craig-day

followup https://github.com/zendesk/private_gem/pull/10


RFE: handle block devices with no filesystem

Assuming a PV is a block device and lacks a filesystem, how will we handle?

Proposal: You're already using rsync - so why not use the --sparse --copy-devices options to copy the contents of a device


Use with Node v14: cannot read error of undefined

I tried all sorts of things using SVGO in Node 14 with "type": "module"
The only way I could actually access optimize was doing this:
import Svgo from 'svgo'

Svgo.prototype.optimize()

But I get this error when executing:
/node_modules/svgo/lib/svgo.js:27
        if (this.config.error) {
                        ^

TypeError: Cannot read property 'error' of undefined

v1 exports class which should be initialised
import Svgo from 'svgo';

const svgo = new Svgo(config)
svgo.optimize(code).then(result => {})


[rtextures] Add different wrap modes for NPatches

Please, before submitting a new issue verify and check:

[ ] I tested it on latest raylib version from master branch
[x] I checked there is no similar issue already reported
[x] I checked the documentation on the wiki
[ ] My code has no errors or misuse of raylib

Issue description
The current NPatch implementation only stretches the image. There are cases where a repeated mode would be more appropiate, like for the UI element I have attached, where the dotted lines would be better repeated to fit the size with as many dots, rather than stretch a fixed number of dots.
Issue Screenshot

@Aurora2500 Agree, but I'm afraid it implies a level of complexity to be implemented, not trivial. Feel free to work on it if you want.

No answer so I assume nobody is going to work on this feature. Closing.


Nested transactions don't work as expected

original:
try {
  await defaultManager.transaction(async mgr => {
    await mgr.withRepository(repo).insert({ name: "user1" });
    //throw new Error(`abort`);
    await mgr.withRepository(repo).insert({ name: "user2" });
    try {
      await mgr.transaction(async mgr2 => {
        await mgr2.withRepository(repo).insert({ name: "user3" });
        //throw new Error(`abort2`);
        await mgr2.withRepository(repo).insert({ name: "user4" });
      });
    }
    catch (err) {
      // ...
    }
    throw new Error(`abort3`)
  });
}
catch (err) {
  // ...
}

with typeorm-transactional:
try {
  await runInTransaction(async () => {
    await repo.insert({ name: "user1" });
    //throw new Error(`abort`);
    await repo.insert({ name: "user2" });
    try {
      await runInTransaction(async () => {
        await repo.insert({ name: "user3" });
        //throw new Error(`abort2`);
        await repo.insert({ name: "user4" });
      },{ propagation: Propagation.NESTED });
    }
    catch (err) {
      // ...
    }
    throw new Error(`abort3`)
  });
}
catch (err) {
  // ...
}

When using typeorm-transactional test case abort3 does create user3 and user4 which is not desired.
(other cases work as expected)

I think it's already committed in the transaction that makes user3, user4
The reason is that if the propagation property is set toNESTED, new transactions that are unrelated to the parent transaction are opened and executed
I think the propagation property should be REQUIRED to throw an error at the abort3 position and don't create user3, user4

I was looking for a propagation type with implementation that would not commit nested transactions if parent transaction is aborted. I found some workaround around this in the meantime.
Also from your source code there does not seem to be a difference between NESTED and REQUIRES_NEW:
https://github.com/Aliheym/typeorm-transactional/blob/master/src/transactions/wrap-in-transaction.ts#L92


Cleanup examples/museum-guide-low-level/ directory

Some folders are duplicated

Closing because commits are not signed-off


Where the .pth file? I found only the .ckpt file.

Thanks.

Thank you for your interest!
direct link:
u2net.pth
I found out there was a typo in ckpt file link.
I have updated!
Thank you!

Do you train it use DUTS Image Dataset?

Yes I did!
Everything step was same with U-2-Net except for transformation!


Examples link broken

The link in the README to https://hacspec.org/examples/ is broken. I do not immediately see where it should point.

Good catch, thanks! Links were relative to the github repository, not the website. I pushed a fix in #702 that rewrites relative links into absolute ones before pushing the README to the website.


No strings

Compatibility unit testing is not complete, but I wanted to PR this so Adam could have it.
I removed all strings from the types package, which means blocks, etc. should now be compatible with JSON.
This change is neither a hardfork or a softfork

Because Specifier is a byte-array of len 16, compatibility stuff is still needed. "NonSia" is different from [16]byte{'N', 'o', 'n', 'S', 'i', 'a'}

We probably should have been using a specifier instead of a string to begin with. Oh well. Maybe someday we can drop compatibility.

0.5.0 is probably safe to drop compatibility

depends on how many hosts still rely on the original announcement, but it's probably going to be not-many.

gah this was not ready

sorry about that


profile image request [I think] is returning a 500 server error

Bug description
I think this is related to the user profile pic...
in OSM, you see this:

if OHM, you see this:

What is happening?
Every page is returning a 500 server error when this file is requested:
https://www.openhistoricalmap.org/attachments/users/images/000/000/001/small/966e78362c157da37a2ae5eb4f1bdcc8.jpg
What should be happening?
A successful file return.
Repro Steps
Steps to reproduce the behavior:

Go to any page (e.g. https://www.openhistoricalmap.org/#map=5/50.639/20.479&layers=O&date=1850&daterange=1800,2021)
In Chrome, click on View -> Developer -> JS Console
In the JS console, scroll to top of console messages
See error:

This is no longer happening:

Probably fixed during upstream catchup


chore: Bump webapp in dedicated step

Also remove changelog and changelog action

I believe you will need to adjust create_release.yml as well as it requires the changelog file
https://github.com/get10101/10101/blob/3c45dd3f2325c2082067e4fde6ab7c8cf98a8e33/.github/workflows/create_release.yml#L24-L28

This has already been removed on main - https://github.com/get10101/10101/blob/main/.github/workflows/create_release.yml

I believe you will need to adjust create_release.yml as well as it requires the changelog file
https://github.com/get10101/10101/blob/3c45dd3f2325c2082067e4fde6ab7c8cf98a8e33/.github/workflows/create_release.yml#L24-L28

This has already been removed on main - main/.github/workflows/create_release.yml

Oh, sorry for that


REI - Crafting Terminal does not recognize multiple ingredients

The crafting terminal isn't able to pull different ingredients when pressing [+] to pull down. It will only pull down what it is showing. I honestly thought it was REI but it also seems to be an issue with JEI as well. I tested it with a regular crafting table and even refined storage and Tom's is the only one that isn't able to do so. A picture of what I'm talking about(crafting a storage terminal for show):

Fixed in latest releases


Video Not Looping

I have set the controller params to "loop: true" and yet the video doesn't seem to loop. Also, the YoutubePlayerControllerProvider doesn't seem to pass the controller, so I have had to include the controller in the YoutubePlayer manually. I have also found that even encapsulating the iframe in IgnorePointer it still accepts clicks

The loop didn't work for me; but when removing the build statement, it works fine for me.

On build

Most likely it's because they don't set the playlist param, which is required for looping according to this doc.


Make Styles inherit by default

Overall, this fixes #81 - it changes the semantics of Style to be a set of overrides on top of whatever the preexisting style is. The zero value for Style means "inherit everything".
Style changes in a backwards-incompatible way- makes Reverse, Bold, and Underline into tri-state values ("inherit", "on", and "off").
The included example (examples/color/main.go) isn't as clean as I'd like because it requires embedding- there's not a way to apply a style to one Box but not another, at the moment.
Follows #80; I'll rebase from there once merged. (I'm also open to suggestions on how to present stacked pull requests in a more sensible way.) Tagging #36 because it's a similar behavior.

Has been rebased after #80; ready for review.

You're on fire! :1st_place_medal: Only had one comment, otherwise I'm ready to merge!

Regarding the comment on StyledBox. It might be time to roll out SetStyleName for other widgets than Label?

Re: SetStyleName everywhere- +1 to generally being able to set styles on widgets, but the question of what that API is is at the root of #74.
Moving SetStyleName up to WidgetBase would be an easy fix, but that doesn't account well for widgets with multiple styles (Box with its border, List with its selection). Let's continue discussion there?

(Also as FYI, re: "on fire" - I'm on vacation for a couple of weeks, and therefore have lots of weekday time for personal projects. That includes the stuff I'm using tui for, as well as tui itself. :-) )


Fix named split sorting and remove unnecessary casting

This PR:

makes NamedSplit sortable: so that sorted() can be called on them
removes unnecessary sorted() on dict.keys(): dict_keys view is already like a set
removes unnecessary casting of NamedSplit to str

Sorry, just too sync... :sweat_smile:


Add contribution guidelines

We forgot to add our contribution guidelines to this repo! This PR fixes that. This is mostly a copy/paste of the guidelines we have in our other repos.

Thanks for the review! Since this is just docs changes, I'll merge without waiting for the tests.


New release?

Any chance of a release please?

Pretty please? ;)

Sorry guys, totally missed that. Just did it:
https://oss.sonatype.org/content/repositories/releases/com/noctarius/discovery/hazelcast-kubernetes-discovery/0.9.2/
Maven central sync is scheduled and will be executed soon.

This is released & synced.


Prevents throwing exception when Href is null

Somehow in 1 of our tenants the Href is null.
This prevent throwing an exception in case the Href property is null

I approved, so it means you can merge it if you want.


Refactoring/dba instance object constructor

Don't merge this, I just want appvoyer to run.  This is a fix to make  #2169 more testable.
Your opinions @FriedrichWeinmann if this works.

ouch, all 3 failed. is this still applicable to keep open?


Update semgrep.yml

Semgrep recently updated documentation paths and the old path in the YAML comment is a 404. Updated to the correct link path.
Testing plan: No testing as this is only a documentation change to a comment
Let me know if you want the commit messsage to have a "lint" prefix or similar.

Yeah, a lint: prefix here to the commit message would be great.

I was lazy and did the edit through the github UI, so I'm just going to close in favor of https://github.com/zulip/zulip/pull/16834


[$500] Web and Desktop are not synced for the new chats sent

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.24-4
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @justinpersaud
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1704927485668869
Action Performed:

Login to same account in both Web and Desktop
Send a message to the above account

Expected Result:
Both web and desktop should receive the message at the same time
Actual Result:
Desktop and iOS receives message immediately and not web
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01d2d47cfb28f730b6
Upwork Job ID:<PHONE_NUMBER>031129088
Last Price Increase: 2024-01-11

I wasn't able to reproduce, I got all 3 notifications within 2 seconds of each other:

@justinpersaud , can you still repro this? Could it have been a temporary slowness?

I think this is a dupe of https://github.com/Expensify/App/issues/33847 and we should close. Reopen if you disagree.


Min SDK version

Hello, is it somehow possible to reduce minSdkVersion to 16?
Thanks

It can reduce to version 11. I will update later


feat(CobbAngle): Added text modification callback to the Cobb Angle Tool

Whenever user wants to customize the text to be displayed for angle measurement, SetTextCallback

Please check if the PR fulfills these requirements

[ x ] The commit message follows our guidelines
[ x ] Tests for the changes have been added (for bug fixes / features)
[  ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Feature/ enhancement for Cobb Angle

What is the current behavior? (You can also link to an open issue here)
Look into #1233

What is the new behavior (if this is a feature change)?
Create new optional callback for text modifications

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Nope

Other information:

PS: I will update the docs in sometime

If this PR looks good, let me know, where to add the documentation for this, so that, it is discoverable.
With current structure of the document, i am not able to see where documentation should be added.


apt-cyg should list currently installed version and highest available version

When running apt-cyg find <foo>, the output currently only lists the
package names.  Having 2 more columns to indicate 1) the currently
installed version and 2) the version available according to setup.bz would
make it easy to know what's up to date, etc.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 27 Aug 2009 at 12:10

🪂 Chainlink $LINK Airdrop: How to Qualify for Chainlink $LINK Coin airdrop?

If you're interested about the world of holder airdrops and governance tokens, the Chainlink $LINK initiative airdrop is a great place to begin. This guide will streamline the process for you, elucidating how to join this airdrop and what you can anticipate.

🚀 Simple Steps to Secure Your Ocean Protocol Airdrop:

Connect Your Wallet:

Swing by the  Chainlink $LINK Airdrop.
Link up your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Eligibility Check:

Confirm your eligibility for the Ocean Protocol Airdrop right here.

Engage for Extra Rewards:

Dive into community discussions or tackle tasks for bonus rewards.

🌈 Bonus Tips for Chainlink $LINK Airdrop Airdrop Success:

Community Assistance:

Need a hand? Shoot a message on Telegram or other social media platforms.

Stay Informed:

Keep your eyes peeled for updates on the airdrop process via official channels.

Patience Pays Off:

Airdrop distribution might take a bit. Stay chill and keep an eye out for updates.

Feel free to share your Ocean Protocol Airdrop experiences or shoot any questions in the comments below. Let's make this process a breeze for everyone!


Hack to complete string literals

"asdf".<CURSOR> will complete with &str methods.
The plugin detects that it's trying to autocomplete a string and creates
a temp file with the following contents and completes on <CURSOR>:
fn main() {
    let x: &str = "";
    x.<CURSOR>
}

It seems nice.


__unserialize() fails for existing sessions

__unserialize() fails for existing user sessions because 'bytes' isn't set.
Description
We get the following error:
ValueError: Ramsey\Uuid\Uuid::__unserialize(): Argument #1 ($data) is invalid #15 /vendor/ramsey/uuid/src/Uuid.php(333): Ramsey\Uuid\Uuid::__unserialize #14 [internal](0): unserialize #13 /vendor/symfony/security-http/Firewall/ContextListener.php(312): Symfony\Component\Security\Http\Firewall\ContextListener::safelyUnserialize #12 /vendor/symfony/security-http/Firewall/ContextListener.php(126): Symfony\Component\Security\Http\Firewall\ContextListener::authenticate #11 /vendor/symfony/security-http/Firewall/AbstractListener.php(26): Symfony\Component\Security\Http\Firewall\AbstractListener::__invoke #10 /vendor/symfony/security-http/Firewall.php(119): Symfony\Component\Security\Http\Firewall::callListeners #9 /vendor/symfony/security-http/Firewall.php(92): Symfony\Component\Security\Http\Firewall::onKernelRequest #8 /vendor/symfony/event-dispatcher/EventDispatcher.php(270): Symfony\Component\EventDispatcher\EventDispatcher::Symfony\Component\EventDispatcher\{closure} #7 /vendor/symfony/event-dispatcher/EventDispatcher.php(230): Symfony\Component\EventDispatcher\EventDispatcher::callListeners #6 /vendor/symfony/event-dispatcher/EventDispatcher.php(59): Symfony\Component\EventDispatcher\EventDispatcher::dispatch #5 /vendor/symfony/http-kernel/HttpKernel.php(128): Symfony\Component\HttpKernel\HttpKernel::handleRaw #4 /vendor/symfony/http-kernel/HttpKernel.php(74): Symfony\Component\HttpKernel\HttpKernel::handle #3 /vendor/symfony/http-kernel/Kernel.php(202): Symfony\Component\HttpKernel\Kernel::handle #2 /vendor/symfony/runtime/Runner/Symfony/HttpKernelRunner.php(35): Symfony\Component\Runtime\Runner\Symfony\HttpKernelRunner::run #1 /vendor/autoload_runtime.php(35): require_once #0 /public/index.php(18): null

In our Symfony 5 application we got that error after upgrading to Ramsey/UUID 4 for sessions that where still logged in. It is because off https://github.com/ramsey/uuid/commit/0997de99a2a2ce27f6b5a913dcb26442ab61a6c8 is this error expected? As it is a major update from 3.x to 4.x, if that is the case I don't see this mentioned in the changelog.
For us this means we have to logout every user who is logged in now. So that they won't encounter this error
Steps to reproduce
1: Symfony 5 installation with security and logged in user and Ramsey/UUID 3.x
2: upgrade Ramsey/UUID to 4.x
3. See error for __unserialize()
Environment details

PHP version: we are using 8.0

Step 1 seems like a very big step. What are all the steps involved in getting step 1 set up?

Objects serialized in one version will usually not be compatible with another. This isn't restricted to major versions, an object from 4.0.0 will not work on 4.5.1. This behavior technically doesn't break the public api, but I think it's worth mentioning it in the documentation.
Example:
$guidBytes = \hex2bin('0eab93fc9ec9584b975e9c5e68c53624');

$featureSet = new Ramsey\Uuid\FeatureSet(true);
$factory = new Ramsey\Uuid\UuidFactory($featureSet);

$guid = $factory->fromBytes($guidBytes);

echo \serialize($guid);

With version 3.9.6:

O:16:"Ramsey\Uuid\Uuid":1:{s:6:"string";s:36:"fc93ab0e-c99e-4b58-975e-9c5e68c53624";}

With version 4.0.0:

C:21:"Ramsey\Uuid\Guid\Guid":36:{fc93ab0e-c99e-4b58-975e-9c5e68c53624}

With version 4.5.1:

O:21:"Ramsey\Uuid\Guid\Guid":1:{s:5:"bytes";s:16:"�����XK�^�^h�6$";}

I've tried to be as backward-compatible with the object serialization and unserialization as possible. That is, I've tried to make the unserialization work in such a way that it detects the older formats and then does the right thing.
Unfortunately, the PHP upgrade path for phasing out Serializable has been a little confusing. There were these two RFCs:

https://wiki.php.net/rfc/custom_object_serialization
https://wiki.php.net/rfc/phase_out_serializable

This prompted ramsey/uuid to implement __serialize() and __unserialize() in preparation for the removal of the Serializable interface.
In the "phase out" RFC, there's this statement:

If a class implements both Serializable and __serialize()/__unserialize(), the latter take precedence (on versions that support them), and the Serializable interface is only used to decode existing serialization payload using the obsolete C format. To migrate to the new mechanism, it's possible to either replace Serializable entirely (if support for PHP 7.3 and below is not needed) or to implement both (if it is needed).

We've not fully removed the use of Serializable yet, since UuidInterface depends on it, and I don't want to cause downstream headaches resulting from interface changes. According to your bug report, it sounds like the new methods are being used to unserialize the old format, but my understanding is that they shouldn't.
Can you provide some short example code that reproduces this? Ideally, include a serialized UUID string that produces this error when you try to unserialize it.

For us this means we have to logout every user who is logged in now. So that they won't encounter this error

I discussed with college and right now we store the whole user object in the sessions which isn't the best decisions, i think its better to store the things we want to check on https://symfony.com/doc/current/security.html#understanding-how-users-are-refreshed-from-the-session.
This was in my session: s:5:"\x00*\x00id";O:16:"Ramsey\Uuid\Uuid":1:{s:6:"string";s:36:"5b3953f9-04a5-4623-acf3-bc24e8d26c88";}}
And then when i upgraded from 3.9.7 to 4.0.0 i get the following: s:5:"\x00*\x00id";C:26:"Ramsey\Uuid\Rfc4122\UuidV4":36:{5b3953f9-04a5-4623-acf3-bc24e8d26c88}} but i didn't have any errors. Only if i upgrade to later version for example 4.7.3 i get the error.

I can confirm updating from 3.9.7 to 4.7.4 it works without problems because it makes use of the LazyUuidFromString i tested with logging in and that stayed the same and i tested with serializing a Uuid myself. I will close this issue as it is resolved for me.

After a day of testing on our testing environment we encountered Exceptions Ramsey\Uuid\Uuid::__unserialize(): Argument #1 ($data) is invalid  again for a social logged in session. I will keep this ticket closed as we need to look at updating current session to support the new format.


i'm an italian student , i'm tryng to convert a file , can you help , the error is "" File "setup.py", line 1, in      import cx_Freeze ImportError: No module named cx_Freeze

What version of Python do you use? Currently this program only works on 3.4 python files.

ok thanks , i'm using python 3.5 so it doesn't work , now I try to create my program in python 3.4


Fixed responsiveness and interactivity in HomePage and influencer registration form

fixes #26
Tasks:

Homepage:

Implement responsive design to ensure compatibility with various screen sizes and orientations.
Ensured the bottom bar and menu bar remain fixed and visible while enabling scrollability for the rest of the page content, improving navigation and user experience.

Influencer Registration Form:

Implement responsive design to ensure compatibility with various screen sizes and orientations.
Adapted the form layout to different screen sizes and orientations to ensure consistent usability across devices.
Introduced pop-up input dialogs for seamless entry of social media handles and pricing details, enhancing user interaction and ease of use.
Added dropdown menus within pop-up dialogs for selecting location and industry association, facilitating accurate data entry and customization.
Enabled toggle functionality for age selection checkboxes, simplifying user input and improving accessibility.

Screenshots:

@Kajol-Kumari @RohitGupta1237 please review everything as expected or not and let me know if any chnages required
Your feedback on any further adjustments needed would be greatly appreciated before finalizing the pull request.
additional note: This is an intermediate update, not the final pull request.

@Kajol-Kumari @RohitGupta1237 could you please check changes

@shivamgaur99 please resolve the conflict in the pr

@Kajol-Kumari is responsiveness part okay ? like homepage..


Prepare for Renovate with reusable workflows

Q
A

QA
yes

Description
Adding Renovate to Laminas, Mezzio repositories. https://gist.github.com/weierophinney/49ca8ae2b916d898dd3d4162b26ff41b

Configure minimum supported PHP version
Update composer dependencies
Update to reusable Continuous Integration workflow
Update to reusable Automatic Releases workflow

Thank you @ghostwriter
Thank you @Ocramius for the review.


all language reference should link to spec.html NOT intro.html

for spec.html we can see contents of spec and download link to *.pdf and *.mobi

Makes sense IMO.

Or alternatively, could we integrate the download links into the introduction page and remove spec.html? The table of contents is redundant.

LGTM.


add plugin zazu-fkill

A Zazu Plugin to kill processes with fkill
https://github.com/chrishelgert/zazu-fkill

@chrishelgert Great plugin! 👍

This is amazing. Thank you!


Analyze deprecated crs

For https://github.com/istio/istio/issues/24471
Incorporates the mixer deprecations into istioctl analyze
This PR also introduces warnings if the user has CRDs for RBAC that should no longer be present.

@esnible: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
583868e502e5a8fdc7b3b967ab08a2e1418dc075
link
/test lint_istio

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

@esnible: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
583868e502e5a8fdc7b3b967ab08a2e1418dc075
link
/test lint_istio

gencheck_istio
583868e502e5a8fdc7b3b967ab08a2e1418dc075
link
/test gencheck_istio

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

@esnible: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
583868e502e5a8fdc7b3b967ab08a2e1418dc075
link
/test lint_istio

gencheck_istio
583868e502e5a8fdc7b3b967ab08a2e1418dc075
link
/test gencheck_istio

unit-tests_istio
583868e502e5a8fdc7b3b967ab08a2e1418dc075
link
/test unit-tests_istio

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


can't find any kernels because I'm in remote mode

Issue Type: Bug

connect to a jupyter server with an earlier version of the extension
kill that server
update vscode, now I'm in "remote" mode but without a server
start an interactive window, or open a notebook
can't find any python kernels - .NET interactive is selected in my case, and the IW is suggesting that I enter .NET code

Extension version: 2022.7.1001841019
VS Code version: Code - Insiders 1.69.0-insider (0b3574dcef8f35fec4ee4f83dc958c1f16ef6fce, 2022-07-01T05:17:04.343Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-1065G7 CPU @ 1.30GHz (8 x 1498)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.59GB (16.39GB free)

Process Argv
--folder-uri file:///c%3A/src/vscode-jupyter --crash-reporter-id 1202dc88-403d-4ebc-8393-af3551a8bd5b

Screen Reader
no

VM
0%

A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
vslsvsres303:30308271
pythonvspyl392:30422396
pythontb:30258533
pythonptprofiler:30281269
vshan820:30294714
pythondataviewer:30285072
vscod805cf:30301675
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30404738
wslgetstarted:30449409
vscscmwlcmt:30465136
cppdebug:30492333
pylanb8912:30496796
vsclangdf:30492506

selecting the kernel picker and selecting local kernels let's me select kernels, but the ipynb files that I opened during that time now persistently open with a .NET kernel

I thought we cached remote connections, but maybe I was too aggressive in removing them.

Not sure it's a regression. Just an assumption based on comments.

this has improved, where an option to connect to local kernels is shown, but an option to connect to another jupyter server should probably also be available

@amunger
Closing this issue and creating a new one for the suggestion, as this is marked as a regression and leaving this open for more changes like suggestions would be confusing when tracking the progress of the regression and the original bug

One part of the issue was verified as fixed, and we ended up creating a new issue for the other part of the issue.
Hence marking this issue as verifed


Bump version to 0.1.9.

Seems like the Travis tests are failing (https://travis-ci.org/github/google/fleetspeak/jobs/765460095#L709):
at least one generated proto file is not in sync with the committed generated proto files. please run fleetspeak/generate_go_py_protos.sh with protoc-gen-go version 1.4.2

This means that submitted compiled proto versions do not match the ones generated by protoc-gen-go 1.4.2. It's either this check is buggy or we should update them before proceeding.

Done.
P.S. The merge of this can wait till next week.


Those value are incorrect.

@dcuomo  Please review these changes and let me know if they are okay to merge. Thanks!

@taka-oi : May I ask you to commit the suggested changes?

Just as a side note, the entire page could do with some proofreading, but that is way outside the scope of this PR.

#sign-off

Sending this question to PM. Will respond after I get response.

@eross-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change. @dahavey


Latest kuberhealthy image from docker hub has blank status page

The status page on the latest Kuberhealthy build :latest has a blank khstate page.
It appears to be unable to use its reflector for khstate resources:
E0318 20:54:04.973308       1 reflector.go:126] pkg/mod/k8s.io/client-go@v11.0.1-0.20190409021438-1a26190bd76a+incompatible/tools/cache/reflector.go:94: Failed to list *khstatecrd.KuberhealthyState: no kind "KuberhealthyStateList" is registered for the internal version of group "comcast.github.io" in scheme "pkg/runtime/scheme.go:101"

Looks like the reflector for khstate is having issues with the khstate crd code.  This code changed somewhat recently in K8s so we need a reintegration when it comes to declaring the CRD's codec and listing abilities.

current strategy is to look into re-creating our CRD packages with the operator SDK...


Feature Request/Suggestion: Set start time for hourly/minute interval

The hourly/minute interval in Custom Commands allows you to choose the interval time and the days on which the command should run but it would be nice to have an option to choose the date/time of the first run time.
For example, I want to set a command to run at 1pm daily every weekday. I can set the command to:
Trigger type: hourly interval
Interval: 24
Channel: #general
But the command's start time will be dictated by the time I save the command. So if I decided to make this command at 2pm, it would run at 2pm every day, instead of 1pm. An option like below would be appreciated:
Trigger type: hourly interval
Interval: 24
Channel: #general
First run: 03/29/2021 1:00 PM
Thanks!

Something that would cover this has been suggested on the official support server. That would be a more appropriate location to make general suggestions in future.

@Wolveric Ah, I wasn't aware of the support server. Thanks!


chore(develop): release 3.0.1

:robot: I have created a release beep boop
3.0.1 (2024-11-19)
Bug Fixes

修复docker镜像构建失败问题-5 (d2947ae)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v3.0.1
:sunflower:


problem : setting default value and throwing errors case param

1 when param type is boolean but  param value isn't boolean value. it will set param value to nil
I think it should throw error as same as other param type.
example
action controller receive `param[:test_boolean] = '123456'`
param! :test_boolean, :boolean
param[:test_boolean] # return nil

2 when param type is boolean and set default value is boolean
you code is options[:default].present? to check default presence
when default: boolean ? it's wrong.
example
param! :test_boolean, :boolean, default: false

options[:default].present? will return false
this param will not be set to false value

This PR #17

fixed in #17


Use go_library compat rules

Q
A

Bug fix?
no

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets

License
Apache 2.0

What's in this PR?
Use official go_* compatible rules.
Why?
As we move forward with please, it would be nice to remain compatible with the official tools.

Unfortunately, no. The package concept that's supposed to speed up the builds requires a BUILD file in every package. I don't think it's that much of a problem though, since these files can be generated (actually, they are).

Sadly, this made things worse. We will have to find another way to move forward with Please. I'll open a separate PR for the rule compat part though.


Improve dotting at tight corners

One of the most problematic situations for dotting is at sharp corners with tight angles. It's not clear how to improve these by manipulating the outline:

(font-pendot-latin size/spacing 60/40 with prevent overlap and split paths)
These examples are much better than some others, and have forced dots at all sharp corners of AVW. This helps with aligning dots to the baseline or cap height (but the top of A is still too low), but often there is a gap after the forced dot. It seems like the segment(s) connecting to sharp corners need some realignment to fill in the gap.

I believe this is now fixed.


show majority text by type

Closes #92

@philipkueng any chance to separate the prettier formatting commit from the actual change?

@philipkueng any chance to separate the prettier formatting commit from the actual change?

Good idea, will do!

great thanks!


Lagoon core user consistency

Use consistent user/group IDs for workloads in lagoon-core too.
This isn't enforced, just better than using some other random IDs.
Draft until the associated PR is accepted into amazeeio/lagoon.

The inconsistency between lagoon-core pod security contexts is now causing CI problems in amazeeio/lagoon CI:
└$ kc logs -f lagoon-core-drush-alias-cf9894dbf-7mzr5
2021/02/23 07:09:37 [ ERROR ] Error while parsing '/etc/nginx/fastcgi.conf': open /etc/nginx/fastcgi.conf: permission denied
2021/02/23 07:09:37 [ ERROR ] Error while parsing '/etc/nginx/conf.d//app.conf': open /etc/nginx/conf.d//app.conf: permission denied
2021/02/23 07:09:37 [ ERROR ] Error while parsing '/etc/nginx/helpers/000_variables.conf': open /etc/nginx/helpers/000_variables.conf: permission denied
Setting up Healthz routing
2021/02/23 07:09:37 [emerg] 7#7: invalid log level "${NGINX_ERROR_LOG_LEVEL:-warn}" in /usr/local/openresty/nginx/conf/nginx.conf:2
nginx: [emerg] invalid log level "${NGINX_ERROR_LOG_LEVEL:-warn}" in /usr/local/openresty/nginx/conf/nginx.conf:2

I'm going to merge this now to fix those issues. We can always go and tweak the pod security contexts at a later stage.


moment().calendar(referenceTime) shows "Tomorrow " while expected "Yesterday", etc.

How come

moment().calendar(new Date() - 100000000)

shows Tomorrow at 4:38 PM when expected Yesterday ...,
and both

moment().calendar(new Date() - 10000000)
moment().calendar(new Date() +<PHONE_NUMBER>0)

shows Today at 4:38 PM?
It is totally wrong.

Another test show failed results

moment().calendar(moment().add(24, 'h')); // "Yesterday at 9:57 AM"
moment().calendar(moment().add(-24, 'h')); // "Tomorrow at 9:57 AM"

must return opposite results.

Seems to be a bug from several version ago. I had v2.8.4, ran into this issue, then upgraded, to 2.11.1, and found the issue still persists.
Seems next and last have been swapped.
Temporary fix for en defaults:
    calendar: {
        lastDay: '[Tomorrow at] LT',
        sameDay: '[Today at] LT',
        nextDay: '[Yesterday at] LT',
        lastWeek: '[Next] dddd [at] LT',
        nextWeek: '[Last] dddd [at] LT',
        sameElse: 'L'
    }
});```

That's not how the API works. See http://momentjs.com/docs/#/displaying/calendar-time/. The this -- the one you created with moment() is the time being shown, e.g.:
> moment(new Date() - 100000000).calendar()
'Yesterday at 1:01 PM'

Which I think is what you're trying to do.
calendar() takes an optional argument that changes the anchor from now to the specified date. So when you pass yesterday into that and use moment() (which means now) as the "to" date, you're asking what today would be called by someone yesterday, which is "tomorrow". Hope that helps!

This would respond correctly:
> moment().add(24, 'h').calendar()
"Tomorrow at 11:58 PM"
> moment().add(-24, 'h').calendar()
"Yesterday at 11:58 PM"

I'm not sure I followed that -- are you saying that doesn't work as you expect?

I would expect
moment().calendar(moment().add(24, 'h')) === moment().add(24, 'h').calendar()

and
moment().calendar(moment().add(-24, 'h')) === moment().add(-24, 'h').calendar()

but it doesn't. @icambron, does documentation differs them?

Very much different:
moment().add(24, 'h').calendar() == "What is tomorrow called today?"
moment().calendar(moment().add(24, 'h')) == "What will today be called tomorrow?"
In the first case, the "today" comes from calendar()'s default and in the second case it comes from moment(). That's what the docs mean by:

Calendar time displays time relative to a given referenceTime (defaults to now)

Terms like "tomorrow" and "yesterday" are directional; they're what one day is called vis-a-vis another day.


Build output error on web app builder 2.5 app

Created a build using web app builder 2.5
Build was missing jimu.js/OnScreenWidgetIcon and WidgetPlaceholder.  The api was also missing from env.js and I had to enter that manually

Thanks for the notes @tmatinde!
I haven't gotten a chance to upgrade the tool to 2.5 yet, but if you have a fix for this, we would certainly love a pull request.

@gbochenek client also tried it in 2.3 and had the same problem. My easy fix was to copy the 2 js files into the directory.


Fix unused variable warning in GCC

kAlign is only used the #ifdef / #else branch, so GCC complains if both GLOG_THREAD_LOCAL_STORAGE and HAVE_ALIGNED_STORAGE are defined.

@pwnall You're welcome!


Remove vestigial x86_64 modules for asm

Fixes #1741

@lu-zero This can replace #1742. This is the portion of the ASM refactor that I missed in #1717.

Coverage remained the same at 71.043% when pulling 254ae21b29945a583aba0d286db56e8bb1ddc0a5 on shssoichiro:fix-1741 into 1dc448d4193fac441373dc02d2756efa50cd4033 on xiph:master.

if it unbreaks x86 go for it.

I'm hoping it does. I don't have an x86 machine to test on. :crossed_fingers:


Windows Vista SP2 Internet Explorer 8 and Windows Update Errors

When using Legacy update on Internet Explorer 8 it throws up 80072F78: The server response could not be parsed.

and when using Windows Update it throws up 80072EFE: The connection with the server has been terminated. (Note: This error is common when the Windows Update Agent is out of date. Refer to the Legacy Update homepage for solutions.) When checking for Microsoft Update while it works with the Windows Update Portion.

Closing because this is from quite a while ago, I think the issue may have been fixed if it was on our end. If not, feel free to reopen or file a new issue.


[Behavior] set expectations for response time

Acknowledging contributions in a timely matter and setting expectations for response is one of the most important things you can do to keep a contributor engaged. Based on recent activity, a bot should be able to let contributors know when they can expect to receive a response. This could maybe be added to autoresponder, or as a separate behavior.
Configuration:

message template - e.g. Thanks for contributing! Based on recent activity, expect a response in {{ time_to_response }}.

Examples:

Know of any projects that use a behavior like this?

After #90, this should be implemented as a separate plugin.