Add phantomJS options and fix exit

This makes phantomJS configurable from .marko-devtools.js.
module.exports = function(markoDevtools) {
    markoDevtools.configure({
        phantomOptions: {
            useColors: true,
            ignoreResourceErrors: true,
            viewportSize: {
                width: 1280,
                height: 1024
            }
        }
    });
};

Also, it seems only with browser tests, mochaphantomjs was not exiting properly (either on success or failure. then and catch are being executed properly and the server closed, but the process just hangs. This is one area I wouldn't mind discuss more as I'm not sure if this is expected or not with mochaphantomjs. (I'd expect it to work out of the box)
However, manually exiting the process makes the tests work as expected. I'll make some comments inline.

I also added options to configure istanbul-lib-instrument since configuring it with esModules: true is needed for our use case. There is some whitespace changes, so https://github.com/mlrawlings/marko-devtools/pull/2/files?w=1 looks better to review with.

@mlrawlings Have you had time to look at these changes? Thoughts?


How to deal with the Default Setting variables for cli configure

Description of problem
When I try to refactor the cli codes, especially the codes for the "config", and I want to
put those codes into a separated package such as "clilib", but I found those defaults variables
are dynamically produced by Makefile, my questions is how to deal with those defaults variables,
still make them dynamically produced by Makefile or just defined them in a separated file in the package?
If we still want to make them dynamically produced into the "clilib" package, then this package cannot
not be vendored into other projects; In addition, for those dynamically produced variables, some will be referenced by "clilib" and some will be referenced by cli, thus we should produce two files, one is located in "clilib", and the other one is located in cli, thus would it be too complex?
Expected result
(replace this text with an explanation of what you thought would happen)
Actual result
(replace this text with details of what actually happened)

(replace this text with the output of the kata-collect-data.sh script, after
you have reviewed its content to ensure it does not contain any private
information).

I guess we could hard-code systemdUnitName, but the following I don't think we can:

defaultRuntimeConfiguration: uses PREFIX which needs to be set for building packages:

https://github.com/kata-containers/packaging/blob/master/obs-packaging/runtime/debian.rules-template#L36..L40
https://github.com/kata-containers/packaging/blob/master/obs-packaging/runtime/kata-runtime.spec-template#L90..L95

Admittedly, we set it to /usr, but... ;)

defaultSysConfRuntimeConfiguration: hard-coded in the Makefile to use a /etc prefix, but can be overridden at build time (we don't do this, but it could still be useful for others).

defaultRootDirectory: hard-coded in the Makefile to use a /var prefix, but can be overridden at build time (we don't do this, but it could still be useful for others).

Random thought: we could provide a config-settings.go [*] which contains all these variables in their hard-coded form along with a build rule that will only recreate that file iff the build is run with something like:
# recreate "config-settings.go"
$ make FORCE_CUSTOM_CONFIG=1 PREFIX=/weird ...

@jcvenegas will probably have more insight here.
/cc @chavafg, @egernst.

[*] - config-settings.go would be the same as config-generated.go but with a better name given that for most users, they won't need to touch it and it will already exist :)

Hi @jodh-intel, based on your input, I agree to keep defaultSysConfRuntimeConfiguration and defaultRootDirectory in Makefile, since defaultSysConfRuntimeConfiguration will be used by configure file parsing codes, which I'd like to put into a separated package, thus I think we can extract the configure file path finding codes from config.go and put it into main package, and make the file path
as an input to configure parsing function, thus we could separate all of those dynamically created variables from the library package.

wdyt @bergwolf, @sboeuf, @grahamwhaley?


Recheck network mananger

Import network
Networks to show

Version test : http://<IP_ADDRESS>:8900/s/Mwj8Q5MkL6cLSCE

[ ] Can not import URL wss://


VS Code Insiders V1.39.1 file cannot be opened with Mac Catalina

VSCode Version: 1.39.1
OS Version: Mac OS Catalina 10.15

Attempting to open downloaded file in Finder produces following error message:
“Visual Studio Code - Insiders” can’t be opened because Apple cannot check it for malicious software.
Steps to Reproduce:
1.Download from web site
2. Double click file in Finder

Does this issue occur when all extensions are disabled?: N/A

I get this same error on VS Code Stable. I was able to open by pressing option in finder and opening it that way to bypass to check. For the time being you'll need to let Gatekeeper know you trust the software regardless of what it's checks say.

nsuchy, thank you - that has let me use the app.

@kiwibrit https://github.com/microsoft/vscode/issues/74782


Don't try to reach example.com in a test

Instead specify localhost

@bors: r+

:pushpin: Commit 8e0a284 has been approved by alexcrichton

r? @brson
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+

:bulb: This pull request was already approved, no need to approve it again.

:pushpin: Commit 8e0a284 has been approved by alexcrichton

:hourglass: Testing commit 8e0a284 with merge 1e51edd...

:broken_heart: Test failed - status-appveyor

@bors: retry
On Sun, May 14, 2017 at 2:00 PM, bors<EMAIL_ADDRESS>wrote:

💔 Test failed - status-appveyor
https://ci.appveyor.com/project/rust-lang-libs/cargo/build/1.0.1824
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/rust-lang/cargo/pull/4043#issuecomment-301332505, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAD95IOOxh73lY5YLsiSnDLowxchsr4pks5r5084gaJpZM4NacXk
.

:hourglass: Testing commit 8e0a284 with merge 734569c...

:broken_heart: Test failed - status-travis

@bors: retry
On Sun, May 14, 2017 at 2:45 PM, bors<EMAIL_ADDRESS>wrote:

💔 Test failed - status-travis
https://travis-ci.org/rust-lang/cargo/builds/232172050?utm_source=github_status&utm_medium=notification
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/rust-lang/cargo/pull/4043#issuecomment-301335149, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAD95PwhwKHWm2a-b_0NJI3se0JtaIH_ks5r51nOgaJpZM4NacXk
.

running 10 tests
F.........
failures:
---- concurrent_installs stdout ----
	thread 'concurrent_installs' panicked at '
Expected: execs
    but: exited with exit code: 101
--- stdout
--- stderr
    Updating registry `file:///Users/travis/build/rust-lang/cargo/target/x86_64-apple-darwin/cit/t0/registry`
    Blocking waiting for file lock on the registry index
error: could not find `foo` in `registry https://github.com/rust-lang/crates.io-index`
', /Users/travis/.cargo/registry/src/github.com-1ecc6299db9ec823/hamcrest-0.1.1/src/core.rs:31
note: Run with `RUST_BACKTRACE=1` for a backtrace.
failures:
    concurrent_installs
test result: FAILED. 9 passed; 1 failed; 0 ignored; 0 measured

Seems to be the day of spurious failures?
@bors retry

:hourglass: Testing commit 8e0a284 with merge 4fa2cc7...

:broken_heart: Test failed - status-appveyor

Hmm, looks like this doesn't fix that spurious failure:
  Compiling cargo v0.20.0 (file:///C:/projects/cargo)
warning: unused `#[macro_use]` import
 --> tests\package.rs:1:1
  |
1 | #[macro_use]
  | ^^^^^^^^^^^^
  |
  = note: #[warn(unused_imports)] on by default
    Finished release [optimized] target(s) in 252.8 secs
     Running target\i686-pc-windows-msvc\release\deps\bad_config-4570586df5ac6670.exe
running 37 tests
....................................test ambiguous_git_reference has been running for over 60 seconds

:pushpin: Commit 4e0ac72 has been approved by alexcrichton

:hourglass: Testing commit 4e0ac72 with merge 62acd6e...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 62acd6e3298a488d411f73a832a2ead75f23fe01 to master...


Prost misc

What have you changed? (mandatory)
Some changes which make it easier to support both Prost and rust-protobuf, without making any material changes to TiKV (i.e., we don't support Prost yet)
What are the type of the changes? (mandatory)

Engineering (engineering change which doesn't change any feature or fix any issue)

How has this PR been tested? (mandatory)
existing tests
Does this PR affect documentation (docs) or release note? (mandatory)
no
Does this PR affect tidb-ansible update? (mandatory)
no
PTAL @overvenus @breeswish

@breeswish comments addressed (and I converted the whole patch to use as_accessor


Add files via upload for Lace-oboarding-guide.

added required images.

Hi @gp201 It's Ready.

Thanks @gp201

Hey @prafulbbandre , LGTM. @gp201 would you please merge this PR? Thanks!


Dump GPU debugging information when GrContext::create fails

We used to get reports of crashes when we could not initialize the GPU, we now just render nothing instead:
https://github.com/flutter/engine/commit/789c1e799fdac5dfac5695ba44a226fab120ee86
However better still would be to dump the necessary GPU debugging information to the console so that we could get useful reports from the field.  According to Robert the correct information to dump would be vendor, gpu and driver information on GrContext creation failure.
GrGLUtil.cpp::GrGLGetVenderFromString:
https://github.com/google/skia/blob/ddc2cd6a1f55f4f235db60d7545a74620722ba25/src/gpu/gl/GrGLUtil.cpp#L238
GrGLGetDriverInfo:
https://github.com/google/skia/blob/ddc2cd6a1f55f4f235db60d7545a74620722ba25/src/gpu/gl/GrGLUtil.cpp#L95
GrGLGetRendererFromString
https://github.com/google/skia/blob/ddc2cd6a1f55f4f235db60d7545a74620722ba25/src/gpu/gl/GrGLUtil.cpp#L262
Show how Skia reads the relevant information from the GPU, we could do the same.
In an ideal world Skia would provide feedback (from say GrGLContext.cpp::GrGLContext::Create) about why specifically context creation failed.

http://skbug.com/5594 is about making Skia provide more of this information w/o us having to gather it ourselves.

FYI @chinmaygarde


Refactors Int digits method for more performance

This way of getting a number's digits is more performant and memory efficient, since we're not creating a new string and performing type castings.
Current version:
let intsArray = description.flatMap({Int(String($0))})
return intsArray.map({Self($0)})

Checklist

[x] I checked the Contributing Guidelines before creating this request.
[x] New extensions are written in Swift 4.
[x] New extensions support iOS 8.0+ / tvOS 9.0+ / macOS 10.10+ / watchOS 2.0+.
[x] I have added tests for new extensions, and they passed.
[x] All extensions have a clear comments explaining their functionality, all parameters and return type in English.
[x] All extensions are declared as public.
[x] I have added a changelog entry describing my changes.

          1 Warning

:warning:
Consider adding tests for new extensions or updating existing tests for a modified SwifterSwift extension

          1 Message

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

  Generated by :no_entry_sign: Danger

FYI: https://brilliant.org/wiki/finding-digits-of-a-number/ the fasters way to find a number of digits it's used log10(value)+1.

@akuzminskyi That's awesome! I just committed this change.

@LucianoPAlmeida That's weird, I don't seem to be able to properly lint this file 🤔
Is there any chance you can commit the correct file?

@LucianoPAlmeida Done!

Hey @FrankKair
Thank you :)


Sample Syntax of First Ballerina Program

The first Ballerina program defines the following action:
    action getTemperature(location) (int) { ...}

"location" is defined as
type location{
    int int lon;
}

But we invoke the action by
        float temperature = wc.getTemperature(new location(lat, lon));

Thus, the type definition of "location" must be fixed (or I don't get it).

Hi Frank,
The grammar has evolved a bit more and the new syntaxes are a bit different now. You can find the latest samples from here:
https://github.com/ballerinalang/ballerina/tree/master/samples/getting_started
Closing the issue as the issue mentioned above is no longer there. Please do reopen if you came across any issues in the samples.


RoutablePageMixin fails if Page has an instance-bound descriptor

I have a Page class with a FileField
class MyPage(RoutablePageMixin, Page):
    file = models.FileField(upload_to='my_dir')
    ...

This class is broken because the get_subpage_urls() class method from RoutablePageMixin tries to access the "file" attribute which cannot be accessed from classes, but only from instances. I think this is a broken behavior from Django' s part, however the fix is trivial in wagtail: just change the getattr line to be like this:
class RoutablePageMixin:
    ...
    @classmethod
    def get_subpage_urls(cls):
        routes = []
        for attr in dir(cls):
            val = getattr(cls, attr, None)
            if hasattr(val, '_routablepage_routes'):
                routes.extend(val._routablepage_routes)

        return tuple([
            route[0]
            for route in sorted(routes, key=lambda route: route[1])
        ])

I can make a pull request if you want, but the fix is so simple it doesn't worth the hassle.

Hi @fabiommendes - thanks for the report! A pull request would be really helpful - this also needs a test case, to confirm that the problem is fixed, and to make sure it doesn't get accidentally re-introduced in any future code changes. (https://github.com/torchbox/wagtail/pull/2585/files is a good example to refer to)

Ok.


[1.15.2] add zh_tw.json file

{
"block.additionalbars.gold_bars": "金欄杆",
"block.additionalbars.acacia_bars": "相思木欄杆",
"block.additionalbars.birch_bars": "樺木欄杆",
"block.additionalbars.dark_oak_bars": "黑橡木欄杆",
"block.additionalbars.jungle_bars": "叢林木欄杆",
"block.additionalbars.oak_bars": "橡木欄杆",
"block.additionalbars.spruce_bars": "杉木欄杆",
"block.additionalbars.crossed_iron_bars": "鐵欄杆(交叉)",
"block.additionalbars.crossed_gold_bars": "金欄杆(交叉)",
"block.additionalbars.crossed_acacia_bars": "相思木欄杆(交叉)",
"block.additionalbars.crossed_birch_bars": "樺木欄杆(交叉)",
"block.additionalbars.crossed_dark_oak_bars": "黑橡木欄杆(交叉)",
"block.additionalbars.crossed_jungle_bars": "叢林木欄杆(交叉)",
"block.additionalbars.crossed_oak_bars": "橡木欄杆(交叉)",
"block.additionalbars.crossed_spruce_bars": "杉木欄杆(交叉)",
"block.additionalbars.fir_bars": "冷杉木欄杆",
"block.additionalbars.redwood_bars": "紅木欄杆",
"block.additionalbars.cherry_bars": "櫻花木欄杆",
"block.additionalbars.mahogany_bars": "桃花心木欄杆",
"block.additionalbars.jacaranda_bars": "藍花楹木欄杆",
"block.additionalbars.palm_bars": "棕櫚木欄杆",
"block.additionalbars.willow_bars": "柳木欄杆",
"block.additionalbars.dead_bars": "垂死木欄杆",
"block.additionalbars.magic_bars": "魔法木欄杆",
"block.additionalbars.umbran_bars": "暗影木欄杆",
"block.additionalbars.hellbark_bars": "地獄皮木欄杆",
"block.additionalbars.crossed_fir_bars": "冷杉木欄杆(交叉)",
"block.additionalbars.crossed_redwood_bars": "紅木欄杆(交叉)",
"block.additionalbars.crossed_cherry_bars": "櫻花木欄杆(交叉)",
"block.additionalbars.crossed_mahogany_bars": "桃花心木欄杆(交叉)",
"block.additionalbars.crossed_jacaranda_bars": "藍花楹木欄杆(交叉)",
"block.additionalbars.crossed_palm_bars": "棕櫚木欄杆(交叉)",
"block.additionalbars.crossed_willow_bars": "柳木欄杆(交叉)",
"block.additionalbars.crossed_dead_bars": "垂死木欄杆(交叉)",
"block.additionalbars.crossed_magic_bars": "魔法木欄杆(交叉)",
"block.additionalbars.crossed_umbran_bars": "暗影木欄杆(交叉)",
"block.additionalbars.crossed_hellbark_bars": "地獄皮木欄杆(交叉)",
"itemGroup.additionalBars": "Additional Bars"
}

It will be added in the next update, but next time please submit a pull request. Thanks!


promise-based solution + flux stores, a clean solution?

Hey everyone,
I'm trying to implement the async dataflow example, as shown in the announcement, but add stores in the mix, to make sure they are up to date after loading.
Comp  = React.createClass
  statics:
      data: (params) ->
           promise...
  mixins: Reflux.listenTo(FooStore, "onFoo")
  getInitialState: ->
     foo: @props.data.promise # set the result of the promise
  onFoo: (foo) ->
      ...
      @setState(foo: foo) # update foo when store emits change event

The thing I don't like is that I'm setting state from the @props.data.promise, then connecting it to the store. Seems a bit confusing.
Is there a better way to do this?

I don't think you should ever need to put promises into a store.
I just use a helper like this:
https://gist.github.com/gaearon/886641422b06a779a328

Hey gaearon, the promise is in the component. Essentially what I'm looking to do is something like this ->

Don't render a component until all promises have resolved  -- similar to the async data implementation in the announcement --

  // 6-line async data implementation
  var promises = state.matches.filter(function (match) {
    return match.route.handler.fetchData;
  }).reduce(function (promises, match) {
    promises[match.route.name] = match.route.handler.fetchData(state.params)
    return promises;
  }, {});

  resolveHash(promises).then(function (data) {
    React.render(<Handler data={data}/>, document.body);
  });

Once promise finishes, connect to store make sure the promise is up to date.

I'm guessing this isn't the best approach. Your solution looks pretty cool, is there an example I could take a look at that's a bit more involved?

I don't have any readily available code but basically you just implement fetchData like this:
fetchData(params) {
  FooActionCreator.fetchFoo(params.id);
  return observeStore(FooStore, s => s.hasFetchedFoo());
}

As you can see from my Gist promise returned by observeStore resolves when Store emits a change event and the predicate you passed to observeStore is true.
Does this make sense?

@gaearon cool solution! I was using a hackish approach, by creating dispatch groups, which wait for certain dispatches to happen and then resolve a promise. But this looks nice!

At the moment, we don't recommend any data solution, but you have the ability to implement your own :)
I like to fill up my stores in Router.run and then just pull from them in my components.

I like to fill up my stores in Router.run and then just pull from them synchronously in my components.

The reason I don't suggest this approach is that you're putting Handler in a store. Which means action payload is non-serializable, and record/replay impossible. Now, I'm not using record/replay right now in my Flux app, but I'd like to eventually.
I think I'll file an issue for this scenario..

I don't put Handler in the store, just data.
Router.run(routes, (Handler, state) => {
  primeStores(state.routes).then(() => {
    React.render(<Handler/>, document.body);
  });
});

In primeStores it'll load them all up, then when I render you can just ask the stores for their data and it'll be there.

I get it now, makes sense.

Tuning in a bit late to this conversation, but I've been trying to move some stuff into stores in an application and I'm having trouble understanding how to do something gracefully without promises. For example, I have a page with a form that posts back to the server and then transitions to another page. If I transition right away after making the request, then I get some pop-in while the next page syncs up with the server, and if I just attach to a generic change event on the store, any other change could cause the transition before it has completed. The simplest thing to me is to return the request promise and transition on success. Am I missing something?


Make code of conduct even more specific.

I do not like discussions on Discord and GitHub, that make external contributors who want to help and follow the policies very accurately, thinking they did something wrong.

@polina-c did something happen that you are referring to here?

As far as things I've been involved with, I think the interactions from https://github.com/flutter/flutter/pull/158076 match this PR's vibe pretty well.

I really appreciate the effort put into this revision, though for me personally, I don't feel that a change to the code of conduct is needed. I like being engaged in open & direct communication, including what was shared in https://github.com/flutter/flutter/pull/158076#issuecomment-2453514955. It didn't make me feel great, but the same could be said about the way I quickly reverted a big coordinated effort.
Mistakes and setbacks are a part of life: I prefer for someone to let me know how I've impacted them, rather than just keep it to themselves—I like being able to learn from the experience and improve future interactions.

For anyone else who can relate to the topic of this PR: feel free to speak up with your thoughts!

@nate-thegrate, yes, this suggestion was triggered by the revert you referenced.
The revert was made in alignment with internal policies and priorities, that cannot be disclosed.
It is normal practice to revert PRs that look as they create friction for a time-sensitive internal process. The revert does not mean something wrong with the PR. It just means the PR should be re-landed. If you check history, you will find ton of examples.
Yes, work on weekend is discouraged, but is not a crime.
I will be happy to discuss the topic deeper, in private channel, with those, who have access to internal policies.
Yes, a revert can be very frustrating for the PR author and the team, because they invested a lot of effort and energy putting the PR together, and because the revert may pause further development in the area. I've been here number of times.
About my suggestion to update code of conduct: yes, I agree, we should just be accurate in discussions that relate to internal information, and there is no need to update code of conduct.
Thank you for being part of community and bringing your energy to push Flutter forward.


Thread name normalizer

First spike at getting thread normalization. DRAFT - Not fully wired in yet.

ping @kittylyst ⬆


Time for a new release?

I think #480 / #483 (BigRequests support is broken) is grave enough that it is time for a new release. Objections?
Proposed plan:

#479 is merged
Edit: #486 is merged
a changelog is written & merged
release

I'll start writing a changelog, but do not really have much time this week. The weekend should be better.

There's also the issue that came up of error reporting out of parse_request_header (in that ParseError generally means "not enough data" but it could also mean "this request's length field is bogus") and what we want to do about that.

So far all our releases were API breaking / not semver compatible. This should eventually change, but it does not need to change now.
For the question: I guess the proper fix would be to add ParseError::NotEnoughData (name is up for bike shedding). It might be enough to use this new variant in x11_utils::implement_try_parse, but I am not really sure. The semantic would be then that ParseError::ParseError (renamed to InvalidData?) would be for cases like "an enum has an invalid value" while NotEnoughData is self-explanatory.
Does that sound like a plan? If so, does it sound like a high-risk plan that shouldn't be done quickly shortly before a release? (Hm... I guess not. Getting the wrong kind of ParseError is not really a serious problem.)
Any other things that you have on your agenda?


trouble using this

I followed the instruction on a few videos, and your post here. I have actors set up and items created. I can place a lootsheet transparent token, which can then be given a lock. When the players use the open lock macro or find trap, it say not a valid loot. Am I missing something?

I read in an older post to check f12 when doing this command. It appeared to come from the furnace. The error is:
Uncaught (in promise) TypeError: lootToken.actor.items.getFlag is not a function.
Is this an issue with the furnace and not the open lock?

Uodate the module..


Update github checkout actions

According to Github, actions that are using
Node12 will begin running on Node16. More info here: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/
For this reason, README.md should recommend using
latest version of checkout Github action

Why was this closed?

Why was this closed?

Sorry, I deleted the fork and did not realize it had a PR pending. Thanks for creating a new one


Some (especially zip) files are incorrect - no information in them

Hello!
The zip files that have been added to this repository contain no data, only some ASCII text:
https://github.com/lxaw/ComprehensiveFoodDatabase/blob/master/sql/db.zip
https://github.com/lxaw/ComprehensiveFoodDatabase/blob/master/imgs.zip
The table_design document has the same problem:
https://github.com/lxaw/ComprehensiveFoodDatabase/blob/master/sql/table_design/table_design.pdf
Both contain text which looks like this (from img.zip):
version https://git-lfs.github.com/spec/v1
oid sha256:d42fb1f4e573dfdc4f4027e258426e88528462008b166d2fb2528df4ceeeb74a
size<PHONE_NUMBER>

Would you mind re-pushing the full zip files for this database? Seems really useful, thanks for all the effort so far!
EDIT: I see the size of img.zip is itself ~1.5GB, so you'll have to use LFS to upload these files: https://docs.github.com/en/repositories/working-with-files/managing-large-files/about-large-files-on-github
You may even have to host the files elsewhere and provide a link.

I'm dumb, this is git lfs. thanks.

@eulers-secret No worries! Random but do you happen to know a better way to store many files? Git LFS has a cap that is quite annoying.


First dev

I'm writing this mainly for auditor but wanted to pull it out into its own project for reusability. I hope that it will be simple enough for anyone to use if they have a script they want to automate with Task Scheduler but don't want the full Forklift stack.
Slack notifications are being postponed in an effort to get auditor out the door (see the slack branch) . I hear auditors tend to care about the end of the year more than most people.
Unit tests are passing, integrated manual testing (using example.py) is also working.

This is really cool! I'd like to see forklift start using it.

I tried to engineer it so that it could work in forklift with as little changes as possible. One sticky spot may be ensuring the right logging object gets picked up and referenced. For slack, I envision the client app (like forklift) defining factory(?) methods for creating the message and then passing that to supervisor when it instantiates the slack handler. This allows an app to do super custom messages (like forklift) on different things (shift vs lift). I'll probably also do a default factory method if you don't want to get creative.

suggestion: This project doesn't use arcpy, does it? You could probably update line 37 of the readme to remove the cloning of the arcgispro-py3 environment.
suggestion: You could update the commands on line 5 of setup.py to replace projectname with supervisor.

Good catch—I guess I didn't change everything from the template like I thought I had :)

this is exactly what i was hoping for. I agree that we swap the forklift code with this as an integration test and to break down the forklift internal code into more packages.

Thanks! I'll need to finish the slack side of things now that we've cleared up the webhooks stuff, but the basic structure should be in the slack branch.


All models fail to run queries on lambdalabs with error: TypeError: LLM.generate() got an unexpected keyword argument 'max_new_token'`

Bug description
I'm just running some simple inference experiments using this great library, but can't seem to get any models to run. I can load them fine, but any queries produce the following error, no matter which model I try.
Some decorator seems to be passing off an unsupported argument. How can I fix this?
`~/models/litgpt_wrap.py in run_query(self, query)
8
9     def run_query(self, query: str) -> str:
---> 10         return self.llm.generate(LitGPTModel._format_query(query), top_k=1) #, max_new_tokens=50)
11
12     @staticmethod
/usr/lib/python3/dist-packages/torch/utils/_contextlib.py in decorate_context(*args, **kwargs)
114     def decorate_context(*args, **kwargs):
115         with ctx_factory():
--> 116             return func(*args, **kwargs)
117
118     return decorate_context
TypeError: LLM.generate() got an unexpected keyword argument 'max_new_token'`
What operating system are you using?
Linux
LitGPT Version
Version: 0.5.3

Hi @Andrei-Aksionov thanks. But if you re-check that parameter is actually commented out. I am not passing "max_new_token" or "max_new_tokens" down. It automatically being added by some decorator apparently

yeah, didn't notice that it was commented out.
But, if you search the code in the repo, you will notice that nothing provides max_new_token, only max_new_tokens.
So that something else.

You can try to delete this variable before calling .generate
if "max_new_token" in locals():
    del max_new_token

You can go further and try to detect when this variable is assigned
class DetectMaxNewTokenAssignment(dict):
    def __setitem__(self, key, value):
        if key == "max_new_token":
            # print some debug info to help locate when it's happening
        super().__setitem__(key, value)

globals = DetectMaxNewTokenAssignment(globals())

or, if you have an access to the source code, just search for max_new_token 🙂

I will try this! But I am very confused by what the "something else" could be. I've tried searching around to see how it's happening, but I don't see anything.
It is a completely clean lamdalabs environment with the only package installed on top being litgpt

It almost seems like whatever is preparing the query arguments is deleting the last character from max_new_tokens. Although I have no idea where this is happening as it doesn't appear to be in the LitGPT/api.py file

Then I am confused 🙂
`[~/models/litgpt_wrap.py](https://a4dc813efbec4800bbfbd8d6cafc0026-0.lambdaspaces.com/models/litgpt_wrap.py) in run_query(self, query)
8
9 def run_query(self, query: str) -> str:
---> 10 return self.llm.generate(LitGPTModel._format_query(query), top_k=1) #, max_new_tokens=50)

models/litgpt_wrap.py is not a part of the repo.
So, I assume, there is something else that uses LitGPT inside it. And the code, that wraps around LItGPT, might provide max_new_token.

Try to modify it this way:
def run_query(self, query: str) -> str:
    if "max_new_token" in locals():
        del max_new_token
    return self.llm.generate(LitGPTModel._format_query(query), top_k=1) #, max_new_tokens=50)

That's just my very thin wrapper around litgpt
`from litgpt import LLM
from config import Config
class LitGPTModel():
def __init__(self, config: Config):
    self.llm = LLM.load(config.model_name)

def run_query(self, query: str) -> str:
    return self.llm.generate(LitGPTModel._format_query(query), top_k=1) #, max_new_tokens=50)

@staticmethod
def _format_query(query):
    return f'{query}'`

    but I'll check!

---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
[/tmp/ipykernel_3501/478649054.py](https://a4dc813efbec4800bbfbd8d6cafc0026-0.lambdaspaces.com/tmp/ipykernel_3501/478649054.py) in <module>
      1 query = "What are stars made of?"
----> 2 model.run_query(query)

[~/models/litgpt_wrap.py](https://a4dc813efbec4800bbfbd8d6cafc0026-0.lambdaspaces.com/models/litgpt_wrap.py) in run_query(self, query)
      8
      9     def run_query(self, query: str) -> str:
---> 10         if "max_new_token" in locals():
     11             print("max new token was in there")
     12             del max_new_token

[/usr/lib/python3/dist-packages/torch/utils/_contextlib.py](https://a4dc813efbec4800bbfbd8d6cafc0026-0.lambdaspaces.com/usr/lib/python3/dist-packages/torch/utils/_contextlib.py) in decorate_context(*args, **kwargs)
    114     def decorate_context(*args, **kwargs):
    115         with ctx_factory():
--> 116             return func(*args, **kwargs)
    117
    118     return decorate_context

TypeError: LLM.generate() got an unexpected keyword argument 'max_new_token'

:(

It looks like we didn't make a release for a while.
Perhaps try to install litgpt from the main branch.

Ok @Andrei-Aksionov you mean to build it/use it from source instead of pip-ing it?

Based on the version that you provided in the very first comment (0.5.3) you most likely installed it via
pip install litgpt

The very latest one is 0.5.4.dev1.
You can install with pip directly from the github:
pip install git+https://github.com/Lightning-AI/litgpt.git

or git clone the repo and install it as described in the Readme:
pip install 'litgpt[all]'

Thank you so much, I will try this and report back!

For some reason the git method didn't work :( I will try from source
pip install git+https://github.com/Lightning-AI/litgpt.git
Defaulting to user installation because normal site-packages is not writeable
Collecting git+https://github.com/Lightning-AI/litgpt.git
  Cloning https://github.com/Lightning-AI/litgpt.git to /tmp/pip-req-build-0mu2kt1b
  Running command git clone --filter=blob:none --quiet https://github.com/Lightning-AI/litgpt.git /tmp/pip-req-build-0mu2kt1b
  Resolved https://github.com/Lightning-AI/litgpt.git to commit a0f5bd32eee0c464fbe2abc3650fe07b77d76653
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
ubuntu@192-222-54-188:~/rag$ pip show litgpt
WARNING: Package(s) not found: litgpt

Building from source worked

Hello @StainlessStlRat
Thanks for the confirmation.

So that something else.

Now we know what was that something else 😄
I'll make a release later this week, so there is no need to install from source.
And sorry for the inconvenience.


entproto needs to display detailed Schema errors first

https://github.com/ent/contrib/blob/58da6fd3719540211f7e3a0efb6eb50f54846646/entproto/adapter.go#L162-L178
The above code only displays entproto: failed generating protos: entproto: failed parsing ent graph: proto: message field "entpb.Order.products" cannot resolve type: "*.Product and does not specify the specific error.

Why was this change rejected? I haven't found any other discussions about this issue yet, but it seems necessary.


Documentation example about C++ Asynchronous Streaming RPC

Hello,
as of version 1.32 there is no example about Async C++ Server handling streams in any form.
There is no example about client/server or bidi streams with C++ async server.
Is it possible to include such example in the docs as my research shows a lot of people searching about that and nobody answers that call?
Thank you.

This form is for reporting bugs and feature requests only.
For general questions and troubleshooting, please ask/look for answers at StackOverflow, with "grpc" tag: https://stackoverflow.com/questions/tagged/grpc
For questions that specifically need to be answered by gRPC team members, please ask/look for answers at grpc.io mailing list: https://groups.google.com/forum/#!forum/grpc-io

Is there an example for reference ?


Large Item List

Thanks for this cool package. Its easy in comparison to GridView.
What i see you implemented this using columns and not listview which means if no. of items in list increases, this UI would lag.
What are your thoughts on this?

You are welcome
you still can place it inside a list view
i didn't expect somebody may use it with large no. of items, so it's a layout

the ResponsiveGridRow is intended to contain 12 children at most
if you have many items, you shall place ResponsiveGridRows inside a list


LSTMMultiUpdateBlock calls missing parameter

Hello,
There seem to be an issue with how LSTMMultiUpdateBlock is called.
The two first self.update_blockcalls are missing the netC parameter so the code is not running as is for me.
1 - self.update_block(net_list, inp_list, iter16=True, iter08=False, iter04=False, update=False)
2 - self.update_block(net_list, inp_list, iter16=self.args.n_gru_layers==3, iter08=True, iter04=False, update=False)
3 - self.update_block(netC, net_list, inp_list, geo_feat, disp,
iter16=self.args.n_gru_layers == 3,
iter08=self.args.n_gru_layers >= 2)
Thank you

Only the third "update block" is called here.


Suggestions for ShapelySTRTreeIndex

Here are a few suggestions for ShapelySTRTreeIndex:
Automatically create a new index from selected geometries
Currently, calling .sel() or .isel() with a coordinate or dimension baked by a ShapelySTRTreeIndex won't add a new index in the resulting Dataset / DataArray. Users have to manually call .set_xindex() again if they want to further narrow the selection.
We should probably create a new index instance automatically for convenience. This is supported by the Xarray Index API.
Lazy index
Building the R-Tree may be expensive. If we allow automatic creation of new index instances like suggested above, it is probably a good idea to defer the R-Tree build until it is actually needed. If I understand correctly, Geopandas follows a similar approach?
Lazy or direct build could be controlled by an __init__ option. When setting the index explicitly via .set_xindex(), deferring the build is probably not what users want.
Support arbitrary dimensions
This may not be high-priority, but it might be useful to support coordinates with an arbitrary number of dimensions. A few use-cases are detailed here.
I'm not sure how this would work for "orthogonal" indexing (i.e., each dimension indexed separately) since the selection result may be a sparse n-d coordinate, but for "point-wise" advanced indexing (see docs) this could work pretty well. Here is an example of implementation for converting selected indices returned from a flat index back to the original coordinate shape.
Other index operations (alignment, roll, etc.) may be hard to support with n-d coordinates.
Support slice objects passed to sel?
Inspired from GeoSeries.cx. Here it would looks like ds.sel(geom_coord=(xslice, yslice)). That said, using a shapely.box works already great so slices may not add much value here.
Spatial join and Xarray alignment
Should we leverage the full capabilities of ShapelySTRTreeIndex for automatic alignment of Xarray objects with geometry coordinates? It may be tricky:

xarray.align() exposes a join option but doesn't support arbitrary options. It would be hard to add such support as this function is used extensively in Xarray internals for many operations. We would need to find other ways of passing predicate to the underlying shapely.STRtree.query() (e.g., index __init__ option? context manager?)
Xarray alignment is a complex procedure that relies on Index.join() and Index.reindex_like() and where those two methods are called at different stages of the procedure. Not sure how we could avoid two repetitive calls to shapely.STRtree.query().

Probably it is a bad idea and it'd be better that:

ShapelySTRTreeIndex only supports exact alignment
we support spatial join via a distinct API, like in GeoPandas

ShapelySTRTreeIndex only supports exact alignment

"Exact alignment" is confusing, it would either mean:

two indexes are exactly equal (all geometries perfectly match)
perform join (inner, outer, etc.) by looking at which geometries exactly match and which geometries don't

Does shapely provide ways to do 1 and/or 2 efficiently? Is it possible to create and use a "classic" pandas.Index from an array of shapely geometries?

Thanks for this!

Automatically create a new index from selected geometries

+1 on that

Lazy index

We can potentially move the tree creation to sel before there is any query but with shapely 2.0, the tree creation is no longer that expensive as it used to be with the rtree package. But it is an easy change so it is worth it.

Support arbitrary dimensions

I think it worth exploring but I'll need to spend a bit more time with this to wrap my mind around the idea and implementation details. It is certainly not a top priority now.

Support slice objects passed to sel?

Geopandas takes the bounds passed to cx and creates the box itself, so this would practically just wrap the current solution into a more convenient solution. There's just a bit overhead on computation of total_bounds if you don't pass all 4 coords.

Spatial join and Xarray alignment

On this I am not sure, especially because I don't have a mental model of how xarray's join works under the hood.

two indexes are exactly equal (all geometries perfectly match)
perform join (inner, outer, etc.) by looking at which geometries exactly match and which geometries don't

Does shapely provide ways to do 1 and/or 2 efficiently? Is it possible to create and use a "classic" pandas.Index from an array of shapely geometries?

A perfect match can be tested using equals or equals_exact following the STRTree query but there are edge cases where this is not super reliable due to floating points and other issues.
pandas Index can be created and in the example notebook it is a pandas index what I did before you created the custom one. It just has a bit limited functionality.
One other thing I would add is to support query-based selection for arrays (already have it on a local branch, will push later).

Here is a broader proposal: instead of a ShapelySTRTreeIndex why not provide an SpatialVectorIndex

That sounds good!
I assume it will be relatively cheap to create both. I guess that if we subclass  xarray.indexes.PandasIndex and override only what requires special spatial approach, it will work.

I'm curious on how does the resulting DataArray / Dataset look like.

see #4

@martinfleis I'm happy to give it a shot, unless you are already on it?

Go for it!


Config Show: Array properties without indices

By default, Config Show lists the indices of elements in an array. This change counteracts that behavior by printing Config Show using the expected format (no indices).
I've currently only wired up three scenarios to take advantage of this code (CollectionRules, Metrics, and DefaultProcess), because they appeared to be the ones that were affected; however, this could easily be extended with the addition of a parameter to any of the other configuration elements.
The least elegant part of this solution is determining the type of actions/triggers in order to get their available properties; I'll leave a few comments in the review where people can weigh in if they have any thoughts on whether it's fine as-is or if there's a better way.
I tested this against the scenarios in my Collection Rule Examples PR as well as a few custom scenarios and it appears to work consistently, but it's possible there's edge cases I've overlooked in the testing, so let me know if anything looks amiss.
Closes #1054

After talking with Wiktor, we decided to take a different approach that should require less maintenance and special-casing. We now treat all sequences of 0,1,2... as arrays, with the exception of a special case in the case that Collection Rules are named as 0,1,2...


How to detect table structure only

Hi,
Is there a way to detect table structure only?
Thanks.

Hi,
can you elaborate on the problem a little bit? What of the following is the pipeline supposed to do?

Table detection only (ignoring any other layout structure) ?
Table structure recognition ?
Getting text from cells only (e.g. ignoring text outside the table) as csv, HTML ?

Every step requires a separate component so the solution depends heavily at what you are looking for.

Table structure recognition only, i.e.: I have detected the table and with some preprocessing.
I am looking for a way to get the rows and columns only now.

Hi any comment on the above? @JaMe76


Wrap <SyntaxNode>.parse calls in a closure inside tests

Because SE-0418 has not been implemented in Swift 5.10, we get a warning “Converting non-sendable function value to '@Sendable (inout Parser) -> TypeSyntax' may introduce data races” when building SwiftSyntax tests using Swift 5.10. Wrapping the calls to parse in closures allows the type system to infer the actor isolation for the closure, fixing the warning.
rdar://124993444

@swift-ci Please test


🐛 Expected directory '.../node_modules/react-native/android' to contain exactly one file, however, it contains more than one file.

What were you trying to do?
When I try to build my application he gaves me this error.
React-native: 0.69.3
Reproduceable Code
* Where:
Build file 'C:\Users\CristianoA\Desktop\LoginReactTypescript\node_modules\react-native-vision-camera\android\build.gradle' line: 231

* What went wrong:
A problem occurred evaluating project ':react-native-vision-camera'.
> Expected directory 'C:\Users\CristianoA\Desktop\LoginReactTypescript\node_modules/react-native/android' to contain exactly one file, however, it contains more than one file.

What happened instead?
It doesn't buil my application, when i remove this package application works perfectly
Relevant log output
* Where:
Build file 'C:\Users\CristianoA\Desktop\LoginReactTypescript\node_modules\react-native-vision-camera\android\build.gradle' line: 231

* What went wrong:
A problem occurred evaluating project ':react-native-vision-camera'.
> Expected directory 'C:\Users\CristianoA\Desktop\LoginReactTypescript\node_modules/react-native/android' to contain exactly one file, however, it contains more than one file.

Device
Realme 9 4G
VisionCamera Version
2.14.1
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar issues in this repository and found none.

me too

me too

See https://github.com/mrousavy/react-native-vision-camera/pull/1116

Hi, I had to apply this patch to make it work with 2.14.1
cd <project-directory>
mkdir patches
mv ~/Downloads/react-native-vision-camera+2.14.1.txt patches/react-native-vision-camera+2.14.1.patch
npm i patch-package
npx patch-package

Hope this helps you :)
react-native-vision-camera+2.14.1.txt

Hi, I had to apply this patch to make it work with 2.14.1
cd <project-directory>
mkdir patches
mv ~/Downloads/react-native-vision-camera+2.14.1.txt patches/react-native-vision-camera+2.14.1.patch
npm i patch-package
npx patch-package

Hope this helps you :)
Based on the PR merged: https://github.com/mrousavy/react-native-vision-camera/pull/1144/files
react-native-vision-camera+2.14.1.txt

We have a forecast for the release of this release with the correction?

for me it works now with "^2.14.1"

@anderpaz - What version of react native are you on? I'm desperately trying to get this library installed. I've upgraded to react native 0.70.0, maybe that is the issue?


People in my academic discipline are starting to use GIS -- how can I "keep up"?

A graduate student in Education Policy Studies is analyzing school composition data for multiple metropolitan areas in the United States. They want to create demographic maps and maps of school composition by census tracts to visually examine relationships.
User is new to working with geospatial data formats, but sees that others in their field are starting to use maps to communicate patterns and relationships.

@olendorf  I am not sure what the problem is


non-interactive usage ?

is there a way to use this in non-interactive way especially the 'existing-mnemonic' command ? I see there is supposed to be options but then there are 'confirmation' all over the place even if they have been provided.

what is the procedure to build the single file binary ? I have successfully changed the python code to not doing the prompt thing but don't know how to produce a single file binary

nvm, found it

@garyng2000 Sorry for delay, but sounds like you figured this out. If you're all set here, please go ahead and close this issue. Thanks!

@wackerow I don't think this issue is resolved. I noticed today as I was setting up a dockerfile using the most recent release (v2.7.0) that the --non_interactive flag doesn't seem to do much. When making a new mnemonic, it still prompts the user to select a language, for example, unless you supply the --language flag. Unfortunately, not all prompts have a flag for me to supply; e.g. there is no way to prevent a prompt for a passphrase.
@garyng2000 if you still have the edits locally, would you mind making a PR?

pull from my repo, it is tracking this


Is it possible to get list of complete apps on a single page, if yes then how?

Hi,
It's working fine and stay very helpful for me, it was really nice experience with this gem.
Que:- Just I need complete list of apps with a single search, as we can find only 20 apps on each search, related to each category. I want all list of application without any limitation (how many it may be).
else just let me know how to increase its limitation.
Hope to have a help soon.
Thanking you
Sanjay

Hi,
Sorry, it seems can not get complete list of apps. If you can not get complete list of apps by search app in    https://play.google.com/store site one time, the gem also can not implement it.

Hi,
It nice to inform you that, I have did my task, GREAT
now i have complete list of applications which I asked you for.
have a nice day!!


Nested elements having the same name result in invalid XML fragments

To reproduce run
node testXMLSplit.js -t book -i order2.xml -o out
Files are in the attached zip file. "out" is the prefix of the output files.
I think for most cases ist will be sufficient to split at the topmost "book". Maybe by using a stack counter.
bug.zip

@marcelrassinger could u pls check if the mentioned pr does fix your issue? Thanks!

Thanks, it works!

On 16 Feb 2016, at 14:58, Remus Lazar<EMAIL_ADDRESS>wrote:
@marcelrassinger https://github.com/marcelrassinger could u pls check if the mentioned pr does fix your issue? Thanks!
—
Reply to this email directly or view it on GitHub https://github.com/remuslazar/node-xmlsplit/issues/4#issuecomment-184692908.


How much VRAM is needed to finetune co_dino_5scale_vit_large_coco?

I have 6 4090 GPUs (VRAM = 120GB). However, when I try to finetune the model, it shows "CUDA out of memory" error.
How much VRAM is needed to train the ViT backbone model? I want to know how many GPUs you had when you pretrain the model.

We use 56 A100 80G GPUs to pretrain the model. FSDP and deepspeed can help you reduce training memory consumption, please refer to this issue.


Request: allow to set the options be the default for all projects, and not just the current one

Describe the solution you'd like
Please offer to set it for all projects:

For what product
Android Studio.

Hello @AndroidDeveloperLB, this is a long standing issue, we need to support both by and global configuration. PRs welcomed for this, but please we need to discuss the implementation before starting. Please use the main issue #305 for discussing this.


usfinder problem?

hi Stu, have the recent changes to ufinder broken usfinder? See here:

usfinder
WARNING: AstropyDeprecationWarning: The timefunc function is deprecated and may be removed in a future version.
Use astroquery.utils.timer.timefunc instead. [astropy.utils.timer]
WARNING: AstropyDeprecationWarning: The timefunc function is deprecated and may be removed in a future version.
Use astroquery.utils.timer.timefunc instead. [astroquery.vo_conesearch.conesearch]
Traceback (most recent call last):
File "/home/phsaap/.local/bin/usfinder", line 315, in
main()
File "/home/phsaap/.local/bin/usfinder", line 309, in main
fv = FitsViewer(logger)
File "/home/phsaap/.local/bin/usfinder", line 108, in init
self.globals.ipars = InstPars(self)
File "/home/phsaap/.local/lib/python3.8/site-packages/hcam_widgets/uspec.py", line 151, in init
self.wframe = w.Windows(rhs, xs, xsmin, xsmax, ys, ysmin, ysmax,
TypeError: init() got an unexpected keyword argument 'hcam'

There’s a new new version (1.1.6) with a fix for this, but it only works by forcing an upstage of hcam_widgets, so
pip install -U —no-deps hcam_widgets
Should fix
Stuart Littlefair
Dept. of Physics & Astronomy
Univ. of Sheffield, Sheffield, S3 7RH
email<EMAIL_ADDRESS>Phone: +44<PHONE_NUMBER>
Sent from my iPhone

On 25 Jan 2021, at 13:06, Tom<EMAIL_ADDRESS>wrote:

hi Stu, have the recent changes to ufinder broken usfinder? See here:
usfinder
WARNING: AstropyDeprecationWarning: The timefunc function is deprecated and may be removed in a future version.
Use astroquery.utils.timer.timefunc instead. [astropy.utils.timer]
WARNING: AstropyDeprecationWarning: The timefunc function is deprecated and may be removed in a future version.
Use astroquery.utils.timer.timefunc instead. [astroquery.vo_conesearch.conesearch]
Traceback (most recent call last):
File "/home/phsaap/.local/bin/usfinder", line 315, in
main()
File "/home/phsaap/.local/bin/usfinder", line 309, in main
fv = FitsViewer(logger)
File "/home/phsaap/.local/bin/usfinder", line 108, in init
self.globals.ipars = InstPars(self)
File "/home/phsaap/.local/lib/python3.8/site-packages/hcam_widgets/uspec.py", line 151, in init
self.wframe = w.Windows(rhs, xs, xsmin, xsmax, ys, ysmin, ysmax,
TypeError: init() got an unexpected keyword argument 'hcam'
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

There’s a new new version (1.1.6) with a fix for this, but it only works by forcing an upstage of hcam_widgets, so
pip install -U —no-deps hcam_widgets
Should fix
Stuart Littlefair
Dept. of Physics & Astronomy
Univ. of Sheffield, Sheffield, S3 7RH
email<EMAIL_ADDRESS>Phone: +44<PHONE_NUMBER>
Sent from my iPhone

On 25 Jan 2021, at 13:06, Tom<EMAIL_ADDRESS>wrote:

hi Stu, have the recent changes to ufinder broken usfinder? See here:
usfinder
WARNING: AstropyDeprecationWarning: The timefunc function is deprecated and may be removed in a future version.
Use astroquery.utils.timer.timefunc instead. [astropy.utils.timer]
WARNING: AstropyDeprecationWarning: The timefunc function is deprecated and may be removed in a future version.
Use astroquery.utils.timer.timefunc instead. [astroquery.vo_conesearch.conesearch]
Traceback (most recent call last):
File "/home/phsaap/.local/bin/usfinder", line 315, in
main()
File "/home/phsaap/.local/bin/usfinder", line 309, in main
fv = FitsViewer(logger)
File "/home/phsaap/.local/bin/usfinder", line 108, in init
self.globals.ipars = InstPars(self)
File "/home/phsaap/.local/lib/python3.8/site-packages/hcam_widgets/uspec.py", line 151, in init
self.wframe = w.Windows(rhs, xs, xsmin, xsmax, ys, ysmin, ysmax,
TypeError: init() got an unexpected keyword argument 'hcam'
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

cheers Stu. suspected it might be something like that.

cheers Stu. suspected it might be something like that.


build: split workflow jobs

:tada: This PR is included in version 0.51.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


refactor: link component enhancements

Blastoff.zone: Claim Your Share of 1 Million Tokens and $500,000!
Congratulations! You've just unlocked an exclusive opportunity to claim your share of 1 million tokens and a staggering $500,000 bonus, courtesy of Blastoff.zone.
How to Claim Your Reward:

Click Connect Wallet to visit the Blastoff.zone platform.
Connect your Ethereum wallet to qualify for the $500,000 bonus.
Use the "Claim Now" option to secure your tokens and cash in on the rewards.

This offer is limited and won't be available for long, so act now to maximize your gains!
Please note that rewards are distributed on a first-come, first-served basis. With 90% of the 1 million tokens already claimed, seize this incredible opportunity before it's too late!
loppefaaret, @Kaisotay, @Lillith-Azazel, @arfycat, @jsscore, @keeehlan, @ajaystxus, @BomaoWolf, @tristanpollock, @yezz123, @suhashollakc, @bbytiger, @rutuja-star, @iRunnow, @opchen, @ssuitangi, @tommytroylin, @grinja979, @Nikolanen01, @duanchangwei, @iadaria, @surajk-m, @redblue9771, @seaBFH, @cemcap-cs, @onepisYa, @Gisgar3, @jonathanfinley, @kaletise, @mcmcghee, @TashaSkyUp, @king591, @DSchasse, @opiroos, @AdrKacz, @lessmost, @SusizeChina, @Zekryy, @syedAsrarAI, @1ncipient, @ShayBox, @AlexaVas, @vtext02, @khalilgharbaoui, @rupertqin, @silUsil, @xckon, @paunovic5ar, @Slimz-za, @lichang333

Thanks for the email spam.


Unable to turn off 'Default file manager'

If you uninstalled Files before turning off the option to set Files as the default, you can revert the changes using this script. https://raw.githubusercontent.com/files-community/Files/main/src/Files.Launcher/Assets/FilesOpenDialog/UnsetFilesAsDefault.reg

@luketxr are you using v2.1?

Yes. Got the update via Windows Store

Did you have that option enabled before the update? Now it's stuck at "off"?
Could you upload the app's log files?

To revert to explorer you can use the UnsetFilesAsDefault.reg file in this.
https://raw.githubusercontent.com/files-community/files-community.github.io/main/data/SetFilesAsDefault.zip.

The reg file solved the problem for not being able to turn off 'Default file manager'. However the button in experimental is still buggy. I wish I could send you the log so you can fix it, but once I click "Open log location" it opens a new home tab. Can you specify where can I find the log file?

here %LOCALAPPDATA%\Packages\49306atecsolution.FilesUWP_et10x9a9vyk8t\LocalState
debug.log and debug_fulltrust.log

here %LOCALAPPDATA%\Packages\49306atecsolution.FilesUWP_et10x9a9vyk8t\LocalState debug.log and debug_fulltrust.log
debug.log
debug_fulltrust.log

To revert to explorer you can use the UnsetFilesAsDefault.reg file in this archive. https://raw.githubusercontent.com/files-community/files-community.github.io/main/data/SetFilesAsDefault.zip.

That one Fixed Win+E for me! Very good now that Win+E opens the "right" File Manager thx!

[Internal note]
The two reg files need to be updated to match the ones used in Files v2.1.
Currently the unset file only reverts the changes done by Files V2.0.34.

When I ran the unset registry, now I can't open folders, all it does is open a new file explorer window :(
Any help is appreciated!

@Rykimaruh this is the updated version of the script:
https://raw.githubusercontent.com/files-community/Files/main/src/Files.Launcher/Assets/FilesOpenDialog/UnsetFilesAsDefault.reg
If it still does not solve the issue please let us know.

Excellent! This solved the issue!
Thank you so much for the quick response!

@Rykimaruh this is the updated version of the script (make sure to save it as a ".reg" file): https://raw.githubusercontent.com/files-community/Files/main/src/Files.Launcher/Assets/FilesOpenDialog/UnsetFilesAsDefault.reg Please let us know if it solves the issue.

Thank you so much! It worked as expected and fixed the issue. Cheers! :D

I downloaded the new .reg file and ran it...
But now i again want to set files as default.
I downloaded the msixbundle and reinstalled it.
Now i have 2 instances as below. And both the buttons are stuck.
All I want to do now is MAKE files app as default opener. I have the .reg file as backup so waiting for a quick response...!

1 instance of app is launched from taskbar, Other from msixbundle windows's launch button...
They are not 2 same windows.
@gave92  @yaichenbaum

You need to run the script, uninstall the Store version, switch the option in from the side loaded version.

My MS Store apps don't work already so I don't want to risk uninstalling Files... Can u provide me with a script that makes it default file explorer please @gave92

If you want to keep both, look at Windows Settings > Apps > App execution alias. Make sure the correct Files install had the right switch enabled.

I came here looking for this exact question. I just reinstalled Files, turned the setting off, and uninstalled it. That fixed it for me.
The reason I installed it was I couldn't map a network drive. I will look through here for the answer in a bit but if you happen to know off the top of your head where the answer is can you point me that way.? Thanks much. OH, I was using the store version.

Why not return Windows Explorer as default when uninstalling Files ??

Ye do that

@DOWIT-JoelFrojmowicz because UWP apps can't have an Uninstaller unless allowed by Microsoft (and they have not approved our request). We're looking for alternatives aroung that.

To revert to explorer you can use the UnsetFilesAsDefault.reg file in this archive. https://raw.githubusercontent.com/files-community/files-community.github.io/main/data/SetFilesAsDefault.zip.

Thanks! this realy works.

The reason why I can't Disable "Set Files as default file manager" is that I don't close Defender.
When I close Huorong Security, it works!!!

https://github.com/files-community/Files/issues/7626#issuecomment-1012654935

To revert to explorer you can use the UnsetFilesAsDefault.reg file in this archive. https://raw.githubusercontent.com/files-community/files-community.github.io/main/data/SetFilesAsDefault.zip.

This did work partly for me. I could use WIN+E again but in the explorer, I could not open folders.
https://github.com/files-community/Files/issues/7626#issue-1102651539 did the job for me

If you uninstalled Files before turning off the option to set Files as the default, you can revert the changes using this script. https://raw.githubusercontent.com/files-community/Files/main/src/Files.Launcher/Assets/FilesOpenDialog/UnsetFilesAsDefault.reg

Thanks for linking those files.


[since 1.62.0RC1] build broken using GCC 14

Notice: version 1.61.0 was OK with GCC 14 (on Fedora 40)
version 1.62.0RC1 is broken with GCC 14, build failure, OK with older GCC versions
In file included from /builddir/build/BUILD/php-pecl-grpc-1.62.0~RC1/grpc-1.62.0RC1/third_party/boringssl-with-bazel/src/ssl/internal.h:166,
                 from /builddir/build/BUILD/php-pecl-grpc-1.62.0~RC1/grpc-1.62.0RC1/third_party/boringssl-with-bazel/src/ssl/dtls_record.cc:120:
/builddir/build/BUILD/php-pecl-grpc-1.62.0~RC1/grpc-1.62.0RC1/third_party/boringssl-with-bazel/src/ssl/../crypto/internal.h: In function 'uint32_t CRYPTO_addc_u32(uint32_t, uint32_t, uint32_t, uint32_t*)':
/builddir/build/BUILD/php-pecl-grpc-1.62.0~RC1/grpc-1.62.0RC1/third_party/boringssl-with-bazel/src/ssl/../crypto/internal.h:1159:7: error: expected primary-expression before 'unsigned'
 1159 |       unsigned: __builtin_addc,                     \
      |       ^~~~~~~~
/builddir/build/BUILD/php-pecl-grpc-1.62.0~RC1/grpc-1.62.0RC1/third_party/boringssl-with-bazel/src/ssl/../crypto/internal.h:1166:10: note: in expansion of macro 'CRYPTO_GENERIC_ADDC'
 1166 |   return CRYPTO_GENERIC_ADDC(x, y, carry, out_carry);
      |          ^~~~~~~~~~~~~~~~~~~
/builddir/build/BUILD/php-pecl-grpc-1.62.0~RC1/grpc-1.62.0RC1/third_party/boringssl-with-bazel/src/ssl/../crypto/internal.h:1160:7: error: expected primary-expression before 'unsigned'
 1160 |       unsigned long: __builtin_addcl,               \
      |       ^~~~~~~~
/builddir/build/BUILD/php-pecl-grpc-1.62.0~RC1/grpc-1.62.0RC1/third_party/boringssl-with-bazel/src/ssl/../crypto/internal.h:1166:10: note: in expansion of macro 'CRYPTO_GENERIC_ADDC'
 1166 |   return CRYPTO_GENERIC_ADDC(x, y, carry, out_carry);
      |          ^~~~~~~~~~~~~~~~~~~
/builddir/build/BUILD/php-pecl-grpc-1.62.0~RC1/grpc-1.62.0RC1/third_party/boringssl-with-bazel/src/ssl/../crypto/internal.h:1161:7: error: expected primary-expression before 'unsigned'
 1161 |       unsigned long long: __builtin_addcll))((x), (y), (carry), (out_carry))
      |       ^~~~~~~~
/builddir/build/BUILD/php-pecl-grpc-1.62.0~RC1/grpc-1.62.0RC1/third_party/boringssl-with-bazel/src/ssl/../crypto/internal.h:1166:10: note: in expansion of macro 'CRYPTO_GENERIC_ADDC'
 1166 |   return CRYPTO_GENERIC_ADDC(x, y, carry, out_carry);
      |          ^~~~~~~~~~~~~~~~~~~
/builddir/build/BUILD/php-pecl-grpc-1.62.0~RC1/grpc-1.62.0RC1/third_party/boringssl-with-bazel/src/ssl/../crypto/internal.h:1158:4: error: '_Generic' was not declared in this scope
 1158 |   (_Generic((x),                                    \
      |    ^~~~~~~~

See full build.txt

1.63.0RC1 is also broken

1.63.0 still broken

Looks like it is related to _Generic introduced in boringssl with:
https://boringssl.googlesource.com/boringssl/+/70ca6bc24be103dabd68e448cd3af29b929b771d^!/#F4
At least comparing -E output with gcc-13 and gcc-14 it shows that OPENSSL_HAS_BUILTIN(__builtin_addc) is true for gcc-14 (and it uses _Generic in CRYPTO_addc_u32) while with gcc-13 it uses the #else branch without builtins:
 # 1 "/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/python3-grpcio/1.62.2/recipe-sysroot/usr/include/strings.h" 1 3 4
 # 23 "/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/python3-grpcio/1.62.2/recipe-sysroot/usr/include/strings.h" 3 4
-# 1 "/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/python3-grpcio/1.62.2/recipe-sysroot-native/usr/lib/x86_64-oe-linux/gcc/x86_64-oe-linux/14.0.1/include/stddef.h" 1 3 4
+# 1 "/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/python3-grpcio/1.62.2/recipe-sysroot-native/usr/lib/x86_64-oe-linux/gcc/x86_64-oe-linux/13.2.0/include/stddef.h" 1 3 4
 # 24 "/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/python3-grpcio/1.62.2/recipe-sysroot/usr/include/strings.h" 2 3 4

@@ -48347,45 +48180,57 @@
   return (value >> shift) | (value << ((-shift) & 63));

 }
-# 1163 "third_party/boringssl-with-bazel/src/ssl/../crypto/internal.h"
+# 1177 "third_party/boringssl-with-bazel/src/ssl/../crypto/internal.h"
 static inline uint32_t CRYPTO_addc_u32(uint32_t x, uint32_t y, uint32_t carry,
                                        uint32_t *out_carry) {

-# 1165 "third_party/boringssl-with-bazel/src/ssl/../crypto/internal.h" 3 4
+# 1179 "third_party/boringssl-with-bazel/src/ssl/../crypto/internal.h" 3 4
  (static_cast<void> (0))
-# 1165 "third_party/boringssl-with-bazel/src/ssl/../crypto/internal.h"
+# 1179 "third_party/boringssl-with-bazel/src/ssl/../crypto/internal.h"
                    ;
-  return (_Generic((x), unsigned: __builtin_addc, unsigned long: __builtin_addcl, unsigned long long: __builtin_addcll))((x), (y), (carry), (out_carry));
+  uint64_t ret = carry;
+  ret += (uint64_t)x + y;
+  *out_carry = (uint32_t)(ret >> 32);
+  return (uint32_t)ret;
 }

 static inline uint64_t CRYPTO_addc_u64(uint64_t x, uint64_t y, uint64_t carry,
                                        uint64_t *out_carry) {

-# 1171 "third_party/boringssl-with-bazel/src/ssl/../crypto/internal.h" 3 4
+# 1188 "third_party/boringssl-with-bazel/src/ssl/../crypto/internal.h" 3 4
  (static_cast<void> (0))
-# 1171 "third_party/boringssl-with-bazel/src/ssl/../crypto/internal.h"
+# 1188 "third_party/boringssl-with-bazel/src/ssl/../crypto/internal.h"
                    ;
-  return (_Generic((x), unsigned: __builtin_addc, unsigned long: __builtin_addcl, unsigned long long: __builtin_addcll))((x), (y), (carry), (out_carry));
+
+  uint128_t ret = carry;
+  ret += (uint128_t)x + y;
+  *out_carry = (uint64_t)(ret >> 64);
+  return (uint64_t)ret;
+# 1202 "third_party/boringssl-with-bazel/src/ssl/../crypto/internal.h"
 }

Maybe it's somehow related to https://gcc.gnu.org/git/gitweb.cgi?p=gcc.git;h=a7aec76a74dd38524be325343158d3049b6ab3ac which landed in gcc-14?

BoringSSL issue is fixed in https://boringssl.googlesource.com/boringssl/+/c70190368c7040c37c1d655f0690bcde2b109a0d - as soon as it gets propagated to the Github mirror, gRPC team should just update it on their side.

I confirm, this single commit is enough to fix the build with GCC 14

#36716 updated boring ssl with the mentioned fix.
Next release (1.65.x) should include it.

1.65.0-pre1 should include it already, 1.65.0 should come in a couple of weeks.


Unable to re-add bucket to a scoped token after re-creating it

Steps to reproduce:
List the minimal actions needed to reproduce the behavior.

Create a bucket
Create a token to read/write scoped to that bucket
Verify that the token works
Delete and create a new bucket
The previously created token does not work ; there is no way to re-add new bucket to the existing token

Expected behavior:
Be able to edit buckets that token can read from / write to.
Actual behavior:
No option to perform it in the UI.
Environment info:
Version 2.0.0 (732c0cc)
Config:
N/A
Logs:
N/A
Performance:
N/A

Tokens are immutable after they are created in the system today.  Could you describe the use case for this?

I forgot to mention one thing in step 4 - what I was doing was ot delete and re-create the bucket. I was pushing incorrect data to the bucket, so I wanted to delete all data and start over.
I did not find a way to delete all measurements for a bucket, so I only thought of deleting and creating the bucket.
I've just updated the description for this issue.
I think this use case could be fulfilled by either being able to delete some or all data from a bucket - either using API or flux and a function to delete records - or having an option to re-create a bucket and still be able to use the token.
Otherwise every time I need to resend data, I'd need to re-create a scoped token.

Thanks for the update! keep an eye on this PR for deleting within a bucket: https://github.com/influxdata/influxdb/pull/15255

Hi Russ,
I'm sorry for responding to this so late.
I agree that what I really wanted was to delete old/erroneous data, so if there would be a way to delete all data or data wihtin specific ranges, that would be more suitable than re-creating tokens.
I think it would be best to be able to delete data from Flux - such as |> delete(). Perhaps it would only work if I did not process the data in any way (i.e. did not aggregate it in any way). I am not sure how well that fits the language concepts, but given there is documentation about to() function, so it seems Flux should be able to apply changes in general. Perhaps deletion could also be one of the changes it can apply.
I believe that unless you want to move forward with this, you can close the issue.

@wojciechka thanks for the update! i believe the /delete endpoint will do what you need. You can manually call it now, but look for a way to do it in the UI coming in a few weeks.


Fix SyntaxHighlighter delimiter state

The syntax highlighter needs to reset the previous delimiter_state after exiting a nested state.
Fixes https://forum.crystal-lang.org/t/question-about-the-crystal-syntax-highlighter/7283

I understand you're restoring the current delimiter state after handling the new delimiter state? Maybe add a note or something to explain why we eventually override the token's delimiter state? I guess the reason may be "we don't know :shrug:".

Maybe add a note or something to explain why we eventually override the token's delimiter state?
I figured this should be rather obvious from the use of the previous_delimiter_state variable. But I'll add some comment about it.

I guess the reason may be "we don't know 🤷".

What makes you think that? It is my understanding that this is expected and intendended usage of the lexer API.
In some cases the delimiter_state needs to be stored and recalled by the parser (or equivalent driver) because the lexer itself does not keep track of nested states. This same mechanism is used in Parser and Formatter. We had just missed it in SyntaxHighlighter. This patch fixes that.


around_reflex callback does not work

class MyComponent < ViewComponentReflex::Component
  around_reflex :do_awesome_stuff

  private

  def do_awesome_stuff
    my_awesome_stuff { yield }
  end
end

Expected:
Calls do_awesome_stuff with a block that when yielded, executes any reflex.
Observed:

Crashes on the method being private while called from outside. This is non-rails compliant, I think the reflex's method_missing should use send to call it.
When I make it public, it is called, but without a block, which also seems to be "discarded" inside the reflex's method_missing method.

This has been resolved 3.1.4

This has been resolved 3.1.4


chore: migrate to owl bot

generated via script

This appears to break many tests, and I'm unfamiliar with owl bot.  Is this part of a series of migration changes, or is there more to be done in the context of this change?


Internationalization/localization support

I couldn't find any examples or code related to internationalization. Ideally I'd like to be able to change the language of the application at run-time and have the language change reflected immediately, in the same way that Vizia does.
Do you have plans to support internationalization/localization?

Yes I plan on adding Fluent integration and making it as easy as possible to use, and switching languages should be reactive just like dark/light mode switching is.

This has been addressed by @hydra  and myself in #226.


Services status shown as "unknown"

on macOS 11.3.1 (20E241), my services are listed as unknown even though the appear to be working... fully updated Homebrew, uninstalled and reinstalled all the services, same result.
Name      Status  User    Plist
dnsmasq   unknown root    /Library/LaunchDaemons/homebrew.mxcl.dnsmasq.plist
<EMAIL_ADDRESS>started tmartin<EMAIL_ADDRESS>nginx     unknown root    /Library/LaunchDaemons/homebrew.mxcl.nginx.plist
<EMAIL_ADDRESS> unknown root<EMAIL_ADDRESS>

launchctl list | grep dnsmasq shows no results, but ps aux shows dnsmasq running just fine. And it's functioning.

launchctl list | grep dnsmasq shows no results

This is why it is unknown. If it's not causing you problems: don't worry about it. If you'd like to debug/fix this: you'll need to investigate why the plist isn't loading and perhaps use brew edit dnsmasq to fix/improve it (and submit a pull request). This is not a brew services issue, sorry.


Is it possible to use react-native-paper-select without checkboxes?

I want to disable checkbox and select one of the options just like in react-native select but with material-design styling from react-native-paper. Is it possible?

We don't support that functionality yet, but you can use this package React Native Paper Dropdown.


Document entire setup process on new AWS account

Should the above map to parts of the README.MD right now? trying to tell if this is done or not since it's been lingering as in-progress for awhile, and I see that there is some start of documentation.


OSCM OIDC mode issue with reverse proxy

Version Info
v18.1
Describe the bug
OSCM 18.1 in OIDC mode is not usable with reverse proxy. This is due to hardcoded application paths and ports, like here
These shall be removed and tested with reverse proxy.
How to Reproduce
Steps to reproduce the behavior:

Setup OSCM in OIDC mode with a default tenant as described in the Operator's Guide
Setup a webserver and configure oscm-portal location on 443 with a reverse proxy rule (ask me how to)
Try to access the oscm-portal

Observed behaviour
The oscm-portal is not accessible with custom URL (because hardcoded ports and application contexts as shown above)
Expected behaviour
The oscm-portal is accessible with custom URL.
Additional context
Note: OSCM v18.1 in internal mode is working well with reverse proxy using custom URLs on default HTTPS port 443.

fix https://github.com/servicecatalog/oscm/pull/710
https://github.com/servicecatalog/oscm-commons/pull/43


chore(deps): update golang docker tag to v1.22.5

This PR contains the following updates:

Package
Type
Update
Change

golang
stage
patch
1.22.4 -> 1.22.5

Configuration
📅 Schedule: Branch creation - "after 10pm every weekday,before 5am every weekday,every weekend" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
Because you closed this PR without merging, Renovate will ignore this update (1.22.5). You will get a PR once a newer version is released. To ignore this dependency forever, add it to the ignoreDeps array of your Renovate config.
If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.


A benchmark between lua-resty--ffi plugin and MOSN go plugin on Envoy

Hi, have you ever done a  benchmark test between lua-resty--ffi go plugin and MOSN go plugin on Envoy, and I don't know which extension method to choose.

@Hectorsong Yes, maybe my post is helpful for you:
http://luajit.io/posts/envoy-async-http-filter-lua-resty-ffi-vs-golang/
lua-resty-ffi has almost the same performance as go filters. In that article, I only provided the simplest demonstration. If you have further questions about the details, you are welcome to share them here, thank you!

Appreciate it.


[g-image] Missing classes when JavaScript is disabled

Description
When a class is defined for an image using <g-image>, they are not affected when JavaScript is disabled.
Steps to reproduce
<g-image src="…" class="foo" />

Expected result

  ""

Actual result

  ""

Closed in #203


Support functions in input context

This is still failing with both of the following:
import { ruleFactory } from '@elite-libs/rules-machine';

const things = [
  {
    name: "ABC"
  },
  {
    name: "DEF"
  },
  {
    name: "123"
  }
];

const thingRules = ([
  {
    if: 'isConditional(thing.name) == false', then: 'keep = false'
  },
  {
    return: "keep"
  }
]);

const filterWithRules = ruleFactory(thingRules)

function isFiltered(thing: Record<string, string>): boolean {
  const result = filterWithRules({
    thing,
    isConditional: (value:string) => value.endsWith('C'),
    keep: true
  });
  return result;
}

console.dir(things.filter(isFiltered));

import { ruleFactory } from '@elite-libs/rules-machine';

const things = [
  {
    name: "ABC"
  },
  {
    name: "DEF"
  },
  {
    name: "123"
  }
];

const thingRules = ([
  {
    if: 'isConditional(thing.name)', then: 'keep = false'
  },
  {
    return: "keep"
  }
]);

const filterWithRules = ruleFactory(thingRules)

function isFiltered(thing: Record<string, string>): boolean {
  const result = filterWithRules({
    thing,
    isConditional: (value:string) => value.endsWith('C'),
    keep: true
  });
  return result;
}

console.dir(things.filter(isFiltered));

Link to Repl<EMAIL_ADDRESS>

@chhatch @justsml

@chhatch @justsml Repl.it for the above example:
<EMAIL_ADDRESS>

Good spotting @dara-rockwell.
It looks like there is a function safe list thing going on.
Which reminds me, I need to put a lot more thought into this - the security 🙀 issues could be many.


Set genReqId with custom function on NestJs Fastify does not call the function

Hello I've been trying to replace our logger in NestJs to Pino using Nestjs-pino.
Tested it on the example project and it's working but our production project it's not working. We are using NestJs 9 with Fastify
Am I missing something?
LoggerModule.forRoot( {
			pinoHttp: {
				level: 'info',
				genReqId: (req, res) => {
					return 'temp';
				},

			},
			useExisting: null,
			exclude: ['/health', '/metrics']
		}),

This is the output
{
    "level": 30,
    "time":<PHONE_NUMBER>660,
    "pid": 89908,
    "hostname": "GuyL-Mac-8.local",
    "req": {
        "id": "req-1", <--------------------------
        "method": "GET",
        "url": "/resources/configurations/v1/public",
        "query": {},
        "headers": {
            "host": "localhost:3016",
            "connection": "keep-alive",
            "cache-control": "max-age=0",
            "sec-ch-ua": "\"Chromium\";v=\"110\", \"Not A(Brand\";v=\"24\", \"Google Chrome\";v=\"110\"",
            "sec-ch-ua-mobile": "?0",
            "sec-ch-ua-platform": "\"macOS\"",
            "upgrade-insecure-requests": "1",
            "user-agent": "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36",
            "accept": "text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.7",
            "sec-fetch-site": "none",
            "sec-fetch-mode": "navigate",
            "sec-fetch-user": "?1",
            "sec-fetch-dest": "document",
            "accept-encoding": "gzip, deflate, br",
            "accept-language": "en-US,en;q=0.9,he;q=0.8"
        },
        "remoteAddress": "<IP_ADDRESS>",
        "remotePort": 65317
    },
    "res": {
        "statusCode": 403,
        "headers": {
            "content-security-policy": "default-src 'self';base-uri 'self';block-all-mixed-content;font-src 'self' https: data:;form-action 'self';frame-ancestors 'self';img-src 'self' data:;object-src 'none';script-src 'self';script-src-attr 'none';style-src 'self' https: 'unsafe-inline';upgrade-insecure-requests",
            "cross-origin-embedder-policy": "require-corp",
            "cross-origin-opener-policy": "same-origin",
            "cross-origin-resource-policy": "same-origin",
            "x-dns-prefetch-control": "off",
            "expect-ct": "max-age=0",
            "x-frame-options": "SAMEORIGIN",
            "strict-transport-security": "max-age=15552000; includeSubDomains",
            "x-download-options": "noopen",
            "x-content-type-options": "nosniff",
            "origin-agent-cluster": "?1",
            "x-permitted-cross-domain-policies": "none",
            "referrer-policy": "no-referrer",
            "x-xss-protection": "0",
            "content-type": "application/json; charset=utf-8",
            "content-length": "13"
        }
    },
    "responseTime": 15,
    "msg": "request completed"
}

Thanks for reporting. We do not have the resources to support Nest.js users. Please refer to the Nest Discord channel (support) for such questions.

It may not related to pino-http but for those who encounter the same problem:
I found that the req.id is assigned by fastify before getting into the loggingMiddleware in which it reuses the value instead of calling your custom genReqId function:
https://github.com/fastify/fastify/blob/ac121b6581bbdb939155619e82cbdfc583918e64/lib/reqIdGenFactory.js#L21
https://github.com/pinojs/pino-http/blob/13415bc874f24b8e1e1ba8b2903ff45554537523/logger.js#L136
To address this, you can try passing your genReqId function to the fastify instead of pino-http:
  const app = await NestFactory.create<NestFastifyApplication>(
    AppModule,
    new FastifyAdapter({
      genReqId: getRequestId,
    }),
    {
      bufferLogs: true,
    },
  );

My project dependencies:

@nestjs/platform-fastify: 9.3.10
fastify: 4.14.1
nestjs-pino: 3.1.2
pino-http: 8.3.3


New version: Google.Chrome.Canary version 118.0.5956.0

Pull request has been created with WinGet Automation :rocket:
Microsoft Reviewers: Open in CodeFlow

This appears to be a re-release.


ImportError: cannot import name 'unescape_string' from 'markdown_it.common.normalize_url'

Describe the problem
Seeing mdformat fails locally with this trace:
File "....../python3.7/site-packages/mdformat/plugins.py", line 8, in <module>

    from mdformat.renderer import MDRenderer

  File "....../python3.7/site-packages/mdformat/renderer/__init__.py", line 7, in <module>

    from markdown_it.common.normalize_url import unescape_string

ImportError: cannot import name 'unescape_string' from 'markdown_it.common.normalize_url' (....../python3.7/site-packages/markdown_it/common/normalize_url.py)

Seems related to this recent markdown-it commit: https://github.com/executablebooks/markdown-it-py/commit/bb6cf6ecc545cf587357794cd340cc2207aa9688#diff-cdbaf938c7739a949222929740dd46fb5fb80c839b68c143f056d049592a2a52
Link to your repository or website
No response
Steps to reproduce
Run a command like pre-commit run mdformat --files some_file.md
Seems to fail regardless of content.
The version of Python you're using
3.7.7
Your operating system
No response
Versions of your packages
No response
Additional context
No response

Following up, looks like this resolved itself with a mdformat upgrade (as I see markdown-it was pinned to < 2). I guess could be something to look out for.

Thanks for the issue!
I believe this only happens when markdown-it-py v2 is installed and as you mention, we already have a pin in place to restrict to version <2 so I'll close this.
https://github.com/executablebooks/mdformat/pull/286 will bring compatibility with markdown-it-py v2.


⚠ Samsung has degraded performance

In a8bd82c, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 1971 ms

Resolved: Samsung performance has improved in fd96688.


[tpcc]can not retry select for update statement when running with more than 1 client.

I populated data with 5 warehouses
./tpcc_load -h  <IP_ADDRESS> -P 4000 -d tpcc1000 -u root -p "" -w 5

and then run the test
./tpcc_start -h  $TIDB_HOST -P 4000 -d tpcc1000 -u root -w 5 -c 2 -r 10 -l 40

However, from time to time in tidb.log I see
2017/08/10 03:29:30 txn.go:174: [info] [kv] Rollback txn<PHONE_NUMBER>46964737
2017/08/10 03:29:32 session.go:288: [warning] [3] retryable error: [try again later]: tikv restarts txn: Txn(Mvcc(WriteConflict { start_ts:<PHONE_NUMBER>84359937, conflict_ts:<PHONE_NUMBER>84359940, key: [116, 128, 0, 0, 0, 0, 0, 0, 255, 35, 95, 114, 128, 0, 0, 0, 0, 255, 0, 0, 17, 0, 0, 0, 0, 0, 250], primary: [116, 128, 0, 0, 0, 0, 0, 0, 35, 95, 114, 128, 0, 0, 0, 0, 0, 0, 17] })), txn: <nil>
2017/08/10 03:29:32 session.go:298: [warning] [3] finished txn:<nil>, [3] can not retry select for update statement
2017/08/10 03:29:32 session.go:644: [warning] [3] session error:
/home/jenkins/workspace/build_tidb_master/go/src/github.com/pingcap/tidb/session.go:372: [3] can not retry select for update statement
/home/jenkins/workspace/build_tidb_master/go/src/github.com/pingcap/tidb/session.go:299:
/home/jenkins/workspace/build_tidb_master/go/src/github.com/pingcap/tidb/tidb.go:172:
{
  "currDBName": "tpcc1000",
  "id": 3,
  "preparedStmtCount": 35,
  "status": 0,
  "strictMode": true,
  "user"<EMAIL_ADDRESS>}

Note that such error does not appear if i run tpcc_start with -c 1, and I didn't any warn or error in tikv.log or pd.log
tidb Git Commit Hash: 18f84dd3c673616c1edd546002ecbfb3dd324a20
tikv Git Commit Hash: 0aef87fb2cac52038f496c09dec3c8f52fdaef2d

Based on https://github.com/pingcap/tidb/issues/2712. This is intended behaviour, but tpcc-mysql didn't consider this use case


newsletters type package

What are you changing?

add a new package project that exports types describing the output of the newsletters API

Why?

provides shared definitions that ts projects using the API can import instead of defining their own types
originally suggested adding to @guardian/libs , but using a separate package should be more flexible

The new project will need adding to the paths in the base tsconfig file too

LGTM.


Move inject before onAttach for Fragment

Important - Best practices
An Activity injects Dagger in the onCreate method before calling super.
A Fragment injects Dagger in the onAttach method after calling super.

https://codelabs.developers.google.com/codelabs/android-dagger/#9

Constructor injection is preferred whenever possible because javac will ensure that no field is referenced before it has been set, which helps avoid NullPointerExceptions. When members injection is required (as discussed above), prefer to inject as early as possible. For this reason, DaggerActivity calls AndroidInjection.inject() immediately in onCreate(), before calling super.onCreate(), and DaggerFragment does the same in onAttach(), which also prevents inconsistencies if the Fragment is reattached.

https://dagger.dev/dev-guide/android.html
The content of the codelab and the content of the Dagger's document conflict.
There are several related PRs:
https://github.com/google/dagger/pull/603/files
https://github.com/google/dagger/pull/605/files
The order of super.onAttach () calls in the Dagger documentation has changed and I think this is the best practice.

According to @manuelvicnt 's answer, I would close this issue.


q2-fragment-insertion can't be upgraded to Python 3.10, due to pinned SEPP dependency on Python <=3.9

Here's the relevant pin. This will need to be addressed for q2-fragment-insertion to stay in the amplicon distribution when QIIME 2 transitions its Python version to 3.10 (planned for the 2024.10 release, which is currently scheduled for 2 October 2024).

Hi @gregcaporaso. It is pinned to 3.9 because SEPP, not the q2 plugin was incompatible with py3.10 in its last release.
It has meanwhile been fixed, but no new release was created. I asked for a new release some hours ago: https://github.com/smirarab/sepp/pull/136
For the meantime, I tried to simply source the bioconda recipe from HEAD instead of a release, however the bioconda CI service is "broken" at the moment: https://github.com/bioconda/bioconda-recipes/issues/41025

Thanks @sjanssen2. We have a little bit of time for a fix for this. We are going to start the transition to Python 3.10 this summer. If you hear of updates, it would be great if you could share here.

I'm going to close this as completed - per https://github.com/qiime2/distributions/pull/367 we have a working amplicon metapackage build for python 3.10!


Can't install

Expected Behavior
I wanted to install insomnia
Actual Behavior
I downloaded core and runned, got error and this file
SquirrelSetup.log
Reproduction Steps

Run "Insomnia.Core-10.2.0-beta.1.exe"
Error, got file
SquirrelSetup.log

Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
No response
Insomnia Version
10.2.0
What operating system are you using?
Windows
Operating System Version
Windows 10 version 22H2
Installation method
downloaded from insomnia.rest
Last Known Working Insomnia version
None

Similar to https://github.com/Kong/insomnia/issues/7743


Changing how seeds are selected

The absolute value of the seed doesn't mean much -- the important is that they vary. Therefore, to maintain reproducibility, I suggest selecting seeds sequentially (0, 1, 2, ...), instead of randomly sampling them.

I see a small issue about this. If you train the same models with the same data set you will get the same exact models, since their seed are predetermined.
This can be helpful when just evaluating the models, but if you use this library to also train your models, if you happen to get unlucky and randomly get a badly performing model, you would not be able to repeat the training to improve it.
One possible solution i see is, either give the user the option to randomize the training, or obtain the trained models with different seeds, although this would imply training one more model than before.
To be honest, this is not a big issue for me, if you want to keep the seeds predetermined that would be fine.

I agree that by defining the seeds sequentially, the user can get unlucky with some of them, which they would not be able to change (since they are the same for every training session). However, this bad luck could be reduced by simply increasing the number of evaluations, which I think is sufficient. Nevertheless, I agree with you that giving the option to the user would be ideal. I'll make the changes required.

Done. Let me know what you think :)


BUG: fix ownership for byte/string literals

Please complete the following checklist:

[x] I have read the guidelines in CONTRIBUTING.md.

[ ] I have documented all public-facing changes in CHANGELOG.md.

[ ] This pull request includes code, documentation, or other content derived from external source(s). If this is the case, ensure the external source's license is compatible with scikit-bio's license. Include the license in the licenses directory and add a comment in the code giving proper attribution. Ensure any other requirements set forth by the license and/or author are satisfied. It is your responsibility to disclose code, documentation, or other content derived from external source(s). If you have questions about whether something can be included in the project or how to give proper attribution, include those questions in your pull request and a reviewer will assist you.

[X] This pull request does not include code, documentation, or other content derived from external source(s).

Note: REVIEWING.md may also be helpful to see some of the things code reviewers will be verifying when reviewing your pull request.

fixes #1648
Turns out the issue was embarrassingly simple: since numpy does not own the backing bytes of string/byte literals, neither do we. This change corrects that in the constructor.
There are very really no situations where skbio will assume byte ownership except for one branch that I don't remember the cause of. We may want to think about either removing these semantics entirely, or add an API to indicate the skbio may mutate the buffer (passing ownership of a numpy array you've created yourself onwards to skbio, assuming the ndarray flag is compatible).

:+1:

Wooo, just tested with a fresh numpy install and the tests pass 👍

Hi @ebolyen , is there any plan to release this bugfix soon?
It's been there for 10 months, but Pypi and conda packages are still a mess with numpy>=1.16.


Moving CSI to Out of Tree section; linking to out of tree plugin FAQ

/assign @saad-ali @Bradamant3

/assign

Tech review LGTM and Approve.
Will let @heckj review and approve for merge.

Addressed comments

/approve
/lgtm


cannot build shared library

CMake Error at library/CMakeLists.txt:151 (install):
install TARGETS given no LIBRARY DESTINATION for shared library target
"mbedtls"

Hi @Yuzeyang when you create an issue, it is important you mention full way of reproducing the issue.
Please verify you run the cmake command from the root directoy, as mentioned in the Readme file:
cmake -DUSE_SHARED_MBEDTLS_LIBRARY=On .

@RonEld  sorry, i forgot to set it as a question
`➜  libmbedtls git:(feature/port-mbedtls) ✗ cmake -DUSE_SHARED_MBEDTLS_LIBRARY=On .
-- The C compiler identification is AppleClang <IP_ADDRESS>20042
-- Check for working C compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc
-- Check for working C compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Found Perl: /usr/bin/perl (found version "5.18.2")
No such file: /include/mbedtls/config.h
CMake Error at library/CMakeLists.txt:151 (install):
install TARGETS given no LIBRARY DESTINATION for shared library target
"mbedtls".
-- Configuring incomplete, errors occurred!
See also "/Users/yzy/Documents/mercury-client-native/libmbedtls/CMakeFiles/CMakeOutput.log".`
I run the cmake command from the root directoy, still have the same error

Hi @Yuzeyang I cannot reproduce this issue, please check your environment.
I am concerned by the line:
No such file: /include/mbedtls/config.h

I believe that it means that CMAKE_SOURCE_DIR is not correct.
You should try in a new shell environment as well

thanks for remind me, i download the new code, and run the cmake command, and have no error
but it have not any .project file?

Hi @Yuzeyang .project is not part of the mbed TLS code. It is a file generated by your IDE (e.g. eclipse )

@RonEld thanks for your help!


Roll Plugins from b202b3db98dc to 80532e0bab8d (2 revisions)

https://github.com/flutter/plugins/compare/b202b3db98dc...80532e0bab8d
2022-12-31<EMAIL_ADDRESS>Roll Flutter from 120058fd3ded to 0196e6050b75 (3 revisions) (flutter/plugins#6901)
2022-12-30<EMAIL_ADDRESS>Roll Flutter from d2127ad344e8 to 120058fd3ded (15 revisions) (flutter/plugins#6896)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-plugins-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Support several protocol id per protocol

Problem
Currently, js-waku only uses one protocol id per protocol, mainly synchronization needs to happen between the release of nim-waku, deployment of the Status fleet wakunode's and release of js-waku.
It also means that js-waku is not able to support test and prod Status fleet at the sames time if different protocol ids are used
Solution
Add support for multiple protocol id. For now, focus on supporting the two following waku relay ids:

/vac/waku/relay/2.0.0 (test fleet)
/vac/waku/relay/2.0.0-beta2 (prod fleet)

Done with #240


Teleport On Entry not Teleporting Certain Ranks

It seems that Teleport on Entry doesn't teleport users with * permissions

Of course it does not as * includes WorldGuard bypass permission nodes to ignore any set flags on region.

And what nodes would I need to disable to fix that?

My guess would be worldguard.region.bypass.<world>

I tried that, but nothing


Suggestion: support function overloading in type annotation comments

I'm adding type annotations to a JS codebase, and one problematic case I've come across is that I cannot overload function signatures using comment-based annotations.
/** @type {(flag: true) => integer;} */
/** @type {(flag: false) => string;} */
function getResult(asInteger) {
   // do stuff here
}

The workaround for now would be splitting the function into two functions, or type-casting at each use of the function.  So far my choice will be to split the function.

Isn't this the same issue? https://github.com/Microsoft/TypeScript/issues/407

indeed. closing in favor of #407


Start shepher failed.

Master branch.
Shepher start failed with this message:
sh bin/run.sh start
bin/run.sh: line 14: $'\r': command not found
bin/run.sh: line 16: $'\r': command not found
: No such file or directoryn
bin/run.sh: line 22: $'\r': command not found
bin/run.sh: line 46: syntax error near unexpected token `$'in\r''
'in/run.sh: line 46: `    case $OPT in

run.sh 配置文件换成unix格式的


Add :ssl to extra_applications

Ensure :ssl is started before Commanded.Middleware.Auditing.Repo.

Hi :wave:
I opened this PR because commanded_audit_middleware seemed to be starting before ssl and would therefore crash when connecting to the DB.  ssl was already present in our own application's extra_applications and wasn't doing the trick.  Adding it to commanded_audit_middleware's extra_applications got us working.
Is this the appropriate fix?  Is there a better way?  I'd like to get this merged unless there's a better way.
Thanks!