{ungoogled-,}chromium,chromedriver: 130.0.6723.91 -> 130.0.6723.116

Note: This PR was once again delayed due to the tarball not available for multiple days after release.
Gentoo started generating and publishing such tarball themselves which we could in theory switch to just like AlpineLinux did.
Given nixpkgs has different constraints and our electron source builds already implement source fetching, we (@networkException and I) instead started working on migrating to source fetching for chromium as well.
This effort should be ready in a few days (hopefully before branch-off so there is less to backport) and will allow us to update chromium even faster than in the past few months, and especially the last handful releases.

https://chromereleases.googleblog.com/2024/11/stable-channel-update-for-desktop.html
This update includes 2 security fixes.
CVEs:
CVE-2024-10826 CVE-2024-10827
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 354993 run on x86_64-linux 1

1 package failed to build:

gotenberg

37 packages built:

aws-azure-login
chromedriver
chromium
chromium.sandbox
fast-cli
flaresolverr
grafana-kiosk
mermaid-cli
mermaid-filter
percollate
pinnwand
pinnwand.dist
playwright-test
puppeteer-cli
python311Packages.great-tables
python311Packages.great-tables.dist
python311Packages.pytest-playwright
python311Packages.pytest-playwright.dist
python311Packages.shiny
python311Packages.shiny.dist
python312Packages.great-tables
python312Packages.great-tables.dist
python312Packages.pytest-playwright
python312Packages.pytest-playwright.dist
python312Packages.shiny
python312Packages.shiny.dist
selendroid
selenium-server-standalone
sharedown
single-file-cli
sitespeed-io
undetected-chromedriver
ungoogled-chromium
ungoogled-chromium.sandbox
vhs
vscode-extensions.yzane.markdown-pdf
wayback

gotenberg is already failing on master, so that isn't a new breakage.


Prep for Release v0.25.4

This PR does the following:

prep for release v0.25.4

Tugboat has finished building the preview for this pull request!
Link:

https://pr800-emszzvewjqezwdob09ot7dkeapdy5mbv.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/61b23f4c6e958a76e4f2d171

Can you remove the line - Fixes the Github Actions CI / Lint task by pinning the Typescript version. from the Changelog? I had to revert the fix because it broke again.

Tugboat has finished building the preview for this pull request!
Link:

https://pr800-emszzvewjqezwdob09ot7dkeapdy5mbv.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/61b23f4c6e958a76e4f2d171


⚠ Bofrost.es has degraded performance

In 4512d93, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1674 ms

Resolved: Bofrost.es performance has improved in 4f4e98b after 6 minutes.


Import unv mesh

Import of set 82 from .unv file added to enable mesh creation.

Thank you for your contribution :)


Viewing the cross tag errors

When browsing to http://lib.haxe.org/t/cross, I get an error message:
website.controller.HomeController.tag()
All other tags (including empty/fake ones) work normally.

I also encountered this issue. "cross" is the most-used tag so I think it's important to fix.

@ibilon looks like it's not quite fixed by your PR: Invalid field access : toSemver, could you look into this?

Note: this affect http://lib.haxe.org/all too
The only reason this could happen should be because a project wasn't removed but all of its version were.

It's now fixed.

Indeed!

flixel seems to be missing from that page? There's only flixel-addons etc, but flixel core has the tag too, see http://lib.haxe.org/p/flixel/.

@Gama11 it's far to the bottom but it's there too

Hm, you're right.. I thought the libs were sorted by downloads, but it looks like that's only true for the ones at the top for some reason?

Hmm it should have been and it's the case in the /all and most tags.
http://lib.haxe.org/t/game also has the issue, maybe the nulls are affecting the sorting algorithm.

Want me to open a separate issue for this? This got a bit off-topic.

@Gama11 please do


getSleepSamples() and getBloodPressureSamples() Not working

getSleepSamples() and getBloodPressureSamples() never return any data. I've triple checked that my app has permission to read data from HealthKit and that data is present however both functions never return any data. I think it's those functions particularly because calling getStepCount() with the same setup/options works fine
CODE:
//Pedometer permissions stuff
const HKPERMISSIONS = AppleHealthKit.Constants.Permissions;
//Options for all Apple Healthkit features
let OPTIONS = {
permissions: {
read: [HKPERMISSIONS.StepCount, HKPERMISSIONS.SleepAnalysis, HKPERMISSIONS.BloodPressureDiastolic, HKPERMISSIONS.BloodPressureSystolic],
write: [HKPERMISSIONS.StepCount]
}
};
let BLOODOPTIONS = {
unit: 'mmhg',	// optional; default 'mmhg'
startDate: (new Date(2018,5,20)).toISOString(), // required
endDate: (new Date()).toISOString(),	// optional; default now
ascending: false,	// optional; default false
limit:10, // optional; default no limit
}
let SLEEPOPTIONS = {
startDate: (new Date(2018,5,20)).toISOString(), // required
endDate: (new Date()).toISOString(), // optional; default now
//limit:10, // optional; default no limit
};
...
AppleHealthKit.initHealthKit(OPTIONS, (err, res) => {
if(this._handleHealthKitError(err, 'initHealthKit')) {
return;
}
// healthkit initialized...
          //Blood Pressure Example

          AppleHealthKit.getBloodPressureSamples(BLOODOPTIONS, (err: Object, results: Array<Object>) => {
            if (err) {
              return;
            }
            console.warn("BLOOD: " + results);
          });

      //Sleep Analysis

      AppleHealthKit.getSleepSamples(SLEEPOPTIONS, (err: Object, res:Array<Object>) => {
        if (err) {
          console.warn("ERROR: " + err)
        return;
        }
        console.warn("SLEEP: " + res)
      });

...
}

I looked more into it and it seems whatever StartDate you enter is pushed forward by 1 month in the obj-c (endDate was always whatever date i entered). Working around it by just entering a start date of 1 month earlier than it should be but looks like something to fix in the next release

@XaviSmith Do you mind sharing in more detail how you were able to make this issue work on your end?
I'm having the same issue with getWeightSamples. I'm getting an empty array when I try to run and Alert on results.
`let d = new Date();
const PERMS = AppleHealthKit.Constants.Permissions;
const options = {
permissions: {
read: [
PERMS.StepCount,
PERMS.Weight,
PERMS.SleepAnalysis,
],
},
date: d.toISOString(),
startDate: d.toISOString(),
endDate: d.toISOString(),
limit: 1
};
// Permissions
AppleHealthKit.initHealthKit(options, (err, results) => {
if (err) {
console.log('HealthkitInitError: ', err);
return;
}
});
// Weight
AppleHealthKit.getWeightSamples(options, (err, results) => {
if (err) {
console.log('WeightError: ', err);
return;
}
Alert.alert(JSON.stringify(results), options.startDate)
});
`
FYI: I have no issues getting data from getStepCount. Only having issues with any of the Samples methods like getSleepSamples, etc...
Attached I'm sharing a screenshot of the alert with the empty array for results and startDate (this is on my iPhone, not on the simulator). I'm also sharing a screenshot of my healthdata with weight value that matches the startDate.
BTW: I'm using JSON.strigify because otherwise I get the following error: "json value of type nsmutabledictionary cannot be converted to nsstring".
Thanks in advance.


Update CHANGELOG.md for v0.2.0 release

Adds missing Changelog entry for v0.2.0 release.

BTW this was completed at time of release, however the commit is not present in main. See - https://github.com/juju/terraform-provider-juju/commit/e06f3e532845f272be067e0f6b13c2f2ff890a10


Fix sec+nanosec reporting; use time_nanosecs

Signed-off-by: Eric Windisch<EMAIL_ADDRESS>

Coverage remained the same at 88.119% when pulling 5b77843443e9da8715062f2e119e9bb8415dc587 on ewindisch:fix_nanosecs into dd64589153c3ee25826ccd7bc7f75baef7cf6313 on iopipe:master.

Coverage remained the same at 88.119% when pulling ffeeabd628191d5c00d48279432756b0e5a93146 on ewindisch:fix_nanosecs into dd64589153c3ee25826ccd7bc7f75baef7cf6313 on iopipe:master.

Coverage remained the same at 88.119% when pulling 12120fd23d7059dcad105ab1fcc80dd6cc34536e on ewindisch:fix_nanosecs into dd64589153c3ee25826ccd7bc7f75baef7cf6313 on iopipe:master.

Coverage remained the same at 88.119% when pulling 6c4939b33ef77335102d4fec67e34bdd8996dd15 on ewindisch:fix_nanosecs into dd64589153c3ee25826ccd7bc7f75baef7cf6313 on iopipe:master.


Dockerize repo

Dockerize for easier reproducibility

That would be great. I just need time to do it. I was hoping that this file https://github.com/sdpython/ensae_teaching_cs/blob/master/.circleci/config.yml would be enough to help people install everything they need to make it work.

Nice ! I can try to Dockerize it if you want. Just let me know

i am trying to see whether i can help on this issue. to do that, first try to dockerize all the repo (although the final image would be mega fat), then try to simplify / remove things :)
i have friends using the package, and they complain to have problems with installs sometimes. So my though was that having an image downloadable from the hub directly if possible( i believe it's free from open source projects) ? will smooth the usage.
Let me know this is useful :+1:

I assume you are looking for an image on linux? I can host the image on my website if needed.

i am a linux user , yeah, but most of my friends are on windows. so maybe just focus on a dockerfile, then some text to explain how to generate an image from the file, and then launch a container from the image. if you can host the image, that would be awesome :)


Error occur when qr code install

When i want to install some games by qr code, an error occurs. Plz help me to resolve this problem. Contact me via email, I will give you a photo from my 3ds about this problem

https://github.com/Steveice10/FBI/issues/89


simplify Reader::into_stream

Using try_unfold means we don't need to track the errored bit ourselves.

@benesch thanks!


Add docs for OpenSergo fault-tolerance spec v1alpha1

Add zh-cn docs for OpenSergo fault-tolerance spec v1alpha1.
(English version TBD)

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


[bug] spans are not properly nested when using multiple auto instrumentors

Describe the bug
When using multiple auto instrumentors, spans from distinct auto-instrumentors show up in different traces
To Reproduce

use langchain and openai auto instrumentors in the same application
Expected behavior
the openai spans should be nested in tthe same trace as langchain spans

Additional context
See comments here https://github.com/Arize-ai/openinference/issues/1062#issuecomment-2456467995

Hello - I'm working with @codefromthecrypt on using langchain telemetry. I filed this discussion in langchain about allowing context propagation within callback handlers which I think is required for a proper approach to fix this.
https://github.com/langchain-ai/langchain/discussions/27954
In the meantime, it would be possible to autoinstrument entry points such as BaseLLM to activate context via run_id, but ideally the library provided a cleaner mechanism for it.

Hello - I'm working with @codefromthecrypt on using langchain telemetry. I filed this discussion in langchain about allowing context propagation within callback handlers which I think is required for a proper approach to fix this, feel free to chime in if appropriate.
langchain-ai/langchain#27954
In the meantime, it would be possible to autoinstrument entry points such as BaseLLM to activate context via run_id, but ideally the library provided a cleaner mechanism for it.

Hey @anuraaga yeah I started taking a look at this yesterday, was trivial to get nesting correct in openai so look for that soon re issue #1061. But langchain, as you mentioned is not. We currently don't instrument any of the entry points or methods, because there are so many and they are changing all the time. But instead hook into the callback handlers like you mentioned. This make context propagation non trivial. Thanks for starting the discussion with them, will follow along in the thread and see if we can get a fix in when they support it.

Note this is partially resolved in #1121 will keep this open for tracking any changes on the langchain front. For future context, using a proxy I was able to get properly get context and wrap original function calls with context.with however, since generate requests to llms are queued in langchain and not executed till later the context was lost by the time openai (for example) was actually called.


[SPARK-38879][PYTHON][TEST] Improve the test coverage for pyspark/rddsampler.py

What changes were proposed in this pull request?
This PR add test cases for rddsampler
Why are the changes needed?
To cover corner test cases and increase coverage
Does this PR introduce any user-facing change?
No - test only
How was this patch tested?
CI in this PR should test it out

Can one of the admins verify this patch?

Merged to master.


feat: inner Title split d/m;

Опишите проблему
Оптимизация:

Мелкий рефакторинг внутренних компонентов (Title): сплит д/м

Ожидаемое поведение
Без изменений

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

28 of 28 (100.0%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 82.321%

Totals

Change from base Build<PHONE_NUMBER>9:
0.01%

Covered Lines:
10646

Relevant Lines:
11806

💛  - Coveralls


MiniController animation functionally won't work after parent widget rebuilds.

As listener is attached to Minicontroller inside initState widget life cycle method but Mini player is very likely to be recreated because of parent widget rebuilds ex. Safe Area change ( Device Navigation keys hide, show)  , MediaQuery and others so.
Opacity test 0 <= opacity <= 1.0 fails too because after safe area height changes.

I couldn't recreate this issue. The controller is always working for me.

Opacity test 0 <= opacity <= 1.0

This can happen if you use a special Curve such as Curves.easeInOutBack.


Request: keep the menu icon hidden when re-launching the app

Currently, if Maccy has the menu bar hidden and it is re-launched, the menu bar icon is restored.
I personally think it would be more intuitive to leave the icon hidden (as set in the preferences) when re-launching the app and open the preferences window instead (or do nothing at all)

That's an old feature that was added back when there was no preference window. @artginzburg What do you think about dropping this feature?

Hey @p0deje , I'm sorry to bother you, but is there any "news" about this request? Thank you for considering it in the first place BTW!

I wanted to get feedback from @artginzburg before implementing it.

That's an old feature that was added back when there was no preference window. @artginzburg What do you think about dropping this feature?

Sorry for the long wait.
Well, I think the preferences window that is opening on re-launch may get quite annoying for the user. Instead, it would be cool to temporarily show the menu bar item with the clipboard history menu (I've seen this feature in some other app. This is handy).
So, if the user forgets that he already has the app running and re-launches it, he won't be annoyed by the preferences window popping up or by the necessity to manually hide the menu bar item again. At the same time, if he, for example, forgets the combination he set to open the clipboard history, he can just Spotlight > Maccy > ⌘ , and take a look at the right hotkey.

@artginzburg your idea actually sounds much better than mine

I'm going to just popup Maccy menu when the application is opened again. But keep the menu bar visibility intact. Sounds good?

@p0deje sounds perfect to me, thank you very much!


Routes not following DNS RFC won't get LE certificate

Describe the bug
Routes starting with an alphanumeric value are not valid as per RFC 1035 - DNS
https://www.ietf.org/rfc/rfc1035.txt

E0207 09:32:23.384826       1 route.go:726] failed to accept ACME authorization: failed to create exposing Service /2012example-org-acme-g7kj6: Service "2012example-org-acme-g7kj6" is invalid: metadata.name: Invalid value: "2012example-org-acme-g7kj6": a DNS-1035 label must consist of lower case alphanumeric characters or '-', start with an alphabetic character, and end with an alphanumeric character (e.g. 'my-name',  or 'abc-123', regex used for validation is 'a-z?')

To Reproduce
Steps to reproduce the behavior:

create a route that starts with an alphanumeric value
enable letsencrypt on the route
the route gets created in openshift
acme-controller fails because the name of the route starts with a alphanumeric value

Expected behavior
route gets letsencrypt certificate and passes validation

workaround:
recreate the same route and prefix the label of the route with x- this will satisfy the acme controller and everything works as intended

I think we can close this one as this has been a Openshift related issue


MSC3395: Synthetic appservice events

Rendered
Initial implementation for Synapse here
Initial implementation for the matrix-bot-sdk here

@Half-Shot what sort of review are you looking for on this? It's made it onto the SCT board, but without context.

oob conversation has lead to a realization of not being sure why this is on the board - removing.


Whitespace at the bottom of the edit-master page...

I know this is a bit of a petty request and I am sorry for that.  What is important is that I know it is.  (:
How do I add white space to the bottom fo the edit-master page.  It is very uncomfortable for me to go way down there with my mouse and eyes when typing in information on my laptop.  Also when I hit the add button on a record it pushes off the bottom of the page.  I want to be able to scroll it up a bit (or a lot) on the page.  Again.. really sorry to bother you all with this.  Formatting is not my specialty.

Sorry I do not get what is your problem.

I see the issue here being two seperate issues:
user comfort
You don't want to have the content on the bottom of the screen, so there should be whitespace added. This is heavily user specific, therefore this will not be implemented. If you have necessity for that, please try to achieve that in the browser directly.
Adding records pushes off of the page
If the viewport is 100% full to the bottom, adding records will push "off the screen" to the bottom. This is unintended an will most likely be fixed by the 2.0 release as it will be a 100% rewrite of the frontend.

Because everyone loves to type at the very bottom of their screen.  And adding a few rows of white space at the bottom that allows them to scroll up or a setting to expand the size of the footer is.....  and adding the white space really has no effect on those people that do like to type at the very bottom of their screen.
I was actually just asking if there was a suggested or easy place to inject code to create that white space.  I do not mind hacking the software locally.  When I have time I will play with it and post my results here.  Thank you for responding!!  (:


Docs: Authorization - Granting user access to API resources

We need documentation that explains how to grant users access to API resources provided by installed packages through RBAC authorization.
This will enable users, other than cluster admins themselves, to create custom resource objects for consuming the provided API from installed operators/cluster extensions.
Specifically, the documentation covers:
1. Create default ClusterRoles for API/CRD access (if not provided by the bundle itself):

Default "view" ClusterRole: Grants users permission to view all custom resource objects/instances of specified API resources across the cluster.
Default "edit" ClusterRole: Allows users to modify all custom resource objects/instances of specified API resources within the cluster.
Default "admin" ClusterRole: Provides users with full permissions over all custom resource objects/instances of specified API resources in the cluster.

2. Grant user access to API resources:

Option 1: Binding default ClusterRoles to users:

ClusterRoleBinding: Grants users' permissions across all namespaces.
RoleBinding: Grants users' permissions within a specific namespace.

Option 2: Extending default Kubernetes roles:

Aggregation Labels: By adding aggregation labels to custom resource definitions, users with existing RoleBinding or ClusterRoleBinding for default Kubernetes roles automatically gain access to equivalent "view/edit/admin" ClusterRoles for the provided APIs/CRDs.

work on this is in PR #1407


Dark mode not working for java page

Dark mode not working for java page: https://opentelemetry.io/docs/languages/java/automatic/configuration/#enable-resource-providers-that-are-disabled-by-default

@svrnm not sure if we want to fix this - since I also saw that we want to document env vars only?

@chalin we got ourselves a fun one

Oh yes, thanks for reporting that -- it is one I forgot. I should be a quick fix. If not, I'll let this be given that you'll be dropping the use of the shortcode that generates those page elements, is that right?

if it is a quick fix, maybe let's fix it even if we decide to drop that (except the dropping is a quick thing in itself)


Rename Rect.inset() to Rect.inflate()

The name .inset() is confusing because a positive number makes the rectangle bigger, whereas normally one "insets" a shape by shrinking it. Changing the method name to inflate() would clear up the confusion.
Also, while we're at it, please add a static EMPTY Rect with min=FLOAT_MAX and max=FLOAT_MIN. This makes it easier to compute bounding boxes through successive unioning.

@viridia I hope https://github.com/bevyengine/bevy/pull/13452 does the trick. I would appreciate any suggestions on documentation updates, since I'm not sure if the doc comments in these methods would need some rewording.


feat: switch to http-types and add http-service-h1

Removes http-service-lambda, as the lambda-runtime is still not
updated to<EMAIL_ADDRESS>Currently http-service-hyper is broken due to a missing compat implementation to transform a http_types::Body into a hyper::Body

Fixing http-service-hyper is blocked on https://github.com/hyperium/hyper/issues/1704


Add WP Edu

@thientran235 Regrettably, this request has some signs of fraud. While it does not mean that the request is incorrect, I have to close it without merging.
Please apply for free educational license with an official document which proves that you are a student. You can do it using "OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students.
Thank you for the understanding!


chore: reformat for 80 columns; fix spacing

shell style guide specifies 80 columns
since echo adds spaces between its arguments the line that was already
split resulted in an extra space:
Verify installed headers created only  expected directories.

This change is

Consider hooking something like shfmt to the build.  We should have the robots do the drudgery of placing whitespace: https://github.com/mvdan/sh


Implement option to disable sorting on ChoiceSet

Currently, ChoiceSets always sort choices, which is fine most of the time, but sometimes you want an arbitrary order. A good example of this is Oracle Curses - it would make more sense if they were sorted Minor -> Extreme.
This merge adds a "sort" boolean to the ChoiceSet RE, which defaults to true. When set to false, the prompt will follow the order declared in the ChoiceSet RE itself.
Left is "sort": true (or omitted), right is "sort": false

(First real code merge, hope I haven't broken anything 🙏)

Personal opinion is that explicit choice sets should not be sorted, and queried ones should be. But this is shark's baby, so up to him to decide.

Personal opinion is that explicit choice sets should not be sorted, and queried ones should be, rather than introducing a sort parameter. But this is shark's baby, so up to him to decide.

Yeah, I think that'd make for the best results.

#3678 was chosen instead, using the given arbitrary order whenever there is a choice list. While this should solve your issue, thank you for making this PR and bringing up the issue.


Updated missing strings

Only left strings that won't translate cause bug
PR Checklist

Please check all that apply to this PR using "x":

[x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[x] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

Fixes
Issue Number:
What is the current behavior?
What is the new behavior?
Fixed most of missing strings, but some of them still won't translate
Other information

Thanks @kodxana !


Add support for browser.SetTimeout

Hi,
Surf is a very nice library. For my use case it's missing a way to add a Timeout when opening a connection. The underlying http.client does support a Timeout (https://golang.org/pkg/net/http/#Client), so I went ahead and added to the browser. It works as expected in my code:
browser := surf.NewBrowser()
browser.SetTimeout(10 * time.Second)

If you consider merging this PR, let me know if something needs to be changed.
Regards,
C.

Hi,
I read this in the Contribution.md file:
"Pull Requests:
Code changes are made to the dev branch. Once a milestone has been reached the branch will be merged in with master, and a new version tag created. Do not make your changes against the master branch, or they will more than likely be ignored."
However, there is no dev branch. Do you prefer me to resend the PR applied to the v2 branch?
Thank you,
C.

I'll have to change the contribution file. I think we recently decided to scrap the idea of submitting PR against the dev branch.


Test against Django 1.11

Now that Django 1.11a1 has been released:
https://www.djangoproject.com/weblog/2017/jan/17/django-111-alpha-1/

Current coverage is 100% (diff: 100%)

Merging #610 into master will not change coverage

@@           master   #610   diff @@
====================================
  Files           4      4
  Lines          83     83
  Methods         0      0
  Messages        0      0
  Branches        8      8
====================================
  Hits           83     83
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update d1bd5a9...1961c7a

You're welcome :-)


Moderator: remove prompt from interface

This is related to #202 where we're moving the prompt management to the planner. Since the moderator gets called before the planner there will be no prompt to pass in. The moderator should just moderate based off the users input which we can pass in. The new interface should look like this:
export interface Moderator<TState extends TurnState> {
    /**
     * Reviews an incoming utterance and generated prompt before it's sent to the planner.
     * @remarks
     * The moderator can review the incoming utterance for things like prompt injection attacks
     * or the leakage of sensitive information. The moderator can also review the generated prompt
     * to ensure it's appropriate for the current conversation.
     *
     * To approve a prompt, simply return undefined. Returning a new plan bypasses the planner and
     * redirects to a new set of actions. Typically the moderator will return a new plan with a
     * single DO command that calls `AI.FlaggedInputActionName` to flag the input for review.
     *
     * The moderator can pass any entities that make sense to the redirected action.
     * @param context Context for the current turn of conversation.
     * @param state Application state for the current turn of conversation.
     * @param ai AI system that's calling the moderator.
     * @param input Input to moderate.
     * @returns An undefined value to approve the prompt or a new plan to redirect to if not approved.
     */
    reviewPrompt(
         context: TurnContext,
         state: TState,
         ai: AI<TState>,
         input: string
    ): Promise<Plan | undefined>;

    /**
     * Reviews a plan generated by the planner before its executed.
     * @remarks
     * The moderator can review the plan to ensure it's appropriate for the current conversation.
     *
     * To approve a plan simply return the plan that was passed in. A new plan can be returned to
     * redirect to a new set of actions. Typically the moderator will return a new plan with a
     * single DO command that calls `AI.FlaggedOutputActionName` to flag the output for review.
     *
     * The moderator can pass any entities that make sense to the redirected action.
     * @param context Context for the current turn of conversation.
     * @param state Application state for the current turn of conversation.
     * @param ai AI system that's calling the moderator.
     * @param plan Plan generated by the planner.
     * @returns The plan to execute. Either the current plan passed in for review or a new plan.
     */
    reviewPlan(
         context: TurnContext,
         state: TState,
         ai: AI<TState>,
         plan: Plan
    ): Promise<Plan>;
}

Passing in the AI system makes everything consistent with the new Planner interface as well (#206)

P0


How do i start?

I am confused on how to start using it. I have installed it. But how to start
Do i have to use the storybook pattern like you did in the examples.
Please guide me or show me resources to read on how to start.
Thanks

hello! sorry for the late reply. I also just realized that there is no "Usage" section in the README. I'll try to add that. You don't have to use Storybook, for example, to use it in your React app, you can use this:
import React from 'react';
import moment from 'moment';
import Datatable from 'react-bs-datatable';

const header = [
  { title: 'Username', prop: 'username' },
  { title: 'Name', prop: 'realname' },
  { title: 'Location', prop: 'location' },
  { title: 'Last Updated', prop: 'date' }
];

const body = Array.from(new Array(57), () => {
  const rd = (Math.random() * 10).toFixed(1);

  if (rd > 0.5) {
    return {
      username: 'i-am-billy',
      realname: `Billy ${rd}`,
      location: 'Mars',
      date: moment()
        .subtract(1, 'days')
        .format('Do MMMM YYYY')
    };
  }

  return {
    username: 'john-nhoj',
    realname: `John ${rd}`,
    location: 'Saturn',
    date: moment()
      .subtract(2, 'days')
      .format('Do MMMM YYYY')
  };
});

const Table = () => (
  <Datatable
    tableHeaders={header}
    tableBody={body}
  />
));

Wow thank you so much
This really helps me as a beginning 😂
Thanks.
Itedjere Godspower
On Thu, May 21, 2020, 06:09 Try Ajitiono<EMAIL_ADDRESS>wrote:

hello! sorry for the late reply. I also just realized that there is no
"Usage" section in the README. I'll try to add that. You don't have to use
Storybook, for example, to use it in your React app, you can use this:
import React from 'react';
import moment from 'moment';
import Datatable from 'react-bs-datatable';
const header = [
{ title: 'Username', prop: 'username' },
{ title: 'Name', prop: 'realname' },
{ title: 'Location', prop: 'location' },
{ title: 'Last Updated', prop: 'date' }
];
const body = Array.from(new Array(57), () => {
const rd = (Math.random() * 10).toFixed(1);
if (rd > 0.5) {
return {
username: 'i-am-billy',
realname: Billy ${rd},
location: 'Mars',
date: moment()
.subtract(1, 'days')
.format('Do MMMM YYYY')
};
}
return {
username: 'john-nhoj',
realname: John ${rd},
location: 'Saturn',
date: moment()
.subtract(2, 'days')
.format('Do MMMM YYYY')
};
});
const Table = () => (

));
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Imballinst/react-bs-datatable/issues/62#issuecomment-631883932,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF25MBNN5YNCGPLA2HFYBIDRSSZQVANCNFSM4NA3I63Q
.

all good @Itedjere! Please let me know if you have further questions 👍


Added a new page about security and updated front page.

Looking for anything else I may have missed as well as places to emphasize more.

About two spaces after a period: I learned from typing courses to put two spaces after a period when using a typewriter. Those days are long behind us now, and modern typography no longer requires or recommends two spaces after a period. You'll notice multiple spaces collapse in the markdown output, so it doesn't change the appearance of the html.

TLDR: put one space after a period, not two.


If you switch the camera on and off multiple times, it will crash when exiting or disconnecting.

Describe the bug
If you switch the camera on and off multiple times, it will crash when exiting or disconnecting.
SDK Version
SDK: The latest version
Server: 1.5.2
git clone from main
git clone https://github.com/livekit/client-sdk-swift.git

iOS/macOS Version
16.2
Steps to Reproduce
Continuously switch the camera on and off, and it will crash after disconnecting or unpublishing after about 15-20 times.
Screenshots

Thanks, will fix this !

I've updated the main branch, can you try it out.

Great, it's fixed


Dropdown button

Button with a dropdown ability

Is this updated in latest dist?

:+1:

@Eun Spectre.css now provides pure CSS dropdown menu. https://picturepan2.github.io/spectre/#dropdowns


Proposal to join Cloud Native Computing Foundation

Jaeger team has made a proposal to donate Jaeger to Cloud Native Computing Foundation whose member projects include Kubernetes, Prometheus, OpenTracing, etc.

presentation slides
formal proposal: https://github.com/cncf/toc/pull/42

Pinging people from our recent (and still open) survey: if you are currently using Jaeger and support its transition to CNCF, please comment on the CNCF TOC proposal issue.
The comment can be just one sentence, e.g. We are using Jaeger at {organization name} and we support Jaeger becoming a CNCF member project {because ...}.
@jkandasa @sunfaces @jbdalido @princeop @pavolloffay @mabn @jpkrohling @nlamirault @JodeZer @prestonprice57 @jrbury @objectiser @sloev @hwinkel @Madhu1512 @yuekui2 @valichek @dianvaltodorov @ZhouZiHe @LoungeFlyZ @jeluard @diegofernandes @d-ulyanov @jyothepro @yqf3139 @tomersimis @ruinanchen @szdavid92 @anuptalwalkar @hekike @sul4bh @Strandedpirate @julianste @awhiteside @nklmish @sweatybridge @kevinearls @felixbarny @hzariv @nlamirault @longXboy @drzero42 @xdralex @philipgian @bharat-p @pvlugter @Dieterbe @frankgreco

We are using Jaeger at stagemonitor and we support Jaeger becoming a CNCF
member project
Yuri Shkuro<EMAIL_ADDRESS>schrieb am Fr., 25. Aug. 2017 18:16:

Jaeger team has made a proposal to donate Jaeger to Cloud Native
Computing Foundation http://cncf.io whose member projects include
Kubernetes, Prometheus, OpenTracing, etc.

presentation slides
https://docs.google.com/presentation/d/1lDNdFxr58q-Fz4-yGXiei1iFxTDYq9LTQcJZ0x5BOec/edit?ts=5968dcd3#slide=id.g24b4bc2494_1_0
formal proposal: cncf/toc#42 https://github.com/cncf/toc/pull/42

Pinging people from our recent (and still open) survey
https://github.com/uber/jaeger/issues/207: if you are currently using
Jaeger and support its transition to CNCF, please comment on the CNCF TOC
proposal issue https://github.com/cncf/toc/pull/42.
The comment can be just one sentence, e.g. We are using Jaeger at
{organization name} and we support Jaeger becoming a CNCF member project
{because ...}.
@jkandasa https://github.com/jkandasa @sunfaces
https://github.com/sunfaces @jbdalido https://github.com/jbdalido
@princeop https://github.com/princeop @pavolloffay
https://github.com/pavolloffay @mabn https://github.com/mabn
@jpkrohling https://github.com/jpkrohling @nlamirault
https://github.com/nlamirault @JodeZer https://github.com/jodezer
@prestonprice57 https://github.com/prestonprice57 @jrbury
https://github.com/jrbury @objectiser https://github.com/objectiser
@sloev https://github.com/sloev @hwinkel https://github.com/hwinkel
@Madhu1512 https://github.com/madhu1512 @yuekui2
https://github.com/yuekui2 @valichek https://github.com/valichek
@dianvaltodorov https://github.com/dianvaltodorov @ZhouZiHe
https://github.com/zhouzihe @LoungeFlyZ https://github.com/loungeflyz
@jeluard https://github.com/jeluard @diegofernandes
https://github.com/diegofernandes @d-ulyanov
https://github.com/d-ulyanov @jyothepro https://github.com/jyothepro
@yqf3139 https://github.com/yqf3139 @tomersimis
https://github.com/tomersimis @ruinanchen
https://github.com/ruinanchen @szdavid92 https://github.com/szdavid92
@anuptalwalkar https://github.com/anuptalwalkar @hekike
https://github.com/hekike @sul4bh https://github.com/sul4bh
@Strandedpirate https://github.com/strandedpirate @julianste
https://github.com/julianste @awhiteside https://github.com/awhiteside
@nklmish https://github.com/nklmish @sweatybridge
https://github.com/sweatybridge @kevinearls
https://github.com/kevinearls @felixbarny
https://github.com/felixbarny @hzariv https://github.com/hzariv
@nlamirault https://github.com/nlamirault @longXboy
https://github.com/longxboy @drzero42 https://github.com/drzero42
@xdralex https://github.com/xdralex @philipgian
https://github.com/philipgian @bharat-p https://github.com/bharat-p
@pvlugter https://github.com/pvlugter @Dieterbe
https://github.com/dieterbe @frankgreco https://github.com/frankgreco
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uber/jaeger/issues/354, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACEDCNLCTHr3UNUHE-oL_Qx8hrzrNOQ5ks5sbvNQgaJpZM4PC37m
.

@felixbarny do you mind adding the comment to the actual TOC proposal issue? https://github.com/cncf/toc/pull/42

Lol, sorry. Yes, I'll do.
Yuri Shkuro<EMAIL_ADDRESS>schrieb am Fr., 25. Aug. 2017 18:41:

@felixbarny https://github.com/felixbarny do you mind adding the
comment to the actual TOC proposal issue? cncf/toc#42
https://github.com/cncf/toc/pull/42
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uber/jaeger/issues/354#issuecomment-324973858, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACEDCB7TnQCsbM-YCgrLYdXeIhAXgz2Lks5sbvk2gaJpZM4PC37m
.

@yurishkuro great work!! added my commends to the CNF proposal.


Unable to add plugin

I am trying to add the plugin using following commands but getting error.
phonegap plugin add phonegap-plugin-barcodescanner
Error: Cannot find plugin.xml for plugin 'phonegap-plugin-barcodescanner'. Please try adding it again.

I have just tested and worked fine.
Can you provide your phonegap version?


add-nullability

@JulianBissekkou Hi! maybe add branch and make prerelease version like in other packages? Thanks for PR and your job 😃

@Overman775 A prerelease was pushed a few weeks ago. Include version 1.0.10-nullsafety.0 for that :D


EPIC: OpenSearch: new content

What should we add to our content collections?
We already have good content from OpenSearch in NodeJS. Use the same subject to bring some python examples in tutorial format. So we can start to have collections for different languages. Reuse the same kaggle data set would be recommended.
Here are some ideas:

[ ] Write search queries with Python in OpenSearch
[ ] OpenSearch: Get data mapping with Python
[ ] Use Aggregations with Python in OpenSearch (metric, bucket, pipeline)
[ ] OpenSearch and logs (investigate how it is with aiven product)

Is this something you would work on yourself?
[x] I plan to open a pull request for this

I have added OpenSearch: Get data mapping with Python for the sample-dataset tutorial for now. I will use to make reference to OpenSearch: write search queries.


Error catching and Typescript: Object is of type 'unknown'

Since I upgraded my project to Typescript 4.4+, any catch error is considered as unknown and its type has to be verified.
This article well explain the situation: https://bobbyhadz.com/blog/typescript-object-is-of-type-unknown
I have the case with this library. Considering the following code sample:
try {
  const mangoCard = await mango.Cards.get(card.cardId);
  // Stuff...
} catch (error) {
  if (error.Type === 'ressource_not_found') {
    // Stuff...
  }

  throw error;
}

Gives me the following Typescript error:
error TS2571: Object is of type 'unknown'.

208     if (error.Type === 'ressource_not_found') {
            ~~~~~

This should be something solvable by using instanceof with the right error instance type. For example with a Joi validation error catch:
import Joi, {
  ValidationError,
} from 'joi';

try {
  await schema.tailor(context.method).validateAsync(context.data, {
    abortEarly: false,
  });
} catch (e) {
  if (e instanceof ValidationError) {
    throw new BadRequest('invalid data', { errors: e.details });
  }

  throw e;
}

However, I did not find any typing representing the MangoPay error containing the Type property on this library.
Did I miss something or is it something that need to be updated on that project? :thinking:

Hey @soullivaneuh,
I am going to add an ApiError type that should help.
type ApiError = {
            Message: string;
            Type: string;
            Id: string;
            Date: number;
            errors: [string, string];
        }

Also, you can define your custom error handler when instantiating the API:
const api = new mangopay({
    clientId: 'placeholder',
    clientApiKey: 'placeholder',
    errorHandler: function (errorMessage, errorBody: mangopay.models.ApiError) {
        console.log(errorMessage)
        console.log(errorBody)
    }
});

Fix in release 1.30.1

Currently have 1.31.1, but I don't find the ApiError export:
import {
  ApiError
} from 'mangopay2-nodejs-sdk';

How am I supposed to use it?

Currently have 1.31.1, but I don't find the ApiError export:
import {
  ApiError
} from 'mangopay2-nodejs-sdk';

How am I supposed to use it?
import * as Mangopay from 'mangopay2-nodejs-sdk';
Mangopay.models.ApiError


Adding bower.json

To register this as a bower package:

bower register theaterjs git@github.com:Zhouzi/TheaterJS.git
bower info theaterjs
@Stephn-R  said that he wanted to also help with this from here

those dependencies should be for dev purposes only

Thank you for your contribution @devanp92 I'll have a look at bower's documentation. Thank you for the heads-up @Stephn-R, actually it's my bad, I messed up with the dependency declaration in the package.json file. I'll fix it.

Thank you @devanp92 and @Stephn-R, TheaterJS is now available as a bower component bower install theaterjs


slow message loading when topic is compacted and filter is oldest

Issue submitter TODO list

[X] I've looked up my issue in FAQ
[X] I've searched for an already existing issues here
[X] I've tried running main-labeled docker image and the issue still persists there
[X] I'm running a supported version of the application which is listed here

Describe the bug (actual behavior)
We have several compacted (cleanup policy: compact) topics in our clusters, when we try to load messages in any of them using the "oldest" filter, it takes a significant amount of time to load (around 1 minute or more). Worthy to note that this doesn't occur when selecting the "newest" filter.
The topic size is around 17MB.
We monitored the CPU and memory of kafbat and increased it significantly in efforts to get a faster load time, but it didn't help. The app logs and console logs looks normal
We have significantly larger topics that are not compacted and they load the messages instantly, regardless of which filter.
Expected behavior
messages in compacted topics are loaded faster, like normal topics that have delete cleanup policy
Your installation details
running the latest main 1196f9f docker image deployed in EKS as a normal k8s deployment without helm.
Steps to reproduce
create a compacted topic and load it with some significant data
Screenshots

Logs
No response
Additional context
No response

Similar problem we noticed: for compact topics, oldest is working fine for us. But when we switched to use newest, it took forever to load.

@Haarolean this should be fixed due to https://github.com/kafbat/kafka-ui/pull/406

@Haarolean this should be fixed due to #406

@wanisfahmyDE @xuyuanz please verify, the image is available under ghcr.io/kafbat/kafka-ui:main

@wanisfahmyDE so does it work now?

yes switched to use kafka-ui:main.  Now it loading forever for "oldest" messages.

Similar problem we noticed: for compact topics, oldest is working fine for us. But when we switched to use newest, it took forever to load.

We are facing the same issue with helm chart version 1.4.5, when switched to main, it was the other way round.

Probly 'other way around' issue is related to #249, lol. I'll hide these comments as this is most likely unrelated. This issue is open which means we haven't fixed that, yet. Please upvote the issue and stay tuned!


[CentOS 7, 17.1.1-2] Several GFE issues

GFE finally loaded for me, but it has a lot of problems.
A) The map is stuck in the midwest. This is a problem for me being in the northeast.
B) Every time I click a button to try to do something, it gives an unhandled event loop exception. This goes for just about any button on it. This makes the mode unusable.
Any ideas? I want it centered on the BOX area and want to use it to export KML grids of snowfall accumulation maps this coming winter. (Which leads me to another question: can the CWA boundaries be changed for this purpose or can I export grids going beyond my CWA? I remember last summer when playing with a prior version I could draw outside the boundaries of my CWA.)

How did you get GFE to load?

I honestly have no clue. For about 10 days it wouldn't load and then it randomly started loading on Friday, but not actually doing anything.
Sent from my iPhone

On Jul 23, 2017, at 12:49 AM, airbusman5514<EMAIL_ADDRESS>wrote:
How did you get GFE to load?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I cannot seem to change views, I got it to load as well!


Webhook single item

Process first and only element in the NotificationItems array.
See documentation.

JSON and HTTP POST notifications always contain a single NotificationRequestItem object.
SOAP notifications may contain up to six NotificationRequestItem objects, in case the events triggering notifications happen within a very short period of time.

Acceptance Criteria

Update the WebhookController to process first and only element in the array

@gcatanese Quick discussion: Would it be an idea to raise a ticket and have the libraries/SDKs return a single item (abstract away the logic?)
See: https://github.com/Adyen/adyen-dotnet-api-library/blob/develop/Adyen/Model/Notification/NotificationRequest.cs#L35
We can already do the aggregation on library-level here, what is your opinion on this? ^


helper function to create a new environment

Sorry, but I don't want to pollute the global namespace by defining a function. A better option would be to create a static method on the Dotenv class that could be used instead.

That sounds like a better idea, ill create a PR later!


Epub support

As the title says
Almost done at the moment - Just sorting out some metadata

http://www.lab99.com/web-advice/how-to-create-an-ebook-part-4
https://ebooks.stackexchange.com/questions/1183/what-is-the-minimum-required-content-for-a-valid-epub
https://www.creating-ebooks.com/
https://web.archive.org/web/20201126065023/https://www.thebookdesigner.com/2015/07/4-ways-to-create-an-epub-ebook/
https://www.lifewire.com/create-epub-file-from-html-and-xml-3467282

In beta testing..

http://xahlee.info/js/html5_non-closing_tag.html

Still in beta, but I think it works well enough (though it needs more care)


VisualQnA UI docker image has backend address hardcoded(not being replaced by envs)

VisualQnA UI image (Gaudi) is not working with the set_env.sh set from https://github.com/opea-project/GenAIExamples/blob/main/VisualQnA/docker/gaudi/README.md

Thanks!

Thanks for reporting this issue. I will try to reproduce and give feedback later.

@ezelanza I can reproduce your issue.
I fix it with this PR https://github.com/opea-project/GenAIExamples/pull/701. After updating the setting in compose.yaml for the backend endpoint, it will override the default url setting in VisualQnA/docker/ui/svelte/.env. Please check it out.

Thanks, @wangkl2 for that! After replacing the  compose.yaml as you mentioned, I see the variable correctly changes it to my public_ip (I've covered it for security reasons), but the request from the UI to the backend service seems to be blocked by CORS policy.

It works when I test it from any terminal
~ % curl http://${host_ip}:8888/v1/visualqna -H "Content-Type: application/json" -d '{
     "messages": [
      {
        "role": "user",
        "content": [
          {
            "type": "text",
            "text": "What'\''s in this image?"
          },
          {
            "type": "image_url",
            "image_url": {
              "url": "https://www.ilankelman.org/stopsigns/australia.jpg"
            }
          }
        ]
      }
    ],
    "max_tokens": 300
    }'
{"id":"chatcmpl-MUqRQ3gWyBjAJqnn4nLuvP","object":"chat.completion","created":1725029828,"model":"visualqna","choices":[{"index":0,"message":{"role":"assistant","content":"The image shows a stop sign at the corner of a street, with a traditional Chinese archway or gateway in the background. There is also a black car driving past the stop sign. The scene suggests a blend of Western and Eastern architectural styles. "},"finish_reason":"stop"}],"usage":{"prompt_tokens":0,"total_tokens":0,"completion_tokens":0}}%  ```


Global require is not supported

Repeat the bug

Prepare the sample bookmarklet. It includes a require call.
Run the bookmarklet on https://chat.openai.com/chat.

The following error is logged:
VM639:formatted:7 Uncaught Error: Dynamic require of "https://etok.codes/require/raw/main/examples/showdown/mod.ts" is not supported
    at <anonymous>:1:201
    at <anonymous>:1:266
    at <anonymous>:1:1559

Fwiw, I looked into the JS code using the Chrome browser tools. I suspect the following piece of code could be part of the problem since it throws the error that was logged above.
    var p = (r=>typeof require < "u" ? require : typeof Proxy < "u" ? new Proxy(r,{
        get: (o,e)=>(typeof require < "u" ? require : o)[e]
    }) : r)(function(r) {
        if (typeof require < "u")
            return require.apply(this, arguments);
        throw new Error('Dynamic require of "' + r + '" is not supported')
    });

It could be that it is not permitted to call require not permitted in an esbuild bundle no matter the definition of the global require function.

It might be possible to write a custom Plugin for injecting custom behavior for dynamically imported scripts.

After some time has passed, it is clear that this is expected behavior. In future cases, the advice is: to find a way to bundle the code with the rest of the bookmarklet.

Note: Due to this limitation, it is impossible to serve the bundled JS for convenience by creating a simple testing bookmarklet with an import to another script. The testing bookmarklet may have to be a library that is capable of listening to changes and running the new bookmarklet code as it changes.


Use X-Forwarded-For for geo location

Hi
is it possible to tell goaccess to use x-forwarded-for field for geo location?  Otherwise, it shows the location of the reverse proxy.
I'm using goaccess 1.6 in combination with caddy.

Yes, that's what I meant. Client IP is in XFF and not in "remote_ip" field of caddy logs.

You should be able to use a custom format with XFF. Please take a look at this post.

Thanks, that works :)
For anyone else stumbling over this: In the log-format "remote_ip":"%h" needs to be replaced with "X-Forwarded-For":"%h"


Mirror doesn't work

Raycast version:
Description
Steps To Reproduce

The current behavior
gen.lib.rus.ec mirror doesn't work in my country. There should be a feature that allows us to specify the Libgen mirror.
The expected behavior

What extension are you talking about?

Looks like this is for https://www.raycast.com/yz3440/library-genesis.
It depends on libgen from NPM, which chooses from just 2 mirrors – https://www.npmjs.com/package/libgen#usage-choosing-a-mirror.
@havhav31 I'd suggest opening an issue on https://github.com/dunn/libgen.js about this. It looks like others have had similar issues. Also looks like it's out of maintenance, but a PR adding a new mirror to the list here might be accepted.
Then an upgrade to the libgen dependency on the extension should just work.


[Typescript] add base color palette type to components

first step toward #24778
These are the proposed components that should support color palette in the theme
(primary, secondary, error, info, success, warning).
Likely to appear in Dashboard and used as status

Chip
Badge
TimelineDot

Common

Button
ButtonGroup
Icon
IconButton

Form controls

LinearProgress
Form*
Input*
Radio
Checkbox

Once the list is finalized, then fix the types and missing classes for the components.

[ ] I have followed (at least) the PR section of the contributing guide.

It's not just TypeScript change though. You're adding new colors at runtime that weren't available before.
Paraphrased from internal chat:
I'd just remove the common color type. I don't think these colors make sense on all components. Let each component decide which colors it implements e.g. <AppBar color="error" /> is a bit too heavy in my opinion.

which one that I add is runtime?

All the *Classes.ts related changes. Hence the bundle size increase.

we can just say that if the component has color prop, these base palette can be used. it is a lot easier for consumer and less issue open than number 1.

I generally don't agree with this approach. Just because something is possible does not imply that it should be possible. Constraints are a good thing because they remove choices one has to make. If we agree that the full AppBar in an error state is bad, then we should remove that option. It's related to the concept of the "pit of success".

@eps1lon what's about we list the components that make sense to support base palette color eg. button, badge, textfield, ...
for the components the does not make sense, I see 2 ways.

don't change anything and if someone open an issue about color type, we response that they need to disable type on the line by themselves.
change the implementation back to support only some colors. ex, for AppBar

// current
/* Styles applied to the root element if colors comes from palette. */
    ...(styleProps.color &&
      styleProps.color !== 'default' &&
      styleProps.color !== 'inherit' &&
      styleProps.color !== 'transparent' && {
        backgroundColor: theme.palette[styleProps.color].main,
        color: theme.palette[styleProps.color].contrastText,
      }),

// new
/* Styles applied to the root element if colors comes from palette. */
    ...((styleProps.color === 'primary' || styleProps.color === 'secondary') && {
        backgroundColor: theme.palette[styleProps.color].main,
        color: theme.palette[styleProps.color].contrastText,
      }),

I think that it is ok if we change the implementation to something like:
...styleProps.color !== 'inherit' && {
  backgroundColor: theme.palette[styleProps.color].main,
  color: theme.palette[styleProps.color].contrastText,
}),

for the components that support colors, as that would mean that teams that want to extend the colors prop, can easily do it by adding new colors in the palette, and extending the color prop. I agree that we should restrict the proptypes & props interfaces for the components to support by default only the colors that we want to support by design.

I agree with the framing that @eps1lon is pushing. TypeScript is one aspect of the change, but it's not fundamentally about TypeScript. I think that it's about a different tradeoff: flexibility vs. soundness.

@siriwatknp the list looks solid. I would maybe add the Fab and Switch components as well as part of the Common/Form controls respectively.

@siriwatknp the list looks solid. I would maybe add the Fab, ToggleButton and Switch components as well as part of the Common/Form controls respectively.

Switch sounds good to me but I doubt ToggleButton and Fab. I haven't seen success or error UI for these two components before.

For the color demos, usually, we have only been shown 1 or 2 custom colors. Do we need to show all the colors of the palette? It seems to make the code preview of the demo more challenging to process for the developers. And with live edit #24640, developers will be able to quickly try the value they need. If your intent is visual regression, then I would propose we add custom visual snap (but not in the docs).

My intention is to show that the component support the color in the palette. What if the demo show one more color like success.
<Button color="secondary">

What if the demo show one more color like success.

We could have two, yeah, and for the button, maybe them all (it's a button, quite a unique component, but not sure it would be better). For the others, I would encourage againt.
In any case, I think that we should always have an inline preview of the demo. If the demo becomes too long preventing it to display (limit around 20 lines). Then it's not flying, and two colors is likely more than enough.

In any case, I think that we should always have an inline preview of the demo. If the demo becomes too long preventing it to display (limit around 20 lines)

I checked, all of the Color demo has inline preview so 2 colors sounds good.

The contrastText of the success color feels wrong (white works a lot better)

Agree, it will be fixed in #26817 😊


FossId: Explicitly install the ORT authenticator

The authenticator is needed to add credentials to the Git URL passed
to FossID. Recent changes in the scanner infrastructure have obviously
caused that the ORT authenticator has not always been installed when a
new FossID scan is created. The scan then fails due to missing
credentials.

FYI @mnonnenmacher, that looks like an interesting side-effect of switching to scan-by-repo. However, I guess it was always wrong to implicitly rely on the ORT authenticator to be installed.

@oheger-bosch the commit lacks a signed-off-by.

@oheger-bosch the commit lacks a signed-off-by.

Sorry about that. Fixed now.

FYI @mnonnenmacher, that looks like an interesting side-effect of switching to scan-by-repo. However, I guess it was always wrong to implicitly rely on the ORT authenticator to be installed.

Agreed. If the authenticator is invoked, the caller needs to ensure that it is installed. Maybe, in addition to the installAuthenticatorAndProxySelector() helper function, there could be another function to query credentials for a given URL that calls the authenticator and ensures that it is installed?

there could be another function to query credentials for a given URL that calls the authenticator and ensures that it is installed?

Yeah, maybe a thin ORT-specific wrapper around Authenticator.requestPasswordAuthentication could make sense...


Not able to post bulkedit

Hey,
I have added a UI layer which can directly access the config file. I am not able to post to the /api/config/bulkedit.
My post call is in this format.
{
"EditRequest" : [{
			   "Name":"email5",
			   "Type":"notification",
			   "Text":"notification email5 { email =<EMAIL_ADDRESS>		           "Delete": false

			}]

}

But I keep getting the
"json: cannot unmarshal object into Go value of type conf.BulkEditRequest" error. Kindly help me out.

Be warned, this bulk edit feature is "experimental" and not something we actually ended up using. That being said it may work fine, but be aware you in a bit of uncharted territory.
A conf.BulkEditRequest is a slice of EditRequest:
// BulkEditRequest is a collection of BulkEditRequest to be applied sequentially
type BulkEditRequest []EditRequest

// EditRequest is a proposed edit to the config file for sections. The Name is the name of section,
// Type can be "alert", "template", "notification", "lookup", or "macro". The Text should be the full
// text of the definition, including the declaration and brackets (i.e. "alert foo { .. }"). If Delete
// is true then the section will be deleted. In order to rename something, specify the old name in the
// Name field but have the Text definition contain the new name.
type EditRequest struct {
	Name   string
	Type   string
	Text   string
	Delete bool
}

So your body should just be an array of those objects at the root. Not a dictonary at the root (and there is no EditRequest key either).
So I think it should just be:
[{
			   "Name":"email5",
			   "Type":"notification",
			   "Text":"notification email5 { email =<EMAIL_ADDRESS>		           "Delete": false

			}]

Closing and assuming solved unless I hear otherwise.


Button disappears

Describe your chaotic idea
The moment you try to click on a button, it moves away or disappears. Users have to chase the button to click it, creating a playful but frustrating experience.
Add Screenshots or Sketches
it keeps moving over the screen

Confirm

[X] I understand that my idea will add to the chaos and I'm okay with it
[X] I agree to follow this project's Code of Conduct
[X] I want to work on this feature
[X] I'm GSSOC'24 Extd registered Contributor
[X] I'm Hacktoberfest registered Contributor

Seems interesting
Wait for #2 to be completed then you can continue with your implementation
till that time search for new issues!
Happy Coding 🚀

I would like to work on this project. please assign it to me.

Its already assigned to the creator, waiting for #2 to be completed then I will assign it
You can search for new issues @Rishita-2714  which I will be happy to assign!

@vansh-codes shall i get started on this?

Yes!

Is this issue completed?
Please share the PR link if its completed


Trojan:Win32/Wacatac.H!ml

A bit too much malware
Heya, cool project but I'm seeing a bit too much malware for my personal tastes.
Maybe y'all could do Ninja Lite, with just the build tools, without any of the Privilege Escalation, Defense Evasion, write calls to NTUSER.DAT, or other sketchy process behaviors exhibited by ninja.bootstrap.exe and ninja.exe?

I think I'd prefer that version.

--DJ

Hey guys. What's going on here?


chore: add test coverage for footer_win_opts

Add test coverage for https://github.com/mikesmithgh/kitty-scrollback.nvim/pull/182

:tada: This issue has been resolved in version 4.2.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add multi-GPU support for finetuning

Currently, when using multiple GPUs, the progress bars are not moving and it doesn't seem like anything is happening, despite the fact that the GPUs are in use. If they're indeed benchmarking the models then this progress should be displayed.

Does this mean that using multiple GPUs does indeed work, simply that the progress bar is not showing as intended? (in which case the title might be misleading)

Yes, multiple GPUs do work, but the command-line output is simply stuck at "Finetuning model: 0it [00:00, ?it/s]".

Closing for now, since the crucial point is using multi-GPUs for large decoder models, and vLLM supports that - this has been added in https://github.com/ScandEval/ScandEval/releases/tag/v12.0.0.


PHOENIX-7069 : Disable metadata caching re-design in Permission ITs based on HBASE-28391

Created https://issues.apache.org/jira/browse/PHOENIX-7296
Tests pass locally.


Add mockgen target for multicluster packages

Add a new make target mockgen to allow developer to
generate mock files for multicluster packages automatically
Re-generate MC client codes with existing license file
Update check-codegen.sh to check MC client codes as well

/skip-all

@jianjuns  seems the code tidy check is failed due a change on the workflow, I am checking the root cause, I will update the code after I verify it locally.


3rd PR for StorSimple 8000 series device ARM APIs.

…pportpackage, failover, networksetting, timesetting, backup, restore, clone.
This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

@veronicagg Hi, (a) All these APIs are reviewed by ARM board.
(b) They are also deployed to ARM (as private preview). Currently post private preview release is happening and GA will be happening soon.
(c) For "XmsExamplesProvidedValidation, TrackedResourcePatchOperationValidation", both the examples and patch call are a work in progress along with the work for the SDK review for these.
Actually we had ~85 APIs which are all deployed to ARM. They are all reviewed by ARM board too. Since the count of APIs was huge, we earlier just considered a limited set of APIs and got them reviewed here and in the first and second PR. In the process, we fixed issues found for them for all the other APIs too. After this PR we are planning to have 2 more PRs. For all the APIs(including the ones in the coming PRs), we will be adding the examples in the last PR, as by that time the SDK related test recordings will be done and we will be using them to generate the examples. Along, with the examples, we would also be adding the patch call for "managers" too in the last PR(as again that operation is not used now and hence we would need to test it completely before adding it here and is already a planned worked item before we sign-off on the swagger spec for all our APIs).
Given this, please do let me know if there is any other issues with this PR or if not, we can get this merged.

@manaas-microsoft thanks for the context. I've added a few more quetsions and would like to get input from @ravbhatnagar on some of the above.

No modification for Python

No modification for NodeJS

No modification for Ruby


Lifted structure loss

This PR refers to Issue #102 and it is implementing the LSL described in the article https://arxiv.org/pdf/1511.06452.pdf

Thanks for the PR! Looking forward to adding this to the tfsim losses. I left some comments on the loss function.

@owenvallis implemented the changes requested, although i'm a little lost about the following snippet:
# Get negative distances
    negative_dists, _ = negative_distances(
        negative_mining_strategy, pairwise_distances, negative_mask
    )

since both of results are not being used anywhere else

@owenvallis implemented the changes requested, although i'm a little lost about the following snippet:
# Get negative distances
    negative_dists, _ = negative_distances(
        negative_mining_strategy, pairwise_distances, negative_mask
    )

since both of results are not being used anywhere else

Good point! Looks like we're missing the columns-wise concat in the logsumexp. You had it in the previous commit, so pulling that back in I think it should be:
    # Reorder pairwise distances and negative mask based on positive indices
    reordered_pairwise_distances = tf.gather(negative_distances, positive_indices, axis=1)
    reordered_negative_mask = tf.gather(negative_mask, positive_indices, axis=1)

    # Concatenate pairwise distances and negative masks along axis=1
    concatenated_distances = tf.concat([pairwise_distances, reordered_pairwise_distances], axis=1)
    concatenated_negative_mask = tf.concat([negative_mask, reordered_negative_mask], axis=1)

    # Compute (margin - neg_dist) logsum_exp values for each row (equation 4 in the paper)
    neg_logsumexp = tfsim_losses.utils.logsumexp(margin - concatenated_distances, concatenated_negative_mask)

@owenvallis implemented changes discussed, and also fixed for flake8, I'll try writing unit tests for the loss too. ALready subscribed for CLA/google action too

Thanks! Looks like the CLA is working for the current commits, but you will need to update the author on the older commit.
Let me know once the tests are ready and I'll run the checks and merge the PR.

Also I keep getting this error when writing tests:
FAILED tests/losses/test_lifted_structure_loss.py::TestLiftedStructLoss::test_lifted_struct_loss_test_mode_eager - TypeError: lifted_struct_loss() got multiple values for argument 'distance'
FAILED tests/losses/test_lifted_structure_loss.py::TestLiftedStructLoss::test_lifted_struct_loss_test_mode_graph - TypeError: in user code:
for a test like this (just example one, not the correct):
import tensorflow as tf
from absl.testing import parameterized
from tensorflow.python.framework import combinations
from tensorflow.keras.losses import Reduction
from tensorflow_similarity import losses
from . import utils

@combinations.generate(combinations.combine(mode=["graph", "eager"]))
class TestLiftedStructLoss(tf.test.TestCase, parameterized.TestCase):
    def test_config(self):
        lifted_obj = losses.LiftedStructLoss(
            reduction=Reduction.SUM,
            name="lifted_loss",
            distance="cosine",
        )
        self.assertEqual(lifted_obj.distance.name, "cosine")
        self.assertEqual(lifted_obj.name, "lifted_loss")
        self.assertEqual(lifted_obj.reduction, Reduction.SUM)

    def test_lifted_struct_loss(self):
        """Tests the LiftedStructLoss with different parameters."""

        labels = tf.constant([1, 2, 2, 1], dtype=tf.int32)
        embeddings = tf.random.normal([4, 10])
        expected_loss = 1

        lifted_obj = losses.LiftedStructLoss(reduction=Reduction.SUM)
        loss = lifted_obj(labels, embeddings)

        self.assertAlmostEqual(loss, expected_loss)

Interesting, I'll have to look into that more. Maybe the kwargs also contain the distance and that is getting passed twice in the init call or something? I'll try and take a look at it tomorrow if I get a chance.

havent thought about that, will try to do some stuff too

Had some progress today, we have some problems in the implementation and the distance thing was missing arguments that were coming from call. Working on it, expect to push some things tomorrow

@owenvallis just implemented some sample tests, now they're running fine, errors are fixed although the loss values are really weird, so im not sure they are correct. Also did not finish to implement test cases.

new pr #342


Partial refactor of _SequenceCollectionBase

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
761 unchanged lines in 12 files lost coverage.
Overall coverage decreased (-0.001%) to 91.609%

Files with Coverage Reduction
New Missed Lines
%

.nox/test-3-10/lib/python3.10/site-packages/cogent3/util/misc.py
15
94.19%

.nox/test-3-10/Lib/site-packages/cogent3/util/misc.py
15
94.19%

.nox/test-3-11/lib/python3.11/site-packages/cogent3/util/misc.py
15
94.19%

.nox/test-3-11/Lib/site-packages/cogent3/util/misc.py
15
94.19%

.nox/test-3-9/lib/python3.9/site-packages/cogent3/util/misc.py
17
93.84%

.nox/test-3-9/Lib/site-packages/cogent3/util/misc.py
17
93.84%

.nox/test-3-10/Lib/site-packages/cogent3/core/alignment.py
110
94.89%

.nox/test-3-10/lib/python3.10/site-packages/cogent3/core/alignment.py
111
94.85%

.nox/test-3-11/lib/python3.11/site-packages/cogent3/core/alignment.py
111
94.85%

.nox/test-3-11/Lib/site-packages/cogent3/core/alignment.py
111
94.85%

Totals

Change from base Build<PHONE_NUMBER>:
-0.001%

Covered Lines:
176828

Relevant Lines:
193024

💛  - Coveralls


[SPARK-10093][SQL] Avoid transformation on executors.

This is kind of a weird case, but given a sufficiently complex query plan (in this case a TungstenProject with an Exchange underneath), we could have NPEs on the executors due to the time when we were calling transformAllExpressions
In general we should ensure that all transformations occur on the driver and not on the executors.  Some reasons for avoid executor side transformations include:

(this case) Some operator constructors require state such as access to the Spark/SQL conf so doing a makeCopy on the executor can fail.
(unrelated reason for avoid executor transformations) ExprIds are calculated using an atomic integer, so you can violate their uniqueness constraint by constructing them anywhere other than the driver.

Merged build triggered.

Merged build started.

Test build #41160 has started for   PR 8285 at commit 8d3b048.

Test build #41160 has finished for   PR 8285 at commit 8d3b048.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41160/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #41161 has started for   PR 8285 at commit e8b8240.

Test build #41161 has finished for   PR 8285 at commit e8b8240.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41161/
Test FAILed.

Merged build finished. Test FAILed.

It seems this one has been subsumed.


remove init files

Summary

This PR cleans up all __init__.py files, which are no longer needed now that we have a setup.py
Testing

Existing units tests and pyright
Notes

Comments

[N/A] Add an x between the brackets if you commented your code well!

I still want to see if the sphinx docs need these __init__.py files. Still trying to fix some issues that occurred with the docs over the past few PRs, so lets hold off on merging this for at least a little bit

Roger, holding off.

__init__ files, unfortunately, need to be present for the sphinx docs to work. Thoughts on closing this PR? @tmf97

Rip, yeah closing


Fix memory leak in profiler

Automatically delete DevStat object at shutdown

ok, that libdc1394 problem with build server now...

Please rebase and get rid of other people's commit

I have very bad experience with rebasing. The squash should just have my
changes.
On Mon, Aug 28, 2017 at 5:27 PM Eric Junyuan Xie<EMAIL_ADDRESS>wrote:

Please rebase and get rid of other people's commit
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-mxnet/pull/7617#issuecomment-325519310,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKts_e208dC0BOoZWp8nQT8Ca_eILoDgks5sc1rqgaJpZM4PC6My
.

If we make it so that Profiler doesn't get destructed while its member functions are being called, this fix won't be necessary right?

It still has a memory leak without this, or are you referring to the add ref count?  Holding a ref count is pretty harmless even so.

A shared_ptr copy has non-trivial overhead, specially since this is in critical path

I think compared to the horrendous mutex lock below it, it is trivial.
On Wed, Aug 30, 2017 at 11:37 AM Eric Junyuan Xie<EMAIL_ADDRESS>wrote:

A shared_ptr copy has non-trivial overhead, specially since this is in
critical path
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-mxnet/pull/7617#issuecomment-326080879,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKts_YfiEc-3SE2k_4D7FCE2mH-KavMjks5sdavugaJpZM4PC6My
.

This isn't compiling

I'm ooo sick ATM.  There is a problem with the build machine last I
checked. They are working on it.
On Wed, Aug 30, 2017 at 11:18 PM Eric Junyuan Xie<EMAIL_ADDRESS>wrote:

This isn't compiling
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-mxnet/pull/7617#issuecomment-326200854,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKts_X_m4TIxOkzslYvCIqPL8BJ3YS7Wks5sdlBIgaJpZM4PC6My
.

Yes that device file needs to be deleted on the build server. It is causing
OpenCV unit to fail. Not related to my code.  Sandeep was looking into it.
  There is a problem with the build machine last I
checked. They are working on it.
On Wed, Aug 30, 2017 at 11:18 PM Eric Junyuan Xie <
<EMAIL_ADDRESS>wrote:

This isn't compiling
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-mxnet/pull/7617#issuecomment-326200854,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKts_X_m4TIxOkzslYvCIqPL8BJ3YS7Wks5sdlBIgaJpZM4PC6My
.

I just went onto every build server and attempted to fix the problem myself...


bison: 3.6.4 -> 3.7.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/bison/versions.
meta.description for bison is: "Yacc-compatible parser generator"
meta.homepage for bison is: "https://www.gnu.org/software/bison/"
meta.changelog for bison is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 3.7.1 with grep in /nix/store/fprhpkn9m6dpjqbckx42nk49bcjiv5fs-bison-3.7.1

directory tree listing: https://gist.github.com/d35924b007f07a526902bad746d60651

du listing: https://gist.github.com/030f4fb1441dc9cfe040b1364dbcde37

Rebuild report (if merged into master) (click to expand)

122560 total rebuild path(s)

35629 package rebuild(s)

35487 x86_64-linux rebuild(s)
31354 i686-linux rebuild(s)
24980 x86_64-darwin rebuild(s)
30739 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
AMB-plugins
AusweisApp2
CoinMP
DisnixWebService
EBTKS
EmptyEpsilon
FIL-plugins
Fabric
LASzip
LASzip2
Literate
MMA
NSPlist
OVMF
OVMF-CSM
OVMF-secureBoot
PlistCpp
QuadProgpp
R
SDL
SDL2
SDL2_gfx
SDL2_image
SDL2_mixer
SDL2_net
SDL2_ttf
SDL_Pango
SDL_gfx
SDL_gpu
SDL_image
SDL_mixer
SDL_net
SDL_sixel
SDL_sound
SDL_stretch
SDL_ttf
SPAdes
SkypeExport
Sylk
Xaw3d
_0x0
_1password
_2048-in-terminal
_20kly
_2bwm
_389-ds-base
_3llo
_3mux
_3proxy
_6tunnel

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fprhpkn9m6dpjqbckx42nk49bcjiv5fs-bison-3.7.1 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A bison https://github.com/r-ryantm/nixpkgs/archive/ff2364032d182abc3d01179c148b0932e78b386d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fprhpkn9m6dpjqbckx42nk49bcjiv5fs-bison-3.7.1
ls -la /nix/store/fprhpkn9m6dpjqbckx42nk49bcjiv5fs-bison-3.7.1/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings

@ofborg eval


Add an estimate helper that estimates RCU and WCU

Might look like...
operation = add('users', item={
    'id': 50,
    'team_id': 288,
    'username': 'sunshie',
    'state': 'active'
})

est = estimate(operation)

print(est)

{ 'rcu': 0, 'wcu': 45, 'units': 'kbs' }

Will need...

[ ] Create a function called estimate that takes in a dynamof.core.model.Operationas its argument
[ ] Inside estimate implement the logic to calculate the RCU/WCU of the operation given you can look in at the operations arguments and type.
[ ] Return the estimate 🍻

On the topic of calculating the RCU/WCU here are some helpful resources

medium post - calculating-a-dynamodb-items-size-and-consumed-capacity
stack overflow question on the topic
Zac Charles online tool
Zac Charles code for his tool


Primitive collections

Does project Valhalla offer primitive collections for free, or we'd better have them now?

We plan to provide the primitive collection with a separate module after the operation of the basic collection library is stable.
Kala 0.x and 1. x will be compatible with Java 8. In the future, Kala 2.0 will be fully refactored with Java 23 (or 29) to adapt to the new features (primitive classes, records, sealed interfaces, etc ).

Cool!!


[stable/grafana] Fix sed replacing special sources

Use sed negation pattern to avoid replacing special sources like -- Grafana --
Signed-off-by: Andrey Voronkov<EMAIL_ADDRESS>
What this PR does / why we need it:
Currently, sed replacing special sources like -- Grafana --
Which issue this PR fixes

fixes https://github.com/helm/charts/pull/22213#issuecomment-650063935

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign
/ok-to-test

/test pull-charts-e2e

/retest

/retest

@Antiarchitect
Could you rebase the chart version to 5.3.2 please? It seems helm member resolved some test-infra issues.

@zanhsieh Done

/lgtm


Get initial implementation that builds (probably will require no imports)

Got it working with imports, leveraging https://esbuild.do!


Add slither for l1 contracts

What ❔

Why ❔

Checklist

[ ] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[ ] Tests for the changes have been added / updated.
[ ] Documentation comments have been added / updated.

@StanislavBreadless as we discussed with @vladbochok could you please review the following findings
MailboxFacet._deriveL2GasPrice(uint256,uint256) (contracts/state-transition/chain-deps/facets/Mailbox.sol#163-182) performs a multiplication on the result of a division:
	- l1GasPriceConverted = (_l1GasPrice * s.baseTokenGasPriceMultiplierNominator) / s.baseTokenGasPriceMultiplierDenominator (contracts/state-transition/chain-deps/facets/Mailbox.sol#[16](https://github.com/matter-labs/era-contracts/actions/runs/8425289708/job/23071021468?pr=280#step:9:17)6-167)
	- batchOverheadBaseToken = uint256(feeParams.batchOverheadL1Gas) * l1GasPriceConverted (contracts/state-transition/chain-deps/facets/Mailbox.sol#173)
MailboxFacet._deriveL2GasPrice(uint256,uint256) (contracts/state-transition/chain-deps/facets/Mailbox.sol#163-182) performs a multiplication on the result of a division:
	- l1GasPriceConverted = (_l1GasPrice * s.baseTokenGasPriceMultiplierNominator) / s.baseTokenGasPriceMultiplierDenominator (contracts/state-transition/chain-deps/facets/Mailbox.sol#166-167)
	- pubdataPriceBaseToken = L1_GAS_PER_PUBDATA_BYTE * l1GasPriceConverted (contracts/state-transition/chain-deps/facets/Mailbox.sol#[17](https://github.com/matter-labs/era-contracts/actions/runs/8425289708/job/23071021468?pr=280#step:9:18)0)
Reference: https://github.com/crytic/slither/wiki/Detector-Documentation#divide-before-multiply

and let us know how we should proceed. Should it be fixed or ignored?  If fixed, then could you add a fix into this PR?
Thank you.

@dnkolegov we can ignore it. If we used what slither proposed, the calculations would indeed be slightly more precise, but IMHO the way it is now is a bit easier to read & reuse (i.e. whenever we need l1 gas price we can use l1GasPriceConverted)


Space-travellers-app: missions page

The following were carried out

Develop reducer and action creators for mission page
Fetch and display mission info from API
Display mission info on missions page and style it

Hi @Adams-Ghub  ,
You have done great work implementing all the tasks. There is nothing else to say than it is now time to merge.
Status: Approved

Highlights

Linter ✅
PR documentation ✅


Add targets and patches for NSS

These changes would allow NSS to move off the branch and use master.
Changes in hints don't have to be merged I can remove them.
I noticed that the unit-tests target doesn't work. I adapted the Makefile in kremlib to make it work. All other changes are NSS only and should be pretty minimal.
@beurdouche Can you have a look at this and let me know what you think?

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


Combat - Fleeing

Added enemy and player fleeing to the combat system. Enemies should have a chance to run when the player is sufficiently strong (twice enemy's strength?), and it should be an option for players. Enemies should be able to block the way (agility dependent?) for player fleeing, and if the flee is successful the player should move to the previous area they were in. Not sure if enemy fleeing should move the enemy to a different area or just remove it from the game.

Implemented as of commit 0cd74ac07e68a1ff821e69f163d580f02f87e400.
Made enemies disappear when they flee, and players move to a random area, not the previous area, if they successfully flee.


Specifically dependent continunants without bearer

Description of the issue
As a specifically dependent continuant (SDC) depends on an independent continuant, every SDC should have an has bearer axiom. But currently a lot of them do not have.
Ideas of solution

For imported classes check whether axioms were accidentally not imported:

class
bearer

specifically dependent continuant
independent continuant?

quality
independent continunant?

relational quality
independent continunant?

realizable entity
independent continunant?

disposition
independent continunant?

function
independent continunant?

role
independent continunant?

Add has bearer axioms to OEO defined SDC classes:

class
bearer

policy instrument
government (not yet defined). Alternatively: organisation?

rotor diameter
wind energy converting unit or wind rotor

hub height
wind energy converting unit

planned availability
power plant

unplanned availability
power plant

resolution
data set

spatial resolution
georeferenced data set (not yet defined)

temporal resolution
time series

state of matter
portion of matter

demand
person or organisation or object

energy storage
artifical object

greenhouse effect dispostion
'portion of matter' 'has state of matter' value gaseous

volatility
portion of matter

agent
person or organisation

good role
continuant

market exchange role
organisation?

organisation role
aggreate of people (not yet defined[^1])

waste role
object aggregate

Workflow checklist

[ ] I discussed the issue with someone else than me before working on a solution
[ ] I already read the latest version of the workflow for this repository
[ ] The goal of this ontology is clear to me

I am aware that

[ ] every entry in the ontology should have a definition
[ ] classes should arise from concepts rather than from words

[^1]: Maybe an intermediate class in between object aggregate and population.

OEO Dev meeting 44:
@fabianneuhaus validates the BFO classes, with Martins help
@l-emele implements the one that are obvious. There unclear ones will be discussed later.

We have at least two BFO classes, that we modified for the energy usecase, which are quality and role.
For both, we should add 'has bearer' some (energy or 'portion of matter'), I guess.
And therefore, this axiom should also be added to the parent classes realizable entity and specifically dependent continuant.
And the same axiom should be added to demand and energy demand (direct subclasses of realizable entity).
@l-emele @fabianneuhaus  @MGlauer do you agree?

I agree.

'Specifically Dependent Continuant' SubClassOf: has_bearer some 'Independent Continuant'>

'Specifically Dependent Continuant' SubClassOf: has_bearer some ('Independent Continuant'> or energy).

class
bearer

specifically dependent continuant
'independent continuant' or energy

quality
'independent continunant' or energy

relational quality
'independent continunant'

realizable entity
'independent continunant' or energy

disposition
'independent continunant'

function
'independent continunant'

role
'independent continunant' or energy

demand
'independent continunant' or energy

Quality has_bearer some (energy or 'portion of matter') seems to be too narrow to me. E.g., consider the reservoir of a pumped-storage hydroelectricity installation. That reservoir (it could be a lake or a large cave) is an immaterial entity, more specifically a site. Nevertheless it has qualities, e.g., a volume -- which contradicts the proposed axiom.

True. I think, we wanted to narrow the axiom for energy only: energy subClassOf 'has bearer' some 'material entity'.[^1]  Hence, a site cannot not be bear of an energy in that case. Only the portion of rock that forms the site, to stay with the example.
reservoir is actually classified as artificial object in OEO. Do we have to reconsider that classification?
[^1]: Energy is a quality of material entities which manifests as a capacity to perform work (such as causing motion or the interaction of molecules)

demand | person or organisation or object

This comes from the def of demand obviously. person is subclass of object, i.e. demand SubClassOF 'has bearer' some object or organisation?

good role | continuant

We cannot extend the bearer of the subclass compared to the parent class. Instead 'independent continunant' or energy, like its parent property role, I'd suggest.

Quality has_bearer some (energy or 'portion of matter') seems to be too narrow to me. E.g., consider the reservoir of a pumped-storage hydroelectricity installation. That reservoir (it could be a lake or a large cave) is an immaterial entity, more specifically a site. Nevertheless it has qualities, e.g., a volume -- which contradicts the proposed axiom.

True. I think, we wanted to narrow the axiom for energy only: energy subClassOf 'has bearer' some 'material entity'.1 (I mixed up material entity and portion of matter in my comment, sorry). Hence, a site cannot not be bear of an energy in that case. Only the portion of rock that forms the site, to stay with the example. reservoir is actually classified as artificial object in OEO. Do we have to reconsider that classification?

I'll open a separate issue for that.

Any feedback to my latest suggestions? It would be great to solve this.

I agree to your suggestions.

If there are no objections, I'll implement.

Go for it! :rocket:

My attempt is to put the axioms into oeo-import-edits.owl, yet an answer to #1484 is needed first.

from oeo-dev-54:
since 'has bearer' is inherited by its parent, we'll try sdc <= ((has_bearer some ic) OR ((has_bearer some energy) AND (role OR ...))

from oeo-dev-54: since 'has bearer' is inherited by its parent, we'll try sdc <= ((has_bearer some ic) OR ((has_bearer some energy) AND (role OR ...))

We could implement the suggested solution, but that entailed to put EVERY differentiation into the axiom for the superclass specifically dependent continuant, and to also maintain that axiom.
The more feasible solution seems to be to only explicitly state the bearer, where there´s no differentiated substructure needed and a false inheritance not a risk.

I will also delete the already existing wrong inherited axioms.

The bearer of policy instrument needs a discussion, see #1507

The good role is currently defined as role of a continuant, which is obviously an artifact and exceeds the 'energy or independent continuant' restriction. I'll fix that, too.


Issue with ai_protocol in int mbedtls_net_connect()?

Ran into the issue with ai_protocol value in mbedtls_net_connect() function. Used as is on IPv4 enabled ARM platform the call to mbedtls_net_connect() returns error -0x066. That is caused by the line
hints.ai_protocol = proto == MBEDTLS_NET_PROTO_UDP ? IPPROTO_UDP : IPPROTO_TCP;

which when function is called with MBEDTLS_NET_PROTO_TCP for “proto” sets the hints.ai_protocol to IPPROTO_TCP which is defined as 6.
Setting hints.ai_protocol manually to 0 makes everything work fine. I see the same done in Google’s GCP IoT library for example, where the mbedtls_net_connect() is reimplemented and the hints.ai_protocol is set to 0 as well.
Is this a bug in the code? I didn’t check if this works on Linux/Win or Cygwin as we are not using mbedTLS on those platforms.
mbedTLS version we are using is:
mbedTLS version: mbed TLS 2.16.6 branch released 2020-04-14
The code in question:
/*

    Initiate a TCP connection with host:port and the given protocol
    */
    int mbedtls_net_connect( mbedtls_net_context *ctx, const char *host,
    const char *port, int proto )
    {
    int ret;
    struct addrinfo hints, *addr_list, *cur;

    if( ( ret = net_prepare() ) != 0 )
    return( ret );

    /* Do name resolution with both IPv6 and IPv4 */
    memset( &hints, 0, sizeof( hints ) );
    hints.ai_family = AF_UNSPEC;
    hints.ai_socktype = proto == MBEDTLS_NET_PROTO_UDP ? SOCK_DGRAM : SOCK_STREAM;
    hints.ai_protocol = proto == MBEDTLS_NET_PROTO_UDP ? IPPROTO_UDP : IPPROTO_TCP;

    if( getaddrinfo( host, port, &hints, &addr_list ) != 0 )
    return( MBEDTLS_ERR_NET_UNKNOWN_HOST );

    /* Try the sockaddrs until a connection succeeds */
    ret = MBEDTLS_ERR_NET_UNKNOWN_HOST;
    for( cur = addr_list; cur != NULL; cur = cur->ai_next )
    {
    ctx->fd = (int) socket( cur->ai_family, cur->ai_socktype,
    cur->ai_protocol );
    if( ctx->fd < 0 )
    {
    ret = MBEDTLS_ERR_NET_SOCKET_FAILED;
    continue;
    }

Suggested enhancement
Not sure if the line in question is acting differently between IPv4 and IPv6 capable platforms. On IPv4 it needs to be modified to actually work.

I don't think ai_protocol has anything to do with IPv4 vs IPv6: that's the job of ai_protocol, and AF_UNSPEC allows replies for either protocol. ai_protocol is important if port is a symbolic name, but I think it generally doesn't influence the result if port is numeric, apart from perhaps a validtiy check.
I don't think changing to hints.ai_protocol = 0 would be acceptable because it would break mbedlts_net_connect(ctx, "www.example.com", "https", MBEDTLS_NET_PROTO_TCP).
A fully-capable POSIX platform should accept IPPROTO_TCP with a numeric proto (or a symbolic proto that is known for TCP), but it's certainly conceivable that an embedded stack would not recognize it.
What operating system or TCP/IP stack are you using? What value are you passing for proto? A short, full program would help so that we could run it and test.

Function is called with "MBEDTLS_NET_PROTO_TCP" for "proto" as specified in original post:
'''
which when function is called with MBEDTLS_NET_PROTO_TCP for “proto” sets the hints.ai_protocol to IPPROTO_TCP which is defined as 6.
'''
We are running on Mentor Graphics Nucleus OS 2014.12.

Sorry, I meant to ask what value you're passing for port. (I fixed my previous message to avoid future confusion.) In particular, it matters whether it's symbolic or numeric.

Port is passed as a character string containing port number, not service type. In our particular case, it is string "443".


Student form finder: add outcome for EU new students 2017-18

https://trello.com/c/IL5uba9q/467-27-march-changes-to-the-form-finder-for-eu-new-students
** Do not put live before 27 March 2017 **
Add outcome for EU new 2017-18 students
CHANGED
<% content_for :title do %>
EU full-time student
<% end %>
<% content_for :body do %>
You can't apply for the 2017 to 2018 academic year yet.
<% end %>
TO
<% content_for :title do %>
EU full-time student
<% end %>
<% content_for :body do %>
To apply for a Tuition Fee Loan use form EU17N.
Academic year | Form

| -
2017 to 2018 | EU17N - form and notes (PDF, xxxKB)

##Additional information
You may need to include additional information on the following forms.
Form | When to use the form

| -
EUTFLR - form (PDF, 40KB)  | To change the amount of loan you want to apply for
Evidence factsheet (PDF, 79KB) | List of evidence to send with your application - for example, proof of identity or income
Certifier checklist (PDF, 59KB) | Get the person who verifies your evidence to sign this form
EUCO1 - form (PDF, 137KB) | To update your course, name and other details

<%= render partial: 'when_you_can_apply.govspeak.erb' %>
<%= render partial: 'where_to_send_your_forms_non_uk.govspeak.erb' %>
<% end %>
REASON
New students from the EU can now apply for the 2017-18 academic year.

Superseded by PR #2957