No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:9005' is therefore not allowed access

I am using aws-serverless-express module for using lambda function with express js.
const server = awsServerlessExpress.createServer(app, null, binaryMimeTypes)
exports.handler = function (event, context, callback) {
awsServerlessExpress.proxy(server, event, context);
}
After that i have create AWS api gateway API with resource and method POST.
when I login , getting error- No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:9005' is therefore not allowed access. The response had HTTP status code 403.

Try specifying * in the swagger instead of example.com
On Thu, Aug 17, 2017, 2:40 AM webistan<EMAIL_ADDRESS>wrote:

I am using aws-serverless-express module for using lambda function with
express js.
const server = awsServerlessExpress.createServer(app, null,
binaryMimeTypes)
exports.handler = function (event, context, callback) {
awsServerlessExpress.proxy(server, event, context);
}
After that i have create AWS api gateway API with resource and method POST.
when I login , getting error- No 'Access-Control-Allow-Origin' header is
present on the requested resource. Origin 'http://localhost:9005' is
therefore not allowed access. The response had HTTP status code 403.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/awslabs/aws-serverless-express/issues/90, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABy6l-I_3TIVVhB50fpUkQPrT3gMkQ5Kks5sZAptgaJpZM4O6BC5
.

where do I excactly find the swagger file ?

This was assuming you were using the example - swagger can be found here https://github.com/awslabs/aws-serverless-express/blob/master/example/simple-proxy-api.yaml. You can either configure OPTIONS at the API Gateway layer, or configure it at the Lambda layer (simpler) by using the cors middleware, see https://github.com/awslabs/aws-serverless-express/blob/master/example/app.js#L4.

I'm actually not using the example, so I got confused a little, the lambda layer would be more appropriate in my case, but I'm using the cors middleware in the app.js and still getting the same error ...

Verify your OPTIONS is going through to Lambda and is returning that
allowed origin.
On Thu, Aug 17, 2017, 12:33 PM sami karakoese<EMAIL_ADDRESS>wrote:

I'm actually not using the example, so I got confused a little, the lambda
layer would be more appropriate in my case, but I'm using the cors
middleware in the app.js and still getting the same error ...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/awslabs/aws-serverless-express/issues/90#issuecomment-323171991,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABy6l1SPnypsJAjMlLxSQihi9OOsViFQks5sZJWAgaJpZM4O6BC5
.

I can actually catch the OPTIONS request and I'm setting "Access-Control-Allow-Origin" : "*", but the It seems like the next incomming POST request does give me the 502 error.
here is my console log:

POST https://[awsdomain]/dev/[route]/ 502 ()
confirm:1 XMLHttpRequest cannot load https://[awsdomain]/dev/[route]/. No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'https://[unkown aws domain]' is therefore not allowed access. The response had HTTP status code 50

I am actually communication with another lambda express backend on the POST request, but this other backend has not the same domain like the unkown one ... Does lambda itself communicate with any other server under any circumstance ? I hope these are just some missconfigurations of my express app

Does lambda itself communicate with any other server under any circumstance

No.

I hope these are just some missconfigurations of my express app

This seems to be the case. It just occurred to me that "No 'Access-Control-Allow-Origin'" error is sometimes a red herring when an error occurred before API Gateway could handle it, and the error response does not include CORS configuration. It sounds like your second lambda express backend has some error.

when hit POST request by POSTMAN so working fine but in my angularjs code POST request convert to OPTIONS request in browser so it can't be reach server geetting same error 500 mention above.
how can we solve this issue.
$http({
url: API,
method: "POST",
data:  {"name":"abc"},
dataType: 'json',
async: false,
headers: {"content-type": "application/json"}
}).success(function(data, status) {
console.log(data);
})
.error(function (error){
console.log(error);
});

can we create multiple function with proxy  AWS API GATEWAY when using aws-serverless-express in AWS lambda function?

Yes, you just need to configure additional Resources on your API
On Wed, Aug 23, 2017, 6:11 AM webistan<EMAIL_ADDRESS>wrote:

can we create multiple function with proxy AWS API GATEWAY when using
aws-serverless-express in AWS lambda function?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/awslabs/aws-serverless-express/issues/90#issuecomment-324325892,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABy6l8naTquUm5quQU4Odv3Oeiv0Pacrks5sbCUbgaJpZM4O6BC5
.

only proxy resource work fine if we create custom resource with method and used than getting error:
missing authentication.
How is it possible for custom resource used in aws-serverless-express in lambda function?

solved issue by create resources with new proxy resources.
like this:
/login
/{proxy}
/dashboard
/{proxy}
This mechanism solved issue for allowed to different resources communicate different lambda function.

Access to XMLHttpRequest at 'https://pwyv0x4l7l.execute-api.us-west-2.amazonaws.com/dev/login' from origin 'http://<IP_ADDRESS>:3000' has been blocked by CORS policy: Request header field sessionid is not allowed by Access-Control-Allow-Headers in preflight response.
login:
handler: controller/usercontroller.login
events:
- http:
path: /login
method: post
cors:
origin: 'http://<IP_ADDRESS>:3000'
headers:
- Content-Type
- X-Amz-Date
- Authorization
- X-Api-Key
- X-Amz-Security-Token
- X-Amz-User-Agent
- Access-Control-Allow-Headers
allowCredentials: false
function response(statusCode, message) {
return {
statusCode: statusCode,
headers: {
'Access-Control-Allow-Origin': 'http://<IP_ADDRESS>:3000' // Or use wildard * for testing
},
body: JSON.stringify(message)
};
}
can someone help me now why i am getting this error


Official page is down

I'm trying to access the official page (http://qtip2.com) but it seems to be broken.

I also faced this. 502 Bad Gateway response is coming on multiple resources request.

Hi everyone,
Apologies for this, I've been notified by a few of you but haven't had a
chance to take a look yet. If you need the docs they're in the wiki section
in the mean time.
Thanks for your patience.
On Fri, 12 Oct 2018, 06:54 AshutoshTiwari2<EMAIL_ADDRESS>wrote:

I also faced this. 502 Bad Gateway response is coming on multiple
resources request.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/qTip2/qTip2/issues/850#issuecomment-429212578, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AALeO9KQHIkUVzWDPpINapl9zFQpEoqzks5ukC6wgaJpZM4XYZ_q
.

--

*Craig Michael Thompson *
a Front-end Web Developer / Consultant
JavaScript development, together with HTML5/CSS3 and emerging WHATWG
standards
Email: http://www.craigsworks.com
Phone: +(44) 7878 062 458 <+447878062458>
Twitter*: @craigsworks https://twitter.com/craigsworks*

This is still down but I find the internet archives version helps:
https://web.archive.org/web/20180828195006/http://qtip2.com/

28 days later, the site is still down. :(

Fixed guys... finally!


When using WhereIn in a field alongside with a range operator on another field, it throws an exception

Hi,
After reading the documentation on query limitations I understood that the current behavior is a bug, but as I'm not 100% sure, I'm rather opening the issue as a Support request.
I'm trying to filter a collection by some document ids (less than 10) and by a property of the document, to know which users logged in to my application exactly 30 days ago, with both range operators filtering on the same field.
So, my understanding is that I'm not going against any limitation, as there's nothing in the documentation that says I can't query an "in" in a field and a range in another field.
Here's an example of what I'm trying to do:
var minValue = DateTime.UtcNow.Date.AddDays(-30);
var maxValue = minValue.AddDays(1);
var ids = new List<string> { "437f6e3b47144e948453f83f6477ed53", "548170046e1749b2896f16de971a34df" };

var collection = this.db.Collection("users");
var query = collection.WhereIn(FieldPath.DocumentId.ToString(), ids);
query = query.WhereGreaterThanOrEqualTo("lastConnection", minValue);
query = query.WhereLessThan("lastConnection", maxValue);

var querySnapshot = await query.GetSnapshotAsync();

Executing this code in a .NET Core 3.1 project, with Google.Cloud.Firestore 2.3.0 referenced, I get the following exception:

Grpc.Core.RpcException: Status(StatusCode="InvalidArgument", Detail="Cannot have inequality filters on multiple properties: [key, lastInteraction]", DebugException="Grpc.Core.Internal.CoreErrorDetailException: {"created":"@1608741029.523000000","description":"Error received from peer ipv4:{AN IP HERE}","file":"T:\src\github\grpc\workspace_csharp_ext_windows_x64\src\core\lib\surface\call.cc","file_line":1062,"grpc_message":"Cannot have inequality filters on multiple properties: [key, lastInteraction]","grpc_status":3}")
at Grpc.Core.Internal.ClientResponseStream2.MoveNext(CancellationToken token)    at Google.Api.Gax.Grpc.AsyncResponseStream1.MoveNextAsync(CancellationToken cancellationToken)
at System.Linq.AsyncEnumerable.g__Core|295_0[TSource](IAsyncEnumerable1 source, Action1 action, CancellationToken cancellationToken) in d:\a\1\s\Ix.NET\Source\System.Linq.Async\System\Linq\Operators\ForEach.cs:line 31
at System.Linq.AsyncEnumerable.g__Core|295_0[TSource](IAsyncEnumerable1 source, Action1 action, CancellationToken cancellationToken) in d:\a\1\s\Ix.NET\Source\System.Linq.Async\System\Linq\Operators\ForEach.cs:line 31
at Google.Cloud.Firestore.Query.GetSnapshotAsync(ByteString transactionId, CancellationToken cancellationToken)

Am I doing something wrong or is this a bug?
Thank you very much

Firstly, I'm on vacation at the moment (as is the rest of the .NET client library team) so please forgive any delays in response. But I suspect this is more of an issue with the documentation of Firestore queries rather than a bug in the .NET library - I expect this is a server-side query limitation.
I'll check your query when I'm back at work (January 5th) but until then you might want to try performing the same query with another language (or in the Firestore console if that supports custom queries - I can't remember or check right now). I suspect you'll get the same error.

@jskeet thank you for getting back to me, especially while being on vacation.
You were absolutely right, I tried with Node.js and got the same error.
Here's my Node.js code
const { Firestore } = require('@google-cloud/firestore');
const firestore = new Firestore({ projectId, keyFilename })

async function quickstart() {
    const listOfReferences = ['5f87094d1face4007522c7a6', '5fa2928e9ca37c007671f268'];
    const collection = firestore.collection('PrivateInfluencersInfo');
    const minDate = new Date(2020, 11, 3, 23, 0, 0, 0);
    console.log('minDate', minDate);
    const maxDate = new Date(2020, 12, 4, 23, 0, 0, 0);
    console.log('maxDate', maxDate);
    const query = collection
    .where('__name__', "in", listOfReferences)
    .where('lastInteraction', '>=', minDate)
    .where('lastInteraction', '<', maxDate)
    ;
    let cont = 0;
    await query.get()
        .then(function (querySnapshot) {
            querySnapshot.forEach(function (doc) {
                // doc.data() is never undefined for query doc snapshots
                console.log(doc.id, " => ", doc.data());
                cont++;
            });
            console.log('total', cont);
        })
        .catch(function (error) {
            console.log("Error getting documents: ", error);
        });
}
quickstart();

And here's the error

Error getting documents:  Error: 3 INVALID_ARGUMENT: Cannot have inequality filters on multiple properties: [key, lastInteraction]
at Object.callErrorFromStatus (C:\Dev\node-firestore\node_modules@grpc\grpc-js\build\src\call.js:31:26)
at Object.onReceiveStatus (C:\Dev\node-firestore\node_modules@grpc\grpc-js\build\src\client.js:327:49)
at Object.onReceiveStatus (C:\Dev\node-firestore\node_modules@grpc\grpc-js\build\src\client-interceptors.js:305:181)

Then, I don't know how to proceed, if I should close this issue and take it somewhere else, or if I should let you handle it internally.
I will fix my .NET code to workaround that error.
Enjoy your vacation. There's no rush for you to get back to me :)

Thanks for testing that. I'll close this issue, but when I'm back at work I'll raise this with the Firestore team to clarify the documentation.
Thanks for raising it so clearly.


Hello, I've been trying to know what mod keep crashing my world, please help

I don't know why it keeps randomly crash while playing. Really hope i can get some help
Minecraft version: 1.20.1
Fabric version: 0.16.9
Enhanced Falling Tree version: 0.5.0
java.util.ConcurrentModificationException
at java.base/java.util.ArrayList$Itr.checkForComodification(ArrayList.java:1095)
at java.base/java.util.ArrayList$Itr.next(ArrayList.java:1049)
at knot//dev.architectury.event.EventFactory$1.handleInvocation(EventFactory.java:61)
at knot//com.google.common.reflect.AbstractInvocationHandler.invoke(AbstractInvocationHandler.java:87)
at knot/jdk.proxy2/jdk.proxy2.$Proxy255.render(Unknown Source)
at knot//net.minecraft.class_757.handler$zpp000$architectury$renderScreenPost(class_757.java:20674)
at knot//net.minecraft.class_757.method_3192(class_757.java:945)
at knot//net.minecraft.class_310.method_1523(class_310.java:1219)
at knot//net.minecraft.class_310.method_1514(class_310.java:802)
at knot//net.minecraft.client.main.Main.main(Main.java:250)
at net.fabricmc.loader.impl.game.minecraft.MinecraftGameProvider.launch(MinecraftGameProvider.java:480)
at net.fabricmc.loader.impl.launch.knot.Knot.launch(Knot.java:74)
at net.fabricmc.loader.impl.launch.knot.KnotClient.main(KnotClient.java:23)
at org.prismlauncher.launcher.impl.StandardLauncher.launch(StandardLauncher.java:100)
at org.prismlauncher.EntryPoint.listen(EntryPoint.java:129)
at org.prismlauncher.EntryPoint.main(EntryPoint.java:70)
logs: https://mclo.gs/bYdpXXj
Here are the mods work work with Architectury I think

Are you sure the main problem is with my mod? The crash log doesn't point to it, and the screenshot indicates that you're just missing the Architectury API

Hey, I'm not good at coding but I've learned about Java a little bit, you should use the ConcurrentHashMap, instead of Hashmap in Java, switching to ConcurrentHashMap is easy. I keep getting concurrentmodificationexception crashin the modpack.
This is your code, hope you can change all the HashMap to ConcurrentHashMap:
package me.adda.enhanced_falling_trees.api;
import java.util.HashMap;
import java.util.Iterator;
import java.util.Map;
import java.util.Optional;
import java.util.function.Supplier;
import net.minecraft.class_2680;
import net.minecraft.class_2960;
public class TreeRegistry {
private static final Map REGISTRIES = new HashMap();
public static Supplier register(class_2960 resourceLocation, Supplier treeTypeSupplier) {
REGISTRIES.put(resourceLocation, (TreeType)treeTypeSupplier.get());
return treeTypeSupplier;
}
public static Optional getTreeType(class_2680 blockState) {
return REGISTRIES.values().stream().filter((treeType) -> {
return treeType.enabled() && treeType.mineableBlock(blockState);
}).findFirst();
}
public static Optional getTreeType(class_2960 resourceLocation) {
return Optional.ofNullable((TreeType)REGISTRIES.get(resourceLocation));
}
public static class_2960 getTreeTypeLocation(TreeType treeType) {
Iterator var1 = REGISTRIES.entrySet().iterator();
  Map.Entry entry;
  do {
     if (!var1.hasNext()) {
        return null;
     }

     entry = (Map.Entry)var1.next();
  } while(entry.getValue() != treeType);

  return (class_2960)entry.getKey();

}
}

Ah, well, in general this has a place to be, so yes, I'll review it in more detail.


I recommend KOReader, but it has some problems on DPT-RP1 ...

Just to see if there is someone can solve the problem.
Since most people hack their dpt-rp1 to read books, I recommend the app KOReader, which works like Kindle (the device, not the app). Here are a snapshot from my dpt-rp1.
This is the setting pane to adjust the DPI (ignore the error dialog box):

However, the bad news is that the android version KOReader seems doesn't refresh the e-ink screen correctly. When you touch the screen, it does responds (you can see it through logcat), but it often dose not refresh the content. You have to push the menu button to force it to refresh.
In fact, this is so annoying that I can barely use it.
I list it here so that someone capable can fix the little bug. Please info me if I bothered anyone.

Here is the "solution" I am currently using: Install Wake Lock and set it to SCREEN_DIM_WAKE_LOCK and autostart on boot. This should make KOReader usable but probably has an impact on battery life.
Acquiring a wakelock keeps /sys/power/asleep to on instead of snooze while using the device. If you press the power button so that the screen with "DIGITAL PAPER" is visible, the state will be standby as without the wakelock so I think the battery impact should only be there while using the device.
Another option might be to play around with the snooze_timeout_millis in /etc/dp_configurations/StandbySettingsConf.xml. I haven't tried anything like that, just found the file later on. I guess that a real solution should be part of KOReader though as other apps seem to refresh the screen normally.

Thanks a lot!
I have contacted the maintainers of KOReader. It seems the refresh problem always happens on the Android device with an e-ink screen. Since they don't have many brands of e-ink Android devices available, there is no general way to fix it.
I'm not familiar with Android, so I searched about Wake Lock on the Internet. And according to your description,  I guess this is how dpt-rp1 works: Every time 1000ms after you stop operating, dpt-rp1 switches the e-ink screen off to save power. Since KOReader doesn't "reactivate" the screen before refreshing, most of the refresh operations are enqueued and eventually lost.
The KOReader refresh function locates in the file /data/data/org.koreader.launcher/files/ffi/framebuffer_android.lua, I guess I might fix the problem by modifying it. At least I hope so.
I don't know Android, and I don't know lua either.

I have to say, at lest, the WakeLock method works.

I just tried the method with /etc/dp_configurations/StandbySettingsConf.xml and it also worked. Executing the following in adb shell has essentially the same effect as the wakelock method:
su
mount -o rw,remount /system
busybox vi /etc/dp_configurations/StandbySettingsConf.xml
# change snooze_timeout_millis to 86400000
cd /data/system
mv ConfMgr.db ConfMgr.db_bak
mv ConfMgr.db-journal ConfMgr.db-journal_bak
reboot

I will be using this method now instead.

Setting snooze_timeout_millis to 10000 instead of 86400000 also works for pretty much all relevant operations and might be better for the battery. If you want to fix this in KOReader, keeping a wakelock until all operations are finished is probably a good idea.

I tried the  /etc/dp_configurations/StandbySettingsConf.xml at first, but I didn't delete the files ConfMgr.db and ConfMgr.db-journal, so it didn't work.
Thanks. Now it works without the WakeLock app.

I also tried KOReader on my android smartphone and compared the two adb logcat. The DPT has a lot of messages like W/PowerManagerService( 2149): Going to sleep due to screen timeout (uid 1000)..., W/PowerManagerService( 2149): [PhoneWindowManager] Screen turned off...TRANSITION_WAKEUP_TO_SNOOZE and D/PowerManagerService( 2149): ReleaseSuspendBlocker: PowerManagerService.WakeLocks so I thought that maybe this would be the reason.

Thanks for invitation! I'll write a new wiki page about it sooner. @HappyZ

Yes please!

@Zack-Q thanks! looks great!


Update Namespace.ValidateNamespace to add workspace to arguments

Signed-off-by: Josh<EMAIL_ADDRESS>Description
in order for the non current context to be used by ValidateNamespaces so that it can create namespaces in the workspaces kubernetes context we need to pass in the workspace and then use workspace.KuberneteContext() to look up the correct context.
Type of change
This pull request fixes a bug in Radius and has an approved issue (issue link required).
Fixes: #5698

Thanks for your contribution @jhandel! You should run make generate. Please see: https://github.com/radius-project/radius/actions/runs/7847801524/job/21417564330?pr=7154.

Thanks for your contribution @jhandel! You should run make generate. Please see: https://github.com/radius-project/radius/actions/runs/7847801524/job/21417564330?pr=7154.

sorry about that.. I thought I read somewhere that the generated code wasn't checked in so it could always be re-generated so I kind of ignored that assuming the build would run the generator..
Anyways, generated the updated code and checked..

Thanks for your contribution @jhandel !


Jacoco badge generator

Would be nice to have badges showing code coverage for our main branch and maybe also add a codecoverage in the review of pull requests?
https://github.com/marketplace/actions/jacoco-badge-generator

An alternative solution instead of using jacoco badge generator we can upload code coverage files to
https://about.codecov.io/
https://github.com/marketplace/actions/codecov


Buy now button redirection

As when user click on buy now button it will redirect it directly to payment page, but the current scenario is it does not navigate to payment page

Congratulations, @root-0101! 🎉 Thank you for creating your issue. Your contribution is greatly appreciated and we look forward to working with you to resolve the issue. Keep up the great work!We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

Merged in #289


Improve vae test mode

With this patch VAE example test time is reduced:
1:21.58 --> 8.387

jenkins, test this please.

LGTM!


Fixed a broken job board links Closes #183

changed the non-existing job posting link with the actual job board link.

Thanks, will do more in the future


[Bug][Build] The build of plugin failed for wrong git2go version

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
$ make build-plugin
Usage:
  build all plugins:              scripts/compile-plugins.sh [golang build flags...]
  build and keep one plugin only: PLUGIN=jira scripts/compile-plugins.sh [golang build flags...]
Building plugin ae to bin/plugins/ae/ae.so
Building plugin dbt to bin/plugins/dbt/dbt.so
Building plugin feishu to bin/plugins/feishu/feishu.so
Building plugin gitee to bin/plugins/gitee/gitee.so
Building plugin gitextractor to bin/plugins/gitextractor/gitextractor.so
Building plugin github to bin/plugins/github/github.so
Building plugin gitlab to bin/plugins/gitlab/gitlab.so
Building plugin icla to bin/plugins/icla/icla.so
Building plugin jenkins to bin/plugins/jenkins/jenkins.so
Building plugin jira to bin/plugins/jira/jira.so
Building plugin org to bin/plugins/org/org.so
Building plugin refdiff to bin/plugins/refdiff/refdiff.so
Building plugin starrocks to bin/plugins/starrocks/starrocks.so
Building plugin tapd to bin/plugins/tapd/tapd.so
# pkg-config --cflags  -- libgit2
Package libgit2 was not found in the pkg-config search path.
Perhaps you should add the directory containing `libgit2.pc'
to the PKG_CONFIG_PATH environment variable
No package 'libgit2' found
pkg-config: exit status 1
Makefile:25: recipe for target 'build-plugin' failed
make: *** [build-plugin] Error 2

After sudo apt-get install libgit2-dev

$ make build-plugin
Usage:
  build all plugins:              scripts/compile-plugins.sh [golang build flags...]
  build and keep one plugin only: PLUGIN=jira scripts/compile-plugins.sh [golang build flags...]
Building plugin ae to bin/plugins/ae/ae.so
Building plugin dbt to bin/plugins/dbt/dbt.so
Building plugin feishu to bin/plugins/feishu/feishu.so
Building plugin gitee to bin/plugins/gitee/gitee.so
Building plugin gitextractor to bin/plugins/gitextractor/gitextractor.so
Building plugin github to bin/plugins/github/github.so
Building plugin gitlab to bin/plugins/gitlab/gitlab.so
Building plugin icla to bin/plugins/icla/icla.so
Building plugin jenkins to bin/plugins/jenkins/jenkins.so
Building plugin jira to bin/plugins/jira/jira.so
Building plugin org to bin/plugins/org/org.so
Building plugin refdiff to bin/plugins/refdiff/refdiff.so
Building plugin starrocks to bin/plugins/starrocks/starrocks.so
Building plugin tapd to bin/plugins/tapd/tapd.so
# github.com/libgit2/git2go/v33
../../go/pkg/mod/github.com/libgit2/git2go/v33@v33.0.6/Build_system_dynamic.go:12:3: error: #error "Invalid libgit2 version; this git2go supports libgit2 between v1.3.0 and v1.3.0"
 # error "Invalid libgit2 version; this git2go supports libgit2 between v1.3.0 and v1.3.0"
   ^~~~~
Makefile:25: recipe for target 'build-plugin' failed
make: *** [build-plugin] Error 2

What you expected to happen
The build can be successful
How to reproduce
make build-plugin on ubunut Ubuntu 18.04.6 LTS
Anything else
No response
Version
v0.12.0
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

@lshmouse please check the docs: https://devlake.apache.org/docs/Plugins/refdiff#development

@lshmouse please see the libgit2 installtion docs: https://devlake.apache.org/docs/Plugins/refdiff#development

Thanks your pointing out.  The builds are passed, thanks~


hexo-generator-search not work in this theme

I followed official docs of hexo-generator-search, but the search bar of theme-landscape still can not search locally, it will turn to google search. Can anyone tell me how to use hexo-generator-search in this theme? Or it will be great if anyone can tell me how to local search in landscape-theme without using hexo-generator-search. I am using github pages to show my blogs.

I have the same question.

the readme of hexo-generator-search has told you how to do. use the theme which used [hexo-generator-search](https://github.com/wzpan/hexo-generator-search) or write script and page by yourself.

您好，收到了~感谢您的来信，祝您生活愉快！


Can't retrieve information from UPDATE QUERY

I just upgraded to 0.2.0 and just hit the first bug.
If you update an entry, the result stream is never going to be called and it is impossible to access the information property which is needed for informations like the last insert id.
To test it, take the following gist:
https://gist.github.com/wzrdtales/f679c3251ea3488a7fec

This should be fixed as of 5643167a54 and v0.2.1.

@mscdex Awesome, can confirm that this works now :)


General Updates 13

⏱ Before you start

[x] Have you checked if a similar PR has already been requested?
[x] Have you built and ran the app?

↗ Related/Fixed issues

Closes #186
Closes #189

📄 Description
Official General Updates round 13
💭 Motivation and Context
Todo

[ ] Add click event to activities button in activities section in home page.
[ ] Add click event to feed items in activities page.
[ ] Handle OAuth Restriction exception properly.
[ ] Completely update and improve indivisual issue/PR page.
[ ] Add repository Insights page
[ ] Add organization Peoples page
[ ] Add organization Projects page
[ ] Add filtering feature (reposityries, users, notifications, ...)
[ ] Update indivisual Discussion page
[ ] Update indivisual Project page
[ ] Add repository Actions page
[ ] Add search page
[ ] Correct a word 'oction' to 'octicon'
[ ] Fix activities page
[ ] Add multilingual resources (en-US) English
[ ] Add multilingual resources (en-UK) 'Real' English
[ ] Add multilingual resources (es-ES) Spanish
[ ] Add multilingual resources (ja-JP) Japanese
[ ] Add multilingual resources (nl) Dutch

📸 Assets (if appropriate):
None
React with a 👍 if you'd like to see this feature implemented

New color codes:

#0d1017
#151b22
#21262e
#30373d
#484f58
#6e7681
#8b939e
#afbac5
#c9d1d9
#f0f6fc
#051d4d
#0d2d6b
#0d419d
#1158c7
#206feb
#388bfd
#58a6ff
#79c0ff
#a5d6ff
#cae8ff
#271051
#3c1e70
#553098
#6e40c9
#8957e5
#a371f7
#bc8cff
#d2a8ff
#e2c5ff
#edddff
#42062b
#5d103e
#7d2457
#9e3670
#bf4b8a
#db61a2
#f778ba
#ff9bce
#ffbedd
#ffdaec
#490402
#67090d
#8e1619
#b62324
#da3633
#f85149
#ff7b72
#ffa198
#ffc1ba
#ffdad5
#3d1300
#5a1e02
#752d09
#9b4216
#bd561d
#db6d28
#f0883e
#ffa657
#ffc680
#ffdeb3
#341a00
#4b2a00
#693e00
#825303
#9e6a03
#bb800a
#d29922
#e3b341
#f2cc60
#f8e3a1
#04260f
#053a15
#0f5322
#186c2e
#228636
#2ea043
#3fb950
#56d364
#7ee787
#aff5b4

New color codes:

what are these for? @onein528

Coloring some places such as Issue/PR state labels, normal labels

Coloring some places such as Issue/PR state labels, normal labels.
You don't agree?

idk. I suppose it might not be in accordance with the Microsoft Fluent Design System guidelines. @onein528

I know it. but are there colors of fluent design system?

I know it. but are there colors of fluent design system?

@onein528 The AccentColour variables are fluent.

Open:
Close:
Draft:
Pending:
PRO:
there are many.

I think color color isn't important and what is important is how to display with those colors for uses.

I think color color isn't important and what is important is how to display with those colors for uses.

sure then.

@onein528?

Can you build GU13?

@DeveloperWOW64 Can you build GU13 instead of schema-gen branch?

Okay - done.. now the errors about Configuration Manager are done, but now I have these errors:
https://discord.com/channels/935562861701390336/982985508634366002/996302622946512927

@onein528

See this.
https://devblogs.microsoft.com/dotnet/csharp-11-preview-updates/#remove-parameter-null-checking-from-c-11
'paramerter null-checking' is no longer supported. it is removed in preview SDK.
We need to fix.

'paramerter null-checking' is no longer supported. it is removed in preview SDK.
We need to fix. Should be 'latest'

Okay - but wouldn't that only be a temporary fix? @onein528

I Updated csproj.
Then please build. if you cannot, tell me.

WOAH @onein528 Nice!
Fixed the error!

Changed a lot since I've last been able to run the app..

what exactly?

what exactly?

@onein528 Sidebar, shell, etc. very nice!

@onein528 When will you implement localisation resources?

before merged this.

Simplified

All roght. Ready to add multilingual resources to some pages such as Intro Page.xaml

All right. Ready to add multilingual resources to some pages such as Intro Page.xaml

Okay! Sadly I can only read spanish and not write so we'll have to wait for @BobbyESP. does @luandersonn know Brazilian Portugese?

All right. Ready to add multilingual resources to some pages such as Intro Page.xaml

Okay! Sadly I can only read spanish and not write so we'll have to wait for @BobbyESP. does @luandersonn know Brazilian Portugese?

@onein528

Yes! I will mention. I am going to prepare the default language, en-US.

@Gaoyifei1011 Can you do Chinese?

@Gaoyifei1011 Can you do Chinese?

Yes, I can. I submitted a PR about Chinese Simplified translation a few months ago.

Yes, I can. I submitted a PR about Chinese Simplified translation a few months ago.

Ah, I remember! But we've changed the UI so much it might not be valid anymore..

@onein528 You still working on multilingual resources?

All right. Ready to add multilingual resources to some pages such as Intro Page.xaml

Okay! Sadly I can only read spanish and not write so we'll have to wait for @BobbyESP. does @luandersonn know Brazilian Portugese?

Yes

Ohh guys. So sorry for being very inactive with this :( Img having some issues but dont worry. I can translate the app!

It's fine! @luandersonn has internship apparently so he is not online either. You will LOVE to see what's changed since you last built; the UI has improved so significantly!

You guys are doing a good work, well done

Cannot add resources...

Why? Option is greyed out?
@onein528

YES

I have seen that before.. so weird

I will investigate.

@onein528 When clicking on activities/marketplace (buttons I added) they don't fill with the accent colour

also, activities gives NullReferenceException.

Should I delete those button? I think it doesnt make any sense.

Should I delete those button? I think it doesnt make any sense.

@onein528 Why? Marketplace is queryable and Activities cannot go underneath the recent feed as it is different info. Build the app to see changes in a4b088b81f5f221adae9d29dd78b9971450c3c6b.

No I think I should remove all buttons

No I think I should remove all buttons

But what would we do then? Personally I disagree.

Right.
I have to create new Actovity page then?

Right. I have to create new Actovity page then?

Create a new one? Why?

Create a new one? Why?

@onein528


[rbac]clone roles Management

Is your feature request related to a problem? Please describe.
It's incredibly difficult to manage, create roles.
Describe the solution you'd like
The ability to clone a role or copy/export/import roles. Or just fix the UI so I can paste permissions as text and have them work. Then we could easily test roles before.
Describe alternatives you've considered
Scraping values from the DB directly.:(
Additional context
Having to comb through permissions one-by-one and paste them in to merge is a brutal waste of time. Instead if we could just clone gamma and add what we actually want.

@junlincc This feature existed in version 0.36. Does not anymore in 0.37.
Not sure why. Perhaps @villebro could comment.

@dpgaspar

I re-looked at v 0.37.1, the function of copy role works fine.
I have noticed it only allowing copying of Public roles when using AUTH_TYPE setup, even when the logged in user is an Admin. Perhaps that's what the other are facing as well.

close this since no recent follow-up.


ImportError: No module named parse

Traceback (most recent call last):
  File "/usr/local/bin/bloodhound-python", line 11, in <module>
    load_entry_point('bloodhound==1.0.4', 'console_scripts', 'bloodhound-python')()
  File "/usr/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 489, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/usr/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 2852, in load_entry_point
    return ep.load()
  File "/usr/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 2443, in load
    return self.resolve()
  File "/usr/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 2449, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "build/bdist.linux-x86_64/egg/bloodhound/__init__.py", line 26, in <module>
  File "build/bdist.linux-x86_64/egg/bloodhound/ad/domain.py", line 29, in <module>
  File "build/bdist.linux-x86_64/egg/dns/resolver.py", line 19, in <module>
ImportError: No module named parse

I am sure i've install parse successfully.

I expect this is a problem with your installation. Could you try with python 3?


feature : maybe the /run command make this better

Feature : /run command
this command run and launch of the Shorcuts(PWA's) and Installed Chrome Web Apps (alike SoundCloud) or alike Twitter Installing Popup
PWAs
Hamburger Menu > More Tools > Create Shortcut... > Run As Window (tada :tada: You have web-app installed your chrome)
or
Chrome Web App Store
going to chrome web app store (not chrome extensions)
install what you want (:tada: tada, you have official installed app)
or
Installing Popup
alike Twitter or YouTube Web Apps, show us popup to confirm installing
Why i'm explain these three way?
because some extension just launch Chrome Web App Store and didn't recognize any other else app :-1:
but i think you can make it better if launch other way's apps :+1:
And/Or
launching the extensions make so powerful :star_struck:
At The End
Thanks for must powerful this tool :heart:

Aha, Alson using /install command to create shortcut from current web-page or if have the installable chrome-web-app, move user to the chrome web app page and if have installing confirm, triggering it (maybe) :thinking:
Best Wishes for You :wink:


6 tests fail on FreeBSD

log

The error in leanlaketest_buildArgs seems to be because this check is wrong:
https://github.com/leanprover/lean4/blob/53af5ead5359eeb47e1c7a62381000337b5d04c0/src/lake/tests/buildArgs/test.sh#L6-L10
Is it correct to say that FreeBSD uses the same sed as Darwin even though the uname is different?

The leanlaketest_serve error is similar:
https://github.com/leanprover/lean4/blob/53af5ead5359eeb47e1c7a62381000337b5d04c0/src/lake/tests/serve/test.sh#L4-L8
is apparently needed because the tail command on mac doesn't have the --pid option. Presumably it has the same fix as the other one.


feat(ci): check cargo install

Motivation
See https://github.com/foundry-rs/foundry/issues/5788
Ensure that cargo install works for all binaries as some workspace dependencies hide features that need to be explicitly declared by crates.

This job can be replaced by just cargo hack check which runs cargo check on all crates. Installing with optimizations is not necessary.


BIOS support

Hi! Does this chip support different bios besides cerbios? I want to make a 128mb ram mod on my xbox 1.6, and for this I need xblast bios for ram tests. For 1.6 there is a modified xblast. Now I only have a softmod, and I’m still thinking about which modchip to choose is best. Thanx in advanse.

Yes, i have run Evox. Try using the packer utility (preferably the python script at the repository).

Please feel free to test any other Bios and notify if there is  one that doesn't work


Proposal to add new feature to the user endpoint - Passwordless

Passwordless will allow you to login in using just an email, the service will send a link to the submitted email and once the owner clicks the link the service will publish new message which contains session to the specified topic.

I think the idea of passwordless login is sound. The naming of endpoints might change and the implementation itself. e.g SendMagicLink and VerifyToken or something of that nature. I think also topic shouldn't be a thing we put in the request because it exposes implementation details. Instead you want something like ID, and it might be more appropriate to store in the cache or similar for a given TTL.

Hi @asim
Did some changes to the code as your suggestions (see the commit)
Code needs some refactoring though.

Will come back and review this soon

Might ask @crufter or @domwong also to have a look

What I’m trying to achieve here is when the user clicks the login button the user/SendMagicLink  kicks in with an email as payload, then it does the necessary checks and after that it subscribes to a topic (currently this code uses the old stream and m3o now has three new options: new stream, mq and events). The login page won’t update/redirect until the m3o client receives a session object which indicates that the user has clicked the MagicLink sent to his/her email.
There are some issues here with the link itself, currently when the user clicks the MagicLink it will redirect him/her to a page that shows “not authorized” message i believe because i didn’t attach M3O user API token with the MagicLink, so i think we need some sort of 3rd party app that consume the link instead then publish to the topic.

Taking a high level look, I like the approach in general; I like the use of the broker for pubsub. Need to be wary of leaking goroutines though so need to make sure to break out of the loop after some timeout. Thumbs up from me in that it's definitely something worth progressing but will need a proper review and testing - @crufter might be better placed for that

I'd like to get this merged this week. @lambdaR can you please resolve the conflicts then we'll review and merge.

Since M3O will introduce App hosting soon, I am leaning toward the idea of removing the VerifyToken endpoint and replace it with passworldless.m3o.com App instead, basically a web server that receives a request from the link that have been sent to user’s email, do the necessary checks and display green check symbol if successful or a red X if the token was invalid or any failed checks.
In short, converting the VerifyToken endpoint to a web server hosted on passworldless.m3o.com.

We have user.m3o.com for any app based stuff for verification, but equally apis should be standalone and the functionality shouldn't depend on third party apps that we can't guarantee will exist

What 3rd party? M3O should provision and maintain the passworldless.m3o.com in this case and it is dedicated only to the Passwordless feature proposed.

This repo is totally separate to any form of hosting. If the API itself or the endpoints are not useful without a UI then its an issue

Ok, Is there a chance that M3O can internally provide a mechanism to generate a one time token (token for accessing endpoints)? In this way we can send to the user an email with a link that holds two tokens, one for accessing (one time access) the VerifyToken endpoint and the second one to authorize access for the account.

There is an otp service in this repo you can call for that

I will keep this branch for my personal use and move the PR to  #292


[Feature Request] Add Sudo Support on Windows

Description of the problem or steps to reproduce
Since Sudo wasn't really a thing on Windows, micro has no support for this, and will happily tell this to the user when trying to use:

However, since 24H2, Windows added sudo as an option. It behaves pretty similar, eg by running sudo <my-command> it would show a UAC prompt, and on success run my-command in an administrator instance of that commandline.
Additionally, even if 24H2 is not being used, GSudo is basically the exact same.
It would be a great addition to support sudo when either 24H2's Sudo or Gsudo is present.
Specifications
Commit hash: /
OS: Windows 11
Terminal: /

I am not available to look much into adding support, but I tried checking what had to be specifically done to add support months ago. Micro runs dd when saving files using sudo but dd is not preinstalled. I wrote about it in a comment in a different issue: https://github.com/zyedidia/micro/issues/2660#issuecomment-2267289804

What exactly is micro internally using on windows to save files? Couldn't it just do the exact same it is already doing, but instead with this sudo/gsudo command as a prefix? Sadly i don't know anything about the go language or windows internals itself, otherwise i'd gladly help however i can.

Micro writes to files using Go standard library functions in all platforms when saving without sudo, so the same thing cannot be done as a command using sudo. The actual command run with sudo in all platforms except Windows is like echo "new file content" | sudo dd bs=4k of=file.txt, but micro uses functions to write input that sudo passes to dd instead of echo "new file content" |.
dd is a program used like a command that is usually preinstalled in Linux distributions. It's not available by default in Windows so it may be inconvenient if micro uses dd in Windows. Micro may just use a different command instead of dd ... in Windows to add support, but I don't know what command can be used.

As @niten94 already wrote, it's unfortunately not done by allowing/enabling sudo on Windows, because it would require additionally dd (coreutils) for Windows. The latter one isn't impossible, because they already have been ported (e.g. gnuwin32 coreutils, rust coreutils), but need to be installed too.
Or it requires a fully different approach for storing files with extended privileges like ShellExecute (see here: https://blog.hadenes.io/post/how-to-request-admin-permissions-in-windows-through-uac-with-golang/).
The easiest way for us would be option one, since the mechanism doesn't need to be rewritten or extended and we just need to enable the sudo support and give hints that additional tools are required.

Micro may just use a different command instead of dd ... in Windows to add support, but I don't know what command can be used.

If all that is needed is an alternative native command, can't we just use powershell for that?
I googled a bit, and tested a bit of powershell script in a freshly installed windows-10 vm (only gsudo installed), and could observe the following:

Maybe this could be used on windows?

Maybe this could be used on windows?

That's like running micro itself with super user rights and using Go's standard functionalities to open and write, but this isn't what we want...at least not in this form. The extended privileges shall only be used for the file save process, not for the rest of micro.
This is the current approach to store the file:
https://github.com/zyedidia/micro/blob/56c1f75bad017a817e7ecf7e5b3bb0df7c4a8623/internal/buffer/save.go#L37
Currently I don't know if there is a different alternative other than dd under Windows to use the stdin as input and write to a file.
BTW:

Or it requires a fully different approach for storing files with extended privileges like ShellExecute (see here: https://blog.hadenes.io/post/how-to-request-admin-permissions-in-windows-through-uac-with-golang/).

Currently realized, that we don't even need sudo, because Windows already offers runas, which can be used via the native exec.Command() without the need of ShellExecute. And the runas can be stored into sucmd.

in this case, my basic "idea" wasn't to run the entirety of micro as sudo, but rather something like
cmd = exec.Command(config.GlobalSettings["sucmd"].(string), "powershell", "-command", "powershell -command", "& {...the-command...}")

(assuming that syntax is correct)
since the saving-process is done via a command anyway.
But if the runas method is more convenient that is probably better than this awfully long powershell-script.

But the PowerShell isn't available by default either or am I wrong? micro can be executed under Windows without the PowerShell. So we end up in a situation where something additionally needs to be installed anyway. Then I would recommend to go with a coreutils alternative (mentioned here: https://github.com/zyedidia/micro/issues/3541#issuecomment-2491877196) and the dd path should most probably be configurable.

both windows 10 and 11 come with powershell pre-installed. it should therefore be available on every pc that isn't stuck 200 years ago.

That command does not seem to work from what i can see. Dunno what the '-' means either.
Does the 4k block size matter? If not, the simplest way would probably be the command:
powershell -command "string-content" | Out-File -FilePath "path/to/file"

This would create the given file (if it does not exist), and afterwards write the given content to it.

I think @JoeKar was trying to read the data written to PowerShell standard input so Get-Content -Path '-' was written. Each string passed except the first argument are passed as one argument to the program in the first argument (exec.Command: sudo, sudo: powershell), so arguments have to be passed like in the syntax that @JoeKar wrote. The file content is written to the standard input of cmd in line 39, 63 and 64 below, so it does not have to be included in arguments:
https://github.com/zyedidia/micro/blob/56c1f75bad017a817e7ecf7e5b3bb0df7c4a8623/internal/buffer/save.go#L39-L64
I do not know much about .NET and PowerShell but I tried reading documentation and I do not think cmdlets can be used. The output encoding is handled in line 63 but encoding seems to be converted when using pipes and Out-File^1 in PowerShell. Set-Content -AsByteStream cannot be used in PowerShell 5.1.^2
I did not check anything with error handling but I think this can be done. The code can also be passed in Base64.
using namespace System.IO
$f = [File]::Open($args[0],
    [FileMode]::OpenOrCreate -bor [FileMode]::Truncate, [FileAccess]::Write)
$i = [System.Console]::OpenStandardInput()
$i.CopyTo($f)
$i.Dispose()
$f.Dispose()

I am not sure if Go would do that different, but whenever i run this inside a powershell terminal, it just fails because it takes that '-' literally, but expects an actual filesystem path it seems:

The file content is written to the standard input of cmd in line 39, 63 and 64 below, so it does not have to be included in arguments:

Wouldn't that require both of those processes (A: Creating the file, and B: Writing to it) require admin permissions? I am pretty sure just running the creation process as sudo won't automatically give the process access to write to it.

Honestly, i think i totally lack the options to test something like this. I have no idea what [System.Console]::In.ReadToEnd() does, or specifically i cannot test it, since in a console it just... freezes, probably since it does something entirely else in the Go Program context.
If someone else with go & ps knowledge and the ability to build micro could test this all, it would probably be way easier than me trying to fiddle my way into this without knowing anything about... anything really.

I have no idea what [System.Console]::In.ReadToEnd() does, or specifically i cannot test it, since in a console it just... freezes, probably since it does something entirely else in the Go Program context.

It has nothing to do with Go and it doesn't freeze in the first place, it just waits for input, since we try to read from the stdin.
Anyway, I created my own Win10 VM for a faster test and was able via...
echo test | powershell -command "[System.Console]::In.ReadToEnd() | Set-Content -Path test.txt
...to forward the echoed string into the file, but most probably we will run into trouble, since it just takes the string and doesn't interpret it as byte stream. With runas additional user information must be provided as parameter and can't be set within the sucmd...otherwise the whole string will be searched as command, so it will increase complexity especially when it shall execute a command like this as one-liner.

Sorry, I think my previous comment had unnecessary information and was confusing. I have Windows 10 installed on my personal machine so I can build micro and test, but replacing dd ... only with PowerShell code is more complicated than I thought.

But to be honest, I never used PS before. If you find a nice way to create a short replacement for the dd-approach without dd itself but with Windows internals...why not.

Anyway, I created my own Win10 VM for a faster test and was able via...
echo test | powershell -command "[System.Console]::In.ReadToEnd() | Set-Content -Path test.txt
...to forward the echoed string into the file, but most probably we will run into trouble, since it just takes the string and doesn't interpret it as byte stream.

I do not think there is anything else available on Windows that can be used by default. I tried to somewhat search and learn about .NET and PowerShell, but I do not think there is a short command that can be used. I used Console.OpenStandardInput in the code I wrote because Console.In is a TextReader, but the type of the object returned does not have a method like ReadToEnd.
I have only tested the code with $i.CopyTo($f) using gsudo on micro, but there are 3 ways the file can be written. The commands can be written in one line, but I think the first and third one look the least worst:
trap { break }
$f = [IO.File]::Open($env:OUTFILE, [IO.FileMode]::OpenOrCreate, [IO.FileAccess]::Write)
$i = [Console]::OpenStandardInput()
$i.CopyTo($f)
$i.Dispose()
$f.SetLength($f.Position)
$f.Dispose()

trap { break }
$i = [Console]::OpenStandardInput()
& {
    $b = [byte[]]::new(4096)
    while(($c = $i.Read($b, 0, $b.Length)) -gt 0) {
        [ArraySegment[byte]]::new($b, 0, $c)
    }
} | Set-Content -Encoding byte -LiteralPath $env:OUTFILE
$i.Dispose()

trap { break }
$i = [Console]::OpenStandardInput()
$ms = [IO.MemoryStream]::new()
$i.CopyTo($ms)
Set-Content -Encoding byte -Value $ms.ToArray() -LiteralPath $env:OUTFILE
$ms.Dispose()
$i.Dispose()

I have no experience with measuring performance so something wrong is done, but these results came out when I ran this script using busybox-w32:
set -u -e

if [ $# -ne 2 ]; then
    prog=`basename -- "$0"`
    printf '%s\n' "usage: $prog dest src" >&2
    exit 1
fi

d=`dirname -- "$0"`

export OUTFILE="$1"
for s in "$d"/*.ps1; do
    printf '%s\n' "${s#$d/}"
    cmd=`cat "$s"`
    time powershell -command "$cmd" < "$2"
    cmp -- "$2" "$1"
    read _
done

> busybox sh bench.sh samples\micro-README.md out.txt
copyto.ps1
real    0m 0.16s
user    0m 0.12s
sys     0m 0.07s

scbuf.ps1
real    0m 0.31s
user    0m 0.28s
sys     0m 0.07s

scval.ps1
real    0m 0.20s
user    0m 0.06s
sys     0m 0.15s

With runas additional user information must be provided as parameter and can't be set within the sucmd...otherwise the whole string will be searched as command, so it will increase complexity especially when it shall execute a command like this as one-liner.

I realized this when testing but I think additional parameters have to provided when using gsudo too. I do not know how the command is escaped, but gsudo runs it in a shell that is somehow detected automatically. cmd.exe is used as the shell when gsudo -d is run. The path has to be set in an environment variable if using PowerShell because arguments after -command "cmd" are included in cmd, so there is nothing to mind about escaping paths in arguments.

Sorry, I am probably the one who has searched the most in the thread, but I did not update on details I have found and what I was going to do. I think I should have at least said something even if short but I tried creating a pull request and adding support.

I do not know why I just realized this now, but is there any reason to not run micro itself with sudo and pass an internal flag like this?
sucmd := config.GlobalSettings["sucmd"].(string)
if runtime.GOOS == "windows" {
    exe, err := os.Executable()
    if err == nil {
        cmd = exec.Command(sucmd, exe, "-write", name)
    } else {
        return err
    }
}


GHxxx: Removed unnecessary parameter

Replaces PR #1649 as that one seems dead.

@Roemer,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@Roemer Great! Could you also add a test to verify that it indeed is dead?
The change itself is already tested in Should_Remove_Trailing_Slashes.


ARM support

ARM builds would be needed to support for example Raspberry Pi.

...but manylinux tag does not support ARM?

Thanks for making that great Python installer.
When will there be a version for raspberry pi?
Best regards
Gerhard

I hope that time will come fast.
I have seen some packages that could be installed with pip
Examp. https://github.com/samjabrahams/tensorflow-on-raspberry-pi
Do they create their own toolchain?

Yes, they have created their own special Docker image with all the correct dependencies and it runs on actual RPi hardware. Tensorflow itself has some tools to generate the package after the build.
What I would need to make this happen:

similar Docker image as the Cent OS 5 manylinux image is but with some RPi /ARM baseline which is common for all devices - this will probably require QEMU based image or some image with cross-compilation toolkit like Scratchbox2
I must be able to run the images on Travis
afaik the resulting wheels cannot be distributed via PyPI so I would need to host them manually somewhere

This is not a task that can be done over a weekend, it will require most probably several weeks, maybe months, of work.

Raspberry pi have a build system where they try to build all the pypi packages -- piwheels.
piwheels website: https://www.piwheels.hostedpi.com/
blogpost about what it is: http://bennuttall.com/piwheels-building-a-faster-python-package-repository-for-raspberry-pi-users/
FAQ: https://www.piwheels.hostedpi.com/faq.html
They are receptive to PRs to include Debian dependencies here: https://github.com/bennuttall/piwheels/
I guess all your dependencies aren't in the Debian they use though? Also, compiling on real hardware can take a long time for big C++ libraries.

(opencv-python piwheel for rpi)[https://www.piwheels.hostedpi.com/simple/opencv-python/]

I have no idea. I do not maintain those packages and I don't know how they were created.

We haven't built Python 2 wheels - we might do in future. But there's not much future left.

@bennuttall only use case I have for python2 wheels is for dependencies for Inkscape extensions, as Inkscape still uses python2.

I try to build this project on QEMU based docker image, it quite slow and often make docker build timeout. it's really painful...

when Can I "pip install opencv_contrib_python" in  Raspberry Pi?

I should add, I recently added the *-headless builds from this repo too (thanks to @skvark's fine work!), so you can "pip install opencv_contrib_python_headless" too. I watch this repo, so when there's a new release I try to add it to piwheels as soon as I'm able, but there'll usually be some delay.

@bennuttall  I am sorry that :
I have use pip install opencv_contrib_python-<IP_ADDRESS>-cp35-cp35m-linux_armv7l.whl and It seems OK, but when I
python  import cv2
It told me that:
Traceback (most recent call last):   File "<stdin>", line 1, in <module>   File "/home/pi/.pyenv/versions/3.5.2/lib/python3.5/site-packages/cv2/__init__.py", line 7, in <module>     from . import cv2 ImportError: libhdf5_serial.so.100: cannot open shared object file: No such file or directory
even
sudo apt-get update sudo apt-get install libhdf5-dev sudo apt-get update sudo apt-get install libhdf5-serial-dev
It didnot work ...
How can I makr it work ?

@cooldeaaap1 libhdf5-dev should be pulling in libhdf5-100 which in turn provides /usr/lib/arm-linux-gnueabihf/libhdf5_serial.so.100 (you don't actually need the *-dev packages for installation - those are for building - but they pull in the underlying library anyway, so no harm).

@waveform80 what should I do to make opencv-contrib-python work ?

@cooldeaaap1 when you say sudo apt-get update; sudo apt-get install libhdf5-dev libhdf5-serial-dev didn't work, do you mean installing them didn't work, or importing cv2 after installing them didn't work? If the former, then I'd need to see the error from apt-get. If the latter I'd have to ask if you were running the latest Raspbian or not?

@waveform80
Thank for your help , I change to the latest Raspbian , I want opencv-contrib-python==<IP_ADDRESS>;but I got error
ImportError: /usr/local/lib/python3.5/dist-packages/cv2/cv2.cpython-35m-arm-linux-gnueabihf.so: undefined symbol: hb_buffer_get_glyph_infos
change to others ,like <IP_ADDRESS> ,it works, Is it a bug ?

PyPi does not support ARM wheels yet.

@skvark, thanks for the great work. I am confused when you say pypi doesn't support arm, it does. Unless I am misunderstanding you, if you look at something I published sslam-0.0.3-cp27-cp27mu-linux_armv7l.whl it is a wheel file for armv7. I did nothing special, here is my setup.py, just a simple c extension. I did build the package on a real raspberry pi, not travis, because that is only x86. Also, my stuff is trivial compared to opencv. Again, I might be misunderstanding your comment. :smile:

@waveform80 I am sorry that : I  had better to use "<IP_ADDRESS>" ,because I want to use a prject https://github.com/AirtestProject/Airtest
and it's requirements contains "opencv-contrib-python <IP_ADDRESS>" https://github.com/AirtestProject/Airtest/blob/master/requirements.txt

@skvark, thanks for the great work. I am confused when you say pypi doesn't support arm, it does. Unless I am misunderstanding you, if you look at something I published sslam-0.0.3-cp27-cp27mu-linux_armv7l.whl it is a wheel file for armv7. I did nothing special, here is my setup.py, just a simple c extension. I did build the package on a real raspberry pi, not travis, because that is only x86. Also, my stuff is trivial compared to opencv. Again, I might be misunderstanding your comment. 😄

Yeah, you can upload arbitrary ARM builds to PyPI nowadays. My comment was a bit misleading. What I mean is that there is no universal support for different ARM based boards. Yes, I could start building these packages for ARM on a Raspberry Pi but that does not mean that the packages are compatible with other boards.
Therefore, it does not make any sense for this repository to start supporting one product line (RasPi) as there are many different ARM based boards. For example almost all mobile phones which are running Android are ARM based (!). It would be cool to run pip install opencv-python on my Sailfish OS phone, but that's not just yet possible with the current state of Python package management. Please see https://github.com/pypa/manylinux/issues/84.
However, thanks to @bennuttall's and @waveform80's great work, you can use pre-built wheels on Raspberry Pi's. For more information, see https://www.piwheels.org/.

I'm in the same situation described before. After installing opencv-contrib-python-<IP_ADDRESS> using piwheels and importing cv2 in python3 it shows:
ImportError: libhdf5_serial.so.100: cannot open shared object file: No such file or directory
I have try to reinstall libhdf5 and libhdf5_serial but it didn't resolved the problem. I'm using Ubuntu Mate in RPI. Any insights? Thanks

I don't maintain Raspberry Pi builds. Please open an issue to https://github.com/bennuttall/piwheels.

Can Travis build for ARM?

According to https://github.com/travis-ci/travis-ci/issues/2790, only cross-compiling to Android is available out of the box.
https://github.com/Pi4J/pi4j/blob/master/.travis.yml is an example showing it's possible to cross-compile for Pi with a downloaded "rpi-tools" package.
As per Running Travis CI tests on ARM architecture | tomaz.me, it's also possible to run tests with qemu.
Finally, https://bennuttall.com/piwheels-building-a-faster-python-package-repository-for-raspberry-pi-users/ and https://www.piwheels.hostedpi.com/ say that the piwheels project compile everything from PyPI (though, if the latter is correct, only for py3.4 and py3.5 -- looks strange, maybe the info is outdated). So all we need to do is provide an sdist that they can run pip wheel <pkg> on.

Finally, https://bennuttall.com/piwheels-building-a-faster-python-package-repository-for-raspberry-pi-users/ and https://www.piwheels.hostedpi.com/ say that the piwheels project compile everything from PyPI (though, if the latter is correct, only for py3.4 and py3.5 -- looks strange, maybe the info is outdated). So all we need to do is provide an sdist that they can run pip wheel  on.

We attempt to build everything from PyPI (and keep everything that succeeds :). As OpenCV is a little more tricky (as you noted there's no sdist that works "out of the box") we handle that separately; specifically I watch this repo and when @skvark makes a new release I build them on my Pis and import them into piwheels when they're done (usually a couple of days later!). Tensorflow is also handled separately (specifically, the maintainers send Ben the pre-built wheels and he imports them).
We only handle py34 and py35 at the moment because we (currently) only support Raspbian Jessie (py34) and Raspbian Stretch (py35). At some point, when I've got a spare couple of weekends, we'll add py27 to that mix but it's not a trivial addition (the build queue logic is already fairly complex and multiple major versions makes things even more complex).
We probably won't be supporting anything more recent until Raspbian Buster makes an appearance (there was a vague attempt at supporting py36 via Ubuntu images on the builders, but that just caused more trouble than it was worth because some py36 users had built their own on Raspbian, some were using Ubuntu, and thus each had different lib versions installed so we're sticking strictly to Raspbian for now).

Anyway, as noted above - if you've any issues with the piwheels builds, do feel free to open issues at https://github.com/bennuttall/piwheels

To resolve the "cannot open shared object file" error, see https://blog.piwheels.org/how-to-work-out-the-missing-dependencies-for-a-python-package/

Travis now supports natively building for the ARM architecture: https://blog.travis-ci.com/2019-10-07-multi-cpu-architecture-support
It's currently in alpha so be aware of possible problems.

In particular, no language: python support at the moment (so you have to build/install python yourself and such).

I wonder, why not to use cross compilation?

@skvark There is an option to get dedicated Arm servers via the WorksOnArm project. Please see the https://github.com/WorksOnArm/cluster/issues/209 for details. Tagging @vielmetti for visibility.

There has also been some progress in getting wheels built for some Python projects for arm64 - notably this report from numpy at https://twitter.com/pypackages/status/1274444742621888513 .
There's a couple of options through the Works on Arm effort mentioned above:

shared CI on Drone Cloud w/native arm64 support
shared CI on Travis CI w/native arm64 support
dedicated CI on a dedicated machine directly through Works on Arm

the preference for dedicated systems is for projects that have some compute density needed - ideally you'd want to build more than one package, or be working on a system that is a critical piece not yet ported but that lots of other things depend on, etc. Much of the time, Drone or Travis is a more effective way of getting CI done than dedicated systems.

Hello, I guess this entire thread is yet focused on embedded devices such as raspberry pi however is there any further plan to support ARM64 architecture and corresponding wheels since there is no wheel compiled for macOS / arm64? Thanks!

Please use the search function, there is a dedicated issue for arm64.

The latest release includes experimental arm64 (aarch64) Linux wheels. Older arm variants will not be supported. Please use PiWheels or compile from source if you need armv7 / armv6 wheels.
Linux arm64 (aarch64) tracking issue: https://github.com/skvark/opencv-python/issues/366
Apple Silicon arm64 tracking issues: https://github.com/skvark/opencv-python/issues/429

The latest release includes experimental arm64 (aarch64) Linux wheels. Older arm variants will not be supported. Please use PiWheels or compile from source if you need armv7 / armv6 wheels.
Linux arm64 (aarch64) tracking issue: https://github.com/skvark/opencv-python/issues/366
Apple Silicon arm64 tracking issues: https://github.com/skvark/opencv-python/issues/429


add focusAfterSuggestionSelected optional prop, default true

I'm running into a problem on mobile devices where I want to blur the input after a selection, mainly so that the device's on screen keyboard will hide after an option is picked.
If I blur in the onSuggestionSelected event callback handler, the findDOMNode(this.refs.input).focus(); in onSuggestionMouseDown ends up focusing the input again.
Right now I've resorted to using a timeout in onSuggestionSelected to blur the input. I'd prefer a simple option to control the behavior.

I have the same problem. Thanks @hughlomas for your PR

@hughlomas Sorry it took me so long to get back to you. It totally make sense to hide the keyboard on mobile devices after selection. The only issue I can see with this approach is that it will affect desktop users as well, i.e. if you set focusAfterSuggestionSelected={false}, desktop users won't get the focus after selection. Does this mean we need to handle mobile-only behaviour separately? What are your thoughts on this? cc @r3to

hi @moroshko! In my use case, I want to remove the focus on desktop and mobile devices after the user has selected one of the suggestions. I guess it would be more intuitive to just have a focusAfterSuggestionSelected option. So, if the behavior on mobile and desktop should differ, this could be easily changed per configuration: focusAfterSuggestionSelected={isDesktop}

@r3to It all makes sense. I'm just curious:

why are you not interested in having the focus back on desktop when suggestion is selected?
how would you implement isDesktop?

@moroshko

I'm building a meteorological site, where the Input field for the location also acts as the title for the current forecast. If a valid location is entered/selected, the input field will be styled slightly different, than when it's foucssed. I'll send you a link to the website, once we go live :-)
I'd probably use a user agent sniffing method to define isDesktop, knowing that this isn't a very stable and future-proof way. I'd probably use https://github.com/kaimallea/isMobile for that (haven't used it yet). I think isDesktop shouldn't be defined by react-autosuggest, but by the component, that uses `Autosuggest``

oh, and btw: thanks for your work!

@r3to Thanks!!
@hughlomas Could you please resolve the conflicts so that I could merge this PR?

@hughlomas @r3to I just added a focusInputOnSuggestionClick prop to 3.0.0-beta.2. You are welcome to upgrade to 3.0:
npm install<EMAIL_ADDRESS>--save

Sorry I missed your messages, @moroshko, I am not a super active github user and it had been a while; I forgot about this. Thank you for merging it in!


Deprecate requirement for Jaeger Thrift exporter

With the ingest now supporting OTLP over HTTP, it doesn't make much sense to keep Jaeger Thrift exporter as a required exporter.
Jaeger Client repositories themselves are unmaintained, deprecated and a few of the OpenTelemetry libraries still depend on them (e.g. Node.js). This might produce warnings from vulnerability checks without having a fast way to fix the issues.
The proposal is to lower the wording from MUST offer to MAY offer.

👍 It will reduce amount of work for Splunj OTel .NET AutoInstrumentation. By default we will have otlp over http probably.
FYI: @signalfx/gdi-dotnet-approvers

Would be great if it could be dropped but using "MAY" will lead to conconsist UX across Splunk Otel SDKs. I think we should either drop it from all libs or none. We could opt for dropping it from all but allowing users to install it manually as an additional package if they need.

We could use MAY offer, but add an extra condition that the distro MUST mention that this exporter is deprecated; preferably using a log statement.

The scenario I have in mind is someone instrumenting multiple services written in different languages and being surprised that Jaeger works for one but not for the other. They'd eventually find the deprecation statement in docs but still doesn't eliminate the potentially bad UX. Either way, I'll leave it to actual GDI members to decide and go back into my infra cave :)

Agree, ideally I'd strip any mentions to it completely :smile:

But this brings in the question that if we are getting rid of parts of the spec, whether we should have guidelines for distro authors when something should give a deprecation warning and when it's OK to remove something completely.
For example with splunk-otel-js we want to do a 2.0 release soon, it would be great chance to incorporate breaking changes like this.

Personally, I suggest doing it in two phases.
First for GDI spec version 1.x: MAY offer + deprecation notice
Second for GDI spec version 2.x: remove it 😉 The instrumentation libraries would have to bump the major version as well.
Side note: As part of spec version 2.x we could also remove SPLUNK_METRICS_* env vars in favor of OTel env vars.

@seemk Do we want to use this issue to track deprecating jeager-thrift-splunk in our distros?
@mzajacsplunk FYI

@seemk Do we want to use this issue to track deprecating jeager-thrift-splunk in our distros?

We could make 1.5 release of the spec, and then distros will just have to implement it to conform to the new version. And then we just close this issue.

@MrAlias Are the issues with milestone 1.5 required for v1.5.0?

@MrAlias Are the issues with milestone 1.5 required for v1.5.0?

They can be bumped to the next release if need be.

I will try to create a new release before end of next week and create issues for spec compliance in directly our repositories (e.g. https://github.com/signalfx/splunk-otel-java). Closing this one.


support adding/maintaining dependencies with user defined node_modules

Currently quarkus-web-bundler allows adding dependencies through either mvnpm or webjars.
Please provide another option by using npm directly i.e. the user should be able to add a dependency through npm install. Maybe it can work like this:
Add a new config in application.properties, e.g.:

option:

// scans the node_modules directory
// provided by the user in the given directory
quarkus.web-bundler.dependencies.external.node-modules
=web/app/node_modules

option:

// When quarkus-web-bundler creates its package.json,
// it also merges the user-supplied package.json dependencies,
// e.g. only the content of "dependencies" and "save-dependencies" are merged":
quarkus.web-bundler.dependencies.external.package-json
=web/app/package.json

What would you benefit from using npm directly? I think if you need this, you should use Quinoa


Fixed RAPID routing model time step.

Fixed RAPID routing model time step in FOC files.
THESE CHANGES NEED TO BE PROPOGATED TO MASTER.
Thanks to Yeosang Yoon for finding the error.

Looks great! Thanks @emkemp


LAL-1.2

This license has several names
Licence Art Libre, Lizenz Freie Kunst, Free Art License

v1.3 has been endorsed by FSF: https://directory.fsf.org/wiki/License:LAL-1.3
→ goes to whitelist then


Allow the user to specify the baseline log file

Currently the baseline log file is determined automatically by the parser, based on the version of the parsed log and the available baseline logs.
To implement this the following is needed:

Adding an optional parameter to the parser's command line arguments.
Verifying that the specified file exists and is a valid log file
A baseline should only contain options for the default cf. Handle files that are not compliant (ignore other cf-s? Reject? Other?)
Write unit tests to test this capability

@Yuval-Ariel - Please update the description above and add motivation / use case for this feature request. Thanks

i've updated the description. The expected behavior regarding comparing a baseline LOG file which has multiple cfs is indeed problematic.
i think the expected behavior should be to compare cfs which similar names if they exist and leave out the rest. e.g. baseline has default, cf_1 cf_2 and the LOG parsed has default and cf_1. so compare default and cf_1 and leave out cf_2 from the baseline diff.

@udi-speedb , the way it currently works is also fine


[W socket.cpp:663] [c10d] The client socket has failed to connect to [::ffff:<IP_ADDRESS>]:29500 (errno: 110 - Connection timed out).

Reminder

[X] I have read the README and searched the existing issues.

Reproduction
#!/bin/bash
NPROC_PER_NODE=4
NNODES=1
RANK=0
MASTER_ADDR=<IP_ADDRESS>
MASTER_PORT=29500
CUDA_VISIBLE_DEVICES=0,1,2,3 python -m torch.distributed.run
--nproc_per_node $NPROC_PER_NODE
--nnodes $NNODES
--node_rank $RANK
--master_addr $MASTER_ADDR
--master_port $MASTER_PORT
src/train.py examples/full_multi_gpu/llama3_full_sft.yaml
Expected behavior
full lora
System Info
No response
Others
No response

you need to configure correct master addr and port


Update helm chart install commands in chart README

With the new OCI images we need something like this: helm upgrade --install --wait vault oci://ghcr.io/bank-vaults/helm-charts/vault. This won't work with the current version however, we need to upgrade the unsealer image tag to 1.20.4 and tag a new version of the chart!

https://github.com/bank-vaults/vault-helm-chart/blob/main/vault/README.md


Gradient color map needed for continuous variables

Notes from 4/4/2022 EDA UX meeting:
Categorical variables will use a discrete color map
Continuous variables will use a discrete gradient color map
Default # of bins for continuous variables = 8. This matches the max number of bins allowed for categorical data
We will allow users to change the total number of bins
We don't need to define a maximum number of bins for continuous variables. If the user chooses to have a very large number of bins, the color map will look like a continuous gradient & legend may get annoyingly long but this is OK

I've removed the "map" label as this ticket is to create a gradient color map with can be used broadly in different viz tools. I will create another ticket specific to using continuous "Main" variables on the map viz tool

Addressed with web-components PR
Nothing for outreach to QA - this ticket was addressed by adding colormaps to web-components that can be accessed by the developers.


Help needed for Folder structure: Perception Module Apollo 5.0

Hello,
Can someone guide me to understand the folder structure of Perception module of Apollo 5.0?
Especially the folder structure of perception/camera/
I find a lot of folders(like for example: lib, testdata, camera.......) but not sure of which is used for which purpose? Is it mentioned somewhere. It is very difficult for me to understand :(
Please guide!!!!!

The general introduction is as follows.
.
├── app        \\  main entrance
├── common     \\ common program
├── lib         \\ Library, used for traffic lights, obstacle detection and other functions
├── test        \\ test case
└── tools       \\ Tool for displaying the recognition results of lane lines and traffic lights

If you want to introduce in more detail, you can refer to link.  This document is still being improved.

Thank you so much @daohu527
Where should I refer to the main code for traffic lights, obstacle detection and lane detection? In App or Lib?
Because I see codes in both these folders.

In "modules/perception/onboard/component", there are
modules/perception/onboard/component/trafficlights_perception_component.h
modules/perception/onboard/component/detection_component.h
modules/perception/onboard/component/lane_detection_component.h

Thank you again @daohu527
To be more specific, if modules/perception/onboard/component/trafficlights_perception_component.h is the main code, then what is _https://github.com/ApolloAuto/apollo/tree/r5.0.0/modules/perception/camera/lib/traffic_light_  ?
This folder also contains core components of implementations divided into folders like preprocessor, detector, and tracker.

If you have many steps, you will eventually need an entrance. Reading the code will help you understand it.

Hello @daohu527
I did not understand your answer. Can you explain?

@weidezhang not sure how you got unassigned :(

@daohu527 is right，modules/perception/onboard/component/ is entrance of a module, for example traffic lights, obstacle detection.modules/perception/camera/lib/traffic_light is specific functions to implement. Reading code will help more.

@panfengsu Which means "modules/perception/camera/lib/traffic_light" contains the core elements of the implementation?
What does entrance of a module mean?
I am new to Apollo, so sorry for silly questions.

an interface of a module

ok @panfengsu
If I have to change the configurations like "min_crop_size", "crop_method", "crop_scale" ,"is_bgr" and test the traffic light detection module, then I need to change in this file  right?

apollo/modules/perception/production/data/perception/camera/models/traffic_light_detection/detection.pt

Thank you :)
Testdata folder contains the configurations and data for the test cases in camera/test/ folder right?

Testdata is use for function test
camera/test is use for unit test

ok but I am confused why a code in camera/test has reference to Testdata
For eg below:
init_options.conf_file = "config.pt";
init_options.root_dir =
"/apollo/modules/perception/testdata/"
"camera/lib/traffic_light/detector/recognition/data/";
What is it fetching from Testdata?
Also, where the test cases in testdata folder? I can only see the conf and data defined which is used by the code in camera/test folder.

You are right, camera/test is used for functional test and reads models and parameters from Testdata. Actually I haven't run the test case, you can try it first.
Traffic light detection uses a deep learning model, you can first understand the relevant information.

ok thanks for the clarification.
Is there a procedure to run the test cases? I don't know how to do it


Allow one to bound the size of output shards when writing to files.

This fixes #22129 by possibly writing multiple shards per bundle.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @abacn

Unrelated failure in apache_beam.runners.portability.fn_api_runner.fn_runner_test.FnApiRunnerTestWithGrpcAndMultiWorkers.test_pardo_large_input

Run Python PreCommit

Run Python PreCommit

Hi ! is this available for WriteToParquet as well ? I couldn't find a way to do it with Parquet files

This has not been plumbed through for WriteToParquet, but the underlying infrastructure should work there (see the changes to sdks/python/apache_beam/io/textio.py -- just adding arguments and passing them through). I'd be happy to help you with a pull request.


Ability to combine version & publish commands together for protected branches

Problem
Hello there
In monorepos with main branch protected with pull request necessity, I think it is better to only add changesets to the pull request (summary & bump types can be reviewed by contributors also) & finally by merging to the main branch, changesets github action would "version & publish" to the registry itself.
Right now, as I inspected the whole doc and API, after merging to the main branch, again another PR is created by the action & by merging the subsequent PR, publishing would happen. I think the PR creation by the action in this scenario is redundant completely.
It would be great if we could config the action in a way so that "version & publish" commands are run one after another.

I've seen a number of issues that I think relate to your vision, like https://github.com/changesets/action/issues/77. I'll try follow up and collate them all.
The issue with having "version & publish" happening in the same step is the need for the bump commit (package versions being updated) by Changesets. To support a workflow without making a commit, we'd need to add an option and logic to only track the Package versions by git tags only. Package versions in package.json files would not be updated.

Yeah, the main problem here is that you need to bump versions and remove the changeset files from git - both of those usually require an additional commit to be created. If that would happen whenever you merge something to your base branch then it would become quite cumbersome to work with and more prone to race conditions as people are able to branch off your base at any point in time. However, I think that I've seen people using such custom workflows with Changesets - so it's definitely possible.
While for some projects that extra PR might be redundant - for other projects it's quite nice as "batching" multiple PRs into one is desirable. Imagine that you'd have two concurrent PRs introducing major changesets for package A. Your intention is probably to release a single major version of your package and not two.

@Andarist
I don't think race condition might happen in my use case, because "versioning & publishing" is a kind of task that is tracked by one person at a time, and it is done rarely in a month.
But the problem of additional commit is rational. I think that it would be more clear if I summarize my own workflow in my CI:
1- In my repository, direct push to the main branch is impossible. The branch is protected with pull request necessity.
2- Every time we create a PR, and push to it, some validation workflows run (tests, type checking, yarn dedupe check).
3- At the end of a PR, if we want to also make a publish, we create changesets and let the bump type & summary become reviewed by other developers.
4- When review is completed, the PR developer should call version command on changeset so that package.json & CHANGELOGS are updated.
5- At this moment, after merging to the main branch is done, a telegram message is sent via a bot with info about releases (The info is not available right now!).
I changed my mind and think that the version should be manually run by the developer, but it would be great that the bot had some config to not let PRs with changesets merge to the main branch, so that the developer doesn't forget to run the version command.
About step 5, I wanted to suggest that a release object in publishedPackages output of the action containing the summary, tag & title would come in handy. I would rather take the PR if possible.

The idea of making temporary markdown files (so-called changesets) for saving bump types and summaries before versioning is also great, and in my use case makes commenting and reviewing easier.
But in my opinion, these md files are temporary and I don't like them to be present in my commit history.
That's why I want the config mentioned above for the bot to avoid PRs with changesets to be merged to the main branch.

I changed my mind and think that the version command should be manually run by the developer, but it would be great that the bot had some config to not let PRs with changesets merge to the main branch, so that the developer doesn't forget to run the version command.

This probably would be doable. However, I also wonder - could we automate running changeset version as some kind of pre-merge hook? Does GitHub support executing git hooks when landing PRs using their UI/API?

About step 5, I wanted to suggest that a release object in publishedPackages output of the action containing the summary & bump type would come in handy. I would rather take the PR if possible.

I don't think that including the summary would be OK here - there are certainly character limits when it comes to action outputs and this would risk exceeding those limits. The bump type probably could be added though.

The idea of making temporary markdown files (so-called changesets) for saving bump types and summaries before versioning is also great, and in my use case makes commenting and reviewing easier.

Yeah, this is IMHO one of the main appeals of Changesets.

My vote would simply be a flag in the changeset config that auto merges the PR (say automerge: true). This way you still have the PR with the needed changes and removal of changeset file without the developer having to manually click approve.

you need to bump versions and remove the changeset files from git - both of those usually require an additional commit to be created

Thinking a little more abstract, do the changesets need to be cleaned up?
We could use git tags and commit history to determine newer changesets since the last release. This would also remove the need for any commits. Devs could periodically clean up old changeset files for hygiene, but not strictly necessary 🤔

Whew, took far too long to figure this out, but I have a solution for those who want to auto-merge their Changset "Version Packages" PRs like our team does:
.github/workflows/auto-merge.yml:
name: Auto Merge

on:
  workflow_run:
    workflows: ["Release"] # the name of your release action
    types:
      - completed

jobs:
  release:
    # prevents this action from running on forks
    if: github.repository == 'your-org/your-repo'
    name: Release
    timeout-minutes: 15
    runs-on: ubuntu-latest
    steps:
      - name: "Merge pull request"
        uses: "actions/github-script@v6"
        with:
          script: |
            const owner = "your-org";
            const repo = "your-repo";
            const head = "changeset-release/main";
            await new Promise(r => setTimeout(r, 2000));
            const { data } = await github.rest.pulls.list({ owner, repo, head, per_page: 1 });
            if (data.length) {
              const pull_number = data[0].number;
              await github.rest.pulls.merge({ owner, repo, pull_number, merge_method: "squash"});
            }

This will immediately merge your changset PR so you can do one-push publishing of packages

Ok, I've finally managed to get this working. It's a bit hacky but seems reliable:
name: Release

on:
  push:
    branches: main

concurrency:
  group: ${{ github.workflow }}-${{ github.ref }}
  cancel-in-progress: true

jobs:
  release:
    # prevents this action from running on forks
    if: github.repository == 'your-org/your-repo'
    name: Release
    timeout-minutes: 15
    runs-on: ubuntu-latest
    steps:
      - name: Checkout Repo
        uses: actions/checkout@v3
        with:
          # This makes Actions fetch all Git history so that Changesets can generate
          # changelogs with the correct commits
          fetch-depth: 0
      - name: Setup Node.js
        uses: actions/setup-node@v3
        with:
          node-version: 17.x
          cache: "npm"
      - name: Install dependencies
        run: npm install --frozen-lockfile
      - name: Build
        run: npm run build
      - name: Create release pull request
        id: changesets
        uses: changesets/action@v1
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
          NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
      - name: Merge pull request
        if: steps.changesets.outputs.pullRequestNumber > 0
        uses: actions/github-script@v6
        with:
          script: |
            const owner = "your-org";
            const repo = "your-repo";
            const head = "changeset-release/main";
            const pull_number = "${{ steps.changesets.outputs.pullRequestNumber }}"
            await github.rest.pulls.merge({ owner, repo, pull_number, merge_method: "squash"});
      - name: Publish to npm
        if: steps.changesets.outputs.pullRequestNumber > 0
        uses: changesets/action@v1
        with:
          publish: npm run release
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
          NPM_TOKEN: ${{ secrets.NPM_TOKEN }}


ACM-2463 created mce-gather script to sync with acm's must-gather repo

Signed-off-by: Erin Murphy<EMAIL_ADDRESS>

@eemurphy Can you share instructions for how to build and run the must-gather image to validate? I'm basically trying the following and having issues
docker build -t quay.io/jakobgray/backplane-must-gather:dev .
docker push quay.io/jakobgray/backplane-must-gather:dev
oc adm must-gather --image=quay.io/jakobgray/backplane-must-gather:dev

I get this empty error:
Error running must-gather collection:
    gather never finished for pod must-gather-bbjlx: openshift-must-gather-wrbjf/must-gather-bbjlx unexpectedly terminated: exit code: 1, reason: Error, message:


Union of @boundary Types

I can't find details about Union support: neither documented nor reported as unsupported.
But I can't make it work:
Graph1
type Album @boundary {
  id: ID!
}

type Artist @boundary {
  id: ID!
}

union CatalogEntity = Album | Artist

type FeedCard {
  type: String!
  catalogEntity: CatalogEntity
}

type Query {
  (...)
  feed: [FeedCard]
}

Graph2
type Album @boundary {
  id: ID!
  title: String!
}

type Artist @boundary {
  id: ID!
  name: String!
}

When I query with
feed {
    type
    catalogEntity {
      __typename
      ... on Artist {
        id
        name
      }
      ... on Album {
        id
        title
      }

    }
  }
}

I receive
"message": "Cannot query field \"title\" on type \"Artist\"".

Did I miss something? maybe declare the Union in graph 2 but how?

Hi @pasdeloup, at this stage we don't support @boundary on union types. It is a little hidden but mentioned here

Thanks @lucianjon , sorry I really missed that FAQ indeed!


Added viewer privacy notice with links to /from ToU.

Purpose
To Add Viewer Privacy notice
Fixes UML-780
Approach

new handler
twig file with a privacy notice
routing
behat tests

Learning
GDS styling: https://design-system.service.gov.uk/styles/
Checklist

[X] I have performed a self-review of my own code
[ ] I have added relevant logging with appropriate levels to my code
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[X] I have added tests to prove my work
[ ] The product team have tested these changes

Coverage decreased (-0.03%) to 80.788% when pulling 2c213dc00908503e58665efc2eca9bb07adc5de3 on UML-780-Viewer-Privacy-notice into d0adbc7f51e68373a24d3c555aaae05155f065ea on master.


Allow kernel argument to take custom struct by value

By specializing template<T> struct set_kernel_arg when expanding BOOST_COMPUTE_ADAPT_STRUCT, kernel input argument can now take custom struct by value. This potentially utilizes constant memory of the compute device

Coverage remained the same at 84.019% when pulling e1b96b36457997087952df710ceb722ff4ca5e8f on rosenrodt:pr-kernel-pass-struct-by-value into 36c89134d4013b2e5e45bc55656a18bd6141995a on boostorg:master.

It looks like Apple platform regards custom struct as invalid kernel argument. That's weird because my 2016 Macbook works just fine


Error in compilation: SG_CONSTRAINT

I'm trying to compile the latest version (develop) and shows this error:
shogun/src/shogun/preprocessor/RFFPreprocessor.cpp:26:17: error: ‘castable’ was not declared in this scope
26 |   SG_CONSTRAINT(castable()));

@javiermaldonadoc could you give the complete error message?

i've solve this issue after erase all my local copy and reinit the local repo. I think my local copy was broken somehow.... Thanks!


fetchgit fetches submodules sequentially instead of in parallel

Describe the bug
fetchgit clones each submodule in sequence instead of in parallel, which is unnecessarily slow for repos that have a lot of submodules (such as bat which has 64 direct submodules).
At first I thought this was because git's submodule.fetchJobs setting defaults to 1, but it turns out fetchgit doesn't even use that, it instead recursively clones the submodules by hand. @nbp it looks like you added this 10 years ago; is there any reason we can't just use git submodule update --init --recursive today?
I don't know why it doesn't just use git's built-in submodule handling, doing so would make fixing this trivial. Parallelizing the manual clone loop seems a lot more complicated (especially because of the problem of mixed output).
To Reproduce
Steps to reproduce the behavior:

Build bat.src from scratch. (Edit: bat was recently updated to not use submodules anymore. This can be reproduced using an older version)

Expected behavior
It should clone the repo in a reasonable amount of time.
Notify maintainers
Last few changes to fetchgit/nix-prefetch-git were @Mic92 and @bhipple
Metadata

system: "x86_64-darwin"
host os: Darwin 19.4.0, macOS 10.15.4
multi-user?: no
sandbox: no
version: nix-env (Nix) 2.3.4
channels(lily): "nixpkgs-20.09pre226586.571212eb839"
nixpkgs: /Users/lily/.nix-defexpr/channels/nixpkgs

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute: fetchgit
# a list of nixos modules affected by the problem
module:

See #238467

Simplified in https://github.com/NixOS/nixpkgs/pull/238733


About results on CASIA V1

Very happy to see your work!
I  have tested your code on casia v1 and gotten dice coeff only 0.642 today? May I ask the best model.pth you provided is it able to reach 0.73 of the paper?
Noting that I have used the casia v1 from：https://github.com/namtpham/casia1groundtruth with both tamper images and gt，and the model is the best model you provided, and the batch size is 1.
Thank you very much if you can answer my questions！

In our paper, 0.73 is F_1, not dice coeff.

In our paper, 0.73 is F_1, not dice coeff.
I used the code to comput f_1,but i got the same value as dice coeff. And it is found that the calculation process of f1 and dice coeff is the same. what‘s wrong with that？ I am confused now，or can you provide the code you compute F_1? Thanks!!!

This work was completed a year ago, and some details of the code can no longer be found.
But, I can give you some advices.
You try to convert the output of model to binary result, f_1 may be better.

This work was completed a year ago, and some details of the code can no longer be found. But, I can give you some advices. You try to convert the output of model to binary result, f_1 may be better.

Thanks. I recently found a problem, when I print the true_masks label, I find that there are decimals (float)? For an image segmentation problem, should the mask just contain 0, 1 labels? Did you do some operaations to true_mask, or need to binarize true_masks? Thanks again!

Which dataset does true_mask come from?

Which dataset does true_mask come from?

It comes from the SAN_8 you provided. And i have tried many ways to compute F1, its value is always below the value you mentioned.

Sorry to bother you again, recently, I trained your network from scratch by using SAN_8w, after 45 epochs. I only got 60% F1 when val dataset is casiav1.  I wonder how many epochs you set as your aticle does not mention it.

It's weird. In my experiments, i use total training epochs 45.
What about trying to increase the training epoch?

Why did I get a f1 score of 0.4 with the author's weight and 0.2 with my own training weight? Can I see your code for testing f1 scores