Fix CUDA CMake warning settings ("warning: style of line directive is a GCC extension")

/home/ryan/cudagrad/src/operations.cu:17:3: warning: style of line directive is a GCC extension
   17 |   helloFromGPU<<<1, 1>>>();
      |   ^~
/home/ryan/cudagrad/src/operations.cu:18:3: warning: style of line directive is a GCC extension
   18 |   cudaDeviceSynchronize();
      |   ^~
/home/ryan/cudagrad/src/operations.cu:19:3: warning: style of line directive is a GCC extension
   19 | }
      |   ^
/home/ryan/cudagrad/src/operations.cu:20:3: warning: style of line directive is a GCC extension
   20 |
      |   ^
/home/ryan/cudagrad/src/operations.cu:23:3: warning: style of line directive is a GCC extension
tmpxft_00001daf_00000000-6_operations.cudafe1.stub.c:4:3: warning: style of line directive is a GCC extension
In file included from tmpxft_00001daf_00000000-6_operations.cudafe1.stub.c:1:
/home/ryan/cudagrad/src/operations.cu:15:3: warning: style of line directive is a GCC extension
   15 |
      |   ^
In file included from tmpxft_00001daf_00000000-6_operations.cudafe1.stub.c:1:
/home/ryan/cudagrad/src/operations.cu:15:3: warning: style of line directive is a GCC extension
   15 |
      |   ^
tmpxft_00001daf_00000000-6_operations.cudafe1.stub.c:2:3: warning: style of line directive is a GCC extension

Cant reproduce


#9147

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[ ] Feature
[x] Refactoring

Needs QA check

@kodadot/qa-guild please review

Context

[x] Closes #9147

Did your issue had any of the "$" label on it?

[x] My DOT address: Payout

Screenshot 📸

[] My fix has changed UI

Copilot Summary

I am a bit lost, @daiagi @roiLeo any idea why this build is failing? its only 1 character change

I am a bit lost, @daiagi @roiLeo any idea why this build is failing? its only 1 character change

safe to ignore. https://github.com/kodadot/nft-gallery/pull/9166#pullrequestreview-1850341638

mkey then
thanks!
pay 20 usd

😍 Perfect, I’ve sent the payout
💵 $20 @ 6.93 USD/DOT ~ 2.886 $DOT
🧗 13Qx65nLd6SwdtjrRyuoEtp9CKXhF651xdHBPaXcvhwKm4N1
🔗 0xe98cced2edcdb4b94012f88b626d100574c37e9f2e3fe011582f787bc9c18e29
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


fix: concat raw query for virtual module request

Plugin.resolveId returns resource path without query and used directly as the new request for uno.css virtual module, raw request query will lost and may disturb some other compile processes rely on this.
In my opinion, raw request query should be preserved after resolve, to avoid problems mentioned above.

I think the query should be handled within plugin's resolveId

I think the query should be handled within plugin's resolveId

Yes I also considered about that, but the query should be also processed here, because the rule for load match vfsModules with a resource path without query which is provided by include condition of webpack ruleSet, any result with query returned by plugin's resolveId will break this match.
include (id) {
  return id != null && plugin.__vfsModules!.has(id)
},

So I guess plugin's resolveId returning a id without query is intentional designed.

I updated this pr to accept the result with query returned by plugin's resolveId, and should be worked with https://github.com/unocss/unocss/pull/1142


add switch parameter for writing the output as HTML

...or some other markup language. I mean, for a few measly extra lines... why the hell not?

cool fact of the day for ya .. the dot above an i or j is called a TITTLE (stop laughing)

ima have to look that up in the Urban Dictionary to confirm something. Added to my TODO list.
As for the xkcd... well, this is legit. It would make visual parsing easier, and it should take just a handful of lines to get it rolling. About 5 according to my estimation, without looking at the code. That's of course if I take the simplest route, but I think adding some pretty CSS in the output would be nice, and it wouldn't take many extra lines. I could even write TWO styles, and add an EXTRA switch parameter for toggling dark/light mode. Even better: if I'm already including CSS, why not provide it in a separate file instead of embedding it, to make it more modular? It would only take another couple short steps from there, and users would then be able to write their own styles if they wanted! While I'm at it, I could even add some good ol' JavaScript in, to add controls for filtering the output and all. Sounds great, doesn't it?! But, something bugs me: if I'm already dealing with HTML + CSS + Javascript, why not turn the whole thing into a WEB APP?  I'm concerned for your safety ... the last commit was a week ago. Have you been trapped in the neighbor's garage or something? Please advise

Aww. The pants missed me! Sorry, I got myself stuck in a toilet paper tube for so many days that I look smaller now. I was also ill for a few days, and after that I just got lost in my research of random stuff of questionable usefulness!
I suspect this issue will remain open for some time... :laughing:

readme:

Additionally, it can catch one type of syntax error (for now), and includes that information in the report

includes should be singular

claustromaniac .. is this you: https://thechive.files.wordpress.com/2018/08/daily_picdump_2917_640_high_15.jpg :grinning:


Feat: Add default keymaps

Resolves #287.
In this PR I introduce the following changes:

keymap settings are all grouped together in a common field in the config table
keymaps can be disabled globally or individually
global keymaps for most gitlab.nvim API functions are set
keymaps for common actions in the reviewer windows are set
warnings are shown for removed keymap settings fields

Some of the default keybindings in this PR are different from the current defaults. The following are the differences and the reasons for the suggested changes. Some of them are more useful than others and I can revert any of those to the current default in main if you prefer to keep the breaking changes to the minimum. Of course I can remap any of these in my personal config. However, I think that some of these suggested mappings nicely override the behaviour of builtin normal mode commands in non-modifiable buffers and that other users might like these mappings too. It would be nice if more users could review this PR and express their opinions about the suggested changes.
Global keybindings

choose_merge_request: mapping changes from glb to glc as this is a better mnemonic and is now free for mapping since "glc" is no longer needed for a global create_comment mapping
the funciton gitlab.review has the start_review keymap field and the mapping changes from glr to glS. glr is the common prefix for add_reviewer and delete_reviewer, so with glr alone the user needs the wait for timeoutlen for gitlab.review to be called. start_review and glS are more explicit.
create_mr: is mapped to glC as this is a clearer mnemonic than glO

Popup

perform_action is mapped to ZZ as the bulitin behaviour of ZZ is not needed (we don't really need to write the current file for the popup windows), but I think that it nicely captures what we want to do: apply changes and close the window. This is similar how the commit buffers are handled when vim is the default editor for git. I'm also planning to add the ZQ mapping that would close the window without saving the buffer contents to the temp_registers, so that it would logically complement ZZ.

Discussion tree

jump_to_reviewer changes from m to a - The point is to have the same keymap for jumping back and forth between the reviewer and discussion tree, but I'd like to keep the m for setting marks in the reviewer.
refresh_data changes from a to <c-r> - this is a common keybinding for refreshing in browsers and other applications. It's a modifying keybinding, so it's free to remap in a non-modifiable buffer.
switch_view changes from S to c - I'd like to add some functionality around accepting suggestions for which the S will be a more suitable mnemonic. We could change switch_view to change_view to make this one also more natural.
toggle_resolved changes from p to -. I think it might be useful to add some preview functionality to the discussion tree, which would make it possible to for example see suggestion diffs. For this the p would be a nice mnemonic. - is used for toggling in other plugins, e.g., fugitive.

Other notes

Some of the code that I've added is quite repetitive - adding global keymaps could probably be refactored to use a loop and get rid of some of the duplication, but I guess this can be done in a separate issue.
For the reviewer keymaps, I've only used the functions create_comment_suggestion and create_multiline_comment, since create_comment for one line is just a special case of a "multiline" comment. I think in the future we could remove the create_comment function altogether to reduce the code complexity. What do you think?

One problem with the newly set reviewer keymaps is that they are buffer-local but not window-local (window-local mappings are not possible in (n)vim). That means if I'm working on a MR and want to modify one of the files open in the reviewer, I press DiffView's mapping gf, I get the file opened in a separate tab, but the reviewer mappings are still in place, which is not desirable - I really want create_comment/suggestion to be mapped to s and c, but then I cannot use these keys to modify the file.
I'll try to fix this with the help of autocommands.

One problem with the newly set reviewer keymaps is that they are buffer-local but not window-local (window-local mappings are not possible in (n)vim). That means if I'm working on a MR and want to modify one of the files open in the reviewer, I press DiffView's mapping gf, I get the file opened in a separate tab, but the reviewer mappings are still in place, which is not desirable - I really want create_comment/suggestion to be mapped to s and c, but then I cannot use these keys to modify the file.
I'll try to fix this with the help of autocommands that would fire when entering and leaving the specific window, not just the specific buffer.

This is now fixed by deleting the keymaps within the already existing set_callback_for_reviewer_leave function.

This looks great, thank you for working on it and sorry it took me so long to get to the review. I've left some suggestions but mostly this looks great to me! 🚀

Hi. Thanks for the review. I'm on vacation and I'll get back to this next week.


V 0 28 0

(insert scope and description here)
Items:

[ ] Ran test & live-tests
[ ] Implemented (new) unit tests
[ ] Removed credentials
[ ] Removed argument pointing to staging
[ ] Updated documentation

For release:

[x] Bumped version
[x] Added changelog
[x] Updated announcement banner

Also replaced code font with links to the attributes and functions we're talking about.


Challenge (Create a Form Element): Remove root relative path from action URL in example

Challenge: https://www.freecodecamp.org/learn/responsive-web-design/basic-html-and-html5/create-a-form-element
This is a very minor "issue". But I have seen a few people on the forum adding the / in front of the absolute URL and I think it is because of the example code given. I would suggest we just remove the slash.
Example code:
<form action="/url-where-you-want-to-submit-form-data">
  <input>
</form>

So just this instead:
<form action="url-where-you-want-to-submit-form-data">
  <input>
</form>

Not sure if, or where, we teach about paths. But I think most paths used in the curriculum are absolute or relative.

@lasjorg shall I take this up?

Is anyone dealing with this issue? I could take upon it and make the PR if it's okay for everyone.

Is anyone dealing with this issue? I could take upon it and make the PR if it's okay for everyone.

I am. I'll let you know if it doesn't work for me,


Feature Request: Add a toggle to use old-style Namespaces

I propose a flag that allows me to use old-style namespaces. If you use older versions of C# for targeting console, the code produced by the generator doesn't work as it's not valid code. I would be willing to make a PR.
Examples:
Currently Generated :
namespace MyNamespace;
public void MyClass{}

Desired Generation or Optional Formet:
namespace MyNamespace {
    public void MyClass{}
}

you can just specify the c# version in the csproj, whats the reason you are using older versions?


Code snippet for tutorial on folding  technique (BAEL-2949)

Since we are providing javadoc let's complete it. Javadoc also for the variables, etc. Also let's format the code, there are files with 2 line feeds instead of one and a lot of line feeds in the end as well.


"Download CSV" and "Download Text" returning a file with written only "undefined"

We're using opendistro-for-elasticsearch-kibana:1.7.0 docker image.
When we use SQL Workbench with LIMIT 10000 and indices with about 150 fields:

it's possible to get a full correct CSV file using _opendistro/sql?format=csv API,
the "Download CSV" and "Download Text" options return a 9 bytes file with just the string "undefined",
the "Download JSON" and "Download JDBC" options work instead,
in some of the cases, after first using "Download JSON", the "Download CSV" option starts working from the same query (maybe the faster response for catching helps the browser),
tested with Chrome and Firefox.

Unfortunately I'm not able to reproduce this bug with sample data, but I'll be happy to provide any additional information.

Hi @steadfastgaze thank you for reporting this issue, we will take some investigation and make a fix to it when we have resources. To quick break the blocker, could you try to download the file(s) for a second time if the downloaded file is "undefined"? That might help.

Hi @chloe-zh, thank you for support and nice catch, downloading the file for a second time (after the first time fails as undefined) works!


LibUsb prints out error messages when multiple devices are in use.

LibUSB should not generate error messages when opening two Azure Kinects at the same time.

See https://github.com/microsoft/Azure-Kinect-Sensor-SDK/issues/485 for a more specific description to this error.

See #485 for a more specific description of this error.


Update for current EventStore

Requires a non-empty sharding columns list (use pkey)
Default port is now 1100.
Fix some typos in comments

signed the CLA

Thanks again, @markstur and @stevemart!


trying to put this in a modpack update for Steve's New Glaxy bud sadly wont be able to

Describe the bug
crashes mc on boot
To Reproduce
Steps to reproduce the behavior:

put mods in mods folder
run game
game break, get error

Expected behavior
mod work :(
Screenshots
crash report file attached
crash-2021-05-19_03.16.41-client.txt
I suppose i will have to leave this mod out of the pack for now, i would love to add it in if it gets fixed looks amazing!

Outdated Advanced Rocketry. Version 2.0.0 or higher required. Use previous version of OCRocketry, might work


What is LayoutAnimationRepository?

I just want to know what LayoutAnimationRepository is. Preferably I'd like a copy of the code it represents.
I just need it so I can finally compile this library outside of the node_module folder and make a few edits so I can hopefully fix the way scrolling on drag displaces the item being dragged,.
Please and thank you.

@JonasMVeit it's something of a global layout animation repository, which he is using to add experimental animation to react-native-reanimated 2.x.x, which won't be needed once upgraded to react-native-reanimated 3.x.x as the 3.x.x version handles layout animations perfectly for Lists.
You can see a demo of the changes in my PR: #550


Proposal: CareWithShare

Description

CareWithShare

The DApp is a decentralized crowdfunding platform built on the Kaia Blockchain, modeled after GoFundMe, where users can create charity campaigns, raise funds, and contribute to causes in a secure, trustless, and transparent manner. Each campaign operates through smart contracts that handle the collection of funds, monitor the fundraising progress, and ensure proper fund allocation based on preset conditions (e.g., reaching the funding goal or deadlines).

Campaign creators can withdraw funds only when certain goals are met, while contributors can request refunds if the target isn’t reached.

Bounty tracks:

Defi
Socialfi

Approved. A couple notes:

this is a typical ICO fundraise, nothing new
i recommend you to implement a timelock on the amount raised and withdrawal period to further protect funders

Thank you, i will think more about that function!


Configure retry time for external keystone

Add more control on how often operator will check availability
of external keystone endpoint.

recheck

/gcbrun

recheck

Why there is a need to monitor external keystone?

I don't like the fact that keystone controller is used to manage and perform health check or readiness check of external services. It should be only used to deploy and manage keystone deployment in k8s. Keystone address should be pass as parameter to services that use it, regardless if it is local one or external one.

Moreover operator should work only having access to k8s api, but now it has to have access to some external service. That might be problematic if we run it locally so not in the cluster.

There is an use case for using existing keystone, I understand it does not perfectly fit to native way of handling resource, but  does k8s resources always operates without need of accessing external resources?  Yes we are waiting for readiness of keystone, as other services waits for postgres or swift. That is not typical event based watch which is dictated of external nature of this resource. The services which uses keystone, get information like address from keystone resource status, I did not change that. I also reused already existing keystone client, with required changes introduced to it. Not sure if understand what you mean by running locally? The external keystone related code (including checking if it possible to fetch token) is only activate when the ExternalAddress property is set for keystone resource, it is not a default behavior, must be user decision.

However fixing that will require a lot of work, so I will approve this change.
How would you address this use case differently? I can work further on this.


Gitleaks ignoring git safe.directory

Describe the bug
When running gitleaks at Gitlab CI it returns the following error and workaround:
ERR fatal: unsafe repository ('/opt/src' is owned by someone else)
ERR To add an exception for this directory, call:
ERR
ERR 	git config --global --add safe.directory /opt/src

However, the workaround doesn't seem to be taking effect as the error keeps happening.
To Reproduce
gitlab-ci.yml file
sast:gitleaks:
  extends: .external
  stage: security (sca/sast)
  script:
    - apk add git
    - make gitleaks
  artifacts:
    when: always
    reports:
      secret_detection: "$REPORT_PATH/secret-detection.json"

Execution output
touch "output/reports/secret-detection.json"
git config --global --add safe.directory /opt/src
docker run -v /builds/application:/opt/src \
	zricethezav/gitleaks:latest detect \
	--source "/opt/src" \
	--report-path "/opt/src/output/reports/secret-detection.json"

Expected behavior
Add directory to exception list and don't fail
Screenshots

Basic Info (please complete the following information):

OS: gitlab-runner 14.9.0~beta.68.g1283960c
Gitleaks Version: 8.7.0

Additional context
Add any other context about the problem here.
cc @zricethezav

Root cause: The owner of the directory is different than the (current) user, performing git/gitleaks
(as in the example below, root and gitleaks)

the first run gitleaks detect --source "/src" got error

the fix git config --global --add safe.directory should be placed at same docker/user as the gitleaks  command

the second run gitleaks detect --source "/src" is successful

$ docker run -it --entrypoint "" \
                 -v /home/ec2-user/Hello-World:/src \
                 zricethezav/gitleaks:latest bash
bash-5.1$ ls -lah /src
total 4K
drwxr-xr-x    3 root     root          32 Apr 19 13:36 .
drwxr-xr-x    1 root     root          17 Apr 19 13:40 ..
drwxr-xr-x    8 root     root         163 Apr 19 13:36 .git
-rw-r--r--    1 root     root          13 Apr 19 13:36 README
bash-5.1$ id
uid=1000(gitleaks) gid=1000(gitleaks)
bash-5.1$ gitleaks detect --source "/src"

    ○
    │╲
    │ ○
    ○ ░
    ░    gitleaks

1:41PM ERR fatal: unsafe repository ('/src' is owned by someone else)
1:41PM ERR To add an exception for this directory, call:
1:41PM ERR
1:41PM ERR      git config --global --add safe.directory /src
bash-5.1$ git config --global --add safe.directory /src
bash-5.1$ gitleaks detect --source "/src"

    ○
    │╲
    │ ○
    ○ ░
    ░    gitleaks

1:41PM INF scan completed in 56.652421ms
1:41PM INF no leaks found
bash-5.1$

fixing for GitLab CI:

override entrypoint to run the fix git config --global --add safe.directory

test:
  stage: test
  image:
    name: zricethezav/gitleaks:latest
    entrypoint: [""]
  before_script:
    - git config --global --add safe.directory $CI_PROJECT_DIR
  script:
    - gitleaks detect

This is an environment issue rather than a gitleaks issue. If you need support on getting your environment set up I will be offering commercial support soon. Closing this issue for now.
Thank you @tuan1986le for providing a solution.

Thank you, @tuan1986le. Works perfectly


LL-1320 Adds missing styles for unconfirmed operations

Adding the unconfirmed state to operations, both on the operation list and operation details while at the same time reworking the SVG icons (thanks @bscholving) since there were some issues with masking and react native.

Type
Feature, UI Polish
Context
https://ledgerhq.atlassian.net/browse/LL-1320
Parts of the app affected / Test plan
Operation list row and Operation details for confirmed/unconfirmed operations

@Arnaud97234


chore(workflow): migrate release tagging to github actions

Why is this PR required? What issue does it fix?:
The PR migrates the release tagging process from travis to github actions as per this issue.
What this PR does?:

add downstream tagging to release
remove travis file
update build status badge

Does this PR require any upgrade changes?:
If the changes in this PR are manually verified, list down the scenarios covered and commands you used for testing with logs:
Any additional information for your reviewer?:
Mention if this PR is part of any design or a continuation of previous PRs

GR_TOKEN secret needs to be added to the repo before merging
travis app integration can be removed from the repository
Dependent on #86

Checklist:

[ ] Fixes #
[x] PR Title follows the convention of  <type>(<scope>): <subject>
[ ] Has the change log section been updated?
[ ] Commit has unit tests
[ ] Commit has integration tests
[ ] (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
[ ] (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

@kmova Can you add the GR_TOKEN to this repository


PitCrew send/receive pit strategy

Relates to #50.

[ ] Read pit strategy menu from the game telemetry and send it to the server
[ ] Web clients to send pit strategy command back to RaceDirector (and all other connected clients)

Log it

[ ] Open the pit menu when receiving a pit strategy

Create a RaceDirector.DeviceIO project (and RaceDirector.DeviceIO.Interface) to send key strokes

[ ] Expose pit strategy request as an Observer for other plugins (e.g. voice commands) to send pit strategy requests

Create PitCrew.Interface project

[ ] Docs

R3E fields:
public struct Shared
{
  // What is currently selected in pit menu, and array of states (preset/buttons: -1 = not selectable, 1 = selectable) (actions: -1 = N/A, 0 = unmarked for fix, 1 = marked for fix)
  public Constant.PitMenuSelection PitMenuSelection;
  public PitMenuState PitMenuState;
}

public struct PitMenuState
{
  public int Fuel;
  public int FrontTires;
  public int RearTires;
}

ACC fields:
public struct SPageFileGraphic
{
  public float MfdFuelToAdd; // How much fuel to add on the MFD
  public int MfdTyreSet; // # of tyre set on the MFD
  public Wheels<float> MfdTyrePressure; // Tyre pressures on the MFD
}


[Feedback] Stop including error messages in the feedback body

Related command
az feedback
Description
Stop including error messages in the feedback body and explicitly ask the user to remove sensitive information.
Additional information
Microsoft internal tracking ID: SIR12534797

Az Feedback refinement to fix an issue.


on upgrade of git branch check against remote

Compares a branch ref after first fetching. This should be the only case it needs to first fetch.

Comment and waiting for tests, otherwise +1.


ProtocolError

I am seeing a ProtocolError when adding the interation to HomeAssistant.
HomeAssistant logs show a 500 Internal Server Error.
The error messages seem to be very similar to
https://github.com/travisghansen/hass-opnsense/issues/45
I am on 23.1.1, not sure of the recent fixes would also fix my issue.

same here
| 23.1.1_2

same here as well
opnsense: 23.1.1_2
hass-opnsense: v0.1.13

I just released v0.1.14 (with the fix that was in main). Give it a try and we'll see if that fixes the issue.

Was experiencing the same issue after updating opnsense to 23.1.1_2.  Just applied the update above and can confirm this resolves the issues.  Thanks!!!

Was experiencing the same issue after updating opnsense to 23.1.1_2. Just applied the update above and can confirm this resolves the issues. Thanks!!!

same, Thanks @travisghansen

@travisghansen thank you for your work! works again


Roe 382 individual bo validation

https://companieshouse.atlassian.net/browse/ROE-382
Spec update
Also updated the handling of the country list as this will require both overseas and uk countries.
The nature of control fields all need to be tested together so added an umbrella nature of control variable as what is in the dto is too fine grained.

As a general question, which part of the code changes address AC 2 in the linked story:
Given that I am on the BO individual page
When I enter more than 50 characters to the "nationality", "residential_address" and "residential_service_address" fields
Then validation fires to let me know 50 is the maximum amount of characters allowed

The address field except country and postcode are already 50 characters max but needed to update and test nationality.


Repeated menu underlined shortcut keys

There are repeated underlined characters in menu items, at least under menu Selection.

Version: 1.58.2
OS: Linux x64 4.19.0-17-amd64

Steps to Reproduce:

Press Alt + S
Press S

There are multiple menu items with an underlined S character.

In general, it might be good idea to revise menu items for underlined characters.

No, these issues are not talking about the same thing. Not at all.

I don't think its great to change these now and we already handle this gracefully, but leaving this up to @bpasero and @alexdima


trackRequestResult & angular guard not returning cached data

Which @ngneat/elf-* package(s) are the source of the bug?
requests-status, requests-cache
Is this a regression?
No
Description
Hi,
We just begin to use elf (instead of akita) for a simple need of cache (store will come later).
In our case we have a route behind a canActivate Guard.
If we use trackRequestResult with cacheResponseData: true, the next call to the request return an EMPTY Observable.
This appears to be because the query never trigger the 'complete' state so we got nothing from this code :
if (result.fetchStatus === 'fetching' && preventConcurrentRequest && !options?.skipCache && !stale) {
  return getRequestResult(key).pipe(
    filter((requestResult) => requestResult.fetchStatus === 'idle'),
    switchMap((requestResult) => {
      if (requestResult.isError) {
        return throwError(() => requestResult.error)
      }
      if (requestResult.isSuccess) {
        return of(requestResult.responseData)
      }
      return EMPTY
    }),
    take(1)
  )
}
)

We go in cause if return true but got nothing cause filter((requestResult) => requestResult.fetchStatus === 'idle' emit nothing.
Please provide a link to a minimal reproduction of the bug
No response
Please provide the exception or error you saw
No expetion, no error, just nothing

Please provide the environment you discovered this bug in
A service API with the request tracked

// todos.service.ts

getTodos(): Observable<Todos> {
  return this.httpClient.request<Todos>(...).pipe(
    trackRequestResult(['todos'], { cacheResponseData: true })
  );
}

A component behind a route with canActivate guard
const routes: Routes = [
  {
    path: '/todos',
    component: 'TodosComponent',
    canActivate: ['TodosGuard']
  }
]

In this guard use the service to call the API request (the switch hide business logic, simplified here)
canActivate(_route: ActivatedRouteSnapshot, _state: RouterStateSnapshot): Observable<boolean> {
  return this.todosService.getTodos().pipe(switchMap(() => of(true)));
}

In the component, call the same request
ngOnInit(): void {
  this.todosService.getTodos().subscribe() // or | async in html;
}

The component never get the cached data (done by the guard)
Anything else?
We override the request-result.ts fro mthe lib and it seems that the query from the guard never goes through the complete here :
return source.pipe(
  tap({
    error(error) {
      console.log('error ' + key)
    },
    next(data) {
      console.log('next ' + key)
    },
    complete() {
      console.log('complete ' + key)
    },
    finalize() {
      console.log('finalize ' + key)
    }
  })
)

We have the next and the finalize (we had it) but never the complete.
If we subscribe in the guard instead of returning it, it seems to work.
canActivate(_route: ActivatedRouteSnapshot, _state: RouterStateSnapshot): Observable<boolean> {
  this.todosService.getTodos().pipe(switchMap(() => of(true))).sbuscribe();
  return of(true);
}

No solution found till now for us.
Do you want to create a pull request?
No

But EMPTY completes the observable - https://github.com/ReactiveX/rxjs/blob/master/packages/rxjs/src/internal/observable/empty.ts#L65

I guess the issue is that it doesn't return a boolean


Get list of classes as they appear in stylesheet

If I have a css file index.css and I wanted to get a list of all the classes as they appear in the file how would I do that?
I am able to get a list of rules and selectors but how can I get the class name of each selector?

Hi. It's not possible. simplecss designed just for style matching.

Ah okay thanks, if possible could you point me in the right direction to do that?

I guess you can use cssparser, but it's pretty heavy and low-level.
On the other hand, you can simply fork simplecss and make Selector's internals public. The main problem is that simplecss stores a selector as an AST and class selectors are converted/desugared into [style~=value].


Revert "Pass route state to onUpdate callback"

Reverts rackt/react-router#2507

Why?

@ryanflorence
Oops, probably should have referenced the specific comments in the PR.
Per @mjackson - comment thread starting at https://github.com/rackt/react-router/pull/2507#commitcomment-14387913, plus related discussion on https://github.com/facebook/react/issues/5459#issuecomment-156510527.
As the intended use case for Flux integration looks to be an anti-pattern, it seemed best to back this out for now to prevent people from doing that.


Unable to mock the setting of an environment variable

Attempting to test the following code:
def method5() {
    def item1 = env.ITEM1 ?: 'default'
    def item2 = "value-${item1}"
    env.setProperty('ITEM2', item2)
    item2
}

With the following unit test:
    def 'test method5'() {
        setup:
            explicitlyMockPipelineVariable('env')
            getPipelineMock('env.getProperty').call('ITEM1') >> 'item1Value'
        expect:
            myExample.method5() == 'value-item1Value'

I continue to get the following error:
groovy.lang.MissingPropertyException: No such property: ITEM2 for class: com.homeaway.devtools.jenkins.testing.PipelineVariableImpersonator

When testing it a bit differently using the following method:
    def 'test method5-2'() {
        setup:
            explicitlyMockPipelineVariable('env')
        when:
            myExample.method5()
        then:
            1 * getPipelineMock('env.getProperty').call('ITEM1') >> 'item1Value'
            1 * getPipelineMock('env.setProperty').call('ITEM2', 'value-item1Value')
    }

I get the same error.
I've successfully unit tested cases for retrieving environment variables, but code that includes setting using 'setProperty' fails with the error above.
I'm thinking the best route is to create my own mock object for environment, but I'm not sure what class to Mock i.e. what the Environment API class is in Groovy.
            def mockEnv = Mock(Object)
            myExample.getBinding().setVariable('env', mockEnv)

Any advice is appreciated. Thanks.

I'm sorry it took so long to get back to you on this!
This is a common slip-up: "Pipeline Variable" does not mean a variable in a pipeline, but specifically "The symbol name given to a Java class that has implemented Jenkins GlobalVariable extension point."
env is not that - it's just* a regular Groovy variable - so use .setBinding(...) on your Script to populate it, not explicitlyMockPipelineVariable(...).
See a full explanation & discussion in this FAQ issue comment: https://github.com/homeaway/jenkins-spock/issues/20#issuecomment-452765199

The 2.1.0 release of jenkins-spock expanded the documentation for working with variables: https://javadoc.io/static/com.homeaway.devtools.jenkins/jenkins-spock/2.1.1/com/homeaway/devtools/jenkins/testing/JenkinsPipelineSpecification.html#using-pipeline-variables


Expect all expressions to have a type after type checking.

Rather than skipping expressions that have no type in post-type checking
verification, verify that they do have a type.
There is one exception to "all" expressions having a type. We serialize
the un-typechecked raw values of @objc enums, and thus after
deserialization we do not have types for these. We cannot distinguish
these in the verifier, though, so we just allow any integer literal
through without a type.

@swift-ci Please smoke test

@swift-ci Please smoke test


content on UI5con website updated with travel information and evening…

… event content

 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.


python312Packages.azure-mgmt-batch: 17.2.0 -> 17.3.0

Changelog: https://github.com/Azure/azure-sdk-for-python/tree/azure-mgmt-batch_17.3.0/sdk/batch/azure-mgmt-batch
Description of changes

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.

Result of nixpkgs-review pr 296932 run on x86_64-linux 1

4 packages built:

python311Packages.azure-mgmt-batch
python311Packages.azure-mgmt-batch.dist
python312Packages.azure-mgmt-batch
python312Packages.azure-mgmt-batch.dist


Enable parse-backticks on search and subscription pages

Closes #6806
Also enables backticks on subscription pages

Test URLs

https://github.com/search?q=repo%3Arefined-github%2Frefined-github+is%3Aissue&type=issues
https://github.com/search?q=repo%3Arefined-github%2Frefined-github+is%3Apr&type=pullrequests
https://github.com/search?q=repo%3Acommunity%2Fcommunity+backticks+in+titles&type=discussions
https://github.com/notifications/subscriptions

Screenshot

Search - Issue
Before (Search - Issue)

After (Search - Issue)

Search - PR
Before (Search - PR)

After (Search - PR)

Search - Discussions
Before (Search - Discussions)

After (Search - Discussions)

Subscriptions
Before (Subscriptions)

After (Subscriptions)

Thank you! Please also add those URLs in the Test URLs list at the bottom of the file

Added search and subscription test URLs 👍

Fixed backticks for discussion searches, PR should cover all search types now


[Event-Request] Table 36 "Sales Header" - OnBeforeLookupSellToContact

Hello,
We would like to request the addition of the event OnBeforeLookupSellToContact(Rec, xRec, IsHandled) in procedure OnLookup() of field(84; "Sell-to Contact"; Text[100]) in Table 36 "Sales Header":
trigger OnLookup()
var
    Contact: Record Contact;
    ++ IsHandled: Boolean;
begin
    ++ IsHandled := false;
    ++ OnBeforeLookupSellToContact(Rec, xRec, IsHandled);
    ++ if IsHandled then
    ++ exit;

    if "Document Type" <> "Document Type"::Quote then
        if "Sell-to Customer No." = '' then
            exit;

[IntegrationEvent(false, false)]
local procedure OnBeforeLookupSellToContact(var SalesHeader: Record "Sales Header"; xSalesHeader: Record "Sales Header"; var IsHandled: Boolean)
begin
end;

Background: same event as OnBeforeLookupSellToContactNo(Rec, xRec, IsHandled), so including the IsHandled so we can skip the rest of the code in the OnLookup() procedure.
Thanks in advance,
Jerry Tyssen
Cegeka

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 41235.


Python-3 projects depending on matplotlib & plot-directive fail indeterministically

If a python-3 project needs matplotlib's plot-directive extension, fails on install because the native-library freetype2 is not installed:
Notice that freetype is indeed installed, so python2 projects are not affected by this bug
(see ie build: https://readthedocs.org/builds/xray/1889182/ collected by #1021)
The error produced is the following:
Setup Output
...

requirements
-----
...
BUILDING MATPLOTLIB
            matplotlib: yes [1.4.2]
                python: yes [3.4.0 (default, Apr 11 2014, 13:05:11)  [GCC
                        4.8.2]]
              platform: yes [linux]

REQUIRED DEPENDENCIES AND EXTENSIONS
                 numpy: yes [not found. pip may install it below.]
                   six: yes [six was not found.]
              dateutil: yes [dateutil was not found. It is required for date
                        axis support. pip/easy_install may attempt to
                        install it after matplotlib.]
                  pytz: yes [pytz was not found. pip will attempt to install
                        it after matplotlib.]
               tornado: yes [tornado was not found. It is required for the
                        WebAgg backend. pip/easy_install may attempt to
                        install it after matplotlib.]
             pyparsing: yes [pyparsing was not found. It is required for
                        mathtext support. pip/easy_install may attempt to
                        install it after matplotlib.]
                 pycxx: yes [Official versions of PyCXX are not compatible
                        with matplotlib on Python 3.x, since they lack
                        support for the buffer object.  Using local copy]
                libagg: yes [pkg-config information for 'libagg' could not
                        be found. Using local copy.]
              freetype: no  [The C/C++ header for freetype2 (ft2build.h)
                        could not be found.  You may need to install the
                        development package.]
    OPTIONAL LATEX DEPENDENCIES
                dvipng: yes [version 1.14]
           ghostscript: yes [version 9.10]
                 latex: yes [version 3.1415926]
               pdftops: no

============================================================================
                        * The following required packages can not be built:
                        * freetype

@ankostis I'm wading through old tickets and discovered that this was never responded to yet. Sorry for this! We updated the matplotlib on the server and some of its configuration lately so I'm wondering if that issue still exists for you and needs further investigation by readthedocs?

Sorry, v cannot provide confirmation any more.
Have switched to using Python-2 for working with matplotlib in rtd.
Feel free to close it.


Bold Statement by default?

Hi!
I've just tried this colorscheme and have a very small problem. Take the following C code as an example:

Some keywords are bold while others are not, that seems a little bit weird to me. And I believe it was because Statement's are not default to bold.
https://github.com/blazkowolf/gruber-darker.nvim/blob/3d1bf7f9f112e70e8123e1d5b85a8df2ffeca93e/lua/gruber-darker/highlights/vim.lua#L185-L192
I'm not sure if such setting is intentional or not, but i think it would be nice to have all keywords highlighted in a consist way.
Thanks!

Hi @Pokryton! Thanks for your interest in the colorscheme. I'll do some digging to see if this is expected behavior based on the original gruber-darker-theme this colorscheme is derived from.
It may take me a bit to get around to it as I've got work and other projects I'm working on simultaneously, but I will get to it.


Migrating Project to Swift 5

Running the Swift 4.2 and the Swift 5 migration assistant.

^^ bump on this?

Ah, I did the same thing just now at https://github.com/blanksblanks/abandoned-strings to get this thing compiling on XCode 11
@ijoshsmith bump
btw I was linked to this project via https://www.avanderlee.com/xcode/unused-localized-strings/ 🙂


Transition Error: Cannot read property 'getTransitionName' of null

Ionic 2, IOS issue. Reproducible in desktop browser too.
Steps:
First I need to navigate more than 2 levels, then come back by pressing back instantly (regular speed) and you will get empty black screen in desktop browser or white screen in ios app.
Error stack:
nav-controller-base.js:289 Uncaught TypeError: Cannot read property 'getTransitionName' of null
at NavControllerBase._postViewInit (nav-controller-base.js:289)
at NavControllerBase._viewTest (nav-controller-base.js:348)
at NavControllerBase._nextTrns (nav-controller-base.js:192)
at ti.resolve (nav-controller-base.js:142)
at NavControllerBase._trnsFinish (nav-controller-base.js:462)
at t.invoke (polyfills.js:3)
at Object.onInvoke (ng_zone.js:236)
at t.invoke (polyfills.js:3)
at e.run (polyfills.js:3)
at NgZone.run (ng_zone.js:126)

@dheran can you test if the problem persist in nightly?
npm install ionic@nightly --save

@manucorporat I've never seen this stack trace when the black screen occurs in reference to .  I'm able to reproduce this in the browser and on iOS #8885
Testing ionic-angular@nightly now

I ran into this issue as well. However, as @curtisblanchette points out, after upgrading to ionic-angular@nightly, the problem doesn't occur anymore.
I will keep monitoring it...

I will close this issue for the moment. please feel free to reopen it.

I installed ionic-angular@nightly module. But to me it does not work, the problem persists. What could I do wrong?

Alas, the problem persists indeed. I applied a different work-around for now.

I was getting a similar issue too with the same stack trace on rc.1. Seems to be fixed with nightly so far.

Hi Ionites,Did this fix the problem?
I installed ionic-angular@nightly module ,the problem persists in my project .
I want this problem to be definitely fixed!!!

Odd because it works for me using<EMAIL_ADDRESS>but not the normal rc.1 release when I nav in and out of my views. Not sure why.

@TongeJie thanks for the repo! I will check it out this week! Reopening!!

@TongeJie I was able to reproduce the problem with your repo, then noticed it does not use nightly:
    "ionic-angular": "2.0.0-rc.1",

After some testing with the conference app running the latest nightly I also cannot reproduce this issue anymore. Because of this I am going to close this issue for now. Thanks everyone!

Hi, I'm currently getting this error (rc3), I suspects it's some sort of race condition where using Loading.
Runtime Error
Cannot read property 'getTransitionName' of null
Stack
TypeError: Cannot read property 'getTransitionName' of null
at OverlayPortal.NavControllerBase._postViewInit (http://localhost:8100/build/main.js:25387:67)
at OverlayPortal.NavControllerBase._viewTest (http://localhost:8100/build/main.js:25460:18)
at OverlayPortal.NavControllerBase._nextTrns (http://localhost:8100/build/main.js:25283:25)
at ti.resolve (http://localhost:8100/build/main.js:25230:19)
at OverlayPortal.NavControllerBase._trnsFinish (http://localhost:8100/build/main.js:25574:9)
at t.invoke (http://localhost:8100/build/polyfills.js:3:13422)
at Object.onInvoke (http://localhost:8100/build/main.js:32618:37)
at t.invoke (http://localhost:8100/build/polyfills.js:3:13373)
at e.run (http://localhost:8100/build/polyfills.js:3:10809)
at NgZone.run (http://localhost:8100/build/main.js:32508:62)
Ionic Framework: 2.0.0-rc.3
Ionic Native: 2.2.6
Ionic App Scripts: 0.0.46
Angular Core: 2.1.1
Angular Compiler CLI: 2.1.1
Node: 6.9.1
OS Platform: Windows 10
Navigator Platform: Win32
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.99 Safari/537.36


[FIX] Update npmignore

As reported here: https://github.com/salesforce-ux/theo/pull/114#issuecomment-323154368
It seemed that beta3 brakes during installation as we didn't updated .npmignore before release to include the needed files. This should fix it.
I've seen you included the readme.md when publishing, so I've also added the cli.md

CC @aputinski


Add 3rd MPPT string

I have a 15k outdoor with 3 MPPT strings, the total wasn't adding up for me, this commit adds the 3rd string for me.

Thanks for the pull request.  If you have a chance please also add current and voltage support for the third string.


Non-persistent topic subscription metrics

Fixes #13645
Motivation
Non-persistent topic doesn't have subscription metrics
Modifications
Expose a new non-persistent subscription metric: pulsar_subscription_msg_drop_rate
Verifying this change
This change added tests and can be verified as follows:
org.apache.pulsar.broker.stats.PrometheusMetricsTest#testNonPersistentSubMetrics
Documentation
Check the box below or label this PR directly (if you have committer privilege).
Need to update docs?

[ ] doc-required
(If you need help on updating docs, create a doc issue)

[x] no-need-doc
(Please explain why)

[ ] doc
(If this PR contains doc changes)

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

@codelipenghui  PTAL

@Jason918 Can this pr be merged now?

Depends by #15674, so cherry-pick to branch 2.10


Level of Nodes

💥 Proposal
Given a Undirected Graph with V vertices and E edges, Find the level of node X. if X does not exist in the graph then print -1.
Have you read the [Contributing Guidelines on Pull Requests]
Yes.
(https://github.com/TesseractCoding/NeoAlgo/blob/master/CONTRIBUTING.md#reporting-new-issues)?
Kindly assign this issue to me under GSSoC'21.

/assign


Add link to WCAG to Applied Accessibility Index Page

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

This commit adds a link to the W3 WCAG guidelines page. I believe this is a small, but useful enhancement to this page.

Blocked by #38308

That makes sense. In a previous module we linked to Wikipedia so i figured it was fair game.
This is absolutely not a necessary link.
Sent from my iPhone

On May 7, 2020, at 12:42 PM, Tom<EMAIL_ADDRESS>wrote:

@moT01 commented on this pull request.
I think we want to avoid using external links in the challenges as much as possible. So I don't think this should be added. Anyone else can feel free to weigh in.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Yes, I know there's some hyperlinks lingering around out there, but I think that the general hope is to remove them. You can leave this open for now if you want and maybe someone else will share an opinion.

@levlaz We actually are not wanting to add any more external links outside freeCodeCamp.org at this point.  I am going to close this PR now.  Users can search on their own to find out more about WCAG.
We do appreciate your efforts here and look forward to future contributions.
Thank you for understanding.


Fix a couple of minor display issues for the Variation and StructuralVariation 'Genes and regulation' pages

Apologies Laurent, Ridwan has pushed something to master that should not go on release, so this pull request needs to be on postreleasefix/84 again please.


M365 Learning Pathway Full Screen Issue

We noticed that with the default/regular view, when we try to see all the steps/assets in a M365 Learning pathway playlist, in order to jump to specific steps/assets directly, the background of the steps is in black which is good because it's easy to read. Unfortunately, when you expand the view of the playlist so that it's in full screen mode, the background of the steps are no longer black but rather transparent, so it's very difficult to read the steps. Could you please have a look into this:
View when not in full screen mode with black background for steps/assets (print screen shot 1)
View when in full screen mode with transparent background for steps/assets: (print screen shot 2)
Could you please have a look into this as this is affecting multiple users.
I can be reached on:
<EMAIL_ADDRESS>

@chillininmtl this is an issue that was resolved in a fix for #606. Please check your version of Learning Pathways. The latest version with that fix is version 4.3.1


Add details for a couple more prod devices

This PR adds some more details for prod devices:

Bastion ID for 720A9DEAD4391942
Full details for 720A9DEAD4390C1D (replacement for lost/stolen device 720A9DEAD441410E, marked removed in #281)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (main@58a41b0). Learn more about missing BASE report.

Additional details and impacted files
@@          Coverage Diff           @@
##             main    #282   +/-   ##
======================================
  Coverage        ?   2.22%
======================================
  Files           ?      21
  Lines           ?    1575
  Branches        ?       0
======================================
  Hits            ?      35
  Misses          ?    1529
  Partials        ?      11

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Adding mostly-empty docs

This update mirrors the structure from the proposal doc (https://docs.google.com/document/d/1cctIY0GPrvC2k3-BfDU0j7bVrzitua84ATSJWV7247w/edit?usp=sharing) (minus Recipes, still undecided) so we can see it live.

I'm not clear on what the Travis-CI error is indicating - I am assuming that if the Netlify preview looks good then I've done my job. Can someone let me know if there's anything I need to fix on my end?

Fantastic! Do not worry about travis, it is indeed irrelevant for updates to the docs. The failure is spurious.

Thanks!
You can merge this as-is and we can go from there - or if you'd prefer I work at a more granular level I can wipe the pages that already have some draft content.
Offhand this includes the History section, Installing extensions, and all of Starting a project:
https://deploy-preview-2892--openrefine-docs.netlify.app/manual/starting

Looks good to me

@thadguidry does this look good to you?

👍


...would consider to integrate automatic dependency updating

We could integrate something like http://greenkeeper.io/ or https://github.com/peerigon/updtr to keep the gulp-modular dependencies up to date.

I like Greenkeeper because it integrates nicely by providing PRs. But we have to introduce tests and integrate Travis beforehand.


consider Extism to simplify the DX

Hi, this is really cool! I read your note about the alloc and free functions needing implementation by each wasm module and thought perhaps Extism could help simplify this from a developer experience standpoint.
I'm curious if this was something you have already looked into and if so, where it fell short.
The project already has great support to compile plugins from 8 languages and good documentation for users to learn from.
We have an active discord if you'd like to chat! https://extism.org/discord

Nice, had no idea about Extism or Dylibso, seems to be a game changer, will definetly have a look at it.

awesome! please feel free to reach out if you have any questions. would love to hear what you think 🙌


translation don't work

the translation process don't work properly, i fix it

This was fixed in https://github.com/evanmiller/ChicagoBoss/commit/50bc9056a811398a56a6d51e08caf8602fbe4436 and later. Closing the pull request, thanks for the report!


TikTok request: auto rename [~.unknown_video] to [~.mp4]

Checklist

[x] I'm reporting a site feature request
[x] I've verified that I'm running youtube-dl version 2019.12.25
[x] I've searched the bugtracker for similar site feature requests including closed ones

Description

TikTok video files downloaded through youtube-dl always end with the file name [.unknown_video]. However, when downloading a video through the app, it downloads an [.mp4] file.
This is a request for youtube-dl to automatically rename [.unknown_video] files to [.mp4] files for TikTok.

When is that old pull request to fix the extractor getting merged?

It's still happening.

This is still a problem in 2020.06.06, but the issue is closed. Has it been replaced/superseded by another?

I am getting this error on numerous videos while downloading an Odysee playlist. Some get the proper .mp4 extension, others do not. Will run a rename command to correct this. After doing so, is there a way to go back and embed the thumbnails without downloading the video again for all the files that got the .unknown_video extension?


Use Conventional Commits

Hey @merikan, i've noticed this section doesn't have any releases. That is because you don't use Conventional Commits. These commits are automatically detected. CI will automatically resolves the next released version and publishes it.

Hi @denysdovhan I didn't know that, I will make sure to use that in the future. ☺
How should I proceed? I read the documentation and the recommendation was to do a rebase. Do you want me to rebase and force push (branch rules probably forbids) or should add just push an empty feat-commit to trigger a release?

I think we should add a note about conventional commits to our contributing guidelines.
 Let's treat all the previous commits as WIP.


Replace default command sentinel character

! is used by a surprisingly-large number of IRC bots as their command sentinel, meaning that as currently implemented, users of DRC cannot interact with bots of that type.
@ is out too, as is # for obvious reasons both. Can't be anything used for formatting, so *, _, ~ are all out. It should be something pretty obscure, as well. Options:
requires the SHIFT key:

&cmd
^cmd
?cmd
+cmd

these don't require the SHIFT key:

;cmd
,cmd
.cmd
'cmd
=cmd

(I think I prefer the non-SHIFT-required variants...)

Went with ;. Watch out: the previous ! character may have become muscle memory (it had for me)!


Sonoff Mini DIY mode with firmware 3.5, 3.6

Hi!
Sonoff mini in DIY mode with firmware 3.5 and 3.6 do not detected by Home Assistant.
Is there any hope that these devices will be included in the list of tested devices?
Thank You!

Check this:
https://github.com/itead/Sonoff_Devices_DIY_Tools/issues/75#issuecomment-609449006

Thank You!


Comment Action Not Working Because of Redesign on Instagram

The bot can't write any comments correctly, because instead of selecting send on the screen, it selects @, I guess it's because of the new Instagram design.

im on the same tip has there been a fix for this yet?

Use the recommended version of Instagram


feat(aws): allow service and region to be specified in aws options.

PR Checklist:

[x] I have run npm test locally and all tests are passing.
[x] I have added/updated tests for any new behavior.
[x] If this is a significant change, an issue has already been created where the problem / solution was discussed: N/A (small change)

PR Description
This PR picks up where #2188 and #2339 left off to allow aws signing for API Gateway requests which require service and region as part of the signature. aws4 supports adding both service and region so this is a simple change.
Many thanks to @NewGyu who originally wrote this code which I've simply applied to the latest master!

apologies. I did run tests locally, but there appears to be a problem. I'm looking into it.

There we go!

When can we get this merged? we need 'region' to sign AWS API Gateway url with custom DNS

When this will be merged? I need to sign4 request to API Gateway, and the service param is needed to create the signature.

Please merge

I see service for aws4 but no region https://github.com/request/request/blob/e81c9b1dadffe692ac6797fbf7e0bdba557ebfdc/request.js#L1369

we need this feature. Please merge


Update scalameta, semanticdb-scalac to 4.4.31

Updates

org.scalameta:scalameta
org.scalameta:semanticdb-scalac

from 4.4.30 to 4.4.31.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #1372.


Improved documentation

Improved README file
Added instructions and a lot of different resources that are friendly to both beginners and advanced users alike.
Added .github/FUNDING.yml. (Sponsorships will need to be turned on, in order for a Liberapay link/Sponsor button to appear in the repository)

Gah, sorry it took so long to get back to this. Thank you for the PR!

Gah, sorry it took so long to get back to this. Thank you for the PR!

Mind if I work on the README of the actual client as well?

Mind if I work on the README of the actual client as well?

if you want to, sure!


Cheat sheet

WIP cheat sheet that customers can use while they are working with us.
Maybe some day something for https://developers.redhat.com/cheat-sheets/
Can be generally handy but needs additional eyes and contributions
source: https://hackmd.io/6-i34q7uR_2um-yH1trmbA

should we add a link in README.md in root or when users ask for something like this in slack, we point them?


KYAN-289 add new sizes to 'size' dropdown

'Extra small' and '2 extra small' sizes were added to 'common/size' dropdown. Unlike other items, they are not present in Bulma, so new CSS classes were created and placed on the top level of atomic system (as they apply to any component that uses the dropdown). Existing CSS variables provided by t.font-size atom were used.
Motivation and Context
https://teamds.atlassian.net/browse/KYAN-289
Screenshots (if appropriate)

Extra small is:
12px in Kyanite:

0.875rem (14px) in streamx-dev:

2 Extra Small is:
10px in Kuanite:

0.75rem (12px) in streamx-dev:

Upgrade notes (if appropriate)

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue) labeled with bug
[x] New feature (non-breaking change which adds functionality) labeled with enhancement
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] My code follows the code standards of this project.
[ ] My change requires updating the documentation. I have updated the documentation accordingly.

Added screenshots for Kyanite


Investigate flexible in-memory 2's complement

I've decided to go with sign-magnitude, but I'm kind of curious how 2's complement stacks up. Perhaps I'll write a prototype when NBKFlexibleWidthKit is done.

I bet I could do it in a week, because actual two's complement is so much more straightforward than as-if two's complement.

I'll add a sentence or two about flexible-width semantics:

Flexible Width Integers
The transformation is performed with respect to the current bit width. The bit width may change depending on the value's normalization behavior.

Can't tell a lie; I hope in-memory two's complement compares well (or at least good enough).

Another thing I dislike about sign-magnitude-as-if-two's-complement is that the digit type ends up being Int when you'd really want somethings like Signed<UInt>, which is how I implemented it in ANK.

That said, signed two's complement normalization is quite troublesome compared to magnitude normalization.

Storing the sign bit separately makes normalization as simple as the unsigned case, at the cost of bit castability. It's also weird and wasteful in the case of resizable integers (#46). Hm. I wonder if there's a better layout? Perhaps somehing like:

struct ResizableInteger<Digit> {

	var body: Array<Digit.Magnitude>
	var tail: Digit
}
``

I never got around to evaluating its viability, but I made two quick tests/observations:

2x formTwosComplement() on lhs and rhs makes long division ≤ 1.3x slower
2x formTwosComplement() on the product makes long multiplication ≤ 1.2x slower

Dunno. Signed arbitrary precision integers are shades of gray no matter what you do :frowning_face:

Hm. I think I'll use the next couple of days to try it out, because I think it could be neat and I want it to work out. Sign-magnitude-wannabe-two's-complement makes me weep inside.

I haven't gotten to comparing performance yet, but there's something compelling about accessing words with O(1) complexity. Is it as compelling as accessing the magnitude with O(1) complexity? Dunno, but the un/signed symmetry is neat.

Well, if it is viable then viability does not come easily...
// MARK: scrappy 2's vs feature/NBKFlexibleWidth (less is brrr)

add: ( 78,  32) vs (91, 55)
sub: ( 79,  29) vs (92, 49)
mul: (164, 122) vs (88, 58)
div: ( 94,  46) vs (57, 20)

The inability to bit cast un/signed values (due to normalization) is quite sad.

Hm. I could write a generic FlexibleWidth<Digit> model where Digit is either Int or UInt. That would be the most straightforward un/signed two's complement option (in my opinion). I've actively tried to avoid it, however, simply because there's only two interesting specializations of it. It wouldn't improve IntXL's performance, but it also wouldn't require two private helper protocols.

I think I'll use the next couple of days trying out the generic approach. I'm fine with incrementally making IntXL viable, so long as doing so does not increase the project's complexity disproportionately.

My first impression of a generic FlexibleWidth<Digit> is not great, as the models are obviously quite different. One can undeflow, the other can not. They also normalize differently. I'm sure the fixed-width-but-resizable storage could be made generic, but I don't think it suits flexible-width.


test: hypothesis property

What I did

First pass at #351
Uses ghostwriter to create some property based Hypothesis tests.
How I did it

hypothesis write <name of module>
How to verify it

Checklist

[ ] All changes are completed
[ ] New test cases have been added
[ ] Documentation has been updated

I thought #351  was about providing tooling for users to use hypothesis testing in their ape tests?

I thought #351  was about providing tooling for users to use hypothesis testing in their ape tests?

It is, that issue needs more definition

@NotPeopling2day What is the status on this? Is this still something we want?

Think should close this for now


Feature request - bridge slash commands

Which version of matterbridge are you using?
v0.9.1-dev / Mattermost v3.5.1
Please describe the expected behavior.
On Mattermost side we can define some slash commands which can be useful for all protocols. For example if I've a slash commands /wiki it can be interesting to use it on other protocols.
If it is not enought clear let me know i'll detail more

I haven't tested the slash commands yet, isn't the output visible on the other bridges when you execute a slash command ?


Delete Playlists

Can't seem to delete playlists.

Found it! It was reported previously! My fault for not searching first. .....It would be nice to be able to delete from the player menu.


feat: implement ngrok library

Signed-off-by: João Vanzuita<EMAIL_ADDRESS>

lgtm, going to merge this since its only going into my branch


For Adding Flutter Dev resources

I would like to add flutter related  material and resources

Yes, you are most welcomed.
But don't repeat any already exist resources.

Just forked this repo and make changes.
after that make a PR abut your updates.
If you like this repo don't forgot to star it.


How to call the "thing" that uniquely identifies transactions

As we well know the Monero codebase uses a hash over the bytes of a transaction as the "thing" that is used to uniquely identify the transaction.
Variables and parameters that hold this are not consistently named. It's a race between id and variants like e.g. txid on the one side and hash, tx_hash, h or similar on the other side.
I would be glad if we could agree and standardize on one or the other for the Seraphis wallet.
Personally I currently lean towards id. I think a sematic name that tells what something is beats a name that tells in a very technical way how it was calculated and relies on my knowledge that hashes serve to identify transactions.

I prefer 'transaction id' if I have to choose, but I also like 'tx hash' because it reinforces that it is a hash of the entire tx.

"Transaction id" is pretty much settled on in the cryptosphere.  I like the shortened version--txid--as it would be widely understood by neophytes.

I support the use of id. I think what it actually is under the hood is better kept abstracted away from its name.

Seen as decided (we use id in Seraphis wallet coding) after discussion in the first workgroup meeting


GitHub release too big for jsDelivr

I'm trying to load this package from jsDelivr CDN but it fails because the package is over 50 MB. Are there any files that could be removed to reduce the package size?

https://www.jsdelivr.com/package/npm/phaser should work.
<EMAIL_ADDRESS>

The CDN is for including the build files in your games, rather than the entire repo. Even so, the entire Phaser repo is only 19MB.


Call to undefined method GuzzleHttp\Psr7\Request::withParentHeader()

Working fine on localhost, not on live server running PHP 5.5.38

Just update to the latest version of guzzlehttp/guzzle & guzzlehttp/psr7.

Would like to help here.. but I see no feedback provided after the comment of @alexeyshockov
Also no version of guzzle/guzzle guzzle/psr7 or php is provided..
I am closing the issue.. If you are still having issues feel free to provide more info and we can reopen it.


Various image gallery layouts

Now, that's your image gallery:
img {
image 1
image 2
}

They are shown one below another. This is not enough.
From the very start I planned to have a way to specify the gallery's layout here:
img <layout> {
image 1
image 2
}

Some possible layouts:

Content arrangement:

default
grid
horizontal
carousel (why?)
text-to-the-right
tile

Gallery placement:

stretch
right
left

You can combine them like grid right (similar to transclusion selectors).

Left and right considered harmful.
I don't like them. When using ‘right’, I actually want to put the picture on the opposite side, not the right side. On RTL wikis it should be the left side, for example.
Replace ‘right’ with ‘end’? ‘opposite’?
I need your suggestions.

5 days have passed, no suggestions. I am about to cry. Whatever.
What about the grid layout? How many columns is it going to have? I think we need layouts grid2 and grid3 instead.

@bouncepaw I believe you can leave it to flexbox, and maybe give a glance on Bootstrap grid system

Replace ‘right’ with ‘end’? ‘opposite’?

start, end, as in SVG. But arbitrary left and right are still desired IMHO.

@chekoopa Bootstrap 5 uses start and end btw

start and end sound good, I choose them

As of 2065460db6a1586880a80c9414320e7cf3dcee70, the layouts are there. I've added only a few of them, but it is easy to add many more if there is a need.
I'll close this issue once there is documentation for the layout syntax.

Alright, the doc is there.
New issues shall be opened for new layout requests


Fix frequency of rare bee products

This is a proposed fix for #17. Note that I've also changed numbers that were not wrong, to add a second digit to them so that there's no question of what they were supposed to be (e.g., 0.9 that really was supposed to be 90% becomes 0.90 in this patch).

I see no problems here, though the 1.10 branch is not used, and also has no releases. It may be easier and clearer to just delete it. (@mezz ?)

We should remove the 1.10 branch, I'll do that.

Actually it looks like there may be changes there that are not in the current 1.11/1.10 branch.
I'll leave it, I don't want to delete history.
Most PRs should just target the latest version.


Error with last update

2023-03-21 18:37:43 ERROR    Crashing with uncaught exception:
Traceback (most recent call last):
File "/usr/local/lib/python3.10/site-packages/bluetti_mqtt/server_cli.py", line 87, in start
loop.run_forever()
File "/usr/local/lib/python3.10/asyncio/base_events.py", line 600, in run_forever
self._run_once()
File "/usr/local/lib/python3.10/asyncio/base_events.py", line 1896, in _run_once
handle._run()
File "/usr/local/lib/python3.10/asyncio/events.py", line 80, in _run
self._context.run(self._callback, *self._args)
File "/usr/local/lib/python3.10/site-packages/bluetti_mqtt/server_cli.py", line 100, in run
sys.exit('Could not find the given devices to connect to')

And me too...
s6-rc: info: service s6rc-oneshot-runner: starting
s6-rc: info: service s6rc-oneshot-runner successfully started
s6-rc: info: service fix-attrs: starting
s6-rc: info: service fix-attrs successfully started
s6-rc: info: service legacy-cont-init: starting
cont-init: info: running /etc/cont-init.d/00-banner.sh
Add-on: Bluetti to MQTT
MQTT bridge between Bluetti and Home Assistant
Add-on version: 1.4.0
You are running the latest version of this add-on.
System: Debian GNU/Linux 11 (bullseye)  (aarch64 / raspberrypi4-64)
Home Assistant Core: 2023.3.5
Home Assistant Supervisor: 2023.03.1
Please, share the above information when looking for help
or support in, e.g., GitHub, forums or the Discord chat.
cont-init: info: /etc/cont-init.d/00-banner.sh exited 0
cont-init: info: running /etc/cont-init.d/01-log-level.sh
cont-init: info: /etc/cont-init.d/01-log-level.sh exited 0
s6-rc: info: service legacy-cont-init successfully started
s6-rc: info: service legacy-services: starting
s6-rc: info: service legacy-services successfully started
[20:44:45] INFO: Reading configuration settings...
[20:44:47] INFO: Starting bluetti-mqtt...
Scanning....
Found AC200M2242000434992: address DD:7B:53:EB:53:7B
s6-rc: info: service legacy-services: stopping
s6-rc: info: service legacy-services successfully stopped
s6-rc: info: service legacy-cont-init: stopping
s6-rc: info: service legacy-cont-init successfully stopped
s6-rc: info: service fix-attrs: stopping
s6-rc: info: service fix-attrs successfully stopped
s6-rc: info: service s6rc-oneshot-runner: stopping
s6-rc: info: service s6rc-oneshot-runner successfully stopped

Could not find the given devices to connect to

This happens sometimes, I think when the BT connection isn't closed cleanly.  It will eventually reconnect, sometimes after a while. You can reset Bluetooth manually if you have access to the HAOS console with the command systemctl restart bluetooth or just reboot the host.  Working on a solution.
@Dengor1984 Turn scan off in the configuration tab.  Scanning is meant to be toggled on for initial configuration to find the bt mac address, then turned off.  With the latest update, the addon is shutdown after the scan is performed.

With the latest update, the addon is shutdown after the scan is performed.
Thanks for your work and this additional information. I was wondering why it was stopping after upgrading tó 1.4.0.
Now anything is working fine again. :-)

Could not find the given devices to connect to

This happens sometimes, I think when the BT connection isn't closed cleanly. It will eventually reconnect, sometimes after a while. You can reset Bluetooth manually if you have access to the HAOS console with the command systemctl restart bluetooth or just reboot the host. Working on a solution.
@Dengor1984 Turn scan off in the configuration tab. Scanning is meant to be toggled on for initial configuration to find the bt mac address, then turned off. With the latest update, the addon is shutdown after the scan is performed.

Thanks it's solved my issue!


[Icon request] 💨 Tailwind CSS

Tailwind CSS has the VS Code language identifier tailwindcss and it can be manually assigned to CSS files that have this framework's specific syntax.
Here's their SVG logos.
Thank you :)

theres already tailwind css?
https://github.com/catppuccin/vscode-icons/blob/main/icons/tailwind.svg

there is, but the languageId isnt used, i'll add it later :)

should be live :)


WIP: Followups P2P

Description
__
Just ignore this ftm, it contains work related to the P2P follow ups. Since the follow ups milestone has not started yet, and the v1 milestone is not merged to main yet, this will just confuse you.
At the moment of writing this, this PR has added:

[ ] Documentation

[x] Document with diagrams how the socket module works
[ ] Document with diagrams how the network module works

[ ] Wire codec swapping out for protobuff
[ ] Refactor the socket module to implement the Module interface.

To be filled and continued in due time

Closed in favor of #65


Fix routing bug

This PR will not build, as it is dependent on the changes in https://github.com/MemberJunction/MJ/pull/620

@JS-BC  LGTM - update the branch from next and merge it


Request to Modify Vehicle Controls During Record Playback

CARLA version: 0.9.13
Platform/OS: Linux, Ubuntu 20.04
Problem you have experienced: Try to modify the control of Ego during record playing, but not work.
What you expected to happen: Ego vehicle change its control, and stop playing record.
I used Carla's video recording function to record a simulation process. The content of this record is roughly a Ego vehicle controlled by BasicAgent driving on the Town02 map.
I use the client.replay_file to play this record and obtain the behavior (control) of the Ego vehicle at each time point (tick). However, when I tried to modify the Ego vehicle's controls via the following code, this did not take effect. The Ego vehicle still follows the control in the record.

def replay():
    client = carla.Client("localhost", 2000)
    client.set_timeout(Settings.client_timeout)
    client.load_world(Settings.map)
    world = client.get_world()
    settings = world.get_settings()
    settings.synchronous_mode = True
    settings.fixed_delta_seconds = 1 / Settings.fps
    world.apply_settings(settings)

    # set tm to be sync (async default)
    tm = client.get_trafficmanager()
    tm.set_synchronous_mode(True)
    world.tick()
    client.replay_file(
        "recording.recd", 0, 0, 0
    )
    world.tick()

    vehicle = None
    actors = world.get_actors()
    for actor in actors:
        if actor.type_id.startswith("vehicle"):
            vehicle = actor
            break

    if not vehicle:
        print("[ERROR] Failed to find vehicle actor")
        raise ValueError
    while True:
        spectator = world.get_spectator()
        transform = vehicle.get_transform()
        spectator_transform = carla.Transform(
            transform.location + carla.Location(z=30), carla.Rotation(pitch=-90)
        )
        spectator.set_transform(spectator_transform)

        vehicle_control = vehicle.get_control()
        vehicle_control.steer = 0.5 # Just set the direction to turn left
        vehicle.apply_control(vehicle_control)

        world.tick()

So, is there any way to modify the Control of the Ego vehicle during the record playback process? Or switch to manual control mode after a certain point in time?
Related issues: #3624

No, the point of the replay, as it's name implies, is to play something again, exactly as it was recorder in the first place. Any attempt to control a replayed actor is overwritten by the replayer itself


Error message from running create_map.py

We recorded a lane.csv with lane_recorder.py and then tried to convert it into map with create_map.py. Here is error msg from running the python script:
    landmark.waypoint.id = last_central_lane.id.id
AttributeError: 'RepeatedCompositeFieldContainer' object has no attribute 'id'

Here is how we modified the script (in the section of creating default_end_of_waypoint) to make it work:
    poi = POI()
    landmark = poi.landmark.add()
    landmark.name = "default"
    waypoint = landmark.waypoint.add()
    waypoint.id = last_central_lane.id.id
    waypoint.s = last_central_lane.length
    waypoint.pose.x = last_central_lane.central_curve.segment[0].line_segment.point[-1].x
    waypoint.pose.y = last_central_lane.central_curve.segment[0].line_segment.point[-1].y
    print "landmark: {}".format(landmark)

    #landmark.waypoint.id = last_central_lane.id.id
    #landmark.waypoint.s = last_central_lane.length
    #landmark.waypoint.pose.x = last_central_lane.central_curve.segment[0].line_segment.point[-1].x
    #landmark.waypoint.pose.y = last_central_lane.central_curve.segment[0].line_segment.point[-1].y

Closed the issue due to solution is provided in #4106. Please open an new issue if there are additional question! :)


fix: make redirects work as intended

Issue #50 showed that redirects aren't working properly. They go off of the url's path, which introduces a couple of problems. For example, when testing /dist/latest-v20.x, the worker checks if nodejs/dist/latest/v20.x is in the REDIRECT_MAP object.
This is in a pretty rough testing state still, but I wanted to open the conversation on this and the approach we want to take
I got rewrites working but it's not exactly nice (see diff). There are a few takeaways I got from tinkering that I think we should keep in mind:

The redirectPaths.json file has the bucket path as the key, so I think we should convert the url to the bucket path and then check if the bucket path is in the REDIRECT_MAP rather than checking for the url (eliminates possibility of looking for redirects for paths such as nodejs/dist/...)
#39 noticed this but restating since I think it's noteworthy. We really don't have any paths deeper than 3 directories that trigger a rewrite. They're all paths such as /download/release/latest/ or /docs/latest or /api.

The code in the diff is working and something we could go with, but I wanted to get others' opinions on it and any other possible approaches before opening it for review.
Fixes #50

Can we simply make it a hot-fix for #50 and then open the discussion on another issue for potential improvement paths? It's just that we want to unblock the release of this 👀

@MoLow fixed in #53


Add typings for custom elements v1 API

Source: https://www.w3.org/TR/2016/WD-custom-elements-20160830/
As suggested by comments on https://github.com/Polymer/polymer/pull/3954

Hi @rictic, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

👍
Can you resolve the merge conflicts? Thanks

Sure thing, rebased!

Rebased again!

Hm, looks like it was failing due to the use of Promise. Changed to PromiseLike as I see other code here using.

Erp, hadn't updated the baseline. Amended that commit.


UWP apps don't show the icon

ℹ Computer information
Operating System
Windows 10 Pro Insider Preview 64-bit
CPU
Intel Core i7 @ 1.80GHz	51 °C
Kaby Lake-U/Y 14nm Technology
RAM
16.0GB
Motherboard
TIMI TM1905 (U3E1)
Graphics
LQ156M1JW01 (1920x1080@60Hz)
Intel UHD Graphics (Unknown)
2047MB NVIDIA GeForce MX250 (Unknown)	48 °C
SLI Disabled

PowerToys version: Release v0.20.1
PowerToy Utility: PowerToy Runs
Running PowerToys as Admin: yes
Windows build number: [run "winver"] 20190.1000

📝 Provide detailed reproduction steps (if any)

…Open Powertoy Run, using Alt+Space
…Write UWP app name (ie. Spotify, Whatsapp..)
…The UWP apeears in the results but with no icon

✔ Expected result
The UWP apeears in the results but with icon, available and working in previous version of powertoy
❌ Actual result
No icon available to all UWP apps
📷 Screenshots

@bmatcoimbra Can you please share logs files located in %userprofile%/appdata/local/microsoft/powertoys/powertoys run/logs?

Yes,
I attach logs from month ago (where it work) and from yesterday
It only happens to UWP apps,  win32 apps (software) works flawless.
Thanks
2020-07-10.txt
2020-08-17.txt

@bmatcoimbra Can you please check if the following step fix this issue:

Close powertoys
Go to %userprofile%/appdata/local/microsoft/powertoys/PowerToys Run
Delete Cache folder
Restart powertoys.

And please upload the logs as well after trying this.

Problem solve awesome!!
Maybe an option of automatic clear the cache  in every start would be a permanent solution.
Log in attach
2020-08-18.txt
thanks

The issue is because the Package location is changed (maybe due to updates) and since we are caching these programs, the old location persists. For example, I experienced this issue when the location for Maps changed from
C:\\Program Files\\WindowsApps\\Microsoft.WindowsMaps_10.2005.1.0_x64__8wekyb3d8bbwe to C:\\Program Files\\WindowsApps\\Microsoft.WindowsMaps_10.2006.11.0_x64__8wekyb3d8bbwe

@somil55 As discussed at standup.  Lets create a separate issue to propose removing the cache, and measure the performance impact of reindexing the applications on cold start.
@crutkas ^^FYI


Poor Model Performance on HM3D Dataset in Cross-Domain Setting

Upon transferring the model trained on the MP3D (Matterport3D) dataset for evaluation on the HM3D dataset, I have noticed a substantial decline in its performance under the cross-domain setting. However, I am uncertain about the root cause of this issue. Could you please provide any relevant configuration files or offer suggestions to pinpoint the problem?
Here's a comparison of the outcomes:
Evaluation results on the HM3D dataset:
2024-04-03 20:58:52,680 Object categories: 28
2024-04-03 20:58:52,680 step input size: 512
2024-04-03 23:11:15,812 Average episode reward: 2.9890
2024-04-03 23:11:15,812 Average episode distance_to_goal: 5.2411
2024-04-03 23:11:15,812 Average episode success: 0.3100
2024-04-03 23:11:15,812 Average episode spl: 0.1413
2024-04-03 23:11:15,812 Average episode softspl: 0.2381
2024-04-03 23:11:15,812 Average episode collisions.count: 10.4160

In comparison, the model performed notably better within the MP3D dataset:
2024-03-06 06:43:06,888 Object categories: 28
2024-03-06 06:43:06,888 step input size: 512
2024-03-06 15:32:29,139 Average episode reward: 4.8421
2024-03-06 15:32:29,140 Average episode distance_to_goal: 4.0260
2024-03-06 15:32:29,140 Average episode success: 0.4998
2024-03-06 15:32:29,140 Average episode spl: 0.1613
2024-03-06 15:32:29,140 Average episode softspl: 0.2159
2024-03-06 15:32:29,140 Average episode collisions.count: 15.7809

Hi, it seems that you evaluated on 28 categories on HM3D. But the standard testing setup in HM3D only performs evaluation on 6 object goal categories: chair, couch, potted plant, bed, toilet and tv.

Thank you for your reply. I've identified and corrected the issue: the object goal indices differed between the MP3D and HM3D dataset.

Hi @yusirhhh, I noticed that the model.net.obj_categories_embedding.weight is manually set by a fixed size of 28 here with the nn.Embedding shape. How did you adjust the nn.Embedding shape when loading the pre-trained model for evaluation? Did you just load the first 6 rows for HM3D? Could you share how you adjusted the MP3D setting to work with HM3D? Thanks!


Inquiry on Data Collection Step

Is it possible to release the code for data collection? Is each dataset collected using one seed or multiple seeds? Thanks in advance!

I am also wondering if it is possible to release the code for data collection, I've tried running URL Benchmark and generating the code myself, however, the results didn't match with the paper. Thank you!


Consider a cf_include feature or a way to override the default cf_ignore feature

What's the user value of this feature request?
Some developers need files, like the revision directories (.git, etc), which are by default excluded by the cf push command (source). An example is a developer who wants to use an application monitoring tool (like Sentry) which would correlate an issue to a particular .git release.
Who is the functionality for?
Any developer who wishes to correlate application monitoring with revision releases.
How often will this functionality be used by the user?
Unknown. Interested developers might use it on every application push or deployment.
Who else is affected by the change?
No one. Only users who explicitly want to include .git directories, etc.
Is your feature request related to a problem? Please describe.
I just want to include the .git directory with the app when I push the app, so I can track issues to a specific git commit.
Describe the solution you'd like
something like:

provide a .cf_include file to explicitly include files or directories which are normally ignored  Source
or
provide via a command line argument "--include-directories" to explicitly include directories

Describe alternatives you've considered
I've consider other ways of adding files to the app post deployment, with live sync plugins or via the API.
Additional context
https://about.gitlab.com/2019/01/25/sentry-integration-blog-post/ CTRL-F "Find and fix bugs faster with release and commit tracking"
I've never contributed to an OSS project or programmed in Go, but this seems really approachable and solves a personal itch.

Hello @jmsmtn , Here’s a solution you can implement right away with the beta of version 7 of cf: https://github.com/cloudfoundry/cli/releases/tag/v7.0.0-beta.24
In CF CLI 7 we have a feature called labels, which allows you to add snippets of metadata to an app, you could use this to store git information about the current version of the app pushed to cloud foundry.
For example, to store the commit of the code with the app, this would work:
cf push MYAPPNAME ... && \
cf7 set-label app MYAPPNAME commit=$(git rev-parse HEAD)

This will only update to label with the newest commit sha if the push is successful, preserving your current git sha as an app-label named commit.
And then to retrieve this info:
cf7 labels app MYAPPNAME | awk '$1 == "commit" { print $2 }'


bug: inconsistent  dynamic loggerLink

Provide environment information
2024/03/09 16:24:58 ➜  eval-ai git:(main) ✗ npx envinfo --system --binaries --browsers --npmPackages "typescript,next,react,@trpc/server,@trpc/client,@trpc/react,@trpc/react-query,@trpc/client,@trpc/next,@tanstack/react-query"

Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y) y
npm notice Beginning October 4, 2021, all connections to the npm registry - including for package installation - must use TLS 1.2 or higher. You are currently using plaintext http to connect. Please visit the GitHub blog for more information: https://github.blog/2021-08-23-npm-registry-deprecating-tls-1-0-tls-1-1/
npm notice Beginning October 4, 2021, all connections to the npm registry - including for package installation - must use TLS 1.2 or higher. You are currently using plaintext http to connect. Please visit the GitHub blog for more information: https://github.blog/2021-08-23-npm-registry-deprecating-tls-1-0-tls-1-1/

  System:
    OS: macOS 14.3.1
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 21.03 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.19.0 - ~/.nvm/versions/node/v18.19.0/bin/node
    Yarn: 1.22.1 - ~/.nvm/versions/node/v18.19.0/bin/yarn
    npm: 10.2.3 - ~/.nvm/versions/node/v18.19.0/bin/npm
    pnpm: 8.6.11 - /usr/local/bin/pnpm
  Browsers:
    Chrome: 122.0.6261.112
    Edge: 122.0.2365.80
    Safari: 17.3.1
  npmPackages:
    @tanstack/react-query: ^4.36.1 => 4.36.1
    @trpc/client: ^10.45.1 => 10.45.1
    @trpc/next: ^10.45.1 => 10.45.1
    @trpc/react-query: ^10.45.1 => 10.45.1
    @trpc/server: ^10.45.1 => 10.45.1
    next: ^14.0.4 => 14.1.3
    react: 18.2.0 => 18.2.0
    typescript: ^5.3.3 => 5.4.2

Describe the bug
I want to dynamically control whether to log the trpc requests in the console or not, and I managed to do this using a state management library(jotai).
But there is still something wield, that is, my global trpc router (e.g. get all the posts upon t3-app starts) cannot be changed its behavoiur upon first initialization.
Reproduction repo

To reproduce
// trpc/react.tsx

export const api = createTRPCReact<AppRouter>({})

export function TRPCReactProvider(props: { children: React.ReactNode }) {
  const queryClient = getQueryClient()
  const [trpcReactLogEnabled] = useAtom(trpcReactLogEnabledAtom)

  const create = () => {
    console.log({ trpcReactLogEnabled })

    return api.createClient({
      transformer,
      links: [
        loggerLink({
          enabled: (op) =>
            // in client, we cannot directly use env.xxx
            trpcReactLogEnabled &&  // <-- core
            (process.env.NODE_ENV !== "production" ||
              (op.direction === "down" && op.result instanceof Error)),
        }),

        // 这个必须放log下面才有用
        unstable_httpBatchStreamLink({
          url: getUrl(),
        }),
      ],
    })
  }

  const trpcClient = useMemo(create, [trpcReactLogEnabled]) // <-- core
  // const trpcClient = useCallback(create, [trpcReactLogEnabled])() // same as the above
  // const [trpcClient] = useState(create()) // do not use this, it wont change

  return (
    <QueryClientProvider client={queryClient}>
      <api.Provider client={trpcClient} queryClient={queryClient}>
        {props.children}
      </api.Provider>
    </QueryClientProvider>
  )
}

// store.ts

import { atomWithStorage } from "jotai/utils"

export const trpcReactLogEnabledAtom = atomWithStorage(
  "trpc.react.log.enabled",
  true,
)

// layout.tsx

// <-- × it always logs if the default log option is true, or always not
const { data: convs } = api.core.listConv.useQuery()

 // <-- √  it will logs if we turn on the `trpcReactLogEnabledAtom` or not
const updateConv = api.core.updateConv.useMutation()

Additional information
No response

i dont understand what you're bug reporting - how is this related to t3 app?
Sounds like you should file a feature request to trpc to add support for this.


⚠ Bofrost.lu has degraded performance

In d9bcb94, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1027 ms

Resolved: Bofrost.lu performance has improved in 35d5a9c after 6 minutes.


doom-{variable,unicode}-font no longer applied at startup

With the following font configuration in config.el:
(setq doom-font (font-spec :family "Terminus" :size 14 :weight 'normal)
      doom-big-font (font-spec :family "Terminus" :size 28 :weight 'normal)
      doom-unicode-font (font-spec :family "Noto Color Emoji")
      doom-variable-pitch-font (font-spec :family "sans" :size 13))

(and most of the rest of my config disabled just in case), after starting Emacs with --fg-daemon and opening a graphical (pgtk on wayland) frame using emacsclient, Emacs renders the test emoji in init.example.el as a replacement character (box containing "01f642") instead of an emoji. This worked correctly before commit d4dec3565870b96abff0f08f1b638b9d527a7086 (confirmed by rolling back), and still works correctly if I launch a standalone graphical Emacs directly instead of using the daemon. I can also fix it by calling doom/reload-font, which instantly makes the emoji character appear.
Prior to that commit, I needed to set doom-unicode-font (as above) to avoid the same problem. So it looks like doom-unicode-font (and the other fallbacks applied automatically after that commit) aren't working correctly.
I suspect what changed is that before that commit doom-init-extra-fonts-h called set-fontset-font after loading the theme, which happened after opening the first frame, while now the equivalent code runs at startup (before there is a graphical frame). But I haven't fully confirmed that, or figured out why initializing things earlier breaks.
(I only very rarely need this to work, and really only noticed because I checked if I could drop doom-unicode-font from my config after that commit... I'm really only filing this issue in case the fix is obvious to someone more familiar with Emacs fontsets and how they interact with frames, or in case calling doom/reload-font on frame creation is a reasonable fix.)
What did you expect to happen?
Same font setup in a graphical frame on emacs with --fg-daemon and without.
What actually happened?
doom-unicode-font does not work correctly with emacs --fg-daemon, but works without it.
Additional details:
Please let me know if I left anything out above or below.
Steps to reproduce:

Install a basic fixed-width font such as http://terminus-font.sourceforge.net/ that definitely does not support emojis, and a font such as Noto Color Emoji that does
Set doom-font to the former, doom-unicode-font to the latter
Run emacs --fg-daemon
Run emacsclient -n -c to open a graphical frame (I'm running this in a Wayland compositor)
Open https://github.com/hlissner/doom-emacs/blob/develop/init.example.el#L33 and look at the test emoji

System information:
M-x doom/info from inside an Emacs with the font display issue: https://pastebin.com/wc1cDtGG
The Emacs I'm running is from https://copr.fedorainfracloud.org/coprs/deathwish/emacs-pgtk-nativecomp/ (an Emacs 28 snapshot to get pgtk and nativecomp).

Hi! (thanks y'all for the amazing work!) I had this same issue and upgraded doom to latest but that didn't fix it (reloading the font does work, as per above comments).
What seems to have fixed it was disabling the unicode module in init.el. As far as I can tell, fonts are correctly selected now.

My config (`doom/info`)

generated    May 25, 2021 18:22:36
system       MacOS 11.3.1 Darwin 20.4.0 x86_64 mac
emacs        27.2 ~/.emacs.d/ -> ~/.emacs.d/
doom         2.0.9 HEAD -> develop daa50557a 2021-05-25 00:11:58 -0400 ~/.doom.d/ -> ~/.doom.d/
shell        /bin/zsh
features     NOTIFY KQUEUE ACL GNUTLS LIBXML2 ZLIB TOOLKIT_SCROLL_BARS XIM MODULES THREADS JSON PDUMPER GMP
traits       gui server-running envvar-file
modules      :completion company (ivy +fuzzy +prescient +childframe +icons) :ui deft doom doom-dashboard (emoji +unicode) hl-todo indent-guides ligatures modeline ophints (popup +defaults) treemacs vc-gutter vi-tilde-fringe workspaces zen :editor (evil +everywhere) file-templates fold multiple-cursors snippets :emacs dired electric (ibuffer +icons) undo vc :term vterm :checkers syntax :tools (eval +overlay) lookup magit pdf :os macos :lang emacs-lisp ess json javascript markdown (org +roam) sh :config (default +bindings +smartparens)
packages     (emojify) (org-fancy-priorities) (forge) (pandoc-mode) (treemacs-evil) (treemacs-projectile) (treemacs-magit) (treemacs-icons-dired) (treemacs-all-the-icons) (csv-mode) (ivy-bibtex) (org-ref)


Non-commiting users in Responsible dropdown

I'm a sole developer that started using GitBlit + Jenkins. I've got GitBlit setup so that the admin user creates and owns all the repositories and the jenkins user has RW+ to all repositories. This way my id is a standard user id with RWC, otherwise I can't create ticket branches, and can't make changes admin changes to the repo.
When I create a new ticket, I set the responsible user to my id. However the admin and jenkins id's show up in that dropbox. Would it be possible to add a user property that would disable id's from showing up as able to receive tickets?

The Responsible list is based on push permissions.  Your Jenkins and Admin users have push permission to the repository so they are available choices.  I agree that this is noisy but I'm not sure we can change that behavior without causing harm to other installations.

I see your point. There isn't a clean way to add a role to every user in an existing installation without it causing disruption. A one time script that is given the option to run the first time an admin logs in would do the trick but then you have a one-off piece of code that is part of the login sequence.


”Măgura Ilvei”, ”Arșița” are listed in the drop-down menu

Description:
When user writes ”Măgura Ilvei”, ”Arșița” the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Măgura Ilvei"

Expected results
The location appears in the drop-down menu as "Măgura Ilvei, Bistrița-Năsăud (Măgura Ilvei)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Arșița"

Expected results
The location appears in the drop-down menu as "Arșița, Bistrița-Năsăud (Măgura Ilvei)".