Inserting a document in 2 different collections throws an error

Hi Everyone,
I was just trying out LokiJS for a project and found something that feels like a bug. If it isn't - sorry for wasting your time.
I was (for fun) going to benchmark the difference between a unique constraint and an index. When inserting the same document in two different collections, I get an error.

    throw new Error('Document is already in collection, please use update()');

How can my collections know of each other? To me it makes intuitive sense for them to be completely separated, unless joined.
Please see the code below:
var loki = require('lokijs')
var fs = require('fs')

var db = new loki('testBench');

//We create two collections - one with a unique constraint and one with an index
var nodesUnique = db.addCollection('nodesUnique', { unique: ['technicalPath'] });
var nodesIndex = db.addCollection('nodesIndex', { indices: ['technicalPath'] });
console.log(db.listCollections());

var doc = JSON.parse(fs.readFileSync('./cfr.json', 'utf8'));

// Get rid of the root node to just insert the records
var docDeeper = doc.nodes;

//Insert records in our unique index way of working
var readCfr = nodesUnique.insert(docDeeper);
nodesUnique.update(readCfr);
console.log(db.listCollections());
console.log(nodesUnique.by("technicalPath", "/2/1/1/1/"));

//Insert records in our binary index way of working
var readCfr2 = nodesIndex.insert(docDeeper);
nodesIndex.update(readCfr2);
console.log(db.listCollections());
console.log(nodesIndex.find({technicalPath: "/2/1/1/1/" }));

When you insert the object a field is added called something like $loki (I can’t remember the exact name but similar to this). This meta data field is used internally to keep track of the object. If you try to insert the same object elsewhere in Loki you get this error.

When you insert the object a field is added called something like $loki (I can’t remember the exact name but similar to this). This meta data field is used internally to keep track of the object. If you try to insert the same object elsewhere in Loki you get this error.

Thanks for the reply, but that doesn't seem to be the case. $loki is being maintained in readCfr or readCfr2. When adjusting the code to create a separate document, it's still showing the same behavior.
var loki = require('lokijs')
var fs = require('fs')

var db = new loki('testBench');

//We create two collections - one with a unique constraint and one with an index
var nodesUnique = db.addCollection('nodesUnique', { unique: ['technicalPath'] });
var nodesIndex = db.addCollection('nodesIndex', { indices: ['technicalPath'] });
console.log(db.listCollections());

var doc = JSON.parse(fs.readFileSync('./cfr.json', 'utf8'));
//Debug code to check if this worked
//for (var exKey in doc) { console.log("key:" + exKey + ", value:" + doc[exKey]); }

// Get rid of the root node and just insert the records
var docDeeper = doc.nodes;
var docDeeper2 = doc.nodes;

//Insert records in our unique index way of working
var readCfr = nodesUnique.insert(docDeeper);
nodesUnique.update(readCfr);
console.log(db.listCollections());
console.log(nodesUnique.by("technicalPath", "/2/1/1/1/"));

//Insert records in our index way of working
var readCfr2 = nodesIndex.insert(docDeeper2);
nodesIndex.update(readCfr2);
console.log(db.listCollections());
console.log(nodesIndex.find({technicalPath: "/2/1/1/1/" }));

It does work if I go all the way back and read it again from Disk, but if I check the length of the document (docDeeper), nothing has changed after insert.
var loki = require('lokijs')
var fs = require('fs')

var db = new loki('testBench');

//We create two collections - one with a unique constraint and one with an index
var nodesUnique = db.addCollection('nodesUnique', { unique: ['technicalPath'] });
var nodesIndex = db.addCollection('nodesIndex', { indices: ['technicalPath'] });
console.log(db.listCollections());

var doc = JSON.parse(fs.readFileSync('./cfr.json', 'utf8'));
var doc2 = JSON.parse(fs.readFileSync('./cfr.json', 'utf8'));
//Debug code to check if this worked
//for (var exKey in doc) { console.log("key:" + exKey + ", value:" + doc[exKey]); }

// Get rid of the root node and just insert the records
var docDeeper = doc.nodes;
var docDeeper2 = doc2.nodes;

//Insert records in our unique index way of working
var readCfr = nodesUnique.insert(docDeeper);
nodesUnique.update(readCfr);
console.log(db.listCollections());
console.log(nodesUnique.by("technicalPath", "/2/1/1/1/"));

//Insert records in our index way of working
var readCfr2 = nodesIndex.insert(docDeeper2);
nodesIndex.update(readCfr2);
console.log(db.listCollections());
console.log(nodesIndex.find({technicalPath: "/2/1/1/1/" }));

It seems that insert() injects metadata directly into the object you provide to it, without creating an internal copy. (@willaaam In your first example, try var docDeeper2 = Object.assign({}, docDeeper);.)
Sounds like a bug to me. If there are performance reasons why insert() doesn't shallow clone, documentation should warn about the side effects.
Relatedly, are the results from by() / find() / etc... shallow-cloned? If not, it would be possible to directly modify values (including internal fields like $loki) without an insert().

When you create the collection it’s a documented option if you want to clone the inserts and results.
Similar when you call data() on your results set there is an option to clone and an option to remove metadata.

@RobertGary1 Thanks, you're correct.
https://rawgit.com/techfort/LokiJS/master/jsdoc/Loki.html#addCollection

Name
Type
Attributes
Default
Description

clone
boolean

false
specify whether inserts and queries clone to/from user

cloneMethod
string

'parse-stringify'
'parse-stringify', 'jquery-extend-deep', 'shallow, 'shallow-assign'

@willaaam try
var nodesUnique = db.addCollection('nodesUnique', { unique: ['technicalPath'], clone: true, cloneMethod: 'shallow' });
var nodesIndex = db.addCollection('nodesIndex', { indices: ['technicalPath'], clone: true, cloneMethod: 'shallow' });

Don't want to stray too far off topic, but I think only collection.chain() has the option of cloning retrieved data? It looks like collection.by() and collection.where() will always return direct references to internal objects.
(Probably intended, but unexpected to API consumers and perhaps worth documenting more clearly?)


Update larg4services_dune.fcl

update protodunevd geo from v1 to v2

Hi @vgalymov, what a coincidence! I made the exactly same update in my commit last week: https://github.com/DUNE/dunesim/commit/c29dd7a1b13694b7bc47350aa010ca9743552132
I'm going to close this PR.


make returning errors consistent

Some functions return -1 and set errno, others returns -errno. We should be consistent, e.g. make all functions return -errno.

I'm not quite following the need for this -errno stuff. Usually, this is done - in the form of PTR_ERR() and friends - when we're returning a pointer, but we are doing this all over the place not just for that instance. There shouldn't be any need to do this via a "0 is success, otherwise you get an errno". There's quite a bit of code that finds itself switching the signs and it's a little hard to follow.

Is PTR_ERR and friends acceptable to use in user space? What we want to achieve here is to have a consistent way across functions  for returning error codes, I'm not particular how we do it, I'm open to suggestions.

I haven't seen it in userspace before TBH.
So far, there isn't really that much code that returns pointers, is there?
So I'd be inclined to not do the negative thing overall. Agreed on consistency for sure!
I would be inclined to return "errno" - so 0 for success, otherwise ENOENT etc.
Does that sound reasonable? Another preference?

But if we return,
0   - Success

0 - Failure

Then this will not address the case where function want to return success non-zero value e.g size
I think -errno solves this issue. So

=0 - Success
<0   -  Failure

That's a good point. But I'm also not sure we have too many of those. The int get_foo(size_t *sizep) would work for those, but it's a bit less elegant.
I really don't think -errno is appropriate for the public API - I don't know offhand of any other public APIs that do this. And given that, and how many places we'd have to be careful about public versus internal, I think it's best to stick with one of the standard C-library methods.
Having said that, if both you and Thanos think we should do -errno even for the public API we can go with that.

No I did not meant to say this behaviour for public API's. I meant it for internal functions.  I thought this issue was opened for internal functions and not public API's.

I was planning to address both internal and public APIs with this issue. We've been returing -errno simply because this is how the kernel does it. The benefit of this approach is that, as Swapnil said, we can return 0 for success and >0 for values. TBH I don't know whether there is a standard way public libraries APIs solve this.
Let's enumerate our options (we can also state pros/cons for each option):

Return -errno for error, >= 0 for success/value.
Return -1 for error, set errno.
Use PTR_ERR/ERR_PTR.
Pass pointer to function to communicate error.

Personally I find option 1 the simplest and least invasive in the code. Shall we have a poll on Slack? :)

I'm really loathe to set a new precedent for the public API here

Fixed by series ending in #433


nlopt

Hi,
Can FASTER be used with NLOPT instead of Gurobi?

Hi @hs1110 ,
No, NLOPT is not implemented in FASTER, only Gurobi
Hope this helps


Finishing dialogue

Hi, I'm new with scribble and I have a question. How can I close a dialogue and not restart them when the last message is finished?. I have seen that all the demo file examples are a loop and I don't know how to end the dialogue. I ask here because I have searched YouTube videos and I have not found anyone who does it, everyone gives examples that are loops.

This isn't a question that I can answer in detail. Scribble is just a text renderer. The logic that you're using to control where, and why that text is being drawn is out of scope.


Fix Avatar has no tooltip in thread report header and displaying "fake" text in WS chat

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/19825
PROPOSAL: https://github.com/Expensify/App/issues/19825#issuecomment-1568682796
Tests

Create new room from any workspace (if not exist)
Send any message in room chat
Thread that message
Verify that tooltip showing thread creator's email when hover avatar on thread report header or on LHN
Create a Workspace chat (if not exist)
Repeat 2->4
Go to Search page
Search threads above
Verify that tooltip showing creator's email for those threads in Search page

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

Offline tests

This tooltip is not affected by network
QA Steps

Same as Tests

[ ] Verify that no errors appear in the JS console

PR Author Checklist

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

Screenshots/Videos

Web

https://github.com/Expensify/App/assets/9639873/dd850494-dc23-4fc9-8111-6bc3a162f115

Mobile Web - Chrome

N/A

Mobile Web - Safari

N/A

Desktop

https://github.com/Expensify/App/assets/9639873/7154ade3-60ab-43f1-ac27-9505a6e2a074

iOS

N/A

Android

N/A

🚀 Deployed to staging by https://github.com/puneetlath in version: 1.3.27-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/puneetlath in version: 1.3.27-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.3.27-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Azure ML Studio

Hello,
I'm getting started here so this is more of a question, but how does this compare to Azure ML studio?
Thanks so much

AzureML studio (which is the new UI) or Azure ML studio (classic)? Here's the documentation for classic - https://docs.microsoft.com/en-us/azure/machine-learning/compare-azure-ml-to-studio-classic
AML and DevOps work together, AML has a workspace boundary that tracks individual assets and artifacts. Devops provides the code and pipeline management.
The AML studio is the new UI for Azure Machine Learning service.


The lidar_centerpoint is now built with stderr

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I'm convinced that this is not my fault but a bug.

Description
$ colcon build --symlink-install --cmake-args -DCMAKE_BUILD_TYPE=Release -DCMAKE_EXPORT_COMPILE_COMMANDS=ON --catkin-skip-building-tests --parallel-workers 8  --packages-up-to lidar_centerpoint
Starting >>> autoware_auto_geometry_msgs
Starting >>> autoware_auto_mapping_msgs
Starting >>> autoware_lint_common
Starting >>> tier4_debug_msgs
Finished <<< autoware_lint_common [0.46s]
Starting >>> autoware_cmake
Finished <<< autoware_cmake [0.47s]
Finished <<< autoware_auto_geometry_msgs [9.07s]
Starting >>> autoware_auto_perception_msgs
Finished <<< autoware_auto_mapping_msgs [9.74s]
Starting >>> autoware_auto_planning_msgs
Finished <<< tier4_debug_msgs [12.4s]
Finished <<< autoware_auto_planning_msgs [22.9s]
Starting >>> tier4_autoware_utils
Finished <<< autoware_auto_perception_msgs [25.4s]
Finished <<< tier4_autoware_utils [17.4s]
Starting >>> lidar_centerpoint
[Processing: lidar_centerpoint]
--- stderr: lidar_centerpoint

 	Cannot retrieve the public link of the file. You may need to change
	the permission to 'Anyone with the link', or have had many accesses.

You may still be able to access the file from the browser:

	 https://drive.google.com//uc?id=1KFhmA4oFT6CtZx5806QeMzn5H2tKa3oD

 	Cannot retrieve the public link of the file. You may need to change
	the permission to 'Anyone with the link', or have had many accesses.

You may still be able to access the file from the browser:

	 https://drive.google.com//uc?id=1iyk5VoQ4uNBGPZwypVZIMjSuSYAI1RxP

 	Cannot retrieve the public link of the file. You may need to change
	the permission to 'Anyone with the link', or have had many accesses.

You may still be able to access the file from the browser:

	 https://drive.google.com//uc?id=13aYPRHx17Ge4BqxzW9drAUSWTppjtUV5

 	Cannot retrieve the public link of the file. You may need to change
	the permission to 'Anyone with the link', or have had many accesses.

You may still be able to access the file from the browser:

	 https://drive.google.com//uc?id=14PJ_L3Jpz6Yi8GzoctVOEbGWcaCLArGp
...
---
Finished <<< lidar_centerpoint [43.2s]

Summary: 9 packages finished [1min 34s]
  1 package had stderr output: lidar_centerpoint

This problem seems to occur neceessary models are not downloaded.
Expected behavior
The lidar_centerpoint is built with no stderr.
Actual behavior
The lidar_centerpoint is now built with stderr.
Steps to reproduce
$ git clone git@github.com:autowarefoundation/autoware.git
$ cd autoware
$ mkdir src
$ vcs import src < autoware.repos
$ colcon build --symlink-install --cmake-args -DCMAKE_BUILD_TYPE=Release -DCMAKE_EXPORT_COMPILE_COMMANDS=ON --catkin-skip-building-tests --parallel-workers 8 --packages-up-to lidar_centerpoint

Versions

Ubuntu20.04
ROS2 galactic
autoware.universe: https://github.com/autowarefoundation/autoware.universe/tree/bb96ae7b56085a6054ad1ade970bad399faef35a

Possible causes
No response
Additional context
No response

@h-ohta cc @Shin-kyoto Probably the same reason.
https://github.com/autowarefoundation/autoware.universe/pull/871

@kenji-miyake Thank you for your information. I tried it, and I found that problems looks like resolved.
I'll make PR to fix this issue.


Set up persona support

fix #23

[x] set up personas support
[x] add modal to choose the persona
[ ] add a safeGetSituation behaviour (by adding the function from ngc to @publicodes/tools)
[x] refactor flags pattern matching (with decoder ?)

[!WARNING]
Seems to have some drops on performance: sliders aren't as fluid as they are currently in prod. Howerver, I'm not sure of the reason yet.

Closed in favor of #31


serix: fix check of objTypeDenotation in RegisterInterfaceObjects

In RegisterInterfaceObjects we check if all objs that were given, have the same objTypeDenotation, but we don't check that if we pass the objs one by one by calling this function multiple times.
This needs to be fixed.

My fault, we already fixed that. But I cleaned up some stuff and added a testcase now.


When autoscend is installed via git checkout this script errors on on svn update

Just an FYI.  If autoscend, or any of the, github hosted, dependencies of ttpack are installed via git checkout then running svn update on ttpack errors out.
For now I've commented out the autoscend entry in my copy of dependencies.txt.

Oh, thank you. I haven't looked at it in a while. Nice to see they finally implemented the git checkout.

fixed.
thank you for the notification.
I added a manifest so it will now install correctly using
git checkout https://github.com/taltamir/ttpack.git

and also I cleared all git dependencies from this project. instead instructing the user to manually install them as necessary.
You should make sure to use svn delete ttpack and on any dependency this broke. as well as git delete. then reinstall them. Sorry about the mess there, hopefully the transition from svn checkout to git checkout doesn't have any more bumps along the way


Feature Deletion with Layer Deletion

Features still remain visible on grid after layer has been deleted. They are removed after clicking on flow/control layer of a different level.


fix: Use unique workflow names for state machine integ tests

Issue #, if available: N/A
Description of changes:
Concurrent test runs in the same account would fail because test_state_machine_definition.py uses the same hardcoded state machine name in the test cases. This fixes it by generating a unique name using unique_name_from_base from the sagemaker SDK which we are using the the sagemaker integ tests.
Sample error:
tests/integ/test_state_machine_definition.py:72: AssertionError
------------------------------ Captured log call -------------------------------
ERROR    stepfunctions:stepfunctions.py:208 A workflow with the same name already exists on AWS Step Functions. To update a workflow, use Workflow.update().
=========================== short test summary info ============================
FAILED tests/integ/test_state_machine_definition.py::test_task_state_machine_creation

Most of the lines changed in this PR are trailing whitespace removals.
Testing
Ran the test suite:
$ tox -e py38 -- -s -vv tests/integ/test_state_machine_definition.py

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-sEHrOdk7acJc
Commit ID: 7a0ae00e163adb67f11e57cc028298772de2b4ff
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-sEHrOdk7acJc
Commit ID: 874723706cdf0bb19cdfb998770ab6186289d9a6
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


FOB Deleted on server restart or mission restart

Bug report
Basic Information
So, the T90 used in the RHS AFRF is bugged because of a T14 Track or something making a hardkill trigger making it invincible. Rather than report the bug to you, because of its RHS's issue. I decided to remove it from the template and maybe add in some cup stuff to supplement its removal. I also changed the sector capture size and the fob size in a small amount so I could build FOB closer to the airfields and have fixed-wing frames.
I load the new PBO on the server and it deleted all my placed assets for the FOB I had, I expected something like that. No biggie, so I made a new one. Well my server went through its restart and the FOB that was built now was deleted.
So anything that is placed by us in the build mode is deleted, including the FOB building itself anytime the server is shutdown or the mission isnt active.
Map used:
Tanoa
Mods used:
RHS Suite
F-18
HAFM
CUP
CBA
Dedicated or local game:
Dedicated
Source of the mission file (URL):
Steam release // KP Liberation Tanoa
Individual Things
Did you edit anything within the mission files?
Yeah.
If yes, what did you edit exactly?
As mentioned above, I've added Zeus, I made Zeus have access to all items. I've added some stuff to the RHS AFRF OPFOR template. I removed the T-90. I set the sector capture radius to 150 and the FOB build radius to 150.
Short Issue Description
FOB area and placed buildings, items, vehicles are deleted once the server is reset. All of them, placed by the build menu in the FOB build area are missing.
Expected Behaviour
When the server is turned back on or the missions is reselected to play, the FOB's buildings, Items and Vehicles should be there.
Steps to Reproduce the Issue
Started server, started mission, placed items. Waited 5 minutes, shut mission down, reloaded mission, items are gone.
Screenshots, Serverlog or any other helpful Information
12:04:00   Error position: <_classnamesToSave) &&                   >
12:04:00   Error Undefined variable in expression: _classnamestosave
12:04:00 File mpmissions__cur_mp.Tanoa\scripts\server\game\save_manager.sqf..., line 547
12:04:00 Error in expression <ob_range * 2)) select {
((typeof _x) in _classnamesToSave) &&
Seems to be relevant. I can provide a bigger log if need.

That's not irrelevant, that's your issue. You made a mistake in your edits. For this kind of support join the Discord server, as it's no bug in the framework. Recheck your edits.


ControlzEx 6.0.0 is not compatible with MahApps.Metro 2.4.10

The latest MahApps (2.4.10) is not compatible with the newest ControlzEx 6.0.0 or the version range needs to be updated.

True. MahApps v3.0 will be compatible. It's in the oven. However, you can use nightly builds for now if you need the latest features.


React Native source is not highlighted properly

React native refuses to recognize *.jsx files, So all react native source has a *.js extension. This is causing highlighting issues with code like this:
// example.js
class Example extends React.Component {
   render() {
    return (
      <View>
        <Text>{`${this.props.startDate}`}</Text>
      </View>
    );
  }
}

Is rendered by javascript mode like this:

I'm not sure if there's actually a bug here or not (since it renders OK with JSX mode). AFAICT, JSX won't be part of the ECMAScript spec. Maybe there's still something that can happen in CodeMirror to help alleviate this unfortunate design decision in React Native?

This is a problem of deriving the language from the extension, and then using extensions that are usually associated with another language. I don't think CodeMirror can be expected to automatically handle that. The solution is probably set up your editor to use jsx mode for .js files.

Hmm, that's too bad, I don't have control over the render mode on Bitbucket Server.


Install Django

For front end framwork

https://docs.djangoproject.com/en/4.0/topics/install/

https://docs.djangoproject.com/en/4.0/intro/tutorial01/

New github Repositoyr:
https://github.com/tbaranoski/Trading_Quant_Front_End
Plan is to eventually implement as submodule to this project


investigate creating the incrementer/decrementer DOM element based on the accumulator

Right now it's always a span.  which is fine, but maybe it'd be worth allowing it to inherit, for instance if the accumulator was an H1, then maybe so should the other thing.

'inherit' : false

If it inherits something that isn't inline displayed, that's what would need to be set when inheriting.  But what should it inherit? Only the DOM object type or more?


Export DITA-OT jars as maven arctifacts

Description
Exporting all DITA-OT jars as Maven artifact would help a lot in building Maven projets that are using DITA-OT for building their documentation by removing the dependencies on DITA jars with system. Publishing DITA-OT maven artifacts on Central should be linked to the distribution of a parent pom in the DITA-OT release folder
User's project pom:    pom.xml.txt
DITA-OT parent pom: pom.xml.txt
For this to work, the only additional step needed on the user's computer is to install the DITA-OT pom with:
pushd %DITA_HOME% & mvn clean install & popd
See this project: zosrothko/DITA-OT-SAMPLE as example.

How do you handle the fact that DITA-OT cannot be run from a JAR and requires that the resource files must be on disk?

Since I know nothing about DITA-OT internals, I have no clue to answer your question. But from a pure Java perspective, a command like
java -cp target/classes:dita-ot-3.6/lib/*:dita-ot-3.6/plugins/org.dita.pdf2/lib/*:dita-ot-3.6/plugins/org.dita.pdf2.fop/lib/*:dita-ot-3.6/plugins/org.dita.index/lib/*  com.scort.mi.dita.mojo.Main

can be run from a jar by setting the CLASSPATH environment variable as above or setting the Class-Path attribute of the MANIFEST.MF as above, and set the Main-Class  attribute of the MANIFEST.MF to com.scort.mi.dita.mojo.Main.
May be you could briefly expose why DITA-OT has such strong and unusual constraints, just for understanding?

Various third-partly libraries used by DITA-OT expect e.g. imports to combine resources to be file references or in some cases URIs. Replacing those libraries or coding workarounds takes time.

Ok, got it... a rather long and dispendious time consuming process...

Ok, got it... a rather long and dispendious time consuming process...


Security Alert Another device is attempting to sign in..

I get the warning
"Security Alert
Another device is attempting to sign in using your Plex account. You should only continue if you know this device and intend to grant it access."
The login is from central Europ and I am definitely not located there.
I only get this warning when clicking the link from this Github.
Is there a reason for this?

Yes, the login process is handled by the backend which is located in Europe/US and routed based on load/location. I'm working on the new fronted that will handle login locally but it won't make it more secure, just will get rid of the warning.


[DO NOT MERGE] Add Swagger spec and the examples for ManagedDatabase 2021-11-01-preview

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[x] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
End of February 2022.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
End of February 2022
If updating an existing version, please select the specific langauge SDKs and CLIs that must be refreshed after the swagger is published.

[x] SDK of .NET (need service team to ensure code readiness)
[x] SDK of Python
[x] SDK of Java
[x] SDK of Js
[x] SDK of Go
[x] PowerShell
[x] CLI
[???] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that label "WaitForARMFeedback" will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

please fix the validation errors:
Swagger Avocado Failing after 2m — Swagger Avocado failed
Swagger SpellCheck Failing after 2m — Swagger SpellCheck failed


[Storage] Remove msrest dependency

Now that the generated code does not use msrest the dependency can be removed from all Storage packages.

API change check
API changes are not detected in this pull request.


fix(jobs): darkside proc box not working

Fixes #3087
In fact, the bug likely occurred before #3001 though

Perhaps blame https://github.com/quisquous/cactbot/commit/8fc0673d32fd941aded29470cd781749509cb855


Re-enable dependency on segmented algorithms on CircleCI

Reduced the workload on some tests that were on the verge of timing out, hope that's alright here.

bors merge


Rework sanity ignore list

convert to a single file with links for different ansible version
sort list alphabetically

error is:
Running sanity test "ignores"
--
ERROR: Found 1 ignores issue(s) which need to be resolved:
ERROR: tests/sanity/ignore-2.14.txt:15:1: File 'plugins/modules/cbr_policy.py' does not exist
See documentation for help: https://docs.ansible.com/ansible-core/2.14/dev_guide/testing/sanity/ignores.html


spot check: Gravity Force Lab 2.2.0-rc.3

@emily-phet @terracoda @jessegreenberg @arouinfar @kathy-phet @samreid @chrisklus, gravity-force-lab/2.2.0-rc.3 is ready for RC testing.  This is the third RC test, and so we just need to spot check the issues found in the previous RC.
Document issues in https://github.com/phetsims/gravity-force-lab/issues and link to this issue.
This simulation is outfitted with:

All PhET-iO featured, including a designed api
Sound
Interactive Descriptions (alternative input and auditory descriptions)
Gesture Accessibility with support for mobile iOS

Assigning to @ariel-phet for prioritization.

General RC Test

What to Test

Click every single button.
Test all possible forms of input.

Test all mouse/trackpad inputs.
Test all touchscreen inputs.

If there is sound, make sure it works.
Make sure you can't lose anything.
Play with the sim normally.
Try to break the sim.
Test all query parameters on all platforms. (See QA Book for a list of query parameters.)
Download HTML on Chrome and iOS.
Make sure the iFrame version of the simulation is working as intended on all platforms.
Make sure the XHTML version of the simulation is working as intended on all platforms.
Complete the test matrix.
Don't forget to make sure the sim works with Legends of Learning.
Check this LoL spreadsheet and notify AR or AM if it not there.
If this is rc.1 please do a memory test.
When making an issue, check to see if it was in a previously published version
If there is a console available, check for errors and include them in the Problem Description.
As an RC begins and ends, check the sim repo. If there is a maintenance issue, check it and notify developers if there is a problem.

Issues to Verify

[ ] phetsims/molarity#222
[ ] phetsims/phet-io-wrappers#330
[ ] phetsims/phet-io-wrappers#317
[ ] phetsims/axon#287
[ ] phetsims/studio#102
[ ] https://github.com/phetsims/gravity-force-lab/issues/180
[ ] phetsims/studio#103

These issues should have the "status:ready-for-qa" label. Check these issues off and close them if they are fixed. Otherwise, post a comment in the issue saying that it wasn't fixed and link back to this issue. If the label is "status:ready-for-review" or "status:fixed-pending-deploy" then assign back to the developer when done, even if fixed.

Link(s)

Simulation
iFrame
XHTML
Test Matrix
Legends of Learning Harness

PhET-iO RC Test

What to Test

Make sure that public files do not have password protection. Use a private browser for this.
Make sure that private files do have password protection. Use a private browser for this.
Make sure standalone sim is working properly.
Make sure the wrapper index is working properly.
Make sure each wrapper is working properly.
Launch the simulation in Studio with ?stringTest=xss and make sure the sim doesn't navigate to youtube
For newer PhET-iO wrapper indices, save the "basic example of a functional wrapper" as a .html file and open it. Make sure the simulation loads without crashing or throwing errors.
Load the login wrapper just to make sure it works. Do so by adding this link from the sim deployed root:/wrappers/login/?wrapper=record&validationRule=validateDigits&&numberOfDigits=5&promptText=ENTER_A_5_DIGIT_NUMBER

Further instructions in QA Book
Please at least load all wrappers except studio in the wrapper suite with internet explorer.
make sure to do a complete metacog recording/playback test. Ask @zepumph if you need help. (this doesn't need to be done as much this time around. Make sure to do a test, and any others relating to issues to verify)

Link(s)

Wrapper Index
Test Matrix

Accessibility (a11y) RC Test

What to Test

Specific instructions can be found below.
Make sure the a11y feature doesn't negatively affect the sim in any way.
Test all possible forms of input.

Test all mouse/trackpad inputs.
Test all touchscreen inputs.
Test all keyboard navigation inputs (if applicable).
Test all forms of input with a screen reader (if applicable).

Issues to Verify

[x] Might be an issue with the radio button group

These issues should have the "status:ready-for-qa" label. Check these issues off and close them if they are fixed. Otherwise, post a comment in the issue saying that it wasn't fixed and link back to this issue. If the label is "status:ready-for-review" or "status:fixed-pending-deploy" then assign back to the developer when done, even if fixed.

Link(s)

a11y View
Simulation
Test Matrix

Screen Readers
This sim supports screen readers. If you are unfamiliar with screen readers, please ask Katie to introduce you to screen readers. If you simply need a refresher on screen readers, please consult the QA Book, which should have all of the information you need as well as a link to a screen reader tutorial made by Jesse. Otherwise, look over the a11y view before opening the simulation. Once you've done that, open the simulation and make sure alerts and descriptions work as intended.

Platforms and Screen Readers to Be Tested

[ ] Windows 10 + Latest Chrome + Latest JAWS
[ ] Windows 10 + Latest Firefox + Latest NVDA
[ ] macOS + Safari + VoiceOver
[ ] iOS + Safari + VoiceOver (Please test this! Especially with the Ruler interaction)

Critical Screen Reader Information
We are tracking known screen reader bugs in this Google Document. If you find a screen reader bug, please check it against this list.

Keyboard Navigation
This sim supports keyboard navigation. Please make sure it works as intended on all platforms by itself and with a screen reader.
Final Requests

[x] If this sim is in this list, make an issue in website to ask if PhET research page links need updating. Please assign to @terracoda and @emily-phet.

FAQs for QA Members

There are multiple tests in this issue... Which test should I do first?
Test in order! Test the first thing first, the second thing second, and so on.

How should I format my issue?

We typically assign the developer who opened the issue in the QA repository.

My question isn't in here... What should I do?
You should:

Consult the QA Book.
Google it.
Ask Katie.
Ask a developer.
Google it again.
Cry.

[ ] Win 10 chrome w/ Jaws and Phet-io
[ ] Win 10 firefox w/ NVDA and Phet-io
[ ] Win 10 edge  and Phet-io

[ ] Mac 10.15 chrome  and Phet-io
[ ] Mac 10.15 safari w/ VO  and Phet-io

[ ] Win 10 IE  and Phet-io
[ ] Mac 10.12 safari  and Phet-io

[ ] ChromeOS  and Phet-io
[ ] iPadOS 13 w/ VO  and Phet-io

@KatieWoe this is the current top testing priority (along with any ES-6 supporting testing that might come up)

I made a few updates to the initial comment. Please refresh to get added issues to verify.

Thanks for the speedy turn around. I'll try to get a new RC tonight over in https://github.com/phetsims/gravity-force-lab/issues/262


Input type should allow only Positive numbers and not negative

Is it possible to allow only Positive numbers while entering the cell , - ve sign should not be considered as input eg -1.

You can use a mask to force the user to enter a required format.
https://bossanova.uk/jsuites/javascript-mask


TypeScript 3.8 lib.d.ts updates

[ ] Ensure that PRs are merged in.
[ ] Sync with https://github.com/Microsoft/TSJS-lib-generator/

It's a# project last time I saw, doesn't it? 😅

Presumably, this includes #34065 and #33707.

This is done now.


Browser gets stuck on the starting screen

Hello,
I downloaded the latest Homebrew to my SD card yesterday, and opened it via YouTube app exploit. Then I installed this Homebrew browser and opened it, and it gets stuck on the starting screen, and I can't do anything except turning off my 3DS XL (not new). So I can't even take a look at this browser D:

Ditto. Not working on NN3DS either.

Confirmed; this is a known bug with http:c on new versions of the homebrew exploits. Far as I'm aware, the fix is going to involve rewriting all of the networking code to utilize sockets instead, which'll be fun.

@zeta0134 It seems to have fixed itself on 0.1.1. Found out I was using the 0.1 version. However there are still no apps.
Also, if you're having trouble with hosting apps and games, I can try to set up something for ya if you'd like. I can try to get some kind of hosting.

FTP is actually considerably more complicated to support than straight HTTP. The basic issue is that right now, the homebrew browser is calling http:c, which is an OS service provided by the 3DS. It's pretty basic, only supports HTTP 1.0, and is broken in the current exploits. Sockets are better supported, and would allow the use of any regular protocol without too much fuss; it's what we should have been using all along, but http:c was easy to get working quickly at the time.

@zeta0134 Okay. Question: what would happen if you converted all of the files on the server to raw text and have the 3DS interpret that and make those into correctly named and extension'd files?

That would overcomplicate things. The act of turning network data into saved files isn't the problem, it's the transport layer that gets us that network data in the first place. Once we have the data, spitting it out to the SD card is much easier if it is already in the correct format.
Right now the browser basically works in reverse; the homebrew list itself is a static file, and the browser parses that and turns it into the menu client-side. All the network stack has to be able to do is download files. That's it. Everything else falls into place when that mechanism is working, and that's what's currently broken. :) (This also greatly simplifies the server setup, as literally any static file serve will do.)
The idea is of course to continue using HTTP requests. What needs to change code-wise is how those requests are made; right now we're relying on the OS to make HTTP requests on our behalf, which is broken in currently exploits. Sockets are not broken, so we simply need to make HTTP requests over sockets, and everything will magically work again. All that really takes is time; I had the thing working with sockets originally, and threw it out because the code was awful and poorly written. (raises hand in guilt)  Now that I have a better idea of how I want to structure things, it should be pretty easy to do. ...y'know, next week, when I'm not buried in IRL work.
tl;dr: I know how to fix the issue, it's non-trivial and will take me time, and I am tied up with IRL things until sometime next week. Fixes are incoming, you have my word. Patience please. :)

@zeta0134 Okay ^^ As this is free, developers are allowed to take as much time as they need ^^

@zeto0134 But FTBrony does it fine. With my understanding of what you're saying, the 3DS can't download from a server. But it's well demonstrated that an FTP server works. Just use a computer server to download the actual applications then FTP them over.

@cyandotapps Have you tried updating to fix the problem?

@mattycfp The Nintendo 3DS is actually capable of downloading things from the internet.
And yes, I tried, but this had no effect. Still the same.

I know it is capable :P
Maybe you installed it incorrectly...?
Sent from my iPhone

On Aug 29, 2015, at 3:18 PM, Cyan.Apps<EMAIL_ADDRESS>wrote:
@mattycfp The Nintendo 3DS is actually capable of downloading things from the internet.
And yes, I tried, but this had no effect. Still the same.
—
Reply to this email directly or view it on GitHub.

@mattycfp That's a whole other project on its own, largely orthogonal to the homebrew browser itself. The only thing they'd both have to know is the file layout on the server, which is presently a static file share.
I may put together a PC-side downloader at some point, but as with @zeta0134, real life takes priority, so don't bank on it. It would then be possible to FTP the downloaded files to the 3DS using other apps like FTBrony. It would be about the same amount of effort to do this than to just fix the browser in the first place, so that's what we plan to do.

@zeta0134 After the update by Nintendo to system firmware 10.0.0-27, the Homebrew Browser (the application!) works. However, there will never be a page of downloadable applications showing up.


WIP: Makefile iteration

Can't get the makefile in #14 to work locally. Including some optimizations while debugging.
Tried to minimize docker layers while also making greater use of cache. Fixes the errors I was getting while trying to build with #14. I've got a few ideas to make the build go even faster, but I figure I can do those when there are less urgent things to get on. Works in my local tests!

Since you moved generateversion.py and rundocker.sh to the /src dir now they are being bundled into the apk. We don't need those files.
V/PythonActivity(11678): Extracting private assets.
V/python  (11678): extracting kolibri-0.6.dev020170922004728_git-py2.py3-none-any.whl
V/python  (11678): extracting generateversion.py
V/python  (11678): extracting _load.html
V/python  (11678): extracting main.py
V/python  (11678): extracting logo.png
V/python  (11678): extracting sitecustomize.py
V/python  (11678): extracting rundocker.sh
V/python  (11678): extracting loading-spinner.gif
V/python  (11678): extracting kolibri/
V/python  (11678): extracting kolibri/__main__.py
.
.
.

The docker build stalls on me after # APK Kolibri-<IP_ADDRESS>9271638-debug.apk available in the bin directory

I'm not sure why cleanup is done in this step.
Step 7/9 : RUN useradd -l kivy &&   chown -R kivy:kivy /home/kivy &&  su kivy -c make updatedependencies
 ---> Running in 32c1e982fac6
# Make bin directory, where APK is held. Empty if it exists.
mkdir -p bin
rm -f bin/*.apk 2> /dev/null
rm -rf ./src/kolibri 2> /dev/null
 ---> 676916a9fb41
Removing intermediate container 32c1e982fac6

The docker build stalls on me after # APK Kolibri-<IP_ADDRESS>9271638-debug.apk available in the bin directory

Actually it stalls for maybe 5-10 min and then it continues
 ---> 33bb0de0a1f4
Removing intermediate container f9681bd54a67
Successfully built 33bb0de0a1f4
Successfully tagged kolibrikivy:latest
bash ./src/rundocker.sh
Changing owner from 0:0 to 1000:1000

I wonder if it just timesout?

Since you moved generateversion.py and rundocker.sh to the /src dir now they are being bundled into the apk. We don't need those files.

huh. I didn't realize that the entire src directory was being unpacked into the apk, I thought it was restricted to src/kolibri. Do you know where this is specified?

The docker build stalls on me after # APK Kolibri-<IP_ADDRESS>9271638-debug.apk available in the bin directory
Actually it stalls for maybe 5-10 min and then it continues

For me it stalls for a little while (maybe 30 secs?) then proceeds. Not sure if that's a feature of docker (to have somone do some work on the container image's final container) or a bug :man_shrugging: @aronasorman ? 5-10min  sounds insane.

I'm not sure why cleanup is done in this step.

Again, didn't know all of src was being emptied into the apk. I think it'd be a good idea to enxapsulate it into a kolibri dir
Will patch typo
The I moved things into the src dir to minimize the number of COPYs and therefore layers of the docker image. Might not be work.

huh. I didn't realize that the entire src directory was being unpacked into the apk, I thought it was restricted to src/kolibri.

Well we need the other resources besides the kolibri dir that is extracted from the whl file. Like the images, main.py, etc.

Well regarding I'm not sure why cleanup is done in this step., I do not know why that make recipe is called again if it only called in one place


Deinterlace filter

I see annoying stripes when I transmit interlaced video from my android device to PC trough Scrcpy.
Please, add ability to switch on/off deinterlace filter via hotkey and command line key.
I think it is not difficult due to ffmpeg usage in Scrcpy.

A deinterlace filter operates on a source interlaced video, where all (typically 3, Y U V) planes are interlaced separately. As soon as the video is "captured", the result in a progressive (non-interlaced) video, with the "striped" being encoded as the actual content. It could not be "deinterlaced" correctly after this process. In other words, you must deinterlace before capturing.


Could not find module `Haste'

Just installed hastec, and it fails on any Haste or Fursuit imports.  Upgraded from GHC 7.4.1 to 7.4.2 just in case, same error.  Any ideas?
# cat HelloWorld.hs
import Haste
main = writeLog "Hello, world!"

# hastec -v HelloWorld.hs
Using binary package database: /home/skligys/.haste/haste-pkg/package.cache
wired-in package ghc-prim mapped to ghc-prim-<IP_ADDRESS>-7d3c2c69a5e8257a04b2c679c40e2fa7
wired-in package integer-gmp mapped to integer-gmp-<IP_ADDRESS>-af3a28fdc4138858e0c7c5ecc2a64f43
wired-in package base mapped to base-<IP_ADDRESS>-6e4c9bdc36eeb9121f27ccbbcb62e3f3
wired-in package rts mapped to builtin_rts
wired-in package template-haskell not found.
wired-in package dph-seq not found.
wired-in package dph-par not found.
*** Chasing dependencies:
Chasing modules from: *HelloWorld.hs
hastec: panic! (the 'impossible' happened)
  (GHC version 7.4.2 for x86_64-unknown-linux):
        Could not find module `Haste'
Locations searched:
  Haste.hs
  Haste.lhs

I'm getting this exact error (after successfully running haste-boot --force) - what should I try to fix this?


Fix the --ide-option handling.

Pull Request Description

Important Notes

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[x] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[x] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

weirdly enough, this was last changed by @vitvakatu:
https://github.com/enso-org/enso/blame/0f4f8a0542a246b3d83f461d4ad26f759a24d5ac/app/ide-desktop/lib/client/watch.ts#L113
maybe there's an inconsistency before OSes?

@somebody1234
I think it was always supposed to work with the -- separator between flags to the electron runner and IDE flags, but there was a bug. @vitvakatu solved it by removing the --, but in the meantime, I fixed -- parsing on my branch; hence when it landed, it caused the bug to reappear (this time another way around).
@vitvakatu Do you have any further insight?

@vitvakatu

I believe you're right, @mwu-tow. Do you need QA for this PR?

Yes, it'd be great if anyone other than me confirmed that this works. :)

I'll check it now


Using external sort is broken in 0.6.1

In 0.6.1, when using external data and sorting capabilities, the sort ascending / descending component never renders as the checks rely on comparing that.state.sortSettings.sortDirection to 'asc' or 'desc'.
Problem is, the sortDirection key is internal value, normally set by griddle when NOT using external sorting. There's no way to set it from the outside, hence sorting component never renders (it thinks there's no sorting applied at all).

Yeap i have the same issue here

Sorry your encountering this -- in regard to #449, is this only happening with External sorting?

+1
I did some digging:
Because of the return in here:
if (this.props.useExternal) {
    this.props.externalChangeSort(column, this.props.externalSortColumn === column ? !this.props.externalSortAscending : true);
    return;
}

This other part is never reached:
var sortDirectionCycle = columnMeta.sortDirectionCycle ? columnMeta.sortDirectionCycle : [null, 'asc', 'desc'];
 var sortDirection = null;
 // Find the current position in the cycle (or -1).
var i = sortDirectionCycle.indexOf(this.state.sortDirection && column === this.state.sortColumn ? this.state.sortDirection : null);

// Proceed to the next position in the cycle (or start at the beginning).
i = (i + 1) % sortDirectionCycle.length;

 if (sortDirectionCycle[i]) {
            sortDirection = sortDirectionCycle[i];
 } else {
            sortDirection = null;
}
 var state = {
            page: 0,
            sortColumn: column,
            sortDirection: sortDirection
};

 this.setState(state);

Therefore, sortDirection is always undefined, so this part in gridTitle.jdx fails:
if (that.props.sortSettings.sortColumn == col && that.props.sortSettings.sortDirection === 'asc') {
      columnSort = that.props.sortSettings.sortAscendingClassName;
      sortComponent = that.props.useGriddleIcons && that.props.sortSettings.sortAscendingComponent;
 } else if (that.props.sortSettings.sortColumn == col && that.props.sortSettings.sortDirection === 'desc') {
      columnSort += that.props.sortSettings.sortDescendingClassName;
      sortComponent = that.props.useGriddleIcons && that.props.sortSettings.sortDescendingComponent;
}

Hope this helps

@ryanlanciaux Could you pls fix it?

someone please make a pull request for this fix

+1

+1

I worked around this with a subclass and patch hack. I'm sure there are better ways but might be useful for someone.
import Griddle from 'griddle-react';

export default class Grid extends Griddle {
  oldGetSortObject = this.getSortObject;
  getSortObject = () => {
    const sortObject = this.oldGetSortObject();
    sortObject.sortDirection = sortObject.sortAscending ? 'asc' : 'desc';
    return sortObject;
  };
}

thanks @aclowes , it works.

I got the sort icon working by using #486

Has anyone had more success with 0.7.0?

Confirming - upgrading to 7 fixed this for me

Thanks for checking -- Will close this out for now then :)


Problem using BelongsToOneRelation

Good morning.
I'm trying to query a Model that has defined a BelongsToOneRelation. Though it looks quite simple it is not working at all when I use 'withGraphFetched'
Here is my model:
import { Model } from 'objection';

export class UserAircraft extends Model {
	idusersAircrafts: string;
	fkIdusers: string;
	fkIdaircrafts: string;

	static get tableName() {
		return 'users_aircrafts';
	}

	static get idColumn() {
		return 'idusers_aircrafts';
	}

	static get relationMappings() {
		const Aircraft = require('../aircrafts/aircrafts.model').Aircraft;
		const User = require('./users.model').User;

		return {
			aircraft: {
				relation: Model.BelongsToOneRelation,
				modelClass: Aircraft,
				join: {
					from: 'users_aircrafts.fk_idaircrafts',
					to: 'aircrafts.idaircrafts'
				}
			}
		};
	}
}

When running a query like:
const query = UserAircraftModel.query();
query.select('*');
query.withGraphFetched({ aircraft: true });

I get a null property aircraft in each record, though when using a join query like this:
const query = UserAircraftModel.query();
query.select('*');
query.innerJoin('aircrafts', 'users_aircrafts.fk_idaircrafts', 'aircrafts.idaircrafts');

I do get the data from the aircrafts table. I have output the query that get executed in each sample and those are:
With Graph:
{
  method: 'select',
  options: {},
  timeout: false,
  cancelOnTimeout: false,
  bindings: [ 'e3088190-7d68-4b0f-be99-7caffb90bc43' ],
  __knexQueryUid: 'mYLs8NJHsMrupgSgU3XWI',
  sql: 'select * from "users_aircrafts" where "fk_idusers" = ?'
}

With Join:
{
  method: 'select',
  options: {},
  timeout: false,
  cancelOnTimeout: false,
  bindings: [ 'e3088190-7d68-4b0f-be99-7caffb90bc43' ],
  __knexQueryUid: 'hA3Zi6Cm-NTsW6SoZyjSE',
  sql: 'select * from "users_aircrafts" inner join "aircrafts" on "users_aircrafts"."fk_idaircrafts" = "aircrafts"."idaircrafts" where "fk_idusers" = ?'
}

It is clear that using graph it is not even querying the aircrafts table, can you please let me know what I'm doing wrong?
Thanks in advance
Juan Antonio Santana

You are using knexSnakeCasemappers but still using snake_case in your relation mappings.

That was the issue, thanks!!


[BUG] Inconsistent Result with dropdown filter.

Description
Use example JSON and typed to filter the list. On H, it shows only the first option whereas with Ho it shows the first 2 option.
[   {     "label": "Home Town",     "value": "home_town"   },   {     "label": "Working City",     "value": "working_city"   },   {     "label": "Destination Wedding",     "value": "dest"   } ]
Drive Video
Important Details

Version: Cloud
OS: MacOs 10.14.6
Browser: Firefox 80.0.1 (64-bit).

This is because Fuse.js is configured to use fuzzy matching, with a threshold of 0.6. If we want it only filter exact matches we need to configure the threshold to 0.
What should be the correct behavior?

@daniel-shuy great catch! If it's the default behaviour of the library let's not change it. We were just confused as to why this was happening. Although this wasn't via code, this really was a great contribution. Thank you! 🎊


There is no Function App in the resource list when executing "Deploy to Function App..." command from command palette

OS: All
Azure Resource Extension Build Version: 20220515.1
Azure Function Extension build Version: 20220513.15
Repro Steps:

Create and open a project in VS Code.
Create a Function App.
F1 -> Select "Azure Functions: Deploy to Function App...".
Check whether the Function App shows in the resource list or not.

Expect:
The Function App shows in the resource list.
Actual:
There is no Function App in the resource list.

More Info:
This issue also reproduces for other commands, such as "Create Slot.../Add New Setting.../Browse website" and so on.

Ran into this one myself while testing the student account 😅

@fiveisprime it should be fixed now. If you're on the latest versions of RGs and Functions and it still repros let me know.

This issue still occurs for me, on a fresh install of VSCode (v1.67.2), with fresh installs of extensions on latest versions.


Add HTTP error code, error status to responseStreamChunk Error

When using unitary-to-streaming requests, it is hard to reason about HTTP error codes based on a chunk that contains error: it only contains the text of the error response.
Instead, it would be useful for the error field of a chunk to be structured:

grpc_code containing the code of the RPC error (to avoid the pain of extracting it from the error string)
http_code that is from HTTPStatusFromCode
message containing the current string of the error
http_status containing a textual representation of the grpc_code or http_code.

Yay, thanks @kdima!


potential widget: mathbox.js

[This library] (https://github.com/unconed/MathBox.js/) looks amazing. Creates beautiful 3D mathematical animations and interactives with webgl.
Admittedly, I don't know much about it yet, and there's a lot of moving parts, so translating a simple API to R might be tricky, but it's worth thinking about.

Agree it is very nice;  thanks for adding to this.  Trying to figure out how to prioritize these.  Much might just depend on the ease of conversion and my schedule for the week.


For VDDK help text - replace the example with a format - step 2

Reference: https://issues.redhat.com/browse/MTV-1552
Rephrasing the text messages based on doc and UX review.
Few screenshots

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 36.16%. Comparing base (13484d0) to head (6de5690).
Report is 124 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1390      +/-   ##
==========================================
- Coverage   36.81%   36.16%   -0.65%
==========================================
  Files         158      158
  Lines        2548     2577      +29
  Branches      599      614      +15
==========================================
- Hits          938      932       -6
- Misses       1428     1450      +22
- Partials      182      195      +13

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

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests
JS Bundle Analysis - Avoid shipping oversized bundles


Create User Personas and User Stories

Need to create these so I can try and help get understanding of the softwares features from the end-users view

These should be finished by the end of tomorrow. I have initially delayed doing this because I thought i would have to change my requirements at some point down the line. I'm happy I quickly realised this would be a mistake as i can just go back and change/amend them later depending on if my requirements change or not;

Have created User Stories and User Personas. These are useful as they allow me to put myself in my end users shoes and understand their thoughts/motivations.


[Textfield] when disabled - the text color also applied to adornment text (-webkit-text-fill-color)

Again, not sure whether it's an intension or not, but in v4 you could have a disabled Textfield adorned with an enabled  Button.  In v5 the adornment text get the color of the input text because of the new css property of .MuiInputBase-root.Mui-disabled class:

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

Current Behavior 😯
V5:

Expected Behavior 🤔

V4:

Steps to Reproduce 🕹

https://codesandbox.io/s/material-ui-issue-forked-c9sz7
Context 🔦

Of course I can solve it many other ways, I just wanted to know whether this behaviour is known and intended.
Your Environment 🌎

$ npx @material-ui/envinfo
npx: installed 2 in 4.398s

  System:
    OS: macOS 10.15.7
  Binaries:
    Node: 12.16.3 - ~/.nvm/versions/node/v12.16.3/bin/node
    Yarn: 1.22.4 - /usr/local/bin/yarn
    npm: 6.14.4 - ~/.nvm/versions/node/v12.16.3/bin/npm
  Browsers:
    Chrome: 87.0.4280.88
    Edge: Not Found
    Firefox: 82.0.2
    Safari: 14.0.1
  npmPackages:
    @emotion/react: ^11.1.2 => 11.1.2
    @emotion/styled: ^11.0.0 => 11.0.0
    @material-ui/core: ^5.0.0-alpha.18 => 5.0.0-alpha.18
    @material-ui/icons: ^4.9.1 => 4.9.1
    @material-ui/lab: ^5.0.0-alpha.18 => 5.0.0-alpha.18
    @material-ui/styled-engine:  5.0.0-alpha.18
    @material-ui/styles:  5.0.0-alpha.18
    @material-ui/system:  5.0.0-alpha.18
    @material-ui/types:  5.1.0
    @material-ui/unstyled:  5.0.0-alpha.18
    @material-ui/utils:  5.0.0-alpha.18
    @types/react:  16.9.55
    react: ^16.13.1 => 16.14.0
    react-dom: ^16.13.1 => 16.14.0

@aepp Thanks for the report, the change comes from #23332. What about we scope the color to only the <input> element, avoiding side effects on the adonement?
diff --git a/packages/material-ui/src/InputBase/InputBase.js b/packages/material-ui/src/InputBase/InputBase.js
index ee4055ee84..8807b8874b 100644
--- a/packages/material-ui/src/InputBase/InputBase.js
+++ b/packages/material-ui/src/InputBase/InputBase.js
@@ -50,7 +50,6 @@ export const styles = (theme) => {
       '&$disabled': {
         color: theme.palette.text.disabled,
         cursor: 'default',
-        WebkitTextFillColor: theme.palette.text.disabled, // Fix opacity Safari bug
       },
     },
     /* Styles applied to the root element if the component is a descendant of `FormControl`. */
@@ -128,6 +127,7 @@ export const styles = (theme) => {
       },
       '&$disabled': {
         opacity: 1, // Reset iOS opacity
+        WebkitTextFillColor: theme.palette.text.disabled, // Fix opacity Safari bug
       },
       '&:-webkit-autofill': {
         animationDuration: '5000s',

@oliviertassinari if you ask me - sounds like a valid solution to me. The developer has to configure the look of the adornment. Regardless of the state of the input. At least that's how I've been treating adornments - as separate components which have to be styled separately, always.


Fail any DevWorkspace when webhooks are not present on cluster

Note: I'm not sure this PR should even be merged. The added checks may catch some simple configuration errors (e.g. webhooks accidentally get deleted) but does nothing to address the main reason we do any checking for webhooks (i.e. preventing unauthorized access). I'm opening this PR for discussion in case there's something I'm missing, but it seems kind of pointless IMO.
What does this PR do?

Validate that webhooks exist on cluster when reconciling a DevWorkspace, and fail the workspace if they're not present
Rework validateCreatorID check to check that both mutating and validating webhooks exist and that they're older than the DevWorkspace.
Extract two functions: killWorkspace, which deletes a workspace's deployment and marks it as failed, and failWorkspace which just cuts out the four lines we use to fail workspaces everywhere in the main reconcile loop.

The PR is split into two commits:

The first commit is a half-way solution, doing the detailed check when a workspace has the restricted-access annotation set.
The second commit extends the check above to all DevWorkspaces and does some light refactoring

What issues does this PR fix or reference?
Fixes https://github.com/devfile/devworkspace-operator/issues/305
Is it tested? How?

Create a DevWorkspace on the cluster
Delete the mutating webhook: kc delete mutatingwebhookconfigurations.admissionregistration.k8s.io controller.devfile.io
Edit the DevWorkspace created in step 1
Workspace should be failed, workspace deployment should be deleted.

Closing this PR as it's out of date with the current repo and I don't think there's much value here.


Unexplained errors while replying to a conversation

Version
botbuilder-integration-aiohttp>=4.9.1
Describe the bug
We have deployed a teams bot with python SDK. Intermittently when the users are trying to converse with the bot we get exceptions in the SDK
concurrent.futures._base.CancelledError
if "MemberNotFoundInConversation" in e.args[0]:
IndexError: tuple index out of range
although if the same user retries the same message again , right away everything works fine. Due to this we are unable to scale the bot further.
To Reproduce
Steps to reproduce the behavior:
Intermittent issue, Start the bot and monitor the conversation logs
Expected behavior

Give clear error message explaining where is the issue
Bot should respond to the user

Screenshots
Error type 1
[2020-09-11 04:23:37 +0000] [12] [DEBUG]
[on_turn_error] unhandled error:
DEBUG:aiohttp.web:
[on_turn_error] unhandled error:
Traceback (most recent call last):
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/bot_adapter.py", line 128, in run_pipeline
context, callback
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/middleware_set.py", line 69, in receive_activity_with_status
return await self.receive_activity_internal(context, callback)
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/middleware_set.py", line 79, in receive_activity_internal
return await callback(context)
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/activity_handler.py", line 68, in on_turn
await self.on_message_activity(turn_context)
File "/teams-connector/bot/teams_bot.py", line 112, in on_message_activity
await turn_context.send_activity(row['text'])
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/turn_context.py", line 170, in send_activity
result = await self.send_activities([activity_or_text])
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/turn_context.py", line 217, in send_activities
return await self._emit(self._on_send_activities, output, logic())
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/turn_context.py", line 295, in _emit
return await logic
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/turn_context.py", line 212, in logic
responses = await self.adapter.send_activities(self, output)
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/bot_framework_adapter.py", line 728, in send_activities
raise error
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/bot_framework_adapter.py", line 714, in send_activities
activity.conversation.id, activity.reply_to_id, activity
File "/usr/local/lib/python3.7/site-packages/botframework/connector/aio/operations_async/_conversations_operations_async.py", line 529, in reply_to_activity
request, stream=False, **operation_config
File "/usr/local/lib/python3.7/site-packages/msrest/async_client.py", line 115, in async_send
pipeline_response = await self.config.pipeline.run(request, **kwargs)
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_abc.py", line 159, in run
return await first_node.send(pipeline_request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_abc.py", line 79, in send
response = await self.next.send(request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_requests.py", line 106, in send
return await self.next.send(request, **kwargs)
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_abc.py", line 79, in send
response = await self.next.send(request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_abc.py", line 79, in send
response = await self.next.send(request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_requests.py", line 85, in send
await self.driver.send(request.http_request, **kwargs)
File "/usr/local/lib/python3.7/site-packages/msrest/universal_http/async_requests.py", line 91, in send
return await super(AsyncRequestsHTTPSender, self).send(request, **requests_kwargs)
File "/usr/local/lib/python3.7/site-packages/msrest/universal_http/async_requests.py", line 79, in send
await future
concurrent.futures._base.CancelledError
Error Type 2
[on_turn_error] unhandled error: tuple index out of range
DEBUG:aiohttp.web:
[on_turn_error] unhandled error: tuple index out of range
Traceback (most recent call last):
File "/teams-connector/bot/teams_bot.py", line 162, in get_member_details
turn_context, turn_context.activity.from_property.id
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/teams/teams_info.py", line 175, in get_member
connector_client, conversation_id, member_id
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/teams/teams_info.py", line 264, in _get_member
conversation_id, member_id
File "/usr/local/lib/python3.7/site-packages/botframework/connector/aio/operations_async/_conversations_operations_async.py", line 731, in get_conversation_member
request, stream=False, **operation_config
File "/usr/local/lib/python3.7/site-packages/msrest/async_client.py", line 115, in async_send
pipeline_response = await self.config.pipeline.run(request, **kwargs)
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_abc.py", line 159, in run
return await first_node.send(pipeline_request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_abc.py", line 79, in send
response = await self.next.send(request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_requests.py", line 106, in send
return await self.next.send(request, **kwargs)
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_abc.py", line 79, in send
response = await self.next.send(request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_abc.py", line 79, in send
response = await self.next.send(request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_requests.py", line 85, in send
await self.driver.send(request.http_request, **kwargs)
File "/usr/local/lib/python3.7/site-packages/msrest/universal_http/async_requests.py", line 91, in send
return await super(AsyncRequestsHTTPSender, self).send(request, **requests_kwargs)
File "/usr/local/lib/python3.7/site-packages/msrest/universal_http/async_requests.py", line 79, in send
await future
concurrent.futures._base.CancelledError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/bot_adapter.py", line 128, in run_pipeline
context, callback
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/middleware_set.py", line 69, in receive_activity_with_status
return await self.receive_activity_internal(context, callback)
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/middleware_set.py", line 79, in receive_activity_internal
return await callback(context)
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/activity_handler.py", line 68, in on_turn
await self.on_message_activity(turn_context)
File "/teams-connector/bot/teams_bot.py", line 75, in on_message_activity
email_id, name, tenant_id = await self.get_member_details(turn_context)
File "/teams-connector/bot/teams_bot.py", line 165, in get_member_details
if "MemberNotFoundInConversation" in e.args[0]:
IndexError: tuple index out of range
`
Additional context
we used below starter code, and more or less have the same structure
https://github.com/microsoft/BotBuilder-Samples/tree/main/samples/python/57.teams-conversation-bot

Thank you for the report, @saikiran2603
I would like to help but I have a few questions:

Are you saying you have only seen this error in logs and have not reproduced the error yourself?
Are the errors showing up in app insights or somewhere else?
Shall we assume your bot is only accessed from Teams and not Web Chat, Emulator, etc.?

@v-kydela thanks !!

Yes the error is not reproducible at will. If i test it some times i do get this error , but i cannot pinpoint any reason / event which causes this error, it appears to be on random.
I can see these errors in my docker container logs. Also once i get this error my application is designed to respond to the user saying , the bot faced some issues etc , like shown in 57 Teams conversation bot example. This message does get sent to the user whenever i see this error in the logs
Yes its a Teams bot only. and we are not using emulator yet.

I have raised a similar request in botbuilder sdk repo as well, since i didnt know which is the right repo to raise this issue , https://github.com/microsoft/botbuilder-python/issues/1384

@saikiran2603 - I will go ahead and address the issue here for now. I can see that this is an intermittent issue, but it may still be necessary to come up with some semblance of a way to reproduce the issue so we have a way to tell if it's "fixed." For example, if you talked to the bot for 10 minutes without getting an error would that be an indication that the issue is resolved? Or if you sent 100 messages? Or if you opened 10 threads with the bot? Or if you signed out and back in 10 times?
Once you've gained some confidence around being able to tell if the issue is present or not, you can see if the issue occurs in other configurations and that will help us pinpoint the cause of the issue. For example:

Does the issue occur if you deploy without a Docker container?
Does the issue occur if you run the bot locally?
Does the issue occur if you use the sample without your modifications?

@v-kydela Sure that sounds good !
to answer your questions ,

I havent tested it but i suspect it does. I will test this today and get back,
Yes
I need to test this. I will report back on this.

@saikiran2603 - Have you tested the sample without your modifications?

@saikiran2603 - Are you still working on this?

@v-kydela yes i tested it without any modifications and without docker, but i am still unable to reproduce the issue at will .. it does not occur always in both the situations

These are the logs when i was able to re produce the issue
Case 1
[2020-09-24 05:31:05 +0000] [13] [DEBUG]
[on_turn_error] unhandled error: tuple index out of range
DEBUG:aiohttp.web:
[on_turn_error] unhandled error: tuple index out of range
Traceback (most recent call last):
File "/teams-connector/bot/teams_bot.py", line 162, in get_member_details
turn_context, turn_context.activity.from_property.id
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/teams/teams_info.py", line 175, in get_member
connector_client, conversation_id, member_id
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/teams/teams_info.py", line 264, in _get_member
conversation_id, member_id
File "/usr/local/lib/python3.7/site-packages/botframework/connector/aio/operations_async/_conversations_operations_async.py", line 731, in get_conversation_member
request, stream=False, **operation_config
File "/usr/local/lib/python3.7/site-packages/msrest/async_client.py", line 115, in async_send
pipeline_response = await self.config.pipeline.run(request, **kwargs)
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_abc.py", line 159, in run
return await first_node.send(pipeline_request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_abc.py", line 79, in send
response = await self.next.send(request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_requests.py", line 106, in send
return await self.next.send(request, **kwargs)
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_abc.py", line 79, in send
response = await self.next.send(request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_abc.py", line 79, in send
response = await self.next.send(request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_requests.py", line 85, in send
await self.driver.send(request.http_request, **kwargs)
File "/usr/local/lib/python3.7/site-packages/msrest/universal_http/async_requests.py", line 91, in send
return await super(AsyncRequestsHTTPSender, self).send(request, **requests_kwargs)
File "/usr/local/lib/python3.7/site-packages/msrest/universal_http/async_requests.py", line 79, in send
await future
concurrent.futures._base.CancelledError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/bot_adapter.py", line 128, in run_pipeline
context, callback
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/middleware_set.py", line 69, in receive_activity_with_status
return await self.receive_activity_internal(context, callback)
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/middleware_set.py", line 79, in receive_activity_internal
return await callback(context)
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/activity_handler.py", line 68, in on_turn
await self.on_message_activity(turn_context)
File "/teams-connector/bot/teams_bot.py", line 75, in on_message_activity
email_id, name, tenant_id = await self.get_member_details(turn_context)
File "/teams-connector/bot/teams_bot.py", line 165, in get_member_details
if "MemberNotFoundInConversation" in e.args[0]:
IndexError: tuple index out of range
[2020-09-24 05:31:05 +0000] [13] [DEBUG] None
DEBUG:aiohttp.web:None
Case 2
[on_turn_error] unhandled error:
DEBUG:aiohttp.web:
[on_turn_error] unhandled error:
Traceback (most recent call last):
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/bot_adapter.py", line 128, in run_pipeline
context, callback
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/middleware_set.py", line 69, in receive_activity_with_status
return await self.receive_activity_internal(context, callback)
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/middleware_set.py", line 79, in receive_activity_internal
return await callback(context)
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/activity_handler.py", line 68, in on_turn
await self.on_message_activity(turn_context)
File "/teams-connector/bot/teams_bot.py", line 112, in on_message_activity
await turn_context.send_activity(row['text'])
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/turn_context.py", line 170, in send_activity
result = await self.send_activities([activity_or_text])
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/turn_context.py", line 217, in send_activities
return await self._emit(self._on_send_activities, output, logic())
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/turn_context.py", line 295, in _emit
return await logic
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/turn_context.py", line 212, in logic
responses = await self.adapter.send_activities(self, output)
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/bot_framework_adapter.py", line 728, in send_activities
raise error
File "/usr/local/lib/python3.7/site-packages/botbuilder/core/bot_framework_adapter.py", line 714, in send_activities
activity.conversation.id, activity.reply_to_id, activity
File "/usr/local/lib/python3.7/site-packages/botframework/connector/aio/operations_async/_conversations_operations_async.py", line 529, in reply_to_activity
request, stream=False, **operation_config
File "/usr/local/lib/python3.7/site-packages/msrest/async_client.py", line 115, in async_send
pipeline_response = await self.config.pipeline.run(request, **kwargs)
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_abc.py", line 159, in run
return await first_node.send(pipeline_request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_abc.py", line 79, in send
response = await self.next.send(request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_requests.py", line 106, in send
return await self.next.send(request, **kwargs)
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_abc.py", line 79, in send
response = await self.next.send(request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_abc.py", line 79, in send
response = await self.next.send(request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/site-packages/msrest/pipeline/async_requests.py", line 85, in send
await self.driver.send(request.http_request, **kwargs)
File "/usr/local/lib/python3.7/site-packages/msrest/universal_http/async_requests.py", line 91, in send
return await super(AsyncRequestsHTTPSender, self).send(request, **requests_kwargs)
File "/usr/local/lib/python3.7/site-packages/msrest/universal_http/async_requests.py", line 79, in send
await future
concurrent.futures._base.CancelledError
[2020-09-23 04:02:41 +0000] [13] [DEBUG] None
DEBUG:aiohttp.web:None
Although i still cannot figure out when this issue occours and at what configuration, it still appears on random and the error message does not help me figure out what might be the issue either :(

@saikiran2603 - Yes, I already understood that you cannot reproduce the issue at will. I thought we had agreed that we needed a way for you to determine if the issue is present or not, though. What method had you been using before to see if the issue occurs while testing locally, etc.? Are you sending a certain amount of messages, for example?

I thought we had agreed that we needed a way for you to determine if the issue is present or not, though.

Yes i still am on the same page.

What method had you been using before to see if the issue occurs while testing locally, etc.? Are you sending a certain amount of messages, for example?

I would just start conversing with the bot , and sometimes i would see the error after hours of testing or sometimes much quicker. I could not figure out a pattern there. And i cannot replicate the issue every time.

@saikiran2603 - If the error occurs in your code but not the sample then that would narrow down the cause of the issue to just the changes you've made. You may email your code to me if you'd like to share it privately. I can see in your stack trace references to a file called "/teams-connector/bot/teams_bot.py" which I'm guessing must be in your project.

@saikiran2603 - Are you still working on this?

Yes , i am mailing the code to you ..

@saikiran2603 - I haven't received an email yet. What address did you mail it to?

@saikiran2603 - Do you still need help?

I am closing this issue due to inactivity. We are still happy to provide help if you need any, though it may be a good idea to make a new post if that's the case. Please have a look at this to find out the best places to ask for help, and consider asking questions on Stack Overflow.

Having the same issue.


remove req/rec/opt distinctions in DH sections?

@mslarae13 please check the interface at https://data.dev.microbiomedata.org/submission/samples
I think you will find that some of the MIxS as-is Required or Recommended columns are colored white, not yellow or magenta, because MIxS doesn't consider them required or recommended for soil samples.
We could either

leave the section names as they are which will be informative but not enforced
move those columns to MIxS modified and assert that they are required or recommended

I don't think there is any action to do here. If there are slots whose required or recommended status need to be updated, please open an issue in the submission-schema repo.


CSL Styles having underlined entities should be handled

JabRef version
5.15 (latest release)
Operating system
Windows
Details on version and operating system
No response
Checked with the latest development build (copy version output from About dialog)

[X] I made a backup of my libraries before testing the latest development version.
[X] I have tested the latest development version and the problem persists

Steps to reproduce the behaviour
Note: burning issue

Open a library in JabRef
Connect to an OO document
Select the CSL Style "Modern Language Association 7th edition (underline)" from the Select Style option
Cite any entry.
You would notice that the journal name is not underlined, although the style specifies it:

Appendix
...

Log File
Paste an excerpt of your log file here

Fixed in https://github.com/JabRef/jabref/pull/11577/commits/d0a329ff8f9ba5577503c6ec77f6231c80a1098d


preview

firstly, i'm really glad you forked this! kudos
however, i noticed a bug with the preview window
Error executing vim.schedule lua callback: .../lazy/symbols-outline.nvim/lua/symbols-outline/hover.lua:43: attempt to concatenate field 'bg_hl' (a nil value)
stack traceback:
        .../lazy/symbols-outline.nvim/lua/symbols-outline/hover.lua:43: in function 'handler'
        ...r/neovim/HEAD-ae8ca79/share/nvim/runtime/lua/vim/lsp.lua:1516: in function 'fn'
        vim/_editor.lua:343: in function <vim/_editor.lua:342>

I notice the same

amazing, thank you


docs: Add info about adding global script tags

Per this slack discussion, wanted to add some docs to our FAQ about adding global script tags.

Perfect, thanks!! 🙌


SliverAppBar backgroundColor issue : a bottom line is visible

Steps to Reproduce

Execute flutter run on the code sample

Expected results:
No visible red line.
Actual results:
A visible red line when toolbarHeight is different than kToolbarHeight.
My app is under flutter 3.3.4 but this example is under 3.7.5 and issue still reproductible.
Issue visible on emulator and real device (Pixel 7 for example).

Code sample

import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(primarySwatch: Colors.blue),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  final SliverOverlapAbsorberHandle appbar = SliverOverlapAbsorberHandle();

  @override
  void dispose() {
    appbar.dispose();
    super.dispose();
  }

  double get appbarHeight => kToolbarHeight + 10;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: CustomScrollView(
        slivers: <Widget>[
          SliverAppBar(
            titleSpacing: 0,
            elevation: 0,
            pinned: true,
            backgroundColor: Colors.red,
            toolbarHeight: appbarHeight,
            title: SizedBox(
              height: appbarHeight,
              child: Container(
                color: Colors.white,
              ),
            ),
          ),
          const SliverToBoxAdapter(
            child: Center(
              child: Text("Can you see a little red line above ? "),
            ),
          ),
        ],
      ),
    );
  }
}

Logs

No logs

No issue(s)

[✓] Flutter (Channel stable, 3.7.5, on macOS 13.1 22C65 darwin-arm64, locale fr-FR)
    • Flutter version 3.7.5 on channel stable at /Users/earminjon/fvm/versions/3.7.5
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision c07f788888 (6 days ago), 2023-02-22 17:52:33 -0600
    • Engine revision 0f359063c4
    • Dart version 2.19.2
    • DevTools version 2.20.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at /Users/earminjon/Library/Android/Sdk
    • Platform android-33, build-tools 33.0.2
    • ANDROID_HOME = /Users/earminjon/Library/Android/Sdk
    • Java binary at: /Users/earminjon/Library/Java/JavaVirtualMachines/corretto-11.0.17/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment Corretto-<IP_ADDRESS>.1 (build 11.0.17+8-LTS)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14C18
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (version 2022.1)
    • Android Studio at /Users/earminjon/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/221.6008.13.2211.9514443/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] IntelliJ IDEA Ultimate Edition (version 2022.3.2)
    • IntelliJ at /Users/earminjon/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 72.1.4
    • Dart plugin version 223.8617.8

[✓] IntelliJ IDEA Ultimate Edition (version 2022.3.2)
    • IntelliJ at /Users/earminjon/Library/Application Support/JetBrains/Toolbox/apps/IDEA-U/ch-0/223.8617.56/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] Connected device (3 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554 • android-arm64  • Android 13 (API 33) (emulator)
    • macOS (desktop)             • macos         • darwin-arm64   • macOS 13.1 22C65 darwin-arm64
    • Chrome (web)                • chrome        • web-javascript • Google Chrome 110.0.5481.177

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Red line visible :

Red line not visible :

@EArminjon Can you check this comment if it works for you?

@EArminjon It's closed because it's marked as a duplicate (please see https://github.com/flutter/flutter/issues/112629#issuecomment-1267657830). I'm not quite sure but you may check that linked issue and see if it's similar to this case.
If it is the case, please follow up on it for further updates. You can leave there your comment and react an upvote to the original comment will help to raise the issue.