[FR] - Need a setter to set FCM host for Functional tests/Service Level tests.

Summary:
I recently integrated FCM SDK to send Push Notifications. The integration was very simple and smooth but while I started writing Functional Tests, I got to know that I can't give FCM host while constructing a FcmMessaging object. The FCM_URL is hardcoded in the FirebaseMessagingClientImpl class, which is restricting me from writing functional tests.
Solution:
While constructing the FcmMessaging object or while initializing FirebaseApp there can be an option to pass the FCM host. If the host is not passed then the default hardcoded host must be used.
Describe alternatives you've considered
Currently, due to this, I'm blocked from writing FTs. I'm currently only dependent on UTs

Hey @yashjain12yj, might be too late for you since it's currently november but I implemented a work-around for this issue using reflection.
/**
   * Override the internal fcm.googleapis.com URL with a gateway URL. This function is needed due to
   * the FirebaseClient interface missing a property to configure the FCM url.
   *
   * @throws Exception ReflectionBased errors, could be encountered once the {@link FirebaseClient}
   *     interface is changed.
   */
  private void overrideFcmUrl() throws Exception { // NOSONAR
    var instance = FirebaseMessaging.getInstance();
    var clientSupplierField = instance.getClass().getDeclaredField("messagingClient"); // NOSONAR
    clientSupplierField.setAccessible(true); // NOSONAR
    var clientSupplier = (Supplier<?>) clientSupplierField.get(instance); // NOSONAR
    var messager = clientSupplier.get(); // NOSONAR
    var urlField = messager.getClass().getDeclaredField("fcmSendUrl"); // NOSONAR
    setFinal(messager, urlField, gatewaySendUrl); // NOSONAR
  }

  /**
   * Helper function to override private final fields in Objects.
   *
   * @param instance Object that contains {@link Field}
   * @param field Field that should be changed
   * @param newValue any value
   * @throws Exception ReflectionBased errors, could be encountered once the {@link FirebaseClient}
   *     interface is changed
   */
  private void setFinal(Object instance, Field field, Object newValue) throws Exception { // NOSONAR
    field.setAccessible(true); // NOSONAR
    Field modifiersField = Field.class.getDeclaredField("modifiers"); // NOSONAR
    modifiersField.setAccessible(true); // NOSONAR
    modifiersField.setInt(field, field.getModifiers() & ~Modifier.FINAL); // NOSONAR
    field.set(instance, newValue); // NOSONAR
  }

This effectively rewrites the fcmSendUrl which can then be used for testing. Hope this is of any help!

+1, we are also experiencing the issue described in this ticket

Any update on this issue? The PR is waiting for review. Thanks!


Fix panic message boxes

Problem: when alvr-server panics, it displays a message box with the panic info. But the rfd::MessageDialog replaces the whole message box content with just "An error occuried" which is not really informative. I played around it a little bit and it seems that it does not accept the panic info as the message box content.
Solution: Shorten the info to display. After that the message box is displayed normally.

Could you do a before and after

Before:

After:

I believe the issue is that rfd does not support new lines in the string. Not actually about the panic info in particular. Try calling alvr_common::show_e() with a new line in it.

@SniperJoe ping. Please do more tests as I suggested, as the code of this PR doesn't really explain why it fixes the issue.

@zarik5 tested with the long error with removed line breaks. Still not showing correctly.

@zarik5 also tried to show a short error with one \n, all good


Convert no-for-in rule to use walk function

PR checklist

[x] Addresses an existing issue: fixes #680

Overview of change:

Updates the template used for generating new rules to use the walk function.
Updates the no-for-in rule to use the walk function.

#680 covers all the rules,  so this is just the first of hopefully many...bandwidth permitting. 😄

Thanks for the review @JoshuaKGoldberg


RFC 74: Remove logger package from standard library

Remove the logger package from the standard library and move to its own library under the ponylang org.
https://github.com/ponylang/rfcs/blob/main/text/0074-remove-stdlib-logger.md

https://github.com/ponylang/logger has been created and the 1.0.0 release is undersay.


fix: App Crash on LoginFragment (double tap)

Fixes #1242
Changes:
Made the default snackbar message non null. Which was the reason
Screenshots for the change:

@haroldadmin I updated it . review it

@theLimitBreaker Can you please update the PR to resolve conflicts? Also, please change the default message to a blank string, and before displaying the Snackbar check if the message is blank using the .isNotBlank() stdlib function.

Yes I was going to update .
Regarding blank string. I thought setting default message to "not-set" would be more readable .
Setting it to blank message this is the need

@haroldadmin , setting "''" in nav-graph doesn't make it empty or blank string , instead it will show
'' as string in SnackBar .

You can leave it as "". It shows a red line when you do this, but the project builds and runs just fine.

That's not a good practice. Are you sure?

@iamareebjamal @liveHarshit Need your opinion here


Removed references to GOPROXY; now using default with Go 1.13.x

Description
Our dependency updater started to get 404s back when trying to run commands such as go get github.com/0xAX/notificator@master.  This seems to be coming from our current GOPROXY of https://gocenter.io.  Using the official proxy of https://proxy.golang.org, the problem goes away.
Now that we've officially moved to Go 1.13.x, we don't even necessarily need to set GOPROXY:

The GOPROXY environment variable may now be set to a comma-separated list of proxy URLs or the special token direct, and its default value is now https://proxy.golang.org,direct. When resolving a package path to its containing module, the go command will try all candidate module paths on each proxy in the list in succession. An unreachable proxy or HTTP status code other than 404 or 410 terminates the search without consulting the remaining proxies.

So, this PR removes any GOPROXY setting/exporting and relies on the Go 1.13.x default instead.
Setup
make go_deps_update to ensure that you don't get the 404 errors we were getting before.
Code Review Verification Steps

[ ] Request review from a member of a different team.

Created https://github.com/jfrog/gocenter/issues/29 to track this and add support of dynamic versions in https://gocenter.io


Only allow grab-and-drag on title of widgets

Right now you can grab almost anywhere in a widget and drag it to another location. This makes it very hard to scroll the widget pane on a mobile device because there's almost no place that's safe to touch.

It would also be nice to be able to copy-and-paste text from inside widgets.


Reword messages in gene phenotype page.

Paul K. is quite keen to make the messages consistent.
Fix ENSEMBL-4535.

https://www.ebi.ac.uk/panda/jira/browse/ENSEMBL-4535


Fix battery not showing for some devices

Adds bat-compatibility boolean checking from configuration file.
Fixes: https://github.com/Alexays/Waybar/issues/2459

Compatibility swaps full battery module recognition to at least show how much power is left.

LGTM, can you add the new option to the man and Github wiki?

Sure

Done

Thanks!


Submarine-556. [WEB] Connect workbench with database for metric

What is this PR for?
Connect workbench with database for metric.
What type of PR is it?
[Feature]
Todos
What is the Jira issue?
https://issues.apache.org/jira/browse/SUBMARINE-556
How should this be tested?
https://travis-ci.org/github/kobe860219/submarine/builds/709419937
Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

@pingsutw  @liuxunorg  Please help me review this pr, thanks a lot.

@kobe860219 You forgot convert the PR title to uppercase. SUBMARINE-556. xxx
Remember next time.


Test poly fails with  {todo,extract_test_case,memoize_function}

Given the function:
poly(X) ->
    case X of
        1 -> X;
        _ -> X
    end.

Inferring the type of this function when stored in a separate file works as expected. But when contained in tycheck_simple.erl, the test fails with:
     Failure/Error: {error,
                        {todo,extract_test_case,memoize_function},
-module(test).
                        [{dnf_var_ty_list,normalize_coclause,5,
                             [{file,
                                  "/Users/swehr/devel/etylizer/src/erlang_types/dnf_var_ty_list.erl"},
                              {line,31}]},
                         {constraint_set,meet,2,
                             [{file,
                                  "/Users/swehr/devel/etylizer/src/erlang_types/constraint_set.erl"},
                              {line,49}]},
                         {ty_rec,normalize_miss,3,
                             [{file,
                                  "/Users/swehr/devel/etylizer/src/erlang_types/ty_rec.erl"},
                              {line,558}]},
                         {ty_rec,normalize,3,
                             [{file,
                                  "/Users/swehr/devel/etylizer/src/erlang_types/ty_rec.erl"},
                              {line,534}]},
                         {constraint_set,meet,2,
                             [{file,
                                  "/Users/swehr/devel/etylizer/src/erlang_types/constraint_set.erl"},
                              {line,46}]},
                         {constraint_set,meet,2,
                             [{file,
                                  "/Users/swehr/devel/etylizer/src/erlang_types/constraint_set.erl"},
                              {line,49}]},
                         {ty_rec,normalize_miss,3,
                             [{file,
                                  "/Users/swehr/devel/etylizer/src/erlang_types/ty_rec.erl"},
                              {line,558}]},
                         {ty_rec,normalize,3,
                             [{file,
                                  "/Users/swehr/devel/etylizer/src/erlang_types/ty_rec.erl"},
                              {line,534}]}]}

@albsch See PR #100

A simpler test case which exhibits the same bug:
-spec foo5(Foo) -> Foo.
foo5({_}) -> ok.

In #116, the TODO was removed and that resulted in unsound tally results. So this bug needs to be fixed in main before #116 can be merged.

The reason is the following:
normalize_coclause(Size, PVar, NVar, Tuple, Fixed, M) ->
  case dnf_ty_tuple:empty() of
    Tuple -> [[]];
    _ ->
      case ty_ref:is_normalized_memoized(Tuple, Fixed, M) of
        true ->
          % TODO test case
          error({todo, extract_test_case, memoize_tuple}); %[[]];
        miss ->
          % memoize only non-variable component t0
          dnf_ty_tuple:normalize(Size, Tuple, PVar, NVar, Fixed, sets:union(M, sets:from_list([Tuple])))
      end
  end.

When we traverse the types during a tally calculation, we need to memoize the whole component including the variables, otherwise we calculate unsound results.

Works with #119 .


No module named 'magic'

I've just started to use fvid again, but:
E:\>fvid -i Documents.rar -e -o Documents.mp4
Traceback (most recent call last):
  File "c:\users\tran bich dung\appdata\local\programs\python\python39\lib\runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "c:\users\tran bich dung\appdata\local\programs\python\python39\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\Tran Bich Dung\AppData\Local\Programs\Python\Python39\Scripts\fvid.exe\__main__.py", line 4, in <module>
  File "c:\users\tran bich dung\appdata\local\programs\python\python39\lib\site-packages\fvid\fvid.py", line 16, in <module>
    import magic
ModuleNotFoundError: No module named 'magic'

I don't know what happened. I tried to install 'magic':
E:\>py -m pip install magic
ERROR: Could not find a version that satisfies the requirement magic
ERROR: No matching distribution found for magic

And upgrade fvid to the latest master branch:
E:\>py -m pip install --upgrade git+https://github.com/AlfredoSequeida/fvid
Collecting git+https://github.com/AlfredoSequeida/fvid
  Cloning https://github.com/AlfredoSequeida/fvid to c:\users\tran bich dung\appdata\local\temp\pip-req-build-zyhpx15q
  Running command git clone -q https://github.com/AlfredoSequeida/fvid 'C:\Users\Tran Bich Dung\AppData\Local\Temp\pip-req-build-zyhpx15q'
Requirement already satisfied: bitstring>=3.1.6 in c:\users\tran bich dung\appdata\local\programs\python\python39\lib\site-packages (from fvid==1.0.0) (3.1.7)
Requirement already satisfied: pillow>=7.2.0 in c:\users\tran bich dung\appdata\local\programs\python\python39\lib\site-packages (from fvid==1.0.0) (8.0.1)
Requirement already satisfied: tqdm>=4.49.0 in c:\users\tran bich dung\appdata\local\programs\python\python39\lib\site-packages (from fvid==1.0.0) (4.54.1)
Requirement already satisfied: cryptography>=3.1.1 in c:\users\tran bich dung\appdata\local\programs\python\python39\lib\site-packages (from fvid==1.0.0) (3.2.1)
Requirement already satisfied: pycryptodome>=3.9.8 in c:\users\tran bich dung\appdata\local\programs\python\python39\lib\site-packages (from fvid==1.0.0) (3.9.9)
Requirement already satisfied: six>=1.4.1 in c:\users\tran bich dung\appdata\local\programs\python\python39\lib\site-packages (from cryptography>=3.1.1->fvid==1.0.0) (1.15.0)
Requirement already satisfied: cffi!=1.11.3,>=1.8 in c:\users\tran bich dung\appdata\local\programs\python\python39\lib\site-packages (from cryptography>=3.1.1->fvid==1.0.0) (1.14.4)
Requirement already satisfied: pycparser in c:\users\tran bich dung\appdata\local\programs\python\python39\lib\site-packages (from cffi!=1.11.3,>=1.8->cryptography>=3.1.1->fvid==1.0.0) (2.20)
Building wheels for collected packages: fvid
  Building wheel for fvid (setup.py) ... done
  Created wheel for fvid: filename=fvid-1.0.0-py3-none-any.whl size=9930 sha256=23a8357e046bb26ea95a09bc2dd433ab5a1e4872a064e09f19ed726fba392229
  Stored in directory: C:\Users\Tran Bich Dung\AppData\Local\Temp\pip-ephem-wheel-cache-xf5q5eny\wheels\7d\29\2f\3abea42ac756b48077987d878a53a93f6ec3b1cee1c8560187
Successfully built fvid
Installing collected packages: fvid
  Attempting uninstall: fvid
    Found existing installation: fvid 1.0.1
    Uninstalling fvid-1.0.1:
      Successfully uninstalled fvid-1.0.1
Successfully installed fvid-1.0.0

But it still doesn't work...
Any help appreciated. Thanks!

Anyway, installing python-magic and zfec requires Microsoft Visual C++ 14.0, fvid should've included those out-of-the-box.
Are you sure you installed MIcrosoft Visual C++ 14.0, because I installed it on my backup windows computer with Microsoft Visual C++ 14 and it worked fine. Also, how would fvid include those out-of-the-box? That's an appealing option, but I don't know how to do that.

I got it going. I downloaded a precompiled zfec and installed it. Apparently, on Windows, you have to install python-magic-bin to fetch libmagic's DLL.


wsl commands result in lines with carriage returns

From PowerShell, running
wsl cat file1 > file2
wsl echo text3 > file2

results in a file2 with carriage returns. If I am inside WSL-Ubuntu and run the commands (without wsl), the resulting file does not have carriage returns, which is the desired behavior.

This is because of how powershell handles piping from "native commands". When you use wsl cat ... > file2, cat is running inside wsl, but > file2 is being handled by powershell.
File an issue over at the powershell repository!


[Feature Request] support multi scraping config

#11

🎉 This issue has been resolved in version 1.23.0 🎉
The release is available on:

v1.23.0
GitHub release

Your semantic-release bot 📦🚀

看你更新了，我更新了 docker容器，下载 完成执行自动任务有这个报错！
ikaros日志：


Update node modules

Update eslint:

Turned off the no-prototype-builtins rule which was erroring our use of foo.hasOwnProperty
Removed a bunch of unnecessary gloabal defines

Updated rimraf (no changes required)
Updated gulp-tap (no changes required)

Thanks for the pull request @hpinkos!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

Could someone review this? @tfili @mramato?

I'm surprised no-prototype-builtins is on by default.  Weird.

Thanks @hpinkos!  We'll need to publish a new version of eslint-config-cesium at some point, but that can happen separately.


cats-parse port of URI

@rossabaker is your current intent to start fresh over in https://github.com/typelevel/cats-uri ?

More or less.  I'd like to take a closer look at scala-uri before investing too much into a cats-uri, but I think there's consensus that there's value to a URI outside http4s, and also a consensus that our 0.21 version stinks.


Example image could be clearer

The last example, under the heading Changing the Bend of a Cardinal Spline, shows how changing the tension parameter adjusts the curve.  However, each tension parameter is displayed on a single image and in a single color - and so it is very difficult to see

how the parameters affect the shape
which shape belongs to which parameter

At a minimum please change the image to use a different color for each spline.  Perhaps change the color of each label to match the line color it belongs to as well?
Thank you.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: a0d031d9-c5f1-cf5e-9b33-2259d8c7cc81
Version Independent ID: c2b984bd-0baf-88cf-83fd-2f3000be8942
Content: How to: Draw Cardinal Splines - Windows Forms .NET Framework
Content Source: dotnet-desktop-guide/framework/winforms/advanced/how-to-draw-cardinal-splines.md
Product: dotnet-framework
Technology: dotnet-winforms
GitHub Login: @adegeo
Microsoft Alias: adegeo

Thank you for this feedback. I'll add it to our backlog, however, I don't think this is a high priority at the moment. Cheers!


Run configurations' behavior is weird

I tried to log-in as a (new) guest.

Open Run Configuations dialog

"Run" and "Close" buttons are partially overlapped by main panel.
Plus(+) button has unintended active border.

Close dialog without applying it

"undefined" configuration is displayed in run configurations.

Re-open Run Configuations dialog

Run Configurations UI is totally broken.

@minsung-jin
Please check this issue.
Thanks.

Hmm. Above 2 and 3 issues are resovled also? (or not reproduced?)

It's being reproduced and tested.

Not currently reproduce. When reproduce'll check again.


HH-150/HH-151: placeholder image

Description

Add placeholder (background fallback) image to Cards, Hero and Page
Customizable placeholser image(s) can be added to RHHC config : (see another pr https://github.com/City-of-Helsinki/react-helsinki-headless-cms/pull/43)

Issues
https://helsinkisolutionoffice.atlassian.net/browse/HH-150
https://helsinkisolutionoffice.atlassian.net/browse/HH-151
Closes
DEV-XXX:
Related
Testing
Automated tests
Manual testing
Screenshots
Additional notes

LGTM, but I haven't seen the changes of RCHC yet.
There is 1 console.log that should be removed

RHHC pr is here if you would like to check it: https://github.com/City-of-Helsinki/react-helsinki-headless-cms/pull/43


Implement fetch stream retry

Description
Add a retry layer on top of the failover layer.
How was this PR tested?
Added a bunch of unit tests

totally agree


fix(Menu): theme bugs

Warnings

:warning:
There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

  Generated by :no_entry_sign: dangerJS

Menu dropdown indicator position is broken.

Items with icons are too tall.

Fails

:no_entry_sign:
All of your entries in CHANGELOG.md should be in the **Unreleased** section!

  Generated by :no_entry_sign: dangerJS


[new] Add apvlv pdf reader

Build locally ok

installed in nix store and runs ok

Default pdf when none is specified in cli:

When opening one:

Hello,
Missing the magic words, please.
And thanks for your time.
Cheers,

Why is the 0.1.5 release from https://github.com/naihe2010/apvlv/releases not being used? As I understand you went with the release listed on http://naihe2010.github.io/apvlv/ ; it's probably better to use the GitHub release as it seems more recent and such sites are often out of date.
Aside from that it looks good to me. Not sure if the chmod is needed but I haven't checked.

Hello,

Why is the 0.1.5 release from https://github.com/naihe2010/apvlv/releases not being used? As I understand you went with the release listed on http://naihe2010.github.io/apvlv/ ; it's probably better to use the GitHub release as it seems more recent and such sites are often out of date.

I tried with the latest release from github first.
It compiles (with gtk3 and some other dependencies).
But it does not run correctly (freeze, or the window to open the file to open does simply not appear).

Aside from that it looks good to me.

Cool.

Not sure if the chmod is needed but I haven't checked.

I will double check.
Cheers,

Not sure if the chmod is needed but I haven't checked.

I will double check.

It is not. Removed.
Thanks


Missing option to determine visibility of TypeNode

Currently, there is now direct way to test the visibility of a TypeNode in a macro, except for converting it to a Path by passing it to other macro methods (which seems rather clunky).
A quick way to determine the visibility of a TypeNode would be helpful for the cases in which a TypeNode is directly returned (e.g. TypeNode#ancestors).
Pull request #11696 implements the TypeNode#private? method, but another approach could be to instead use a visibility method, which returns a SymbolLiteral.
However, a class can only be private or public, so a SymbolLiteral is not necessary, except maybe for consistency with other visibility methods (which on the other hand can also return a :protected).
Changing the PR to a SymbolLiteral would be unproblematic, though.

I think it would make more sense to copy #visibility from Def and Macro. We don't need another way to express the same thing.

Ruby went with separate methods instead of visibility as well https://rubyreferences.github.io/rubychanges/3.1.html#methodunboundmethod-public-private-protected

So sounds like separate methods is the way to go? @Hadeweka, able to pick this back up?

Note that this change was reverted in Ruby 3.2 (perhaps only temporarily): https://rubyreferences.github.io/rubychanges/3.2.html#methodpublic-protected-and-private-are-removed

So sounds like separate methods is the way to go? @Hadeweka, able to pick this back up?

I could change the pull request to also add TypeNode#public?, but as protected types don't seem to exist, it would only return the oppositve value of private? (and therefore not technically be necessary).

I don't think the addition of predicate methods (#public?, #protected, #private?) necessarily means deprecation of #visibility. They serve different purposes and there are use cases for both.

Ah right, one that I can think of it reconstructing the node, which would be a lot easier with #visibility. In that case, we should probably add both the predicate methods and #visibility to TypeNode then yea?

So is the consensus to add both #visibility and #public?/#private? to TypeNode?
If so, I can update my pull request.

@Hadeweka I don't suppose this is something still on your radar?

Whoops, kind of forgot about this over time, sorry.
I actually managed to circumvent the problem for my projects, but if it's still relevant for someone, I could finally update my pull request 😃

That would be great yes if you have the time :)

I rewrote the pull request (while accidentally force-pushing to the newest Crystal commit to the branch, sorry for that):
https://github.com/crystal-lang/crystal/pull/11696


Docker container tags

Every container running tests for a student submission will be tagged with the student's github username.

Looks like there is a test you missed in this update:
# github.com/autograde/aguis/web_test [github.com/autograde/aguis/web.test]
394web/hooks_test.go:47:43: cannot use runner (type *mockRunner) as type ci.Runner in argument to web.GithubHook:
395	*mockRunner does not implement ci.Runner (wrong type for Run method)
396		have Run("context".Context, *ci.Job) (string, error)
397		want Run("context".Context, *ci.Job, string) (string, error)

Coverage decreased (-0.08%) to 46.156% when pulling 9a5ab5c3487fec40fb446f70706b313d41c00599 on container_tags into a681f8f36939fb632ee41f3648ffa36b4569eb00 on master.


fix(react-bindings): multiple client async start handling while loading

One other thought I had was maybe we should move the loading being set into a useEffect. It might be consistent with react principles and safer for future changes.
Let me know if you agree and I can change it accordingly.
Didn't want to make a larger change without discussing first.

Hey @nadavshatz, thanks for the PR. I don't quite follow what it is solving however.
initializeAsync is making a network request and then setting isLoading to false once it is resolved.
My understanding of the changes in this PR is that if a second call to useStatsigInternalClientFactoryAsync is made, it will just immediately resolve and set isLoading to false, regardless if the network request is still in flight.

For some reason on a second call or rerender (not sure the reason I just see a debugger being called again inside the use memo) then it just returns since the ref exists but the loading isn't being set since the code isn't reached for that instance. Maybe each call creates another instance of the is loading state.

After some more research it is indeed considered bad practice to modify state while in a useMemo.
Working with ChatGPT even provides this useEffect version which is much much safer.
I updated my PR to it and added  to the test which actually fails with the old implementation.

lastly - we can use this if we don't want the strict mode wrapper.
https://testing-library.com/docs/react-testing-library/api/#reactstrictmode
I just didn't want to change a global setting.


The just api/init command fails when executed repeatedly in succession

Description

The load_sample_data.sh script used to prepare the development environment loading the sample data is not completely idempotent. It often fails to create the ES indices if they already exist when it's expected to succeed every time it is run.
Reproduction

If desired, try starting from a fresh environment running the recreate command

just api/recreate

Then run the problematic command several times

just api/init

See it constantly fails in one of the ES index creation steps

just curl-post '{"model": "audio", "action": "CREATE_AND_POPULATE_FILTERED_INDEX", "destination_index_suffix": "init"}'
STATUS_CODE=$(curl -X POST -H 'Content-Type: application/json' -d '{"model": "audio", "action": "CREATE_AND_POPULATE_FILTERED_INDEX", "destination_index_suffix": "init"}' -o /dev/stderr -w "%{http_code}" 'http://localhost:50281/task'); if [ $STATUS_CODE -lt 200 ] || [ $STATUS_CODE -ge 300 ]; then echo "Status: $STATUS_CODE"; exit 1; fi
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   195  100    93  100   102    841    922 --:--:-- --:--:-- --:--:--  1772
{"message": "Failed to schedule task due to an internal server error. Check scheduler logs."}Status: 500
error: Recipe `curl-post` failed on line 76 with exit code 1
error: Recipe `create-and-populate-filtered-index` failed on line 94 with exit code 1
error: Recipe `init` failed on line 53 with exit code 1

Can you also post the logs from the ingestion_server and indexer_worker services when this appears? Those will spell out the actual exception and where in the code it was raised.


Conflicts usage: Java Platform Support warn unused method from @RestController class in Spring project

I am currenly working on a Spring boot project on vscode. Normally, the Spring Boot Tools extensions will recognize properly and not warning any method in @RestController or repositories classes.
However, this morning I tried the extension and got the warning which is from the Java PlatForm Support extension in vscode. After a while of seeking the cause of the warn, I turned off the extension and the warn will disappear

Enabled the extension

Disabled the extension

I think it would be great if it does not conflict if I use this extension with the spring boot project.

Hey @KhoaLee, are you still facing this issue?

Closing this issue due to inactivity, please feel free to reopen this issue if you think this problem still persist.


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 1408 ms

Resolved: Bofrost.lu performance has improved in 9699987 after 13 minutes.


ci: upload jaq executable to GitHub releases

Add a new workflow to upload jaq executable files to GitHub releases.
I have tested on my fork.

Do you know how I can test this workflow without creating a release?

Hi @eitsupi, thanks again for this PR. Given that this is a quite security-sensitive change, I decided to fully reimplement this functionality in order to understand how to use this safely. This is done in #107. I will prefer my PR to yours and hope that you understand. Still thank you for the inspiration. :)


tests/net/socket/getaddrinfo: assert logic is backwards

Describe the bug
Assert logic in https://github.com/zephyrproject-rtos/zephyr/blob/master/tests/net/socket/getaddrinfo/src/main.c#L23 is backwards.  You can run this test without access to a local dnsmasq server and it passes.
To Reproduce

Make sure QEMU net-tools are started, but no local dnsmasq server is available.
cd tests/net/socket/getaddrinfo
mkdir build && cd build
cmake -DBOARD=qemu_x86 ..
make run
test passes(?!?)

Expected behavior
Test should fail if no local dnsmasq server is running
Impact
Test is not working
Screenshots or console output
SeaBIOS (version rel-1.12.0-0-ga698c8995f-prebuilt.qemu.org)
Booting from ROM..***** Booting Zephyr OS v1.14.0-rc3-297-g6fcf986e7775 *****
Running test suite socket_getaddrinfo
===================================================================
starting test - test_getaddrinfo_ok
PASS - test_getaddrinfo_ok
===================================================================
starting test - test_getaddrinfo_no_host
PASS - test_getaddrinfo_no_host
===================================================================
Test suite socket_getaddrinfo succeeded
===================================================================
PROJECT EXECUTION SUCCESSFUL

Environment (please complete the following information):

OS: Arch Linux
Toolchain: Zephyr SDK 0.10.0
SHA: https://github.com/zephyrproject-rtos/zephyr/commit/0734b4fcaaea62e5f887d932a79b31507d61429d

I have a feeling more of the socket tests have this same issue.

Please note that all the programs under tests/net are suppose to be self contained, they are not suppose to connect to outside world. The programs in samples/net/ are different in this respect as those ones will need outside connectivity to function properly.
That said, I tried to run this test for native_posix and it fails with this message
ASSERTION FAIL [net_if_get_link_addr(iface)->addr != ((void *)0)] @ zephyr/subsys/net/ip/net_if.c:2870
I will investigate what is going on here as the test is suppose to be run in native_posix according to its testcase.yaml file. I will also look what the test is actually doing or trying to do.

@jukkar can you try and run this test for qemu_x86?  Without any setup of local dnsmasq it will PASS.  Which is not correct.  Take a look here at the assert checks:
https://github.com/zephyrproject-rtos/zephyr/blob/master/tests/net/socket/getaddrinfo/src/main.c#L23

BTW, if you look at the asserts, there is clearly a bug there.  Removing the bug tag doesn't change this.  Are we to ship LTS with incorrect tests?

Ugh.  Re-reading this.  I guess this test is written as a failure?  Why does this even exist?  Samples would do compile testing for sanity.
This test is a waste of CPU cycles.
I'll close, but this is silly.

To clarify why this test is silly:
The whole reason I started investigating this, is that an actual DNS query sent via the Socket-based APIs which is never returned by the server will hang the Zephyr device.  We have an edge gateway setup where containers are loaded which handle NAT64, DNS64, joining BLE 6lowpan devices as well as OpenThread.  This issue was identified when the DNS64 container spins up a bit later than the BLE joiner.   The perfect example of this would be when the gateway software is updated and it requires a restart, but the IoT nodes are up and running at the time.
The K_FOREVER semaphore taken here and a bit lower are the cause of the hang: https://github.com/zephyrproject-rtos/zephyr/blob/master/subsys/net/lib/sockets/getaddrinfo.c#L119
This test has no real network setup, so DNS subsys lib returns EAI_CANCELLED when the query cannot be sent.  This test checks exact that: that a query cannot be sent and nothing else.

Lets not close this as there is an issue. As I said, I will investigate how to fix it.

There are too may assignees so setting only one. @mike-scott, please set only one assignee at a time as otherwise it is not clear who will start to fix the issue.

@jukka Apologies.  I'm too used to PRs where that spot is for reviews.  Thank you for taking a look here and at the new bug I opened.

This test has no real network setup, so DNS subsys lib returns EAI_CANCELLED when the query cannot be sent. This test checks exact that: that a query cannot be sent and nothing else.

Well, it tests at least something, specifically what it could (easily) test. And for what it tested, the logic is correct, so updating the ticket title.


/usr/bin/dock cannot allocate memory

I am getting the following error when trying to launch a unit on the coreos cluster:
Apr 20 13:33:14 ip-10-21-1-80.ec2.internal bash[14077]: 511136ea3c5a: Download complete
Apr 20 13:33:14 ip-10-21-1-80.ec2.internal bash[14077]: b3553b91f79f: Pulling metadata
Apr 20 13:33:14 ip-10-21-1-80.ec2.internal bash[14077]: b3553b91f79f: Pulling fs layer
Apr 20 13:33:14 ip-10-21-1-80.ec2.internal bash[14077]: b3553b91f79f: Error downloading dependent layers
Apr 20 13:33:14 ip-10-21-1-80.ec2.internal bash[14077]: c9e98f90d17d: Error pulling image (latest) from abrkn/ebs-sidekick, endpoint: https://registry-1.docker.io/v1/, ApplyLayer fork/exec /usr/bin/docker: cannot allocate memory
Apr 20 13:33:14 ip-10-21-1-80.ec2.internal bash[14077]: c9e98f90d17d: Error pulling image (latest) from abrkn/ebs-sidekick, ApplyLayer fork/exec /usr/bin/docker: cannot allocate memory
Apr 20 13:33:14 ip-10-21-1-80.ec2.internal bash[14077]: time="2015-04-20T13:33:14Z" level="fatal" msg="Error pulling image (latest) from abrkn/ebs-sidekick, ApplyLayer fork/exec /usr/bin/docker: cannot allocate memory "

What might be the cause of this error? top doesn't seem to bring up any memory issues and df / shows only 2% disk usage.

cannot allocate memory

@olalonde that means that there's no enough free memory to create a new container in that node

RAM memory?

Yes

@olalonde I'm going to close this, as the error indicated points to the host's memory as the issue. If there's anything we can help with, let us know!


[Annoucement] Moving from Microsoft.Extensions.WebEncoders.Core to System.Text.Encodings.Web

As of RC2 the types from Microsoft.Extensions.WebEncoders.Core have been moved to System.Text.Encodings.Web. In the process there were several API changes.
IHtmlEncoder -> HtmlEncoder
.HtmlEncode -> .Encode
IJavaScriptStringEncoder -> JavaScriptEncoder
.JavaScriptStringEncode -> .Encode
IUrlEncoder -> UrlEncoder
.UrlEncode -> .Encode
The parameter ordering for some of the Encode methods has also changed.

We are closing this issue because no further action is planned for this issue. If you still have any issues or questions, please log a new issue with any additional details that you have.


will correct Up the UI for the GOA card

there is imperfection in the goa beach card

there is repetitive location and read More .
will correct it up .Pls  do assign and label it with hacktoberfest.

@harshvardhansb pls label it with hactoberferst.

@harshvardhansb label it with hactoberfest


Version 1.5.2 is incompatible with mongoid version <5.0.0

The latest version DatabaseCleaner breaks when not using Mongoid 5:
https://github.com/DatabaseCleaner/database_cleaner/commit/ca58d24232f05835303f111d204a49a6845ff950
Method exist only in 5.x:
https://github.com/mongodb/mongoid/blob/bb206b86bc3315cdb38d3597393205f7d1c0bb31/lib/mongoid.rb#L63-L66
We're using Mongoid 4.0.2 and with DatabaseCleaner version 1.5.2 is breaks with:
undefined method default_client' for Mongoid:Module (NoMethodError) /home/tb/.rvm/gems/ruby-2.2.1@rails41/gems/database_cleaner-1.5.2/lib/database_cleaner/mongoid/truncation.rb:31:in session'
/home/tb/.rvm/gems/ruby-2.2.1@rails41/gems/database_cleaner-1.5.2/lib/database_cleaner/moped/base.rb:29:in db_version' /home/tb/.rvm/gems/ruby-2.2.1@rails41/gems/database_cleaner-1.5.2/lib/database_cleaner/moped/truncation_base.rb:26:in collections'
/home/tb/.rvm/gems/ruby-2.2.1@rails41/gems/database_cleaner-1.5.2/lib/database_cleaner/moped/truncation_base.rb:14:in clean' /home/tb/.rvm/gems/ruby-2.2.1@rails41/gems/database_cleaner-1.5.2/lib/database_cleaner/base.rb:46:in clean_with'
/home/tb/workspace/web/site_testing/features/support/env_overrides.rb:133:in `AfterConfiguration'
It would be better e.g. to detect existence of "default_client" and "default_session" or just the Mongoid version...
Best regards,
Thomas Balsløv.

@P9GIT Hi, thanks for the bug report.
Could you try this patch locally?
def session
  Mongoid::VERSION > "5.0.0" ? ::Mongoid.default_client : ::Mongoid.default_session
end

You'd need to modify this file: https://github.com/DatabaseCleaner/database_cleaner/blob/master/lib/database_cleaner/mongoid/truncation.rb#L30-L32
Please let me know.

Hi!
Thanks for the patch :)
It still breaks however, but this works:
def session   ::Mongoid::VERSION > "5.0.0" ? ::Mongoid.default_client : ::Mongoid.default_session end

@P9GIT Could you submit a PR with this patch? I'd like to release a new version as soon as possible.

Sure, uh, but I don't know how. Never worked with github actually, just making issues ;)
How do I do it?

@P9GIT Please follow the steps described here: https://github.com/DatabaseCleaner/database_cleaner/blob/master/CONTRIBUTE.markdown
If something is not clear with the steps please let me know and I can help.

OK, thanks. Seems relatively straight forward :)

Hmm, I think that was it!


Add profile

Heyy !! it's my first open-source contribution thanks to you ✌.

@Monilprajapati Thank you so much for your contribution! Your efforts are greatly appreciated and will go a long way in improving our project. Please feel free to share it with others and help us grow even further!
Welcome to the world of Open Source.


[Question] After an update to TBMQ 2.0 I got messed up 2 new entities

Hi @dmytro-landiak
After an update to TBMQ 2.0, I can see 2 new fields:
subscription.subscriptions
unauthorized-client.unauthorized-clients
Here are some screenshots

I believe the entity names shouldn't be like this, and also there should not be an error at "unauthorized-client.unauthorized-clients" section
So I think I probably did something wrong at the ./tbmq-install-and-run.sh
Can u pls advice on what could be wrong?
Also highlighted is the piece of code that I changed according to the documentation

Here is the redis section at docker.compose.yml

As always, thank you in advance!

Hey @dmytro-landiak , I did not miss it running first time, but there could be an error executing it indeed.
However I did it again, here some logs from console:
wget -O tbmq-upgrade.sh https://raw.githubusercontent.com/thingsboard/tbmq/release-2.0.0/msa/tbmq/configs/tbmq-upgrade.sh &&
sudo chmod +x tbmq-upgrade.sh && ./tbmq-upgrade.sh
--2024-12-03 15:25:31--  https://raw.githubusercontent.com/thingsboard/tbmq/release-2.0.0/msa/tbmq/configs/tbmq-upgrade.sh
Resolving raw.githubusercontent.com (raw.githubusercontent.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 4097 (4.0K) [text/plain]
Saving to: ‘tbmq-upgrade.sh’
tbmq-upgrade.sh                                                              100%[=============================================================================================================================================================================================>]   4.00K  --.-KB/s    in 0s
2024-12-03 15:25:31 (53.4 MB/s) - ‘tbmq-upgrade.sh’ saved [4097/4097]
[sudo] password for prdmzgateway:
Docker Compose version is: V2
2.0.0: Pulling from thingsboard/tbmq
Digest: sha256:afc1cec150c60cade1367ab6e684671a7f9f766bc66cca183f8a36c6e92bdaa2
Status: Image is up to date for thingsboard/tbmq:2.0.0
docker.io/thingsboard/tbmq:2.0.0
Docker Compose file backup created: docker-compose.yml.bak
Trying to replace the TBMQ image version from [1.4.0] to [2.0.0]...
TBMQ image line updated in docker-compose.yml with the new version
WARN[0000] /home/prdmzgateway/docker-compose.yml: version is obsolete
[+] Stopping 1/0
✔ Container prdmzgateway-tbmq-1  Stopped                                                                                0.0s
WARN[0000] /home/prdmzgateway/docker-compose.yml: version is obsolete
"docker inspect" requires at least 1 argument.
See 'docker inspect --help'.
Usage:  docker inspect [OPTIONS] NAME|ID [NAME|ID...]
Return low-level information on Docker objects
docker: no name set for network.
See 'docker run --help'.
WARN[0000] /home/prdmzgateway/docker-compose.yml: version is obsolete
? Going to remove prdmzgateway-tbmq-1 Yes
[+] Removing 1/1
✔ Container prdmzgateway-tbmq-1  Removed                                                                                0.2s
WARN[0000] /home/prdmzgateway/docker-compose.yml: version is obsolete
[+] Running 1/1
✔ Container prdmzgateway-tbmq-1  Started
Opened the portal again, giving the same error, also corrected the redis url at /tbmq-upgrade.sh as you suggested

can you execute the following commands and show me the output?
docker compose ps | grep "postgres" | awk '{ print $1 }'

Example: downloads-postgres-1
Put in the command below instead of downloads-postgres-1 the returned name from the above command.
docker inspect -f '{{ range .NetworkSettings.Networks }}{{ .NetworkID }}{{ end }}' downloads-postgres-1

Example: 049d97c42572af92b538f0735ae6c000596ffcfd2cbfdef64874c6360b73eab9

Here what I got
prdmzgateway@prdmzgateway:~$ docker compose ps | grep "postgres" | awk '{ print $1 }'
WARN[0000] /home/prdmzgateway/docker-compose.yml: version is obsolete
prdmzgateway-postgres-1
prdmzgateway@prdmzgateway:~$ docker inspect -f '{{ range .NetworkSettings.Networks }}{{ .NetworkID }}{{ end }}' prdmzgateway-postgres-1
d30a11fa1d82e826711166111960bec020042b8a56bd8c9ee8fd5f81f6769c31

maybe try running the following command that should have been executed by the script. I have replaced the values to use the default ones and your network id.
Successfully removed device publish messages from Postgres!
Updating data from version 1.4.0 to 2.0.0 ...
Upgrade finished successfully!
[InsertUnauthorizedClientQueue] Queue polling was interrupted.
[DeleteUnauthorizedClientQueue] Queue polling was interrupted.
[LatestTimeseriesQueue] Queue polling was interrupted.
[InsertUnauthorizedClientQueue] Queue polling was interrupted.
[LatestTimeseriesQueue] Queue polling was interrupted.
[InsertUnauthorizedClientQueue] Queue polling was interrupted.
[LatestTimeseriesQueue] Queue polling was interrupted.
[DeleteUnauthorizedClientQueue] Queue polling was interrupted.
[DeleteUnauthorizedClientQueue] Queue polling was interrupted.
[TimeseriesQueue] Queue polling was interrupted.
[TimeseriesQueue] Queue polling was interrupted.
[TimeseriesQueue] Queue polling was interrupted.
Closing JPA EntityManagerFactory for persistence unit 'default'
HikariPool-1 - Shutdown initiated...
HikariPool-1 - Shutdown completed.

yes the logs you show are fine and seems you upgraded successfully.

Alright nice, yea, seems like everything now look good, thank you @dmytro-landiak


Mod count (Downloaded and Available)

Geode Version
v2.0.0-beta.22

Suggestion
It would be a nice addition if the browser told you how many mods there are and how many you have installed.

fixed on new index :3 (check the installed tab for installed mod counts)


Ember build failures cause timeouts

Ember builds that fail due to missing or out of date dependencies don't cause Rails errors.
Instead, the request will timeout, in which case all bets are off and the failures that are rendered are largely red-herrings.

I’ve modified https://github.com/quaertym/ember-cli-dependency-checker/blob/master/index.js to run in `preBuild` (I didn't think errors in its `init` hook would be handled by `buildError`)

I’m forcing a build failure in a project’s `preBuild` by omitting a dependency.

No matter what I do, I can’t get `ember-cli-rails-addon`’s `buildError` to handle the error

GitHub
quaertym/ember-cli-dependency-checker
ember-cli-dependency-checker - Ember CLI addon for checking missing node and bower dependencies before running ember commands
(edited)

rwjblue [10:06 AM]
why would dep checker want to run in prebuild?

seanpdoyle [10:06 AM]
to check dependencies before a build?

rwjblue [10:06 AM]
`init` is before building :wink:

seanpdoyle [10:06 AM]
Also I was unsure if it was throwing exceptions too early for `ember-cli-rails-addon` to catch them

seanpdoyle [10:07 AM]
I’m also fish out of water and desperately trying anything

rwjblue [10:07 AM]
but it is also before any other non-build related commands

rwjblue [10:07 AM]
if we move this check to preBuild, we loose much of the power of the dependency checker

rwjblue [10:07 AM]
because then it only checks for things that run builds

seanpdoyle [10:07 AM]
ok, I buy that

rwjblue [10:07 AM]
(which is not everything)

rwjblue [10:07 AM]
the key (like with bundler) is that is is ALWAYS ran

seanpdoyle [10:07 AM]
what can the rails addon do to notice that exception?

seanpdoyle [10:08 AM]
it’s killing the process before it even knows about the rails addon

rwjblue [10:08 AM]
how is it spawning?

rwjblue [10:08 AM]
then you need to check exit code of the spawned process

rwjblue [10:08 AM]
(and its stderr contains the error output)

seanpdoyle [10:09 AM]
`ember-cli-rails` currently calls `Kernel.spawn` on

https://github.com/thoughtbot/ember-cli-rails/blob/master/lib/ember-cli/app.rb#L285

which splits `ember build —watch` into its own process

GitHub
thoughtbot/ember-cli-rails
ember-cli-rails - Easily integrate Ember CLI and Rails

seanpdoyle [10:10 AM]
the problem is that the rest of `ember-cli-rails` depends on the rails addon to communicate ember-side errors by catching them in `buildError` and writing them to a file

seanpdoyle [10:11 AM]
TL;DR the gem doesn’t do anything to check if the `spawn` call of `ember build` fails

rondale_sc [10:11 AM]
@seanpdoyle:  are you able to `node debug $(which ember) build` and catch a `debugger` inside the error hook? (edited)

rwjblue [10:11 AM]
@seanpdoyle: then you need to spawn a thing that does what you want

rwjblue [10:11 AM]
@seanpdoyle: for example, spawn another ruby script that runs `ember build` and emits the error.json file you need

seanpdoyle [10:11 AM]
does `ember build` only write bad stuff (errors and whatnot) to `stderr`?

rwjblue [10:11 AM]
¯\_(ツ)_/¯

rwjblue [10:12 AM]
you tell me

seanpdoyle [10:12 AM]
What is an `error.json`?

rwjblue [10:12 AM]
I dunno!

rwjblue [10:12 AM]
https://embercommunity.slack.com/archives/ember-cli/p1446649834002151
Sean Doyleseanpdoyle
the problem is that the rest of `ember-cli-rails` depends on the rails addon to communicate ember-side errors by catching them in `buildError` and writing them to a file
Today at 10:10 AM

seanpdoyle [10:12 AM]
ok, just making sure its not a std ember-cli thing

rwjblue [10:12 AM]
I just picked `errors.json` as "to a file"'s name

seanpdoyle [10:12 AM]
gotcha

seanpdoyle [10:13 AM]
ok, just to clarify, ember-cli-dependency-checker *DOES* fail too early for the addon to notice. therefore, we need to come up with another way to handle pre-build failures gem-side

rondale_sc [10:13 AM]
@rwjblue: why do you need to spawn?

seanpdoyle [10:13 AM]
@rondale_sc: that’s the way the gem currently does it

rwjblue [10:13 AM]
@rondale_sc: I DON'T!!!
:joy:1

rwjblue [10:13 AM]
@rondale_sc: AFAIK that is what it was already doing

seanpdoyle [10:13 AM]
these are all my/our problems, not his

rondale_sc [10:13 AM]
Oh I see you mean the gem is calling `spawn` to fire off the build, yes?

rwjblue [10:14 AM]
RE: https://embercommunity.slack.com/archives/ember-cli/p1446649989002167
Sean Doyleseanpdoyle
ok, just to clarify, ember-cli-dependency-checker *DOES* fail too early for the addon to notice. therefore, we need to come up with another way to handle pre-build failures gem-side
Today at 10:13 AM

rwjblue [10:14 AM]
not quite right

rondale_sc [10:14 AM]
^ kk, I think I'm following along now.

rwjblue [10:14 AM]
it isn't about early

rwjblue [10:14 AM]
its that this isn't a build error

rwjblue [10:14 AM]10:14
and only buildErrors are caught by the `buildError` hook

Closed by https://github.com/thoughtbot/ember-cli-rails/commit/3703693e22b06df5be31aa9ca7c77c7df8ce8835 and https://github.com/ember-cli/ember-cli/commit/8c93854ae8b9222aff5406c7211d935ceb49c6d6.
Newer versions of ember-cli will redirect stderr to our build errors file, which ember-cli-rails will detect and present to the user.


[Bug] - Deleting Planned Activity does not re-add the supplies back.

I noticed that there is a bug when you delete a planned activity with supplies, it does not add the used supplies back to the supplies list.
Re-Creation Steps:

Create a planned activity with supplies selected for the planned activity.
Delete the planned activity.

You will notice that the used supplies quantities are not added back to supplies in the garage/car.
Please let me know if you need additional information.

Next release:

You'll be able to delete the requisition history for within the plan/service/repair/upgrade record and it will replenish the supply quantity, you can also delete the entire record and it will replenish all of the requisitioned supplies.

Released.


cloudwatch-logs module lambda with ssm_enable = "True" is missing the wrapper.sh script needed to function

From what I have read in the aws docs it would need to be in your folder/repo somewhere and added as a layer.

The wrapper.sh is in our serverless repo , also you can create the layer from our Coralogix-Lambda-SSMLayer.


Repo sync for protected CLA branch

The pull request is created from master637653350755125007 to master to fix git push error for protected CLA branch

@ghogen : Thanks for your contribution! The author(s) have been notified to review your proposed change.


feat(api): added api_key_cmd configuration option

This PR adds a feature to the plugin that introduces an alternative method of providing the OpenAI API key. In addition to providing the key via an OPENAI_API_KEY environment variable, the plugin also supports getting the API key from the output of arbitrary commands, configured via the api_key_cmd configuration parameter.
This allows users to securely provide the API key, without having to write it in clear-text to disk. It also prevents the API key from being readable by processes with access to the initial environment variables of neovim.
The default behaviour is to use the OPENAI_API_KEY environment variable, so as to not break backwards compatibility.
The README includes a couple of examples of how to use op or gpg to inject the secret into neovim.
Disclaimer: This is my first lua code since... 2005 or something, so I'm sure there's a prettier way of doing what I did here. Critique is welcome ;)
Since configuration wasn't initialized when the the api.lua file was being run, I had to introduce a setup() function to the api.lua file.

Great work, thanks! Could you just fix the styling problem to pass checks?

Great work, thanks! Could you just fix the styling problem to pass checks?

Absolutely, just pushed a fix!


What are the rules around what prettier version is used?

If I run prettier from the version installed in my project (from npm install --save-dev prettier), the output is different to that of the VS Code extension. I can see that the prettier version in the extension is fixed.

Is it possible for the extension to use the local prettier if it exists instead of the one it provides?
If not, is it possible to update the prettier version in this extension and if so, are you just going to have to manually update it all the time when a new prettier comes out?

Thanks for the great extension.

It should be possible to modify the extension to use a local version. I would even add a globally installed one.

use local version
use global version
use packaged version: Works out of the box and everyone can't sudo or have a local version. This one needs to be updated in the extension itself.

I think it is possible to update the dependency by going into vscode's extension folder, find this extension and npm i prettier@latest. Restart vscode.
I expect prettier to have a lot less updates in the future, it is really new and will likely stabilize. The global version I mentioned might not be necessary (SOON)

Great I'll try it out. I'll close this in favour of #29 (which I did not see). Thanks @CiGit


FocusEvent relatedTarget should be optional

FocusEvent relatedTarget is defined to always return something but there are times when it may be null.
Refereces

https://developer.mozilla.org/en-US/docs/Web/API/FocusEvent#Properties
https://codepen.io/wezm/pen/MqRROW

https://github.com/reasonml-community/bs-webapi-incubator/blob/e936b2e23d59583aef5e5cec0739777856021f93/src/dom/events/FocusEventRe.re#L9

Fixed. Thanks!


Luis app is not returning intents property in json

I have noticed this change luis app since 2 days where I see json returned after querying luis app does not have intents property anymore . Here is the screenshot of what I see when I query my model.

Has anything changed recently ? as I see some UI updates on luis UI as well?

@rjgmail88 You need to make sure to set the verbose querystring property to true. I'm closing this issue, feel free to reopen if the problem continues after you set verbose=true.


Removes the automatic module name (JPMS) because of a name clash.

We created overlapping modules names in VAVR 0.10.* The only way to fix that (without renaming packages in a backward incompatible way) is to remove the automatic module name. Starting with VAVR 2.0, we will ship proper modules.

Codecov Report

Merging #2487 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2487      +/-   ##
============================================
- Coverage     95.06%   95.04%   -0.02%
  Complexity     5400     5400
============================================
  Files            89       89
  Lines         12761    12761
  Branches       1582     1582
============================================
- Hits          12131    12129       -2
- Misses          452      454       +2
  Partials        178      178

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/io/vavr/concurrent/FutureImpl.java
73.68% <0%> (-1.51%)
27% <0%> (ø)

Continue to review full report at Codecov.

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


Height is too large

Hallo,
I use the owl-carousel script to display some images in the sweet alert PopUp.
The images are displayed in a row (float).
But what happens is ( I think so ) sweetarlert2 calculates for each image the hight and I get a to large popup .... the min-hight of my pop up is to large.
Is there a way to solve this problem like... set min-hight manually  or something else?
I use sweetalert2 v5
best regards

I'm not sure I understand your request, please create jsfiddle so I can see and reproduce the issue.

Thank you for the fast response.
I just send you a email with screenshots.

Thank you @quma for the email with the detailed explanation. This issue should be fixed in https://github.com/limonte/sweetalert2/releases/tag/v5.0.3 without any changes in your code.
Please try the latest swal2.

WOW thank you !
after I use the sweetalert2-common.js it works !!!

Nice :)


Hi master:

when i use this I get this error in cmd:

<EMAIL_ADDRESS>test F:\ThunderDown\ae-to-json-master\ae-to-json-master
npm run open-ae; npm run test-build-run; npm run clean-root;

npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\
node_modules\npm\bin\npm-cli.js" "run" "open-ae;" "npm" "run" "test-build-run
;" "npm" "run" "clean-root;"
npm ERR! node v6.2.0
npm ERR! npm  v3.8.9
npm ERR! missing script: open-ae;
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     https://github.com/npm/npm/issues
npm ERR! Please include the following file with any support request:
npm ERR!     F:\ThunderDown\ae-to-json-master\ae-to-json-master\npm-debug.log
npm ERR! Test failed.  See above for more details.
Can you  help me ?

My system is  window7

F:>cd F:\ThunderDown\ae-to-json-master\ae-to-json-master
F:\ThunderDown\ae-to-json-master\ae-to-json-master>npm install

<EMAIL_ADDRESS>prepublish F:\ThunderDown\ae-to-json-master\ae-to-json-master

npm run compile-ae

<EMAIL_ADDRESS>compile-ae F:\ThunderDown\ae-to-json-master\ae-to-json-master

npm run transpile; npm run after-effects;

npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\
node_modules\npm\bin\npm-cli.js" "run" "transpile;" "npm" "run" "after-effect
s;"
npm ERR! node v6.2.0
npm ERR! npm  v3.8.9
npm ERR! missing script: transpile;
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     https://github.com/npm/npm/issues
npm ERR! Please include the following file with any support request:
npm ERR!     F:\ThunderDown\ae-to-json-master\ae-to-json-master\npm-debug.log
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\
node_modules\npm\bin\npm-cli.js" "run" "compile-ae"
npm ERR! node v6.2.0
npm ERR! npm  v3.8.9
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>compile-ae: npm run transpile; npm run after-effects;
npm ERR! Exit status 1
npm ERR!
npm ERR!  Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the ae-to-json package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm run transpile; npm run after-effects;
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs ae-to-json
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls ae-to-json
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     F:\ThunderDown\ae-to-json-master\ae-to-json-master\npm-debug.log
npm WARN optional Skipping failed optional dependency /chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture: fseve
<EMAIL_ADDRESS>npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\
node_modules\npm\bin\npm-cli.js" "install"
npm ERR! node v6.2.0
npm ERR! npm  v3.8.9
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>prepublish: npm run compile-ae
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>prepublish script 'npm run compile-ae'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the ae-to-json package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm run compile-ae
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs ae-to-json
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls ae-to-json
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!      (C:\Program Files\nodejs\node_modules\npm\lib\utils\lifecycle.js:245:16)
95 verbose stack     at emitTwo (events.js:106:13)
95 verbose stack     at EventEmitter.emit (events.js:191:7)
95 verbose stack     at ChildProcess. (C:\Program Files\nodejs\node_modules\npm\lib\utils\spawn.js:24:14)
95 verbose stack     at emitTwo (events.js:106:13)
95 verbose stack     at ChildProcess.emit (events.js:191:7)
95 verbose stack     at maybeClose (internal/child_process.js:850:16)
95 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:5)
96 verbose pkgid<EMAIL_ADDRESS>97 verbose cwd F:\ThunderDown\ae-to-json-master\ae-to-json-master
98 error Windows_NT 6.1.7601
99 error argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "install"
100 error node v6.2.0
101 error npm  v3.8.9
102 error code ELIFECYCLE
103 error<EMAIL_ADDRESS>prepublish: npm run compile-ae
103 error Exit status 1
104 error Failed at the<EMAIL_ADDRESS>prepublish script 'npm run compile-ae'.
104 error Make sure you have the latest version of node.js and npm installed.
104 error If you do, this is most likely a problem with the ae-to-json package,
104 error not with npm itself.
104 error Tell the author that this fails on your system:
104 error     npm run compile-ae
104 error You can get information on how to open an issue for this project with:
104 error     npm bugs ae-to-json
104 error Or if that isn't available, you can get their info via:
104 error     npm owner ls ae-to-json
104 error There is likely additional logging output above.
105 verbose exit [ 1, true ]

@Halishen sorry the version of the after-effects module that this module uses did not support Windows. If you'd like to create a PR introducing the newest version of after-effects I'd love it.

Thank you  very much!Good luck!


Hide Prev at first step or if there is no actual prev step

After the dialog is loaded we are in first step so there is no prev step, in that situation if the previous button id by default disabled then it will make better in ux.

Hi @manchumahara
This has been resolved as of version 0.0.8 available on NPM.
If the tours activeStep is equal to 0, the previous button is disabled and a "disabled" class is added to the element. Feel free to customise this in CSS to hide the button entirely.
Thanks


Orange Liqueur and Triple Sec are interchangeable in many drinks

Triple Sec and orange liqueur are interchangeable in many drinks. And in margaritas, Triple Sec is preferred. In my opinion, there should be a way to combine them.
If not, a way to tweak recipes would also suffice so I can swap them myself.

Yes! I'll want to combine them. Just need to figure out how
I could remove triple sec and just use orange liqueur, but I don't think the experience would be that great.
Tweaking recipes is a great feature. Thanks!

There is now a version  (v1.17.0) that combines interchangeable ingredients. It should be avaliable on Google Play Store and F-droid in a couple of days.


Leter level changes recorded sometimes

If I type a s d f, then hit back in history it shows a s d, then back again, a s...
There needs to be a threshold of change required for a history entry. There also should be the option to only record history upon generation. Normally, if we're making adjustments between generations, we want to go back to the last generation. Not a ctrl+z level typing undo.

whereswaldodude

Hi.
Looking for something like this....


Add handlers to allow loading/caching stream management state

This is work in progress to figure out how this api should work.

How should we handle anything left in the unacked queue if resumption failed?


Upgrade packse to 0.3.15

Includes https://github.com/astral-sh/packse/pull/179 for #3225
uv pip compile scripts/scenarios/requirements.in -o scripts/scenarios/requirements.txt --refresh-package packse --upgrade
cargo dev fetch-python
source .env
./scripts/sync_scenarios.sh

This needs a BUILD_VENDOR_LINKS_URL update too, should we add that to the script?

Ah thanks for the note. We probably should!

We could add a separate upgrade-packse script that runs the requirements update and does this replacement?


A nasty bug it seems

I am getting a crash with an error below. Adding a line
print('compiler error') makes it go away!
Error message:
third_party/tensorflow/stream_executor/lib/statusor.cc:34 - Attempting to fetch value instead of handling error Unimplemented: Unimplemented MHLO -> HloOpcode: %s%19 = "mhlo.broadcast_in_dim"(%18) {broadcast_dimensions = dense<[0, 2]> : tensor<2xi64>} : (tensor<64x16xf32>) -> tensor<64x16x16xf32>

You're missing the code to reproduce the problem. Could you provide it, please?
I'm also assuming this is on GPU? What jax/jaxlib versions are you using?

Closing in lieu of internal bug b/173408588, since the bug reproduction involves research code that the author does not want to share on Github.


MINOR: Rename benchmark crate

Which issue does this PR close?

N/A
Rationale for this change

Consistency & avoid confusion with same crate in Ballista
What changes are included in this PR?

Rename benchmark crate
Are there any user-facing changes?

No

Does this PR break compatibility with Ballista?
No

Thanks all!


Fix image ordering

Images were off by for their respective steps. Fix for that.

Thanks for the pull request. 👍  Images for steps should be resolved now.


epel issue while trying to run perfkit on RHEL7.9 static VM

Traceback (most recent call last):
  File "/home/dgavinaa/perfkitbenchmarker/perfkitbenchmarker/pkb.py", line 1311, in RunBenchmarkTask
    RunBenchmark(spec_for_run, collector)
  File "/home/dgavinaa/perfkitbenchmarker/perfkitbenchmarker/pkb.py", line 1088, in RunBenchmark
    DoProvisionPhase(spec, detailed_timer)
  File "/home/dgavinaa/perfkitbenchmarker/perfkitbenchmarker/pkb.py", line 760, in DoProvisionPhase
    spec.Provision()
  File "/home/dgavinaa/perfkitbenchmarker/perfkitbenchmarker/benchmark_spec.py", line 718, in Provision
    vm_util.RunThreaded(self.PrepareVmAfterBoot, self.vms)
  File "/home/dgavinaa/perfkitbenchmarker/perfkitbenchmarker/background_tasks.py", line 647, in RunThreaded
    return RunParallelThreads(target_arg_tuples,
  File "/home/dgavinaa/perfkitbenchmarker/perfkitbenchmarker/background_tasks.py", line 582, in RunParallelThreads
    return _RunParallelTasks(
  File "/home/dgavinaa/perfkitbenchmarker/perfkitbenchmarker/background_tasks.py", line 556, in _RunParallelTasks
    raise parallel_exception_class(
perfkitbenchmarker.errors.VmUtil.ThreadException: The following exceptions occurred during parallel execution:
Exception occurred while calling PrepareVmAfterBoot(<IP_ADDRESS>):
Traceback (most recent call last):
  File "/home/dgavinaa/perfkitbenchmarker/perfkitbenchmarker/background_tasks.py", line 252, in Run
    self.return_value = self.target(*self.args, **self.kwargs)
  File "/home/dgavinaa/perfkitbenchmarker/perfkitbenchmarker/benchmark_spec.py", line 977, in PrepareVmAfterBoot
    vm.PrepareVMEnvironment()
  File "/home/dgavinaa/perfkitbenchmarker/perfkitbenchmarker/linux_virtual_machine.py", line 525, in PrepareVMEnvironment
    self.SetupPackageManager()
  File "/home/dgavinaa/perfkitbenchmarker/perfkitbenchmarker/linux_virtual_machine.py", line 1861, in SetupPackageManager
    self.RemoteCommand(
  File "/home/dgavinaa/perfkitbenchmarker/perfkitbenchmarker/linux_virtual_machine.py", line 1058, in RemoteCommand
    return self.RemoteCommandWithReturnCode(*args, **kwargs)[:2]
  File "/home/dgavinaa/perfkitbenchmarker/perfkitbenchmarker/linux_virtual_machine.py", line 1074, in RemoteCommandWithReturnCode
    return self.RemoteHostCommandWithReturnCode(*args, **kwargs)
  File "/home/dgavinaa/perfkitbenchmarker/perfkitbenchmarker/linux_virtual_machine.py", line 1146, in RemoteHostCommandWithReturnCode
    raise errors.VirtualMachine.RemoteCommandError(error_text)
perfkitbenchmarker.errors.VirtualMachine.RemoteCommandError: Got non-zero return code (1) executing rpm -q epel-release || sudo yum install -y http://anglxd00001.nomura.com/pulp/isos/NOMURA/Library/custom/perfkitbenchmark/perfkitbenchmark-repo/epel-release-latest-8.noarch.rpm
Full command: ssh -A -p 22<EMAIL_ADDRESS>-2 -o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no -o IdentitiesOnly=yes -o PreferredAuthentications=publickey -o PasswordAuthentication=no -o ConnectTimeout=5 -o GSSAPIAuthentication=no -o ServerAliveInterval=30 -o ServerAliveCountMax=10 -i /home/dgavinaa/.ssh/id_rsa_perfkit -o ControlPath="/tmp/perfkitbenchmarker/runs/c0e38e7a/ssh/%h" -o ControlMaster=auto -o ControlPersist=30m rpm -q epel-release || sudo yum install -y http://anglxd00001.nomura.com/pulp/isos/NOMURA/Library/custom/perfkitbenchmark/perfkitbenchmark-repo/epel-release-latest-8.noarch.rpm
STDOUT: package epel-release is not installed
Loaded plugins: enabled_repos_upload, package_upload, product-id, search-
              : disabled-repos, subscription-manager
Examining /var/tmp/yum-root-Rc1sdF/epel-release-latest-8.noarch.rpm: epel-release-8-16.el8.noarch
Marking /var/tmp/yum-root-Rc1sdF/epel-release-latest-8.noarch.rpm to be installed
Resolving Dependencies
--> Running transaction check
---> Package epel-release.noarch 0:8-16.el8 will be installed
--> Processing Dependency: redhat-release >= 8 for package: epel-release-8-16.el8.noarch
--> Finished Dependency Resolution
Error: Package: epel-release-8-16.el8.noarch (/epel-release-latest-8.noarch)
           Requires: redhat-release >= 8
           Installed: redhat-release-server-7.9-6.el7_9.x86_64 (@rhel-7-server-rpms)
               redhat-release = 7.9-6.el7_9
           Available: redhat-release-server-7.0-1.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.0-1.el7
           Available: redhat-release-server-7.1-1.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.1-1.el7
           Available: redhat-release-server-7.2-9.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.2-9.el7
           Available: redhat-release-server-7.3-7.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.3-7.el7
           Available: redhat-release-server-7.4-18.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.4-18.el7
           Available: redhat-release-server-7.5-8.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.5-8.el7
           Available: redhat-release-server-7.5-8.el7.1.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.5-8.el7.1
           Available: redhat-release-server-7.6-4.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.6-4.el7
           Available: redhat-release-server-7.7-10.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.7-10.el7
           Available: redhat-release-server-7.8-2.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.8-2.el7
           Available: redhat-release-server-7.9-3.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.9-3.el7
           Available: redhat-release-server-7.9-5.el7_9.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.9-5.el7_9
**********************************************************************
yum can be configured to try to resolve such errors by temporarily enabling
disabled repos and searching for missing dependencies.
To enable this functionality please set 'notify_only=0' in /etc/yum/pluginconf.d/search-disabled-repos.conf
**********************************************************************

 You could try using --skip-broken to work around the problem
 You could try running: rpm -Va --nofiles --nodigest
Uploading Enabled Repositories Report
Loaded plugins: product-id, subscription-manager
STDERR: Error: Package: epel-release-8-16.el8.noarch (/epel-release-latest-8.noarch)
           Requires: redhat-release >= 8
           Installed: redhat-release-server-7.9-6.el7_9.x86_64 (@rhel-7-server-rpms)
               redhat-release = 7.9-6.el7_9
           Available: redhat-release-server-7.0-1.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.0-1.el7
           Available: redhat-release-server-7.1-1.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.1-1.el7
           Available: redhat-release-server-7.2-9.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.2-9.el7
           Available: redhat-release-server-7.3-7.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.3-7.el7
           Available: redhat-release-server-7.4-18.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.4-18.el7
           Available: redhat-release-server-7.5-8.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.5-8.el7
           Available: redhat-release-server-7.5-8.el7.1.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.5-8.el7.1
           Available: redhat-release-server-7.6-4.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.6-4.el7
           Available: redhat-release-server-7.7-10.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.7-10.el7
           Available: redhat-release-server-7.8-2.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.8-2.el7
           Available: redhat-release-server-7.9-3.el7.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.9-3.el7
           Available: redhat-release-server-7.9-5.el7_9.x86_64 (rhel-7-server-rpms)
               redhat-release = 7.9-5.el7_9

command used;
/usr/local/bin/bin/python3.10 ./pkb.py --benchmarks=unixbench --benchmark_config_file=unixbench.yaml --static_vm_file=localtest.json --os_type="rhel8" --ip_addresses=EXTERNAL


Change Project Name

While I really like the project name R2DBC for Rust Relation Database Connectivity the abbreviation is taken from the Java Ecosystem that this project is inspired by. To avoid conflicts we should probably change the project name

Decided to go with RSDBC. First, Rust uses rs for its file extension so its a nice node to Rust. Second, its not atypical for people to say squared when two words start with the same letter (at least in the US) and in our case Rust Relational or R-squared. So I think this works for both Rust Relational Database Connectivity as well as just Rust Database Connectivity.


feat: Add support for Google Service Accounts and GCE VM network configuration

Description
feat: Adds support for Google Service Accounts and GCE VM network configuration

Google Life Sciences executors can now be run with user-specified service accounts
As a result, application default credentials can be used on GCE without requiring the GOOGLE_APPLICATION_CREDENTIALS environment variable.
Added flags to support user-specified service accounts, VM network and subnetwork
Updated tests and documentation

QC

[X] The PR contains a test case for the changes or the changes are already covered by an existing test case.
[X] The documentation (docs/) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).

Thanks a lot!


"ReactBingmaps" does not work with next - css file/module from node_modules is not supported

Bug report
Describe the bug
"ReactBingmaps" does not work with next - css file/module from node_modules is not supported
To Reproduce
Code repo is here: "https://github.com/jim-king-2000/nextbugrepro".
Steps to reproduce the behavior, please provide code snippets or a repository:

Create a sample next project. (see next.js tutorial.)
npm i react-bingmaps
Add "import { ReactBingmaps } from 'react-bingmaps';" into "pages/index.js".
npm install --save @zeit/next-css
create next.config.js and copy the following code:

// next.config.js
const withCSS = require('@zeit/next-css')
module.exports = withCSS()

See the error:

./node_modules/react-bingmaps/lib/components/ReactBingmaps/ReactBingmaps.css 1:0
Module parse failed: Unexpected token (1:0)
You may need an appropriate loader to handle this file type.

.react-bingmaps{
| width: 100%;
| height: 100%

Expected behavior
There should be no error.
Screenshots
./node_modules/react-bingmaps/lib/components/ReactBingmaps/ReactBingmaps.css 1:0
Module parse failed: Unexpected token (1:0)
You may need an appropriate loader to handle this file type.

.react-bingmaps{
| 	width: 100%;
| 	height: 100%

Additional context
It seems that we can use css file/module directly in "pages/index.js". But if the css file/module comes from "node_modules", next fails.

duplicate issue

The workaround below does not work. Please help.

@timneutkens ，could you please take a look again?

@Purii, would you please take a look?

Use the following hacking in next.config.js
if (typeof require !== "undefined") {
  require.extensions[".less"] = () => {};
  require.extensions[".css"] = (file) => {};
 }

const withCSS = require('@zeit/next-css')
module.exports = withCSS()


feat: support visit of stage repository

Purpose of the PR
support visit of stage repository

Main Changes

Verifying these changes

[ ] Trivial rework / code cleanup without any test coverage. (No Need)
[ ] Already covered by existing tests, such as (please modify tests here).
[ ] Need tests and can be verified as follows:

xxx

Does this PR potentially affect the following parts?

[x]  Nope
[ ]  Dependencies (add/update license info)
[ ]  Modify configurations
[ ]  The public API
[ ]  Other affects (typed here)

Documentation Status

[ ]  Doc - TODO
[ ]  Doc - Done
[x]  Doc - No Need

Currently, the issue of missing the common1.0.1 package has been fixed.
The current errors are:
1. JDK8 is not supported
2.  Test phase

Currently, the issue of missing the common1.0.1 package has been fixed. The current errors are: 1. JDK8 is not supported 2. Test phase

skip the pd/store module while we use java8?
list the related test we could choose to skip first

Currently, the issue of missing the common1.0.1 package has been fixed. The current errors are: 1. JDK8 is not supported 2. Test phase

skip the pd/store module while we use java8?
list the related test we could choose to skip first

the first problem we have to handle them now
like in action add：
if java8
mvn compile -pl hugegraph-server #specify the needed modules
else
mvn compile # all modules


Typo in depth_img = depth_frame.reshape((kinect.depth_frame_desc.Height, kinect.depth_frame_desc.Width, 4)).astype(np.uint8)

Thanks for compiling this set of useful mapper functions!
I think the value 4 in this line https://github.com/KonstantinosAng/PyKinect2-Mapper-Functions/blob/bc63d39962c9df6fc7fac3eb2cf9bc48c1bb2c03/mapper.py#L58
should be changed to 1

Yes, you are right, thank you very much. I will update it.

I think if the 1 is omitted from depth_img,
depth_img = depth_frame.reshape((kinect.depth_frame_desc.Height, kinect.depth_frame_desc.Width)).astype(np.uint8)

then the subsequent two lines
align_depth_img = np.zeros((1080, 1920, 4), dtype=np.uint8)
align_depth_img[:, :] = depth_img[depthYs, depthXs, :]

would have to change to:
align_depth_img = np.zeros((1080, 1920), dtype=np.uint8)
align_depth_img[:, :] = depth_img[depthYs, depthXs]

You are right i must have copied them from color_2_depth_space and forgot about them. I updated the code. Thanks again.


Add configurable ENV variables for use with Helm

Description
Allows for adding ENV variables to the relevant containers to tweak configuration. Any repeated ENV variables (i.e. you list FELIX_LOGSEVERITYSCREEN twice) are currently handled by Kubernetes by taking the last assignment and using that.
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Deploy preview for calico ready!
Built with commit 5e9b515df523464892de817cd98e630fdbc7c6ee
https://deploy-preview-2525--calico.netlify.com


Support for Covariates

thank you for the fantastic work! I wanted to ask if there are any schedule to support covariates (both feat_dynamic_real and past_feat_dynamic_real). These features would be extremely helpful for my project and likely for many others as well.
Thanks again for your great work!

@Houyikai if you mean support for exogenous covariates -- we are planning to make this available at some point, but the timeline has not yet been finalized.


[SP-6274] [PDI-19754] - Job entry "Delete files" fails if the files that needs to be deleted do not exist

This PR fixes below issue
If files exist, the delete is performed. But if files do not exist, then execution fails.

:white_check_mark: Build finished in 27m
Build command:
mvn clean verify -B -e -Daudit -amd -pl engine

:exclamation: No tests found!
:information_source:  This is an automatic message

Closing as it's no longer to be backported (directly) to this branch.


Description prop missing in NativeSelect

Description
Would be nice to be able to use the description prop for the NativeSelect component.
Additional Information
No response

Thanks for your request!
We have some internal refactoring planned for this week, but we'll get on this soon enough!


Any way get current version in package executor of web:lib?

I am trying to get the current version of the library to use it in the outputPath path of package executor. Is there a way to get the version in workspace.json to do something like
"package": {
  "executor": "@nrwl/web:package",
  "options": {
    "project": "package.json",
    "outputPath": "dist/libs/element-base/{version}",
    "entryFile": "libs/element-base/src/index.ts",
    "tsConfig": "libs/element-base/tsconfig.lib.json"
  }
}

Hi @harwee, actually there is no way to achieve this and we're not planning to add this feature right now as we're focusing on something else, but if this feature is needed by some other people we can plan it. I leave it open for now.

Sounds like this can be solved using postTargets option that we are planning to add here #167

I want to bump that issue. postTargets are nice, but sometimes you want to retrieve version without creating a new release. I feel like this should be possible. The implementation details are open for discussion, but I feel like programmatic solution would be great start (since I assume it already exists internally). Something like:
import { getProjectVersion } from '@jscutlery/semver'

async function foo() {
  console.log(await getProjectVersion('my-project-name'));
}

would solve many problems for people who need something more "custom".

This would be very useful, as I want to write information into a source file to include in the release. I want the tag version that would be used, to be added to a version.ts file per project/library, so I can export that in my code and then use it in UIs and other places.

Hi, are there any news about this argument?
For now, I've prepared a little script that uses git-describe to write a json file into each application that needs to access the tag version.
`const { gitDescribeSync } = require('git-describe')
const { writeFileSync } = require('fs')
const APPS_FOLDER = './apps/'
const APPS_VERSION = ['main-app', 'secondary-app']
function recordVersions() {
APPS_VERSION.forEach((app) => {
const appFolder = APPS_FOLDER + app
recordVersion(appFolder, app)
})
}
function recordVersion(appFolder, appName) {
const gitInfo = gitDescribeSync({ match: ${appName}* })
const versionInfoJson = JSON.stringify(gitInfo, null, 2)
writeFileSync(appFolder + '/git-version.json', versionInfoJson)
}
recordVersions()`
There are several problems about this workaround:

the script must be called by hand each time an application has been released
the git-version.json files must be committed before pushing the updates

Is there a way to resolve this 2 problems?
Thank you

Is there any updates on this, i would like to access the version so i can use that in other executors.


Fix init script

Added a check for docker-compose and added +x permission within git

THank you @ynixon, this is a nice idea!


Hello, hwo can help me, tune triggers in unity for vrchat

What are you talking about?

@User12344 You're best off looking up the VRChat discord for help on that. Triggers in VRChat are unique to that particular game's scripting, and is not a general Unity question that would relate to this repository (which generally does not handle scripting by default to begin with).

If you want to discus topic not related to bugs/improvements welcome to gitter


⚠ Carbubu has degraded performance

In 4ff33dc, Carbubu (https://carbubu.fr) experienced degraded performance:

HTTP code: 200
Response time: 3271 ms

Resolved: Carbubu performance has improved in c88f643 after 6 minutes.


Replaced staging.nuget.org with int.nugettest.org

The staging.nuget.org is deprecated and it is recommended to switch to int.nugettest.org (see https://github.com/NuGet/NuGetGallery/issues/5627).

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/policies/NuGet-FAQ.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@harumburum thanks for your contribution!


workflows/automate_merge_message: fix diff point

Currently it does a diff between M and B, we want a diff between H and B
M
|\
| \
C  H
|  |
|  H'
| /
|/
B

H would be the tip of the branch that was merged by M. Tbh H' is a bit of a confusing way to talk about the previous commit on the branch, H1, H2 might've been more clear? 😅


Font-family + font-size stats

Will it be able to gather stats on unique font-family / font-size combinations some day?

@t32k Is this thing working now or you just closed the issue?

StyleStats enables you to know the following metrics:

Total Unique Font Families
Unique Font Families
Total Unique Font Sizes
Unique Font Sizes

Is it enough? What metrics do you want? Please let me know more detail. :)

It's useful to =to know permutations of font families and font sizes – something like:

helvetica – 8px, 10px, 12px;
arial – 12px, 16px, 18px;
That question was about that kind of data)

Hmm, StyleStats can only parse CSS files, it means it cannot analyze cascaded style on the HTML page.
It can font metrics as follows.
body {
  font: italic small-caps normal 13px/150% Arial, Helvetica, sans-serif;
}


Update GSgnnNodeModelInterface interface

Issue #, if available:
Description of changes:
Both predictions and embed_dict are either tensors or dictionaries of tensors
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

do we need tests to trigger the code?

do we need tests to trigger the code?

Added


feat: Add HCL support

This PR will add support for HCL via tree-sitter-hcl. Thank @MichaHoffmann, for the hard work to make it possible! :rocket:
Fix #238

There's some information on atom nodes at https://difftastic.wilfred.me.uk/adding_a_parser.html#configure-parsing, but I think you want string_lit and heredoc_template.

There's some information on atom nodes at https://difftastic.wilfred.me.uk/adding_a_parser.html#configure-parsing, but I think you want string_lit and heredoc_template.

heredoc_template seems should not be treated as atoms maybe?
With heredoc_template

Without heredoc_template

@Xuanwo Unfortunately difftastic doesn't support interpolated string literals today. It assumes strings are atoms.
I'm already doing the same thing in other languages, e.g. JavaScript treats template_string as an atom.
It would definitely be nice to fix all these cases at some point, but that's a separate task :)

I'm already doing the same thing in other languages, e.g. JavaScript treats template_string as an atom.

Thanks for explaining!


Fix typo in iss.vim

paramter -> parameter

Since it's an obvious fix I'll include it and Cc Jason.


Option to read new description from stdin

Will add tests if you approve of API

@WhyNotHugo any opinion on this? I use this feature everyday


WIP: Support Redshift groups in grant statements

Prototype support for Redshift groups within grant statements. Rationale follows.
Users work as normal, but it seems Redshift does not support groups in dialect of postgresql system information functions (eg, https://docs.aws.amazon.com/redshift/latest/dg/r_HAS_TABLE_PRIVILEGE.html). As such, we have to take a deep dive into pg_class's aclitem[] framework, checking for ACLs connected to group <groupname> in the underlying stringlike structure (eg, https://docs.aws.amazon.com/redshift/latest/dg/r_PG_DEFAULT_ACL.html).
This is, needless to say, elaborate and extremely messy. I'm open to better ways to extract this information if they exist, but I do not presently see these in Redshift's documentation.

Unfortunately, the generic syntax uses several unsupported commands in Redshift (https://docs.aws.amazon.com/redshift/latest/dg/c_unsupported-postgresql-functions.html). I'll see if I can make these work using Puppet regular expression tools instead. If not, we'll need to shelve this until Redshift rebases its servers onto postgresql >= 8.4.

Updated to reflect the review comments above and several fixes from manual testing in Redshift. This also fixes ALL TABLES IN SCHEMA for groups in its UNLESS clause, by changing the SELECT statement from a singleton to a check for FALSE = ALL.


PR for #3899 providing a TypeScript TypeDefs plugin

Hi @dotansimha,
Here is a PR for the Typedefs plugin.
I created it as a draft for now as the test is failing and I'm stuck.
The test is created from https://graphql-code-generator.com/docs/custom-codegen/contributing/ and looks similar as other tests.
But for some reason expect(result.content).toBeSimilarStringTo(..); in the test is complaining TypeError: expect(...).toBeSimilarStringTo is not a function. I just don't see what I'm doing wrong unfortunately.

We already have a plugin that prints the schema in SDL so we can improve it that outputs a code file with a typeDefs export if file extension is TS or JS.
For example, Fragment Matcher plugin works in similar way;
https://graphql-code-generator.com/docs/plugins/fragment-matcher#configuration

OK if there is an easier way to integrate in a existing plugin that would be nice as well.
Then this one can be closed.


Update "define.alias" API description

The old readme says that the first parameter is the new name of old path, but according to the test case, the first parameter should be the old path and the second one is the new name.

Good catch, thank you for fixing!


RuntimeError: common/data/coco_detr.py not available in detrex configs

hello，I meet this problem when training customer data

hello，I meet this problem when training customer data

hello, please make sure you've installed detrex correctly~, after installing detrex, the config in configs/common will be linked to detrex/config/configs, therefore you can use the interface detrex.config.get_config("common/data/coco_detr.py") in projects

hello，I meet this problem when training customer data

@NSMBGhost hello, please make sure you've installed detrex correctly. After installing detrex, the configs under configs/common will be soft linked to detrex/config/configs, therefore, you can use the interface detrex.config.get_config("common/data/coco_detr.py") in projects.

I have intalled detrex according to the tutorials and runed these command correctly
$ git clone https://github.com/IDEA-Research/detrex.git
$ cd detrex
$ git submodule init
$ git submodule update
$ python -m pip install -e detectron2
$ pip install -e .
and the directory is the following picture:

but still not solve the problem,do I miss something?

hello，I meet this problem when training customer data

@NSMBGhost hello, please make sure you've installed detrex correctly. After installing detrex, the configs under configs/common will be soft linked to detrex/config/configs, therefore, you can use the interface detrex.config.get_config("common/data/coco_detr.py") in projects.

I have intalled detrex according to the tutorials and runed these command correctly $ git clone https://github.com/IDEA-Research/detrex.git $ cd detrex $ git submodule init $ git submodule update $ python -m pip install -e detectron2 $ pip install -e . and the directory is the following picture:  but still not solve the problem,do I miss something?

Please check the directory detrex/config to see if there is and dir named detrex/config/configs. It will look like:

And after building detrex, there will be the dir named build, but I didn't see it in your directory.

Seems like the build process is right~, maybe you can check if you are using right environment.

hello，I meet this problem when training customer data

@NSMBGhost hello, please make sure you've installed detrex correctly. After installing detrex, the configs under configs/common will be soft linked to detrex/config/configs, therefore, you can use the interface detrex.config.get_config("common/data/coco_detr.py") in projects.

I have intalled detrex according to the tutorials and runed these command correctly $ git clone https://github.com/IDEA-Research/detrex.git $ cd detrex $ git submodule init $ git submodule update $ python -m pip install -e detectron2 $ pip install -e . and the directory is the following picture:  but still not solve the problem,do I miss something?

Please check the directory detrex/config to see if there is and dir named detrex/config/configs. It will look like:
And after building detrex, there will be the dir named build, but I didn't see it in your directory.

When I use pycharm to debug this project, I will report an error. But today I used vscode, and it has miraculously run successfully. Now I want to train myself with coco type custom data, but I encounter difficulties. There are only four types of data sets. It seems that detectron2 does not support putting custom data sets directly in the coco dataset directory. How can I modify the code to run successfully

Yes, seems like using pycharm debug mode will meet some strange bugs here. Actually you can try to use your own dataset by hacking the train_net.py, you can pass your own dataloader here.
And you can also follow the detectron2 custom dataset tutorials to see how to register your own datasets into the dataset directory

thanks a lot！

As there is no more activity, I am closing the issue~ Feel free to reopen it if necessary.


Refactor the code to not be dependant on day.js

Task: Use built-in JavaScript functions to calculate the date. As this is just a refactoring of the code, it should perform just as with day.js. Changing the structure is fine, although the structure can remain as is. Here is the unfinished snippet I have:
const vscode = require('vscode');
const dayjs = require('dayjs');
const { yourXmas } = vscode.workspace.getConfiguration('xmasTime');

// Helpers

/**
 * @param {string} targetDate
 * @returns {{ ms: number, days: number, hours: number, minutes: number}}
 */
function getDifference(targetDate) {
    // ...

    return {
        ms: diffMs,
        days: diffDays,
        hours: diffHours,
        minutes: diffMinutes,
    };
}

function getUnit(difference) {
    // ...
}

// Get the time left and return the text

function getTimeLeft() {
    // ...
}

module.exports = { getTimeLeft };

Make sure to work on this issue in the se2-v2 branch!


fix: resolve jx-apps.yml explicitly

for versions and values files so that things are completely specific in the
jx-apps.yml file
fixes #45

/approve

/retest


Visualize sender of message

Is your feature request related to a problem? Please describe.
Sent messages look just like received messages.
Describe the solution you'd like
display sent messages on the right and received messages on the left
Describe alternatives you've considered
display a name on every message (needed for groups)

resolved by #253

Not really in group chats?

That is true, I'll edit the message


Russian Phone Numbers

Hello, added Russian phone number formats, which starting with +7 or 8
Home numbers with actual region codes

Good PR!

Module already added in #397


fix(forms): vertically align labels for switches

Closes #1072
Live Preview - Checks and radios

@Lausselloic I chose 0.4375rem in order to be an integer in px as well: 7px which should be closer to what could use a designer. However it is maybe less well-centered than the one you proposed initially: 0.413rem, a little bit difficult to evaluate sometimes.


Effectful

Codecov Report

Merging #17 into master will increase coverage by 11.79%.
The diff coverage is 90.47%.

@@             Coverage Diff             @@
##           master      #17       +/-   ##
===========================================
+ Coverage   39.89%   51.69%   +11.79%
===========================================
  Files          17       21        +4
  Lines         198      236       +38
===========================================
+ Hits           79      122       +43
+ Misses        119      114        -5

Impacted Files
Coverage Δ

src/main/scala/http4s/extend/ErrorAdapt.scala
0% <ø> (ø)
:arrow_up:

src/main/scala/http4s/extend/laws/IsoLaws.scala
100% <100%> (ø)

src/main/scala/http4s/extend/Iso.scala
100% <100%> (ø)

src/main/scala/http4s/extend/package.scala
100% <100%> (ø)

src/main/scala/http4s/extend/Effectful.scala
78.57% <78.57%> (-21.43%)
:arrow_down:

.../main/scala/http4s/extend/laws/EffectfulLaws.scala
98.14% <91.66%> (-1.86%)
:arrow_down:

...scala/http4s/extend/types/MkExceptionDisplay.scala
92.3% <92.3%> (ø)

src/main/scala/http4s/extend/types/MkVoid.scala
92.85% <92.85%> (ø)

... and 4 more

Continue to review full report at Codecov.

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