Clarify: Is a minio Client thread-safe?

Are minio client objects thread-safe? Or are users perhaps encouraged to create one client instance per goroutine?

Are minio client objects thread-safe? Or are users perhaps encouraged to create one client instance per goroutine?

You are expected to hold locks in your application and share the same minio client object. @mcandre

Are minio client objects thread-safe? Or are users perhaps encouraged to create one client instance per goroutine?

You are expected to hold locks in your application and share the same minio client object. @mcandre

In other words, minio-client object is thread safe.

@harshavardhana says "you are expected to hold locks" translation: NOT threadsafe.
then @krishnasrinivas quotes this saying "In other words, minio-client object is thread safe."
Super confusing contradictory answers. Please clarify:

Is locking needed to use a client object safely from multiple threads
Are multiple client objects required for concurrent requests

If this is documented somewhere, I cannot find it.

Sorry for the confusion!

Is locking needed to use a client object safely from multiple threads

Locking is not needed to use a client object safely from multiple threads

2. Are multiple client objects required for concurrent requests

No. You can use the same client object for concurrent requests.

@krishnasrinivas thank you for the clarification :) It might be good to mention this in the docs too.


Export current Script variables in main catches

When we have the script bail out, let's export out the main objects and include a stack trace.

closing out, we have good logging and it would be hard to try to export out the objects that are in the middle of creation if that is where the problem is at.


added partial refund functionality

This PR adds partial refund functionality to our server-side GA integration. Currently, customers can only do full refunds with our server-side GA integration. Partial refunds are not Spec'd yet - but this fix follows the traditional patterns of our ecommerce spec, and is likely to fall within Spec when (if) we do add partial refunds to the spec. In the interim, this works as a nice, non-breaking, partial refund feature - requested by Motley Fool - and it mimics the pattern already in place in our A.js GA integration.

LGTM


Creating binary files with lower case extensions

Is it possible to to create binary files where extensions PJX, SCX, VCX etc. are forced to be in lower case?
Now I have differences, because regenerated binary files have file extensions in upper case:

Maybe is there already some parameter for this?

Hi Darko:
If you are using all the files included in the FoxBin2Prg project, then this is already implemented from the beginning and enabled by default, no parameters needed.
In the same directory of FoxBin2Prg you'll find another EXEs:

fb2p_diff.exe => To make direct scx/vcx comparisons (text conversions are made in background)
filename_caps.exe => This program handles the automatic UPPER/lowercase conversions of the files. It uses a filename_caps.cfg configuration file (autodocumented inside) preconfigured for converting to lowercase the extension of the files converted.

When you make a conversion to text or to binary, a LOG is generated explaining the conversion process, which name is the same as the file selected for convertion plus ".log".
In example, if I convert to text a file named "FB2P_TEST.VCX", then a LOG file named "FB2P_TEST.VCX.LOG" is created, which include in the logging the information about the caps convertion (or not), like this example:
****************************************************************************************************
CONVERSION PROCESS  2018-03-14T09:36:08
****************************************************************************************************
> USING THIS SETTINGS: C:\DESA\FOXBIN2PRG\FOXBIN2PRG.CFG  => C:\DESA\foxbin2prg\pruebas varias\v<IP_ADDRESS>\fb2p_test.vcx
	c_OriginalFileName:           C:\DESA\foxbin2prg\pruebas varias\v<IP_ADDRESS>\fb2p_test.vcx

	* Names capitalization program [C:\DESA\FOXBIN2PRG\filename_caps.exe] found
	c_foxbin2prg.changefileattribute >> [C:\DESA\foxbin2prg\pruebas varias\v<IP_ADDRESS>\fb2p_test.VC2] lnRet = 0, dwFileAttributes_Orig = -1

	Converting file C:\DESA\foxbin2prg\pruebas varias\v<IP_ADDRESS>\fb2p_test.VC2...
		- File: C:\DESA\foxbin2prg\pruebas varias\v<IP_ADDRESS>\fb2p_test.VC2 (7.34/7.34 KiB)

In this case you can check that the log says that "* Names capitalization program [C:\DESA\FOXBIN2PRG\filename_caps.exe] found", so uses it automatically.
I suggest using the EXE version of FoxBin2Prg, it's faster, but make sure you are using all the EXEs included and that have not copied only foxbin2prg.exe (or prg, which can't work alone).

OK, thank you for this explanation.
I have copied
foxbin2prg.exe
filename_caps.exe
filename_caps.cfg
fb2p_diff.exe
foxbin2prg.cfg

to some folder, which is added to system PATH variable, and now when I call foxbin2prg.exe I don't see upper / lower case differences.
My foxbin2prg.cfg looks like this:
extension: pj2=pjm
extension: vc2=vca
extension: sc2=sca
extension: fr2=fra
extension: lb2=lba
extension: mn2=mna
extension: db2=dba
extension: dc2=dca

I also created small script to easily create text / binary files for opened VFP project.

Do not forget to copy fb2p_diff.cfg too  :D
Best Regards!

But I don't see fb2p_diff.cfg file anywhere in project?

Oh, sorry, I see that didn't include it. You can find it now. This is a link to it, so you don't need to search for it:
https://github.com/fdbozzo/foxbin2prg/blob/master/fb2p_diff.cfg
This CFG just include the configuration to 2 Diff tools, one of which is included with FoxBin2Prg (Winmerge Portable)

OK, i copied also fb2p_diff.cfg, thank you :)


derive Default for Volatile, WriteOnly and ReadOnly

ran rustfmt
derive Default for Volatile, WriteOnly and ReadOnly

Thank you and sorry for the delay!

Published as v0.2.7


fix(token): use native token url for openshift token

Since we are now talking to OSO proxy for OSO interactions, we have move away from talking to keycloak to understand if the user has connected her OSO
DO NOT MERGE - this is a sample!

replace by https://github.com/fabric8-ui/ngx-login-client/pull/101


Clicking on OUTPUT tab crashes VSCode

Type: Bug
When I click on the name of the OUTPUT tab in the VSCode panel, it can take several consecutive clicks, but this action consistently leads to an instant crash. More rarely, I've noticed it happen when switching from the Output tab to the Terminal tab. What's more, I can reproduce this even when I open VSCode in Restricted Mode via code /SafeMode. I've gotten VSCode from Homebrew.

Attached are the Console.app logs that I've gathered around the time of the
crash (with /SafeMode on, and Wifi off), ± 1 second.
vscode_crash_console_logs.txt

In this case, it's hard to use the Bisect tool to find the cause of the error because the app crashes hard every time.

Can you launch from the command line and add  --disable-extensions as an argument?

Yep, still happening 🙃 Even after removing the ~/Library/Application Support/Code directory. Is there any other way to factory-reset it?

Does https://code.visualstudio.com/docs/setup/uninstall#_clean-uninstall help?

Alas, no. But at least this time I got an Apple crash report out of it.
crashreport.txt

/extC++

It looks like this is caused by the C++ extension. Please file the issue to the C++ extension repository. Make sure to check their issue reporting template and provide them relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!


Fix target errors in auto_debugging

See: https://github.com/google/BIG-bench/issues/872

Thank you for making these corrections!


Tooltip does not work inside Modal

Description of the bug
Tooltip does not work when placed inside the Modal component.
Steps To Reproduce
Place a Tooltip inside the Modal component.
Codesandbox: https://codesandbox.io/p/sandbox/goofy-sun-hznv5g
Additional Information
No response

This might be related to #500

This is still an issue @Barsnes , made a minimal codesandbox to showcase the behaviour of Tooltip in Modal. I played a bit around with the Tooltip-component earlier this week, and I got it working by using the root prop in FloatingPortal.
https://codesandbox.io/p/sandbox/designsystem-modal-5ctlqr

This is still an issue @Barsnes , made a minimal codesandbox to showcase the behaviour of Tooltip in Modal. I played a bit around with the Tooltip-component earlier this week, and I got it working by using the root prop in FloatingPortal.
https://codesandbox.io/p/sandbox/designsystem-modal-5ctlqr

Hi, add portal={false} to the tooltip: https://codesandbox.io/p/sandbox/designsystem-modal-5ctlqr

Ah, perfect!

Ah, perfect!

I added a not about this in #2053, so we will remember to add it to the documentation later on


Change color margin space

Hi wysaid :)
in that picture
how can i change the top and bottom black margin color??
or.. how to remove the margin??
I appreciate always for your help 👍

https://github.com/wysaid/android-gpuimage-plus/blob/master/library/src/main/java/org/wysaid/view/CameraGLSurfaceView.java#L105

OMG.. Thanks a lot :)

<org.wysaid.view.CameraRecordGLSurfaceView         android:id="@+id/myGLSurfaceView"         android:layout_width="match_parent"         android:layout_height="warp_content" />
How to match the preview size??
no margin top and bottom

Ask someone good at Android GUI.

@wysaid
Hi,
Can I change the color of 'clear color'
the current updated version??

@ssy06005  You can implement this your self.
Just call glClearColor


Remove typo from pstn-usage-report.md

Closes https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/8949.

@CarolynRowe PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Help getting USPS API Key

Hello, I signed up with USPS's web tools API, and they sent the username and password to my email— yet I see nothing about how I can obtain an API key. Please guide me to the next steps

You should be able to use the Web Tools User ID (Username) listed in the registration e-mail as the API key.

Thanks @LiterallyLarry , for the quick response and easy fix!


"Segmentation fault (core dumped)" while testing with uWS.us_listen_socket_close (v16.5.0)

Hi again Alex,
This error only happens on work deploy build machines (running Ubuntu Xenial), and not on my laptop (which runs mac os mojave), so it's difficult for me to debug it.
Node version: 10.15.3
uWS version: 16.5.0
I think I've narrowed it down to this area in my tests:
describe('message handler', () => {
  const originalOpenHandler = sockets._openHandler;

  beforeEach(() => {
    sockets._opened = null;
  });

  afterEach(() => {
    sockets._openHandler = originalOpenHandler;
    sockets._opened = null;
    if (sockets.listenSocket) {
      uWS.us_listen_socket_close(sockets.listenSocket);
      sockets.listenSocket = null;
      sockets.listen(conf.websockets.port);
    }
  });

  /* Open is considered complete once authentication has resolved. */
  it('waits for open to complete', done => {
    withSandbox(async sandbox => {
      uWS.us_listen_socket_close(sockets.listenSocket);

      const onMessageSpy = sandbox.spy(sockets, '_onMessage');

      await new Promise(resolve => {
        /*
          We intentionally avoid making this an async function,
          to mimic the current source code behavior.
        */
        const wrappedOpenHandler = (ws, req) => {
          setTimeout(() => {
            sockets._openHandler(ws, req);
            onMessageSpy.should.not.be.called();
            resolve();
          }, 500);
        };

        sockets._openHandler = wrappedOpenHandler;

        sockets.listen(conf.websockets.port);

        const ws = createClient({ token: tokenFor('frank') });
        testMessageWithWs(ws, subscribeMessage);
      });
      done();
    });
  });

  it('returns early if authentication fails', done => {
    withSandbox(async sandbox => {
      const error = Error('uh oh');
      sandbox.stub(sockets, '_onConnection').callsFake(async () => {
        await Promise.delay(100);
        throw error;
      });
      sandbox.spy(sockets, '_onMessage');

      const ws = createClient({ token: tokenFor('frank') });
      testMessageWithWs(ws, subscribeMessage);
      await Promise.delay(500);

      sockets._onConnection.should.be.called();
      sockets._onMessage.should.not.be.called();

      done();
    });
  });
});

^ This is a nested scope inside of a bigger integration test body; sockets is the name of the server. Outside of this, sockets (the server) was already started with a call to #listen:
this.apiWsB = this._buildWebSocketBehavior();

this.apiWss = uWS
  .App()
  .ws('*', this.apiWsB)
  .listen(port, token => {
    if (token) {
      this.listenSocket = token;
      log.info(`Listening to port ${port}`);
    } else {
      log.error(`Failed to listen to port ${port}`);
    }
  });

^ But in the test above, I am purposely closing the connection, and trying to reconnect before continuing testing.
Please let me know if I can help clarify anything

I can now replicate the issue in a docker container (the image is just node:10.15.3)

uWS.us_listen_socket_close is a direct unchecked wrapper of us_listen_socket_close so any misuse leads to undefined behavior (it may crash, it may not crash, it may run buggy, etc).
You have to be very careful with that function

Thanks! This seems to be an issue with the way my tests are configured so I'll be closing this. Happy New Year!


chore: Slight formatting fixes in exploring-data.mdx

Add missing space, correct spacing for bold formatting
SUMMARY
Slight formatting fixes in documentation. Adding missing space, and correcting bold formatting.
TEST PLAN
Review formatting changes
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #12846 (d1aae0d) into master (bf2cd56) will decrease coverage by 0.69%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #12846      +/-   ##
==========================================
- Coverage   64.08%   63.38%   -0.70%
==========================================
  Files         488      488
  Lines       30175    30160      -15
==========================================
- Hits        19338    19118     -220
- Misses      10837    11042     +205

Flag
Coverage Δ

python
63.38% <ø> (-0.70%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

superset/db_engines/hive.py
0.00% <0.00%> (-85.72%)
:arrow_down:

superset/databases/commands/create.py
32.65% <0.00%> (-59.19%)
:arrow_down:

superset/sql_validators/postgres.py
50.00% <0.00%> (-50.00%)
:arrow_down:

superset/views/database/mixins.py
59.64% <0.00%> (-22.81%)
:arrow_down:

superset/db_engine_specs/hive.py
73.84% <0.00%> (-17.31%)
:arrow_down:

superset/databases/commands/update.py
85.71% <0.00%> (-8.17%)
:arrow_down:

superset/db_engine_specs/presto.py
81.38% <0.00%> (-6.71%)
:arrow_down:

superset/databases/schemas.py
93.40% <0.00%> (-6.05%)
:arrow_down:

superset/databases/dao.py
94.11% <0.00%> (-5.89%)
:arrow_down:

superset/databases/api.py
86.55% <0.00%> (-5.47%)
:arrow_down:

... and 20 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 bf2cd56...d1aae0d. Read the comment docs.

Codecov Report

Merging #12846 (d1aae0d) into master (bf2cd56) will decrease coverage by 0.69%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #12846      +/-   ##
==========================================
- Coverage   64.08%   63.38%   -0.70%
==========================================
  Files         488      488
  Lines       30175    30160      -15
==========================================
- Hits        19338    19118     -220
- Misses      10837    11042     +205

Flag
Coverage Δ

python
63.38% <ø> (-0.70%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

superset/db_engines/hive.py
0.00% <0.00%> (-85.72%)
:arrow_down:

superset/databases/commands/create.py
32.65% <0.00%> (-59.19%)
:arrow_down:

superset/sql_validators/postgres.py
50.00% <0.00%> (-50.00%)
:arrow_down:

superset/views/database/mixins.py
59.64% <0.00%> (-22.81%)
:arrow_down:

superset/db_engine_specs/hive.py
73.84% <0.00%> (-17.31%)
:arrow_down:

superset/databases/commands/update.py
85.71% <0.00%> (-8.17%)
:arrow_down:

superset/db_engine_specs/presto.py
81.38% <0.00%> (-6.71%)
:arrow_down:

superset/databases/schemas.py
93.40% <0.00%> (-6.05%)
:arrow_down:

superset/databases/dao.py
94.11% <0.00%> (-5.89%)
:arrow_down:

superset/databases/api.py
86.55% <0.00%> (-5.47%)
:arrow_down:

... and 20 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 bf2cd56...d1aae0d. Read the comment docs.

Welcome @neilsmiller and thanks for this!

Welcome @neilsmiller and thanks for this!

For future reference @neilsmiller the title of PR's has to to be pre-pended with something like chore or bug etc for the Github PR linter to be happy! I've renamed it this time for you

For future reference @neilsmiller the title of PR's has to to be pre-pended with something like chore or bug etc for the Github PR linter to be happy! I've renamed it this time for you

@srinify got it, thanks for letting me know!


command-line plugin doesn't recalculate width size for floats; forces horizontal scrollbars

I have a something like this:

cd foo/bar

In other words, I have my Prism code section in a figure, and I float the entire figure on the left side, telling it that it can take up to 50% of the page. And it looks great! Obviously on most screens this figure is far smaller than the half the screen, so there is no scrollbars whatsoever.
But then if I turn on the command-line plugin, I have a problem:

cd foo/bar

The command-line plugin adds a left section with the *nix command prompt. Great---that's what I wanted. Unfortunately the size of the floated figure does not expand horizontally, even though there is plenty of room on the left half of the screen. Because the  section needs more room horizontally than the
 element gives it, the  section develops horizontal scrollbars. But they are unnecessary---the entire figure should have simply expanded horizontally!
In summary, the addition of the command-line plugin makes the  element require more room with the addition of the command prompt, but the outer
 element apparently still only reserves enough space for the  without the command prompt, resulting in unnecessary horizontal scrollbars.

Hi @garretwilson,
Thanks for reporting this! Could you provide a simple online example that demonstrates this behavior?

I'll provide you with complete HTML source code:
<!DOCTYPE html>

	<meta charset="UTF-8" />
	<title>Prism Floating Command Line Bug</title>
	<link type="text/css" rel="stylesheet" href="prism.css" />

mkdir doc
cd helloworld/src
javadoc -d ../doc com.example

To reproduce this issue you should download Prism from http://prismjs.com/download.html using the "coy" theme for all languages, and include the following plugins:

Line Highlight
Line Numbers
Keep Markup
Command Line

Now my course is being translated to Portuguese, and we have a class using this next week. Still no movement on this? Anything I can do to get the ball rolling? Can I contribute some money somewhere to get someone to look at this? Thanks.

Hi @garretwilson,
Sorry for the late response, I was really busy in the last couple of months.
The command-line plugin uses the property float: left to position the command prompts. Unfortunately a float usually does not change the width of the parent. I've tried many things to solve this with floats and I came to the conclusion, that it is pretty much impossible.
But I found a working solution without float using inline-blocks Branch Commit. It is a bit of a hack. Maybe you could test it and if it works I would create a PR.

I found a bug in my solution. Here is the new commit. It is quite possible, that this will break some stuff. We should test it extensively, before we apply it to the Prism.

Hi, @zeitgeist87. Thanks for looking into this. I've been so busy writing this course that I'm only now getting a chance to look back at the status here.
 Or do I need to download your fork and try that?

@garretwilson I know it's been a while, but the commit referenced isn't in Prism, just his fork. You would need to copy the code in manually to your codebase to test if it resolves your issue. If it does, we can pull it in.
If you're not really having a problem anymore, we can close this issue. Let me know.

I'm still having this problem. In fact it cropped up just a few days ago. I had an example bash input in a figure that I had made be less than half the page horizontally. Once I turned on the command line plugin, even though the figure had plenty of space, it forced horizontal lines. If I remember correctly, it doesn't account for the extra command-line prompt that appears on the left. Unfortunately I don't have time to try pulling in the test fix right now. But it's not resolved, so it shouldn't be closed.


refactor: integrate vue-i18n-routing

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
As mentioned here https://github.com/intlify/routing/pull/79#issuecomment-1793307251 we want to integrate vue-i18n-routing in this repo.
We probably want the tests from that repo as well, so I still need to work on that.
@kazupon
There is likely still quite some code that can be removed, and maybe the overall directory structure should be changed, let me know what you think.
I also removed the exclusions/splits that were added to support Vue 2, would this cause issues for Nuxt I18n Bridge/7.4? I removed it to lower the build size.
Also could you take a look at https://github.com/BobbieGoede/i18n/blob/refactor/integrate-routing-4/src/runtime/routing/utils.ts, I'm not sure if the checks are still relevant for Nuxt/Vue 3 as I don't fully understand the difference between I18n, VueI18n and Composer 😅

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

@BobbieGoede

I also removed the exclusions/splits that were added to support Vue 2, would this cause issues for Nuxt I18n Bridge/7.4? I removed it to lower the build size.

You don't have to worry about that. :)
Nuxt I18n Bridge / Nuxt I18n v7.4 can provide the composable used by Nuxt I18n v8 for the existing Nuxt I18n v7.3 code base. You no longer need to worry about vue-i18n-routing. /cc @wattanx

Also could you take a look at https://github.com/BobbieGoede/i18n/blob/refactor/integrate-routing-4/src/runtime/routing/utils.ts, I'm not sure if the checks are still relevant for Nuxt/Vue 3 as I don't fully understand the difference between I18n, VueI18n and Composer 😅

I think you can remove them if they are not used in Nuxt I18n.
I18n is an instance created by createI18n, and APIs such as t can be used via the global property.
The Composer is the skeleton exported by useI18n and others that provide Vue composition API-based API.
VueI18n is a legacy API that provides an API compatible with vue-i18n v8. It's wrapped with Composer.

Thanks for the feedback!

I think you can remove them if they are not used in Nuxt I18n.

Some of it is still being used but I have now removed the unused parts and cleaned/refactored the rest of the utils.

Great! 👏
LGTM!

I think all that's left to do is to move/add the relevant tests from vue-i18n-routing, hopefully I have time to work on that in the next few days. Will let you know when that's done!

@kazupon
I have added most of the tests from vue-i18n-routing. Some of the features/tests don't quite work the same in @nuxtjs/i18n (https://github.com/nuxt-modules/i18n/pull/2686/files#diff-1e1fee9c39238ac92309cc638084d2fa0d3a898b6ea2f4f98de2d01188447306R105-R108 and https://github.com/nuxt-modules/i18n/pull/2686/files#diff-1e1fee9c39238ac92309cc638084d2fa0d3a898b6ea2f4f98de2d01188447306R120-R124) not sure how and whether those should be tested.
Some of the tests had to be adjusted due to https://github.com/nuxt-modules/i18n/issues/2581.
I haven't added the extends tests, related to extending vueI18n and router. Not entirely sure how those could be tested accurately within the module.
Moving the tests also revealed an existing bug as seen here https://github.com/nuxt-modules/i18n/pull/2686/files#diff-1e1fee9c39238ac92309cc638084d2fa0d3a898b6ea2f4f98de2d01188447306R46-R56. When using strategy no_prefix the routes are not being escaped, this case was not being tested in vue-i18n-routing. I will create a new issue to track this bug.

@BobbieGoede
Thank you for your efforts!

I have added most of the tests from vue-i18n-routing. Some of the features/tests don't quite work the same in @nuxtjs/i18n (https://github.com/nuxt-modules/i18n/pull/2686/files#diff-1e1fee9c39238ac92309cc638084d2fa0d3a898b6ea2f4f98de2d01188447306R105-R108 and https://github.com/nuxt-modules/i18n/pull/2686/files#diff-1e1fee9c39238ac92309cc638084d2fa0d3a898b6ea2f4f98de2d01188447306R120-R124) not sure how and whether those should be tested.

This may be due to a slight difference in behavior between the vue-router used in vue-i18n-routing and the routing inside nuxt.
nuxt uses vue-router, but internally nuxt wraps it and tweaks routing a bit.
The difference may affect the test.

I haven't added the extends tests, related to extending vueI18n and router. Not entirely sure how those could be tested accurately within the module.

extendI18n does not need to be tested if it works with nuxt i18n specs.
(BTW, I was thinking of providing vue-i18n-routing as a routing library to Vue community that uses only vue + vue-router) 😅

extendI18n does not need to be tested if it works with nuxt i18n specs.

It seems to be working 😅

(BTW, I was thinking of providing vue-i18n-routing as a routing library to Vue community that uses only vue + vue-router) 😅

What do you mean exactly? If there's any changes planned for vue-i18n-routing I can help 😄
I have no more changes for this PR, it can be merged if everything looks good!

What do you mean exactly? If there's any changes planned for vue-i18n-routing I can help 😄

With using vite-plugin-pages, I wanted to give like nuxt i18n routing by using vue-i18n-routing for vue community.
https://github.com/hannoeru/vite-plugin-pages
And I wanted to give i18n routing on SSR with using vike
https://github.com/vikejs
I wanted to give these to the vite community via vue-i18n-routing, but my time is limited. 😅


Fix apply configuration generation on macOS

I noticed this while looking into #240
Commit 1
On at least macos 14.1 (sonoma) the paste utility seems to require a filename to be specified; before this change, the API_DIRS variable fails with the following error:

usage: paste [-s] [-d delimiters] file ...

Commit 2
This avoids us spinning up a separate shell when evaluating API_DIRS, since GO_MODULE already contains the value we want.
This change also changes the variable assignments; make variables are very confusing, but := should only evaluate at startup while = will evaluate every time the variable is used.
See these docs for how = works, and these for :=. From the first link:

Another disadvantage is that any functions referenced in the definition will be executed every time the variable is expanded. This makes make run slower; worse, it causes the wildcard and shell functions to give unpredictable results because you cannot easily control when they are called, or even how many times.

Thanks Ash! I am just dreaming of a Mac! 😉 I have also verified that this still works on my Windows-Ubuntu.
/lgtm
/approve


Uploadify: if there is no Flash Player there will be no automatic fallback

Is there anybody how use Uploadify? Or are we about to drop Uploadify support?

Simply Add 'onFallback' function...
'onFallback': function () {
alert('Flash was not detected.');
}


Rollback - k8s version not reflected on clusters after rancher server rollback

What kind of request is this: bug
Steps to reproduce:

Launch rancher server v2.1.6
Create a cluster with k8s version - v1.11.6
Upgrade rancher server to v2.2.0-rc2.
For the cluster created already, upgrade k8s version to v1.12.5/ v1.13.1
Rollback the rancher server to v2.1.6

Expected Result:
kubernetesVersion under rancherKubernetesEngineConfig and appliedSpec should reflect the upgraded one, which is under gitVersion
Actual Result:
kubernetesVersion under rancherKubernetesEngineConfig and appliedSpec reflects older version.

Other details that may be helpful:
Environment information

Rancher version: v2.1.6 to v2.2.0-rc2
Installation option (single install/HA): single install

Cluster information

Cluster type : DO/AWS/Azure
Kubernetes version (use kubectl version):

before upgrade of rancher- v1.11.6
after upgrade of rancher - upgraded to v1.12.5/ v1.13.1

@sowmyav27 I am able to reproduce this issue when I rollback to the etcd backup taken before the upgrade. This is expected, because this data doesn't have latest cluster state. Also, don't think v1.12.5 is supported version for rancher/rancher:v2.1.6. So we'd sync up information from v1 k8s node, and it would reflect in version under cluster API but not in our saved RKE config.
These kind of rollbacks would only happen when upgrade fails for some reason, and user wants to rollback to previous data and restore the system. So I think we should be okay with this, @alena1108 @sangeethah can confirm.

Confirming. When rancher rolls back, it rolls back to the old etcd snapshot. It means that any changes done on the next rancher version prior to rollback, can get lost once rollback is performed.

wont fix


[Candidate 1v1 The Valley ZH v1] Supply piles in center are not mirrored

Issue Description
Supply piles in center are not mirrored.
Expected behavior
Supply piles in center are mirrored.
Screenshots

Done.

Fixed with https://github.com/xezon/GeneralsRankedMaps/commit/747cf94bec785adbb3c3c488250fcc705a17a85d


Deploy specific app

Server side fix for https://github.com/remind101/empire/issues/615.
Two things I wanted to call out:

The change to AppsFindOrCreateByRepo. I had talked to Barrett about this being a design choice on your guys end. The change I made makes sense to me, but I just started working with this stuff today. Let me know if there is some implication I'm unaware of by unlinking them.
emp deploy -a some-app remind101/acme-inc will only work if some-app exists. This seemed like a much simpler choice than creating the app if it doesn't exist. If we create the app based on "-a" then that subtly changes the meaning of "-a" in relation to all other commands that use it. I'll defer to whatever you guys prefer on this though.

This looks great! I'll play around with it a bit today. Couple of initial comments coming.

This is great! Thanks!


TOC to LT (#1)

update to camera

typo

formatting

Controller support

Move to the real colors ;D

features manager es6 import

link update

Fix #1862

Fix #1861

Fix #1860

Changed "Table of contents" to "Lesson Topics"

It's more intuitive to call the list of sections for a single lesson to be called "Lesson Topics" because "Table of contents" usually
refers to the list of lessons, not the list of things in  a lesson.
Co-authored-by: Raanan Weber<EMAIL_ADDRESS>Co-authored-by: David Catuhe<EMAIL_ADDRESS>

Up?

Jared, to facilitate easier discussion and merging it is customary to make many small changes rather than one large change. In this case I'd suggest that you withdraw/delete the PR and create a branch with a PR for each issue. I can work with you on it if you have questions. Thanks!

I'm finally going to do something about that.


chore: make pollInterval configurable

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

/lgtm


fullRobot animation

Reported by anonymous on 28 Mar 2011 11:01 UTC
The "fullRobotR3" (Modelica.Mechanics.MultiBody.Examples.Systems.RobotR3.fullRobot) example doesn't give a meaningful animation. The animation works in Dymola if the "extra" value of all shapes in the example is set to zero. According to the documentation, the extra value should not have any influence.
Since the model is part of the MSL the animation should work without any Dymola specific settings.

Migrated-From: https://trac.modelica.org/Modelica/ticket/509

Modified by dietmarw on 28 Mar 2011 13:02 UTC

Comment by dietmarw on 28 Mar 2011 15:01 UTC
duplicate of #510


Can't compile even though portaudio is installed

I'm using Mac OS.
I have added portaudio as a dependency in a project and I  have the following error, both when using version 0.6.3 from crates.io and and from the git master:
Compiling portaudio v0.6.3 (https://github.com/RustAudio/rust-portaudio.git#dc1f284f)
error: could not find native static library `portaudio`, perhaps an -L flag is missing?
Could not compile `portaudio`.

However, I have installed portaudio with port:
$ ls /opt/local/lib | grep portaudio
libportaudio.2.dylib
libportaudio.a
libportaudio.dylib
libportaudiocpp.0.dylib
libportaudiocpp.a
libportaudiocpp.dylib

I have added it to the library path:
$ echo $LIBRARY_PATH
/opt/local/lib/

I have also tried cargo rustc -- -L/opt/local/bin/ without success... apparently, the flags added for rustc are not used to compile dependencies.
I have also tried to install portaudio with Homebrew (so that it is in the more usual location, in /usr/local/lib/) but it doesn't work either.
Anyway, shouldn't the build process download portaudio and compile it if it is not found?

Have you tried cargo clean after installing portaudio, and before compiling?

Same here, even after cargo clean.

I am now using version 0.6.4 and it works flawlessly.

I had a build issue with the same error as listed above.  I think I have tracked down the problem and have a workaround. I'm on a few days old, clean install of macOS 10.11.5. I don't have ports or brew installed. I'm running the current beta of Xcode.
The stable version of PortAudio won't build on my setup, so I went with the daily snapshot from July 11. This builds fine on my system and I installed it under /usr/local. The 0.6.4 version of rust-portaudio fails to build against this setup with the error generated in the original parent issue.
I did some poking around and it looked to me like the build was always trying to do a static build no matter what settings and environment vars I supplied. I then discovered that if I commented out the "if pkg_config::Config::new:: ..." section in build.rs, the build worked fine. So, I checked out pkg-config-rs and did a build of it. The build completes successfully but the tests fail. I discovered that this is caused by the fact that I didn't have the pkg-config program installed on my system.
The fix: I downloaded pkg-config latest from https://pkg-config.freedesktop.org/releases/ and built and installed with a configuration of,
./configure --prefix=/usr/local --with-internal-glib
I then did a clean clone of rust-portaudio and ran cargo build and cargo test with no issues. All of the example programs run as they are supposed to.

Similar problem. Running on mac OSX 10.11.5. portaudio is installed (/usr/local/lib/)

Thanks a lot everyone, I'll leave this open until a fix for the build.rs script lands. I won't have time to do this myself for a little while yet, so if anyone is considering making the fix feel free to open a PR.

if anyone is considering making the fix feel free to open a PR

The "fix" I tried is not really a fix, it's a quick workaround. I replaced this:
if pkg_config::Config::new().atleast_version("19").find("portaudio-2.0").is_ok() {
    return;
}

With:
// if pkg_config::Config::new().atleast_version("19").find("portaudio-2.0").is_ok() {
    return; // <- return is still here
// }

I'm not sure it's what you want. Should we do something conditional like cfg!(target_os = "macos")?

Just found a solution. Just run brew install pkg-config too (and cargo clean if you have some mess). I will send a PR for readme.

Yeah, I also feel that any fix here should just be in the readme and mention how to install pkg-config. The real problem seems to me to be in pkg-config-rs, so perhaps an issue needs to be opened there too.

I've just tried this locally on my machine, and I noticed a small issue. I'm on a fairly fresh machine that didn't previously have PortAudio installed, and am using rust-portaudio 0.7.0.
I initially ran cargo build and got the standard "error: could not find native static library portaudio, perhaps an -L flag is missing?" message. I then did brew install portaudio && brew install pkg-config. Trying cargo build led to the same error message. At first I thought there must be something wrong with the build script or environment (which is what led me to this thread). However, after some experimenting, I determined that if I closed my terminal and reopened it, and then re-ran cargo build, things worked as expected.
I'm not sure what the issue exactly was though. Can anyone reproduce it/any ideas?


domain without ALLOW-FROM should error

These two are equivalent, but the second one should throw:
frameguard({
  action: 'SAMEORIGIN'
})

frameguard({
  action: 'SAMEORIGIN',
  domain: 'https://example.com'
})

Done in #19, which will be deployed in frameguard@4 (see #20 for that).


Bug: Coupon get applied even if invalid

Expected behavior:
As an user I expect the coupon that get automatically applied to the cart, to be valid for my order
Actual behavior:
Coupon that are automatically applied do not take into consideration the "days" in which the coupon are valid, and therefore can be applied, but invalid (an error popup will come up as soon as they are applied).
Reproduce steps:

Create a coupon with recurrence on a different day that today
Make the coupon auto applicable
Go on the website
Add something to the cart that will make the coupon applicable
see the coupon being applied even if it should not (as it is the wrong day of the week)

Version:
3.4.4-beta-26
Additional Information:
I have not personally checked if all other settings (minimum price, correct items, etc..) work properly.

Thanks for the reminder - this has been on my list to circle back to.
I've opened a PR with a fix


Update serverDiff description in README.md

Making the serverDiff setting more clear in the readme

Was already in the middle of updating. But thank you anyways.


Fix/auth-error

Goal of this PR : Close https://github.com/aeecleclair/Titan/issues/354
Description of the added features :

[x] Improving error message
[x] Logging them
[x] Displaying them on toast

Codecov Report
Attention: Patch coverage is 0% with 58 lines in your changes are missing coverage. Please review.

Project coverage is 54.71%. Comparing base (32264c5) to head (d418d94).

Files
Patch %
Lines

lib/auth/providers/openid_provider.dart
0.00%
58 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #364   +/-   ##
=======================================
  Coverage   54.71%   54.71%
=======================================
  Files         168      168
  Lines        3670     3670
=======================================
  Hits         2008     2008
  Misses       1662     1662

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


Don't reuse the same variable for different purposes

Description
Don't reuse the same variable for different purposes
It is just local change, but yet will help the linters to figure out what can go wrong
Type of change

[x] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library

/lgtm

/approve


Update engine linter to require a bug URL on FLUTTER_NOLINT lines

Because we tend to tack // FLUTTER_NOLINT lines onto the first available line after the copyright header, and because we tend to copy existing copyright headers into newly-created files, there are very good chances that the linter opt-out comment ends up accidentally copied along with the header.
Further, it's often unclear why a given file is opted out of the linter. We should update ci/bin/lint.dart to verify that when a FLUTTER_NOLINT comment is found, it is followed by a GitHub issue URL, or we fail the lint for the file.

Fixed by flutter/engine#21922.


Reformat quasiquote spec according to commonmark

This commit formats the quasiquote markdown document with Prettier
(https://prettier.io/) to make it compatible with commonmark so that it
renders normally in Docusaurus. To make the tables render correctly, we
alsoy escape pipe characters | for Pat.Alternative and Type.Or

Before this PR

After this PR


[DRAFT] Exporter for TFLite

This draft is for TFLite Exporter.
Related: #1879
Signed-off-by: seongwoo<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1892. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @mhs4670go, nntrainer/utils/node_exporter.cpp does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

@jijoongmoon Could you tell me about this error?

:octocat: cibot: @mhs4670go, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1892-202204270116260.84354090690613-3b2dfca6b9da817877edc232fb1419dea815df9a/.

:octocat: cibot: @mhs4670go, nntrainer/utils/node_exporter.cpp does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

@jijoongmoon Could you tell me about this error?
You can check where is the error.
http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1892-202204270116260.84354090690613-3b2dfca6b9da817877edc232fb1419dea815df9a/report/doxygen_tag.txt

:octocat: cibot: @mhs4670go, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1892-202204271124580.61964893341064-df35a81a346cfbc26981fe184defb76ff82055ed/.

:octocat: cibot: @mhs4670go, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1892-202204281818000.48909997940063-a2ec84a5752fb74ca3de0fe5dfd676d87f2ca0f4/.

:octocat: cibot: @mhs4670go, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1892-202204282241020.74452090263367-7f41e66cc29ada9c15afc6d3cfd08f9f08ad0ef8/.

:octocat: cibot: @mhs4670go, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1892-202204290007390.9612820148468-d9abdfb72df3ce2d79061560111d7e7e2396f9b7/.

:octocat: cibot: @mhs4670go, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1892-202204290033290.77930402755737-c2e3acf0cb041bc5a0b2d4d5df0aaf373f1636d0/.

:octocat: cibot: @mhs4670go, nntrainer/compiler/tflite_opnode.h does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

:octocat: cibot: @mhs4670go, nntrainer/compiler/tflite_opnode.h does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

:octocat: cibot: @mhs4670go, nntrainer/compiler/tflite_opnode.h does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

:octocat: cibot: @mhs4670go, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1892-202204291601400.38410592079163-c488192721acb94d1a522914b950ad7e08f193db/.

:octocat: cibot: @mhs4670go, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1892-202204300021290.96123194694519-dfbb2fef0a71fee7f86a50fc845bdee4256709a8/.

:octocat: cibot: @mhs4670go, nntrainer/compiler/tflite_opnode.h does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

:octocat: cibot: @mhs4670go, nntrainer/models/neuralnet.h does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

:octocat: cibot: @mhs4670go, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1892-202205061634090.013375043869019-ce863435d030e2c3172600e3722952dc844df05a/.

:octocat: cibot: @mhs4670go, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1892-202205061905060.87092304229736-f85716624245ddf87e816d96c87ecd8e91808fe4/.

@jijoongmoon FYI, you can get resnet_test.tflite by ./nntrainer_resnet18 fake 1 128 1

:octocat: cibot: @mhs4670go, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1892-202205101545230.86554408073425-3a45f581ffa717aa65263d38c0fb0816ea6da1c9/.

:octocat: cibot: @mhs4670go, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1892-202205101622280.59861707687378-ef1d9c1b594e895bd37470702d5f6209217d4b03/.

:octocat: cibot: @mhs4670go, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1892-202205112006340.039571046829224-a32dac395b9f12dcefcf505fd29accb4057b3fb0/.


[Changelog] Update for Hammer-10

[skip ci]

@miq-bot add_label documentation

@miq-bot add_label hammer/yes

@carbonin can you review/merge as time permits.

Hammer backport details:
$ git log -1
commit 467000ca2917ad000fc3d788dd67cc80a3d2cec4
Author: Nick Carboni<EMAIL_ADDRESS>Date:   Mon Aug 12 10:26:13 2019 -0400

    Merge pull request #19128 from JPrause/hammer10_cl

    [Changelog] Update for Hammer-10

    (cherry picked from commit 0f43a31b954030ea052cb110e0ad22b519537059)


pytorch version

When I convert it to the Pytorch version, I have the following problems. Can you help me solve them？thanks
error:
x_gap = x.mean(axis=2,keepdims=True).mean(axis=3,keepdims=True)
TypeError: mean() received an invalid combination of arguments - got (axis=int, keepdims=bool, ), but expected one of:

()
(torch.dtype dtype)
(tuple of ints dim, torch.dtype dtype)
didn't match because some of the keywords were incorrect: axis, keepdims
(tuple of ints dim, bool keepdim, torch.dtype dtype)
(tuple of ints dim, bool keepdim)
didn't match because some of the keywords were incorrect: axis, keepdims

Please refer to torch API which uses the parameters dim and keepdim.

Please refer to torch API which uses the parameters dim and keepdim.

I have tried many times and found no success. Do you know how to convert？thanks
x_gap = x.mean(axis=2,keepdims=True).mean(axis=3,keepdims=True)

As the error info mentioned:  the combination of arguments (axis, keepdims) is not valid, the keywords were incorrect;
As the error info suggested: you should use (dim, keepdim) instead, because of the difference between MegEngine API and torch API.

Please check my unofficial PyTorch implementation.


Make the LIVD_Supplemental table work for us

Prerequisities:   #2055 and then #2054.  Note: no coding involved in this ticket.
The LIVD_Supplemental table is our way of tracking whether a particular diagnostic device is meant for at-home / unproctored / etc usage.
https://github.com/CDCgov/prime-reportstream/blob/master/prime-router/metadata/tables/LIVD-Supplemental-2021-06-07.csv
The concept is ok, but needs a bit more work, and a process wrapped around it.
Once #2054 is done, then we'll ensure we have valid test_kit_name_ids in our system.
Acceptance Criteria

Query the covid_result_metadata table and find all the test_kit_name_ids we are receiving, and who is sending them.   Work with @anshulkumar-usds  to contact the senders and get the values for is_otc, is_proctored and is_home.   Populate the LIVD-Supplemental table, and have someone else double check your work.
Ask @anshulkumar-usds to add an Onboarding question, such that we find out this information earlier in the process.
There's a playbook out there somewhere for how to update the LIVD table.   Add to it this process to update the Supplemental table at the same time, eg, to turn task 1 above into an operational process.

I think this has been superceded both by @carlosfelix2 work to combine LIVD Supplemental with LIVD, and by the hope that OTC and related info can be provided by HHS.  Closing.


Improve storefront price pattern

That won't fly. A regexp with $ and . will not work due to the special character handling. So we have to make it rather a full regexp and get rid off the variables to be able to express the \ quoting.
Example: $ \d{1,3}([,]\d{3})*.\d{2}


Users/arunkm/symboltask/make symbolservertype to be explicitly selected

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


Does this play nicely with Better Errors?

Or is this just for the log file?

I have no idea about that because I have no experience using it.
Any proposals are welcome.


Add static version file for skuba-update

skuba-update version is only correct when rpm
building. This is because there is hidden black
magic in the spec file, generating a VERSION
file based on the value of the Version of the
rpm. This Version is edited in place in the
spec file by a script, triggered from the Makefile,
which itself generates the version based on
git describe information.
This PR ensures that the VERSION file is always
present in our repo, by generating it directly
during any "make" step, and adding a
pre-commit hook to ensure that the VERSION
file is always correct.

Rebased

I am abandonning this, I don't see much value in it. If it really had some value, we would probably have merged it by now. Anyone can resurrect this if necessary.


cleanups around checkpoint-recovery

Motivation
few refactors to simplify and untangle the code
Description

cleaned up unit tests:

removed implicit usage of globals (expectedCheckpoint)
reordered tests to avoid tests branching on conditions like if tc.fail (tests for error conditions are separate now)

moved loading checkpoint out from the Cobra command to the app start routine
removed preserve *checkpoint.PreserveData from App struct - it's a local variable now
moved few checks from node package to checkpoint for better testability + added tests

Test Plan
tests pass
TODO

[x] Explain motivation or link existing issue(s)
[x] Test changes and document test plan
[ ] Update documentation as needed
[ ] Update changelog as needed

Bors merge

git on windows failed
bors merge

Flaky TestPartition
bors merge

bors merge

bors merge


No comments pane

Type: Bug
I cannot find the comments pane anywhere. It doesn't matter which setting I use for it to appear.

VS Code version: Code 1.71.0 (784b0177c56c607789f9638da7b6bf3230d47a8c, 2022-09-01T07:25:10.472Z)
OS version: Linux x64 5.15.60-1-MANJARO
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4790 CPU @ 3.60GHz (8 x 3870)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
0, 0, 1

Memory (System)
15.52GB (11.27GB free)

Process Argv
/a/bitbean/smg-auto/dealsmgr/api-server

Screen Reader
no

VM
0%

DESKTOP_SESSION
xfce

XDG_CURRENT_DESKTOP
XFCE

XDG_SESSION_DESKTOP
xfce

XDG_SESSION_TYPE
x11

Extensions (32)

Extension
Author (truncated)
Version

gitdiffer
aag
0.0.3

Bookmarks
ale
13.3.1

better-toml
bun
0.3.2

dart-code
Dar
3.48.1

flutter
Dar
3.48.0

vscode-eslint
dba
2.2.6

vscode-deno
den
3.13.1

githistory
don
0.6.19

permute-lines
ear
1.1.0

EditorConfig
Edi
0.16.4

prettier-vscode
esb
9.8.0

vscode-solution-explorer
fer
0.7.1

go
gol
0.35.2

css-to-jss
inf
1.0.7

plantuml
jeb
2.17.4

code-eol
jef
1.0.12

docthis
joe
0.7.1

docomment
k--
0.1.31

csharpfixformat
Leo
0.0.84

csharp
ms-
1.25.0

python
ms-
2022.14.0

vscode-pylance
ms-
2022.8.50

jupyter
ms-
2022.8.1002431955

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.9

remote-containers
ms-
0.251.0

remote-ssh
ms-
0.84.0

remote-ssh-edit
ms-
0.80.0

remote-wsl
ms-
0.66.3

vscode-remote-extensionpack
ms-
0.21.0

gitlink
qez
1.2.4

vscode-yaml
red
1.10.1

The Comments view only shows when an extension contributes either comments or commenting ranges.

The described behavior is how it is expected to work. If you disagree, please explain what is expected and what is not in more detail. See also our issue reporting guidelines.
Happy Coding!

I briefly scanned the information here and there to try and figure it out and so far I have been unsuccessful.

The comments that show in the Comments view are not inline code comments, but instead comments that show in a comments UI element, such as code review comments. You as a user cannot add such comments without first installing an extension, such as GitHub Pull Requests and Issues, that enables them.

Here's an example of what these comments look like:

Okay - thank you so much! I have all the Git/Github/CodeLense integration stuff turned off, so I wasn't able to see those.


Select time grain "Original Value" is not working on first attempt

The first attempt to choose "Original Value" as time grain results in "Day" being selected.
This is very confusing for both beginners for power users who might not even check teh result of their actions.
How to reproduce the bug

Go to Explore with any time-series chart. Make sure  that the set time grain is not "Day". Maybe set it to "Week" and then refresh the page.
Click on Time Grain
Select "Original Value"
See how the Time Grain is set to "Day" instead
repeat step 3
See how now the time grain is set as desired

Expected results
Selecting "Original Value" works on the first attempt.
Actual results
It takes 2 attempts after loading/reloading the page.
It seems that "Original Value" is only set correctly, if "Day" was set before.
Screenshots

Environment
We use the official Docker images of 1.4.1 and 1.4.2, respectively.

affected browsers:

Chrome 99.0.4844.84, 64 bit
Edge 99.0.1150.36, 64 bit
98.0.2, 64 bit

superset version: 1.4.1; 1.4.2
python version: python --version
node.js version: ??
any feature flags active:

    FEATURE_FLAGS = {
      # stable:
      "THUMBNAILS": True,
      "THUMBNAILS_SQLA_LISTENERS": True,
      "SQLLAB_BACKEND_PERSISTENCE": True,
      "ENABLE_TEMPLATE_PROCESSING": True,
      "DASHBOARD_CROSS_FILTERS": True,

      # experimental:
      "ALERT_REPORTS": True,
      "ALERTS_ATTACH_REPORTS": True,
      "DASHBOARD_NATIVE_FILTERS": True,

      # unclassified:
      "ENABLE_EXPLORE_DRAG_AND_DROP": True,
      "ENABLE_DND_WITH_CLICK_UX": True,

      # development:
      "DASHBOARD_NATIVE_FILTERS_SET": True
    }

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

The controls are a bit different now as of 2.0.1 and possibly earlier (you just (X) out the time grain, rather than selecting "Original Value") but in any case it seems to work just fine now, and I could not reproduce this issue. Closing as fixed, but let me know if you encounter it in newer versions. Thanks!


Hello, do you support Android IOS?

Hello, do you support Android IOS?

android   socket  and  ios socket   Support?  thinkyou

Yes. Support android socket and ios socket.

Hello, can I have an demo please? Thank you very much

我在手册里面并没有找到 安卓 和ios d的 demo   帮忙分享一个吧  QQ674483879

No demo.

那 能不能说一下   安卓需要用哪个库  才能连接到 workerman

Please use english, that can help all people to understand each other. Thanks.

Hello, can you tell me what code framework Android needs to use the workerman framework?

https://github.com/koush/AndroidAsync
https://github.com/daltoniam/Starscream
https://github.com/facebook/SocketRocket

Thank you very much!


feat: custom key for cache

When the user does not disable the use of cache (cache-disabled:true), the user can customize the cache key.
For example: https://github.com/wtto00/karma-android-launcher/actions/runs/6380253195/workflow
Here I am running 14 tasks simultaneously, and each task will execute the following shell commands:
# Install emulator tool
sdkmanager --install emulator
# The android-31 in each task is different, ranging from android-21 to android-34
sdkmanager --install "system-images;android-31;default;x86_64"
# Create emulator
avdmanager -s create avd -n android_emulator_name -k "system-images;android-31;default;x86_64"
# Start emulator
emulator -avd android_emulator_name -no-window -noaudio

However, all the cache keys for these tasks are the same, so the cache for the 'system-images' I installed is not saved, or only one of them is cached. They will overwrite each other.
In addition, each task will create an emulator, which is a time-consuming task to create and launch the emulator from scratch. If it can be cached, it will be much faster.
If it is possible to customize the cache key, all the data for different tasks can be cached, and the next execution will be much faster. It won't need to download and create the emulator from scratch every time.

I will release soon and include v3.
Thanks @wtto00

@wtto00
I released v3.2 that include this change!
https://github.com/amyu/setup-android/releases/tag/v3.2


Fix ansible modules documentation URL

Previous URL now gives a 404. The content at the new URL is slightly
different but roughly similar.

Dear @rezib
Nice catch ! Thanks a lot for this PR 😊
Merging right now. This will be propagated to online documentation at next build, probably next week.
Have a nice weekend !

You're welcome, thanks for the quick merge!
Bye


Make drag n drop smoother

Following this example (source) I made the drag n drop smoother
Check it out

Hey @teldosas: This is awesome! I remember wanting to implement this, but at that time ReactDND didn't have this example. Can you please resolve the conflicts before I can review this? I'd suggest not adding any file from dist-* and dist folder in this PR.

@prakhar1989 It should be ready now :)

I wish the ReactDND examples included some example for testing these interactions. The current test suite lacks any unit tests for this behavior 😥

It should be ready :)

Great! Thanks for the addressing the comments!


Issue branch

I am trying to import some external APIs by adding dependencies in the build.gradle file, and import changes. The tests will all pass without adding the dependencies(before the last commit) initially. The issue is that after I adding the dependencies, and no modifications to other code, but some junit tests will fail. But if I do manual testing, there seems no such errors.

Hi @crizyli, thank you for submitting a PR! Just to confirm, are you sure this is the correct repo you are trying to merge your changes into? Maybe nus-cs2103-AY1718S2 or CS2103JAN2018-W15-B1 was what you were looking for? ;)

Lewis Koh<EMAIL_ADDRESS>周四下午1:57写道：

Hi @crizyli https://github.com/crizyli, thank you for submitting a PR!
Just to confirm, are you sure this is the correct repo you are trying to
merge your changes into? Maybe nus-cs2103-AY1718S2
https://github.com/nus-cs2103-AY1718S2 or CS2103JAN2018-W15-B1
https://github.com/CS2103JAN2018-W15-B1/main was what you were looking
for? ;)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/se-edu/addressbook-level4/pull/848#issuecomment-373269346,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbUKdB-R4P1frKe21gyoO4DuLIb1Na0Fks5tegKtgaJpZM4Srj6p
.

Oh, sorry! I am trying to merge the changes into CS2103JAN2018-W15-B1
https://github.com/CS2103JAN2018-W15-B1/main Can I re-create the PR and
edit the link in the issue?

We do not have access to the other repo. Please close this PR and issue, and resubmit it to the correct location, thank you.

Do I need to close the issue too? Because Prof Damith asked me to post an issue in the upstream repo se-edu/addressbook-level4 and ask for help from the  AB4 developers. Maybe I should create the PR to CS2103JAN2018-W15-B1
https://github.com/CS2103JAN2018-W15-B1/main instead?

Do I need to close the issue too? Because Prof Damith asked me to post an issue in the upstream repo se-edu/addressbook-level4 and ask for help from the AB4 developers. Maybe I should create the PR to CS2103JAN2018-W15-B1
https://github.com/CS2103JAN2018-W15-B1/main instead?

Yes @crizyli you need to create a PR against your team's fork CS2103JAN2018-W15-B1 and the PR should contain only the code that you modified when trying to add dependencies. After creating that PR, you can post the link to that PR when you post your problem here.

Closing this PR.


gpu: add process GPU utilizationon MIG slices indexed by the parent GPU

Fix #1293
Currently, the nvml and dcgm libs do not provide the power consumption of individual MIG slices, only the total GPU power consumption for all MIG slices combined.
Giving that, the problem in #1293 is because the existing code wrongly assigns GPU power consumption to each MIG slice, because it is using MIG ID instead of the GPU ID to set the process GPU utilization.
Note that, the Ratio power model's logic involves calculating the ratio utilization of a process and multiplying it by the resource power consumption. So, with each process GPU utilization, total GPU utilization, and GPU power consumption, we can calculate the GPU power consumption for each process.
To fix the problem in #1293, we need to index the process GPU utilization with the parent GPU ID (not with the MIG ID). This way, the Ratio power model can correctly distribute the parent GPU power consumption among the processes in the MIG slices based on their GPU utilization.
Future work:

[ ] add the MIG info as labels to the process's GPU utilization to be able to add this label to the prometheus metrics

To fix the problem in https://github.com/sustainable-computing-io/kepler/issues/1293, we need to index the process GPU utilization with the parent GPU ID (not with the MIG ID). This way, the Ratio power model can correctly distribute the parent GPU power consumption among the processes in the MIG slices based on their GPU utilization.

I don't think this is sufficient. The problem in #1293 is about multiple GPUs. Each GPU may report different energy readings. The current ratio model uses aggregate utilization and energy among multiple GPU devices, this leads to problems when the GPUs are configured differently in terms of cooling, frequency, and SM spec.

@rootfs, you're correct.
Even if we calculate the MIG power before applying the Ratio Power Model, we'll still face this issue that the Ratio Power model uses the aggregated system data....
Can we merge this PR and fix the problem to calculate the Ratio Per GPU later?

@rootfs, you're correct.
This PR does not fix the problem in #1293
This PR fix some other problems.

@marceloamaral let's merge this one first. When you have time, can you also fix the multi GPU ratio in #1292?


Promise support

I would be nice to have the Promise support:
await generify(source, dest, data)
await generify(source, dest, data, onData)

The promise will be resolved after all the onData (if any)

I would improve the interface a bit and maybe support also async iterators. wdyt?

Would you like to take a stab at a PR?


Dear mise users

This is an issue for when mise users create issues or pull requests to aqua-registry.
We're aware that mise uses aqua-registry as backend, but aqua-registry is a registry for aqua, not mise.
So we don't accept bug reports that we haven't reproduced using aqua.
If you face any problem using mise, please ask mise community for help first.
Before creating issues or pull requests to this repository, please confirm that you can reproduce the problem using aqua.
And if you can reproduce them using aqua, please write how to reproduce it in issues or pull requests.

aqua version
OS and CPU Architecture
aqua.yaml
command and outputs
expected behaviour
actual behaviour

Thank you.

I am not happy this is happening to you but I'm also not sure what to do about it. You certainly should not be needing to deal with mise-specific issues though.


Why there is sometimes appearing error on tau[0] == tau[1]

I want to ask when I set optimization = 2, why there will be sometimes appearing errors about tau value.
Thank you!

Interesting, it would be nice if you could send me some sample data which raises this issue. tau1 and tau2 should always be different, but in this case they end up being set at the same value by the optimiser. I can also see that there is a bug in the output (the error message is not formatter properly).

Hi, I didn't manage to reproduce the issue; this is what I did
checked python version:
python --version  (or python3 --version)
should be 3 and above, mine is Python 3.7.0
cloned repository as ledapy
git clone https://github.com/HIIT/Ledapy ledapy
copied your sample in ledapy as test.csv
then in python
import ledapy
import numpy
import scipy.io as sio
import matplotlib.pyplot as plt
from numpy import array as npa
filename = 'ledapy/test.csv'
sampling_rate = 100  # note: what is the sampling rate?
csv_data = numpy.genfromtxt(filename, delimiter=',')
phasicdata = ledapy.runner.getResult(csv_data, 'phasicdata', sampling_rate, downsample=4, optimisation=2)
plt.plot(phasicdata)
plt.show()

I get a result that seems correct

The sampling_rate is 128
and I set the downsample = 8
Sorry I forget to give you the information

Ok I see, it may take me a long time to debug this issue, also considering that I don't have a Matlab license right now to compare results with Ledalab.
A temporary solution could be to use a downsample value of 2, which works in this case. The downsample value can be adjusted depending on the size of the input (the data segment of the sample data is already quite short, so it doesn't need much downsampling).

Hi. Thanks for the port, much appreciated!
I also have problems with Tau1 == Tau2. I have a GSR signal with duration 117sec with sampling rate 125Hz. For this signal I tested deconvolution with varying sampling rates and optimization iteration counts, here is the result:
Sampling rate 5Hz and optimizations 1: Success
Sampling rate 5Hz and optimizations 2: Success
Sampling rate 5Hz and optimizations 3: Success
Sampling rate 5Hz and optimizations 4: Success
Sampling rate 8Hz and optimizations 1: Success
Sampling rate 8Hz and optimizations 2: FAILED! (tau1 == tau2 == 1.000000)
Sampling rate 8Hz and optimizations 3: FAILED! (tau1 == tau2 == 1.000000)
Sampling rate 8Hz and optimizations 4: FAILED! (tau1 == tau2 == 1.000000)
Sampling rate 12Hz and optimizations 1: Success
Sampling rate 12Hz and optimizations 2: FAILED! (tau1 == tau2 == 1.000000)
Sampling rate 12Hz and optimizations 3: FAILED! (tau1 == tau2 == 1.000000)
Sampling rate 12Hz and optimizations 4: FAILED! (tau1 == tau2 == 1.000000)
Sampling rate 17Hz and optimizations 1: Success
Sampling rate 17Hz and optimizations 2: FAILED! (tau1 == tau2 == 1.000000)
Sampling rate 17Hz and optimizations 3: FAILED! (tau1 == tau2 == 1.000000)
Sampling rate 17Hz and optimizations 4: FAILED! (tau1 == tau2 == 1.000000)
Sampling rate 21Hz and optimizations 1: Success
Sampling rate 21Hz and optimizations 2: Success
Sampling rate 21Hz and optimizations 3: Success
Sampling rate 21Hz and optimizations 4: Success
Sampling rate 24Hz and optimizations 1: Success
Sampling rate 24Hz and optimizations 2: FAILED! (tau1 == tau2 == 1.000000)
Sampling rate 24Hz and optimizations 3: FAILED! (tau1 == tau2 == 1.000000)
Sampling rate 24Hz and optimizations 4: FAILED! (tau1 == tau2 == 1.000000)

I simply interpolated my original signal for different rates and then ran command:
phasic = ledapy.runner.getResult(yy,'phasicdata',rate,downsample=1,optimisation=N_opt)
where N_opt was 1-4. So there is something weird going on with the algorithm. I'm running Python 3.7.3.
In case you/someone wants try, there are data for a good and a bad case:
Sampling rate 21Hz and optimizations 3: Success:
https://drive.google.com/open?id=18SQmjUokPcLFeRHna_hawkEypvWoGOj5
Sampling rate 24Hz and optimizations 3: FAILED!:
https://drive.google.com/open?id=1miaOM0VwC72wmFRJh3K95yutAeyWBQh5

Hello, I think I found the issue, the latest version (1.1) should now work, at least with the files you provided. You could run your full test suite to make sure the results are now as expected.

All my tests for this data passed using version 1.1. However, I still get warning
analyse.py:42: RankWarning: Polyfit may be poorly conditioned  [x, newerror, step] = cgd_linesearch(x, olderror, direction, error_fcn, h)
for some sampling rates, which indicates some numerical stability/convergence issues. Sampling rate still have some unexpected effects on computations.
But there are no errors and results seem ok, so this issue appears fixed for now. Thank you!

Alright, I'll have a look. Does the final output (returned by getResult) look particularly different when you get that warning?

The data seems similar so it's not bad despite getting the warning.
Here are couple of plots for the "FAILED" data above. I ran phasic computing using different sampling rates (identical input data) with optimization steps set at 3 and no downsampling:

First shows the Pearson correlations and the second estimated signals with all "bad rank" result highlighted [I also shifted signal by stepping 0.015 in y-dir for easier visualization].

Very nice, thanks! I'll have a look to see how Ledalab deals with that fitting. In the meantime I can close this issue

Hello, I did investigate, actually I found a slight issue in that function when comparing with Matlab. I updated the package to version 1.2. Not sure if it will fix the warning, but at least now that function behaves the same as in Matlab

I think you fixed it, no more warnings and the results seem more consistent (higher cross-correlations). Here are the same plots with the latest version:

Alright, thank you for running the tests!


Does Spring cloud kubernetes service discovery works for selected namespaces?

Currently I see, spring.cloud.kubernetes.discovery.all-namespaces=true -> discovers the services from all the namespaces in the entire cluster.
spring.cloud.kubernetes.discovery.all-namespaces=false -> discovers the services in the current namespace, i,e the namespace in which the application has been deployed.
Is there any property something like - spring.cloud.kubernetes.discovery.namespaces= namespace-a, namespace-b, namespace-c
So that I can discover services only in specific namespaces as needed.

correct, either current namespace or all namespaces are currently supported, no selectable ones yet.

@Haybu is there a way that one could customize / extend the Kubernetes Client with code to enforce certain namespaces?


[GHSA-fwvg-2739-22v7] Miniflare vulnerable to Server-Side Request Forgery (SSRF)

Updates

Affected products

Comments
Harassment

Hi there @mrbbot and @mskowroncf! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our highly-trained Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory


Adding Initial Markdown Support

Hello! This is a pull request that fixes GMOTGIT/GMOT-SSG#8.
The changes that I made were:

Update the file handling so that md files can be accepted and written to HTML
When creating paragraphs from an md file, check for markdown syntax using regex and replace the markdown syntax with the appropriate HTML
Update the README with information about how to use the markdown feature and an example

Please let me know what you think of the changes and if there is anything you would like added/removed.

Hi Gustavo, thanks for the feedback. I made some changes based on your comment, could you take a look and let me know what you think? Thanks.


Sign In is not working

In the recent version 4.8.1 of Emulator,
I'm running the Authentication sample Bot, in that while Signing in , the emulator is not redirecting to a browser window to generate the token, instead it is opening the explorer window.
Is there any fix or workaround for this issue.
because of this issue I'm unable to debug my project, also this issue is not there in the previous version,

Hey @dmvtech could you take a look at this one? Thanks

@JeganBas If not already done, can you confirm this reproduces after selecting Clear state from the File menu?
If so, can you please grab a fiddler trace. You can send to me at<EMAIL_ADDRESS>

Even after clearing the state it's the same results,  the emulator keep opening the explorer window instead of browser while signing in.  I'm unable to use fiddler here. Suggest me a different solution please

I am able to reproduce. To clarify; it is opening File Explorer (explorer.exe).

Looks like a regression #1397
@tonyanziano

@dmvtech  yes  , its opening the File explorer only, what could be the fix for this ?

Thanks for looking into this @dmvtech .
I'll dig around a bit once I get an auth sample working. It's possible that something changed inside Web Chat and that's the cause, but that's just a hunch at the moment.

I tried to reproduce it, but I can't on my end 🤔

This is on v4.8.1.
@JeganBas if you downgrade to a previous version, like 4.8.0 or 4.7, do you still experience this issue? If we could narrow it down to a certain version then we can look at the delta in the past commits and see if there is anything that might have caused it.

Hi @tonyanziano , I've downgarded to Emulator 4.7 and now i can able to sign In, so the problem is only in 4.8.1

Thanks for testing that out @JeganBas ,
We can take a look at what changed between 4.7 and 4.8.1 and see if there are any obvious commits that might have caused this behavior.

hi, if you set the path to ngrok in the emulator settings, the Oauth Card will show the expected result: the prompt to login:

After multiple tests, I noticed the issue happens when the path is empty.

I'm now able to reproduce this reliably.
Video for reference
This is indeed a bug in the sense that clicking the sign-in link should not open the file explorer, however, that being said, the following still remains true:

When reproducing this bug, as you can see in the video I posted, as soon as the sign-in card is generated, an error message is printed out in the Emulator log panel stating that the Emulator was unable to start up ngrok due to the ngrok path being wrong or blank in the app setting page. This is the correct behavior, and receiving an actual OAuth token without configuring ngrok is still impossible and will remain this way.

When ngrok is not configured, clicking on the sign-in card should still cause the Emulator to give the bot a token, however it will be an emulated token (not generated by the token service) prefixed with emulatedToken_ that the user can use for testing purposes

Once I fix this bug, the user will once again be able to get an emulated token from the Emulator when ngrok is not configured

This fix will be in the next nightly build (usually gets published around 11 PM PST).

@tonyanziano thanks for updating, i will  validate the new build and update you


Filter Templates by Cicero Version

Is your feature request related to a problem? Please describe.
The TemplateLibrary component should have the capability to filter by cicero version
Describe the solution you'd like
The component should accept a semver expression as a prop to dictate which cicero version to filter
Describe alternatives you've considered
Open to input
Additional context
N/A

Do you mind if I try to tackle this issue?

Please use this package to filter versions:
https://www.npmjs.com/package/semver
A semver.satisfies filter will ensure that the user can specify the versions they would like.

@irmerk This issue should be moved to Cicero (Transfer issue button works great for things like that) with a alpha-0.20 label.

I believe @dselman was discussing this as a feature in TemplateLibrary

I believe @dselman was discussing this as a feature in TemplateLibrary

I understood TemplateLibrary to be: https://github.com/accordproject/cicero/blob/master/packages/cicero-core/src/templatelibrary.js

Ah no, it's linked in the original post: https://github.com/accordproject/cicero-ui/tree/master/src/TemplateLibrary

Ah no, it's linked in the original post: https://github.com/accordproject/cicero-ui/tree/master/src/TemplateLibrary

Hence my question, is this Cicero UI? Cicero? or both?

@dselman and @DianaLease may have input.

Refer to #181 before starting this issue.


Create separate messages source

Move C based message handling to a separate source file.

This is done.


Post-README tweaks

Sync templating with desired state of operator repositories.
Add fix to non-functional README-checking action.
Add minor tweaks to fix issues with rendered READMEs.

Thanks! Yes, the variable will be set in single operators and is not further used here.


How does the loss function train the network?

Hello, how do the two Lcls and Lreg loss functions in the paper train the network? Are they separate training networks? Or do two functions turn into one to train the network?

It's the same as faster R-CNN. Please refer to it for details.
BTW, this repo is deprecated. Please use our new repo: https://github.com/yihui-he/KL-Loss


Right click on a draggable point keeps the point in dragging mode untill click

same as this one https://github.com/apache/echarts/issues/14109.  it says the issue may got fixed in echart v5.4.0  but I do have this problem. I am using volkovlabs-echarts-panel v4.3.0.

This issue is still open, and it's not clear if it was fixed. You may do your own event handling to override the default if it's the issue.
@mikhail-vl  got it. I will try to override the default action. Thanks again for your help.

@alpglide Also, please reply in the ECharts issue, that it's still an issue for them to investigate.

Let me know if there is anything else.


Update README

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

Comparison is base (2d1b905) 92.99% compared to head (82b2757) 92.99%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #6   +/-   ##
=======================================
  Coverage   92.99%   92.99%
=======================================
  Files           6        6
  Lines         714      714
=======================================
  Hits          664      664
  Misses         50       50

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


Allow EDTF date to refine basic date with time

If a start_date or end_date has been qualified to the fullest extent possible (down to the day), treat it as a daylong range in EDTF when comparing it to the start_date:edtf or end_date:edtf tag, and compare it both ways to see if there’s any overlap.
Originally, I wanted to just append TXX:XX to the basic date, but either EDTF.js or the EDTF specification apparently doesn’t allow unspecified digits (X) within time specifiers. It also wasn’t practical to ignore the time when parsing the EDTF date, because the EDTF date can contain a whole range or set of dates with time precision, not just one. Fortunately, at least EDTF.js seems to recognize 24:00:00 as a synonym for midnight that does not roll over into the following day, per ISO 8601-1:2019/Amd 1:2022.
Fixes OpenHistoricalMap/issues#764.

@erictheise let me know if there’s anything I can do to make this easier to review. Thanks!


Fixed #3 : Migrated to Requests

Using browser framework to view a website is a bad idea. (heavy framework for just requesting a website)

Changes Proposed :

Migrated the code to use requests module.

Ping @SubhrajitPrusty

Ping @SubhrajitPrusty :1st_place_medal:

Why not we make two versions, one with requests, another with broswer frameworks. Will you make a backup branch then accept this PR so that I can work on this?

Second PR will include both the things you asked :)

Deleting the PR as the maintainer is good with wget.


merge from template-node-app

README.md

docs(README): update from template

:tada: This PR is included in version 2.0.165 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


How to run Wox's Web Search plugin w/ customizable keyword and {q} param replace?

📝 Reuse/port Wox's Web Search plugin w/ action keyword and {q} param
This should work very similar to Wox's Web Searches plug-in, including:

customizable action keyword
URL w/ param replacement of {q} for the search
customizable icon

As Wox user for the last 10 years the web searches plug-in is 90% of why I love Wox and haven't yet switched to PT Run. I'd be happy to implement it as a plug-in if it's not in the works. I ❤ whoever pulled in Wox to PT btw.
Also, it's highly possible it's already there and I'm missing it?

If you'd like to see this feature implemented, add a 👍 reaction to this post.

tracked by #3245


how to create an sleep func ?

    async sleep(ms: number) {
        let timeout = GameRules.GetGameTime() + math.min(ms, 1 / 30) / 1000
        let routine = coroutine.running()
        this.queue.set(routine,timeout)

        while (GameRules.GetGameTime() < timeout) {
            coroutine.yield(routine);
        }

        this.queue.delete(routine)
        return Promise.resolve()
    }

im try to  yield  the coroutine, and then resume it in this.queue on game global Think. but it not work . have you any idea?

It should be something like this:
declare function now(): number;

// NOTE: using a map here, but you probably want a min-heap for performance reasons
const sleeps = new Map<() => void, number>();

async function sleep(ms: number): Promise<void> {
    // Create promise and add its resolving callback to a map
    return new Promise(resolve => {
        sleeps.set(resolve, now() + ms);
    });
}

function think(): void {
    // Iterate over all active sleeps
    for (const [unsleep, timeout] of sleeps) {
        // Check if this sleep has expired
        if (now() >= timeout) {
            // Trigger the promise resolution
            unsleep();
            // Remove sleep from map
            sleeps.delete(unsleep);
        }
    }
}

But since I recognize you are doing this for Dota I would recommend you just use the Timers library instead:
async function sleep(ms: number) {
    return new Promise(resolve => {
        Timers.CreateTimer(ms / 1000, () => {
             resolve();
        });
    });
}

Thanks  alot, your scheme I think is better than the timer library, thank you


Problem with os.clock and schedulers

Hey,
I wanted to measure the time of how long the code needs and tried to use os.clock. It looks like that os.clock and schedulers aren't working together nicely.
I created for quicker testing a scheduler that runs 5s later
Before starting: 15.744292
-- run scheduler 5s later
Written from scheduler: 15.863173

How you see, there is a small problem with them.

What was the call to ScheduleTask? It takes the time to wait in Ticks, which is ~1/20 of a second.

Sorry, the 5s are the time to wait - 100 ticks.

As @worktycho already said - show us your code

Here is the code
function Initialize(Plugin)
    Plugin:SetName("TestPlugin")
    Plugin:SetVersion(1)

    PLUGIN = Plugin

    -- Command Bindings
    cPluginManager.BindCommand("/test", "test", CmdTest , " test")

    LOG("Initialised " .. Plugin:GetName() .. " v." .. Plugin:GetVersion())
    return true
end

function CmdTest(a_Split, a_Player)
    START = os.clock()

    local CALLBACK =
            function(a_World)
                a_Player:SendMessageInfo("start = " .. START)
                a_Player:SendMessageInfo("current = " .. os.clock())
            end

    a_Player:GetWorld():ScheduleTask(100, CALLBACK) -- run task in 5s
    a_Player:SendMessageInfo("Running...")
    return true
end

I have tested similar code (in the Debuggers plugin: https://github.com/mc-server/MCServer/blob/master/MCServer/Plugins/Debuggers/Debuggers.lua#L1609-L1620 ) and it works alright for me.
But then I remembered that clock() behaves differently on Windows (my tests) and on Linux (Your tests?) - on Windows it returns the wall time elapsed since program start, while on Linux it returns the amount of time that the program has spent actively using the CPU since its start. This means that you should not use os.clock() for timing purposes, you should use os.time instead.
If you change your code to work on a console command and log its output to the console, you should be able that the log line timestamps are the specified time apart. Or just test the Debuggers plugin's sched console command.

http://www.lua.org/pil/22.1.html


Is it possible to set BarBackgroundColor to custom color

Hi Is there a way to make the BarBackgroundColor to work
e.g.
bottomBarPage.BarBackgroundColor = Color.Orange;
Currently I am using
bottomBarPage.BarTheme = BottomBarPage.BarThemeTypes.DarkWithAlpha;

I am new to .NET sorry if this is a stupid question

found it.
in style.xml
#000000


Beginning of project modularization using SPM

Codecov Report

Merging #31 (26729fe) into develop (561eed3) will increase coverage by 0.04%.
The diff coverage is 80.00%.

@@             Coverage Diff             @@
##           develop      #31      +/-   ##
===========================================
+ Coverage    77.12%   77.16%   +0.04%
===========================================
  Files           46       46
  Lines         1132     1134       +2
===========================================
+ Hits           873      875       +2
  Misses         259      259

Impacted Files
Coverage Δ

...enes/CollectionView/CollectionViewController.swift
45.94% <ø> (ø)

CV/Scenes/CollectionView/CollectionViewModel.swift
100.00% <ø> (ø)

CV/Scenes/CollectionView/DataSource.swift
100.00% <ø> (ø)

...enes/CollectionView/CollectionViewModelTests.swift
100.00% <ø> (ø)

...es/CollectionView/Mocks/ViewModelClientMocks.swift
100.00% <ø> (ø)

CVTests/Scenes/Image /ImageInteractorTests.swift
100.00% <ø> (ø)

CVTests/Scenes/Image /ImagePresenterTests.swift
100.00% <ø> (ø)

...Tests/Scenes/Image /Mocks/ImageProviderMocks.swift
85.71% <ø> (ø)

Modules/Sources/Common/URL.swift
100.00% <ø> (ø)

Modules/Sources/Networking/ViewModelClient.swift
50.00% <ø> (ø)

... and 6 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


BackdropProps is missing from Dialog API docs

BackdropProps is not listed on https://material-ui.com/api/dialog/

[ ] The issue is present in the latest release.
[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

Expected Behavior 🤔

Steps to Reproduce 🕹

Steps:

Context 🔦

Your Environment 🌎

Tech
Version

Material-UI
v4.?.?

React

Browser

TypeScript

etc.

Any other props supplied will be provided to the root element (Modal).

Duplicate of #18288


Optimize byte array encoding

Copy the entire byte array instead of iterating over one byte at a time.  This makes CBOR much much faster at encoding messages with large binary blobs.
Benchmarking on this data, a ROS2 sensor_msgs/Image of 640x480 RGB:
var benchmarkData = {
  header: {
    stamp: {
      sec: 1,
      nanosec: 1e8
    },
    frame_id: ""
  },
  height: 480,
  width: 640,
  encoding: "RGB",
  is_bigendian: 0,
  step: 640,
  data: Uint8Array.from(Array.from({length: 640 * 480 * 3}, () => 128))
};

master
CBOR encode: 20.234ms
fast_byte_array_encoding
CBOR encode: 1.288ms

Coverage increased (+0.003%) to 99.265% when pulling 1a6de156903322c39734c613def5f19e00d1a934 on mvollrath:fast_byte_array_encoding into 65dc49611107db83aff8308a6b381f4d7933824b on paroga:master.


Create option to specify type for Query service

This change enables the specification of a service type for the query service created by a Jaeger resource.  It addresses an issue noted by some commenters on #990, though I don't believe it fixes the original issue raised there.

This is good to be merged, pending only a godoc for the possible values for the new property.

Thank you for your contribution!


Synology Note Station Client 2.2.1-553

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.0 schema?

Microsoft Reviewers: Open in CodeFlow

Looks like I have to do the same changes as #19681 anyway.


i18n (Spanish translations)

Right now, part of the website is translated in Spanish.
For example: https://techqueria.org/es/contact/
We would like to get the entire website translated.
How to contribute to translations are explained here: https://github.com/techqueria/website/blob/master/.github/CONTRIBUTING.md#translations
If you are bilingual and would like to contribute, let us know which page you would like to translate so we can create a separate issue to track the progress.

✅ By submitting this issue, I have verified the following

[x] Checked to see if the issue has already been discussed before 🤔
[x] Made sure enough details were provided if proposing a new feature 🔍

Hi @fvcproductions, I'm going to help translating some of the content. 👍

Hey, @fvcproductions! I'd love to get started contributing with some translations. I can work on all of the "Support Us" pages.

@omarnyte Brilliant - you can open a PR when you're ready and feel free to ask me any questions you have on how to submit translations

@fvcproductions

@fvcproductions, I'm having some trouble hitting /es/donate-space/, /es/volunteer/, and /es/supporters/. In production, those routes display a 404. Locally, they show a blank page with a Cannot GET error.  /es/support-us/ works as expected. I've tried adding the URLs to the sitemap on config.toml, but I'm still not hitting any of those routes. How can I configure those URLs?

Hola, I would love to contribute to this project. Any other words of advise to get started ?
Thanks,

@cyberfernandez Thanks for reaching out - contributing guidelines are here so I would use that to get started https://github.com/techqueria/website/blob/master/.github/CONTRIBUTING.md#translations and you're welcome to ask me any questions!


Question: Can I use rzls as Language Server (LSP) in other IDEs like Omnisharp?

I couldn't find anyone in the community using the razor language server project rzls as LSP server even though it exists in this repo also couldn't find any documentation on how to use the language server without VS-Code. I've limited knowledge in LSP so I want to ask if rzls is compatible with LSP? Can I use it in any LSP Client (e.g., Neovim) like I use omnisharp?
I use omnisharp server in Neovim like that omnisharp_binary --languageserver --hostPID neovim_PID.
Appreciate the time you took to answer the question :)

rzls is compatible with LSP. And the code in this repo is MIT licensed and can be used in any way that is compatible with that license. Note that binaries distributed with C# - Visual Studio Marketplace are released under a different license: License Agreement | Visual Studio Marketplace.


UI: Rely on property ordering from the service

Currently in CategoryUtil, an ordering value is assigned to the various properties within a category.  Per Rene, the service is now returning the properties in the desired order within each category.  We should be able to eliminate the optional attribute 'orderInCategory' from the ConnectorProperty model - and just assume that the ordering returned from the service is correct, and display the category properties in that order.  The assignment of ordering in CategoryUtil can be removed.

PR #84 is pending

PR merged


Create cookbook and write usage recipes

Usages that need documenting:

[ ] Creating masks from vector labels and csvs
[ ] Creating and visualizing graph objects
[ ] Convert a pixel-coordinate csv to a georegistered object
[ ] Convert a georegistered geojson to a pixel-coordinate csv

May make sense to wait to do these until the entire 0.1.0 codebase is complete so I can add more to the list.

Issue moved to CosmiQ/solaris #21 via ZenHub


Max coins to hold & max time to hold

Suggestion for a parameter that sets the max number of different coins that can be held, so that the bot not runs amok and spends all your funds on a ton of coins.
Also a parameter that sells the coin if held for longer than x minutes/hours could be useful for getting rid of stale coins that does not move.

This would be awesome

100% Agree with this. I have made mods to my clone of the code to achieve this. FYI the Binanace timestamp is ms not s; so divide by 1000.

Only max time left to code


Added application/octet-stream

Issue Link :link:

https://github.com/Alamofire/AlamofireImage/issues/370
Goals :soccer:

Add application/octet-stream to acceptable content types. This causes iOS 13 to behave like previous versions.
Implementation Details :construction:

Testing Details :mag:

Hi @rafaelc0sta, thank you for taking the time to put this together. I have just pushed f48f8cfe into master while maintaining your attribution. It will go out as part of AFI 4.0.0 here shortly.
Cheers. 🍻


No line breaks after header

Line breaks are not inserted after a header.
The webdemo and preview image show a different behaviour, no idea what's going on here.

That's very weird. I also have no idea what's going on and I cannot reproduce the issue.


IDFA to GAID

I updated the payload key to gaid as well, but just wanted to verify thats ok on your data processing side?
closes #62

Yes, the update of the payload key to gaid is fine on the Lytics side.


Register package

@JuliaRegistrator register()

Registration pull request updated: JuliaRegistries/General/89637
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" c0e96cb950881fc4f3386301693c354fcf8c0861
git push origin v0.1.0


Redesign of the category properties config

Codecov Report

Merging #3 (248797e) into main (76188d9) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##                main        #3   +/-   ##
===========================================
  Coverage     100.00%   100.00%
  Complexity        35        35
===========================================
  Files              8         8
  Lines             99        99
===========================================
  Hits              99        99

Flag
Coverage Δ

backend
100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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 76188d9...248797e. Read the comment docs.

I've adapted the distance of the labels and highlighted the "Add new Category property" in a different colour with a different margin.