Build C compiler errors when building from source on RHEL v8.9

Due to issues running switcher 0.8.1 om REL8 (see issue 104 I tried to build it from source using the 0.8.1 source package.  The OS I'm running is RedHat Enterprise Linux 8.9 on x86_64.
x@y: /dev/shm/kubeswitch-0.8.1$ uname -a
Linux x.a.b 4.18.0-513.9.1.el8_9.x86_64 #1 SMP Thu Nov 16 10:29:04 EST 2023 x86_64 x86_64 x86_64 GNU/Linux

x@y: /dev/shm/kubeswitch-0.8.1$ cat /etc/redhat-release
Red Hat Enterprise Linux release 8.9 (Ootpa)

However when running the make all of the downloaded source package I'm running into issues with the C compiler invocation. No matter if I try to compile using clang or gcc.
Compiler versions I've tested with are:
gcc
x@y: /dev/shm/kubeswitch-0.8.1$ gcc -v
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/libexec/gcc/x86_64-redhat-linux/8/lto-wrapper
OFFLOAD_TARGET_NAMES=nvptx-none
OFFLOAD_TARGET_DEFAULT=1
Target: x86_64-redhat-linux
Configured with: ../configure --enable-bootstrap --enable-languages=c,c++,fortran,lto --prefix=/usr --mandir=/usr/share/man --infodir=/usr/share/info --with-bugurl=http://bugzilla.redhat.com/bugzilla --enable-shared --enable-threads=posix --enable-checking=release --enable-multilib --with-system-zlib --enable-__cxa_atexit --disable-libunwind-exceptions --enable-gnu-unique-object --enable-linker-build-id --with-gcc-major-version-only --with-linker-hash-style=gnu --enable-plugin --enable-initfini-array --with-isl --disable-libmpx --enable-offload-targets=nvptx-none --without-cuda-driver --enable-gnu-indirect-function --enable-cet --with-tune=generic --with-arch_32=x86-64 --build=x86_64-redhat-linux
Thread model: posix
gcc version 8.5.0 20210514 (Red Hat 8.5.0-20) (GCC)

clang:
c@y: /dev/shm/kubeswitch-0.8.1$ clang -v
clang version 16.0.6 (Red Hat 16.0.6-2.module+el8.9.0+19521+190d7aba)
Target: x86_64-redhat-linux-gnu
Thread model: posix
InstalledDir: /usr/bin
Found candidate GCC installation: /opt/rh/gcc-toolset-13/root/usr/lib/gcc/x86_64-redhat-linux/13
Selected GCC installation: /opt/rh/gcc-toolset-13/root/usr/lib/gcc/x86_64-redhat-linux/13
Candidate multilib: .;@m64
Candidate multilib: 32;@m32
Selected multilib: .;@m64

go
x@y: /dev/shm/kubeswitch-0.8.1$ go version
go version go1.21.6 linux/amd64

Building using the default C compiler (clang) it fails using the following issues:
x@y: /dev/shm/kubeswitch-0.8.1$ make all
> Format
> Test
ok  	github.com/danielfoehrkn/kubeswitch/pkg/config/migration	(cached)
ok  	github.com/danielfoehrkn/kubeswitch/pkg/config/validation	(cached)
> Check
Executing golangci-lint
/dev/shm/kubeswitch-0.8.1/hack/tools/bin/golangci-lint
INFO [config_reader] Config search paths: [./ /dev/shm/kubeswitch-0.8.1 /dev/shm /dev / /home/x]
INFO [lintersdb] Active 7 linters: [errcheck gosimple govet ineffassign staticcheck typecheck unused]
INFO [loader] Go packages loading at mode 575 (exports_file|files|imports|types_sizes|compiled_files|name|deps) took 369.488726ms
INFO [runner/filename_unadjuster] Pre-built 0 adjustments in 6.044975ms
INFO [linters_context/goanalysis] analyzers took 0s with no stages
INFO [runner] Issues before processing: 12, after processing: 0
INFO [runner] Processors filtering stat (out/in): cgo: 12/12, skip_dirs: 12/12, exclude: 12/12, exclude-rules: 0/12, autogenerated_exclude: 12/12, filename_unadjuster: 12/12, path_prettifier: 12/12, skip_files: 12/12, identifier_marker: 12/12
INFO [runner] processing took 348.727µs with stages: identifier_marker: 128.002µs, autogenerated_exclude: 105.193µs, path_prettifier: 56.306µs, skip_dirs: 40.314µs, exclude-rules: 12.234µs, cgo: 1.966µs, nolint: 983ns, fixer: 692ns, max_same_issues: 545ns, skip_files: 436ns, filename_unadjuster: 405ns, severity-rules: 331ns, exclude: 281ns, diff: 223ns, max_per_file_from_linter: 163ns, source_code: 150ns, max_from_linter: 127ns, sort_results: 106ns, uniq_by_line: 105ns, path_prefixer: 83ns, path_shortener: 82ns
INFO [runner] linters took 219.884616ms with stages: goanalysis_metalinter: 219.490396ms
INFO File cache stats: 0 entries of total size 0B
INFO Memory: 7 samples, avg is 40.3MB, max is 67.3MB
INFO Execution took 598.978578ms
Executing go vet
Executing gofmt/goimports
Check for license headers
All checks successful
# runtime/cgo
clang-16: error: argument unused during compilation: '-arch x86_64' [-Werror,-Wunused-command-line-argument]
make: *** [Makefile:34: build-switcher] Error 1

Building using the (gcc) it fails using the following issues:
x@y: /dev/shm/kubeswitch-0.8.1$ export CC="gcc"
x@y: /dev/shm/kubeswitch-0.8.1$ make all
> Format
> Test
ok  	github.com/danielfoehrkn/kubeswitch/pkg/config/migration	(cached)
ok  	github.com/danielfoehrkn/kubeswitch/pkg/config/validation	(cached)
> Check
Executing golangci-lint
/dev/shm/kubeswitch-0.8.1/hack/tools/bin/golangci-lint
INFO [config_reader] Config search paths: [./ /dev/shm/kubeswitch-0.8.1 /dev/shm /dev / /home/x]
INFO [lintersdb] Active 7 linters: [errcheck gosimple govet ineffassign staticcheck typecheck unused]
INFO [loader] Go packages loading at mode 575 (compiled_files|deps|exports_file|files|imports|name|types_sizes) took 364.056768ms
INFO [runner/filename_unadjuster] Pre-built 0 adjustments in 6.909621ms
INFO [linters_context/goanalysis] analyzers took 0s with no stages
INFO [runner] Issues before processing: 12, after processing: 0
INFO [runner] Processors filtering stat (out/in): path_prettifier: 12/12, skip_dirs: 12/12, identifier_marker: 12/12, cgo: 12/12, autogenerated_exclude: 12/12, exclude: 12/12, filename_unadjuster: 12/12, skip_files: 12/12, exclude-rules: 0/12
INFO [runner] processing took 338.257µs with stages: autogenerated_exclude: 114.188µs, identifier_marker: 95.274µs, path_prettifier: 62.914µs, skip_dirs: 45.893µs, exclude-rules: 13.415µs, cgo: 2.387µs, nolint: 959ns, filename_unadjuster: 637ns, max_same_issues: 479ns, skip_files: 393ns, fixer: 303ns, severity-rules: 233ns, diff: 232ns, exclude: 229ns, max_from_linter: 143ns, sort_results: 109ns, source_code: 105ns, uniq_by_line: 102ns, path_shortener: 94ns, max_per_file_from_linter: 87ns, path_prefixer: 81ns
INFO [runner] linters took 188.482193ms with stages: goanalysis_metalinter: 188.09972ms
INFO File cache stats: 0 entries of total size 0B
INFO Memory: 7 samples, avg is 40.0MB, max is 66.8MB
INFO Execution took 562.844673ms
Executing go vet
Executing gofmt/goimports
Check for license headers
All checks successful
# runtime/cgo
gcc: error: x86_64: No such file or directory
gcc: error: unrecognized command line option '-arch'; did you mean '-march='?
make: *** [Makefile:34: build-switcher] Error 1

The strange thing is however that the go binary is being built nevertheless as hack/switch/switcher_linux_amd64 and this still seems to work well - at least using local store.
So I'm wondering what's causing the compile error and what are the consequences of the failed compile (as the go binary is built anyhow).

Thanks for the detailed report. I'll try to look into the issue.


Update Redhat family spec file, minimum version python-zmq 14.5

see https://github.com/saltstack/salt/issues/28184


Mocking better (or at all!) in a fully static/non-nullable Dart

We talked about this a little in Seattle today.
Mockito uses noSuchMethod to metaprogram/implement missing methods in runtime based on features/API from the Java Mockito and uses Matcher to define what is legal.
Already, this breaks down in strong-mode:
import 'package:mockito/mockito.dart';

class Animal {
  String eat(Food food) { ... }
  void move({int x, int y}) { ... }
}

class MockAnimal extends Mock implements Animal {}

void main() {
  var animal = new MockAnimal();
  when(animal.eat(any)).thenReturn('Yum!');
  //              ^^^
  //              Is type: Matcher, which is not of type Food.
}

This used to work pre-strong mode, but now requires a hacky typed API:
// Approximately how `typed` is implemented
typed(matcher) {
  _recordTypedMatcherForLaterUse(matcher);
  return null;
}

void main() {
  var animal = new MockAnimal();

  // Ugly
  when(animal.eat(typed(any)).thenReturn('Yum!');

  // Even uglier. We can't guarantee what order named parameters will be called on, so we
  // need users to manually record the name of the argument.
  when(animal.move(x: typed(any, name: 'x'), y: typed(any, name: 'y')).then(...);
}

So we lose the nice idiomatic Dart feel and more error-prone boilerplate is involved.
Unfortunately, even this fix relies on a hacky solution that always returns null... What happens when we introduce non-nullables, i.e., imagine this:
// Assume this is in a version of Dart with non-nullability by default.
void main() {
  when(animal.eat(typed(any)).thenReturn('Yum!');
  //             ^^^^^^^^^^^
  //             The only value legal here is Food. How do we say "any food"?
}

We maybe can get around this by again, being bad, assuming this works in static Dart:
dynamic typed(matcher) {
  _recordTypedMatcherForLaterUse(matcher);
  return new Object() as dynamic;
}

I've outline an alternative API idea in https://github.com/dart-lang/sdk/issues/27562 that could help, if we could do something like:
T any<T>() => implement(T, _AnyMatcher);
//            ^^^^^^^^^^^^^^^^^^^^^^^^^
//            Create a new anonymous subclass of T, i.e.
//            class _Anonymous${T} extends _AnyMatcher implements T {
//              noSuchMethod(_) {}
//            }

This would allow removing the typed API, as well as working properly/legally in strong-mode.
/cc @srawlins

Note that is could also be done properly with code generation, but it seems really unfortunate to require a code-generation step for something like this - maybe I could change my mind if we had some static meta-programming/macros.

The API is hacky. It uses the same method call for two things:
when(animal.eat(any)).thenReturn('yum');
...
animal.eat(hay);

That's going to be hard to do in a typed setting because the first call is not type-correct.
It used to work because the type might be wrong, but the call actually works because noSuchMethod don't care about types or signatures at all.
Maybe it would be better if it was instead written:
when(animal).eat(any).thenReturn('yum');

That way the "programming interface" for the animal is not the animal itself, it's a new object returned by when which can program the animal.
You can even use cascades then:
when(animal)..eat(any).thenReturn('yum')..isEating.thenReturn(true);

(if that makes sense at all).

/cc @srawlins who can update plans for mockito in the future

We found a solution for the near term, because @srawlins is very smart.
If we ever want to introduce non-nullability, we will need a real solution.


fastwalk: don't clean the path argument to Walk on Windows

On Windows don't attempt to clean the path argument to Walk since the existing clean logic (cleanRootPath) transforms paths like "C:" => "C:" which are not equivalent.
This logic only existed to make the joining of paths simpler and should probably be removed since we shouldn't be modifying user provided paths.
TODO: Investigate if anything relies on the current clean logic and remove it if nothing does.
Fixes: https://github.com/charlievieth/fastwalk/issues/37

Fixes: https://github.com/junegunn/fzf/issues/4027


Postlight custom scripts

This PR includes:

changes to the overall template config.
modifications to the template structure to reflect new features such as flow and css-modules.
Read Me update to showcase the usage as well as description of the fork.

👍
Looks good to go and ready for usage


add JanuriDP

hey there, that is my work for week 1 hehe
if i can, i will participate on week 2 challange.
peace

Your pull request needs further work because of the following reason(s):

Your file has a github link but it's (probably) not to your profile.

Once you have resolved the issues, comment on this request to be re-judged
This is an automated response, beep boop.
if you think there is a mistake, contact a maintainer

45df39c825b65a85391b97cdf15f0085e42cbdf8

i dont understand
https://github.com/januridp/ that link is mine

Its bothered by the trailing slash.  Youre right though. Ita fine


Feat login

There is a failed SSR hydration in this branch

There is some margin here in the root div, i think it should be removed to match the figma, otherwise looks good to me!

Additionally the toast box is at the bottom, perhaps @yunruu @shishirbychapur ya'll can take a look at this too

Additionally the toast box is at the bottom, perhaps @yunruu @shishirbychapur ya'll can take a look at this too

whats wrong?

the toast can be fixed later imo, currently the page isn't functional yet, and the error message is generic. for now, i think it's fine


Use RabbitMQ recommended configuration

https://www.rabbitmq.com/priority.html#overview
https://docs.celeryq.dev/en/latest/userguide/routing.html#rabbitmq-message-priorities

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 88.086%

Files with Coverage Reduction
New Missed Lines
%

safe_transaction_service/history/services/collectibles_service.py
2
90.91%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
5708

Relevant Lines:
6480

💛  - Coveralls


Create and activate a virtual environment could specify more details about the creation of the virtual environment folder.

Hello,
I would like to share feedback on this documentation on the "Create and activate a virtual environment" section which says: "In a suitable folder, run the following commands to create and activate a virtual environment named .venv." After reviewing this with a customer, the paraphrasing "in a suitable folder" is not clear enough as is not confirming if the root folder of the project or any other folder of the project is a valid one for the virtual environment, this would help to prevent creating the venv folder in a path that could cause conflicts in your python project. Therefore, we believe that providing some samples of suitable paths would be helpful to clarify this step.
Thank you!

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 8b1ff97a-0ca7-c91b-5b45-370212407a51
Version Independent ID: b60719ca-3e64-8a00-75bb-dda1d8119b21
Content: Create a Python function from the command line - Azure Functions
Content Source: articles/azure-functions/create-first-function-cli-python.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@v-jesume
Thanks for your feedback! We will investigate and update as appropriate.


Implemented LED stacks and removed cache, updated tests

Now led_player and shows fully restore lower priority led states when they stop or when a mode ends.

Looks very good to me. Two small notices:

You may use self.debug_log instead of if self.debug: self.log.debug()
Please fix the two PEP8 style issues


Tree View bug when opening multiple files from command line.

If I open a single file with Atom from a command line (atom some_file.cpp), it opens Atom, displays the current directory and its contents in the Tree View and displays the file which was passed as the argument.  So, everything works properly.

However, if I open multiple files (atom some_file1.cpp some_file2.cpp some_file3.cpp), it opens Atom, display the files which were passed as the arguments, but does something strange in the Tree View:

As you can see, instead of the current directory and its contents, it displays each file from the argument list as a directory.  However, they're all empty.  If I close the displayed files, there is no way to re-open them by clicking those empty directories.... they're just empty artifacts.  If I right click and show each of those directories in the explorer, it opens the proper current directory in the explorer.  If I right click and choose to open in the new window, it'll open a new instance of Atom with the selected file displaying and proper Tree View on the left.
Note that this only occurs if you are doing this with existing files.  If any of the arguments is a non-existing file, then you get a mixture of the two cases:

If all of the arguments are non-existing files, then proper Tree View is displayed, as none of the files actually exist, so they're just displayed and need to be saved.
I've tried this on two different machines and its the same result.  So, sounds like a bug.
Windows 7 SP1 & Windows 10
Atom 1.0.19

Thanks for the detailed report. I noticed you also opened https://github.com/atom/tree-view/issues/632 -- I don't think there's a need to have two issues track the same problem, so I'm closing this in favor of that issue.

Yeah, sorry.  I wasn't sure if both trackers are monitored or merged somehow, so I kind of did a shotgun effect there.


feat(devcontainer): add the ability to use GitHub Codespaces and VS Code remote containers

Description
This project is marked as open to contribution for Hacktoberfest, and as such I figured it would be desirable for contributors to have a simplified means of getting up and running with the developer setup. With the addition of the devcontainer configuration, this project is now compatible with GitHub Codespaces as well as VS Code Remote Containers. Contributors who use one of these options now get all of the dev setup steps completed for them, and you get peace of mind that contributions will come from setups with the correct usage of eslint, prettier, vitest, etc.
Additional context
Demo using GH Codespaces
Demo

What is the purpose of this pull request?

[ ] Add quotes
[ ] Bug fix
[x] New Feature
[ ] Add more tests
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123 or closes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

@AnilSeervi Great! Any idea why the build (12.x) check is not making progress? It's been unchanged for like 12 hours :smile:

@AnilSeervi Great! Any idea why the build (12.x) check is not making progress? It's been unchanged for like 12 hours 😄

No idea @andrewbrey.I haven't even put 12.x node version in the workflow and still it expects for it.

@AnilSeervi perhaps you could merge this PR (without that check being completed) and take a look at your GH actions setup separately? :smile:


Docs/fix doc

just some jsdoc and markdown linting fixes
i saw that the project is not using prettier to format files, i could add it if you want.

Added prettier thanks for the suggestion!


OIDC Federation Test Case

Test that includes logout flow also has been sent with the PR: https://github.com/wso2/product-is/pull/7039. Hence closing this.


Messaging: BigQuery Migration (EU)

Context
We missed EU users in a previous deprecation notice. Luckily, we're extending the deadline anyway, so we can roll an update out to them which still gives them time.
Audience / Targeting
Tomas has provided a list of impacted EU users.
Resources, timeline, responsibilities
A single message, essentially combining messages from https://github.com/PostHog/meta/issues/137 and https://github.com/PostHog/meta/issues/147
Preview
Coming soon

Cohort: https://app.posthog.com/cohorts/48235

For your approval, @tomasfarias

Looks perfect! I would perhaps omit the "you can disable the old export app" as we disable that as part of the migration, and if users disable it early they could miss data. This data can be re-exported, but I rather err on the side of having more data than less data.

Done, and scheduled!


Universal Question with Variable Supporting Text.

It is important to have universal questions that can improve the experience of resident and help program coordinators pull data that can be measure across departments. However some universal questions are not appropriate for certain programs. For example, this is our universal address question "Provide your current address. If currently experiencing housing instability, please list nighttime address." This question is not appropriate for programs targeting homeowners. Maybe we can use universal questions with variable supporting text. The second sentence in the example question would be part of variable supporting text.

moving to backlog as there is a workaround using static text.  I'm sure there are other use cases for this that will come up in the future.


White vertical line in calendar

Hi, I'm using the latest version (7.0.4) and experiencing issue similiar to #26, #205, #227
Here is the screenshot (using iPhone 5s device)

And here is my setup
    func setupCalendarView() {
        calendarView.minimumLineSpacing = 0
        calendarView.minimumInteritemSpacing = 0
        calendarView.allowsMultipleSelection = true
        calendarView.isRangeSelectionUsed = true
        calendarView.cellSize = calendarView.frame.size.width / 7

        calendarView.register(
            UINib(nibName:"DatePickerHeader", bundle: Bundle.main),
            forSupplementaryViewOfKind: UICollectionElementKindSectionHeader,
            withReuseIdentifier: "CustomHeader"
        )

        let startDate = Date()
        let endDate = Calendar.current.date(byAdding: .day, value: 1, to: startDate)!
        calendarView.selectDates([startDate, endDate])
    }

I have a view inside cell for coloring selected date, the view constraint is set to fill the cell (top, left, bottom, right = 0).
I have also tested on iPhone 7 (Simulator) and its working fine
However, iPhone 7 Plus (Simulator) shows ~1px overlap (see the red circle)

Any advice regarding this issue? Am I missing something?
Great project btw, like it ;)

That vertical line is the same as you will get if you implement any other UICollectionView.
To get rid of it, simply set the width of the calendarView to be something that is divisible by 7 such that the value is not a recurring decimal value.
Look on This Link on how this guy does it with his collectionView
On that link look for the line that says ->
Things I learn:

    All kinds of things with UICollectionView (didSelect, didHighlight, shouldSelect, shouldHighlight, de/selecting cells programatically, etc.)
    One issue that made me stuck for days was there was tiny pixel gap between date cells that I couldn't remove. I posted the solution here: http://stackoverflow.com/a/42574952/2105910

I tried to apply the solution, it seems like it only works if the calendar is full width (same as device width)
In my case, calendar is showed like a modal which is why i wrapped calendar inside another view with padding (like the photo in my previous post), it's not working if i divide my wrapper by 7
// By changing 'view' to my 'wrapperView'
var itemWidth: CGFloat {
    get {
        return floor(wrapperView.frame.size.width / 7)
    }
}

And also if i remove my wrapperView and apply full width to my calendar, my header is also resized which make white line appear in left side of my header

The answer is not to divide by seven. The answer is to set the calendar's width to a size that is divisible by seven without a recurring decimal place.
in your code, can you tell me what is the value of this? -> wrapperView.frame.size.width

wrapperView.frame.size.width is 315 regardless of devices
I then apply this
        let gap = wrapperView.frame.size.width - (itemWidth * 7)
        calendarView.widthAnchor.constraint(equalTo: wrapperView.widthAnchor, constant: -gap).isActive = true
        calendarView.cellSize = itemWidth

Hey can you join gme here? https://gitter.im/patchthecode/JTAppleCalendar

I apply this as you suggested and its working fine right now. Haven't spot any issue so far
Thanks


adds toJSON method to orbitdb address class

Currently json serializing an orbitdb address results in the json value { root: "<root>", path: "<path>" } which throws an error when fed back into address.parse.
Adding a toJSON method that returns this.toString() would make the json serialized address again parseable by the address class.
This is small change but is breaking the behavior of how json.stringify serializes the address currently.

any reviewers?

Correct me if I'm wrong. I think this can be closed.


[Improvement] Make the coordinator client managed by CoordinatorClientFactory singleton

Code of Conduct

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

Search before asking

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

What would you like to be improved?
We should make the coordinator client managed by CoordinatorClientFactory singleton like the ShuffleServerClientFactory does.
How should we improve?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Hi @zuston
It's been a while since this issue existed and I'd like to help it if nobody is working on it currently.

Feel free to pick up

How about let's me try and contribute the issue if no body try it.


[0.11.0-rc1] Document set rules changed

Description
Hi!
I discovered this inconsistency in behavior between v0.11.0-rc1 and v0.10.0 . Here is the example:
#let titlepage(
  title: "Title",
) = {
  set document(title: title)

  align(center, text(1.5em, weight: 100, title))

  pagebreak()
}

#let disclaimer(
  author: "Me",
) = {
  set document(author: author)
  set page(
    margin: (left: 30mm, right: 30mm, top: 40mm, bottom: 40mm),
    numbering: none,
    number-align: center,
  )

  v(75%)
  text(lorem(20))

  v(15mm)
  grid(
      columns: 2,
      gutter: 1fr,
      "Arrakis, " , author
  )
}

#titlepage()

#disclaimer()

v0.10.0 compiles it but v0.11.0-rc1 gives this error:
error: document set rules must appear before any content
  ┌─ ../thesis-template-typst/thesis_typ/disclaimer.typ:7:2
  │
7 │   set document(title: title, author: author)
  │   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Here is the original opened issue.
Reproduction URL
No response
Operating system
Linux
Typst version

[X] I am using the latest version of Typst

Yes, this was a bug fix that I forgot to include in the changelog. I'll add it to the full release changelog.
Previously, Typst unintentionally allowed set document directly after a page break, but just ignored it. For it to work, set document really must be the first thing and affect everything, with no title page or anything before it and no unaffected content after it.


Added innovation logging to LPE.

Closes #4762 .

Note baro/vision/mocap innovations are not logged since there isn't a place to squeeze them in existing ekf_innovation message.

@jgoppert Nice!

@tumbili I think ideally we would want an innovation vector for each sensor.

@jgoppert Yes, also we can rename the topic to "estimator_innovations" I guess.


add sonatype maven packages support (future referance)

Sonatype has recently updated its UI and added a score to the packages on their repository

For this, we first need to support Java.
We need examples for installing commands and links to packages.


How can we convert response of dbClient.PostFind(postFindOptions) into struct?

I am unable to convert the response into struct. below is my code ..
findResult, _, err := dbClient.PostFind(postFindOptions)
if err != nil {
fmt.Println("err", err)
}
b, _ := json.MarshalIndent(findResult, "", "  ")
how can we convert findResult into struct? please suggest the solution.
Thanks in advanced!

The findResult is a FindResult struct, that is what is returned by PostFind.

Thanks Rich for your quick reply. Actually I am getting 'Docs' from FindResult struct and want to have docs values in my customized struct where i can apply my logic.

Actually I am getting 'Docs' from FindResult struct and want to have docs values in my customized struct where i can apply my logic.

Sorry that wasn't clear to me from the original question.
You have 2 choices:

write a converter to accept the built-in Document struct type and use GetProperty/GetProperties to retrieve the properties you need and apply them to your own struct type.
use raw IO with PostFindAsStream to bypass the built-in types and deserialize directly into your own structs.


Syft creates two LicenseRef- that differ only by case

What happened:
In the following file:
syft-container5.spdx.json.txt
Syft has created two LicenseRef- that differ only by case:
LicenseRef-BSD-3-clause-generic

LicenseRef-BSD-3-clause-GENERIC

This is invalid SPDX. See https://github.com/spdx/tools-java/issues/143
What you expected to happen:
Syft should create two distinct indentifiers.
Steps to reproduce the issue:
Anything else we need to know?:
Environment:

Output of syft version: syft-0.86.1

Today based on the output I'm seeing this:
  {
   "licenseId": "LicenseRef-BSD-3-clause-GENERIC",
   "extractedText": "BSD-3-clause-GENERIC"
  },
  {
   "licenseId": "LicenseRef-BSD-3-clause-generic",
   "extractedText": "BSD-3-clause-generic"
  },

Syft should create two distinct indentifiers.

Wouldn't better behavior be that we have a single identifier? Or are you saying that we really should keep the two distinct nodes and add a suffix to ensure they are unique? For example:
  {
   "licenseId": "LicenseRef-BSD-3-clause-GENERIC-1",
   "extractedText": "BSD-3-clause-GENERIC"
  },
  {
   "licenseId": "LicenseRef-BSD-3-clause-generic-2",
   "extractedText": "BSD-3-clause-generic"
  },

(note the 1 and 2 suffix on the licenseID)

If the extracted text is exactly the same, yes it is best to have a single identifier.
If the extracted text is different, you need to have two distinct identifiers.
In this particular case, you might consider or not that extracted text is the same (as it differs only in case).
My only request is that you never create two indentifiers that only differ in case, because that is invalid SPDX, as the indentifiers are case-insensive according to the SPDX standard.

Hi @vargenau, thanks for the update. I think we have a path forward to de-duplicate identifiers in a case-insensitive way. We'll put this issue on our backlog to fix when we are able.

I think we can leave the license structs that are attached to the package alone in the core syft model and make this a deduplication concern in the spdx format encoder, where we can take one or more strategies (either hashing the id or extracted text content and using as a suffix, or something else).


[flake-update] Enable GitHub auto-merge

https://stackoverflow.com/a/72259998/428586

Closed in #105


tpm2_getmanufec: check for null arguments in Base64Encode and Retriev…

…eEndorsementcredentials
With HashEKPublicKey now possibly returning NULL, call chain should
check for that possibility.
Fixes: 2b6bb441f478 ("tpm2_getmanufec: fix issues")
Signed-off-by: Jerry Snitselaar<EMAIL_ADDRESS>

Redid commit with spaces in place of indents


UOT-121280 add tags and custom properties to search

I have not been able to login to qlik dev server to add info on apps so that I can test tags in search and multiple objects in custom properties array.


kube-vip arm/v7 images in dockerhub ?

arm/v7 images are not available for all kube-vip versions.
Here is the current status in dockerhub (https://hub.docker.com/r/plndr/kube-vip/tags?page=1&ordering=last_updated):

0.3.0 --> arm/v7 image available but kube-vip version 0.3.0 is not yet mentioned in the documentation (https://kube-vip.io/hybrid/#changelog) so I don't really know if I can start using it or not?
0.2.3 --> arm/v7 image not available
0.2.2 --> arm/v7 image not available
0.2.1 --> arm/v7 image looks available but I'm not able to pull it:

root@master-02:~# podman pull plndr/kube-vip:0.2.1
Completed short name "plndr/kube-vip" with unqualified-search registries (origin: /etc/containers/registries.conf)
Trying to pull docker.io/plndr/kube-vip:0.2.1...
  unsupported docker v2s2 media type: "application/vnd.oci.image.layer.v1.tar+gzip"
Trying to pull quay.io/plndr/kube-vip:0.2.1...
  unauthorized: access to the requested resource is not authorized
Error: 2 errors occurred while pulling:
 * Error initializing image from source docker://plndr/kube-vip:0.2.1: unsupported docker v2s2 media type: "application/vnd.oci.image.layer.v1.tar+gzip"
 * Error initializing source docker://quay.io/plndr/kube-vip:0.2.1: Error reading manifest 0.2.1 in quay.io/plndr/kube-vip: unauthorized: access to the requested resource is not authorized

0.2.0 --> arm/v7 image not available

0.3.0 is the recommended release.. I just haven't had the opportunity to finish it before the holidays... I'll try and complete it today/tomorrow.

All images should now be auto-generated.


Crashed while redirecting

I have tabNavigator and three stackNavigators one per tab.
In some action I go from one tab (and stack of course) fo another tab and stack within it and then I got this error "Cannod read property setNativeProps of undefined"
.\node_modules\react-navigation\node_modules\react-native-tab-view\src\TabViewPagerPan.js
252 line of
this.props.position.setValue(toValue); // toValue = 0;
failed on
.\node_modules\react-native\Libraries\Animated\src\AnimatedImplementation.js
1796 line
if (this._component.setNativeProps) {
Thank you for helping

software
version

react-navigation
1.0.0-beta.11

react-native
0.44.1

node
v7.7.4

npm
4.1.2

So after short investigation I've found that it was issue in my code:
in same moment I was call
navigation.dispatch(NavigationActions.reset({         index: routes.length - 1,         actions: routes.map(route => NavigationActions.navigate({routeName: route, params: { reset: true }}))     }));
and
navigation.navigate(navigation.state.params.returnRoute);
obviously in this case nav inserted wrong parameters for animation (or react-native-tab-view).
It can be resolved by dirty hack with InteractionManager.runAfterInteractions for example.


{humble}: delete unneeded patch and appends

Missing cstdint include bug is fixed in rosbags project. https://github.com/ros2/rosbag2/pull/1321

The same patch is added in master/humble. I think the PR is applicable to master/humble.
For iron/jazzy/rolling the patch is not included.

Merged into kirkstone: https://github.com/ros/meta-ros/commit/b4db6314bae1c78498158e267200a9c15a739cad

@robwoolley, I think this patch is not needed and it's causing the compilation to fail, please check the issue for further informations.


Use of a table alias does not work with soft delete

GORM Playground Link

https://github.com/go-gorm/playground/pull/541
Description

Our REST API supports an expression filter parameter which is parsed and turned into generated sql code using table aliases. This works fine as long as the model doesn't support soft deletes. If the model supports soft delete the query fails due to the generated is null clause for the soft delete column being generated with as "[table-name]".[soft-delete-column]".

Linking to PR with fix:
https://github.com/go-gorm/gorm/pull/5841

Hi @robhafner, may I ask if you are still working on this issue? If not, I would like to work on it.

Hi @yaofeng-wang, feel free to work on it as I have not had time lately to take another look at it. Thanks!


weights link is broken

can you please reupload your weight file?

can you please reupload your weight file?

https://github.com/yashk2810/Image-Captioning/blob/master/weights/time_inceptionV3_1.5987_loss.h5?raw=true
This link works


Incorrect annotation

Please see the failing test: tests/pdk/finfet/test_annotation.py
The netlist below is mapped to a level shifter:
        mp29 v4 vbias2 v2 vccx p w=2.16e-6 m=1 nf=12
        mp33 vbias2 vbias2 vbias1 vccx p w=1.44e-6 m=1 nf=8

This is wrong as the transistor widths are not the same.

@kkunal1408 , this is the issue from the LDO amp that I was showing earlier Today. Annotation should require matched transistor parameters m/nf/w/nfin.

@kkunal1408 , this is ready for review.


Performance difference for GEMM & flash-attention between reported and reproduced in CI

Current CI runs POC branches for flash-attention perf_attn and for GEMMs triton_perf_poc and runs provided run_all.sh scripts. Custom libigc is installed libigc1_1.0.24994.16243-igc+releaseinternal1_amd64.deb. However, there is a significant performance difference between reproduced and reported.
For example in flash attention CI reproduction gets about 2.5TFlops for Z:1,H:32,N_CTX:16384,D_HEAD:64, while reported value is close to 81 TFlops. Similar difference exists for GEMM results.
Results can be accessed as CI artefacts here (including raw summary files from run_all.sh): https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/9637082068

I can reproduce reported numbers in local IDC (8 cards, MAX 1550) server for both cases with:

pytorch-gpu-dev-0.5.1
gfx-driver-ci-comp_igc-24994

Also similar data when changing to use custom libigc1_1.0.24994.16243-igc+releaseinternal1_amd64.deb.

The CI workflow looks good, not found big difference in dumped ir and asm files between CI data and local ones.

Does this machine dedicated ? I see there are 2 GPU cards with max frequency 1600, Can you check their health using like: xpu-smi health -l or xpu-smi dump -d 0 -m 0,1,2 -i 1 -n 5

I think CPU should have limited impact.
Latest build failed to benchmark gemm and attention (llvm-target): https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/9830987255
But it cannot be reproduced in other branch(like a mirror branch): https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/9833038783
Needs to double confirm with this.

This benchmarks on POC branches will not be merged into the main branch directly, We are integrating them into Triton benchmark like softmax now.
Related:
https://github.com/intel/intel-xpu-backend-for-triton/pull/1539
https://github.com/intel/intel-xpu-backend-for-triton/pull/1597

Revisit after integrating to the POC branch to the default one.

Depends on https://github.com/intel/intel-xpu-backend-for-triton/issues/1450 and https://github.com/intel/intel-xpu-backend-for-triton/issues/1451

since #1450  is closed, please check there should not be perf diff @ESI-SYD @LiyangLingIntel

Update: GEMM done
FlashAttention blocked by it's unavailability on the llvm-target branch.
Assertion `detail::isPresent(Val) && "dyn_cast on a non-existent value"' failed.

Tracked in: https://github.com/intel/intel-xpu-backend-for-triton/issues/1758

Update:
Triton's FlashAttention wip #1758
XeTLA's FlashAttention done #1877

Remaining FA performance gap tracked in https://github.com/intel/intel-xpu-backend-for-triton/issues/1758.


[POOL-402] Check blockWhenExhausted in hasBorrowWaiters

Hi, I am a user of Apache commons pool2. I found that the performance of returnObject(K) would be improved a lot if I do a pre-check getBlockWhenExhausted() in hasBorrowWaiters(). The flame graphs are shared in the following link for comparison. Thank you!
https://drive.google.com/drive/folders/11NLW4nzGMxxPwO_xjUa_nnZGciMrkpfn?usp=sharing

Coverage decreased (-0.03%) to 84.995% when pulling 9809f6f7735ae242002ed50142c7389ebb655a06 on Cp-John:branch-refine-returnObject into 90eab321ad32868dc80c42d7d08ac086d44146a0 on apache:master.

Thanks for the patch.  Looks good to me and will definitely improve performance for pools configured not to block when exhausted.

Created placeholder ticket: https://issues.apache.org/jira/browse/POOL-402

Looks harmless and, as @psteitz pointed out, looks like it will improve performance for some pools. Created placeholder issue, and looked at coverage.

Even though it shows the line as covered, I believe not all branches of the condition statement are covered. Not sure if it'd be easy to add a unit test for when getBlockWhenExhausted() is true and when it's false (maybe to avoid regressions)?
Thanks Bruno

Hi Bruno! Thanks for your prompt response.
I have added two unit tests for when getBlockWhenExhausted() is true and when it's false.
Cheers,
Cao Peng

Hi~ @psteitz @kinow Can someone help to merge this PR if there are no other problems. Thanks!

I'll take a look over this long weekend (which is now in the US for me). TY for your patience .

Hi All.
I do not think this PR is ready yet because the tests pass when the main changes are not applied.
The new tests must fail in this case, otherwise they do not prove that the changes to main fix or change anything.
IOW, in TDD terms: Write and run the tests FIRST, see them fail, change the code in main, run the test again, if they now pass, all is well.

Ah, good catch Gary! :+1:

Looks to me like the tests were added just to satisfy coverage nits.  This is a performance enhancement, not a bug.  Junit is not a microbenchmark framework, so it makes no sense to require unit tests that fail then succeed to show performance improvement.

Ah, I lost track that this was about performance, that's where JMH helps.

The flame graphs posted above support the claim of improved performance.

+1 mea culpa @psteitz . I simply saw no tests and asked if unit tests could be added.


Configuring Active Directory ldap-resource creates unneeded lines in nslcd.conf starting with pam_authz_search

Environment

Declarative Onboarding Version: 1.22.0-3
BIG-IP Version: 15.1.3

Summary
Configuring AD authentication with DO creates lines starting with pam_authz_search in nslcd.conf that breaks group-membership lookup. Uncommenting these lines manually fixes remote authentication.
Steps To Reproduce
Steps to reproduce the behavior:

Submit the following declaration:

    "schemaVersion": "1.0.0",
    "class": "Device",
    "async": true,
    "label": "my BIG-IP declaration for declarative onboarding",
    "Common": {
        "class": "Tenant",
        "myAuth": {
            "class": "Authentication",
            "enabledSourceType": "activeDirectory",
            "fallback": false,
            "remoteUsersDefaults": {
                "partitionAccess": "Common",
                "terminalAccess": "disabled",
                "role": "no-access"
            },
            "ldap": {
                "bindDn": "CN=BigIP-SA,OU=ServiceAccounts,OU=BIG-IP,OU=Services,DC=foo,DC=bar,DC=com",
                "bindPassword": "hunter2",
                "bindTimeout": 30,
                "checkBindPassword": true,
                "checkRemoteRole": true,
                "filter": "(sAMAccountName=*)",
                "groupDn": "groupName",
                "groupMemberAttribute": "member",
                "idleTimeout": 30,
                "ignoreAuthInfoUnavailable": true,
                "ignoreUnknownUser": true,
                "loginAttribute": "sAMAccountName",
                "port": 636,
                "referrals": false,
                "searchScope": "sub",
                "searchBaseDn": "DC=foo,DC=bar,DC=com",
                "searchTimeout": 30,
                "servers": [
                    "ad01.foo.bar.com",
					"ad02.foo.bar.com"
                ],
                "ssl": "enabled",
                "version": 3
            }
        },
        "Administrators": {
            "class": "RemoteAuthRole",
            "attribute": "memberOF=CN=F5-Admins,OU=Access,OU=Groups,DC=foo,DC=bar,DC=com",
            "console": "tmsh",
            "lineOrder": 1001,
            "role": "admin",
            "remoteAccess": true,
            "userPartition": "all"
        }

    }
}

Observe the following error response:
After enabling LDAP debug (tmsh modify auth ldap all debug enabled), relevant lines from /var/log/secure

Jul  9 15:02:58 mybigip.foo.bar.com info nslcd[23473]: nslcd: [3c9869] <authz="testuser"> DEBUG: ldap_simple_bind_s("CN=BigIP-SA,OU=ServiceAccounts,OU=BIG-IP,OU=Services,DC=foo,DC=bar,DC=com","***") (uri="ldaps://ad01.foo.bar.com:636")
Jul  9 15:02:58 mybigip.foo.bar.com info nslcd[23473]: nslcd: [3c9869] <authz="testuser"> DEBUG: set_socket_timeout(30,500000)
Jul  9 15:02:58 mybigip.foo.bar.com info nslcd[23473]: nslcd: [3c9869] <authz="testuser"> DEBUG: ldap_result(): CN=testuser,OU=Personal,OU=Internal,OU=Users,DC=foo,DC=bar,DC=com
Jul  9 15:02:58 mybigip.foo.bar.com info nslcd[23473]: nslcd: [3c9869] <authz="testuser"> DEBUG: trying pam_authz_search "(&(&(sAMAccountName=testuser)((sAMAccountName=*))(|(host=mybigip.foo.bar.com)(host=localhost.localdomain)(host=\*)))"
Jul  9 15:02:58 mybigip.foo.bar.com info nslcd[23473]: nslcd: [3c9869] <authz="testuser"> DEBUG: myldap_search(base="DC=foo,DC=bar,DC=com", filter="(&(&(sAMAccountName=testuser)((sAMAccountName=*))(|(host=mybigip.foo.bar.com)(host=localhost.localdomain)(host=\*)))")
Jul  9 15:02:58 mybigip.foo.bar.com info nslcd[23473]: nslcd: [3c9869] <authz="testuser"> ldap_search_ext() failed: Bad search filter

Expected Behavior
I should be able to login with a user in the specified RemoteAuthRole
Actual Behavior
I am not able to login, before I comment out these lines from /etc/nslcd.conf and restart nslcd service
#pam_authz_search (&(&(sAMAccountName=$username)((sAMAccountName=))(|(host=$hostname)(host=$fqdn)(host=\)))
#pam_authz_search (&(&(sAMAccountName=$username)((sAMAccountName=*))(member=groupName))
After that is done I am able to login as expected.

Declarative Onboarding does not directly modify the nslcd.conf file. It uses tmsh to modify auth ldap properties. Do you know what tmsh properties are tied to those lines in the configuration file?

This issue is being automatically closed because there has been no response to our request for more information from the original author. With only the information that is currently in the issue, we don't have enough information to take action. Please reach out if you have or find the answers we need so that we can investigate further.

Sorry for the late reply, this issue is now resolved.
By removing the following lines in the Terraform-configuration the lines starting with pam_authz_search were removed from nsld.conf;
"checkBindPassword": true,
"filter": "(sAMAccountName=*)",
"groupDn": "groupName",
"groupMemberAttribute": "member",


Requests to the Azure metadata service take minutes to time out

Bug Report

import path of package in question: "github.com/Azure/azure-sdk-for-go/sdk/azidentity"
SDK version: v1.2.0

Specify the exact commit if possible; one way to get this is the REVISION
column output by dep status "github.com/Azure/azure-sdk-for-go.

output of go version: go version go1.19.1 linux/amd64

What happened?

When using a custom credential chain with a managed identity credential, requests to the Azure metadata take minutes to time out instead of 1 second. This seems to be caused by the fact that...
https://github.com/Azure/azure-sdk-for-go/blob/f8cb16d25c435299b972737b2021aa1854d95e07/sdk/azidentity/default_azure_credential.go#L100
...which is set when using the default chain isn't set when using a custom chain.

What did you expect or want to happen?

I expected the metadata request to time out after 1 second when not running on Azure so that the next credential in the chain is attempted.

How can we reproduce it?

package main

import (
	"context"
	"fmt"

	"github.com/Azure/azure-sdk-for-go/sdk/azcore"
	"github.com/Azure/azure-sdk-for-go/sdk/azidentity"
	"github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/resources/armresources"
)

func main() {
	clientID := azidentity.ClientID("abcd1234-...")
	subscriptionID := "foo"
	opts := azidentity.ManagedIdentityCredentialOptions{ID: clientID}

	managed, err := azidentity.NewManagedIdentityCredential(&opts)
	if err != nil {
		panic(err)
	}

	cred, err := azidentity.NewChainedTokenCredential([]azcore.TokenCredential{managed}, nil)
	if err != nil {
		panic(err)
	}

	client, err := armresources.NewResourceGroupsClient(subscriptionID, cred, nil)
	if err != nil {
		panic(err)
	}

	res, err := client.Get(context.TODO(), "foo", nil)
	if err != nil {
		panic(err)
	}

	fmt.Println(*res.ID)
}

The code above blocks for more than two minutes at client.Get() on my machine.

Anything we should know about your environment.

I think this problem is hard to reproduce if on your dev machine you get a fast network failure for <IP_ADDRESS>. On my machine, requests to this address are dropped silently, in which case it would make sense that the SDK would give up after a short amount of time and try another credential rather than hang for a long time.

DefaultAzureCredential sets that timeout because it's designed to help developers get started and is commonly used in environments where IMDS isn't available. Other credentials don't impose similar timeouts because there isn't an obvious good default value, and we don't want to take control over timeouts from applications. You can set your own with timeout with context:
// the timeout applies to the client method call, including authentication
ctx, cancel := context.WithTimeout(context.Background(), 5*time.Second)
defer cancel()
res, err := client.Get(ctx, "foo", nil)

@chlowell thanks for responding quickly.
I think we're talking about two different things. Your code snippet sets a timeout on the Get() request. I'm talking about setting a timeout on the request the SDK does to IMDS before calling Get().
Say I define a custom credential chain with IMDS as the 1st credential and Azure CLI as the 2nd one. In that case, when running this code on an environment without IMDS I want the request to IMDS to time out quickly so that Azure CLI auth is attempted. I don't necessarily want to time out the Get() request quickly, or maybe even at all - the two timeouts are different AFAICT.

Thanks, I did overlook that distinction! You could use a tiny wrapper to apply a timeout only to authentication:
import (
	"github.com/Azure/azure-sdk-for-go/sdk/azcore"
	"github.com/Azure/azure-sdk-for-go/sdk/azcore/policy"
	"github.com/Azure/azure-sdk-for-go/sdk/azidentity"
)

type wrapper struct {
	cred *azidentity.ManagedIdentityCredential
}

func (w *wrapper) GetToken(ctx context.Context, opts policy.TokenRequestOptions) (azcore.AccessToken, error) {
	c, cancel := context.WithTimeout(ctx, 5*time.Second)
	defer cancel()
	return w.cred.GetToken(c, opts)
}

However, this doesn't work with ChainedTokenCredential because you need an unexported type to signal the timeout error is non-fatal. This is a good argument for exporting that type; I'll open a PR.
In the meantime, if the Azure CLI isn't available in your non-dev environment, you could put AzureCLICredential ahead of managed identity in the chain. It fails fast when the CLI isn't installed.

Thanks @chlowell. Yes, I already have a workaround in place. Wanted to report this to improve things for other users.
Thanks for looking into this!

@chlowell are you talking about exporting the credentialUnavailableError type so that we can use a wrapper and check the error type in user code? AFAICT this won't be enough because of the following check:
https://github.com/Azure/azure-sdk-for-go/blob/7511cc07bf639e2f1e6e2b36eca5a056da4e8d33/sdk/azidentity/managed_identity_client.go#L178-L181
We return a credentialUnavailableError only when cancel isn't nil, and cancel gets assigned only when c.imdsTimeout > 0:
https://github.com/Azure/azure-sdk-for-go/blob/7511cc07bf639e2f1e6e2b36eca5a056da4e8d33/sdk/azidentity/managed_identity_client.go#L165-L169
Am I missing something?
BTW, I'm happy to open a PR once we figure out the right solution.

Yes, that's what I have in mind. With CredentialUnavailableError exported, the wrapper can signal the chain to try another credential after a managed identity timeout. Then the wrapper needs only to know whether a timeout has occurred. It doesn't have to rely on the wrapped credential for that because it has the Context which would have timed out. I'm imagining something like this:
func (w *wrapper) GetToken(ctx context.Context, opts policy.TokenRequestOptions) (azcore.AccessToken, error) {
	c, cancel := context.WithTimeout(ctx, time.Second)
	defer cancel()
	tk, err := w.cred.GetToken(c, opts)
	// special case: timeout; signal the chain to keep iterating
	if ctxErr := c.Err(); errors.Is(ctxErr, context.DeadlineExceeded) {
		err = &azidentity.CredentialUnavailableError{Message: "managed identity timed out"}
	}
	// otherwise, propagate the wrapped credential's return
	return tk, err
}

We chose to export an error constructor rather than an error type for this in azidentity v1.3.0-beta.2. Updating my examples above, a wrapper would now look like this:

import (
	"context"
	"errors"
	"time"
	"github.com/Azure/azure-sdk-for-go/sdk/azcore"
	"github.com/Azure/azure-sdk-for-go/sdk/azcore/policy"
	"github.com/Azure/azure-sdk-for-go/sdk/azidentity"
)

type wrapper struct {
	cred *azidentity.ManagedIdentityCredential
}

func (w *wrapper) GetToken(ctx context.Context, opts policy.TokenRequestOptions) (azcore.AccessToken, error) {
	c, cancel := context.WithTimeout(ctx, time.Second)
	defer cancel()
	tk, err := w.cred.GetToken(c, opts)
	if ctxErr := c.Err(); errors.Is(ctxErr, context.DeadlineExceeded) {
		// timeout: signal the chain to try its next credential, if any
		err = azidentity.NewCredentialUnavailableError("managed identity timed out")
	}
	return tk, err
}

Yes! #19859 implemented a more robust wrapper for DefaultAzureCredential to use internally and added that wrapper as an example for ChainedTokenCredential.


Add files via upload

@vsoch Yes you are right. There should be a separate folder - I've made the change, hope it'll work now.

I love the d3! Did you mean for the dependencies (boxplots.css and boxplots.js) to be in a test_files folder? Running a webserver that uses the $PWD as root would return 404 for those files and you wouldn't see the visualization. Are all the files supposed to be in some test or example folder?

Hi @mwei97 could you write something about what this PR is attempting to implement? I am currently unclear what feature you are trying to add to the project or what bug you are trying to fix.


Only typescript ~4.2 or ~4.3 version

Please read and mark the following check list before creating a pull request (check one with "x"):

[x] I read and followed the CONTRIBUTING.md guide.
[x] I read and followed the New Feature Checklist guide.

Short description of what this resolves:
This MR fixes #5893. Angular currently only supports typescript 4.2. or 4.3 but not 4.4. Semantic versioning lead to the case, that npm installed v4.4.

is this project actively maintained? @yggg?


[BUG]: Returns empty p tags string when there is no text just line breaks! required validation not working

Description
_A bug: When i press enter and add line breaks it returns empty p tags string like

. on it angular required validation does not work. please fix it, if there is no text then angular required validation should work. thanks
Version Information
ngx-editor: 7.0.4
angular: 10.0.4

Browser

[x] Chrome -  87.0.4280.66

Use custom validators. Example
import {Validators} from 'ngx-editor'

new FormControl('', Validators.required())

Some other validators https://sibiraj-s.github.io/ngx-editor/#/reactive-forms?id=validators

Closing this issue. Feel free to continue the discussion If you have any doubts.

I have used this validator but still facing issue, can you please tell me the version of ngx-editor please?

Oh. I get it. I misunderstood earlier. The validator just check if the editor is empty or not, this is same as Angular's required validator.
It doesn't trim the value before validation.
You might wan't to implement your own custom validator for custom .
See here for examples https://github.com/sibiraj-s/ngx-editor/blob/master/src/lib/validators.ts.
Let me know if you need any help with that.

Okay, I will submit a PR later after handling this issue. Others will also benifit from it.

Thanks. I am just not sure whether it has to added to required validator, since I added the behaviour to match angular's required validator.
I don't use angular that heavily. how is this handled in other places?

I have put some thought over it. I think this is not just about text content. the editor will have custom views, images etc. Considering those into account, I suppose deciding when the editor is TRULY empty is purely subjective .
Exposing bare default would be the best way to go and for custom needs they can implement their own solution which is also supported by angular.

Okay, I will submit a PR later after handling this issue. Others will also benefit from it.

and for this. I am open to adding a section in the docs adding custom code examples.


Is there major compatitbility issues?  Repeated Errors

I have spent over 10 hours trying to set up a bot simply to like tweets based on keywords.  I keep getting invalid syntax "" line 1.  I was only able to setup something that retweeted tweets from certain users.  What am I doing wrong?  About to throw my PC out the window and not retrieve it

Do you have a sample code

What's the full traceback?

@Negaunee0024  how did u get around the 401 errors with authenticating to use POST endpoints such as retweeting? I keep getting 401 errors on my oauth1a authentication
thanks

That's not related to this issue.
See the FAQ section about this in the documentation.

heres my codebase homies, and my traceback is:
`
Traceback (most recent call last):
File "/Users/flakibaby/PROJECTS/peyote-sentiment-twitterbot/like.py", line 23, in
user = getUserInfo()
File "/Users/flakibaby/PROJECTS/peyote-sentiment-twitterbot/like.py", line 20, in getUserInfo
user = y.get_user(username='im_rickjamez')
File "/usr/local/lib/python3.9/site-packages/tweepy/client.py", line 1660, in get_user
return self._make_request(
File "/usr/local/lib/python3.9/site-packages/tweepy/client.py", line 166, in _make_request
response = self.request(method, route, params=request_params,
File "/usr/local/lib/python3.9/site-packages/tweepy/client.py", line 122, in request
raise Unauthorized(response)
tweepy.errors.Unauthorized: 401 Unauthorized
'
`
import tweepy
import config
import time
creating client returning client
def getClient():
x = tweepy.Client(
# bearer_token=config.bearer_token,
                   consumer_key=config.consumer_key,
                   consumer_secret=config.consumer_secret,
                   access_token=config.access_token,
                   access_token_secret=config.access_token_secret
                   )
return x

testing if i can get full user info
def getUserInfo():
y = getClient()
user = y.get_user(username='jack')
return user.data
user = getUserInfo()
print(user)
`

Again, that's not related to this issue.


Anchor links are not scrolled to automatically

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[ ] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
Anchor links do not scroll to their position on the page (in fact, no scrolling happens).
Steps to reproduce
I tried reproducing on https://codesandbox.io/s/docusaurus, but they ignore links entirely.
Locally:

Go to some page and copy some heading anchor link (e.g. http://localhost:3000/dbux/advanced/terminology#cgr).
Go to some other page.
Paste and go to the copied link.

I end up on the right page, but it does not scroll down to the correct heading.
I tried the same on the official page (e.g. https://docusaurus.io/docs/configuration#site-metadata), and there it works correctly.
Expected behavior
When going to an anchor (hash) link of some sub heading on a page, it should scroll to the sub heading right away.
Actual behavior
The link take you the correct page, but there is no scrolling.
Your environment

Public source code: https://github.com/Domiii/dbux/tree/master/docs
Public site URL: (not public yet)
Docusaurus version used: 2.0.0-beta.14 and 2.0.0-beta.13
Environment:
<EMAIL_ADDRESS>Google Chrome | 96.0.4664.110 (Official Build) (64-bit) (cohort: Stable)
Revision | d5ef0e8214bc14c9b5bbf69a1515e431394c62a6-refs/branch-heads/4664
OS | Windows 10 Version 20H2 (Build 19042.1415)

Reproducible demo
(cannot reproduce since direct links don't work at all on codepen)
Self-service

[X] I'd be willing to fix this bug myself.

Duplicate of https://github.com/facebook/docusaurus/issues/5272. We also have a notice here: https://docusaurus.io/docs/cli#docusaurus-start-sitedir

Please note that some functionality (for example, anchor links) will not work in development. The functionality will work as expected in production.


Release-1.8: Cherry-pick of #2243

What this PR does / why we need it:
This needs to be released in v1.8.1, containing https://github.com/kubeflow/training-operator/pull/2243.
Which issue(s) this PR fixes (optional, in Fixes #<issue number>, #<issue number>, ... format, will close the issue(s) when PR gets merged):
Part-of: https://github.com/kubeflow/training-operator/issues/2241
Checklist:

[ ] Docs included if any changes are user facing

/assign @kubeflow/wg-training-leads


make the replication applier auto-start for the RocksDB engine if con…

…figured so

@jsteemann
2017-11-11T21:10:31Z [31361] INFO [reportRole@ClusterFeature.cpp:221] {cluster} Starting up with role AGENT
2017-11-11T21:10:31Z [31361] ERROR [runServer@arangod.cpp:218] arangod terminated because of an exception: cannot disable non-optional feature 'RocksDBRecoveryFinalizer'

Setting arangod/RocksDBEngine/RocksDBRecoveryFinalizer.cpp to optional


chore: add script to help run ADO extension locally

Details
It can be difficult to debug the ADO extension locally. We can run dist/pkg/index.js, but we haven't been mapping input variables into the environment.
This PR adds a script so that we can more easily run the extension locally. It's not a full substitute for running in an agent environment - but it can be helpful in some scenarios.
Running yarn start will run the bundled index.js script with the following task inputs:

any default values from task.json inputs
any additional values in local-overrides.json (these take precedence)

Here is the output I see when I run yarn start:
PS C:\Users\karansin\accessibility-insights-action\packages\ado-extension> yarn start
yarn run v1.22.4
$ node scripts/run-locally.js
run-locally.js is setting up input outputDir to value _accessibility-reports
run-locally.js is setting up input siteDir to value .
run-locally.js is setting up input scanUrlRelativePath to value /
run-locally.js is setting up input maxUrls to value 100
run-locally.js is setting up input scanTimeout to value 90000
run-locally.js is setting up input singleWorker to value true
run-locally.js is setting up input url to value https://markreay.github.io/AU/before.html
beginning task execution below

installing runtime dependencies...
[1/4] Resolving packages...
warning Lockfile has incorrect entry for<EMAIL_ADDRESS>Ignoring it.
warning Lockfile has incorrect entry for<EMAIL_ADDRESS>Ignoring it.
warning Lockfile has incorrect entry for<EMAIL_ADDRESS>Ignoring it.
warning Lockfile has incorrect entry for<EMAIL_ADDRESS>Ignoring it.
...<further task output>

Before this PR, running yarn start usually crashes because the url parameter is not mapped into the environment. You can get around this by mapping environment variables yourself, e.g. npx cross-env ACT=true INPUT_URL=https://markreay.github.io/AU/before.html INPUT_SCANTIMEOUT=90000 INPUT_maxurls=5 INPUT_outputDir=_accessibility-reports yarn start but this is quite a hassle.
Motivation
make it easier to see the impact of small changes locally
Context
n/a
Pull request checklist

[ ] Addresses an existing issue: Fixes #0000
[ ] Added relevant unit test for your changes. (yarn test)
[ ] Verified code coverage for the changes made. Check coverage report at: <rootDir>/test-results/unit/coverage
[ ] Ran precheckin (yarn precheckin)
[ ] (after PR created) The Accessibility Checks (pull_request) check should fail. All other checks should pass.

Codecov Report

Merging #1027 (98ceabc) into main (71dd10b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1027   +/-   ##
=======================================
  Coverage   89.69%   89.69%
=======================================
  Files          44       44
  Lines        1058     1058
  Branches      139      139
=======================================
  Hits          949      949
  Misses        109      109

Continue to review full report at Codecov.

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


Trused advisor checks fail if the default region in the profile is not the one expected

depending on the partition ( US gov cloud, standard or china) a different region must be used for trusted advisor checks.
that said the checks for trusted advisor use whatever is set as the default in the profile. this can cause the following errors
For standard AWS the endpoint should be /support.us-east-1.amazonaws.com
For US Gov Cloud the endpoint it should be support.us-gov-west-1.amazonaws.com
the partition can be determind vi a the get_partition_for_region call in boto3 https://boto3.amazonaws.com/v1/documentation/api/latest/reference/core/session.html using the default region then making the calls to the correct endpoint for that partition.
7.26 [extra726] Check Trusted Advisor for errors and warnings - trustedadvisor [Medium]
Could not connect to the endpoint URL: "https://support.us-east-2.amazonaws.com/"
Could not connect to the endpoint URL: "https://support.us-east-2.amazonaws.com/"
Could not connect to the endpoint URL: "https://support.us-east-2.amazonaws.com/"
Could not connect to the endpoint URL: "https://support.us-east-2.amazonaws.com/"
Could not connect to the endpoint URL: "https://support.us-east-2.amazonaws.com/"
Could not connect to the endpoint URL: "https://support.us-east-2.amazonaws.com/"
Could not connect to the endpoint URL: "https://support.us-east-2.amazonaws.com/"
Could not connect to the endpoint URL: "https://support.us-east-2.amazonaws.com/"
Could not connect to the endpoint URL: "https://support.us-east-2.amazonaws.com/"
Could not connect to the endpoint URL: "https://support.us-east-2.amazonaws.com/"
Could not connect to the endpoint URL: "https://support.us-east-2.amazonaws.com/"
Could not connect to the endpoint URL: "https://support.us-east-2.amazonaws.com/"
Could not connect to the endpoint URL: "https://support.us-east-2.amazonaws.com/"
Could not connect to the endpoint URL: "https://support.us-east-2.amazonaws.com/"
Could not connect to the endpoint URL: "https://support.us-east-2.amazonaws.com/"
Could not connect to the endpoint URL: "https://support.us-east-2.amazonaws.com/"

Thanks for the heads up @prmarino1m, could you please share what command are you using for each case? For those checks pretty tight to a region, in most cases, -r region must be used to force where all calls are going. For example, I'm testing it with commercial with no region set up in ~/.aws/config and I get the same error as you show here. So I have to specify -r us-east-1 to make it work properly. For GovCloud you have to use -r us-gov-west-1. Let me know if it works.

Hi @prmarino1m, could you please share what commands are you using for each failing case?
We are investigating it in order to fix it or define the cocrrect way to execute this check.
Thank you!

We are closing this issue. Feel free to reopen it if you can provide us more information.
Thank you.

@jfagoagas  Sorry for the delayed response here are the options i used in prowler "-M html,csv,text -p  -r us-gov-west-1" that produced this output. Also the default region set in the profile was us-gov-west-1
Also as i pointed out this can be avoided if prowler

Thank you @prmarino1m , we're investigating it.

The way this check extra726 is written it takes region from the command line and if in your case it is getting us-east-2 it is because you probably are running an old version of prowler since that is not even the default commercial region. Please test the latest release or the master repo code with -M html,csv,text -p -r us-gov-west-1 -f us-gov-west-1 or even for both gov regions -M html,csv,text -p -r us-gov-west-1 -f 'us-gov-west-1 us-gov-east-1'. We don't have a way to test GovCloud at this moment so please let us know if that works.

We are closing this issue since we didn't have any feedback. Please, feel free to reopen it if you have any comment.


pythonPackages.thumbor: 6.4.2 -> 6.5.1

Motivation for this change
The current thumbor build on master: https://hydra.nixos.org/build/76000076
This is mainly caused by several dependency bumps (e.g.
pythonPackages.tornado to v5 in dd936ccdb9bb04436f4e045998f305de12a67eb2)
that broke the setup.py from version 6.4.
Bumping to the latest stable version (6.5.1) fixes these issues.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option 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"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

nox-review is still in progress and it seems to take a while locally, I'll update the PR as soon as its ready %)


Issue: Images are not showing up in Mailchimp

Description
Issue: Images are not showing up in Mailchimp. Need to contact support.

Meh. Issue was due to Brave Browser blocking imagery.


Client references json.JSONDecodeError, only available in Python 3.5+

I'm assuming by use of future you mean to support lower Python versions?

https://github.com/uservoice/uservoice-python/pull/18


Proposal file for new dataset (localizer)

I am training localizer on a new dataset COIN. In the config file it expects a proposal file (similar to thumos14_tag_val_normalized_proposal_list.txt for THUMOS14).
In case of THUMOS14, it is just fetched from some other place (https://github.com/open-mmlab/mmaction/blob/master/data_tools/thumos14/fetch_tag_proposals.sh).
How to generate this file in the first place, for the new dataset in my case?

same question

https://github.com/yjxiong/action-detection

@iuhiyuh
Hi！I wonder if  you have worked out this problem by SSN?
When I run "python gen_proposal_list.py DATASET FRAMES_PATH"
an error occurs :
"KeyError: 'video_validation_0000893'".


About Page

Related Issue

issue #45
Description
Created an about page

Screenshots
https://github.com/dakshsinghrathore/Brighter-Beginnings/assets/133582773/1beeaba8-a325-4d7a-937a-f9d8c766b399

@dakshsinghrathore  Do I need to make more changes?

@bhavya-anand19 remove the home button and  add sufficient padding to cards.

@bhavya-anand19 remove the home button and add sufficient padding to cards.

Okk


background colors fonts w/ Acrylic - do not display correctly

Version: 1.0.92
Platform: win32 10.0.18362
Plugins: clickable-links, save-output
When enabling Acrylic background - fonts with a non-default background color (highlight) do not display properly. the font and background color are the same - so you see blocks, and not letters.
with Acrylic enabled:

without Acrylic enabled (how it should be):

so this is why... and i though the problem was the theme colors...

You can kind of get around it with Appearance -> From Color Scheme instead of Theme, but it isn't quite the way things ought to look, I guess.

Duplicate of #646


[2.x] Bump CI to JDK 21

Closes #7731

Shall reduce the scope of it. Let's leave launcher-package for later.

The only outstanding scripted test is stdlib-unfreeze, otherwise everything works for JDK 21.
Are there any standard library methods that are not in Scala Library 2.13.11 but are added later? As a part of the test checks whether sbt properly handles methods newly added to standard library.

Are there any standard library methods that are not in Scala Library 2.13.11 but are added later?

Yes. Nothing public in the source code, but we've added things that are private[scala] or some subpackage. (And those are then public in bytecode, as expected.)

Yes. Nothing public in the source code, but we've added things that are private[scala] or some subpackage. (And those are then public in bytecode, as expected.)

Thanks for the heads up. I guess we can do something similar to this to access these private methods.
And indeed after some searching, I found one such private method: https://github.com/scala/scala/commit/8077afc3c4ccdcf0f785f9d732977e1eb6437f3f


Allow uploading large files via the web UI

Is your feature request related to a problem? Please describe.
I tried to upload a 13GB dataset file (for this repo), and after waiting a couple of hours for it to upload, it gave a "Payload too large" error.
Describe the solution you'd like
I'd like to be able to upload large dataset files using the web UI.
Describe alternatives you've considered
I'm guessing there's some sort of command line tool for cases like this, but I'd prefer to not have to install something just to upload a file - it's bad UX. There's no fundamental technical reason why the web UI can't handle large files, so it doesn't seem like a good idea to put limits like this in place.
Additional info
If the team is for some reason adamant about not allowing upload of large files via the web UI, then at the very least it would be good to stop the user with an error when they pick the file, rather than after the upload is complete. I.e. check the size of the blob with JS before uploading rather than uploading and checking size on the server.
If it's actually already possible to upload large files via the web UI (and I've just misunderstood the process), then please consider this issue a request for better UX in guiding the user toward doing that. After creating the dataset repo I just clicked across to the "files and versions" tab, and then clicked "Add file > Upload file".
Thanks!

cc @julien-c

For @SBrandeis (and cc @Pierrci)
I think we said we would remove the 5GB limit on file upload no? (But WDYT of keeping a 20GB limit as larger than that files aren't CDN-distributed and it's just not something that we want to support IMO)

Thanks for the report @josephrocca the plan is to support file uploads up to 20GB and check the blob size client-side for files larger than 20GB.
WDYT?

Any progress on this? Just tried again, but this time I split the file into a couple of ~6GB files to be safe. This time I got a 504 error for one of them, and a 500 error for the other. I have a symmetric gigabit connection, but the uploads took 1 or 2 hours, so it seems a bit slower than it perhaps should be, and maybe that caused the 504 timeout, but I'm not sure about the 500 error. The speed issue could just be due to the fact that I'm based in Singapore.

This has been implemented/shipped 20 days ago but we had server connectivity issues earlier today so maybe it was a temporary hiccup?

@julien-c I just tried again and got a 500 error when it was >80% complete (after about 40-50 minutes). I'm just uploading a 6GB tar file, and it uploads at about 2 megabytes/sec. A 600MB file uploads fine. If you can somehow simulate the 2 MB/s upload speed (devtools might alow you to do this?) then my guess is that you'd be able to replicate it. Seems like it has to do with some sort of timeout on the server-side.

Hello @josephrocca, we fixed the underlying issue preventing uploads of large files to the Hub
You should now be able to upload files up to 20GB as stated by @julien-c earlier

@SBrandeis Hi Simon, I just tested 4 times, and it always fails after 40-50 minutes with net::ERR_CONNECTION_CLOSED. Here's a video of the last few seconds as it fails:
https://user-images.githubusercontent.com/1167575/152629860-58cdbd5c-3d42-49a8-8695-9ee8258a511d.mp4
Previously it would show a red error box in the upload UI, but now there's no indication of the error on the web page itself. Here's what Chrome reports in the console:

As mentioned earlier, it's just a 6GB tar file. My average upload speed is about 1.5MB/s. You should be able to emulate this upload speed with Chrome's DevTools IIRC.

@SBrandeis Any progress on this? I just tried to upload a 480mb file and that gave me a error (code 500). I tried a second time and it worked. I tried the above-mentioned 6GB tar again a couple of times but it still doesn't work. So seems flaky, and maybe worse as files get larger? Can you reproduce the errors that I'm talking about, or is it working okay on your end?
To quickly create a ~6GB tar to reproduce:
dd if=/dev/zero of=upload_test bs=1M count=6000
tar -cf upload_test.tar upload_test

And network connection throttling (to match my ~1.5 megabytes/s upload) is in Chrome DevTools:

Hello @josephrocca
There was indeed an issue preventing the upload of files larger than ~5GB, I'm sorry for the inconvenience
We've released a fix yesterday, and I've been able to successfully upload a 6GB file via the UI today, with network throttling to simulate a low bandwidth: https://huggingface.co/datasets/sbrandeis/test-ui-upload/tree/main
Error reporting is still meh (you have to send the whole file over the network for the browser to display the error), we're working on improving this
Please let me know if you still experience issues with web file uploads

@SBrandeis Thanks! The lack of early errors probably isn't such a big issue now that the upload limit is much higher, although I guess it would be pretty annoying to wait for several hours and get an error, for those that don't know about the limits. And I guess it is basically a one-liner to fix this - i.e. the non-minified equivalent of if(s.size > max) showTheError() here:

But feel free to close this if the current state is considered good enough for now (I see that a revamp is going to happen at some point which might make any extra work here not worth it).

(for context @josephrocca on that linked comment, we're going to probably ~reimplement the LFS part of git in the browser 🤯 )

Still getting code-500 errors here and there :/ This was for a 3.5GB tar file:

Tried again and it worked the second time though. No idea what caused it, but one difference was that during the first upload I was creating other small files (e.g. readme, etc.) while it was running. I was adding the other files via the "Create a new file" (and pasting in text), not the "Upload file" button. During the second upload attempt I did not make any changes/additions to the repo. But if that were the problem, then I'd expect to have seen this error message instead.

Yes we do not support concurrent commits while uploading/committing a file through the UI

should the backend lock the repo while there's an upload in progress?

I believe @SBrandeis has a PR open to relax this requirement


services/ftp: Use multiple connections instead

I have to admit that we (reviewers) made a mistake.
Reusing the same connection makes everything complex:

We can't use async API because we can't init the stream during build

Thus, we can't use put_with_stream because the requirement for the reader is different.

We can't return the reader/dirstream to users because they referred to FtpStream.

Thus, we have to read them in memory slower than multiple connections.

I plan to merge this PR and leave this improvement as new issue.

Use multiple connections instead of reusing the same one.
In the future, we can implement a connection pool for FTP connections.

Originally posted by @Xuanwo in https://github.com/datafuselabs/opendal/issues/581#issuecomment-1236049315

Cc @ArberSephirotheca, who may be interested in this issue.

I am happy to work on this issue, please assign it to me :).


New feature - extend search to other tables in other model driven apps

[Enter feedback here]
My customer is disappoi8nted with dataverse search as they cannot extend the search to tables in another model driven app. IS this a feature that can be added?

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 23b0ff5c-e77e-6cf1-5a0e-86a62908d73d
Version Independent ID: e58c39e4-dc33-3639-14e7-a970a214b0ed
Content: Overview of Extended Security Updates for Windows Server 2008, 2008 R2, 2012, and 2012 R2
Content Source: WindowsServerDocs/get-started/extended-security-updates-overview.md
Product: windows-server
Technology: getting-started
GitHub Login: @dknappettmsft
Microsoft Alias: daknappe

Hi, GitHub issues in this repo are not for product feature requests and are for Windows Server, not Power Platform (which Dataverse search is). As this is not related to the article, I'll close it.
#please-close


Added a method to initialize a FlagSet

This can be used to initialize a FlagSet given a pflag.FlagSet.
Useful for converting pflg.FlagSets to FlagSets.
Signed-off-by: alejandrox1<EMAIL_ADDRESS>

Came up with this as a possible solution for converting a pflag.FlagSet to a legacy.FlagSet, such as in https://github.com/kubernetes/kubernetes/blob/master/cmd/kube-proxy/app/server.go#L454

Thank you @rosti @sttts for your comments.
Just made the name change

/lgtm
/approve

Thinking about this, I'm not sure it's necessary.
If we use legacyflag, it can be used for all the flags on a component.
It's more consistent and easier to reason about to just interface with one flag library for all the flags, so why not just switch everything over to legacyflag all at once?
In other words, I see how this new method could be useful for an incremental migration to legacyflag, but why can't the migration be atomic (and do we really want to be in the business of another incremental migration)?


Unable to request for api access key

Describe the bug
Captcha in https://mjml.io/api constantly giving error - Invalid Captcha answer, please try again.
To Reproduce
Steps to reproduce the behavior:
Try to submit an api access request.
Additional context
I am trying to get an api access key. But the captcha fails every time, not being able to submit the request. I have sent the link to a co-worker, he is failing too.

We're working on a fix on our side for that. We'll update here asap

I've subscribed to the issue but I end up checking the registration often, the anticipation is building. 😄

Hi there I am also experiencing this issue, is there a workaround like I could email you directly with my details to request access to the API? Many thanks

Hello guys, sorry but we can't take individual requests for API access. The issue will be resolved soon.

No worries, perfectly understandable, thanks for your reply on this anyway:)

Hi @jack-ellis-verse @danielloader @shimulch, everything is fixed concerning the api access form 👍

Brilliant, thank you for letting us know!:-)


strconv.Atoi: parsing "conversation": invalid syntax upon item creation

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

Basically, when I try to create an object in my database, it gives me the following error strconv.Atoi: parsing "conversation": invalid syntax
Expected Behavior 🤔

I would expect the object to be created.
Steps to Reproduce 🕹

conversation schema:
package schema

import (
    "entgo.io/ent"
    "entgo.io/ent/schema/edge"
    "entgo.io/ent/schema/field"
    "time"
)

// Conversation holds the schema definition for the Conversation entity.
type Conversation struct {
    ent.Schema
}

// Fields of the Conversation.
func (Conversation) Fields() []ent.Field {
    return []ent.Field{
        field.Time("created_at").
            Default(time.Now),
        field.Time("updated_at").
            Default(time.Now),
    }
}

// Edges of the Conversation.
func (Conversation) Edges() []ent.Edge {
    return []ent.Edge{
        edge.From("users", User.Type).
            Ref("conversations"),
        edge.To("chats", Chat.Type),
    }
}

chat schema:
package schema

import (
    "entgo.io/ent"
    "entgo.io/ent/schema/edge"
    "entgo.io/ent/schema/field"
    "time"
)

// Chat holds the schema definition for the Chat entity.
type Chat struct {
    ent.Schema
}

// Fields of the Chat.
func (Chat) Fields() []ent.Field {
    return []ent.Field{
        field.Text("message").
            MaxLen(1024),
        field.Time("created_at").
            Default(time.Now),
    }
}

// Edges of the Chat.
func (Chat) Edges() []ent.Edge {
    return []ent.Edge{
        edge.From("conversation", Conversation.Type).
            Ref("chats").
            Unique(),
        edge.From("sender", User.Type).
            Ref("sent_chats").
            Unique(),
    }
}

I fetch a conversation like so:
        client.Conversation.
        Query().
        Where(conversation.ID(id)).
        WithUsers().
        Only(context.Background())

And when I try to create a chat with the fetched conversation like so:
        client.Chat.
        Create().
        SetSender(sender).
        SetConversation(conversation).
        SetMessage(message).
        Save(context.Background())

I get the following error:
strconv.Atoi: parsing "conversation": invalid syntax

Your Environment 🌎

Tech
Version

Go
1.20.7

Ent
0.12.3

Database
PostgreSQL

Driver
github.com/lib/pq

Hey @Breigner01 👋🏻
Can you please provide a reproducible example using https://github.com/ent/bug? Thanks

After 2 days trying to reproduce this bug, I cannot manage to reproduce it.
This is what I did: Atoi Failed Bug Reproduction
If you can tell me what stuff I may have done wrong to cause this issue I would be really appreciative but this is as close as I could get to what I have in my program without including the whole gin part.
So I'm kinda at a loss there getting frustrated alone because I can't reproduce a bug even when copy pasting the code that I use.

By the way,
I don't know if this is relevant but my program is running in docker.
In theory in shouldn't change anything but idk at this point, I'm trying to find the one thing that's causing this issue and this is one of the differences with the attempted bug reproduction.
I'll dockerize it right away to try it out but I'm not hoping it'll fix anything.

Well the small test code had been dockerized but it doesn't reproduce the error, so I guess that was for nothing.

So, I just found where the bug is.
It actually comes from a hook that I didn't setup correctly.


Update issue

I experienced the following behaviour, can't provide the exact console output but as I remember from my mind:
If you do a composer update and the magento-core package is going to be updated, the magento-core-composer-installer

first undeploys/deletes all core-files from the magento-docroot
composer fetches the new package version (which can last for several minutes on slow connections) and afterwards
the composer-installer redeploys the updated magento-core.

Even if the updated package is already in the composer-cache, this still leaves magento in a void-state for ~30 seconds.
This is a nightmare on production-environments because in the meantime between undeploy and redeploy the magento-installation is left in an "empty" state.
How to work around this issue?

Hi, you should use a different deploy strategy - you really should not be doing a composer install on a live environment or at least on the publicly accessible code. Look in to using Capistrano or something similar, or even running the install locally, building a package and deploying that.
Even without this package, you will experience the same problem with just composer, there will be a small gap where it is removing, extracting and writing files after updates.

Good point. Unfortunately not economical for smaller projects to setup capistrano deployments. Seems like to live with this restriction and place a index.html to show a maintenance page.

If you invest a little time with Capistrano it will make your life a lot simpler. We have a sort of skeleton project that we start all projects from with the Capistrano config already in there, also including base Composer config. Then when we initialise a project all we have to do is change the server address.
Capistrano is still manual also, so you don't have to pay for anything. Just run cap production deploy
Anyway i'll close this now as it's not something I want to solve with this tool.


Dutch translation

Added dutch translations for Create metalurgy,
Just DM me if there are mistakes and ill look into it

I have merged your PR just right now, but for the next time. Can you put ur nl_nl.json file here src/main/resources/assets/createmetallurgy/lang and not in src/generated/resources/assets/createmetallurgy/lang
Which is reserved for data gen files only


xSQLServer: Added VS Code workspace settings file

Pull Request (PR) description

Added VS Code workspace settings file with formatting settings matching the Style Guideline (issue #645). That will make it possible inside VS Code to press SHIFT+ALT+F, or press F1 and choose 'Format document' in the list. The PowerShell code will then be formatted according to the Style Guideline (although maybe not complete, but would help a long way).

This Pull Request (PR) fixes the following issues:
Fixes #645
Task list:

[x] Change details added to Unreleased section of CHANGELOG.md?
[x] Added/updated documentation, comment-based help and descriptions in .schema.mof files where appropriate?
[ ] Examples appropriately updated?
[x] New/changed code adheres to Style Guidelines?
[ ] Unit and (optional) Integration tests created/updated where possible?

This change is

Codecov Report

Merging #646 into dev will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##            dev   #646   +/-   ##
===================================
  Coverage    95%    95%
===================================
  Files        29     29
  Lines      2943   2943
===================================
  Hits       2797   2797
  Misses      146    146

Reviewed 2 of 3 files at r1, 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

@PlagueHO Thanks for the review! 😄


Arno Privitelli patch 1 1

Something is off here... In the index file you took "Programming_02.html", but then you worked on "Algorithms_02.html". I suggest that you solve this issue and send another pull request


Providing descriptions of cards in the datasource

Sometimes I want to add a note/comment to a card, but the CSV format doesn't really specify a way of doing this.
What first comes to mind is just adding a blank comment line above each row, as you would in most programming languages (e.g. #, // or whatever). Currently, cards.py will ignore every row that starts with a '#', but it will also think it's a row of data (that should just be skipped). Maybe this can be improved?
Alternatively, we could add a special column (maybe something like @description) that is only for this specific purpose. So any content in that column would just be filtered out of the data when building. This solution doesn't really make it easy to actually read the description, though. I think I would much prefer being able to comment on a blank row.

If using Google Sheets you can add comments to specific cells. That's a good solution, but I would like being able to do it directly in a CSV too.

Additionally, using # to place comments on empty rows would lead e.g. GitHub to render the entire file incorrectly.

This issue cannot be resolved unless it is decided that following the CSV "standard" is not required.
I'm currently of the opinion that it should be, or at least be compliant enough that GitHub renders it properly. So that leaves the @description idea as the only option.


result.links get failed

My code:
const HCCrawler = require('headless-chrome-crawler');
(async () => {
const crawler = await HCCrawler.launch({
evaluatePage: () => ({
title: $('title').text(),
}),
onSuccess: result => {
console.log(Got ${result.links.length} links.);
},
});
await crawler.queue('http://jsgyq.jinshan.gov.cn/html/'); // Queue a request
await crawler.onIdle();
await crawler.close();
})();
the result is :Got 2 links.
but ,actually there are lots of links in 'http://jsgyq.jinshan.gov.cn/html/'
who can tell me the reason , thanks.

i'm here still waiting…thanks

You could change the maxDepth option, I think the default is two, so will only go two layers deep


Software Management: Get settings

Description
First software settings command. This one lists settings.
Steps to Test
npm run build && ./dist/bin/vip-settings-software-get.js @2891 --format json

Thank you both!

Preferably the order in the result table would be: WordPress > PHP > MU. This is the UI order of tabs, and was suggested in the design meeting.

Fixed

Only WP can be "pinned" right now. And even though we get it in the api response, I think we should probably avoid the language. Perhaps flip it around and have a column for "Managed Updates Enabled" that is true only for WP if unpinned.

How about this? I recently read that this sort of columns should not exist and be grouped for better UX.
┌────────────┬───────────┬───────────────────────┐
│ name       │ slug      │ version               │
├────────────┼───────────┼───────────────────────┤
│ WordPress  │ wordpress │ 6.0 (managed updates) │
├────────────┼───────────┼───────────────────────┤
│ PHP        │ php       │ 7.4                   │
├────────────┼───────────┼───────────────────────┤
│ MU Plugins │ muplugins │ staging               │
└────────────┴───────────┴───────────────────────┘

Instead of current property, maybe use version?

Fixed

In the json output, could we maybe add the slug property to the software object? Would help with scripting.

I added slug to all. I would rather not change the columns based on the format. I am fine with introducing some other options to add/remove columns though (regardless of the output format). Any suggestions here?

IDK if this is enforced / normal or not, but does the CLI have to "hang" after running it and require cntrl-c to get out? Feels a bit strange to not just get the response in the terminal output, esp with like --format json. Seems like it would make scripting a bit tricky. Ignore me if this is standard though.

Yeah this is some weird VIP cli thing. It actually seems to work fine when scripting:
./dist/bin/vip-config-software-get.js @2891 --format json | jq

Also works fine for other formats I tested.

The other thing - we already have config command, having a settings command is going to be confusing so I propose we move it under config

Good idea. I moved it there.


Salvage expedition deletes you without warning and you cannot leave

Description
We used the cargo shuttle to do a salvage expedition and after we were done the shuttle wouldn't allow us to leave. We were just going to wait out the timer but the game never warned us of the timer running out and we all got despawned along with the cargo shuttle.
Reproduction
Go into salvage expedition with cargo shuttle
Try to leave
Screenshots
I've made a quick video on what happened: https://youtu.be/mdvMZTvbVHY
Additional context

shadowcommander reported that some of the announcements didn't play so it might've been the states desyncing somehow so it never progressed.
For the leaving bit I might just have it leaveable if everyone is onboard for now as previously you could leave at any time and abandon people.

the exact same thing happened to my salv buddies on miros, and it can probably end the round due to.cargo not being available

Okay it's really stupid why it happens gonna fix it rn.


Hello, you mentioned in this paper that the code will be published, how long will it take to publish it?

Hello, you mentioned in this paper that the code will be published, how long will it take to publish it?

I am highly interested to see the released code.


Evaluate Datashader trajectory support

Check if this enables fast trajectory rendering.
See https://datashader.org/user_guide/Trajectories.html

Should be possible using custom aggregators but there are no examples so far https://github.com/holoviz/datashader/issues/969


GLM update

Updates GLM library files to release <IP_ADDRESS> (https://github.com/g-truc/glm/releases/tag/<IP_ADDRESS>)
Resolves https://github.com/tangrams/tangram-es/issues/2074

Updating GLM seems to be causing crashes and incorrect behavior, even after restoring the default initialization behavior that was removed in version <IP_ADDRESS>. This will need to be tracked down and fixed eventually if we want to ever update GLM (which we probably should).
But for now, it is simple enough to work around the immediate issue of missing build support for GCC 7.4 instead. Closing this PR in favor of #2092