How to use the route object in the ts file

Environment
system:Window
dependencies:{
"nuxt": "^3.6.5",
"pinia": "^2.1.4",
}
Reproduction
https://github.com/HW-maintenance/HW_maintenance_web.git
Describe the bug
I have a problem when i send http request
I need to call a third-party interface, and when the third-party token becomes invalid, I need to jump to the login page
I hope to write the logic as a common method, but nuxt does not expose the route object
I thought about posting a new feature pr, but before doing that, I'd like to be able to discuss it as an issue
Additional context
the Generic http request method
import type { WatchSource } from 'nuxt/dist/app/compat/capi'
import type { KeysOf } from 'nuxt/dist/app/composables/asyncData'
import type { Method } from 'types/commonModel'
import type { HttpConstructor } from 'types/commonModel/http'

class HttpsUtils<DataT> {
  key: string | undefined
  params: URLSearchParams | undefined
  lazy: boolean | undefined
  immediate: boolean | undefined
  pick: KeysOf<DataT> | undefined
  watch: WatchSource[] | undefined
  baseURL: string | undefined = ''
  header: HeadersInit | undefined = {}

  constructor({ key, lazy, immediate, pick, watch, baseUrlFlag, defaultFun }: HttpConstructor<DataT>) {
    const runtimeConfig = useRuntimeConfig()
    this.key = key
    this.lazy = lazy
    this.immediate = immediate
    this.pick = pick as KeysOf<DataT>
    this.watch = watch
    this.baseURL = runtimeConfig.public[baseUrlFlag] as string | undefined
    defaultFun && (this._default = defaultFun)
  }

  _default = (): DataT => {
    return {} as DataT
  }

  transform = (input: DataT): DataT => {
    return input?.data ? input?.data : input
  }

  _onRequest = () => { }
  _onRequestError = () => { }
  _onResponse = () => { }
  _onResponseError = () => { }

  requestData = async (url: string, method: Method, param: { body?: any; params?: any }) => {
    const res = await useFetch(url, {
      method,
      server: false,
      headers: this.header,
      baseURL: this.baseURL,
      params: param?.params,
      body: param?.body,
      lazy: this.lazy,
      immediate: this.immediate,
      pick: this.pick,
      watch: this.watch,
      default: this._default,
      transform: this.transform,
      onRequest: ({ request, options }) => {
        // 设置请求头
        options.headers && ((options.headers as any).Authorization = bearerStore().token)
      },
      onRequestError: ({ request, options, error }) => {
        // 处理请求错误
      },
      onResponse: ({ request, response, options }) => {
        // 处理响应数据
        if (response._data.code === 500) {
          ElMessage.error(response._data.msg)
          bearerStore().token = null
        }
      },
      onResponseError: ({ request, response, options }) => {
        // 处理响应错误
        ElMessage.error('网络错误')
      },
    })
    return res
  }
}

export default HttpsUtils

Logs
No response

You should call useRoute within a context like a Vue component, composable or Nuxt plugin.
Ambiently it will not work for good reason - we need to be careful not to share state between different incoming requests.


feat: set output parameter wayback_url upon save attempt

Reattempting because I shouldn't have recompiled on the last PR attempt. Apologies!

I found it handy for a project in which I'm using this action to set an output parameter upon a save attempt to either an empty string or the Wayback Machine URL depending on whether the save was successful, so that the URL is more easily available to other steps.
I'm new to JavaScript/GitHub Actions development so I'm sure this needs edits/refactoring, but I figured I'd make my work available for implementation as it might be useful for someone else.

Thanks for the contribution. I've left a couple of comments and suggestions in the code, and I've got a couple more question:

What version of yarn did you use? The changes in yarn.lock look like churn and seem to be unnecessary
Can you add a simple test for this in test/wayback.spec.ts?

No problem.

All I can see is "yarn": ">=1.22.0" in package-lock.json, which appears to be the final minor release for yarn 1.x. I'm not familiar with yarn but cursory research suggested that either yarn.lock or package-lock.json should be committed for projects that have both, so I committed it since it was already included in the repository. I can certainly roll it back.
I'll give it a shot. It'll definitely need a second set of eyes though.

It appears that rolling back the yarn.lock file to the latest one here broke it. I still get this error even when using the exact version that appears to be running in the CI builds, v1.22.5. It does work on my machine if I run yarn install --frozen-lockfile --force.
After looking at the docs for @actions/core, the only way of testing this is to monitor stdout calls to ensure that one is made that we'd expect. A model for this is here, but I'm unable to run tests locally to ensure they pass before committing because I get the following errors when attempting to run the exact yarn jest --maxWorkers=2 --ci --coverage call that is run in the CI builds:

  ●  › works

    TypeError: Cannot read property 'headers' of undefined

      61 |   private handleErrorResponse(response: AxiosResponse): void {
      62 |     // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-assignment
    > 63 |     const error: string = response.headers?.['x-archive-wayback-runtime-error'];
         |                                    ^
      64 |     if (error) {
      65 |       switch (error) {
      66 |         case 'AdministrativeAccessControlException':

      at WayBack.handleErrorResponse (src/wayback.ts:63:36)
      at WayBack.save (src/wayback.ts:55:12)
      at Object.<anonymous> (test/wayback.spec.ts:34:5)

  ●  › polls

    TypeError: Cannot read property 'headers' of undefined

      61 |   private handleErrorResponse(response: AxiosResponse): void {
      62 |     // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-assignment
    > 63 |     const error: string = response.headers?.['x-archive-wayback-runtime-error'];
         |                                    ^
      64 |     if (error) {
      65 |       switch (error) {
      66 |         case 'AdministrativeAccessControlException':

      at WayBack.handleErrorResponse (src/wayback.ts:63:36)
      at WayBack.save (src/wayback.ts:55:12)
      at Object.<anonymous> (test/wayback.spec.ts:47:5)

The same issue happened when I attempted to write a test on the updated getArchiveUrl function. So, long story short, I have no clue how to test this because tests on existing unchanged code aren't passing.

Great deep dive on the testing status. I think I've got another solution though. We're not interested in whether or not process.stdout.write gets called, just that core.setOutput is. So we can utilize the mock in test/setup.ts and add an additional mock for core.setOutput, then check the status of that.
I'm not too sure what's going on with yarn. I checked what I get locally with what's in your branch, and I wasn't able to get the same results. I regenerated the lockfile on my end and committed it.

Thank you for fixing that up for me! Hopefully it wasn't too much work; if it was, my apologies. Some of this stuff is way out of my depth but I figured it might prove useful to someone else like it did to me! Thanks again.

Not a problem. Thanks for the contribution, and I'm glad you're finding it useful!


Bitwarden Support

Full and short name of product
Bitwarden (self-hosted)
Let us know the long, full name and, if it has one, the short name (for example, PowerShell and pwsh).
Does this product have LTS versions? What are the intervals between each LTS version?
As far as I know, there is no LTS.
What is the website for the product and for its version information?
https://bitwarden.com/help/bitwarden-software-release-support/ - says

"Support for Bitwarden Server releases is available for a duration of 12 months following the initial release."

Releases are mentioned here: https://bitwarden.com/help/releasenotes/
Additional context
Add any other context or screenshots about the product which you want us to add endoflife.date
https://en.wikipedia.org/wiki/Bitwarden
Worth noting that this is about the self-hosted version and that people can also use the hosted version with no updates required.
Also, there's an unofficial fork called vaultwarden that's also selfhostable.
Cheers

We should use the name “BitWarden Server” for the product.


Suggestion: Config option for light level indicator

Would like to be able to adjust at what light level the Red X shows up for use with:
https://www.curseforge.com/minecraft/mc-mods/peacefulsurface

Added in v4.2


Various fixes for the native build

@N3xed:
Following set of native build fixes:

Introduction of a new rustc cfg: ESP-IDF version. Derived by grepping the generated bindings.rs file for three constants (coming from esp_version.h: ESP_IDF_VERSION_MAJOR, ESP_IDF_VERSION_MINOR and ESP_IDF_VERSION_PATCH.

Why? Because it works regardless of the concrete build type: be it pio or native. As to why we are not basing the version on Git tags, that might be difficult as there is e.g. no proper git tag for ESP-IDF master. Als git describe returns nothing for master. Using the above integers help us e.g. "prepare" the codebase for a release to be branched off from ESP-IDF master, as these are correctly set there (to 4.4.0)

Bugfix (in resources/cmake_project/CMakeLists.txt) as to how the SDKCONFIG and SDKCONFIG_DEFAULTS were passed to CMake. Your approach was not really working for me. The current one is copy-pasted from google

Enhancement of the kconfig code:

Support for string values (in addition to tristates). First use case is the CONFIG_IDF_TARGET parameter, which is the MCU essentially. Before that, it was derived in a different way for pio vs native. Bugfixes in CfgArgs so that string parameters are properly handled
NOTE, important: Bugfix: the json-based kconfig values were not really having the CONFIG_ prefix, so I am now stripping it from the .config based kconfig values as well. We might do the other way around as well, especially in light of the fact that some CfgArgs might NOT come from kconfig values, so having the ESP-IDF kconfig values always starting with esp_idf_config_* as opposed to just esp_idf_ is giving us a better namespace separation of sorts.

Pull of common code from build_pio.rs and build_native.rs into the main build.rs file:

Bindgen run
Propagation of CinclArgs, LinkArgs and CfgArgs
ESP-IDF version grepping
Derivation of the MCU string (from kconfig ) and computation of the MCU configs (esp32, esp32s2, esp32c3)
Regexp-based filtering of kconfig values. Might be useful in future and is one of your open items, I think
Patchsets (~the "master" patchset needs to be re-generated as it doesn't apply cleanly on ESP-IDF master currently, but that would be a separate commit~) EDIT: Fixed now.

Again, some Clippy fixes (sorry!)

@N3xed: By the way, any preference / opinions here? That one is a bit important:

NOTE, important: Bugfix: the json-based kconfig values were not really having the CONFIG_ prefix, so I am now stripping it from the .config based kconfig values as well. We might do the other way around as well, especially in light of the fact that some CfgArgs might NOT come from kconfig values, so having the ESP-IDF kconfig values always starting with esp_idf_config_* as opposed to just esp_idf_ is giving us a better namespace separation of sorts.

@N3xed: By the way, any preference / opinions here? That one is a bit important:

NOTE, important: Bugfix: the json-based kconfig values were not really having the CONFIG_ prefix, so I am now stripping it from the .config based kconfig values as well. We might do the other way around as well, especially in light of the fact that some CfgArgs might NOT come from kconfig values, so having the ESP-IDF kconfig values always starting with esp_idf_config_* as opposed to just esp_idf_ is giving us a better namespace separation of sorts.

I think it's better without the *_config_* in between. Just using cfg! or #[cfg] already implies that is a configuration option and just the word config doesn't really say that these came from the sdkconfig. As for namespacing: I think there will be only very few cases where we add them manually so I think having name collisions isn't really an issue we'll have. Also I think it shouldn't really matter for the user where these cfgs came from (which it would if we namespace them).


freebsd.mtree: Needs explicit callPackage

Description of changes
Otherwise we get an infinite recursion.
This fixes the bootstrap of FreeBSD, which uses this tool from NetBSD.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@alyssais following https://github.com/NixOS/nixpkgs/pull/305864#issuecomment-2081580848, I confirmed this fixes evdev-proto.


Show/keep "Add single comment" button on existing comments during review

Description
Currently, you can only answer to a comment, either before a reviews starts or after it was submitted (or have a random comment floating in the review that cannot be resolved.
Before review:

During review:

After review:

So I would like to retain the ability to just comment on an existing comment, without it being part of the new review.
Example URLs
https://github.com/ST-DDT/faker/pull/21

As far as I know, this is a GitHub limitation and not a UI limitation. The server knows that a review is in progress so any "single comments" will always be appended to it.
A related feature request:

https://github.com/refined-github/refined-github/issues/5394


[gui] immutable AndroidManifest.xml

Issue details
I try to modify the AndroidManifest.xml, but it seems the manifest is read-only. How do I make the manifest writeable?
Jadx version
1.4.7
Java version
20.0.2
OS

[X] Windows
[ ] Linux
[ ] macOS

forgive my ignorance


Use map coordinates for spraying

Changes spray system to use MapCoordinates, allowing targeting of sprayables between grids and fixing a crash when using a sprayable from a vehicle, fixing  #9321.

I still saw some issues but I'll make a pr later.

Oh, my bad. Should probably have waited. I guess we could also just revert it?

It's fine it'll take me 2 minutes.


Detect if a parameter's location (i.e., in value) changes

@christophermanning please review?
Changing location is analogous to changing type, and should constitute a breaking change. This also prevents two parameters with the same name but different locations from masking each other.
Per @gburt's comment on #8.

LGTM


IGAPP-1328: Toolbar not sticky

can only be reproduced on webnext

Tested. Actually, why does the Aside have a z-index of 10? is there Any situation where this actually makes sense?

no clue. you can create a ticket and remove it. will merge this urgent fix now :)


make test target is needed

cache should have an individual test and added to Jenkins

For the last available update on the master branch,
make run VCD=0 does not run simulation without generating vcd file.
If you would like to run simulation with or without generating vcd file using only VCD macro, I suggest that you remove
`define VCD
from .vh files of the testbench.
What do you think?

I agree

I thought they'd already been removed

this has been done


LSTM over on MyActionMaskModel

Demo using an LSTM layer on top of MyActionMaskModel
https://docs.ray.io/en/releases-2.5.1/rllib/rllib-models.html?highlight=LSTM#built-in-auto-lstm-and-auto-attention-wrappers

Test cases:

[ ] Ray-provided auto-LSTM demo works as-written (direct copy/paste)
[ ] Auto-LSTM on top of Ray example action mask model
[ ] Ausot-LSTM on top of custom action mask model

Case 3 error message:
(RolloutWorker pid=16662) 2023-10-11 10:45:57,338       ERROR worker.py:861 -- Exception raised in creation task: The actor died because of an error raised in its creation task, ray::RolloutWorker.__init__() (pid=16662, ip=<IP_ADDRESS>, actor_id=306c78b76f7263aba54196f301000000, repr=<ray.rllib.evaluation.rollout_worker.RolloutWorker object at 0x7f7309c66530>)
(RolloutWorker pid=16662)   File ".../python3.10/site-packages/torch/nn/modules/linear.py", line 96, in __init__ [repeated 5x across cluster] (Ray deduplicates logs by default. Set RAY_DEDUP_LOGS=0 to disable log deduplication, or see https://docs.ray.io/en/master/ray-observability/ray-logging.html#log-deduplication for more options.)
(RolloutWorker pid=16662)     self._update_policy_map(policy_dict=self.policy_dict)
(RolloutWorker pid=16662)   File ".../python3.10/site-packages/ray/rllib/evaluation/rollout_worker.py", line 1985, in _update_policy_map
(RolloutWorker pid=16662)     self._build_policy_map(
(RolloutWorker pid=16662)   File ".../python3.10/site-packages/ray/rllib/evaluation/rollout_worker.py", line 2097, in _build_policy_map
(RolloutWorker pid=16662)     new_policy = create_policy_for_framework(
(RolloutWorker pid=16662)   File ".../python3.10/site-packages/ray/rllib/utils/policy.py", line 142, in create_policy_for_framework
(RolloutWorker pid=16662)     return policy_class(observation_space, action_space, merged_config)
(RolloutWorker pid=16662)     TorchPolicyV2.__init__(
(RolloutWorker pid=16662)     model, dist_class = self._init_model_and_dist_class()
(RolloutWorker pid=16662)   File ".../python3.10/site-packages/ray/rllib/policy/torch_policy_v2.py", line 488, in _init_model_and_dist_class
(RolloutWorker pid=16662)     model = ModelCatalog.get_model_v2(
(RolloutWorker pid=16662)   File ".../python3.10/site-packages/ray/rllib/models/catalog.py", line 613, in get_model_v2 [repeated 2x across cluster]
(RolloutWorker pid=16662)     raise e
(RolloutWorker pid=16662)     instance = model_cls(
(RolloutWorker pid=16662)     super(LSTMWrapper, self).__init__(
(RolloutWorker pid=16662)   File "/home/dylanrpenn/clockpunch/punchclock/nets/action_mask_model.py", line 95, in __init__
(RolloutWorker pid=16662)     self.action_head = nn.Linear(last_layer_size, num_outputs)
(RolloutWorker pid=16662)     self.weight = Parameter(torch.empty((out_features, in_features), **factory_kwargs))
(RolloutWorker pid=16662) TypeError: empty() received an invalid combination of arguments - got (tuple, dtype=NoneType, device=NoneType), but expected one of:
(RolloutWorker pid=16662)  * (tuple of ints size, *, tuple of names names, torch.memory_format memory_format, torch.dtype dtype, torch.layout layout, torch.device device, bool pin_memory, bool requires_grad)
(RolloutWorker pid=16662)  * (tuple of SymInts size, *, torch.memory_format memory_format, Tensor out, torch.dtype dtype, torch.layout layout, torch.device device, bool pin_memory, bool requires_grad)

Auto-LSTM is possibly not the right way to go (not possible). May have to write a custom model that does action masking + LSTM.
https://discuss.ray.io/t/issue-creating-custom-action-mask-enviorment/4940/6?u=dylan906

Auto-LSTM is possibly not the right way to go (not possible). May have to write a custom model that does action masking + LSTM. https://discuss.ray.io/t/issue-creating-custom-action-mask-enviorment/4940/6?u=dylan906

Auto-LSTM definitely not the way to go. Making new custom model that has FC->LSTM->action_mask

Test cases
Case 1
Summary:
Model is using default from custom model definition and inserted defaults from Ray's other processes.
Model:
class MaskedLSTM(TorchRNN, nn.Module):
    """Fully-connected layers feed into an LSTM layer."""

    def __init__(
        self,
        obs_space: gym.spaces.Space,
        action_space: gym.spaces.Space,
        num_outputs: int,
        model_config: dict,
        name: str = None,
        fc_size: int = 64,
        lstm_state_size: int = 256,
    ):

Config:
param_space = {
    "framework": "torch",
    "env": "my_env",
    "horizon": None,  # environment has its own horizon
    "env_config": env_config,
    "model": {
        # Specify our custom model from above.
        "custom_model": "MaskedLSTM",
        # Extra kwargs to be passed to your model's c'tor.
        # "custom_model_config": {},
        "custom_model_config": {
            # "num_outputs": 2,
            "fcnet_hiddens": [6, 6],
            "fcnet_activation": "relu",
            "fc_size": 5,
            "lstm_state_size": 20,
        },
    },
}

Output (excerpt):

(RolloutWorker pid=15229) model_config = {...
'fcnet_hiddens': [256, 256], 'fcnet_activation': 'tanh', ...
 'lstm_cell_size': 256, ...
'custom_model': 'MaskedLSTM', 'custom_model_config': {
'fcnet_hiddens': [6, 6], 'fcnet_activation': 'relu', 'fc_size': 5, 'lstm_state_size': 20
}, ...
}

(RolloutWorker pid=15229) self.fc_size = 64
(RolloutWorker pid=15229) self.lstm_state_size = 256
(RolloutWorker pid=15229) hiddens = [256, 256]
(RolloutWorker pid=15229) activation = tanh


feature(CLI) Add --plainText option to CLI

Signed-off-by: Jerome Simeon<EMAIL_ADDRESS>Issue #184
Add a --plainText option to the draft and normalize commands in the CLI.
Example run:
bash-3.2$ cat test/data/nestedblockquote.md
Once upon a time

> This is a\
> linebreak.
> This is a quote.
>
> Heading Two
> ----
> This is more text.
> Ordered lists:
> 1. one
>    this has more in it
>    - nested list
>    - etc.
> 2. two
> 3. three
>
> This should be a line
>
> ----
> This should be more text
> > This should be a nested blockquote
> with more things in it
> >
> > And a second paragraph in the nested blockquote
>
> The end of the blockquote

bash-3.2$ ../markdown-cli/index.js normalize --sample test/data/nestedblockquote.md  --plainText --cicero
9:26:36 PM - info:
Once upon a time

This is alinebreak.This is a quote.

Heading Two

This is more text.Ordered lists:
1. one
   this has more in it
   -  nested list
   -  etc.
2. two
3. three

This should be a line

This should be more text

This should be a nested blockquotewith more things in it

And a second paragraph in the nested blockquote

The end of the blockquote

The end
bash-3.2$ ../markdown-cli/index.js parse --sample test/data/nestedblockquote.md --cicero --output data.json
9:27:05 PM - info: Creating file: data.json
9:27:05 PM - info:
{
  "$class": "org.accordproject.commonmark.Document",
  "xmlns": "http://commonmark.org/xml/1.0",
  "nodes": [
    {
      "$class": "org.accordproject.commonmark.Paragraph",
      "nodes": [
        {
          "$class": "org.accordproject.commonmark.Text",
          "text": "Once upon a time"
        }
      ]
    },
.....
bash-3.2$ ../markdown-cli/index.js draft --data data.json --cicero --plainText
9:27:25 PM - info:
Once upon a time

This is alinebreak.This is a quote.

Heading Two

This is more text.Ordered lists:
1. one
   this has more in it
   -  nested list
   -  etc.
2. two
3. three

This should be a line

This should be more text

This should be a nested blockquotewith more things in it

And a second paragraph in the nested blockquote

The end of the blockquote

The end

Coverage decreased (-0.4%) to 88.565% when pulling 2a2caeb856fcfda19f04b1c596206b60ceabe627 on jeromesimeon:js-feature-cli-plainText into c9816d9b1fb1c70f04cf6af4a5b08a06499be019 on accordproject:master.

Looks like I have a bug for line breaks... I'll take a look at that.

Looks like I have a bug for line breaks... I'll take a look at that.

I filed as: https://github.com/accordproject/markdown-transform/issues/193


[TS convertion] - Verification message

@daigof Just merged in the Storybook 5 --> 6 branch, let me know if any of the conflicts are confusing!


Updates to accessions.tsv (12788 new, 1 removed) seqs downloaded 2021-08-04

Thanks!  :)

Sorry for the delay Angie - I was away on holiday! Hope you are able to get some time off this summer as well - and thanks as always for the update!

Thank you Emma!  :)  I was in fact away on vacation all last week, and haven't managed to do another update yet this week, but - tomorrow!  :)  Glad you got some time off too.


Fix PR comment bot

's gets misrecognized as end of string and caused GHA workflow to fail

still failed :(

oh it runs on main


[FeatureRequest] Options Chain Data missing strike/expiration

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem? Please describe.
The options chain data needs strike/expiration, otherwise I have to query a separate endpoint
Describe the solution you'd like.
include strike/expiration
Describe an alternate solution.
include strike/expiration
Anything else? (Additional Context)
No response

@Brett55 thank you for the request. I have routed this feature request to related teams.

In fact, the symbol name tells the attributes of the options. So, I would rather vote to add a helper function in SDK than adding payloads to the API. Here is the python snippet recently posted in the slack community.
from datetime import datetime

s = "AAPL240906C00227500"
expiration_date = datetime(
    year=2000 + int(s[-15:-13]),
    month=int(s[-13:-11]),
    day=int(s[-11:-9])
)


Make functionality of @Traced more clear

Resolves #217

@fmhwong please also take a look to see whether you have comments.


Allow filtering of war mode mismatch groups in 'Custom'?

I use the custom group finder a lot, and want to be able to ignore WM On groups, can you add the ability to filter out war mode mismatches?

That's not really spam-filtering related I think. But theres the Premade Groups Filter addon which can work together with LFGSpamFilter. You can put not warmode into the advanced query to filter out warmode groups.

That's not really spam-filtering related I think. But theres the Premade Groups Filter addon which can work together with LFGSpamFilter. You can put not warmode into the advanced query to filter out warmode groups.

It's not, but you can only really filter on so many things now (since you can't filter on the group name anymore), and this addon already filters on almost all of them, except war mode.

True, group name is not readable by addons, but many other parameters are. Since there's already an addon dedicated to generic filtering and it works together with LFGSpamFilter, I'd like to keep LFGSpamFilter focused solely on spam filtering and reporting.

Ok.. I'm going to fork this project then for my own needs. Thanks for your work!


"Rename" NetworkPolicy: allow-to-seed-apiserver -> allow-to-runtime-apiserver

How to categorize this PR?

/area networking security
/kind enhancement
What this PR does / why we need it:
This PR makes gardenlet's NetworkPolicy controller not only deploy the allow-to-seed-apiserver NetworkPolicy but the same one with the new allow-to-runtime-apiserver name. This is needed since the controller is planned to be also used by gardener-operator in garden!=seed clusters.
In addition, the istio-system namespace in seed clusters is now labeled with gardener.cloud/role=istio-system, and all istio-ingress* namespaces are now labeled with gardener.cloud/role=istio-ingress for easier identification.
Which issue(s) this PR fixes:
Part of #7352
Special notes for your reviewer:
In the future, the gardenlet's NetworkPolicy controller will manage more NetworkPolicy resources.
Release note:

The configuration for `gardenlet`'s `NetworkPolicy` controller must now be submitted via `.controllers.networkPolicy` instead of `controllers.seedAPIServerNetworkPolicy`.

The `allow-to-seed-apiserver` `NetworkPolicy` is now deprecated and replaced by the new `allow-to-runtime-apiserver` `NetworkPolicy`. Components running in the seed cluster that need to talk to the `kube-apiserver` should be labeled with `networking.gardener.cloud/to-runtime-apiserver=allowed`.

The `istio-system` namespace in seed clusters is now labeled with `gardener.cloud/role=istio-system`. All `istio-ingress*` namespaces are now labeled with `gardener.cloud/role=istio-ingress`.

/assign

/retest

/test pull-gardener-e2e-kind-ha-multi-zone-upgrade

I added a new commit (b847ca7) to simplify the logic in one of the reconciler's mapping functions.


Crash when IPv6 interface is disabled in the otThreadDiscover() callback

Describe the bug
If otIp6SetEnabled(sInstance, false) is called from the callback supplied to otThreadDiscover() the device crashes. More info below.
To Reproduce

Git commit id: 2ce3d3bf0
IEEE 802.15.4 hardware platform: Silicon Labs BRD4186C
Build steps: Regular build steps in Silicon Labs Studio Thread project
Network topology: Single device

Add the following code to the application triggered by some user action such as button press or a CLI command:
{
 otIp6SetEnabled(sInstance, true);
 otThreadDiscover(sInstance, 0, OT_PANID_BROADCAST, false, false,  scanResult, 0);
}

void scanResult(otActiveScanResult *aResult, void *aContext)
{
    if (aResult == NULL) {
        otIp6SetEnabled(sInstance, false);
    }
}

Triggering the action results in a crash in MeshForwarder::UpdateSendMessage()
Expected behavior
Device should not crash
Console/log output
N/A
Additional context
When Thread scan completes  the following calls happen
MeshForwarder::UpdateSendMessage() -->Get<Mle::DiscoverScanner>().HandleDiscoveryRequestFrameTxDone() --> DiscoverScanner::HandleDiscoverComplete()->mHandler(nullptr, mHandlerContext);
The mHandler() points to the discovery  callback. If the callback calls
otIp6SetEnabled(mOTInst, false)
then it in turn calls MeshForwarder::Stop() which sets mSendMessage to nullptr
When the call stack returns to MeshForwarder::UpdateSendMessage() it tries to access mSendMessage in RemoveMessageIfNoPendingTx(*mSendMessage) resulting in a hard fault.

Thanks @selissia.  Submitted https://github.com/openthread/openthread/pull/8111 which should help resolve this.


Paths for native modules and binaries imported by a renderer process being changed to .webpack/main instead of .webpack/renderer

Preflight Checklist

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Issue Details

Electron Forge Version:

6.0.0-beta.50

Electron Version:

8.1.1

Operating System:

Windows 10 (1903)

Last Known Working Electron Forge version::

N/A

Expected Behavior
When using trash (dependency for deleting files and moving them to the current OS' native recycling bin), the executables macos-trash and windows-trash.exe should be copied over to .webpack/renderer/native_modules and the path to these executables in the main_window/index.js file should reflect the renderer path.
Actual Behavior
macos-trash and windows-trash.exe are properly copied over to .webpack/renderer/native_modules but the path in main_window/index.js was rewritten to .webpack/main instead of .webpack/renderer. When trying to delete a file, it throws a an error:
Uncaught (in promise) Error: spawn C:\path\to\testElectronTrashApp\.webpack\main/../native_modules/windows-trash.exe ENOENT
    at Process.ChildProcess._handle.onexit (internal/child_process.js:264)
    at onErrorNT (internal/child_process.js:456)
    at processTicksAndRejections (internal/process/task_queues.js:80)

To Reproduce

Create a new project and install trash and string-replace-loader.

npx create-electron-app testElectronTrashApp --template=webpack
cd testElectronTrashApp
npm install trash

Create an empty text file named temp.txt
In webpack.rules.js add the following lines (workaround for default export):

{
    test: /parsers\.js$/,
    loader: 'string-replace-loader',
    options: {
      search: 'module.exports = parsers;',
      replace: 'export default parsers;',
    }
  }

In the main renderer file import trash and try to delete temp.txt:

import trash from 'trash';
trash('C:\\path\\to\\testElectronTrashApp\\temp.txt')

Run the app: npm start

Additional Information
It seems like the fix is fairly straightforward. In the Webpack.ts file under the Webpack plugin, on line 100 inRendererDir should be passed to the assetRelocatorBaseDir function.

Hi @Mac-Genius @ysdexlic  @MarshallOfSound  I'm also runnig into the same issue, how can I fix it or go around it untill it's fixed?
Thanks!

hey @mohyddintash, I fixed by following some the step left in the comment and a bit more:

As said by @Mac-Genius, "In the Webpack.ts file under the Webpack plugin, on line 100 inRendererDir should be passed to the assetRelocatorBaseDir function.", you can find this file in the node_modules although it's not the safest thing to do to fix the issue (it'll be overwritten every time you install deps)
After that fix, a new bug arose about a directory not existing. If you come across this the fix is - in the same file at line 86 and line 93 they reference any_folder. As the any_folder directory didn't exist this was erroring. I changed it to be the name of my main window main_window as a hack (any directory in the renderer directory should work) but the real fix would be to stop whatever is moving up a directory (adding ../ to the path) as they've clearly added any_folder in an attempt to negate it.

I'm not working on the project anymore but after resolving these two issues I ran into even more but I think they were quite library specific.
I hope this helps, let me know if you have any more questions.

@ysdexlic Thanks fo taking the time to reply! in fact i did the manual fix for line 100, but haven't came across the issues yet, as I haven't yet run make, so will keep an eye. However, the thing I was trying to do failed so I switched my approach.
I do have a question though and hopefully you or someone could help out!
Basically I have a code that requires a native module which reads a RFID tag, (It works in main process but It's blocking so I need to run it in parallel, sadly it didn't work in the rendere windowr).
So, I need a way to be able to run that code as another process using fork but from my built project, currently I am forking it from a directory outside the project and using an absolute path, it works but It doesn't feel right cause I have to make sure to use same electron version there and also run electorn-rebuild twice.
I am tihnking of creating a new renderer that contains my RFID code and that renderer (if native_modules/ still work after that fix above), I can then fork the RFID renderer's index.js from my main renderer if that made any sense? Does it sound right? I'm a bit confused..

@ysdexlic @mohyddintash Sorry I haven't replied sooner, I'm not very active on GitHub. I manually patched it myself in my node_modules but I don't have a good workaround. When I get some time I can put in a PR with some tests.
As for your question @mohyddintash, you may want to look at Web Workers. It's OS level multi-threading that could allow you to run your RFID code in a separate thread (branched off your main thread) that doesn't require you to have a renderer thread just for that.

@Mac-Genius I did try that, but couldn't get it to work, then I came across a post on electron-orge's github issues, where someone mentioned that Web Workers will not work with native modules, can you confirm?
The docs say that it's not guaranteed to work.
Is there a way to know if the module I want to use will not work without going through the implementation?

@mohyddintash Hmm, I'm not sure. Is it safe to run in a renderer thread then? Or is it only recommended to run on the main thread?

@Mac-Genius Not sure, I did try running it but failed, since the library I'm using for the RFID uses Atomic.wait which isn't supported in the browser, so I tried to mess with the library's source code, but didn't succeed.
Anyway, I finally managed to get it to work using web workers! I did like the docs suggest if you want to use native modules in workers and added this which is supposed to make it thread safe:
process.dlopen = () => {
    throw new Error('Load native module is not safe')
}`

let worker = new Worker();

Also, kudos to @ysdexlic for mentionintg the any_folder issue above cause I ran into it later, but in my case, to get rid of the error, I modified the code to look like:
assetRelocatorBaseDir(inRendererDir = true) {
    if (this.isProd) {
      return `process.resourcesPath + "/" + (__filename.includes(".asar") ? "app.asar" : "app") + "/.webpack/${inRendererDir ? 'main' : 'renderer/'}"`;
    }

    return JSON.stringify(_path.default.resolve(this.webpackDir, inRendererDir ? 'main' : 'renderer', inRendererDir ? '.' : ''));
  }

Hi @Mac-Genius @ysdexlic @MarshallOfSound I'm also runnig into the same issue, how can I fix it or go around it untill it's fixed?
Thanks!

Using native module in renderer, you can copy from node_modules manually, not perffect, duplicate with renderer/native_modules (e.g. better-sqlite3):

yarn add copy-webpack-plugin -D
webpack.plugins.js
const ForkTsCheckerWebpackPlugin = require('fork-ts-checker-webpack-plugin');
const CopyPlugin = require('copy-webpack-plugin');

module.exports = [
  new CopyPlugin({
    patterns: [
      { from: './node_modules/better-sqlite3/build/Release/better_sqlite3.node', to: '../native_modules/build/Release/better_sqlite3.node' },
    ],
  }),
  new ForkTsCheckerWebpackPlugin(),
];

renderer.tsx
const Database = require('better-sqlite3');
const { app } = require('electron');
const dbPath = app ? app.getAppPath() + 'foobar.db' : 'foobar.db';
console.log(dbPath);
const db = new Database(dbPath, { verbose: console.log });
console.log(db);

Tested in start and package.


fix bug in tagging.py with spacy en model

This PR fixes a bug in the chatterbot/tagging.py file
Implementation

A description of the bug and solution can be found in this stack overflow thread

Basically one of the lines in tagging.py is updated to cater for when chatterbot tries to load a model titled en. It ensures that the correct naming of the English model is used, i.e. en_core_web_sm

hi everyone how are you doing?


[CI] Update ci environment

In this pull request, docker/install/ubuntu_install_paddle.sh is modified to upgrade paddlepaddle to version 2.1.3 which is released this month.
In PaddlePaddle 2.1.3, we provide a new API paddle.disable_signal_handle() to disable system signal capturing in PaddlePaddle framework, which may cause conficts with AutoTVM.
Thanks for contributing to TVM!   Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.

Let’s get it in quickly

Would you mind opening a CI upgrade thread in the issues?

Merging is as per request @junrushao1994.

@jiangjiajun can you please send a PR to update python/gen_requirements.py?

@jiangjiajun can you please send a PR to update python/gen_requirements.py?

OK https://github.com/apache/tvm/pull/9098


HTML Compliance - Dashboard

HTML compliance (aside from the active widgets).
Rename duplicate id's.
Fix available widgets collapsible section href with id.
Where obsolete replace name attribute with id.

This change has been reverted since it broke the "click on the wrnech to configure" widget actions.


ensure on voice channel permissions

anyone who generates a voice lobby, should have full rights to edit that voice lobby created.

Permissions already given
issue complete


Pulumi panic on pulumi up

    panic: interface conversion: interface {} is []interface {}, not *schema.Set
    goroutine 143 [running]:
    github.com/terraform-providers/terraform-provider-aws/aws.setAutoscalingTags(0xc000010a40, 0xc0010d3500, 0x17, 0x4749900)
        /home/travis/gopath/pkg/mod/github.com/pulumi/terraform-provider-aws@v1.38.1-0.20200616201417-ed844a7bd9d5/aws/autoscaling_tags.go:83 +0xe49
    github.com/terraform-providers/terraform-provider-aws/aws.resourceAwsAutoscalingGroupUpdate(0xc0010d3500, 0x4eacba0, 0xc00047af00, 0x0, 0x0)
        /home/travis/gopath/pkg/mod/github.com/pulumi/terraform-provider-aws@v1.38.1-0.20200616201417-ed844a7bd9d5/aws/resource_aws_autoscaling_group.go:939 +0x4b8
    github.com/hashicorp/terraform-plugin-sdk/helper/schema.(*Resource).Apply(0xc000484900, 0xc000931ae0, 0xc000d53da0, 0x4eacba0, 0xc00047af00, 0x1, 0x0, 0x0)
        /home/travis/gopath/pkg/mod/github.com/hashicorp/terraform-plugin-sdk@v1.13.1/helper/schema/resource.go:316 +0x263
    github.com/hashicorp/terraform-plugin-sdk/helper/schema.(*Provider).Apply(0xc00049e180, 0xc000a6d818, 0xc000931ae0, 0xc000d53da0, 0x0, 0x0, 0x0)
        /home/travis/gopath/pkg/mod/github.com/hashicorp/terraform-plugin-sdk@v1.13.1/helper/schema/provider.go:294 +0x99
    github.com/pulumi/pulumi-terraform-bridge/v2/pkg/tfbridge.(*Provider).Update(0xc000692b60, 0x64cff00, 0xc000fd85a0, 0xc000ecc2a0, 0xc000692b60, 0x4e29901, 0xc000f95a80)

Hello all, I encountered this error above.
I am not sure what happened but this script worked last week witout any issues

Hi @aj-garcia05
My apologies for this being the case - we believe that #1011 will take care of this issue but we are testing this right now. I will alert you to this issue being closed and the version of the provider it has been fixed in. We believe this will be fixed today (June 22, 2020)
If you want to be unblocked for now, you can remove the tags from the autoscaling group and your infrastructure will be deployed as expected :/ I am sorry for this being the case
Thanks
Paul

I see. Thanks for responding quickly. I thought there is something wrong with my code. I'll try your fix. by the way, I hope this helps. This is the current npm packages that are being installed on my project.
And also, I'm using the latest version of pulumi. 2.4.0
        "@pulumi/aws": "^2.9.1",
        "@pulumi/awsx": "^0.20.0",
        "@pulumi/eks": "^0.19.0",
        "@pulumi/kubernetes": "^2.1.1",
        "@pulumi/pulumi": "^2.4.0",
        "@types/glob": "^7.1.1",
        "aws-sdk": "^2.658.0",

Hi @aj-garcia05
The issue is actually pulumi/aws - and has been broken for 2 of the releases but I didn't grok what the problem was until Friday and we are code reviewing the issue right now
P.

I see. Thanks! I just rolled back to my previous version and it worked fine. Thanks for clarifying. :)

I presume you may have been on pulumi-aws 2.6.x ?

@stack72 sorry for the late response.
Yes. I was using 2.6 before the error happened.


Rider + Godot 4 Debugging - Get to Equal Godot 3.5 Quality

I decided to try debug the Godot 4 branch to see if it could be made equivalent to Godot 3.5 in short order.  Instead, I debugged the debugger.
For some reason, while you can set up the ability to launch Godot 4.x (tried 4.1.1 and 4.2.1) from Rider, it does not attach the debugger.  You can attach it post-launch with Run -> Attach to Process, and selecting the Godot/C7 process, but it's a bit tedious.  Still arguably faster than VSCode's debugging was last time I tried it, but not nearly as seamless as Godot 3.5 (half the time is the human responding to the UI, half is loading the list of processes).
In addition to trying two Godot 4.x versions, I tried both Rider 2023.1.6 (with its most-recent-available Godot plugin), and the latest Rider 2024.1 with the latest Godot plugin.  I also tried removing the .idea folder and re-generating it, and rebuild the project after each change.  Neither IDE could attach to Godot 4.x automatically.
Output from both 3.5 and 4.2.1, with Rider 2023.1.6 (newlines added for readability):
Godot 3.5 (Working)

"C:\Program Files\JetBrains\JetBrains Rider 2023.1.6\bin\runnerw.exe"
"C:\Program Files\JetBrains\JetBrains Rider 2023.1.6\lib\ReSharperHost\windows-x64\dotnet\dotnet.exe"
exec
"C:\Program Files\JetBrains\JetBrains Rider 2023.1.6\lib\ReSharperHost\JetBrains.Debugger.Worker.exe"
--mode=client --frontend-port=53736 "--plugins=C:\Users\QuintillusCFC\AppData\Roaming\JetBrains\Rider2023.1\plugins\rider-godot\dotnetDebuggerWorker;
C:\Program Files\JetBrains\JetBrains Rider 2023.1.6\plugins\rider-unity\dotnetDebuggerWorker"

"C:/Programming/Godot_v3.5.2-stable_mono_win64/Godot 3.5.2 x64.exe" --path "C:/Development/Civ Related Projects/Prototype/C7"
Process ""C:/Programming/Godot_v3.5.2-stable_mono_win64/Godot 3.5.2 x64.exe" --path "C:/Development/Civ Related Projects/Prototype/C7"" started.

Godot Engine v3.5.2.stable.mono.official.170ba337a - https://godotengine.org
OpenGL ES 3.0 Renderer: AMD Radeon RX 6600M
Async. shader compilation: OFF

Mono: Log file is: 'C:/Users/QuintillusCFC/AppData/Roaming/Godot/mono/mono_logs/2024-04
-12_00.25.46_24572.log'
Hello logger!

The only differences are the frontend-port, which seems to increment, and that 3.5 uses OpenGL ES rather than Vulkan.  Rider 2024.1 has a slightly different sequence, but seeing the same output when the Rider version is the same makes me think it's something odd about Godot.
Link to the Godot plugin issues list: https://github.com/JetBrains/godot-support/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc  Found a few that seemed sort-of related but no silver bullets.  It seems like it's theoretically supporting Godot 4 at this point although support was added incrementally so how complete that is, I'm not sure.  It may also make sense to try the maintainer's Dodge the Creeps game (https://github.com/van800/godot-demo-projects/tree/net6/mono/dodge_the_creeps) which now runs on Godot 4.2 to see if that debugs automagically or not.

Confirmed that the same thing happens with "Dodge the Creeps" with the Rider 2023.1.6.  Will it happen with the latest as well?  If so, that implies it's time to report an issue and not keep debugging C7 debugging.

Confirmed it occurs the same way with the latest Rider/Godot plugin and "Dodge the Creeps".  Set a new personal high score of 22 seconds.

Got a quick answer from the support ticket I filed; it needs to be a .NET Executable configuration type in Godot 4, not the Godot-icon "Start and Debug" type.  I have updated our Wiki instructions to note that (while still leaving the Godot 3.5 instructions as well).


Rancher v2.3.5 hardening guide needs to be validated for Kubernetes v1.16

Rancher v2.3.5 hardening guide needs to be validated for Kubernetes v1.16.
https://rancher.com/docs/rancher/v2.x/en/security/hardening-2.3.5/

With v2.3.5 of the Rancher hardening guide the goal is to align the guide with the version of Kubernetes covered in CIS Kubernetes Benchmark v1.5.0. With that goal a hardening guide for Kubernetes v1.16 might be covered in future revisions of the CIS Kubernetes Benchmark.


Can't get or fetch or create an instance for Zend\Db\Adapter\Adapter when running skeleton application on ibmi

The ZF2 skeleton application, using Zf2.1.1 is throwing the following exception after deploying to the ibmi. I have confirmed that mysql and php are working properly by issuing simple mysql_connect and mysql_query calls using the same db credentials being used in the ZF2 app. I also create a simple php script using PDO to connect to mysql and that is working properly as well.
 Feel free to NOT contributing next time, thanks :-)

usefullness of my comment was "anyone who want to reproduce the issue, is free to install CentOS 7 and follow tutorial step by step"
since I was not warned "ZF2 tutorial will not run on CentOS 7", I believe it is a bug

That's indeed better now ;-)
Marco Pivetta
http://twitter.com/Ocramius
http://ocramius.github.com/
On 20 February 2015 at 10:54, Ilya Shipitsin<EMAIL_ADDRESS>wrote:

usefullness of my comment was "anyone who want to reproduce the issue, is
free to install CentOS 7 and follow tutorial step by step"
since I was not warned "ZF2 tutorial will not run on CentOS 7", I believe
it is a bug
—
Reply to this email directly or view it on GitHub
https://github.com/zendframework/zf2/issues/3726#issuecomment-75220544.

**
Finally got it working
**
don't know why but commenting these lines $resultSetPrototype->setArrayObjectPrototype(new Album()); Made it working No other changes were done.

To add when I re-did the tutorial by downloadng the ZF2 skeleton zip from the github it worked like a charm.
Earlier I setup the ZF2 via the composer command line and also the main doc root had an extra folder in parent which I think was causing the issue for me.
Regards,

@flyingbuddha - today I had this problem. Its not obvious at all. Thank you that you mensioned. Why zend cannot add a check and write a normal message that database is not configured? If those checks are too expensive for production, then at least for development mode there should be those. How could I find out quickly this without answers of other people?

If those checks are too expensive for production, then at least for development mode there should be those. How could I find out quickly this without answers of other people?

The proper way to check this is to create a test that instantiates all defined services as a pre-deploy step. This isn't something the framework can do for you.

This is because your adapter object is not loaded. So update your line in application.config.php as follows.
'config_glob_paths' => array(
'config/autoload/{,.}{global,{,.}local}.php',
),

You mean
'config_glob_paths' => [
    realpath(__DIR__) . '/autoload/{,.}{global,{,.}local}.php'
],

some systems (ex. IBMi) need realpath(DIR) in order to pickup these files.
However, never version of the skeleton have been updated and this thread is over 5 years old 😄


feat: opensearch api

This is the initial moto implementation of AWS Opensearch serverless api. I have added 12 API endpoints. Can I get a feedback on this.

This PR would resolve my enhancement request #7759


reverse/2 makes a mistake

v1.25.8
?- X = [_],  reverse([""|Y],B).
   X = [_A], Y = [], B = [[]]
;  X = [_A], Y = [_B], B = [_B,[]]
;  X = [_A], Y = [_B,_C], B = [_C,_B,[]]  % _B,_C !
;  ... .

?- X = [_A],  reverse([""|Y],B).
   X = [_A], Y = [], B = [[]]
;  X = [_A], Y = [_B], B = [_B,[]]
;  X = [_A], Y = [_B,_B], B = [_B,_B,[]]  % _B,_B ?

Naming has been changed
v1.25.20
?- X = [_A],  reverse([""|Y],B).
   X = [_A], Y = [], B = [[]]
;  X = [_A], Y = [_B], B = [_D,[]]
;  X = [_A], Y = [_B,_C], B = [_F,_G,[]]
;  X = [_A], Y = [_B,_C,_D], B = [_H,_I,_J,[]]

I guess _B = _D signifies step-by-step _B = _G, _B = _J and so forth


Initial pass at wandb Ludwig integration

This is initial pass at a basic integration of wandb with Ludwig.  It currently mirrors tensorboard events, stores hyperparameters, syncs artifacts, and stores evaluation results.
I'm using the python "black" code formatter which seems to be clashing with some of the existing code.  Is there a preferred formatter to use?

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

Just tried to sign the CLA and it's blank...

Just tried to sign the CLA and it's blank...

@briankhsieh can you please look into the CLA issue?

This is initial pass at a basic integration of wandb with Ludwig. It currently mirrors tensorboard events, stores hyperparameters, syncs artifacts, and stores evaluation results.
I'm using the python "black" code formatter which seems to be clashing with some of the existing code. Is there a preferred formatter to use?

Thanks for this contribution, will look into it shortly.
As for formatter, we usually use PyCharm's one with the line length set to 79.

It looks like cla-assistant.io is having intermittent issues. It worked for me this morning. Can you try again later?

@briankhsieh got it this time, just took a little while to load.

@w4nderlust I'm using VSCode, if you have any ideas on settings I'm all ears.  I'll do some googling as well.  Other than formatting, any thoughts on the pull?

@vanpelt sorry for the delay on this, was out for conferences. Will look into it by end of day Friday.

I'm really sorry for the long delay @vanpelt , I had some personal issues that kept me away from working on Ludwig, now I'm back. The PR looks good to me as a starting point, just added a couple formatting suggestions. Regarding VSCode, I don't use it, but likely you can set the linter to 80 columns somehow (note sure this is still valid but https://stackoverflow.com/questions/29968499/vertical-rulers-in-visual-studio-code ). Do you prefer if I merge it now or do you want to keep on working on it adding the missing features?

Hey @w4nderlust no worries.  Let me take a pass at getting formatting setup properly and address your comments.  I'll have something shortly.

Ok, @w4nderlust addressed your feedback.   Let me know if I missed anything, I'm still seeing DeepSource issues.  Would love to get my linter / formatter setup correctly in vscode but not sure how best to do it.

I looked at the deepsource complaints. They are msotly related with global and import.
Doing import more than one time in multiple threads would be a problem?
If not, maybe you could do the same thing I'm doing with the horovod import here: https://github.com/uber/ludwig/blob/master/ludwig/models/model.py#L91-L94 Assigning the imported library as a reference variable in self, so that in the rest of the class of the function you just call self.wandb.
It would look like:
try:
    import wandb
    wandb_obj = Wandb()
    setattr(wandb_obj, 'wandb', wandb)
    return wndb_obj

What do you think?

Sorry for the delay on this, I'll hopefully get to it this week.

Ok @w4nderlust finally got to this.  Let me know if you need any other changes!

@w4nderlust Ping

Hey @w4nderlust any updates here?

Checked it out, lgtm, will test it later this week and it everything works fine as expected will merge. please give me some instructions for testing, like an example command or something like that ;)

Awesome!  To test it, create a wandb account here:  https://app.wandb.ai  then run wandb login on your machine.  Then just run a Ludwig job with the —wandb flag.

@w4nderlust any updates here?

@w4nderlust I'm sure the holidays are hectic.  Would love to get this in, do you have an eta?

@w4nderlust I'm sure the holidays are hectic. Would love to get this in, do you have an eta?

Tested it and it works fine, thank you so much :)
Can I ask you one final little piece? As I'm going to port Ludwig to TF2, I would really fancy a testcase for this integration so that I'm sure it keeps working after the porting.

I created a PR for documentation that I believe contains the additional calling points defined here (assuming no additional static methods). You are welcome to add list your contribution/integration flag and links here: https://github.com/uber/ludwig/pull/607

Thanks for your review @dsblank , i agree on those methods being class methods even if self is never used, It's more flexible.

THanks a lot for your work and your patience. Sorry if it took me longer than I expected to get to this, but in the end we merged it :)

Thanks, I'll be playing with it!

Thanks for that great integration guys! Is it now fully working?

Yes!


SharePoint 2013 Database Server Requirements

According to the following article SharePoint 2013 database servers (including SQL Server 2014) must be installed on Windows Server 2012 or 2008SP2.
https://github.com/MicrosoftDocs/OfficeDocs-SharePoint/blob/live/SharePoint/SharePointServer/install/hardware-and-software-requirements-0.md#minimum-software-requirements
@SerdarSoysal
Is it really true that I have to install an older version of Windows Server for the databases than is supported by SharePoint 2013 FE and App Servers (2012R2) or is this information outdated?
Can I not simply go by the supportability Matrix of the SQL Server version (e.g. SQL Server 2014 supports up to Windows Server 2019)?

@alexjebens Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@alexjebens Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

I would also like to add that the same paragraph does not list support for SQL Server 2014 meaning that this document contradicts itself.

I would also like to add that the same paragraph does not list support for SQL Server 2014 meaning that this document contradicts itself.

Hello @alexjebens Alexander, how are you? Currently looking into this...

Hello @alexjebens Alexander, how are you? Currently looking into this...

This article needs to be updated to follow the supported OS guidelines of SQL Server and should probably just link to https://docs.microsoft.com/en-us/troubleshoot/sql/general/sql-server-windows-8-later-versions.

This article needs to be updated to follow the supported OS guidelines of SQL Server and should probably just link to https://docs.microsoft.com/en-us/troubleshoot/sql/general/sql-server-windows-8-later-versions.


Enhancement: Allow X509Certificate to be passed rather than files & password

It currently doesn't seem to be possible to pass an X509Certificate retrieved from the windows certificate store meaning the cert & key must be passed as files with the key password being required in code. This completely ruins the security model since its incredibly easy to decompile and get the password. Passing an X509 certificate would allow the certificate to be installed with the password and then retrieved and used in code without the need for the password.
What I'd like to be able to do is something like:
var cert = GetCertificate("XXXXXXXXXXXXXXXXXXXXXXXXXXX");

var config = new Dictionary<string, object> {
	{ "bootstrap.servers", brokerList },
	{ "ssl.certificate.location", cert}
};

static private X509Certificate GetCertificate(string thumbprint)
{
	var certStore = new X509Store(StoreName.My, StoreLocation.LocalMachine);
	certStore.Open(OpenFlags.ReadOnly);
	try
	{
		var certCollection = certStore.Certificates.Find(X509FindType.FindByThumbprint,
		Regex.Replace(thumbprint, @"\s|\W", "").ToUpper(), false);
		if (certCollection.Count > 0)
			return certCollection[0];
	}
	finally
	{
		certStore.Close();
	}
	return null;
}

Hi!
There is a pull request on librdkafka where someone implemented in memory configs for ssl. You specify private key, cert and CA with byte arrays. https://github.com/edenhill/librdkafka/pull/1338
I've been trying to implement against that with customization to confluent-kafka-dotnet to access the new functions in librdkafka #1338. I haven't been successful with that mostly because my C/C++ knowledge is a bit rusty.
I would really like to see a solution for this.

That pull request was implemented by my colleague. He did give me a compiled DLL with the changes for confluent-kafka-dotnet that I managed to get working. I'm not sure if he submitted a pull request here for those changes though!
We didn't actually end up using kafka!

Hello,
In our organisation we have implemented Two-Way SSL/TLS (Kind of Example 2 in https://github.com/mhowlett/confluent-kafka-dotnet/tree/security/examples/Security).
We have created a producer application using confluent .Net Kafka client - 0.11.6 and it will be deployed on Micorsoft Azure. We are facing an issue in passing the private key location as with Azure it is not possible to export the private key of certificate once it is installed and due to security reasons, we are not allowed to deploy the private key file along with the application.
Is there any workaround for this?
Thanks.

This is not going anywhere is it?

There's ongoing work for this in librdkafka, stay tuned

Hi Guys, any updates on passing thumbprint instead of a cert ?

Hello @edenhill, I have seen you have implemented this in the librdkafka, any ETA on when this will be ported here?

this was added here: https://github.com/edenhill/librdkafka/pull/2309/files


Add automated firmware builder to custom coreboot section

I made an automated Full ROM and altfw UEFI builder: https://github.com/ChocolateLoverRaj/scripts. It is much easier for simple things like changing the logo and timeout.

This just makes it easier for clueless people to brick their devices

Explain

You are enabling people that don't know how to compile coreboot to flash custom builds which always has the chance of bricking

Closing for now, the room for error is too high.

People who don't know how to build coreboot can still use the firmware Utility script to have the same chance of bricking their Chromebook.

Compiling coreboot is kind of like a screening, it filters out the people who know what theyre doing. If a user does not know how to compile coreboot they should not be playing with flashrom.

Right, but the firmware utility script automatically flashes firmware for you. You need to manually flash custom ROMs and some zero clue how and end up doing random thinggs.
On Thu, Oct 12, 2023 at 9:57 PM, Rajas Paranjpe @.***(mailto:On Thu, Oct 12, 2023 at 9:57 PM, Rajas Paranjpe < wrote:

People who don't know how to build coreboot can still use the firmware Utility script to have the same chance of bricking their Chromebook.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you modified the open/close state.Message ID: @.***>


Missing learning rate in entropic_md_update function(?)

Not sure about this one. Don't we need the learning rate as a parameter inside the function to compute the formula stated in the cell above?

As I said in the readme. This time I wanted to leave a bit more work to you. But leaving out some of the arguments in only some of the function might be too confusing so thanks for adding this!


How to map PurchaseStatus enum for payment state value coming from verifyreceipt response to to manually call completePurchase method?

Steps to reproduce
Hello There,
I'm checking an in app purchase package scenario for one use case.
Initially i make a buyNonConsumable method to purchase a subscription.
After that I make an api call from my server to verifyreceipt which is giving me "payment_state":1 and "acknowledgement_state": 0,
After then i check for a property like (purchaseDetails.pendingCompletePurchase) which is giving me true and then I'm making a call for completePurchase method. But somehow at that moment my app closed.
So to manage it next time manually I'm having all the data I need in purchaseDetails object in my server, except PurchaseStatus enum.
So can anyone tell me from where can I map it that? Cause even the PurchaseStatus Class of the package is not providing any proper details how to map it or which states are this
So please add the states required or mapped for PurchaseStatus enum in docs.
Expected results
We need to map have the doc for states required or mapped for PurchaseStatus enum.
Actual results
We dont have states required to map for PurchaseStatus enum.
Code sample
PurchaseDetails purchaseDetails = PurchaseDetails(
              productID: (activePlanResponse.value.activeResponseItem?.plan
                  ?.productId ?? '').toString(),
              purchaseID
                  : (activePlanResponse.value.activeResponseItem?.plan
                  ?.orderId ?? '').toString(),
              status
                  : getPurchaseStatusById(id: state.toInt()),
              transactionDate
                  : (activePlanResponse.value.activeResponseItem?.plan
                  ?.transactionDate ?? '').toString(),
              verificationData: PurchaseVerificationData(
                  source: activePlanResponse.value.activeResponseItem?.plan
                      ?.source ?? '',
                  localVerificationData: activePlanResponse.value
                      .activeResponseItem?.plan?.localVerificationData ?? '',
                  serverVerificationData
                      : activePlanResponse.value.activeResponseItem?.plan
                      ?.serverVerificationData ?? ''));
          await purchaseDetails.completePurchase(purchaseDetails);

Above values I'm fetching from my server and mapping but don't know how to map this PurchaseStatus value.
I'm getting an error like following when try to map status manually,
updatePurchaseProducts error :'package:in_app_purchase_android/src/in_app_purchase_android_platform.dart': Failed assertion: line 176 pos 7: 'purchase is GooglePlayPurchaseDetails': On Android, the purchase should always be of type GooglePlayPurchaseDetails.
Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.10.5, on macOS 13.4 22F66 darwin-x64, locale en-IN)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] VS Code (version 1.85.2)
[✓] Connected device (3 available)
[✓] Network resources
• No issues found!

@fldvlpr
Based on the error you are getting, check if https://github.com/flutter/flutter/issues/82645#issuecomment-841853349 helps in your case or not.


Implement the MultiOnEmpty group

Group retrieved using multi.onCompletion().ifEmpty() with the following methods:
* failWith (Throwable, Supplier<Throwable>)
* continueWith (T, Supplier<T>)
* switchTo (Multi<T>, Supplier<Multi<T>>)

Moved to onCompletion().ifEmpty()


direct drivable area segmentation with yolact

Thanks for your wonderful job！I have implemented direct drivable area segmentation with yolact_darknet53_config.And the test video below shows that it really works.But for the speed,is there any solution that use more lightweight model to be the backbone,for example MobileNets?
https://youtu.be/ZzLeWhUnpmI

Some people in #27 are trying out mobilenet, but I think you already saw that. Other than that, no I don't have plans to implement backbones other than the 4 we already have. However, adding your own backbone should be pretty easy. Check out the code in backbone.py and you should be able to follow along. If you do decide to try to implement it yourself, feel free to ask questions if you're stuck.
Though, mobilenet (because it's so lightweight) might actually struggle a lot with YOLACT, because we leave localization to the network.
Actually--if your application is drivable area segmentation, wouldn't it be better to use a semantic segmentation method? YOLACT makes compromises to be able to do instance segmentation, so if you don't need instance segmentation you're much better off just using semantic segmentation. (Though correct me if I'm wrong. I'm not familiar with drivable area segmentation)

Yep!Thanks for your prompt reply.As you said, it's better to use semantic segmentation methed  to implement coarse drivable area segmentation especially taking speed into account.
But other classes will be added in the forthcoming experiment,for example alternative drivable area and car.Semantic segmetation method is hard to find the every lane‘s boundary when number of lane and driving car’s perspective is uncertain.So I think Instance segmentation method should be adopted.
Above all,thanks again for your contribution.

That makes sense. By the way, if you want it, there's actually a semantic segmentation branch in YOLACT used only for training. See #73 for where you can expose that branch.

Thank you for the quick response!
I will try the experiment based on your instance segmentation method when alternative drivable area is added.Then test if the model works.of course,I will try the method that you suggest.
Thanks again~

Here is the test image, green area is the direct drivable area;and the other is the alternative drivable area.it seems fine,of course it depends the labeled datasets

@pandamax  thanks for sharing the idea even i wanted to have a look into it , so as the author suggested did u try the semantic segmentation method for driveable area
@dbolya  the link you have pointed in your comment can be used for semantic segmentation right ?
i wanted to do lane marking detection also

@abhigoku10 Yeah we already have a semantic segmentation module built in. You just need to expose it like in #73.

@pandamax  did u perform the instance segmentation or semantic segmentation for your dirveable are a application , if so what is the code changes made to obtain semantic segmentation

@pandamax how many images have you trained you model with to get the output??
@dbolya  to get drive-able and non drive-able area should we train the architecture with label

@abhigoku10 Yeah, train with two instance annotations per image: one for the drivable area, and the other for the non-drivable error. Then, ignore the instance segmentation output of the model and just use the semantic segmentation.

@pandamax how many images have you trained you model with to get the output??
@dbolya to get drive-able and non drive-able area should we train the architecture with label
In my experience, I got the datasets from BDD100K, you can use the drivable-area data and convert the data format to cocostyle, here is a script may help you. And train dataset is 70K, val datasets is 10K.

meanwhle, I opt the instance segmentation method.

hey @pandamax  script which you have  suggested to convert to cocostyle is not available

@dbolya  any method on training for these low images , any params we can tune so to see the model generated

@abhigoku10 Wait you're training from scratch from scratch? i.e. not even imagenet pretraining? There are almost as many parameters as pixels in your training data...
You can try changing learning rate and stuff like that, but I doubt you'll get anywhere.

@dbolya i am using resent50 imagenet pretrained model for training with kitti 200 images annotated with driveable area as a separate class

@abhigoku10 Right, but you realize that even Mask R-CNN does abysmally on KITTI? There are so few images that the best methods are traditional computer vision hand-crafted algorithms. Machine learning techniques just don't work as well with so few images.
You can try reducing the learning rate, but it's probably just going to overfit.

@dbolya @pandamax  i trained the model using BDD100k data set i am getting the exact boxes but the mask is zero , even during training the mask value during validation is zero , but i am getting proper boxes of the road from the model currently the training set consists of 90k images

@abhigoku10 Huh, that shouldn't be happening. Maybe you have a bug somewhere in your config, or if the dataset was not originally in COCO format, a bug in conversion?

@abhigoku10 here is script you to check if your data format is cocostyle and visualize your dataset. That may do some help.

@pandamax @dbolya  after a through evaluation i had set this (set 'freeze_bn': True in the config). which i made it false now its working on BDD100k dataset , now i shall train on the KITTI dataset and update the results

@dbolya @pandamax i have got better results after training using the dataset . I have few queries
1.the mask value for the road surface detection during validation mAP is low its in the range of 5-10
2. i tried this for lane detection and i get the same the mask value of the range 0-1
3. Any metrics to validate the output obtained of driveable area and alternate driveable area

@pandamax did you actually manage to add road objects to your direct drivable area segmentor?


Config refactor to Typesafe Config

This PR is a refactor of the configuration mechanism of the Kafka Monitor from json/map to Typesafe Config.
Why?

I needed to override several configuration values with environment variables (I'm using it within Docker). Typesafe Config makes this much easier with its ${?MY_ENV_VAR} syntax.
less verbose configuration syntax
A more structured type to indicate the configuration (Config instead of Map). Also no more casting of values when retrieving a property, Typesafe Config has getString, getBoolean, getList etc. methods

closing this PR, as I think it's better to approach this a different way.