added optional options to noprocess

This enables the option &noprocess to be expanded to &noprocess=id,hl
Example:
[PlayStore](https://play.google.com/store/apps/details?id=org.jitsi.meet&hl=de&target=_blank&noprocess)
Current result link: https://play.google.com/store/apps/details?id=org.jitsi.meet&hl=de&target=_blank - not opening in new tab
This will work as intended by not processing the id, however the target also gets no processed.
Using
[PlayStore](https://play.google.com/store/apps/details?id=org.jitsi.meet&hl=de&target=_blank&noprocess=id)
Will skip the id processing, and still process the target. Resulting in a better end result link.
New result link: https://play.google.com/store/apps/details?id=org.jitsi.meet&hl=de - opening in new tab successfully
This is in reference to the following issues:
https://github.com/getgrav/grav/issues/2839 - FIXED
https://github.com/getgrav/grav/issues/2882
https://discordapp.com/channels/501836936584101899/501854266873348112/729728029063905345

theres some bugs found in discord currently testing

[noprocess](https://play.google.com/store/apps/details?id=org.jitsi.meet&hl=de&target=_blank&noprocess)
result:  `https://play.google.com/store/apps/details?id=org.jitsi.meet&hl=de&target=_blank`
should have `target=_blank` in url and **not open in a new tab**

[noprocess=id](https://play.google.com/store/apps/details?id=org.jitsi.meet&hl=de&target=_blank&noprocess=id)
result:  `https://play.google.com/store/apps/details?id=org.jitsi.meet&hl=de`
should have `id=org.jitsi.meet&hl=de` in url and **open in a new tab**

[only target](https://play.google.com/store/apps/details?target=_blank)
result:  `https://play.google.com/store/apps/details`
should  **open in a new tab**

[Browser](https://meet.weikamp.biz/Support?rel=nofollow&target=_blank)
result:  `https://meet.weikamp.biz/Support`
should  **open in a new tab**

Test cases above

be great if you could add these to the test suite @ricardo118

Also will need some updates to learn to show how it can be used with options

I added some tests, and used those to confirm the functionality is good.  perhaps you can use these for future reference.

I needed to remove this "automatic extraction of links attributes" so I removed all of them in the configuration.
Un fortunately, as there is no attributes, the configuration element is no more an array but "null" and I got Exception "in_array() expects parameter 2 to be array, null given" (/system/src/Grav/Common/Page/Markdown/Excerpts.php at line 131) :
            // Loop through actions for the image and call them.
            foreach ($actions as $attrib => $value) {
                if (!in_array($attrib, $skip)) {
                    $key = $attrib;

                    if (in_array($attrib, $valid_attributes, true)) { // <---- this one is the 131
                        // support both class and classes.
                        if ($attrib === 'classes') {
                            $attrib = 'class';
                        }
                        $excerpt['element']['attributes'][$attrib] = str_replace(',', ' ', $value);
                        unset($actions[$key]);
                    }
                }
            }

My workaround without changing the code is to add a random string as a valid attribute in the hope no website will use it as valid attribute (but as I choose it randomly, chances are really tiny).
To fix it in the code, I changed the 117th line as follows (adding the ?: [] at the end) :
            $valid_attributes = $grav['config']->get('system.pages.markdown.valid_link_attributes') ?: [] ;

@ricardo118 Can you take a look?

ya will do

I needed to remove this "automatic extraction of links attributes" so I removed all of them in the configuration.
Un fortunately, as there is no attributes, the configuration element is no more an array but "null" and I got Exception "in_array() expects parameter 2 to be array, null given" (/system/src/Grav/Common/Page/Markdown/Excerpts.php at line 131) :
            // Loop through actions for the image and call them.
            foreach ($actions as $attrib => $value) {
                if (!in_array($attrib, $skip)) {
                    $key = $attrib;

                    if (in_array($attrib, $valid_attributes, true)) { // <---- this one is the 131
                        // support both class and classes.
                        if ($attrib === 'classes') {
                            $attrib = 'class';
                        }
                        $excerpt['element']['attributes'][$attrib] = str_replace(',', ' ', $value);
                        unset($actions[$key]);
                    }
                }
            }

My workaround without changing the code is to add a random string as a valid attribute in the hope no website will use it as valid attribute (but as I choose it randomly, chances are really tiny).
To fix it in the code, I changed the 117th line as follows (adding the ?: [] at the end) :
            $valid_attributes = $grav['config']->get('system.pages.markdown.valid_link_attributes') ?: [] ;

(I was going to submit a pull request with this change but did not find the Excerps.php file in develop or 1.7 branches so I don't know how to fix it in the github code).

i just tested this, and while yes what you say does happen, i fixed the error without having any issues by simply setting the config to
valid_link_attributes: [] instead of valid_link_attributes: null.
However, admin saves it as null, so i believe the check suggested here is valid.  Pushed a change here https://github.com/getgrav/grav/commit/2813934d21b04ef5c9a084dfea3e0873588d69e7


docs(ic): improved description of href and subprotocolBody fields

How the href should be formatted is still not clear enough. I tried to describe that in more detail also including parts where the connector configuration of the data provider is relevant.

Updated references to the public path based on feedback from @matgnt, so that it's closer to the naming in the settings/configuration of EDC.


MNT: Re-rendered with conda-smithy 2.2.1

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Cheat Engine Error 126

When adding the compiled plugin to Cheat Engine im getting an error stating that 'The plugin dll could not be loaded: 126'

Refer to #4


Hi i having an issue with ionic native that the page with the map will show for 2 seconds and then its the page close itself. Thanks and sorry for any inconvenience caused

I'm submitting a ...  (check one with "x")

[x] question
[ ] any problem or bug report
[ ] feature request

If you choose 'problem or bug report', please select OS:  (check one with "x")

[x] Android
[ ] iOS

cordova information:  (run $> cordova plugin list)
com-sarriaroman-photoviewer 1.1.16 "PhotoViewer"
cordova-plugin-camera 4.0.2 "Camera"
cordova-plugin-device 2.0.1 "Device"
cordova-plugin-geolocation 4.0.1 "Geolocation"
cordova-plugin-googlemaps 2.2.5 "cordova-plugin-googlemaps"
cordova-plugin-ionic-keyboard 2.0.5 "cordova-plugin-ionic-keyboard"
cordova-plugin-ionic-webview 1.1.16 "cordova-plugin-ionic-webview"
cordova-plugin-nativeaudio 3.0.9 "Cordova Native Audio"
cordova-plugin-nativegeocoder 3.1.1 "NativeGeocoder"
cordova-plugin-splashscreen 5.0.2 "Splashscreen"
cordova-plugin-whitelist 1.3.3 "Whitelist"

If you use @ionic-native/google-maps, please tell the package.json (only @ionic-native/core and @ionic-native/google-maps are fine mostly)
Current behavior:

Expected behavior:

Screen capture or video record:

Related code, data or error log (please format your code or data):
The person who share your project files on Github (or other git repository) is in faster lane than other people.
Please share your project files on Github or others(Bitbucket, Gitlabs...etc).
If you don't want to share your project files, please create a demo project, then share it.

Screen captures, and/or native logs(such as Logcat, xcode logs) are appreciate.

Giving much information, you are waiting time is less.
Thank you for your cooperation.

If you can't solve your problem by yourself, please share your project files on github.

ping

okay2.. 1 moment im pushing my code

hi @wf9a5m75  i did invite you in my repo. the maps is on pages/map. Sorry for the troubled

I can't move ahead.

ping

Just login no username or password required

Okay, then how can I reach to the map page?

Well, since you don't reply, I change the code like this.
export class LoginPage {

    constructor(public navCtrl: NavController, public navParams: NavParams) {
    }

    ionViewDidLoad() {
        console.log('ionViewDidLoad LoginPage');
    }

    actionLogin(){
        this.navCtrl.setRoot('MapPage', {}, {animate: true, direction: 'forward'})  <-- here
    }

}

And run it.

actually i did like you and it works.. but when open a modal to go to next page with map then map appear 2 seconds and the apps freeze..

hi @wf9a5m75 i solved it by close the modal first then redirect to the page with map.. thanks a lot for your help earlier

Sounds good


fix cds-dbm build for @sap/cds >= 5.4

- requires build task definition
{
    "use": "cds-dbm/dist/build/postgres-cf"
    . . .
}

An upcoming CDS version will support the registration of a custom build task provider.
This allows you to determine the required custom build tasks based on the project context
respectively CDS configuration instead of explicit definition.
E.g. a custom build task provider can inspect the configured database kind in
order to create a PostgressDB build task if "postgresql" has been configured.
This will also allow the definition of a build task based on the "for" alias
instead of the fully qualified module path provided by "use".

Hi @mikezaschka ,
maybe you can make @vobu and me also an approver.

Hi @LotharBender,
I've pulled your branch and updated all dependencies. But when I then run the test I get a lot of errors. One of them is:
 FAIL  test/PostgresAdapter.test.ts (273.102 s)
  ● PostgresAdapter ›  drop ()  › + dropAll: false + should remove all cds based tables and views from the database

    TypeError: Cannot read property 'requires' of undefined

      58 |
      59 |     beforeEach(async () => {
    > 60 |       cds.env.requires.db = Object.assign({ kind: 'postgres' }, options.service)

maybe you can try gregorwolf/cds-dbm/tree/fix/issue-34.
CU
Gregor

Hi Gregor,
sorry for delay....
I can reproduce the issue, but I get the very same errors when testing the main branch using @sap/cds version 5.1.5.

npm run build
npm test

Regards,
Lothar


Is there any way to run otel autoinstrumentation without the --require or --import flag for JS/TS?

[x] This only affects the JavaScript OpenTelemetry library
[ ] This may affect other libraries, but I would like to get opinions here first

When setting up NodeJs with Typescript, the documentation mandates using the --require flag.
Is there any way to avoid this pattern, by simply running the code in instrumentation.ts first before other code by means of importing it (e.g. in app.ts)?
I am concerned autoinstrumentation by means of --require or --import would affect readability.

You could do something like require("./instrumentation") as your first line of import in your index.ts or the main file that gets executed first


Feat: Add git branches to Algernon

PR is still a draft, not reviewing. Also the test coverage seems to have decreased quite a bit. Have you written test for your stuff yet?

Tests are done, but are currently still marked as expected failures until SquitBrowser is added to the CI on this branch. That should resolve the coverage issue.
Review can wait until marked ready for review...


[BUG?]seems to never focus on the correct note location

Hi again
Coming back to map-view after a while
I still dont seem to get it  (and im a GIS professor ;-))
see video: https://paste.xinu.at/gmQ3S/
if I have this in the FM yaml:
---
tags:
- review
- באר-שבע
status: 5
rating: 3
location: [ 31.371154785156, 34.823097229004 ]
counter: null
---
# .דלפק

חנות שתמחה בפיתות שף

and I issue map view open map view am i not supposed to get a zoomed view of the actual note location , ie 31.371154785156, 34.823097229004 . currently it only opens in every not the same world view

what am I missing here :)

Welcome back! ונחמד לראות עברית כאן :)
AFAIK all the old bugs of not opening at the right state are long gone. I'd love to help but I just need to know a few details:

Is this the latest Map View version?
How are you opening Map View? Is it from the note's menu? The global ribbon icon? Something else?

It might be the "focus in map view" menu item that you're looking for (from the note's context menu), rather than the global command of opening Map View with the default state.

Oh sorry I just noticed you also pasted a video.
Try to right-click the 'location' line and click "show on map", or alternatively open the note context menu (the 3 dots on the far right), where you should see "focus  in Map View". That should do what you're aiming for.

אהלן :)
thanks for much for your kind and swift response!
I see now yeah thats much more of what I had in mind
it seems though that it still is opened pretty zoomed out for example for a location in NYC it opens like this

as opposed to when I manually zoom in into the actual location (ie Katz deli:)

is that by design?
thanks so much again for you epic work on this , its very much appreciated
תודה רבה :)

Can you play around with your "default zoom for 'show on map' action" in the setting? It's possible that the default doesn't look so great on your desktop.

yup that did it 😊
thanks so much!
שיהיה שבוע נפלא


Package SDK Improvements

This is the master plan to improve our SDK experience for package writers.
 and NPM publishing. It is constantly breaking and we are not testing it in our daily operations. Instead, rely on clone target repo + npm install. This flow is tested by travis
[ ] automatically surface git repo under projects as extensions
[x] allow editing of git repo packages in editor
[x] make native dependencies optional
[x] make --cloud default and local build optional

Hi pelikhan,
I found that if I updated the main.ts of package on github, I need to wait at least 20 mins to see the change in makecode.
Is there anyway to preview the blocks now?

@jxgx072037 you can take a look at https://makecode.com/blog/github-packages, it's a new feature where you can import your repo and make modifications directly in the editor

Fixed with github extension?


Metadata encoding unexpectedly changed

https://github.com/bincode-org/bincode/commit/b34ee9541d0054a839b67fa8e39a488e4ff4dc8b
All metadata written prior than v0.2.12 are not compatible with later versions.

Fixed in v0.3.x


Update run_first.py

Pull Request Title

Related Issue

Description

Type

[ ] Bug Fix
[ ] Feature Enhancement
[ ] Documentation Update
[ ] Code Refactoring
[ ] Other (please specify):

Proposed Changes

Screenshots / Code Snippets (if applicable)

How to Test

Checklist

[ ] The code compiles successfully without any errors or warnings
[ ] The changes have been tested and verified
[ ] The documentation has been updated (if applicable)
[ ] The changes follow the project's coding guidelines and best practices
[ ] The commit messages are descriptive and follow the project's guidelines
[ ] All tests (if applicable) pass successfully
[ ] This pull request has been linked to the related issue (if applicable)

Additional Information

path file is not correct

The paths updated in your commit corresponds to the processed versions of the resume(s) and job description(s). Kindly recheck if this is a valid change and corresponds to an open bug or a new feature.


Re-export react-dom/server

https://github.com/overlookmotel/react-async-ssr/issues/19
Re-exports top-level react-dom/server API.

I'm closing this PR.
Please see https://github.com/overlookmotel/react-async-ssr/issues/19#issuecomment-531476525 for my rationale.


Compiled code example

This seems like such a cool project!  While I'm planning to mess around to understand it - I think if you showed the App example as a compiled button it would allow users to see it all in action.

Good point. There's an example of the generated code now in the code (though v1 is different).


Charter needs an update

I have updated the website with the 8th gathering sign up: https://github.com/Praqma/web-codeall/issues/64
However, the part of the Charter about Gatherings needs an update. It states that Praqma is covering all costs and that gathering are hosted at companies. It needs an update.
http://alliance.praqma.com/charter/
"At least once every half year, Praqma is responsible for organizing a gathering, that shall be open to all members.
The gatherings may be hosted at facilities and venues at the members’ organizations. Expenses for executing the gatherings shall be covered by Praqma.
Each alliance member is entitled to register two self-appointed (chief and geek) organization representatives to participate, free of additional cost, at the gatherings.
Each alliance member is entitled to send more participants to the gatherings than just the two designated roles. However, additional gathering participants may be charged a meet-up fee to cover the venue and catering costs of the gathering."

Members page, see #67 for updates.

@lakruzz made a small rewrite to make it fit the new code camp format... if the text isn't easy to grasp or understand, a think a slightly bigger change is needed.
... just tried edit to new format, with out to many changes.


git clone with a depth of 500 to fix the version

Current Travis CI builds use a version that starts with "0.0.0" instead
of "0.3.0":

[info] Packaging /home/travis/build/fthomas/scala-steward/modules/core/.jvm/target/scala-2.12/scala-steward-core_2.12-0.0.0-167-fe8f4da7.jar ...

I guess sbt-dynver can't correctly determine the version with a clone
depth of 50 (which is the default). This sets the git clone depth to
500 to fix the version.

[info] Packaging /home/travis/build/fthomas/scala-steward/modules/core/.jvm/target/scala-2.12/scala-steward-core_2.12-0.3.0-238-2d603bfa.jar ...

Looks good. 262 commits left before 0.4.0 need to be released. :-)


Ensure prepare.sh works for hosts with a bond interface

Found while testing in SoftLayer, where physical hosts have bonds
instead of individual network interfaces.
Signed-off-by: Kyle Mestery<EMAIL_ADDRESS>

@huikang Found this in testing today.


Test content of Download Application as PDF response

Context
Introducing middleware to add compression recently caused PDF downloads to fail.
PDFKit doesn't modify the status of the response if it encounters errors, it probably should but we don't control that.

Changes proposed in this pull request
Adds a way to read the text from the PDF response and verify the candidate name and support reference are present.

Guidance to review

Link to Trello card

Things to check

[x] This code doesn't rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] New environment variables have been added to the Azure config

The test breaks for me locally. We can pair on it when you have some time. 👍

Shall we close this for the time being?

Closing as we think we're experiencing this bug in tests https://github.com/wkhtmltopdf/wkhtmltopdf/issues/2148 also this https://github.com/pdfkit/pdfkit/pull/469 might solve the testing by status code issue.


Refactor Codebase to take out the files to root folder instead of having them in app folder

Currently, all the files of the repo are contained in the app folder. They should be present in the root folder like other Python Projects.

Fixed in #46


Enabled multi-nodes pruning of ResNet50 on cpu device.

Type of Change
Example enabling
API not changed
Description
Enabled multi-nodes pruning of ResNet50 on cpu device.
Expected Behavior & Potential Risk
results on multi-nodes is comparable with single node.
How has this PR been tested?
follow readme
Dependency Change?
no library dependency introduced or removed

Please update spell check dict.


DAWN-695 ⁃ Deferred transaction bugs

cancel_deferrerd is able to cancel transaction which will be created in the same action after the call to cancel_deferred
it's possible to create multiple transactions with the same sender_id
minor bug: print() doesn't work in deferred transactions (nothing appears in console)
apply_context::execute_deferred uses default_max_gen_trx_count instead of actual parameter

Case 1 and 2 are verified by the existing api_tests/deferred_transaction_tests unit tests, while case 3 and 4 are verified by the additional unit test https://github.com/EOSIO/eos/pull/2173


magic.solana.signMessage fails on react-native

✅ Prerequisites

[ ✅ ] Did you perform a cursory search of open issues? Is this bug already reported elsewhere?

Mentioned in https://github.com/magiclabs/magic-js/issues/354 but closed - I do not think the issue is resolved.

[ ✅ ] Are you running the latest SDK version?
[ ✅ ] Are you reporting to the correct repository (magic-sdk)?

🐛 Description
The web codesandbox here https://codesandbox.io/s/ma-otp-forked-8yt5e7 works but copied line-for-line into react native, and changing
-import { Magic } from "magic-sdk";
+import { Magic } from "magic-sdk/react-native

-    await magicClient.auth.loginWithMagicLink({
-      email: email,
-    });

+    await magicClient.auth.loginWithSms({
+      phoneNumber: phoneNumber,
+    });

Seems to break it.
The Magic client instantiates,  Logging in with magic works, but the signMessage method throws that error no matter what I provide as an argument.
For example:
      const bytes = new Uint8Array([72, 101, 108, 108, 111]); // "Hello"
      const signature = await magicClient.solana.signMessage(bytes);

throws
 [Error: Magic RPC Error: [-32603] Internal error: unexpected type, use Uint8Array]

Do you have an example of this working in a react-native environment?
🤔 Expected behavior
signMessage returns a signed message
😮 Actual behavior
Throws
 [Error: Magic RPC Error: [-32603] Internal error: unexpected type, use Uint8Array]

🌎 Environment

Software
Version(s)

magic-sdk/react-native
10.0.0

ios
15.5

iPhone
13 pro

npm
8.12.2

react-native
>=0.68.1

Hey @kelly-ry4n. Thx for reporting this. I'm taking a look now and shall get back to you shortly.

@kelly-ry4n, the issue is caused by JSON.stringify mangling the original object structure. I'll make a patch soon, and it may requires you to update the Magic RN SDK

Thanks @Ethella -- Do you have an ETA on the patch fix?

Thanks @Ethella -- Do you have an ETA on the patch fix?

It should be out today. I will leave a note in the thread.

:rocket: Issue was released in<EMAIL_ADDRESS>:rocket:


Investigate blank user agent strings in analytics

Some are legit, some are bots.

Write a script for removing listens and subs from a particular ip
Run said script

Blank user agents are now blocked by https://github.com/Pinecast/pinecast-lambda/commit/665c64c88661e80b3b3f4a60bdfa9aaa3676a7d8


Right Rotate may be wrong

Ckks scheme, I tried to use RotateColumns () function, but got the wrong answer.
I tried to find the cause of the error, and finally found that when commenting out part of the code of the RotateColumns () function, the result will be correct. But this may slow down the efficiency of rotation.
The commented code is:
// if utils.HammingWeight64(k) <= utils.HammingWeight64((eval.ckksContext.n>>1)-k) {
					eval.rotateColumnsLPow2(ct0, k, evakey, ctOut)
				// } else {
				// 	eval.rotateColumnsRPow2(ct0, (eval.ckksContext.n>>1)-k, evakey, ctOut)
				// }

So I guess the cause of the error may be related to right rotation.

Hello Dylan,
Thank you for your feedback. Could you tell me more about the parameters and the rotations amount that you used so that I can look into it?

Thank you for your reply.
I didn't find the error when I tried. Maybe I made a mistake.


Include URL in library information

Closes #20
Usage
https://github.com/cybozu/LicenseList/assets/84154073/21d4bcf8-9ce2-4776-b886-c1d605c2282c
import SwiftUI
import LicenseList

struct ContentView: View {
    @Environment(\.openURL) private var openURL

    var body: some View {
        NavigationStack {
            List(Library.libraries) { library in
                NavigationLink(library.name) {
                    Text(library.licenseBody)
                        .padding(.horizontal, 16)
                        .navigationTitle(library.name)
                        .toolbar {
                            ToolbarItem(placement: .topBarTrailing) {
                                Button {
                                    if let url = library.url {
                                        openURL(url)
                                    }
                                } label: {
                                    Image(systemName: "link")
                                }
                            }
                        }
                }
            }
            .navigationTitle("License")
        }
    }
}

I think it would be better to add usage instructions to the README.
### Example for UIKit

```swift
import LicenseList

// in ViewController
let vc = LicenseListViewController()
vc.title = "LICENSE"

// If you want to anchor link of the repository
vc.licenseListViewStyle = .withRepositoryAnchorLink

navigationController?.pushViewController(vc, animated: true)
```

### Example for SwiftUI

```swift
import LicenseList

struct ContentView: View {
    var body: some View {
        NavigationView {
            LicenseListView()
                // If you want to anchor link of the repository
                .licenseListViewStyle(.withRepositoryAnchorLink)
                .navigationTitle("LICENSE")
        }
    }
}
```

@Kyome22
Fixed!


Receiver address not respected in TRX with multiple token transfers

Hi,
here is a transaction were a program airdropped tokens to different Solana accounts.
This transaction is related to a StarLaunch IDO, the Token distrubution is somtimes correctly recognized by staketax and sometimes missing, but in this case even the wrong amount is booked.
Reproduce
Take whatever receiver address you want from the transaction, generate a report, the SNS amount will always be the one from the most top action in the transaction. So for the provided transaction it's always ~ 38.91 SNS

Thanks for nice details.  I should have a fix in pretty soon.

Fixed and deployed with https://github.com/hodgerpodger/staketaxcsv/commit/82e556f8becbff3c724eba1c7d929672568a9baf


Bump go to 1.22.4 to fix ray-operator vulnerabilities

Why are these changes needed?
Vulnerability scanning shows Critical and Medium severity vulnerabilities in ray-operator v1.2.0-rc.0 image due to go version 1.22.3. ray-operator image: gcr.io/gke-release-staging/ray-operator@sha256:56f36fe9a805a271b218f16df009f06a1d785d53ff4b27a5bb2e5c33f52e7d97. This PR bumps the go toolchain to 1.22.4 to fix these errors.
Related issue number

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[x] This PR is not tested :(

cc: @kevin85421 @andrewsykim


Maven capability not picked up

Dear,
maybe it is the expected behavior, still I couldn't find any reference to it. I have maven installed as suggested by this link https://maven.apache.org/guides/getting-started/windows-prerequisites.html . After installing the agent on that server, I do not see in the list of capabilities an entry called maven pointing to mvn file. Maven build step is expecting this capability to be set. Is this behavior expected or is this a bug?
Thanks

Please excuse me, issue is actually presenting itself in the windows vnext agent. However, if anyone can help or point me to the right direction, any help will be appreciated.

Capabilities basically try to find the tool.  Typically that's by being in the path.
Is maven resolvable via PATH?  If you type maven from a cmd prompt?
Next detail is what account is that build agent running as (running as a service?)  The default is network service - if it's in the PATH of your logged in account, perhaps network service can resolve it via it's paths.  If you can resolve it, you can try changing the account it runs as (note, you have to re-run agent config to do this since it sets ACLs).
Finally you can add that capability in the web UI on the agent so it routes there but note that when the task runs, it still needs to locate/find maven.

I just had the same problem. This is how I fixed the issue:

Remove the registered agent
Go to the environment variables on your server in 'System'
Add two variables "MAVEN_HOME" and "M2_HOME" with both as value the installation directory of maven. In my case is this: "C:\Program Files\Java\apache-maven-3.3.9"
Restart server
Register agent again and voila, the agent should have the capabilitie: "MAVEN_HOME", "M2_HOME" and "maven"


Add metadata_max config to help performance on devices with large blocks

Alright, hopefully I've done this correctly. Inspired by our conversation on #203.
Not sure of the best way to validate performance "properly", if there's a test bench already setup I should be using, but I am still seeing a 10x increase in directory traversal/file deletion and a 2x-4x speedup in file reads with this code and a metadata_max of 4096 and inline_file_max of -1.
Commit message:
We have seen poor read performance on NAND flashes with 128kB blocks.
The root cause is inline files having to traverse many sets of metadata
pairs inside the current block before being fully reconstructed. Simply
disabling inline files is not enough, as the metadata will still fill up
the block and eventually need to be compacted.
By allowing configuration of how much size metadata takes up, along with
limiting (or disabling) inline file size, we achieve read performance
improvements on an order of magnitude.

Also, as far as backwards compat/superblock inclusion goes, I've been able to move backwards and forwards between custom/default values with no data loss, errors or asserts. I don't know the codebase well enough to be sure, but I believe since these new limits only come into play when deciding to compact, adding new blocks, or rewriting files, they will be silently applied only when required, and old blocks can be traversed fine.

This looks great!
The only question I have is do we need inline_file_max if we have metadata_max? It is currently based on block_size/8, but could instead be changed to metadata_max/8. I'm not sure a separate inline_file_max is useful.

Not sure of the best way to validate performance "properly", if there's a test bench already setup I should be using

Unfortunately not at the moment. It's on my TODO list to put together a set of benchmarks similar to the tests, and would be the first step in addressing the current performance issues.
Performance hasn't been a high priority for littlefs, and the only real basis for performance during development was theory, so the current performance issues probably shouldn't have been a surprise. I believe "you are what you test" is how the saying goes...

all our files are strictly larger than the inline file size, so I want to be able to disable them entirely

Ah, if you create a file larger than the inline file size, it should never get written out to disk as an inline file (unless there's a bug, or you call lfs_file_sync while the file is still inline size).
Due to other conditions with multiple open files, open inline files are stored entirely in the file cache. So in theory, when you write a file larger than cache_size, by the time littlefs needs to flush the data to disk it will have already decided the file should not be inlined.

Aha, and that explains why we had the perf issues at 128k blocks and 4k cache - the inline file limit is 16k, so it will always be written out before un-inlining.
Fix commits incoming shortly!

Looks perfect 👍 Thanks for following all the unwritten style rules.
Will bring this in on the next minor release.

Assuming it passes CI ;)
But it did pass the first time

perf issues at 128k blocks and 4k cache - the inline file limit is 16k

Hmmm, the actual check is against min(0x3fe, cache_size, block_size/8) (0x3fe is due to a 10-bit size field). So in theory it still should not have been written out as inlined:
https://github.com/littlefs-project/littlefs/blob/1a59954ec64ca168828a15242cc6de94ac75f9d1/lfs.c#L2966-L2969

Hmm, are directories/dir entries stored as files, and thus inline-able? That's the only other possibility if they were indeed being inlined as that 0x3fe shows.

Nope, littlefs doesn't do directory inlining. I wonder if it might have been a bug.

Thanks for the PR!

Thanks for the PR!


Create await utils function to make async calls sync

is await likely to be coming to the Swift language? Might be worth naming your function something else if so

Warnings

:warning:

Any changes to library code should be reflected in the Changelog.
Please consider adding a note there and adhere to the Changelog Guidelines.

Messages

:book:
Danger

  Generated by :no_entry_sign: Danger Swift against b4ebf06a8bcedb3182bcf6b8a6ddd5acba3edf40

@orta this is a really good call, there is a proposal for await (https://gist.github.com/lattner/429b9070918248274f25b714dcfc7619), then is possible it becomes part of the language at one point.
Then I think it might make sense to rename it, thank you for the suggestion.


Freezing layers with eqx.tree_map (not parameters in a single layer)

Hello all,
Running around in circles a bit here. I'm trying to implement the 'Stochastic Depth' layer in the Compact Convolutional Transformers paper.
This is basically dropout applied to a sequence of layers as opposed to elements of a weight matrix.
To do this, I thought I could tree-map-mask the pytree list of layers, somehow indexing a random percentage of them to target (akin to dropout).
I'll save pasting a lot of useless workings, but I thought something like this should work

key = jr.key(0)

# Layers to drop (should end up with all the layers after, but some with all arrays having their gradients stopped)
layers = tuple([eqx.nn.Linear(1, 1, key=key) for _ in range(4)])

rate = 0.5 # Drop half of the layers
ix_of_layers_to_drop = jr.choice(key, jnp.arange(len(layers)), (int(rate * len(layers)),))

layers_ = eqx.tree_at(
   # Return nodes of layers (that are linear layers themselves) that should be replaced in layers with 'frozen' equivalents
    lambda layers: [layers[i] for i in ix_of_layers_to_drop],
    # Only stop gradients of arrays
    eqx.filter(layers, eqx.is_array),
    # Replace leaves with 'frozen' leaves
    replace_fn=lambda x: jax.lax.stop_gradient(x)
)
assert len(layers_) == len(layers) # Not really testing much...

does this make sense?
One hiccup may be that jax.lax.stop_gradient works on PyTrees, meaning that the replace function directly replaces entire linear layers in this case (the array-leaves of them at least!). I've applied this function to the array-only PyTrees of the linear layers which should be correct.
Cheers
Jed

It looks lke you have a tracer (ix_of_layers_to_drop) that you're using to try and index into a list (layers[i]). You've not mentioend what your error message is but I suspect this is what you're bumping into?
(Side note, I don't see the connection between dropout and freezing layers -- from your link it looks more like certain values are randomly set to zero.)

Sorry totally missed how this works, fixed it with the true method of the module. Cheers!


Put the (*Server).disableAutoGroupTags field in the OpenAPI struct

Is your feature request related to a problem? Please describe.
The disableAutoGroupTags field is only about OpenAPI and does not impact runtime.
Describe the solution you'd like
It should be placed in the OpenAPI struct instead of the Server
As this is non-critical for our users, I would be happy if a newcomer takes this issue :) It should be a 20 lines change.

As grouping is a part of server, we should keep it related to the Server struct


Manually running Pipeline appears to connect gitlab timeout error

I manually run Pipeline and I get an error connecting gitlab repo timeout. The picture is as follows, I initially suspected the root size of my repo, my repo storage size is 1G

@gitlawr

Useful
Info

Versions
Rancher v2.1.6 UI: v2.1.19

Route
global-admin.clusters.index

Close in favor of https://github.com/rancher/rancher/issues/20191


resolve project not being set in offline context creation

What type of PR is this?
/kind bug
What does does this PR do / why we need it:
when we are creating offline context - we are not setting the project which could be a major bug when using any command in non-cluster mode.
Which issue(s) this PR fixes:
No issue
PR acceptance criteria:
Not sure how to test this
~~ - [ ] Unit test

[ ] Integration test

[ ] Documentation ~~

[X] I have read the test guidelines

How to test changes / Special notes to the reviewer:

adding @mik-dass as he recently updated this file


Question about custom dataset

Thank you so much for providing such excellent research!
I am currently attempting to reproduce your paper (https://github.com/zc-alexfan/hold/blob/master/docs/custom.md), but I’ve encountered some issues and would like to ask a question.
I am working in a pyenv environment and have set up all the virtual environment configurations correctly. I have completed up to the step of using the METRO hand tracker (right-hand only; CVPR paper method), and I skipped Hamer since I don't think it’s necessary.
Now, when I try to run the code for Object pose estimation, specifically the command
cdroot; pycolmap scripts/colmap_estimation.py --num_pairs 40 --seq_name $seq_name

I get the following error message:
{'model': {'name': 'superglue',
           'sinkhorn_iterations': 50,
           'weights': 'outdoor'},
 'output': 'matches-superglue'}
[2024/12/19 20:59:53 hloc INFO] Skipping the matching.
[2024/12/19 20:59:53 hloc WARNING] The database already exists, deleting it.
[2024/12/19 20:59:53 hloc INFO] Creating an empty database...
[2024/12/19 20:59:53 hloc INFO] Importing images into the database...
[2024/12/19 20:59:55 hloc INFO] Importing features into the database...
100%|________________________________________________________________________________________________________________________________________________________________________________________________________________________| 151/151 [00:00<00:00, 1932.23it/s]
[2024/12/19 20:59:55 hloc INFO] Importing matches into the database...
100%|______________________________________________________________________________________________________________________________________________________________________________________________________________________| 6040/6040 [00:03<00:00, 2000.83it/s]
[2024/12/19 20:59:58 hloc INFO] Performing geometric verification of the matches...
[2024/12/19 20:59:59 hloc INFO] Running 3D reconstruction...
[2024/12/19 21:00:11 hloc ERROR] Could not reconstruct any model!
Traceback (most recent call last):
  File "scripts/colmap_estimation.py", line 33, in <module>
    colmap_pose_est(seq_name, num_pairs)
  File "./src/colmap/colmap_utils.py", line 321, in colmap_pose_est
    visualization.visualize_sfm_2d(model, images, color_by="visibility", n=5)
  File "/home/hold/generator/Hierarchical-Localization/hloc/visualization.py", line 16, in visualize_sfm_2d
    reconstruction = pycolmap.Reconstruction(reconstruction)
ValueError: [reconstruction.cc:86] Check Failed: ExistsReconstruction(path) cameras, images, points3D not found at None

Could you please advise me on how to resolve this?
best regards

It seems that there is no COLMAP point clouds reconstructed. Can you check if the input of COLMAP make sense? Also, I suggest to try our preprocessed dataset first before creating a custom dataset.

Thank you for the quick response!
I checked the code and tried reducing the num_pairs to 30, and fortunately, it worked perfectly!
It seems that some parameter tuning may be required during the process of creating the custom dataset.
Thank you again! :)


Updated gradebook delete command

added tests for gradebook add parser

Coverage increased (+3.4%) to 69.096% when pulling b72ebfd2691ea7e446375e1fe81be3a8bcf5744b on tristyxxnana:UpdatedGradebookDeleteCommand into 5d0a065616291da82dda25163cd9524a396a97c7 on CS2113-AY1819S1-T16-1:master.


cat not set multi scopes when add oauth2 plugin on a service

Summary
SUMMARY_GOES_HERE
Steps To Reproduce

create a service
create a route for this service
add oauth2 plugin on this service
I want  "scopes": [“email”, "phone", "address"]，but only return  "scopes": ["address"]

use config.scopes[] is the same as config.scopes
"config.scopes[]=email"
"config.scopes[]=phone"
"config.scopes[]=address"
offical docs
https://docs.konghq.com/hub/kong-inc/oauth2/
Enabling the plugin on a Service

$ curl -X POST http://kong:8001/services/{service}/plugins \
    --data "name=oauth2"  \
    --data "config.scopes=email" \
    --data "config.scopes=phone" \
    --data "config.scopes=address" \
    --data "config.mandatory_scope=true" \
    --data "config.enable_authorization_code=true"

{
    "created_at":<PHONE_NUMBER>,
    "config": {
        "refresh_token_ttl": 1209600,
        "enable_client_credentials": false,
        "mandatory_scope": true,
        "provision_key": "LBOuaaUVSbwdsD3COjcMO6Uy3y8KoX9x",
        "accept_http_if_already_terminated": false,
        "hide_credentials": false,
        "enable_implicit_grant": false,
        "global_credentials": true,
        "enable_authorization_code": true,
        "enable_password_grant": false,
        "scopes": [
            "address"
        ],
        "anonymous": null,
        "token_expiration": 7200,
        "auth_header_name": "authorization"
    },
    "id": "46333a6f-75d7-4548-89fa-a5789d25d21c",
    "service": {
        "id": "009c1dcd-52c1-4d43-916d-96b052198db3"
    },
    "enabled": true,
    "protocols": [
        "grpc",
        "grpcs",
        "http",
        "https"
    ],
    "name": "oauth2",
    "consumer": null,
    "route": null,
    "tags": null
}

Additional Details & Logs

Kong version (2.0.4)
Kong debug-level startup logs ($ kong start --vv)
Kong error logs (<KONG_PREFIX>/logs/error.log)
Kong configuration (the output of a GET request to Kong's Admin port - see
https://docs.konghq.com/latest/admin-api/#retrieve-node-information)
Operating system (docker kong 2.0.4)

I could not reproduce with latest version. So I think it is fixed. Thus closing this. Please reopen if you still see this.


Format function comments with JSDOC comment styles.

See comments above is_array fucntion for reference.

Created pr for that https://github.com/Hetarth02/js-array-helpers/pull/14


Nick: Watch madloki.pro

Nick requests the watch of the watch_keyword madloki\.pro. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

madloki\.pro has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by cigien in Charcoal HQ


SOLR-13751: add BooleanSimilarityFactory

Description
This PR exposes Lucene's BooleanSimilarity in Solr.
Solution
Please provide a short description of the approach taken to implement your solution.
Tests
Please describe the tests you've developed or run to confirm this patch implements the feature or solves the problem.
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[x] I am authorized to contribute this code to the ASF and have removed any code I do not have a license to distribute.
[x] I have developed this patch against the master branch.
[ ] I have run ant precommit and the appropriate test suite.
[x] I have added tests for my changes.
[ ] I have added documentation for the Ref Guide (for Solr changes only).

Thanks for comments so far! I've created https://github.com/apache/lucene-solr/pull/867 to replace this one.


Read/convert column orientated spreadsheets

Looking to read/convert column orientated spreadsheets into JSON with keys in the first column.  Does SheetJS support this spreadsheet layout/format?
Thanks,
Mike Brown

The utility functions are row-oriented, but it's not too hard to add a flag to support column-oriented accesses.  They aren't that complex, so you can actually change them yourself while waiting!
The actual sheet_to_json definition is pretty short: https://github.com/SheetJS/js-xlsx/blob/master/bits/90_utils.js#L1 .  The header loop iterates over the first row, the core loop iterates over the rows and the inner loop iterates over the sheets.  Those can be flipped for column-oriented access.

Thanks for the pointer.  Much appreciated.

Can you please provide the ode to proceed


dbHost not recognized

Hi whenever i type in npm start it says "dbHost not recognized."

Hello @evan9523,
Thanks for opening the issue.
Could you please provide a screenshot or the full error displayed in the terminal?

Hello I am having the same issue. 'db host' is not recognized as a command
screenshot attached

Hi! @thisistuhinsen and @evan9523 in your package.json can you change
"scripts": {
    "start": "dbHost=localhost node ./bin/www"
  },

to
"scripts": {
    "start": "export dbHost=localhost; node ./bin/www"
  },

Now its showing export not recognized as a command

@thisistuhinsen and @evan9523 change it to
"scripts": {
    "start": "node ./bin/www"
  },

Closing the issue as it has been resolved

Also, spruce.divy.work user page is broken @DivySrivastava


Update catalog_date field in traject using the publishing job files

Publishing profiles
refs https://github.com/pulibrary/marc_liberation/issues/822#issuecomment-719602951
Bibliographic Enrichment -> Create date subfield 950b
Physical Items Enrichment -> Create date subfield 876d
Electronic Inventory Enrichment -> Activation date subfield 951w

Similar to the Single Record if there is a 876 field don't use the 951 fields.
If there are no 876 or 951 then use the bib create date: Bibliographic Enrichment 950b.

closed by #945


[WFLY-15850] Bump Undertow version to 13.0

https://issues.redhat.com/browse/WFLY-15850

@fl4via please review

Supersedes: https://github.com/wildfly/wildfly/pull/14595

I have added an approval as we have a long chain of PRs in the queue depending on this one.


Implement Telnyx SMS adapter

What does this PR do?
Implements a new SMS Adapter for sending via Telnyx
Test Plan
I've successfully sent a test SMS using the new Telnyx adapter.
Related PRs and Issues
Closes #13
Have you read the Contributing Guidelines on issues?
Yes

@dcowan-london, would you please fix the linting error?

@stnguyen90 @abnegate Sorry to nag, but following up on this?

Thank you!


Added Confirm Button With Confetti Animation

This pull request introduces the implementation of the Confirm Button With Confetti Animation, a fun and engaging interactive button for users. The purpose of this PR is to add this exciting button to the repository and make it available for users to enjoy. 🎊😊
Fixes: #690

Hi, there seems to be a merge conflict, please resolve it


Add new E2E tests

This is a preliminary PR, not ready for checkin yet.
Attempts to replicate the behavior of copying precompiled test executables
to the host, and executing them there.
Includes the ability to set up IIS (a prerequisite for the tests) on windows hosts.
As currently coded, will only work when both the tester and the target are windows hosts.
Clearly needs updating to work with linux testers, as that's the target environment.<!--

New contributors are highly encouraged to read our
CONTRIBUTING documentation.
Both Contributor and Reviewer Checklists are available at https://github.com/DataDog/datadog-agent/blob/main/docs/dev/contributing.md#pull-requests.
The pull request:

Should only fix one issue or add one feature at a time.
Must update the test suite for the relevant functionality.
Should pass all status checks before being reviewed or merged.

Commit titles should be prefixed with general area of pull request's change.
-->

What does this PR do?

Move init to SetupSuite and remove globals
Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

These changes were integrated in the target branch


OCM-2360: Add support for importing a classic cluster

Adds support for running terraform import to pull in a cluster.
Usage:
Add a minimal definition of the cluster to the .tf file:
resource "rhcs_cluster_rosa_classic" "mycluster" { }

Run import on that resource, passing the cluster's id:
$ terraform import rhcs_cluster_rosa_classic.mycluster 24q7p8vthmm5erql10nn3qovq45juhsk
rhcs_cluster_rosa_classic.mycluster: Importing from ID "24q7p8vthmm5erql10nn3qovq45juhsk"...
data.rhcs_policies.all_policies: Reading...
rhcs_cluster_rosa_classic.mycluster: Import prepared!
  Prepared rhcs_cluster_rosa_classic for import
rhcs_cluster_rosa_classic.mycluster: Refreshing state... [id=24q7p8vthmm5erql10nn3qovq45juhsk]
data.rhcs_policies.all_policies: Read complete after 1s
data.aws_caller_identity.current: Reading...
data.aws_caller_identity.current: Read complete after 0s [id=765374464689]

Import successful!

The resources that were imported are shown above. These resources are now in
your Terraform state and will henceforth be managed by Terraform.

/retest

/cc @nirarg

/hold
Tracking down an issue w/ the STS fields after a subsequent apply()

/retest

@JohnStrunk this is look great
Great job on investigating this
Please let me know when all validation are done so we can merge

/unhold
/cc @nirarg

/approve
/lgtm


typo fix in percona-xtradb-cluster/README.md: duplicated word "image image"

line 98:
'image' is duplicated.

/assign @paulczar

/assign @timstoop
Hello, This typo is very obvious, Please can you approve it? It's here already one week.
Thanks a lot!

/ok-to-test
/lgtm


FilePanel edit

Suppress the Edit icon and restrict server access unless ALL of the following conditions are met:

User has SiteAdmin role
File is text; not binary
Editing is enabled in a filepanel config setting
File size does not exceed 100k

We are not going to implement this right now because in Paas this is not required and in JWS/FOSS, Web Engg team do not want passwords to show up. One way to implement is to remove the file content when user clicks edit on the file containing passwords and display a message like this
"Please copy paste the updated contents here."
Here are the files which already were modified to update a property file without passwords.
FileView.txt
FilePanelService.txt
Toolbar.txt


Server creation failing due to $node fields being null

Panel or Daemon: Panel
Version of Panel/Daemon: v0.7-rc2
Server's OS: Ubuntu 16.04
Your Computer's OS & Browser: Windows, Google Chrome (Using postman to make API requests)

Add Details Below:
I was playing with server creation in the API, I am quite sure my request is right, I have looked in the StoreServerRequest class and all my fields are correctly validated, so that's not the problem.
However when I then make a request I get:
{
    "errors": [
        {
            "code": "NoViableNodeException",
            "status": "400",
            "detail": "No nodes satisfying the requirements specified for automatic deployment could be found."
        }
    ]
}

However with the deployment settings I am passing on with the request there should definitely be a node viable. So I went to investigate.
I figured out the error was thrown from the FindViableNodesService class. I then changed a couple things so the if statement in the for loop in the handle method looked like this:
if (($node->sum_memory + $this->memory) > $memoryLimit || ($node->sum_disk + $this->disk) > $diskLimit) {
    throw new NoViableNodeException($node->sum_memory + $this->memory . ' | ' . $memoryLimit . ' | ' . $node->memory);
    continue;
}

But then the result was:
{
    "errors": [
        {
            "code": "NoViableNodeException",
            "status": "400",
            "detail": "256 | 0 | "
        }
    ]
}

Which made me curious, so I went ahead and put this above the original if statement.
if (is_null($node->memory)) {
  throw new NoViableNodeException('test');
}

Result:
{
    "errors": [
        {
            "code": "NoViableNodeException",
            "status": "400",
            "detail": "test"
        }
    ]
}

I did one more test and found out that $node->disk is also null.
Is this a bug, or am I missing something?

What are the allocation limits on that node for disk and memory?

These are my allocation limits:

The request I was sending:
Endpoint: /api/application/servers
Method: POST

{
	"name":"testwithapi",
	"user": 1,
	"egg": 1,
	"docker_image": "quay.io/pterodactyl/core:java",
	"startup": "java -Xms128M -Xmx{{SERVER_MEMORY}}M -jar {{SERVER_JARFILE}}",
	"environment": {
		"SERVER_JARFILE": "server.jar",
		"VANILLA_VERSION": "latest"
	},
	"limits": {
		"memory": 256,
		"swap": 0,
		"disk": 256,
		"io": 500,
		"cpu": 400
	},
	"deploy": {
		"locations":[1],
		"dedicated_ip": false,
		"port_range": ["25565"]
	}
}

Is there an open port 25565 on any node in location 1?

Yes, there is.
Also if there wouldn't be one it makes no sense for those to be null.

I cannot reproduce this at all, and it makes no sense how it could be getting null back unless there is some type of data-integrity issue in the database.
Can you manually run this command for me and let me know what it outputs?
select `nodes`.`id`, `nodes`.`memory`, `nodes`.`disk`, `nodes`.`memory_overallocate`, `nodes`.`disk_overallocate`, IFNULL(SUM(servers.memory), 0) as sum_memory, IFNULL(SUM(servers.disk), 0) as sum_disk from `nodes` inner join `servers` on `servers`.`node_id` = `nodes`.`id` where `nodes`.`public` = 1 and `nodes`.`location_id` in (1)```

+----+--------+------+---------------------+-------------------+------------+----------+
| id | memory | disk | memory_overallocate | disk_overallocate | sum_memory | sum_disk |
+----+--------+------+---------------------+-------------------+------------+----------+
| NULL |   NULL | NULL |                NULL |              NULL | 0          | 0        |
+----+--------+------+---------------------+-------------------+------------+----------+
1 row in set (0.00 sec)

This probably explains why the fields are null, but I don't get why it would properly display on the webpage.

Can you show your nodes and locations table for me (stripping out sensitive info as needed).

Nodes:
https://hastebin.com/ufitiqudim.sm
Locations:
MariaDB [ptero]> select * from locations;
+----+-------+------+---------------------+---------------------+
| id | short | long | created_at          | updated_at          |
+----+-------+------+---------------------+---------------------+
|  1 | Test  | Test | 2017-12-06 19:20:53 | 2017-12-06 19:20:53 |
+----+-------+------+---------------------+---------------------+
1 row in set (0.00 sec)

And are there any servers on this node?

No, not currently.
Here you can see the allocations and that there are no servers assigned to them.


Configure fails for Mac using gcc 4.6

Hi,
I tried to build CTK on Mac OS X Lion using gcc 4.6. I get the following configure error:
CMake Error at CMakeLists.txt:631 (message):
Failed to compile DGraph application.
Change Dir: /Users/Andi/Desktop/Arbeit/ctk_bin/Utilities/DGraph
Run Build Command:/usr/bin/make
[ 50%] Building CXX object CMakeFiles/DGraph.dir/DGraph.cpp.o
g++-mp-4.6: error: x86_64: No such file or directory
g++-mp-4.6: error: unrecognized option '-arch'
make[2]: *** [CMakeFiles/DGraph.dir/DGraph.cpp.o] Error 1
make[1]: *** [CMakeFiles/DGraph.dir/all] Error 2
make: *** [all] Error 2
Any ideas?
Thanks
Andreas

What CMake are you using ? can you try with a recent cmake ? (2.8.7)

Ok I tried CMake 2.8.7 but the error remains the same.
On 08.02.2012, at 17:32, Julien Finet wrote:

What CMake are you using ? can you try with a recent cmake ? (2.8.7)

Reply to this email directly or view it on GitHub:
https://github.com/commontk/CTK/issues/137#issuecomment-3870558

I finally got a solution. I just removed all entries where -DCMAKE_OSX_ARCHITECTURES was set (i.e. in CMakeLists.txt, CMake/ctkBlockSetCMakeOSXVariables.cmake and CMake/ctkBlockCheckDependencies.cmake). Now the build was successful.


Update new image registry path

What type of PR is this?
/kind failing-ci
What does this PR do / why we need it:
To fix CI failure, Updating new test container registry path
Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes - CI failure due to deprecated test container registry
How to test changes / Special notes to the reviewer:
CI should pass

CI will still fail because openshift CI job config still uses the master branch docker file which has the deprecated image registry path. So merging it forcefully


Mirror order conditions as False, Unknown, True

This is a breaking change on Conditions.Mirror(). Originally this call ordered conditions as False, True, Unknown then picked the first of that list to reflect. However all the use cases (we see) assume that if there is reasourceA that depends on the status of resourceB then resourceA reconciler calls:
  resourceB.Status.Conditions.Mirror(ResourceBSpecificSubConditionOfA)

and expects that the call returns the most sever non True condition if any or return a True condition otherwise. So the original behavior that returned True even if there was Unknown condition was unexpected.
This patch changes the order of the status to False, Unknown, True. But keeps the original logic that if the ReadyCondition is True then the rest of the conditions are ignored even if there are False conditions in the list.

/hold
I want to see that the behavior change I noticed in nova-operator after this lib-common change are OK https://github.com/openstack-k8s-operators/nova-operator/pull/283

/unhold
https://github.com/openstack-k8s-operators/nova-operator/pull/283 now shows that nova-operator actually have a bug before this fix. And after this fix the test needs to be adapted as it made wrong assumptions.


Add a reference to Legacy Applications in the Xresources configuration file

I had gotten a bit confused about how to use an Xresources configuration file to set the value of the so-called "Legacy Applications" field in gnome-tweaks. This commit hopefully clarifies the situation.

Looks good, thanks @adamroyjones !


python3Packages.av: fix build

Description of changes
Many of the tests depend on sample files fetched over HTTP and they keep adding more. Since they support a caching mechanism, use it to pre‐fetch all the used samples instead.
It might be nice to have a package for the full FATE suite, but it’s over 1 GiB in size so I just did this for now.
Fixes: ca258d2d393eb240fd4eddf9a40093e422c33116
Going to run a full nixpkgs-review on a master‐based version of this, but it’s probably fine to merge optimistically.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] 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.

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

We might want to reconsider running these tests at all if they require such maintenance overhead or imply huge build time closures.

In this current PR it’s like 16 MiB of samples and one bash script to keep them updated in case they add more; I think it’s fine. The overhead was manually disabling every test that wants a sample file, which should no longer be necessary. There’s upstream issues at https://github.com/PyAV-Org/PyAV/issues/623 and https://github.com/PyAV-Org/PyAV/issues/955 (hi @mweinelt).
At some point it might be nice to run the full FATE test suite against our FFmpeg builds, really, at which point we’d have to start thinking about size.

This breaks Home Assistant’s weird PyAV fork thing… I’ll look into it tomorrow.

cc @mweinelt

I just talked with @mweinelt on Matrix and we’re going to try replacing Home Assistant’s outdated PyAV fork with the real thing. Will report back on the nixpkgs-review results when I have them.

Alright, this has no regressions vs. Hydra on master (except whisper-ctranslate2 which doesn’t look related so it’s probably an issue with my regression testing methodology?). Should be good to merge.


Add TestRunResultProcessor for JUnit-style XML output

Smack will be configurable with optional TestRunResultProcessor instances, which will operate on the outcome of the executed tests. This commit adds a processor that generates an XML file that mimics the JUnit-style reporting.
When Smack is configured to use the new processor introduced by this commit (org.igniterealtime.smack.inttest.util.JUnitXmlTestRunResultProcessor) then a file naed test-results.xml will be created in the directory identified by the system property logDir.

For brownie points, I added a step in the workflow of this repo that processes the JUnit-style XML output, and generates UI elements from that (there's an additional 'check' in this PR, and the summary of the build shows a table, as shown below).
I'd love to be able to group the test outcome per XEP, but I'm not sure if that's possible with this particular tool. I've asked the author in https://github.com/mikepenz/action-junit-report/issues/1080

The regex change to the job has changed the reporting from

MoodIntegrationTest.testNotificationAfterFilterChange (Normal)

to

MoodIntegrationTest.XEP-0107: User Mood/MoodIntegrationTest.testNotificationAfterFilterChange (Normal)

Which is more information, but unsure if it's actually nicer.

Merged. We'll change the reporting separately, if we're able.


Document static authority.

Signed-off-by: Ville Aikas<EMAIL_ADDRESS>
Summary
Document the static authority added in https://github.com/sigstore/policy-controller/pull/69
Fixes: #188
Also, consolidate the description of Authorities to a single block rather than redefining in both key and keyless, and
now in newly added static sections.
Release Note

Document the static authority added in https://github.com/sigstore/policy-controller/pull/69

Documentation

@ltagliaferri @hectorj2f would you mind taking a look?


Preserve the <WorkSpace>/apis/ folder between syft versions

Description
Closes OpenMined/syft-internal-issues#221
Affected Dependencies
List any dependencies that are required for this change.
How has this been tested?

Describe the tests that you ran to verify your changes.
Provide instructions so we can reproduce.
List any relevant details for your test configuration.

Checklist

[ ] I have followed the Contribution Guidelines and Code of Conduct
[ ] I have commented my code following the OpenMined Styleguide
[ ] I have labeled this PR with the relevant Type labels
[ ] My changes are covered by tests

instead of doing this can we just move the apis dir somewhere, and then bring it back?


pythonTerminalEnvVarActivation should reproduce python-dotenv comments behaviour

Hey, one thing I've noticed is that inline comments, by which i mean something like:
key=value # this is an inline comment

aren't supported by pythonTerminalEnvVarActivation. That's understandable as I'm not sure it's generally supported by the dotenv format, however the python-dotenv library does support these types of comments.
For me this was really confusing as I automatically got opted into the the experiment without my knowledge when changing to a new machine and then suddenly my tests were no longer working and it wasn't clear why.
Suddenly, my env values were being piped through with the comments attached even though on a different machine with the same version of python-dotenv (1.0.0) they were coming through as expected. The reason was of course that the values were now coming from vscode when the terminal session was started, and python-dotenv doesn't override pre-existing env var values by default.
I think this could also catch other people out so thought it was worth floating the idea that these types of comments are supported.
Version info:
vscode-python: v2024.3.10611008 (pre-release)
Version: 1.87.0 (Universal)
Commit: 019f4d1419fbc8219a181fab7892ebccf7ee29a2
Date: 2024-02-27T23:42:56.944Z (2 days ago)
Electron: 27.3.2
ElectronBuildId: 26836302
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 23.3.0

Closing in favor of https://github.com/microsoft/vscode-python/issues/18307.


fix: Remove Future type hint from return type which fails to type check when overridden

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #132 🦕

Thank you! This will fix #127, #128, #129, #130, #131 after the next release too.


Managed VM platform

Root issue for constructing a managed VM platform support:

[ ] service model compatibility for managed VMs
[ ] service registry and event handling for VMs
[ ] docker-based sidecar for VMs
[ ] traffic capture for VMs
[ ] discovery services for VMs

This is a duplicate of #333 . The solution is pretty much the same. And finally, lets not pollute the VMs with docker unless absolutely necessary.

No, this is specifically for GCE environment. I mentioned docker to ease deployment (dealing with DEBs and RPM is too much for beta)

Dude, VMs is the same whether it's GCE or AWS or Bluemix softlayer.
On Fri, May 19, 2017 at 7:31 PM Kuat<EMAIL_ADDRESS>wrote:

No, this is specifically for GCE environment. I mentioned docker to ease
deployment (dealing with DEBs and RPM is too much for beta)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/istio/manager/issues/797#issuecomment-302832948, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AH0qd0gNDrs-N6Hrd1fWzf3qL5bXn0HHks5r7iZAgaJpZM4NhGsF
.

--
~shriram


[SPARK-10121] [SQL] [WIP] When add jar, also explicitly set the classloader of metadataHive's HiveConf inside metadataHive's state.

https://issues.apache.org/jira/browse/SPARK-10121
This will make sure the HiveConf inside metadataHive's state actually loads the class. Otherwise, if we add jar in one thread and the query using the custom class in another thread, we may see ClassNotFoundException.

Merged build triggered.

Merged build started.

Test build #41385 has started for   PR 8364 at commit 4aecbba.

I am closing it. The fix is not right.

Test build #41385 has finished for   PR 8364 at commit 4aecbba.

This patch fails Spark unit 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/41385/
Test FAILed.


Fix issue where arraycolumns can't be saved with jsonlines

Closes issue raised in #213.
Simplified to_jsonl to simply wrap to_pandas, which addressed the issue.

Codecov Report

Merging #214 (03092c7) into dev (a72cdae) will increase coverage by 0.06%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev     #214      +/-   ##
==========================================
+ Coverage   86.21%   86.28%   +0.06%
==========================================
  Files          47       47
  Lines        3134     3127       -7
  Branches      615      612       -3
==========================================
- Hits         2702     2698       -4
+ Misses        317      314       -3
  Partials      115      115

Flag
Coverage Δ

unittests
86.28% <100.00%> (+0.06%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

meerkat/columns/tensor_column.py
84.61% <100.00%> (+0.54%)
:arrow_up:

meerkat/datapanel.py
93.43% <100.00%> (+0.68%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a72cdae...03092c7. Read the comment docs.


Language and code example around the "thread-staticness" of Thread.CurrentPrincipal is misleading

Issue description
In multiple places, the page refers to Thread.CurrentPrincipal as being "thread-static" and says that it will not work in async scenarios. There is even an included code example intended to demonstrate this:
// Create a ClaimsPrincipal and set Thread.CurrentPrincipal
var identity = new ClaimsIdentity();
identity.AddClaim(new Claim(ClaimTypes.Name, "User1"));
Thread.CurrentPrincipal = new ClaimsPrincipal(identity);

// Check the current user
Console.WriteLine($"Current user: {Thread.CurrentPrincipal?.Identity.Name}");

// For the method to complete asynchronously
await Task.Yield();

// Check the current user after
Console.WriteLine($"Current user: {Thread.CurrentPrincipal?.Identity.Name}");

However, Thread.CurrentPrincipal is not, in fact, thread-static. It behaves more like AsyncLocal and flows nicely across awaits. Simply running the code snippet demonstrating the problem shows that it works fine on both .NET Framework and .NET Core (at least in the scenarios I tried) :-)
We were migrating to ASP.NET Core with a large codebase that used Thread.CurrentPrincipal pretty heavily. All we had to do to get things working was set this in a middleware upfront and it flows nicely just like it did before.

Can you create an issue from the doc page? That will connect the doc to the issue. Go to the bottom of the page and select This page .


Datetime

Is there is any updates in Datetime component.?!
May i know the stable version which supports select,upload, number and datetime.

This is now resolved with beta.1 release!  We are now using the flatpickr module which is much easier to deal with.


⚠ Minecraft has degraded performance

In cfc2e43, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1096 ms

Resolved: Minecraft performance has improved in 73b503e.


Suggest splitting CamelCase into 2 functions

It can make a big difference in usability to avoid using booleans in function signatures, since it's impossible to tell what true or false mean when you're just reading code.
I suggest using CamelCase and PascalCase as function names (where PascalCase is the  first upper and camel is first lower).
I'd also like to have any functions that need to use the intiialisms list to have an alternate form that takes a customized list.
I'm happy to make PRs for any of this.

I can't recall the exact use case; The reason for the bool was something like "it's easier to swap a bool than to swap an entire function".  However, this was years ago and I can't attest to the quality of my rationale (especially because I was less functional-minded).  I'd be happy to implement your suggestion, and I greatly appreciate the PR offer.

Cool, yeah, it also makes it easier to use in templates (which happens to be my use case)

Closed by #4


Upgrade to React Native 0.57.5

Fixes #115 and probably a bunch more. I also updated all of the CLI-level dependencies.

Investigating the Semaphore failure now. Tests passed on my machine.

Tests pass. Will need to address CI failures later.


Added ripper for Myreadingmanga.info

Category
This change is exactly one of the following (please change [ ] to [x]) to indicate which:

[ ] a bug fix (Fix #...)
[X] a new Ripper
[ ] a refactoring
[ ] a style change/fix
[ ] a new feature

Description
A ripper and unit test for Myreadingmanga.info
Testing
Required verification:

[ ] I've verified that there are no regressions in mvn test (there are no new failures or errors).
[ ] I've verified that this change works as intended.

[ ] Downloads all relevant content.
[ ] Downloads content from multiple pages (as necessary or appropriate).
[ ] Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.

[ ] I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

[ ] I've added a unit test to cover my change.

Coverage increased (+0.07%) to 42.238% when pulling 543cd09d4f5a846a8f40d54318d6ec646f8bab55 on cyian-1756:myreadingmanga into 0d01c10bc713646cf8eb9d302168cae068df304d on RipMeApp:master.


Full delve item forum export support

Closes #992
Add's full support for fossils and resonators.
I refactor the visitor generation code while I was there, there was alot of boilerplate, should be even simpler to generate Visitors from now on.

@nqrcqn Please test this, should allow you to sell fossils and resonators.

Yes, fixes fossils

Because this PR updates the hover for fossils to fix Scarabs and Splinters (and other items in the future) I'm defaulting the item hover header to the currency one.
Closes #998 too.
@nqrcqn Can you check it with the latest artifact from my latest commit. Thanks.

Can't comment on the defaulting but Scarabs have the same white/gray background as Offering/Divine Vessels, so I think those should be Rarity.Normal at least

I just realized splinters aren't added to forum export at all, so that explains having no currency background as well.

Bumping this PR for a merge please @Stickymaddness

I'm testing the latest build, one of the recent merges broke tab refreshing. Tabs do not refresh unless I close and restart Procurement.

@nqrcqn If you could release a detailed bug report in a new that would be amazing. 👍

The only detail is clicking Refresh All/Used Tabs does not refresh the tabs anymore.


🐛 [Bug report]: Add MaterialUI icon to TechStack list

Descrcibe the bug?
Due to some issue, MaterialUi icon has been removed from techstack. The idea of this issue to bring it back with appropriate icon
Steps to reproduce the bug?
NA
Expected behavior
ReactPlay uses MaterialUI so it should be listed under techstack
Desktop (Please provide your system information)
NA
Mobile (Please provide your device information)
No response
Screenshot / Screenshare
No response
Relevant log output
No response
Code of Conduct

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

It means you want to add material ui icon in this list of tech stacks. Right?

I can take up this


Insufficient debug info when Warning: The following definitions had errors while loading.

I've been trying to update an older project to work with newer codebase. During the install, I got the following warning/error:
The following definitions had errors while loading.  Press Proceed to try to load them again (they may work on a second pass):
[list of classes with errors]

That is to be expected, but the issue is that this warning/error does not (as far as I can tell) provide any information about the individual errors in the given definitions, which is very unhelpful for someone trying to fix those errors.
There is a variable called errorDefinitions which contains the definitions mentioned in the message above, but no trace of the exact errors.

Yes we know and we faced this (suboptimal) design. It is bad. This behavior is a legacy.
Can you tell us on which version of Pharo you are experiencing it?

Can you also provide a little case showing your problem so that we can use it to see how to fix it.


Laravel search only specific word

Hi ,
I have a database with arabic records now i want to search exact word like Words that include
شَهْر
But excluding
شَهْرَيْنِ
Words that include
سَنَة
And
سِنِين
Laravel approach
 Verse::where('text','like','%'.$string.'%')->get()
Your Package
Verse::searchAr('text',$string)->get();
your package also giving same result , i mean search for all similar words which contain شَهْر but i want to search where word is only شَهْر
any soultion or suggestion

"select * from verses where ( REPLACE(ذَٰلِكَ, 'ـ', '') REGEXP 'text')" here is the sql result of my query

Actually i want to search specific words in Quran i used many tricks and regular expressions its give me result but i want more result in depth like get the word سَنَةٌ to be more generic by removing the last vowel ٌ to become سَنَة so it will match all versions like سَنَةٍ etc
And then make it more generic to get السَنَة including بالسَنَة

$results=\App\Models\Verse::with('surah')->where('text',$string)
        ->orwhere('text','REGEXP','(^|[[:space:]])'.$string.'[[:space:]]?')
        ->orwhere('text','LIKE','[[:<:]]'.$string.'[[:>:]]')
        ->orwhere('text','REGEXP','(^|[[:space:]])'.$string.'([[:space:]]|$)')->get();

this query gives me 80% of the results but still 20% is remain

Oops! It looks like I did not mention it here :-(
The stripHarakat method in version 6.2 of the Ar-PHP library (released on 20 Jun 2021) can remove the last Harakat alone:

Change log note
Method documentation
Live example


Fixing Composer

@rochamarcelo Hey man I don't understand composer too well, what does this do? Will this fix #22

With my new pull request It should work, packagist does not allow uppercase characters.
You must submit the package to packagist.org.

This pull request is also needed to solve issue #22


HttpAsyncHook: cannot do an await response.text() or response.json() after run: "Connection closed"

Describe the bug
After the change in version 1.15.2 of astronomer-providers, it seems the HttpAsyncHook closes the connection after returning from run(...). This means the body of the ClientResponse object can no longer be read, since it will raise a "Connection closed" error. Immediately before this fix, i.e. in 1.15.2, one could do a response.text() or response.json().
To Reproduce
Steps to reproduce the behavior:

Create an HttpAsyncHook with a "GET" method: hook = HttpHookAsync(method="GET", ...)
Do an await on its run method: response = await hook.run(...)
Try to read the body of the response, also awaiting: payload = await response.json()
Error: aiohttp.client_exceptions.ClientConnectionError: Connection closed

Expected behavior
The calling code should be able to read the response body.
Additional context
I'm running this code inside an Airflow Trigger.

Additional information: I believe the commit that changed the behavior is in this PR

Hi @andresferrari , could you please provide the code that led to this failure? I'm not able to reproduce the same behavior.
I changed the code in https://github.com/astronomer/astronomer-providers/blob/1.17.3/astronomer/providers/http/triggers/http.py#L65-L76 as the following to simulate the situation you mentioned.
        while True:
            try:
                response = await hook.run(
                    endpoint=self.endpoint,
                    data=self.data,
                    headers=self.headers,
                    extra_options=self.extra_options,
                )
                break
            except AirflowException as exc:
                if str(exc).startswith("404"):
                    await asyncio.sleep(self.poke_interval)

        print(response)
        payload = await response.json()
        print(payload)
        yield TriggerEvent(True)

Also commented out the code here so that we can be sure that we're running in the triggerer.

The DAG I'm using to test this is https://github.com/astronomer/astronomer-providers/blob/1.17.3/astronomer/providers/http/example_dags/example_http.py and set the HTTP hook host to https://www.google.com/

Or is there anything I misunderstood? Thanks!

@andresferrari Thanks for your prompt response. Let me check whether I can reproduce and see how I can fix it. Thanks!

I tried to add a keep_response argument in HttpHookAsync in this https://github.com/astronomer/astronomer-providers/pull/1330 for address this


Clashes with live_view 0.17.x

Describe the bug
When trying to update a project to live_view 0.17.2 there is a clash with slot/2 when using Surface
function slot/2 imported from both Surface.API and Phoenix.LiveView.Helpers, call is ambiguous
How to reproduce it
Upgarde live_view to 0.17.x
The behavior you expected
Should still compile
Your Environment
Surface: v0.6.0
LiveView: v0.17.2
Elixir: v1.12.2

Fixed in #546.


Locate and get access to the best API that we can pull active traffic data from

We want this to be as granular as possible. Best case scenario is that we can pull data on a street by street basis. Google maps API comes to mind, but I'm not sure if Google exposes that feature.
If there is no data that can provide a reasonably granular representation of active traffic, another approach may be to draw data from multiple APIs.

Best candidate I've found so far is Here's API. Here's a sample response and a stack overflow post describing the symbols.
It includes live road information on

Current speed of traffic
Active "Free flow" speed (Not sure what the difference is between this and traffic speed)
A "Jam Factor" rating, between 0.0 to 10.0 (10 being the road is closed) indicating the quality of travel.

It's a good start, but still missing some necessary data to realistically approximate amount of cars on the road. For example, a "Jam Factor" rating of 9 can have wildly different meanings in terms of emissions if the data is describing an inter-city road vs an 8 lane LA freeway.
Perhaps an API or dataset that tells us the capacity for traffic for a given road would be of help, but I've yet to find one.

We'll be using the above AI along with OpenStreetMap's Overpass API.

Re-opening this. It looks like the Here API has a few bugs on gathering road data. It seems the bounding box and proximity resources are a bit off. Some streets inside the bounding box/proximity are missing, while many streets nearby the bounding box are included in the JSON results.
In addition, Here may have trouble cooperating with OpenStreetMap's Overpass API. The Overpass API breaks roads up into edges between nodes, while Here doesn't. This means traffic data over a road that stretches many miles may be completely off.

We will be using the HERE API after all. @fjonass 's research shows that the bounding box is merely a suggestion of the data you want, and HERE will return the "most important roads" nearby. In addition, we are able to get road shape information afterall thanks to @Tattomoosa finding this endpoint https://developer.here.com/documentation/traffic/dev_guide/topics_v6.1/example-flow-sh-frc.html
This endpoint is still missing two key pieces of information: road type (highway, residential, etc.) and lane count. Both of these are very helpful for modeling emission data. Fortunately, Overpass API does give us this information. Unfortunately, it doesn't seem there's any simple method to get a 1-1 correspondance between road information provided by HERE and road information provided by Overpass.
The best we can do at the moment is to compare the road name strings, but even those aren't 1-1 since names may be provided by differing sources. For example, HERE may claim a road's name is "SE Lolo Pass Rd" while Overpass may claim it's "se lolo pass rd." (Notice differing capitalization and punctuation). The best option here would be to use some sort of "auto-correct" string algorithm to determine if the two differing names are "close enough".

I've done the string comparison you're talking about, using levenshtein distance.


Codebase improvement: Updated Imports for New JSX Transform in React

🔧 Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[x] Codebase improvement
[ ] Other (if none of the other choices apply)

🚨 Checklist

[x] I've read the guidelines for contributing
[x] I've added necessary documentation (if appropriate)
[x] I've ensured that my code additions do not fail linting or unit tests (if applicable)

Description
This pull request implements updates to the imports in the codebase, leveraging the new JSX transform in React as outlined in the official React blog.
By adopting the new JSX transform, we can benefit from improved performance and better compatibility with future React versions.

I merged it but workflows and package.json should be updated too, missed that


eds-core-react is not tree-shaking as expected

Test shows that a single import from eds-core-react imports the whole library. Should use tree-shaking to avoid this.

Build results after only importing Button:

Did some more testing here, and its due to typescript importing core-react using the commonjs bundle (which is what typescript operates with internally). commonjs modules as we know are not tree-shakeable....
Other libraries have solved this by providing named explicit named paths for components. Such as Material-UI

Closing this issue as we have a seperate issue for named paths #1043

Closing this issue as we have a seperate issue for named paths #1043


[Improvement][Authentication] Improvement k8s resource center statement

Purpose of the pull request
fix: Improvement
Brief change log
Due to [DSIP-42] Add dolphinscheduler-aws-authentication module, resource center statement is outdated.
will directly link to related to avoid inconsistency.
Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Please create an issue and link to this PR, and correct the title.

It was related to
https://github.com/apache/dolphinscheduler/issues/16122,
for somehow reason it can not be tagged.


Fixed CI state issue

Signed-off-by: aisuko<EMAIL_ADDRESS>Description
We got the CI issue with go statticcheck, but when I was check it at local, it doesn't report any error. So, I'd like to update the CI process to follow the latest official Github Action example and to solve this.https://staticcheck.io/docs/running-staticcheck/github-actions/

This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

I need to fix the new error were checked by the staticcheck Github action on our code. As we can see the staticcheck logic is working well. https://github.com/meshery/meshery-linkerd/runs/4451238855?check_suite_focus=true

Can you still make the changes asked in static check , so that all tests pass before we merge?
If it works fine, we can reuse the same github action across all adapters

Can you still make the changes asked in static check , so that all tests pass before we merge?
If it works fine, we can reuse the same github action across all adapters

No problems. I will fix the error later.

Please help to take a look guys and after we merge this one. The confused on other PR can be solved.


Implement drag and drog to move files using react-dnd

fixes #145

Use react-dnd library to add support to drag and drop on FilesTree

Hi @shashigharti can you please test and review?

Currently if you install react-dnd you will have the error:
Compiled with problems:
ERROR in ./node_modules/react-dnd/dist/core/DndProvider.js 28:0-48

Module not found: Error: Can't resolve 'react/jsx-runtime' in '/home/aivuk/frictionless/application/node_modules/react-dnd/dist/core'
Did you mean 'jsx-runtime.js'?
BREAKING CHANGE: The request 'react/jsx-runtime' failed to resolve only because it was resolved as fully specified
(probably because the origin is strict EcmaScript Module, e. g. a module with javascript mimetype, a '*.mjs' file, or a '*.js' file where the package.json contains '"type": "module"').
The extension in the request is mandatory for it to be fully specified.
Add the extension to the request.

I did need to change the line 28 of node_modules/react-dnd/dist/core/DndProvider.js from:
import { jsx as _jsx } from "react/jsx-runtime";
to
import { jsx as _jsx } from "react/jsx-runtime.js";

Currently if you install react-dnd you will have the error:

Looks like this is a well know problem https://github.com/react-dnd/react-dnd/issues?q=is%3Aissue++react%2Fjsx-runtime and there are some suggestions to solve it as using an older version of the react-dnd library or using React 18. I tried to migrate  the current React v17 from the application, but some things broke.


Add baremetal-operator in required project & sort

Adding baremetal-operator in the required project
and sort the list.

[ ] Appropriate testing is done and actually running
[ ] Appropriate documentation (README in the role, main README is up-to-date)

recheck

/approve
re-ordering the operators will make finding any missing ones easier - thanks for that extra step, @Sandeepyadav93 !


TCA crash : Release build with micro-frameworks app

Description
We are currently using TCA along with Tuist in our application. In particular, a bit a la isowords we have split our app in multiple frameworks. However, in release builds (debug works fine), TCA crashes when some reducers are in different frameworks. This is the graph of how we have set up the app to reproduce the bug.

Thanks in advance for the help 🙌
Checklist

[X] I have determined whether this bug is also reproducible in a vanilla SwiftUI project.
[X] If possible, I've reproduced the issue using the main branch of this package.
[X] This issue hasn't been addressed in an existing GitHub issue or discussion.

Expected behavior
App shouldn't crash in Release mode
Actual behavior
App crashes in release mode. See attached video:
https://user-images.githubusercontent.com/26744253/202856365-75cd0ba7-b3f4-48f8-b590-0c2d86b49b7c.mp4
Steps to reproduce
For simplicity and clarity, I have created a minimal project to reproduce the bug : https://github.com/PierreCapo/tca-frameworks-bug.
You can follow the video attached to exactly reproduce the crash.
The Composable Architecture version information
0.45.0
Destination operating system
iOS 14 +
Xcode version information
14.0
Swift Compiler version information
swift-driver version: 1.62.8 Apple Swift version 5.7 (swiftlang-<IP_ADDRESS>.4 clang-14<IP_ADDRESS>)
Target: arm64-apple-macosx13.0

Hi @PierreCapo, it is not clear that this is an issue with the library. The project works fine when putting all the files into a single target. So, I am going to move it to a discussion and maybe someone who has experience with Tuist can help out.
I would also suggest asking on Tuist's discussions. It could be an issue in Tuist, Xcode or maybe even Swift.


Update cri and docker vendors

Update cri and docker to recent commits for mips enablement
Fixes: #3839
Signed-off-by: Phil Estes<EMAIL_ADDRESS>

Codecov Report

Merging #3852 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3852      +/-   ##
==========================================
- Coverage   42.35%   42.34%   -0.02%
==========================================
  Files         130      130
  Lines       14683    14683
==========================================
- Hits         6219     6217       -2
- Misses       7539     7540       +1
- Partials      925      926       +1

Flag
Coverage Δ

#linux
45.76% <ø> (-0.02%)
:arrow_down:

#windows
37.82% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

snapshots/btrfs/btrfs.go
57.39% <0%> (-0.9%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 45fb5ae...252d655. Read the comment docs.


♻ Add static CSS class names

Thank you for contributing to Homarr! So that your Pull Request can be handled effectively, please populate the following fields (delete sections that are not applicable)
Category
Refactoring only
Overview

Added static class names for easier customization

Requested by @blassley

Does this work when you build ? I hope the css clasnames in inline styles are not minimized / optimized

I did not, because we already have other CSS names with are used on runtime - they wouldn't work too if they were "optimized". Also, simply renaming names won't give you any performance improvement. AFAIK Mantine only uses them because they are not specific elements - hence creates random unique hashes for component.

I did not, because we already have other CSS names with are used on runtime - they wouldn't work too if they were "optimized". Also, simply renaming names won't give you any performance improvement. AFAIK Mantine only uses them because they are not specific elements - hence creates random unique hashes for component.

I thought the JavaScript bundler thingy would rename everything to symbols so it takes less characters in the resulting JavaScript

Nope, works as expected (check preview deployment above)

Nope, works as expected (check preview deployment above)

Yeah I understand it works I was just curious about optimization


Refactor testcases expandable

Description

Migrates test cases for Expandable component from Enzyme to react testing library

Which issue(s) does this PR relate to?

Closes #861

Testing

npm run test

Trade-offs

N/A

Screenshots

N/A

Checklist

[ ] If any new components were added, there are exported from packages/index.ts
[x] This PR is associated with a JIRA and mentions in the commit message footer ("Closes …")
[ ] This PR contains breaking changes and states in the commit message body ("BREAKING CHANGE: …")
[x] I have reviewed the changes and provided detail to the sections above

@nataliepina @vacas5 have updated according to review comments. Please check


the server doesn't show info about the client

On this gif ( https://raw.githubusercontent.com/constverum/ProxyBroker/master/docs/source/_static/cli_serve_example.gif ) the server prints an info line when a client connects.
The current version doesn't do that, though it would be very useful. I tried the command that is on the GIF.

It would be nice if a range of error log levels were produced, currently the only meaningful output you get is when you use debug logging.
proxybroker --log DEBUG serve --host <IP_ADDRESS> --port 8888 --types HTTP HTTPS  --lvl High