Add log message to show running or fail status exactly

Add or fix some log messages

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

Project coverage is 49.42%. Comparing base (7f010a1) to head (27fe417).
Report is 1 commits behind head on main.

Files
Patch %
Lines

src/storage.rs
0.00%
2 Missing :warning:

src/ingest.rs
50.00%
1 Missing :warning:

src/peer.rs
50.00%
1 Missing :warning:

src/publish.rs
50.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #699      +/-   ##
==========================================
- Coverage   49.44%   49.42%   -0.02%
==========================================
  Files          29       29
  Lines        8313     8319       +6
==========================================
+ Hits         4110     4112       +2
- Misses       4203     4207       +4

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


pipeline test

JIRA link (if applicable)
N/A
Change description
pipeline test
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

pr not needed, pipeline works


Datapoint Graph: Provide option to download data in CSV or XLSX

Currently, widget provides a way of saving it as image.
A good improvement will be to allow saving it as CSV, XLSX which can be used for further analysis or for sharing within teams.

We will integrate a export component into core and then will integrate it here.


Increase VirtualizedList batch size on iOS and Android

Details
I used React Devtools to profile ReportScreen rendering and find places where we can save rendering time.
Increase VirtualizedList batch size (https://github.com/Expensify/App/pull/19345/commits/19234f2cd6fc69426ec4bfaad2f11ee58e6920b4)
Using a low batch size can be good for responsiveness but cause a lot of excessive rendering. This is because each batch render will cause all items in the list to re-render. We do have memoization setup to minimize this cost, but it is still not small. Note that rendering cells is pretty fast and that rendering a batch of 10 is almost the same speed as rendering a single one (this is because all items still have to render on each batch).
This is all renders from the VirtualizedList component when loading a ReportScreen.
Before:

Total time: 2034 ms
Number of renders: 40
Average render time: 50.85 ms
Max render time: 76.7 ms
After:

Total time: 464.1 ms
Number of renders: 10
Average render time: 46.41 ms
Max render time: 100.9 ms
With this change we save 1569.9ms (77% reduction) in total time spent rendering, with similar average render time at the cost of slightly higher max render time (100 ms vs 76 ms, which is still acceptable).
Fixed Issues

$ https://github.com/Expensify/App/issues/15163
PROPOSAL: N/A
Tests

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

@mountiny Update to only set batch size on web. I moved the batchSize prop to index.js which is only used for web.

Thanks! asking for a C+ review

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/mountiny in version: 1.3.73-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 1.3.73-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/mountiny in version: 1.3.74-0 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/chiragsalian in version: 1.3.74-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Can api listen my own message ?

Hello,
When I reply to others, I would like to log my reply message
Is it possible to make this work ?
Since the api can listen others message, the behaviors are the same, right ?
Thanks.

Hey! Yes you can, take a loot at selfListen inside setOptions. Just do api.setOptions({selfListen: true}).

Thanks a lot :+1:

Hi ! Is it possible to listen what time do I read others' messages ?
Thanks.

Set listenEvents to true and check for the read_receipt event.

Yes. I have set both of listenEvents and selfListen to true. It's ok to listen other people's read_receipt, but it's not ok when I read other people's message. It could not listen my own read_receipt.

Why do you need to know when you read your own messages?

Currently I am developing a messaging mobile app based on facebook-chat-api.
This app could give you a push notification which indicates someone's emotion (e.g: happy, angry, sad ... ) when someone sends a message to you.
The reason that I develop this app is for researching some HCI issues, so I would like to log the users read time and response time to see how this functionality influence users :-)

I think you have some misunderstanding.
I want to know when do I read "other person's" messages.

Hi , I have another question.
Is it possible to login by facebook login api (e.g. user just simpily clicks facebook login button) without giving email and password to access the facebook-chat-api functionality ?
Thanks

No

You know when you read other people's messages based on how the user interacts with the UI if I'm understanding correctly. You're in control of how the user sees the message. I don't see why this has to do with this api.
As @Schmavery said, this is a hard limitation enforced by FB. We can't bypass it.


Literature Survey Summary

Steps for simple literature reading

[ ] Collect 5 most recent Multi Object Tracking papers with highest impact from papers with code
[ ] Read 5 paper abstracts
[ ] Read 5 paper introductions
[ ] Read 5 paper conclusions
[ ] Summarize each of the 5 papers
[ ] Choose most relevant 2 papers for the next week reading

Papers found on PWC Multi Object Tracking state of the art leader-board

[ ] https://paperswithcode.com/paper/a-simple-baseline-for-multi-object-tracking
[ ] https://paperswithcode.com/paper/towards-real-time-multi-object-tracking
[ ] https://paperswithcode.com/paper/detect-and-track-efficient-pose-estimation-in
[ ] https://paperswithcode.com/paper/a-baseline-for-3d-multi-object-tracking
[ ] https://paperswithcode.com/paper/real-time-multiple-people-tracking-with


Different from pandas index at read_csv in case of specified names

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Any
Modin version (modin.__version__): 0.8.0
Python version: 3.7.5
Code we can use to reproduce:

if __name__ == "__main__":
    import pandas
    import modin.pandas as pd
    from modin.pandas.test.test_io import (
        create_test_pandas_dataframe,
        TEST_CSV_FILENAME,
        df_equals,
    )

    create_test_pandas_dataframe().to_csv(TEST_CSV_FILENAME)

    md_df = pd.read_csv(TEST_CSV_FILENAME, names=["c1", "c2", "c3", "c4", "c5"])
    pd_df = pandas.read_csv(TEST_CSV_FILENAME, names=["c1", "c2", "c3", "c4", "c5"])

    try:
        df_equals(md_df, pd_df)
    except Exception as e:
        print(e)  # DataFrame.index are different

    md_df = pd.read_csv(
        TEST_CSV_FILENAME, names=["c1", "c2", "c3", "c4", "c5"], usecols=[0, 1, 2, 3, 4]
    )
    pd_df = pandas.read_csv(
        TEST_CSV_FILENAME, names=["c1", "c2", "c3", "c4", "c5"], usecols=[0, 1, 2, 3, 4]
    )

    df_equals(md_df, pd_df)  # everything is okay

Describe the problem
Here is the output of the code above:
DataFrame.index values are different (100.0 %)
[left]:  RangeIndex(start=0, stop=5, step=1)
[right]: Float64Index([nan, 0.0, 1.0, 2.0, 3.0], dtype='float64')

As we can see the indices are different. In case names parameter is specified read_csv will start read data right from the first line of the file and that line pandas indicates with nan value at index, when Modin just creates RangeIndex. But when usecols is specified at the same time as names Pandas creates RangeIndex the same as Modin.

This might be a similar issue to #2239.

This is quite an old issue, so we need to figure out whether it still applies.

I was able to create the dataset used: test.csv
Unfortunately, it looks like both df_equals statements are now failing. For the first check we get this:
DataFrame.iloc[:, 0] (column name="c1") values are different (80.0 %)
[index]: [nan, 0.0, 1.0, 2.0, 3.0]
[left]:  [col1, 0, 1, 2, 3]
[right]: [col1, 0, 1, 2, 3]

And for the second check after using usecols we are getting the following error:
DataFrame.iloc[:, 1] (column name="c2") values are different (80.0 %)
[index]: [0, 1, 2, 3, 4]
[left]:  [col1, 0, 1, 2, 3]
[right]: [col1, 0, 1, 2, 3]

It looks like Modin reads in the numbers as numpy.int64 by default where was pandas is reading them in as strings.


generate logging with gapic updates

@onneon I'm not sure why this was opened, going to close it out.


plugin incompatible

IDEA's browse repositories cannot search for the plugin. And install plugin from disk error：plugin is incompatible with this installation

try it now


Error by installing

When installing with "rails g solidus_multi_domain:install" after successful "bundle install", I got the error "uninitialized constant Spree::Tracker (NameError)". The output is:
$ rails g solidus_multi_domain:install
/home/carlos/.rvm/gems/ruby-2.3.1/gems/solidus_multi_domain-1.5.0/app/models/spree/tracker_decorator.rb:1:in <top (required)>': uninitialized constant Spree::Tracker (NameError) from /home/carlos/.rvm/gems/ruby-2.3.1/gems/solidus_multi_domain-1.5.0/lib/spree_multi_domain/engine.rb:11:in block (2 levels) in activate'
from /home/carlos/.rvm/gems/ruby-2.3.1/gems/solidus_multi_domain-1.5.0/lib/spree_multi_domain/engine.rb:10:in glob' 	from /home/carlos/.rvm/gems/ruby-2.3.1/gems/solidus_multi_domain-1.5.0/lib/spree_multi_domain/engine.rb:10:in block in activate'
from /home/carlos/.rvm/gems/ruby-2.3.1/gems/solidus_multi_domain-1.5.0/lib/spree_multi_domain/engine.rb:9:in `each'
...
Help please !!!

I'ḿ using ruby 2.3.1p112 (2016-04-26 revision 54768) [x86_64-linux] with Rails <IP_ADDRESS>.

The master version of Solidus moved trackers to an extension: https://github.com/solidusio-contrib/solidus_trackers
Adding that dependency to your Gemfile should fix this error. I'll make a note of this in the README until we can come up with a better solution.

That dependency solved the error above... Thanks you.


Support Relative Imported Interfaces

Hello! I have fixed the problem References to objects between files fake as empty objects and passed ci and format.
I want to know your thoughts and will focus on this project.
Thanks!

@googlebot I fixed it.

Thanks for your contribution, this is an absolutely fantastic start! One issue I see is that because this change uses file system modules that aren't available in a browser, then this would likely break the documentation site (google.github.io/intermock/)
Another approach is to pass a function (for discussion sake we can call it resolution) which does the resolution of file imports. Essentially this could live on the Options object passed into mock. https://github.com/google/intermock/blob/1cde0067e1098243016d5f96f42f4d1e3d0c0e97/src/cli/index.ts#L73
I think your code would not need too many modifications other than to move it to the cli file and then use setImportExportSpecifier as the resolution function, I believe.
https://github.com/google/intermock/blob/master/src/cli/index.ts
What are your thoughts?

Thanks for your contribution, this is an absolutely fantastic start! One issue I see is that because this change uses file system modules that aren't available in a browser, then this would likely break the documentation site (google.github.io/intermock/)
Another approach is to pass a function (for discussion sake we can call it resolution) which does the resolution of file imports. Essentially this could live on the Options object passed into mock.
https://github.com/google/intermock/blob/1cde0067e1098243016d5f96f42f4d1e3d0c0e97/src/cli/index.ts#L73
I think your code would not need too many modifications other than to move it to the cli file and then use setImportExportSpecifier as the resolution function, I believe.
https://github.com/google/intermock/blob/master/src/cli/index.ts
What are your thoughts?

Don't worry. I have slightly edited webpack config to ignore fs in browser, ran npm run docs-build , opened index.html and checked the html worked well 😃.
And I changed tsconfig.ts's  lib option from "es2016" to "es2020" to support String.prototype.matchAll.
What do you think of it? I am looking forward to your reply 🚀.

Thanks for modifying the webpack config! I should clarify my prior statement which is that I'm not sure how other users of this package are utilizing it and by adding a filesystem API to Intermock we will automatically make the module Node only by default. At the very least, it will require users to modify their own build settings, and it warrants bumping Intermock all the way to 1.0.
I'm leaning strongly towards keeping Intermock web compatible by default. While that necessitates the expense of bundling in an entire compiler, there could be lots of use cases out in the wild like that. An import resolver function passed in through the Options could be a happy middleground.
What are your thoughts given the opinion expressed above?

Thanks for modifying the webpack config! I should clarify my prior statement which is that I'm not sure how other users of this package are utilizing it and by adding a filesystem API to Intermock we will automatically make the module Node only by default. At the very least, it will require users to modify their own build settings, and it warrants bumping Intermock all the way to 1.0.
I'm leaning strongly towards keeping Intermock web compatible by default. While that necessitates the expense of bundling in an entire compiler, there could be lots of use cases out in the wild like that. An import resolver function passed in through the Options could be a happy middleground.
What are your thoughts given the opinion expressed above?

I agree with you. Passing a function to the option is a compatible approach. I have added resolveFileImports in cli option and importsResolver in Intermock general options, and all work well.
What do you think of it? I am looking forward to your reply 🚀.

Excellent work, thank you so much! One last administrative thing, can you resolve this? https://github.com/google/intermock/pull/58#issuecomment-891848287

@googlebot I fixed it.

force push your PR branch

Excellent work, thank you so much! One last administrative thing, can you resolve this? #58 (comment)

This has been solved by force-pushing the 981377660LMT:master branch from dd14d03 to 2958296 I checked .I changed the email used in git commit to ensure consistency with the one in CLA.

Thank you again!

For me that issue is still occurring, my intermock version is 0.2.5. Here's my test:

Was this every released? I see the change in the source but not in version 0.2.5

@ryanmcdermott @fluffyspace
Has this version been released? If I still import type in 0.2.5 it comes out as an empty object.


Django admin UI removed in later Netbox v4 versions

Hello,
please add an option to add otp tokens over the normal UI. As stated in the netbox issue #16136, the Django admin ui will be removed. https://github.com/netbox-community/netbox/issues/16136
@marco012024 Could you implement this in your fork? This would be wonderful.

Hello, thank you for your issue.
I've been working about it.
The plugin is based on the django-otp project. That project provides device management in the admin section. It is not so easy to use models from django-otp in the new interface due to the management of restrictions in NetBox. Anyway it would be better to decouple from django-otp and transfer the main functionality to the plugin.
I have tried it and it's possible. As result I see that the device management in the new UI should work.

Release v1.2.0 adds OTP device management in the NetBox interface.


DLP base schema information

Is the DLP base schema information correct as it seems to be exact same list as in PowerApps including "App Name" field with description "Unique Identifier of the PowerApp"?
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 98e67d79-53b0-8303-d759-c2a3dd971556
Version Independent ID: d4f73e5c-1960-e1b9-e94c-8cf65eb5671a
Content: Data loss prevention activity logging - Power Platform
Content Source: power-platform/admin/dlp-activity-logging.md
Service: power-platform
GitHub Login: @pnghub
Microsoft Alias: gned

@petepuu
Taking to email.


Add new method ListProfilesBySubscriptionId and tests

Based on some swagger changes, the SDK changes include:

Fixing type from Profile to Endpoint in CreateEndpointAsync method.
Adding new method ListProfilesbySubscriptionId.
Adding new tests for the new Profile method.

Hi @injyzarif, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (inzarif). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

This should go to AutoRest branch


Create play

@yafa369 could you please give a brief description on the PR...

Seems like spam and has nothing to do with the repo. Please close


Clear the Screen should do Exactly That

VSCode Version: Latest
OS Version: Windows 10

Steps to Reproduce:

Open VSCODE terminal CMD
Run a few Commands,
Type CLS

it should clear the screen and not put everything on a new line.
Might not be a serious issue but it bugs the hell out of me  :)

Reproduces without extensions: Yes

This is being discussed upstream https://github.com/sourcelair/xterm.js/issues/106

Not sure if this is exactly the same behavior as what the issue was opened with but here's my observations.
Doing a cls does clear the screen but it's annoying to see the previous history coming back after running another command. For example, if I run a Unit Test and do a cls, I don't expect to see previous tests coming back after a cls but they do and this makes the terminal a little hard to follow. I'm running version 1.18.0-insiders which has the new terminal rendering engine.

eyep, have to destroy terminal before every test call. upstream or not this is ridiculous.

Just ran in to this issue myself. Every time I F5 my PowerShell script, all the previous output that used to be in the terminal returns, along with the new output. If I perform cls/Clear-Host it goes away until I F5 again, and the old content returns with new output appended. And so on...
Here's my about window:

Any update ?
The console cannot provide wrong information !

https://github.com/xtermjs/xterm.js/issues/106 got closed on June 1st, as the problem appears to be with https://github.com/rprichard/winpty.
https://github.com/Microsoft/vscode/issues/45693 lists all the winpty issues affecting vscode, and this one is among them.
I checked the winpty repo for open issues mentioning "cls" or "clear" but nothing stands out as related to this. I don't have enough expertise to open an issue with winpty and provide details that would be relevant, but maybe that's what needs to be done to move forward on this?

@TheKojukinator https://github.com/Microsoft/vscode/issues/45693 will be looking at an alternative to winpty.

This is likely related to winpty, tracking in https://github.com/Microsoft/vscode/issues/45693


[vcpkg_host_path_list] add SET subcommand

This is a minor change with no serious code changes.

Thanks :)


O(N^2) performance when reading XML with SequentialAccess

Describe the bug
When reading from an XML column with CommandBehaviour.SequentialAccess, read times of a value balloon to O(N^2).
I first observed this bug about 7.5 years ago on .NET Framework and wrote a blog post at https://codingrange.com/blog/performance-optimization-or-why-does-my-sql-query-take-an-hour-to-execute
I recently re-tested this on both .NET Framework and .NET 7, as well as on both System.Data.SqlClient and Microsoft.Data.SqlClient. This behaviour is present in all four combinations.
To reproduce

On Windows, install .NET 7 and .NET Framework 4.8 or higher.
Install SQL Server. I used SQL Server 2022 Developer Edition.
Make sure that the local user has permissions to create and drop databases with Integrated Security (Windows Authentication).
Clone and build https://github.com/yaakov-h/SqlXmlBenchmark
Run the executable as follows:

SqlBenchmarker.exe Microsoft 1
SqlBenchmarker.exe Microsoft 2
SqlBenchmarker.exe Microsoft 3
SqlBenchmarker.exe Microsoft 4
SqlBenchmarker.exe Microsoft 5

Note that the insertion times grow fairly linearly, but the read times grow exponentially.
Expected behavior
Read times grow linearly with a linear increase in data size.
Further technical details
Microsoft.Data.SqlClient version: v5.0.1
.NET target: .NET Framework (multiple versions), .NET 7.0.1 (x64, 97203d38ba), probably others
SQL Server version: SQL Server 2014, 2022, probably others
Operating system: Windows 10, Windows Server 2022, probably others.
Additional context
If the SQL SELECT statement is altered from:
SELECT [{0}] FROM [{1}] WHERE [{2}] = @pkvalue;
to:
SELECT CONVERT(NVARCHAR(max), [{0}]) FROM [{1}] WHERE [{2}] = @pkvalue;
then the application will demonstrate the expected linear increase in performance.
This bug was filed on Microsoft Connect (RIP) in 2015, but seems to have been lost to the sands of time.
A performance trace with dotTrace suggests that SqlStreamingXml.GetChars is the function to blame here, spending a large amount of time in StringBuilder.get_Chars(int) and StringBuilder.Remove(int, int).

Hi @yaakov-h , thanks for bringing this to our attention and thanks for providing the repro sample.  We'll take a look at it after our release.

This is an interesting one. The reason that the problem exists for xml and not strings is that the xml route goes through a dedicated reader type, SqlStreamingXml. The problem is that in GetChars there is a bunch of code that writes to a stringbuilder and then removes the chars it has just written to keep the memory used by the builder low. This isn't a good use of a string builder and that's what is consuming the time.
How important is fixing this to you? It's certainly something that it would be nice to change but it's a quite niche use-case and the obvious workaround of using a string and wrapping it in your own xml reader seems reasonable.

We’ve been running the CONVERT to NVARCHAR(MAX) workaround since 2015 😄


Property 'info' does not exist on type 'Logger' and duplicate outputs

HI I am getting this error with below code. Seems I cannot access logger
const KafkaClient = new Kafka({
  // kafkajs will add a extra id string to end of this
  clientId: 'ms-consumer',
  brokers: kafkaVars.host.split(','),
  ssl: {
    rejectUnauthorized: false,
    key: kafkaVars.clientCertKey,
    cert: kafkaVars.clientCert,
    ca: kafkaVars.trustedCert
  },
  logLevel: level(),
  logCreator: StandardLogger
});
KafkaClient.logger().info('some error')

Property 'info' does not exist on type 'Logger'.

I also noticed if I use comsumer.logger().info('Subscriber 'testSubscriber1' successful request. {}') the logs work but I am seeing multiple outputs:

2019-10-02T20:06:50.510Z INFO [Consumer] Subscriber 'testSubscriber1' successful request. {}
2019-10-02T20:06:50.510Z INFO [Consumer] Subscriber 'testSubscriber1' successful request. {}

Hi @Awatatah, I think your custom logger is wrong. From the error message (Property 'info' does not exist on type 'Logger'), I can see that it tries to access an object called Logger.

Feel free to re-open this issue if you need


WIP: Add reader for fOLD toolbox xls files

The fOLD Toolbox (https://github.com/nirx/fOLD-public) provides precomputed sensitivity of different standard optode positions to brain locations. Until we have proper photon migration #62, then these precomputed values will be useful.
There is no license for the fOLD files so I guess I cant include them. But I have written a reader for them. Just working on an API now and then will need an example etc.

Codecov Report

Merging #371 (e4ca408) into main (c9db009) will decrease coverage by 43.53%.
The diff coverage is 15.45%.

@@             Coverage Diff             @@
##             main     #371       +/-   ##
===========================================
- Coverage   95.62%   52.08%   -43.54%
===========================================
  Files          49       51        +2
  Lines        1760     1870      +110
  Branches      232      237        +5
===========================================
- Hits         1683      974      -709
- Misses         31      830      +799
- Partials       46       66       +20

Impacted Files
Coverage Δ

mne_nirs/io/fold/tests/test_fold.py
13.41% <13.41%> (ø)

mne_nirs/io/fold/_fold.py
21.42% <21.42%> (ø)

mne_nirs/channels/tests/test_roi.py
14.28% <0.00%> (-85.72%)
:arrow_down:

mne_nirs/channels/tests/test_short.py
17.39% <0.00%> (-82.61%)
:arrow_down:

...nal_enhancement/tests/test_negative_correlation.py
17.50% <0.00%> (-82.50%)
:arrow_down:

mne_nirs/statistics/tests/test_glm_type.py
18.58% <0.00%> (-81.42%)
:arrow_down:

mne_nirs/utils/tests/test_io.py
20.00% <0.00%> (-80.00%)
:arrow_down:

...erimental_design/tests/test_experimental_design.py
24.48% <0.00%> (-75.52%)
:arrow_down:

mne_nirs/visualisation/_plot_quality_metrics.py
21.42% <0.00%> (-75.00%)
:arrow_down:

mne_nirs/visualisation/tests/test_visualisation.py
25.53% <0.00%> (-74.47%)
:arrow_down:

... and 24 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 c9db009...e4ca408. Read the comment docs.


[FEATURE] Add labels with hdfs host for local storage total_hadoop_write_data metrics

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I have searched in the issues and found no similar issues.

Describe the feature
We will write apps of different businesses apps into different HDFS.
For further iterative optimization, we need to expose the write capacity of
different HDFS to evaluate the total write capacity of different business lines.
Motivation
No response
Describe the solution
No response
Additional context
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

#1107 done


Document enhancement request for private endpoint dns zone

Customer may want to add private DNS zone to their private DNS server. According to this ticket: https://github.com/MicrosoftDocs/azure-docs/issues/91695. The default hostname could be a sub domain (i.e. ‘1.azurestaticapps.net’, ‘2.azurestaticapps.net’). I suggest we explain more details in the document. Thanks.
Every Static webapp will have a default hostname that ends with some DNS suffix. For some apps that will be ‘azurestaticapps.net’, for others it could be a sub domain (i.e. ‘1.azurestaticapps.net’, ‘2.azurestaticapps.net’). This is by design and allows the platform to be more resilient to DNS outages.
For private endpoints to work, you need to have an intermediate CNAME of .privatelink.. As a result, if you have a default hostname for your SWA that ends with ‘1.azurestaticapps.net’ then the private endpoint dns zone should be ‘privatelink.1.azurestaticapps.net’. This ‘1’ is not specific to private endpoints but just a result of the default hostname having it.
The default hostname and subdomain of your Static Web App are decided at create time and will remain constant until the app is deleted.

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

ID: e0ff6e01-b10d-83b4-2faa-d0aabce7e554
Version Independent ID: 92496f20-fdf2-4299-db90-5c95d0d88fcc
Content: Configure private endpoint in Azure Static Web Apps
Content Source: articles/static-web-apps/private-endpoint.md
Service: static-web-apps
GitHub Login: @burkeholland
Microsoft Alias: buhollan

@rachelxj-ms Thanks for your feedback.
Assigning to content author  @burkeholland  for review.

Hi,
do we have an update on this or any more specific information in regards to potential subdomains?
I guess it would be enough for customers if they'd at least know which privatelink DNS zones they need to create.
Cheers,
Andre

This is causing some confusion here as well.

Can n be 1,2,... up to what? in privatelink.{n}.azurestaticapps.net?

This would help to know which private dns zones to create

Does anybody know if there is a policy that can account for this variance over PE's that's linked to staticSites (groupId: staticSites, privateLinkServiceId contains Microsoft.web/staticSites)?

If I am to follow https://learn.microsoft.com/en-us/azure/cloud-adoption-framework/ready/azure-best-practices/private-link-and-dns-integration-at-scale to implement automatic registration of private endpoint into centrally managed private dns zones the policy need to know which private dns zone to register to...

Related:

https://github.com/Azure/Enterprise-Scale/issues/1073
https://github.com/Azure/terraform-azurerm-caf-enterprise-scale/issues/482
https://github.com/Azure/ALZ-Bicep/issues/330

Thanks for your feedback and your contribution to Azure docs.
Feedback for this repository is moving away from GitHub to a system specific to the Microsoft Learn platform. Issues for this repository will soon be disabled, and additional comments from GitHub will no longer be possible. However, we are now tracking and triaging this issue in the new feedback system.
To learn more about our feedback systems, please see Provide feedback for Microsoft Learn content.
#please-close


Promote DisableDNSProviderManagement to GA

How to categorize this PR?

/area open-source
/kind cleanup
What this PR does / why we need it:
The DisableDNSProviderManagement feature gate is promoted to GA and is now unconditionally enabled.
Code has been cleaned up regarding usage of DNSEntry and DNSProvider which is not used anymore since UseDNSRecord was promoted to GA or now with promotion of DisableDNSProviderManagement.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Follow-up of #6142
Release note:

The `DisableDNSProviderManagement` feature gate has been promoted to GA and is now unconditionally enabled. If the `shoot-dns-service` extension is deployed, please make sure following prerequistes are given for a smoothly transition:
 - The `shoot-dns-service` extension must be installed in a version >= `v1.20.0`.
 - The controller deployment of the `shoot-dns-service` sets `providerConfig.values.dnsProviderManagement.enabled=true`
 - Its admission controller (`gardener-extension-admission-shoot-dns-service`) is deployed on the garden cluster

/uncc

/assign

There are a few more occurrences of the external-dns-management dependency. What about them? Can they be dropped as well now?

I still have problems to run the e2e tests locally to understand the failure of the managed seed test.

There are a few more occurrences of the external-dns-management dependency. What about them? Can they be dropped as well now?

I will take a look at them after I fixed the e2e test. Any hint is welcome.

Thank you for the PR!. While doing joint review with @acumino, we have the following comments:
...

@ary1992 Thanks for your review. All requested changes have been addressed.
The failing e2e test ManagedSeeds worked for me locally, let's wait and see if Prow gets the same result.

There are a few more occurrences of the external-dns-management dependency. What about them? Can they be dropped as well now?

@rfranzke
The remaining external-dns-management imports are related to the calculation of the required extensions.
The DNSProvider related parts cannot be dropped without loosing the ControllerInstallations for the dns-external extension. It is still needed for the shoot-dns-service and shoot-cert-service.
I see three alternatives:

Provide some dependency mechanism for extensions to other extensions. I.e  if shoot-dns-service is installed, also install the dns-external extension.
Change the deployment policy to Always in its ControllerRegistration
The dns-controller-manager could be deployed by the shoot-dns-service. But this would cause substantial migration, configuration and testing efforts.

We need also to discuss which component should be responsible for deploying the CRDs for DNSProvider, DNSEntry, and DNSOwner.

@MartinWeindel I think we should drop support for registering external-dns-management via ControllerRegistration since it's no longer a supported extension for Gardener (only indirectly via shoot-dns-service).
I think the clean solution would be that shoot-dns-service itself deploys external-dns-management. Introducing a dependency mechanism or using Always policy are workarounds FMPOV. If you think this is too much of an effort for now, then I propose to go ahead with Always deployment policy of external-dns-management but remove all .spec.resources from the ControllerRegistration (Gardener shall not understand DNSProvider any longer).

We need also to discuss which component should be responsible for deploying the CRDs for DNSProvider, DNSEntry, and DNSOwner.

Same like above, it should be shoot-dns-service.

@rfranzke
I suggest to move the cleanup work for the DNSProvider registration to a separate PR, after either the ControllerRegistration has been changed to policy Always or the deployment has moved completely to the shoot-dns-service extension.

@rfranzke I suggest to move the cleanup work for the DNSProvider registration to a separate PR, after either the ControllerRegistration has been changed to policy Always or the deployment has moved completely to the shoot-dns-service extension.

Sure. For now, probably it makes sense to still document (or at least announce via release note) that external-dns-management must still be registered as an extension via dedicated ControllerRegistration in case shoot-dns-service is used.

Thank you for the PR!. While doing joint review with @acumino, we have the following comments:
...

@ary1992 Thanks for your review. All requested changes have been addressed. The failing e2e test ManagedSeeds worked for me locally, let's wait and see if Prow gets the same result.

Thank you for the changes. I don't have any further comments.
/lgtm

ping @MartinWeindel, please rebase so that we can go ahead with this PR

/assign

/milestone v1.52


chore(main): release 0.9.2

:robot: I have created a release beep boop
0.9.2 (2024-12-18)
Bug Fixes

license metadata (57a1899)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.9.2
:sunflower:


samesite and secure attributes on cookies

After a while google chrome gives a warning: "SameSite" attribute must be either lax,none etc. also "Secure" attribute must be sent by cookie.
Anyone get these warnings?

Are you sure this is caused by this package and not any of your API calls?

Closing as there's no response


Unable assign populated loaded type to original entity

Describe the bug
Cannot assign populated loaded type to original entity, it no error before<EMAIL_ADDRESS>but occur after<EMAIL_ADDRESS>Stack trace
Type 'Loaded<MemberNotification, "notification">' is not assignable to type 'MemberNotification'.
  The types of 'toObject(...).notification' are incompatible between these types.
    Type 'EntityDTO<Loaded<Notification, never>>' is not assignable to type 'string'.ts(2322)

To Reproduce
import { BaseEntity, Entity, ManyToOne, PrimaryKey, Ref } from "@mikro-orm/core";
import { MikroORM } from "@mikro-orm/postgresql";
import ormConfig from "orm.config";
export class Base<T extends { }, PK extends keyof T> extends BaseEntity<T, PK> {
  constructor() {
    super();
  }
}
@Entity()
class MemberNotification extends Base<MemberNotification, 'id'> {
  @PrimaryKey({comment: 'Primary key', autoincrement: true})
    id!: string;

  @ManyToOne({nullable: true})
    notification?: Ref<Notification>;
}

@Entity()
class Notification extends Base<Notification, 'id'> {
  @PrimaryKey({autoincrement: true})
    id!: string;
}

async function test(){
    const orm = await MikroORM.init(ormConfig);
    const em = orm.em
    const queryRet = await em.findOne(MemberNotification,{},{populate:["notification"]})
    const test:MemberNotification = queryRet // Occur compiler error
}

Steps to reproduce the behavior:
No description
Expected behavior
No error occurred.
Additional context
Versions

Dependency
Version

node
18.12.0

typescript
4.9.4

mikro-orm
5.6.0 - 5.6.1

@mikro-orm/postgresql
5.6.0 - 5.6.1

It was caused by this commit, and it is surfacing only because of you use the ORM base entity that adds the serialization methods. It is kinda the same problem as #3277.


feat: add helper component to show formated amount

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.43%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5987

Relevant Lines:
14451

💛  - Coveralls


Building Collaborations

Description
Leverage opportunities to accomplish the technical work of the Division/ITL/NIST by developing collaboration with external partners through, for example, CRADAs, contracts, grants, SBIRs, and recruiting guest researchers, IPAs, students, or post-docs.
Success Measures
Supervisor typically finds: Evidence of development and contributions of collaborations and exchange of expertise to maximize NIST's technical impact.

DevOps COI work is part of this critical element: https://github.com/aj-stein-nist/okrs/issues/35.


"n" parameter does not return multiple responses

I'm encountering an issue with the "n" parameter. It appears not to be functioning as expected since it does not return multiple responses. According to the documentation, the "n" parameter correlates to the number of output sequences to return for the given prompt.
{
    "input": {
        "prompt": "A prompt...",
        "sampling_params": {
            "max_tokens": 300,
            "temperature": 0.85,
            "top_k": 40,
            "n": 3,
            "presence_penalty": 1.2
        }
    }
}

Expected:
The API should return three (3) distinct answers.
Actual:
Despite setting "n" to 3, only one response is returned.
{
    "delayTime": 161,
    "executionTime": 4182,
    "id": "sync-302d9141-abf8-4335-9499-0caf70777c81-u1",
    "output": [
        [
            {
                "text": " A response",
                "usage": {
                    "input": 274,
                    "output": 26
                }
            }
        ]
    ],
    "status": "COMPLETED"
}

I've also encountered another confusing outcome when I attempted to use the "stream" parameter.
{
    "input": {
        "prompt": "### Instruction:\nSimply output the answer to 1+1\n\n### Response:\nSure, 1+1=",
        "stream": true,
        "sampling_params": {
            "max_tokens": 50,
            "temperature": 0.85,
            "top_k": 40,
            "n": 3,
            "presence_penalty": 1.2
        }
    }
}

Expected:
As the 'stream' option is set to 'true', I expected it to handle the 3 responses set by 'n'.
Actual:
Results are unexpected; it appears multiple responses are received but only the first contains text while the others are empty or single characters.
{
    "delayTime": 177,
    "executionTime": 1512,
    "id": "sync-0bc6241b-4350-4664-9d09-cd5d2ef6e59a-u1",
    "output": [
        [
            {
                "text": "2",
                "usage": {
                    "input": 32,
                    "output": 1
                }
            },
            {
                "text": "",
                "usage": {
                    "input": 32,
                    "output": 1
                }
            },
            {
                "text": "",
                "usage": {
                    "input": 32,
                    "output": 1
                }
            },
            {
                "text": "",
                "usage": {
                    "input": 32,
                    "output": 2
                }
            },
            {
                "text": "",
                "usage": {
                    "input": 32,
                    "output": 2
                }
            },
            {
                "text": ".",
                "usage": {
                    "input": 32,
                    "output": 2
                }
            },
            {
                "text": "",
                "usage": {
                    "input": 32,
                    "output": 2
                }
            },
            {
                "text": "",
                "usage": {
                    "input": 32,
                    "output": 2
                }
            },
            {
                "text": ".",
                "usage": {
                    "input": 32,
                    "output": 3
                }
            }
        ]
    ],
    "status": "COMPLETED"
}

Unclear whether this is correct or a bug. Any suggestions or potential solutions would be appreciated.
Thanks! 🤗

Added in the latest update, thank you for your feedback!


Catalog entry without a time axis

We have some data that is just 2D and has no time axis. But the catalog seems to not allow submitting such a request, this should be fixed.

That's unusual, non-timeseries data is pretty common. Could be 2D, or volumetric 3D, etc. I'm not familiar with the catalog standard/spec so don't really have any suggestions.  You could put a datetime as a metadata of when the data was created or so, but shouldn't require a time axis for that.

yes, providing creation time should suffice, sorry for the confusion!

is this also done and only needs approval?

Hi @baloola, so to clarify: when the data refers to a single point in time, you recommend to use the fields Date:Creation Date, instead of the Time axis?

@mari-s4e I'd worry about using creation date, as that usually refers to the time the dataset was calculated, especially with modeled data this can be years after the fact.
While we could implement a different approach for datasets that cover only one point in time, I do like keeping things consistent. Any reason we cannot just use the Time Axis component, provide same begin and end time for such datasets?


Merge updates from development

Updates to config, can now add/delete boards through UI

Testers before merge would be appreciated!

Only thing that appears to not work is it doesn't actually delete boards you select to delete. All effects work.

Hi!
Exams are almost over, so I'll pick this up again in about a week. Switch
branch for multiple devices, I think it's available to all.
On Thu, May 17, 2018, 20:03 blassbasti<EMAIL_ADDRESS>wrote:

Is this already merged into the master repo or do i still have to add it
myself?
Really want to get more than one device to work!!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/not-matt/Systematic-LEDs/pull/24#issuecomment-389973331,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ae5azKopPvEz8mX_4sPEYx8NNHO8LlWWks5tzclzgaJpZM4S3S-6
.


Hotplates need an overhaul or should be replaced with chem heaters.

Currently with hotplates you just put a beaker on the hotplate and you hope you've waited long enough for it to heat up. There's nothing telling. It doesn't show you if it's on or off, doesn't tell you the temperature of the reagents and no estimate on how long it might take. Hotplates should have a UI or they should be replaced with chem heaters.

There's nothing telling.

You can hear when the reaction occurs, and usually the color of the solution will have changed.

It doesn't show you if it's on or off,

Isn't it always on when there's a beaker on it? There should probably be like a power LED on the sprite to indicate this.

doesn't tell you the temperature of the reagents and no estimate on how long it might take.

It's a hotplate. It's not aware of what's in the beaker, it just heats it. If you want to know temperature, a thermometer makes more sense.

Hotplates should have a UI or they should be replaced with chem heaters.

Not everything needs a UI. Hotplates are simple enough that they definitely don't need one.

and usually the color of the solution will have changed.

the beaker's solution isnt visible when on the hotplate so you have to do the, alt click shift click click back onto hotplate because meth isnt ready, shuffle

the beaker's solution isnt visible when on the hotplate so you have to do the, alt click shift click click back onto hotplate because meth isnt ready, shuffle

Yeah true. It should probably put the actual beaker entity on the hotplate instead of a sprite layer.

You can hear when the reaction occurs, and usually the color of the solution will have changed.

Yes but it's a guessing game of when it's going to happen.

Isn't it always on when there's a beaker on it? There should probably be like a power LED on the sprite to indicate this.

There's no visual indicator to show there's a state change if it's on or off other than there's a beaker on it.

It's a hotplate. It's not aware of what's in the beaker, it just heats it. If you want to know temperature, a thermometer makes more sense.

Not everything needs a UI. Hotplates are simple enough that they definitely don't need one.

It's a game. Not everything has to be logical. Chem heaters showed you what the temperature was and allowed you to change target temperature/power.


Unable to withdraw LP positions

UI sets a limit of 0 preventing liquidity from being withdrawn.

Fixes #897


Can we have a new component like this one https://mui.com/material-ui/react-grid2/?

In Radix primitives that wouldn't make sense, as these are unstyled primitives.


Entities added to "info_entities" are not centered

Which version of Room Card?
ROOM-CARD  1.08.03
Describe the bug
I added several entities to the "info_entities" list, including icons and plain text. The text elements are centered along with the header, but the icons are "floating." All icons are from the "Material Design Icons" collection.
Expected behavior
centered icons
Screenshots

Desktop (please complete the following information):

OS: Win11
Browser Chrome
Version 126.0.6478.62

YAML
- entity: binary_sensor.dvizhenie_v_gostinoi
    show_icon: true
    icon:
      template:
        icon: >
          if (entity.state == 'on') return 'mdi:home';  else return
          'mdi:home-variant-outline';
  - entity: sensor.smartmi_evaporative_humidifier_water_level
    show_icon: true
    icon:
      template:
        styles: >
          if (entity.state >= 70) return 'color: green';   if (entity.state >=
          20) return 'color: blue';   else return 'color: orangered';
        icon: >
          if (entity.state >= 70) return 'mdi:cup';   if (entity.state >= 20)
          return 'mdi:cup';   else return 'mdi:cup-off-outline';
  - entity: sensor.gostinaia_vlazhnost_pochvy_soil_moisture
    show_icon: true
    icon:
      template:
        styles: >
          if (entity.state >= 80) return 'color: green';   if (entity.state >=
          60) return 'color: orangered';   else return 'color: red';
        icon: >
          if (entity.state >= 70) return 'mdi:flower-tulip';   if (entity.state
          >= 20) return 'mdi:flower-tulip';   else return
          'mdi:flower-tulip-outline';
  - entity: sensor.esphome_web_b7d4fc_co2
    show_icon: true
    icon:
      template:
        styles: >
          if (entity.state >= 1200) return 'color: red';    if (entity.state <
          1200) return 'color: green';
        icon: |
          'mdi:molecule-co2';
  - sensor.0xa4c13826b0d2f8ba_temperature
  - sensor.0xa4c13826b0d2f8ba_humidity

I noticed the same the info entities shifted to the left noticeably, this happened after i updated ha container, although hacs autoupdates the addons so maybe I just rebuilt after this updated, anyway, it drives me up the wall, it was the very first thing i noticed after the container rebuilt.

i don't think it's anything in yaml I haven't changed the cards in over a half a year, but maybe ha changed some interpretation of yaml? for reference
first card yaml: https://o.o5.ddns.net/GDYGA
2nd card yaml: https://o.o5.ddns.net/HFmk0

I noticed the same the info entities shifted to the left noticeably, this happened after i updated ha container, although hacs autoupdates the addons so maybe I just rebuilt after this updated, anyway, it drives me up the wall, it was the very first thing i noticed after the container rebuilt.

i don't think it's anything in yaml I haven't changed the cards in over a half a year, but maybe ha changed some interpretation of yaml? for reference
first card yaml: https://o.o5.ddns.net/GDYGA
2nd card yaml: https://o.o5.ddns.net/HFmk0

This is something different. Might be your theme or that you use a stack in card.

@almirus Can you share your entire yaml of the card?

Close because of no answer to questions


Add ZoneSync method #52

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
All new code requires tests to ensure against regressions.

Description of the Change

According to syncZone Api, the http call support is added.
Why Should This Be In The Package?

people might need it.
Benefits
Adds a value to project  with sync zone.
Possible Drawbacks

Verification Process

unit tests are added to check functionality.
Applicable Issues (Optional)

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Added authorization_code flow authenticators for torii provider.

Extended torii provider for refreshing tokens in oauth2 authorization_code flow.

Thanks for the contribution!
However, this looks like it is handling a very specific case that is likely only relevant to your application but not everyone's Torii-based authenticators? This should likely remain in your application code rather than in the base library itself. Please let me know if I'm missing something.

I've opened a PR because this is not only relevant to my application but it's a common problem for everyone that use an oauth2 autentication system with authorization_code flow (PKCE).
This flow is useful if you develop app or webapp and you don't want to let users share theirs passwords within the mobile app.
Useful links:

https://hackernoon.com/strengthening-oauth2-for-mobile-f4f3925dbf18
https://auth0.com/docs/flows/concepts/auth-code-pkce

Sure, refresh tokens are a good thing in general but I don't know what Torii providers people that use the Torii authenticator use and whether those providers/services use refresh tokens at all?

Closing this as it's out of scope for ESA.


latest dependencies

latest dependencies

Only failing on Node 6 and lower:
https://travis-ci.org/socketio/socket.io-redis/builds/594885163?utm_source=github_status&utm_medium=notification

@darrachequesne

If that is the case you should remove node 6 from travis.yml and add node 10 and node 12.
Also, it is a bad idea to specify "latest" for the version in production code for wide use, because you never know when a breaking change may be made in a new version of a dependency.

Please do not use latest in production. Stay safe!


As a user, when viewing the Address object, I would expect the Household Account field to just be labeled Account, since NPSP now supports organizational accounts for addresses as well.

Included in beta release version 3.37 (Beta 3) and higher

Included in production release version 3.37


Reverted to AllocConsole method for focus

See #110, I'm having this exact problem using the Win key, the start menu appears when I switch workspaces with Win + {1..9} (which is annoying of course). Not sure why @rickbutton made this change when this was working properly.

Isn't this related to #195 too?

I personally think it'd be better to recommend using modifiers like Win + Shift instead of using Win by itself.

The thing is most users (like me) come from other unix tiling window managers like xmonad, dwm or i3, and we all usually use the windows key as the mod key. So it will be very inconvenient having to use a different mod key when you switch OS (I dual boot windows for dotnet and linux as my main driver).
Also, the Win key itself has no useful functionalities behind other than opening the start menu. In case this is still unconvincing we should either: mention that the use of the win key is not recommended due to this problem, or do something to prevent the start menu from opening.
This seems to be the easiest solution to this problem, but if there's a better way to do it I have no problem

Strongly echo the importance for the Windows key to be (properly) bindable for keybind parity with Unix tiling windows managers. I imagine there is a fairly significant chunk of the userbase here that use workspacer because they are coming from a Unix system with a proper tiling wm and want their workflow to stay the same.
As a current stopgap, if you can live with not being able to open the start menu with the Windows key, I have found that I can use Open-Shell to disable the start menu popping up when using keybindings with the Windows key (#110).
Under the "Controls" tab of the Open-Shell settings, I just set Win and Shift+Win to do nothing (and then keep left click to open the W10 start menu so it's still accessible). The lack of start menu is eased if you use a dmenu/Rofi style launcher like Wox or Keypirinha, with my preference being the latter.
Note however that this does not solve conflicts with some Windows shortcuts. I can map Win+F just fine (typically a Window feedback shortcut) for example, but to use Win+L for vim-style navigation keys, I had to use a registry hack to disable the typical lock shortcut.

Thank you @imswebra, I chose "Nothing" instead:
. I'll be working on adding a workaround to this in the docs with the information you provided. I'm closing this PR.


Traffic Sign Recognition

what is the issue getting on tensorflow version upgrade

I will try again sir, By evening I will try to do it in tenserflow  version 2


all: failed to run make.bash on darwin arm64

What version of Go are you using (go version)?

$ go version

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?
$ GOROOT_BOOTSTRAP=/Users/cuonglm/sdk/go1.16 ./make.bash
Building Go cmd/dist using /Users/cuonglm/sdk/go1.16. (go1.16 darwin/arm64)
Building Go toolchain1 using /Users/cuonglm/sdk/go1.16.

Building Go toolchain2 using go_bootstrap and Go toolchain1.
go tool dist: FAILED: /Users/cuonglm/sources/go/pkg/tool/darwin_arm64/go_bootstrap install -gcflags=all= -ldflags=all= -i cmd/asm cmd/cgo cmd/compile cmd/link: signal: segmentation fault

What did you expect to see?
make.bash run successfully.
What did you see instead?
make.bash failed after commit 5ff7ec98b7727b3641df25200345b1aa50b6ff35
cc @dr2chase @jeremyfaller @cherrymui

FYI, using the bootstrap binary produces the same error:
GOROOT_BOOTSTRAP=$(pwd)/../../go-darwin-arm64-bootstrap ./make.bash

Also observed on ppc64le:
[root@rajalakshmi-workspace ~]# go version
go version go1.15.8 linux/ppc64le
[root@rajalakshmi-workspace ~]# git clone https://github.com/golang/go.git
Cloning into 'go'...
remote: Enumerating objects: 29, done.
remote: Counting objects: 100% (29/29), done.
remote: Compressing objects: 100% (28/28), done.
remote: Total 463350 (delta 2), reused 1 (delta 1), pack-reused 463321
Receiving objects: 100% (463350/463350), 238.53 MiB | 26.31 MiB/s, done.
Resolving deltas: 100% (370164/370164), done.

Building Go toolchain2 using go_bootstrap and Go toolchain1.
go tool dist: FAILED: /root/go/pkg/tool/linux_ppc64le/go_bootstrap install -gcflags=all= -ldflags=all= -i cmd/asm cmd/cgo cmd/compile cmd/link: signal: segmentation fault (core dumped)
[root@rajalakshmi-workspace src]#

@cuonglm Can you please make the issue summary more generic to include power architecture?

@cuonglm Can you please make the issue summary more generic to include power architecture?
linux/ppc64le

I think it’s ok to add more failure in comments. Also there’s note in original CL https://go-review.googlesource.com/c/go/+/293392

Also observed on Linux/Arm64

I have observed this on all non-intel arches supported by Fedora, that being linux/arm64, linux/arm32, linux/s390x and linux/ppc64le with various bootstrap GC versions, gcc&co. That makes me believe this is some internal GC issues introduce somewhere between Feb 27th and now based on builds that I periodically run. Trying to bisect it now.


Parse errors in directives doesnt show up

Steps to Reproduce:
In a notebook execute the following:

Expected behavior:
It should show some sort of Parse Error saying that lsmagic doesnt take any argument
Actual Behavior:
If there is a parse error during parsing the directives, the command is passed to the Kernel which displays confusing error:
https://github.com/dotnet/try/blob/91ff6e81150397499a8d215bd80cdab18192a5c7/Microsoft.DotNet.Interactive/KernelBase.cs#L179


Documentation

Documentation for deployment needs to be properly written: https://github.com/PrivateAIM/documentation

An first version was pushed to the documentation repo


:package: Update denops.vim to v1.0.0-alpha.4

No need to update immediately. apologize for it; m(_ _)m
https://vim-jp.slack.com/archives/C01N4L5362D/p1625290486070300


Boilerplate

i'm still unsure on the workflow with pull request so i'll just leave it here. Merge if you feel it looks ok.


[KED-1358] conda install kedro

Description
Anaconda is a popular package manager for many data scientists cause of its ability to resolve non-python dependencies, like Cuda.  Many tools like MLFLOW rely heavily on anaconda in order to ensure compatibility when serving model on Databricks or Azure.
Context
Many pypi packages can be automatically compiled to be install as conda packages using conda skeleton python kedro, conda convert --platform all /path/to/package/ and conda upload path/to/package/`.  For kedro this fails, as the pypi .tar.gz does not have the test_requirements.txt.
I have made a workaround by building specifically from the GitHub repo, which I have made availible.
Possible Implementation
Change what is uploaded to pypi to ensure the test_requirements.txt is uploaded.
Possible Alternatives
meta.yml
{% set name = "kedro" %}
{% set version = "0.15.8" %}

package:
  name: "{{ name|lower }}"
  version: "{{ version }}"

source:
  git_rev: 0.15.8
  git_url: https://github.com/quantumblacklabs/kedro.git

build:
  number: 0
  script: "{{ PYTHON }} -m pip install . --no-deps --ignore-installed -vv "

requirements:
  host:
    - pip
    - python
  run:
    - python

test:
  imports:
    - kedro

about:
  home: "https://kedro.readthedocs.io/"
  license: "BSD"
  license_family: "APACHE"
  license_file: "LICENSE.md"
  summary: "A Python library that implements software engineering best-practice for data and ML pipelines."
  doc_url: "https://kedro.readthedocs.io/"
  dev_url: "https://github.com/quantumblacklabs/kedro/tree/develop"

@marcusinthesky Thank you for the feature request! We have a ticket in our backlog and will let you know once it is done :)

This is currently what is keeping me from using kedro seriously, I don't want the mix of conda and pip installs to result in a broken/unstable environment. Otherwise looks like an amazing project, thanks for open-sourcing your hard work!

I feel similarly to d-chambers, regarding mixing of pip and conda. Thank you to the people working on the pull request on conda-forge!

Hey guys, good news: we're on conda! 😃 (thanks @marcelotrevisani for the help in getting this pushed on conda-forge)
conda install kedro will install the latest Kedro version, 0.15.9, and we'll be ensuring that future releases will also be added as they come out.
Please be mindful that we haven't gone and retroactively added older versions, we recommend you use the new release, particularly as the last 3 or so releases have been hotfixes. If there's any reason why you can't update and you strongly need the conda installaton, rather than PyPI, let us know in a separate issue and we can have a look at getting that sorted for you.

Just to let you all know that it is not necessary to manually create PR to update the recipe on conda-forge because we have some bots which check from time to time for new releases. The bot will automatically create a PR with the new version. :)

This is currently what is keeping me from using kedro seriously, I don't want the mix of conda and pip installs to result in a broken/unstable environment. Otherwise looks like an amazing project, thanks for open-sourcing your hard work!

It is nice to have everything packaged with conda indeed. However, if you really need a package which is not (yet) on conda-forge you have some options:

You can open a PR on conda-forge and package the missing project, or just open an issue there requesting to package that project.

Or you can install the pacakge with pip but asking pip to no install the dependencies, in that way it will not mess with your environment. Something like pip install --no-deps PKG_NAME

Thanks, I didn't know about the no-deps flag.


Work In Progress – OpenTracing Brave Bridge

Early code building an OpenTracing Brave bridge.
There's testing and tests still to go, and some of the new things in Brave-3 I'm not entirely confident of yet, but I do believe this is along the right track.
All spans are created with the local tracer. CS+CR and SR+SS annotations are done around the inject and join methods. Not sure yet whether that approach will work.
Haven't started on the async side of affairs either.
@adriancole @kristofa

@michaelsembwever @adriancole @kristofa just wanted to mention that I have great interest in this PR but know far less about Brave than any of you. That said, please pull me in about the OpenTracing side if it would be helpful.

@michaelsembwever Nice, but I'll have to do some reading on the OpenTracing api :)

re-opening this fresh as the destination is now master


Any chance of adding rebinding chat key support for Valheim Chat Tweaks?

Any chance of adding rebinding chat key support for Valheim Chat Tweaks?
I would like to rebind the chat key to anything other than Enter.
Thanks.

I'll look into it. Sorry for the late response, I was taking a break from coding after my exams in school.

I added this

Great, thanks a lot! :smile:

After installing the mod, how do I change the binding? I can't seem to find any readme on how to use it.

you can either do one of these things:
-install configuration manager for bepinex and press f1 to change the config file (recommended way)
-go to Valheim\BepInEx\config\kenx00x.ChatTweaks.cfg and edit the chatkey

Cool, thanks!


Wrapping exceptions

Describe the bug
Wrapping the exceptions in ServiceCreation file instead of rethrowing error

resolved in #27


Fix attributes being dropped in certain circumstances

Whenever format_lat is triggered and extrapolates data to the poles, the attributes were being dropped.

Thanks for fixing this, Ben!
Could you update the changelog and add yourself to the CITATION.cff file?

Done, thanks @BSchilperoort!


Support .net style iterators/enumerators

One of the issues with the library at the moment is that the iterators used to iterate over the results do not follow .net's iterator paradigm as defined by IEnumerable & IEnumerator which makes it difficult to use.
Goals:

Minimize breaking changes (keep existing API for now)
Add new enumerable/results for those that want to use it.

See #408


Multiple Response Groups

Is there a way to supply multiple ResponseGroups, similar to how you can in the Amazon scratchpad ?
Maybe by marking the AmazonResponseGroup with the FlagsAttribute

i change this in the next version

If you accept pull requests I can try to put something together
On Apr 21, 2016 2:17 AM, "Tino Hager"<EMAIL_ADDRESS>wrote:

i change this in the next version
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/tinohager/Nager.AmazonProductAdvertising/issues/2#issuecomment-212762728

Yes i accept pull requests 👍


fake steam id doesnt work

fake steam id doesnt work status or amx_who still shows as BOT

It's works as intended. It's shouldn't affect any thirdparty plugins. It's used just for statistics system like hlstats and so on.


csg_boltzmann: re-implemented --excl option (fixes #177)

I don't like the N^2 loop, but better than nothing.
@ruehle, @MrTheodor : review please!

Hi,
For --no-map it causes Segmentation Fault:
csg_boltzmann --top topol_new.xml --excl ooo --no-map

Program received signal SIGSEGV, Segmentation fault.
0x000000000040d8bd in CsgBoltzmann::EvaluateTopology(votca::csg::Topology*, votca::csg::Topology*) ()

I am not exactly sure how this should work, I tried on a very simple example of butane and it generates such excl list:
# atomistic: BUT cg: BUT cgmap: map.xml
1 3 4
2 3 4

I would expect that it will only generate excl for CG molecules so:
1 2

Test files:
https://gist.github.com/MrTheodor/da3f6de77e1143934708

Hmm, isn't the --excl option to write exclusions to be used for the atomistic simulation?
I am not sure if it makes sense to combine it with --no-map, but let me check what 1.2.4 was doing.

Ok, getting same results as with v1.2.4.
There were a couple of things wrong in the new exclusion list scheme, too, pointer comparison is dangerous ;-)
In v1.2.4 in --excl + --no-map segfaults, too, so I disable that combination as it doesn't make much sense anyhow.


i have a problem "add etcd member to the cluster"

hello~
i have a question about  "add etcd member to the cluster"
my kubernetes version 1.12.1
and os is ubuntu 16.0.4
i did
kubectl exec -n kube-system etcd-${CP0_HOSTNAME} -- etcdctl --ca-file /etc/kubernetes/pki/etcd/ca.crt --cert-file /etc/kubernetes/pki/tcd/peer.crt --key-file /etc/kubernetes/pki/etcd/peer.key --endpoints=https://${CP0_IP}:2379 member add ${CP1_HOSTNAME} https://${CP1_IP}:2380
but i got
Unable to connect to the server: dial tcp <IP_ADDRESS>:6443: i/o timeout
this error.. firewalld was all down and i copied cert files already..
please help me thank you

It seems your command is wrong, check this out:
/etc/kubernetes/pki/tcd/peer.crt

Was that problem fixed?

thank you for your kindness.
i misintepret the url.
i did https://VIP:30000
it did not work.
however,
"curl -k https://VIP:16443 http://vip:16443/ " is worked between master
servers
thank you
2018년 10월 25일 (목) 오전 10:37, cookeem<EMAIL_ADDRESS>작성:

Was that problem fixed?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cookeem/kubeadm-ha/issues/36#issuecomment-432883429,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZTXGGs8FTdzt-lQjS03qsAUcnDLaICTks5uoRXogaJpZM4XxrPd
.

show me the kube-dashboard status, run command below:
kubectl get pods -n kube-system

my dashboard status is running like below
root@k8s-master01:~# kubectl get pods -n kube-system
NAME                                    READY   STATUS    RESTARTS   AGE
calico-node-7gdv6                       2/2     Running   0          22h
calico-node-8f2rf                       2/2     Running   0          22h
calico-node-97ms9                       2/2     Running   0          22h
coredns-576cbf47c7-92hb5                1/1     Running   1          23h
coredns-576cbf47c7-jb54r                1/1     Running   1          23h
etcd-k8s-master01                       1/1     Running   3          23h
etcd-k8s-master02                       1/1     Running   3          22h
etcd-k8s-master03                       1/1     Running   1          22h
kube-apiserver-k8s-master01             1/1     Running   3          22h
kube-apiserver-k8s-master02             1/1     Running   1          22h
kube-apiserver-k8s-master03             1/1     Running   2          22h
kube-controller-manager-k8s-master01    1/1     Running   2          22h
kube-controller-manager-k8s-master02    1/1     Running   2          22h
kube-controller-manager-k8s-master03    1/1     Running   2          22h
kube-proxy-gvsf7                        1/1     Running   0          22h
kube-proxy-nczlh                        1/1     Running   0          22h
kube-proxy-pzq52                        1/1     Running   0          22h
kube-scheduler-k8s-master01             1/1     Running   3          23h
kube-scheduler-k8s-master02             1/1     Running   2          22h
kube-scheduler-k8s-master03             1/1     Running   1          22h
kubernetes-dashboard-7954d796d8-lqms7   1/1     Running   0          22h
i set the vip like this.
"ifconfig eth0:1 <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast
<IP_ADDRESS> up"
in my master server i got this result like below
root@k8s-master01:~# curl -k https://<IP_ADDRESS>:30000
https://<IP_ADDRESS>:30000

<!doctype html>     <link rel="icon" t*
~~~~

and in my local pc got this result
rohyr@rohyr-Samsung-DeskTop-System:~$ curl -k
https://<IP_ADDRESS>:30000 https://<IP_ADDRESS>:30000
*curl: (7) Failed to connect to <IP_ADDRESS> port 30000: *
2018년 10월 25일 (목) 오후 12:42, cookeem<EMAIL_ADDRESS>작성:

show me the kube-dashboard status, run command below:
kubectl get pods -n kube-system
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cookeem/kubeadm-ha/issues/36#issuecomment-432903941,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZTXGDwmsthRUhNtElSKa5c_K8xP6Vt_ks5uoTMTgaJpZM4XxrPd
.

Yes, the dashboard is normal, make sure the port not block by your cloud provider, you should check your cloud provider’s firewall settings

thanks you~!
i'll check my cloud setting.
i'm using digital ocean cloud.
thank you very much!!! your kubernetes project is really helpful to me
2018년 10월 25일 (목) 오후 8:18, cookeem<EMAIL_ADDRESS>작성:

Yes, the dashboard is normal, make sure the port not block by your cloud
provider, you should check your cloud provider’s firewall settings
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cookeem/kubeadm-ha/issues/36#issuecomment-433012443,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZTXGPxG0p9OKMJcaEihKQ8Idt3hOT4Gks5uoZ4ggaJpZM4XxrPd
.

You are welcome, I think this issue should be closed


Update Dockerfile to fix integration tests

Which issue does this PR close?

Closes https://github.com/apache/arrow-datafusion/issues/1979
Rationale for this change

Fix integration tests
What changes are included in this PR?

Update Dockerfile to use new crates
Are there any user-facing changes?

No

Thanks for fixing the tests 👍

x-post from another PR:
I checked the pipeline definition the integration test is not part of it. I think at some point we should include it. @andygrove @alamb should we do that now? I guess one concern might be if we want it to block the whole DataFusion pipeline if this fails

It seems I was unable to run the integration tests using this PR locally due to an arrow compilation error:
run:
dev/integration-tests.sh

Finally get:
error: could not compile `arrow`

Caused by:
  process didn't exit successfully: `rustc --crate-name arrow --edition=2021 /usr/local/cargo/registry/src/github.com-1ecc6299db9ec823/arrow-10.0.0/src/lib.rs --error-format=json --json=diagnostic-rendered-ansi,artifacts,future-incompat --crate-type lib --emit=dep-info,metadata,link -C opt-level=3 -C linker-plugin-lto -C codegen-units=1 --cfg 'feature="comfy-table"' --cfg 'feature="csv"' --cfg 'feature="csv_crate"' --cfg 'feature="default"' --cfg 'feature="flatbuffers"' --cfg 'feature="ipc"' --cfg 'feature="prettyprint"' --cfg 'feature="rand"' --cfg 'feature="test_utils"' -C metadata=d397f8e267aeac42 -C extra-filename=-d397f8e267aeac42 --out-dir /tmp/ballista/target/release/deps -L dependency=/tmp/ballista/target/release/deps --extern bitflags=/tmp/ballista/target/release/deps/libbitflags-5182ee06b0261ea5.rmeta --extern chrono=/tmp/ballista/target/release/deps/libchrono-0093b5093d2d1ae8.rmeta --extern comfy_table=/tmp/ballista/target/release/deps/libcomfy_table-57c2aefe72826335.rmeta --extern csv_crate=/tmp/ballista/target/release/deps/libcsv-a9cbe2362b55a77e.rmeta --extern flatbuffers=/tmp/ballista/target/release/deps/libflatbuffers-5c4a4352ba21817e.rmeta --extern half=/tmp/ballista/target/release/deps/libhalf-d38738f000b3c670.rmeta --extern hex=/tmp/ballista/target/release/deps/libhex-7b34d2c291b6ed12.rmeta --extern indexmap=/tmp/ballista/target/release/deps/libindexmap-125fff6d7b2ad6d4.rmeta --extern lazy_static=/tmp/ballista/target/release/deps/liblazy_static-be01e6c7b419929c.rmeta --extern lexical_core=/tmp/ballista/target/release/deps/liblexical_core-86d4f438bc1a25c8.rmeta --extern multiversion=/tmp/ballista/target/release/deps/libmultiversion-133ec2520e3b23c5.rmeta --extern num=/tmp/ballista/target/release/deps/libnum-2f517b9ac25baa4d.rmeta --extern rand=/tmp/ballista/target/release/deps/librand-1ae75b019c023c85.rmeta --extern regex=/tmp/ballista/target/release/deps/libregex-e51999868ae7fc39.rmeta --extern serde=/tmp/ballista/target/release/deps/libserde-bb6f55394b459156.rmeta --extern serde_derive=/tmp/ballista/target/release/deps/libserde_derive-f7dad95eddccdc1b.so --extern serde_json=/tmp/ballista/target/release/deps/libserde_json-12e43b16bda7b1dc.rmeta --cap-lints allow` (signal: 9, SIGKILL: kill)
warning: build failed, waiting for other jobs to finish...
error: build failed
The command '/bin/sh -c cargo build $RELEASE_FLAG' returned a non-zero code: 101

@realno We have an issue open for adding these tests to CI [1]. I remember there was an effort to implement this some time ago but there were some technical challenges, IIRC. It would be great if someone could have another go at this.
[1] https://github.com/apache/arrow-datafusion/issues/321

@yjshen Could you file a follow-on issue if you are still not able to compile, with some more detail about the error.


Mongo.Pool

Is pool still available? Can't seem to use it with v2.1.. And if it's not used anymore, what is the alternative?

Found it. Just need to pass name to start_link.
For those who tried to make it work with phoenix:
worker(Mongo, [ [database: "test", name: :mongo] ])


fix(theme): theme was not applying, big refactor was needed

Summary
How did you test this change?
Images/Videos

Current dependencies on/for this PR:

main

PR #200   👈

PR #201

This comment was auto-generated by Graphite.


Fix network stack

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: davivel:x: ggdiez

👍 Great job!! @ggdiez @davivel


Update addresses.json

[ ] ⚠ The title contains the address? Please make sure that the title of this PR includes the address

Address
B62qq7BzBtJ1HPu6fMztfkKggJYBYypRJvU3BMGkfke2VFQgfmmLkrd
Evidence
Memo: airdropmina.info - Airdrop #1
Link
[The link to the explorer e.g. https://minaexplorer.com/wallet/B62qq7BzBtJ1HPu6fMztfkKggJYBYypRJvU3BMGkfke2VFQgfmmLkrd

@nerdvibe Hey! When will this PR be approved?

sorry for the delay, will merge this now.
@wjdfx @EmrePiconbello , I will make you admins too of this repo, so you can merge PRs too :)

(will do in a couple of hours)


changes

Would you mind describing this change for commit logs and tracking. Why is it needed and what it does. Thanks!

Also please fill out the CLA.

Closing due to lack of response.


pagination should be easier

currently, pagination requires many lines of code:
async with Client() as client:
    users, resp, _ = await client.list_users()
    while users:
        for user in users:
            print(user.profile.login)
        users, _ = await resp.next() if resp.has_next() else (None, None)

my 2 line PR would reduce some of the boilerplate code (the last line above ^^^) to:
        users, _ = await resp.next()

it'd be even nicer to have an async generator to either return each page, or each object in the page (eg, Users). basically take the 5 lines above and move them into the SDK itself. then use async for to get the users:
# in the SDK `UserClient` class:
async def page_users(self):
    users, resp, _ = await self.list_users()
    while users:
        for user in users:
            yield user
        users, _ = await resp.next()

# call the SDK code:
async for user in client.page_users():
    print(user.profile.login)

this is more Pythonic, and similar to how other Okta SDKs work, eg, Node.js (https://github.com/okta/okta-sdk-nodejs#serial-or-parallel-synchronous-work), and I think Java, too, but I can't tell from the example.
Node.js example https://github.com/okta/okta-sdk-nodejs#list-all-org-users
// Node.js example

// You can also use async iterators.
for await (let user of client.listUsers()) {
    console.log(user);
}

my code doesn't handle errors, but it could. error handling should use try--it's more Pythonic.
see also https://github.com/okta/okta-sdk-python/issues/326#issuecomment-1314496367

it looks like this was rolled back due to breaking tests. is there a plan to fix the tests?


Unhandled Exception: SqliteException(1): index activities_index already exists

I am already using indexes in a while with .moor files and in it, I have a long list of indexes which the first is:
CREATE INDEX activities_index ON activities(uid);
I often use migrations to bump the database scheme, but none of them affected this table and I didn't have any problems with the changes.
But now, and for a while, I got this error every time I open the app:
E/flutter ( 5822): [ERROR:flutter/lib/ui/ui_dart_state.cc(177)] Unhandled Exception: SqliteException(1): index activities_index already exists
E/flutter ( 5822): #0      DatabaseImpl.execute (package:sqlite3/src/impl/database.dart:110:7)
E/flutter ( 5822): #1      _VmDelegate._runWithArgs (package:moor/src/ffi/vm_database.dart:160:11)
E/flutter ( 5822): #2      _VmDelegate.runCustom (package:moor/src/ffi/vm_database.dart:170:11)
E/flutter ( 5822): #3      _ExecutorWithQueryDelegate.runCustom.<anonymous closure> (package:moor/src/runtime/executor/helpers/engines.dart:79:19)
E/flutter ( 5822): #4      _ExecutorWithQueryDelegate._synchronized (package:moor/src/runtime/executor/helpers/engines.dart:25:26)
E/flutter ( 5822): #5      _ExecutorWithQueryDelegate.runCustom (package:moor/src/runtime/executor/helpers/engines.dart:76:12)
E/flutter ( 5822): #6      QueryEngine.customStatement.<anonymous closure> (package:moor/src/runtime/api/query_engine.dart:301:23)
E/flutter ( 5822): #7      QueryEngine.doWhenOpened.<anonymous closure> (package:moor/src/runtime/api/query_engine.dart:98:64)
E/flutter ( 5822): #8      _rootRunUnary (dart:async/zone.dart:1194:47)
E/flutter ( 5822): #9      _CustomZone.runUnary (dart:async/zone.dart:1097:19)
E/flutter ( 5822): #10     _FutureListener.handleValue (dart:async/future_impl.dart:150:18)
E/flutter ( 5822): #11     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:703:45)
E/flutter ( 5822): #12     Future._propagateToListeners (dart:async/future_impl.dart:732:32)
E/flutter ( 5822): #13     Future._completeWithValue (dart:async/future_impl.dart:536:5)
E/flutter ( 5822): #14     Future._asyncCompleteWithValue.<anonymous closure> (dart:async/future_impl.dart:574:7)
E/flutter ( 5822): #15     _rootRun (dart:async/zone.dart:1186:13)
E/flutter ( 5822): #16     _CustomZone.run (dart:async/zone.dart:1090:19)
E/flutter ( 5822): #17     _CustomZone.runGuarded (dart:async/zone.dart:994:7)
E/flutter ( 5822): #18     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:1034:23)
E/flutter ( 5822): #19     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
E/flutter ( 5822): #20     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)

It happens even when I create the database for the first time!
In one of my migrations, I dropped and recreated a table, but it was long ago and I am not sure if the error started at that time.
I use:
  sqflite_sqlcipher: ^1.0.0+6
  moor:
  encrypted_moor:
    git:
      url: https://github.com/simolus3/moor.git
      path: extras/encryption
      ref: 348fb655cefb1ee777c76935cdb0c873886c393e

  sqlcipher_flutter_libs:
    git:
      url: https://github.com/simolus3/sqlite3.dart.git
      path: sqlcipher_flutter_libs
  sqflite: ^1.1.7

dev_dependencies:
  moor_generator:
  build_runner:

Any suggestions? Thank you! : )

I close this because the error stopped showing up by itself after a while.
Thanks for your time! : )


Where can I get a deepspeed checkpoint for the bing finetune example?

The documentation mentions that a TF, PT, or HF checkpoint will work for the bing finetune example, but it looks like to run the code as is and try to use any of those checkpoints, it'll still run this section of the code
https://github.com/microsoft/DeepSpeedExamples/blob/master/BingBertSquad/nvidia_run_squad_deepspeed.py#L831
Which ends up saying
"Unable to find model state in checkpoint"

Even it I try to insert an arg
args.ckpt_type = "HF"

It still tries to run that section of code somehow.
So it looks like you need a deepspeed checkpoint, otherwise there is a
ValueError: Should NOT use --preln if the loading checkpoint doesn't use pre-layer-norm.

error
I also tried altering the code so it uses convert_ckpt_to_deepspeed but then I run into ValueError: Invalid ckpt_type.. So somehow that function is not able to use HF checkpoints.

Fixed by removing --preln , removing quotes from the .sh file, and adding --deepspeed_transformer_kernel
All this and more is actually documented here
https://www.deepspeed.ai/tutorials/bert-finetuning/#loading-huggingface-and-tensorflow-pretrained-models
I foolishly was trying to get the instructions to work at the top of the page before moving on to the next sections.


Fix Vint error in java maker's s:ReadClassPathFile

This also prefers Python 3 instead of 2.

/cc @wsdjeg

LGTM


protobuf not exist

Currently I'm using unity 5.3.8f. after loding 'FourD' scene I got almost 14 error and all are because of protobuf not exist or  type of namespace protobuf not exist error. What can I do to remove this error?

after installing protobuf error is gone.


Update MD file on new branch 2017-11-20_13-31-03

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

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


input doesn't have styles

Expected behavior
I wanna get styles for input like in docs:

Actual behavior
But I get this:

Other styles work fine.
I do not understand what I'm doing wrong.

Is there a way to set style={} for DatePicker component?

You can use the customInput property to make this happen - at least it worked for my purposes
https://github.com/Hacker0x01/react-datepicker/blob/e052b4ad349c92670b63c0a19d69a6a5f1fb6396/src/index.jsx#L573
<DatePicker
  selected={this.state.starts_at}
  timeFormat="HH:mm"
  dateFormat="LLL"
  onChange={(day) => this.handleDateChange('starts_at', day)}
  customInput={<input style={INPUT_STYLE} />}
/>


Plugin crashed a server.

I just installed the plugin of the latest build (v11), but after I put this plugin in the /config/mods directory, then my server crashes with the following error:
[07-24-2022 16:43:58] [I] 1 mods loaded.
[07-24-2022 16:43:58] [I] Server loaded. Type 'help' for help.
[07-24-2022 16:43:58] [E] java.lang.RuntimeException: Error loading mod moreCommands
        at mindustry.mod.Mods.contextRun(Mods.java:707)
        at mindustry.mod.Mods.lambda$eachClass$27(Mods.java:695)
        at arc.struct.Seq.each(Seq.java:192)
        at mindustry.mod.Mods.eachClass(Mods.java:695)
        at mindustry.server.ServerLauncher.init(ServerLauncher.java:74)
        at arc.backend.headless.HeadlessApplication.mainLoop(HeadlessApplication.java:65)
        at arc.backend.headless.HeadlessApplication$1.run(HeadlessApplication.java:53)
Caused by: java.lang.NoSuchFieldError: net
        at moreCommandsPlugin.init(moreCommandsPlugin.java:35)
        at mindustry.mod.Mods.lambda$eachClass$26(Mods.java:695)
        at mindustry.mod.Mods.contextRun(Mods.java:705)
        ... 6 more

Versions:
[07-24-2022 16:51:39] [I] Version: Mindustry 7-pre-alpha official / build 135.2
[07-24-2022 16:51:39] [I] Java Version: <IP_ADDRESS>

Plugin is not available for v7 for now, i working on thé new update

@aloyen

Issue pending.... It will closes now.

Issue pending.... It will closes now.


I have some trouble

Ask your question here:
After install it,show the error and I　cannt open the web.

current kubespider is a non-webui application.

I will read the usage again.


Confusing to use with other 'heavy' (transpiling) loaders

Thanks for trying to tame bootstrap's complexity!
As noted in #3, it is challenging to use this with other loaders including Babel, JSX.

At the very least some README points about this would help.
I also don't really understand why this is an issue :)  With other loaders, I can add them to the 'loaders' section, and add a test.

Updated README.


complied

Failed to execute goal org.apache.maven.plugins:maven-shade-plugin:3.1.0:shade (default) on project spark-atlas-connector-assembly: Error creating shaded jar: duplicate entry: META-INF/services/com.hortonworks.spark.atlas.com.fasterxml.jackson.core.ObjectCodec -> [Help 1]
[ERROR]

Any update on this issue? I am facing the same issue.


Bad UTF-8 encoding 0xff Error  (at offset 0)

I am getting this error while trying to open a .json file.
So my application downloada zip file and extracts it with the archive module
extractFiles() async{
    final dir = await Storage().localPath;
    final _directory = Directory(dir+"/bizzapp");
    List<FileSystemEntity> _files;
    _files  = _directory.listSync(recursive: true, followLinks: false);
    // Read the Zip file from disk.
    List<int> bytes = new File(_files[0].path).readAsBytesSync();

    // Decode the Zip file
    Archive archive = new ZipDecoder().decodeBytes(bytes);

    // Extract the contents of the Zip archive to disk.
    for (ArchiveFile file in archive) {
      String filename = file.name;
      if (file.isFile) {
        List<int> data = file.content;
        new File(dir+'/bizzapp/' + filename)
          ..createSync(recursive: true)
          ..writeAsBytesSync(data);
      } else {
        new Directory(dir+'/bizzapp/' + filename)
          ..create(recursive: true);
      }
    }
  }

then trying to open the file
getFilesFromDevices() async {
    final dir = await Storage().localPath;
    final _directory = Directory(dir+"/bizzapp/MData");
    setState(() {
      _files = _directory.listSync(recursive: true, followLinks: false);
      print({"File Path ",_files.map((f)=>f.path)});
    });
  }

 //on button click calling this method
 Future<String> loadAssetFromJson(path) async {
    return await rootBundle.loadString(path);
  }

After clicking on the button i am getting this error

E/flutter ( 3141): [ERROR:flutter/lib/ui/ui_dart_state.cc(148)] Unhandled Exception: Exception: FormatException: Bad UTF-8 encoding 0xff (at offset 0)
E/flutter ( 3141): # 0      _Utf8Decoder.convert (dart:convert/utf.dart:518:13)
E/flutter ( 3141): # 1      Utf8Decoder.convert (dart:convert/utf.dart:311:13)
E/flutter ( 3141): # 2      Utf8Codec.decode (dart:convert/utf.dart:59:56)
E/flutter ( 3141): # 3      AssetBundle._utf8decode
package:flutter/…/services/asset_bundle.dart:79
E/flutter ( 3141): # 4      _IsolateConfiguration.apply
E/flutter ( 3141): # 5      _spawn.
package:flutter/…/foundation/isolates.dart:115
E/flutter ( 3141): # 6      Timeline.timeSync (dart:developer/timeline.dart:161:22)
E/flutter ( 3141): # 7      _spawn
package:flutter/…/foundation/isolates.dart:113
E/flutter ( 3141): # 8      _startIsolate. (dart:isolate/runtime/libisolate_patch.dart:303:17)
E/flutter ( 3141): # 9      _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:171:12)

I tried opening the extracted file from another json application and it failed,
I tried connecting phone to the pc and checked storage and there is no bizzapp folder in the local storage while its clearly visible in the mobile's file manager. i sent the file from another medium to the pc and opened it and it opened correctly
This is really weird, Is there anything wrong with the way i am extracting the files?

It looks like the file is not a valid UTF-8 file and so the codec is choking on it. UTF-8 supports an optional byte-order mark at the beginning of the file which can help with endianness and signalling whether a byte stream is utf-8, utf-16, etc.
UTF-8 files that include this mark should begin with the (hex) bytes fe bb bf. Note that there's no ff byte there. The optional UTF-16 BOM looks like fe ff, which would come in as ff fe if the consuming system has the opposite endianness of the producing system. Are you sure your file isn't UTF-16?
As a fix, I'd suggest opening the file in a hex editor and just removing the leading 0xff 0xfe bytes.


Doc update: clarify implications of default version config option

The version string is not just date but combination of date and time
In one of my projects we were running multiple replicas of frontend app and each replica on start up was running building the project before serving the app. Although the code is same which is being built is same in each of those replicas but the output of build is not same because builds did not complete at the same time and hence this version field was not same in the replicas.
Whenever the user's request is served from a different replica, the service worker will update itself.
This PR does not solve this problem but it communicates it in the documentation.

@diwakar-s-maurya yeah, that makes sense. Also can use '[hash]' string for consistent builds. Default Date didn't mean to produce something consistent. It's good to clarify that, yes.

Thanks 👍