AttributeError: 'DataFrame' object has no attribute 'dtype'

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug, a feature request, or a significant problem with documentation (for small docs fixes please send a PR instead).
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

I'm using the FashionMNIST code tutorial in   https://goo.gl/GtM6ov:
Mac OS 10.13.3:
Source:
TensorFlow version (1.8.0):
*Python 3.6.5 :: Anaconda, Inc.**:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

train_input_fn = tf.estimator.inputs.numpy_input_fn(
x={"pixels": features.values/255},
y=labels,
batch_size=100,
num_epochs=3,
shuffle=True)
feature_columns = [tf.feature_column.numeric_column("pixels", shape=784)]
classifier = tf.estimator.LinearClassifier(
feature_columns = feature_columns,
n_classes = 10,
model_dir = "./models/linear1"
)
classifier.train(input_fn=train_input_fn)

AttributeError                            Traceback (most recent call last)
 in ()
----> 1 classifier.train(input_fn=train_input_fn)
~/anaconda3/lib/python3.6/site-packages/tensorflow/python/estimator/estimator.py in train(self, input_fn, hooks, steps, max_steps, saving_listeners)
361
362     saving_listeners = _check_listeners_type(saving_listeners)
--> 363     loss = self._train_model(input_fn, hooks, saving_listeners)
364     logging.info('Loss for final step: %s.', loss)
365     return self
~/anaconda3/lib/python3.6/site-packages/tensorflow/python/estimator/estimator.py in _train_model(self, input_fn, hooks, saving_listeners)
841       return self._train_model_distributed(input_fn, hooks, saving_listeners)
842     else:
--> 843       return self._train_model_default(input_fn, hooks, saving_listeners)
844
845   def _train_model_default(self, input_fn, hooks, saving_listeners):
~/anaconda3/lib/python3.6/site-packages/tensorflow/python/estimator/estimator.py in _train_model_default(self, input_fn, hooks, saving_listeners)
851       features, labels, input_hooks = (
852           self._get_features_and_labels_from_input_fn(
--> 853               input_fn, model_fn_lib.ModeKeys.TRAIN))
854       worker_hooks.extend(input_hooks)
855       estimator_spec = self._call_model_fn(
~/anaconda3/lib/python3.6/site-packages/tensorflow/python/estimator/estimator.py in _get_features_and_labels_from_input_fn(self, input_fn, mode)
689   def _get_features_and_labels_from_input_fn(self, input_fn, mode):
690     """Extracts the features and labels from return values of input_fn."""
--> 691     result = self._call_input_fn(input_fn, mode)
692     # TODO(anjalisridhar): What about the default DistributionStrategy? Perhaps
693     # using any input is alright in that case. There is also a
~/anaconda3/lib/python3.6/site-packages/tensorflow/python/estimator/estimator.py in _call_input_fn(self, input_fn, mode)
796       kwargs['config'] = self.config
797     with ops.device('/cpu:0'):
--> 798       return input_fn(**kwargs)
799
800   def _call_model_fn(self, features, labels, mode, config):
~/anaconda3/lib/python3.6/site-packages/tensorflow/python/estimator/inputs/numpy_io.py in input_fn()
194         num_threads=num_threads,
195         enqueue_size=batch_size,
--> 196         num_epochs=num_epochs)
197
198     batch = (
~/anaconda3/lib/python3.6/site-packages/tensorflow/python/estimator/inputs/queues/feeding_functions.py in _enqueue_data(data, capacity, shuffle, min_after_dequeue, num_threads, seed, name, enqueue_size, num_epochs, pad_value)
390     elif isinstance(data, collections.OrderedDict):
391       types = [dtypes.int64
--> 392               ] + [dtypes.as_dtype(col.dtype) for col in data.values()]
393       queue_shapes = [()] + [col.shape[1:] for col in data.values()]
394       get_feed_fn = _OrderedDictNumpyFeedFn
~/anaconda3/lib/python3.6/site-packages/tensorflow/python/estimator/inputs/queues/feeding_functions.py in (.0)
390     elif isinstance(data, collections.OrderedDict):
391       types = [dtypes.int64
--> 392               ] + [dtypes.as_dtype(col.dtype) for col in data.values()]
393       queue_shapes = [()] + [col.shape[1:] for col in data.values()]
394       get_feed_fn = _OrderedDictNumpyFeedFn
~/anaconda3/lib/python3.6/site-packages/pandas/core/generic.py in getattr(self, name)
4370             if self._info_axis._can_hold_identifiers_and_holds_name(name):
4371                 return self[name]
-> 4372             return object.getattribute(self, name)
4373
4374     def setattr(self, name, value):
AttributeError: 'DataFrame' object has no attribute 'dtype'
This is the error I'm getting.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
Exact command to reproduce

Thanks for your response. I've got the issue solved. Thank you

@attitude95 please share how your issue got solved.


The log location is wrong

At least for my OS, the logs are under are under C:\WindowsAzure\Logs\Plugins\Microsoft.Azure.Diagnostics.PaaSDiagnostics<version>\

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

ID: b3a5910a-2b07-f14a-e5b4-afdef8ec1366
Version Independent ID: 5d9f6b8c-d991-dac6-8d15-010f06a678b4
Content: Troubleshooting Azure Diagnostics extension
Content Source: articles/azure-monitor/platform/diagnostics-extension-troubleshooting.md
Service: azure-monitor
Sub-service: diagnostic-extension
GitHub Login: @rboucher
Microsoft Alias: robb

@ciufudean Thanks for the comment. We are looking into it and we will update you soon.

@ciufudean I have verified by adding the extension to VM  and it shows location of log file as "C:\WindowsAzure\Logs\Plugins\Microsoft.Azure.Diagnostics.IaaSDiagnostics" . Can you confirm if  the log location provided for the cloud service ?

@rboucher Can you please review this and update the document accordingly?

@ciufudean   I'm checking into this location with the development team to understand what has changed.  Clearly if you are seeing it in a different place, something is going on.

Still waiting to hear back.  Pinged dev again.

@ciufudean Dev says they didn't change the location so I'm not sure why it's an issue on your side.  It's going to vary based on what you are hooked to.

Hi, the straight question is if they say or if they tested my issue.
Anyway, I am not working in this area anymore, so do as your conscience
dictates :)
On Fri, Apr 10, 2020 at 3:38 AM Rob Boucher Jr (Microsoft) <
<EMAIL_ADDRESS>wrote:

@ciufudean https://github.com/ciufudean Dev says they didn't change the
location so I'm not sure why it's an issue on your side. It's going to vary
based on what you are hooked to.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/41219#issuecomment-611818606,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AESRV6WBJOIXUKD7NZ2BTJLRLZTBPANCNFSM4JDPLK6A
.

@cuifudean They tested and looked at the configuration of the underlying components.  It's probably related to what you were using. If you remember, let me know if it was Virtual Machines, Cloud Services or Service Fabric.  If you don't remember that's fine.  Thanks for responding.

#please-close


Error in TestPackage serialization

The code for serialization in TestPackage.WriteXml works only when a Settings element is present. When there are no settings (which is rare) a package containing three subpackages
Package
  Subpackage1
  Subpackage2
  Subpackage3

is incorrectly serialized as if it were
Package
  Subpackage1
    Subpackage2
       Subpackage3

This is probably due to incorrect handling of EndElement in the code

:tada: This issue has been resolved in version 2.0.0-beta4 :tada:
The release is available on:

GitHub Release
NuGet Package


docs: Fix inconsistencies

Description
Changes proposed in this pull request (what was done and why):

I may or may not have found copy/paste errors in our docs, pls check: Do we really want to talk about logs here?

Changes refer to particular issues, PRs or documents:

Traceability

[ ] The PR is linked to a GitHub issue.
[ ] New features have a milestone set.
[ ] New features have defined acceptance criteria in a corresponding GitHub Issue, and all criteria are satisfied with this PR.
[ ] The corresponding GitHub issue has a respective area and kind label.
[ ] The follow-up issues (if any) are linked in the Related Issues section.
[ ] Adjusted the documentation if the change is user-facing.
[ ] The feature is unit-tested
[ ] The feature is e2e-tested

will check this out tomorrow, didn't manage to do it today


tests: add coverage for common/helpers.go

This also changes the helpers.go to enable mocking all the different architectures.  If you have a better idea for the name of the newly created variable I'll be glad to hear it :)
The runtime.GOARCH is actually a constant that is taken from sys package https://golang.org/pkg/runtime/#pkg-constants
Refs #419

Is sys.GOARCH also a constant or a function/variable ? I can't find it's definition.
I'm wondering if the tests could be overriding that instead?

Is sys.GOARCH also a constant or a function/variable ? I can't find it's definition.
I'm wondering if the tests could be overriding that instead?

It seems to also be a constant that is configured by during GO installation on the system I suppose?
You mean that it might be possible to modify that constant somewhere instead of mocking it here like I did? Sounds doable, I can try that if we'd rather prefer it that way.

I can see it running go env and it seems to be possible to change using os.Setenv.

Is sys.GOARCH also a constant or a function/variable ? I can't find it's definition.

It's constant, thus it cannot be modified. Defined in /usr/lib/golang/src/runtime/extern.go.


Avoid view integrations tracking loading state manually

Right now, it seems like both react-apollo and angular2-apollo are tracking loading state by doing manual bookkeeping. But this doesn't seem optimal, because:

Apollo Client already keeps track of loading state internally
Some options to watchQuery, for example returnPartialData, might return data while loading is still true, confusing the view integrations

I think we should have a way to get the loading state from apollo directly, so that other libraries don't have to guess what it is.
@jbaxleyiii @kamilkisiela do you have some ideas for how it should be best returned? Some ideas:

Apollo Client returns a queryId, and has a getLoadingState method
The observable returned by watchQuery has a getLoadingState method
The result from the observable, which is currently a GraphQLResult with data and errors, can get a new property, loading

I think (3) is the best, would doing that be enough to remove the need to keep track of loading state in react-apollo and angular2-apollo?

@stubailo Totally agree that loading state should be delivered to UI integrations directly from ApolloClient.
3rd option seems to be the best one.

would doing that be enough to remove the need to keep track of loading state in react-apollo and angular2-apollo

Yes, it would :+1:

@stubailo It seems like https://github.com/apollostack/apollo-client/pull/385 has added the queryId to the result of calling watchQuery, so the queryId can now be used to read directly from the state of apollo.queries.

Oh, awesome! I didn't even notice. I should publish a new version with all of these changes.

@stubailo haha yeah same, I was about to open up a PR for that tweak, but noticed the new changes in master beat me to it! I added a note to the changelog - https://github.com/apollostack/apollo-client/pull/405

@stubailo Do we stay with queryId and apollo.queries or there's going to be that 3rd option implemented?

The result from the observable, which is currently a GraphQLResult with data and errors, can get a new property, loading

apollostack/angular2-apollo#43

@stubailo @kamilkisiela I would love for this to be done!

This should be fixed through #467 and then we can work on implementing the required changes within the view layers


Changing the background color of the navigation bar

I wish to know if there is any way to change and customise the background colour of the navigation bar created by the SPFakeBar library?

You can add to blurView your subview.

I am sorry I am new to IOS development, may I ask how to do that ?


fix CI pipeline

[x] Adds Github workflow for unit tests
[x] fixes broken link in README
[x] all green CI
[x] remove .dev from version (this currently breaks jupyter-releaser, but is being fixed).

@Zsailer I fixed that flake8 hook failing in pre-commit and re-enabled it in CI. It seems weird that we need so much configuration and compatibility code to get these various linters working together. Was it ever discussed why we need both black and flake8? I think we should consider dropping one of these two to prevent further effort being wasted here.
If this hasn't been discussed yet, I'll probably bring it up in the next Server meeting.

@Zsailer Something else that's weird: when you commented out the pre-commit step in CI, it also disabled almost all of the tests from running in CI. Not sure why that happened.

Thanks!


Feature request: Add CompletionProvider

Hi, thanks for the extension.
Could you add a CompletionProvider like https://github.com/MrLuje/Mocking.Helpers in your extension?

Hmm... I may have to download Visual Studio Community to give that extension a try, I haven't used Visual Studio in some time.
From what I can tell given the screenshots on Mocking.Helpers, however, Speedy Moq is trying to solve a different problem-- I don't want to type out mocks character by character, substitution/completion by substitution :D
I want to focus on writing my test logic, and not the <>><>()() ugliness of the (completely necessary) mocking libraries.
I'm not quite ready to put the "wontfix" label on this issue, or close it, yet, but I'm leaning that direction. I'll give it a little time to think through.

I've had a couple of days to think about this, and the more I think about it, the less useful I think code completion would be in this extension. I thank you for your feedback!


Specify some input or env that allows a user to override the default GITHUB_REPOSITORY env var

Is your feature request related to a problem? Please describe.
Using this toolkit and your create-issue action, I am unable to create an issue in a different repository than the action's repo. Our team is attempting to put as much automation in its own repository as possible to separate repo concerns and this functionality would be great!
Describe the solution you'd like
I can pass env vars of repo and owner, or a repository value with a / delimited string
Describe alternatives you've considered
I tried to override the GITHUB_REPOSITORY var. Looking at the Context class, it does look like process.env.GITHUB_REPOSITORY is checked. Unfortunately, the actions documentation explicitly says that GITHUB_ variables will be ignored, so I'm not sure how else to proceed.
Additional context
I do apologize if I missed a setting somewhere that facilitates this! For context, here is the relevant portion of our action using create-issue:
      - name: Create an issue
        uses: JasonEtco/create-an-issue@v2
        env:
          GITHUB_TOKEN: ${{ secrets.CREATE_ISSUE_TOKEN }}
          GITHUB_REPOSITORY: github/heart-services <-- this doesn't get overwritten
          START_DATE: ${{ steps.date.outputs.start }}
          END_DATE: ${{ steps.date.outputs.end }}
          USER_NAME: ${{ steps.get_first_responder.outputs.user }}
          LABELS: first-responder
        with:
          filename: .github/ISSUE_TEMPLATE/first_responder.md
          assignees: ${{ steps.get_first_responder.outputs.user }}

Thank you 😄 . Assuming you think this could be a useful feature, I'm happy to open a PR with whatever approach you think is best.

Hey, sorry this slipped last my radar. If you're still interested in this behaviour, an issue in JasonEtco/create-an-issue would be better, since it really means adding an option there.
Totally supportive of that! But note that GitHub Actions creates API tokens that are scoped to the repository they're being run in - so you'd also need to pass a personal access token that has access to the repo you want to open an issue in.

Totally missed your reply, apologies! We ended up simply adding the action to the repo in question. Maintaining PATs we're forced to rotate, and tied to team members who will certainly not be on the team forever seems like more of a hassle than its worth.
Thanks for getting back to me 😄

Maintaining PATs we're forced to rotate, and tied to team members who will certainly not be on the team forever seems like more of a hassle than its worth.

I definitely agree, I think you picked the best approach 👍  🚀


Plugin not compatible with latest version of Jetbrains IDEs (2023.2)

What would you like to be added
Support for the latest version of Jetbrains IDEs
Why is this needed
The plugin does not work on the latest version of the IDE (in my case GoLand 2023.2)
Implementation details

Same here. GoLand 2023.2 as well.

It says the latest plugin requires "Plugin 'Open Policy Agent' (version 'v0.10.0') is not compatible with the current version of the IDE, because it requires build 231.* or older but the current build is IC-232.8660.185".

@vgramer let me know if I can support you in this in any way 😃


file box related issue

Hi,
After opening any selected file inside File Box and then trying to move them into another folder inside File Box an error message appears which says that the deletion of the original file after moving it to the newer location was impossible. The copy process is also not done. The desired operation can be done only after closing and restarting the Civilizer server. Thanks.

I couldn't reproduce the issue on Mac OS X.
But I could reproduce the issue on Windows 8.1 only on a special case.
When I opened a zip file from File Box and when I tried to delete the file while my zip application is still holding that file, I couldn't delete it. ( This is a normal behavior for Windows. )
Still your information is missing details (no mention about OS version, browser, other important situational facts like the app was still running while you moved or deleted the file, etc) but it looks like your OS is Windows.
On Windows, for example, you open a .docx file with MS word from the browser, and you try to delete the file while still the MS word running (sometimes you may not notice it is still running...), and then the file can't be deleted or moved. Also this behavior may differ depending on your Windows version. I could still move a file on Windows 8.1, but it wouldn't let me delete the same file. Other Windows version even may not let you move files...
Even though, say, you properly terminated the app, but you couldn't still move the file. However, If that's an application issue, it should happen on other platforms like Mac OS X.

The desired operation can be done only after closing and restarting the Civilizer server

This is what I can't explain about. Sorry...
Well, Civilizer doesn't do any special operations in terms of moving or deleting files. It just asks OS to do that.
What I can tell for now is that this seems like an OS related issue and Civilizer can't do much about it... or more evidences are required to show that it was actually an application issue.


Update UpdatePolicyConnectionConfiguration to be 1:1:1 - Part 2

Details

Fixed Issues

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

Prerequisite:

Your workspace need to connect with Xero.

Go to Accounting.
Go to Import => Tracking Category, try to turn on or off the switch and verify that it works as expected.
Try to change the tenant and verify that it worked as expected.
Go to Import => Taxes, turn on and off switch, verify that it works as expected.

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

Offline tests

Same as Tests.
QA Steps

Same as Tests.

[x] 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: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb 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 either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[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 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(theme.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 the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[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.

Screenshots/Videos

Android: Native

https://github.com/user-attachments/assets/ac84a7af-be5b-48f5-a491-5d7a0d3e34e7

Android: mWeb Chrome

https://github.com/user-attachments/assets/c7fe2422-9ffe-4538-8fdb-d08fd9e8b9dd

iOS: Native

https://github.com/user-attachments/assets/98c22522-db35-4c0b-8681-d73d1bf6c31e

iOS: mWeb Safari

https://github.com/user-attachments/assets/7e695b6f-4b7e-46a8-9d52-520e2f8bc85d

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/89b839c9-6e65-4999-99b5-a0d1acd73f6b

MacOS: Desktop

https://github.com/user-attachments/assets/74442186-2c98-4411-af04-d56c9b1de740

It's basically ready for review, I'll complete the screenshots/videos later.

Updated all screenshots/videos.
All yours @dangrous

🚀 Deployed to staging by https://github.com/dangrous in version: 9.0.25-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/dangrous in version: 9.0.25-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Manipulating dates does not update value returned by toISOString and toJSON functions

Using toISOString and toJSON functions outputs the original date instead of the manipulated date.
http://jsfiddle.net/wdnomhr8/

So, this feedback is coming a year and a half later (sorry), but I think the problem that you are seeing here is that the original date being parsed is actually October 30 if you are in UTC -4 or further West. This is because when the default Moment constructor is used, it will convert from the offset, to the local time. Thus, for me in -5:
//October 30 at 11 PM is actual day parsed
moment("2014-10-31T04:00:00.000Z").format()
"2014-10-30T23:00:00-05:00"

//add one day, October 31 at 11 PM
moment("2014-10-31T04:00:00.000Z").add(1, 'days').format()
"2014-10-31T23:00:00-05:00"

//Start of October 31
moment("2014-10-31T04:00:00.000Z").add(1, 'days').startOf('day').format()
"2014-10-31T00:00:00-05:00"

The parsing guide has a section on this:
http://momentjs.com/guides/#/parsing/local-utc-zone/


db: reuse ROTxn reference in LocalState

This is not going to work due to MDBX transaction lifetime issues when using LocalState in worker threads.


Remove DepositData

Currently DepositData is:
{
    # Amount in Gwei
    'amount': 'uint64',
    # Timestamp from deposit contract
    'timestamp': 'uint64',
    # Deposit input
    'deposit_input': DepositInput,
}

We already discussed removing timestamp. Further, we can put amount inside DepositInput because (unlike timestamp) it is known ahead of time and can be signed over by the proof_of_possession.
This would allow us to reduce to two objects for deposits: Deposit and DepositInput. (We could then rename DepositInput to DepositData for consistency.)

Nice! Hooray for simplification.

We already discussed removing timestamp.

To clarify to those who weren't in the call, the original justification for timestamp was to put a time limit on deposit inclusion so that deposit receipts cannot be "double-spent". However, this is no longer necessary because we are forcing eth1 deposits to be processed in order.


Deploy also to Gradle plugin portal (#629)

Summary
This PR makes possible to apply the Spring Cloud Contract Gradle plugin with short syntax (as requested in #629):
plugins {
  id 'org.springframework.cloud.contract' version 'XXX.RELEASE'
}

Please note. This change keeps backward compatibility with 2.x.
Implementation
The feature is achieved by:

Adding an alias for the Gradle plugin id - making it compatible with Plugin portal requirements.
Configuring effecting publishing to Plugin portal with a dedicated plugin.

Additional configuration

Valid gradle.publish.key and gradle.publish.secret properties have to be added in CI/CD configuration (e.g. in ~/.gradle/gradle.properties.
The CI/CD server should call publishPlugins (directly or that task can be - after testing - made a requirement for the general release task).

Remarks/concerns

This solution keeps backward compatibility with the 2.x line - the old plugin id is still available. There's just an extra id alias which effectively will be the main one in the future.
It's perfectly fine to have more than one plugin defined in a single JAR - see my test plugins and their configuration.
I wasn't able to test it in that project, so some glitches can occur (however, in general, the configuration like that has chance to work - as mine which it is based on).
I connected releasing to the portal to releasing to Maven Central. You may change it if desired.
I propose you to release some RC version first (before the final one or maybe even a milestone - by temporary slightly modifying release.gradle) to ensure the correctness
Once successfully released and tested the old plugin id could be marked as deprecated to be eventually removed in some future version.

Update. I switched from master to 2.1.x which seems to be the main branch to merge commits to.

The test failures are rather not related to my changes:
Your job ran 1393 tests with 3 failures

    org.springframework.cloud.contract.stubrunner.StubConfigurationSpec - org.springframework.cloud.contract.stubrunner.StubConfigurationSpec

java.lang.NoSuchMethodError: org.junit.platform.commons.util.ReflectionUtils.tryToReadFieldValue(Ljava/lang/Class;Ljava/lang/String;Ljava/lang/Object;)Lorg/junit/platform/commons/function/Try;

org.springframework.cloud.contract.stubrunner.StubRunnerPropertyUtilsSpec - org.springframework.cloud.contract.stubrunner.StubRunnerPropertyUtilsSpec

java.lang.NoSuchMethodError: org.junit.platform.commons.util.ReflectionUtils.tryToReadFieldValue(Ljava/lang/Class;Ljava/lang/String;Ljava/lang/Object;)Lorg/junit/platform/commons/function/Try;

org.springframework.cloud.contract.stubrunner.spring.cloud.consul.StubRunnerSpringCloudConsulAutoConfigurationSpec - org.springframework.cloud.contract.stubrunner.spring.cloud.consul.StubRunnerSpringCloudConsulAutoConfigurationSpec

java.lang.NoSuchMethodError: org.junit.platform.commons.util.ReflectionUtils.tryToReadFieldValue(Ljava/lang/Class;Ljava/lang/String;Ljava/lang/Object;)Lorg/junit/platform/commons/function/Try;

That's my fault. I'll fix it and please rebase against master

Much better after rebase :)

We can call $ ./gradlew publishPlugins -Pgradle.publish.key=<key> -Pgradle.publish.secret=<secret> to upload the plugin


677 bug major profile page mobile visualization problems

Motivation and context
This is needed since the donations tab was not sticking to the design and was not responsive at all.
It fixes #677.
There are still things that are not ready:

The table that's showing the donation amounts is hardcoded with the userDonations.amount, because I didn't know how to check if there are recurring donations, donations by card or donations by bank
The campaign that is showing is just the first campaign from the campaigns list.

This is the actual design from figma.
https://www.figma.com/file/MmvFKzUv6yE5U2wrOpWtwS/Podkrepi.bg?node-id=10340%3A31392

Important notes
Also changed the Dockerfile to use node:16.14.2-alpine3.15 and the .env.local.example to be localhost:8080/auth instead of the actual dev keycloak server.

Screenshots:

Before
After

Made them both use useMemo hooks, however they are currently chained, because first the type filters get applied and then the result of it is passed as a dependency to the date filtering so it can filter 'over' the type filters.
It's still not that readable and someone might get confused. If you have better ideas I am open.


About the training epochs

Hi,
In your paper, you said training the depth and pose network takes around 30 epochs. Training the residual flow network takes around 200 epochs. I have prepared the data for training using prepare_train_data.py file.  How many iterations are 30 epochs and 200 epochs? I saw the examples in Readme are 350000 iterations and 400000 iterations.
Best,
Zhai

Ok, thank you for your reply. In your training processing, you made --seq_length=3. Does three pictures count as a training sample? And after I use prepare data file provided by you, for example, for flow, the formatted data does not contain all of KITTI raw dataset sequences, just part of scenes. Is right? We can calculate the training samples using for training depth and pose network, 350000/30=11666?

@yzcjtr And I reproduce the depth results when the training iterations is 150k. How many iterations can I reproduce the flow results?

I think it's okay if you follow the scripts I provided for formatting the data. As for the flow experiment, we provided details in the paper and I suggest you adopt a validation set.


Error when exporting to WASM

When I run this cli command to generate wasm binary:
GOOS=js GOARCH=wasm go build -o output/html/aneka_guess_number/script/app.wasm cmd/html/aneka_guess_number/app.go
I got this error:
vendor/github.com/fatih/color/color.go:21:5: undefined: isatty.IsTerminal

I tried to resolve this by bump up the version of github.com/fatih/color, from v1.7.0 to v1.9.0. After bumping up the version, I can compile to WASM. I create the related PR to bump up the version. #34

It's my first time creating PR to open source project, please correct me if there is mistake. Thanks!

Thank you!
Merged #34


renaming a column on postgresql generates wrong code.

instead of renaming from old_column_name to new_column_name it does it to old_column_name
using dbeaver 3.6.2 as extension on eclipse 4.5.2
generated code example
ALTER TABLE table_name RENAME COLUMN old_column_name TO old_column_name;

Did not test it but looking at the source code it looks like on the end of the file: https://github.com/serge-rider/dbeaver/blob/80beac9ae06c0798fe93f36c805ad45e784feb1b/plugins/org.jkiss.dbeaver.ext.postgresql/src/org/jkiss/dbeaver/ext/postgresql/edit/PostgreTableColumnManager.java#L165
instead of DBUtils.getQuotedIdentifier(column) it should be command.getNewName() or maybe better  DBUtils.getQuotedIdentifier(command.getNewName())
on DB2TableColumnManager.java it is only command.getNewName()

Thanks for report


keylogger

could you please help me to find and add a keylogger
<EMAIL_ADDRESS>

No this is the exact reason I removed the keylogger.


GoLang InsecureSkipVerify

In the Go sample code, after making the effort to load the root CA, you turn off certificate verification with the InsecureSkipVerify flag.  This defeats much of the benefit of SSL!

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

ID: 77a94e6f-a799-494f-f2f9-171c1804e5f4
Version Independent ID: 0ca5e2ae-5889-61ef-3ce7-5c447f7519a6
Content: Configure SSL connectivity to securely connect to Azure Database for MySQL
Content Source: articles/mysql/howto-configure-ssl.md
Service: mysql
GitHub Login: @ajlam
Microsoft Alias: andrela

@aunger, thanks for the feedback. Unfortunately, I'm not as familiar with GoLang. Is the recommendation to remove InsecureSkipVerify: true?

I'm no Go expert either, but yes, I think it's clear you shouldn't set that flag.  From the docs:
        // InsecureSkipVerify controls whether a client verifies the
        // server's certificate chain and host name.
        // If InsecureSkipVerify is true, TLS accepts any certificate
        // presented by the server and any host name in that certificate.
        // In this mode, TLS is susceptible to man-in-the-middle attacks.
        // This should be used only for testing.
        InsecureSkipVerify bool


Update README.md

Thank you :-)

Could I say contributed in NodeOS?

Sort of.

So I'm ready to go for a job interview now. Bye!

So I'm ready to go for a job interview now. Bye!

That's not fair, and you know it. F*ck yourself :angry:

@piranna Let's keep it clean and respectful please.

@piranna Let's keep it clean and respectful please.

Yeah, I'm sorry for my distasteful response, but being ironic or not, that an empty profile without a background just fix a typo and want to go for a job interview as NodeOS commiter, although it's true, it's not professional by trying to get some benefict and reputation from the work of others.

I'll try to moderate my tone in the future and not take some sort of things as personal offenses, I promise.

@piranna Thank you, you're a hugely important part of this project and therefore your words have a lot of weight.
Honestly, if this contributor is able to convert a single contribution into a paying job then I would be more impressed than upset.

Cleaned thread. Locking.


Closes #200 substitute command line arguments for defaults

This simple patch make arguments specified on the command line override defaults specified in the Python script. The only messy part is allowing required arguments to be omitted on the command line. This is done by iterating through the actions and mutex groups adjusting the required and nargs field as appropriate.
I have based this patch on the master rather than the 1.0.4 release branch because the latter appears broken, as described here.
Support for this functionality has been expressed here, here and here

Hi @chriskiehl,
Thanks for the review. :)
The high level tour of the code is pretty quick. In short, it calls argparse.parse_args aka self.original_parse_args with the supplied command-line arguments, and overwrites argparse's default values with the result. The only issue that arises is how to deal with validation errors on the command line. In particular, you don't want to enforce required arguments at this stage but rather just leave them empty and let Gooey enforce the requirement. Ideally, argparse would have some arrangement where you can tell it what to enforce (like parse_known_args doesn't enforce 'unknown argument'). But as a fallback that works right now, the code iterates through the arguments and replaces any that are required (ie required == True or nargs in ['+', None]) with non-required equivalents, then calls argparse.parse_args, and finally restores the original values.
When you say move it out of the decorator, do you just mean put it into a separate function? Otherwise I'm afraid  I think it actually needs to happen in the decorator. I did write an earlier version of this functionality that didn't do the work inside the decorator and it became much more messy and complicated. Overwriting the defaults before calling config_generator.create_from_parser seems the neatest solution.
Excuse me for missing something, but what boundaries can you see that need enforcing?
Tests and documentation... OK yes I'll figure something out there.
And finally, enabling via a flag. I'm not opposed to what you say but could I first explain why I chose to make it default behaviour? As it stands, if you call a Gooey-enabled script and provide command-line arguments, those are silently ignored. It seems no worse to me that they be silently used to fill in values before the Gooey window appears, If that behaviour is to be an option, then consistency would suggest that when the option is disabled, the script should return an error code if it is called with arguments (other than --ignore-gooey). What do you think?

lgtm. Nice cleanup! One last thing, could you swap this to point at the release branch (1.0.4-release)?

Oh, also, yeah, please squash your commits down into one.


CustomSecuritySample - Forms auth returns 503 error on Reports portal

I have successfully deployed the forms auth, able to register and login with admin user against Reportingservices endpoint. However , post auth (redirect from logon page) it loads an 503 error page on /reports page. Please point to the module that may be causing this behavior.

Running into the same problem.
My implementation worked on 2014, 2016 and 2017 SQL Server.
I tried with 2019 now and /ReportServer works fine. Can open reports, etc.
But when I do access /reports I get blank page and response is Internal Server Error 500.
Checked server event logs and also all logs in '\LogFiles' folder.

RSPortal_2020_07_28_12_25_05.log log is empty.
RSHostingService...log include 2020-07-28 12:18:35.7379|INFO|9|Starting Process: C:\Program Files\Microsoft SQL Server Reporting Services\SSRS\Portal\RSPortal.exe and I do see the task running.

Any idea where I can further look or debug?

Found out the issue. Was machine key problem.
The original ps1 Script provided to create a machine key use lower case values.
I did some digging and found some posts where they mentioned this.
So reason was: could not read machine key correctly.
Solution was to add case sensitive entry in rsreportserver.config in <configuration> right below the <Dsn> setting:
<MachineKey
ValidationKey="[KEY]"
DecryptionKey="[KEY]"
Validation="AES"
Decryption="AES" />

I had those lower-case key and attributes before. Strange thing is that it worked with the service, only resportal.exe refused to work. After change to the config above it worked.

I'm still having the same issue even after I made changes as you suggested. Also, the Security Sample does not ask you to add the machine key to rsreportserver.config. Can you please let me know if you did some customization on top of the security sample?
Thanks

Hello. hard to say as I developed a whole implementation in our product so we can open reports with our users and roles. Best you can do is try and check all existing logs as the new UI is hard to debug imho. Male sure reportserver service runs completely before checkout UI

I'm still having the same issue even after I made changes as you suggested. Also, the Security Sample does not ask you to add the machine key to rsreportserver.config. Can you please let me know if you did some customization on top of the security sample?
Thanks

Hello. hard to say as I developed a whole implementation in our product so we can open reports with our users and roles. Best you can do is try and check all existing logs as the new UI is hard to debug imho. Male sure reportserver service runs completely before checkout UI

My issue is that I did not included the machine key inside the <system.web>in Microsoft.ReportingServices.Portal.WebHost.exe.config. Once I added the <sytem.web></system.web>, no more issues.

My issue is that I did not included the machine key inside the <system.web>in Microsoft.ReportingServices.Portal.WebHost.exe.config. Once I added the <sytem.web></system.web>, no more issues.

Good to know you solved it, but strange is that I did not do this, just checked.


fix: remove New Relic from public Dockerfile

Description:
Since New Relic is an edX-specific thing, we already inject this code in our internal Dockerfile repo by default.
Remove New Relic injection in Dockerfile code.
Merge checklist:

[ ] All reviewers approved
[ ] CI build is green
[x] Changelog record added
[ ] Documentation updated (not only docstrings)
[ ] Commits are squashed

I think you'll need to rebase onto latest master to pick up the pip/pip-tools fix, and squash the commits with a new commit message to pass the commit-linter.

Try rebasing again? I made a test PR (https://github.com/openedx/edx-cookiecutters/pull/180) and CI passed, so I think whatever dependency issue there was has since been resolved.


crowdin-cli: Add passthru.tests.version

Motivation for this change
Add automated testing to crowdin-cli package.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

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


Switch back to rest serializer

This is ready to go except that I would like to get confirmation that this in fact fixes #132.
Fixes #128
Fixes #130
Fixes #132

@benkonrath
_this.store.pushPayload('user', response)
store.js:1851 Uncaught TypeError: serializer.pushPayload is not a function

message: "Assertion Failed: You cannot use store#pushPayloadwithout a modelName unless your default serializer definespushPayload"

in console it does show _this.store.pushPayload so it may just be that Im not invoking it correctly (followed ember DS.store doc)..

@erichonkanen try:
let data = _this.store.normalize('user', response);
_this.store.pushPayload('user', data);

@benkonrath @dustinfarris I just ran in to (almost) the #128 problem, and this patch fixed it for me.

@erichonkanen any update?

@dustinfarris Let's just deal with #132 separately if it still needs to be addressed.

@dustinfarris  What do you think about cutting a new release. Maybe 1.1?


Major upstream release

Problem

Major upstream release, package needs tom adjustments

Solution

Adjustments made!

PR Status

[x] Code finished and ready to be reviewed/tested
[x] The fix/enhancement were manually tested (if applicable)

Automatic tests
Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

!testme


Simulation fixes for fleet (mostly VTOL)

@AndreasAntener @tumbili @sanderux @dagar The models in SITL seemed mostly unusable. I've fixed a number of issues with them and the flight performance improved significantly (from unusable to reasonably realistic). Cross-testing, in particular under joystick control, would be most appreciated.
And Merry Christmas! I guess this is my gift for the PX4 VTOL developers.
Note that the control gains and stall speeds became more realistic with these fixes. That's a good hint towards things going into the right direction.

Thanks! I'll give them a try.
As a result I was using SITL for initial development, then graduate to X-Plane HIL for testing (when possible).

@Lorenz thanks!

I'm continuing this here because I cleaned up the commits: https://github.com/PX4/Firmware/pull/6168


CS0655 Compile time error when attempting to set NullValue DateTime property

NEST/Elasticsearch.Net version: <= 7.10.0
Elasticsearch version: N/A
Description of the problem including expected versus actual behavior:
The DateAttribute and DateNanosAttribute types define a NullValue property of type DateTime which is not valid for attributes.
Steps to reproduce:

Create a POCO with a nullable DateTime property
Add the DataAttribute to the property
Autocomplete suggests an option to supply the NullValue but after attempting to set this it results in a compile time error CS0655 'parameter' is not a valid named attribute argument because it is not a valid attribute parameter type.

Expected behavior
A DateTime property should either a) not be exposed on the attribute. We should investigate alternatives such as accepting a string which at runtime is formatted to a date. We need to be mindful that date formats used for the field likely need to be respected.

Attributes are being removed in v8.0 due to lack of support with System.Text.Json interop + complexity in maintaining these.


The problem of objects penetrating each other after the gripper clamps the object

Hello, I want to collect the torque information generated by the collision at the joint, but after the gripper clamps the object, the object will pass through other objects. What should I do?

This is information about the object：

Hi, did you try to add <self_collide>1</self_collide> to the link of the object?

oh，thanks for your advice, the problem has been solved.

Great, that's good to hear! May I asked what resolved it?

I used gazebo to re-export the more detailed parameter configuration， and set the <self_collide>1</self_collide>, then the object will colide the table, the force and torque change as well.


tests: bluetooth: tester: Allow to pass int instead of bool in comp cmd

This will allow to choose more variants of cdp in future if needed.

@PavelVPV the general expectation is that PRs which should get merged during the feature freeze should have a "Fixes #..." reference to make it clear what bug they are fixing. "This will allow to choose more variants of cdp in future if needed." sounds more like enabling some future functionality rather than fixing a clear bug, so it'd help to have the actual bug that this is fixing more formally defined.


GitLab Strategy

I created gitlab strategy and it could be added to this opauth repo
here's new strategy https://github.com/ppatrik/opauth-gitlab
Tested with GitLab CE 8.7.0

can you create a pull requests for your changes? id assume, this way it gets worked into theproject faster


React reveal with React Component

Hi,
Thank you so much for your librarie.
But have any example with a Custom React Components use case please ?

Well I'm planning to document this use case a bit more thoroughly. Basically you can either:

Wrap your component in a div tag:

<Fade>
  <div>
     <YourComponent />
  </div>
</Fade>

Expose DOM ref in your component

<Fade refProp="inputRef">
  <YourComponent />
</Fade>

inputRef is taken from example docs but could be anything you want

use withReveal Higher Order Component. That's the proper way to inject reveal functionality in custom component. Docs are coming soon but you can take a look at this source to get the idea how it works.

Greaat ! :+1:


Fix ctc when using volatile option

Fix following issue.
https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/chainer-jp/k6B7LPSPsbU/3vt33rTNFwAJ

LGTM!


orgainzation settings: Fix bot owner profile display.

Clicking on the 'Owner' value for a row in the list of bots does
nothing, and causes a blueslip error.
This is because the map object in which we store the users have
integer keys, while we pass the owner id as string.
This is fixed by parsing the owner id to integer before passing it
on.
Fixes #14107.

Testing Plan:
Manually, in the browser UI.
GIFs or Screenshots:
Before: No profile is displayed.
Now:

This is great, merged after fixing the spelling of "organization", thanks @agrawal-d!


Docker Container Debugging: File Not Found

I'm running a docker container with Alpine running Ruby 3.1 and Rails 7.0 where the docker-compose.yml uses the following command to start the debug session:
sh -c "rm -f tmp/pids/server.pid && rdbg -O --host=<IP_ADDRESS> --port=1234 -c -- bundle exec rails server -b <IP_ADDRESS> -p 3000"

When I run docker compose up I get the container up and running with a debugger ready for a connection to attach. In my VS Code launch.js I have the following configuration:
{
  "name": "Attach with rdbg",
  "type": "rdbg",
  "request": "attach",
  "debugPort": "1234",
}

When I start the debugger using the above configuration it launches the debugger and immediately stops on the bin/rails file. I can then continue/step through the process using VS Code's debugger tools but it doesn't actually show the debugger line in VS Code and instead says Unable to open '<file name>'. File not found.. It says that for every file that the debugger stops on.
In my docker-compose.yml I define the volumes for the application like so:
.:/app:cached

I'm not sure what I need to do in order to get the full integration of VS Code's debugger with a rails app running on a docker container.
Let me know what other information would be useful to diagnose my issue and I can add that. Thanks!

Thank you for the report. I need to prepare repro env...

I could repro this feature without Docker. I'm trying to fix, but it can be later version, I can fix.

https://github.com/ruby/debug/pull/600 may solve this issue.

Awesome! Thank you for this quick turnaround! Based off of that pull request does this mean I would be able to set the sourceReference in my launch.json to tell vscode where the debugger's file system is?

I've updated the extension and gem to the latest versions and am getting some parts of it to work! Using the same configurations as I mentioned at the beginning of this issue I can now add a line with debugger which will cause a break when the code is run. However when this happens vscode opens a readonly of the file with the debugger line (which I assume is being sent from the docker container).
Is there a way to use path mappings to tell vscode to use the local files when displaying the debugging info?
Another thing that I'm seeing is when I add a vscode breakpoint (the red dot) then I see this error in my docker container output:
DEBUGGER: Debugger can attach via TCP/IP (<IP_ADDRESS>:1234)
DEBUGGER: wait for debugger connection...
DEBUGGER: Connected.
DEBUGGER: ReaderThreadError: No such file or directory @ rb_check_realpath_internal - /app/c:\Users\jpau\Documents\sites\fees\app\controllers\application_controller.rb
["/usr/local/bundle/gems/debug-1.5.0/lib/debug/session.rb:1600:in `realpath'",
 "/usr/local/bundle/gems/debug-1.5.0/lib/debug/session.rb:1600:in `resolve_path'",
 "/usr/local/bundle/gems/debug-1.5.0/lib/debug/session.rb:1373:in `clear_line_breakpoints'",
 "/usr/local/bundle/gems/debug-1.5.0/lib/debug/server_dap.rb:233:in `process'",
 "/usr/local/bundle/gems/debug-1.5.0/lib/debug/server.rb:72:in `block (3 levels) in activate'",
 "/usr/local/bundle/gems/debug-1.5.0/lib/debug/server.rb:244:in `setup_interrupt'",
 "/usr/local/bundle/gems/debug-1.5.0/lib/debug/server.rb:70:in `block (2 levels) in activate'",
 "/usr/local/bundle/gems/debug-1.5.0/lib/debug/server.rb:419:in `block (2 levels) in accept'",
 "/usr/local/lib/ruby/3.1.0/socket.rb:810:in `block (2 levels) in accept_loop'",
 "/usr/local/lib/ruby/3.1.0/socket.rb:807:in `each'",
 "/usr/local/lib/ruby/3.1.0/socket.rb:807:in `block in accept_loop'",
 "/usr/local/lib/ruby/3.1.0/socket.rb:805:in `loop'",
 "/usr/local/lib/ruby/3.1.0/socket.rb:805:in `accept_loop'",
 "/usr/local/bundle/gems/debug-1.5.0/lib/debug/server.rb:417:in `block in accept'",
 "/usr/local/lib/ruby/3.1.0/socket.rb:782:in `tcp_server_sockets'",
 "/usr/local/bundle/gems/debug-1.5.0/lib/debug/server.rb:393:in `accept'",
 "/usr/local/bundle/gems/debug-1.5.0/lib/debug/server.rb:49:in `block in activate'"]
DEBUGGER: Disconnected.

The file on my local would be just:
c:\Users\jpau\Documents\sites\fees\app\controllers\application_controller.rb

And on the container would be:
/app/app/controllers/application_controller.rb

Is there anything I need to do so the built in vscode break points reference the correct the path mappings?

https://github.com/ruby/vscode-rdbg/issues/32#issuecomment-1101292879
I patched together a working demo of this extension in a remote vscode environment - with it debugging a dockerized rails app.
The issue I have now is that I want multiple mappings. The main rails directory is my working directory that I mapped to a docker volume. I can debug with breakpoints fine with my hack job mentioned in #32. But I also have a ruby gems docker volume that is handled by docker that I am more or less oblivious of. I would want the debugger to have a second read only mapping, something like:
{
  "remoteRoots": [
    {
      "remoteRoot": "/path/to/app",
      "localfs": true
    }, {
      "remoteRoot": "/path/to/gems",
      "localfs": false
    }
  ]
}

I believe the localfs is an internal(ish) vscode configuration, so I think that option applying to different directories might be a pipe dream. But at the very least maybe multiple remote mappings could be supported? If my internal ruby gem files opened in edit mode - that wouldn't be the worst thing.

https://github.com/firien/vscode-rdbg/tree/multi-remote-root
well I just did it, branch above will map multiple directories. The ruby gems files load as editable - but 🤷♂.
{
  "type": "rdbg",
  "name": "Attach with rdbg",
  "request": "attach",
  "debugPort": "12345",
  "localfs": true,
  "sourceMappings": [
    {
      "remoteRoot": "/rails_app" //this is dockerized rails app
      // with localRoot missing - it defaults to workspaceFolder
    },
    {
      "remoteRoot": "/usr/local/bundle", //ruby gems in docker
      "localRoot": "/home/user/.local/share/docker/volumes/app_gems/_data" //local mapping of ruby gem docker volume
    },
  ]
}


Transfer fix!

You guys merged so fast I didn't notice it, please merge this bugfix for the transfer part to work correctly 😄

I think I said these words before... I'm going to repeat it: I Love You.
LOL....
Thanks for pushing all these fixes. Being very busy for the past 1 month. I'll start adding some features once I can find some breathing space once again :)

😄 Yes you said it before, but thank you!


Performance link is broken

Please check the performance  review document URL, It leads to a 404 page.

[Here is the link] (http://handbook.hackerbay.io/people-operations/performance-reivew/README.md)

Whew did you find the broken link. Can you please send PR.


rename set_relay_on to set_is_relay_on

I found that the UI switch was not switching the relay on or off, even though I see is_relay_on gets changed in the logs upon clicking. I think set_relay_on isn't getting called and actually changing the duty cycle, since it would be looking for set_is_relay_on instead.

👍 yup that's a silly mistake. I'll merge and rip a new release to pypi

New version is available: https://pypi.org/project/pioreactor-relay-plugin/0.4.1/
I think you'll have to uninstall and install the plugin again to get the latest version.


Form and text alignment

1.the login page placeholder color light
2, the wallet information page to edit the name icon and view the private key icon size adjustment, remain the same
3.wallet backup, harvesting two page icon size and left and upper and lower space adjustment, keep the same
4.wallet information, wallet backup, harvesting three pages of content, the first line of text and border spacing adjustment, keep consistent
5.top navigation toggle node dot and icon space adjustment, and the Console icon before the center

merged with 28e1ac6c


[RFC]support RBAC

this will be done in some configuration file rather than DDL dialect in ClickHouse
We may reuse some existed libraries, for example, casbin-rs.

[x] 1.0 BRSL(Base Resources Spec Language)

[x] casbin-rs investigation(use or not use?)
[x] RFC for lang

[ ] 2.0 In-Base Auth Implementation

[ ] primary user/password impl
[ ] popular 3rd-party auth platform

@jinmingjian I'm glad that the casbin-rs community is helping us to do this, so I update our primary plans based on the discussions before here.
Basically, we need two kinds of policies, which:

describes which users have access to which databases/tables, in particular access patterns (e.g. read/write/update/...),
describes which users have access to which computing resources, how many quotas, and the priorities.

These two rules might be described by:
[policy_definition]
data = user, table_id, action
resource = user, type, quota, priority

In addition, we need to find a way to dynamically update the policies during runtime. This might be done by a housekeeper which loads the policy file immediately after it changes.

@whjpji
So, this is based on my understanding of the policy requirement. I think this would need two Casbin enforcers based on the two policy requirements.
For the first case, we could simply use a Access Control List based policy if all users have not so much databases/access patterns in common. Otherwise if there are many users who have access to a specific database with the same access patterns, we could optimize it to use a RBAC model. For example, the model in this case could be:
[request_definition]
r = usr, id, act  # user, database-id, access pattern

[policy_definition]
p = usr, id, act # policy user, database-id, access pattern

[role_definition]
g = _, _

[policy_effect]
e = some(where (p.eft == allow))

[matchers]
m = g(r.usr, p.usr) && r.id == p.id && keyMatch(r.act, p.act)

And the policy could be:
p, database1_read, read
p, database1_admin, *
... (other policies)

g, user1, database1_read
g, user2, database1_read
g, user3, database1_admin
... (other policies)

This would mean user1 and user2 have read access to database1 while user3 has admin access to it.
For the second case, assuming quota means quota limits we can create a similar RBAC model for this which will use the priority effect. An example model could be:
[request_definition]
r = usr, quota  # user, quota used

[policy_definition]
p = priority, usr, quota, eft # policy priority, user, quota limits, effect (allow or deny)

[role_definition]
g = _, _

[policy_effect]
e = priority(p.eft) || deny

[matchers]
m = g(r.usr, p.usr) && (r.quota<p.quota)

And the policy could be:
p, 10, res1_group1, 100, allow
p, 10, res1_group2, 50, allow
... (other policies)

g, user1, res1_group1
g, user2, res1_group2
... (other policies)

So, when we send an enforcement request with user and quota used parameters - it compares if the user has access to the computing resources and if the current used quota is lesser than the quota limits. Here res1_group1 maybe some specific computing resources with quota limit of 100. And since user1 has a role as that, it would have a quota limit of 100.
This is just a rough idea, I believe this can be made better. Please correct me if anything is wrong.
(cc: @hsluoyz @hackerchai @PsiACE)

@whjpji thanks for updating detailed thoughts for this issue. I think we should have a unified model rather than two. And there are some dynamic requirements like quote here.

@rushitote thanks for sharing your great idea! It is a good material to reference!

Based on the current capability of TB, I have a primary plan to support access control:

First to add a system.user table to store user infos (user ID, username, password, etc.). At the first handshake of the client and server, the user is verified by password.
Then each query needs to be associated with a specific user, which is useful to verify whether the user has access to the queried table.
Next, introduce casbin to determine whether the user should be authorized to access the table in read/write/update/delete by some policy rules. BTW, the casbin policy are better to stored in a dedicated table (like system.policy) instead of CSV, because CSV is hard to revise in runtime and will be hard to read or write when the rules become more complicated.
Finally, to make the authentication actually work, connections with SSL/TLS should be introduced to secure that the user from any client is identical to the registered user in db.

@whjpji self-hosted access control is cool, and we thanks for casbin framework for policy as well:)

Blocked on #230, and I'm going to fix it first.


Feature migrate mumukit 2

Fixes #3

@npasserini integration is nearly done. Please review the edge cases and that will be everything by now


[BUG] 2022 session data not available

Describe the issue:
I'm able to load an event schedule for 2022, however I'm not able to load any session data from this season.
I am able to load data from every other year I've tried (I probably haven't tried them all).
(Please note I'm a newbie from CS50x using your API for my final project, so I'm sorry if I've missed something)
Reproduce the code example:
@app.route("/results")
def results():
    year = request.args.get('year')
    session_name = request.args.get('session')
    event = request.args.get('event')
    try:
        session = fastf1.get_session(int(year), event, session_name)
        session.load()
    except (ValueError, KeyError):
        return render_template("error.html", error='Session data not found.')
    df = session.results
    return render_template("results.html", tables=[df.to_html(classes='table table-striped table-dark table-hover table-responsive', header="true", index=False, justify='center')])

Error message:
DEBUG:requests_cache.policy.actions:Cache directives from request headers: CacheDirectives()
DEBUG:requests_cache.policy.actions:Pre-read cache checks: Passed
DEBUG:requests_cache.policy.actions:Post-read cache actions: CacheActions(expire_after=datetime.timedelta(seconds=43200))
core           INFO     Loading data for Bahrain Grand Prix - Race [v3.3.9]
INFO:fastf1.fastf1.core:Loading data for Bahrain Grand Prix - Race [v3.3.9]
req            INFO     No cached data found for session_info. Loading data...
INFO:fastf1.fastf1.req:No cached data found for session_info. Loading data...
_api           INFO     Fetching session info data...
INFO:fastf1.api:Fetching session info data...
DEBUG:requests_cache.policy.actions:Cache directives from request headers: CacheDirectives()
DEBUG:requests_cache.policy.actions:Pre-read cache checks: Passed
DEBUG:requests_cache.policy.actions:Post-read cache actions: CacheActions(expire_after=datetime.timedelta(seconds=43200), send_request=True)
DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): livetiming.formula1.com:443
DEBUG:urllib3.connectionpool:https://livetiming.formula1.com:443 "GET /static/2022/2022-03-20_Bahrain_Grand_Prix/2022-03-20_Race/SessionInfo.jsonStream HTTP/11" 403 None
DEBUG:requests_cache.policy.actions:Cache directives from response headers: CacheDirectives(max_age=3600)
DEBUG:charset_normalizer:Encoding detection: utf_8 is most likely the one.
DEBUG:requests_cache.policy.actions:Pre-write cache checks: disabled status
DEBUG:requests_cache.session:Skipping cache write for URL: https://livetiming.formula1.com/static/2022/2022-03-20_Bahrain_Grand_Prix/2022-03-20_Race/SessionInfo.jsonStream
DEBUG:charset_normalizer:Encoding detection: utf_8 is most likely the one.
logger      WARNING     Failed to load session info data!
WARNING:fastf1.fastf1.core:Failed to load session info data!
DEBUG:fastf1.fastf1.core:Traceback for failure in session info data
Traceback (most recent call last):
  File "/home/matt7/Documents/coding/Projects/cs50_final_project/.venv/lib/python3.10/site-packages/fastf1/logger.py", line 151, in __wrapped
    return func(*args, **kwargs)
  File "/home/matt7/Documents/coding/Projects/cs50_final_project/.venv/lib/python3.10/site-packages/fastf1/core.py", line 1462, in _load_session_info
    self._session_info = api.session_info(self.api_path,
  File "/home/matt7/Documents/coding/Projects/cs50_final_project/.venv/lib/python3.10/site-packages/fastf1/req.py", line 445, in _cached_api_request
    data = func(api_path, **func_kwargs)
  File "/home/matt7/Documents/coding/Projects/cs50_final_project/.venv/lib/python3.10/site-packages/fastf1/_api.py", line 1665, in session_info
    raise SessionNotAvailableError(
fastf1._api.SessionNotAvailableError: No data for this session! If this session only finished recently, please try again in a few minutes.
req            INFO     No cached data found for driver_info. Loading data...
INFO:fastf1.fastf1.req:No cached data found for driver_info. Loading data...
_api           INFO     Fetching driver list...
INFO:fastf1.api:Fetching driver list...
DEBUG:requests_cache.policy.actions:Cache directives from request headers: CacheDirectives()
DEBUG:requests_cache.policy.actions:Pre-read cache checks: Passed
DEBUG:requests_cache.policy.actions:Post-read cache actions: CacheActions(expire_after=datetime.timedelta(seconds=43200), send_request=True)
DEBUG:urllib3.connectionpool:Resetting dropped connection: livetiming.formula1.com
DEBUG:urllib3.connectionpool:https://livetiming.formula1.com:443 "GET /static/2022/2022-03-20_Bahrain_Grand_Prix/2022-03-20_Race/DriverList.jsonStream HTTP/11" 403 None
DEBUG:requests_cache.policy.actions:Cache directives from response headers: CacheDirectives(max_age=3600)
DEBUG:charset_normalizer:Encoding detection: utf_8 is most likely the one.
DEBUG:requests_cache.policy.actions:Pre-write cache checks: disabled status
DEBUG:requests_cache.session:Skipping cache write for URL: https://livetiming.formula1.com/static/2022/2022-03-20_Bahrain_Grand_Prix/2022-03-20_Race/DriverList.jsonStream
DEBUG:charset_normalizer:Encoding detection: utf_8 is most likely the one.
core        WARNING     Failed to load extended driver information!
WARNING:fastf1.fastf1.core:Failed to load extended driver information!
DEBUG:fastf1.fastf1.core:Exception while loading driver list
Traceback (most recent call last):
  File "/home/matt7/Documents/coding/Projects/cs50_final_project/.venv/lib/python3.10/site-packages/fastf1/core.py", line 2252, in _drivers_from_f1_api
    f1di = api.driver_info(self.api_path, livedata=livedata)
  File "/home/matt7/Documents/coding/Projects/cs50_final_project/.venv/lib/python3.10/site-packages/fastf1/req.py", line 445, in _cached_api_request
    data = func(api_path, **func_kwargs)
  File "/home/matt7/Documents/coding/Projects/cs50_final_project/.venv/lib/python3.10/site-packages/fastf1/_api.py", line 1513, in driver_info
    raise SessionNotAvailableError(
fastf1._api.SessionNotAvailableError: No data for this session! If this session only finished recently, please try again in a few minutes.
INFO:werkzeug:<IP_ADDRESS> - - [15/Jul/2024 18:48:15] "GET /results?year=2022&session=Race&event=Bahrain%20Grand%20Prix HTTP/1.1" 200 -
INFO:werkzeug:<IP_ADDRESS> - - [15/Jul/2024 18:48:15] "GET /static/styles.css HTTP/1.1" 304 -

I've just noticed that I'm able to load Bahrain 2022 qualifying with the same code. Of the other sessions I've tried, none of the race or sprint sessions load.

This has been reported in #603 and #606 as well. Seems to be an upstream issue in FOM's end so not much we can do here

Apologies, I skimmed the issues so should have checked more thoroughly! Thanks.


Fix delete contact method doesn't clear the errors

Details

When we are deleting a failed add new contact method, the errors still stay because we "don't clear it".
Fixed Issues

$ https://github.com/Expensify/App/issues/17343
PROPOSAL: https://github.com/Expensify/App/issues/17343#issuecomment-1513274023
Tests

Same as QA Steps

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

Offline tests

Same as QA Steps
QA Steps

Go to Settings > Profile > Contact Method
Press/click New contact method
Fill the email with any email that is not the current login email
Fill the password incorrectly
Press/click Add
Verify there is a red dot error at the Settings avatar icon (next to Search Icon)
Open the new contact method you just added
Press/click Remove
Verify the red dot is gone from the Settings avatar icon

[x] 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 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

https://user-images.githubusercontent.com/50919443/232967571-1376172b-055b-4bd7-8d3a-025bf47d3a96.mov

Mobile Web - Chrome

https://user-images.githubusercontent.com/50919443/232967622-28bdd812-9b0b-4cfd-a0e7-0e95198b0398.mov

Mobile Web - Safari

https://user-images.githubusercontent.com/50919443/232967632-0def6169-7e0d-4b47-b3e3-afd0ecfed653.mov

Desktop

https://user-images.githubusercontent.com/50919443/232967679-a9dcbf6a-70fe-4170-bf23-96a21332fd5e.mov

iOS

https://user-images.githubusercontent.com/50919443/232967691-5a7aa0c7-26e0-467e-9f79-c6c0bdedebd4.mov

Android

https://user-images.githubusercontent.com/50919443/232967744-8569d34a-36b9-4884-92b8-2377af8c996b.mov

@alex-mechler @parasharrajat One of you needs to copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

Bump @parasharrajat

: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/alex-mechler in version: 1.3.4-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/jasperhuangg in version: 1.3.4-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Update file size when exporting tables to S3

Moving to 256 mb as discussed in https://github.com/hellonarrativ/spectrify/issues/29

Thank you @paoliniluis !


Logging for felix unclear if vm has no security groups attached.

If a vm has no security groups associated with it, then felix does not add it and deletes the chain associated with it, giving the below log.  Rather than just saying the Endpoint is unready, it would be more helpful if it explained why it was unready i.e. no profile_ids / no security group ids are associated with this vm.
2015-07-08 16:04:39,468 [INFO][21479/227] calico.felix.endpoint 337: Endpoint<IPv6,id=EndpointId,iface=tape9798859-0a> became unready.

To be clear, the end result is that packets to the VM are dropped, which is exactly what we want.  The issue is that it's hard to tell why.

Fixed by #822


copilot-chat ChatHistory controller is neither authenticates nor validates message ownership

Describe the bug
The ChatHistory endpoint is unauthenticated and it is possible to interact with the backend API without any kind of authentication.
It is also possible to query and access chat messages for other users.
To Reproduce
Steps to reproduce the behavior:

Access https://localhost:40443/chatSession/getAllChats/.
This will list all conversations for any user you know the object ID of

Expected behavior
The ChatHistory controller is missing an [Authorize] attribute, and the endpoints should validate that the user requesting the messages owns or has access to them.

Nice call out, lol, but I think there are several notes that indicate not to use this for production and since the sample app is meant to convey the novelty behind using SK for the copilot use case, I probably wouldn't expect the sample app to articulate further with assumptions on a security implementation.
That's my take, I'm a consumer just like you.

As the team is open to PR, I think it would be a great addition if you could make it work.

@IanNorris @dedalo Happy to contribute a PR to this.

@parthvnp - Yes, we are open to PRs and would be great if you can contribute.

@hathind-ms Thanks. I will start working on a fix.

This has been fixed with https://github.com/microsoft/chat-copilot/pull/126


no permite deploy

Does this occur consistently?
Repro steps:

Action: appService.Deploy
Error type: Error
Error Message: "proyectocontrolBanda" is connected to a GitHub repository. Push to GitHub repository to deploy.
Version: 0.24.9
OS: win32
OS Release: 10.0.19045
Product: Visual Studio Code
Product Version: 1.80.1
Language: es

Call Stack
Object.<anonymous> extension.bundle.js:2:2410350
extension.bundle.js:2:2408199extension.bundle.js:2:2408199
S extension.bundle.js:2:2407885
extension.bundle.js:2:2410000extension.bundle.js:2:2410000
k.d extensionHostProcess.js:97:43474
k.withProgress extensionHostProcess.js:97:43337
Object.withProgress extensionHostProcess.js:117:31108
Object.<anonymous> extension.bundle.js:2:2409931
fulfilled extension.bundle.js:2:2407953

"proyectocontrolBanda" is connected to a GitHub repository. Push to GitHub repository to deploy.


CCO domain name and its ownership (and PURL Server)

This moved from Issue 105 starting at https://github.com/CommonCoreOntology/CommonCoreOntologies/issues/105#issuecomment-2243651644
I (Jim) wrote:
I believe this means: CCO IRI http://www.ontologyrepository.com/CommonCoreOntologies/ would be replaced by the PURL https://purl.ieee.org/sa/cco/, which would point to that class in the latest version of CCO at https://opensource.ieee.org/cco/CommonCoreOntologies/
I believe this deprecates use of http://www.ontologyrepository.com, which I don't see we need anymore.
Alan responded:
@jimschoening1 The reason to keep www.ontologyrepository.com (or some other domain that is controlled by the developer community) is that while IEEE, like any organization, has the best of intentions stuff happens in organizations and down the road it may come a time where they no longer host the PURL server. By having the ontology developers/steering committee own the domain that eventuality can be handled by standing up an alternative PURL server and directing the domain to that.
This is not a theoretical concern. OCLC, who ran the PURL server OBO initially used, was an equally reputable and trustworthy organization, but due to internal decisions we were not privy to stopped supporting it. The only thing that let our PURLs continue to be resolved was the forethought in deciding to own our own domain name, and our negotiation with OCLC to be able to use it as a CNAME for their PURL server.
Please don't try to reassure me that IEEE would never do this.  Shit happens.
The cost of a domain name is negligible and the protection it gives in the face of unforeseeable changes is very valuable.
Jim responded: Alan, I agree IEEE is not perfectly trustworthy, but who is this developer group and are they a more trustworthy entity? If IEEE decided to defund the PURL server, I believe OSWG members (Are not We the developer group?) would find a way to keep it going. Ron owns the domain now. If we kept it, shouldn't IEEE own it?
Alan: The people who are most invested in making sure the ontology works are the developers and users. There will be a steering committee for Common Core and they are the ones that should control the domain. Certainly we could have IEEE as one of the administrators. But having them own it puts the domain in the same situation as an IEEE domain. If things go to hell, getting back control is dicey
Jim responded: "Alan: But key CCO (plus domain extensions) stakeholders are voting members of IEEE Ontology Standards Work Group (OSWG). Could OSWG own the domain name (whether it is ontologyrepository or purl.ieee.sa/cco/? OSWG has time-tested Policies and Procedures (P&P) with higher levels of governance. Would that not be more trustworthy than the emerging CCO Governance Board? I realize BFO set up their own governance structure, but members of it can't join ISO as individuals. Also, even if IEEE management (who admittedly has little stake in CCO) were to defund the PURL server, the URL (purl.ieee.org/sa/cco/...) is still stable, since IEEE would have no reason to turn off the subdomain. CCO stakeholders could use that URL to stand up our own PURL server, probably under the ultimate control of IEEE, but not requiring their time or budget. In fact, that was the original request to IEEE, for us to set up a PURL server, but their IT team leader said he could do it for us. We might want to get more control of that server, which we could do a Risk Management chart to show the risk of loss of budget for the PURL Server, and show the risk mitigation that we have volunteers who have demonstrated we can run it if needed. The IT Team Leader would fight us, but I chair the IEEE PURL Server Subgroup and could take this over his head."
Alan responded:
"You wouldn't be able to use purl.ieee.org in the case that IEEE decomissioned the server because that's a subdomain of ieee.org. Subdomains are managed with records attached to the primary domain. To change the ip address of purl.ieee.org would require IEEE to execute that. In the scenario I am talking about having IEEE do anything related to the PURL breaks down. Moreover large organizations won't want their subdomains pointing to resources outside their control.
As I said, I have the most confidence in the developers and active users of the ontology having the right incentives to make resolution work in the long term, and so they should be in control of the domain. The setup would be to have at any time 3 designated members who are able to manage the domain records. Payment is cheap and we've always found someone who is is willing to pay the $15 or $20 to renew the domain yearly. If at some point it seems like there's no responsible party within the community then we can always find a new steward at that point.
I have less confidence in the overarching OSWG lasting as long as the ontologies under it last.
We're going back and forth on this. What I suggest has precedent and has worked for over a decade at OBO, so it's a tested solution. I don't understand why there is so much resistance. Technical setup is trivial."
Jim responded: "Alan, It sounds like you don't object to moving to opaque identifiers ;) These other points are off topic, but deserve further discussion, so I will break them out as new issues."
Alan responded: "Nope. But if there's going to be a change in domain as well both changes ought to be done in one release."

@jimschoening1 As the CCO Governance Board and developers move toward a next release of CCO, we would like to use this issue tracker solely for technical comments and concerns regarding CCO and begin using the discussion feature for other conversations. For this reason, I am closing this thread. Please feel free to start more open-ended discussions or recommend future directions (e.g. regarding use of an existing or future PURL server) here:  https://github.com/CommonCoreOntology/CommonCoreOntologies/discussions
There are a number of distinct issues in this conversation with @alanruttenberg. Three comments:
First, regarding configuration management of CCO now and moving forward, this would be a matter to take up with the CCO Governance Board who manages CCO. The CCO Governance Board holds frequent meetings and working sessions, but could also schedule a break out session. If a dedicated conversation on any matter here is desired, I recommend coordinating with @johnbeve or @mark-jensen regarding a time that works for us to meet.
Second, for discussions regarding any updates to the license of CCO, please see https://github.com/CommonCoreOntology/CommonCoreOntologies/issues/215 where these issues will now be discussed. The Governance Board will, of course, be coordinating directly with IEEE; CUBRC, Inc.; and other parties regarding any recommendations.
Third, CCO will be adopting new IRIs that include opaque local ids in a single future release and socializing this in advance for stakeholders; however, no decision has yet been reached regarding the namespace or other elements of these IRIs and we will be discussing this with you and others going forward. Presently, this is planned to follow the upcoming release, resulting in a 2.0 release.


Can't format console/ shell-session output that includes ANSI colors

(Original issue 1444 created by purplefishies on 2018-06-06T16:50:49.003688+00:00)
I'm trying to take log data from Tmux ( with capture-pane -e ) that includes ANSI color codes that colorize some of the output.
When I run pygmentize -f latex  sample.shell-session it generates Latex with the ANSI codes still in the latex file.
This causes a latex compile failure.

I just managed to hack my way into rendering shell session output's with ANSI colors.
New ANSI-capable styles are available in my upcoming click-extra v3.0.0 release. You should be able to do:
$ pip install click-extra >= 3.0.0

Then use my new ansi-prefixed lexers.
Here is an example in Sphinx:
.. code-block:: ansi-shell-session
    $ hello --help
    Usage: hello [OPTIONS]

      Simple program that greets NAME for a total of COUNT times.

    Options:
      --dummy-flag / --no-flag  [default: no-flag]
      --my-list TEXT
      --count INTEGER           Number of greetings.  [default: 1]
      --name TEXT               The person to greet.
      --time / --no-time        Measure and print elapsed execution time.  [default:
                                no-time]
      --color, --ansi / --no-color, --no-ansi
                                Strip out all colors and all ANSI codes from output.
                                [default: color]
      -C, --config CONFIG_PATH  Location of the configuration file. Supports both
                                local path and remote URL.  [default:
                                ~/.hello/config.{toml,yaml,yml,json,ini,xml}]
      -v, --verbosity LEVEL     Either CRITICAL, ERROR, WARNING, INFO, DEBUG.
                                [default: INFO]
      --version                 Show the version and exit.
      -h, --help                Show this message and exit.

Which renders to:

My Pygments ANSI capable shell and session lexers were just published as part of click-extra 3.0.0 and wrote a quick tutorial at: https://kdeldycke.github.io/click-extra/pygments.html

For the record, inclusion of ANSI-capable lexers are discussed for inclusion in the pygments-ansi-color project at: https://github.com/chriskuehl/pygments-ansi-color/issues/33#issuecomment-1550046730


Mixed PDF export

Test rtl.pdf
Using the built-in pdf export command (or any other one) doesn't handle files with both rtl and ltr lines in it at all. In the editor itself it works fine

Shouldn't happen and it recreates for me too. Seems like it broke in one of the recent versions, will investigate.

Should be fixed in 1.2.1, please let me know if it works for you.

it does. tysm


New logo offer

Hello, I want to design a logo for you as a graphic designer, and if you agree, I want to start with the logo design.

Go ahead and draft something up and post it here. I'd love to have a look.


Various Solaris helpers

Pretty self-explanatory. Please hold off on merging until @scotthain and @curiositycasualty have given a :+1:.

LGTM :+1:

Other than the sun4u/sun4v thing - looks great! We just need to add the possibility of sun4v.

:+1:

@sethvargo This should be ready to :shipit: when you approve.

Woah. I go on vacation for a week and everyone has new pictures!? :smile:


Graph is invisible

Looks like the graph is not visible anymore
https://lccn.lbao.site/predicted/biweekly-contest-109

@baoliay2008 Ya, Please look into this issue. The graph is invisible from yesterday. Main Graph as well as participants Graph.

Appreciate your feedback, I'll promptly address this matter.

I have verified
last 5 contests
graph is visible & working fine
Thank you @baoliay2008
. https://lccn.lbao.site/predicted/weekly-contest-355
. https://lccn.lbao.site/predicted/biweekly-contest-109
. https://lccn.lbao.site/predicted/weekly-contest-354
. https://lccn.lbao.site/predicted/weekly-contest-353
. https://lccn.lbao.site/predicted/biweekly-contest-108


Chore: Update to Electron 25.2.0

I noticed the latest VSCode was no longer able to access secrets properly on KDE Neon Unstable, which is using a preview of KDE 6.
Looking through the logs, I noticed the desktop environment was detected as KDE 4. This is because the underlying Chromium version of Electron 22 did not support KDE 6 yet.
This update to Electron (and because of that, Chromium), adds full support for KDE 6.

@microsoft-github-policy-service agree

Thanks for the contribution, we don't accept runtime updates from external contributions since there are tasks that cannot be completed by community. This will be resolved with https://github.com/microsoft/vscode/pull/180466


[WIP] Version 3 (support transaction and subcollection)

Main new features of version 3 are support transaction and subcollection.
Some API will be changed from version 2.

[ ] transaction

[x] Implement
[ ] Add test

[ ] subcollection

[ ] Implement
[ ] Add Test

[ ] Change some API

[ ] Change some public methods and property to private or protected.

[ ] Fix README
[ ] Add CHANGELOG

Codecov Report

Merging #7 into master will increase coverage by 2.62%.
The diff coverage is 98.36%.

@@            Coverage Diff             @@
##           master       #7      +/-   ##
==========================================
+ Coverage   95.69%   98.31%   +2.62%
==========================================
  Files           2        2
  Lines          93      119      +26
  Branches       10       16       +6
==========================================
+ Hits           89      117      +28
  Misses          1        1
+ Partials        3        1       -2

Impacted Files
Coverage Δ

__tests__/util.ts
100% <100%> (ø)

src/index.ts
98.03% <98.3%> (+3.3%)
:arrow_up:

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 a23afd7...beeb239. Read the comment docs.


Make it easier to switch thumbnail CDNs

The current code is hardcoded to a specific server (speech, vanwa), so it's not just a matter of switching URLs.  It needs to handle different output formats, plus speech requires extra input too.

[ ] Figure out a way to make the switch easier (common interface?)
[ ] Next, choose either auto fallback when one fails, or manually flip the switch

I think a big part of the problem is a brave / adblock list that we turned up on. Maybe if we get the failed to fetch, we can mention to try disabling that. It's called "AdGuard Français" , and on Brave, it's on: brave://adblock/

Since we have our own now, don't think this is important.


[Suggestions]

Please add purchase list and bombzone hints from GOESP

ok

Purchase list has been added.


Registering subscriptions for destruction

This updates rqt_plot to register its subscriptions for destruction with RclpySpinner.
Depends on PR https://github.com/ros-visualization/rqt/pull/184

Obsolete, see ros-visualization/rqt#184 for details.