Marius/sort date

Fix

Styles now override Ant-defaults.
index.css -> index.scss.
DOM now has correct nesting of elements.

Changes

Changed every color-field to us SCSS variables.
Changed size of Card to diversify from table, and clear whitespace.
Introduced accent color and changed some elements to use it.
Increased visibility of breadcrumbs.
Updated colorscheme
Removed double search-text (we should adhere to the field moving up above the box when clicked/typed)

Add

Implemented SCSS variables for colors across all SCSS files.
Sorting of table on date now works.
Icons to Card to alleviate whitespace and make it less boring.

Conditional render now added. Other fixes also implemented.


Reviewer assing task fails on PRs from forks

The task fails with this error:
 Fetching configuration file from the source branch
Fetching changed files in the pull request
Identifying reviewers based on the changed files
Identifying reviewers based on the author
"per_author" is not set; returning no reviewers for the author.
Adding other group membres to reviewers if group assignment feature is on
Group assignment feature is disabled
Randomly picking reviewers if the number of reviewers is set
Requesting review to LysandreJik, osanseviero
Error: HttpError: Resource not accessible by integration

Example run: https://github.com/huggingface/huggingface_hub/runs/5659079126?check_suite_focus=true
ping @muellerzr

In any case with the simplification of the repo I was thinking the auto reviewer might not be very necessary anymore. WDYT @muellerzr @LysandreJik?

I actually completely agree with you Omar. Before it was good because we had essentially 3 different libs at once, so it was needed for delegation. Now that we no longer have that it can be removed. Feel free to assign it to me and I can get this in today!


Add build step

Reverts the previous merge that removes destructuring usages in favor of a build step similar to clay-kiln, in order to ensure ES5 output compatible with all browsers.
Adds the following npm scripts:

npm run prepublishOnly for a production build before publishing to npm
npm run build or npm run watch for a development build

Only the output bundle will be published to npm, rather than the source/test suite.

Coverage remained the same at 100.0% when pulling fc9fd0c4d6f5b9d0110e1f3778195447b2ed1bb4 on Entercom:add-build-step into 99791ffeaea3b41372b938517067a3d79e3b03c5 on clay:master.


Hide debugging functions in Land contract

Useful for debugging/development but should be private when deployed.

Privatized.


How to use glib.MenuModel?

I'm making a menu for an application window and can't figure out how to use glib.MenuModel to add sections, either there is a missing binding or I'm not understanding GTK3, or both.
I create a menu with glib.MenuNew(). Then I need to add sections, but menu.AppendSection takes a string and a MenuModel, not a menu. I see no way of converting from *glib.Menu to *glib.MenuModel, and I see no way of creating a MenuModel directly. According to the docs it is an abstract class, but where is its implementation?
Here is some example code from my application. The lines with AppendSection do not work, because section is a *glib.Menu instead of a *glib.MenuModel:
func NewMenu(model *Model, cmdNames string) *Menu {
	menu := Menu{}
	menu.m = glib.MenuNew()
	section := glib.MenuNew()
	sectionName := ""
	names := strings.Split(cmdNames, " ")
	for _, name := range names {
		if name == "" {
			continue
		}
		if name == "-" {
			// create a section
			menu.m.AppendSection(sectionName, section)
			section = glib.MenuNew()
			continue
		} else {
			cmd := FindCommand(model, name)
			if cmd.IsNull() {
				if !HasLoc(name) {
					log.Warnw(`key for menu is neither command nor valid translation`, "key", name)
					continue
				}
				sectionName = Tr(name)
			}
			// now we know it's a valid command
			section.Append(Tr(cmd.Title()), `win.`+cmd.CommandAction().GetName())
		}
	}
	menu.m.AppendSection(sectionName, section)
	return &menu
}

So, menu and model are wrapper classes you made yourself?
Anyway, it looks like a mistake has been made implementing the Menu wrapper. It seems that a Menu is actually an extension of the MenuModel. See here under "Object Hierarchy": https://developer.gnome.org/gio/stable/GMenu.html#GMenu-struct

So, just taking the pointer of the underlying MenuModel should work fine:
menu := glib.MenuNew()
sectionA := glib.MenuNew()
sectionB := glib.MenuNew()
menu.AppendSection("A", &sectionA.MenuModel)
menu.AppendSection("A", &sectionB.MenuModel)

Anyway, maybe it might be better to make an interface for MenuModel, then this would be a bit easier, but it's not really necessary.

Yes, sorry, it's not a minimal example. My app has its own wrappers for various things like db access, translation and undo/redo.
I hope you can fix this, because MenuModel seems to be used a lot.

Well, like said, for now there is a solution. Just do that for now.

Could you do me the favor and write a minimal working example with the workaround I provided? Would take some work of my shoulders, as I am currently also working on other stuff.

I don't understand the workaround. This doesn't seem to work:

menu := glib.MenuNew()
sectionA := glib.MenuNew()
sectionB := glib.MenuNew()
menu.AppendSection("A", &sectionA.MenuModel)
menu.AppendSection("B", &sectionB.MenuModel)

Anyway, maybe it might be better to make an interface for MenuModel, then this would be a bit easier, but it's not really necessary.

Menu contains a MenuModel, so just passing a reference to the underlying MenuModel should satisfy section *MenuModel.  I just checked whether it compiles. So, unless my VS Code lied to me, this should compile. Like I said, I am currently doing other things, which is also why I asked for a minimal example.


man facepalming emoji brings woman facepalming

How to reproduce:
enter facepalm in search

click on the man

click on the image on the right, then paste in twitter, see a woman facepalming glyph and a "man icon"

What I expected is the glyph from here
https://emojipedia.org/man-facepalming/
which is 🤦♂
I am using version 2.0.1 on firefox 74.0.1.

This extension appears to be unmaintained. Try https://github.com/johannhof/emoji-helper/issues/127#issuecomment-498349096

Yes, after creating the issue I realized that it is probably futile. But the other one brings a warning that it has not been evaluated by Mozilla.


How do I implement Page navigation using the Tabs Components?

About docs
Hi! What's the easiest way to implement Tab Components just to link to other pages in the way that both vercel.com and react.geist-ui.dev do? I'm surprised this isn't an example in the Tab Component library?
Languages

[x] English
[ ] Chinese

Description
Hi! What's the easiest way to implement Tab Components just to link to other pages in the way that both vercel.com and react.geist-ui.dev do? I'm surprised this isn't an example in the Tab Component library?
Page Links (Optional)

@Twoy519, Do you use Next.js?

Yeah Next.js

@ofekashery Yes I am using Next.js

@Twoy519 I think you can refer to the code for the implementation
Tab Menu
https://github.com/geist-org/react/blob/18c1afbc14d8d7ba7c6352d6a0084211d29d6f07/lib/components/menu/menu-sticker.tsx#L43

Hi @Twoy519, it might help you:
import React from 'react';
import { useRouter } from 'next/router';
import { Tabs } from '@geist-ui/react';

const Menu = () => {
  const router = useRouter();

  return (
    <Tabs value={router.pathname} onChange={(route) => router.push(route)}>
      <Tabs.Item label="Overview" value="/" />
      <Tabs.Item label="Projects" value="/projects" />
      <Tabs.Item label="Integrations" value="/integrations" />
      <Tabs.Item label="Activity" value="/activity" />
      <Tabs.Item label="Domains" value="/domains" />
      <Tabs.Item label="Usage" value="/usage" />
      <Tabs.Item label="Settings" value="/settings" />
    </Tabs>
  );
};

export default Menu;


Add environment parameter in config

Description
Checking the current environment / flagging by environment is a common use cases for projects I'm involved in, so this adds environment awareness to feature flags. You can set environment as a field in the Config passed to Init, then check them in rule (e.g. rule: env == "dev")
I realize I need to address tests/docs, but wanted to get feedback on the approach first to see if it was acceptable.

Changes include

[ ] Bugfix (non-breaking change that solves an issue)
[x] New feature (non-breaking change that adds functionality)
[ ] Breaking changes (change that is not backward-compatible and/or changes current functionality)

Closes issue(s)

Resolve #175
Checklist

[x] I have tested this code
[x] I have added unit test to cover this code
[x] I have updated the documentation (README.md and /docs)
[x] I have followed the contributing guide

Coverage increased (+0.004%) to 94.173% when pulling e137bcf1c18a057a62712784b5a6c41e98e1d5bf on neonmoose:main into a7d56d3fb35d5afa3c9d55296dadb889fa09d0bb on thomaspoignant:main.

Thanks for the feedback @thomaspoignant ! I think I've addressed everything, including the cleanup, and adding tests and docs for the change. Let me know if there's any other changes you think are needed.

Hey @thomaspoignant, yep - with this solution, env would be specified in the rule.
The advantage of having it be specified in the rule is that it allows for more flexibility for various use cases. In particular:

Enabling for multiple (but not all) environments. This is a very common use case on teams I've been on - for example to enable a flag in local development environments and staging environments but not in production

e.g. rule: (env == "dev") or (env == "staging")

Enabling the flag based on certain rules only in certain environments. For example it's enabled for all users in non-production environments, but in production it's only enabled for specific users

e.g. rule: (env != "prod") or (user_id == 1234)

I think having environment separate from rule, while might be easier in the simplest case, prevents it from being used in cases like the examples above. This would make it much less useful/powerful in my opinion.

My only concern is that we cannot have 2 different configurations for the same flag, and it could be a limitation.

I'm not sure what you mean by this - can you give an example?


Static pages: remove the partners and about us pages

Delete the routes, controller actions and views (and any applicable tests) for About Us and Partners pages as they are not required.

Can you assign this to me?

@LindseyB done

These pages are linked from the home and also the tenanted home, do we just want to remove those pages from the non-tenanted? or both?
http://localhost:3000/about_us vs http://localhost:3000/alta/about_us or should all be removed?

Ah good question. Let’s leave the about us and partners pages for the root org. Just remove for the orgs.


Fruit Ninja Game

Is your feature request related to a problem? Please describe.
No, my feature request is not related to a problem
Describe the solution you'd like
I want to make the fruit ninja game using HTML, CSS, and JavaScript.
Describe alternatives you've considered
No alternatives
Additional context
Nil

@Vanshiii203 Instead of fruit can I make a Ball ninja game? Like ball will be present in place of fruit.

Hey, @Vanshiii203 please answer the above-asked question.

Yes you can


Fix: Fixed crash that would occur when trying to display tags

Resolved / Related Issues

[X] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
Closes #issue...

Validation
How did you test these changes?

[X] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[ ] Are there any other steps that were used to validate these changes?

This may fix #12041 as well.


Process multiple XLS/XLSX files from a S3 bucket

Description
I am looking for IO catalog to read multiple XLS/XLSX files at once and load save them as parquet file under another bucket.
Context
Often we receive multiple excel files as datasets from clients and as of now, we are loading them individually through pandas which is inefficient. It will save lots of time to handle all files at once.
Possible Implementation
I have some work done in this direction and will like to contribute for review and help others in the organisation.
Possible Alternatives
(Optional) Describe any alternative solutions or features you've considered.

I have recently had the same kind of issue, and the kedro documentation didn't help me with it. Normally my dataset consists of multiple files in a bucket, and not a single file in a bucket. My solution for now is to create a separate pipeline step that looks up all the files in the cloud service and compiles them down to one local pickle dataset. That pipeline step has no input.

Thanks for raising this! @iver56 and @NeerajMalhotra-QB, we're releasing kedro 0.15.5 soon and we've come up with a way to load a directory of files or folders with something called the PartitionedDataSet. It will solve this issue in a file type agnostic way. I'm going to close this issue and the PR for this in the meanwhile.
But please check out the docs for this: https://kedro.readthedocs.io/en/latest/04_user_guide/08_advanced_io.html#partitioned-dataset


Fix: Generate Sales Invoice

Generate sales invoice option not working.
Please fix this issue.

Hello. The team added Generation of Sales Invoice per hour interval. Don't forget to set Generate Limit under Tail Settings also.

Please check #13
Please change generate limit to generate time.  For example if we set 12PM every day. System automatically check that time and there is any pending receipt sytem will generate automatically to invoice.

@sammish The team will check on generation by time.


Origin/lynex

Please add the related 2 labs in course to this chapter (add description text and link to lab folder) as experiment, template could follow chapter 7 and 8. This will boost the in action experience and book's system style.

代码实例换为，方便各章统一格式
code

汉化这个表格

chapter 6.5 还需要统一格式

Google Pathway and MOE distributed training have new pattern? like Dynamic/ Sparse activation? 如果有新pattern是否需要加入到pipeline并行后做趋势对比和展望？

好像缺少一块内容，task placement? （可以融合进算法对比或者框架或者新开section）例如在模型和pipeple并行里，这块有个优化问题，记得有google paper提过优化，以及有paper auto tuning这个过程。

Hello 山哥，我们先merge一版再开新PR否？

Please merge this to avoid long pending, please solve the conflict and solve the format issues.

Please refine in next PR


The default start option seemed to have led to a game breaking bug

I selected default from the little dragon in the room in Helgen.  I went outside and walked around a bit. Then I slept in the room.  When I woke up this seems to have triggered the vanilla execution scene but from the perspective of the people of Helgen.  When the carts came to a stop basically the game bugged out and the scene stopped progressing.  A few of the characters from the scene were there but doing nothing.  Then I walked around Helgen a bit and the rest of the Characters loaded but nothing was happening.  The issue was during this scene speaking with NPC's did not result in dialogue so it seemed like the game was unplayable.
I guess this is all from the mod Alternate perspective which I've never tried before.  It seems pretty cool but I would think could easily lead to bugs judging on how finicky the game can get is crashing the cart in the opening scene.
By the way, next time I tried a new start I choose the vanilla (Dragonborn) start and going to the keep with the dragon attacking the Dragon was just sitting on the ground for a while blocking the path and not attacking.  Eventually he took off though and the scent progressed.

Hi!

When the carts came to a stop basically the game bugged out and the scene stopped progressing. A few of the characters from the scene were there but doing nothing. Then I walked around Helgen a bit and the rest of the Characters loaded but nothing was happening. The issue was during this scene speaking with NPC's did not result in dialogue so it seemed like the game was unplayable.

In regards to your first issue, I have never experienced this in any of my play throughs, nor did I experience it testing it 3 times just now.  As such, I must ask If you have modified the list in any way? I am going to make some issue templates to make asking this easier.

By the way, next time I tried a new start I choose the vanilla (Dragonborn) start and going to the keep with the dragon attacking the Dragon was just sitting on the ground for a while blocking the path and not attacking. Eventually he took off though and the scent progressed.

This is something I have experienced before, even without Alternate Perspective. It doesn't happen every time, however. This is something I'm comfortable pinning on script load during this scene, as it's known to be (as you mentioned) the most script-intense part of the game. I've confirmed that there's no incorrect conflict resolution on the quest or packages in this part of the game.

If the first issue mentioned in your post happens to anyone else on an unmodified version of the list, we'll revisit this. Otherwise, I was unable to produce that issue.
Other issues mentioned here have been fixed for the next version.


Babili not working in 3.0.0

Do you want to request a feature or report a bug?
bug

What is the current behavior?
uglify js breaks spread operator in webpack3 only.  I am assuming at the moment it is uglify as I can disable it and things appear to start working.  I am still llooking into the details.
const r = Object.keys(selected).reduce((...args) => {
    console.log(args.length); // 4
    handleProcessReduction(selected, ...args);
  }, {});

function handleProcessReduction(selected, ...args) {
  console.log('p: ', args.length); // 1
}

I can see it is not doing the right thing:
return Object.keys(e).reduce(function() {
            for (var r = arguments.length, t = Array(r), n = 0; n < r; n++)
              t[n] = arguments[n];
            console.log(t.length), u.apply(void 0, [e, t]); // [e, t] not [e].concat(t)?
          }, {});

If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?
When using previous versions it would output with a concat and would work fine.
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
Noting #5110 and #5107 -- I thought they were different but looking deeper it appears it's the entire problem.  Updating webpack doesnt work because uglify is breaking spread operation in function arguments.  All of those errors appear to be due to the arguments not being there because they all get added to the first argument.
Update oops - meant babili not uglify

Not using babili and switching back to uglify fixes this.

report this to the babili plugin/loader.


Help - Scroll to top?

Question: Is there a way to scroll the flatlist to the top?
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

There is no built-in functionality to scroll the list to the top.
Can you describe the problem that you are facing in more details so I can see if there is a problem on the module side or it is just a feature request.
You can always get reference of the container you put the Flatlist inside and add a button that will set that container scrollTop to zero when clicked.


Update protect-azure-kubernetes-service.md

Cosmetic change in CLI style, keep consistency

@vhorne

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

Thanks @petetian


[Doc] add contributors

Type of Change
Documentation
Description
add contributors
Expected Behavior & Potential Risk
n/a
How has this PR been tested?
n/a
Dependency Change?
n/a

ci reject the word "Życzyński" even it's in dictionary.


Uncaught RangeError: Array buffer allocation failed

Hello!
this is my issues
Atom: 1.31.1 x64
Electron: 2.0.7
OS: Unknown Windows version
Thrown From: atom-typescript package 12.6.3
Stack Trace
Uncaught RangeError: Array buffer allocation failed
At buffer.js:38

RangeError: Array buffer allocation failed
    at new ArrayBuffer (<anonymous>)
    at typedArrayConstructByLength (<anonymous>)
    at new Uint8Array (native)
    at new FastBuffer (buffer.js:38:5)
    at Function.Buffer.alloc (buffer.js:245:10)
    at fs.readFile (ELECTRON_ASAR.js:530:29)
    at /packages/atom-typescript/node_modules/resolve/lib/async.js:125:13
    at /packages/atom-typescript/node_modules/resolve/lib/async.js:10:20
    at ELECTRON_ASAR.js:294:9
    at _combinedTickCallback (internal/process/next_tick.js:131:7)
    at process._tickCallback (internal/process/next_tick.js:180:9)

Commands
     -6:52.5.0 core:move-down (input.hidden-input)
  2x -6:51.9.0 core:backspace (input.hidden-input)
 19x -6:49.3.0 core:move-down (input.hidden-input)
 27x -6:47.9.0 core:move-up (input.hidden-input)
 10x -6:45.8.0 core:move-down (input.hidden-input)
     -6:37.7.0 core:move-up (input.hidden-input)
 12x -6:36.5.0 core:move-down (input.hidden-input)
 25x -6:34 editor:select-to-beginning-of-word (input.hidden-input)
undefined -6:32.7.0 core:select-left (input.hidden-input)
     -6:26 editor:toggle-line-comments (input.hidden-input)
     -6:25.3.0 core:save (input.hidden-input)
     -4:48.8.0 autocomplete-plus:confirm (input.hidden-input)
     -4:48.7.0 autocomplete-plus:activate (atom-text-editor.editor.is-focused)
     -0:27.1.0 pane:show-next-recently-used-item (input.hidden-input)
     -0:27 pane:move-active-item-to-top-of-stack (input.hidden-input)
     -0:17.2.0 core:backspace (input.hidden-input)

Non-Core Packages
an-old-hope-syntax 1.0.4
atom-css-comb 3.3.1
atom-ide-ui 0.13.0
atom-ternjs 0.18.3
atom-typescript 12.6.3
autocomplete-java 1.2.7
css-color-name 0.4.0
file-icons 2.1.25
ide-typescript 0.7.6
indent-guide-improved 1.4.13
javascript-snippets 1.2.1
styri-syntax 0.2.0

This seems to be thrown from the depths of Atom's internals (or, more precisely, Node's), and it seems to complain about being unable to allocate memory. My guess would be either your system memory was low at the moment, your Atom setup is somehow corrupted, or your system is completely unstable due to hardware problems. None of which sounds like a problem with the package. Please feel free to correct me if I'm wrong.


Stricter check for OpenGL version support

We use Qt's OpenGL support and OpenGL via GLEW. We have seen at least
one instance where there was a mismatch between these two and the
version detected by GLEW was < 2.0, which is the minimum version
required by Orbit. We therefore add a stricter check to also directly
check the version obtained by glGetString(GL_VERSION).
Tested: Manually ran Orbit locally.
Bug: http://b/178349019

@beckerhe Could you take a look what you think about this? I'm not quite happy with the change, as this is somewhat a shot in the dark.

@beckerhe Could you take a look what you think about this? I'm not quite happy with the change, as this is somewhat a shot in the dark.

Well yes, it's a shot in the dark. But I also don't see what we can do more at this point - without investing a lot more time. I like the change; all of it. The FATAL will give us better insights in case there is another crash. And the error message should be helpful to the user.


How to obtain Map Origin, Latitude/Longitude Coordinates, and Direction?

Hello, I am currently using OpenLane-V2 and have a question regarding the dataset.
Do you have data related to the map origin, latitude/longitude coordinates, and direction?
Any guidance or pointers towards relevant documentation would be greatly appreciated.
Thank you for your assistance!

We don't have a detail documentation about this. This dataset is built on the Argoverse 2 and nuScenes datasets. The localization information reletive to the global(city) coordinate can be found in ego pose.
The coordinate between city coordinate and GPS coordinate like WGS84 can be easily transformed. You can find something at #52.


Fix typo in secret.md

In following lines it's “username”, to maintain context consistency, I think it should be "username" here.

Doesn't look like a bug to me.

All right, thanks.


Typo "Azure SQL server"→"Azure SQL Server"

https://docs.microsoft.com/en-us/sql/t-sql/statements/create-database-transact-sql?view=sql-server-ver15&tabs=sqlpool

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

Closing per https://github.com/MicrosoftDocs/sql-docs/pull/5663#issuecomment-721231378.


Menu.html not working

Hi again xD, the module is not detecting the menu.html, any way I could debug this??

You can set breakpoint on appconfig.menu
https://github.com/viewflow/django-material/blob/master/material/frontend/apps.py#L88
https://github.com/viewflow/django-material/blob/master/material/frontend/templates/material/frontend/base.html#L73
When you solved it, can you please list here, what was the problem root.

@kmmbvnr  so the current_module is not been defined anywhere so the menu is never rendered


Fixup SubscriberClient.StopAsync when Token is Already Cancelled

If the token passed in as the 'hard stop' token is already cancelled
then the registration will never fire and the client will peform a
soft stop instead. This can take quite some time.
The fix is to check after we have registered on the token if the
token has been cancelled, and if so cancel the hard stop token source
too.

This should already be tested in the ImmediateStop test; so I'm slightly surprised that this might be a problem. I thought the Register method called the provided function immediately if the token is already cancelled.
I'll do a little more checking.

Hmm. Looking at the docs for Register it does say it should call it. I've seen the client take a long time to stop if the token was cancelled before StopAsync was called though and fixed it with something similar to this code outside of StopAsync.

I just tried this code on .NET Core and Full Framework, both on Windows, and in both cases the registered function was called:
var token = new CancellationToken(true);
token.Register(() => Console.WriteLine("Called!"));
Thread.Sleep(500);
Console.WriteLine("Done");

Please can you provide some more information about the environment in which you're seeing this problem?

@iwillspeak you say that you've added some code outside of StopAsync which prevents the client taking a long time to stop. Please can you share what this code looks like, as it may help point towards an explanation.
And what code are you executing within the subscription callback? If this is long-running, and doesn't properly honour cancellation, this can slow down stopping.

The subscribe callback can take a few seconds, and only checks the cancellation token at the start. That might be the source of the issue. A for the stop it looked something like:
var stopTask = subscriber.StopAsync(stopCts.Token);
using (var registration = token.Register(() => stopCts.Cancel()))
{
    if (token.IsCancellationRequested)
    {
        stopCts.Cancel();
    }
    await stopTask;
}

Guessing that it was the processing block that was causing the slow stop then. The change to the cancellation must have  been a red herring.

Thanks for closing. I think you're right that the callback is probably the problem.


Support cast <> String to interval

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
In DataFusion, we support SQL syntax like INTERVAL 3 months that is then converted into Intervals (I think originally contributed by @ovr).
However we currently only support converting individual values (rather than arrays)
Describe the solution you'd like
I would like the arrow cast kernel https://docs.rs/arrow-cast/32.0.0/arrow_cast/cast/fn.cast.html to support casting to/from strings and intervals
DataType::Interval(_) <----> DataType::Utf8/DataType::LargeUtf8
There is some subtlety related to parsing intervals (like if a given interval type can fit a specific range and precision)
Describe alternatives you've considered
Additional context
The datafusion parse_interval logic is here: https://github.com/apache/arrow-datafusion/blob/1f7885bb48dd33ce7b9df995214393bbff080e08/datafusion/common/src/parsers.rs#L115-L266
I think it would be fairly straightforward to port (along with tests) to arrow-rs
I don't think arrow should have the logic that tries to auto-detect the type of the interval -- in DataFusion I think we could simply change the code to try parsing the string as different types of intervals and use the one that first succeeds.

Related PR in datafusion: https://github.com/apache/arrow-datafusion/pull/5117

I'm interesting in this issue, may I take it?

That would be great -- thank you @doki23


Create Boavizta SCI CPU IMP

Name
bovizta.cpu.sci
Model

Create an IMP to represent the Boavizta Component Model see https://doc.api.boavizta.org/Reference/component_route/ specifically for a CPU
Calculates the emissions associated with the use of a specific CPU, e.g. Intel XEON, Graviton etc...
Returns the data in an SCI Impact Metric (just E and M for now)

Model Params
The IMM takes several inputs in the raw form
model
This is the type of chip so one call to the API will only return data for ONE type of chip.
{
   "core_units": 24, # this is the number of cores the utilization rate is refereeing to. A utilization of 100% on a 24 core system means you're using all 24 cores 100%.
   "name": "Intel Xeon Gold 6138f", # The specific CPU this refers to.
}

usage
"days_use_time": 1,         # duration in days/hours/mins
"time_workload": 50,      # %age util during that duration
"usage_location": "FRA", # region

other

verbose
allocation (linear/total)

Example YAML
  - component:
      model:
        path: bovizta.cpu.sci
      config:
        verbose: true
        allocation: total
        name: Intel Xeon Gold 6138f
        core_units: 24
      observations:
        shared:
          location: FRA                                    # usage_location
        series:
            -   datetime: 2023-07-06T00:00
                duration: 15s                               # translated into days_use_time
                cpu: 0.34                                      # translated into time_workload
            -   datetime: 2023-07-06T00:15
                duration: 15s
                cpu: 0.12
            -   datetime: 2023-07-06T00:30
                duration: 15s
                cpu: 0.01
            -   datetime: 2023-07-06T00:40
                duration: 15s
                cpu: 0.78

Observations

Will be in the format of CPU utilization, durations.
They additionally have a location which is either used to give more accurate embodied or if it's used for grid emissions we can ignore since we want to handle grid emissions ourselves.

Multiple Observations
series:
    -   datetime: 2023-07-06T00:00
        duration: 15s
        cpu: 0.34
    -   datetime: 2023-07-06T00:15
        duration: 15s
        cpu: 0.12
    -   datetime: 2023-07-06T00:30
        duration: 15s
        cpu: 0.01
    -   datetime: 2023-07-06T00:40
        duration: 15s
        cpu: 0.78

We need to call the API multiple times, once per observation.
Then sum up the outputs into one Impact Metric.
Perhaps we provide our own "debug" mode which

Eventually, we will need to convert the Energy numbers that are returned into Carbon with our own Grid Emissions data source that provides real-time grid emissions.
series:
    -   datetime: 2023-07-06T00:00
        e: 123 # convert this to O with the I for this specific time and duration using own own data source of grid emissions
        m: 34
    -   datetime: 2023-07-06T00:15
        e: 12 # convert this to O with the I for this specific time and duration
        m: 34
    -   datetime: 2023-07-06T00:30
        e: 13 # convert this to O with the I for this specific time and duration
        m: 34
    -   datetime: 2023-07-06T00:40
        e: 234 # convert this to O with the I for this specific time and duration
        m: 34

Outputs
This is the raw output from Boavizta
{
  "gwp": {
    "manufacture": 970, # embodied (m) in gCO2eq
    "use": 2100,
    "unit": "kgCO2eq"
  },
  "pe": {
    "manufacture": 13000,
    "use": 71020, # energy (e) in kWh
    "unit": "MJ"
  },
  "adp": {
    "manufacture": 0.15,
    "use": 0.000354,
    "unit": "kgSbeq"
  }
}

ImpactMetric (SCI style) will look like so for now:
{
 'e': xxx
 'm': xxx
}

Unit Tests
Unit Tests

Calling with observations calls the API and returns a value for e and m.
Call with a single observation returns a single impact metric with the same timestamp and duration.
Call with multiple observations returns a multiple impact metrics with the same timestamp and duration as the original observation.
If you don't provide mandatory inputs it returns an error (please decide what the mandatory inputs are in terms of config and observations).
When the Boavizta API doesn't work (400/500 whatever) it returns an error.
If you provide an cpu name type that isn't supported, returns an error

@jawache
Can you give me an explanation on
series:
    -   datetime: 2023-07-06T00:00
        e: 123 # convert this to O with the I for this specific time and duration using own own data source of grid emissions
        m: 34
    -   datetime: 2023-07-06T00:15
        e: 12 # convert this to O with the I for this specific time and duration
        m: 34
    -   datetime: 2023-07-06T00:30
        e: 13 # convert this to O with the I for this specific time and duration
        m: 34
    -   datetime: 2023-07-06T00:40
        e: 234 # convert this to O with the I for this specific time and duration
        m: 34

should the calculate endpoint return an array sorted by the datetime?

@gnanakeethan as per your comment above, yes return an array as you specify.

[ ] For embodied carbon, use LINEAR and take manufacturing as it is returned.
[ ] For energy the default is to take it as is from the PE section.
[ ] Update to the new spec and return an array as you describe above.
[ ] Very important, the unit tests above are new, this task won't be complete unless the unit tests are complete as per described above.

[x]  For embodied carbon, use LINEAR and take manufacturing as it is returned.
[x] For energy the default is to take it as is from the PE section.
[x] Update to the new spec and return an array as you describe above.
[] Very important, the unit tests above are new, this task won't be complete unless the unit tests are complete as per described above.


Rails 4.2, boolean-type MySQL columns coming accross as 0/1, not true/false

Is this one of the known problems in 4.2? If so, can be closed, but reporting just in case.
I have a column created in a migration as "t.boolean", in a MySQL database.
After fetching an object from this table, in Rails 4.1 and previous, the values show up as either boolean true or false. In Rails 4.2, the values show up as integer 0 or 1.
Of course, this causes bugs from software expecting true or false.
I guess I can probably work around this by some custom hook in my model that converts 0 and 1 to true and false on read, but it's messy.

there's probably more casting related regressions due AR changing its type-casting its guts in 4.2 ... we're still waiting for contributors to finish up or sponsor these: https://github.com/jruby/activerecord-jdbc-adapter/issues/599

Sweet, thanks!  Is there an ETA for a release that includes these improvements?


Custom entity view class fails bytecode analysis when using 2 stack slot types

A simple entity view like the following fails to work because our analysis currently assumes every parameter consumes a single stack slot:
    @EntityView(Document.class)
    static class CustomDocumentView {

        private final long id;
        private final String name;

        public CustomDocumentView(long id, String name) {
            this.id = id;
            this.name = name;
        }

        @IdMapping
        public long getId() {
            return id;
        }

        public String getName() {
            return name;
        }
    }

The stack slot of the parameter name is 3 instead of 2, which leads to issues in the analysis of the constructor.

The issue is flagged with "workaround available" label. Can you please elaborate on the nature of the workaround?

The workaround is to not use primitive types. In Kotlin, this probably means using nullable types i.e. Long?


Hard delete of package doesn't delete package owner requests

Repo:

Create package
Add new owner to the package
Hard delete the package through the Admin panel
Go to Account->Manage Packages

You'll see 500 server error due to NullReferenceException in OwnerRequestsListItemViewModel. The cause is that the PackageOwnerRequest still exists in the DB, but the PackageRegistration was deleted.

It looks like hard deletes also don't delete from the PackageRegistrationOwners table, but this doesn't block any UI or cause exceptions.

Hard deletes should delete from the PackageRegistrationOwners table according to this
https://github.com/NuGet/NuGetGallery/blob/master/src/NuGetGallery/Services/PackageDeleteService.cs#L25

I checked locally and it does in fact delete from the PackageRegistrationOwners table--I think if it didn't, the commit would fail.

Closing - should be fixed with #4898


Adding Identity Server 4 Admin UI

https://github.com/skoruba/IdentityServer4.Admin
https://github.com/brunohbrito/JPProject.IdentityServer4.AdminUI

Error
Sorry, there was an error : unauthorized_client
Unknown client or client not enabled
Client Application Access
Below is the list of applications you have given access to and the names of the resources they have access to.
You have not given access to any applications

Hi @stopcorp,
You can pull the latest code and start the ClassifiedAds.Migrator again to seed the Identity Server configuration for client applications automatically.
I've also finished the Admin UI, you can take a look at the Identity Server https://localhost:44367 to configure application settings directly via UI.
Cheers,

Thx. But i decided to set up on an other architecture.


Better default sort order, add option to specify order of columns in header

It's hard to know a good order for the header column. Alphabetical never seems to be ideal, frequency is nice, but sometimes all fields are present all the time. It should default to do nothing more than the order they come back in (to better match find semantics), and add an option to specify the order. The order will be specified as an array of field names (for now, only specify top-level fields), and only those fields will be displayed in the results.

Resolved in bcf0348da07ed7fa1acb616280008156fe0a3528


400 for create-file

I've just upgraded from 0.10.0 to 0.11.0 to give fine-tuning a spin (hopefully gpt-4 fine-tuning will be announced on monday!)
But when running:
(wkok.openai-clojure.api/create-file
 {:purpose "fine-tune"
  :file (clojure.java.io/file "one.jsonl")}
 {:api-key openai-key})

I get a 400 for the options object.
ExceptionInfo
   status: 400
   {:request-time 876,
    :request
    {:user-info nil,
     :as :text,
     :headers
     {"Accept" "application/json",
      "Authorization"
      "Bearer 🐻 ",
      "content-type"
      "multipart/form-data; boundary=hatoBoundaryQTa_33CmKkdIY26ZhtH98PDN6JM1t0",
      "accept-encoding" "gzip, deflate"},
     :server-port nil,
     :url "https://api.openai.com/v1/files",
     :http-request
     #object[jdk.internal.net.http.HttpRequestImpl 0x166de34c "https://api.openai.com/v1/files POST"],
     :uri "/v1/files",
     :server-name "api.openai.com",
     :version :http-1.1,
     :query-string nil,
     :body
     #object[java.io.PipedInputStream 0x41bf8c3c "java.io.PipedInputStream@41bf8c3c"],
     :scheme :https,
     :request-method :post},
    :http-client
    #object[jdk.internal.net.http.HttpClientFacade 0x3b2f4320 "jdk.internal.net.http.HttpClientImpl@5913bcdb(9)"],
    :headers
    {"openai-organization" "user-me",
     "server" "cloudflare",
     "content-type" "application/json",
     "access-control-allow-origin" "*",
     "content-length" "177",
     "alt-svc" "h3=\":443\"; ma=86400",
     "openai-version" "2020-10-01",
     "strict-transport-security" "max-age=15724800; includeSubDomains",
     "openai-processing-ms" "77",
     "connection" "keep-alive",
     "cf-cache-status" "DYNAMIC",
     "cf-ray" "820d386008c96718-AMS",
     "date" "Sat, 04 Nov 2023 13:31:57 GMT",
     "x-request-id" "93f9bb55cb879c92f269b550f44fb1e7"},
    :status 400,
    :content-type :application/json,
    :uri "https://api.openai.com/v1/files",
    :content-type-params {},
    :version :http-1.1,
    :body
    "{\n  \"error\": {\n    \"message\": \"Additional properties are not allowed ('options' was unexpected)\",\n    \"type\": \"invalid_request_error\",\n    \"param\": null,\n    \"code\": null\n  }\n}\n"}

I hadn't tried this on 0.10.0 yet, so I don't think it's a regression.
Any ideas?

Interesting, maybe OpenAI is more strict lately with request validation, but nonetheless, I fixed it so we do not ad an options key into the multipart fields.
Released v0.11.1

Cheers! Thanks for the quick turnaround :)


identifier blinks either host or world in hubble observe

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
A bug happened!
Hello. I have problem with network policies.
in a hubble observe -f i see
Dec 31 02:08:48.502: <IP_ADDRESS>:46538 (host) <- fraima-gatekeeper/gatekeeper-audit-6b8d6d54d7-wjbxs:9090 (ID:726615) to-stack FORWARDED (TCP Flags: SYN, ACK)
Dec 31 02:08:48.502: <IP_ADDRESS>:46538 (world) -> fraima-gatekeeper/gatekeeper-audit-6b8d6d54d7-wjbxs:9090 (ID:726615) to-endpoint FORWARDED (TCP Flags: ACK)
Dec 31 02:08:48.503: <IP_ADDRESS>:46538 (world) -> fraima-gatekeeper/gatekeeper-audit-6b8d6d54d7-wjbxs:9090 (ID:726615) to-endpoint FORWARDED (TCP Flags: ACK, PSH)
Dec 31 02:08:48.503: <IP_ADDRESS>:46538 (host) <- fraima-gatekeeper/gatekeeper-audit-6b8d6d54d7-wjbxs:9090 (ID:726615) to-stack FORWARDED (TCP Flags: ACK, PSH)
Dec 31 02:08:48.503: <IP_ADDRESS>:46538 (host) <- fraima-gatekeeper/gatekeeper-audit-6b8d6d54d7-wjbxs:9090 (ID:726615) to-stack FORWARDED (TCP Flags: ACK, FIN)
Dec 31 02:08:48.503: <IP_ADDRESS>:46538 (world) -> fraima-gatekeeper/gatekeeper-audit-6b8d6d54d7-wjbxs:9090 (ID:726615) to-endpoint FORWARDED (TCP Flags: ACK, FIN

I don't understand, it's normal that the identifier blinks either host or world, but network policies not work without setup iptables.
cilium-config -->
# -- Configure the kube-proxy replacement in Cilium BPF datapath
# Valid options are "disabled", "probe", "partial", "strict".
# ref: https://docs.cilium.io/en/stable/gettingstarted/kubeproxy-free/
kubeProxyReplacement: strict

k8sServiceHost: ${k8s_api_server_fqdn}
k8sServicePort: ${k8s_api_server_port}

cluster:
  # -- Name of the cluster. Only required for Cluster Mesh.
  name: ${cluster_name}
  # -- (int) Unique ID of the cluster. Must be unique across all connected
  # clusters and in the range of 1 to 255. Only required for Cluster Mesh.
  id: ${cluster_id}

ipam:
  mode: kubernetes
  operator:
    clusterPoolIPv4MaskSize: ${node_cidr_mask}
    clusterPoolIPv4PodCIDRList:
      - ${service_cidr}

operator:
  resources:
    limits:
      cpu: 1000m
      memory: 1Gi
    requests:
      cpu: 100m
      memory: 128Mi

resources:
  limits:
    cpu: 4000m
    memory: 4Gi
  requests:
    cpu: 100m
    memory: 512Mi

# -- Configure service load balancing
loadBalancer:
  algorithm: random
  mode: snat

wellKnownIdentities:
  # -- Enable the use of well-known identities.
  enabled: true

l2NeighDiscovery:
  enabled: true

# -- Allows to explicitly specify the IPv4 CIDR for native routing.
# When specified, Cilium assumes networking for this CIDR is preconfigured and
# hands traffic destined for that range to the Linux network stack without
# applying any SNAT.
# Generally speaking, specifying a native routing CIDR implies that Cilium can
# depend on the underlying networking stack to route packets to their
# destination. To offer a concrete example, if Cilium is configured to use
# direct routing and the Kubernetes CIDR is included in the native routing CIDR,
# the user must configure the routes to reach pods, either manually or by
# setting the auto-direct-node-routes flag.
#
# Deprecated in favor of ipv4NativeRoutingCIDR, will be removed in 1.12.
# nativeRoutingCIDR:

# -- Allows to explicitly specify the IPv4 CIDR for native routing.
# When specified, Cilium assumes networking for this CIDR is preconfigured and
# hands traffic destined for that range to the Linux network stack without
# applying any SNAT.
# Generally speaking, specifying a native routing CIDR implies that Cilium can
# depend on the underlying networking stack to route packets to their
# destination. To offer a concrete example, if Cilium is configured to use
# direct routing and the Kubernetes CIDR is included in the native routing CIDR,
# the user must configure the routes to reach pods, either manually or by
# setting the auto-direct-node-routes flag.
# ipv4NativeRoutingCIDR:
ipv4NativeRoutingCIDR: "<IP_ADDRESS>/8"

# -- Configures the use of the KVStore to optimize Kubernetes event handling by
# mirroring it into the KVstore for reduced overhead in large clusters.
enableK8sEventHandover: true

# -- Configure the encapsulation configuration for communication between nodes.
# Possible values:
#   - disabled
#   - vxlan (default)
#   - geneve
tunnel: "disabled"

endpointRoutes:
  # -- Enable use of per endpoint routes instead of routing via
  # the cilium_host interface.
  enabled: true

# -- Enables masquerading of IPv4 traffic leaving the node from endpoints.
enableIPv4Masquerade: false

# -- Enables masquerading of IPv6 traffic leaving the node from endpoints.
enableIPv6Masquerade: false

# -- Enable hostPort service support.
hostPort:
  enabled: true

# Disable iptables

# -- Enable Layer 7 network policy.
l7Proxy: false

# TODO: Add documentation
enableIdentityMark: false

# -- Configure whether to install iptables rules to allow for TPROXY
# (L7 proxy injection), iptables-based masquerading and compatibility
# with kube-proxy.
installIptablesRules: false

bpf:
  # -- Configure the eBPF-based TPROXY to reduce reliance on iptables rules
  # for implementing Layer 7 policy.
  tproxy: true

  # -- Enable native IP masquerade support in eBPF
  masquerade: true

nodePort:
  # -- Enable the Cilium NodePort service implementation.
  enabled: false
  bindProtection: false
  autoProtectPortRange: false
  enableHealthCheck: false

hubble:
  enabled: true
  metrics:
    enabled:
      - dns
      - drop
      - tcp
      - flow
      - icmp
      - http

  relay:
    enabled: true

  ui:
    enabled: true

localRedirectPolicy: true

externalWorkloads:
  # -- Enable support for external workloads, such as VMs (false by default).
  enabled: true

my main task is to work the full functioning of Cilium without Iptables. How can I fix this error without connecting iptables back?
Cilium Version
1.12.0
Kernel Version
Linux master-5dddc7a4-1 5.15.0-56-generic #62-Ubuntu SMP Tue Nov 22 19:54:14 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Kubernetes Version
1.23.12
Sysdump
No response
Relevant log output
No response
Anything else?
No response
Code of Conduct

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

I don't understand, it's normal that the identifier blinks either host or world

That may be a red-herring. I suspect we report world as the identity in some cases because we haven't resolved it to host yet.

but network policies not work without setup iptables.

What do you mean by that exactly?

I suspect that masquerading may also be playing a role here. That is to say, when traffic enters the node and gets SNAT'd to the host's IP, Cilium will continue to treat that traffic as "from world". However, for the reverse path, we may not recognize that while the response is destinated for a "host" IP, the connection will actually be reverse-SNAT'd (ie DNAT'd) after Cilium handles the traffic, and that the connection corresponds to the original connection from outside the node. AFAIK this should not have any policy consequences.

From our experience, we have determined that such an error can be reproduced if there are no marking rules from cilium in iptables.

Why were the marking rules from Cilium missing in iptables?

because we set
installIptablesRules: false

The primary task was to make cilium work only with ebpf without using iptables, because there are requirements from IS - do not touch the iptables configuration (they configure it themselves)


Add missing packages to Conda-Forge meta.yaml

Summary
Upon testing conda-forge installation for version 0.0.101 , there are few
missing packages that can possibly lead to ModuleNotFound Errors.
Adding packages to meta.yaml, should allow for conda-forge to install
these additional packages by default.
Additions

Added Regex , pydot , networkx==2.6.3 , ipython , jinja2 to meta.yaml

Changes
The wget package is called wget on Pypi and python-wget on anaconda . Hence the change in t
Todo List

[ ] Confirm Jinja2 version required for AIModelshare
[ ] Confirm Pydot version requirted for AIModelshare

@mikedparrott
As discussed on slack , I have opened the PR .
I tried to check what is the lower bound requirement for Jinja2 and Pydot for our library but could not find them .
Can you share your  thoughts on how to find this information ?


Creation of subfolders while uploading

fixes #41
Swapped the 'create Folder' checkbox to be a dropdown allowing the user to choose between:

Folder (creation of a folder for the files)
Current (upload files to the current location)
Subfolder (create a subfolder for each file)
It will check the uploaded files and try to determine automatically which mode to select.

Structural changes to upload.vue:

created regular expressions for filtering Images and Video files.  They are used in a few places so figured abstract that out.
added in the upload location types
Added in suggested upload location based on files
Split the pendingUpload function into folder creation and file upload.  This was necessary to support the subfolders.  I wanted to use the existing functionality of the uploadMixin and the fact that the existing uploaded folders properly set annotations.

Make sure you remove the ability to upload to to the current folder.  It might require changing the input from a dropdown back to a checkbox if there are only 2 options.

Swapped back to a boolean createSubfolders now that we talked about removing uploading to the current location.
Edited it so it will automatically suggest createSubFolders for lists of movie files
When clicking on createSubFolders a warning comes up to clarify what it is doing and suggest it only be used for video files.


fetch seems to timeout in Reeder.app with >9000 unread items

I have currently 10441 unread items, if i try to refresh in Reeder.app on my iPad, it says syncing, but doesn't seem to fetch new items. It looks like something times out.
Workaround: mark stuff as read, until >9000 unread items.
Log:
Jul 14 23:57:49 miniflux miniflux[16981]: [2018-07-14T23:57:49] [INFO] [Middleware:Fever] User #1 is authenticated
Jul 14 23:57:49 miniflux miniflux[16981]: [2018-07-14T23:57:49] [DEBUG] [Storage:SetLastLogin] userID=1 took 7.786745ms
Jul 14 23:57:49 miniflux miniflux[16981]: [2018-07-14T23:57:49] [DEBUG] [Fever] Fetching unread items for userID=1
Jul 14 23:57:50 miniflux miniflux[16981]: [2018-07-14T23:57:50] [DEBUG] [HTTP] MYIP POST /fever/?api&groups
Jul 14 23:57:50 miniflux miniflux[16981]: [2018-07-14T23:57:50] [INFO] [Middleware:Fever] User #1 is authenticated
Jul 14 23:57:50 miniflux miniflux[16981]: [2018-07-14T23:57:50] [DEBUG] [Storage:SetLastLogin] userID=1 took 3.668065ms
Jul 14 23:57:50 miniflux miniflux[16981]: [2018-07-14T23:57:50] [DEBUG] [Fever] Fetching groups for userID=1
Jul 14 23:57:50 miniflux miniflux[16981]: [2018-07-14T23:57:50] [DEBUG] [Storage:Categories] userID=1 took 266.470212ms
Jul 14 23:57:51 miniflux miniflux[16981]: [2018-07-14T23:57:51] [DEBUG] [Storage:Feeds] userID=1 took 281.546712ms
Jul 14 23:57:51 miniflux miniflux[16981]: [2018-07-14T23:57:51] [DEBUG] [HTTP] MYIP POST /fever/?api&feeds
Jul 14 23:57:51 miniflux miniflux[16981]: [2018-07-14T23:57:51] [INFO] [Middleware:Fever] User #1 is authenticated
Jul 14 23:57:51 miniflux miniflux[16981]: [2018-07-14T23:57:51] [DEBUG] [Storage:SetLastLogin] userID=1 took 9.133284ms
Jul 14 23:57:51 miniflux miniflux[16981]: [2018-07-14T23:57:51] [DEBUG] [Fever] Fetching feeds for userID=1
Jul 14 23:57:51 miniflux miniflux[16981]: [2018-07-14T23:57:51] [DEBUG] [Storage:Feeds] userID=1 took 68.142468ms
Jul 14 23:57:51 miniflux miniflux[16981]: [2018-07-14T23:57:51] [DEBUG] [HTTP] MYIP POST /fever/?api&saved_item_ids
Jul 14 23:57:51 miniflux miniflux[16981]: [2018-07-14T23:57:51] [INFO] [Middleware:Fever] User #1 is authenticated
Jul 14 23:57:51 miniflux miniflux[16981]: [2018-07-14T23:57:51] [DEBUG] [Storage:SetLastLogin] userID=1 took 3.730083ms
Jul 14 23:57:51 miniflux miniflux[16981]: [2018-07-14T23:57:51] [DEBUG] [Fever] Fetching saved items for userID=1
Jul 14 23:57:51 miniflux miniflux[16981]: [2018-07-14T23:57:51] [DEBUG] [EntryQueryBuilder:GetEntryIDs] condition=e.user_id = $1 AND e.starred is true, args=[1] took 79.110623ms
Jul 14 23:57:56 miniflux miniflux[16981]: [2018-07-14T23:57:56] [DEBUG] Sys=223886K, InUse=223886K, HeapInUse=210936K, StackSys=544K, StackInUse=544K, GoRoutines=18, NumCPU=1
Jul 14 23:58:26 miniflux miniflux[16981]: [2018-07-14T23:58:26] [DEBUG] Sys=251882K, InUse=251882K, HeapInUse=237552K, StackSys=544K, StackInUse=544K, GoRoutines=18, NumCPU=1

how can I debug that or can you help me?
Newest release at time of writing, freebsd 11.1, postgres (PostgreSQL) 10.3

I am using:
$ miniflux -version
2.0.x-dev

which is the version 2.0.9 for freebsd from the release section.

Looks like the same issue as #81

Can you try again with the latest version? many improvements have been made in the mean time.


The theme doesn't change when i change the song

When a new song plays the colors of the theme don't change.
They do change when i go to a song after it and go to the previously mentioned song.
there's the video of encountering the issue
https://gyazo.com/08de45e24fb06b43d373ad86a1a60557
win 10
spicetify 2.9.2
dynamic theme 3.5

I'm getting this behaviour too.

After reading a bit more, my guess is the problem is here: https://github.com/JulienMaille/spicetify-dynamic-theme/blob/f843ec3b99f44d71021855e334be846297c3dff2/default-dynamic.js#L13
See here how spicetify-cli made a "temporary" update to their built-in extensions to avoid using the Hermes protocol: https://github.com/spicetify/spicetify-cli/pull/1543
Unfortunately I don't have the knowledge or time currently to read into this and work out a solution to make in a PR. I might check back later but if not, good luck to anyone working on this!


remove the '-u' flag on the script bang declaration

this allows the evaluation of variables whether bound or not.  So that we can give a friendly message.
for safety, also place the evaluation of ${!1} in quotes

Yep! Was the unquoted ${!1} evaluation which was giving me grief. Thanks so much for fixing it, looking forward to using the new scripts!


Update ImageMagick 7.0.10 to -18

Microsoft Reviewers: Open in CodeFlow

The new binary path in the Url, to the exe does not exist.


Error: build-essentials missing

I get the following error when running gut:
[localhost] A dependency appear to be missing on localhost: build-essential

Looking through the code, this means either gcc or make is missing. I have both:
$ gcc --version
gcc-5 (Homebrew gcc 5.2.0 --without-multilib) 5.2.0
$ make --version
GNU Make 3.81

Am I missing something?

Same problem here. My target destination is a freebsd, but...
$ gcc --version
gcc (FreeBSD Ports Collection) 4.8.5
Copyright (C) 2015 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
$ make --version
usage: make [-BeikNnqrstWwX]
            [-C directory] [-D variable] [-d flags] [-f makefile]
            [-I directory] [-J private] [-j max_jobs] [-m directory] [-T file]
            [-V variable] [variable=value] [target ...]


TypeScript: Variable is used before being assigned / Block-scoped variable used before its declaration

What are the steps to reproduce this issue?
You can either download only the JSON or setup FastAPI project that produces the OpenAPI JSON.
JSON is here:
openapi.json
Note: You can skip this part if you only want to use the downloaded JSON for orval:

Download example FastAPI project
fastapi-example.zip
Unzip project, in terminal go to directory and run following commands:
python3 -m venv venv
source venv/bin/activate
pip install -r requirements.txt
uvicorn main:app --reload

Now you should be able to access OpenAPI JSON in http://localhost:8000/openapi.json
Run orval --input http://localhost:8000/openapi.json --output ./api.ts
OR
Run orval --input [download_dir_path]/openapi.json --output ./api.ts
What happens?
There should be this line of code:
export const ProposalCalculationRowUpdatePartialCostCategory = {...ManualRowCostCategoryEnum,...ProductCostCategoryEnum,} as const;

Which has TS errors saying "Block-scoped variable 'ManualRowCostCategoryEnum' used before its declaration." and "Variable 'ManualRowCostCategoryEnum' is used before being assigned.", same for ProductCostCategoryEnum. These are also clear JS errors as const variables cannot be referenced before assignment.
What were you expecting to happen?
Not have any errors that cause JS code to break (in this case ReferenceError) when running the file.
Any other comments?
Code is from fairly large project I'm working on so I simplified it as much as possible. The way Pydantic models are named and such seem to affect whether orval knows how to generate valid code so don't be too confused if things are named weirdly in the example project. However, it probably should still work, no matter how the schemas are named?
It seems that Orval 6.7.0 has fixed most (but not all) of these issues as with versions 6.0.0-6.6.0 I got around 117 exact same errors but with >=6.7.0 only 24 (when using the original code of my project).
What versions are you using?
Operating System: OSX / Mac Sonoma 14.2.1
Package Version: 6.25.0
Typescript Version: 4.7.4
Browser Version: Chrome 120.0.6099.216

Thanks to @AffeJonsson fixed for 6.29.0


99th_percentile_frame_rasterizer_time_millis in [flutter_gallery_ios__transition_perf, flutter_gallery__transition_perf_e2e_ios32] regression

https://flutter-flutter-perf.skia.org/e/?begin=1602654448&end=1602714273&keys=X1ff0283333c366cd3e7c124d9966ab3e&num_commits=50&request_type=1&xbaroffset=22001
Started at commit https://github.com/flutter/flutter/pull/68128, but I'm wondering if it could be due to https://github.com/flutter/flutter/pull/68148.

@jonahwilliams Is there an expected performance change from #68128?
@kf6gpe In this issue, too, you suspect the engine roll.  Do you think there is an error in our performance tests where attribution is incorrectly recorded?

The change linked only uninstalls applications after the driver script has run. It should not be affecting performance metrics

@jonahwilliams Unless the time it takes to uninstall the app is now being incorporated into the measurement.  The only other explanation is that our performance tests are incorrect, right?

The apps were being uninstalled before, so that seems unlikely. There was only a period of 5-10 commits where they weren't always uninstalled

This appears to be another case of misattribution.  The commit that perf tool points to doesn't seem to create a problem when tested locally.  Here were the results I got:
33903c17e18f6ebbc523f472ebdc02cdf0356061 "99th_percentile_frame_rasterizer_time_millis": 47.611,
767fb98a7b3b75bb6545cd707b9ad5103ae12df7 "99th_percentile_frame_rasterizer_time_millis": 47.24

I tested https://github.com/flutter/flutter/pull/68148 locally and saw a change of 5ms which doesn't match the 30ms increase shown in skia performance.  I'll dig deeper.
ed2a4809dae2c8faa643a55897d824c2006627fd - "99th_percentile_frame_rasterizer_time_millis": 46.346,
12bea61c2902784fec73b7f5f2ff3415f873fead - "99th_percentile_frame_rasterizer_time_millis": 41.092,

Hey @keyonghan I'm getting a 30ms jump in skia performance monitoring that I can't reproduce locally.  Is there something messing with the devicelab device?  Amber alert, etc?

Unfortunately we currently have no way to remotely monitor/dismiss popped up windows for the ios32 device..

What device are you running on locally @gaaclarke ?

That's not what the graph is showing, the graph is showing that there is a regression for the 32bit device and 64bit device.  Which makes it sound less likely to be a hardware issue, but still doesn't explain why I can't reproduce it locally.

Here's a recap of where we are:

The author of the commit expressing the regression is confident that it isn't to blame.  I agree it is very very unlikely the cause.
Local testing of that commit doesn't produce the regression
An engine roll was suspected, local testing shows no problem there
The regression appears on 32bit devices and 64bit devices which seems to suggest it isn't a hardware problem with one device (things like amber alerts would show up on both devices, however).  It also lessons the likelihood that the problem is device specific.

There is a chance that this problem for some reason don't show up in my iOS 13 iPhone 8 but does show up in both the devicelab devices.  @keyonghan what is the 64bit device we are using in devicelab?

All other iphones are with 64bit (go/flutter-devicelab-hardware).

I'm not sure how to proceed here.  The commit associated with the regression is one that the author doesn't believe caused a regression; I agree with the assessment; local testing doesn't show a regression, in fact it shows an improvement; yet the regression shows up on 2 of our devicelab units.  I have no plausible explanation why the commit in question would potentially cause a regression let alone cause a regression on one type of device and not another.
I don't even know why the engine roll was suspected since it has no proximity to the appearance of the regression.  If someone could suggest a plausible hypothesis for this regression I could continue exploring this otherwise I'm just stabbing around in the dark.

I'm going to revert https://github.com/flutter/flutter/pull/68128 and see if the number drop back down.

Please do not revert devicelab stability fixes

If the tests cannot function properly with the IPA/APK being uninstalled after each run, they need to be disabled.

The benchmarks clearly show a regression at that commit. Looking at it, I agree that if that commit is responsible then there's something wrong with the benchmark itself. We should revert the commit, let it sit for five or six commits to check the efffect, then re-land this commit.
If, during the revert period, we see the benchmarks revert back to where they were, we should investigate what's wrong with the benchmark itself.

You know, now that I think about it.  If a regression in performance happened after we started uninstalling the app, one likely culprit would be deleting the shader cache which could result in slower performance.  Thus a dip in performance would be expected.

I was finally able to reproduce this at 33903c17e18f6ebbc523f472ebdc02cdf0356061!  it requires running the test multiple times,  I went from ~46ms to ~6ms because of the shader cache.
So, this test needs the be re-baselined and we need to add a new test that measures performance with a full shader cache.  I'll file a bug for that and close the one I created about misattribution.  I'll also close the PR of the revert, thanks @jonahwilliams and @cbracken  for the help.  That one was a doozie.

@kf6gpe it looks like the tests auto rebaseline?  Can you make sure that it's baselined correctly please.

Awesome find. Thanks for digging into this @gaaclarke !


Updated secp256k1 to latest

Since we have no continuous integration build running for this module there is no way for us to tell whether the code compiles.

Ideally before merging any code into any project going forward we will have some continuous integration check that it compiles and even better runs a suite of tests verifying that the existing behavior has not been interrupted unexpectedly.

Before approving this pull request I am going to look into why Circle CI is not reporting build results here.

Here are posted the results for the latest builds on secp256k1
https://app.circleci.com/pipelines/github/ProofOfWorkCompany/secp256k1?invite=true

@KatrinaAS CodeFactor recommends the following changes be made to eliminate possible errors:
https://www.codefactor.io/repository/github/proofofworkcompany/secp256k1/pull/6

After speaking with Katrina I will merge this since it is simply rebasing off the latest version of the module.


How do I get the confirm / cancel button on IOS and customize the confirm / cancel button text?

Hi guys, I noticed that on android there're default ok / cancel buttons on the pop-up. But on IOS I don't see any confirm / cancel buttons, there're no props for me to customize any, has anyone figured out how to do it on IOS?

hello, thanks for asking! The ios date time picker does not have any confirm / cancel buttons, you have to create and style those by yourself.
Thank you!


feat: add correct link to article in new site

Thanks to the changes in electronjs.org-new we can now generate a
direct link to the exact article a user is visiting to make the
transition from one to another easier.

Rel https://github.com/electron/electronjs.org-new/commit/cf657d5f30068b14d6d1079c0a2be7c8890ab523

@nornagon @erickzhao told me you were interested in this feature!

@ckerr that's because the new docs live in a separate domain (https://beta.electronjs.org). In the production site we have Fastly that acts as a proxy and any request that goes to /docs/latest hits that domain (internally).
Not sure if I explained my self correctly. Happy to chat 1:1 about this because we will have to do the same for the blog soon-ish!
Also I've added a new "related commit" where I replace the search index with the one used to crawl the new docs so we are pointing more people to the new docs.


Converting to your plugin to react.js component

Hi,
I am just trying to use your plugin as a component in my reactjs application. I have abit changed your plugin slightly and pushed the codes to here
I then build the plugin by using
vue-cli-service build --target wc --name my-app src/App.vue
using the js file inside dis folder
I am trying to run my react app but I get below error when I use the js file inside my react app
import "components/PolygonCropper/my-app.js";
I am getting below error when I compile my react.js application
Failed to compile.

./src/components/PolygonCropper/my-app.js
  Line 139:4:      Expected an assignment or function call and instead saw an expression  no-unused-expressions
  Line 3990:475:   Expected an assignment or function call and instead saw an expression  no-unused-expressions
  Line 3990:1058:  Expected an assignment or function call and instead saw an expression  no-unused-expressions
  Line 3990:1238:  Expected an assignment or function call and instead saw an expression  no-unused-expressions
  Line 3990:1435:  Expected an assignment or function call and instead saw an expression  no-unused-expressions
  Line 37299:5:    Expected an assignment or function call and instead saw an expression  no-unused-expressions
  Line 37304:10:   Unexpected use of 'self'                                               no-restricted-globals
  Line 37304:33:   Unexpected use of 'self'                                               no-restricted-globals
  Line 37388:18:   'Vue' is not defined                                                   no-undef
  Line 37949:16:   Unexpected use of 'self'                                               no-restricted-globals
  Line 37949:36:   Unexpected use of 'self'                                               no-restricted-globals

Can you help me convert your plugin to a react component successfully?

Sorry this plugin is only for Vue application.

Sorry this plugin is only for Vue application.

Hi, I can successfully build your plugin as a web component (a js file) now I am trying to put this js file inside my react, I tried helmet and react-script-loader but without any success.
I am looking for  a way to integrate this into my react application!

Hi, I can successfully build your plugin as a web component (a js file) now I am trying to put this js file inside my react, I tried helmet and react-script-loader but without any success.
I am looking for  a way to integrate this into my react application!


Add .gitattributes file to streamline install as dependency

As discussed in this thread and recommended by The League it's possible to automatically exclude files and directories from a release package on GitHub. This results in those files and directories being excluded when the project is downloaded by composer as a dependency, speeding up the install and not wasting the user's time and disk space.
Here's one for Dotenv, using laravel/framework's .gitattributes file as an example.

:+1:

Good idea :+1:


Enforcing No Argument Modification in Testing Utilities

Is your feature request related to a problem? Please describe.
For problems that require students to write functions, we sometimes want to impose that the students' functions should not modify some or all of their arguments. To currently enforce this, a copy needs to be made of the arguments that aren't meant to be modified before running the students' function. Then, after the function is run, the copy needs to be compared to the arguments that were passed to the function. This occurs in cases where the arguments are data structures of some kind (e.g. Array, Stack, etc.).
Describe the solution you'd like
Since this process is easily generalizable, I believe that this should be added to the testing utilities. In particular, it could be added to the run_generic_test function.
The first question is how the user should specify which arguments should be checked for modification. This could be specified in a parameter enforce_no_mod:

By default, enforce_no_mod is False indicating that all of the arguments can be modified.
If enforce_no_mod is True, then none of the arguments can be modified and should all be checked.
Lastly, enforce_no_mod can be a list of bool of equal length to the existing params parameter. enforce_no_mod[i] = True means params[i] should be checked for modification. enforce_no_mod[i] = False means that it should not.

The second question is how run_generic_test will know how to copy any arguments that need to be checked. This can be specified by a list copy_fns such that copy_fns[i] is a function that can be used to make a copy of params[i]. Since params can be a single value (i.e. not a list), it may be good to allow copy_fns to be a single function also. This should be provided a default value as to not have to change existing testing code (None is sufficient).
The third question is how run_generic_test will know how to compare the argument passed to the students' function and its copy. This also can be specified by a parameter copy_eqs_fns that operates in a similar manner to copy_fns except where copy_eqs_fns[i], in the instance it's a list, is a function that checks the equality of params[i] to its copy. This should be provided a default value as to not have to change existing testing code (None is sufficient).
Describe alternatives you've considered
The alternative is the current process described in my answer to the first prompt.
Additional context

An additional reason to take this approach is that it will allow the tests display an error message describing that arguments were modified when they should not have been. This could not be easily accomplished by offloading this complexity to the user through modification to custom_comparator to include an optional params=None parameter. In such a case, the tests would simply fail despite the logged output and expected output being the same.


Addon doesnt start anymore

The problem
Error: Image ghcr.io/brenner-tobias/cloudflared/amd64:4.0.2 does not exist for addon_9074a9fa_cloudflared
What version of Cloudflared has the issue?
4.0.2
What was the last working version of Cloudflared?
4.0.1
What type of installation are you running?
Home Assistant OS
Add-on YAML Configuration
No response
Anything in the logs that might be useful for us?
No response
Steps to reproduce the issue
Start 4.0.2 in Homeassistant.
Additional information
No response

had no problem with it; however was previously 4.0 and needed to reset/delete the tunnel so I uninstalled the addon and then rebuilt anything on a fresh install of the addon; may try that.

Deleting the addon, and reactivating it solved it, somehow something hangs it seems.


Adjust L1 resolution gas limits

Currently the gas limits in the CrossDomainMessenger.sendMessage calls are arbitrary chosen.
We need to measure the cost of them and adjust accordingly.

@fredo what do we do here?

I think it is fine to stick with the current abitrarily chosen value (1000000). we have seen gas usage of around 800k per L1. not priority right now

Ok, then closing this for now. We can reopen or make a new issue once it becomes relevant.


Created Dail-e Lambda Function

This function will currently do the following:

Scrap WOTD XML Feed for Today's word
Locate today's WOTD in database- must run query to find ID. Set result to variable wordID.
Set today's Word of the Day to active.
Find any other active WOTD items that are not today's word.
Loop through active WOTD items that are not supposed to be active and set to inactive.

To Do:

[ ] Add functionality to handle multiple languages. Currently hard coded for Spanish.
[ ] Research and add capability to schedule function to run.
[ ] Add in lastActive timestamp.

For lastActive timestamp, would it be best is I stripped out the time and just included a date in YYYY-MM-DD format? I think the date should suffice for our purposes and the time adds complexity. I think DynamoDB can support this. Any thoughts?

Oh weird. Could of sworn those indents are in my file on my laptop. I'll fix.


Improvements to context/account/clouds

This feature is not well documented and is not necessarily straightforward to use.
This is due to wanting to maintain compatibility with xplat CLI whilst also exposing new functionality.
Related to this question - https://github.com/Azure/azure-cli/issues/1559#issuecomment-267110057.

My suggestion is to:

Remove the context commands for now
Ensure that we have an environment variable I can set to indicate which cloud I'm currently using

When we add profile support:

Add a default profile attribute to the cloud configuration
Ensure that we have an environment variable I can set to override the currently selected profile


USB port

Hi @headkaze
I have a question. The thunderbolt type c should be set to “Internal” or "TypeC+Sw". I don't know how to set it correctly. Thank you for helping

I have a similar question, what is the difference between 'TypeC+Sw' and 'TypeC'?

Hi guys, you need to check your IORegistry for the value assigned by the system. On my system, due to the fact that I have both XHC as the USB3 controller and XHC as the USBC controller, the system (Mojave latest) set the value of UsbConnector to 9. Find out your value with IORegistryExplorer tool and set it accordingly.
Normally, the value is 9 by default for such controllers, such as on my embedded USB-C on my Intel NUC9i7BEH2 but depends on hardware on your motherboard, of course.
When I was building my SSDT-UIAC.aml the values that UsbConnector can accept are (per https://github.com/corpnewt/USBMap/ page)
0: Type A connector
1: Mini-AB connector
2: ExpressCard
3: USB 3 Standard-A connector
4: USB 3 Standard-B connector
5: USB 3 Micro-B connector
6: USB 3 Micro-AB connector
7: USB 3 Power-B connector
8: Type C connector - USB2-only
9: Type C connector - USB2 and SS with Switch
10: Type C connector - USB2 and SS without Switch
11 - 254: Reserved
255: Proprietary connector

Thanks @mackonsti, out of interest do you know if choosing the correct connector for a port in Hackintool makes a difference to the actual port or it it purely cosmetic/reference in Hackintool?

Hi @aptonline I remember discussing this with @headkaze over at InsanelyMac and he was quite clear about this, better always use the true electrical type of the connector. A hardware USB 3.0 connector should be marked as UsbConnector=3 for example, and in the case of USB-C if we see the choices here (above) you see UsbConnector=9 as being the first and most likely correct option. I think "Switch" refers to the internal driver system, not a physical switch obviously, so "9" is a good place to start. Hopefully when you fix all this, you can test external devices with success (and expected behaviour) On my Intel NUC8i7BEH2 the connector is used for display purposes with a cable USB-C to MiniDP with excellent results.

I have a similar question, what is the difference between 'TypeC+Sw' and 'TypeC'?

If you pluging in your USB-C cable, rotating it 180 degrees then pluging it back in lights up two separate rows green in Hackintool, then you have TypeC for both of those rows. If the same row lights up green whichever way the cable is plugged in, you have TypeC+Sw.
As for not having ports set correctly, I experienced issues with my USB dongles randomly ejecting when they were set incorrectly so there must be some magic happening.

@jamesfawcett - my hack has a single physical USB-C port that is a USB 3.1 port.  When I rotate a USB2 device in the USB-C port, the logical port retains a single "HS10" identity.  When I rotate a USB3 device in the USB-C port, the logical port identity alternates between "SS03" and "SS04" depending on the physical connector orientation.  Since my USB-C port's USB2 "orientation" behavior is different from my USB-C port's USB3 "orientation" behavior, is it correct to label HS10's Connector as "TypeC+sw" and to label the SS03 and SS04 Connectors as "TypeC" ?
I currently have my USB-C port logical ports (for the single USB-C physical port) defined as follows (as viewed in Hackintool)
  Type    Name    Location ID     Port    Connector       Comment
  XHC     HS10    0x14600000      0x0A    TypeC+Sw        Front USB-C (USB2)
  XHC     SS03    0x14A00000      0x13    TypeC           Front USB-C (USB3)
  XHC     SS04    0x14B00000      0x14    TypeC           Front USB-C (USB3)

@jamesfawcett - my hack has a single physical USB-C port that is a USB 3.1 port.  When I rotate a USB2 device in the USB-C port, the logical port retains a single "HS10" identity.  When I rotate a USB3 device in the USB-C port, the logical port identity alternates between "SS03" and "SS04" depending on the physical connector orientation.  Since my USB-C port's USB2 "orientation" behavior is different from my USB-C port's USB3 "orientation" behavior, is it correct to label HS10's Connector as "TypeC+sw" and to label the SS03 and SS04 Connectors as "TypeC" ?
I currently have my USB-C port logical ports (for the single USB-C physical port) defined as follows (as viewed in Hackintool)
  Type    Name    Location ID     Port    Connector       Comment
  XHC     HS10    0x14600000      0x0A    TypeC+Sw        Front USB-C (USB2)
  XHC     SS03    0x14A00000      0x13    TypeC           Front USB-C (USB3)
  XHC     SS04    0x14B00000      0x14    TypeC           Front USB-C (USB3)


[Suggestion] Possibility to placed the projector facing the other way

Right now when you placed the projector, it always face the way you looking at. I would like to have a way to placed the projector the other way.
Why?
Right now you always have to be in the back of the projector to be able to place corectly the projector facing the good way. If you are on a close room, you will have to mine a hole in the wall, to go in the back question to place it right. And at the end, it take a bit more time even if you are on a open space, because you have to go little ferther in each dirrection, to place the projectors...
How can this be done in the mod?
There have multiples way you can do that, I think this can be the new default way to place directly without any issue, but this can be done like multiples other mods does, by a Shift Right-click question to invert the placement way.
Version: 1.0.0 Beta 3
Minecraft: 1.16.5
Forge: 36.2.1

Already in place for the next release, per this commit: 4928b4598776a78beb140ed6021cfc54d687de5c
It'll be activated when you hold shift, per the suggestion.


OverrideOptionsOnConnectionHandle must provide HttpRequest as callback data

Describe the feature
Adding custom SetHttpClientFactory and custom CurlHttpClient if great but there is no link between client and served request in function OverrideOptionsOnConnectionHandle.
Use Case
Implementing simultaneous and dynamic SSL checking and verification via SSL_CTX_set_verify (check for some connections, discard for other connections, etc.) with CURLOPT_SSL_CTX_FUNCTION is not possible because HttpClientFactory is static and there is no way to understand which option to use in OverrideOptionsOnConnectionHandle.
Proposed Solution
Declare OverrideOptionsOnConnectionHandle with additional parameter HttpRequest* and when invoked in CurlHttpClient::MakeRequest pass "request.get()" along with curl connection handler.
Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request
[X] This feature might incur a breaking change

Thanks for taking the time to create this feature request. This is something that we might want to add in the future, but it is not a priority right now. If you are interested in having this additional parameter added please 👍 the original comment so we can gauge interest.


Add filter method to Array class

Designed in a similar fashion like sort_custom
instead of using FuncRef for the sake of consistency.
The error isn't always accurate, sometimes it is actually
due to the wrong number of arguments passed into the filterer
method.

I screwed this one up by deleting the source branch. Created a better one with a proper feature branch instead and as a draft PR. Won't happen again. Sorry. ##30874
Closing this one.


Add a warning if vagga.yml is present in the directory hierarcy

🙍

Well, I don't think it should. Mostly because if we support both yml and yaml there is an ambiguity of which is chosen when both are present.
I think the times when more than three character extension was a problem have already passed.

By the way, if it is just common case for mistakes you're doing, we can add a warning if there is vagga.yml but not vagga.yaml. Is it something you appreciate?

It totally is. I guess, warning when there are both files can be helpful too

Okay, I'm reopening the issue and marking it "easy". If no-one provides a pull request I'll fix it later.


[COASTAL-97] (WIP) Alert at time of day

https://tidepool.atlassian.net/browse/COASTAL-97
@rickpasetto since you built the alerting system, I wanted to get your input on this addition. It allows for an alert at a specific time of day and it may repeat.
Also see: https://github.com/tidepool-org/Loop/pull/468

@rickpasetto is adding this feature with https://github.com/tidepool-org/LoopKit/pull/440


Positioning Issue: Tip Window appear to be more bottom on Android 11

Run the demo app on Google Pixel 5(Android 11) and you will see what similar to below. The ToolTip position seems to be more "bottom" than expected.

I'm sure that is not related to night theme, and not even related to gravity.

I don't know if this is related to Pixel (5) or Android 11, though.

using AVD Pixel 3a API 30 x86 can cause similar problem.

Android 8 and Android 10 cannot re-produce the issue, no matter what device it is. I guess the problem comes along with Android 11 (API 30)

Facing the same issue on Google Pixel 2 XL (Android 11)
Found any solution @samuelchou ?

So I banged my head against this issue, it seems to only be happening on Android 11 (not on Android 12).
Looks like you need to subtract the status bar height to get it aligned correctly.
val tooltip = Tooltip.Builder(context).anchor(this, 0, determineYOffset(context), true)

private fun determineYOffset(context: Context): Int =
          if (Build.VERSION.SDK_INT == Build.VERSION_CODES.R)
              -((context as? MainActivity)?.statusBarHeight ?: 0)
	else
              0

Also, it looks like if the device has a notch/cut out, you need to compensate for its height as well. Based on the example below:
private fun determineYOffset(context: Context): Int =
    if (Build.VERSION.SDK_INT == Build.VERSION_CODES.R) {
        val activity = context as? MainActivity
        val statusBarHeight = activity?.statusBarHeight ?: 0
        val cutoutHeight = activity?.window?.decorView?.rootWindowInsets?.displayCutout?.safeInsetTop ?: 0
        cutoutHeight - statusBarHeight
    } else {
        0
    }

@zarsky When I tested out your method on a device with a notch the cutoutHeight was equal to statusBarHeight which made yOffset 0 again. Can you specify for which case you needed cutoutHeight as well?
Also, I got statusBarHeight using the following method.
val metrics: WindowMetrics = activity.windowManager.currentWindowMetrics
val insets = metrics.windowInsets.getInsets(WindowInsets.Type.statusBars())
val statusBarHeight = insets.top


artist features are combined into new artists

hey :)
when my songs are tagged with more than one artist, musikcube will take those two artists and turn them into a new one in the artists library bar, but I don't want to have 14 different menus of the same artist and instead have one song that is tagged with "artist1, artist2" shown under both artist1 and artist2 instead of a new artist called "artist1 artist2"
I'll attach an example

This issue is maybe related to my request. Please use the album artist as a way to sort by artists, that way when I e.g. have a genre CD called Classics V2, I can but that name as an album artist. That way not all titels of this genre CD get listed as a separate artist in the list that @alinawallpainter showed.

Yep, in this case you can just change the view from "artist" to "album artist" and it should fix your issue.

where does one change the vieuw from artist to album artist ?


Broken Profiler

I dont think its working correctly?

I confirm. Missing some info

Is your ini FreezeRestartTime > 0?

Profiler without TMain (not working)

Profiler with T-MAIN working

Bug Begin maybe on this one:
https://github.com/Sphereserver/Source-X/commit/89e27bfd3e578e67b5c8973890e9c5d952edc977
(I'm not able compile it on my PC)
Or this one https://github.com/Sphereserver/Source-X/commit/75a07b4498bd62837c6f51afa968fb5c8789df64

Profiler without TMain (not working)
Profiler with T-MAIN working
Bug Begin maybe on this one: 89e27bf (I'm not able compile it on my PC)
Or this one 75a07b4

how do you activate Tmain?

how do you activate Tmain?

you can't, I compiled an old build

Confirm Fix on dev branch

We now Have this on boot

Usefull?

Guess so, i added it lol


CodeGuru for Repo Analysis

Once the project progresses further and the actual repositories are in a good state, one of the basic features we will want to add is a Dependabot-like feature that uses CodeGuru to run analysis on our users' code.

Note: we'll probably create a feature branch for this and create separate dev branches for api, ui, infra separation.

Originally posted this as CodeWhisperer - sorry I mixed up services. I meant CodeGuru.


use nicknames when possible

when the moved message's author has a nickname, the executed webhook's username should be set to the author's nick

When a message is moved, if the author of the message has a nickname, the executed webhook should use that nickname for the username. For example, if the original author is known by the nickname "ChatMaster," the webhook's username should be set to "ChatMaster" instead of the author's default username. This approach helps maintain consistency and personalization in the communication process, ensuring that the user's preferred nickname is reflected kik usernames to explore in all automated updates.


feat(agenda): add new module for job scheduling with agenda

Information

Type
Breaking change

Feature
Hopefully not

Usage example
see readme
Todos

[x] Tests
[x] Coverage
[x] Example
[x] Documentation

Hello @ochrstn,
Thanks for this PR and Good jobs ;)
I added some simplification suggestion. Tell me if you have any question.
See you
Romain

Hello @ochrstn,
Thanks for this PR and Good jobs ;)
I added some simplification suggestion. Tell me if you have any question.
See you
Romain

Thanks a lot for your feedback and suggestions. I already learned a lot just by reading it and I will try to implement the improvements as soon as possible 👍

Hello @ochrstn
Can you check the unit test please :)

Hello @ochrstn
Can you check the unit test please :)

As soon as I use PlatformTest.bootstrap(Server, ...) in the integration tests, a lot of errors occur in the tests of other modules. What is the reason for this? 🙀 I'm a little lost right now.  @Romakita Maybe you have a hint for me?

As soon as I use PlatformTest.bootstrap(Server, ...) in the integration tests, a lot of errors occur in the tests of other modules. What is the reason for this? 🙀 I'm a little lost right now. @Romakita Maybe you have a hint for me?
Yes. This is the problem when you test something that load a library from a hook. Once the module is loaded by a unit test, it will be loaded for the following ones. It is therefore necessary to provide a means of activating Agenda only at the desired time and especially a destruction function to stop the jobs.

Ok. What we can do is to activate Agenda only by an optin from the configuration. The advantage for the developer is to define by configuration which instance is able to run Agenda task if you deploy your code on aws/heroku with multiple VM.
By default @tsed/agenda won't load providers ^^. I'll add comment on your PR.

Got it. The database connection initialized by Agenda is not closed correctly....

Pull Request Test Coverage Report for Build 885191237

61 of 61 (100.0%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.005%) to 96.07%

Totals

Change from base Build 884583166:
0.005%

Covered Lines:
7980

Relevant Lines:
7988

💛  - Coveralls

@ochrstn Thanks for the Contribution. The next point is to add the documentation on the Ts.ED website :)

:tada: This PR is included in version 6.53.0 :tada:
The release is available on:

v6.53.0
GitHub release

Your semantic-release bot :package::rocket:


Suggested refactor of WXT#1238

This PR is for the changes I mentioned here: https://github.com/wxt-dev/wxt/pull/1238#issuecomment-2515154584
I realized after posting that comment that you probably didn't understand what I was talking about, so I figured I'd do the refactor myself lol.
Once you merge this PR, the changes will show up in your WXT PR. Alternatively, I can do this refactor in a separate PR for WXT directly, instead of doing it into your fork if you'd like.
After this is merged, inside zip.ts, you can just add the following to filter them out from sources.
  if (wxt.config.zip.zipSources) {
+   const entrypoints = await findEntrypoints();
+   const skippedEntrypoints = entrypoints.filter(entry => entry.skipped);
+   const excludeSources = [...wxt.config.zip.excludeSources, skippedEntrypoints.map(entry => relative(wxt.config.zip.sourcesRoot, entry.inputPath))]
    await wxt.hooks.callHook('zip:sources:start', wxt);
    // ...
    await zipDir(wxt.config.zip.sourcesRoot, sourcesZipPath, {
      include: wxt.config.zip.includeSources,
-     exclude: wxt.config.zip.excludeSources,
+     exclude: excludeSources,

Actually, changing my mind. This is useful to be merged without your changes, so I'm going to make a PR directly into WXT for the change.


7 Function Framework binaries should have language-specific aliases

Most of the Functions Frameworks are invoked using either functions-framework or functions_framework terminal commands.
This is fine for environments where only one FF is installed, but environments with multiple FFs (i.e. FFs for multiple languages on the same machine, or multiple FF versions for a single language) make it harder to select specific FFs.
For example:
Node.js directory
$ ls
README.md         all-tests.xml     index.js          node_modules      package-lock.json package.json      test              test.html

Ruby FF (...wait, what?)
$ functions-framework --target=hello_gcs --signature-type=event --port 8089
/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/universal-darwin19/rbconfig.rb:229: warning: Insecure world writable dir /Users/anassri/Desktop in PATH, mode 040776
I, [2020-10-05T18:15:25.641369 #83511]  INFO -- : FunctionsFramework v0.5.1 server starting.
I, [2020-10-05T18:15:25.641410 #83511]  INFO -- : FunctionsFramework: Loading functions from "./app.rb"...
Traceback (most recent call last):
	5: from /usr/local/bin/functions-framework:23:in `<main>'
	4: from /usr/local/bin/functions-framework:23:in `load'
	3: from /Library/Ruby/Gems/2.6.0/gems/functions_framework-0.5.1/bin/functions-framework:19:in `<top (required)>'
	2: from /Library/Ruby/Gems/2.6.0/gems/functions_framework-0.5.1/lib/functions_framework/cli.rb:111:in `run'
	1: from /Library/Ruby/Gems/2.6.0/gems/functions_framework-0.5.1/lib/functions_framework/cli.rb:135:in `start_server'
/Library/Ruby/Gems/2.6.0/gems/functions_framework-0.5.1/lib/functions_framework/cli.rb:135:in `load': cannot load such file -- ./app.rb (LoadError)

One possible fix is to add aliases to the existing FF binaries - e.g. functions-framework-<LANGUAGE>, and maybe functions-framework-<LANGUAGE>-<VERSION>. (N.B: I suspect the former is more common than the latter.)

Note: This is an addition - so functions-framework would still work, but may point to an arbitrary FF binary if run on a machine with multiple FFs installed.

The latest version (0.7.0) of the Ruby framework provides both functions-framework-ruby and functions_framework, but deprecates the latter. (i.e. it works, but prints a warning suggesting that users use functions-framework-ruby instead.) Additionally, I just finished updating the Ruby GCF buildpack to use functions-framework-ruby instead of functions-framework. That's the behavior I'd recommend in general.

@ace-n I think this global functions-framework binaries only happened to very specific languages, i.e. python and ruby. Thus I don't think other languages need specific aliases.
Do you mind filing an issue to not use the generic functions-framework name in language-specific repos as you see the issue? I think that will make more progress.

Done (for Python and Node - since the Node FF can be installed globally via npm install -g)


[AWARE] pwd replace by $(pwd) and added new URL for the Census dataset

I replaced pwd by $(pwd) to actually compare the output of pwd and not the pwd string
I added a working URL for the Census1990 dataset

@Baunsgaard

Ups. I guess it worked for the wrong reason. Thanks for the catch.

I have merged it and it is pushed to master. Thanks for the update!


Introducing ferris-says 0.2.0

I've recently been toying with Rust and came across ferris-says.  I liked the concept and was inspired to work on a few enhancements.  Highlights are below:

Output more closely matches cowsay
Additional arguments can be provided to the command as an alternative to standard input
Graceful handling of multi-byte characters (should fix #2)
Addition of integration tests

Given the scope of changes I've tentatively deemed this version 0.2.0.  I'll defer to your input and guidance on versioning.  I have a few other ideas in mind, mostly providing parity with cowsay/cowthink but this is sufficient for now.  :-)
I look forward to your feedback and thoughts.  Thanks!

Hi! Sorry I don't have email notifications for github turned on and I forget to check them on here 😅 This looks great and helps bring the code a little more up to date. I like this and will merge as is. Thanks 😄

No problem at all.  Thanks! 👍


Windows 7 kill command not working

Hello Lorenzo: The # system "kill -9 #{pid}" command is not working on Windows 7 64 bit. This line is located in the server.rb file at line #23 in the \lib\shutup folder.
I found that substituting the following line after commenting out the above line works fine however:
system "taskkill /PID #{pid} /F"
I'm using the gem to shutdown the rails server.

Hello @davidrails3165, thank you for the suggestion. The gem was not built for windows use but would be great to extend it. Do you think you can open a pull request or shall I take care of it?


Allow to format output for lxc profile list

Handy to have if we have to deal with lots of profiles.

jenkins: test this please


WIP: not having a recommended question in the know base

use the fallback_not_found_relevant_contents configuration in the .toml (project configuration)
fixed: #117

@schumannc @lgabs check if this is what you were trying to address in PR #143

This PR does not address the same as #143. #143 already address the problem of answering a default text in case of zero documents retrieved from the vector store, without passing any prompt to the llm (e.i. GPT).
As I and @schumannc  suggested in issue #117 , this solution with a default answer is more appropriate for default behavior, since a second solution discussed via prompt engineering actually works with no fallback treatment in the code, and instructions in the prompt template would solve that (as I've shown in this issue comment with some links). Besides, the default answer solution it's a much controlled way to have as a default behavior, instead of delegating this responsability to the user when making a prompt.
I did not see any objection to the default fallback option in the issue discussion, hence my PR proposing it.