text:
Add setTimeout() method

This PR adds a Nmap::setTimeout() method which allow to override the default 60s timeout.

[x] Doc
[x] Tests

LGTM, thanks!

Thanks !

source: gharchive/pull-request

authors: EmmanuelVella willdurand

repo: willdurand/nmap

license: mit

license_type: permissive

license_source: bigquery


text:
Weights path

Adds optional weight_save_path to trainer to remove the need for a checkpoint_callback when using cluster training

Passed on GPUs
tests/test_models.py::test_running_test_pretrained_model_ddp PASSED                                                                                [  3%]
tests/test_models.py::test_running_test_after_fitting PASSED                                                                                [  6%]
tests/test_models.py::test_running_test_without_val PASSED                                                                                [  9%]
tests/test_models.py::test_running_test_pretrained_model PASSED                                                                                [ 12%]
tests/test_models.py::test_running_test_pretrained_model_dp PASSED                                                                                [ 16%]
tests/test_models.py::test_gradient_accumulation_scheduling PASSED                                                                                [ 19%]
tests/test_models.py::test_multi_gpu_model_ddp PASSED                                                                                [ 22%]
tests/test_models.py::test_optimizer_return_options PASSED                                                                                [ 25%]
tests/test_models.py::test_single_gpu_batch_parse PASSED                                                                                [ 29%]
tests/test_models.py::test_early_stopping_cpu_model PASSED                                                                                [ 32%]
tests/test_models.py::test_no_val_module PASSED                                                                                [ 35%]
tests/test_models.py::test_no_val_end_module PASSED                                                                                [ 38%]
tests/test_models.py::test_simple_cpu PASSED                                                                                [ 41%]
tests/test_models.py::test_amp_single_gpu PASSED                                                                                [ 45%]
tests/test_models.py::test_cpu_restore_training PASSED                                                                                [ 48%]
tests/test_models.py::test_amp_gpu_ddp PASSED                                                                                [ 51%]
tests/test_models.py::test_cpu_slurm_save_load PASSED                                                                                [ 54%]
tests/test_models.py::test_loading_meta_tags PASSED                                                                                [ 58%]
tests/test_models.py::test_dp_output_reduce PASSED                                                                                [ 61%]
tests/test_models.py::test_model_saving_loading PASSED                                                                                [ 64%]
tests/test_models.py::test_model_freeze_unfreeze PASSED                                                                                [ 67%]
tests/test_models.py::test_amp_gpu_ddp_slurm_managed PASSED                                                                                [ 70%]
tests/test_models.py::test_cpu_model_with_amp PASSED                                                                                [ 74%]
tests/test_models.py::test_cpu_model PASSED                                                                                [ 77%]
tests/test_models.py::test_all_features_cpu_model PASSED                                                                                [ 80%]
tests/test_models.py::test_single_gpu_model PASSED                                                                                [ 83%]
tests/test_models.py::test_multi_gpu_model_dp PASSED                                                                                [ 87%]
tests/test_models.py::test_amp_gpu_dp PASSED                                                                                [ 90%]
tests/test_models.py::test_ddp_sampler_error PASSED                                                                                [ 93%]
tests/test_models.py::test_multiple_val_dataloader PASSED                                                                                [ 96%]
tests/test_models.py::test_multiple_test_dataloader PASSED                                                                                [100%]

repo: williamFalcon/pytorch-lightning


text:
.set_beta(0) assigns weight=NaN to parameters with zero weight causing plot errors

Describe the bug
When the initial data contains samples with zero weight, these will turn into NaNs when running .set_beta(0). This will then result in errors when trying to plot the prior.
MWE
ns = NestedSamples(root="/path/to/anesthetic/tests/example_data/pc")
for i in range(10):
    ns['logL'][i] = -np.inf
    ns['weight'][i] = 0
prior = ns.set_beta(0)
prior.plot_1d(['x0', 'x1'])

Error output
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-24-31aeaee3a714> in <module>
----> 1 prior.plot_1d(['x0', 'x1'])

~/.virtualenvs/py38env/lib/python3.8/site-packages/anesthetic/samples.py in plot_1d(self, axes, *args, **kwargs)
    256
    257         for x, ax in axes.iteritems():
--> 258             self.plot(ax, x, *args, **kwargs)
    259
    260         return fig, axes

~/.virtualenvs/py38env/lib/python3.8/site-packages/anesthetic/samples.py in plot(self, ax, paramname_x, paramname_y, *args, **kwargs)
    157                     if ncompress is None:
    158                         ncompress = 1000
--> 159                     return kde_plot_1d(ax, self[paramname_x],
    160                                        weights=self.weight,
    161                                        ncompress=ncompress,

~/.virtualenvs/py38env/lib/python3.8/site-packages/anesthetic/plot.py in kde_plot_1d(ax, data, *args, **kwargs)
    322     weights = kwargs.pop('weights', None)
    323     ncompress = kwargs.pop('ncompress', 1000)
--> 324     x, w = sample_compression_1d(data, weights, ncompress)
    325     kde = gaussian_kde(x, weights=w)
    326     p = kde(x)

~/.virtualenvs/py38env/lib/python3.8/site-packages/anesthetic/utils.py in sample_compression_1d(x, w, n)
    344         i = w.index
    345     else:
--> 346         i = numpy.random.choice(w.index, size=n, replace=False, p=w/w.sum())
    347
    348     # Define sub-samples

mtrand.pyx in numpy.random.mtrand.RandomState.choice()

ValueError: probabilities contain NaN

Expected behavior
I'd say that .set_beta should carry over the zero weight instead of assigning NaNs. Then the w/w.sum() in the sample compression wouldn't produce the plotting error. Thoughts?
Additional context
Cobaya allows to specify external priors with which you can exclude parts of a multi-parameter space. This will create entries with log(0) likelihood and with 0 weight

A workaround is to create a mask:
mask = np.isfinite(prior.weight)
prior[mask].plot_1d(['x0', 'x1'])

which then works fine. However, it would be nicer if this extra step wouldn't be necessary...

@williamjameshandley this explains a lot of the errors I was getting. I think NestCheck still has this issue, so should we carry this workaround to #63 or create a PR for nestcheck?

Hi @lukashergt. I think the issue is not the zero weight, but in fact the -np.inf logLs.
in the function decorated by @beta.setter, you can see that there is a line including self.beta*self.logL, and it is this which generates the error. The fix to this would be in the same vein as #71, namely to define  0 * np.inf == 0 in this case (via a np.where), as the infinity is a 'logarithmic infinity', and thus subdominant to beta's 0.
Feel free to submit a PR with this single line fix (ideally with the initially failing test above which then passes with the fix).
One further point is that your line ns['weight'][i] = 0 doesn't actually do anything, since ns['weight'] is different from ns.weight, and it is the latter which is used for computing the weighted sums. This possible anti-pattern was introduced in #55 and #56. I'm not sure of the best way to ameliorate this clearly confusing state of affairs. One could:

drop the weight as a column (but this then makes it less clear that it is a weighted frame)
make the weight column read-only (don't know if this is easy to do in pandas, and would likely cause no end of bother down the line)
synchronise the weight column with the weight attribute more effectively. This would require someone with an understanding of how the weighted data frames work in anesthetic/weighted_pandas.py which is currently just me at the moment (AFAIK).

Would be keen to hear both @lukashergt and @appetrosyan 's views on this.

One further point is that your line ns['weight'][i] = 0 doesn't actually do anything, since ns['weight'] is different from ns.weight, and it is the latter which is used for computing the weighted sums.

What exactly do you mean by "doesn't do anything"? It does change the weight column as can be seen if you do print(prior.weight) at the end of my MWE. What is less desirable is that it also changes ns.weight and ns.logL (because prior is a simple copy not a deep copy of ns)...
But in any case, here I'm only doing this for the MWE to create the sample that can be used to show the error.

It does change the weight column as can be seen if you do print(prior.weight) at the end of my MWE.

Nope, wrong. It actually doesn't change the numbers, but it does change the way it displays the numbers making it look like the weights changed to zero when in fact they are the same numbers as before which were tiny to begin with...
No clue, why the display would change...
logL seems to be changed to -inf, though, which indeed must be the thing that drives the error.

So the issue is that there is a hidden attribute in the WeightedSeries and WeightedDataFrame called _weight.
The hidden attribute is necessary for WeightedSeries, as if you want the pandas equivalent of a Series you can only have one column, so the weights have to be an attribute. This is forwarded as a readable attribute via a @property, and the correct behaviour is also enforced for slicing.
The same was originally true for WeightedDataFrame, but this was slightly counterintuitive as you would not see the weights in the same way that you see nlive or other derived NestedSamples columns. The contents of _weight is therefore copied into a new column whenever you change _weight using the anesthetic functions, but if you go and change this new column manually it has no effect on the weighted calculations (like mean, std, median etc)

drop the weight as a column (but this then makes it less clear that it is a weighted frame)

I don't like this solution. It is useful to see the weight column when looking at the full table.

make the weight column read-only (don't know if this is easy to do in pandas, and would likely cause no end of bother down the line)

Well, it seems like this is already the case, right? Trying to overwrite the weight column actually doesn't work...

The original issue is closed with #85 , however, I'm not sure whether the discussion about the weight column should be kept open...?

Raised as a new issue #86 for further discussion there

source: gharchive/issue

repo: williamjameshandley/anesthetic

license: MIT

license_source: github-api


text:
feat: add TriliumNext via optional extra successor repo

because the package name collides but the fork is consensual, and to avoid forcibly upgrading trilium users to the fork, we could make use of an optional override repo.
Closes #1209
This draft PR attempts to demonstrate this usage of the facility by creating an additional repository in 11-successors containing an entry for the trilium package pointing to the TriliumNext fork.  In this way the default behaviour leaves trilium to use and follow the original repository, but if users want to move to the fork (or choose to install and track the fork) they will need to enable the 11-successors repository.  For example:
head -1 /etc/deb-get/01-main.repo |sed 's/01-main/11-successors/g' |sudo tee /etc/deb-get/11-successors.repo

This will only work after the repository exists, but then an update will instantiate the new repository override definition and (once the cache for trilium is removed or expires) existing trilium users will get an "upgrade" onto the fork, which will be presented as the only trilium package because of the override.  To test this out before the PR is merged (since the CI test is currently broken anyway, and in any case would need extending to enable the 11-successors repo in the test logic!) it is possible to enable the repo from the PR branch thus:
head -1 /etc/deb-get/01-main.repo |sed 's/01-main/11-successors/g;s/wimpysworld/philclifford/;s/\/main\//\/add-triliumnext\//' |sudo tee /etc/deb-get/11-successors.repo
## This is helpful/needed if your cache is still live
# sudo rm /var/cache/deb-get/trilium*
deb-get update

With the original copied to 99-local.d under a different name you'd then see something like this and be able to install the fork:
deb-get show trilium-old trilium
 [...]
  [+] Including local package trilium-old
  [!] Please consider contributing back new entries, an issue (or raise a PR) directly at https://github.com/wimpysworld/deb-get/pulls
Trilium Notes
  Package:	trilium-old
  Repository:	99-local
  Updater:	deb-get
  Installed:	No
  Published:	0.63.7
  Architecture:	amd64
  Download:	https://github.com/zadam/trilium/releases/download/v0.63.7/trilium_0.63.7_amd64.deb
  Website:	https://github.com/zadam/trilium/
  Summary:	Trilium Notes is a hierarchical note taking application with focus on building large personal knowledge bases.
TriliumNext Notes
  Package:	trilium
  Repository:	11-successors
  Updater:	deb-get
  Installed:	0.90.12
  Published:	0.90.12
  Architecture:	amd64 arm64
  Download:	https://github.com/TriliumNext/Notes/releases/download/v0.90.12/TriliumNextNotes-v0.90.12-linux-x64.deb
  Website:	https://github.com/TriliumNext/Notes
  Summary:	An open-source, cross-platform hierarchical note taking application with focus on building large personal knowledge bases

This seems to me the most flexible and respectful approach.
Of course users are at liberty to take this definition and use it as an override in 99-local or add it to their own external repository as an override.

Documenting the opportunity to move to the fork needs mentioning in more than just a line in the release notes if this is acceptable.  I'd suggest a note in the 01-main definition and maybe a note in the wiki linked there with how to enable the 11-successors repo.
As it stands this proposal would allow similar overrides to be added to the one repo for other forks, but that would mean all-for-one-and-one-for-all.  It might be better to have one override repo per fork (or set of forks, or ...). Happy to rename the new repo to 11-triliumnext.

repo: wimpysworld/deb-get


text:
Does this work with Vim as an LSP?

Basically, I want to be able to do this:
  windicss = {
    cmd = { "windicss-language-server", "--stdio" },
    filetypes = { "aspnetcorerazor", "astro", "astro-markdown", "blade", "django-html", "edge", "eelixir", "ejs", "erb", "eruby", "gohtml", "haml", "handlebars", "hbs", "html", "html-eex", "jade", "leaf", "liquid", "markdown", "mdx", "mustache", "njk", "nunjucks", "php", "razor", "slim", "twig", "css", "less", "postcss", "sass", "scss", "stylus", "sugarss", "javascript", "javascriptreact", "reason", "rescript", "typescript", "typescriptreact", "vue", "svelte" },
    init_options = {
      userLanguages = {
        eelixir = "html-eex",
        eruby = "erb"
      },
    },
    on_new_config = function(new_config)
      if not new_config.settings then
        new_config.settings = {}
      end
      if not new_config.settings.editor then
        new_config.settings.editor = {}
      end
      if not new_config.settings.editor.tabSize then
        -- set tab size for hover
        new_config.settings.editor.tabSize = vim.lsp.util.get_effective_tabstop()
      end
    end,
    root_dir = util.root_pattern('windi.config.js', 'windi.config.cjs', 'windi.config.ts', 'tailwind.config.js', 'tailwind.config.cjs', 'tailwind.config.ts', 'postcss.config.js', 'postcss.config.ts', 'package.json', 'node_modules', '.git'),
    settings = {
      tailwindCSS = {
        lint = {
          cssConflict = "warning",
          invalidApply = "error",
          invalidConfigPath = "error",
          invalidScreen = "error",
          invalidTailwindDirective = "error",
          invalidVariant = "error",
          recommendedVariantOrder = "warning"
        },
        validate = true
      },
    },
  },

require'lspconfig'.windicss.setup{}

...and then for it to work with Neovim via https://github.com/neovim/nvim-lspconfig

I would also welcome a windi lsp support in neovim!

updates?

Any news? It would be awesome to use this in my projects.

You would need to split the backend part from the extension and then make the extension call the backend while allowing the backend to run as its own thing

authors: Suyashtnt benwoodward dedani olgam4

repo: windicss/windicss-intellisense


text:
server: use the standard golang pprof hooks

Hello @jazzdan, @maiamcc,
Please review the following commits I made in branch nicks/httppprof:
28f8722b83c95ab2c897a47f99eae882a6288fe6 (2020-01-03 15:13:52 -0500)
server: use the standard golang pprof hooks

ya, i think we should remove it on the next release

repo: windmilleng/tilt


text:
Fix UnhandledPromiseRejectionWarning error

Using http://downloads.mongodb.org rejects with an error:
(node:21921) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 7): Error: Protocol "http:" not supported. Expected "https:"

This fixes #26

Unfortunately this was already know from PR#21: https://github.com/winfinit/mongodb-download/pull/21#commitcomment-22810016

@winfinit Can you please take a look at this PR? There are some dependent module that are broken because of this issues and it's a blocker. Thx!

+1

@david-cho-lerat-HL2 it's not the same problem,  #21 fixes domain and uri but brokes schema replacing  https  by  http that is deprecated on mongodb distribution servers for now.
@winfinit Please merge this changes ASAP

+1

@orn0t I know this wasn't the main focus of #21, but I had commented in #21 about the problem fixed by #27

Can we merge this then please and release it? It's a blocker for a lot of us.

+1

+1

+1

Hey @winfinit what is the status? Based on the fact there was one line changed in the PR and this is pending merge and release for 9 days now, maybe you should consider adding some more people to the contributors.

@mihaerzen Your fork unfortunately doesn't seem to fix the problem;

Only the source is changed and the built version is included while installing via NPM
The URL fastdl.mongodb doesn't seem to fix the problem for me, getting error: "Unable to get signature content" - StatusCode 403.

First problem is off-course easily solvable, the second one is not. Do you have any ideas what's causing this 403 response, my bet would be it has something to do with the mongodb server configuration. Are there any other aliases for this page we can use?

Hey @sjorssnoeren!

I have no idea how to generate the build. My assumption was some CI does this when the release is created. Or do you maybe know how to do it?
This PR does not use fastdl.mongodb. It literally just adds s to https. You might want to try installing the master version and then changing this line to match my PR https://github.com/winfinit/mongodb-download/blob/master/built/mongodb-download.js#L13 . After I've done this I had no more issues with this module. Maybe your issue is somewhere else?

Hi @mihaerzen
Thanks for your response! I've never used TypeScript, but my guess is you can just change it in the distributed files (I don't think CI handles this in this repository). I would recommend to change it directly in built directory and cross fingers it doesn't break a .js.map file
Checking your solution with https right now!

@mihaerzen I double checked, I still receive errors for https://downloads.mongodb.org. For some reason I keep receiving 403's.
I use latest version of mockgoose, works locally, fails in CircleCI Node 6.

@mihaerzen You were right, I wrongly configured npm to use this fork. Sorry about that.

got same thing here https://gitter.im/nestjs/nestjs?at=5a20df37540c78242d6d6342 that is happening when runnin this one https://github.com/nestjs/nest/tree/master/examples/10-mockgoose with process.env.NODE_ENV = 'test' and without having mongodb installed as the project sust using Mockgoose thing

hopefully it will fix that https://github.com/winfinit/mongodb-download/issues/26#issuecomment-348403520 as well

I am using the Mockgoose7.3.3 and I was able to run my tests successfully but suddenly it starts complaining "(node:9520) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 6): Error: Protocol "http:" not supported. Expected "https:".
I would like to know the reason why it worked previously and why it suddenly started complaining the above.
And also what should be the work around for this, please help me with this.
Thanks!

+1

+1

@samathan Use shrinkwrap or npm 5 and pin use the git repo address pinned to this pr’s commit id

authors: MalhotraVijay alexandrusavin david-cho-lerat-HL2 flx-l franky-continu kuncevic mihaerzen orn0t sallas samathan sjorssnoeren

repo: winfinit/mongodb-download

license: ISC


text:
update License to 2017

Happy New Year!

It's almost 2018 but thanks. :)


text:
Add support for icon picker

Add support for an icon picker field. This took inspiration from https://github.com/hasinhayder/vue3-icon-picker but was modified greatly in the end.

Offers support for extending the fonts library dynamically.
A few notes:

json will be added to the page multiple time if more than 1 icon picker is included.
there are significant issues with how we handle font awesome 6, 4 (-o), wn- and icon-. We seems to be on FA 6.1.1 with the latest being 6.2.0. An update to this would be required, and maybe at this point re-work how we provide support for all the legacy, so that is also becomes easier to update and maintain.
because of the above, and because i do not know what FA was used in the forked repo, I am not sure if the icon picker as it stands in this PR covers all the icons in 6.1.1. This should be improved on upon updating FA

@robertalexa this looks awesome. Do you have any screenshots on how this looks in the current backend theme?
As for upgrading Font Awesome, our current fork is using FA 6.1.1 as I made the upgrade in May. There was a manual element to it - I tried to use the LESS edition of the Font Awesome source files, but I believe I had to make substantial edits for us to cover all our legacy bases. If we need to upgrade again, I can have another crack at doing it and either make it more automated or at least document what I did.

@bennothommo The picker looks the same on both default and tailwindui theme.
I had an attempt and thought that it would be great to instead use a variable (the branding colours that the user selects) but I have discovered inconsistencies between the use of those brandings between the default and tailwindui theme. So i have left this for now.
The light blue looks like it belongs on both UIs

@LukeTowers any news on this?

@jaxwilko it needs to be loaded from a YAML file like a nestedform form config instead of a config file. I can do that sometime this week if @robertalexa can't.

@LukeTowers What difference does that make if neither of these options is the real good long term one? The real good one is to actually handle this "library" of icons as part of the update process of fontawesome at winter's end and decomissions the hardcoded config/yaml approach.
But to do it the right way now, it requires font awesome updated, the shims done once, in 1 place, and married with an existing FA, with regular, solid and brands categories in mind, and with the update process defined and documented.

Regardless of the long term improvements to the icon updating process for the Winter core the API for this particular form widget will remain the same:
icon:
    label: Icon
    type: iconpicker
    libraries:
        fontawesome: ...
        materialui: ...
    # or
    libraries: ~/modules/backend/formwidgets/iconpicker/meta/libraries.yaml

It will not be a config file at any point.
Another change that needs to be made is renaming app.js to iconpicker.js and App.vue to IconPicker.vue and let app = createApp(App) to let iconPicker = createApp(IconPicker)

form:
    fields:
        icon:
            label: 'Icon'
            type: 'iconpicker'
            default: 'far icon-address-book'
            libraries:
                -
                    id: 'far'
                    title: "Font Awesome Regular"
                    prefix: "far icon-"
                    listicon: "far icon-circle"
                    icons:
                        - "far icon-address-book"
                        - "far icon-address-card"
        icon2:
            label: 'Icon'
            type: 'iconpicker'
            default: 'far icon-address-book'
            libraries: ~/modules/backend/formwidgets/iconpicker/meta/libraries.yaml
        icon3:
            label: 'Icon'
            type: 'iconpicker'

Use the above to test. These are the current variations supported using the approach suggested by @LukeTowers

NOTE: the libraries are not scoped by a library parent. They share equal weight. In the context of FontAwesome; far, fas and fab are "individual" libraries

The above example shows:

manual library definition
file based library
no library defined - it will then default to the default path in \Backend\FormWidgets\IconPicker::DEFAULT_LIBRARIES
no default icon defined - it will then default to far icon-address-book - see _iconpicker.php

In the grand scheme of things, the libraries.yaml file should be rebuild as part of FA update process. I can't guarantee that the current list contains all the icons available in the current (winter) FA version, but I can guarantee that everything that is included is 100% supported.

@LukeTowers I have implemented changes based on your feedback.
    fields:
        icon:
            label: 'Icon'
            type: 'iconpicker'
            default: 'far icon-angryyy'
            libraries:
                -
                    id: 'far'
                    title: "Font Awesome Regular"
                    prefix: "far icon-"
                    listicon: "far icon-circle"
                    icons:
                        - "far icon-address-book"
                        - "far icon-address-card"
        icon2:
            label: 'Icon'
            type: 'iconpicker'
            default: 'far icon-angry'
            libraries: ~/modules/backend/formwidgets/iconpicker/meta/libraries.yaml
        icon3:
            label: 'Icon'
            type: 'iconpicker'

First icon picker open (no value, set default, unavailable in the library):

Second icon picker open (no value, set default, available in the library)

Third icon picker open (no value, no set default, first icon in library)

Awesome work @robertalexa, thanks! Could you put together a PR to the docs to document it?

authors: LukeTowers bennothommo jaxwilko robertalexa

repo: wintercms/winter


text:
fix: deleted defaultSubjectValue

PR Submission Checklist for internal contributors
What's new in this PR?
Issues
"New Appointment" overwrites the "CLDR::" Prefix on empty conversation titles
Solutions
Deleted defaultSubjectValue = "New Appointment";
Testing
How to Test
Create a meeting without a title, and check for the CLDR:: "Current Date" conversation Title in Wire
PR Post Merge Checklist for internal contributors

[ ] If any soft of configuration variable was introduced by this PR, it has been added to the relevant documents and the CI jobs have been updated.

References

https://sparkbox.com/foundry/semantic_commit_messages
https://github.com/wireapp/.github#usage
E.g. feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764.

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

authors: CLAassistant KevinJaroschewskiCONPORT

repo: wireapp/outlook-addin


text:
Introduce Plugin API and refactor extension management

Alternative to #67

Will go ahead and merge it @bitxon . We can rework before 1.0.0 if needed

repo: wiremock/wiremock-testcontainers-java


text:
Variable name conflict causing dupes to break after #439

Duping some things like wire teleporters now errors with addons/advdupe2/lua/advdupe2/sv_clipboard.lua:1273: bad argument #2 to '__add' (Vector expected, got table)

I swapped out the pcall to a ProtectedCall so the error prints the full stack:
[advdupe2] addons/advdupe2/lua/advdupe2/sv_clipboard.lua:1266: bad argument #2 to '__add' (Vector expected, got table)
  1. __add - [C]:-1
   2. unknown - addons/advdupe2/lua/advdupe2/sv_clipboard.lua:1266
    3. ProtectedCall - [C]:-1
     4. func - addons/advdupe2/lua/advdupe2/sv_clipboard.lua:1478
      5. unknown - addons/ulib/lua/includes/modules/hook.lua:260

This might be unrelated to the PR, but either way it's an deeper issue that's been revealed now.
https://github.com/wiremod/wire/blob/master/lua/entities/gmod_wire_teleporter.lua#L20
Teleporter defines a field LocalPos. AdvDupe2 stores dupe data directly on the entity, so it's likely it was trying to access this incorrect LocalPos field instead of the AdvDupe2 data.

Oh wow, yeah seems like that's the issue, maybe the advdupe2 values should be named better? Although i don't know how that'd work with backwards dupe compat etc

Should change the issue title and description to reference that instead, although if you can confirm that it doesn't happen on the previous commit, then that's even more mysterious.

Looking deeper into it, it seems this might be due to the implementation of backwards compatibility with AdvDupe1.

@wrefgtzweve
Nope. It does not seem to be my patch that is related only to bone manipulations. Wire teleporter is not a ragdoll :D

It's weirdly inconsistent which made me believe the pr broke it, my bad for that

The issue is still present, that pr didn't solve it for me
[AdvDupe2Notify]	Pasting...
[AdvDupe2Notify]	addons/advdupe2/lua/advdupe2/sv_clipboard.lua:1244: attempt to compare nil with number
[AdvDupe2Notify]	Finished Pasting!

I think the pr fixed it for the Queue'd case (multiplayer), but maybe the singleplayer case isn't fixed.

This is from accessing Queue.Revision which gets its value in function AdvDupe2.LoadDupe, in a dupe info struct whose field revision is set in function AdvDupe2.Decode, where that field is impossible to be nil, so this is likely some sort other issue.
SP and MP both use the Queue. I tested in SP.

Hm, maybe check that you're not using the workshop version, @wrefgtzweve ?

I'm on commit 7f061801bf7bbdfdb573ed23410591d48c1e93ae, restarted my game, i don't have it on workshop either.

Are you willing to share the dupe to help? Any other considerations?

Oh wait, looking at the error im getting / posted, it's a different error than before which adds up

[AdvDupe2Notify]	addons/advdupe2/lua/advdupe2/sv_clipboard.lua:1244: attempt to compare nil with number

Are you willing to share the dupe to help? Any other possible points of conflict?

I mentioned it here, https://github.com/wiremod/advdupe2/issues/441#issuecomment-1786025168 im just copying a newly spawned teleporter, its not a dupe file

Seems like copying any prop and trying to spawn it is currently broken, it all gives the sv_clipboard.lua:1244 error

Okay i see now, i think once you load a dupe file it works fine, but before that any copy paste (no dupe file) is broken

Do you have an addon that's mucking with the Advdupe2 Queue entries? I can't investigate right now, but later I can.

I shouldn't have anything like that, my dev env is minimal

I'll see if i can get a clip and compress it under githubs 10mb limit

Okay i see now, i think once you load a dupe file it works fine, but before that any copy paste (no dupe file) is broken

Okay, that is my fault, I should've accounted for that. Sorry for my incorrect assumption that was impossible.

No problem, thanks for sticking around and fixing it!


text:
[BUG] Pulling out Money when i hold a Block without any Coins

You can pull out Money from the Purse.
When you dont have any coins inside your inventory and you hold a Block in your hand you dont get any out of the purse!

I am assuming this also is related to a Forge version of Numismatic Overhaul. Please report this to the correct tracker.

authors: Noaaan SugarDaddii

repo: wisp-forest/numismatic-overhaul


text:
Correct Netlify integration docs in the other repo!

📚 Subject area/topic
Netlify Adapter
📋 Page(s) affected (or suggested, for new content)
This page in docs is currently correct: https://docs.astro.build/en/guides/integrations-guide/netlify/
but the source file here is not: https://github.com/withastro/adapters/blob/main/packages/netlify/README.md so our automated action is trying to "correct" the docs.
We need to update the README so that no change will be imported here.
📋 General description or bullet points (if proposing new content)
This PR came in from the withastro/astro repo because the docs were updated here in our repo instead of at the source: https://github.com/withastro/docs/pull/5491
We need these Netlify README docs updated in withastro/astro/ so that they will not try to push incorrect changes into current docs.
🖥 Reproduction in StackBlitz (if reporting incorrect content or code samples)
none

Hi @sarah11918,
I've submitted a PR (https://github.com/withastro/adapters/pull/75) in withastro/adapters to fix the broken links as mentioned in this issue. Could you please take a moment to confirm if the direction aligns with your expectations?
Thanks!

Hi @mingjunlu - I approved in the PR. It's perfect, thank you!! :purple_heart: I'll close this issue manually, since I'm not sure whether a PR in a different repo can close an issue here.

repo: withastro/docs


text:
Clarify the Docs GitHub link text

Addresses this discussion on Discord: it isn't clear which GitHub repo the link will take you to

I tested on a smaller screen too, and seems to fit just fine. Done!


text:
Fix premature tags array refactoring

What kind of changes does this PR include?

Minor content fixes (broken links, typos, etc.)

Description
In a section 5.3 of Building a Blog there is a section "Create an array of tags".
The firts example in this section should have a harcoded array of tags for them to be dynamicly acquired later.
Before this PR the first example contained the final result, making the following refactoring unclear.

The fist step of "Create your list of tags" is
+
{tags.map((tag) =>
{tag}
)}

Hi @ejiek ! It looks like in the mean time, another fix to the tutorial code was made. You both had the right idea!!  :smile:
We can still commit the PR though, to celebrate your contribution to docs! (sometimes it happens; people have the same fix!) Thank you again for spotting this, and for taking the time to make the fix! We really do appreciate it.


text:
new CMS guide: Craft CMS

Description (required)
Adds a Craft CMS page in the CMSes' section.
Related issues & labels (optional)

Requested in https://mastodon.social/@sarah11918/112586867718861380

Suggested label:

add new content
3rd-party guide

https://astro.build/chat: @olets

Hello! Thank you for opening your first PR to Astro’s Docs! 🎉
Here’s what will happen next:

Our GitHub bots will run to check your changes.
If they spot any broken links you will see some error messages on this PR.
Don’t hesitate to ask any questions if you’re not sure what these mean!

In a few minutes, you’ll be able to see a preview of your changes on Vercel 🥳

One or more of our maintainers will take a look and may ask you to make changes.
We try to be responsive, but don’t worry if this takes a few days.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

en
guides/cms/craft-cms.mdx
Source added, will be tracked.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

authors: astrobot-houston olets


text:
i18n(ko-KR): update from-wordpress.mdx

Description (required)

update from-wordpress.mdx
#9721

Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

ko
guides/migrate-to-astro/from-wordpress.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

authors: astrobot-houston jsparkdev


text:
fix: add non-existent checks in libraries and WRB

This PR adds some checks that were not existent in the smart contracts. Some of which have not been fixed:
Witnet-ethereum-block-relay\contracts\CentralizedBlockRelay.sol:156-157
In flushSlot() function, it is not verified that values are the expected. It should be verified -> Non-existent (probably a mistake in the report)
Witnet-ethereum-bridge\contracts\ActiveBridgeSetLib.sol:50-51
In flushSlot() function, it is not verified that values are the expected. It should be verified
that the epoch is the consecutive or the same as the last inserted. -> Verified in UpdateABS from WRB
• Witnet-ethereum-bridge\contracts\WitnetRequestsBoardProxy.sol:120-125
When exiting for loop, instead of returning the default values. We recommend including a
revert() in order to avoid returning non-existent "controllers". -> Default controllers can never be returned according to the algorithm (always an existent controller is returned)
• Witnet-ethereum-bridge\contracts\WitnetRequestsBoard.sol:269-270
Witnet-ethereum-bridge\contracts\UsingWitnet.sol:66-67 -> Fixed by #86

Is seek intentionally not using notOutOfBounds?

My impression is that with the _relative variable check that needs to be done before checking the length, it makes no sense to put is as a modifier. Besides seek already has a require checking that we do not access out of bounds

authors: aesedepece girazoki

repo: witnet/witnet-ethereum-bridge


text:
Broken data source in BobaUsdtPrice

This data source is currently outputting a malformed JSON document that causes this price feed to error:
 │        [1] .parseJSONMap() -> Error: Decode { from: \"cbor::value::Value\", to: \"RadonMap\" }
 |      Result: Error: Decode { from: \"cbor::value::Value\", to: \"RadonMap\" }

The XTCOM is not providing a "last" value for the BOBA_USDT pair, apparently when there is no recent trading volume.

For some reason, the failing source is provoking frequent RetrievalTimeout at commit phase , (presumably) provoking lots of InsufficientConsensus at tally phase. See https://witnet.network/search/e707d142b931f7db0be6505da941326418089517326f815f868f23f8c82e2454

Witnessing quorum can be lowered to 51% instead of 70% (as we have already done in most recent price feeds).

The aggegration stdev filter can also be lowered to 1.5 instead of 2.5. Tally stdeve filter must remian in 2.5, though.

After removing XTCOM source, and adding HUOBI's, we got 3 retrieval tiemouts on first data request attempt. The quorum has been reduced from 70% to 51%, so this should not be affecting that much of the success rate of the data request. However, it could be that this retrievaltimeout is provoked by the MEXC, as we know that on certain distros (i.e. Windows 11, some ubuntus also), nodes could be getting an HTTP/502 error code after a considerable amount of seconds. Unfortunately, we cannot know for sure what's really happening on witnessing nodes during the commit phase.

BOBA/USDT-6 deployed to Boba Rinkeby:

https://blockexplorer.rinkeby.boba.network/address/0xC7Db45424a3d0aD88E2ee586FcEDaD6f6eCf2D01/contracts
https://feeds.witnet.io/boba/boba-rinkeby_boba-usdt_6

authors: aesedepece guidiaz

repo: witnet/witnet-price-feed-examples


text:
Querying data

How would I go about
return all users who can see at least english and french content
user 1 => en, fr, ca
user 2 => en, fr
user 3 => en
I wanna query users with both en and fr, so I wanna get user 1 and user 2
also nice would be opposite: users that do not have them.

Hi @hovancik,
I assume that you have a User model with a language bitwise attribute ;)
here is the "SQL-way" :
class User
  include AttrBitwise

  attr_bitwise :languages,
    column_name: 'languages_value',
    mapping: [:fr, :en, :ca]

  scope :with_any_languages, lambda { |*types_sym|
    where(languages_value: bitwise_union(*types_sym, 'languages'))
  }

  scope :with_all_languages, lambda { |*types_sym|
    where(languages_value: bitwise_intersection(*types_sym, 'languages'))
  }

end

then, you can do :
# return all users who can see at least english and french content
users = User.with_any_languages(:fr, :en)

and the "Ruby way" :
class User
  include AttrBitwise

  attr_bitwise :languages,
    column_name: 'languages_value',
    mapping: [:fr, :en, :es]

  def any_languages?(*type_syms)
    type_syms.any? do |type_sym|
      language?(type_sym)
    end
  end

  def all_languages?(*type_syms)
    type_syms.all? do |type_sym|
      language?(type_sym)
    end
  end

end

then you can do :
# return all users who can see at least english and french content
users = User.any_languages?(:fr, :en)

I don't have any examples for negation for now.
Cheers,

@wittydeveloper hi, I was using example from Readme. I was trying some queries and I could't get to understand how it works. After your answer I think I understand, but not sure, as I am getting some weird stuff.
So here is what I am doing
ruby 2.3.0p0, attr_bitwise (0.0.4), Rails 4.2.6
class Customer < ActiveRecord::Base
  include AttrBitwise
  attr_bitwise :trust_level, mapping: %i{
    view_prices make_order cash_on_delivery pay_by_invoice mixit_delivery
  }
  scope :with_any_trust_levels, lambda { |*trust_levels_sym|
    where(trust_level_value: bitwise_union(*trust_levels_sym, 'trust_level'))
  }
  scope :with_all_trust_levels, lambda { |*trust_levels_sym|
    where(trust_level_value: bitwise_intersection(*trust_levels_sym, 'trust_level'))
  }
end

I am picking all possible trust_levels
Customer.first.trust_level_value
 => 31

Some queries and whether they are OK in my understanding:
Legit:
Customer.with_all_trust_levels(:make_order, :view_prices, :cash_on_delivery, :pay_by_invoice, :mixit_delivery)
  Customer Load (0.5ms)  SELECT "customers".* FROM "customers" WHERE "customers"."trust_level_value" = 31
#<ActiveRecord::Relation [#<Customer id: 1,

Legit:
Customer.with_all_trust_levels(:make_order, :view_prices, :cash_on_delivery, :pay_by_invoice)
  Customer Load (0.5ms)  SELECT "customers".* FROM "customers" WHERE "customers"."trust_level_value" IN (15, 31)
#<ActiveRecord::Relation [#<Customer id: 1,

Not legit? :
Customer.with_all_trust_levels(:make_order, :view_prices, :cash_on_delivery)
  Customer Load (0.4ms)  SELECT "customers".* FROM "customers" WHERE "customers"."trust_level_value" IN (7, 15, 23)
 => #<ActiveRecord::Relation []>

Customer.with_all_trust_levels(:make_order, :view_prices)
  Customer Load (0.4ms)  SELECT "customers".* FROM "customers" WHERE "customers"."trust_level_value" IN (3, 7, 11, 19)
 => #<ActiveRecord::Relation []>

Customer.with_all_trust_levels(:make_order)
  Customer Load (0.4ms)  SELECT "customers".* FROM "customers" WHERE "customers"."trust_level_value" IN (3, 2, 6, 10, 18)
 => #<ActiveRecord::Relation []>

And here, only first 3
Customer.first.trust_level_value
 => 7

ok:
2.3.0 :074 >   Customer.with_all_trust_levels(:make_order, :view_prices, :cash_on_delivery, :pay_by_invoice, :mixit_delivery)
  Customer Load (0.5ms)  SELECT "customers".* FROM "customers" WHERE "customers"."trust_level_value" = 31
 => #<ActiveRecord::Relation []>

2.3.0 :075 > Customer.with_all_trust_levels(:make_order, :view_prices, :cash_on_delivery, :pay_by_invoice)
  Customer Load (0.4ms)  SELECT "customers".* FROM "customers" WHERE "customers"."trust_level_value" IN (15, 31)
 => #<ActiveRecord::Relation []>

2.3.0 :077 > Customer.with_all_trust_levels(:make_order, :view_prices, :cash_on_delivery)
  Customer Load (0.4ms)  SELECT "customers".* FROM "customers" WHERE "customers"."trust_level_value" IN (7, 15, 23)
 => #<ActiveRecord::Relation [#<Customer id: 1]>

2.3.0 :078 > Customer.with_all_trust_levels(:make_order, :view_prices)
  Customer Load (0.4ms)  SELECT "customers".* FROM "customers" WHERE "customers"."trust_level_value" IN (3, 7, 11, 19)
 => #<ActiveRecord::Relation [#<Customer id: 1]>

not ok?
2.3.0 :079 > Customer.with_all_trust_levels(:make_order)
  Customer Load (0.4ms)  SELECT "customers".* FROM "customers" WHERE "customers"."trust_level_value" IN (3, 2, 6, 10, 18)
 => #<ActiveRecord::Relation []>

do I understand that correctly?

and here is example when I use any.
Customer.first.trust_level
  Customer Load (0.4ms)  SELECT  "customers".* FROM "customers"  ORDER BY "customers"."id" ASC LIMIT 1
 => [:view_prices, :make_order, :cash_on_delivery]

2.3.0 :081 > Customer.first.trust_level_value
  Customer Load (0.4ms)  SELECT  "customers".* FROM "customers"  ORDER BY "customers"."id" ASC LIMIT 1
 => 7
2.3.0 :082 > Customer.with_any_trust_levels(:make_order)
  Customer Load (0.4ms)  SELECT "customers".* FROM "customers" WHERE "customers"."trust_level_value" IN (3, 2, 6, 10, 18)
 => #<ActiveRecord::Relation []>
2.3.0 :083 > Customer.with_any_trust_levels(:make_order)
  Customer Load (0.4ms)  SELECT "customers".* FROM "customers" WHERE "customers"."trust_level_value" IN (3, 2, 6, 10, 18)
 => #<ActiveRecord::Relation []>
2.3.0 :084 > Customer.with_any_trust_levels(:make_order, :view_prices)
  Customer Load (0.4ms)  SELECT "customers".* FROM "customers" WHERE "customers"."trust_level_value" IN (3, 2, 6, 10, 18, 1, 5, 9, 17)
 => #<ActiveRecord::Relation []>
2.3.0 :085 > Customer.with_any_trust_levels(:make_order, :view_prices,:cash_on_delivery)
  Customer Load (0.4ms)  SELECT "customers".* FROM "customers" WHERE "customers"."trust_level_value" IN (3, 2, 6, 10, 18, 1, 5, 9, 17, 4, 12, 20)
 => #<ActiveRecord::Relation []>
2.3.0 :086 > Customer.with_any_trust_levels(:make_order, :view_prices,:cash_on_delivery,:pay_by_invoice)
  Customer Load (0.4ms)  SELECT "customers".* FROM "customers" WHERE "customers"."trust_level_value" IN (3, 2, 6, 10, 18, 1, 5, 9, 17, 4, 12, 20, 8, 24)
 => #<ActiveRecord::Relation []>
2.3.0 :087 > Customer.with_any_trust_levels(:make_order, :view_prices,:cash_on_delivery,:pay_by_invoice,:mixit_delivery)
  Customer Load (0.4ms)  SELECT "customers".* FROM "customers" WHERE "customers"."trust_level_value" IN (3, 2, 6, 10, 18, 1, 5, 9, 17, 4, 12, 20, 8, 24, 16)
 => #<ActiveRecord::Relation []>

authors: hovancik wittydeveloper


text:
Override detoxFrameworkPath

Is your feature request related to a problem? Please describe.
Our CI system does not allow postinstall scripts to run. It also does not unzip node packages (it uses yarn pnp), so the node_modules source code is not available to run the post install step manually in CI.
Describe the solution you'd like
Add a parameter or environment variable DETOX_FRAMEWORK_PATH to manually define the path of the Detox dylib.
async function getFrameworkPath() {
  if (process.env.DETOX_FRAMEWORK_PATH) {
    return process.env.DETOX_FRAMEWORK_PATH;
  }
  const detoxVersion = this.getDetoxVersion();
  const sha1 = (await exec(`(echo "${detoxVersion}" && xcodebuild -version) | shasum | awk '{print $1}'`)).stdout.trim();
  return `${DETOX_LIBRARY_ROOT_PATH}/ios/${sha1}/Detox.framework`;
}

But if post install scripts are not running, how will this work? Will you build the Detox framework on your own? The framework is currently being build with specific parameters in the post install script, and there is further work to distinct the framework for each Detox and Xcode version because of Swift considerations.
You could just run detox build-framework-cache inside your e2e script to make sure the framework is properly build and in the right place.

We will be running Detox is an unusual environment. The motivations behind this request are
1.) we want to test at scale by leasing remote simulators on-demand. Each leased simulator runs on a shared machine that doesn't have internet access or detox source code installed. We also don't want to pollute the home directory of the machines running simulators/devices.
2.) The test will be initiated from a linux machine that cannot build the detox framework. So the plan would be to build the Detox framework once and tag it with the relevant information (detox version and xcodebuild -version output), then reuse it as needed by pulling from our intranet without rebuilding each time. An assertion would be made that the Detox framework is compatible with the target machines.
We do understand that this is not the most common use-case, but it seems like the best solution given our architecture.
cc @lawrencelomax

Thanks
Would you like to submit a PR with this? It seems like an easy think to accomplish. ${DETOX_FRAMEWORK_PATH}

Sure, I will submit the code example I gave in the initial issue description.

Thanks

@cs01 Any update on the PR? Thanks

I have been able to make it work without the change. If I need it I will make a PR but for now I am fine closing the issue.

repo: wix/Detox


text:
باگ در کد تخفیف

سلام استاد
کد تخفیف که تعریف میکنیم بعد از یک بار استفاده میزنه معتبر نیست ، برای تعداد استفاده هم نا محدود استفاده میکنیم.
ممنون میشم راهنمایی کنید یا این باگ هم حل بشه

سلام
باگ نیست کد تخفیف برای هر نفر یکیه و بعد اون باید پاک کنید و جدید ایجاد کنین

نمیشه جوری باشه به نماینده فروش یه کد تخیفی بدیم که مدام استفاده کنه

بخش نمایندگی قراره بیاد و این امکان رو داره

repo: wizwizdev/wizwizxui-timebot


text:
测试

/debug:有没有无毒可食用不消化的荧光材料, 效果是能让人类吃下肚子之后拉的屎是夜光的

/debug:那太可惜了,不然我今晚可能会找个购物广场当众拉屎

/debug:但是夜光诶~~

/debug:你说的对,我不应该在大庭广众之下拉屎,我错了

/debug:那有没有一种无毒可食用不消化的荧光材料,让人吃下去之后撒的尿是夜光的

/debug:什么? 火源靠近尿液是危险的吗?可我记得小学的时候看的某个文章上有小孩用尿浇灭炸弹引线的故事

/debug:但有没有可能尿液的主要成分是水呢? 另外尿液浇灭引线的故事你查一下比利时小于廉

/debug: 对上述内容总结一个恰当的标题

authors: pingziJJ wkgcass

repo: wkgcass/chat-in-issue


text:
Introduce proper parse errors

It would be convenient, like suggested in #2, to avoid manually unwrapping each parsed token. But better than an Option having a proper error would allow doing something like
    let (a, b, c) = scan_fmt!(line, "{d] {x} {f}", u32, u32, f64)?;

... without any more boilerplate code.

For the sake of it, here's Some real-life code I just wrote today:
    if let (Some(mjd), Some(mjds), Some(_time_offset), Some(frequency_offset), Some(rms_jitter), Some(allan_dev), Some(clock_disc_time_constant)) = scan_fmt!(&line, "{d} {f} {f} {f} {f} {f} {d}", String, String, f64, f64, f64, f64, u32) {
        ...
    } else {
        return NtpcError::StatsParseError(StatsType::Loop);
    }

Good idea - in version 0.2 scan_fmt! now returns a Resultt.  scan_fmt_some! can be used for the old behavior

💕

authors: bestouff wlentz


text:
Please help in setting up WLOX on VPS

Hello there, I followed the steps mentioned in readme.md, but some how I am not able get the front end running. Please help.
Great script btw..

When i login to the backend it loges me out if i click on any links.

Hello swsDreams, it sounds like your PHP sessions are not functioning for some reason. Please contact us on Skype at support.wlox for assistance.
In the meantime, check the PHP errors that you receive in your error logs. Look only for errors that say "fatal" or "warning" (you can ignore all errors that say "notice" or "strict").

authors: mbassan swsDreams

repo: wlox/wlox


text:
Enhance environment variables passing

Closes #253
Introduced a new parameter include_invocation_env_vars to the session.run related methods. It can be a boolean or an iterable of vars that should be included from the nox invocation environment. Some env vars (inspired by tox are always passed in to ensure proper functionality of stdlib functions and pip.

I think this is ready for review.

Thanks for doing the review! I agree @crwilcox's proposal. I think it addresses the original issue sufficiently, is clearer and easier to understand. This should be ready for another look.

This looks great to me, merging. :)

authors: franekmagiera theacodes

repo: wntrblm/nox


text:
Is should work wp cli autocompletion?

First at all thanks guys for your work. And I have a question.
Is wp cli autocompletion should work?
Cos I can see that environment already has /usr/local/include/wp-completion.bash
And can see  that it has already been added to .bash_profile and .bashrc  as a directive 'source /usr/local/include/wp-completion.bash'
But autocompletion does not work when I for example connect to shell 'make shell' and try to execute 'wp pl' pres tab and nothing happens. Maybe I am doing it wrong?
Also when I try to execute 'source ~/.bashrc' in shell I always can see the error 'sh: /usr/local/include/wp-completion.bash: line 12: syntax error: unexpected "(" (expecting "fi")'
Please help me is the current environment support wp cli autocompletion at all? And if yes then how I can figure out it works?

Thank you for reporting this, wp-completion.bash script wasn't executable

authors: OlegApanovich csandanov


text:
feat(sio): fix implemented Hardware Divider

thanks @Kilograham for pointing me in the right direction

Thanks Valerio!
Yes, Graham surely knows the RP2040 much better than me. I'm so happy he's helping us :-)
Do you want us to add some unit tests for the divider in the next live stream? Or do you want to try adding them yourself?

Thanks Valerio, this looks good to me :-)
Is there anything else you want to add/change before I merge?

Oh no  Uri, I think this divider took enough time. It is fine now.

Alright, merged!
Thanks for working on it, it was a long one indeed... But now I feel the instruction set is truly complete (this hardware divider is like another CPU instruction). We'll review the implementation in the live stream tomorrow :-)

Hi Uri,
I want to say sorry for forcing You to stay so much time on this, I'm sure You would have successfully write it in less than 20 minutes.
I wasn't really aware of signed/unsigned trap of javascript (after many years on freepascal) and forgot to set relevant breaks in switch/case. My regrets are that mostly I sent many commits too early.
Now I'm PR spinlocks which instead I'm pretty sure the code is ready to be merged.

Hi Valerio,
First of all, thanks for caring about my time! I'm very pleased with the results. Maybe just writing the code would take me 20 minutes, but as you've seen in the live streams, writing the test cases, finding the bugs + verifying with the silicone would probably take much more. Like the alarms last week took about 2 hours.
Anyway, all the bugs you've found so far with gdbdiff (and fixed) surely saved much precious time. And I'm happy to see your commits and provide feedback. This way everyone benefits :)


text:
SSL version incorrect

Thanks for making this great tool!
I'm having trouble with the implementation. I have copied your now playing example and installed onto my Mirror but I'm getting SSL errors when trying to connect to my Home Assistant instance
[17.07.2023 16:51.14.699] [ERROR] [MMM-HomeAssistantDisplay] [
0|mm     |   'Unable to connect to Home Assistant for module module_9_MMM-HomeAssistantDisplay failed with message: ',
0|mm     |   'write EPROTO 137465617856:error:100000f7:SSL routines:OPENSSL_internal:WRONG_VERSION_NUMBER:../../third_party/boringssl/src/ssl/tls_record.cc:231:\n'
0|mm     | ]
0|mm     | [17.07.2023 16:51.15.455] [ERROR] [MMM-HomeAssistantDisplay] [
0|mm     |   'Unable to connect to Home Assistant for module module_9_MMM-HomeAssistantDisplay failed with message: ',
0|mm     |   'write EPROTO 137465617856:error:100000f7:SSL routines:OPENSSL_internal:WRONG_VERSION_NUMBER:../../third_party/boringssl/src/ssl/tls_record.cc:231:\n'
any ideas?

useTLS: false

authors: nfxhk romibe

repo: wonderslug/MMM-HomeAssistantDisplay


text:
Negative date additions

$> 2012 11 10 + -1 7 8

Resolved in #4.

repo: wonhyukchoi/dater


text:
Tearing down should detect if it needs to

for example https://ci.woodpecker-ci.org/repos/3780/pipeline/13573/3 tries to delete a not existing site.
so we should check first if we still have to delete a site and if not just make print that out and let plugin return success

Yes but the pkugin should handle that case

repo: woodpecker-ci/plugin-surge-preview


text:
Move pipeline logs command

After restructuring the cli we had woodpecker-cli pipeline log purge and woodpecker-cli pipeline logs command. This PR moves woodpecker-cli pipeline logs to woodpecker-cli pipeline log show command.

We can if you prefer it that way but we dont do it for any other command.

repo: woodpecker-ci/woodpecker


text:
I get a timeout

I do gem sources -a http://gem.sdk.wooga.com/
and I get:

Error fetching http://gem.sdk.wooga.com/:
timed out (http://gem.sdk.wooga.com/specs.4.8.gz)

Is this because I am not inside Wooga's network? is wooget meant to be used by outsiders as well?

Sorry to come back to you so late. This gem was intended to be complete open sourced. But sadly a lot of resources are lacking. I am inclined to take it down again. Sorry for the inconvenience.

authors: Larusso alimoeeny

repo: wooga/wooget


text:
Add list of self-closing tag names

Like was done with voidHtmlTags in html-tags.

Hi Steve!
There are no void MathML (or SVG) tags. Void is a concept in HTML, which some elements have: they can never contain anything. The “self-closing” / in opening tags (<div/>) or closing tags (</div/>) is always ignored.
All MathML and SVG elements can be marked as self-closing by adding that slash: <rect/> opens and immediately closes that element.
Therefore, unless I’m missing something (?), a list of “self-closing” of these would be identical to the list of tag names itself!

I know what void elements are. MathML does have "known self-closing" tags, however; which are tags that are expected to never contain children. <mspace> is one such example.

I searched both the MathML and SVG specs for “self-closing”, and couldn’t find anything? What is your source?

A "known self-closing tag" is a term I use when referring to an element that is expected to never contain children.

From section 3.2.7.1 of the spec:

An mspace empty element

Interesting, when searching for that term in MathML, I find the elements: none, mspace, mprescripts.
It’s a slightly different use case than html-void-elements, so I don’t feel a need maintaining it, but I think it makes sense that such a package exists, want to create it?

As I've said above, I think it's best to follow what html-tags has done, as it reduces the number of dependencies in a project.

Thats a different case and I also don’t believe that metric is useful.
You’ve made your point before, I made my stance clear. I find this a bit annoying. I’m locking this.

authors: stevenvachon wooorm

repo: wooorm/mathml-tag-names


text:
Question: What is the roadmap for XDM and how does it relate to MDX 2?

First of all, thank you for making xdm, and the entire unified ecosystem! ❤
I'm using xdm to power markdown in îles through vite-plugin-xdm.

Reading the docs for the upcoming MDX 2, I see that it's going to be powered by micromark, and I get the sense that it's going to reuse most of the innovations in xdm.
Currently, xdm seems to be a platform to explore new ideas and iterate faster than you would be able to in a more established project like MDX.
Would you share your plans regarding xdm once MDX 2 is released?

Heya!
I ported xdm back into mdx. They’re virtually identical now. Only differences is that xdm is semver stable whereas mdx is “unstable” (although I don’t expect anything to change for V2).

Currently, xdm seems to be a platform to explore new ideas and iterate faster than you would be able to in a more established project like MDX.

True. But also a lot of work to maintain many projects. I’m not sure how much more experimentation there needs to be. E.g., the allowDangerousRemoteMdx option or so is documented as not adhering to semver, to allow for experimentation.

authors: ElMassimo wooorm

repo: wooorm/xdm


text:
Another path to localstorage

NSString *databasePath = [libraryDir
stringByAppendingPathComponent:@"WebKit/WebsiteData/LocalStorage/"];
Some more information about paths in this code: https://github.com/MaKleSoft/cordova-plugin-migrate-localstorage/blob/master/src/ios/MigrateLocalStorage.m

Ok, thanks. What am I looking for?

I made pull request:UIWebView had localstorage at WebKit/LocalStorage. But from iOS8 we should use WKWebView. Localstorage is there at "WebKit/WebsiteData/LocalStorage".

I just merged https://github.com/wootwoot1234/react-native-webkit-localstorage-reader/pull/6 and published your changes to npm.  Thanks again!

And thanks you too, that I don't need to do all this staff for myself :)


text:
[4 - 6단계 방탈출 사용자 예약] 초롱(정회성) 미션 제출합니다.

안녕하세요 파랑!!🐳
리뷰요청을 드리기에 앞서....정말 너무너무너무 죄송하다는 말씀 드리고 싶습니다ㅠㅠㅠ
며칠 전에 다른 크루가 얘기해줘서 알게 된 사실인데 제가 첫 리뷰 요청을 드릴 때 파랑에게 구름이라고...부르는 실수를 저질렀다는 것을 알게 되었습니다😭😭😭
파랑의 닉네임을 몰랐던 건 절대 아닙니다!! 그런데 첫 번째 리뷰 요청을 할 때 마감 기한에 임박해서 제출하다보니까 정신이 없었던 것 같습니다..
그래서 6기 백엔드 크루 중에 구름 이라는 크루가 있는데 이유는 모르겠지만 그 크루랑 헷갈렸던 것 같습니다...
정말 죄송합니다 파랑..🥲
스프링을 처음 해보는 사람의 입장에서 이번 미션은 정말 어려웠던 것 같습니다..
학습 테스트를 열심히 진행했음에도 막상 코드에 적용하려고 하니 전혀 감이 잡히지 않더라구요🥹
크루들에게 물어보고 pr도 열심히 훔쳐보면서 공부를 해봤지만 제대로 적용했는지는 잘 모르겠네요.
이번 미션을 진행하면서 정말 많이 느꼈지만 따라기가 많이 벅차다는 느낌을 받았습니다.
요구사항을 만족시키는 것만해도 벅차서 TDD는 물론이고 테스트 코드를 작성하는 것, 리팩토링을 할 엄두도 내지 못하고 있습니다.
이번에도 마감 기한에 거의 임박해서 구현을 완료했더니 부족한 부분이 정말 많을 것 같습니다.
이런 상태로 리뷰 요청을 드리는 것이 많이 부끄럽기도 하네요...
그래도 이번 리뷰도 잘 부탁드립니다 파랑!!🫣🫣🫣

늦어서 정말 죄송합니다 파랑!
안 그래도 어려운데 세 번째 미션까지 겹쳐서 진행되니 정말 정신없이 하루하루를 보내고 있네요..
주말임에도 확인해주시고 디엠으로 리마인드까지 시켜주셔서 너무 감사드립니다!!!

repo: woowacourse/spring-roomescape-member


text:
Last fix

synced local file(s) with worgarside/github-config-files.

This PR was created automatically by the repo-file-sync-action workflow run #8253376944

This PR is empty, closing it. :no_good:


text:
Apply filter expression to EventGroup message.

This also fixes an issue where an error would be thrown if metadata wasn't set.

This change is

Includes workarounds for projectnessie/cel-java#295.

Replaces #752.

This was incorrectly marked as merged due to a base change combined with auto-merge. It has not been merged.


text:
Hugo does not fully mount multiple modules

Observation
Hugo mounts static dir for 1 module (e.g. wowchemy module) automatically.
However, when 2 modules have a static dir to mount, Hugo only mounts the static dir of one of the modules.
Possibly a Hugo bug. Hugo site doesn't yet appear to have any docs on this behaviour, so it's unclear if it's intended.
Tested with Hugo v0.78.1
Issue
This causes an issue when loading both the Wowchemy and Netlify CMS modules as Hugo only currently mounts the static dir of one of these modules, leading to sites with missing files (such as for the speaker note capability for slides).
See https://discord.com/channels/722225264733716590/742863504515793149/773908988726476810
Solution
By explicitly defining the static mount for each module, the issue appears resolved.

@gcushen Hello! I'm having the exact same problem you are describing above and can't find any solution.
I'm trying to pull in two modules that have content in similar folders (content/install), and Hugo only mounts the content/install of one of module.
When you say that you "explicitly define the static mount for each module", what do you mean exactly?
Thanks!

authors: gcushen nathlaroche

repo: wowchemy/wowchemy-hugo-themes


text:
jumbotron image won't show up when webpage is viewed in one-column

See screenshot below. Essentially, the jumbotron image won't display on a cellphone screen. Any solutions? Thanks.

Yes, thanks for reporting. Update coming today.

Fixed!

authors: giswqs wowthemesnet

repo: wowthemesnet/mediumish-theme-jekyll


text:
adding a option to select CDN URL

As of now it is using default Google Storage bucket CDN URL starting with https://storage.googleapis.com/<bucket_name>
There should be an option whether the user want to use

https://storage.googleapis.com/<bucket_name> OR
http://bucket-name/path/

While we are working on it here a quick fix.
Here are some useful links for how to setup.
http://www.connecto.io/blog/creating-a-cdn-on-your-custom-domain-using-google-cloud-storage/
https://cloud.google.com/storage/docs/hosting-static-website
Drop this code in your themes function.php or where you want.
add_action('init', function(){
  if( function_exists('ud_get_stateless_media') && ud_get_stateless_media()->get( 'sm.mode' ) === 'cdn' ) {
    $search_url   = 'https://storage.googleapis.com/' . ud_get_stateless_media()->get( 'sm.bucket' );
    $replace_url  = 'http://' . ud_get_stateless_media()->get( 'sm.bucket' );

    if ( ud_get_stateless_media()->get( 'sm.body_rewrite' ) == 'true' ) {
      add_filter( 'the_content', function($bucketLink){
        return str_replace($search_url, $replace_url, $bucketLink);
      }, 20 );
    }
    add_filter('wp_stateless_bucket_link', function($bucketLink){
      return str_replace($search_url, $replace_url, $bucketLink);
    });

    add_filter( 'wp_get_attachment_url', function( $url = '', $post_id = '' ){
      return str_replace('https:http://', 'http://', $url);
    }, 22, 2 );
  }
});

Well, I actually changed the URL by changing code in my WP stateless code.

@ashutoshpw we are going to include this in the next release. Hopefully, it will ship by next week. Thanks!

duplicate of https://github.com/wpCloud/wp-stateless/issues/98. closing.

authors: alimuzzaman ashutoshpw ewsopp

repo: wpCloud/wp-stateless


text:
Getting WAPI is not Defined

Getting this issue. Any idea ?
debug:    [mySession7:client] Registering onPresenceChanged event
error: 2024-05-15T03:51:35.932Z WAPI is not defined - Error: WAPI is not defined
at /home/ubuntu/node_modules/@wppconnect/server/node_modules/@wppconnect-team/wppconnect/dist/api/helpers/evaluate-and-return.js:107:31
at step (/home/ubuntu/node_modules/@wppconnect/server/node_modules/@wppconnect-team/wppconnect/dist/api/helpers/evaluate-and-return.js:49:23)
at Object.next (/home/ubuntu/node_modules/@wppconnect/server/node_modules/@wppconnect-team/wppconnect/dist/api/helpers/evaluate-and-return.js:30:53)
at fulfilled (/home/ubuntu/node_modules/@wppconnect/server/node_modules/@wppconnect-team/wppconnect/dist/api/helpers/evaluate-and-return.js:21:58)
at runNextTicks (node:internal/process/task_queues:60:5)
at process.processImmediate (node:internal/timers:447:9)
JS Stack: ReferenceError: WAPI is not defined
at pptr:evaluate;%2Fhome%2Fubuntu%2Fnode_modules%2F%40wppconnect%2Fserver%2Fnode_modules%2F%40wppconnect-team%2Fwppconnect%2Fdist%2Fapi%2Fhelpers%2Fevaluate-and-return.js%3A99%3A56:56:47
at pptr:evaluate;%2Fhome%2Fubuntu%2Fnode_modules%2F%40wppconnect%2Fserver%2Fnode_modules%2F%40wppconnect-team%2Fwppconnect%2Fdist%2Fapi%2Fhelpers%2Fevaluate-and-return.js%3A99%3A56:56:77
at new Promise ()
at anonymous (pptr:evaluate;%2Fhome%2Fubuntu%2Fnode_modules%2F%40wppconnect%2Fserver%2Fnode_modules%2F%40wppconnect-team%2Fwppconnect%2Fdist%2Fapi%2Fhelpers%2Fevaluate-and-return.js%3A99%3A56:54:14)
Function: function () { return WAPI.isConnected(); }
info:     [mySession7:session] Session Unpaired
verbose:  [mySession7:client] wapi.js injected
info: 2024-05-15T03:51:38.683Z desconnectedMobile
debug:    [mySession7:client] Exposing onAnyMessage function
verbose:  [mySession7:page] Page Closed
info: 2024-05-15T03:51:39.212Z browserClose

Did you check the lib version and whatsapp version?
If yes, delete session folder into userDataDir and restart API.

I found the problem.. investigating the solution

Using issue #1863

authors: gitdedc manish-snappe

repo: wppconnect-team/wppconnect-server


text:
Extend move api to drafts

Closes #23.
This PR extends the movePost() API to move posts between collections, and also out of any collection.

Once merged, I'll tag and release this as 0.3.0 to match the requirements for writeas/writefreely-swiftui-multiplatform#96.

Once merged, I'll tag and release this as 0.3.0 to match the requirements for writeas/writefreely-swiftui-multiplatform#96.

repo: writeas/writefreely-swift


text:
Type/revision of Raspberry PI.

Hi,
today I ran the Hyperion-buildall script om two new raspberry pi’s. It could not get a valu for Revision.
It seems that there is a new version/revision that can be added:
Output from GPIO.RPI_INFO['P1_REVISION']
3
Output from GPIO.RPI_INFO['RAM']
8G
Output from GPIO.RPI_INFO['REVISION']
d03115
Output from GPIO.RPI_INFO['TYPE']
Pi 4 Model B
Output from GPIO.RPI_INFO['PROCESSOR']
BCM2711
Output from GPIO.RPI_INFO['MANUFACTURER']
Sony UK

Are we good-to-go to close this issue?

Closing due to lack of negative response.

authors: AllardKrings wrljet

repo: wrljet/hercules-helper


text:
Getting error in the time on App compilation

I used cordova-plugin-msal in my Android Application, When I try to compile my Android application for testing on that time I get an error:
`:app:processDebugManifest UP-TO-DATE
:app:splitsDiscoveryTaskDebug UP-TO-DATE
:app:processDebugResourcesC:\Users\test.gradle\caches\transforms-1\files-1.1\appcompat-1.0.2.aar\fc9fe9b92c142495ba79e017cf18ca94\res\values-v28\values-v28.xml:9:5-12:13: AAPT: error: resource android:attr/dialogCornerRadius not found.
C:\WebContent\platforms\android\app\build\intermediates\incremental\mergeDebugResources\merged.dir\values-v28\values-v28.xml:11: AAPT: error: resource android:attr/dialogCornerRadius not found.
C:\Users\test.gradle\caches\transforms-1\files-1.1\appcompat-v7-27.1.1.aar\9d7a7dfeba2eb16cae5bc7d90fa05488\res\values\values.xml:251:5-69: AAPT: error: resource android:attr/fontVariationSettings not found.
C:\Users\test.gradle\caches\transforms-1\files-1.1\appcompat-v7-27.1.1.aar\9d7a7dfeba2eb16cae5bc7d90fa05488\res\values\values.xml:251:5-69: AAPT: error: resource android:attr/ttcIndex not found.
Exception in thread "stderr" java.util.UnknownFormatConversionException: Conversion = 'P'
at java.util.Formatter$FormatSpecifier.conversion(Formatter.java:2691)
at java.util.Formatter$FormatSpecifier.(Formatter.java:2720)
at java.util.Formatter.parse(Formatter.java:2560)
at java.util.Formatter.format(Formatter.java:2501)
at java.util.Formatter.format(Formatter.java:2455)
at java.lang.String.format(String.java:2940)
at com.android.build.gradle.internal.aapt.AaptGradleFactory$FilteringLogger.shouldDowngrade(AaptGradleFactory.java:178)
at com.android.build.gradle.internal.aapt.AaptGradleFactory$FilteringLogger.error(AaptGradleFactory.java:144)
at com.android.ide.common.process.LoggedProcessOutputHandler.handleOutput(LoggedProcessOutputHandler.java:44)
at com.android.ide.common.process.TeeProcessOutputHandler.handleOutput(TeeProcessOutputHandler.java:59)
at com.android.builder.png.AaptProcess$NotifierProcessOutput.handleOutput(AaptProcess.java:457)
at com.android.builder.png.AaptProcess$NotifierProcessOutput.err(AaptProcess.java:415)
at com.android.builder.png.AaptProcess$ProcessOutputFacade.err(AaptProcess.java:332)
at com.android.utils.GrabProcessOutput$1.run(GrabProcessOutput.java:104)
`

Hello, and sorry for the trouble you're having.
I've never seen this issue before, but let's try to isolate it. What happens if you create a blank project in your platform (whatever it is, be it vanilla Cordova or Ionic, React, Phonegap, etc) and install only this plugin? Does the issue still occur, or does the exact error change?

authors: Neotrixsx wrobins

repo: wrobins/cordova-plugin-msal


text:
Spanish translation

Spanish translation

Thanks!

authors: latra wsargent

repo: wsargent/docker-cheat-sheet


text:
Fix Appearing Extra Buttons when refreshing the API Definition page of an API Product

This update contains the fix for the issue https://github.com/wso2/product-apim/issues/12836.
The fix was made to stop appearing 'Edit' button and 'Import Definition' button with the refreshing of the API Definition page of an API Product.

Failing 7 UI test cases are not related to the change in this PR and they are known test failures


text:
Styling login ui

is it also possible to remove jade dependency from package.json since we have removed jade files? it deprecated and giving vulnerable issues


text:
Merge release-4.7.0-m11 branch back to 4.7.x

Purpose
$subject

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: chamathns:x: wso2-jenkins-botYou have signed the CLA already but the status is still pending? Let us recheck it.

authors: CLAassistant chamathns


text:
[APIM][3.1.0][3.2.0] - Default search in publisher and devportal is not the API name search

Description:
In the document [1, 2] it is mentioned that the default search is name search. But the default search in APIM publisher or devportal is a content search. If you don't specify a search prefix such as name, context, version, etc, then it is considered as a content search. So when it is considered as a content search, both API and document media types are considered. But if you want to search only the APIs based on their names, then you need to run the search query with the name: prefix.
[1] https://apim.docs.wso2.com/en/3.1.0/learn/consume-api/discover-apis/search/#search
[2] https://apim.docs.wso2.com/en/latest/learn/consume-api/discover-apis/search/
Affected Product Version:
wso2am-3.1.0, wso2am-3.2.0
Steps to reproduce:

Create multiple APIs with and without similar API names.

Similar names:  E.g: remedyAPI1, remedyAPI2, etc.
Different names: E.g: test, testAPI, etc.

Search the APIs in the publisher by just entering a few characters of the API name. E.g: "te".
The APIs will not be filtered based on the name.
Case 1: Without the "name" keyword

Search the APIs in the publisher by entering a few characters of the API name with the keyword name:. E.g: name:t

Case 2: With the "name" keyword

Unless we use name: the API name based filtering is nor working as expected.

Hi @saranki,
I think we used the term Unified Search when developing this search feature for APIM 3.0.0 by @msm1992 . Shall we use the term Unified Search instead of saying Content Search ?
@bhathiya @tgtshanika @chanaka3d  WDYT ?

+1

authors: bhathiya saranki tmkasun


text:
Oracle DB Docs are not Up-To-Date

The Oracle database-related configuration docs are not been updated to instruct APIM 3.0.0 users.

Updated Doc: https://apim.docs.wso2.com/en/latest/InstallAndSetup/SettingUpDatabases/ChangingDefaultDatabases/changing-to-oracle/


text:
fixes A1487

Purpose
Updated the list of secure Cipher Suites in [1].

[1] https://apim.docs.wso2.com/en/4.0.0/install-and-setup/setup/reference/supported-cipher-suites/

Updated based on Ayoma's feedback in https://github.com/wso2/docs-apim/pull/6573


text:
[APIM-4.0.0] Invalid username reference for TopicConnectionFactory url in jndi-cp.properties.j2

Description:
Currently,  jndi-cp.properties.j2 file is referring only the apim.event_hub.username as the username to construct the TopicConnectionFactory when some conditions are true. This will break the configuration if email user is enabled in the deployment.toml and also when event_hub configurations are applied(TM seperation use case)
It will result constructing the endpoint setting the username as admin@wso2.com@carbon.super
Ideally this value should be admin!wso2.com!carbon.super to be success with the configurations.
Hence we need to refer apim.throttling.jms.username instead.
Steps to reproduce:

Configure APIM-4.0.0 with email-user enabled configs
Apply event_hub configurations.
Start the server and check  jndi-cp.properties file.

Affected Product Version:
APIM-4.0.0
Environment details (with versions):
N/A

duplicate

authors: chaminda nandika


text:
[Publisher][UX][Listing] Styling issue in the api listing card in tenant mode.

fixed in 3.0.0 beta

authors: chamilaadhi menakaj


text:
Add new column to the IDN_OAUTH2_ACCESS_TOKEN table and alter the CON_APP_KEY constraint

Describe the issue:
The organization bound access token feature is given with IS 7.0.0 by adding a new column AUTHORIZED_ORGANIZATION to the IDN_OAUTH2_ACCESS_TOKEN table. Also the CON_APP_KEY unique key constraint also need to be altered to include that newly introduced column.

After this migration, the tokens issued before the migration execution and the token received after the execution, should work.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!

authors: isharak sadilchamishka


text:
Performance Degradation During Authenticator Update with a Large Number of Associated Applications

*Describe the Issue:
A significant delay has been observed in the execution of authenticator update requests (PUT requests) when a large number of applications are associated with the authenticator. Although no errors were logged, a considerable number of repeated SELECT queries  were observed under the same correlation ID, leading to extended execution times
Detailed Observations:

The delay occurs during the IDP update [1] process, where two listeners, doPreUpdateIdP [2] and doPostUpdateIdP, are invoked.
doPreUpdateIdP is responsible for validating and updating applications associated with the authenticator before the authenticator itself is updated.
The repeated database calls to retrieve basic application information are triggered during this validation process. [3]
A large number of SELECT queries are executed repeatedly, indicating a potential inefficiency in the data retrieval mechanism, especially in scenarios with many associated applications.
Additionally, the absence of database indexing may contribute to the slow execution times.

To mitigate this performance issue, it is recommended to optimize the data retrieval process during the IDP update. This could involve:

Reducing the number of repetitive SELECT queries.
Implementing database indexing where appropriate.
Enhancing the logic in doPreUpdateIdP to minimize redundant operations, especially when dealing with large numbers of - associated applications.

[1]
https://github.com/wso2/carbon-identity-framework/blob/v5.17.5/components/idp-mgt/org.wso2.carbon.idp.mgt/src/main/java/org/wso2/carbon/idp/mgt/IdentityProviderManager.java#L2217
[2]
https://github.com/wso2/carbon-identity-framework/blob/v5.17.5/components/application-mgt/org.wso2.carbon.identity.application.mgt/src/main/java/org/wso2/carbon/identity/application/mgt/listener/ApplicationIdentityProviderMgtListener.java#L51
[3]
SELECT ID, TENANT_ID, APP_NAME, USER_STORE, USERNAME, DESCRIPTION, ROLE_CLAIM, AUTH_TYPE, PROVISIONING_USERSTORE_DOMAIN, IS_LOCAL_CLAIM_DIALECT,IS_SEND_LOCAL_SUBJECT_ID, IS_SEN
D_AUTH_LIST_OF_IDPS, IS_USE_TENANT_DOMAIN_SUBJECT, IS_USE_USER_DOMAIN_SUBJECT, ENABLE_AUTHORIZATION, SUBJECT_CLAIM_URI, IS_SAAS_APP, UUID, IMAGE_URL, ACCESS_URL, IS_DISCOVERABLE FROM SP_APP WHERE ID = ?|

SELECT ID, NAME, VALUE, DISPLAY_NAME FROM SP_METADATA WHERE SP_ID = ?

SELECT INBOUND_AUTH_KEY, INBOUND_AUTH_TYPE, PROP_NAME, PROP_VALUE,INBOUND_CONFIG_TYPE FROM  SP_INBOUND_AUTH WHERE APP_ID = ? AND TENANT_ID = ?

SELECT STEP_ORDER, AUTHENTICATOR_ID, IS_SUBJECT_STEP, IS_ATTRIBUTE_STEP FROM SP_AUTH_STEP INNER JOIN SP_FEDERATED_IDP ON SP_AUTH_STEP.ID=SP_FEDERATED_IDP.ID WHERE APP_ID = ?

SELECT AUTH_TYPE FROM SP_APP WHERE ID = ? AND TENANT_ID = ?

SELECT CONTENT, IS_ENABLED FROM SP_AUTH_SCRIPT WHERE APP_ID = ?

SELECT IS_USE_TENANT_DOMAIN_SUBJECT, IS_USE_USER_DOMAIN_SUBJECT, ENABLE_AUTHORIZATION, IS_SEND_AUTH_LIST_OF_IDPS, SUBJECT_CLAIM_URI FROM SP_APP WHERE TENANT_ID= ? AND ID = ?

SELECT PROVISIONING_USERSTORE_DOMAIN, IS_DUMB_MODE FROM SP_APP WHERE TENANT_ID= ? AND ID = ?

SELECT IDP_NAME, CONNECTOR_NAME, IS_JIT_ENABLED, BLOCKING, RULE_ENABLED FROM SP_PROVISIONING_CONNECTOR WHERE APP_ID = ? AND TENANT_ID = ?

SELECT IDP_CLAIM, SP_CLAIM, IS_REQUESTED, IS_MANDATORY, DEFAULT_VALUE FROM SP_CLAIM_MAPPING WHERE APP_ID = ? AND TENANT_ID = ?

SELECT ROLE_CLAIM, IS_LOCAL_CLAIM_DIALECT, IS_SEND_LOCAL_SUBJECT_ID FROM SP_APP WHERE TENANT_ID= ? AND ID = ?

SELECT SP_DIALECT FROM SP_CLAIM_DIALECT WHERE TENANT_ID= ? AND APP_ID = ?

SELECT IDP_ROLE, SP_ROLE FROM SP_ROLE_MAPPING WHERE APP_ID = ? AND TENANT_ID = ?

SELECT AUTHENTICATOR_NAME FROM SP_REQ_PATH_AUTHENTICATOR WHERE APP_ID = ? AND TENANT_ID = ?

SELECT REG_PATH_ID FROM REG_PATH WHERE REG_PATH_VALUE=? AND REG_TENANT_ID=?

SELECT REG_PATH_ID FROM REG_PATH WHERE REG_PATH_VALUE=? AND REG_TENANT_ID=?

SELECT REG_VERSION FROM REG_RESOURCE WHERE REG_PATH_ID=? AND REG_NAME=? AND REG_TENANT_ID=?

How To Reproduce:

Log in to the management console.
Add an email authenticator as a federated authenticator. https://is.docs.wso2.com/en/5.9.0/learn/configuring-email-otp/
Create an application.
Assign the email authenticator as the federated authenticator for the application.
Enable correlation logs. https://is.docs.wso2.com/en/5.11.0/setup/working-with-product-observability/#configuring-product-observability
Execute an API call to update the authenticator.
ex:

curl --location --request PUT 'https://localhost:9443/t/carbon.super/api/server/v1/identity-providers/<IDP_ID>/federated-authenticators/RW1haWxPVFA' \
--header 'Content-Type: application/json' \
--header 'Authorization: Basic YWRtaW46YWRtaW4=' \
--data-raw '{
    "authenticatorId": "RW1haWxPVFA",
    "name": "EmailOTP",
    "isEnabled": true,
    "isDefault": true,
    "properties": [
        {
            "key": "EmailOtpLength",
            "value": "6"
        },
        {
            "key": "EmailOtpExpiryTime",
            "value": "6"
        },
        {
            "key": "Email",
            "value": "aaaaaa@gmail.com"
        },
        {
            "key": "EmailAPI",
            "value": ""
        },
        {
            "key": "OnlyNumerjjjicCharactersForOtp",
            "value": "true"
        }
    ]
}'

Increase the number of applications using the email OTP as a federated authenticator.
Validate that the number of DB calls increases proportionally, leading to delayed response times.

Ex:
Expected behavior:
The authenticator update request should execute efficiently, regardless of the number of associated applications. Database calls should be optimized to avoid unnecessary repetition, and appropriate indexing should be applied to improve query performance. The update process should be completed within a reasonable timeframe without performance degradation.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: IS 5.10.0
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

@DilshanSenarath Can you share the current update on this please.


text:
Change 'Default Service Provider' and 'Resident IDP' names in user portal into somthing meaningful for endusers

Change 'Default Service Provider' and 'Resident IDP' names in user portal into somthing meaningful for endusers.

The Default Service Provider was renamed to Self Care Portal. We can keep the Resident IDP under System Consent

authors: omindu sagara-gunathunga


text:
ID token in OIDC flow and self contained access token should have different configurations to define the requested claims

ID token in OIDC flow and self contained access token should have different configurations to define the requested claims

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!

authors: isharak prabath


text:
Developer Mode For the Identity Server

A suggestion to have a developer mode for the WSO2 Identity Server. The developer mode will include all the standard features, plus developer-friendly features.
An example of a developer-friendly feature would be the Cache Invalidation API.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!

authors: isharak tharindu-b-hewage


text:
[DOC] Bulk endpoint is not properly documented.

At present bulk endpoint is documented only in [1] and documented only about creating users in bulk. In addition to this updating and deleting in bulk needs to be documented. For more info refer to [2].
Sample bulk delete request:
curl -v -k --user admin:admin --data '{"failOnErrors":2,"schemas":["urn:scim:schemas:core:1.0"],"Operations":[{"path":"/Users/5d4ff8f6-e714-4138-aa6f-fa2443cd09e0","method":"DELETE"},{"path":"/Users/1f9ed88b-d0c2-4fda-af32-54a3a8a73316","method":"DELETE"}]}' --header "Content-Type:application/json" https://localhost:9443/scim2/Bulk
[1] - https://docs.wso2.com/display/IS570/apidocs/SCIM2-endpoints/index.html#!/operations#BulkEndpoint#createBulkUsers
[2] - https://malalanayake.wordpress.com/2013/05/28/scim-bulk-endpoint-operations-in-wso2-identity-server/

related: #4693

Need update the doc related to PATCH and PUT operations as well.
Refer https://github.com/wso2/product-is/issues/6887 to see how PATCH works right now.

authors: hauntingEcho malithie somindatommy


text:
Re-Captcha for security questions based password recovery not working

Steps to reproduce:
Configure security questions based password recovery and add below configs

Enable reCaptcha for Security Questions Based Password Recovery
Max Failed Attempts for ReCaptcha

Captcha is not shown after 2 attempts with incorrect answers for security questions.

This issue is no longer there in the latest IS. Tested for both tenant mode and super tenant mode.

authors: GANGANI madurangasiriwardena


text:
Email verification on update not working for secondary userstores.

Description:
Supporting email account verification for an updated email address [1] is not working for users in secondary userstore.
The request fails with 500 Internal server error and the response body as {"schemas":"urn:ietf:params:scim:api:messages:2.0:Error","detail":"Error while updating attributes of user: USERSTORE01/mathee1","status":"500"}
Affected Product Version:
5.10.0-beta2
5.7.0
[1] https://docs.wso2.com/display/IS570/Supporting+Email+Account+Verification+for+an+Updated+Email+Address

Fixed in https://github.com/wso2-support/identity-governance/pull/182


text:
Fix integration tests

Fixes https://github.com/wso2/product-is/issues/11327.
Tests need to use groups claim as it suits instead of the legacy role claim.

Closing and opening https://github.com/wso2/product-is/pull/11560.


text:
error: getaddrinfo RNOTFOUND null null:80

前四步都完成了，手机就是无法访问pc在hosts文件中配置的域名，当我访问 pc的Ip:9888 的时候就报上面的那个错，请问怎么解决？

对了，我的 spy-debugger 版本：

@jintangWang 请问解决了吗？我也遇到同样的问题

@PatWu16 连接同一个网络，设置手动代理，我用了360随身wifi不行，电脑连手机热点也不可以

close

repo: wuchangming/spy-debugger


text:
Does not work.

Script does not work. Cannot be installed via pip, and when running the executable, the example of "wudz-lit -f "C:\file.txt" -s -d -x 100" just shows the help menu.

All issues fixed and releases updated.

authors: oncoremarco wuddz-devs

repo: wuddz-devs/wuddz-lit


text:
CompilerAdapter 类

请问CompilerAdapter 类的作用是什么

@kezhian 为了覆盖掉aspectjtools依赖中的类，让它支持多线程。

authors: kezhian wurensen


text:
java.lang.VerifyError: Expecting a stackmap frame at branch target 10

After successfully obfuscation unable to run jar, below is the error:
Error: A JNI error has occurred, please check your installation and try again
Exception in thread "main" java.lang.VerifyError: Expecting a stackmap frame at branch target 10
Exception Details:
  Location:
    com/my/app/Start.a([Ljava/lang/String;)V @3: ifne
  Reason:
    Expected stackmap frame at this location.
  Bytecode:
    0x0000000: b800 469a 0007 04b8 005c bb00 1e59 2ab7
    0x0000010: 0048 4c2b b600 4a2b 1218 b600 494d 2b12
    0x0000020: 17b6 0049 4e2b 1219 b600 493a 042c c600
    0x0000030: 232c 1201 b600 579a 001a 2dc6 0016 1904
    0x0000040: c600 11bb 0031 592c 2d19 04b7 005e b800
    0x0000050: 5f2b 120d b600 49b8 003f 2b12 0cb6 0049
    0x0000060: b800 402b 1209 b600 49b8 0041 2b12 0eb6
    0x0000070: 0049 b800 422b 120a b600 49b8 0043 2b12
    0x0000080: 0fb6 0049 b800 442b 120b b600 49b8 0045
    0x0000090: 2b12 11b6 0049 3a05 013a 0619 05c6 004f
    0x00000a0: 1905 2bb8 003c 3a06 1906 c600 0e19 06b6
    0x00000b0: 003d 9900 06a7 003f 1906 c600 1819 06b6
    0x00000c0: 003d 9a00 1019 06b6 003e 2bb8 003b 57a7
    0x00000d0: 0025 b200 3a12 07b9 0065 0200 0112 0712
    0x00000e0: 0303 b800 6003 b800 5ca7 000b 1205 2bb8
    0x00000f0: 003b 3a06 1906 c6ff a519 06b6 003d 99ff
    0x0000100: 9d2b 1216 b600 493a 0719 07b8 0053 b600
    0x0000110: 543a 0819 08c1 0033 9900 0e19 08c0 0033
    0x0000120: b800 62a7 0060 1908 c100 3799 0058 bb00
    0x0000130: 2c59 b700 5912 08b6 005a 1907 1215 b600
    0x0000140: 5804 3212 06b6 0058 0332 b600 5a12 04b6
    0x0000150: 005a b600 5b3a 0912 143a 0abb 002c 59b7
    0x0000160: 0059 190a b600 5a19 09b6 005a b600 5b3a
    0x0000170: 0b19 0bb8 0053 b600 54c0 0033 3a0c 190c
    0x0000180: b800 62a7 0014 3a07 b200 3a12 0219 07b6
    0x0000190: 0055 b900 6603 002b 1212 b600 493a 0719
    0x00001a0: 07b8 0051 2b12 13b6 0049 3a08 1210 1908
    0x00001b0: b600 5799 0029 bb00 2059 b700 4d3a 0919
    0x00001c0: 092b 1906 b600 4ea7 0012 3a0a b200 3a19
    0x00001d0: 0ab6 0052 b900 6502 00a7 0026 bb00 1f59
    0x00001e0: b700 4b3a 0919 092b 1906 b600 4ca7 0012
    0x00001f0: 3a0a b200 3a19 0ab6 0055 b900 6502 00b1
    0x0000200:
  Exception Handler Table:
    bci [257, 387] => handler: 390
    bci [257, 387] => handler: 390
    bci [257, 387] => handler: 390
    bci [257, 387] => handler: 390
    bci [447, 455] => handler: 458
    bci [485, 493] => handler: 496

        at java.lang.Class.getDeclaredMethods0(Native Method)
        at java.lang.Class.privateGetDeclaredMethods(Unknown Source)
        at java.lang.Class.privateGetMethodRecursive(Unknown Source)
        at java.lang.Class.getMethod0(Unknown Source)
        at java.lang.Class.getMethod(Unknown Source)
        at sun.launcher.LauncherHelper.validateMainClass(Unknown Source)
        at sun.launcher.LauncherHelper.checkAndLoadMain(Unknown Source)

pom.xml plugin:
          <plugin>
                <groupId>com.github.wvengen</groupId>
                <artifactId>proguard-maven-plugin</artifactId>
                <version>2.3.1</version>
                <executions>
                    <execution>
                        <phase>package</phase>
                        <goals>
                            <goal>proguard</goal>
                        </goals>
                    </execution>
                </executions>
                <configuration>
                    <obfuscate>true</obfuscate>
                    <injar>${project.build.finalName}.jar</injar>
                    <outjar>${project.build.finalName}-obfs.jar</outjar>
                    <includeDependencyInjar>false</includeDependencyInjar>
                    <includeDependency>true</includeDependency>
                    <options>
                        <option>-dontnote</option>
                        <option>-dontshrink</option>
                        <option>-dontoptimize</option>
                        <option>-dontpreverify</option>
                        <option>-dontskipnonpubliclibraryclasses</option>
                        <option>-dontskipnonpubliclibraryclassmembers</option>
                        <option>-allowaccessmodification</option>
                        <option>-keep public class com.my.app.Start{public static void main(java.lang.String[]);}</option>
                    </options>
                    <libs>
                        <lib>${java.home}/lib/rt.jar</lib>
                        <lib>${java.home}/lib/jce.jar</lib>
                        <lib>${java.home}/lib/ext/jfxrt.jar</lib>
                    </libs>
                    <archive>
                        <manifest>
                            <mainClass>Start</mainClass>
                            <packageName>com.my.app</packageName>
                        </manifest>
                    </archive>
                </configuration>
            </plugin>

I have tried latest versions of this plugin but still same issue.

Seems like an issue with Proguard core itself. You need to report this there.
https://github.com/Guardsquare/proguard/issues

authors: lasselindqvist ypkkhatri

repo: wvengen/proguard-maven-plugin


text:
🛑 WW Connect is down

In ac84ca8, WW Connect (https://wellnesswayconnect.com) was down:

HTTP code: 500
Response time: 3071 ms

Resolved: WW Connect is back up in f47b1e3 after 27 minutes.

repo: wwsqaured/up


text:
Update default manifest version for Firefox to 3

Feature Request

[!WARNING]
If you're trying to run wxt -b firefox --mv3, use wxt build -b firefox --mv3 instead to build your extension for production and load the extension manually to test Firefox MV3 extensions.
For now, I would recommend not using Firefox for development, or using MV2 (the default) instead.

Before this can happen, the following changes are required:

[x] MV3 out of experimental status v109 added MV3 support
[x] #229
[ ] Dev mode support: https://bugzilla.mozilla.org/show_bug.cgi?id=1864284

Is your feature request related to a bug?
#226

[!WARNING]
If you're trying to run wxt -b firefox --mv3, use wxt build -b firefox --mv3 instead to build your extension for production and load the extension manually to test Firefox MV3 extensions.
For now, I would recommend not using Firefox for development, or using MV2 (the default) instead.

I have released my extension on Firefox add-ons for Desktop and Android as MV3.
Instantly regretted it.

Requires users to give extension access to all websites. It's not automatically allowed as in MV2.
Firefox Android really lacks MV3 compatibility

Releasing soon MV2 for Firefox instead as it resolves both of the issues from above.

Leaving this as a note. Firefox is not ready, maybe in a few years.

Yeah, I just released my rewrite as MV3 extension and the permissions are weird... Probably more secure, but less user friendly, for sure. I'm not supporting Android, so that's not a problem for me.
I'm gonna close this issue for now so I don't have to keep looking at it.

But https://wxt.dev/guide/multiple-browsers.html and development console have no information about this 😂 I spend so many time to debug why firefox with mv3 not work...

WXT should exit with an error message when you try and run wxt -b firefox --mv3

Really? Why I do not encounter any error message?

Huh 😖

@yunsii Can you add this log to your project? Inside node_modules/wxt/dist/cli.js, do a ctrl+F for "Dev mode does not support Firefox MV3" and add this log before the if statement:

console.log(wxt.config.browser, wxt.config.manifestVersion)

Then re-run the command that's failing and let me know what is logged?

@aklinker1 It seems a bug with WSL?

https://github.com/wxt-dev/wxt/blob/86b7162d99acbf6a9c92f959cffc60255cc30fe6/src/core/runners/wsl.ts#L8-L22
And why exit with an error message at beginning of dev command?

It seems a bug with WSL?

@yunsii This is intentional. web-ext doesn't work well in WSL: https://github.com/mozilla/web-ext/issues/2108#issue-779469169

@aklinker1 So how about exit with an error message at start of dev command?

Firefox MV3 dev mode (wxt -b firefox --mv3) does not work, see bugzilla.mozilla.org/show_bug.cgi?id=1864284 for more details. You have a two options:

Use --mv2, the default, instead when developing against firefox wxt -b firefox --mv2
Build for production, wxt build -b firefox --mv3, and load the extension manually into Firefox to test it

So it is a upstream bug, use this workaround for now.

@dword-design nope, still not working. HTML pages are broken, not content scripts.

authors: aklinker1 dvlden yunsii

repo: wxt-dev/wxt


text:
How to build this project?

Hey bro,
After fresh checkout the code doesn't really build in Xcode, it can not find Refresh and VideoPlayer package, is there any specific command I need to run?
I'm using Xcode version 13.2 (13C90), which is the latest one
(I'm an android developer recently started learning iOS)

It turns out to be an issue with Xcode, https://developer.apple.com/forums/thread/696504
After upgrade to 13.2.1, problem solved

repo: wxxsw/SwiftUI-WeChat


text:
Defer compile

Defers template parsing and compilation until first use. Provides a pretty drastic drop in app startup time for non-precompiled case.

+1

Looks good, but we need tests for this.

Nevermind, @wycats says the existing tests should cover this.

authors: kpdecker syntacticx wagenet