WIP: migrate Census to Airtable

deets later, just need deploy

@rperlin-ela if you have a minute, once this deploys give the census stuff a test. nothing new, but i've done the first steps of migrating to Airtable (waaaay easier to work with than Sheets). it's using the "Census" table i created in AT. once we get all this prepped and merged, i can delete the puma and tract lookup tables and all the fields pointing to them.
next step will be to use the actual census data from Airtable. the current deploy is still hitting Sheets.

@rperlin-ela if you have a minute, once this deploys give the census stuff a test. nothing new, but i've done the first steps of migrating to Airtable (waaaay easier to work with than Sheets). it's using the "Census" table i created in AT. once we get all this prepped and merged, i can delete the puma and tract lookup tables and all the fields pointing to them.
next step will be to use the actual census data from Airtable. the current deploy is still hitting Sheets.

Sorry, just got to this, but looking fine

On Jan 6, 2021, at 2:27 PM, Jason Lampel<EMAIL_ADDRESS>wrote:
@rperlin-ela https://github.com/rperlin-ela if you have a minute, once this deploys give the census stuff a test. nothing new, but i've done the first steps of migrating to Airtable (waaaay easier to work with than Sheets). it's using the "Census" table i created in AT. once we get all this prepped and merged, i can delete the puma and tract lookup tables and all the fields pointing to them.
next step will be to use the actual census data from Airtable. the current deploy is still hitting Sheets.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/Language-Mapping/language-map/pull/154#issuecomment-755559170, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMNKB5G7HX6TQWM5LTI3N4DSYS2R7ANCNFSM4VX6276Q.

Sorry, just got to this, but looking fine

On Jan 6, 2021, at 2:27 PM, Jason Lampel<EMAIL_ADDRESS>wrote:
@rperlin-ela https://github.com/rperlin-ela if you have a minute, once this deploys give the census stuff a test. nothing new, but i've done the first steps of migrating to Airtable (waaaay easier to work with than Sheets). it's using the "Census" table i created in AT. once we get all this prepped and merged, i can delete the puma and tract lookup tables and all the fields pointing to them.
next step will be to use the actual census data from Airtable. the current deploy is still hitting Sheets.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/Language-Mapping/language-map/pull/154#issuecomment-755559170, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMNKB5G7HX6TQWM5LTI3N4DSYS2R7ANCNFSM4VX6276Q.

ok the upcoming deploy in a few will be Sheets-free. i'm hitting the font stuff (for endo labels in map) and census data from Airtable now, so you'll want to test those.
the tricky part was getting it to load without delay since AT has a 100 records per second limit, so for the tracts data it was like 4 seconds for the first load by default. however, i didn't want to add more to the map stuff on load since there's A LOT going on, so my loose hack was to load it just-in-case user does census stuff in:

explore
details
or local (obviously)

it's basically loading behind the scenes, but it would be nice if didn't have to load at all until user interacts w/the census stuff.
anyhoo, i'm ready to merge once you've greenlighted, then we can say adios to Google Sheets.

ok the upcoming deploy in a few will be Sheets-free. i'm hitting the font stuff (for endo labels in map) and census data from Airtable now, so you'll want to test those.
the tricky part was getting it to load without delay since AT has a 100 records per second limit, so for the tracts data it was like 4 seconds for the first load by default. however, i didn't want to add more to the map stuff on load since there's A LOT going on, so my loose hack was to load it just-in-case user does census stuff in:

explore
details
or local (obviously)

it's basically loading behind the scenes, but it would be nice if didn't have to load at all until user interacts w/the census stuff.
anyhoo, i'm ready to merge once you've greenlighted, then we can say adios to Google Sheets.

Poking around and everything so far is looking all good to me, so green light from here. Sheets free!

On Jan 6, 2021, at 5:48 PM, Jason Lampel<EMAIL_ADDRESS>wrote:
ok the upcoming deploy in a few will be Sheets-free. i'm hitting the font stuff (for endo labels in map) and census data from Airtable now, so you'll want to test those.
the tricky part was getting it to load without delay since AT has a 100 records per second limit, so for the tracts data it was like 4 seconds for the first load by default. however, i didn't want to add more to the map stuff on load since there's A LOT going on, so my loose hack was to load it just-in-case user does census stuff in:
explore
details
or local (obviously)
it's basically loading behind the scenes, but it would be nice if didn't have to load at all until user interacts w/the census stuff.
anyhoo, i'm ready to merge once you've greenlighted, then we can say adios to Google Sheets.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/Language-Mapping/language-map/pull/154#issuecomment-755763859, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMNKB5AOZJEYXFH7N4FQGDDSYTSEBANCNFSM4VX6276Q.

Poking around and everything so far is looking all good to me, so green light from here. Sheets free!

On Jan 6, 2021, at 5:48 PM, Jason Lampel<EMAIL_ADDRESS>wrote:
ok the upcoming deploy in a few will be Sheets-free. i'm hitting the font stuff (for endo labels in map) and census data from Airtable now, so you'll want to test those.
the tricky part was getting it to load without delay since AT has a 100 records per second limit, so for the tracts data it was like 4 seconds for the first load by default. however, i didn't want to add more to the map stuff on load since there's A LOT going on, so my loose hack was to load it just-in-case user does census stuff in:
explore
details
or local (obviously)
it's basically loading behind the scenes, but it would be nice if didn't have to load at all until user interacts w/the census stuff.
anyhoo, i'm ready to merge once you've greenlighted, then we can say adios to Google Sheets.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/Language-Mapping/language-map/pull/154#issuecomment-755763859, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMNKB5AOZJEYXFH7N4FQGDDSYTSEBANCNFSM4VX6276Q.

Woo. Merging.
I'll see if I can tackle the two outstanding bugs tonight.

Woo. Merging.
I'll see if I can tackle the two outstanding bugs tonight.


TBI - EXT - Add unit test to ide-extension

Description (include screenshots)
Increase unit test coverage for module sap-guided-answers-extension.
Value
Code quality
Architecture Elaboration
Notes
Tasks

[ ] Add unit tests to increase test coverage

@Klaus-Keller FYI - Do you want to fill out the details?

Done

Has been fixed a while ago:

https://sonarcloud.io/component_measures?metric=coverage&selected=SAP_guided-answers-extension%3Apackages&id=SAP_guided-answers-extension


CR3 Help

while (true) {
BYTE bytes[4] = { 0 };
DWORD i = 0;
if (VMMDLL_VfsReadU(this->vHandle, const_cast("\misc\procinfo\progress_percent.txt"), bytes, 3, &i, 0) == VMMDLL_STATUS_SUCCESS) {
if (atoi(reinterpret_cast(bytes)) == 100) {
break;
}
}
sleep_for(milliseconds(100));
}

When CR3 patching fails, I attempt to repatch it, but every time I start retrieving the data from 0 again, and I can never successfully patch CR3. Some systems behave this way. Normally, on the second attempt, the retrieval should directly start from 100. How should I handle this issue?

Restart pc, it just means it got paged out.

cold restart, and make sure youre restarting your target computer

cold restart, and make sure youre restarting your target computer

It's still not resolved. Could this be related to the system? Windows 10 Professional 22H2 19045.4780 is not working, but Windows 10 Professional 22H2 19045.3803 is working

Probably not. Just page issues.

Its beacause your probably initializing all plugins each time, so progress has to start from 0 again


namespace创建总是会覆盖所有的环境和集群，是否可以做到独立创建 ？

你的特性请求和某个问题有关吗？请描述
在apollo的某个appid下面的某环境某集群下新增一个namespace时候，总是会在这个appid下面的所有环境&所有集群下面创建对应的namespace
清晰简洁地描述一下你希望的解决方案
能否做到在x环境y集群下创建，就只有x环境和y集群有这个namespace
清晰简洁地描述一下这个特性的备选方案
其它背景
在这里添加和这个特性请求有关的背景说明、截图

That would make the namespace management complex. And if a namespace is created but not released, then it is invisible to the client, which can achieve the same effect of not creating it at all.

@nobodyiam thanks for your reply. is it possibly to make the namespace created independent ? We have found that only independent namespace created could solve the probolem we met. Maybe if it is planned to makie it independent or it would violate the apollo design ? Thanks

Would you please help to describe the actual problem you met?


[BUG] invalid context is returned

Describe the bug
When context is retrieved from a callback, the provided context is invalid.
To Reproduce
exampleContext, _ := context.WithCancel(context.Background())
	session := azuretls.NewSessionWithContext(exampleContext)

	session.CallbackWithContext = func(ctx *azuretls.Context) {
		exampleContextFromCtx := ctx.Context()
		if exampleContextFromCtx == exampleContext {
			fmt.Println("match")
		} else {
			fmt.Println("no match")
			session1 := azuretls.NewSessionWithContext(exampleContextFromCtx)
			_, err := session1.Get("https://httpbin.org/headers")

			if err != nil {
				log.Fatal(err)
			}
		}
	}

	_, _ = session.Get("https://httpbin.org/headers")

Expected behavior
The request inside the callback should work, but errors with a "canceled" message, even though context wasn't canceled. This is because an invalid context is being returned.
Screenshots

Desktop (please complete the following information):

OS: Windows
Browser: Chrome
Version: 123

It's patched v1.4.8


(Request) - Specify relative column width in Add-WordTable

When I add a table, I'd like to be able to specify the relative widths of each of the columns, such as:  "30:50:20" for 30%, 50%, 20% of the 1st, 2nd, and 3rd columns.  (Implicitly, I'm assuming "-autofit window" is used)
Thank you!

Please check example 10 and new version 0.4.3. I am not sure if that's what you wanted and whether the expected values work correctly as is.

Awesome! Thank you so much :)
I can confirm that that's what I was trying to describe.
For quick ref for others, I used:
Add-WordTable -WordDocument $WordDocument -DataTable $qa -Design ColorfulGrid -Percentage $true -ColummnWidth 60, 30
Although I was just wondering, what is the meaning of the numbers in the "ColumnWidth" parameter, if I used the flag " -Percentage $false " instead?
Would that be pixels, inches, or centimetres?

I would say it's Pixels.

But ... when I look at what Word shows when you go into propeties in each table in Word10 example I'm really not sure it works as expected. Did you get the 60/30 to be consistent for you? And according to your expectations?

I will test this in the next few days and let you know.


Implemented comparing Naive and DatafrogOpt algorithms.

Note: this pull request is based on my previous pull request.
Changes:

Added a new Algorithm “type” Compare that runs both Naive and DatafrogOpt and compares the reported errors. If they differ, reports differences via logging as error and panics.
I have also added a test for testing the comparison algorithm. However, for some reason, it is not picked up if I run cargo test from the root directory.

Example output:
RUST_LOG=polonius_engine=trace cargo run -- inputs/issue-47680/nll-facts/main -a compare
    Finished dev [unoptimized + debuginfo] target(s) in 0.08s
     Running `target/debug/polonius inputs/issue-47680/nll-facts/main -a compare`
DEBUG 2018-05-29T14:12:21Z: polonius_engine::output: Naive and optimized algorithms reported the same errors.
--------------------------------------------------
Directory: inputs/issue-47680/nll-facts/main
Time: 0.030s
# borrow_live_at

left a comment about the match you're doing but this looks great :)


Isolates and Google-Ml-Kit-plugin

This issue draws over https://github.com/bharat-biradar/Google-Ml-Kit-plugin/issues/86
Considering how amazing this repo is and impact of this issue on its performance, it may be a good idea to explore usage of isolates.
For example let's take a simple example with TextRecognition. You have:

Camera Widget: Many FPS with high resolution according to resolution preset e.g. ResolutionPreset.medium.
Text Recognition by MLKit
Annotations above Camera Widget
Your in-house processing of recognized text

Having all of this run on main isolate sounds scary. This issue is the only mention of isolate within this repo.  This sets main/UI isolate free from being sluggish when doing resource-intensive operations.
I wish this would be discussed further.
For example, for points 2. & 4. above

2.Text Recognition by MLKit
4.Your in-house processing of recognized text

It could be argued that these can happen on different isolates.
In fact 2. is occurring in platform-plugin land (native) So that's probably its own isolate.  Not sure about this though.
For 4., it's probably a good idea to run it on a different isolate only if there's heavy in-house operations on recognized text.
But what about 1. & 3.?

1.Camera View: Many FPS with high resolution according to resolution preset e.g. ResolutionPreset.medium.
3.Annotations above CameraView

For 1. it's actually Camera pub package, which is actually a platform plugin too.
For 3., I think it's okay to create/destruct UI widgets at this high frequency? after all this is what flutter is good at with its generational garbage collector and UI rendering engine.
What are your thoughts on isolate usage? This is not just for performance gains within a feature of the 4 points above, but also to prevent any load on main UI thread/isolate.

@om-ha: this sounds like a great idea, could you send a PR implementing what you propose?
Otherwise we will close this one for inactivity until someone steps in to implement it.

Running the plugin on isolate would take away the flexibility. Developer can always run the functions exposed in the api on different isolate should they prefer it. Hence we are closing this issue.

BTW, I just ran across this open source app that is using our  google_mlkit_barcode_scanning plugin and they are using isolates. Here is a very good example how to do it if you are interested:

https://github.com/openfoodfacts/smooth-app/blob/develop/packages/smooth_app/lib/pages/scan/mkit_scan_helper.dart#L17-L20
https://github.com/openfoodfacts/smooth-app/blob/develop/packages/smooth_app/lib/pages/scan/mkit_scan_helper.dart#L50-L151
https://github.com/openfoodfacts/smooth-app/blob/develop/packages/smooth_app/lib/pages/scan/mkit_scan_helper.dart#L50-L151

For anyone who is coming to this in 2024, those links above don't exist anymore. This is how I was able to implement multiple detectors with Isolates:
class IsolateAIParams {
  SendPort port;
  InputImage image;

  RootIsolateToken token;
  IsolateAIParams(
      {required this.port, required this.image, required this.token});
}

class AIResult {
  final List<DetectedObject> detectedObjects;
  final List<Face> detectedFaces;
  final List<ImageLabel> detectedLabels;
  final RecognizedText detectedText;

  AIResult({
    required this.detectedObjects,
    required this.detectedFaces,
    required this.detectedLabels,
    required this.detectedText,
  });
}

class AIService {
  static final ObjectDetector _objectDetector = ObjectDetector(
      options: ObjectDetectorOptions(
    mode: DetectionMode.stream,
    classifyObjects: true,
    multipleObjects: true,
  ));
  static final FaceDetector _faceDetector = FaceDetector(
    options: FaceDetectorOptions(
      enableContours: true,
      enableLandmarks: true,
    ),
  );
  static final ImageLabeler _imageLabeler =
      ImageLabeler(options: ImageLabelerOptions());
  static final TextRecognizer _textRecognizer = TextRecognizer();

  static Future<AIResult?> performAIRequest(InputImage image) async {
    try {
      final receivePort = ReceivePort();
      final rootToken = RootIsolateToken.instance;
      if (rootToken == null) {
        throw Exception('Root token is null');
      }
      final isolateAIParams = IsolateAIParams(
          port: receivePort.sendPort, image: image, token: rootToken);

      await Isolate.spawn(_performIsolateRequest, isolateAIParams);

      final result = await receivePort.first;

      if (result is AIResult) {
        return result;
      } else {
        return null;
      }
    } catch (e) {
      debugPrint('Error: $e');
      return null;
    }
  }

  static void _performIsolateRequest(IsolateAIParams params) async {
    BackgroundIsolateBinaryMessenger.ensureInitialized(params.token);

    final inputImage = params.image;

    try {
      final results = await Future.wait([
        _objectDetector.processImage(inputImage),
        _faceDetector.processImage(inputImage),
        _imageLabeler.processImage(inputImage),
        _textRecognizer.processImage(inputImage)
      ]);

      if (results.isEmpty) {
        throw Exception('No results');
      }

      final result = AIResult(
        detectedObjects: results[0] as List<DetectedObject>,
        detectedFaces: results[1] as List<Face>,
        detectedLabels: results[2] as List<ImageLabel>,
        detectedText: results[3] as RecognizedText,
      );

      Isolate.exit(params.port, result);
    } catch (e) {
      debugPrint('Error: $e');
      Isolate.exit(params.port, "No results From Isolate");
    }
  }
}

To call it:
final file = File('path')
final inputImage = InputImage.fromFile(file);
final aiResult = await AIService.performAIRequest(inputImage);


Add baseline benchmark for expr evaluator

Add baseline benchmark for expr evluator vs direct go code.
The current implementation is between 40 - 80 x slower than
directly written (or generated) go code
The expr language does not implement OR/AND short circuit
So it will evaluate all conditions for OR even though first condition
is already true.
If you factor in this optimization, then the current implementation
is ~40x slower.

This change is

Jenkins job mixer/presubmit passed

Jenkins job mixer/presubmit passed

Jenkins job mixer/presubmit passed


Add Go

https://github.com/iexuxs-m/AttractingContributors-Foods

I want to keep the template clean, so will not merge this PR


syscall: Go Build Fails on Windows With Long Paths

Opening by request in #3358
What version of Go are you using (go version)?
go version go1.8beta2 windows/amd64

What operating system and processor architecture are you using (go env)?
Win64
What did you do?
Tried to compile several files, many with long names, in a folder via go build
What did you expect to see?
Successful compile
What did you see instead?
go build github.com/cretz/myproj: C:\dls\go1.8\go\pkg\tool\windows_amd64\compile.exe: fork/exec C:\dls\go1.8\go\pkg\tool\windows_amd64\compile.exe: The filename or extension is too long.

My guess is that problem is that the magic we added to the os package doesn't help the syscall package, since syscall (where the exec happens) doesn't use os.
Maybe we need to move the os package's support routines (building the UNC paths) into an internal support package usable by both os and syscall.
/cc @quentinmit

And /cc @alexbrainman

@bradfitz - I think this would be handled by the caller of Exec instead of in syscall directly since they are opaque strings and the difference between a file path and any other arbitrary string is determined by the caller. I do agree that this and the os package should share the same path-fixing code. I also agree it should be an internal function for now until more thought is given to expose (maybe os/exec.OSSafePath but it's really annoying to abstract a Windows -only issue).

Why not have filepath.LongPath() function to be used for these situations?

@harshavardhana If the go std lib handles windows long filepaths, which I think is great, I don't think it would be good to expose it anywhere except (maybe) in syscall, as this is a really platform specific issue where the fix doesn't need to affect API.

Yeah, no new public API.

@harshavardhana If the go std lib handles windows long filepaths, which I think is great, I don't think it would be good to expose it anywhere except (maybe) in syscall, as this is a really platform specific issue where the fix doesn't need to affect API.

:+1:

We very consciously did not make syscall transparently handle long paths, because it would be confusing if the resulting syscall was called with different arguments than the caller intended.
Probably the right thing to do is to teach os/exec to use the same long path handling as os.

Actually, os/exec just uses os.StartProcess. So this is in the os package after all.
I'll send out a fix.

Not trivial, it turns out. Somebody should tackle it during Go 1.9. https://golang.org/cl/34725 might be a good starting point. It has some debugging notes in it and a test.

If I was to guess, the UNC meses with the quote logic; guess only.

Note: #21145, just closed as a dup of this issue, has an handy Powershell script that creates a folder that triggers the problem.

After reviewing #21145 I think the CL that Brad had started ( https://golang.org/cl/34725 ) is barking up the wrong tree. In that CL it attempts to fixup the process name to use the UNC path equivalent that the rest of the os functions can use. However, as noted in the CL, CreateProcess doesn't accept UNC paths for the process name.
Looking at this issue again the process name is short C:\dls\go1.8\go\pkg\tool\windows_amd64\compile.exe and less then 250 chars. This means that no long path fixup would be applied even if it could work.
I suspect that the error is actually referring to the argument length (Windows doesn't really differentiate between the two in error message).
From https://msdn.microsoft.com/en-us/library/windows/desktop/ms682425(v=vs.85).aspx
lpCommandLine [in, out, optional]
The command line to be executed. The maximum length of this string is 32,768 characters, including the Unicode terminating null character. If lpApplicationName is NULL, the module name portion of lpCommandLine is limited to MAX_PATH characters.

Two ways to fix this would be to instrument the Go compilers to take a new argument "-args-on-stdin=true" flag where the exec would read from stdin for args or pass the argument list file to the executable "@C:\Temp\arglist" where the given file is read off of disk and used as arguments.

It would be nice to solve the general problem, but perhaps Windows does not provide a mechanism for that.  If we can only solve the problem for cmd/go, then of course we should do that.
For the record, GCC and friends do this by treating any argument that starts with '@' as a response file.  The file is turned into a sequence of white-space separated strings, and the response file argument is replaced by the resulting set of arguments.  This seems to be a standard Windows concept, documented at https://msdn.microsoft.com/en-us/library/windows/desktop/aa367156(v=vs.85).aspx .

I’m running into this issue and I’d like to contribute a fix.
If the maintainers would like me to teach the toolchain about response files, I’m happy to take a swing at it, but I have some questions:

It seems like it may be worthwhile to teach the flag package how to understand response files.  If that is reasonable, my approach would be when looking for flags, also treat strings that start with @ specially.  Is it reasonable to make that change or would it need to be guarded behind some check to ensure existing invocations which pass strings start start with @ in a position where a flag would be legal continue to work?  It so, I assume adding a new function like AllowResponseFile(bool) is the way to go?
How do you want to do word splitting in the response file? Would we want to support multiple arguments per line in the response file or say that each argument is delimited by a newline.  Regardless of what we choose, how does someone escape a character that would cause word breaking?
If adding this feature to the flag package is unacceptable should I just teach the compiler a new argument like -sourcefiles ?

@ellismg before deciding how to fix this, can you provide me with steps to reproduce it.
Thank you
Alex

@alexbrainman
Sure.  In the case I'm hitting, the problem is just that we construct a command line that is too long (>32K characters) when we try to exec into the compiler.
This works:
C:\Users\Matt G. Ellis\go\src\github.com\ellismg\scratch>dir /B
empty.go
hello.go

C:\Users\Matt G. Ellis\go\src\github.com\ellismg\scratch>type empty.go
package main
C:\Users\Matt G. Ellis\go\src\github.com\ellismg\scratch>type hello.go
package main

import "fmt"

func main() {
    fmt.Printf("hello, world\n")
}
C:\Users\Matt G. Ellis\go\src\github.com\ellismg\scratch>go build

C:\Users\Matt G. Ellis\go\src\github.com\ellismg\scratch>

If you add a bunch of additional files (depending on how deep your source code is, you may need more copies of the dummy file, to construct a command line that is over the windows limit):
C:\Users\Matt G. Ellis\go\src\github.com\ellismg\scratch>for /l %i IN (1,1,2048) DO copy /y empty.go %i.g
o
[...lots of copy spew elided...]

Then go build will fail when trying to run compiler.exe
C:\Users\Matt G. Ellis\go\src\github.com\ellismg\scratch>go build
go build github.com/ellismg/scratch: C:\Go\pkg\tool\windows_amd64\compile.exe: fork/exec C:\Go\pkg\tool\w
indows_amd64\compile.exe: The filename or extension is too long.

Here's my system information:
C:\Users\Matt G. Ellis\go\src\github.com\ellismg\scratch>go version
go version go1.9 windows/amd64

C:\Users\Matt G. Ellis\go\src\github.com\ellismg\scratch>go env
set GOARCH=amd64
set GOBIN=
set GOEXE=.exe
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=C:\Users\Matt G. Ellis\go
set GORACE=
set GOROOT=C:\Go
set GOTOOLDIR=C:\Go\pkg\tool\windows_amd64
set GCCGO=gccgo
set CC=gcc
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0
set CXX=g++
set CGO_ENABLED=1
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config

Locally, I did a hacky thing where I just taught compiler.exe about a -srcfiles argument that would read a response file and then passed every source file via that list.  With that set of changes, I was able to build.
Nothing long path specific here, as @kardianos surmised.  Just generating a very long command that ends up being over a Windows limit.

Something like flag.(*FlagSet).ResponseFiles(bool) makes sense to me, with flag.ResponseFiles calling that method on CommandLine.
The response files should be parsed exactly as the Microsoft tools parse them.  I believe they permit multiple options on a single line, and permit quoting with backslash, single quote, and double quote.

the problem is just that we construct a command line that is too long (>32K characters)

@ellismg thank you for providing the instructions. I can confirm we have command line that is longer than 32K characters.
I do not see any other solution but feeding command line in a "response file". I just wander, maybe we should do it automatically. go command can measure length of a command line and use "response file" if it is longer than 32K. Than users don't even need to know about the "response file". Is that what you are proposing to do @ellismg ?
Alex

Yes, that is my plan.  My hope is to start working on the patches early next week.

@ellismg Just so it is explicit, do you plan to focus changes to accept a "response file" in cmd/{go,compile} or flag or both?

@kardianos.  My plan was to teach flag about response files (as something you could opt into) and then opt cmd/{go,compile} into supporting response files and change the part of cmd/go to use a response file when invoking cmd/compile if the command would be too long otherwise.
I’m happy to take whatever approach you all want, I just care about being able to build projects with lots of .go files in them on windows.

Will there be a function that takes an argument list ([]string) and converts it to a response file io.Reader? If so, where would that live? Again, not too worried, but I think it might be good to flesh out a few ideas on the issue tracker so if there are objections / design differences we can catch them before all the code is written.

@kardianos As someone who is still very much a go neophyte I appreciate catching issues early.   Should I share that here or is there some better place to track issues of this form?

Track it here for now. Or you could create a new issue with something like "proposal: add response file parsing to flag package" and reference this issue in it. But here is fine.

any solution to this ?  It is currently un-usable

@davelatimer, not yet.

Got same issue with Golang 1.10
baliance.com/gooxml/schema/soo/sml
go build baliance.com/gooxml/schema/soo/sml: D:\devtools\Go\pkg\tool\windows_amd64\compile.exe: fork/exec D:\devtools\Go\pkg\tool\windows_amd64\compile.exe: The filename or extension is too long.
my env:
windows10 64bit
go version go1.10 windows/amd64

still not working go1.9.5 windows/amd64
go build github.com/hashicorp/terraform/vendor/github.com/terraform-providers/terraform-provider-aws/aws: C:\Go\pkg\tool\windows_amd64\compile.exe: fork/exec C:\Go\pkg\tool\windows_amd64\compile.exe: Der Dateiname oder die Erweiterung ist zu lang.

@lachnerd I saw you are trying to compile the terraform-provider-aws on Windows. I believe, this does not work because because of the extremely long paths and not because of the golang packaging in conda-forge.
If you are wondering how hashicorp builds the windows dependencies, I believe they are are cross-compiling on Linux.

Why is this bug still open?
This issue has been in golang since compile started handing off the contents of entire packages via the commandline. Normally it was possible to just avoid using packages that included a ton of .go files but this past year a number of major projects have started including these larger packages, so it is not unavoidable.
Where is the development for this fix to compile (and its callers) happening at so we can all help get this done and then the fix merged upstream faster?

Why is this bug still open?

Because nobody has sent a (working) patch yet.

Where is the development for this fix to compile (and its callers) happening at so we can all help get this done and then the fix merged upstream faster?

Not sure what you mean. There is no "upstream": this is "upstream". Here is where you submit patches to fix this issue. See: https://golang.org/doc/contribute.html

@ALTree, question about bug being open was hypothetical and snarky. I'll avoid helping you with your semantics since you likely utilize the same resources as most internet normies and understand that being merged upstream means merged into the primary master branch like most.
Question was: where is the development for this fix to compile (and its callers) happening at so we can all help get this done and then merged? Is there anyone working on this? It's still tagged as "unassigned" and "help wanted". Maybe I fail searching through the mailing list archives, but there's no mention about this there either.
The issue with the 32k limit for UNICODE_STRING has already been identified, but there hasn't been any discussion on the proper fix short of ellismg's suggestion of adding an extra commandline parameter to compile and ensuring that its callers use it.

Would it be better to add support for parsing '@'-prefixed files and expanding them into argv similar to gnu-related tools and microsoft's response files?
Should support for this be added to the "flag" package since all commandline parsing on windows is effected?
Or how about adding an argument that lets you specify paths to packages instead of expanding the paths to all .go files for a package?
Or is ellismg's method of exposing an argument that reads source files from a whitespace delimited file the most likely solution to get merged?

Let's get this discussed and done, this oversight about the limitations of the windows environment during compilation for go has been around for too long now.

Okay, I got something working. I modified cmd/go to pass through the os/exec.Cmd.Args []string as JSON to stdin (to known trusted commands that are aware of it) and then added the decoding before flag.Parse() in cmd/compile.
I haven't tested it on Windows yet, but it works on Linux when I override the only-do-this-on-Windows part.

@bradfitz. kk. I'll let you know its success in a short moment here.

kk. Awesome.
@bradfitz, so it took me a bit as I had to spot the code with prints. but the only issue with your patch had was that where you check for the tool being equal to "compile". We'll need to add the GOEXE suffix to the check. At some point we can expand this extension for all the available tools anyways too.
Other than that, though. It works!

@arizvisa, thanks for testing and noting the .exe bug. I've uploaded a new version and also using it with "link".

@bradfitz, hey man. I noticed that one of the reviewers of your patch suggested the idea of going the path of response files. I made a branch that goes down this route, and adds '@'-prefixed response files to the same place that your patch added your fix.
I essentially re-implemented the buildargv function (from libiberty in gcc) in golang, but at the moment it doesn't do recursive response files which I believe gnu-tools supports. It also doesn't explicitly test the tool that's being dispatched to as your patch does.
At the moment its dev'd against go1.10.1. If the community prefers this solution for some reason, though, I can rebase it against master and create a PR so it can be merged upstream. However, I could use another pair of eyes to sanity check how I'm escaping things when writing to the response file (since it's pretty common to have mistakes here).
The branch I'm referring to is at arizvisa/golang#GH-18468

@arizvisa if you want your change looked at, you have to use tools that this this repo uses - https://golang.org/doc/contribute.html on how to contribute.  It  was @ianlancetaylor who suggested to use response files on https://golang.org/cl/110395. If you post your change here, maybe it will be prefered to CL 110395.
Alex

@alexbrainman, I don't want to step on bradfitz's toes as it was his original fix, I was just offering some direction as it was based on his original work and his response to my complaining. It works for me, but I also don't really want to maintain it or anything ;-) as I'm not really a developer.
Plus, I had realized that I implemented it from a GPL-licensed piece of software which I'm pretty sure is a licensing conflict.

I've submitted 17fbb83693d5d4b880bb128d7afdb137840f76ec (https://go-review.googlesource.com/c/go/+/110395). @arizvisa, please double check that it works for you?
I at least made it such that when it runs on our build farm, 10% of the compile/link invocations go through this code path, so it will get some exercise even for non-large build arguments.

@bradfitz, sorry to get back to you so late as I've been ill for a bit.
It seems to work without issue. So, thanks!
One thing not too important but worth thinking about in the future, the way you split lines in expandArgs just culls out the . I know it's dickheadish, but  is actually a valid character in a filename. Regardless of that weirdness tho, bufio.Scanner might be able to be used in order to scan for newlines more effectively. That was the reason why I ripped buildargv out of libiberty anyways.
Regardless though. Thanks for getting this fixed!

<CR> is not really a valid character in a Windows filename per https://msdn.microsoft.com/en-us/library/windows/desktop/aa365247(v=vs.85).aspx which says...

Characters whose integer representations are in the range from 1 through 31, except for alternate data streams where these characters are allowed.

I don't think we need to care about "alternate data streams", which I assume are like Mac "resource forks"?

Ah, the api call suceeds. (Although I'm using touch to avoid writing a program).
[528] user@skinny ~/build/go1.10.2$ uname -a
MSYS_NT-6.3 skinny 2.10.0(0.325/5/3) 2018-02-09 15:25 x86_64 Msys
[529] user@skinny ~/build/go1.10.2$ touch $'hola\nmundo.'
[530] user@skinny ~/build/go1.10.2$ ls -l $'hola\nmundo.'
-rw-r--r-- 1 user None 0 May  6 16:02 'hola'$'\n''mundo.'
[531] user@skinny ~/build/go1.10.2$ echo hola*
hola
mundo.

Result of the file in the old cmd prompt.
C:\users\user\build\go1.10.2\src>dir /x ..\hola*
 Volume in drive C has no label.
 Volume Serial Number is FA87-0BF0

 Directory of C:\users\user\build\go1.10.2

05/06/2018  04:02 PM                 0 HOLAMU~1     hola?mundo.
               1 File(s)              0 bytes
               0 Dir(s)  13,513,748,480 bytes free

Then a streams which isn't related, plus I couldn't get a newline added as a stream (even using the api)
C:\users\user\build\go1.10.2>echo hi > stream:woot
C:\users\user\build\go1.10.2>dir /r stream
 Volume in drive C has no label.
 Volume Serial Number is FA87-0BF0

 Directory of C:\users\user\build\go1.10.2

05/06/2018  04:12 PM                 0 stream
                                     5 stream:woot:$DATA
               1 File(s)              0 bytes
               0 Dir(s)  13,508,710,400 bytes free

I'm not really a mac guy (more of a security guy) so my research into different mac technologies (such as resource forks) are pretty spotty. but alternate data streams are typically used for things like metadata as they have no particular data structure. NTFS exposes some interesting default ADS which one can use to get some pretty interesting side effects out of software on windows. But as an example for a legitimate thing, browsers will add an ADS to a downloaded so that a file can be signalled as being "unsafe".
But regardless about platform differences/semantics, I'm just happythings are fixed and I just have tendencies to note side effects about things which is what that stripping of  looked like to me. :)
The implementation of buildargv that I ripped attempted to parse out these things by handling quoting and backslash-escapes. However in order to use that implementation effectively, when I had to build the response file I spent most of the time trying to figureout how to properly quote, escape, and newline each argument in the response file so that buildargv didn't remove characters that were explicitly specified.
Again though, thanks for fixing this!

Is this supposed to be fixed on 1.10.2? I just upgraded to the latest version and I'm still getting the error.

Edit: I just saw the 1.11 milestone, how can I have access to this fix before the release?

You could patch it in and recompile Go if it's urgent.
But I suppose we could also backport it to Go 1.10.x if we do another one.
@gopherbot, please backport.


menu should reflect what's going on with multi-step job form

The main menu should say "Next" instead of "Run" if that's where the user is at on the form. Or, "Run" should be disabled until the job form is at that point.

This also has some overlap with #218 because of the menu/form relationship


Hello, I'm new to OpenFeature. Ask, if I need to use GoFeatureFlag as a Provider, is this Operator applicable and how to disable FlagdProxy?

Tagging the Go Feature Flag maintainer @thomaspoignant here but my understand is:
No, this operator uses flagd  as the "backend" and flags defined in flagd format
That said, I know there are some discussions in the community about further commonality / standardisation so perhaps those discussion move this operator closer to being compatible with Go Feature Flag in the future? I would recommend getting involved in the discussion on Slack (#openfeature channel of the CNCF slack)

Hello @oomgomgxx, as @agardnerIT said for now it is hard to use the operator with something else than flagd.

You have some examples in the repo (here the nodeJS example).
That being said, if you are interested in using the open-feature-operator along with GO Feature Flag it is not possible for now due to some limitations on the operator (and also lack of time to look at it 😞 ).
There is an issue open in the GO Feature Flag repo for this https://github.com/thomaspoignant/go-feature-flag/issues/705 you can follow it to see the progress on it, but this is not the top priority right now.


Add support for RAJA versions >= 0.12.0.

@YohannDudouit, you merged this branch into https://github.com/mfem/mfem/pull/1626, so it should be reviewed there. Unless you were planning on reverting the merge into that branch?

For this version, could we update to RAJA::Teams? We are seeing improved performance in some cases. Also perhaps add RAJA-HIP?

@tzanio Do you know if I have to rewrite this PR because we "reverted" it since it was included in #1626 ?

If it was merged in #1626, it will be better to re-write it, i.e. use new commits.

Easy approval,  I just set up the following PR https://github.com/mfem/mfem/pull/1752 to use newer features of RAJA (though there might be more testing to do with that PR).

@artv3 Be careful, I might have to send new commits for this due to the reverted merge of #1626 .

I'm closing this in favor of #1752


Analytics

Analytics module for seeing stats and user activity from admin dashboard

Sorry for being late. Could you please give some detail about this module, some mock-up will be very helpful

Looking at the other platforms:
With the ability to select period of time, range of dates (today, month, year etc)

Users online in the last 30 minutes
Active shopping carts in the last 30 minutes
Products out of stock
Awaiting returns/exchanges (blocking issue: https://github.com/simplcommerce/SimplCommerce/issues/512)
Abandoned shopping carts
New customers
New subscriptions (email advertisements etc.)
Total subscribers
Traffic (visits, unique visits, sources of traffic)
Profits (money earned during selected period of time)
Recent orders
Bestsellers
The most viewed
The most searched

... and a lot more
some graph/chart library would have to be used
Optional integrations:

Google Analytics
Power BI
??


python environment (interpreter, $PYTHONPATH, pylint) can't be correctly set in remote-wsl

Visual Studio Code - Insiders: 1.36.0-insider (user setup)
remote-wsl extension version 0.38.5
wsl1: ubuntu18.04
python environment on wsl : python2.7,python3.6.8,python3.6.6 (my dev repo)
step to reproduce:

set up my dev python environment(with a $PYTHONPATH and python3.6.6) in wsl terminal.
in vscode settings (user,remote,workspace), mannully set python interpreter path, and .env with the content PYTHONPATH=${PYTHONPATH}, and pylint path
launch code-insiders in any dir, like ~/.

actually behavior:
no $PYTHONPATH set in vscode, module definition not working, interpreter not set, no warnning in pylint. can not select python3.6.6 by ctrl+shift+p(no entry for that)
expected:
correctly setup python environment which provides definitions, etc...

the method I tried, but failed:
create a virtualenv in .vnev folder with my devvloping python environment
activite my venv in wsl and launch code, the code can pickup venv settings but still, the python interpreter (should be python3.6.6 instead of python2) and $PYTHONPATH is not right (null)
PS: the whole setup of my working environment has been used on centos and ubuntu for a long time and launch vscode on linux is just perfect with my setup. I am kinda new to wsl using vscode, but I use it on linux all the time.  I just don't know what I could try to figure this out.
Any help will be much appreciated!!

Thanks for letting us know about this, @LimboWK.  We're looking into it.

Sorry for the delay, @LimboWK.  I haven't been able to reproduce any problem with using a .env file to set $PYTHONPATH under WSL (remote).
Here's what I did (via remote to WSL):

create the files:

# <workspace>/.env
PYTHONPATH=${PYTHONPATH}:spam

# <workspace>/spam.py
import sys

for entry in sys.path:
    print(entry)

in the "Debug" sidebar, set the config to "Python: Current File"
run the debugger

In the output ("TERMINAL" pane) I see the following:
$ cd .../gh-6389 ; env PYTHONPATH=:spam PYTHONIOENCODING=UTF-8 PYTHONUNBUFFERED=1 .../python3.5 .../.vscode-server/extensions/ms-python.python-2019.6.22090/pythonFiles/ptvsd_launcher.py --default --client --host localhost --port 54628 .../gh-6389/spam.py
.../gh-6389
.../.vscode-server/extensions/ms-python.python-2019.6.22090/pythonFiles
.../gh-6389
.../gh-6389/spam
<others>

You can see that the contents of the .env file were included when the command ran (env PYTHONPATH=:spam ....  You can also see that the entry from .env showed up in sys.path.
Keep in mind that the contents of the .env file are not used in the terminal normally (e.g. with "Run Python File in Terminal").  However, they are used for the various tools (e.g. pylint).

I also tried the following:

moved that .env file to my home directory
ran the debugger (entry did not show up in sys.path, as expected)
set "python.envFile": "<home dir>/.env" in my workspace settings.json (in WSL)
ran the debugger (entry did show up in sys.path, as expected)

Note that we have several feature requests open related to using the .env file in more places, including for the terminal.

FWIW, I was using the latest release of VSCode and the Python extension.

Also possibly related: #5724

Also, manually changing "python.pythonPath" in my workspace settings.json worked fine.

FWIW, I was using the latest release of VSCode (not insiders) and the Python extension. I had to install the "Remote Development" extension pack manually.

Hello eric, sorry for not catching up quickly here, in fact I did try the release version of vscode and I just can not launch the wsl server when it first starts running that basically why i am using the insider.
I didn't really know what is going on here, but since last time I did changed something.
Now i am using wsl2 in c drive, while previously i was using it in my d:\ drive which was exported and re-imported.
And now I am be able to mannually setup my python interpreter, and I added extrapath for my libs, installed pylint, then the auto completion and linting is working.
I will try do as what you tried to see if I could do it by setting $PYTHONPATH in .env.
Thank for the info ;)

I'm glad things are working better for you now.  Please let us know if there is anything else we can help you with.


Some Champions are Missing

I'm not entirely sure if this is an issue with connecting to the API or not, but about a dozen or so champions are missing when generating item sets. Example here https://imgur.com/OBp3X6F
Going to https://lol-item-sets-generator.org/sets shows that, as of right now, these champions are also missing from the list. https://imgur.com/UcJrEkf

Thank you for your report.
The item sets are generated from the champion.gg datas (they provide a public API).
Apparently, these datas currently omit some champions (like Aatrox, for example).
I think there is not much I can do, except wait for the next build generations (every day at 12 AM) until it gets fixed.
Feel free to re-open this issue if you consider that the issue persists for too long :)

Just bumping this post to notify that this bug is fixed.


iasl: 20180313 -> 20180508

Motivation for this change
Cherry-picked from #40608
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Success on x86_64-linux (full log)
Attempted: iasl
Partial log (click to expand)

installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/7npniq2lclj2j9dll49i5d9iv2a7m1a3-iasl-20180508
shrinking /nix/store/7npniq2lclj2j9dll49i5d9iv2a7m1a3-iasl-20180508/bin/iasl
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/7npniq2lclj2j9dll49i5d9iv2a7m1a3-iasl-20180508/bin
patching script interpreter paths in /nix/store/7npniq2lclj2j9dll49i5d9iv2a7m1a3-iasl-20180508
checking for references to /build in /nix/store/7npniq2lclj2j9dll49i5d9iv2a7m1a3-iasl-20180508...
/nix/store/7npniq2lclj2j9dll49i5d9iv2a7m1a3-iasl-20180508

Success on aarch64-linux (full log)
Attempted: iasl
Partial log (click to expand)

installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/wmmdfa9jgbdjg1g4nxsrpa4ilbkf5x48-iasl-20180508
shrinking /nix/store/wmmdfa9jgbdjg1g4nxsrpa4ilbkf5x48-iasl-20180508/bin/iasl
strip is /nix/store/jk6j4lh9v5mvjdbdc35sj0zffhhf6s56-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/wmmdfa9jgbdjg1g4nxsrpa4ilbkf5x48-iasl-20180508/bin
patching script interpreter paths in /nix/store/wmmdfa9jgbdjg1g4nxsrpa4ilbkf5x48-iasl-20180508
checking for references to /build in /nix/store/wmmdfa9jgbdjg1g4nxsrpa4ilbkf5x48-iasl-20180508...
/nix/store/wmmdfa9jgbdjg1g4nxsrpa4ilbkf5x48-iasl-20180508

This breaks xen and virtualbox: https://hydra.nixos.org/eval/1457705#tabs-now-fail
Reverted in bd6a3b2160e9d7a0b1f2764df04b44db8e97922b

Oh, yikes... Should we file an issue upstream?

Yes, after some investigation to rule out nixpkgs-specific issues.
FWIW, Gentoo currently uses iasl 20171215, Arch and Ubuntu have 20180105.
Maybe we should stick to our current version for now. What's the benefit of being bleeding edge here?


Add support for max-connections to the Undertow connectors

Linux Build 7275 is now running using a merge of fafd14bd65d442b17d378df14bd258dbb0bba259

Windows Build 2329 is now running using a merge of fafd14bd65d442b17d378df14bd258dbb0bba259

Linux with security manager Build 2144 is now running using a merge of fafd14bd65d442b17d378df14bd258dbb0bba259

Linux with security manager Build 2144 outcome was SUCCESS using a merge of fafd14bd65d442b17d378df14bd258dbb0bba259
Summary: Tests passed: 896, ignored: 285 Build time: 0:06:23

Windows Build 2329 outcome was SUCCESS using a merge of fafd14bd65d442b17d378df14bd258dbb0bba259
Summary: Tests passed: 3083, ignored: 367, muted: 1 Build time: 0:48:41

Linux Build 7275 outcome was SUCCESS using a merge of fafd14bd65d442b17d378df14bd258dbb0bba259
Summary: Tests passed: 3085, ignored: 366 Build time: 0:48:42


protbom ref suggestion

Hi there,
Suggesting to add the following documentation reference tables generated out of our loveable protobom protobuf structure,
You can review the current version in ref.
Related to issue
@manifestori If we decide we want this doc i be happy if you help me integrate this in to our uber cool dev container ..
I would like to suggest the addition of documentation reference tables generated from our beloved Protobom protobuf structure. The current version of these tables can be reviewed here.
This suggestion is closely related to issue #162.
@manifestori , if the community decides to include this documentation, I would greatly appreciate your assistance in integrating it into our fantastic development container.
Here is an example of how I currently handle it:
docker run --rm   -v /home/mikey/scribe/DHS/protobom/docs:/out   -v /home/mikey/scribe/DHS/protobom/api/:/protos   pseudomuto/protoc-gen-doc --doc_opt=markdown,protobom-ref.md

Oh this is cool, what do you think about adding a presubmit check to ensure the docs don't go stale? SOmething like what we do with the protos themselves:
https://github.com/bom-squad/protobom/blob/52f2d4b5162dbd15efaf397c5254a3f426e94246/hack/verify-proto.sh#L18-L20

@houdini91 ,
If we note all format specifc data in separate columns, this could enable us to automatically generate the "transfer function" of a whole translator !
The infomration neede would be the meaning of each field for each SBOM format:version.
Is there a way to autogenerate these columns or some labeling capability ?

@dn-scribe, Customizing the pattern might involve forking the protoc-gen-doc project. Let's shift the discussion to the pertinent issue and focus the pull request on a straightforward integration with protoc-gen-doc, establishing a foundation for future enhancements.
I see this documentation more as reference tables rather than detailed specification documentation. While it should capture the specifications and translations, it may lack the higher-level logical translation logic that users might find valuable. It could be worthwhile to include links to relevant tables or fields in the reference table for additional context.

@puerco i added a verify-doc file but i am thinking maybe i merge it in to the verify-proto.sh file?

Can you add hack/verify-doc.sh to the presubmits here?
https://github.com/bom-squad/protobom/blob/7ce33d43003d97ca5e1d20bddb65579c16b73e15/.github/workflows/go-build-and-test.yml#L24-L26

@houdini91 Ill make time this week to finally go over your PR.


Allow normal and slim client work together

Current implementation doesn't allow using Akka.Interfaced and Akka.Interfaced-SlimClient. However a program using Akka.Interfaced could be a slim-client to another program. To make it happen, let them work together.

Not easy because SlimClient has 2 roles.

One is a bridge to actors without akka.net.
The other is a restricted client to the actor system.

Second one can be achievable without pain but first one is not easy to work with Akka.Interfaced on akka.net. For example, SlimClient defines a proxy interface that pretends to be IActorRef which may cause conflict with akka.net.

To make this happen, IActorRef is replaced with IRequestTarget which can be AkkaActorTarget holding IActorRef or BoundActorTarget holding ID of bound actor!


502 Bad Gateway when hitting cloud.internalpositioning.com

Not able to track anything because cloud.internalpositioning.com url says 502 Bad Gateway.

Should be up now


feat: allow adding metadata to versions

Is your feature request related to a problem? Please describe.
I have a requirement to add various metadata to each version, e.g. reason_for_change for auditing compliance.
Describe the solution you'd like
Ideally, the DSL of AshPaperTrail would allow specifying metadata arguments to accept from the action to merge into the version.
Describe alternatives you've considered

Mixins don't seem capable of handling this as an escape hatch.
Storing the metadata directly on the primary resource is a leaky abstraction.

Express the feature either with a change to resource syntax, or with a change to the resource interface
Simple example:
paper_trail do
  metadata [:reason_for_change] # <-- will pull the value from that argument of action
end

It may be important to support more advanced uses, like pulling a nested field from the context and naming the key:
paper_trail do
  metadata [client_ip: ^context([:client, :remote_ip])]
end

Additional context
For implementation, perhaps we could use a special context to store the metadata:
paper_trail: %{reason_for_change: "fix typo"}

which could then be encoded on each version.

Since paper trail is cross-cutting across multiple actions, we'd probably want to use the latter, so ash_paper_trail would extract anything in the paper_trail_metadata context key or something along those lines.


License?

Saying it's free and open source in the readme ain't gonna cut it. You also need a proper LICENSE or COPYING file (and sometimes a license notice on top of each file).
https://choosealicense.com may be a good place to start if you need help.

Now using MIT license.

Great. Thanks for your great work!


LSCL token: Added logo, needs background still

🗃 Github Issue Or Explanation for this PR. (What is it supposed to do and Why is needed)
Enter Issue number here
✅ Checklist

[ ] Github issue details are up to date for people to QA.
[ ] I have tested all my changes.

🕵♂ Notes for Code Reviewer
Technical explanations to help your peers review your work - including environments, credentials, etc.
🙈 Screenshots
For all UI changes

description 1
description 2

img1
img2

👯♀ Paired with
@github-handle or "nobody" if you did not pair.

@gguij004 did it! Thanks!!


Add Silver contest entry with link to tweet

Hi,
I added the tweet link to my image for Silver stage of the contest.
best,
Philippe

Thanks for the 🌱


Make overlays unselectable by using CSS. Fixes #351

Coverage remained the same at 4.224% when pulling def2bf14584c2fd8adf136955e26e1159e9e333e on medihack:let-events-propagate into f1e5342a884f8de86de248b18c5ad6b53593ce66 on cornerstonejs:master.

Thanks!


Inserting OData query options into your SAPUI5 app

Hi,
Thanks so much for these great tutorials!
But where is the source code for this tutorial?
For example, you mentioned northwind/view/Master.controller.js.
Thanks!

Perhaps a link to Build an app from an SAP Web IDE template should be put on the Prerequisites part of the page of Inserting OData query options into your SAPUI5 app

Hello,
Apologies for the long delay, and I am sure you figured it out. For reference, that tutorial is part of a longer series that starts with Build an app from an SAP Web IDE template as you mentioned. This tutorial is not the immediate predecessor which is why it is not listed in the Prerequisites section. Prerequisite and Next Steps are intended to similar to Previous and Next "buttons" for the series.
However, we have noted this a possible usability issue on our side and will make sure to consider it as we are redesigning the tutorial experience.
Thank you!
Meredith


[Windows] Cannot compose metadata in the same category.

Description
I have separate parts of my application that want to add keys to the same Metadata categories.
Executing the following statements should be valid:
BugsnagUnity.Bugsnag.Metadata.Add( "myCategory", new Dictionary<string, string> { { "keyA", "valueA" } } ) ;
BugsnagUnity.Bugsnag.Metadata.Add( "myCategory", new Dictionary<string, string> { { "keyB", "valueB" } } ) ;
Issue
Exception is thrown:
System.ArgumentException: An item with the same key has already been added. Key: myCategory
The Unity C# wrapper around the NativeClient uses a dictionary and naively calls .Add which will fail and throw exceptions if you attempt to add additional keys to the same category.
Environment

bugsnag-unity version: 4.6.4
Unity version: 2018.4.3f1
Operating system name and version: Windows 10
Target platform names and versions: Android/iOS
Initializing bugsnag via the Unity UI or in code? UI

Example code snippet
BugsnagUnity.Bugsnag.Metadata.Add( "myCategory", new Dictionary<string, string> { { "keyA", "valueA" } } ) ;
BugsnagUnity.Bugsnag.Metadata.Add( "myCategory", new Dictionary<string, string> { { "keyB", "valueB" } } ) ;
Expectation: KeyA,ValueA and KeyB,ValueB are merged into the category.
Result: Exception is thrown.

Hi @technicalDeath thanks for the report! This indeed looks like a bug.
We will look to get this fixed in a future release and will let you know when it has.

Note: I was running in Unity editor on windows, but this issue is present on any Unity build platform (Android, iOS, etc.).

@abigail-bugsnag Hi Abigail, I have implemented a fix for this issue which can be found here:
https://github.com/bugsnag/bugsnag-unity/compare/master...technicalDeath:allow_metadata_additions
Perhaps your team would like to integrate it. It has been thoroughly tested on my end on Windows, iOS, and Android.
Cheers!

@technicalDeath
Thanks! The team will take a look.


[BUG]On test environment  Main chain and Side chain can't index block data  each other

Description
The main chain and side chain can’t index each other’s block data. The BP account doesn’t automatically send the ProposeCrossChainIndexing transaction to index the side/parent chain block data. After a period of time, an error Validate block after execution failed: CrossChainValidationProvider will be reported. and the node can't generate block.
Environment

Test Environment
Node: v1.10.0

The issue has been resolved. The cause was due to the parallel execution of the generation block and the generation system transaction, which has now been fixed.


How admin can send a message to broadcast dialog?

1, How admin can send a message to broadcast dialog?
2, how user can get messages of broadcast dialog chat?
I tried to get by API https://api.connectycube.com/chat/Message?chat_dialog_id=63984fad7e97ef002fe72945
but got this error
{
"base": [
"Cannot read properties of undefined (reading 'includes')"
]
}

no activity for a long time, closing.


Confirmation buttons

Hi!
I would suggest to add confirmation buttons to logout or to cancel the creation of new document so you don't have to retype the login informations or the informations of the new document if you missclick on cancel.

Done ! Thanks for the feedback


🐞 - tuiAccordionItemContent SSR issue during hydration

Playground Link
No response
Description

As you notice in the picture, the accordion item content gets repeated twice. (It's created once in the server, and once in the browser, and thus fails to reconcile during hydration).
  @for (item of value().questions; track $index) {
    <tui-accordion [rounded]="false">
      <tui-accordion-item [borders]="null" [open]="true">

          {{ item.question }}

        <ng-template tuiAccordionItemContent>

            {{ item.answer }}

        </ng-template>
      </tui-accordion-item>
    </tui-accordion>
  }

Angular version
18.0.3
Taiga UI version
3.83.0
Which browsers have you used?

[X] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge

Which operating systems have you used?

[ ] macOS
[X] Windows
[ ] Linux
[ ] iOS
[ ] Android

I can't reproduce on https://taiga-family.github.io/preview-landing-components/


update jxcore+sm-<IP_ADDRESS> OSX package's sha

The new sha is considered correct according to https://github.com/jxcore/jxcore/issues/774#issuecomment-170566013


SCSS compilation not working

Hi,
I've updated munee to the last version. Unfortunately, the SCSS compilation doesn't work at all. The file is unchanged. I've only the SCSS source.
I'm developing on a local environment with EasyPHP and PHP 5.5 support.
A set_time_limit(0) was necessary cause I have sometimes this error too ?
Fatal error: Maximum execution time of 30 seconds exceeded in vendor\sabberworm\php-css-parser\lib\Sabberworm\CSS\Parser.php on line 313
Here is a classic declaration :
<link rel="stylesheet" href="themes/restart/assets/scss/app.scss,vendor/font-awesome/css/font-awesome.css?minify=true" />
What do you think it is ?
Thanks for your help

@t-prod: It looks like Munee is erroring out when trying to compile.  The way Munee works (which was an oversight on my part) is that it takes the original content and caches it.  Then when it goes through the process of running it through file types (converting SCSS to CSS) and filters (minifying), it updates that content of the cache.  If there is an error, it's not clearing that cache.  And when you request it again, it sees that there is a cached version and uses that.  Since it errors out and never converts the file, it's serving you the original file.  If you delete the cache every time you will notice that it's erroring out every time.
So with all that in mind, do you have any error log files with more information? It looks like the CSS parser I am using is erroring out on your CSS.  It would be good to have a backtrace for more info.

Thanks for your quick answer. I delete the cache and the second error is Fatal error: Maximum execution time of 30 seconds exceeded in vendor\leafo\scssphp\src\Compiler.php on line 1017.
I've added a set_time_limit(1)
In my htaccess I have the classic rule which works with previous versions :+1:
'#### Munee .htaccess Code Start ####
RewriteRule ^(.*.(?:css|less|scss|js|coffee|jpg|png|gif|jpeg))$ munee.php?files=/$1 [L,QSA,NC]
Munee .htaccess Code End ####'
It's like the scss is totally ignored. All other CSS files or Javascript work fine.
Leafo SCSS is in version 0.1.1 and the compilation with set_time_limit(0) is very very long.
No errors except the fact scss files are ignored.
Any ideas ?

@t-prod: Are you able to work out where in your SCSS this is happening? Start deleting sections and retrying to see what particular part of your SCSS is causing this?  Or maybe provide me a copy of your SCSS to see if I can find something?

Ok I've figured it out. I think is a parsing problem when we work in local.
I use the framework Foundation in SASS mode.
I simply import the foundation.scss located in my vendor path :
@import "../../../../../vendor/foundation/scss/foundation";
but it displays this :
@import url("/localdev/admin/vendor/foundation/scss/foundation/functions");
@charset "UTF-8";
@import url("/localdev/vendor/foundation/scss/components/grid");
@import url("/localdev/vendor/foundation/scss/components/accordion");
@import url("/localdev/vendor/foundation/scss/components/alert-boxes");
@import url("/localdev/vendor/foundation/scss/components/block-grid");
@import url("/localdev/vendor/foundation/scss/components/breadcrumbs");
@import url("/localdev/vendor/foundation/scss/components/button-groups");
@import url("/localdev/vendor/foundation/scss/components/buttons");
@import url("/localdev/vendor/foundation/scss/components/clearing");
@import url("/localdev/vendor/foundation/scss/components/dropdown");
@import url("/localdev/vendor/foundation/scss/components/dropdown-buttons");
@import url("/localdev/vendor/foundation/scss/components/flex-video");
@import url("/localdev/vendor/foundation/scss/components/forms");
@import url("/localdev/vendor/foundation/scss/components/icon-bar");
@import url("/localdev/vendor/foundation/scss/components/inline-lists");
@import url("/localdev/vendor/foundation/scss/components/joyride");
@import url("/localdev/vendor/foundation/scss/components/keystrokes");
@import url("/localdev/vendor/foundation/scss/components/labels");
@import url("/localdev/vendor/foundation/scss/components/magellan");
@import url("/localdev/vendor/foundation/scss/components/orbit");
@import url("/localdev/vendor/foundation/scss/components/pagination");
@import url("/localdev/vendor/foundation/scss/components/panels");
@import url("/localdev/vendor/foundation/scss/components/pricing-tables");
@import url("/localdev/vendor/foundation/scss/components/progress-bars");
@import url("/localdev/vendor/foundation/scss/components/range-slider");
@import url("/localdev/vendor/foundation/scss/components/reveal");
@import url("/localdev/vendor/foundation/scss/components/side-nav");
@import url("/localdev/vendor/foundation/scss/components/split-buttons");
@import url("/localdev/vendor/foundation/scss/components/sub-nav");
@import url("/localdev/vendor/foundation/scss/components/switches");
@import url("/localdev/vendor/foundation/scss/components/tabs");
@import url("/localdev/vendor/foundation/scss/components/thumbs");
@import url("/localdev/vendor/foundation/scss/components/tooltips");
@import url("/localdev/vendor/foundation/scss/components/top-bar");
@import url("/localdev/vendor/foundation/scss/components/type");
@import url("/localdev/vendor/foundation/scss/components/offcanvas");
@import url("/localdev/vendor/foundation/scss/components/visibility");
So the location is wrong where can I correct this ?

I have something in place which rewrites any of the CSS url paths from relative links to absolute for minifying and concatenation purposes. So those @import tags actually shouldn't be rewritten as it runs it through the SCSS compiler and then it goes through and tries to fix any @import tags which are relative to convert to absolute.  I think SCSS isn't compiling your template properly.  Or it's skipping the @import tags because it can't find the files to import.  Are those paths correct?

Maybe it's more the munee.php at the root. Is there any new config variables in a local environment ?
<?php // Include the composer autoload file define('WEBROOT', __DIR__); require 'vendor/autoload.php'; // Echo out the response echo \Munee\Dispatcher::run(new \Munee\Request());
Locations are right cause I used the last version of Munee (I recheck twice).

Okay.  I'm going to give it try myself.  Bringing in Foundation and then trying to compile it via Munee.  I'll get back to you as soon as I can.

Thanks again cause I used Foundation and Munee for a while and I never see an error like this ;)

Sorry for the issues!  Please use the previous version until I can figure this out.

Hello,
Do you have any news about this problem ?
Have you tested it with Foundation ?
Thanks for your answer


[BEAM-8512] Add integration tests for flink_runner.py

This is just a very basic smoke test to ensure that FlinkRunner works as intended with the default settings. This does not test behavior with an actual Flink cluster.


Use SWR for pipeline request

This PR is intended to fix the CI timeout issue, but should also fix #1750

Thanks @bourdakos1. Could you please update the opening description with a brief description of what SWR is and how it might address the CI and log duplication issue?  Although I'm clueless about the technologies used in the frontend, it might be helpful to myself and others to get an idea of how this will help.  Thanks.

On the same line as @kevin-bates, could we actually change the title of the PR to what is actually trying to do, something more like Reduce the number of duplicate calls to backend services.
Also, we have a similar utility that is supposed to handle the calls to backends (@ajbozarth knows that very well). Could you guys collaborate to make sure we don't create two utilities for the same thing.

@ajbozarth I switched to using the metadata service directly. For the error, we can't really do what we are doing now (we could but not recommended)
I know we are replacing error dialogs with toasts in the future, do you have more details on how that would work?

For the error messages we plan to move the non-interactive error message to toast, so any dialog that has a stack trace, links, or buttons other than close would stay dialogs (at least that's the initial plan)

Since this PR is also cleaning up log messages, I guess we can remove this line:
https://github.com/elyra-ai/elyra/blob/59d3eeb4ca2dd18bf2c678eab312c168287f0cac/packages/pipeline-editor/src/PipelineEditorWidget.tsx#L608
to avoid unnecessary logs like


Annotation icon missing on Wagtail 2.11

The annotation icon on the front-end is relying on an icon font that isn't being imported any more in 2.11:

This has been fixed in wagtail-review-ui 0.3.0, but it looks like 0.3.0 has made breaking changes on the JS side (presumably to accommodate the workflow implementation), so we can't upgrade.

Fixed for 0.3 in 9f3f9a85336d60be636876c86ccd9150110c0180


support lichess?

https://github.com/AlekseyLobanov/lichess-export
could combine this to support both
may want to rename the project/repo

didnt end up using the above, seems the API has changed since then and requires auth
should be resolved by #3


Please merge #cde479301a81f9fbaec3fb007457a4ec8e5a65a7 to dev channel

commit(s) to merge: cde479301a81f9fbaec3fb007457a4ec8e5a65a7
merge instructions: https://dart-review.googlesource.com/c/sdk/+/83229
reason: revert a change that introduced regression in Flutter (please see https://github.com/flutter/flutter/issues/23879)
/cc @dgrove @kevmoo @mit-mit @whesse @athomas

Merged in version 2.1.0-dev.9.2

Merged in version 2.1.0-dev.9.2


Exception thrown when registering

Programming error occurring when registering. Registration takes place but the screen shows an exception, as attached in the screenshot.

The migrations are not being applied . Idk why but "python manage.py migrate" is not working, even when I use app name with it .  If you try the command "python manage.py showmigrations" , you won't find the latest migrations in it.
So i dropped the database, created it again , applied migrations and now it is working fine .

https://stackoverflow.com/questions/29902366/django-migration-is-not-applying-the-migration-changes.
Follow this link, or the updated readme file to fix this issue.


How to change Storage Folder Location?

I'm talking about the Folders in:
C:\Users\User\AppData\Local\Keybase
kbfs_block_cache & simplefsdownload

There is someway to change it from the Desktop APP or i will need to do a SymLink? -.-

Bump
...also would like to know how to change this on macOS and Linux.


Adjust style in the map sidebar elements

Feature Description

Tab text elements fonr-size should be 1rem/16px

padding left and right for these icons should be 0.75rem

For the layer icons, please make sure that spacing between two icons equals to 0.75rem/12px (px-2 and ml-1 classes would work)

border-bottom for accordion items (for all accordions in the geohub) should be #D4D6D8 (gray-400)

Accordion chevrons look different in the left and right sidebar. In the left one, they are smaller and look more balanced, so let's use these chevrons for all accordions

Feature Implementation
No response
Feature Alternatives
No response
Feature Impact
No response
Feature Rationale
No response
Feature Considerations
No response
Feature Requirements
No response
┆Issue is synchronized with this Wrike task by Unito

No.5 is not yet fixed


How to escape text

I need to able to escape a text sequence like this:
nbText: hlMd """ openssl req new -days 365 -subj C/=BR/ST=MN/L=Minnesota/O=SKC/OU=/skc.eagan.lab -key client-1.key -out client-1.csr"""
I'm pretty its the = sign it doesn't like.

Nevermind I forgot a closing tag ):


GeckoRate seems wrong

throttling seems wrong. If geckopy.Rate(20) is called, then a node shouldn't publish faster than 20hz, but it happens. It could be an error in my calculating messages/sec too?

fixed


Preproposal: Shared cache and lifetime cycle at proc_macro context

It is a poorly worked idea, but I think it could be put to good use.

share a RwLock  between proc_macros
With the aim of being able to share intermediate representations and to be able to treat the nested ones.
add lifetime synchronization cycle, like webpack or some other bundler
In order to prioritize
Unique identifier for types
To be able to locate and solve

I think that of the types is currently not possible since they resolve after the proc_macro, but some approach can be tried

I'd suggest closing this issues and moving any discussion to https://users.rust-lang.org/ and maybe later https://github.com/rust-unofficial if it goes anywhere.

Ok, I'm closing it and looking for another way more soundness


Adds new command pa app export. Closes #2501

This PR adds the command pa app export.
Closes #2501

Merged manually, thanks you! 👏


CLI: support git URLs as directory arguments

Support values include:

./mydir
file://mydir
Various url schemes for git remote: git://, ssh://, https://, git+ssh://, git+https://
Encode git ref in URL fragment (defaults to main): https://github.com/dagger/dagger#v0.9.2
Not yet supported: tarball-over-https
Not yet supported: interpret local dir as a git repo, by encoding a git ref in the fragment, eg. file://.#main

I would love to add information about supported formats in the --help output, but I don't know how to do that...

I would love to add information about supported formats in the --help output, but I don't know how to do that...

We can add a method to DaggerValue for that so it gets appended to the usage in AddFlag (GetCustomFlagValue). That would show in every flag that uses that type, though. To make a separate section in --help dedicated to this we'd have to customize the help template for it.


Possible (long standing) bugs in logic to review

Some issues we've seen in Pulse that might merit logical changes or tweaks:

https://github.com/18F/pulse/issues/288 (Sep 4, 2015, may not be an issue)
https://github.com/18F/pulse/issues/211 (Jun 4, 2015, but I see this a lot, likely still an issue)
https://github.com/18F/pulse/issues/367 (Jan 20, 2016, prob still an issue)
https://github.com/18F/pulse/issues/272 (Aug 21, 2015, may not be an issue)
https://github.com/18F/pulse/issues/271 (Aug 21, 2015, likely still an issue)
https://github.com/18F/pulse/issues/219 (as of Sep 8, 2016, possibly an issue)

Updated the description with a few more. I'll just update the description if I find others.

And this one about detecting redirect domains: https://github.com/18F/pulse/issues/219#issuecomment-245582986

For review: faces.fta.dot.gov should sayd Yes for Default to HTTPS (its only live endpoint is https://), but does not -- I think because it does a local 301 redirect to a sub-path.


fix(commands): set current device while running project

This PR fix https://github.com/akinsho/flutter-tools.nvim/issues/280

Now flutter_tools_decorations.device works very well with lualine plugin.
Here is my config
{
    "nvim-lualine/lualine.nvim",
    optional = true,
    opts = function(_, opts)
      local function is_available()
        return package.loaded["flutter-tools"] and require("flutter-tools.commands").is_running()
      end

      table.insert(opts.sections.lualine_x, 2, {
        function()
          if not is_available() then
            return
          end
          local device = vim.g.flutter_tools_decorations.device
          local device_name = device and (device.name or device.id) or "DEFAULT"
          return device_name .. " | " .. vim.g.flutter_tools_decorations.app_version
        end,
        color = "DiagnosticOk",
        cond = is_available,
      })
    end,
  }

But when running :FlutterRun command directly on a single default device, I don't know how to get the device properties.
So I fallback to "DEFAULT" for lualine.

Thanks, @bruce3x!


Search and highlight files & directories in Explorer

Could we get a command to search for and highlight (that is, select) files & directories in the Explorer sidebar?

https://github.com/Microsoft/vscode/issues/10026


data: project addition by Sai Kiran Reddy

Add a new project to the list
I want to add a new project to the list
I aim to introduce a new project that will provide a valuable add-on for this project under the gssoc'23 program
I want to add a project called Web Crawler The crawler will traverse websites starting from a given seed URL, following links to discover and crawl additional pages within the defined scope, It will scrape all the front add files like HTML, php except the backend code files to ensure safety!
Code of Conduct

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

Completed. Thanks for your contributions 🎉 ❤


Make the conformance test names unique

Some of the conformance tests have the same name, which doesn't render correctly in the results web page.
Fixes #508.

Note to maintainers, please hold off on merging until the 1.1.0 vote completes.


[TASK] OpenAPI validator plugin implementation

Description:
Implement a compiler plugin to validate a Ballerina service against a OpenAPI documentation and vise versa. Validation should come as compilation errors.
Specification: https://docs.google.com/document/d/1g650UEhFukQk9FVFo0EberhB-DJuv7GCsVO0RQSoE3g/edit?usp=sharing
Following is something similar to the behavior of the expected tool.

No longer relevant


Modal doesn't pass style property to div

Issue description

version #4.6.2
components: Modal

Steps to reproduce issue
<Modal isOpen={isOpen} toggle={this.onToggle} style={{ maxWidth: '95%' }}>

the style property doesn't apply on the final rendered div element. I checked the source code, and found that the problem does exist.
is it intended?

Due to the complexity of the render component, it doesn't look like any extra props are passed through to any of the markup. Which element would you expect to get the styles?
A similar issue with className caused the creation of props such as
className: PropTypes.string, wrapClassName: PropTypes.string, modalClassName: PropTypes.string, backdropClassName: PropTypes.string, contentClassName: PropTypes.string,

I expect the div with 'modal-dialog' class to get the styles. sometimes, it is really a convenience to use an inline style other than class name.

Looks like #339 removed that.


Optimize UX loading +100 keystores on vc

Describe the bug
After fixing https://github.com/ChainSafe/lodestar/issues/4166 now it takes a long time to load secure scrypt keystores.
Note that this UX improvement only affects whales and institutional players:

Home stakers are likely to have < 10 keys
Testnets + Kurtosis are run with either insecure scrypt keystores or mnemonics

Next steps:

[x] There's a bug currently where memory is not released after decrypting the keystores. Must investigate and ensure memory is released
[ ] Instead of decrypting 100 keystores at once, decrypting 5-10 is fine, which provides a significant speed up. A good number would be to decrypt thread count at once
[x] Implement similar strategy to Prysm and Lighthouse where private keys are stored in a single keystore such that subsequent start-ups are fast

@wemeetagain @dapplion Do we still want to keep this issue open to address some of the ideal next steps listed in the original post? Unless I'm mistaken and these have actually been completed?

We can reopen, but the last item is low priority, imo. The big win was @nazarhussain 's recent PR #5043
Last item is:

Instead of decrypting 100 keystores at once, decrypting 5-10 is fine, which provides a significant speed up. A good number would be to decrypt thread count at once

A good number would be to decrypt thread count at once

in node v19.4.0 a new API was added to properly determine this, os.availableParallelism(), v20 will be active LTS in 8 months (2023-10-24)

This was brought up at the March 7, 2022 standup to prioritize the remaining item so institutional type node operators would be able to consider Lodestar in their setup. This is in reference to the tweet made by node operator RockLogic as a blocker to allow them to consider our validator client for their setup: https://twitter.com/Stefa2kEth/status/1628407293325807623?s=20
cc: @dapplion


Encryption with key rotation

Problem: Laravel does not provide a way to rotate encryption keys
Solution:
Allow the framework to maintain multiple encryption keys associated with key_ids, and keep a reference to the key_id used to encrypt the value in the encryption payload.
First, we need to tell the application what factory to use to create the key store. At the moment there are two: KeyRotationKeyStoreFactory and AppKeyKeyStoreFactory
The AppKeyKeyStoreFactory is used by default and it will use your APP_KEY and the value from the config/app.php app.cipher and create a Illuminate\Encryption\KeyStore::class with a Illuminate\Encryption\Key::class with the APP_KEY as the key and the app.cipher as the cipher.
The Encryption service provider then passes this to the Illuminate\Encryption\Encrypter::class and you can encrypt/decrypt away.
The KeyRotationKeyStoreFactory uses the configs at:
Note: I am not recommending that you store your encryption keys in your config/app.php this is probably not the safest place to keep them. I recommend using something like https://www.vaultproject.io/ and have the keys injected into your application.  I am setting the config route that the encryption code will look for these keys. If you do not want to do it this way you can implement your own KeyStoreFactory that pulls the keys from where you store them and provide the new implementation to the application using app.encryption. keyProvider
"app.encryption.keys" = [
  "key-id-1" => [
    'value'  => 'base64:IUHRqAQ99pZ0A1MPjbuv1D6ff3jxv0GIvS2qIW4JNU4=',
    'cipher' => Cipher::AES_128_CBC,
  ],
  "key-id-2" =>  [
    'value'  => 'base64:IUHRqAQ99pZ0A1MPjbuv1D6ff3jxv0GIvS2qIW4JNU4=',
    'cipher' => Cipher::AES_128_CBC,
  ]
]

to create a Key for each config value with the value as the key and the cipher then return a KeyStore with all the keys from the config array.
Every time a value is encrypted, the encrypted value is put in an object Illuminate\Encryption\EncryptionPayload::class along with the iv, mac, and keyId.
The keyId is a reference to the key-id that is used at encryption time.
Every time you rotate a key the old encrypted values keep a copy of the keyId in the EncryptionPayload that refers to the key that the value was encrypted with so that the application can still decrypt the value even if the key is no longer the current key.
The way to set what key you want to use to encrypt moving forward after a key rotation you set the
"app.encryption.currentKey" = "key-id-2"

I like the approach. Consider thinking a little more about the KeyStoreFactory and how to make it extensible. Maybe let people bind whichever KeyStoreFactory they want to a service container alias or something. Idk, it's just kind of having to provide a class path in a config and expecting Laravel to know how to instantiate it.


model: gpt-4 does not exist

I have provided the valid OpenAI key in .env but I guess it is not valid for gpt-4
what to use for 3.5 ? The model gpt-3.5 does not exist
gpt-3 also fails
I am subscribed to GPT4 but I guess the OpenAI keys I get are for 3.5 only.  Should we use a waitlist to get them ?
thanks

Solved:
MODEL="gpt-3.5-turbo"
How to get the GPT4 keys ? thanks

I have provided the valid OpenAI key in .env but I guess it is not valid for gpt-4
what to use for 3.5 ? The model gpt-3.5 does not exist
gpt-3 also fails
I am subscribed to GPT4 but I guess the OpenAI keys I get are for 3.5 only. Should we use a waitlist to get them ?
thanks

Yep, you need to join the waitlist to get access to GPT-4.


Display tables or figures across two columns when two column layout is set globally

Description
Like this article does, place a table across two columns.

Use Case
Need to place a relatively wide table that cannot fit in a single column

Duplicate of https://github.com/typst/typst/issues/553


[MLIR] Add lowering for TF::SeluOp and TF::SeluGradOp

This PR adds the lowering for TF::SeluOp and TF::SeluGradOp to mhlo. Relevant test cases are also added.
This PR also adds TF to TF lowering for the same.
Signed-Off-By: Prateek Gupta<EMAIL_ADDRESS>

Hi @smit-hinsu can you please review this? Thanks!

Hi @smit-hinsu can you please review this? Thanks!

Gentle ping, @smit-hinsu .

Let's remove these ops from this list so that this patterns are used in all contexts:
https://github.com/tensorflow/tensorflow/blob/c83cdf85c5937730af51a384c0cc605bfb8f9214/tensorflow/compiler/mlir/xla/transforms/legalize_tf_with_tf2xla.cc#L226
I am also curious about your usecase that the above didn't work.
Sorry for the delay on this. Thanks for the contributions.

Please re-review @smit-hinsu. I have used DRRs now for the lowering. I am not sure about the code you have pointed. Can you elaborate on this?

Let's remove these ops from this list so that this patterns are used in all contexts:
https://github.com/tensorflow/tensorflow/blob/c83cdf85c5937730af51a384c0cc605bfb8f9214/tensorflow/compiler/mlir/xla/transforms/legalize_tf_with_tf2xla.cc#L226
I am also curious about your usecase that the above didn't work.
Sorry for the delay on this. Thanks for the contributions.

Please re-review @smit-hinsu. I have used DRRs now for the lowering. I am not sure about the code you have pointed. Can you elaborate on this?

How do you use the legalize pass? Do you have xla-legalize-tf-with-tf2xla pass in the pass pipeline?
xla-legalize-tf-with-tf2xla pass in that file is used to legalize ops mentioned in the list. That pass can already legalize Selu op but now that we will have native lowering after this change, we can drop it from the list to keep only one implementation. So, just remove Selu op from the list.

@pr4tgpt  Any update on this PR? Please. Thanks!

@gbaned I am currently working on the changes mentioned by @smit-hinsu in the lower_tf.cc. Will soon this update this PR.

Hi @smit-hinsu please re-review. I have added changes as you suggested. Thanks!

Gentle ping @smit-hinsu.

There were holidays in the US so will get to this on Tuesday or Wednesday.

There were holidays in the US so will get to this on Tuesday or Wednesday.

Gentle Ping @smit-hinsu .

@pr4tgpt Can you please check @smit-hinsu's comments and keep us posted ? Thanks!

@pr4tgpt Can you please check @smit-hinsu's comments and keep us posted ? Thanks!

Sure. I am working on this. Will update the PR soon.

@smit-hinsu Please re-review.

Hi @smit-hinsu, Thank you for your valuable comments. Addressed all. Please re-review. Thanks!

Gentle Ping @smit-hinsu .

Rebased and pushed.

Gentle ping @smit-hinsu.

Hi @smit-hinsu,  I am not sure why tests are failing, can you tell where it is failing? Thanks!

@pr4tgpt  Can you please resolve conflicts? Thanks!

Look at the failure log in this: https://tensorflow-ci.intel.com/job/tensorflow-mkl-linux-cpu-pr/10251/

Hi @smit-hinsu, rebased and merged.

Hi @smit-hinsu can you look into these failing tests. I dont think it is related to my code.


Add download receipt option to redeemed gift cards view

User story
As an Organization admin, I want one place to see all info about gift cards I've funded, including receipts.
Best solution for this problem
Add a download receipt link to the list of gift cards on the redeemed tab of the gift cards settings section.
MVP
We already have the MVP option to download receipts from the transactions list, but it would be much better to have everything in one place.

A single gift card can have multiple receipts, as it can be used for multiple hosts. How should we accommodate for this?
I also don't clearly understand the benefit of having that on this page vs having that in the transactions page, where you can download receipts by transaction or bundled by host.
Also important to note, there was a discussion to have a dedicated page for receipts in the settings (not sure where it's at, ping @Memo-Es @znarf)

A single gift card can have multiple receipts

Ah, I think I meant "gift card payment transaction" then. I thought the gift cards settings would show a list of when the gift cards had been used by the recipient. But maybe that's not the case? I was imagining a similar button to what we see on the transactions list to download the receipt. It could show in both places.
Maybe a better solution is to have a filter on the transactions view for gift card transactions? The problem was, the user is an admin of a large Organisation profile with lots of transactions, and wanted the receipts for the gift card transactions, so had to hunt through the big list of all transactions to find them.
Maybe add a 'gift cards' option here if the profile has any gift cards?

This is the first time I've heard a request for gift card receipts so perhaps it's an edge case and not really a priority.

Maybe a better solution is to have a filter on the transactions view for gift card transactions?

That would be the simplest to implement. Also I'm not sure if it's related, but gift card transactions were missing in the initial release of the transactions page and were added this week.

@znarf @Betree I'd like to work on this.
Is the agreed solution to add a filter to show only gift card transactions?

Is the agreed solution to add a filter to show only gift card transactions?

Yep, that sounds like the way to go!

Is the agreed solution to add a filter to show only gift card transactions?

Yep, that sounds like the way to go!


Corection wrong working as widget

When using in some template this way as widget:
{widget name="ps_linklist" hook="displayFooter"}
then module make no output, beacuse in this case in method renderWidget() parameter $hookName is null and there was no read from $configuration['hook']

For the QA, this fix is valid for old versions of PrestaShop 1.7. On the newest one, the $hookName is properly defined.

Then maybe it should be merged in a branch to release a v2.1.6?
The 3.0.0 version is only compatible with ><IP_ADDRESS> now


AVRO-3377: Deserialization of record of mangled Java class throws ClassCastException

The biggest change in this code is moving the mangling code from the compiler project to the core avro project in order to facilitate mangling at deserialization time.
Jira

[X] My PR addresses the following Avro Jira issues and references them in the PR title. For example, "AVRO-1234: My Avro PR"

https://issues.apache.org/jira/browse/AVRO-3377

Tests

[X] My PR adds the following unit tests OR does not need testing for this extremely good reason:

TestSpecificData.testCanGetClassOfMangledType

Commits

[X] My commits all reference Jira issues in their subject lines. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[X] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain Javadoc that explain what it does

Hey, is it still planned to merge this in 1.11.2? Is there a workaround in the meantime?

PR approved (as code for mangle is moved from SpecificCompiler to SpecificData) with some remarks :

Mangle can transform word with point as "dummy.int.float" into "dummy.int$.float$" for example, as unmangle method only apply on last char (==> "dummy.int$.float" in our case).
Mangle only applies to reserved word, so transform "dummy$" to itself, while unmangle apply to all word (==> "dummy" in our case).
As it's a lot of static method, it may could be extract in external class.

I'm also looking for a workaround until this ticket is implemented.

Hey, any idea of when this will be merged/released or any sample/tips on a workaround?

@hmsantos, i'm sorry, i forgot this PR, at this time, i didn't had right to merge PR.
Nevertheless, you have to resolve conflict before merging.

@kylec32 can you please resolve the conflicts in your PR? Thanks


Add method property to ajaxSettings

jQuery ajaxSettings now has a "method" property.  I've added support for it in the type definitions.

@johnnyreilly

Yup - I see it here: http://api.jquery.com/jQuery.ajax/
Thanks for your contribution @JohnKurlak!

Thanks!


Handling workflow/job cancelling

Is it possible to run the post run step even if the action was cancelled? I have had to delete the generated branches and restart all stuck jobs because of cancelling a running job.

Not currently! Though deleting the generated branch and restarting jobs is not necessary. You can delete the line in the generated file that corresponds to the lock held by the job that was cancelled.
One workaround is to add a timeout (#4) so that eventually, all jobs would complete. Another option is to integrate some GitHub API to check if the lock is held by an active job (requires much more work).


Manifest.write_checksum with checksum_algo: sha512 calls write_sha256

The case switch seems incorrect to me?
https://github.com/tipr/bagit/blob/master/lib/bagit/manifest.rb#L52
Happy to do PR if I'm correct.

Yeah that seems wrong. Feel free to submit a PR.

Yeah that seems wrong. Feel free to submit a PR.


Readiness probe failed: calico/node is not ready: BIRD is not ready: Error querying BIRD: unable to connect to BIRDv4 socket: dial unix /var/run/calico/bird.ctl: connect: connection refused

When I created a k8s cluster in binary mode and deployed cni through the calico plug-in, I found that Pods could not communicate with each other. Then I went to calico's pod events and found the following

Meanwhile, the configuration of my calico ds is as follows

however,i can't find any useful messages in "calico-node-……" pod;
I have collected some information to help me locate the problem. Now I find that the docker0 network card of my virtual machine is in down state, and the restart of docker still does not work after deleting the network card.
How can I solve this problem now? Thanks in advance
Expected Behavior
My Pods can access each other and this warnning could be solved
Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

i bulid this k8s cluster in binary mode which is base on https://blog.csdn.net/weixin_49497353/article/details/128900261;
calico's yaml ->calicoyaml.docx

Context

Your Environment
calico version:3.23.5
k8s version: 1.21.14
docker version: 20.10.17
centos 7.6

Hi @664182893 I'm unfortunately not able to read Mandarin so I can't look at the blog page you linked, but since it's not our official documentation could you instead try setting up your cluster according to our official documentation at https://docs.tigera.io/calico/latest/about/?

Use compiled  calico-node of  binary build images , will have the same problem!

maybe is that issue https://github.com/projectcalico/calico/pull/7556

In my case, this is because the other node's(<IP_ADDRESS>) bird is not running, recreating the pod will fix it.

@664182893 can you please try one of our official tutorials (https://docs.tigera.io/) and check if you still have any issue.

I get this error when following official tutorial https://docs.tigera.io/calico/latest/getting-started/kubernetes/self-managed-onprem/onpremises  . I have IP_AUTDETECTION_METHOD='tailscale*'

From inside the calico node, i can see there is an attempt to speak to the wrong interface:

I don't see any evidence in that output that it's attempting to use the wrong address. What makes you think that?

Closing this issue. Feel free to reopen if you have more information.


Using Image and Line/FilledContours for non-geographic plots

Until we have fully integrated the Image in HoloViews with the data interfaces the existing geoviews Image and Contours plots should also support plotting non-geographic data. Currently this works but has two major issues:

The iris plotting calls do not respect the order of dimensions resulting in mislabeling of the axes.

The aspect on these plot types is set to 'equal' by default which is sensible for 'longitude' and 'latitude' since those exist in the same space but often results in very wide or tall plots.

The aspect issue should be easy enough to fix as we can just detect whether a plot is geographic using the is_geographic utility function in geoviews.element.geo and set the aspect accordingly.
The axis mislabeling is more problematic as it will require us to make changes to iris plotting functions so we can respect the order of coordinates in the cube if desired.

This is now supported, closing.


Rename '@SnapshotAssertions'

I don't like the name of the annotation that enables the snapshot testing capabilities, but I'm not quite sure what would be better.

@EnableSnapshotAssertions
@EnableSnapshotTests
...

Starting from 0.0.6 it will be @EnableSnapshotTests