[Improvement]: Ability to get the list of loaded projects in the Language Server

Description
We need to get a list of open projects from the workspace manager for a feature we are working on in the VS Code Low Code diagram. After a chat with @malinthar, decided to add a method to the interface WorkspaceManager to get the list of projects, the data we require is only the project name, and directory path for the project
Appreciate any input on this
Describe your problem(s)
No response
Describe your solution(s)
No response
Related area
-> Compilation
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

@IMS94 What do you think about adding a new method to the BallerinaWorkspaceManager

@malinthar @CharukaK we had a similar request some time back from the editor team for the project architecture diagram feature. Our advice was to use the workspace VSCode API to get the workspace folders from plugin side and then use those folder paths to get the ballerina projects in the workspace.
Why we cannot recommend getting all the open projects in the workspace manager is because there can be a lot of unrelated projects open there. For example, if a user goto def a std lib, that std lib project also gets opened in the workspace manager. And with the new build server thing coming, there can be even more.
Even from the VSCode side, recommended way for the end users is to add multiple projects to a workspace when working with multiple related projects.

Why we cannot recommend getting all the open projects in the workspace manager is because there can be a lot of unrelated projects open there. For example, if a user goto def a std lib, that std lib project also gets opened in the workspace manager. And with the new build server thing coming, there can be even more.

When investigating the issue it was figured out that from the LS side we can get the folders(filepaths) in the current workspace using the  LanguageServerClient. So we can write a logic to filter the required build projects in the current workspace.


Improve docs for custom transactions and terminology

Docs need to be improved in order to better explain how to use custom transactions.
Also we need a glossary or some explanation of the basic terminology of APM. In this last case maybe we could link to some global APM explanation.

++ to this issue. I implemented the agent recently, and turned to the opbeat-react docs and source for some clues how best to use the agent and structure my transactions. Happy to discuss what I did if that's helpful.


If AzureWebJobsStorage connection string is not set, CLI should provide a helpful error message

If the local function app contains triggers other than HTTP is being used, then the CLI should check the value of ScriptHostConfiguration.HostConfig.StorageConnectionString and provide the following error:
Error: Missing connection string for required setting "AzureWebJobsStorage". Provide a value in appsettings.json or run "func azure functionapp fetch-app-settings <functionAppName>"

Duplicate of #38


Lookup Fields View

Hi, I wanted to know, in cases that the view has a lookup field, which is the view associated with that field. Thanks!

Hello,
For lookup fields in the editable grid, the control does not use any view. The primary field (of the lookup entity) value is being used in the dropdown.


Added pulselang project

@anirudh-hegde hey we are unable to open ,whatever you have pushed ,even no issue no ,Please make a PR again


25-monitor.md

Source:
Do Not Apply
Current Translation:
Do Not Apply
Suggested Translation:
不应用

Suggested Translation:
不适用

I can't find target Sting "Do Not Apply".
There seems to be a reference to a "bb-metadata" from. I don't know where this form is right now, so I can't fix this issue.

Can't fix this issue, because this can't find in xliff.

Hi,
Please refer the below screenshot.

Can't fix this issue, because string can't find in xliff.

can't fix in xliff.


Hello :)

I managed to use this with a Tandy VM-5 monitor.
https://imgur.com/a/qv8d2KG

That's awesome, love it :-)


CB-16504 Adding OUTBOUND LoadBalancerType to 2.54

See detailed description in the commit message.

This looks like a duplicate. Same change is on the branch, probably from 2.53. Closing, please reopen if needed.


black page. Folder lib not found

I have a blank page when I do "ionic serve" for the preview. I looked that the page cannot load some files from the lib directory. In fact lib directory does not exist...

If there is no www/lib directory then bower dependencies are not installed. Please use bower install.


Allow stripping root groups from joined group names.

This PR allows syncing groups under a root group:
With:
apiVersion: redhatcop.redhat.io/v1alpha1
kind: GroupSync
metadata:
  name: keycloak-groupsync
spec:
  providers:
  - name: keycloak
    keycloak:
[...]
      groups:
      - Organizations
      subGroupProcessing: join
      subGroupJoinSeparator: "+"
      subGroupJoinStripRootGroups:
      - Organizations

Groups:
Organizations
\-> my-org
    \-> my-team
Customers
\-> my-cust

Become
my-org
my-org+my-team

@glrf I've gone with a slightly different approach than initially discussed. The groups parameter already existed and allows filtering top-level groups. So I just added a parameter to allow stripping TL group names.
The code is optimised for a minimal diff not for maximal readability since I still felt hope we can upstream at least the join part one day.
I have the time budget to refactor more and track group names/parents more cleanly but I'm not sure if it's worth it and it would mean a huge diff again.


Maximum value for some signals missing

Hello,
I am using cantools with arxml file as input. The maximum values for some signasl is None but arxml has correct Upper limit.
Maximum of signals is None after database.load_file().
Can you help me with this?

Can you help me with this?

yes, if you provide an example file where that happens...


Long addresses display “123dsfds…undefined”

Long account addresses display undefined at the end.

I believe this one was fixed already


How to hide context map of dynamic require?

when i use dynamic require i get this map on chunk
var map = {
"./AboutPage": 247,
"./AboutPage.jsx": 247,
"./DashBoardPage": 249,
"./DashBoardPage.jsx": 249,
"./FeedChannelPage": 251,
"./FeedChannelPage.jsx": 251,
"./HostPage": 253,
"./HostPage.jsx": 253,
"./LoginPage": 255,
"./LoginPage.jsx": 255,
"./ProfilePage": 257,
"./ProfilePage.jsx": 257
}
How could I hide this map and dynamically manipulate this map in code?
please help?

By not using it.
When you use a context, you usually have something like:
"./file/"+aVar+".js"

aVar in this case would be the basename for the file. As you see, the path is constructed dynamically. WebPack can not tell what the paths may look like, and your aVar will indeed contain the raw basename.
So there is no way to hide it, unless you don’t use it, i’m afraid.

My idea is I've try to write some kind of web admin with redux react. And wondering how can I restrict page access of user based on role

That is the job of your backend’s RBAC. A proper RBAC will be able to pick up a malicious attempt and cancel it out.

Well, are there any way to dynamically provide context map by server instead and put it in redux store or something

any suggesstion?

Can you re-open at Stack Overflow for more ideas?


Add anoncreds migration guide

Adds a migration guide controllers to upgrade to anoncreds.

I'm not as close to the AnonCreds changes of late so it might be valuable to get input from others still but I found this document to be quite thorough and informative and, to my knowledge, accurate.

:+1: I think it's good enough for now. We might need to make changes when people start using it, but we can adjust it then.


Support for Scala Client

I have an Akka/Scala, Spark, Kafka, PostgreSQL based application stack, I use Slick FRM for DB communication. Since Prisma support DAL as GraphQL service, I could not find any reference to Scala Client SDK. I like the support for TypeScript, useful for Angular/Node.js, without JVM client, the application stack seems incompatible.
Can Prisma REST APIs can be accesses easily via simple POST/GET/PUT/DELETE method from JVM languages? Anyone point out what is the best way to communicate to Prisma through JVM client (Scala if possible)?

Hi @gopalakrishnan-subramani,
I can understand your situation. We are working hard to make clients for all of the languages. Currently, only these clients are available:

JS client
TS client
Flow client
golang client

We will definitely add the Scala client in the future.

You can consume the prisma API like any other graphql API. The client makes it easy to consume prisma. But it is possible to use it as a graphql endpoint.
You can use something like https://github.com/Jarlakxen/drunk in scala to consume it.


[Bug] Windows Defender detects a Trojan in Nightly VRCX

Describe the bug
After updating to the latest nightly, Windows Defender detects VRCX as a Trojan and quarantines it.
To Reproduce
Steps to reproduce the behavior:

Upgrade to latest nightly at the time of submission of this ticket.
Windows defender will block and quarantine VRCX, stating it is a Trojan.

Expected behavior
Windows Defender should not be detecting VRCX as a Trojan
Screenshots

What version you are running
Latest Nightly at Time of Submission

Wacatac.B1ml is a well known AI detection that can be ignored

go through with the process of update your windows defender security intelligence database and these errors should go away https://github.com/vrcx-team/VRCX/issues/568#issuecomment-1593373667

Instructions worked fine.
https://www.virustotal.com/gui/file/58f57899fb44cce3fe51985c96fe36e279320489690d83d14e81eabaaf1139be/detection
Bitdefender flags it using the same algorithm so you may get more of these tickets. Any idea why this particular build seems to be problematic?

Instructions worked fine.
https://www.virustotal.com/gui/file/58f57899fb44cce3fe51985c96fe36e279320489690d83d14e81eabaaf1139be/detection
Bitdefender flags it using the same algorithm so you may get more of these tickets. Any idea why this particular build seems to be problematic?

its been a common issue, its because of the way the VRCX updater works and has been an ongoing issue, usually we get away with stable being fine as stable is built weeks in advanced before release and submitted to Microsoft for manual review before being published so people have a chance to receive the updated definitions before a false flag can occur. this would be fixed with a Signing certificate but currently the requirements for a EV signing cert is rather insane and overly expensive.

Rescanning it dropped a handful, the executable likely got enough reputation now.


3D plotting

http://juliaplots.org/MakieReferenceImages/gallery//type_recipe_for_molecule_simulation/index.html

Interesting, but model packages should not contain plotting dependencies.


Timeout

Is there any way to set a timeout?
Also, I want to disconnect after checking for mails (if there's mail, after getting them, if there's not after checking). Is there any option to do this?
Thanks.

`var notifier = require('mail-notifier');
var imap = { // your configs ...
connTimeout : 20000, // Time out for connection
authTimeout : 20000 // Time out for authentication
}
notifier = notifier(imap)`

Thanks


remove limit and add max length task description

Details

Remove limit line view task description
Add limit length description when create and update task

Fixed Issues

$ https://github.com/Expensify/App/issues/22493
PROPOSAL: https://github.com/Expensify/App/issues/22493#issuecomment-1632966436
Tests

Create a task with a long description
Open task detail and verify that can view all description

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

Offline tests
The same test

QA Steps
The same test

[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 the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

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

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

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

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[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://github.com/Expensify/App/assets/39086067/0f6d6d90-35ef-4931-ab21-13ce9cf4d3f7

Mobile Web - Chrome
https://github.com/Expensify/App/assets/39086067/0a40f0b9-daed-4756-8f5f-ed56e7807d9a

Mobile Web - Safari
https://github.com/Expensify/App/assets/39086067/15ca3e85-0477-46ec-8cd3-183507c33929

Desktop
https://github.com/Expensify/App/assets/39086067/cd3a1d91-3ab0-4727-8a53-d8a9179fff27

iOS

Android

I think we can deal with the limit when we hit it, we don't need to fix that right now.

@namhihi237 I think we decided not to implement the input limit. @stitesExpensify  let's remove it from this PR right?

@namhihi237 I am not sure how it is working for you on the Android? I thin you haven't tested/updated screenshots after pushing the last changes. numberOfLines={0} won't work on Android.

@namhihi237 I don't think this fix works. It's a code smell, and doesn't make sense to repeat the same component twice?

Thanks I think this works.  I can see we're refactoring numberOfLines in this PR. This would eventually get fixed there, but I think we're good with this change too

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible 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 checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

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

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (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] 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 verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[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] 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 reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/3069065/5af7d875-0be9-43dd-bfaf-eb1869088bb7

Mobile Web - Chrome
https://github.com/Expensify/App/assets/3069065/b6be6d32-b926-4b1e-8f1b-aeea3061d8a9

Mobile Web - Safari
https://github.com/Expensify/App/assets/3069065/88af0032-860c-401e-a418-88a82a2240d4

Desktop
https://github.com/Expensify/App/assets/3069065/e2ea1a21-fb6b-4973-af3a-9fa47c7b0496

iOS
https://github.com/Expensify/App/assets/3069065/1d0cbe57-d74f-4942-8e48-9e78978ca1bf

Android
https://github.com/Expensify/App/assets/3069065/f25096a8-709a-4710-9180-29e34d6e5ab1

@stitesExpensify All yours.
🎀 👀 🎀 C+ reviewed.

Just to confirm @mananjadhav everything is testing well still after the recent changes?

Oops, my GH wasn't updated, Just saw your new checklist. Ignore me!

Yes I checked after the last commit.

: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/stitesExpensify in version: 1.3.44-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@namhihi237 We have a regression when we put a continuous string without any space it overflows the description. https://github.com/Expensify/App/issues/23410#issuecomment-1647960564 Can we please post a PR to fix this please?

🚀 Deployed to production by https://github.com/mountiny in version: 1.3.45-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Fix a couple of grammar type Os  in README.md

I ran across, what I would consider, a couple of grammar type Os while familiarizing myself with this project. If you feel these will help, great! If not, please excuse my ignorance. Thank you for this project.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Already included in previous PR


Cannot catch exception in middleware

I have a middleware that catch OAuth exception then response the error to my desire format. It worked before but now it isn't. And also all status code now being 500 instead of correct code. I'm using lucadegasperi/oauth2-server-laravel package.

https://github.com/dingo/api/blob/master/src/Exception/Handler.php#L179-L194
OAuthException is not instance HttpExceptionInterface so that why it always 500 now. I guess that I have to register an error handler for this.

Just register an exception handler for it with the API.


⚠ Bofrost.de has degraded performance

In 56e6ae3, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1247 ms

Resolved: Bofrost.de performance has improved in c22c287.


The specified item could not be found in the keychain

VS Code Version: 1.55.2 (2021-04-13)
OS Version: Mac OS X Mojave 10.14.6

Steps to Reproduce:

enable preferences sync
authenticate to hotmail and github
problem doesn't occur immediately, but I see this error message in every VS Code window, every morning. Often it appears multiple times (see image).

This was also raised as https://github.com/microsoft/vscode/issues/95701 and closed as no repro. I'm here to provide evidence that it reproduces with v1.55.2. (I was unable to update the original bug report as it's locked).
PS the referenced Troubleshooting Guide does not mention this error.

Hi @john-aws 👋 I'm the new owner of the Auth stack. Just to do some bookkeeping, I'm wondering if you are still seeing this issue?


Custom rafDriver breaks spacebar playback

The rafDriver prop introduced in #374 is a welcome addition for things like rendering frame-by-frame but passing one as__experimental_rafDriver to studio.initialize breaks spacebar play and pause.
My theory, without diving too deep into it, is that DefaultPlaybackController#playDynamicRange, called by useKeyboardShortcuts, is relying on the ticker prop, and the basic rafDriver calls driver.tick(t) infinitely.
I'm assuming that the fix is for either somewhere in the call stack playDynamicRange needs to call driver.tick(t), it needs to be possible to swap out the rafDriver (i.e. not just in initialize, say, for rendering), or I'll need to do the bookkeeping in my own app to determine which "mode" we're in because otherwise theatre breaks. Thoughts?

Here's how I solved this in userland:
const [isRendering, setIsRendering] = useState(false)
useEffect(() => {
  if (isRendering) {
    return
  }
  let tick: number
  const cb = () => {
    rafDriver.tick(performance.now())
    tick = requestAnimationFrame(cb)
  }
  tick = requestAnimationFrame(cb)
  return () => cancelAnimationFrame(tick)
}, [isRendering])


[BUG] Website broken

Describe the bug
Both buttons in website just reload it
Environment:

OS: Windows
Browser [e.g. chrome, safari] Chrome/Edge

Recently, the web research preview is not stable due to a surge of usage. We are trying to improve its concurrent capacity.
Close this issue since it is not a bug with the this codebase.


Why deps.json and dev.json need to be writable?

Description

when run .Net Core cases, the deps.json and runtimeconfig.dev.json file need to be writable, why? when Look into https://github.com/Microsoft/vstest/blob/master/src/Microsoft.TestPlatform.CoreUtilities/Helpers/FileHelper.cs, in line 46, the FileAccess flag is readWrite. Could this be read only?

@qingjuntian Yes, It can be read only, Feel free to raise a PR to fix it.

Seems I do not have permission?

Did you fork the repo? If not please fork it and create a branch with your changes and raise the PR.

I do clone the repo, check out a new branch and get permission denied when I try to push:
"remote: Permission to Microsoft/vstest.git denied to qingjuntian.
fatal: unable to access 'https://github.com/Microsoft/vstest.git/': The requested URL returned error: 403". I also tried this from office and failed with my alias account: qitia

Yeah, you can't push a branch to remote https://github.com/Microsoft/vstest.git/.  You have to fork it(like you did for SNS-Shim), See https://help.github.com/articles/fork-a-repo/

Thanks Satya,
May I know how to install version with this fix? I tried to install Dotnet
Core SDK 2.1.300-preview3-008424 - which I think is the newest build. But
it seems still request write access to the dev.json file.
thanks.
Qingjun
On Thu, Jan 25, 2018 at 4:14 PM, Satya Madala<EMAIL_ADDRESS>wrote:

Closed #1345 https://github.com/Microsoft/vstest/issues/1345 via #1372
https://github.com/Microsoft/vstest/pull/1372.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vstest/issues/1345#event-1441122373, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFfJ7srpo0rWozvYCkxYBHUkTmYJtt2uks5tODfugaJpZM4RMMBC
.

@qingjuntian You are changes released in >= 15.7.0-preview-20180221-13. You can see the version when you run dotnet test.

I learnt while working on another issue that you can open a file in two different process with read mode also. You should specify fileshare while opening file.

Can you explain your scenario?
How are you running into this issue?
Did you test your changes for your scenario?

thanks Satya,
In fact I did not test the change with my scenario - that is why i am
asking here how to install version with this fix.
My scenario is when I run cases (my funtional test case project is
targeting Dotnet core app2.0)
1, the dev.json file is a soft link, which is not writable - I work around
this by copying the real json file
2, I need to run the cases in a parallel way, thus there is a race
condition where two processes try to require the same dev.json file's
writable access, this cause one of the cases failed.
Qingjun
On Mon, Apr 2, 2018 at 10:54 PM, Satya Madala<EMAIL_ADDRESS>wrote:

@qingjuntian https://github.com/qingjuntian You are changes released in

= 15.7.0-preview-20180221-13. You can see the version when you run dotnet
test.

[image: image]
https://user-images.githubusercontent.com/4337332/38200613-b0b3cd64-36b2-11e8-916a-958197e526c7.png
I learnt while working on another issue that you can open a file in two
different process with read mode also. You should specify fileshare
https://msdn.microsoft.com/en-us/library/system.io.fileshare(v=vs.110).aspx
while opening file
https://msdn.microsoft.com/en-us/library/y973b725(v=vs.110).aspx.

Can you explain your scenario?
How are you running into this issue?
Did you test your changes for your scenario?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vstest/issues/1345#issuecomment-377947277,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFfJ7vRrgcSoYTBD7VV9zAGpoLtf-NPDks5tkjuSgaJpZM4RMMBC
.

@qingjuntian I have update my earlier comment.

I learnt while working on another issue that you cann't open a file in two different process with read mode also. You should specify fileshare while opening file.

Because of above reason, your second scenario won't work with your fix.

The default value Fileshare  Enum when opening file is FileShare.Read
right?
https://referencesource.microsoft.com/#mscorlib/system/io/filestream.cs,460
In this case should I specify the fileshare parameter again?
Thanks,
Qingjun
On Tue, Apr 3, 2018 at 10:02 PM, Satya Madala<EMAIL_ADDRESS>wrote:

@qingjuntian https://github.com/qingjuntian I have update my earlier
comment.
I learnt while working on another issue that you cann't open a file in two
different process with read mode also. You should specify fileshare while
opening file.
Because of above reason, your second scenario won't work with your fix.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vstest/issues/1345#issuecomment-378260911,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFfJ7r9-mZR_jQpKSfNdlJw1L7hk3UcNks5tk4D-gaJpZM4RMMBC
.

hello Satya, could you please help check my previous question?
"
The default value Fileshare  Enum when opening file is FileShare.Read
right?
https://referencesource.microsoft.com/#mscorlib/system/io/filestream.cs,460
In this case should I specify the fileshare parameter again?"
Thanks,
Qingjun
On Sun, Apr 8, 2018 at 4:54 PM, qing jun tian<EMAIL_ADDRESS>wrote:

The default value Fileshare  Enum when opening file is FileShare.Read
right?
https://referencesource.microsoft.com/#mscorlib/
system/io/filestream.cs,460
In this case should I specify the fileshare parameter again?
Thanks,
Qingjun
On Tue, Apr 3, 2018 at 10:02 PM, Satya Madala<EMAIL_ADDRESS>wrote:

@qingjuntian https://github.com/qingjuntian I have update my earlier
comment.
I learnt while working on another issue that you cann't open a file in
two different process with read mode also. You should specify fileshare
while opening file.
Because of above reason, your second scenario won't work with your fix.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vstest/issues/1345#issuecomment-378260911,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFfJ7r9-mZR_jQpKSfNdlJw1L7hk3UcNks5tk4D-gaJpZM4RMMBC
.


ppc64le: remove seccomp from docker buildtags

Removes seccomp from ppc64le as a buildtag for now
@justincormack I'm going to take a more in depth look next week, but for now should fix the jenkins issues.
Signed-off-by: Christopher Jones<EMAIL_ADDRESS>

LGTM or we could just fix this next week...

let's merge this for now then, easy enough to enable it again once it's figured out
LGTM


add a detailed validation topic to FAQ

@koegerr i'll have a go at this update and will put you down to review the PR.


Release issue

Hi, i wanted to add your mods to the marketplace but the tar is not valid, you should take a look at sbordeyne's mods releases and workflows

@UwUDev Did y'all ever work out a viable solution for modpack-type repos? Or does it make more sense for me to split/unify this one as I see fit. This one was an issue for the marketplace before because I contained 3 separate but interoperable mods in one place.

@UwUDev My mistake, I didn't look too thoroughly at the workflows page yesterday. I have corrected the issue, and there are now 3 files in the release with the proper formatting. Please let me know if there's anything else that needs doing to get this working.

If it's all good then make a PR on this file https://github.com/balamod/mods/blob/main/index.json
Also make sure that your version tag matches the mod version and check if the version is formatted like this x.y.z

Fixed and sent.


History notfound event

This is a new feature that I factored out of an application I built at Utrecht University. It makes it possible to display a generic "404 not found" view whenever the URL changes but does not match any route known to Backbone.history. In the application, I hacked the feature in by overriding Backbone.history.loadUrl, which is an undocumented method. With this feature added to Backbone itself, that hack would no longer be necessary.
It was already possible to inspect the return value of Backbone.history.start() and display a 404 view if that value was false. However, there was no way to detect a non-matching URL that occurred after a hashchange or pushstate event, for example due to a coding error or a creative user hacking faulty URLs into the page. The notfound event addresses that gap.
Another way to look at it, is that notfound brings parity with the Backbone.sync events: notfound is to route as error is to sync.
I have some specific questions to the reviewer:

Do you agree that the feature is useful enough to justify a couple of extra lines of code?
Do you agree that notfound is an appropriate name for this event?
The event as currently proposed has no payload. Should it? If so, what should that payload be?
The change should be entirely non-breaking (just an extra event, no other changes in behavior), but please do not hesitate to warn me if you think I might be mistaken.

@GammaGames or @RayRaz does either of you have a "big" opinion?

I think it's a nice event to add, no objection!

Considering this usecase could be covered with a catch-all route it strikes me as a nice-to-have feature rather than a must-have.
Having said that, it's only very little code so unless micro-managing code size is a very high priority I don't see any strong arguments against this addition.

Yeah, I agree the event is somewhat redundant with a catch-all route. I didn't think of that at the time... :thinking:
While the added code size is negligible, a "con" might be that this feature introduces a new responsibility, something that must keep working as the surrounding code is edited. Then again, it doesn't seem like a difficult thing to maintain.
If I compare the notfound event to a catch-all route, I do think that the event is a little bit cleaner, so that would be a "pro". A catch-all route for 404 purposes does not seem like something that one would want to navigate to, nor does it seem appropriate to guard with an execute filter. It also seems intrinsically unrelated to any other routes that might be registered in the same router. Then again, a catch-all route is probably about as easy to use as a notfound event.
I could merge this, on the ground that all opinions so far lean to the mildly positive. I could also not merge this, on the ground that mildly positive opinions are not enough. There is something to say for requiring that new features enable workflows that weren't possible before.
Ah, I just realized there is actually such a workflow that is easier to implement with the notfound event. It is also the workflow I implemented in the application I factored this out of. Sometimes, a URL matches a known route but it should still lead to a 404 page. For example, consider /book/10/page/300, which matches the route pattern /book/:bookId(/page/:pageNum). If book 10 has only 160 pages, I might still want that route to lead to a 404 view. Doing so is easy by manually calling Backbone.history.trigger('notfound'), if I happen to listen for that event anyway. If I am handling unknown routes with a catch-all route instead, I will need to take additional steps to ensure that both situations are handled in the same way. Ironically, one possible approach in that case would be to trigger something like notfound on Backbone.history or some other global event bus in both situations.
I am now leaning towards merging, but I'll leave this open for a day, in case anyone can counter the above argument.

Yep so I do something similar triggering a notfound like event.  But I do this on a backbone.radio channel because I use multiple routers..  So if a router matches a route it can still be a 404, but the initial router handles unknown routes and listens for that event to display the 404.  One particular bonus I think I utilize is that depending on the user's permissions some of my routers may or may not be running, so the routers are completely unaware of each other.  However they would all have access to Backbone.history so I suppose that would be fairly the same as a radio channel.
In any case, I think my use case validates the pattern, even if I won't utilize the API.

In any case, I think my use case validates the pattern, even if I won't utilize the API.

Very reassuring, thanks!

This type of interesting insight into other people's usecases is why I like lurking here :smile:


Sending feedback for recurring events not working

Describe the Bug

The feedback endpoint only accepts event slugs which really exist in the database, so it fails for all recurring events where we generate a bunch of date-specific slugs.
Steps to Reproduce

Create recurring event
Retrieve event API endpoint to see which slug the recurring event now has
Try to submit feedback about the event with this specific slug

Expected Behavior

The feedback should be submitted
Actual Behavior

A 404 error occurs: No matching event found for slug.
Additional Information

This is similar to https://github.com/digitalfabrik/integreat-cms/issues/1718, but only from the concept, the solution needs to be different.

Do we need to validate a date-part of the slug? Check that it corresponds to the event?
Another option would be to simply remove the date from the slug, before we retrieve the event from DB. Does it sound okay?

Another option would be to simply remove the date from the slug, before we retrieve the event from DB. Does it sound okay?

Yes, I think removing the date string is perfectly fine, probably only relevant for edge cases to know which exact occurrence of a recurring event the feedback refers to.


Almost discrete (cont'd #821)

As discussed in #1058, open the discussion for the remaining part of #821.

@yhx-12243 Can you explain what this PR is about?  @danflapjax had initially written a "monster" PR #821 with too many results to reasonably deploy in one go.  Part of it was then separated and merged.  A few remaining theorems remained and I had made some comments about them.  Then things sat like this for a long time.  Later @danflapjax may have done more work?  But there was never any followup discussing the remaining comments, so I was waiting for him to split up the work into manageable pieces and re-review them one at a time.
And now you come in and do another big PR.  Please tell me what's the plan here?  And did you even discuss this with @danflapjax ?

@yhx-12243 Can you explain what this PR is about? @danflapjax had initially written a "monster" PR #821 with too many results to reasonably deploy in one go. Part of it was then separated and merged. A few theorems remained pending and I had made some comments about them. Then things sat like this for a long time. Later @danflapjax may have done more work? But there was never any followup discussing the remaining comments, so I was waiting for him to split up the work into manageable pieces and re-review them one at a time.
And now you come in and do another big PR. Please tell me what's the plan here? And did you even discuss this with @danflapjax ?
And also, why repeat this huge list of commits from before???

(i) Because his original pull request #821 is closed instead of being Draft, so we decided to add a continuation, and we can mark it as a draft.
(ii) Because I reuse the branch and remove the merged theorem, only leave the theorem can't deduced by the pi-base now.

When you say "we decided", do you mean "you decided"?  I.e., did you coordinate with @danflapjax ?
Also, the understanding we had from discussion with @danflapjax was that he was going to split this into several smaller PRs to be discussed individually.  Why bypass what he was going to do?
And did you take into account all the pending issues from #821?

I wouldn't mind at all if you were to work on anything that I have begun to bring it to completion, as this is a collaborative project. However, if you do so I ask that you do it with diligence and care. This pull request instead feels reckless. The previous PR was closed with the intention of splitting up the contributions into smaller chunks which can be more efficiently reviewed and revised, but this PR just has the exact same changes without any of the commented suggestions actually incorporated. Again, please feel free to take one or a few of the theorems and include them in a well-made PR of your own, but keep in mind their context when doing so.

Thanks for your intent to get these results added to $\pi$-Base @yhx-12243, but I agree: the discussion in the previous PR was that this needed to be split up into smaller PRs for ease of review. If you'd like to help work on this, please choose a small portion to contribute in a clean PR.


Added /comms/modf, which decomposes floats into integral and fractional outputs

Hope you have power!


Missing fingerprints should have higher priority than canError

When mock car is detected, the canError can show up along with Dashcam mode. This makes troubleshooting with users very difficult. Dashcam mode, Car unrecognized should be shown even if there is a can error resulting of the missing fingerprint.

Isn't the correct solution to make sure CS.canValid is True for the mock car? Then you still get the error in case there is no CAN data at all.

Fixed in ea06941d4d023b82eae8f9d342f972c4744353a1


Added functor laws and example to the functor section

Shouldn't we just link to fantasy-land for the laws? It will blow up all this short descriptions if we start adding all the laws and may be inaccurate or not uptodate with fl.
Was meaning to create a PR to link to all the laws, but was too busy.

I'm also for linking to some other place, but I was not sure what kind of extra resources are at the right level.
The laws are accurate and don't change, as for the conciseness, I created this PR to have the minimal correct description after reading a tweet pointing the danger of oversimplification.
I'll create another PR for monads and we can discuss more before merging if it's the right approach :)

Jop, the laws are accurate, but can easily diverge (with the fl-spec) for
more complicated types. IMO they are not needed to explain what a
functor/or other types are. I think it's best to link to the specification
of https://github.com/fantasyland/fantasy-land for this.
On Mar 23, 2016 07:57, "matthieubulte"<EMAIL_ADDRESS>wrote:

I'm also for linking to some other place, but I was not sure what kind of
extra resources are at the right level.
The laws are accurate and don't change, as for the conciseness, I created
this PR to have the minimal correct description after reading a tweet
pointing the danger of oversimplification.
I'll create another PR for monads and we can discuss more before merging
if it's the right approach :)
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/hemanth/functional-programming-jargon/pull/27#issuecomment-200213411

The laws won't diverge, really. These are not concepts that fl are making up or that we're redefining, they are established concepts that we just want to describe in an approachable way for the JS community.
FL is doing a very good job in describing in a terce way all of the typical algebraic structures found in FP, but for this repo to be useful, I think it should really offer more than concise explanations.
Maybe other of the maintainers of this repo can share their opinion here.

Initially I was opposed to diving into the laws, but this case isn't too confusing to me. Could be another case for a linked wiki for depth on the terms rather than fully explaining in the main document


ResourceExhaustedError in translate example (with GPU)

GitHub issues are for bugs / installation problems / feature requests.
For general support from the community, see StackOverflow.
To make bugs and feature requests more easy to find and organize, we close issues that are deemed
out of scope for GitHub Issues and point people to StackOverflow.
For bugs or installation issues, please provide the following information.
The more information you provide, the more easily we will be able to offer
help and advice.
Environment info
Operating System: Ubuntu 15.10
If installed from binary pip package, provide:
Which pip package you installed.

sudo pip install --upgrade https://storage.googleapis.com/tensorflow/linux/gpu/tensorflow-0.7.1-cp27-none-linux_x86_64.whl

The output from python -c "import tensorflow; print(tensorflow.version)".

I tensorflow/stream_executor/dso_loader.cc:105] successfully opened CUDA library libcublas.so locally
I tensorflow/stream_executor/dso_loader.cc:105] successfully opened CUDA library libcudnn.so locally
I tensorflow/stream_executor/dso_loader.cc:105] successfully opened CUDA library libcufft.so locally
I tensorflow/stream_executor/dso_loader.cc:105] successfully opened CUDA library libcuda.so.1 locally
I tensorflow/stream_executor/dso_loader.cc:105] successfully opened CUDA library libcurand.so locally
0.7.1

If installed from sources, provide the commit hash:
Steps to reproduce
Just run translate.py with 10M pairs training data.
What have you tried?
smaller batch size 32
Logs or other output that would be helpful
(If logs are large, please upload as attachment).
I tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:475] Sum Total of in-use chunks: 3.63GiB
W tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:211] Ran out of memory trying to allocate 32.00MiB.  See logs for memory state
W tensorflow/core/kernels/matmul_op.cc:158] Resource exhausted: OOM when allocating tensor with shape[2048,4096]
W tensorflow/core/common_runtime/executor.cc:1102] 0xe723bd30 Compute status: Resource exhausted: OOM when allocating tensor with shape[2048,4096]
[[Node: gradients_2/model_with_buckets/embedding_attention_seq2seq_2/RNN/MultiRNNCell/Cell0/BasicLSTMCell/Linear/MatMul_grad/MatMul_1 = MatMul[T=DT_FLOAT, transpose_a=true, transpose_b=false, _device="/job:localhost/replica:0/task:0/gpu:0"](model_with_buckets/embedding_attention_seq2seq_2/RNN/MultiRNNCell/Cell0/BasicLSTMCell/Linear/concat, gradients_2/model_with_buckets/embedding_attention_seq2seq_2/RNN/MultiRNNCell/Cell0/BasicLSTMCell/add_grad/Reshape)]]
Traceback (most recent call last):
File "translate.py", line 289, in
tf.app.run()
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/python/platform/default/_app.py", line 30, in run
sys.exit(main(sys.argv))
File "translate.py", line 286, in main
train()
File "translate.py", line 187, in train
target_weights, bucket_id, False)
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/models/rnn/translate/seq2seq_model.py", line 224, in step
outputs = session.run(output_feed, input_feed)
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 315, in run
return self._run(None, fetches, feed_dict)
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 511, in _run
feed_dict_string)
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 564, in _do_run
target_list)
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 586, in _do_call
e.code)
tensorflow.python.framework.errors.ResourceExhaustedError: OOM when allocating tensor with shape[2048,4096]
[[Node: gradients_2/model_with_buckets/embedding_attention_seq2seq_2/embedding_attention_decoder/attention_decoder/MultiRNNCell_3/Cell0/BasicLSTMCell/Linear/MatMul_grad/MatMul_1 = MatMul[T=DT_FLOAT, transpose_a=true, transpose_b=false, _device="/job:localhost/replica:0/task:0/gpu:0"](model_with_buckets/embedding_attention_seq2seq_2/embedding_attention_decoder/attention_decoder/MultiRNNCell_3/Cell0/BasicLSTMCell/Linear/concat, gradients_2/model_with_buckets/embedding_attention_seq2seq_2/embedding_attention_decoder/attention_decoder/MultiRNNCell_3/Cell0/BasicLSTMCell/add_grad/Reshape)]]
[[Node: clip_by_global_norm_2/clip_by_global_norm_2/_9/_3023 = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/cpu:0", send_device="/job:localhost/replica:0/task:0/gpu:0", send_device_incarnation=1, tensor_name="edge_179468_clip_by_global_norm_2/clip_by_global_norm_2/_9", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/cpu:0"]]
Caused by op u'gradients_2/model_with_buckets/embedding_attention_seq2seq_2/embedding_attention_decoder/attention_decoder/MultiRNNCell_3/Cell0/BasicLSTMCell/Linear/MatMul_grad/MatMul_1', defined at:
File "translate.py", line 289, in
tf.app.run()
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/python/platform/default/_app.py", line 30, in run
sys.exit(main(sys.argv))
File "translate.py", line 286, in main
train()
File "translate.py", line 155, in train
model = create_model(sess, False)
File "translate.py", line 132, in create_model
forward_only=forward_only)
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/models/rnn/translate/seq2seq_model.py", line 161, in init
gradients = tf.gradients(self.losses[b], params)
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/python/ops/gradients.py", line 483, in gradients
in_grads = _AsList(grad_fn(wrapped_op, *out_grads))
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/python/ops/math_grad.py", line 431, in _MatMulGrad
math_ops.matmul(op.inputs[0], grad, transpose_a=True))
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/python/ops/math_ops.py", line 951, in matmul
name=name)
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/python/ops/gen_math_ops.py", line 686, in _mat_mul
transpose_b=transpose_b, name=name)
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/python/ops/op_def_library.py", line 655, in apply_op
op_def=op_def)
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 2040, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 1087, in init
self._traceback = _extract_stack()
...which was originally created as op u'model_with_buckets/embedding_attention_seq2seq_2/embedding_attention_decoder/attention_decoder/MultiRNNCell_3/Cell0/BasicLSTMCell/Linear/MatMul', defined at:
File "translate.py", line 289, in
tf.app.run()
[elided 3 identical lines from previous traceback]
File "translate.py", line 132, in create_model
forward_only=forward_only)
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/models/rnn/translate/seq2seq_model.py", line 152, in init
softmax_loss_function=softmax_loss_function)
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/python/ops/seq2seq.py", line 926, in model_with_buckets
decoder_inputs[:bucket[1]])
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/models/rnn/translate/seq2seq_model.py", line 151, in
lambda x, y: seq2seq_f(x, y, False),
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/models/rnn/translate/seq2seq_model.py", line 115, in seq2seq_f
feed_previous=do_decode)
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/python/ops/seq2seq.py", line 691, in embedding_attention_seq2seq
initial_state_attention=initial_state_attention)
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/python/ops/seq2seq.py", line 621, in embedding_attention_decoder
initial_state_attention=initial_state_attention)
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/python/ops/seq2seq.py", line 530, in attention_decoder
cell_output, state = cell(x, state)
File "/home/yy/virtualenv/env1/local/lib/python2.7/site-packages/tensorflow/python/ops/rnn_cell.py", line 663, in call
cur_inp, new_state = cell(cur_inp, cur_state)

@prb12: Did you get a chance to look at this?

Could you please clarify what you mean by
"Just run translate.py with 10M pairs training data." ?
Are you saying that you're trying to run with a batch_size argument of 10 million?
Could you please provide the exact command line you used, along with any changes to translate.py, and it would also be helpful to know what GPU configuration you have (e.g. the output of nvidia-smi)

Closing due to lack of response. Please reopen if you have more information.

Use small model will work.
python translate.py
--data_dir [your_data_directory] --train_dir [checkpoints_directory]
--size=256 --num_layers=2 --steps_per_checkpoint=50

Default parameters in translate are set to quite large values. Large models trained over a long time give good results, but it might take too long or use too much memory for your GPU. You can request to train a smaller model as in the following example.
python translate.py
  --data_dir [your_data_directory] --train_dir [checkpoints_directory]
  --size=256 --num_layers=2 --steps_per_checkpoint=50


Understanding difference between TensorFlow and PyTorch implementations of Unet

I am trying to convert the weights from a segmentation-models-based model that was trained in TensorFlow to PyTorch. I am using a U-net with an efficientnet-b0 backbone.
I noticed a mismatch in the dimensions of the parameters across the PyTorch and and TensorFlow implementations in the convolutional layers of the decoder. What is the source of this discrepancy, and how can I make the architectures match exactly?
Reproducible example:
# set up models

# pytorch
import torch.nn
import segementation_models_pytorch as smp
from functools import partial

pytorch_model = smp.Unet('efficientnet-b0', classes=4, activation=partial(nn.Softmax, dim=0))

# tensorflow
import segmentation_models as sm
tf_model = sm.Unet('efficientnetb0', classes=4, activation='softmax', encoder_freeze=False)

Observe the mismatch in the convolutional layers of the decoder, taking the first one as an example:
layer = tf_test.get_layer('decoder_stage0a_conv')
print(layer.weights[0].shape)
# TensorShape([3, 3, 1952, 256])
print(t.decoder.blocks[0].conv1[0].weight.shape)
# torch.Size([16, 432, 3, 3])

Note how the TensorFlow version has 1952 in channels, and the PyTorch version has 432 in-channels. Why is this the case?

Hii,
Try to convert the tensorflow model to ONNX and from ONNX to pytorch model, hope it will resolve this problem.

I am trying to convert the weights from a segmentation-models-based model that was trained in TensorFlow to PyTorch. I am using a U-net with an efficientnet-b0 backbone.
I noticed a mismatch in the dimensions of the parameters across the PyTorch and and TensorFlow implementations in the convolutional layers of the decoder. What is the source of this discrepancy, and how can I make the architectures match exactly?
Reproducible example:
# set up models

# pytorch
import torch.nn
import segementation_models_pytorch as smp
from functools import partial

pytorch_model = smp.Unet('efficientnet-b0', classes=4, activation=partial(nn.Softmax, dim=0))

# tensorflow
import segmentation_models as sm
tf_model = sm.Unet('efficientnetb0', classes=4, activation='softmax', encoder_freeze=False)

Observe the mismatch in the convolutional layers of the decoder, taking the first one as an example:
layer = tf_test.get_layer('decoder_stage0a_conv')
print(layer.weights[0].shape)
# TensorShape([3, 3, 1952, 256])
print(t.decoder.blocks[0].conv1[0].weight.shape)
# torch.Size([16, 432, 3, 3])

Note how the TensorFlow version has 1952 in channels, and the PyTorch version has 432 in-channels. Why is this the case?

Hello,
I m facing the same problem.
Do you have any solution pelase?
Thanks in advance


Add Codecov integration with Travis CI

Part of #119.

Current coverage is 37.17%

Branch #122 has no coverage reports uploaded yet.

No diff could be generated. No reports for master found.

Powered by Codecov. Updated on successful CI builds.

Yay, it works, now we need to also generate coverage report from functional tests to fulfill ui package coverage

So, I've added code coverage for instrumentation tests too (I hate Gradle and Groovy, everything so  implicit, uh)
And updated Jacoco to latest version, release notes: https://github.com/jacoco/jacoco/releases

Added badge (heh) to show coverage of the master branch (it will show value after merge of this PR).
So far so good, coverage is 82%, we can improve it in future. After merge I'll setup more rules on the codecov.io to also report status check and fail builds.
@vanniktech, @lenguyenthanh PTAL

Super cool @artem-zinnatullin . It's more visualization and now we  can look at code coverage directly on the PR. Please add more rules to get better PRs.

Status check set fail build if code coverage will be less than 82% from now.

Please add more rules to get better PRs.

You mean more checkers like PMD, FindBugs, Lint, etc?

Like this one

Status check set fail build if code coverage will be less than 82% from now.

And some rules like: build fail if we decrease code coverage.

Looking really good and nice :+1:

@artem-zinnatullin & @lenguyenthanh there's also a really nice browser extension

No Safari support :(
On Sun, 3 Apr 2016, 16:26 Niklas Baudy<EMAIL_ADDRESS>wrote:

@artem-zinnatullin https://github.com/artem-zinnatullin & @lenguyenthanh
https://github.com/lenguyenthanh there's also a really nice browser
extension https://github.com/codecov/browser-extension#codecov-extension
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/artem-zinnatullin/qualitymatters/pull/122#issuecomment-204975530

You're using Safari? :D

Why not @vanniktech  😄

It works faster than Chrome, eats less memory and when I work on MacBook it
lives much longer on battery
On Sun, 3 Apr 2016, 17:27 Thanh Le<EMAIL_ADDRESS>wrote:

Why not @vanniktech https://github.com/vanniktech 😄
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/artem-zinnatullin/qualitymatters/pull/122#issuecomment-204985145


Fix issues from yamllint + ansible-lint

Hi Sujith,
After runing yamllint + ansible-lint onyour code (I suppose you did not ;-) ), I encountered several (minor) issues. As this does interfere this my customer, I would appreciate it if you would merge the fixes into your code.
Thanks for the help and your effort for making this code!
Mark

#5


Album year doesn't show correctly (Invalid Date)

Describe the bug
Album year is not shown correctly (Invalid Date)
To Reproduce
Steps to reproduce the behavior:

Install Dynamic theme
Play any song

Screenshots

Desktop Setup

Operating System:  Alt Linux kde 5, Fedora 40 Kde 6
Spotify:  Spotify   installed from flatpak <IP_ADDRESS>1.ge66eb7bc
Spicetify:  2.36.12
Theme:  4.9

Logs

I'm unable to reproduce this on Windows, theme installed from MarketPlace.

Yeah it reproduces cause spotify is blocked in my country, although i can use it without vpn (after i login with vpn). If i turn on vpn and restart spotify it starts to show date correctly. I wonder if there is another way to code getting album year without needing to have vpn to see it, cause anything other than that works correctly without vpn. Also it would be super cool to have ability to customize this wonderful theme directly from spotify (set different background animation settings or turn it off completely). I know that i can do that by modifying css and js files, but that's not so convenient and user friendly


AnnotationSensitivePropertyNamingStrategy Causing Deserialization Errors

I've got a few POJOs that have fields of type ObjectNode.  When I try to deserialize them w/ the Dropwizard ObjectMapper I get the following error from Jackson:
com.fasterxml.jackson.databind.JsonMappingException: Conflicting setter definitions for property "all": com.fasterxml.jackson.databind.node.ObjectNode#setAll(1 params) vs com.fasterxml.jackson.databind.node.ObjectNode#setAll(1 params)

I've tracked it down to AnnotationSensitivePropertyNamingStrategy.  Here's a reproducer:
  public class AnnotationSensitiveExample {
    public String id;
    public ObjectNode json;
  }

  ObjectMapper mapper = new ObjectMapper();
  mapper.setPropertyNamingStrategy(new AnnotationSensitivePropertyNamingStrategy());
  AnnotationSensitiveExample deserialized =
        mapper.readValue("{ \"id\": \"1\", \"json\": { \"foo\": \"bar\", \"baz\": \"bing\" } }", AnnotationSensitiveExample.class);

If you remove the call to mapper.setPropertyNamingStrategy() or add a call to mapper.setPropertyNamingStrategy(null); after the 1st call, deserialization works fine.

Thanks for reporting this issue in the Jackson issue tracker. Let's wait what Tatu has to say. Maybe we can piggyback on a fix in jackson-databind.

Looks like the issue is resolved in Jackson.


Compatible with Keepass 2.41

I can't make this work, even with your packed version in the other post or freshly compiled using latest keepass version (2.41). The EnrollCapture window appears... but just hangs there, it doesn't seem to do anything....am i doing something wrong?

Pls explain in Wiki step by step, how exactly to compile a working plugin for Keepass 2.41 in Visual  Studio 2017?

I try to attach but I receive an error for attach too big.
At the following link on my onedrive (and also the src on github) the version compatible with keepass 2.41 .
https://1drv.ms/u/s!Ajk6HSaqH4NngcFe_cqK-4zyySKeFg
Bye

From: sambul13<EMAIL_ADDRESS>Sent: 2/27/19 7:39 PM
To: fabiocbr75/KeePassFingerprint<EMAIL_ADDRESS>Cc: fabiocbr75<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [fabiocbr75/KeePassFingerprint] Compatible with Keepass 2.41 (#3)
Pls explain in Wiki step by step, how exactly to compile a working plugin for Keepass 2.41 in Visual Studio 2017?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks for uploading. But... it doesn't work stably. It doesn't remember "Fingerprint Key Provider: option in Password Window, one must select it manually each time and press OK. Then CheckFinger window opens, but in 1 sec it crashes KeePass and closes. Only 1 time out of 10 the window stayed open, and it accepted my finger OK.

i try test 3 version 2.41 2.46 2.48
Same erro on CheckFinger
winbioopensession failed: configurationfailure
[https://user-images.githubusercontent.com/14312833/132072490-653b7a5b-19dc-42d8-89a1-18951f4129d2.png]

Windows Hello integration created.

what this mean?

Hello,
To avoid some incompatibilities with the hardware configuration, I created a new plugin that uses the windows hello API. This new plugin is in the KeePassWindowsHello repository. Now all the reader configuration is removed and you have only to set the masterkey.


3c284752-1145-450b-8414-2a38ff81fc8b

this is for automated testing

81a0505f-b744-4649-9359-4ff6a5280640


added solution for CodeForces Problem Number: 1875A

CodeForces Problem Link: https://codeforces.com/contest/1875/problem/A
Problem Number: 1875A

Accepted


Automate the push from github to gitlab

It would be ideal to include this as part of the Kube transition. The goal is to have mirroring repositories. See notes here for greater context: https://colabcoop.atlassian.net/wiki/spaces/IRIS/blog/2023/05/09/2615115777/IRIS+Full+Team+Sync+Release+Planning+May+2023
@JureUrsic - you communicated during the meeting that more research would be needed for this item. Can you also please estimate?

Now that the new dev site is up, this is a very high priority task.

So research showed that mirror Pull is a premium feature on Gitlab.
Here is the easiest solution.
Add the following git hook into your repository. .git/hooks/post-commit.
#!/bin/sh

# Function to check if the current branch is in the list
is_selected_branch() {
  local branch
  for branch in "${selected_branches[@]}"; do
    [[ "$branch" == "$current_branch" ]] && return 0
  done
  return 1
}

# Define the selected branches
selected_branches=("production" "dev" "staging")

# Get the current branch
current_branch=$(git symbolic-ref --short HEAD)

# Check if the current branch is in the selected branches list and the remote name is 'origin'
if is_selected_branch; then
  echo "Pushing branch '$current_branch' to 'kube' remote..."
  git push kube "$current_branch"
fi

exit 0

This will deploy the code to the kube remote when the user commits to the dev, staging, or production branch.

A few thoughts:

Can we use GitHub actions that are triggered on merges to the relevant branches?
In GitHub "dev" is the primary branch; however GitLab uses a 'development' branch for the dev deploy. At the moment we've mirrored this to have a 'development' branch in GitHub but I'd like to remove that and just have the 'dev' branch from GitHub push to the 'development' branch in GitLab.
Do we also want to push tags to GitLab. I don't think they are necessary for any deployments, but is it something that we'd want there as well?

I did a quick search and found 'Mirror to GitLab and trigger GitLab CI' GitHub action which sounds like it would do what we want. Although you'll want to review it to make sure it does, and that the codebase is good. Another option where you'd write your own GitHub action such as https://stackoverflow.com/questions/63308904/push-to-gitlab-with-access-token-using-github-actions. Again, that's just an example there may be better ways of doing that.

I have tested the following to mirror a fork to gitlab.
There are a few setup steps.

Create an SSH key for the mirroring and:

Add the public key into Gitlab Deploy Keys with write access.
Add the private key with the name GITLAB_SSH_KEY to Github Secrets on the repository.

Add the following Github Variables. GIT_USER, GIT_EMAIL, GITLAB_URL

Action workflow .github/workflows/mirror-to-gitlab.yml.
name: Mirror branches to GitLab

on:
  push:
    branches:
      - dev
      - staging
      - production

jobs:
  mirror:
    runs-on: ubuntu-latest

    steps:
      - name: Checkout code
        uses: actions/checkout@v2

      - name: Set up Git
        run: |
          git config --global user.name "${{ vars.GIT_USER }}"
          git config --global user.email "${{ vars.GIT_EMAIL }}"
          git remote add gitlab ${{ vars.GITLAB_URL }}

      - name: Install SSH key
        run: |
          mkdir -p ~/.ssh
          echo "${{ secrets.GITLAB_SSH_KEY }}" > ~/.ssh/id_rsa
          chmod 600 ~/.ssh/id_rsa
          ssh-keyscan gitlab.com >> ~/.ssh/known_hosts

      - name: Push to GitLab - Development branch
        if: github.ref == 'refs/heads/dev'
        run: |
          git push gitlab HEAD:development

      - name: Push to GitLab - Staging branch
        if: github.ref == 'refs/heads/staging'
        run: |
          git push gitlab HEAD:staging

      - name: Push to GitLab - Production branch
        if: github.ref == 'refs/heads/production'
        run: |
          git push gitlab HEAD:production

I don't have access to add secrets or variables to the repository. So I will need somebody with proper access to finish those steps before the workflow can be released.

@jobara I can handle the the Gitlab side if this needs to be expedited?
I just don't have access on the Github side to complete the necessary steps.

@marvinroman if you can send me the secrets, I can add them to the repo.

@marvinroman also, do you want to automatically push tags across to GitLab? I think that's missing from your code snippet.

@marvinroman if you can send me the secrets, I can add them to the repo.

I thought it would be easier to do in the reverse. If you create the SSH key and you put the private key into GITLAB_SSH_KEY you could just pass the public to me. Then I don't have to worry about securely passing you the private key.

@marvinroman if you can send me the secrets, I can add them to the repo.

I thought it would be easier to do in the reverse. If you create the SSH key and you put the private key into GITLAB_SSH_KEY you could just pass the public to me. Then I don't have to worry about securely passing you the private key.

Okay, I guess when we have the format of the script settled we can add the secret. Are there other secrets that need to be added too?

Okay, I guess when we have the format of the script settled we can add the secret. Are there other secrets that need to be added too?

The rest will be variables and not secrets.
GIT_USER=
GIT_EMAIL=
GITLAB_URL="git@git.kube.v1.colab.coop:iris/iris-accessibility.git"


Add support for bindings to overloaded functions

This commit (re)adds support for generating bindings to overloaded
functions. First, we use the symbol name instead of the canonical name
when checking if we have already generated bindings for a given
function. Second, we add a map from canonical names to number of times
we have generated a function with that canonical name. We increment this
number every time we generate a function. When the number is greater
than zero, we append it to the function name so that overloaded
functions get a unique Rust symbol.
Fixes #239
r? @emilio

@bors-servo r+
Perfect, thanks!

:pushpin: Commit 5851026 has been approved by emilio

:hourglass: Testing commit 5851026 with merge bdfab64...

:sunny: Test successful - status-travis

As this solution seems to only use the order of appearance - could this result in conflicts if the order of overrides changes in the source?

As this solution seems to only use the order of appearance - could this result in conflicts if the order of overrides changes in the source?

Yes. In general, if the source changes, the bindings are likely to as well.


1.0.2 causes crash

I had Sync working with version 1.0.1, then updated to 1.0.2 today and now my app crashes with the below.  I reverted back to 1.0.1, and it works again.
*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '*** -[NSDictionary initWithObjects:forKeys:]: count of objects (4) differs from count of keys (0)'
*** First throw call stack:
(
	0   CoreFoundation                      0x0000000103907c65 __exceptionPreprocess + 165
	1   libobjc.A.dylib                     0x0000000105472bb7 objc_exception_throw + 45
	2   CoreFoundation                      0x0000000103850210 -[NSDate isEqual:] + 0
	3   CoreFoundation                      0x00000001038628e4 +[NSDictionary dictionaryWithObjects:forKeys:] + 52
	4   DATAFilter                          0x000000010337033e +[DATAFilter changes:inEntityNamed:localKey:remoteKey:context:predicate:inserted:updated:] + 702
	5   Sync                                0x00000001033a29cc +[Sync changes:inEntityNamed:predicate:parent:inContext:dataStack:completion:] + 1484
	6   Sync                                0x00000001033a3ef2 -[NSManagedObject(SyncPrivate) sync_processToManyRelationship:usingDictionary:andParent:dataStack:] + 1986
	7   Sync                                0x00000001033a335e -[NSManagedObject(SyncPrivate) sync_processRelationshipsUsingDictionary:andParent:dataStack:error:] + 558
	8   Sync                                0x00000001033a45b5 -[NSManagedObject(SyncPrivate) sync_processToOneRelationship:usingDictionary:andParent:dataStack:error:] + 1365
	9   Sync                                0x00000001033a35fd -[NSManagedObject(SyncPrivate) sync_processRelationshipsUsingDictionary:andParent:dataStack:error:] + 1229
	10  Sync                                0x00000001033a2cbc __78+[Sync changes:inEntityNamed:predicate:parent:inContext:dataStack:completion:]_block_invoke + 172
	11  DATAFilter                          0x0000000103370816 +[DATAFilter changes:inEntityNamed:localKey:remoteKey:context:predicate:inserted:updated:] + 1942
	12  Sync                                0x00000001033a29cc +[Sync changes:inEntityNamed:predicate:parent:inContext:dataStack:completion:] + 1484
	13  Sync                                0x00000001033a1c69 __61+[Sync changes:inEntityNamed:predicate:dataStack:completion:]_block_invoke + 121
	14  DATAStack                           0x000000010337d76a __43-[DATAStack performInNewBackgroundContext:]_block_invoke + 58
	15  CoreData                            0x0000000103469a79 developerSubmittedBlockToNSManagedObjectContextPerform + 201
	16  libdispatch.dylib                   0x0000000106939614 _dispatch_client_callout + 8
	17  libdispatch.dylib                   0x00000001069206a7 _dispatch_queue_drain + 2176
	18  libdispatch.dylib                   0x000000010691fcc0 _dispatch_queue_invoke + 235
	19  libdispatch.dylib                   0x00000001069233b9 _dispatch_root_queue_drain + 1359
	20  libdispatch.dylib                   0x0000000106924b17 _dispatch_worker_thread3 + 111
	21  libsystem_pthread.dylib             0x0000000106ca6637 _pthread_wqthread + 729
	22  libsystem_pthread.dylib             0x0000000106ca440d start_wqthread + 13
)
libc++abi.dylib: terminating with uncaught exception of type NSException

@bonebox Hey Jeremy! Damn, this sucks, could you provide me an example of your model and the JSON that you're using, please?

I think I figured out where the problem is occurring, and it actually exists in 1.0.1 as well, but just doesn't cause a crash. I have an entity that that has a one-to-many relationship with itself.  After performing the sync, I noticed that the parent object gets populated ok, but none of the child objects do.  There are 4 of them, which I think is where the count of objects (4) differs from count of keys (0) is coming from.
Here's the data you requested. Let me know if you need anything else.
...
 "element" : {
            "type" : "group",
            "label" : "Triage",
            "elements" : [
                {
                    "type": "select",
                    "label": "Reason for Visit",
                    "reference": "reasonForVisit",
                    "selection": "open",
                    "items": [
                        { "value": "none", "label": "None" }
                    ]
                },
                {
                    "type": "select",
                    "label": "Allergies",
                    "reference": "allergies",
                    "selection": "open",
                    "items": [
                        { "value": "none/no known allergy", "label": "None/No Known Allergy" }
                    ]
                },
                {
                    "type": "select",
                    "label": "Last Vital Signs",
                    "reference": "lastVitalSigns",
                    "selection": "open",
                    "items": [
                        { "value": "none", "label": "None" }
                    ]
                },
                {
                    "type": "select",
                    "label": "Current Medications",
                    "reference": "currentMedications",
                    "selection": "open",
                    "items": [
                        { "value": "none", "label": "None" }
                    ]
                }
            ]
        }
...

Thanks! I'll make a test case to reproduce the issue.
Have a great day (or night🌚)!

Well, I discovered the problem!
I have hyper.isPrimaryKey on the property elementType of the Element entity.  The corresponding JSON property is type (as you can see in my previous post), so I set it up this way according to the README since you have type listed as a reserved attribute.  Once I moved hyper.isPrimaryKey to a different property, it works!  So it seems there is a bug that has something to do with reserved attributes can't also be used as primary keys.

@bonebox We did a bunch of improvements for many-to-many relationships on Sync 1.0.3 would you mind verifying if this is still an issue? Thanks in advance :)

Hey @3lvis sorry, forgot about this.  I just tested it, and it does work using a reserved type as primary key.  What I found though, is in addition to specifying the isPrimaryKey value, you also have to specify the remoteKey value, otherwise it will crash.  That may be worth clarifying in the documentation, as it could be confusing for people, since they normally wouldn't need to specify remoteKey (if the attribute is named appropriately).

Hi @bonebox, you're right. The documentation on using isPrimaryKey in addition to remoteKey could be improved. I'll make a PR fixing it :)
Thanks!

Moved to #101 (documentation missing)


Waypoint: Condense arrays with reduce - currentVal should be wrapped in code tags

In Waypoint: Condense arrays with reduce, the variable currentVal in the third paragraph should be wrapped in code tags, just like in the second paragraph where it was initially mentioned.

The line of code needed to be fixed is here on line 228.

I think 21 minutes is a decent turn-around for a fix.

Sub 20 or bust! :stuck_out_tongue_winking_eye:

@ltegman Well, I had it up in 16, took you long enough to merge it. ;)


Add equals method to ShallowWrapper.

Fixes #122

This is great. Thanks for contributing!
One thing I'd like us to consider:
Crazy thought: What if we had this be an extension of the .is(selector) API instead?
The issue in this case would be the fact that this would actually conflict with #110 unless we had some way of differentiating a JSX element from an object that just looks like a JSX element...
thoughts? cc @blainekasten @ljharb @hartzis
.equals() might be the way to go...

cc @vesln for how we might want to use this in chai-enzyme.

.is(<Node />) is appealing, but it would require similar API in ReactWrapper, and I'm wondering how this would behave:
const Bar = () =>
;
const Foo = () =>
<Bar />
;

// obviously this returns true:
render(<Foo />).is(

);
// should this returns true as well?
render(<Foo />).is(
<Bar />
);

Thanks for the cc @lelandrichardson. I'm personally not that excited about this API in general. My thoughts are that this gives the wrong intention to testing components.
What I think should be the primary goal in testing react components, is that given a certain set of state/props/context, an expected HTML render is asserted. Not an expected JSX call. At that point, you're just duplicating code and making sure it doesn't change. But the purpose of tests is to assert results, not implementations. Because I don't care if code get's refactored in a different way as long as the end result is the same.
If I used this method, I could see somebody doing something like this:
const component = render(

);

component.equals(

) // true

But then, say a refactor happens to this
Foo =
const component = render(

<Foo/>

);

now all of a sudden, the result is the exact same, but my tests have failed. So I just think this encourages the wrong habits. I see the benefit of this to make sure something is rendered that may be dynamic based on a component attribute, but that should be tested in other ways I think.

@lelandrichardson :+1:

@lelandrichardson it should be quite straight forward for chai-enzyme to implement.
expect(xyz).to.equal(xyz)

Great explanation @blainekasten.
Though I don't agree completely that the primary goal of testing react components needs to be html output. Using shallow render you're able to keep the scope of a test to a single component and make assertions about props being passed and what nodes/components will be 'rendered'.
I could see this being useful is for shallow render testing. I also like @vesln's idea, but it does tie you to chai.

I agree with @hartzis . While I think this API could be used in a way that I wouldn't encourage (as you describe), I think the same could be said for most of the APIs on enzyme. This provides the ability for someone to describe what a specific sub-tree of the resulting output looks like, and I think that is powerful.
After giving it some thought, I think this is fine to be on .equals() instead of .is() as well.
As a result, this PR is ready to merge.

rebased onto master.


Refactor statistics extractor to use Robot API visitor

It's proposal of refactor for extracting keywords from result using Robot parsing API visitors.
It should be tiny bit faster on larger files (150Mb+). I did not test it thorougly yet - it's draft.

Thanks @bhirsz, it looks good! If you could also add tests for that it would be great.

@bhirsz I was looking into that one, and one thing is bottering me. Which robotframework version is intended to use with this feature? 3.2.2 seems to be not enough:
File "C:\repo\rfhub2\rfhub2\cli\statistics\statistics_extractor.py", line 15, in visit_keyword
    keyword.body.visit(self)
AttributeError: 'Keyword' object has no attribute 'body'

If this feature needs RFWK 4.0 I will mark it appropriate Milestone.

@MaciejWiczk I apologise for leaving this PR - I was bit busy recently. It should support 3.2 with one change - in RF4 "keywords" attribute was renamed to body. So depending on version you should access it with different name

But when I think about it its possible we could replace that line with self.generic_visit (and it should work for both 3.2 and 4)

I apologise for leaving this PR - I was bit busy recently.

You've been busy, with great results! :)

But when I think about it its possible we could replace that line with self.generic_visit (and it should work for both 3.2 and 4)

Oh cool, I will take a look at this approach and see what's what :)

@bhirsz is this what You mean:
    def visit_keyword(self, keyword):
        try:
            keyword.body.visit(self)
        except AttributeError:
            keyword.keywords.visit(self)
        self.statistics[keyword.name].append(keyword)


Implement remove

Copy and adjust code from Vec::remove()

Thank you for contribution!
Published a new version to Crates.io.


Add support for simple tech tree

Simplest way I can think of to do this is to have technologies that you buy and units that require those technologies in order to build them.

cc @Conaws, suggestions welcome :smile:

[image: Inline image 1]
Just finished implementing a super basic version of this -- gist here
https://gist.github.com/630c44cc44f1f5bd8613184374d36444
Basically, the minimal version just puts things into columns, then
highlights pre-reqs and things it enables in different colors
But I got started on an SVG that draws lines between dependencies
On Fri, Jan 13, 2017 at 2:18 AM, David Whittington<EMAIL_ADDRESS>
wrote:

cc @Conaws https://github.com/Conaws, suggestions welcome 😄
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Zetawar/zetawar/issues/45#issuecomment-272278912, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACljr-iw6lQUU3Hm2K310emDxsp9O4M4ks5rRpGDgaJpZM4LiNqB
.

--
Conor White-Sullivan
347-216-0173
@Conaw http://twitter.com/conaw
Good tests kill flawed theories, we remain alive to guess again.
~ Karl Popper

My bad -- the gist above is just for visually laying out the tech-tree once
you've got it.
For the core implementation of the tech-tree, I figure you'd just have a
ref to pre-reqs  -- can't build tanks before cannons
cost for tech -- simplest thing would be cash, since it's acquired each
turn and it forces a tradeoff between research and building unitys
then each user would have to have the set of tech they'd bought
you could store this as refs at the user level
user/acquire-tech
when you want to purchase something, your options are based on the units
available via your acquired-tech
one option is New-Tech, whose cost is "???", which would generate a modal
with the tech-tree
Another thought -- tech wouldn't just have to be new units, it could
upgrade units as well -- so alpine-skis might allow your infantry units to
move quickly over mountains
On Fri, Jan 13, 2017 at 5:04 AM, Conor White-Sullivan <
<EMAIL_ADDRESS>wrote:

[image: Inline image 1]
Just finished implementing a super basic version of this -- gist here
https://gist.github.com/630c44cc44f1f5bd8613184374d36444
Basically, the minimal version just puts things into columns, then
highlights pre-reqs and things it enables in different colors
But I got started on an SVG that draws lines between dependencies
On Fri, Jan 13, 2017 at 2:18 AM, David Whittington <
<EMAIL_ADDRESS>wrote:

cc @Conaws https://github.com/Conaws, suggestions welcome 😄
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Zetawar/zetawar/issues/45#issuecomment-272278912,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACljr-iw6lQUU3Hm2K310emDxsp9O4M4ks5rRpGDgaJpZM4LiNqB
.

--
Conor White-Sullivan
347-216-0173 <(347)%20216-0173>
@Conaw http://twitter.com/conaw
Good tests kill flawed theories, we remain alive to guess again.
~ Karl Popper

--
Conor White-Sullivan
347-216-0173
@Conaw http://twitter.com/conaw
Good tests kill flawed theories, we remain alive to guess again.
~ Karl Popper

Nice! I like the visualizations. Your implementation suggestions sound pretty similar to what I was thinking too. Being able to enhance existing units would require a little more work, but it's an interesting idea. The easiest way I can think of to implement it would be to add a unit-type-modifier entity that automatically gets applied to an associated unit-type if its pre-reqs are available.


[Istio v1 kiali:v1.17] refreshing kiali page results in 401 status

Rancher Server Setup

Rancher version: v2.5-head 5b2469f3c9
Installation option (Docker install/Helm Chart): Helm chart/docker install

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): RKE1 v1.20.11

Proxy/Cert Details: selfsigned

Information about the Cluster

Kubernetes version: v1.20.11
Cluster Type (Local/Downstream): Downstream custom cluster [oracle_linux_8.4_docker_20.10_packer] and ubuntu node driver cluster
Istio Version: v1.5.901

Describe the bug
kiali works fine on a cluster --> tools --> istio --> kiali if we do not refresh the kiali UI. but if the kiali UI is refreshed, the page goes down and a 401 response is seen in the API requests tab
To Reproduce

Created a downstream custom oracle cluster on an HA rancher server with 3 worker, 2 cp, 3 etcd as a standard user
Installed istio v1 version 1.5.901 version in the downstream cluster
Verified all the workloads were deployed.
From cluster --> tools --> istio v1 --> click on kiali and let kiali load completely
Refresh the page

Result
Upon a refresh, the following response is seen in the console logs
https://<redacted>/k8s/clusters/api/v1/namespaces/istio-system/services/http:kiali:20001/proxy/console/graph/namespaces/?edges=noEdgeLabels&graphType=versionedApp&namespaces=istio-ns&unusedNodes=false&injectServiceNodes=true&duration=60&refresh=15000&layout=dagre

Failed to load resource: the server responded with a status of 401 ()

Expected Result
Refreshing the UI should not result in the 401 error.
Additional context
Installing istio v1 1.5.901 with ubuntu node driver downstream cluster throw the below error when we refresh the kiali UI:
GET https://<redacted>/k8s/clusters/api/v1/namespaces/istio-system/services/http:kiali:20001/proxy/console/overview?duration=60&refresh=15000 500 (Internal Server Error)

clusters.management.cattle.io "api" not found

Similar issue is seen on v2 istio charts as well: https://github.com/rancher/rancher/issues/33739

Moving to 2.5.11

As Istio v1 is deprecated and users are encouraged to start using Istio v2 as well as the fact that this version of Istio is EOL by Istio, we won't be making a fix for this.


LFS-881: NightlyExport should check lastModified instead of created for forms to export

Add json processor option for lastModified
Adjust NightlyExportTask to lastModified date processor rather than created
Add indexing for lastModified and statusFlags to improve export performance

Based off of LFS-724 (#463) as lastModified field is introduced there.


Reconnection errors

Since the last update I just keep getting errors when I try to connect. It just keeps trying to constantly reconnect after that. Any way to fix this?

I think, server was unavailable due to increased traffic, sorry for that.


VTDecompressionSessionDecodeFrame(-12902) error: kVTParamaterErr

Hello,
I'm working on a project for IP camera using Websocket (wss protocol). I am trying to decode a h.264 raw frame using avios. In the wss receive method (binary data) I found the SPS [Unit8Array(14)] and PPS [UInt8Array(4)] so I think I was receiving the correct h.264 raw format. But after that, I was having a problem with the status returned by:
VTDecompressionSessionDecodeFrame(videoSession, sampleBuffer: sampleBuffer, flags: [._EnableAsynchronousDecompression], frameRefcon: nil, infoFlagsOut: &infoFlags)
The status always return H264Error.VTDecompressionSessionDecodeFrame(-12902).
I tried changing the flags but still no working. I have been reading about NAL Unit but I don't have much experience with VTDecompressionSession. I really appreciate for your kind reply.
Here is my implementation of your library:

I create a H264Decoder in viewDidLoad
In the socket received method, i try to decode the h.264 raw frame received.

 I check the videoSession and the sampleBuffer was not null so I don't know which parameter has fault. I am wondering if this is referring to a problem about decompressionOutputCallback because the initVideoSession() method, the line:
outputCallback.decompressionOutputCallback = callback as? VTDecompressionOutputCallback is always nil.
Maybe the reason is the callback is never called.
For more information this is my callback method.
fileprivate func callback(_ decompressionOutputRefCon: UnsafeMutableRawPointer,
                      sourceFrameRefCon: UnsafeMutableRawPointer,
                      status: OSStatus,
                      infoFlags: VTDecodeInfoFlags,
                      imageBuffer: CVImageBuffer?,
                      presentationTimeStamp: CMTime,
                      presentationDuration: CMTime){

    unsafeBitCast(decompressionOutputRefCon, to: H264Decoder.self).decompressionOutputCallback(sourceFrameRefCon, status: status, infoFlags: infoFlags, imageBuffer: imageBuffer, presentationTimeStamp: presentationTimeStamp, presentationDuration: presentationDuration)
}

Hello, I'm working on a project for IP camera using Websocket (wss protocol). I am trying to decode a h.264 raw frame using avios. In the wss receive method (binary data) I found the SPS [Unit8Array(14)] and PPS [UInt8Array(4)] so I think I was receiving the correct h.264 raw format. But after that, I was having a problem with the status returned by: VTDecompressionSessionDecodeFrame(videoSession, sampleBuffer: sampleBuffer, flags: [._EnableAsynchronousDecompression], frameRefcon: nil, infoFlagsOut: &infoFlags)
The status always return H264Error.VTDecompressionSessionDecodeFrame(-12902). I tried changing the flags but still no working. I have been reading about NAL Unit but I don't have much experience with VTDecompressionSession. I really appreciate for your kind reply.
Here is my implementation of your library:
1. I create a H264Decoder in viewDidLoad

2. In the socket received method, i try to decode the h.264 raw frame received.

 I check the videoSession and the sampleBuffer was not null so I don't know which parameter has fault. I am wondering if this is referring to a problem about decompressionOutputCallback because the initVideoSession() method, the line: outputCallback.decompressionOutputCallback = callback as? VTDecompressionOutputCallback is always nil. Maybe the reason is the callback is never called.
For more information this is my callback method.
fileprivate func callback(_ decompressionOutputRefCon: UnsafeMutableRawPointer,
                      sourceFrameRefCon: UnsafeMutableRawPointer,
                      status: OSStatus,
                      infoFlags: VTDecodeInfoFlags,
                      imageBuffer: CVImageBuffer?,
                      presentationTimeStamp: CMTime,
                      presentationDuration: CMTime){

    unsafeBitCast(decompressionOutputRefCon, to: H264Decoder.self).decompressionOutputCallback(sourceFrameRefCon, status: status, infoFlags: infoFlags, imageBuffer: imageBuffer, presentationTimeStamp: presentationTimeStamp, presentationDuration: presentationDuration)
}

Raw frame you are provided seems not to be AnnexB. The Annex B specification solves this by requiring ‘Start Codes’ to precede each NALU. A start code is 2 or 3 0x00 bytes followed with a 0x01 byte. e.g. 0x000001 or 0x00000001