[SPARK-10300] [BUILD] [TESTS] Add support for test tags in run-tests.py.

Re-posting previous PR with a fix for the issue that caused tests to fail in other PRs. For the changes since the last PR, see only the second commit.
I'm not super happy with needing the new module, but I couldn't find an alternative that still allowed us to tag JUnit tests. The extra noise to add the dependencies to all other poms also irks me a little bit.
/cc @pwendell to see if you have a better suggestion...

Merged build triggered.

Merged build started.

Test build #42511 has started for   PR 8775 at commit 9e0b3c4.

Test build #42511 has finished for   PR 8775 at commit 9e0b3c4.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42511/
Test FAILed.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #42514 has started for   PR 8775 at commit f3bb7b4.

Test build #42514 has finished for   PR 8775 at commit f3bb7b4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42514/
Test PASSed.

retest this please

any comments on the new approach?

Merged build triggered.

Merged build started.

Test build #42844 has started for   PR 8775 at commit f3bb7b4.

Test build #42844 has finished for   PR 8775 at commit f3bb7b4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42844/
Test PASSed.

retest this please

Merged build triggered.

Merged build started.

Test build #43062 has started for   PR 8775 at commit f3bb7b4.

Test build #43062 has finished for   PR 8775 at commit f3bb7b4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43062/
Test PASSed.

I'll run the tests once again and push this, since I haven't seen any feedback on top of the previous PR. retest this please

Merged build triggered.

Merged build started.

Test build #43279 has started for   PR 8775 at commit f3bb7b4.

LGTM. Merged into master. Thanks! There might be a small issue with the default value. I will test it locally and send follow-up PR if necessary.

Test build #43279 has finished for   PR 8775 at commit f3bb7b4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43279/
Test PASSed.

@mengxr, looks like the merge commit didn't get pushed (or ASF -> Git mirroring is lagging)?

Yes, this was not pushed. I'll push this later if it's still around.

ok merging.


I have a problem with Real and string , number & logic  tests

Hello, I have a problem with these tests, caould you give me a hint? I have this error:
Lexical error on line 4: Unrecognized text.
Erroneous area:
1: #Real and string#
2:
3: 534.534;
4: "this is a text";
^..^
But I have a rule for string values

Did you use the same rule in the lab? (I think it needs to be changed a little) :)

I have changed it, but no idea why doesn't work, btw, thank you :)

Try to put float before string in .l file


PVS-Studio: The 'then' statement is equivalent to the 'else' statement.

We have found a bug using PVS-Studio tool. PVS-Studio is a static code analyzer for C, C++ and C#: https://www.viva64.com/en/pvs-studio/
We suggests having a look at the emails, sent from @pvs-studio.com.
Analyzer warning: V523 The 'then' statement is equivalent to the 'else' statement. v8-globals.h 685
lib/V8/v8-globals.h:
template <typename TARGET>
inline void TRI_V8_AddMethod(....) {
  // hidden method
  if (isHidden) {
    tpl->Set(name, v8::FunctionTemplate::New(isolate, callback)->GetFunction());
  }
  // normal method
  else {
    tpl->Set(name, v8::FunctionTemplate::New(isolate, callback)->GetFunction());
  }
}

I'm using the latest ArangoDB of the respective release series:

[+] self-compiled devel branch

On this operating system:

[+] Linux
[+] Windows

Thanks. Fixed in the devel branch now.


V22F-672 Fix dlls

This PR installs the msvc runtime dlls into the designated target folder at build time.

Merging with @KungFuDonkey 's approval


Diffview integration cannot show diff for lazy-lock.json file

Description
When I update the lazy-lock.json file in which a few lines in the file change, I'd open Neogit and toggle Diffview to see the diff. But I always get this error: [Diffview.nvim] Failed to create diff buffer: ':0:lazy-lock.json'. The standalone Diffview works just fine without errors.
Neovim version
NVIM v0.9.1
Build type: Release
LuaJIT 2.1.0-beta3
Operating system and version
6.4.4-arch1-1
Steps to reproduce

Update plugins using lazy.nvim, which will then updates the lazy-lock.json file
Open Neogit
Navigate to the lazy-lock.json file, and toggle Diffview

Expected behavior
No response
Actual behavior
The mentioned error is shown
Minimal config
local lazypath = vim.fn.stdpath('data') .. '/lazy/lazy.nvim'
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({
    'git',
    'clone',
    '--filter=blob:none',
    'https://github.com/folke/lazy.nvim.git',
    '--branch=stable',
    lazypath,
  })
end
vim.opt.rtp:prepend(lazypath)

vim.loader.enable()

require('lazy').setup({
  {
    'NeogitOrg/neogit',
    dependencies = {
      'sindrets/diffview.nvim',
      'nvim-lua/plenary.nvim',
    },
    lazy = true,
    config = function()
        require('neogit').setup({
            disable_commit_confirmation = true,
            kind = 'tab',
            auto_show_console = false,
            popup = {
              kind = 'vsplit',
            },
            integrations = {
              diffview = true,
            },
        })
    end,
  },
})

Sounds like an issue for Diffview :\


CSS Modules Naming

CSS Loader creates its class names simply with some sort of hash.
The CSS Modules spec actually uses .folder_file_className_h45H or something similar when attempting to make unique class names. This is preferable since it is now easier to determine what the class is and where it comes from instead of guessing or following an odd rabbit trail to figure it out.
Are there plans to start using the CSS Modules spec for this, or at least give us the option to use it? Can there be?

Nevermind... somehow missed localIdentname all this time. Is there a way to make the [path] portion relative to a certain url so I don't have super long paths in the class name?

ping...
I'm hoping to be able to adjust what [path] outputs. Right now it's 4 folders deep when I only want one: src-client-styles-components instead of being based in the styles path and outputting just components.

you can pass context as query param. pass an absolute path to the root for the naming.

Doesn't seem to be working, and I don't see any mention of context in the documentation...

yep, you're right. Sorry. It's missing here: https://github.com/webpack/css-loader/blob/master/lib/processCss.js#L152

So is it not implemented? I'm not familiar with what that code is doing so I'm not sure what you're showing me. If it isn't implemented, can you implement it? Otherwise, can you give me an example how you can configure it to use the context?

I actually read through the code more and figured out how to do it without the query:
{
	module: {
		loaders: [...]
	},
	output: {...},
	plugins: [...],
	devtool: 'source-map',
	context: ABS_PATH // <- Add it here
}

This then applies to everything, not just the CSS, but that shouldn't be an issue. It's nice to have the option that you now added, though. Thanks.

css-loader?localIdentName=[path][name]--[local]&context=/abs/path/to/directory


Document TERRAFORM_VERSION env var

TERRAFORM_VERSION may be specified per https://github.com/localstack/localstack/blob/c8cfa257b67c5b8f36b10dcc43731da984e13951/localstack-core/localstack/packages/terraform.py#L10

Dupe of https://github.com/thrau/localstack-terraform-init/pull/1


[TimePicker] Update time state on new defaultTime prop.

This pull request try to fix  #3094.
If a TimePicker instance is already mounted and a new defaultTime is passed to it, the time state value doesn't update.
There was already a pull request aiming to fix this and other issues for this component at callemall/material-ui#2027, but it has been waiting for review since Nov 24, 2015.

it has been waiting for review since Nov 24, 2015

Ah, technically it has been waiting for revision since then. Happens frequently. I call them drive by PRs! :D
Anyway, back to business! DatePicker has a value prop and a controlled example, you could model TimePacker off that.

@mbrookes, @oliviertassinari
The TimePicker now adopts a behaviour similar to the DatePicker one.
Tests have been added to prove the fix, although they don't cover all scenarios, it show at least that the change is working :sweat_smile:

Gonna leave @oliviertassinari for the final review, but have to say, awesome work!

@rscnt Wow, thanks for the tests! Could you update the examples? I would be better to remove the warnings from the console (linked to the deprecated methods)

@oliviertassinari docs have been updated, commentaries have been applied and mistakes have been fixed :smile:

@oliviertassinari warnings added and TimePicker/ComplexExample is now using arrow syntax.

@rscnt Thanks! I will have a closer look at this PR, but that look good so far. I have seen some minor issue, like using let where const would work or adding a _ for method, but I wouldn't worry too much about it.
By the way, before we merge, you will have to squats down the commits. (make the history cleaner).

@oliviertassinari 'squashed' :+1:

@rscnt We are almost there :).

@oliviertassinari observations have been applied and conflicts with master@HEAD have been resolved.

Hah, I was using this PR to test error output on gpr, and the errors went away. :laughing:

@rscnt Thanks! Sorry, that's isn't ready to be merged :grin:. I have more feedbacks and we have two conflicts with the master branch (wording and  WindowsListener).

@oliviertassinari removed.

@rscnt Thanks I have done a final review of this PR, aside from my two comments and the Travis fail. I think that we can merge :tada:.
src/utils/date-time.js
  159:11  error  `isAM` is never modified, use `const` instead        prefer-const
  161:11  error  `additional` is never modified, use `const` instead  prefer-const

This looks awesome :heart_eyes: can't wait to see it merged :+1:

@oliviertassinari sorry, linting issues have been fixed.

@rscnt Thanks that's a great feature :rocket:.


HTML in event title

Hi,
Is it somehow possible to use HTML inside the event.title property to add icons?
e.g.
$scope.events = [
    {
      start: ....,
      end: ....,
      title: '<i class="fa fa-whatever"></fa> new event title'
    }, ....
];

use eventRender method:
    $scope.uiConfig = {
      calendar:{
...
            eventRender: $scope.eventRender
...
    }};

    $scope.eventRender = function( event, element, view ) {
    };

Thanks, that helped me a lot!


Parser support for REPLICATION_SLAVE_ADMIN dynamic privilege

REPLICATION_SLAVE_ADMIN is a MySQL dynamic privilege and is the privilege needed to execute START REPLICA, STOP REPLICA, CHANGE REPLICATION SOURCE and CHANGE REPLICATION FILTER.
In MySQL, static privileges are built in to the server and always available to use in grants. Dynamic privileges are registered and unregistered at runtime by server components, and only valid to use in grants when they are registered.
Since this is our first dynamic privilege, and I don't know of any others we need to support yet, I chose to simply hardcode in the privilege name to the parser. This means we can't support any other dynamic privileges yet, but I don't think we need to yet. Doing it this way also means we don't have to build a register/unregister API in GMS for dynamic privileges, so it seemed like an easier way to get started. Alternatively, if we know we want to do more with dynamic privileges, I'm happy to update the parser to accept any string and we can do the validation in GMS.

I think we should just accept any string here, and do the validation in GMS. When I go to implement support, I'm thinking of just adding support for all "standard" MySQL dynamic privileges, and then throwing an error message if someone attempts to use one that hasn't been fleshed out yet.

I'm cool with that. I spent a few mins changing the parser rule on Friday, but hit parser conflicts and had to switch to some other tasks. I'll try to get back to debug changing that rule to accept any string after I get a few more tasks cleared off my plate.

Closing this in favor of https://github.com/dolthub/vitess/pull/222


[Backport] [2.x] [BWC and API enforcement] Reduce the visibility of some existing APIs (#11242)

Backport of https://github.com/opensearch-project/OpenSearch/pull/11242 to 2.x

❌ Gradle check result for 7f88199: FAILURE

https://github.com/opensearch-project/OpenSearch/issues/2775

@andrross @msfroh backport please if you don't mind approving, thank you


Issue + Additional Commentary

I'm new to submitting issues, but I really like your extension. However, a few things I have noticed:

It would be nice if there was a way I could left click on the extension for "options" or "settings".

After registering (loading) "free dictionary data", upon opening settings, the data is unalterable and irremovable (even after clicking "reset to default") and is, therefore added to subsequent dictionary files loaded.

It would also be nice if there was an example TSV, JSON, etc. file embedded into the docs as a reference for custom formatting.

The loading symbol for registering data glitches.

Thank you! This is the first time I've submitted an Issue for any project on GitHub. Tell me if I'm missing anything.

After registering (loading) "free dictionary data", upon opening settings, the data is unalterable and irremovable (even after clicking "reset to default") and is, therefore added to subsequent dictionary files loaded.

Unfortunately, data deletion feature is not supported due to some technical testriction (#51)

It would also be nice if there was an example TSV, JSON, etc. file embedded into the docs as a reference for custom formatting.

That's true. I wrote it.

It would be nice if there was a way I could left click on the extension for "options" or "settings".
The loading symbol for registering data glitches.

What do these mean?

close due to no response.


[Ingenic] Enabling OSD crashes majestic, camera reboots

Any date after June 1st 2023 crashes majestic if OSD is enabled. At least on Ingenic SoC.

Fixed


PLAT-26773 Adding support for converting XDM fields to ParquetIOFields

@adobe/open-source-office

Hey @sshefs02, I see that you are an Adobe employee. You need to be added as a member of this org to get that cla bot check to pass. You can add yourself by following the instructions at https://git.corp.adobe.com/OpenSourceAdvisoryBoard/handbook/blob/master/GitHub-Adobe-Org-Management.md#request-access-to-our-adobe-github-org. This way we can get your adobe ID to github ID mapping. This request will send an invitation to this org which you will have to accept. After acceptance, close and re-open the PR to trigger the check again (or just do a new commit in the pr)


Provide a changelog for 2.0.0

It would be good to learn about the relevant changes in version 2.0.0. It's not listed on the releases page and there is no CHANGELOG file. Thanks!

Thank you @ralfstx for bringing this up!  Since the release, I have been away from the grunt ecosystem and bogged down.  2.0.0 was developed due to the need for some more advanced operations that only work with child_process.spawn. It was a big change to switch from exec to spawn and despite testing, it has been reported to break some behavior with && (#79) in Linux but not Windows.  It should be pretty much identical on the surface but since the entire back-end was replaced and some exec behavior emulated, it was bumped as a major version change to prevent accidental wide-spread breakage.  Hopefully I can pull some time together to get the changelog and release completed.

FYI: I think this is the correct comparison from v1.0.1 to the 2.0.0 commit
• https://github.com/jharding/grunt-exec/compare/v1.0.1...e1ef5d9
Note: There's been 4 commits since 2.0.0 was tagged:
• https://github.com/jharding/grunt-exec/commits/master

I see that 3.0.0 was released about an hour ago, a changelog would be nice as well :)

Diffs for the two more recent majors (changelog is a bit more work and will unfortunately have to wait until I have more time):
v1.0.1 to v2.0.0
v2.0.0 to v3.0.0


CI: Add CI scripts

Create a basic set of CI scripts that just run the static checks for
now.
Fixes #101.
Signed-off-by: James O. D. Hunt<EMAIL_ADDRESS>

Once https://github.com/kata-containers/tests/pull/316 lands, we'll have the docs being auto-checked as well ;)

/me sees https://github.com/kata-containers/tests/pull/316 now ;-)


Requires prehistoric version of wysiwyg-rails (2.6.6)

Somehow installing this ruby SDK requires a very old version of your great wysiwyg-rails – 2.6.6.
Is this a hard requirement or did somebody forgot to update the Gemfile or the corresponding lock file?
Would be great to use the latest and greatest version of wysiwyg-rails together with the ruby SDK!

It should read:
gem.add_dependency 'wysiwyg-rails', '~> 2.6'

To allow for any release in the 2.x series.

Same problem over here. We are still using Froala Editor 2.6.6, while 2.8.4 is long since available.

Same here...

@Holist and others: there's a PR with a fix here: https://github.com/froala/wysiwyg-editor-ruby-sdk/pull/8. You can use my fork until it gets merged: gem 'froala-editor-sdk', '~> 1.2', git: 'https://github.com/kreintjes/wysiwyg-editor-ruby-sdk', branch: 'fix/loosen-locks'

Maybe ping @stefanneculai via email through their support system to get this hurried along? Are you paying customers?
I ended up not using this gem as it's actually quite easy to write your own interface, and more flexible too. There are a few other bugs too that need fixing (See my open issues).

Im a paying customer and opened a ticket on support for this today. thanks @kreintjes for this.

Thanks @stefanneculai :)


[cpptrace] Add v0.3.0

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

@jeremy-rifkin I had trouble using the new libassert port on Windows (VS 17.8) due to cpptrace compile errors - does this PR also fix that?

I'm trying to fix that right now. Cpptrace 0.3.0 has some API changes, hence the errors, but I'm not sure how to get vcpkg to use 0.2.1 for libassert.
I'm surprised you'd be seeing those errors though, since this PR hasn't been merged yet. Did you clone this branch locally or some such thing?

I'm trying to fix that right now. Cpptrace 0.3.0 has some API changes, hence the errors, but I'm not sure how to get vcpkg to use 0.2.1 for libassert.
I'm surprised you'd be seeing those errors though, since this PR hasn't been merged yet. Did you clone this branch locally or some such thing?

Error is using current HEAD of vcpkg. So the current cpptrace port seems to be broken as well.

@nickdademo Can you open an issue on the libassert repo? I'm able to install libassert from the current head.

Regarding this PR, I could use guidance with how to handle libassert needing a specific version of cpptrace.

Regarding this PR, I could use guidance with how to handle libassert needing a specific version of cpptrace.

You can't. The latest versions must integrate in vcpkg.

Well.... I can make a 1.2.2 release for libassert using cpptrace 0.3.0 (and delete the 1.2.1 port). I am somewhat concerned about any future API changes though.
I don't want to sound like I'm complaining but this seems like a pretty egregious shortcoming of a package manager. Out of curiosity is there a reason for not supporting this?

This is a restriction for ports. Basically it is the promise that you can install (any subset of) all ports at once. They are tested together, and they can't set conflicting version requirements.
OTOH users can ask for combining particular versions in manifest mode.
And of course packages can check the versions of dependencies during the build,
Don't discuss with me, I'm just explaining,,,

and delete the 1.2.1 port

... update the port to 1.2.2. Or add a patch.

Thanks for the explanation

A couple checks are still queued but I think I’ve sorted everything out. Thanks again for the help :)

I suggest adopting a more consistent naming convention for files. Specifically, the cmake configs are installed in a folder named libassert, yet the config names lack the lib prefix, which is also absent from the library name. This inconsistency makes it challenging to use the library directly by adding find_package(libassert REQUIRED) in CMakeLists.txt. At least, it happens with VC++2022.
Additionally, it might be beneficial to add a usage hint. For an example, please refer to this: https://github.com/microsoft/vcpkg/blob/master/ports/openal-soft/usage

@ndk Thank you and apologies for the inconvenience! This pr is for cpptrace but that stuff for libassert I think is already taken care of on the v2 dev branch. Another contributor helped me overhaul the cmake there and I'm hoping to release stuff in a couple weeks. A usage file is already specified for this port. If anything else comes up, please let me know and thank you for your interest in the library :) (It's probably best to open issues on the library repositories themselves)


Ignore replaces users feedback

Sending "ignore" as feedback replaces whatever feedback you had on MS.
Not a major issue, but if I'm the only person to send feedback, then I send an ignore, then there is a post with no obvious feedback.

See also #182 and #188.


HackRF seems to send datas but smartphone receive nothing.

Hi !
I need your help with my GPS Spoofing for my Engineering Student Project.
I downloaded the last ephemeris on the Nasa's ftp.
Then created the .bin with: .\gps-sdr-sim.exe -b 8 -e .\brdc0510.20n -l -41.300988,174.775321,100
Then I sent the file to the HackRF with: .\hackrf_transfer.exe -t gpssim.bin -f<PHONE_NUMBER> -s 2600000 -a 1 -x 0
In the console I have this:
call hackrf_set_sample_rate(2600000 Hz/2.600 MHz)
call hackrf_set_hw_sync_mode(0)
call hackrf_set_freq(1575420000 Hz/1575.420 MHz)
call hackrf_set_amp_enable(1)
Stop with Ctrl-C
5.2 MiB / 1.002 sec =  5.2 MiB/second
5.0 MiB / 1.002 sec =  5.0 MiB/second
5.2 MiB / 1.003 sec =  5.2 MiB/second
5.2 MiB / 1.003 sec =  5.2 MiB/second
5.2 MiB / 1.003 sec =  5.2 MiB/second
My smartphone received nothing even if I remove the Wifi Scanning & Perfect Location
I also tried to scan the 1575,420 MHZ with GQRX and I had only noise.
Do you have an idea ?

Same here. Nothing :(

Hello, I posted #240 that might help you.

you, can try using a tiny TCXO (external oscillator). It worked for me to resolve no signal reception...atleast for GPS receivers. The issue of few GNSS receivers in android mobile receiving and still not accepting the spoofed signals, is yet to be resolved.


Add section on oracles

This section is intended to introduce the idea of oracles, so that it doesn't have to keep being explained in the algorithm chapter.
It gives a brief intro to different types of oracle, and uncomputation.
It also has different registers. Which is probably the first time this is used in the textbook (since the prereqs).

Thanks!
I fixed the issues. As you may have noticed, the 'b' was broken on my keyboard yesterday!

Oh no!
Your welcome. :)

----- Original message -----From: James Wootton<EMAIL_ADDRESS>Qiskit/qiskit-textbook<EMAIL_ADDRESS>Hwajung Kang<EMAIL_ADDRESS>Review requested<EMAIL_ADDRESS>[EXTERNAL] Re: [Qiskit/qiskit-textbook] Add section on oracles (#439)Date: Wed, Jun 10, 2020 5:24 AM

Thanks!
I fixed the issues. As you may have noticed, the 'b' was broken on my keyboard yesterday!
—You are receiving this because your review was requested.Reply to this email directly, view it on GitHub, or unsubscribe.

It looks like Travis is stuck.. Admin like @aasfaw might have to force it...

IMO we should leave out phase oracles here, or at least move it to the end of the notebook. I think a common stumbling block is that the promise (in DJ / BV etc.) that the quantum algorithm outperforms the classical algorithm doesn't make sense when you use a phase oracle since it cannot be implemented on a classical computer. I think using classical oracles exclusively would help with this.
My personal preference would have phase oracles only be mentioned after DJ / BV, and be presented as an optimisation of the classical oracle / kickback method (and show they have the same effect). Would be good to know what everyone thinks about this.

You can make a phase oracle out of a Boolean oracle by with an ancilla and a Hadamard and stuff. So maybe we could introduce phase oracles as a special case.

I added some details on why phase oracles are no less weird and quantum than boolean ones.

I don't mind that the concept of phase oracle is introduced before Ch.3 Algorithm section starts...

When I read the textbook first time, I spent some time to figure out how the BV circuit, which used a phase oracle instead of kickback method,( but just noticed that it's been changed, it's not true anymore, though) and thought that it would have been helpful if some explanation was there for that way the oracle was made.

Now I see that the chapter does not use a phase oracle anymore but phase oracle way appears in Grover as well.

So, with the explanation that a phase oracle and the kickback method have the same effect ( which James Wootton just did, I guess ), I think it is good to be mentioned before Algorithm starts.

----- Original message -----From: Frank Harkins<EMAIL_ADDRESS>Qiskit/qiskit-textbook<EMAIL_ADDRESS>Hwajung Kang<EMAIL_ADDRESS>Review requested<EMAIL_ADDRESS>[EXTERNAL] Re: [Qiskit/qiskit-textbook] Add section on oracles (#439)Date: Thu, Jun 11, 2020 9:03 AM

IMO we should leave out phase oracles here, or at least move it to the end of the notebook. I think a common stumbling block is that the promise (in DJ / BV etc.) that the quantum algorithm outperforms the classical algorithm doesn't make sense when you use a phase oracle since it cannot be implemented on a classical computer. I think using classical oracles exclusively would help with this.
My personal preference would have phase oracles only be mentioned after DJ / BV, and be presented as an optimisation of the classical oracle / kickback method (and show they have the same effect). Would be good to know what everyone thinks about this.
—You are receiving this because your review was requested.Reply to this email directly, view it on GitHub, or unsubscribe.

The new version fixed the typo mentions above, and also explicitly points out that the effect is phase kickback.

closes #338


[autoscaler] Fix AWS Autoscaler Tests

Why are these changes needed?
Fix broken AWS Autoscaler CloudWatch unit test stubs and remove broken/outdated EC2 security group and network interface tests.
Checks

[X] I've run scripts/format.sh to lint the changes in this PR.
[X] I've included any doc changes needed.
[X] I've made sure the tests are passing. Note that there might be a few flaky tests.
Testing Strategy

[X] Unit tests
[ ] Release tests
[ ] This PR is not tested :(

License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Code changes LGTM.
p.s.: I noticed several task failures in Travis CI. Some of them can also be seen in Ray's official repo's PRs (e.g., PYTHON=3.6 SMALL_AND_LARGE_TESTS=1) whereas some others are not commonly seen (e.g., LINT=1). After reading the understanding-ci-test-jobs section it doesn't seem to be blocking issue since this PR is within the scope of aws autoscaler.

Code changes LGTM.
p.s.: I noticed several task failures in Travis CI. Some of them can also be seen in Ray's official repo's PRs (e.g., PYTHON=3.6 SMALL_AND_LARGE_TESTS=1) whereas some others are not commonly seen (e.g., LINT=1). After reading the understanding-ci-test-jobs section it doesn't seem to be blocking issue since this PR is within the scope of aws autoscaler.

I noticed the same - it doesn't look like our changes have introduced any new failures, but it would be nice to at least get the LINT job working for all branches in the repo going forward.


Don’t hardcode namespace

Signed-off-by: David Gageot<EMAIL_ADDRESS>

Sure! I was just making this part of the code look like the rest


Shared interface between BaseItem and Factory

I see with 5.0.0 that there is separation between the base item and the factory class. Just looking at the release changelog, I see that the type value is used in both classes. As far as I'm aware, this either should not exist in factory, since it's associated per item type and not per layout type, or it should only be defined once. In the case that the factory does share methods/values used in items, you can instead make the item take the factory in its constructor, and delegate a common interface to the factory.
For example:
https://github.com/AllanWang/KAU/blob/cfac5f0db6527aace725de14f07b19ba85b5e07e/sample/src/main/kotlin/ca/allanwang/kau/sample/PermissionCheckbox.kt#L62-L109
Where both classes implement BindingLayout

@AllanWang the main idea was to do this for the case of the paged list whereas we do not when filling which items there may all pop up. and in that case we won't be able to have an item instance to generate the view for.
maybe we need allow for the factory cache to be a map with the type as key and then the factory as value?
I really want to ensure we do not have to create a item instance just for being able to construct the viewholder

I guess my question is, why do factories have types? Factories seem associated to layouts. If the type is necessary because you imagine that different factories may have the same layout file but different view holders, then why define the type again in items? Why not rely on factory.type?

Hmmm I suppose the main reason was due to the full disconnect between factory and item. and so I could match them together. at least for the factory cache. but I suppose it is better to remove it from the factory and only have it in the item.

removed the type from the factory


Don't shorten the description (as much), include ellipsis when you do

While Mastodon has a relatively low character limit by default, this isn't applied to externally-received posts, so you can show (much) more text there. When you do shorten it, it would be helpful to have an indicator like […] to make it clear the text isn't just cut off.
Longer posts wouldn't be annoying to receive since fediverse software is generally very good at collapsing long posts into a "read more" fadeout automatically. (The current post length is enough to trigger this in the mobile app.)
Mastodon will show media attachments outside of this collapsed section, so no UX breakdown in that regard either.

One reason the current limit concerns me is that it cuts off the funding links of a podcast I follow.
(It doesn't have the necessary metadata to display the "FUNDING" profile attachment, as it's hosted on Spotify.)

 Some podcasts have really long episode descriptions. By default, the Podcast Index returns a truncated description. There is a way to get the full description but it currently isn't used by the ap.podcastindex.org bridge.
If you are interested in seeing the code for the bridge, see https://github.com/Podcastindex-org/pi-activitypub-server

If you are interested in seeing the code for the bridge, see https://github.com/Podcastindex-org/pi-activitypub-server

Thank you, I was looking for that but couldn't find it earlier.
I'll file any issues in the appropriate repository from now on 👍
I'll close this and file it over there a bit later when I get around to it.


fix: sheet getRowVisible lags after filter

close https://github.com/dream-num/univer/issues/3369

Pull Request Checklist

[ ] Related tickets or issues have been linked in the PR description (or missing issue).
[ ] Naming convention is followed (do please check it especially when you created new plugins, commands and resources).
[ ] Unit tests have been added for the changes (if applicable).
[ ] Breaking changes have been documented (or no breaking changes introduced in this PR).

清楚筛选后内渲染不出来，无法滑动表格，路径：1、选中整个表格设置筛选，2、仅筛选任意项，3、拖拽冻结线冻结行、4、清楚筛选条件
2024_9_11 15_37_45 video.webm
2024_9_11 15_40_03 snapshot.json


Migration fails with InvalidBasesError

I'm trying to implement djangocms-blog but when I try to run the migrations the Taggit migration fails like this:
Applying taggit.0001_initial...Traceback (most recent call last):
  File "./manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/Library/Python/2.7/site-packages/django/core/management/__init__.py", line 385, in execute_from_command_line
    utility.execute()
  File "/Library/Python/2.7/site-packages/django/core/management/__init__.py", line 377, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/Library/Python/2.7/site-packages/django/core/management/base.py", line 288, in run_from_argv
    self.execute(*args, **options.__dict__)
  File "/Library/Python/2.7/site-packages/django/core/management/base.py", line 338, in execute
    output = self.handle(*args, **options)
  File "/Library/Python/2.7/site-packages/django/core/management/commands/migrate.py", line 161, in handle
    executor.migrate(targets, plan, fake=options.get("fake", False))
  File "/Library/Python/2.7/site-packages/django/db/migrations/executor.py", line 68, in migrate
    self.apply_migration(migration, fake=fake)
  File "/Library/Python/2.7/site-packages/django/db/migrations/executor.py", line 96, in apply_migration
    if self.detect_soft_applied(migration):
  File "/Library/Python/2.7/site-packages/django/db/migrations/executor.py", line 140, in detect_soft_applied
    apps = project_state.render()
  File "/Library/Python/2.7/site-packages/django/db/migrations/state.py", line 75, in render
    "for more" % new_unrendered_models
django.db.migrations.state.InvalidBasesError: Cannot resolve bases for [<ModelState: 'cms.PageUser'>, <ModelState: 'cms.PageUserGroup'>]
This can happen if you are inheriting models from an app with migrations (e.g. contrib.auth)
 in an app with no migrations; see https://docs.djangoproject.com/en/1.7/topics/migrations/#dependencies for more
File "./manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/Library/Python/2.7/site-packages/django/core/management/__init__.py", line 385, in execute_from_command_line
    utility.execute()
  File "/Library/Python/2.7/site-packages/django/core/management/__init__.py", line 377, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/Library/Python/2.7/site-packages/django/core/management/base.py", line 288, in run_from_argv
    self.execute(*args, **options.__dict__)
  File "/Library/Python/2.7/site-packages/django/core/management/base.py", line 338, in execute
    output = self.handle(*args, **options)
  File "/Library/Python/2.7/site-packages/django/core/management/commands/migrate.py", line 161, in handle
    executor.migrate(targets, plan, fake=options.get("fake", False))
  File "/Library/Python/2.7/site-packages/django/db/migrations/executor.py", line 68, in migrate
    self.apply_migration(migration, fake=fake)
  File "/Library/Python/2.7/site-packages/django/db/migrations/executor.py", line 96, in apply_migration
    if self.detect_soft_applied(migration):
  File "/Library/Python/2.7/site-packages/django/db/migrations/executor.py", line 140, in detect_soft_applied
    apps = project_state.render()
  File "/Library/Python/2.7/site-packages/django/db/migrations/state.py", line 75, in render
    "for more" % new_unrendered_models
django.db.migrations.state.InvalidBasesError: Cannot resolve bases for [<ModelState: 'cms.PageUser'>, <ModelState: 'cms.PageUserGroup'>]
This can happen if you are inheriting models from an app with migrations (e.g. contrib.auth)
 in an app with no migrations; see https://docs.djangoproject.com/en/1.7/topics/migrations/#dependencies for more
File "./manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/Library/Python/2.7/site-packages/django/core/management/__init__.py", line 385, in execute_from_command_line
    utility.execute()
  File "/Library/Python/2.7/site-packages/django/core/management/__init__.py", line 377, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/Library/Python/2.7/site-packages/django/core/management/base.py", line 288, in run_from_argv
    self.execute(*args, **options.__dict__)
  File "/Library/Python/2.7/site-packages/django/core/management/base.py", line 338, in execute
    output = self.handle(*args, **options)
  File "/Library/Python/2.7/site-packages/django/core/management/commands/migrate.py", line 161, in handle
    executor.migrate(targets, plan, fake=options.get("fake", False))
  File "/Library/Python/2.7/site-packages/django/db/migrations/executor.py", line 68, in migrate
    self.apply_migration(migration, fake=fake)
  File "/Library/Python/2.7/site-packages/django/db/migrations/executor.py", line 96, in apply_migration
    if self.detect_soft_applied(migration):
  File "/Library/Python/2.7/site-packages/django/db/migrations/executor.py", line 140, in detect_soft_applied
    apps = project_state.render()
  File "/Library/Python/2.7/site-packages/django/db/migrations/state.py", line 75, in render
    "for more" % new_unrendered_models
django.db.migrations.state.InvalidBasesError: Cannot resolve bases for [<ModelState: 'cms.PageUser'>, <ModelState: 'cms.PageUserGroup'>]
This can happen if you are inheriting models from an app with migrations (e.g. contrib.auth)
 in an app with no migrations; see https://docs.djangoproject.com/en/1.7/topics/migrations/#dependencies for more
File "./manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/Library/Python/2.7/site-packages/django/core/management/__init__.py", line 385, in execute_from_command_line
    utility.execute()
  File "/Library/Python/2.7/site-packages/django/core/management/__init__.py", line 377, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/Library/Python/2.7/site-packages/django/core/management/base.py", line 288, in run_from_argv
    self.execute(*args, **options.__dict__)
  File "/Library/Python/2.7/site-packages/django/core/management/base.py", line 338, in execute
    output = self.handle(*args, **options)
  File "/Library/Python/2.7/site-packages/django/core/management/commands/migrate.py", line 161, in handle
    executor.migrate(targets, plan, fake=options.get("fake", False))
  File "/Library/Python/2.7/site-packages/django/db/migrations/executor.py", line 68, in migrate
    self.apply_migration(migration, fake=fake)
  File "/Library/Python/2.7/site-packages/django/db/migrations/executor.py", line 96, in apply_migration
    if self.detect_soft_applied(migration):
  File "/Library/Python/2.7/site-packages/django/db/migrations/executor.py", line 140, in detect_soft_applied
    apps = project_state.render()
  File "/Library/Python/2.7/site-packages/django/db/migrations/state.py", line 75, in render
    "for more" % new_unrendered_models
django.db.migrations.state.InvalidBasesError: Cannot resolve bases for [<ModelState: 'cms.PageUser'>, <ModelState: 'cms.PageUserGroup'>]
This can happen if you are inheriting models from an app with migrations (e.g. contrib.auth)
 in an app with no migrations; see https://docs.djangoproject.com/en/1.7/topics/migrations/#dependencies for more

What can I do to fix it and get it working?

django.db.migrations.state.InvalidBasesError: Cannot resolve bases for [<ModelState: 'cms.PageUser'>, <ModelState: 'cms.PageUserGroup'>]
This can happen if you are inheriting models from an app with migrations (e.g. contrib.auth)
 in an app with no migrations; see https://docs.djangoproject.com/en/1.7/topics/migrations/#dependencies for more

This doesn't look related to taggit. Also, you're using Django 1.7 which is no longer supported by the DSF. I recommend upgrading.


Service Name with "-" should not be allowed . docker-compose validation for service name fails

Server version - V0.16.3
Steps to reproduce the problem:
Create Environment.
Create a service with name "test-366326" and activate service.
Use "Export Config" for Environment to generate the docker-compose.yml
Use this docker-compose.yml file to create the instances.
Following error is seen:
"Invalid service name "test-366326" - only [a-zA-Z0-9] are allowed"

@alena1108 I think this is resolved in rancherio/cattle@0a5ecaaf93db996c911430153d0682a372d3d65a ?

@vincent99 yes, it was, marking as resolved

Tested with server version - V.0.17.0
When adding service , we are not allowed to have "-" or another special characters in name.
Following error presented to user - "Validation failed: name"


[MODULE] - Bing news search engine integration

Please describe the module you would like to add to bricks
I want to use Bing in refinery.
Do you already have an implementation?
Additional context
Requires paid API key (premium function)

Implemented with #211


Rescheduling problems / execution problems

A user is experiencing the following problems -- also see complete screenshot.

Instead of doing the message immediately, she wanted to reschedule to another moment. Sam asked when (four options), after trying to choose option 2 -- sam kept saying that he did not understand. So she could not continue. Also everytime she gave the response '2' and Sam didn't understand, the four prompts come (press help if ..., press exercise if ...) -- this was considered annoying. Maybe we can not show these four prompts when an individual is in a dialogue?

Later Sam asked if she wanted to continue the dialogue (in between the four prompts). Nothing is said: but Sam later responds 'you finished a part of the preparation phase, good job'

A new dialogue is initiated by Sam, and the user wanted to do it now. The dialogue hower did not continue. She only got reminders that Sam hadn't heard from her in a while. Sam did not respond to her 'verder' reply. Currently, she can therefore not continue. She is stuck in the dialogue.

I think another user (Kimberley Lucas; e-mailadres probably something like that) had something similar. She wanted to do an exercise now, but nothing happened; see screenshot

And something similar that I encountered myself yesterday, when nothing happened when I typed verder


Provide packaged version for major user OS: Linux, Windows, Mac

For systems without network access, users need a pre-packaged version.
one can consider:
https://github.com/pmq20/ruby-packer

I have successfully tested ruby-packer, but generating and supporting architecture dependant packages is a certain effort. while leaving users with the choice of ruby is more generic.


Weird issue about button with icon right

native-base: 2.3.3
react-native: 0.50.4
If there's 1 button, it works normal. But if there're 2 buttons with 1 iconLeft and 1 iconRight, then the UI break

Here is my code:
class Home extends Component {
  render() {
    return (
      <View style={styles.container}>
        <Button iconLeft success>
          <Icon name='arrow-back' />
          <Text>Back</Text>
        </Button>

        <Button iconRight success style={{marginLeft: 10}}>
          <Text>Next</Text>
          <Icon name='arrow-forward' />
        </Button>
      </View>
    )
  }
}

@maphongba008 just simply wrap all your components in <Container/>. Read about NativeBase screen structure here
Gif

@akhil-geekyants  thanks mate, didn't know that I must follow NativeBase structure

@maphongba008 Yes, thats mentioned in the Docs at very beginning section of Components


spec: implement rm_file() for single file deletion

This is going in, thanks!
I wonder, when do you find it necessary to call rm_file rather than rm (which would call _bulk delete with one path) ?

I wonder, when do you find it necessary to call rm_file rather than rm (which would call _bulk delete with one path) ?

DVC always uses rm_file whether deleting a single file or multiple files, and for some remotes it is more optimized than rm() since it is doing a single job and not trying to expand path / check whether it is a directory etc. And s3fs was the only one which didn't support rm_file among the stuff we migrated so far so needed that. But yes, bulk deleting is a potential optimization target for me to use when we do dvc gc, but later ofc


Rename PayForMessage to PayForData

As of https://github.com/celestiaorg/big-dipper-2.0-cosmos/commit/de82d67cc7a0fdb66ba1e63e7ab8be7903372043, we had support for PayForMessage. However, in https://github.com/celestiaorg/celestia-app/pull/371, that was renamed to PayForData. The explorer also needs to support this change.

done


Update index.html and style.css

Created a "features" section on BRL-CAD's main website page

This looks much better, thank you!

unclear how missed, but appears to have dead image links?

Sorry Sir,
Actually the issue is that I have not added the new images in the img file and also not created the pull request of this file

Due to an error updating the main repository this pull request was inadvertently closed and will need to be resubmitted.


Where could I find the example for key generator code?

I use raw java code,
 KeyGeneratorConfiguration keyGeneratorConfiguration = new   KeyGeneratorConfiguration("SNOWFLAKE", "goods_id", new Properties());  tableRuleConfiguration.setKeyGeneratorConfig(keyGeneratorConfiguration);
and I use jpa to insert data in the for loop;
`
    for (int i = 1; i <= 40; i++) {
        Goods goods = new Goods();
        goods.setGoodsName("shangpin" + i);
        goods.setGoodsType((long) (i + 1));
        long goodsId = goodsRepository.save(goods).getGoodsId();
        System.out.println("goods id:" + goodsId);
    }

`
and My database is db_01 and db_02, the table name is goods_0 and goods_1.
The entity code is
`
@Entity
@Table(name = "goods")
public class Goods {
@Id
@GeneratedValue(strategy = GenerationType.IDENTITY)
@Column(name = "goods_id")
private Long goodsId;

private String goodsName;

`
But when i call the insert method, I catch the exception
A different object with the same identifier value was already associated with the session
Do I miss some important configuration?

Addition some extra message
I use Spring JPA under SpringBoot 2.0+, and the sharding jdbc version is 4.0.0-RC1

Please refer to the configurations in JPA-EXAMPLE
@wowojyc

Thank you for your reply
I have read the example code and In fact the code runs well.
However my scene has a bit of difference.
In my application, I need dynamic shard rule, so I must use raw java code instead of spring boot application.properties.
I have several questions.

If I want use distributed primary key generated by sharding jdbc, how the table column like in database schema? Is it necessary set 'primary key ' or Auto increment attributes?
How the JPA entity code like ?
@Id or @GeneratorValue is also necessary? If the answer is yes, which strategy is ok

If I want use distributed primary key generated by sharding jdbc, how the table column like in database schema? Is it necessary set 'primary key ' or Auto increment attributes?
If you want to use use distributed primary key generated by sharding jdbc, setting 'primary key ' or Auto increment  is not necessary.

How the JPA entity code like ?
@id or @GeneratorValue is also necessary?
f you want to use use distributed primary key generated by sharding jdbc,  @id or @GeneratorValue is also not necessary.

The conclusion is that id does not matter if I set primary key with auto increment property and use @id and @GeneratorValue annotations. Am I right?

I use hibernate annotation @Entity, it requires me to specify identifier and I must add it to one property.

Add some extra message.
The application runs well if I remove below setting in the application.properteis
spring.jpa.properties.hibernate.temp.use_jdbc_metadata_defaults=false
I add this setting because of some hibernate exceptions in the log, but I don't know why this will affect the distribute primary key.
The springboot version is 2.0+

@wowojyc  Is your program ok now?
When you use the key generator of ShardingSphere, please do not make Springboot generate key. By set the strategy for @GeneratedValue(strategy = GenerationType.IDENTITY) is ok, if @Id is necessary.
If it's possible, can your give the detail of error logs?

At first.
I meet the exception like this.
spring jpa - a different object with the same identifier value was already associated with the session
And I found the hibernate execute "select last_insert_id()' after my normal insert execute successfully.
After some try, I check my application.properties again.
According to the 解决hibernate执行select last_insert_id问题, I removed the setting and it does work.
The reason why I add this setting is that my database is postgresql and its jdbc connection does not implement createBlob interface.

So your program runs well now? Can you close this issue, if there is no problem?

Yes, but the reason I do not understand clearly, can you give some advice?


javascript error: webpackChunkbuild is not defined

OpenQA.Selenium.WebDriverException: 'javascript error: webpackChunkbuild is not defined
(Session info: chrome=91.0.4472.124)'

same here

I also have the same problem
"javascript error : webpackcunkbuild is not defined"

change "webpackChunkbuild" to "webpackChunkwhatsapp_web_client"

webpackChunkbuild

where should it be changed?

change "webpackChunkbuild" to "webpackChunkwhatsapp_web_client"
It will fix everything.
Yes it is working..

I request you to please change in Wapi.js of this github..

change "webpackChunkbuild" to "webpackChunkwhatsapp_web_client"
It will fix everything.

Thanks a lot! How do you figure it out this fixes? maybe some day other developers could help to fix new issues.

change "webpackChunkbuild" to "webpackChunkwhatsapp_web_client"
It will fix everything.

Thanks a lot! How do you figure it out this fixes? maybe some day other developers could help to fix new issues.

+1 , Help us how to figure these out early.. thanks

This issue solved..

change "webpackChunkbuild" to "webpackChunkwhatsapp_web_client"
It will fix everything.

tysm !


Adding Note Taking Feature for Zoom Meeting Integration

New Features:

To view deployment visit : https://zoomplugin.vercel.app/

Note taking for individual users link

Hello World AI model link

Installation

Clone this github repository to your local and cd to it
Run following command to install dependencies and run development server :

npm install

Setup .env.local file in following format:

DATABASE_URL = mongoDB-URI
NEXTAUTH_SECRET = ""
SDK_KEY = zoom_web_sdk_key
SDK_SECRET = zoom_web_secret

Run following command to generate NEXTAUTH_SECRET hash

openssl rand -base64 32

Run following command to run development server :

npm run dev

Open http://localhost:3000 with your browser to see the app.

Hi @Shaomei @blickly, I have created a pull request for my latest code. Can you please review it?

Thank you Harshank! I was a bit underwater these days for other tasks and
will be flying out today. But Ben and I will find some time for a review
before we head to the airport!
Can you help fill up the tech evaluation for stage 1 ideas
https://docs.google.com/spreadsheets/d/1XL_2VL9zUczrUEZbS2oub75j23yJqQwc7gIjq8GYXls/edit?usp=sharing?
I
don't think I have time to go over it with you today, but I completely
trust your judgement :)
Best,
-Shaomei
On Mon, Sep 11, 2023 at 12:35 PM Harshank @.***> wrote:

Hi @Shaomei https://github.com/Shaomei @blickly
https://github.com/blickly, I have created a pull request for my latest
code. Can you please review it?
—
Reply to this email directly, view it on GitHub
https://github.com/aimpowered/zoomplugin/pull/5#issuecomment-1714466359,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABRKP2JF5VUPJG5TMFQGF3XZ5RYXANCNFSM6AAAAAA4TZ7SAE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thank you Harshank! I was a bit underwater these days for other tasks and will be flying out today. But Ben and I will find some time for a review before we head to the airport! Can you help fill up the tech evaluation for stage 1 ideas https://docs.google.com/spreadsheets/d/1XL_2VL9zUczrUEZbS2oub75j23yJqQwc7gIjq8GYXls/edit?usp=sharing? I don't think I have time to go over it with you today, but I completely trust your judgement :) Best,
…
-Shaomei
On Mon, Sep 11, 2023 at 12:35 PM Harshank @.> wrote: Hi @Shaomei https://github.com/Shaomei @blickly https://github.com/blickly, I have created a pull request for my latest code. Can you please review it? — Reply to this email directly, view it on GitHub <#5 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABRKP2JF5VUPJG5TMFQGF3XZ5RYXANCNFSM6AAAAAA4TZ7SAE . You are receiving this because you were mentioned.Message ID: @.>

Hi Shaomei,
I have already evaluated the top 10 ideas in the sheet. Do you want me to evaluate the rest of the ideas as well?

Yes please!
On Wed, Sep 13, 2023 at 3:26 PM Harshank @.***> wrote:

Thank you Harshank! I was a bit underwater these days for other tasks and
will be flying out today. But Ben and I will find some time for a review
before we head to the airport! Can you help fill up the tech evaluation for
stage 1 ideas
https://docs.google.com/spreadsheets/d/1XL_2VL9zUczrUEZbS2oub75j23yJqQwc7gIjq8GYXls/edit?usp=sharing?
I don't think I have time to go over it with you today, but I completely
trust your judgement :) Best,
… <#m_3757622414438697141_>
-Shaomei
On Mon, Sep 11, 2023 at 12:35 PM Harshank @.> wrote: Hi @Shaomei
https://github.com/Shaomei https://github.com/Shaomei
https://github.com/Shaomei @blickly https://github.com/blickly
https://github.com/blickly https://github.com/blickly, I have created a
pull request for my latest code. Can you please review it? — Reply to this
email directly, view it on GitHub <#5 (comment)
https://github.com/aimpowered/zoomplugin/pull/5#issuecomment-1714466359>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABRKP2JF5VUPJG5TMFQGF3XZ5RYXANCNFSM6AAAAAA4TZ7SAE
https://github.com/notifications/unsubscribe-auth/AABRKP2JF5VUPJG5TMFQGF3XZ5RYXANCNFSM6AAAAAA4TZ7SAE
. You are receiving this because you were mentioned.Message ID: @.>
Hi Shaomei,
I have already evaluated the top 10 ideas in the sheet. Do you want me to
evaluate the rest of the ideas as well?
—
Reply to this email directly, view it on GitHub
https://github.com/aimpowered/zoomplugin/pull/5#issuecomment-1718394168,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABRKP272J2ARHW6QU5RDB3X2IXIHANCNFSM6AAAAAA4TZ7SAE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Closing this PR and will be creating a new one for High Fidelity prototype.


meuzic

This app is great. I think it would be a perfect fit and I would love to talk.
We made a full featured store for pwa's.

We do the hosting - all apps get their own subdomain.
We have a cloud save.
We have the universal login - SSO.
We handle app reviews.
We handle monetization - including purchases and various ad based rewards.
We manually approve each app & update - keeping the apps high quality and maintaining our trust with users.
We provide a real time dashboard for installs, active users, and revenue.
We are USA based.

And it’s all free* for the developers.
Please email me at<EMAIL_ADDRESS>I would love to chat.
If you want to check it out first, go here: https://wickeyappstore.com/app/developer-portal


OIDC crashing because of CSP

Hello,
using the latest commit on branch release I am getting the following error:
Refused to connect to 'https://login.microsoftonline.com/{tenant-id}/.well-known/openid-configuration' because it violates the following Content Security Policy directive: "default-src 'self'". Note that 'connect-src' was not explicitly set, so 'default-src' is used as a fallback.
However with the same setup, when I am using commit 983176a92ecd7c734befb23d739bff7934260675
it works perfectly fine.
Anybody has any idea as to why this is happening? If so how can I fix this?
I understand that it is because of the new added header (CSP) and it is to prevent cross site scripting but how can I fix this problem?

If the JS is running on your page from your server, then you're the one that control the CSP setting.

Currently i am using the csp chrome extension to work around this issue.
I think we can close this issue.


Allow changing a SourceControlResourceGroup's label

Currently, it's not possible to change a SourceControlResourceGroup's label once it's created. From vscode.d.ts:
export interface SourceControlResourceGroup {
		readonly id: string;
		readonlly label: string;
                .......

In Perforce, a changelist has a description which can be edited by the user. To refresh that one changelist's description, an extension would have to do changelist.dispose() the existing changelist, and call SourceControl.createResourceGroup() again with the new description. This has a major side effect, because resource groups are re-arranged in the scm viewlet once an existing one is dispose()'d.
This PR makes it possible to change a SourceControlResourceGroup's without having to call dispose() and createResourceGroup().

@joaomoreno


Why using GLOBAL in source code?

What does it refer to?
If it is useful, I suggest to  use typeof GLOBAL === 'undefined' to check if it exists, because it causes error when using strict mode

It is used when testing in Node.js. It refers to the global object.
I replaced it with a this keyword.
Let me know if it works.

It works now, thanks!
Have widely used in https://github.com/foray1010/Popup-my-Bookmarks/tree/deku

I'm glad that it's useful. I downloaded the extension. Never had one for bookmarks.
I like that it does not use memory when not being used.


make IsBadCharacter func static

@dave3d Just wondering, is there a specific compilation or linker problem that could be solved by the static keyword that you're proposing here?
In general, I think it should be sufficient to place such a local helper function inside an unnamed (anonymous) namespace. Right?
As suggested by C++ Core Guidelines "Use an unnamed (anonymous) namespace for all internal/non-exported entities", https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#Rs-unnamed2
It seems to me that IsBadCharacter is already inside an unnamed namespace.


Get products sorted (manual way) from a collection

Went through all parts of code, searched a lot… I'm totally unable to understand how to retrieve products sorted while querying from a smart collection.
Is this available with specific parameters ?

@benbonnet Did you find a solution? I have the same problem, but in my case I have custom_collections and I need to retrieve their products exactly how I defined them in the panel (manually).

Unfortunately it still seems to be unavailable. We decided to rely on tags (we have an external app the handle our categories, and tags are assigned automatically when they're added to a collection)
It is very far from being ok in that specific context, but product queries by tags are reliable so far

@benbonnet thank you! It's a nice idea for a work around.

@benbonnet I’d like to clarify — are you having this problem with the API in general (regardless of client), or with this gem in particular?

@nwtn I tried only with the Gem. There is any way to see the URL generated for each resource before firing the request to the API?

@nwtn since the issue opened, I can say that we got in depth with the shopify api. The gem is okay if you already know very well the shopify api - it'll be easy to guess how to use the available options.
To be honest we've ended up calling the api directly with rest-client (what you're providing here under the hood), it helped us learn how shopify works

There is any way to see the URL generated for each resource before firing the request to the API?

I haven’t personally tried this, but it should be possible to use collection_path from ActiveResource (which this gem uses under the hood) to see the URLs that will be used. ActiveResource also supports overriding validations, which you could probably use to take a look at the URL.

I'd say it would be advisable to know first the rest api (going though the official doc). The logic (syntax) behind this gem then makes sense. But as an entrypoint, it's been a bit misleading for us

@benbonnet that’s great feedback, thanks. We’ll think about how we can make that expectation clearer.


Problem with casting date from custom field to integer

Hello,
I want to create computed custom field that calculates difference in days between current date and date from my custom field.
I written something like that but it doesn't work.
I don't know Ruby I just googled that function to_i change Date format to Integer.
(Time.now - cfs[21].to_i)
When i try to save issue I received error: 'undefined method `to_i' for Sun, 18 Jun 2017:Date Did you mean? to_time to_s'
When I tried other 2 methods mentions in comunicate I receive other error 'Formuła undefined method `to_time' for nil:NilClass Did you mean? to_i' while saving computed field.
Is there any way using this plugin to achieve effect that I want?

When I tried something like that:
(Date.today - cfs[21]).to_i
I received error 'Formuła expected numeric'
even if field format is Integer or Float.

Hi, try this (Date.today - cfs[21]).to_i if cfs[21].present?

Does it help?

Yes, it's working :D Thank you.

Good morning, before the update, I used the following formula to calculate the number of days elapsed from a certain date to today, you can tell me what it would be now given that it does not work for me, thank you very much.
Number = (Date.current-Date.parse ('% {cf_60}')) / 365
Sprintf ('%. 2f', number)

Hi, replace '%{cf_60}' by cfs[60]
number = (Date.current-Date.parse(cfs[60])) / 365
sprintf('%.2f', number)

Gracis Yakov, this was the first step and I did not get good results.
Copy the complete original formula thinking it could help you understand the problem.
Anyway I have it as follows:
(((Date.today-cfs [63]). To_i if cfs [63] .present?). To_f / 365) .round (2)
Applying what you used in previous comments.
The problem that is presented to me now is when using the result of previous calculation:
If cfs [65]> = 4
"Black"
Else
       If cfs [65]> = 2
       "Red"
       Else
             If cfs [65]> = 1.5
             "Yellow"
              Else
                    "Green"
             End
        End
End
I get the following error:
Error applying formula to field "Calculated Color" - undefined method `> = 'for nil: NilClass
Thank you very much for your predisposition and help

case cfs[65]
  when 1.5...2
    'Yellow'
  when 2...4
    'Red'
  when 4..Float::INFINITY
    'Black'
  else
    'Green'
end

Thank you Yakov, perfect, excellent.


Deleting or renaming a folder doesn't result in talking to userver

Describe the bug
When I delete a file, the errors for the file go away. When I delete the folder, the errors remain. This can be seen in the problems panel or in the red names in the file explorer.
Steps to replicate the bug

Open a project with auto-analysis
Create a folder
Create a file in that folder
Add a syntax error
Delete the folder
The error still remains in the problems panel and the parent directories are still red from the error

The issue seems to be with the client because even adding all the "workspace" server capabilities didn't help.
See https://github.com/microsoft/vscode/issues/60813

Fixed the client with 62eef6471b73fcf507951f1268cd600112769bbe

Fixed the server with future release 1214


AUR: Mess-up with jsoncpp

Trying to build polybar-git off AUR, it wanted me to upgrade jsoncpp - I had to downgrade it for some reason - to 1.8.4-2, what I did.
The build succeeded, and I tried running polybar, which then spit out to me that it couldn't find libjsoncpp.so.19, which is the shared library issued by jsoncpp-1.8.4-1.
Downgrading jsoncpp back to 1.8.4-1 fixes the issue and polybar runs flawlessly.
I don't have the error message anymore, but it was the common "couldn't load shared library" error, with the file I mentioned earlier.

jsoncpp-1.8.4-2 uses libjsoncpp.so.20 for me. Did you compile polybar after upgrading jsoncpp?

See Known-Issues. As @NBonaparte mentioned, you will need to recompile polybar.


feat(gnss_poser): options for gnss pose pub method

Description
Adding feature reffered: #4188
This PR offers three options for publishing gnss_pose:

Publish coming data immediately (buffer_epoch does not have effect on algorithm).
Publish median of the buffer (buffer_epoch setted by user).
Publish average value of the poses in the buffer (buffer_epoch setted by user).

Tests performed
All added options runned through logging simulator.
Effects on system behavior
Different GNSS/INS sensors different accuracies, user should be able to choose the most suitable buffer size and gnss pose publish method for themselves.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The Reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

I have confirmed that it is working as expected. Once the type mistake is corrected, I will approve this pr.

@YamatoAndo i fixed it, sorry about that.


🐛 [flutter_crashlytics] Crashlytics configuration impossible after using flutterfire cli

Bug report
Describe the bug
I initialize FLutterFire in my project with flutterfire cli. After this I can't configure crashlytics correctly.
Steps to reproduce
Steps to reproduce the behavior:

Go to my root project directory
Install cli: dart pub global activate flutterfire_cli
Configure flutterfire flutterfire configure
Add crashlytics from documentation https://firebase.flutter.dev/docs/crashlytics/overview
Run application on simulator -> ok
Try to build app on android: flutter build apk
Get this error:
`Execution failed for task ':app:processReleaseGoogleServices'.

File google-services.json is missing. The Google Services Plugin cannot function without it.
Searched Location:
[myproject]/android/app/src/release/google-services.json
[myproject]/android/app/src/google-services.json
[myproject]/android/app/src/Release/google-services.json
[myproject]/android/app/google-services.json`

Remove this line to build.gradle  apply plugin: 'com.google.gms.google-services'
Get this error:
`A problem was found with the configuration of task ':app:uploadCrashlyticsMappingFileRelease' (type 'UploadMappingFileTask').

No value has been specified for property 'googleServicesResourceRoot'.`

Remove this line to build.gradle  apply plugin: 'com.google.firebase.crashlytics'
flutter build apk works but crashlytics not working with error
The Crashlytics build ID is missing ...

Expected behavior
I want to add flutter fire to my project with flutterfire cli and crashlytics works and flutter build apk works.
Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, 2.7.0-3.1.pre, on macOS 12.0.1 21A559 darwin-x64, locale fr-CH)
[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0-rc1)
[✓] Xcode - develop for iOS and macOS (Xcode 13.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 3.5)
[✓] IntelliJ IDEA Ultimate Edition (version 2017.2.7)
[✓] IntelliJ IDEA Community Edition (version 2021.2.3)
[✓] VS Code (version 1.62.3)
[✓] Connected device (3 available)

• No issues found!

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
Dart SDK 2.15.0-178.1.beta
Flutter SDK 2.7.0-3.1.pre
client_app_sso 1.0.0+1

dependencies:
- cupertino_icons 1.0.4
- firebase_analytics 9.0.2 [firebase_analytics_platform_interface firebase_analytics_web firebase_core firebase_core_platform_interface flutter]
- firebase_core 1.10.5 [firebase_core_platform_interface firebase_core_web flutter meta]
- firebase_crashlytics 2.4.3 [firebase_core firebase_core_platform_interface firebase_crashlytics_platform_interface flutter stack_trace]
- flutter 0.0.0 [characters collection meta typed_data vector_math sky_engine]
- flutter_dotenv 5.0.2 [flutter]
- flutter_secure_storage 4.2.1 [meta flutter]
- flutter_sso 0.0.1 [flutter flutter_appauth flutter_secure_storage json_annotation http path path_provider flutter_dotenv url_launcher firebase_crashlytics firebase_analytics]
- url_launcher 6.0.17 [flutter meta url_launcher_android url_launcher_ios url_launcher_linux url_launcher_macos url_launcher_platform_interface url_launcher_web url_launcher_windows]

dev dependencies:
- build_runner 2.1.5 [args async analyzer build build_config build_daemon build_resolvers build_runner_core code_builder collection crypto dart_style frontend_server_client glob graphs http_multi_server io js logging meta mime package_config path pool pub_semver pubspec_parse shelf shelf_web_socket stack_trace stream_transform timing watcher web_socket_channel yaml]
- flutter_lints 1.0.4 [lints]
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters charcode collection matcher meta source_span stream_channel string_scanner term_glyph typed_data]
- json_serializable 6.1.1 [analyzer async build build_config collection json_annotation meta path pub_semver pubspec_parse source_gen source_helper]

transitive dependencies:
- _fe_analyzer_shared 31.0.0 [meta]
- analyzer 2.8.0 [_fe_analyzer_shared cli_util collection convert crypto glob meta package_config path pub_semver source_span watcher yaml]
- args 2.3.0
- async 2.8.2 [collection meta]
- boolean_selector 2.1.0 [source_span string_scanner]
- build 2.1.1 [analyzer async convert crypto glob logging meta path]
- build_config 1.0.0 [checked_yaml json_annotation path pubspec_parse yaml]
- build_daemon 3.0.1 [built_collection built_value http_multi_server logging path pool shelf shelf_web_socket stream_transform watcher web_socket_channel]
- build_resolvers 2.0.5 [analyzer async build crypto graphs logging path package_config pool pub_semver stream_transform yaml]
- build_runner_core 7.2.2 [async build build_config build_resolvers collection convert crypto glob graphs json_annotation logging meta path package_config pool timing watcher yaml]
- built_collection 5.1.1
- built_value 8.1.3 [built_collection collection fixnum meta]
- characters 1.2.0
- charcode 1.3.1
- checked_yaml 2.0.1 [json_annotation source_span yaml]
- cli_util 0.3.5 [meta path]
- clock 1.1.0
- code_builder 4.1.0 [built_collection built_value collection matcher meta]
- collection 1.15.0
- convert 3.0.1 [typed_data]
- crypto 3.0.1 [collection typed_data]
- dart_style 2.2.0 [analyzer args path pub_semver source_span]
- fake_async 1.2.0 [clock collection]
- ffi 1.1.2
- file 6.1.2 [meta path]
- firebase_analytics_platform_interface 3.0.2 [firebase_core flutter meta plugin_platform_interface]
- firebase_analytics_web 0.4.0+2 [firebase_analytics_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins js]
- firebase_core_platform_interface 4.2.2 [collection flutter meta plugin_platform_interface]
- firebase_core_web 1.5.2 [firebase_core_platform_interface flutter flutter_web_plugins js meta]
- firebase_crashlytics_platform_interface 3.1.10 [collection firebase_core flutter meta plugin_platform_interface]
- fixnum 1.0.0
- flutter_appauth 2.1.0+1 [flutter flutter_appauth_platform_interface]
- flutter_appauth_platform_interface 4.0.0 [flutter plugin_platform_interface]
- flutter_web_plugins 0.0.0 [flutter js characters collection meta typed_data vector_math]
- frontend_server_client 2.1.2 [async path]
- glob 2.0.2 [async collection file path string_scanner]
- graphs 2.1.0 [collection]
- http 0.13.4 [async http_parser meta path]
- http_multi_server 3.0.1 [async]
- http_parser 4.0.0 [charcode collection source_span string_scanner typed_data]
- io 1.0.3 [meta path string_scanner]
- js 0.6.3
- json_annotation 4.4.0 [meta]
- lints 1.0.1
- logging 1.0.2
- matcher 0.12.11 [stack_trace]
- meta 1.7.0
- mime 1.0.1
- package_config 2.0.2 [path]
- path 1.8.0
- path_provider 2.0.7 [flutter path_provider_android path_provider_ios path_provider_linux path_provider_macos path_provider_platform_interface path_provider_windows]
- path_provider_android 2.0.9 [flutter path_provider_platform_interface]
- path_provider_ios 2.0.7 [flutter path_provider_platform_interface]
- path_provider_linux 2.1.2 [flutter path path_provider_platform_interface xdg_directories]
- path_provider_macos 2.0.4 [flutter meta path_provider_platform_interface]
- path_provider_platform_interface 2.0.1 [flutter meta platform plugin_platform_interface]
- path_provider_windows 2.0.4 [ffi flutter meta path path_provider_platform_interface win32]
- platform 3.1.0
- plugin_platform_interface 2.0.2 [meta]
- pool 1.5.0 [async stack_trace]
- process 4.2.4 [file path platform]
- pub_semver 2.1.0 [collection meta]
- pubspec_parse 1.2.0 [checked_yaml collection json_annotation pub_semver yaml]
- shelf 1.2.0 [async collection http_parser path stack_trace stream_channel]
- shelf_web_socket 1.0.1 [shelf stream_channel web_socket_channel]
- sky_engine 0.0.99
- source_gen 1.2.0 [analyzer async build dart_style glob meta path source_span yaml]
- source_helper 1.3.0 [analyzer collection source_gen]
- source_span 1.8.1 [collection path term_glyph]
- stack_trace 1.10.0 [path]
- stream_channel 2.1.0 [async]
- stream_transform 2.0.0
- string_scanner 1.1.0 [charcode source_span]
- term_glyph 1.2.0
- test_api 0.4.3 [async boolean_selector collection meta source_span stack_trace stream_channel string_scanner term_glyph matcher]
- timing 1.0.0 [json_annotation]
- typed_data 1.3.0 [collection]
- url_launcher_android 6.0.13 [flutter meta url_launcher_platform_interface]
- url_launcher_ios 6.0.13 [flutter meta url_launcher_platform_interface]
- url_launcher_linux 2.0.2 [flutter]
- url_launcher_macos 2.0.2 [flutter]
- url_launcher_platform_interface 2.0.4 [flutter plugin_platform_interface]
- url_launcher_web 2.0.5 [flutter flutter_web_plugins meta url_launcher_platform_interface]
- url_launcher_windows 2.0.2 [flutter]
- vector_math 2.1.1
- watcher 1.0.1 [async path]
- web_socket_channel 2.1.0 [async crypto stream_channel]
- win32 2.3.1 [ffi]
- xdg_directories 0.2.0 [meta path process]
- yaml 3.1.0 [collection source_span string_scanner]

@darshankawar I think it is not just an error of flutterfire_cli but a configuration incompatibility between flutterfire_cli and firebase_crashlytics.
I will report this issue in flutterfire_cli too.
Thanks

Hi All,
I am migrating my app for using dart-only initialization. My projects use analytics, crashlytics and etc firebase packages.
For IOS, Crashlytics and analytics work just fine.
For Android, i could do a workaround approach to handle the Crashlytics problem. For more detail, please have a look in
https://github.com/invertase/flutterfire_cli/issues/10
But for Analytics, the below exception is throw.
Missing google_app_id. Firebase Analytics disabled. See https://goo.gl/NAOOOI
After spending some times, I think it should be the problem related to the firebase-android-sdk.
Some users already reported the issue in firebase-android-sdk. Please see the below link.
https://github.com/firebase/firebase-android-sdk/issues/66
@darshankawar
Any suggestion or workaround to solve the issue?


add how to run benchmarks section to readme

Useful for future dev to understand the steps in the  benchmark process.
Part of #280
❗ Blocked by  this devOps Pivotal Story

@demisx  can you also bring back how to run unit tests section

❓ So, is there any value to the team in capturing these benchmark commands in readme, or should we wait until #280 is finished by devOps and close this PR?

I think it would be good to update it with the information about how to run them from #280 as well as how to run them locally if you want to test things. (For example A/B testing performance of two implementations still works locally because those numbers are relative to each other).

Moved to the wiki.


Unable to compile app with flutter 2.8

flutter: 2.8 latest
Tested on: Android and iOS
With latest flutter 2.8, we cannot compile the app anymore with flutter_quill.
../../../../.pub-cache/hosted/pub.dartlang.org/flutter_quill-2.1.1/lib/src/widgets/raw_editor.dart:100:7: Error: The non-abstract class 'RawEditorState' is
missing implementations for these members:
- TextSelectionDelegate.copySelection
- TextSelectionDelegate.cutSelection
- TextSelectionDelegate.pasteText
- TextSelectionDelegate.selectAll
Try to either
- provide an implementation,
- inherit an implementation from a superclass or mixin,
- mark the class as abstract, or
- provide a 'noSuchMethod' implementation.
class RawEditorState extends EditorState
      ^^^^^^^^^^^^^^
../../../../fvm/versions/stable/packages/flutter/lib/src/services/text_input.dart:985:8: Context: 'TextSelectionDelegate.copySelection' is defined here.
  void copySelection(SelectionChangedCause cause);
       ^^^^^^^^^^^^^
../../../../fvm/versions/stable/packages/flutter/lib/src/services/text_input.dart:965:8: Context: 'TextSelectionDelegate.cutSelection' is defined here.
  void cutSelection(SelectionChangedCause cause);
       ^^^^^^^^^^^^
../../../../fvm/versions/stable/packages/flutter/lib/src/services/text_input.dart:973:16: Context: 'TextSelectionDelegate.pasteText' is defined here.
  Future<void> pasteText(SelectionChangedCause cause);
               ^^^^^^^^^
../../../../fvm/versions/stable/packages/flutter/lib/src/services/text_input.dart:979:8: Context: 'TextSelectionDelegate.selectAll' is defined here.
  void selectAll(SelectionChangedCause cause);
       ^^^^^^^^^

+1

Does anybody work on this? I can take a look.

Thanks

+1

While my PR is on review try this
  flutter_quill:
    git:
      url: https://github.com/MagTuxGit/flutter-quill.git
      ref: flutter_28


Support pattern matching for both posts with and without a date in the URL

Ghost already gives the option to pre-pend the date in the post URL, eg; http://myblog.com/2014/12/31/mypost as a configuration setting in the control panel. However, the URL pattern matching does not support both URL with and without dates.
I'll explain why supporting both would be a useful feature.
I initially had a blog being hosted in the free version of wordpress with the domain diogogmt.wordpress.com. I then moved to a self hosted version of ghost on digital ocean with the domain diogogmt.com
My blog was up and running for about 4 years on wordpress and I didn't want simply to shutdown. I then enabled the site redirect feature offered by wordpress.
The issue with the site redirect feature is that it doesn't allow the full post URL to be configured, only the domain and the slug can be changed but the date is part of the URL and can't be removed.
So my wordpress blog was redirecting to my ghost blog with the following URLs:

http://diogogmt.wordpress.com/2014/02/05/mypost -> http://diogogmt.com/2014/02/05/mypost

The issue is that my ghost blog wasn't configured to pre-pend the date in the post URL so the redirects were unsuccessful.
I know I could enable the setting to pre-pend the date in the URL. However, I already published some posts in the new ghost blog without the date in URL and enabling that setting would break the URLs for them.
My proposed solution to the problem is to check for posts slug both with and without the date in the URL so if somebody access http://diogogmt.com/mypost or http://diogogmt.com/2014/05/15/mypost it would fetch the same post.
I've already have a patch with a fix and would appreciate some comments or even if there is a different solution for this problem.
Thanks.

I also have a patch against the stable branch
The URL http://diogogmt.wordpress.com/2014/02/05/cloudstack-4-2-x-loadbalancer-stickness-policies/ will be redirected to http://diogogmt.com/2014/02/05/cloudstack-4-2-x-loadbalancer-stickness-policies/
It is also possible to access the same post without the date in the URL: http://www.diogogmt.com/cloudstack-4-2-x-loadbalancer-stickness-policies/

Hi @diogogmt, I fully understand where you're coming from with wanting to be able to migrate, however, there are serious issues with the approach you've outlined.
Firstly, serving duplicate content at multiple URLs is undesirable (even though Ghost mitigates this by including a canonical URL) and secondly, this approach assumes that the /:slug/ & /:year/:month/:day/:slug/ are the only possible URL patterns, when in fact Ghost supports many, many more. Alternatives are not accessible via the UI, but can be setup by modifying the DB.
I think a much better solution is to add the extra rule you need to either apache or nginx - whichever you are using to host Ghost. It should be relatively straightforward to create a single pattern matching rule which finds URLs which include the date, and redirect to the version without the slug. If you need a hand doing this, why not drop by the #help channel on our slack community?

@ErisDS can you please elaborate on why "serving duplicate content at multiple URLs is undesirable" ?

https://moz.com/learn/seo/duplicate-content highlights a number of issues that search engines face with duplicate content and https://support.google.com/webmasters/answer/66359?hl=en lists a number of reasons details some ways to alleviate the issues.
On top of SEO problems, duplicate content will also play havoc with any analytics you may set up as traffic will be spread out across all of duplicate pages. Other problems may also arise further down the line as you now have more URLs to take care of whenever your site structure or underlying software changes.

Content duplicated across multiple URLs can cause confusion for search engines, social media & analytics... and then I saw @kevinansfield had already replied.

@kevinansfield @ErisDS so your suggestion is to set up a proxy in front of the ghost instance and apply some redirects rules based on the URL fomat?
Do you have an example to share?

@diogogmt if you are self-hosting Ghost & using a custom domain, you should already have a proxy setup in front of Ghost. The documentation is here: http://support.ghost.org/getting-started/#step-4:-advanced-configuration.
If you need help, please visit the #help channel in Slack. GitHub is our issue tracker and we are not able to provide support here.


One should be able to expand/collapse variations in Analysis, too

Perhaps with a context menu. It would conflict with bookmarks, though.

analysis=?    isn't' study chapter the ancestral archetype of all analysis modes?  I might not have enough experience yet with LT.
I also thought bookmarks would eventually or have an issue about making them collapsible.  In-game correspondance analysis, would then just have to bookmark what should be collapsible.  What other usage could there be?  Can all variation be bookmarkes.  or is there a small finite number of bookmark ids possible?. I obviously need to find where bookmarks work and how.

Bookmarks work only on studies. This issue I think I will close. There is no reason to complexify the Tools -> Analysis board (Analysis) feature.


new(static/repo): add new GPG key for Falco packages

What type of PR is this?
/kind content
Any specific area of the project related to this PR?
This PR adds a new GPG key that will be used to sign Falco packages.
The falcosecurity-packages.asc file is a replica of the new key, and will be used in the future to have a non-changing URL pointing to the latest key trusted by the org.
What this PR does / why we need it:
Which issue(s) this PR fixes:
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 78fe62298e31ee23f8bc10ec51c9ba0f2aab9a35


[client/server] Add social features

[ ] Add in-game per-user chat: each player is able to write and receive chat messages from anybody and anywhere as long as they are both connected to the game server
[ ] Add friend requests and list
[ ] Add profiles (username, profile picture)

Related to #21


Action is broken

Hi I'm using the action to test ghidra2dwarf, but it's currently broken.
Example: https://github.com/cesena/ghidra2dwarf/pull/6/checks?check_run_id=1433986748.
Error:
Error: Unable to process command '::set-env name=GHIDRA_INSTALL_DIR,::/opt/hostedtoolcache/ghidra/9.1.2/x64' successfully.
Error: The `set-env` command is disabled. Please upgrade to using Environment Files or opt into unsecure command execution by setting the `ACTIONS_ALLOW_UNSECURE_COMMANDS` environment variable to `true`. For more information see: https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/

Same here, I believe the auto-generated PR #5 would actually fix it.

I tried forking and manually merge the PR #5 https://github.com/meowmeowxw/setup-ghidra-new, but it's still not working https://github.com/cesena/ghidra2dwarf/runs/1527645061

thank you for notification, I'm digging into this.

@meowmeowxw It finally works. I updated setup-ghidra, did clone your project and confirmed it passed the test. https://github.com/er28-0652/ghidra2dwarf/actions/runs/412780309
As @beardypig  mentioned, I merged PR #5, and also I should have compiled TypeScript source code with updated packages by myself to fully affect the update.
May I ask you to run the action again and confirm if it works?

It seems to work, thanks! :+1:


oneof not working

When I use a oneof with this plugin, it reports a syntax error on the last field.
e.g.
message Foo {
  oneof bar {
    optional string baz = 1;
    optional int32 fizz = 2;
  }
}

It reports: "extraneous input fizz, expecting '='"
Plugin version 0.8
see: https://developers.google.com/protocol-buffers/docs/reference/java-generated#oneof-fields

You cannot use optional, required or repeated field modifiers inside of an oneof block.
The only thing to improve here is error message.

Doh! Good call. Thanks!


Don't know how to use this tool, need enhance documentation

Hi, we are a bunch of data scientists. We are trying to leverage prometheus metrics data to do AIOPS. Metrics data are already stored in AWS S3 via Thanos sidecar approach. To make data consumable by Spark, need a tool to convert data. Looks like this tool can be helpful, but it's really hard to understand how to use this tool given neither of us have Go programming experiences...
For instance, we are testing below commands:
go run ./cmd/obslytics export --input-config='{"endpoint":"<IP_ADDRESS>:10901","tls_config":{"insecure_skip_verify":true}}' --match="container_cpu_load_average_10s" --resolution=2h --output-config-file=container_cpu_load_average_10s.parquet --debug --min-time="2021-08-24T20:00:00+00:00" --max-time="2021-08-25T20:00:00+00:00"
The console gets error codes:
level=error ts=2021-08-25T09:32:26.912354Z caller=main.go:104 msg="running command failed" err="unsupported Reader type " exit status 1
No idea why, something related to StoreAPI. We deployed Thanos sidecar and Querier, and then kubectl port-forward querier pod like this: kubectl port-forward thanos-addons-query-7fc47d8cff-tf4rg 10901:10902

Nice 👍🏽

Yup, docs are needed. Help wanted!


github.com/HubSpot/YouMightNotNeedjQuery

I don't know if you saw this before https://github.com/HubSpot/YouMightNotNeedjQuery
Maybe it would be a good idea to merge both if there are cases missing from that repo.

Yep for sure, YouMightNotNeedjQuery is another good guide to reach, I also made a PR to that repo, but unfortunately the maintainers seems not so active. That's also why we started this project.
Basically this project is different with YouMightNotNeedjQuery in these:

Every piece of tips in this guide is used in our projects on PRODUCTION, so feel free to copy them.
Currently we only focused on IE 10+, for it's more featureful. In the near future we don't have enough energy to work with IE 8/9
This project will be actively maintained.

Yeah, I see your point. Thanks!


[DOCS] Difference between v0.1.0, v0.1.1 and v0.1.2

Can someone elaborate on the differences between v0.1.0, v0.1.1 and v0.1.2 I can't find this documented anywhere.

As far as I can tell:
v0.1.0 uses:

No artificial delays or other inbuilt "issues"
-Hat ID: "01"
LeftArm ID: "01"
LeftLeg ID: "01"
RightArm ID: "01"
RightLeg ID: "01"

v0.1.1 uses:

No artificial delays or other inbuilt "issues"
-Hat ID: "02"
LeftArm ID: "02"
LeftLeg ID: "02"
RightArm ID: "02"
RightLeg ID: "02"

v0.1.2 uses:

An artificial 400ms delay added to each request
-Hat ID: "03"
LeftArm ID: "03"
LeftLeg ID: "03"
RightArm ID: "03"
RightLeg ID: "03"

v0.1.3 uses:

No artificial delays or other inbuilt "issues"
-Hat ID: "04"
LeftArm ID: "04"
LeftLeg ID: "04"
RightArm ID: "04"
RightLeg ID: "04"

That said, there is no v0.1.3 image available on ghcr.


Dependabot submodule

Codecov Report

Merging #140 (fc1f29e) into main (298c369) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #140   +/-   ##
=======================================
  Coverage   71.06%   71.06%
=======================================
  Files          12       12
  Lines        1258     1258
=======================================
  Hits          894      894
  Misses        364      364

Impacted Files
Coverage Δ

clippy-sarif/src/bin.rs
93.75% <0.00%> (ø)

Continue to review full report at Codecov.

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


WIP: graalvm 19.2.1 -> 20.0.0 (plus some refactoring)

Motivation for this change
Requested update and refactor https://github.com/NixOS/nixpkgs/issues/83080
Rebased on this pending PR https://github.com/NixOS/nixpkgs/pull/85902
Things done
Still WIP, feel free to give feedback. As of this writing I've only compiled the sdk (core) of graalvm for java8. Java11 support should be much easier. As well as splitting some of the modules (FastR/graal-js/GraalRuby etc) as this is too much for 1 person to maintain.

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

I think we should close this PR, it was already updated

I think we should close this PR, it was already updated

graalvm8, not graalvm8-ce or graalvm8-ee, is not updated.  graalvm8-ce and graalvm8-ee are binary packages while graalvm8 builds from the source.

graalvm8, not graalvm8-ce or graalvm8-ee, is not updated.  graalvm8-ce and graalvm8-ee are binary packages while graalvm8 builds from the source.

good points here. I'm keeping the branch. There was a problem with the embedded llvm compiler in the prebuilt sources. If that isn't resolved in the near future, I will continue this work. I'm still waiting for the java PR to be merged https://github.com/NixOS/nixpkgs/pull/85902 (I see now that it needs rebase), but without the static java, building graalvm from sources wont work. Must admit that in the end, it was bit demotivating that it took so much time for reviews and I had put a lot of hours in this (which is why I'm bit burnt from the graalvm from sources and happy to see the prebuilts working mostly).


Filtering only to .pdb files while indexing sources

Apart from rename, the only real change is addition of line 95 to match only pdb files

@arunkm,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Fails on redirected pages

For example, this page raises an exception:
pyscp.wikidot.Wiki('www.scp-wiki.net')('sandbox').votes

This can probably be fixed by querying www.scp-wiki.net/{page}/noredirect/true instead of www.scp-wiki.net/{page}/.

The URL for the wiki has changed and all you have to do in the example above is switch it to be
pyscp.wikidot.Wiki('www.scpwiki.com')('sandbox').votes


sf initial edits

I wanted to start a PR for docs, so we can have some intermediate discussions here about the docs progress.
So far I am thinking that in installation, we can roll over all the "institutional" instructions into one -- in practice I have also been installing using pip. We can just two sets, one for installing with pip and one for installing from source (I will try to test the from source stuff again).
Other than that, I can confirm that the WSL installation works okay, it is not that experimental.
Finally, I was wondering if we should provide some examples of setting logger to debug and some such, i.e. using the context manager? I am a bit hazy on that personally.

Codecov Report

Merging #51 (541feff) into master (e08ab47) will increase coverage by 13.66%.
The diff coverage is 89.47%.

Additional details and impacted files

@@             Coverage Diff             @@
##           master      #51       +/-   ##
===========================================
+ Coverage   65.04%   78.71%   +13.66%
===========================================
  Files          13       11        -2
  Lines        1711     1602      -109
===========================================
+ Hits         1113     1261      +148
+ Misses        598      341      -257

Impacted Files
Coverage Δ

ggce/utils/utils.py
50.00% <83.33%> (+5.68%)
:arrow_up:

ggce/executors/petsc4py/base.py
97.51% <100.00%> (+97.51%)
:arrow_up:

ggce/logger.py
67.21% <100.00%> (-18.73%)
:arrow_down:

ggce/engine/system.py
74.62% <0.00%> (-17.57%)
:arrow_down:

ggce/model.py
63.97% <0.00%> (-14.32%)
:arrow_down:

ggce/utils/physics.py
80.00% <0.00%> (-13.34%)
:arrow_down:

ggce/engine/terms.py
80.42% <0.00%> (-6.07%)
:arrow_down:

ggce/engine/equations.py
80.18% <0.00%> (-0.69%)
:arrow_down:

... and 5 more

@Chiffafox looks fantastic, should I merge?

Yeah let's do it! The only "core" tutorial still missing is the finite t one, I'll try to get to that one soon -- will open another PR for that.

Hey @x94carbone, still don't see the updated docs (specifically tutorials) on the mainpage (on the right), even though all the updated source files are now part of master -- do we need to do something extra to start hosting the new version?

@Chiffafox ya I didn't deploy yet. I'll tag this later and it will automatically push the docs 👍


Did not received "Import success" after import xml files

Expected Behavior
Received "Import success" messages after import xml files.
Actual Behavior
Did not received "Import success" messages. You can check this photo, I have 18 EHT on my test wallet. Here is my address: 0x5735BA41852f309dF11aF2C27F9c49696eBE66B1
Steps to Reproduce the Problem

Update WALLET_ID (Line 3) and PROVIDER_ID (Line 16 and 82) by new ID get from here for example_v1.5.xml
npm start
curl -v  -F<EMAIL_ADDRESS>http://<IP_ADDRESS>:8900/import

Specifications

Version: on Master branch
Platform: Ubuntu 16.04 run on VMWare 14

I try to import example_gs1.xml and I get error, you can check this photo

Do you have any Alpha TRAC on your wallet (our test tokens)? You need also to have Alpha TRAC in order to test. If not, we will be sending you some today on this address, just let me know.

The amount of 10 000 test tokens (ATRAC) have been sent to your wallet address.

Now, I get "Success" status for the transactions on etherscan. But, I did not get "Import success" messages after import xml files. You can check this photo.
When I import the XML files, I create 3 transaction for each time on etherscan. So, Why 3 transaction not 1 transaction ?

This is completely fine. This message is not displayed in the console, just returned as a result of this method. So you should not see this message in console.
In order to fully test it , you should have another node connected to first one's network, so that import is sent to DH. Then the DC will send the import to DH and start testing - challenging if the DH keeps the data. This is where "No challenges found" comes from. Please read our Incentive model for more information how it works.
Regarding 3 transactions:

writing fingerprint of data on blockchain on import
creating allowance (DC - allowing smart contract to take our money for the job)
creating escrow contract between two parties - otherwise DC may choose not to pay after job is finished. Smart contract guarantees the payout.

Please read the document above, it will be more clear.

Now, I understand how It work. Thank you so much @schrink