fix the dataset field has duplicated records issue

Hello,
after this change tests will not be run during build (and on travis). I have also noticed just now that "check" task also does not depend on "playTest". This way no Travis CI does not check if someone change logic without tests.
If test is not ready maybe a better way would be to use "@Ignore" in that class. Another possibility is to use H2 in memory database with mode MySQL to make fakeApplication working. Of course if something depends on its content, one can provide some content in H2 database.
What do you think about it?
I understand that here you are talking about class https://github.com/linkedin/WhereHows/blob/master/backend-service/test/controllers/DatasetControllerTest.java, right?  I'm not convinced that not running tests during the build (this also applies to web module) is a good way.
Kind regards,
Rafal Kluszczynski

Right now the test is only skipped during Travis build, it is still available for users and developers by using command line.
And we are doing some research to set up the mySQL db on Travis CI now, then we can bring the "playTest" depends back.
Thanks!

I will be looking into the MySql setup for CI for Travis soon...


ETA on Vue 2.0 support

Is there a plan to support Vue 2.0?

Hey @ghost. We don't have any plan to support Vue 2.0 any time soon.
We have moved away from deepstream so we are not actively planning improvements to this repo.


Allow common image tag

I expect to keep all the images in lockstep, so specifying tag versions separately seems an anti-pattern.  This change allows them to be easily kept in sync by taking their common config value from one line.  It retains the ability to set them separately if needed.

One thing you could consider (I've seen done in other charts and have done myself in some) is falling back to use .Chart.AppVersion as the image tag if not overridden. Conceptually it makes sense to me.

One thing you could consider (I've seen done in other charts and have done myself in some) is falling back to use .Chart.AppVersion as the image tag if not overridden. Conceptually it makes sense to me.

Great idea!  I like it; let's use preexisting info wherever possible.  But, help me talk this through a little bit.  The major version is staying the same, and I'm bumping via Update Releases (https://docs.aquasec.com/docs/update-releases).  So, as a user should I be modifying Charts.yaml with that specific version (e.g. 5.0.20190), or would I be able to set .chart.appversion in values.yaml.  Because, as an end-user, I would only ever expect to update the values.yaml file (chart.yaml is metadata set by the packager, not information I'm picking).

Yes you're right; the user wouldn't change Chart.yaml - I think your change still makes sense to allow the user to change all tags at once. I think the benefit of falling back to .Chart.AppVersion is the chart doesn't need to specify a .Values.common.image.tag by default and instead uses the standard AppVersion.

not relevant based on the helm chart design

@niso120b - do you have more information on what is "not relevant"?

Seems like it is pretty common for useful PRs (and issues) to be closed here with little or no comment, as was in my https://github.com/aquasecurity/aqua-helm/pull/99#issuecomment-664842307 and in several examples listed in #92


in search city field need ajax

in search we have city field
i need it must show all  cities in that country selected as AJAX
as well as if city is not selected it must show all members in that country
how can i do that

What do you mean ?
You need to code it in PHP.
If you want we can tell you where to look for but we won't be writing code for you.
You can hire a developer.
You can contribute https://www.patreon.com/bePatron?u=3534366 and maybe you'll get a chance to get your feature added.
If you like the software, it would be appreciated that you give us a star on gitHub and make a nice review on Sourceforge.

Hey @samit78
I would suggest to find someone good at Toptal, Upwork or similar service or dev friend to make this happening.
I totally understand that it can be frustrating for you, however, you have to know that everything here I do is for free, during my free time and I really can't make any custom feature/suggestion requests from users happening. This is totally impossible for me.
All the best with your social dating website!! :tada:


custom_plugins configuration in kong_tests.conf appears to be ignored in helpers.lua

Summary
Setting the custom_plugins configuration in kong_tests.conf will result in the configuration being ignored when running tests using spec/helpers.lua.
I believe it is an oversight, apologies if it is working as intended or a result of improper use of the test helpers.
Steps To Reproduce

Write a plugin in a standalone repository, copying bin/busted, spec/helpers.lua and spec/kong_tests.conf from the kong repository as indicated here.
Try to execute tests using the bin/busted binary and the spec/kong_tests.conf file, specifying custom_plugins.
When running bin/busted, the custom plugin will not be taken into account, custom_plugins will always be equal to: "dummy,cache,rewriter". (https://github.com/Mashape/kong/blob/0.11.0/spec/helpers.lua#L837)

I looked at the template plugin and it passes custom_plugins in the start_kong call, but I still thought that overriding the kong_tests.conf file's configuration was unintended behaviour.
The workaround I found at the moment is to add a condition that checks for conf.custom_plugins before replacing env.custom_plugins with "dummy,cache,rewriter". See here.
I wasn't sure what the intent of that default value was so I didn't want to attempt a different fix without checking if it was actually a bug first.
Additional Details & Logs

Kong version: 0.11
Kong debug-level startup logs (test output):

Error → spec/01-plugin-name/001-test_spec.lua @ 9
plugin-name plugin setup
spec/01-plugin-name/001-test_spec.lua:25:
Error:
/usr/local/share/lua/5.1/kong/cmd/start.lua:62: /usr/local/share/lua/5.1/kong/cmd/start.lua:51: nginx: [error] init_by_lua error: /usr/local/share/lua/5.1/kong.lua:159: plugin-name plugin is in use but not enabled
stack traceback:
	[C]: in function 'assert'
	/usr/local/share/lua/5.1/kong.lua:159: in function 'init'
	init_by_lua:3: in main chunk

stack traceback:
	[C]: in function 'error'
	/usr/local/share/lua/5.1/kong/cmd/start.lua:62: in function 'cmd_exec'
	/usr/local/share/lua/5.1/kong/cmd/init.lua:88: in function </usr/local/share/lua/5.1/kong/cmd/init.lua:88>
	[C]: in function 'xpcall'
	/usr/local/share/lua/5.1/kong/cmd/init.lua:88: in function </usr/local/share/lua/5.1/kong/cmd/init.lua:45>
	/bin/kong:7: in function 'file_gen'
	init_worker_by_lua:40: in function <init_worker_by_lua:38>
	[C]: in function 'xpcall'
	init_worker_by_lua:47: in function <init_worker_by_lua:45>

Kong configuration (kong_tests.conf):

custom_plugins = plugin-name

Operating System: Arch Linux (4.13.3-1-ARCH) (kong/openresty/luarocks installed from source as instructed in the installation guide for kong 0.11.0)

Hi @lavoiedn,
Hmm yes indeed, I see what is going on here... Environment variables intently override configuration settings, including when such settings are lists like the custom_plugins setting. Hence, by hard-coding an environment variable in the spec/helpers.lua module when starting test Kong instances, we are ignoring any such value configured in the test configuration.
At this point, your only bet without maintaining your own patch is to override this environment variable yourself via the supported env argument of start_kong():
helpers.start_kong {
  custom_plugins = "dummy,cache,rewriter,plugin-name"
}

A less than ideal solution, but believe me, we are aware of the crucial lack of testing framework for Kong plugins development, and will be working on it in the upcoming months/year. Ideally, such a testing framework does not communicate such custom settings via hard-coded environment variables...
In the meantime, as this is the intended behavior and your question stands answered, I will be closing this issue, keeping track of this remark internally for when the time comes we address our plugins features and testing framework.
Thank you!

Slightly off topic but testing custom plugins out of the Kong source tree is complicated. On have to copy the spec and the bin folder in the custom plugin development folder. One solution might be to move all that code in a kong.testing lua package and make available as luarock maybe. Also testing plugins that need to call into other servers as part of handling the request, is not straightforward: my plugin calls an external server to do its job, I can make that external server as an API behind kong (which is convenient as I don't need to write a mock server from scratch) but for this to work I need to add my own custom location blocks inside the server block mock_upstream. Are there any plans to improve custom plugins testing.

@iyedbennour Yes, we are aware. We are too busy today to provide such a solution (in fact, all of the modules you mentioned in your comment are note officially supported for testing, aka, use them at your own risk), but this is coming up our list of priorities. When we come up with a testing framework, we intend on making it simple, yet powerful. Stay tuned! :)


*(engine): remove unused errors and dataset

What problem does this PR solve?

Issue Number: close https://github.com/pingcap/tiflow/issues/6849
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

/merge

/run-engine-integration-test

/run-engine-integration-test


Result::unwrap() on an Err value: StateAlreadyQueued'

Bevy version
0.8
Relevant system information
OS: Windows 11
What you did
I attempted to set change my gamestate like so:
// Change the state
if *state.current() != event.0 {
    state.set(event.0).unwrap();
}

What went wrong
Doing so crashes the game with the following error:
Result::unwrap()` on an `Err` value: StateAlreadyQueued'
Additional information
The current workaround i took was to use State::overwrite_set instead like so:
// Change the state
if *state.current() != event.0 {
    // Using [`State::overwrite_set`] instead of [`State::set`] to cirvumvent the 'Aleady Scheduled' bug.
    state.overwrite_set(event.0).unwrap();
}

Edit: To clarify, this system is only being called once per frame and the state isn't currently being changed anywhere else in the code.

This error means that you already changed the state in this frame. The overwrite_* method is made to ignore ongoing transitions.

@mockersf I'm getting this panic after updating to version 0.8, without any code changes related to states or system sets.
When I print the current app state at the app_state.set call site, the current one is not the one I'm scheduling. Logically I can't think of a way my code could possibly schedule the state twice 🤔
app state ResMut(State { transition: Some(Entering(A, B)), stack: [B], scheduled: None, end_next_loop: false })
Let's call the states A, B, and C by their logical order. I'm trying to transition to state C, and as you can see from above, C can't be found from the current state as scheduled or as the current state.

The error is saying there is already a transition ongoing, which you can see by transition not being None.
It doesn't have the best name, it's StateAlreadyQueued as in "there's already a new state queued", not "the state you're trying to set is already queued"

The error is saying there is already a transition ongoing, which you can see by transition not being None.
It doesn't have the best name, it's StateAlreadyQueued as in "there's already a new state queued", not "the state you're trying to set is already queued"

Ah, I see. Thanks for the clarification! Still, how can the SystemSet::on_enter system of state B run before the transition is completed?

I think it's a bug that it was accepted before. https://github.com/bevyengine/bevy/pull/4890 is probably the source of your panic.

I think it's a bug that it was accepted before. #4890 is probably the source of your panic.

Yeah! That must be it. I'll see if there's a better way to handle the three states, maybe by making state B redundant.

I am having a similar problem. Code was fine in 0.7 but now in 0.8 I get the same error.
You can see here the system that handles changing the states (turns in my case). It is only 45 lines:
https://github.com/thephet/BevyRoguelike/blob/main/src/systems/end_turn.rs

This error means that you already changed the state in this frame. The overwrite_* method is made to ignore ongoing transitions.
Without more context, this seems to work exactly as expected 👍

I am not changing the state anywhere else in the code and the system is only called once per frame. The code worked fine in 0.7 but broke after 0.8.

I'm seeing this error when trying to change the state again in a system running on_enter() on 0.8, which didn't happen on 0.7. When moving the system to on_update(), changing the state again does work on 0.8 (but it is less convenient for my usecase).

@NeoRaider by @mockersf 's description, it seems that changing a state in on_enter() is not even meant to work. I take it that the transition from the previous state is meant to be active when on_enter() is triggered. I'm not sure if it's very intuitive though.
In general terms, should the transition be finished before a FSM state is entered? In FSM's I've authored, the transition belongs to the "previous" state (the one where the transition begins), and the next state should not need to be aware of the transition (it has a separate set of transitions).
Pseudocode:
let stateA = { id: 'A', transitions: [ 'B' ]' }

let stateB = { id: 'B', transitions: [ 'C' ]' }

let stateC = { id: 'C', transitions: [ 'D', 'A' ]' }

Using https://github.com/bevyengine/bevy/blob/latest/examples/ecs/state.rs as the inspiration, I find this behavior confusing.
This is enough to reproduce the StateAlreadyQueued
impl Plugin for GameLoopPlugin {
    fn build(&self, app: &mut App) {
        app
            .add_state(GameState::StateA)
            .add_system_set(SystemSet::on_enter(GameState::StateA).with_system(do_state_a))
            .add_system_set(SystemSet::on_enter(GameState::StateB).with_system(do_state_b))
        ;
    }
}

#[derive(Debug, Clone, Eq, PartialEq, Hash)]
enum GameState {
    StateA,
    StateB,
}

fn do_state_a(
    mut state: ResMut<State<GameState>>,
) {
    state.set(StateB).unwrap();
}

fn do_state_b() {
    // no-op
}

Using push instead of set doesn't make any difference.
Using overwrite_set instead does work; but doesn't match the official example in the link above.
btw I am on bevy 0.9.1 on Linux

This error and the entire state stack was removed in #7267.


Large data - follow-up/phase 2 work

This PR contains follow-on work identified as non-critical fixes or wider changes from #204.

See PR #363 as a replacement for this PR


Triggers viewWillAppear()

I just added a few lines of code so from now on viewWillAppear() method will be called. Helpful if you need some preparations done in this method, Ivan 🤠

I @ivanglushko! Thanks a lot for this, but I'm afraid this repo does not accept PRs, as it's simply a collection of sample code from Swift by Sundell. The reason the appearance transition APIs are not included in the Basics article is because I consider it to be beyond the basics of using child view controllers - I'll write a more advanced piece that'll include that, and other related APIs, sometime in the future. Thanks anyway.

@JohnSundell oh I see now :) Glad to help in anyway possible! Thanks for all your work John!


Added a code_of_conduct file

The CODE_OF_CONDUCT.md file is missing in the root repo.  This file helps with how to go about contributing to the open source and also the dos and dont's.

The CODE_OF_CONDUCT.md file is missing in the root repo. This file helps with how to go about contributing to the open source and also the dos and dont's.


Use MSVC intrinsics for better performance

Try to fix https://github.com/cppformat/cppformat/issues/115
Not tested yet due to lacking of MSVC compiler

Merged, thanks!

Applied some changes in https://github.com/cppformat/cppformat/commit/27e0bf72c02e830d5346738e59ea9848c32c2de9 as I think the correct intrinsic to use is _BitScanReverse. Also the intrinsic was not called in release version due to assert.

You're correct thanks.
A question: Why do you disable count_digits(uint32_t n) conditionally when FMT_BUILTIN_CLZLL is not avaliable? I think count_digits(uint32_t n) can coexist with the fallback version of count_digits perfectly since count_digits(uint32_t n) is only an optimization in my opinion.
Note: _BitScanReverse64 is only avaliable on WIN64. Provide count_digits(uint32_t n) should provide much better performance when compiling 32bit version.
In addition, we may implement __builtin_ctzll using __builtin_ctz when __builtin_ctzll when __builtin_ctzll is not avaliable (for example), since 64bit operation is much more slower than 32bit operation on x86 and int (ie int32_t) is much more widely used than long long (ie int64_t).

Why do you disable count_digits(uint32_t n) conditionally when FMT_BUILTIN_CLZLL is not avaliable?

Good point, there is no reason to disable optimized count_digits(uint32_t n) when FMT_BUILTIN_CLZLL is not avaliable. I've fixed this in https://github.com/cppformat/cppformat/commit/cbc46d9e78f6935b83ca233f6f2c2fff533ab1ef.
As for implementing __builtin_clzll in terms of __builtin_clz on 32-bit platforms, this is reasonable although not critical, because, as you mentioned yourself, int is much more widely used than long long and int is already optimized on 32-bit platforms. Besides, 32-bit systems are becoming increasingly rare. But patches are welcome =).

well, what I want to say is that we only provide count_digits(uint64_t n) when __builtin_clz(ll) is not available, which means: if we call count_digits with an int on 32bit platform, we convert it into 64bit long long, do 64bit divisions

Right, but __builtin_clz or equivalent should be available on most platforms. Do you know any popular platform where __builtin_clz is not provided? Not that I'm completely opposed to adding a fallback count_digits(uint32_t) variant (perhaps, by making count_digits a template), just want to make sure that we don't add anything that is almost never used.

Ah I didn't know you're were using intrinsics already for GCC, RapidJson is also big on SSE usage to speed up their lib. MSVC coverage is very welcome. Cool :D

Here is a way to check which intrinsics are available for MSVC (in case you don't know already):
https://msdn.microsoft.com/en-us/library/hskdteyh.aspx

According to the Intel Instruction Set page, Bit Scan Reverse (BSR) is supported on i386+, should be supported in all modern CPUs.
Runtime CPUID check may not be used in this case since CPUID check itself takes time

I agree with @CarterLi that it's better to avoid runtime checks, but thanks for the suggestion @patlecat. I was thinking of adding SSE support, but it adds very little in terms of performance while making implementation much more complicated.

@vitaut  Then why did @miloyip rely on SSE for RapidJson?
You have a library here, so leave it to the developer on what nifty features to enable for his project, then you don't need CPUID, or offer a CPUID tool seperately to make the decision easier for your users.

@patlecat Maybe I'm missing something, but it looks like RapidJson is not using SSE for integer to string conversion: https://github.com/miloyip/rapidjson/blob/master/include/rapidjson/internal/itoa.h
Anyway, patches are welcome =)

Yes, @vitaut . From my implementations, SSE2 does not improve the performance much in integer-to-string conversion so I didn't put that in RapidJSON. However, RapidJSON got performance boost by using SSE2/4 for skipping whitespaces during parsing. It also uses bit scan intrinsics in Grisu implementation (float-to-string conversion) as well.
RapidJSON does not use dynamic dispatching according to CPUID. It simply uses static binding. Some discussions about this can be found here.

Thanks, @miloyip, that's what I thought. You've done a great job with RapidJSON BTW.


Unit Test: Generates archive if target repo doesn't exist

Adds a unit test to specifically test if archive generation is attempted if the target repo doesn't exist.
Related to #782 and PR #789

[x] Did you write/update appropriate tests
[x] Release notes updated (if appropriate)
[x] Appropriate logging output
[x] Issue linked
[x] Docs updated (or issue created)

There are already a few tests that ensure successful migration for GHES... I tried to make this as bare bones as possible to highlight the fact that the target repo is absent.


Unable to run container based on Dockerfile provided

When using the provided Dockerfile, I was able to pass in the .env environment variables in the docker run command, however, after solving that I'm getting an error stating that the storage/site.json file isn't being found.
docker run --env-file ./.env --name pbb -p 8080:8080 pbb
main.go:23: open storage/site.json: no such file or directory

This file is present in the root of the directory at ./storage/site.json. I've noticed the WORKDIR in the Dockerfile image is set to app. I tried moving the storage directory to the app directory, but that gave the same error.

Perhaps I need to specify a volume when running the container

Yeap, this was the issue. Was able to configure the image successfully in a docker-compose file:
  blog:
    container_name: pbb-personal-blog
    env_file:
      - ./.env
    image: pbb
    ports:
      - "8080:8080"
    volumes:
      - /srv/polarbearblog/storage:/app/storage


Evaluation for MegatronT5 Model

Hi I have noticed that there are existing support for some nemo models. But it does not seem that there is a support for MegatronT5 Model. Anyone has ideas how to evaluate this model?

Hi! We don't currently support this, though would welcome contributions adding it.
Perhaps @sergiopperez has input on how simple it would be to extend the current NeMo integration into Megatron-T5?

Hi any updates on how to evaluate T5? Could you please give some clue on how to extend to Megatron-T5?

If you are interested in evaluating a pretrained model such as NeMo Megatron T5 that has been released on Hugging Face, the Hugging Face version is compatible with this library. However we still lack support for models native to the Megatron library.

Closing as NeMo Megatron-T5 support won't be prioritized, though we'd welcome a contribution for it.


Add test directory and target

As a first step I would like to add test sample with unittest. pytest is going to be supported in another Issue.


Document formatting conventions for this repo

Formatting styles vary widely in the C++ community.  Document the formatting style used by this repo.

Namespaces don't indent
Whitespace as per editor config #92
Identifier conventions
Separate test cases with single blank line
Separate test case phases (setup, exercise, verify, teardown) with a single blank line
Test only one behavior per test case

Others?

What do you think about using the Google's C++ style guide?
While it's not necessarily the golden rules, it's comprehensive enough & contain good practices.
Plus, there is a linter that can be used to check the code.

I opened #274 to enforce this with a checker which will cover most of this. It won't cover everything, but it will get us most of the way there.
I'm going to close this in favor of the other one to be clear about what we need.


fix(pinia-orm): $refresh() is not working

🔗 Linked issue
closes #1124

❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (943e554) 99.64% compared to head (7a4562b) 99.64%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1125   +/-   ##
=======================================
  Coverage   99.64%   99.64%
=======================================
  Files          88       88
  Lines        5875     5879    +4
  Branches      536      537    +1
=======================================
+ Hits         5854     5858    +4
  Misses         19       19
  Partials        2        2

Impacted Files
Coverage Δ

packages/pinia-orm/src/model/Model.ts
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[PM-2653] Add automation to release

Add developers to build (--release flag requires it)
add release to maven when merging to master

I'm just looking at the mill docs and it says --release is true by default :thinking:

I'm just looking at the mill docs and it says --release is true by default :thinking:

So my plan about flow and removing snapshot was something about that:

Create release branch , remove snapshot here, make pr to master, remove branch after merge
Create merge back pr from master do develop, attach new snapshot here, merge to develop

So my plan about flow and removing snapshot was something about that:

Create release branch , remove snapshot here, make pr to master, remove branch after merge
Create merge back pr from master do develop, attach new snapshot here, merge to develop

Look mill has a Version which automatically attaches and removes the -SNAPSHOT depending on whether it's a release, but I'm not sure if it's internal.

Look mill has a Version which automatically attaches and removes the -SNAPSHOT depending on whether it's a release, but I'm not sure if it's internal.

Hmm you are talking about something like that - http://www.lihaoyi.com/mill/page/contrib-modules.html#version-file ?
Tbh I would do it in separate pr, as probably geting to proper flow with it would require a little experimentation.

Hmm you are talking about something like that - http://www.lihaoyi.com/mill/page/contrib-modules.html#version-file ?
Tbh I would do it in separate pr, as probably geting to proper flow with it would require a little experimentation.

Yes, you're right it's part of version-file. I see it even has git commands to tag and push, however there could be a very minimalistic approach:
Change build.sc to take the version from the file:
def publishVersion = versionFile.currentVersion().toString

Add the current version on develop and commit:
$ 0.6.0-SNAPSHOT > versionFile/version
$ git add .
$ git commit

Whenever there's a new feature or patch we bump the appropriate version on the release branch or feature PR but leave the -SNAPSHOT in tact.
$ mill versionFile.setNextVersion --bump minor
$ git add .
$ git commit

In the publish script if it's a --release then we remove -SNAPSHOT without committing it.
if [[ "$CIRCLE_BRANCH" == "master" ]]; then
  mill versionFile.setReleaseVersion
  mill mill.scalalib.PublishModule/publishAll \
     ...
fi

This way there's no reason to add and remove it manually.

Yes, you're right it's part of version-file. I see it even has git commands to tag and push, however there could be a very minimalistic approach:
Change build.sc to take the version from the file:
def publishVersion = versionFile.currentVersion().toString

Add the current version on develop and commit:
$ 0.6.0-SNAPSHOT > versionFile/version
$ git add .
$ git commit

Whenever there's a new feature or patch we bump the appropriate version on the release branch or feature PR but leave the -SNAPSHOT in tact.
$ mill versionFile.setNextVersion --bump minor
$ git add .
$ git commit

In the publish script if it's a --release then we remove -SNAPSHOT without committing it.
if [[ "$CIRCLE_BRANCH" == "master" ]]; then
  mill versionFile.setReleaseVersion
  mill mill.scalalib.PublishModule/publishAll \
     ...
fi

This way there's no reason to add and remove it manually.

In any case it would be good to add the workflow description to the README. Maybe halt publishing on master if it's -SNAPSHOT?

In any case it would be good to add the workflow description to the README. Maybe halt publishing on master if it's -SNAPSHOT?

I like this minimalistic approach! Lets go with this way then.
One change i would make it is to bump minor version when merging back from master to develop, that way the version on develop would be unreleased i.e
Now we are working on 0.6.0-SNAPSHOT so the flow to release would be:

create release branch release/0.6.0 targeting master
merge it to master, tag it. (now we have released version 0.6.0
create merge back pr from master to develop bumping minor version to 0.7.0-SNAPSHOT
wdyt ?

I like this minimalistic approach! Lets go with this way then.
One change i would make it is to bump minor version when merging back from master to develop, that way the version on develop would be unreleased i.e
Now we are working on 0.6.0-SNAPSHOT so the flow to release would be:

create release branch release/0.6.0 targeting master
merge it to master, tag it. (now we have released version 0.6.0
create merge back pr from master to develop bumping minor version to 0.7.0-SNAPSHOT
wdyt ?

create merge back pr from master to develop bumping minor version to 0.7.0-SNAPSHOT

You are right, the next change on develop should be a --bump minor, i.e. a new feature, because if we release 0.6.0 then any fixes to that should happen as hotfixes on the master with --bump patch, e.g. 0.6.1. That should be merged back into develop. We shouldn't push more updates to the same snapshot from develop.
Because we would not be committing the version during the release, technically there may not be any change on master to merge back into develop, but I guess a formal PR is a good way to signal that this happened and prevent accidental overwrites.

create merge back pr from master to develop bumping minor version to 0.7.0-SNAPSHOT

You are right, the next change on develop should be a --bump minor, i.e. a new feature, because if we release 0.6.0 then any fixes to that should happen as hotfixes on the master with --bump patch, e.g. 0.6.1. That should be merged back into develop. We shouldn't push more updates to the same snapshot from develop.
Because we would not be committing the version during the release, technically there may not be any change on master to merge back into develop, but I guess a formal PR is a good way to signal that this happened and prevent accidental overwrites.

I had another idea once that would completely automate the version updates: instead of manually bumping the version, a PR would be required to append a line to the CHANGELOG.md and indicate whether it's a major, minor or patch thing, together with the description. When it comes to release, the process would automatically detect the changes since the last one and increment the appropriate parts of the version string (whether once or multiple times I don't know) and commit the changes back.
This way any further change to develop would be an automatic increase in something. And merging the changelog after a hotfix from master to develop would let the developer arrange the order, so in the end if there's a new feature 0.7.0-SNAPSHOT on develop and a hotfix 0.6.1 on master then the hotfix could be merged into the changelog before the feature, which results in 0.7.0-SNAPSHOT being unchanged.

I had another idea once that would completely automate the version updates: instead of manually bumping the version, a PR would be required to append a line to the CHANGELOG.md and indicate whether it's a major, minor or patch thing, together with the description. When it comes to release, the process would automatically detect the changes since the last one and increment the appropriate parts of the version string (whether once or multiple times I don't know) and commit the changes back.
This way any further change to develop would be an automatic increase in something. And merging the changelog after a hotfix from master to develop would let the developer arrange the order, so in the end if there's a new feature 0.7.0-SNAPSHOT on develop and a hotfix 0.6.1 on master then the hotfix could be merged into the changelog before the feature, which results in 0.7.0-SNAPSHOT being unchanged.

One thing I'm not certain about in your workflow is hotfixes:

Release 0.6.0
Merge master into develop with a bump. Now develop is 0.7.0-SNAPSHOT and master is 0.6.0-SNAPSHOT.
Apply a hotfix on master, bumping it to 0.6.1-SNAPSHOT, released as 0.6.1.
Merge master into develop: do you bump to 0.6.2-SNAPSHOT? Or 0.7.0-SNAPSHOT again? Or 0.7.1-SNAPSHOT since it now has a fix it didn't have yesterday? Definitely not 0.8.0-SNAPSHOT.

One thing I'm not certain about in your workflow is hotfixes:

Release 0.6.0
Merge master into develop with a bump. Now develop is 0.7.0-SNAPSHOT and master is 0.6.0-SNAPSHOT.
Apply a hotfix on master, bumping it to 0.6.1-SNAPSHOT, released as 0.6.1.
Merge master into develop: do you bump to 0.6.2-SNAPSHOT? Or 0.7.0-SNAPSHOT again? Or 0.7.1-SNAPSHOT since it now has a fix it didn't have yesterday? Definitely not 0.8.0-SNAPSHOT.

Yep hot fixes are quite tricky. My thinking was to, create hotfix branch and:

either target both master and develop ( but do not change version in develop)
or target master and then merge it back to develop without changing develop versions (in your example leave 0.7.0-SNAPSHOT)

My thinking is that, by having this automation, we can release more often and do not depend on snapshot versions as much. So we will not be required to bump versions with each pr, but have normal release cycle, and as downstream projects will not depend on snapshots as much we will not break ale the nix checks.

Yep hot fixes are quite tricky. My thinking was to, create hotfix branch and:

either target both master and develop ( but do not change version in develop)
or target master and then merge it back to develop without changing develop versions (in your example leave 0.7.0-SNAPSHOT)

My thinking is that, by having this automation, we can release more often and do not depend on snapshot versions as much. So we will not be required to bump versions with each pr, but have normal release cycle, and as downstream projects will not depend on snapshots as much we will not break ale the nix checks.


Change the layout of the RegisterActivity in order to match with the rest of the application.

Currently the RegisterActivity has different colors and style as compared to the rest of the application. You have to make it so that the RegisterActivity looks similar to the Login and Forgot Password Activities. The background color should be white, the EditText's should look similar to the one's in the LoginActivity, use icons for FirstName, LastName and ConfirmPassword  as well at the side. Make sure the I agree to the Terms and Conditions is centered properly. Change the color of the button accordingly as well and make sure it has rounded edges. . Also, shift the back button at the top left of the screen to a bit more left and implement it's functioning. Clicking on it should take you back to the LoginActivity.
THE ID'S OF EVERY ELEMENT SHOULD REMAIN THE EXACT SAME. DO NOT CHANGE THEM.
Attach a screenshot of the new RegisterActivity layout when submitting a PR.

Claim


[skip ci] Update remote data docs to be more user focused

Previously this started with more developer focused information.
I hope that this newer layout puts user interests more in the forefront.

cc @martindurant


C++: Add PostUpdateNode for updates to structs with no chi instructions

In #3118 we added basic IR field flow by attaching PostUpdateNodes to chi instructions. However, updating a field in a struct doesn't always generate a chi instruction. In particular, writing to a struct with a single field doesn't generate a chi instruction. This means we don't catch the following dataflow:
struct A {
  int i;
};
void foo() {
  A a;
  a.i = source();
  A a2 = a;
  sink(a2.i);
}

But we catch it if A is replaced with a struct with more than 1 field like:
struct A {
  int i, j;
};

One problem with adding a PostUpdateNode to the assignment a.i = source() is that the dataflow library requires a predicate getPreUpdateNode that returns the node before the update to its state. For field updates generating a chi instruction we can pick the total operand of the chi (as this represents the memory allocated to the struct), but without a chi instruction it's definately to define a meaningful result of this predicate. So this PR simply implements this predicate as none().
I don't know if this breaks something in the dataflow library, but it does catch the above flow even when A contains a single field.
This PR also adds an additional rule to simpleInstructionLocalFlowStep in order to propagate dataflow to loads with inexact memory operands that arise from stores to single field structs. This rule could be avoided if the overlap was considered MustExactlyOverlap.

Here's the relevant CPP-differences: https://jenkins.internal.semmle.com/job/Changes/job/CPP-Differences/1051/
I'll investigate what's up with boost, drogon and json. There are no result changes, which I guess is unsurprising (I wouldn't expect many real-world cases of structs with a single field).

CPP-differences looks good now: https://jenkins.internal.semmle.com/job/Changes/job/CPP-Differences/1056/


video_player in ios show first picture of video

if we only initialize VideoPlayerController, there will be black screen while it show the first frame in android;
I hope it will act the same as android;

Share your code to help other to know what's happening.

same here only in iOS. any updates??

Seeing the same issue.

The issue for me was that it was behind an opacity.
So this is broken
Opacity(
            opacity: 0.2,
            child: Video..

If I try to seekTo After the video is initialised then the video (also behind an opacity that is set to 0) just seems to disappear. The second i touch the screen it is shown again..... This is only happening on iOS and not android, its really frustrating and I have spent ages trying to fix it

If I try to seekTo After the video is initialised then the video (also behind an opacity that is set to 0) just seems to disappear. The second i touch the screen it is shown again..... This is only happening on iOS and not android, its really frustrating and I have spent ages trying to fix it

Any updates on this?

anything? I need an opacity over my video.

Same problem. No fix for a whole year?

Hi @wanjm,
There's similar open issue #41156 describing this case, please follow-up in it for further updates as it has been prioritized already. So closing this issue as a duplicate of #41156.
If you think this is not the case feel free to write back with relevant details.


Export to xls works but when you do Save As want to make it a Web Page

When you export a file is downloads as an xls (not a problem) when you open it you get a warning (not a problem) but when you try to Save As to the latest version of Excel - xlsx it defaults to Web Page (.htm) (.html)
It's not a big problem but confuses my non-IT customers slightly.
Is there some "Microsoft Magic" going on that it knows it been downloaded from a website and therefore decides for you to default to Web Page?

The ASP.Net framework does not have a standard option to create Excel native (.xls or .xslx) binary files.
The programming trick is that you create a table in HTML format and save that as a file with a ".xls" extension.
Open the file that is created with Notepad or Notepad++ and you will see it for yourself.
Excel understands that is not a binary Excel file, but opens it correctly and displays it as a .xls file. Internally, it remains an HTML text file with an extension .xls.
When you now use Save As, it asks for the conversion to a real Excel binary file and AFAIK there is not an Excel setting that can change this default Excel (safety) behaviour.
A solution for the Reports module would be to use an external library to create binary Excel files directly, but we do not want to create dependability on external libraries.
Another option would be (for you/Excel) to create an Excel/VBA macro that does the Save As without asking

What about this one, EPT?  It seems to have a lot of usage and support out there.
https://www.nuget.org/packages/FastExcel/

@hismightiness That MIT license code looks promising. Small footprint, not too much options, straight forward  ode, recent code updates.
I experiment a bit more to see if that is a nice replacement for the .xls/html table workaround.
Tnx for the link!

My pleasure, sir!  😁


I need the entire original jonyjive

I REALLY love it, can i request a backup of the entire original jonyjive source code?
I enjoy in private and will be never publish it anywhere.
Thanks!
E-mail<EMAIL_ADDRESS>

Sorry, no.


example doc to show syntax formatting problems

Docs Build status updates of commit b2112d4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/syntax-format-example-issue.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 171399a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/syntax-format-example-issue.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

This was just an example doc to give a screen print in an Azure DevOps work item.  Closing POC.


Pictures are lost if sent anonymously

Steps to reproduce

Log out your Open Food Facts account.
Scan a new product and take a front picture
Hit Send.

Expected result
Product appears with photo on server
Result
No photo arrives on the server

how you logout from the android app ?

You switch to a custom version ;)
You wipe the app cache (for various reasons, space being one)
You change your password online

Or you don't login.

ok and the expected behavior is to no able to send product if the user is not connected ?

To be able to send them, even if the user is not connected. Which is not the case.

Thanks @herau
Can we release this as soon as possible ? This results in lost contributions.


about.html includes date

After post build completes successfuly, about.html page also includes a date a la normal blog post, when it should be treated as an immutable page

last commit solved problem


Massively increase tests

You'll probably want to copy in the metadata and data to the tests directory. Or at least some of it.

Tests seem to be increasing.


directline takes too long to get response.

Please refer to issue #2089.
The issue persist but seems better, not sure if the backend of directline is optimized by the team. I add some log to trace the time cost in each phase and found that

The directline restful api "Sending an Activity to the bot" takes <1.8s.
Luis takes around 5s.
A polling GET to receiving Activities from the bot takes around 3s.
So, all together is nearly 10 secs. I use in the simplest way which don't contains any context or state in my bot. Just one question with one reply.
so, is there any way to improve it. thanks in advance.

Hi @zhou-xinghai, have you enabled "High-speed storage" on your Direct Line configuration page?

Just enable this option. Seems don't shorter the time. I will observe it in a longer time.

Can you send a conversation ID or your bot's ID?

hi ,my bot's Microsoft App ID
9f30580a-3c00-4f71-a0b4-c338d248ae62
Another issue is directline send msg operation return " id: 'retry'". As long it happens, all sending msg on this conversation will return this value and my botframework won't  get msg . So can you tell me why return this error.
thanks.

Good news is that your bot is responding reasonably quickly, around 500ms.
It looks like you've configured Direct Line for high-speed storage, but the conversation ID you sent was created before this setting was applied. Conversations that are already in progress will stay on the stack where they were created. Can you try again with a new conversation?
I've added more detailed logging on our side and will have better timing information when you reproduce the problem.

Thanks for your help. It seems faster than before. Can you please take a look at this one conversationId: BhZ3UhDKKJR34kssBdBFjS.

Hi @zhou-xinghai, some things I observed about this conversation:

The longest calls were to POST a message from the client to the bot. These typically took 1-2 seconds.
These POST calls had around 50ms of internal latency. Calling your bot took between 400ms and 500ms.
As of 50ms into the call, the message is available for clients to receive.
As of ~450-550ms into the call, message is delivered to the bot.
The remainder of the call (500-1000ms) is spent making sure the message is reliably delivered to any clients that connected recently. This part of the call does not count against the total delivery time, but you do see it in the API.

I'm planning to refactor the code so those extra 500-1000ms are performed on a background thread, but this is not high-priority work at the moment. Let me know if you have any questions.

Hi dandriscoll,
Thanks for your message. It's OK If you can keep the post message time in less 2 secs and response time to 500ms （the faster the better）.  But that time is surprisingly unstable today.  Below is the time cost I tried. Can you please take a look at the log why it cost me even 25secs to post a activity msg? And sometimes takes around 10secs to response rather than 500ms.
5mr0z5S8PzGKo2zzD2dRvs|0000008  25s sendActivity
5mr0z5S8PzGKo2zzD2dRvs|0000012  3s sendActivity;  10s response
5mr0z5S8PzGKo2zzD2dRvs|0000016  3s sendActivity  12s response
Besides, if I post an activity msg after a while like 10mins, then the post time will take much longer than post msg consistently. I think the stable post and response time is mattered at this stage.

I think these are network issues.
5mr0z5S8PzGKo2zzD2dRvs|0000008 was 25s from your client.
My server saw 13s for this call.
11s of those 13s was spent calling your bot.
Is it possible you're experiencing connection issues?

I am not sure, but I use AWS Server and the network should be more realiable.
So from your log, the 13s was cost on calling my bot? Is it possible that different consersation id take different time? For later, when I start a new conversation, the long latency returns normal.

As to the problem of " After a while like 10mins the post time will take much longer than post msg consistently.". Is it possible that the inactive conversation will turn into sort of "sleep state" while when it reuse again it will take some more time (like allocate the thread or some resources) to activated the consersation session.

@zhou-xinghai, yes, it's possible for your hosting platform to put your application into a sleep state. I don't have any details about how this works in AWS, sorry.


chore(deps): upgrade jackson

Checklist

[x] the contributor license agreement is signed
[x] commit message follows commit guidelines
[x] tests are included
[ ] screenshots are included showing significant UI changes
[ ] documentation is changed or added

Description of change

This upgrades Jackson to the latest version (2.13.1). Previously autotests were still stuck on 1.9.x so it also required upgrading some of the code to use the new API.
Also, to address the assembly deduplication issues, I consolidated the handling of the module-info files to the root build.sbt and tidied up the sub-project build.sbt files.
Lastly, I noticed that we were running the tests (in CI) for a whole package of tests however we were still declaring the individual classes in the TestNG YAML file, so I tidied that up to.

Starting this off as 'draft' as I worked on it a couple of weeks back and all was fine, and then have rebased it back onto develop and am getting a couple of tests failures locally (out by one). So I'm wondering if some of the changes have bumped things - especially the change around PagedSearchResults bug.

Will re-run CodeBuild as it only failed on a single io.github.openequella.rest.Search2ApiTest:
FAILED: collectionTest
--
java.lang.AssertionError: expected [2] but found [3]

That's different to what I saw locally. :thinking:

Same single error again. Odd. Will look at it closer.

Interesting, it could be the order of tests as you mentioned @PenghaiZhang .
Locally I just did a green run of the full io.github.openequella and then attempted to run the com.tle.webtests.test.webservices.rest package which failed in a couple of places. Following that I ran io.github.openequella and had a couple of fails too.
Maybe I need to revert the changes to testng-codebuild.yaml file. :disappointed:

Alright, with that reverted and the changes from develop (to improve the GHA build) I'm aiming for a green CodeBuild and green Old UI from GHA and then I'll take this out of draft.

Alright, ready for review now @PenghaiZhang


New CiviForm landing page

We are pleased to launch the new CiviForm landing page, based on the State of California Design System. Thank you @sophiatareen and others who contributed design and copy to the new site!

Fixes #1.


Custom IV length issues in authenticated encryption

Hi,
I think I found few issues related to IVs of custom length in authenticated encryption:
Order of GCM context initialization (legacy API)
When using legacy API to set the custom GCM IV length the proper order of context initialization is important. However, if a wrong order is used (as in the snippet below), the encryption and decryption succeeds, silently truncating the IV to the default length. I think that the encryption/decryption should fail in such case not to mislead the caller.
unsigned char key[32];
unsigned char iv[16];
EVP_CIPHER_CTX *ctx = EVP_CIPHER_CTX_new();
EVP_CIPHER *type = EVP_CIPHER_fetch(NULL, "aes-256-gcm", NULL);
EVP_CipherInit_ex(ctx, type, NULL, NULL, NULL, 1);
EVP_CipherInit_ex(ctx, NULL, NULL, key, iv, -1);
EVP_CIPHER_CTX_ctrl(ctx, EVP_CTRL_GCM_SET_IVLEN, sizeof(iv), NULL);

I prepared a test for it and a possible fix that forbids increasing the IV length after it has been set in the following PR: https://github.com/openssl/openssl/pull/22507
Perhaps the above logic should be even more strict.
Custom GCM IV length (params API)
I also checked how it works with the current "params API". I took the code from /demos/cipher/aesgcm.c as an example. I was unable to properly decrypt it.
I prepared a test that encrypts data using the legacy API with 16B IV and decrypts it using params API and a possible fix that changes the internal order of context initialization in the following PR https://github.com/openssl/openssl/pull/22507
Custom CCM IV length (params API)
I know that CCM also allows IV length to be customized so I decided to check it as well using /demos/cipher/aesccm.c as an example. In case of CCM I wasn't even able to properly initialize the context.
The test and two fixes similar to GCM are included in https://github.com/openssl/openssl/pull/22507
In all above cases I used the latest openssl from master branch. The tests that I provided fail without the fixes and pass after applying them. I tested it on Linux OS using:
./Configure --strict-warnings
make test

I suppose that beside fixing it, some changes in the /demos/ and documentation may be necessary. Perhaps some other encryption modes/ciphers that I'm unaware of are affected as well.

This should be fixed now. If there is anything remaining please open a new PR.


Barcodes?

Great initiative setting this up.
Does this code support the vision elements like Barcode? I looked into the code and I see only the text function. Can it be easily extended with the other functions like faces, barcodes etc?
https://firebase.google.com/docs/ml-kit/read-barcodes
If not, what would be the best way to bridge to a native code to call Firebase ML Kit?

At the moment the library does not support bar codes, if you wanted to add this feature would have to implement in the native code with a new function to request the qr code. If you want to add this new feature, we would be creating a pull request.


[DNM]net: NET filter

Simple implementation of the network filter based on the hooks.

Incorrect PR, it will be replaced by new one.


add a subdirectory option

add a subdirectory option for visual studio. it will generate a configuration directory when build executable

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jianguang.li seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[bug] URL parssing error in the web server

If you run this simple server example :
import asyncio
from aiohttp import web

@asyncio.coroutine
def handle(request):
    return webResponse(body=request.path.encode('utf8'))

@asyncio.coroutine
def init(loop):
    app = web.Application(loop=loop)
    app.router.add_route('GET', '/', handle)
    srv = yield from loop.create_server(app.make_handler(), '<IP_ADDRESS>', 5555)
    return srv

loop = asyncio.get_event_loop()
loop.run_until_complete(init(loop))

try:
    loop.run_forever()
except KeyboardInterrupt:
    pass

The following requests will get a 200 OK with the considered path as content :
$ curl -X GET http://<IP_ADDRESS>:5555///
/
$ curl -X GET http://<IP_ADDRESS>:555//foo/
/

As you can see, the path of the URL is not well parsed.
This bug resides in the _splitted_path non-data descriptor which uses urlsplit on a path rather than a full URL. The consequence is a second segment interpreted as a network location if the first one is empty.
I've not quite investigated a fix at the moment, but  _splitted_path only being used by raw_path and query_string seems to be a plea to use a fake scheme and netloc for urlsplit to behave the expected way.

I'll take a look


Add support for privacy config v4

Task/Issue URL: https://app.asana.com/0/1202552961248957/1205690187399816/f
Description
Add support for remote config v4
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #3664   👈

This comment was auto-generated by Graphite.


fix(data): remove i18n on meetup links

Actual behavior
Some Meetup links are localized to Spanish with an /es-ES url segment without direct relation to the native language of the country.
Expected behavior
We should get the same behavior across all Meetup links.
Changes
/es-ES url segment removed from Meetup links.

Thanks for your pull-request.


xFormers can't load C++/CUDA extensions

Hi, I have an issue with xFormers.
I built xFormers using cell in the tools section.
Created wheel for xformers: filename=xformers-0.0.18+da27862.d20230331-cp39-cp39-linux_x86_64.whl size=138469800 sha256=38fbe8e437293c6db2bb02a7e7652437fffb6b8fd10afdd8e131e023fecacbae
  Stored in directory: /tmp/pip-ephem-wheel-cache-il4thzwk/wheels/3c/80/98/9d4dcd809f9ada257446b260db163d7ff8e1d92d500baae741
Successfully built xformers
Finished!
Moving .whl to /notebooks/
Here is your wheel file:
/notebooks/xformers-0.0.18+da27862.d20230331-cp39-cp39-linux_x86_64.whl
Installing your new Xformers wheel...
Processing /tmp/tmp.euqA6lL653/xformers-0.0.18+da27862.d20230331-cp39-cp39-linux_x86_64.whl
Collecting pyre-extensions==0.0.23
  Downloading pyre_extensions-0.0.23-py3-none-any.whl (11 kB)
Requirement already satisfied: torch>=1.12 in /usr/local/lib/python3.9/dist-packages (from xformers==0.0.18+da27862.d20230331) (1.12.0+cu116)
Requirement already satisfied: numpy in /usr/local/lib/python3.9/dist-packages (from xformers==0.0.18+da27862.d20230331) (1.23.1)
Collecting typing-inspect
  Downloading typing_inspect-0.8.0-py3-none-any.whl (8.7 kB)
Requirement already satisfied: typing-extensions in /usr/local/lib/python3.9/dist-packages (from pyre-extensions==0.0.23->xformers==0.0.18+da27862.d20230331) (4.3.0)
Collecting mypy-extensions>=0.3.0
  Downloading mypy_extensions-1.0.0-py3-none-any.whl (4.7 kB)
Installing collected packages: mypy-extensions, typing-inspect, pyre-extensions, xformers
Successfully installed mypy-extensions-1.0.0 pyre-extensions-0.0.23 typing-inspect-0.8.0 xformers-0.0.18+da27862.d20230331

Then I reran "Install requirements and download repositories" cell. Then I launched the WebUI and got this
/storage/stable-diffusion/stable-diffusion-webui
WARNING[XFORMERS]: xFormers can't load C++/CUDA extensions. xFormers was built for:
    PyTorch 1.12.0+cu116 with CUDA 1102 (you have 1.13.1+cu117)
    Python  3.9.13 (you have 3.9.13)
  Please reinstall xformers (see https://github.com/facebookresearch/xformers#installing-xformers)
  Memory-efficient attention, SwiGLU, sparse and more won't be available.
  Set XFORMERS_MORE_DETAILS=1 for more details

Now When I try to generate image I get this
Error completing request
Arguments: ('task(jgl25uwqwavjzon)', 'test prompt', '', [], 20, 0, False, False, 1, 1, 7, -1.0, -1.0, 0, 0, 0, False, 512, 512, False, 0.7, 2, 'Latent', 0, 0, 0, [], 0, False, True, False, 'none', 'None', 1, None, False, 'Scale to Fit (Inner Fit)', False, False, 64, 64, 64, 0, 1, False, False, 'none', 'None', 1, None, False, 'Scale to Fit (Inner Fit)', False, False, 64, 64, 64, 0, 1, False, False, 'none', 'None', 1, None, False, 'Scale to Fit (Inner Fit)', False, False, 64, 64, 64, 0, 1, False, False, '1:1,1:2,1:2', '0:0,0:0,0:1', '0.2,0.8,0.8', 150, 0.2, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, False, False, 'positive', 'comma', 0, False, False, '', 1, '', 0, '', 0, '', True, False, False, False, 0, None, None, None, 50) {}
Traceback (most recent call last):
  File "/storage/stable-diffusion/stable-diffusion-webui/modules/call_queue.py", line 56, in f
    res = list(func(*args, **kwargs))
  File "/storage/stable-diffusion/stable-diffusion-webui/modules/call_queue.py", line 37, in f
    res = func(*args, **kwargs)
  File "/storage/stable-diffusion/stable-diffusion-webui/modules/txt2img.py", line 56, in txt2img
    processed = process_images(p)
  File "/storage/stable-diffusion/stable-diffusion-webui/modules/processing.py", line 486, in process_images
    res = process_images_inner(p)
  File "/storage/stable-diffusion/stable-diffusion-webui/modules/processing.py", line 638, in process_images_inner
    x_samples_ddim = [decode_first_stage(p.sd_model, samples_ddim[i:i+1].to(dtype=devices.dtype_vae))[0].cpu() for i in range(samples_ddim.size(0))]
  File "/storage/stable-diffusion/stable-diffusion-webui/modules/processing.py", line 638, in <listcomp>
    x_samples_ddim = [decode_first_stage(p.sd_model, samples_ddim[i:i+1].to(dtype=devices.dtype_vae))[0].cpu() for i in range(samples_ddim.size(0))]
  File "/storage/stable-diffusion/stable-diffusion-webui/modules/processing.py", line 423, in decode_first_stage
    x = model.decode_first_stage(x)
  File "/storage/stable-diffusion/stable-diffusion-webui/modules/sd_hijack_utils.py", line 17, in <lambda>
    setattr(resolved_obj, func_path[-1], lambda *args, **kwargs: self(*args, **kwargs))
  File "/storage/stable-diffusion/stable-diffusion-webui/modules/sd_hijack_utils.py", line 28, in __call__
    return self.__orig_func(*args, **kwargs)
  File "/usr/local/lib/python3.9/dist-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
    return func(*args, **kwargs)
  File "/storage/stable-diffusion/stable-diffusion-webui/repositories/stable-diffusion-stability-ai/ldm/models/diffusion/ddpm.py", line 826, in decode_first_stage
    return self.first_stage_model.decode(z)
  File "/storage/stable-diffusion/stable-diffusion-webui/repositories/stable-diffusion-stability-ai/ldm/models/autoencoder.py", line 90, in decode
    dec = self.decoder(z)
  File "/usr/local/lib/python3.9/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
  File "/storage/stable-diffusion/stable-diffusion-webui/repositories/stable-diffusion-stability-ai/ldm/modules/diffusionmodules/model.py", line 631, in forward
    h = self.mid.attn_1(h)
  File "/usr/local/lib/python3.9/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
  File "/storage/stable-diffusion/stable-diffusion-webui/repositories/stable-diffusion-stability-ai/ldm/modules/diffusionmodules/model.py", line 258, in forward
    out = xformers.ops.memory_efficient_attention(q, k, v, attn_bias=None, op=self.attention_op)
  File "/usr/local/lib/python3.9/dist-packages/xformers/ops/fmha/__init__.py", line 196, in memory_efficient_attention
    return _memory_efficient_attention(
  File "/usr/local/lib/python3.9/dist-packages/xformers/ops/fmha/__init__.py", line 294, in _memory_efficient_attention
    return _memory_efficient_attention_forward(
  File "/usr/local/lib/python3.9/dist-packages/xformers/ops/fmha/__init__.py", line 310, in _memory_efficient_attention_forward
    op = _dispatch_fw(inp)
  File "/usr/local/lib/python3.9/dist-packages/xformers/ops/fmha/dispatch.py", line 98, in _dispatch_fw
    return _run_priority_list(
  File "/usr/local/lib/python3.9/dist-packages/xformers/ops/fmha/dispatch.py", line 73, in _run_priority_list
    raise NotImplementedError(msg)
NotImplementedError: No operator found for `memory_efficient_attention_forward` with inputs:
     query       : shape=(1, 4096, 1, 512) (torch.float16)
     key         : shape=(1, 4096, 1, 512) (torch.float16)
     value       : shape=(1, 4096, 1, 512) (torch.float16)
     attn_bias   : <class 'NoneType'>
     p           : 0.0
`cutlassF` is not supported because:
    xFormers wasn't build with CUDA support
    Operator wasn't built - see `python -m xformers.info` for more info
`flshattF` is not supported because:
    xFormers wasn't build with CUDA support
    max(query.shape[-1] != value.shape[-1]) > 128
    Operator wasn't built - see `python -m xformers.info` for more info
    requires a GPU with compute capability > 7.5
`tritonflashattF` is not supported because:
    xFormers wasn't build with CUDA support
    max(query.shape[-1] != value.shape[-1]) > 128
    Operator wasn't built - see `python -m xformers.info` for more info
    triton is not available
    requires A100 GPU
`smallkF` is not supported because:
    xFormers wasn't build with CUDA support
    dtype=torch.float16 (supported: {torch.float32})
    max(query.shape[-1] != value.shape[-1]) > 32
    Operator wasn't built - see `python -m xformers.info` for more info
    unsupported embed per head: 512

python: 3.9.13
torch: 1.13.1+cu117
xformers: 0.0.18+da27862.d20230331
gradio: 3.16.2
commit: a9fed7c3
checkpoint: 7f16bbcd80

Can you make sure your notebook is up to date? I know the xformers builder used to build the minimized version without the forward attention operators.
Otherwise, something funky might be happening with torch versions: PyTorch 1.12.0+cu116 with CUDA 1102 (you have 1.13.1+cu117)

I'm having the same problem although my error says:
WARNING[XFORMERS]: xFormers can't load C++/CUDA extensions. xFormers was built for:
PyTorch 2.0..0+cu118 with CUDA 1108 (you have 1.13.1+cu117)
I tried updating the launch.py file to:
torch_command = os.environ.get('TORCH_COMMAND', "pip install torch==2.0.0+cu118 torchvision==0.14.1+cu118 --extra-index-url https://download.pytorch.org/whl/cu118")
I also set the requirements_versions.txt to torch==2.0.0
That didn't work, just created a bunch of errors when I ran the notebook so I changed it back. I also tried reinstalling the stable-diffusion-webui folder to no avail.
My notebook is up to date. Not a huge deal but hopefully we someone can get to the bottom of this.

I solved the issue. The problem was that I did not run the "Install requirements and download repositories" block before building xFormers. I updated the notebook, reset storage and built xFormers again. Old errors are gone but I got a new one
TypeError: Descriptors cannot not be created directly.
If this call came from a _pb2.py file, your generated code is out of date and must be regenerated with protoc >= 3.19.0.
If you cannot immediately regenerate your protos, some other possible workarounds are:
 1. Downgrade the protobuf package to 3.20.x or lower.
 2. Set PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=python (but this will use pure-Python parsing and will be much slower).

I downgraded the protobuf package to 3.20.3
Now everything works without any errors but I think that xFormers doesn't work because it takes the same amount of time to generate image as before when I didn't install it.

I had that error, but I don't remember how I fixed it. I might have re-built xformers...

@personxyz what gpu are you using?

@personxyz what gpu are you using?

A6000.

I rebuilt xFormers again and got the same result. It seems that Quadro M4000 is too old to benefit from the xFormers

WARNING[XFORMERS]: xFormers can't load C++/CUDA extensions. xFormers was built for:
PyTorch 2.0..0+cu118 with CUDA 1108 (you have 1.13.1+cu117)
遇到这个问题我的解决方法是: (you have 1.13.1+cu117)这是你的系统内的版本,
重新安装 xFormers 可以按照以下步骤进行：
首先，打开命令行界面。
使用以下命令卸载当前已安装的 xFormers（如果已安装）：
x:\xxx\xxx\python(你的SD目录内的python文件夹),在上面键入CMD
然后在控制台输入以下命令
pip uninstall xformers
完成后打开启动器(我是秋叶的绘世)
高级选项 点进去后跟一键启动并排 有个环境维护,配置pyTorch点开,选择版本,与你的(you have 1.13.1+cu117)这一项版本一致的选择完点击安装即可解决.


panic: double insert of ... when moving one file to another.

this happens when executing a mv from one file to another. ej mv new.jpg new
2017/08/22 18:11:25.770046 s3.INFO Switching from region 'us-east-1' to 'us-west-2'
2017/08/22 18:11:26.184980 main.INFO File system has been successfully mounted.
panic: double insert of filer_public/27/40/27400712-4eab-40ee-84dc-9869b98919fe/new

goroutine 53648 [running]:
github.com/kahing/goofys/internal.(*Inode).insertChildUnlocked(0xc420d15dc0, 0xc422003a40)
	/home/khc/Code/go/src/github.com/kahing/goofys/internal/handles.go:267 +0x626
github.com/kahing/goofys/internal.(*Goofys).Rename(0xc420099320, 0x7f06016fccb8, 0xc42084a2d0, 0xc42084a2a0, 0x0, 0x0)
	/home/khc/Code/go/src/github.com/kahing/goofys/internal/goofys.go:1444 +0x38b
github.com/jacobsa/fuse/fuseutil.(*fileSystemServer).handleOp(0xc420224260, 0xc420530180, 0x7f06016fccb8, 0xc42084a2d0, 0x8c3c00, 0xc42084a2a0)
	/home/khc/Code/go/src/github.com/jacobsa/fuse/fuseutil/file_system.go:156 +0x226
created by github.com/jacobsa/fuse/fuseutil.(*fileSystemServer).ServeOps
	/home/khc/Code/go/src/github.com/jacobsa/fuse/fuseutil/file_system.go:112 +0x12a

what version is this? I cannot reproduce this

please reopen if you can reproduce this with the last release or master

I also met this issue .When update to master, the panic disappear.
But when repeat mv a b ,second,b have existing, will panic on the third mv.
please ref: https://github.com/kahing/goofys/pull/225/commits/e8ae74ca5e3a9283d322f0b441aa7bc54bd1b16d.


Create copilotkit-task-manager.md

What does this PR do?
Design a simple task management app where users can create, update, and delete tasks using voice or text commands
Related PRs and Issues
⭐ 03 - Task Manager (Hacktoberfest Demo)  #604
Vercel Link - https://task-manager-nu-cyan.vercel.app/
Checklist

[x] I have read the Contribution Guide
[x] If the PR changes or adds functionality, I have updated the relevant documentation

@NathanTarbert....please review my pr.

Nice job @Vaishnavi-Raykar!


MM-41267: Keywords prompt playbook action

Summary
This PR adds a new trigger and new action to channels, refactored from the current Playbooks action: when certain keywords are posted in a channel, prompt the user to run a playbook. The configuration is now per-channel, meaning there is not team-wide keywords triggers as there was before when this lived as a Playbooks action.
Please, review by commit and read the commit message and description, they should be self-explanatory.
For some context: although this is a +874 -979 lines PR, you'll find that it's actually not as bad as it looks: a big bunch of the changes are simply deleting code (https://github.com/mattermost/mattermost-plugin-playbooks/commit/881b198410182cc63eb33b9ef65d2cdae4e169b8 and https://github.com/mattermost/mattermost-plugin-playbooks/commit/e1eeca4cb467904047d165dc1bfb21d55f3456f9) and moving code around (https://github.com/mattermost/mattermost-plugin-playbooks/commit/94e5607772e78d044fc43cc879071b6879a0ef54), so hopefully it's not too annoying to review :)
Note: There's one error in the description of https://github.com/mattermost/mattermost-plugin-playbooks/pull/1081/commits/003bb9b9c5ea7b50e5212339e16d0de24b298d5b: the map is not TriggerType->(ActionType->Action) but TriggerType -> []Action.
Screenshots

Channel admins (editable)
Regular members (view-only)

Ticket Link
https://mattermost.atlassian.net/browse/MM-41267
https://mattermost.atlassian.net/browse/MM-41268
Checklist

[ ] Telemetry updated: I'll update it as soon as https://github.com/mattermost/mattermost-plugin-playbooks/pull/1078 is merged
[ ] Gated by experimental feature flag
[X] Unit tests updated

/update-branch

Thanks, @abhijit-singh! I'll fix that in the PR for the LHS categorization action, as I slightly changed the HTML structure there.


⚠ ClamAV has degraded performance

In f0c319f, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 112 ms

Resolved: ClamAV performance has improved in cd22a7c.


Twig_Error_Loader missing

Add use Twig_Error_Loader; otherwise catch Twig_Error_Loader (line 117) is never caught and falls through.

Coverage remained the same at 25.044% when pulling 62a89b8d15254fa4541a5f98e979c45699937bd0 on kitbs:patch-1 into eb65adcffc5d13fd984604eb6669f6c72af50303 on rcrowe:master.


[interop][SwiftToCxx] annotate several value type members as 'noexcept'

This is needed to use generic types with some C++/clang versions as otherwise clang errors out on the lack of noexcept on the destructor.

@swift-ci please test and merge

@swift-ci please test and merge


Question 2

Would it be possible to, in the future, to include information from the logbooks with each entry in the personography?  It would be helpful to have information ready-to-hand that might help differentiate (or equate) adam3 and adam2, for example. We might learn that the memberships for adam3 and adam2 overlap, indicating that they are separate people. Or we might discover evidence that they are the same person. It would also be helpful to have links to the cards in the personography.

We will absolutely include logbook data once we import it - that's already planned for the next phase of development work once the initial version of the personography is tested and moves into production.
If you have ideas about how you would like to see that logbook data displayed, please describe what you want to see or what functionality you need so that I can plan the development work accordingly.   Perhaps we could include a brief listing of membership date ranges or type of membership on the person list display?

Created #37 to address this request.


Notification component support for links

The Carbon Angular Notification component does not currently support the ability to make message content url links as described in the Carbon Design specification https://www.carbondesignsystem.com/components/notification/code.  All messages in z/OSMF (we use Notification Component to issue messages) require the ability to link to additional help content, which makes this linking capability critical to our project.
Thank you.

In the short term you should be able to use a template to display a link in the notification - https://github.com/IBM/carbon-components-angular/blob/master/src/notification/notification-content.interface.ts#L15
Thinking through the API, what would be most useful? Since the content is just a string, would it make sense to support a links: string[] option in the NotificationContent?

As requested by @cal-smith , the version Carbon-components-angular we are using is 2.49.10.  We are looking for linking capability (notification components) in this version.  Thank You.


CSUB-962, CSUB-959, CSUB-969: Polkadot Release v1.1.0 patch

Description of proposed changes

Practical tips for PR review & merge:

[ ] All GitHub Actions report PASS
[ ] Newly added code/functions have unit tests

[ ] Coverage tools report all newly added lines as covered
[ ] The positive scenario is exercised
[ ] Negative scenarios are exercised, e.g. assert on all possible errors
[ ] Assert on events triggered if applicable
[ ] Assert on changes made to storage if applicable

[ ] Modified behavior/functions - try to make sure above test items are covered
[ ] Integration tests are added if applicable/needed

See https://github.com/gluwa/creditcoin3/pull/132#discussion_r1431291054 for detailed test results + screenshots!


Set BottomBar height to 56dp

The height of the bottom bar should be 56dp according to the Material Design Specs.
See #168

Actually forcing the layout_height in the style has no impact in the height of the bottom bar since the items height is defined programatically by params value

I've a fork where I've been working on fixing the height and it turns out to be more tricky because according to the material design spec there should be a 10dp padding for the baseline of the title textview which seems to be quite tricky.
@roughike do you have some input on this? Have you looked into having the height fixed to 56dp at all? Kiitos :-)


CSS degeneration inside Docker only.

Works fine not in docker. Weirdly I'm getting an escape character inside of the container:
Syntax error: atomEscape at position 36
0ffff}]|^--|[^A-Za
^
Running normal docker on Ubuntu

Ok I changed the package manager to "yarn" which fixed it.  I'll create a pull request.


JIT provisioning and persisting User doesn't change anything in the database

Hello !
So I'm having with using the JIT provisioning and the persist_user.
What I'm trying to achieve is that at every connexion of the user, if in the attributes "roles" something has changed, than it changes its roles into the database.
What my code looks like :
in the security.yaml ->
providers:
        saml_provider:
            ##  Basic provider instantiates a user with identifier and default roles
            entity:
                class: App\Entity\User
                property: email

...

main:
            pattern: ^/
            provider: saml_provider
            saml:
                ##  Match SAML attribute 'uid' with user identifier.
                ##  Otherwise, used \OneLogin\Saml2\Auth::getNameId() method by default.
                identifier_attribute: email
                check_path: saml_acs
                login_path: saml_login
                user_factory: saml_user_factory
                persist_user: true
            logout:
                path: saml_logout

in the services.yaml ->
services:
    saml_user_factory:
          class: App\Security\UserFactory

And in my UserFactory.php ->
 public function createUser(string $identifier, array $attributes): UserInterface
 {
        $user = new User();
        $user->setEmail($attributes['email'][0]);
        $user->setFirstname($attributes['firstname'][0]);
        $user->setLastname($attributes['lastname'][0]);
        $user->setTrigram($attributes['trigram'][0]);

        $rolesAttributes = $attributes['roles'];
        if ($_ENV['APP_ENV'] == 'prod') {
            $user->setRoles($this->setProdRoles($rolesAttributes));
        } else if ($_ENV['APP_ENV'] == 'dev') {
            $user->setRoles($this->setDevRoles($rolesAttributes));
        } else {
            $user->setRoles([]);
        }
        return $user;
}

And then for an exemple of the prod roles function ->
public function setProdRoles(array $rolesAttributes): array
    {
        $roles = [];
        if (in_array('roles1', $rolesAttributes)) {
            $roles[] = 'ROLE_USER';
        }
        if (in_array('roles2', $rolesAttributes)) {
            $roles[] = 'ROLE_TEST1';
        }
        if (in_array('roles3', $rolesAttributes)) {
            $roles[] = 'ROLE_TEST2';
        }
        return $roles;
    }

For the use case, what happens is that when a user load for the first time on my application, and in his attribute roles he has only "roles1", then he's going to have 'ROLE_USER' in the database.
But then, in the saml server we add the 'roles2', and what happens is that when he reconnects to the website, he doesn't have the 'ROLE_TEST1' in the database, even though we tried to delete cookies, etc... I have to manually change in the database...
How can i achieve this please ?

Hi @likou31
Does your App\Entity\User class implement the \Nbgrp\OneloginSamlBundle\Security\User\SamlUserInterface?

Hello !
Yes it does implement the \Nbgrp\OneloginSamlBundle\Security\User\SamlUserInterface. But the method setSamlAttributes is empty (i do everything in the factory).

The User Factory is used only when the User Provider throws the UserNotFoundException during authentication. So, it's needed for user creation, not for modification. If the user exists it will be returned by User Provider and setSamlAttributes will be called: https://github.com/nbgrp/onelogin-saml-bundle/blob/fd733c7df79b6ec6a0275324066ddac0a0fb996c/src/Security/Http/Authenticator/SamlAuthenticator.php#L153-L157

Oh ok ! I got it ! I managed to make it work, thank you ! That was it, i had to logout and then log in, and the setSamlAttributes worked perfectly... Thank you !


Automatically ingest log data into ELK stack

ISSUE TYPE

Feature request

SUMMARY
I would like an optional configuration option that allows someone to use the ELK stack to store all logs generated by Docker machines, and perhaps include some useful stats / information within Kibana. This should be optional so that it doesn't prevent devs from using the ELK stack for their own needs.
Goal
Ideally what I want is a way to easily and quickly ingest, parse, search and report on logs that the Docker machines generate. In particular, I want to see logs generated by my PHP applications, and have them stored for longer than the current Docker uptime (while I can see the log directory exists, it'd be nice to leverage the power of ElasticSearch and Kibana for this, given all the building blocks are already there and it's more about plumbing than building something new).
I would be interested in potentially building some part of this, if it's interesting to the Devilbox maintainers.
Thanks for your consideration!

@madmatt Hello,
Even if the https://github.com/amir20/dozzle  project is not really what you expect to have, it could help you as a quick alternative.

Thanks neat, thanks @llaville!

For a console based monitory take a look at https://github.com/jesseduffield/lazydocker

I would also be very much interested in an automatism here. @madmatt if you want to take the lead up on this feel free.

Vector could also be the solution searched !

@madmatt and @cytopia Hello guys
This feature and the one detailed by #722 seems really close.
As I've a solution up and running, I'd like to take the lead (if you are agree)


Create Mobile Nav Slide-In Menu

Create a slide-in menu when hamburger icon is clicked in mobile view, likely from the right as same side as icon.
Site currently has breakpoint to convert nav menu to hamburger, but hamburger does not action when clicked currently.

@peterpme this would be a good first issue as well.

awesome! I think we can do this just by including the bootstrap js:
From the bootstrap docs:
CSS only

<link rel="stylesheet" href="https://stackpath.bootstrapcdn.com/bootstrap/4.1.3/css/bootstrap.min.css" integrity="sha384-MCw98/SFnGE8fJT3GXwEOngsV7Zt27NXFoaoApmYm81iuXoPkFOJwJ8ERdknLPMO" crossorigin="anonymous">

JS, Popper.js, and jQuery

<script src="https://code.jquery.com/jquery-3.3.1.slim.min.js" integrity="sha384-q8i/X+965DzO0rT7abK41JStQIAqVgRVzpbzo5smXKp4YfRvH+8abtTE1Pi6jizo" crossorigin="anonymous"></script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/popper.js/1.14.3/umd/popper.min.js" integrity="sha384-ZMP7rVo3mIykV+2+9J3UJ46jBk0WLaUAdn689aCwoqbBJiSnjAK/l8WvCWPIPm49" crossorigin="anonymous"></script>
<script src="https://stackpath.bootstrapcdn.com/bootstrap/4.1.3/js/bootstrap.min.js" integrity="sha384-ChfqqxuZUCnJSK3+MXmPNIyE6ZbWh2IMqE241rYiqJxyMiZ6OW/JmZQ5stwEULTy" crossorigin="anonymous"></script>


nfs: add support for clients in the StorageClass

The clients parameter in the storage class is used to limit access to the export to the set of hostnames, networks or ip addresses specified.
/cc spuiuk
/assign Madhu-1

/lgtm


Pose Recognition Android Crashes on Startup

Following this guide here.
Build is successful, however on launching the pose recognition app via the emulator, the app instantly crashes.
Running on Android API 34 on Android 14.0 ("UpsideDownCake"). Using emulator via Android Studio.
Has anyone else ran into this issue before?

I think it's been brought up a few times here on GitHub. For some of the vision tasks you absolutely need to run on a physical device.

I think it's been brought up a few times here on GitHub. For some of the vision tasks you absolutely need to run on a physical device.

Yes this is solved. Needs to be ran on ARM64. M1 mac's can emulate it but if running windows you need a physical device.
I'll close this.


Feature: Default lang

Add default locale to settings

https://github.com/vipzhicheng/logseq-plugin-block-calendar/pull/3

A new version released, so close this issue for now.


@llama/env does not export Tokenizer on jsr

error: Uncaught SyntaxError: The requested module 'jsr<EMAIL_ADDRESS>does not provide an export named 'Tokenizers'
import { Tokenizers } from "jsr<EMAIL_ADDRESS>         ^
    at <anonymous> (https://jsr.io/@llamaindex/llamaindex/0.4.0/src/llm/openai.ts:10:10)

unfortunately I don't release JSR anymore since last week, because it has bugs with some npm pacakge

unfortunately I don't release JSR anymore since last week, because it has bugs with some npm pacakges

Does it mean that the support for JSR is discontinued, or it will resume some time later?

blocked by https://github.com/jsr-io/jsr/issues/543


Change how we default feature states & requirement levels

Defaulting to All in AddFlags didn't allow for a user to enable a specific level or state because internally we were ORing with a bitset

missed a test
/hold

/hold cancel

/assign @lionelvillard @n3wscott


Custom Name

Hi,
If i understand correctly the multi-select will only work correctly if the array of objects being supplied has a property called ItemName.  Is that correct or can you specify a setting that allows you to specify the property to be used for the itemName?
For example if i had
this.dropdownList = products;
products object looked like this
[
       {"id":1,"productTitle":"Oracle"},
       {"id":2,"productTitle":"SAP"},
 ];

Hi,
I don't think we should make the object property names generic. There is no way to identify which property belong to the name field and even if we do with the property index number, it would make the data structure unusable for any data manipulations.. Not feasible, hence closing the issue.


axum_sever doesn't work with unreleased version of axum (0.3.0)

Lately Axum has modified the internal implementation of Router and they have started boxing the routers by default (https://github.com/tokio-rs/axum). Unfortunately these new changes don't work with axum_server.
I get following compilation error:
error[E0277]: `(dyn axum::clone_box_service::CloneService<axum::http::Request<axum::body::Body>, Response = axum::http::Response<http_body::combinators::box_body::UnsyncBoxBody<bytes::Bytes, axum::Error>>, Error = Infallible, Future = Pin<Box<(dyn futures_util::Future<Output = Result<axum::http::Response<http_body::combinators::box_body::UnsyncBoxBody<bytes::Bytes, axum::Error>>, Infallible>> + std::marker::Send + 'static)>>> + std::marker::Send + 'static)` cannot be shared between threads safely

See #23.

Keep this issue open until resolved. :+1:

Thanks for the prompt response.

Updated yesterday. Version 0.3 works with axum.


⚠ Bofrost.at has degraded performance

In a16329c, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1346 ms

Resolved: Bofrost.at performance has improved in 992ce3f after 9 minutes.


[Feature]: Labelled results.core.k8sgpt.ai resources

Checklist

[X] I've searched for similar issues and couldn't find anything matching
[X] I've discussed this feature request in the K8sGPT Slack and got positive feedback

Is this feature request related to a problem?
No
Problem Description
No response
Solution Description
The Result objects, when generated, are not labelled.
If using labels, it would be interesting to get more data about the k8sgpt resource who generated it, as well as other important information, such as the Deployment who generated it.
Benefits
The basic idea is to take full advantage of the label selection offered by Kubernetes, simplifying the selection of Result instances generated by a specific driver.
e.g.:
kubectl get results.core.k8sgpt.ai -l k8sgpts.k8sgpt.ai/name=k8sgpt-sample
kubectl get results.core.k8sgpt.ai -l k8sgpts.k8sgpt.ai/backend=openapi

Potential Drawbacks
No response
Additional Information
No response

I'm happy to work on this, by providing a PR.

JFI I marked the k8sgpt Slack discussion as completed although it's not, since I'm not able to join the Slack channel since it's open only to k8sgpt.ai domains.

JFI I marked the k8sgpt Slack discussion as completed although it's not, since I'm not able to join the Slack channel since it's open only to k8sgpt.ai domains.

hmm that's odd @prometherion , we 'll have a look


How can I disable JS minification

Is it possible to disable JS minification?  I really love the minify on save function for this package on css, but it does not minify ES6 well and I want to disable JS minification.  How can I do that?

Im looking to do that same thing. CCS minification is great but I have project config files that don't need compression. Having the option to place a skip comment at beginning of the file would be nice.


Work around for #176 "Scene graph is not properly updated for renderi…

…ng" when adding LightProbe to a Geometry.
It seems the issue happens inside LightProbeFactory. I think this is an acceptable solution for now, since the consequences of the error are pretty bad.
If someone really wants a LightProbe for just one Geometry, they can replicate it by adding the Geometry to a Node and add the LightProbe to that.

Hmm, or is it that now the modification happens from the Swing thread. And
instead it should be added in jME render thread? With the enqueue.
On Tue, 4 Apr 2023, 22:19 Rickard Edén, @.***> wrote:

…ng" when adding LightProbe to a Geometry.
It seems the issue happens inside LightProbeFactory. I think this is an
acceptable solution for now, since the consequences of the error are pretty
bad.
If someone really wants a LightProbe for just one Geometry, they can
replicate it by adding the Geometry to a Node and add the LightProbe to
that.
You can view, comment on, or merge this pull request online at:
https://github.com/jMonkeyEngine/sdk/pull/490
Commit Summary

aa453c3
https://github.com/jMonkeyEngine/sdk/pull/490/commits/aa453c3822a2ec6c2881a49f53c160dde7668192
Work around for #176 "Scene graph is not properly updated for rendering"
when adding LightProbe to a Geometry.

File Changes
(1 file https://github.com/jMonkeyEngine/sdk/pull/490/files)

M
jme3-core/src/com/jme3/gde/core/sceneexplorer/nodes/actions/NewLightPopup.java
https://github.com/jMonkeyEngine/sdk/pull/490/files#diff-11723da5c6b99064b32ba4244250733ca49319b1f09859a72e79198954f16e9e
(7)

Patch Links:

https://github.com/jMonkeyEngine/sdk/pull/490.patch
https://github.com/jMonkeyEngine/sdk/pull/490.diff

—
Reply to this email directly, view it on GitHub
https://github.com/jMonkeyEngine/sdk/pull/490, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB7VJPTPG2MLFTOIO7DM6YTW7RX3RANCNFSM6AAAAAAWTEC5ZA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

My first test was to comment out:
node.addLight(lightProbe);
node.updateModelBound();

And it still happened

This is probably the culprit:
EnvironmentCamera:
@Override
    public void render(final RenderManager renderManager) {
        if (isBusy()) {
            final SnapshotJob job = jobs.get(0);

            for (int i = 0; i < 6; i++) {
                viewports[i].clearScenes();
                viewports[i].attachScene(job.scene);
                renderManager.renderViewPort(viewports[i], 0.16f);
                buffers[i] = BufferUtils.createByteBuffer(
                        size * size * imageFormat.getBitsPerPixel() / 8);
                renderManager.getRenderer().readFrameBufferWithFormat(
                        framebuffers[i], buffers[i], imageFormat);
                images[i] = new Image(imageFormat, size, size, buffers[i],
                        ColorSpace.Linear);
                MipMapGenerator.generateMipMaps(images[i]);
            }

            final TextureCubeMap map = EnvMapUtils.makeCubeMap(images[0],
                    images[1], images[2], images[3], images[4], images[5],
                    imageFormat);
            debugEnv = map;
            job.callback.done(map);
            map.getImage().dispose();
            jobs.remove(0);
        }
    }

I guess proper handling would be to let the LightProbe finish its rendering and then attach it. Or just add it and let the user "update the maps" themselves?

Acceptable temp fix? I'll merge this soon unless there's further discussion.


improvement in UI of bodyFat.html in navbar

I will change the navlinks button border to 2px i.e, neither too sharp nor too curved so, it will give a modern look.
Also the logo is overlapped by the home link when in mobile view, I will remove the ovelap by adding some margin.
Please assign me this issue.

Hey @aksharma27! Can you please share screenshots, so that i can understand your concern properly.

Before: The Home link when hovered was overlapping the logo in mobile and
the text was left aligned.
After: Home link is not overlapping the logo when hovered in mobile view
and here I have aligned the text to center
On Sun, 2 Oct 2022 at 20:59, Sachin Pandey @.***> wrote:

Hey @aksharma27 https://github.com/aksharma27! Can you please share
screenshots, so that i can understand your concern properly.
—
Reply to this email directly, view it on GitHub
https://github.com/skpandey885/HealhUb/issues/133#issuecomment-1264669830,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT2IZEQFDKTQBODFWELULZLWBGS37ANCNFSM6AAAAAAQ25HMEY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Okey @aksharma27! Go ahead and must follow these steps.

Hi @aksharma27 please assign this to me

@skpandey885  please assign me this issue


Question about installation

Hi. This is a fantastic tool, thank you for developing it. Is there a way to install the tool without side-loading the add-in for testing? I would like to use it, but cannot share information from the sheets I'm working on.
Thanks,
Fraser

Hi - no information is shared with any server. All processing is done locally; the server just hosts the code and performs hot updates.

If you felt comfortable doing so, you could also run the server yourself client-side.


Can't find bash ./scripts/pull.sh

I downloaded the following zip but could not find a pull.sh file
wget https://github.com/sapientml/sapientml/files/12593737/sapientml-corpus-0.1.0.zip

Also, it would be great if you could point me to the script used to download the data from kaggle.

@TahaBinhuraib thanks for raising the issue.
We updated the corpus and the documentation.
https://github.com/sapientml/docs/files/13290907/sapientml-corpus-0.1.1.zip
includes pull.sh for downloading datasets from Kaggle.
Thanks again for your contributing!

We have updated the corpus and documentation again.
Please use the following file:
https://github.com/sapientml/sapientml/files/13360498/sapientml-corpus-0.1.2.zip

Thank you very much!