Update earlier-function-dax.md

Docs Build status updates of commit 7453c4b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

query-languages/dax/earlier-function-dax.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@bricker - Thanks for catching this typo!


Other kinds of for loops

Two common patterns of for loops I've encountered in PolyBench:

Loops instantiated with other loop variables (same as #72).

for(let j = k + 1; j < PB_J; j++) // k is another loop bounds and PB_J is compile time constant

Loops range ends with other loop variables

for(let j = 0; j < k; j++) // k is another loop bounds

Non-1 steps for for loops

for(j = 0; j < PB_J; j += 2)

There might be interesting type system things we can do for these loops. Every other for loop should just be turned into a while loop.
For (3), we can do something of the form where only some of the banks are consumed since not all of them are touched.
Don't know what to do for (1) and (2).

Would it work to implement (1) and (2) by using prefix or suffix views and then iterating over those? We would, of course, need to figure out what it means to take a view using an index type…

Well normally, the other loops wouldn't be unrolled so the iterator would be treated as "just" a dynamic number for slicing purposes.
I still don't know of a way to unroll the outer loop without doing the equivalent of the suggestion in #72...
The Jason Cong paper seems to recommend having the inner loop unrolled with a static bound while the outer (controller) loops has a dynamic bound.

I don't think having a prefix view for (2) helps (unless I missed something). What we need is loops with dynamic bounds:
for (let i = 0..j)

Right now, the compiler restricts j to be a syntactic number. The obvious implementation is allowing dynamic bounds and disallowing unrolling of these loops.
As a bonus, we can transitively calculate the TRIPCOUNTs for each of the variables if j or one of its initializers is a static integers. It seems like having the bounds check pass statically calculate the bounds for each expression is the right thing to do in the limit.

I suppose I was implicitly suggesting that loops might be written to use the size of an array for their bounds—sort of like a "foreach" loop. Then, you would do this by first taking a view on the array and then iterating over the entire array. This proposal doesn't really solve the problem; it just moves it into the view creation instead of the iteration…

Yeah, I don't know if that fixes it. See the example in #193 which tries to use both the underlying array and the view together.


Integrate default search params/queries

Things like limit need to be pre-set so we don't get a dump of ridiculous proportions.
Others might be nice as well.

Done!


/send/m.room.message times out in some encrypted rooms

Describe the bug
Pantalaimon is timing out when sending messages in specific encrypted rooms.
To Reproduce
No idea
Expected behavior
Pantalaimon does it's job
Screenshots
Error from matrix-bot-sdk
MatrixLiteClient (REQ-38150) Error: ESOCKETTIMEDOUT
    at ClientRequest.<anonymous> (/home/user/user/node_modules/request/request.js:816:19)
    at Object.onceWrapper (events.js:420:28)
    at ClientRequest.emit (events.js:314:20)
    at Socket.emitRequestTimeout (_http_client.js:769:9)
    at Object.onceWrapper (events.js:420:28)
    at Socket.emit (events.js:326:22)
    at Socket._onTimeout (net.js:482:8)
    at listOnTimeout (internal/timers.js:554:17)
    at processTimers (internal/timers.js:497:7) {
  code: 'ESOCKETTIMEDOUT',
  connect: false
}

Desktop (please complete the following information):

OS: Rasbian
Browser: matrix-bot-sdk
Version: 0.8.0

Additional context
Only happening for one room, 301 users.
[Bot]
# omitted
SSL = True
Notifications = False
ListenAddress = localhost
ListenPort = 8080
IgnoreVerification = True
# had to disable because otherwise pantalaimon crashes when it tries to get stdin input as a system service
UseKeyring = False

no reply in three days

this is why people don't use matrix

Sharing E2EE keys with 300 users which might have multiple devices will take some time, your bot seems to give up before the keys are shared.

Encryption in that specific room worked fine for weeks, I doubt it's that. I suspect the bots session is being blocked by a user but I have no way of checking since it's a headless setup. Installing the dbus library required for panctl prevents pantalaimon from starting

E2EE keys may expire and will need to be re-shared, people delete/add device which expires E2EE keys as well, and the number of devices might have grown which might explain why it needs longer to share the keys. It might be something else, but the first thing to try will be to raise the timeout for your bot.

I can't do that. matrix-bot-js provides no way to increase the ttl for sending events

Can you perhaps confirm that this indeed is the issue using curl to send a message?

There isn't going to be any difference between matrix-bot-js sending a message to the room (MatrixClient.ts#987) and curl sending a message to the room. The issue isn't just the /send/${type}/${track} timing out either, this bug freezes up pantalaimon entirely for a few seconds

I encountered the same issue, briefly, in a room with exactly 2 users, me and my bot.

Yesterday, my bot worked nicely.
Today, I connected with an element-web built and deployed by me. My bot started ESOCKETTIMEDOUT whenever I attemped to send a message.
I closed the tab with this unusual element-web then reopened it.
The bot resumed working correctly.


Add ListOptions to GetAdvancedSecurityActiveCommittersOrg

Adding Pagination support for fetching active committers in an organization.

It looks like you are making a change that affects only Enterprise GitHub, correct?
According to the current documentation URL: https://docs.github.com/en/rest/billing?apiVersion=2022-11-28#get-github-advanced-security-active-committers-for-an-organization
there is no pagination supported by the current endpoint.
Is it different for GitHub Enterprise?
If so, please provide two docs URLs as seen in other methods in this repo.

I think the API endpoint is supported only for enterprise cloud: https://docs.github.com/en/enterprise-cloud@latest/rest/billing?apiVersion=2022-11-28#get-github-advanced-security-active-committers-for-an-organization

Oh, I see... the original URL was wrong then.  Thanks.

@gmlewis can we merge the PR? we got 2 approvals now.


Implement BaseTyped for f64

Short info header:

GFX version: 0.17.1
OS: Windows
GPU: GTX Geforce 1060

Issue
Is there a reason why BaseTyped isn't implemented for f64?
From my limited understanding of the code, adding it to here should do the trick?
There even is a spot for it in the enum...

No reason, PRs are welcome ;)

Will look into it


Create persistent notification if config validation errors occur.

During startup of Home Assistant, if we experience a configuration validation error, capture the message and add it as a persistent notification. That way the user will be notified in the UI that not all components initialized.

Story: https://www.pivotaltracker.com/story/show/116917427

+1

Do we want to group these errors in a way?

add notification tile per platform (with detail of failure)
add failure tile with a list only of failed platforms, components & validation errors


Iddle pause dialog still appearing when in Fullscreen.

Just installed the extension and it won't un-pause the video if it is in Full-screen, making me have to manually click the prompt.
Currently using version: 0.9.2, on Windows Firefox 119.0.1 (64-bit)

This extension stopped working properly a long time ago, use this YT Nonstop extension.

Or even better https://addons.mozilla.org/nl/firefox/addon/yt-nonstop/


timescaledb-tune is looking for wrong postgresql conf file version

I followed the instructions for setting up TimescaleDB on Ubuntu here and installed the package sudo apt install timescaledb-2-postgresql-13.
When I try to run timescaledb-tune, it's searching for postgres 14 config files. Here's the output:
exit: could not find postgresql.conf at any of these locations:
/etc/postgresql/14/main/postgresql.conf
/var/lib/pgsql/14/data/postgresql.conf
/var/lib/postgres/data/postgresql.conf
/var/lib/postgresql/data/postgresql.conf
My config file is located at /etc/postgresql/13/main/postgresql.conf.
I assume the package timescaledb-2-postgresql-13 should be looking for postgresql-13 config files, not postgresql-14? Following the instructions on that page verbatim installs postgres 13.
Running this command appears to work properly:
sudo timescaledb-tune --conf-path=/etc/postgresql/13/main/postgresql.conf

Do you have postgres 14 installed too, it consults pg_config for the version to look for?

Same error with version 14
timescaledb-tune
exit: could not find postgresql.conf at any of these locations:
/etc/postgresql/15/main/postgresql.conf
/var/lib/pgsql/15/data/postgresql.conf
/var/lib/postgres/data/postgresql.conf
/var/lib/postgresql/data/postgresql.conf
timescaledb-tune is looking for version 15 and is not installed in the server

I found the error, please verify the version of psql client, it should be the same version of database.

I found the error, please verify the version of psql client, it should be the same version of database.

To add to the answer from @agrega-dev, if installing postgresql-client via apt-get, make sure the version to be installed matches the installed postgres version. Also, run dpkg -l to verify that you only have one installed version, otherwise you will run into errors with timescale-tune unless you specifically pass it a config file.


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v5.6 to before v5.10.88",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=61fad6816fc10fb8793a925d5c1256d1c3db0cd2",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=7da349f07e457cad135df0920a3f670e423fb5e9"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "61fad6816fc10fb8793a925d5c1256d1c3db0cd2",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "7da349f07e457cad135df0920a3f670e423fb5e9",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "net/packet: rx_owner_map depends on pg_vec\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.10.88 by commit 7da349f07e457cad135df0920a3f670e423fb5e9, it was introduced in version v5.6 by commit 61fad6816fc10fb8793a925d5c1256d1c3db0cd2. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2021-1002707


Operation timed out with STM32L432KCUx

Describe the bug
I am trying to get "blinky" of https://github.com/embassy-rs/embassy/tree/master/examples/stm32l4 running with an STM32L432KCUx developer board.
To Reproduce
Steps to reproduce the behavior:

git clone https://github.com/embassy-rs/embassy
cd embassy/examples/stm32l4
cargo run --release --bin blinky

Expected behavior
Should work.
Stacktrace
    Finished release [optimized] target(s) in 0.06s
     Running `probe-run --chip STM32L432KCUx target/thumbv7em-none-eabi/release/blinky`
(HOST) WARN  insufficient DWARF info; compile your program with `debug = 2` to enable location info
(HOST) INFO  flashing program (8.50 KiB)
(HOST) INFO  success!
Error: An error with the usage of the probe occured

Caused by:
    Operation timed out

Desktop (please complete the following information):

Linux, Fedora 34

Additional context
Add any other context about the problem here.

probe-run is stuck on a quite old probe-rs version. Can you try again with cargo install --git https://github.com/Dirbaio/probe-run?
If it still happens, can you paste the full trace logs? cargo run --release --bin blinky -- -vv

Awesome! Yes, that fixed the issue!
I guess it may be beneficial to release a newer version of probe-run then.

That's not their mistake tho, we didn't release in a long time, which makes it impossible for them to update :)
A release on our side is iminent tho.


Fix snapshot test.

Fix a snapshot test in tessellate-bundler.

Changes Unknown when pulling 5090228c3f66c620a31f4da4e69a489daa9f6088 on fix-some-tests into ** on master**.

Overriding pull request review because tests are broken on master.


Increase version of  "guzzlehttp/guzzle": "^7.5"

I am on Drupal 10.
This is the minimum requirement: "guzzlehttp/guzzle": "^7.5"
Can the SDK work with it?

Ahhh you are way ahead of me.
I see the .0.5.1 release has the guzzle and php versions needed.
https://github.com/jonathantorres/medium-sdk-php/blob/v0.5.1/composer.json


Auto-generate markdown-format report each run

As in KBP code... or something similar to this at least!
https://github.com/HazyResearch/KBP/tree/master/experiment-reports

Good idea!


Callbacks for drawer open and close

We can call these hooks from https://github.com/react-navigation/react-navigation-drawer/blob/0be5b174a39f5a627e546a61067373e6eeeec4a7/src/views/DrawerView.js#L124-L125

@brentvatne Are there plans to implement this? I could really use this at the moment, or alternatively, unmount the drawer when it is closed so its state is reset when opening.

@brentvatne I've needed this in the past and looking for some Hacktoberfest points :) I see there's also #11, I'm assuming implementing this would solve that.
What do you think about this approach?
const MyApp = createDrawerNavigator({
  Home: {
    screen: MyHomeScreen,
  },
  Notifications: {
    screen: MyNotificationsScreen,
  },
}, {
  onDrawerToggled: isOpen => {
    // do something
  }
});

In my use case I wanted to know of both events, so this could allow that. But if you only care about the drawer closing this could be excessive. It could easily be onDrawerOpened and onDrawerClosed. Thoughts? Thanks!

hi @bradbumbalough! sorry for the slow reply. I think the hard thing for me here is I'm not sure exactly what use cases people have in mind. we should enumerate some use cases and then consider how a solution like this will work for those

I stumbled across this issue by way of a Stack Overflow post. What I am looking to do is close the keyboard when the drawer is opened via gesture. I've done this for now by using onNavigationStateChange and checking that action.type === DrawerActions.DRAWER_OPENED. But this is a bit clumsy, so a better API would be much appreciated!

I have a use case that i want to do, whenever i open the drawer and when its finished opening i want an http request to fire to populate the drawer. Is there any callback for when the drawer finishes opening? If anyone knows any method please do share

How do you hide the keyboard when the drawer opens? Seems like that keyboardDismissMode does not have that option (or I didn't find it)

I'm also needing something like this. At the moment I'm using this setup:
App.js:
import Router, { AppNavigator } from './src/routes';

handleDrawerStateChange = () => {
  // src: https://stackoverflow.com/a/52317406/827129
  const defaultGetStateForAction = AppNavigator.router.getStateForAction;

  AppNavigator.router.getStateForAction = (action, state) => {
    switch (action.type) {
      case 'Navigation/DRAWER_OPENED':
        this.setState({ hamburgerActive: true });
        break;

      case 'Navigation/DRAWER_CLOSED':
        this.setState({ hamburgerActive: false });
        break;
      }

    return defaultGetStateForAction(action, state);
  };
}

src/routes.js:
import { createDrawerNavigator, createAppContainer } from "react-navigation";
import MainNav from './components/navigation/MainNav/MainNav';
import DailyDigest from './components/navigation/screens/DailyDigest';
import Settings from './components/navigation/screens/Settings';

export const AppNavigator = createDrawerNavigator(
  {
    DailyDigest: DailyDigest,
    Settings: Settings,
  },
  {
    initialRouteName: 'DailyDigest',
    contentComponent: MainNav,
    drawerWidth: 300,
    drawerPosition: 'right',
  }
);

const AppContainer = createAppContainer(AppNavigator);

export default AppContainer;

My use case is that I have a totally separate hamburger menu (see below). This code lives in App.js, at the same level as my AppContainer.

An issue I'm facing is that the DRAWER_OPENED and DRAWER_CLOSED methods seem to lag quite a bit (you'll notice when I open the side menu, the hamburger switches from a hamburger to a cross immediately (as I hooked up this myself in App.js state). But when closing the drawer (which was done by tapping the grey backdrop on the left) it takes about a second to receive the method fired off and then change the hamburger state from a cross back to a hamburger.
Mainly posting here as my +1 to this being properly supported/documented (I stumbled upon this from this SO post) but also noting the lag here.

Based on @fredrivett (thank you man) solution I find a more fast response solution (the other solution delays secons in some uses).
const defaultGetStateForAction = StackLogadoDrawer.router.getStateForAction;

StackLogadoDrawer.router.getStateForAction = (action, state) => {
  switch (action.type) {
    case 'Navigation/MARK_DRAWER_SETTLING':
      if (action.willShow == false) {
        console.log('CERRADO');
      } else if (action.willShow == true) {
        console.log('ABIERTO');
      }
      break;
    }

  return defaultGetStateForAction(action, state);
};

This is fired just immediately actions occurs (or very near).
It works on gestures or calling openDrawer()
Anyway I think this is a "must have" easy and direct way in the API

This is quite an issue and it hasn't been resolved yet...

My use case for this was because I had two views within the drawer.
If you clicked a nav item it would switch to a sub nav view within the drawer and re-opening the nav would be in that same sub nav state. I needed it to revert back to original state when closed so I manually reset state when close button was clicked but the original state would appear during the close transition which was jarring.
Ideally the component would unmount when closed.

if someone wants to open a PR to add callbacks for open and close please open a pr

case 'Navigation/MARK_DRAWER_SETTLING':
if (action.willShow == false) {
console.log('CERRADO');
} else if (action.willShow == true) {
console.log('ABIERTO');
}
break;
}

thanks!!!

The solution by @RicardoBer worked great up until very recently. Most likely it stopped working because I upgraded React Native (currently on 0.61.1). This is what works for me now is this in case anybody needs it.
componentDidUpdate(prevProps){
  const isDrawerOpen = this.props.navigation.state.isDrawerOpen;
  const wasDrawerOpen = prevProps.navigation.state.isDrawerOpen;

  if(!wasDrawerOpen && isDrawerOpen){
    this.props.isOpen();
  }
  else if(wasDrawerOpen && !isDrawerOpen){
    this.props.isClosed();
  }
}

With hooks
const drawerContentComponent = ({ navigation }) => {
	useEffect(() => {
		console.log(navigation.state.isDrawerOpen)
	}, [navigation])
}

Same as #11

componentDidUpdate(prevProps){
const isDrawerOpen = this.props.navigation.state.isDrawerOpen;
const wasDrawerOpen = prevProps.navigation.state.isDrawerOpen;
if(!wasDrawerOpen && isDrawerOpen){
this.props.isOpen();
}
else if(wasDrawerOpen && !isDrawerOpen){
this.props.isClosed();
}
}

it's work for me, thank
you save my day

useEffect(() => {
    return () => {
        const { isDrawerOpen: wasDrawerOpen } = navigation.state
        console.log(wasDrawerOpen)
    }
}, [navigation])

}

The solution by @RicardoBer worked great up until very recently. Most likely it stopped working because I upgraded React Native (currently on 0.61.1). This is what works for me now in case anybody needs it.
componentDidUpdate(prevProps){
  const isDrawerOpen = this.props.navigation.state.isDrawerOpen;
  const wasDrawerOpen = prevProps.navigation.state.isDrawerOpen;

  if(!wasDrawerOpen && isDrawerOpen){
    this.props.isOpen();
  }
  else if(wasDrawerOpen && !isDrawerOpen){
    this.props.isClosed();
  }
}

It's work for me, thank
you saved my day


doc: Update _kliep.py

Wrong index in KLIEP doc

Codecov Report

Merging #24 (899bb5c) into master (e919420) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #24   +/-   ##
=======================================
  Coverage   94.37%   94.37%
=======================================
  Files          21       21
  Lines        2188     2188
=======================================
  Hits         2065     2065
  Misses        123      123

Impacted Files
Coverage Δ

adapt/instance_based/_kliep.py
92.40% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e919420...899bb5c. Read the comment docs.


feat(type-tests): return void from use function

this removes the implicit any return from the use_old functions. this is a necessary step to enable noImplicitAny in our test packages

Relevant changes are to build-tools/packages/build-tools/src/typeValidator/testGeneration.ts

Unfortunately you won't be able to make the build-tools change and the client change in a single PR since they're in different release groups. You'll have to make the build-tools change then we need to release and integrate it into client.

release and integrate it into client

Is it sufficient to just merge the changes into main and bump the dependency version in this PR, or is there a special release process that must be followed?

Closing as this will be handled by a build-tools bump.


Add low-power support to the e-lock demo application

Signed-off-by: Doru Gucea<EMAIL_ADDRESS>Problem
What is being fixed?  Examples:
NXP E-lock Demo Application was lacking low power support
Change overview
Add low power support for the E-Lock Demo Application
Testing
How was this tested? (at least one bullet point required)

Manually tested and measured the power consumption (<10 uA in low power mode),

Could someone upload the new Docker image (https://github.com/project-chip/connectedhomeip/pull/9120) to the Docker repository?  @woody-apple @andy31415

@saurabhst @Damian-Nordic @msandstedt  @mspang ?


Improve the pull request template

This PR is:

[ ] Adding a new domain
[ ] Updating existing domain size
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[x] Other not listed

[ ] I used the uncompressed size of the site
[ ] I have included a link to the GTMetrix report
[ ] The domain is in the correct alphabetical order
[ ] This site is not a ultra lightweight site
[ ] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[x] Check to confirm

This change to the pull request template aims to:

hide the instructions in the rendered markdown output
make the "Check to confirm" recognized as a tickbox
adds a space after the "GTMetrix Report" to help position the cursor when inserting the link

My hope is that this makes the pull request process smoother for all involved parties.

Thank you I liked that the Instructions are hidden in the rendered output


Can you guys please also provide English Translation?

Hi,
This plugin seems interesting to me. Would you guys please provide a documentation in english also so that everyone can understand?
Thanks

Sorry, because the X5 kernel does not support internationalization temporarily, it cannot be used in countries other than China.
Official documents


YAML import date conversion issue in IAM/Bucket policies

Input
Version: '2012-10-17'
Output
'Version' => #<Date: 2012-10-17 ((2456218j,0s,0n),+0s,2299161j)>

Found that this was actually cuased by an unquoted year-month-day. YAML.load tries to convert/unserialize EVERYTHING! Short of monkeypatching psych; I've switched to YAML.safe_load, and added better input validation and rspec test to catch this, and check for regression. #19


pull request for e2e test at 01/29/2017 01:56:14

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


convert produces catalog keys in wrong order

I don't think this is related to the source volume but it could be. But here's the fsck_hfs output:
   Invalid BSD file type
(4, 0)
   Keys out of order
(4, 4)
Records 5 and 6 (0-based); offsets 0x04AE and 0x04E6
   Keys out of order
(4, 4)
Records 7 and 8 (0-based); offsets 0x0514 and 0x0548
   Keys out of order
(4, 4)
Records 9 and 10 (0-based); offsets 0x057C and 0x05A2
   Keys out of order
(4, 4)
Records 11 and 12 (0-based); offsets 0x05D2 and 0x0602

analyze (with some upgrades to report node and record indexes and more fully describe catalog keys) confirms that the file records are not in the right order in that node.

Upgraded analyze output, with filenames redacted:
- 4:4
- 📄🧵 puts item #469, with name “G…”, in parent ID #468
- 4:5
- 📄🧵 puts item #470, with name “@S…”, in parent ID #464
- 4:6
- 📄🧵 puts item #470, with name “L…”, in parent ID #468
- 4:7
- 📄🧵 puts item #471, with name “P…”, in parent ID #464
- 4:8
- 📄🧵 puts item #471, with name “L…”, in parent ID #468
- 4:9
- 📄🧵 puts item #472, with name “S…”, in parent ID #464
- 4:10
- 📄🧵 puts item #472, with name “P…”, in parent ID #468
- 4:11
- 📄🧵 puts item #473, with name “SU…”, in parent ID #464
- 4:12
- 📄🧵 puts item #473, with name “S…”, in parent ID #468
- 4:13
- 📄🧵 puts item #474, with name “S…”, in parent ID #464

This seems like a continuation of #55. impluse reused the file IDs from the original volume (as a general rule, impluse does not try to fix things that diagnostic tools like fsck don't complain about) but didn't sort the keys for the thread records it created correctly.
I need to check on the key comparator in the thread builder. It might be behaving wrongly when the parentID of two keys is equal—though that happens all the time for folders, and shouldn't be any different for thread records.

Was thinking earlier today about the fact that thread keys don't have names, only node IDs. That's probably the problem here: it's comparing keys that are equal, because they have the same node ID and no node name.
The key comparator is going to need access to the payload record as well, because it's the node name (I think) in the thread record payload that breaks the tie between keys.


Support for multiple slaves?

The readme refers to the slave database in plural form in many places, but looking at the code and example it seems it only supports one slave.
Does this currently support multiple slaves?

There is a section in the readme about possible future enhancements, which is support for multiple slaves. Only wanted to add this feature if there is demand for it.
I will be happy to add the feature.
When supporting multiple slaves we have a few options:

Randomly select a slave instance on startup to balance load.
Use another algorithm to choose a slave on process startup, such as lowest ping time.
Or, name slaves and in the code explicitly select which slave to connect to for selected blocks of code.

Do you have a specific use case that we can address?

I would say a random selection to balance the load. The other two would be nice to have as options in the future.
On Thu, Nov 12, 2015 at 9:47 PM, Reid Morrison<EMAIL_ADDRESS>wrote:

There is a section in the readme about possible future enhancements, which is support for multiple slaves. Only wanted to add this feature if there is demand for it.
I will be happy to add the feature.
When supporting multiple slaves we have a few options:

Randomly select a slave instance on startup to balance load.
Use another algorithm to choose a slave on process startup, such as lowest ping time.
Or, name slaves and in the code explicitly select which slave to connect to for selected blocks of code.
Do you have a specific use case that we can address?

Reply to this email directly or view it on GitHub:
https://github.com/rocketjob/active_record_slave/issues/1#issuecomment-156306723

I could build this behavior directly into active_record_slave, or we could do it in the database.yml as follows:
production:
  database: production
  username: username
  password: password
  encoding: utf8
  adapter:  mysql
  host:     master1
  pool:     50
  slave:
    database: production
    username: username
    password: password
    encoding: utf8
    adapter:  mysql
    host:     <%= %w(slave1 slave2 slave3).sample %>
    pool:     50


Make bug-mining scripts mine bugs chronological so that newer bugs get larger IDs

Currently, the bug-mining scripts work top down (starting with the HEAD revision); they should work bottom up instead.

We won't change the bug IDs. Rather, Defects4J provides the timestamp of each bug. This ensures backward compatibility, avoids confusion, and allows us to later add previously excluded bugs (e.g., after fixing compilation issues or minimizing more patches).


Add support for wasm32 target & silence arithmetic error

Hello! Thank you so much for the awesome library.
I've started using it in a game I'm developing with the Bevy engine (really all I have right is a sphere).
Your library was super simple to get started with and works really well so far.
The only issue I had was rayon and std::time don't work on the wasm32 target yet so I needed to disable those to get it to work in the browser.
Adding cfg(not(target_arch = "wasm32")) everywhere the timer & rayon were used got it working for me.

Codecov Report

Merging #8 (d962e0b) into master (e1d1eec) will decrease coverage by 0.55%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
- Coverage   85.90%   85.35%   -0.56%
==========================================
  Files           6        6
  Lines         596      478     -118
==========================================
- Hits          512      408     -104
+ Misses         84       70      -14

Impacted Files
Coverage Δ

src/manifold_dual_contouring.rs
97.05% <ø> (+0.44%)
:arrow_up:

src/mesh.rs
94.44% <0.00%> (-3.29%)
:arrow_down:

src/cell_configs.rs
95.31% <0.00%> (-1.08%)
:arrow_down:

src/vertex_index.rs
96.36% <0.00%> (-0.74%)
:arrow_down:

src/qef.rs
73.57% <0.00%> (+0.13%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e1d1eec...d962e0b. Read the comment docs.


Pages paths refactor

Describe the task
The pages should have the following paths:
/cif/projects
/cif/project/new
/cif/project/[projectId] : the "Overview tab
/cif/project/[projectId]/attachments
/cif/project/[projectId]/revisions
/cif/operators
/cif/contacts
A pagesRoutes.ts file should be implemented to define these routes. See CIIP's routes.ts file

should /cif/project/new be /cif/project/[revisionId] since starting a new project would create a revision? This route would also be used later for editing a project

/cif/project/new should actually be /cif/project-revision/[revisionId]/edit

Thank you @matthieu-foucault! That fixes the issue I was experiencing during testing yesterday. While reviewing this task, I noticed that the upload attachments is no longer available:

The uploads code is still there, but is just a test page for now, so I didn't want to leave links to it. Links to access an attachments page will be added in #74


Revert ValueProvider class

This reverts commit ee92b96 and 28a0ea8
Manually resolved two minor conflicts:
sdks/python/apache_beam/io/filebasedsource.
sdks/python/apache_beam/runners/direct/direct_runner.p

R: @sb2nov

LGTM

Coverage increased (+0.007%) to 70.576% when pulling 8fdf9aa5632d5e186e7fd41f3eac733beeb0f919 on aaltay:undo into 571631a5ef83ef31ea9f652c5698cea3cc95ad43 on apache:master.


About contribute in pulse

Expected Behavior
Integrate with github project
Is it possible to open a github project to help you as best as we can with focus cards on specific issue or feature needed ?

What exactly do you mean?

to let us see the roadmap etc with specific tasks needed


feat(DOM className): added regex for DOM className

Hello guys,
I noticed that there is no intellicense on for example
document.body.className = "bg-red-500";

const el = document.createElement("div");
el.className = "bg-red-500";

Thanks!


[Server thread/ERROR] [Sponge]: Error occurred while executing command 'mysteryblock list'

http://pastebin.com/gQPAG11J

Resolved


Don't render hidden widgets in markdown visual editor

Fixes #1405
- Summary
Fields using hidden was rendered in the markdown editor with "no control" error if no label attached, else a grey box. Return null when rendering the control for hidden widgets.
- Test plan
After change UI no longer displays hidden components in the markdown visual editor, nor the markdown editor.

This was tested using the registered widget in the bug report, attached below.
CMS.registerEditorComponent({
  // Internal id of the component
  id: "youtube",
  // Visible label
  label: "Youtube",
  // Fields the user need to fill out when adding an instance of the component
  fields: [{
    name: 'id',
    label: 'Youtube Video ID',
    widget: 'string'
  }, {
    name: 'myHiddenField',
    label: 'My hidden field',
    widget: 'hidden'
  }],
  // Pattern to identify a block as being an instance of this component
  pattern: /^youtube (\S+)$/,
  // Function to extract data elements from the regexp match
  fromBlock: function(match) {
    return {
      id: match[1]
    };
  },
  // Function to create a text block from an instance of this component
  toBlock: function(obj) {
    return 'youtube ' + obj.id;
  },
  // Preview output for this component. Can either be a string or a React component
  // (component gives better render performance)
  toPreview: function(obj) {
    return (
      '<img src="http://img.youtube.com/vi/' + obj.id + '/maxresdefault.jpg" alt="Youtube Video"/>'
    );
  }
});

- Description for the changelog
Return null when rendering hidden widgets in the markdown editor.
- A picture of a cute animal (not mandatory but encouraged)

Deploy preview for cms-demo ready!
Built with commit 0a41609a7fb91b95b913a07ba4eeec8a30811b48
https://deploy-preview-1414--cms-demo.netlify.com

Deploy preview for netlify-cms-www ready!
Built with commit 0a41609a7fb91b95b913a07ba4eeec8a30811b48
https://deploy-preview-1414--netlify-cms-www.netlify.com

Hey! To throw in my salt; another approach could be to construct a boolean holding if this widget is hidden or not.
const hidden = field.get('widget') === 'hidden' ? 'display: hidden;' : 'display: block;'
then on the div upon return;
`
<Control {...controlProps}/>
`

@fellasuess It makes sense and i thought about this as well, then i found the following code for other hidden widgets:
   {fields.map((field, i) => field.get('widget') === 'hidden' ? null : (render the component)

https://github.com/netlify/netlify-cms/blob/78e4b829d0ced72f8f38721d38c0d67ccb092aaf/src/components/Editor/EditorControlPane/EditorControlPane.js#L47
Maybe the preferred way is to not execute the rest (by the early return fo null) if the component has no value of being rendered. Anyone else has an opinion on this?

Rendering null is 👍


Play on hover

Hi!
Is it possible to play an animation on hover?
Seriously love this plug in!

hi!
you can set the autoplay to false.
Then add an event listener to the mouseover and call anim.play()


[6.0.0] Resolve peerDependencies warning for react-redux

Self-explanatory.
However, given the amount of diff this generated in the JS source, I figured I'd put it up for a review.

Security Insights

(2) Vulnerable direct dependencies were detected
1 vulns in ini < 1.3.6 via package-lock.json
1 vulns in node-fetch < 2.6.1 via package-lock.json

Error scanning for security relevant changes

Error scanning for potential secrets

Error scanning for cfn validation issues

Action Items

Review dependencies for available updates
See this Splunk dashboard for more CVE details
Review PR for security impact; comment "security review required" if needed or unsure
Verify aviary.yaml coverage of security relevant code

Questions or Comments? Reach out on Slack: #support-infosec.

Security Insights

(2) Vulnerable direct dependencies were detected
1 vulns in ini < 1.3.6 via package-lock.json
1 vulns in node-fetch < 2.6.1 via package-lock.json

Error scanning for security relevant changes

Error scanning for potential secrets

Error scanning for cfn validation issues

Action Items

Review dependencies for available updates
See this Splunk dashboard for more CVE details
Review PR for security impact; comment "security review required" if needed or unsure
Verify aviary.yaml coverage of security relevant code

Questions or Comments? Reach out on Slack: #support-infosec.

Public API Changes
No changes to the public API found for commit 01aff355da2b00cd82e9d09f267e0b12c4376f14
Showing results for 01aff355da2b00cd82e9d09f267e0b12c4376f14

Powered by semver-audit-service. Please report any problems by filing an issue.
Reported by the dart semver audit client 2.2.0
Browse public API.

Public API Changes
No changes to the public API found for commit 01aff355da2b00cd82e9d09f267e0b12c4376f14
Showing results for 01aff355da2b00cd82e9d09f267e0b12c4376f14

Powered by semver-audit-service. Please report any problems by filing an issue.
Reported by the dart semver audit client 2.2.0
Browse public API.

@aviary-wf try again please

@aviary-wf try again please


Add ability to cancel test runs.

Note: I haven't added any unit tests for this functionality yet, but that is probably appropriate.

Note when reviewing: I believe the most critical use case for this for g10 is to get people unstuck from 'waiting' status.  If the case of cancelling any arbitrary tests that are running isn't quite perfect from a user experience, I don't think that is a huge deal at this point.
Functionally, when a test is 'cancelled', the test run is set to a 'cancelling' status. The test runner will finish the currently running test as normal, but then all of the rest of the tests will get a 'cancel' result.  And when completed, the test run will get changed to 'cancelled' status.
For tests in a 'wait' status, cancelling will start a new job, set the waiting test to 'cancel' result instead of 'pass', and then act in the same manner as regular cancellation.
Right now, I have the UI still show the test run as active when in the 'cancelling' status (but not yet 'cancelled').  This best reflects reality because there no way to kill the test that is currently in the process of being run, but also may be a little confusing.  We could have the UI pretend that 'cancelling' is essentially 'cancelled', and then maybe throw out results of the test that was currently running when the test run was cancelled.  But I feel like what we have here is probably good enough for now.
I set the 'cancel' icon to be the same as 'fail' for now.  I think that's what we do in legacy.
In the Demo suite, try out the 'Tests that should be easily cancelled' test (it will pause for X seconds), and the Wait test.

Codecov Report

Merging #78 (1b6ec32) into main (6420555) will decrease coverage by 0.62%.
The diff coverage is 64.00%.

@@            Coverage Diff             @@
##             main      #78      +/-   ##
==========================================
- Coverage   94.34%   93.71%   -0.63%
==========================================
  Files          99      100       +1
  Lines        2422     2468      +46
==========================================
+ Hits         2285     2313      +28
- Misses        137      155      +18

Impacted Files
Coverage Δ

.../inferno/apps/web/controllers/test_runs/destroy.rb
33.33% <33.33%> (ø)

lib/inferno/repositories/results.rb
97.59% <50.00%> (-1.18%)
:arrow_down:

lib/inferno/repositories/test_runs.rb
92.45% <50.00%> (-3.47%)
:arrow_down:

lib/inferno/test_runner.rb
98.96% <87.50%> (-1.04%)
:arrow_down:

dev_suites/dev_demo_ig_stu1/demo_suite.rb
92.56% <100.00%> (+0.89%)
:arrow_up:

lib/inferno/apps/web/router.rb
95.65% <100.00%> (ø)

lib/inferno/entities/test_run.rb
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6420555...1b6ec32. Read the comment docs.

Updated the behavior when you click cancel to make it more obvious what is happening, because it could take a little while for the currently running test to finish:

I spent a little while trying to figure out how to test this functionality and it was a more complicated task than I was prepared for.  I only added a test for the test_run DELETE route.  I may need some direction if you think tests here are necessary @Jammjammjamm.


Update README.md

It seems the external module key word as NO 's'

Coverage remained the same at 3.837% when pulling 6ae8d4cbb368fce00383ccc9b0081dbfb7fce08e on nicomlas:patch-1 into cea3cbabc8986e81526720ed6df96a1262c46eed on chafey:master.

Wow you are right. I guess I wasn't paying attention when changing the README. Thanks!

You're welcome. Thanks to your work :) Great job.
FYI: I believe it is the same on other libs (WebImageLoader etc...)
On Fri, Nov 3, 2017 at 11:35 AM, Erik Ziegler<EMAIL_ADDRESS>wrote:

Wow you are right. I guess I wasn't paying attention when changing the
README. Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/chafey/cornerstoneTools/pull/276#issuecomment-341668600,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACd9sgpiLdxqWU_g1OFcd34qvG8CRyxfks5syuyLgaJpZM4QQ3KY
.

Ugh yeah thanks for the heads up!


Create the ability to 'remove' a pattern

@mcollina could you put your thoughts around how this is done here? I don't mind tackling it tonight as it will allow me to push out a new version of Nanite.

The best way would be to get the matching bucket, remove the relative entry, and then regenerate the bloom filter only with the remaining entries. Is it clear?

@mcollina, sort of, not sure how to regenerate without re adding them all via the .add method

For each pattern, do https://github.com/mcollina/bloomrun/blob/master/bloomrun.js#L35

and you would need to reinitialize https://github.com/mcollina/bloomrun/blob/master/lib/bucket.js#L6-L9

Awesome, if I get stuck I'll ping you.

:+1:

Released in 0.3.0.


confusing unit descriptions for "fee_per_mil" and "fee_rate"

For fee_per_mil we have a definition of "The amount charged per milli-satoshis transferred expressed in millionths of a satoshi.". I interpret the units for this to be (1sat/1milli-sat)*(10^6 micro-sat)/(1sat)=(10^6 micro-sat)/(1milli-sat) .
For fee_rate we have a definition of "The effective fee rate in milli-satoshis. Computed by dividing the fee_per_mil value by 1 million.". If I take the above assumption, then I get ((10^6 micro sat)/(1milli-sat))/(10^6 microsat/1sat)=1sat/1milli-sat , which is not in milli-sat.
Should the definition of fee_per_mil read "The milli-satoshi amount charged per milli-satoshis transferred expressed in micro-satoshi per milli-satoshi"`?
Should the definition of fee_rate be "The milli-satoshi amount charged per milli-satoshis transferred, which is equal to fee_per_mil*10^6"? If this is correct, fee_rate is unitless.
Is there a clearer way to present it?

Note, the above definitions came from ChannelFeeReport.
Note: lnd -h indicates:

bitcoin.feerate, The fee rate used when forwarding payments on our channels. The total fee charged is basefee + (amount * feerate / 1000000), where amount is the forwarded amount. (default: 1)

So, I'm even more confused at this point. If I look at a channel, using lncli feereport, fee_rate=0.000001 and fee_per_mil=1, which contradicts.
Also note, lncli help updatechanpolicy says "fee_rate, the fee rate that will be charged proportionally based on the value of each forwarded HTLC, the lowest possible rate is 0 with a granularity of 0.000001 (millionths)." This seems to be in agreement with ChannelFeeReport, but not lnd -h.

Related to #1523

I was looking into this issue.
A bit confused though.
My understanding of the feerate comes from this line in link.go and this test. To put it here, the expected fee is calculated using,
f.BaseFee + (htlcAmt*f.FeeRate)/1000000

Since both f.BaseFee and htlcAmt are in msat, reversing the equation, the f.FeeRate is then using a uint as a billionth of a satoshi per msat, which is 'nano-sat' per msat, or, nsat / msat, this is more clear in the equation,
msat + (msat *  nsat / msat) / 1e6

And my understanding of the fee_rate is, the amount to charge in nsat to route 1 msat. If that's correct, I'd suggest two changes.

The first is to fix the inaccuracies in the documentations, which is what this issue is about.
A further approach, is to introduce a new type, lnwire.NanoSatoshi, to address the speciality of the fee rate used in routing payments. Also we could refactor some of the variable names.

I think stating it in nanosat per millisat confuses people even more.  It's a fraction.  Just say its in "per million" (and don't abbreviate it as per_mil, since that may be read as per mille, which is just per thousand).  We are used to compute in percent and this should follow this notion only it is in "per million" not "per hundred".  There is also ppm (parts per million) but I'm not sure if it should be used here, since there are no particles.


Add Issue templates

Closes #2
I have added a set of issue templates covering the items in #2.
Looking at the 'rich' views of the templates should show what they will look like to users, these are like html forms rather than a markdown template.
Changes:

Add config.yml to control issue opening
Add issue templates (forms) for discussions, proposed changes, governance and management

@arronlacey (Because I couldn't add you as a reviewer)

What is the purpose of the discussion form vs the change form? I kind of see "change" as meaning changing something that already exists, rather than adding something new.
Would proposing a new feature be a discussion or a change form? We use the discussion form to discuss a proposed new feature, and then a change form to formulate the exact wording that would go in the document, something like that?
Given that we want to describe features as optional/recommended/mandatory (or similar), should the forms directly request the author to propose one of those labels for a given feature?

We use the discussion form to discuss a proposed new feature, and then a change form to formulate the exact wording that would go in the document, something like that?

Something like that is the idea! Do you think it's unnecessary extra, or a useful distinction? I think it helps to see where people are directly looking to update the specification, and where they're having more of an exploratory discussion. But could be convinced either way.

to propose one of those labels for a given feature?

It depends if we want to give any contributor the ability to add labels (if we can I argue we should). Others thoughts?

I think we should keep things simple, maintainers can add labels based on the created issue. If we grow and started getting 10s of issues everday then we should look at more automation, but I don't think we need to over-engineer things now- it's likely to change anyway!
It's not clear to me what the difference between a "change" and a "discussion" is- aren't all changes discussed? If you think it's useful can you add a paragraph or two to the templates (after the This issue form is for ... line) describing when it should be used?

I see the distinction between a discussion and a change proposal quite clearly as,

A discussion is open-ended conversation about anything to do with the project and its management (including governance, for example)
A change is a specific proposal to change the specification. A change could be an addition, modification or removal.

I think both are useful and important, especially as we are proposing having all conversations in issues. Adding some text to clarify that would help.
I don't think we can let anyone modify labels. But also, I think it would be more useful for people playing the 'triage' role to manage categorisation with labels.
I would lean against having a drop-down or label for whether a feature is mandatory or not. I think that can only be decided through consensus and would be part of the text rather than the metadata.

I think we should keep things simple, maintainers can add labels based on the created issue. If we grow and start getting 10s of issues everday then we should look at more automation, but I don't think we need to over-engineer things now- it's likely to change anyway!

Broadly agreed. However, I think there is a very low cost to keeping the templates up to date compared to the benefits of standardisation and approachability. We get some automatic assignment of labels for free so we might as well take adavantage.

thanks for this @JimMadge.
The "proposed change" section in the "Change" template makes it clear there is difference for me from the "Discussion" template. Perhaps we should also ask in the mandatory Details form to state explicitly which files they are intending to change. I think that should help emphasize that this issue is proposing a change to the repo.

Perhaps we should also ask in the mandatory Details form to state explicitly which files they are intending to change. I think that should help emphasize that this issue is proposing a change to the repo.

I think that is a good idea. The idea is this template is for a specific change, so someone creating the idea should be able to identify where those changes need to be made. Also, it saves people contributing to the issue from searching for that text themselves.

@arronlacey @manics @harisood Given it feels like we are moving towards using PRs to propose 'concrete' changes, do we think the change issue template is redundant?

I don't mind too much- if someone wants to use an issue because it's easier for them I think that's fine, but I'm also fine with using a discussion for it.

Perhaps we should also ask in the mandatory Details form to state explicitly which files they are intending to change. I think that should help emphasize that this issue is proposing a change to the repo.

Instead of which file could we ask "which section" / "which feature"?

I think it's okay to have both as well - someone might find it handy to come straight in and be able to suggest a change directly within an issue.

@JimMadge @manics @arronlacey do you think we should set it up so no PRs can be merged until an approved review?

In future I think it'd be good to enforce N approvals as a branch protection rule. For now I don't mind- the people with permission to merge will always have the permission to approve.


Adding LDAP auth support note for Teradata model refresh

Chris Finlan - Confirmed Teradata with LDAP auth is not supported

#sign-off
thanks, @JoshGer !


Changing Cancel and Done button colors

Dear CardIO,
Is there a way to change the colors of the "Cancel" and "Done" buttons on the Navbar for the CardIOPaymentViewController?
I noticed the keepStatusBarStyle property and it was very helpful in keeping the style of our NavBar.
Thanks!

Nothing worked but this. Took almost two hours to figure-out.
NSDictionary *attributes = [NSDictionary dictionaryWithObjectsAndKeys:
                                [UIColor redColor],UITextAttributeTextColor,
                                nil];
    [[UIBarButtonItem appearanceWhenContainedIn:[UINavigationBar class], nil]
     setTitleTextAttributes:attributes
     forState:UIControlStateNormal];

Swift implementation.
cardIOVC?.navigationBar.titleTextAttributes = [NSForegroundColorAttributeName : UIColor.white, NSFontAttributeName : UIFont(name: "AvenirNext-Regular", size: 16)!]         UIBarButtonItem.appearance(whenContainedInInstancesOf: [UINavigationBar.self]).setTitleTextAttributes([NSForegroundColorAttributeName : UIColor.white, NSFontAttributeName : UIFont(name: "AvenirNext-Regular", size: 16)!], for: .normal)


code command support custom directory when attaching to container

When running fm code <sitename>, VSCode currently opens with /workspace as the default directory.
However, users should have the option to change this directory.

[x] Implement the --work-dir flag, allowing users to define their preferred directory.
[x] The default directory should be /workspace/frappe-bench.

When running fm code <sitename>, VSCode currently opens with /workspace as the default directory. However, users should have the option to change this directory.
* [x]  Implement the `--work-dir` flag, allowing users to define their preferred directory.

* [x]  The default directory should be `/workspace/frappe-bench`.

Implemented in #140.


fit() call with no tasks

Hi Everybody,
it seems like my ml backend is receiving the fit signals but no tasks are associated (the first argument is an empty tuple)
The culprit seems to be this line that I can't make sense of.
When a train event is received the object calls the fit method... with an empty tuple as argument, and that's why I receive an empty tuple.
Where are the tasks?
What am I missing?
Thank you in advance

I do use label studio, and digging I just read that since 1.4.1 the behaviour of label-studio changed and instead of calling the _train method of Flask a webhook is called. Is this what's breaking my previously working code?
Is label-studio-ml-backend updated with the latest behaviour of label-studio?

To give even more context, I am running a docker-compose with 4 bridged services:

label-studio
postgresql
label-studio-ml-backend
redis

Basically my LabelStudioMLBase.fit() function is defined as
def fit(self,tasks,**kwargs):
...
and when the Start Training button is pressed in the label-studio interface, the browser calls the label-studio api at the /api/ml/{ml_backend_id}/train endpoint, which queues a job on RQ;
But when the RQ job is processed, what the fit function receives is an empty Tuple for tasks (coherent with the line highlighted in the issue) and the following kwargs dict (some info redacted '...')
{
    "event": "PROJECT_UPDATED",
    "data": {
        "project": {
            "id": 1,
            "task_number": 5,
            "finished_task_number": 5,
            "total_predictions_number": 0,
            "total_annotations_number": 5,
            "num_tasks_with_annotations": 5,
            "useful_annotation_number": 5,
            "ground_truth_number": 0,
            "skipped_annotations_number": 0,
            "title": ...,
            "description": "",
            "label_config": ...,
            "parsed_label_config":...,
        },
        "expert_instruction": "",
        "show_instruction": False,
        "show_skip_button": True,
        "enable_empty_annotation": True,
        "reveal_preannotations_interactively": False,
        "show_annotation_history": False,
        "show_collab_predictions": True,
        "evaluate_predictions_automatically": False,
        "token": ...,
        "result_count": 0,
        "color": "#FFFFFF",
        "maximum_annotations": 1,
        "min_annotations_to_start_training": 0,
        "control_weights": ...,
        "model_version": "INITIAL",
        "data_types": {
            "image": "Image",
            "image_name": "Text"
        },
        "is_draft": False,
        "is_published": False,
        "created_at": ...,
        "updated_at": ...,
        "sampling": "Sequential sampling",
        "skip_queue": "REQUEUE_FOR_OTHERS",
        "show_ground_truth_first": False,
        "show_overlap_first": False,
        "overlap_cohort_percentage": 100,
        "task_data_login": ...,
        "task_data_password": ...,
        "organization": 1,
        "created_by": 1
    },
    "job_id": "1648737854",
    "workdir": "/data/models/1648737854"
}

Am I misunderstanding some step or missing something? Am I supposed to access the jsons and parse them directly from label-studio-ml-backend now?

It seems that you invoke training with webhook. Please check this article to fix the issue.

Exactly! Thank you so much for the link.
So it IS intended behaviour starting from label-studio 1.4.1 correct?
I'll try to implement one of the solutions proposed, but it looks to me like that backward incompatible change should be reflected in this repo's docs at least by not assuming that the fit() method would receive the tasks right?
In any case, thanks for the clarification and the help. If what I suggested up there makes no sense to you we can close this issue.

Yes, it's intended behaviour starting from label-studio 1.4.1.
If you want to enable previous behavior, run LS with this env flag:
ff_back_dev_1417_start_training_mlbackend_webhooks_250122_long=1 label-studio

so, How to use API to get all annotated tasks?
Thanks.

How to use API to get all annotated tasks?

@lglong123 you can use this example


Merging batch/queue updates into release/9.0.0 branch

These updates abstract the batching functionality (delete pages, rescan files, etc...) away from the actual queueing layer introduced in version 9.

Something about the core test suite is now failing but only on Windows and only on PHP 7.2. It's a memory limit error. All other test suites are passing. Since I don't have access to Windows and this is comparably a very low priority to other things I'm ignoring the Windows test failure and merging this into release/9.0.0. This isn't going to merged into develop any time soon, but it's probably good to get it on @mlocati 's radar.


Add missing scm section to glossary view services (NEEDED for release)

In #2592 a new pom was added in open-metadata-implementation/view-services/glossary-author-view
However this is missing the required '' section as identified during review ie
   <scm>
        <connection>scm:git:git://github.com/odpi/egeria.git</connection>
        <developerConnection>scm:git:ssh://github.com/odpi/egeria.git</developerConnection>
        <url>http://github.com/odpi/egeria/tree/master</url>
    </scm>

this may PREVENT the release from shipping as it is required by the release process

This was fixed in #2649


Delete token #12981 from the list

I'm submitting a ...
[ ] bug report
[x] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Hello,
Please can you delete #12981 of the list it's the wrong adress.
Thank you!

#13468


Update common-questions.md

@Rajani-Janaki-Ram : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @Rajani-Janaki-Ram - This PR was opened in the public repo. Would you submit this update in the private repo instead? Thanks. We'll close this PR.


"ERROR Invalid hook call" using ipp with next.js

Describe the bug
Running ipp within a next.js project is giving me an error related to hooks in ipp:
ipp --input ../images --output ./public

  ERROR Invalid hook call. Hooks can only be called inside of the body of a function component. This
       could happen for one of the following reasons:
       1. You might have mismatching versions of React and the renderer (such as React DOM)
       2. You might be breaking the Rules of Hooks
       3. You might have more than one copy of React in the same app
       See https://fb.me/react-invalid-hook-call for tips about how to debug and fix this problem.

 - 1. You might have mismatching versions of React and the renderer (such as React DOM)
 - 2. You might be breaking the Rules of Hooks
 - 3. You might have more than one copy of React in the same app
 - See https://fb.me/react-invalid-hook-call for tips about how to debug and fix this problem.
 - resolveDispatcher (node_modules/@ipp/cli/node_modules/react/cjs/react.development.js:1465:13)
 - Object.useContext (node_modules/@ipp/cli/node_modules/react/cjs/react.development.js:1473:20)
 - useAppState (node_modules/@ipp/cli/dist/ui/dynamic/hooks.js:16:32)
 - Object.useAppStateValue (node_modules/@ipp/cli/dist/ui/dynamic/hooks.js:21:12)
 - exports.Status (node_modules/@ipp/cli/dist/ui/dynamic/Status.js:28:28)
 - renderWithHooks (node_modules/react-reconciler/cjs/react-reconciler.development.js:6036:18)
 -mountIndeterminateComponen (node_modules/react-reconciler/cjs/react-reconciler.development.js:8570:1
                            3)
 - beginWork$1 (node_modules/react-reconciler/cjs/react-reconciler.development.js:9938:16)
 -Object.invokeGuardedCallbackI (node_modules/react-reconciler/cjs/react-reconciler.development.js:115
  pl                           63:10)
 - invokeGuardedCallback (node_modules/react-reconciler/cjs/react-reconciler.development.js:11740:31)

The above error occurred in one of your React components:
    in Unknown
    in InternalApp

React will try to recreate this component tree from scratch using the error boundary you provided, InternalApp.

System information:

OS: macOS 10.14.6
Node version: v14.15.0
IPP version: 1.2.1

Additional context
I suspect it's related to mismatching React versions:
$ npm list react
<EMAIL_ADDRESS>/Users/humphd/repos/app
├─┬<EMAIL_ADDRESS>│ └── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>└── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>
I still get the generated images, but it looks like at least some of the UI related code isn't running due to this.

This does indeed look like an issue with a React version mismatch, unfortunately, npm was never designed to support different version resolution in the same project. The CLI uses Ink to render React in the terminal, which doesn't seem to play nice with the recent update to React that you're using with Next.
I'll check it out to see when an upgrade is possible, in the meantime, you can use the --text flag to use a more basic text renderer, although the above error shouldn't affect the actual processing of images.

Fantastic, thanks --text works great for my use case.

The CLI has been upgraded to React 17. If the issue still persists with the new version, feel free to reopen this issue.

The CLI has been upgraded to React 17. If the issue still persists with the new version, feel free to reopen this issue.


New version: WeasisTeam.Weasis version 4.1.2

[] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/110098&drop=dogfoodAlpha

@microsoft-github-policy-service agree


SliverPinnedHeader and detecting overlapping (i.e. overlapsContent)

Is there a way to detect the overlapping content using a SliverPinnedHeader like you can do with a SliverPersistentHeaderDelegate?  I'm ultimately looking to change the background of the SliverPinnedHeader when content overlaps.

@TyBarthel You can use SliverLayoutBuilder to achieve it.
SliverLayoutBuilder(
  builder: (_, SliverConstraints constraints) {
    return SliverPinnedHeader(
      child: Text('Is overlapped? ${constraints.overlap > 0}'),
    );
  },
),


FXML Support

If our views are built via FXML and then setup on the main fxml widget how do we make this work? Are there examples?

Sorry, no plans for FXML.  I think it is a counterproductive and unnecessary idea, which adds design- and run-time penalty and foreign (XML) dependencies.
This library is Apache licensed, feel free to fork and add features though.


[Request]: How to find the Zoom link

Contact Details
No response
Organizer Request
Hi - hope this is the right place to ask this:
on the "how to attend" page you say "Each calendar entry contains the Zoom link.",
in the calendar there is an entry right now (2022-03-16, 18:15 CET), but i do not see a zoomlink
Code of Conduct

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

It seems to me that the Google Calendar is shared in 'busy' mode. So no event details. Also, I can't see any future one scheduled.

At this point of the program's development, all events are invite-only.  We will be growing the program over time, but it is unlikely that we will have a more open offering before 2023.

hey, I'm also interested in RRC, in case there's no more free slots, is there a place where you've recorded the videos?


User Profile Update fail - TypeError: user.then is not a function

Any profile updates (press update profile button) fails with the below exception. This is true for "Profile Information" and "Change Password".
Stack: Node, Express, Bootstrap, no css or js frameworks, Mocha, MySQL
Node: v6.3.1
TypeError: user.then is not a function
    at exports.accountPut (/Users/ben/WebstormProjects/roadmeer/controllers/user.js:154:8)

Not too sure why we cannot call user.then as user appears return a promise. Interested to hear if anyone can advise.
As a workaround, I have simplified the control flow.
/**
 * PUT /account
 * Update profile information OR change password.
 */
exports.accountPut = function(req, res, next) {
  if ("password" in req.body) {
    req
      .assert("password", "Password must be at least 4 characters long")
      .len(4);
    req.assert("confirm", "Passwords must match").equals(req.body.password);
  } else {
    req.assert("email", "Email is not valid").isEmail();
    req.assert("email", "Email cannot be blank").notEmpty();
    req.sanitize("email").normalizeEmail({ remove_dots: false });
  }

  const errors = req.validationErrors();

  if (errors) {
    req.flash("error", errors);
    return res.redirect("/account");
  }

  let d = {
    email: req.body.email,
    name: req.body.name,
    gender: req.body.gender,
    location: req.body.location,
    website: req.body.website
  };

  if ("password" in req.body) {
    d = { password: req.body.password };
  }

  const user = new User({ id: req.user.id });

  user.save(d, { patch: true })
    .then(() => {
      if ("password" in req.body) {
        req.flash("success", { msg: "Your password has been changed." });
      } else {
        req.flash("success", {
          msg: "Your profile information has been updated."
        });
      }
      res.redirect("/account");
    })
    .catch(err => {
      if (err.code === "ER_DUP_ENTRY") {
        req.flash("error", {
          msg: "The email address you have entered is already associated with another account."
        });
      }
    });
};


Express' card() should gain a sidebar arg

This should provide a decent workaround for the fact that you can't do with layout.sidebar() multiple times.
(Also, for some context, we never a sidebar arg on card() in the 1st place since it's easy enough to do card(layout_sidebar()), but we don't have that luxury currently in express

On 2nd thought, this might not be a great idea (since there is some ambiguity with how to place the sidebar in relation to other card items)


placeholder rendering messed with default value placeholder value

in readme file placeholder spellings are incorrect.. is that intensional?
See attached screen shot..

see attached screen shot..

Here is the solution for the time being, I have used to fix the issue..

@anilios please use latest (1.0.8) version if this works for you.
Tested on iOS and it's working fine with the latest version.

ok thanks, I will check and use.

I am already using 1.0.11

can you please check android crash due to setSpan error?
Crash on Android #12


LSTM weights not loaded correctly in tf.keras

Description
Running into this problem when I:

build model A containing an LSTM layer
save the model weights with tf.keras.save_weights() in TensorFlow format
at some later point create an identical model A' and load the previously saved weights

Inspection by eye shows the lstm weights differ between model A and A' while weights of other layers (e.g. dense) are loaded correctly from file. When running through the same procedure as above but saving the weights in HDF5 format everything works as expected.
System Information:

Have I written custom code: yes
OS Platform and Distribution: Linux Ubuntu 16.04
Mobile device: NA
TensorFlow installed from (source or binary): binary (pip package)
TensorFlow version: 1.10.0
Python version: 3.6.6
Bazel version: NA
GCC/Compiler version: NA
CUDA/cuDNN version: NA
GPU model and memory: NA
Exact command to reproduce:

import tensorflow as tf

input_layer = tf.keras.layers.Input(shape=(None, 5))
lstm_layer = tf.keras.layers.LSTM(units=2)(input_layer)
output_layer = tf.keras.layers.Dense(units=2, activation='softmax')(lstm_layer)

my_model = tf.keras.Model(input_layer, output_layer)

my_model.save_weights("weights_test/my_weights")

print('\nlstm:')
print(my_model.layers[1].get_weights())
print('\ndense:')
print(my_model.layers[2].get_weights())

#lstm:
#[array([[ 0.38913965,  0.4081726 , -0.23971727,  ...
#dense:
#[array([[-0.5567662 , -0.38940358],
#       [ 0.16435587,  1.1092712 ]], dtype=float32), array([0., 0.], dtype=float32)]

new_input_layer = tf.keras.layers.Input(shape=(None, 5))
new_lstm_layer = tf.keras.layers.LSTM(units=2)(new_input_layer)
new_output_layer = tf.keras.layers.Dense(units=2, activation='softmax')(new_lstm_layer)

my_new_model = tf.keras.Model(new_input_layer, new_output_layer)
my_new_model.load_weights("weights_test/my_weights")

print('\nlstm:')
print(my_new_model.layers[1].get_weights())
print('\ndense:')
print(my_new_model.layers[2].get_weights())

#lstm:
#[array([[-0.41712055,  0.67933106, -0.5161066 , ...
#dense:
#[array([[-0.5567662 , -0.38940358],
#       [ 0.16435587,  1.1092712 ]], dtype=float32), array([0., 0.], dtype=float32)]

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

just added the System Information to the issue

@yongtang  Tried different versions of tensorflow. Seems the bug is fixed from 1.11. onward. Thanks and sorry for not checking.

@ptiwald Glad it works. 👍 I will closing this issue, but feel free to reopen if there is still issues related.


Regex tests are to strict for step 9 of TODO project

Describe the Issue
We had a camper on the forum unable to pass step 9.
https://forum.freecodecamp.org/t/learn-localstorage-by-building-a-todo-app-step-9/670010
This was their code
discardBtn.addEventListener("click", () => {

confirmCloseDialog.close()

taskForm.classList.toggle("hidden")}

);

the issue was their formatting not matching what the regex test is expecting here
```js
assert.match(code, /discardBtn\.addEventListener\(\s*('|"|`)click\1\s*,\s*\(\s*\)\s*=>\s*\{\s*confirmCloseDialog\.close\(\s*\);?\s*taskForm\.classList\.toggle\(\s*('|"|`)hidden\2\s*\);?\s*\}\s*\);?/)
```

we should allow for any correct answers to pass and not be strict about formatting here because these are still beginners getting the hang of it.
Affected Page
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures-v8/learn-localstorage-by-building-a-todo-app/step-9
Your code
see explanation above
Expected behavior
see explanation above
Screenshots
No response
System

Device: [e.g. iPhone 6, Laptop]
OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04]
Browser: [e.g. Chrome, Safari]
Version: [e.g. 22]

Additional context
No response

This was actually fixed in #53480 :smile:


POSTGRES_PASSWORD not working

docker run -it -d --name=postgress -e POSTGRES_USER=net123 -e POSTGRES_PASSWORD=passowrd -e POSTGRES_DB=casnio  postgres:9.5-alpine
postgres lunched but it not setting my password net123?
postgres accessed without password even i give poassword in ENV's

You mistyped your password in the env variable "POSTGRES_PASSWORD=passowrd"
$ docker run -it -d --name=postgres -e POSTGRES_USER=net123 -e POSTGRES_PASSWORD=password -e POSTGRES_DB=casnio postgres:9.5-alpine
$ docker exec -it postgres bash
bash-4.4# psql -U net123 -d casnio
psql: warning: extra command-line argument "casnio" ignored
psql: FATAL:  database "b" does not exist
bash-4.4# psql -U net123 -d casnio
psql (9.5.13)
Type "help" for help.

casnio=#

Yes,postgress accessible without password,  i want secure postgres, postgres not asking password.

https://github.com/docker-library/docs/tree/master/postgres#postgres_password

Note 1: The PostgreSQL image sets up trust authentication locally so you may notice a password is not required when connecting from localhost (inside the same container). However, a password will be required if connecting from a different host/container.


Option to remove the Practice with computer button

In some situations it should be hidden automatically, like for Explorer Practice, but there should be an option to hide it until you need it. At least a theme.

Lazily implemented via a theme in v2.3.151


fix: Color consistency

SUMMARY
This PR adds the ability to reset the forced colors of a namespace. It also returns a fresh new color scale instance every time in order for the ordinal scale to work as intended.
This PR is a prerequisite for the related Superset PR https://github.com/apache/superset/pull/17089
TESTING INSTRUCTIONS
Please follow each test case in the related Superset PR https://github.com/apache/superset/pull/17089
ADDITIONAL INFORMATION
Associated issues: https://github.com/apache/superset/issues/11677 https://github.com/apache/superset/issues/16970 https://github.com/apache/superset/issues/16891 https://github.com/apache/superset/issues/16961 https://github.com/apache/superset/issues/16980 https://github.com/apache/superset/issues/16979

Hi @geido, in our last weekly release, we saw a bug where Time Series Area Charts have no fill color. I am guessing it is related to this change. Can you take a look?
Current behavior (superset apache/master, 2021-11-04)

Previous behavior (superset apache/master, 2021-10-28)

Hey @serenajiang! This bug was introduced in https://github.com/apache-superset/superset-ui/pull/1447 and has been fixed by https://github.com/apache-superset/superset-ui/pull/1464. The fix should be included in superset-ui 0.18.23.
CC @villebro


pixi add '<package>=*' no longer works

Checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest version of pixi, using pixi --version.

Reproducible example
$ cat pixi.toml
[project]
authors = ["Matthias Riße<EMAIL_ADDRESS>channels = ["conda-forge"]
description = "Add a short description here"
name = "clams-git"
platforms = ["linux-64"]
version = "0.1.0"

[tasks]

[dependencies]
$ pixi add 'python=*'
  × invalid version constraint: '*' is incompatible with '=' operator'
  ╰─▶ '*' is incompatible with '=' operator'

[ble: exit 1]

Issue description
I want to add a dependency without any specific version. I always want it to get the latest version as of creating the lock file. In the past I've used pixi add '<package>=*' for this purpose, but that no longer works. Manually adding <package> = "*" to the dependencies section in pixi.toml does work though, so I am not sure why pixi add refuses to do it.
Expected behavior
I expected * to continue to be a valid version for pixi add, or alternatively some other way to specify "always the latest version".

As an alternative or complementary feature, it would be nice to be able to specify [package]>=[latest-version] as the dependency to add, without having to figure out the latest version beforehand.
This is similar to what pixi add does already, but without the upper bound on the next major version. In many cases a major version upgrade won't lead to any issues, and if it does I expect the review process in place for updating the lock file to catch those (e.g. through automated testing). The situations where one would want the upper bound are the exception, not the norm, IMO, and those would be determined by the issues found in the lock file update process.

I add #1852 to allow for old style parsing on the command line!


Unnecessary cloning

Is your feature request related to a problem?
There seem to be some unnecessary cases of copying a table in _table.py.
For example, keep_only_columns creates a copy of the table, then calls remove_columns(...) on that copy and returns the result:
Implementation keep_only_columns(...):
    def keep_only_columns(self, column_names: list[str]) -> Table:
        ...
        clone = self._copy()
        clone = clone.remove_columns(list(set(self.column_names) - set(column_names)))
        return clone

However, remove_columns itself already has to - in accordance to our guidelines - create a copy.
Indeed, it does (although very implicitely, by calling Pandas' drop method with the default inplace=false keyword argument):
Implementation of remove_columns(...):
    def remove_columns(self, column_names: list[str]) -> Table:
       ...
        transformed_data = self._data.drop(labels=column_names, axis="columns")
        transformed_data.columns = [name for name in self._schema.column_names if name not in column_names]
        ...
        return Table._from_pandas_dataframe(transformed_data)

Pandas documentation for drop method:
DataFrame.drop(labels=None, *, axis=0, index=None, columns=None, level=None, inplace=False, errors='raise')

Since remove_columns already correctly returns a copy of the table it was called on, it is not clear to me why we need to make an additional copy within out keep_only_columns implementation.
Am I misunderstanding something?
Desired solution
If these additional copies indeed turn out to be unnecessary: Find all occurences of them and remove them,
Possible alternatives (optional)
If the copies actually turn out to be necessary: Close this issue.

:tada: This issue has been resolved in version 0.16.0 :tada:
The release is available on:

v0.16.0
GitHub release

Your semantic-release bot :package::rocket:


When will there be a custom Azure AD RBAC role for managing AAD groups?

We have a need to grant permissions to users to modify group membership of AAD (cloud) security groups. Currently, there is no built-in role or custom role to just grant this level of permission to AAD security groups. The closest we can find is way too much permissions with the Cloud Device Administrator role.

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

ID: 2ced2c12-15c0-9ec3-5a7b-a2498f870e89
Version Independent ID: 6a7ca8e8-f2ca-fa62-6275-e6d4bee8f67f
Content: Preview Azure administrator roles with customizable permissions - Azure Active Directory
Content Source: articles/active-directory/users-groups-roles/roles-custom-overview.md
Service: active-directory
Sub-service: users-groups-roles
GitHub Login: @curtand
Microsoft Alias: curtand

@maxchenz Thanks for your feedback! We will investigate and update as appropriate.

hey @maxchenz I suggest filing this against the product feedback per the feedback section :
https://docs.microsoft.com/en-us/azure/active-directory/users-groups-roles/roles-custom-overview#next-steps
That is the link here : https://feedback.azure.com/forums/169401-azure-active-directory

For a request on this role to be implemented.
That being said you can create a custom role to be able to modify group membership per the docs here : https://docs.microsoft.com/en-us/azure/active-directory/users-groups-roles/roles-create-custom
You can pull from the user admin role : https://docs.microsoft.com/en-us/azure/active-directory/users-groups-roles/directory-assign-admin-roles#user-administrator
That is in particular :

microsoft.aad.directory/groups/members/update
Update groups.members property in Azure Active Directory.

Why can't you just utilize the user administrator role from the docs here :https://docs.microsoft.com/en-us/azure/active-directory/users-groups-roles/directory-assign-admin-roles?

@maxchenz
Please let us know if there are anymore questions within the scope of this git issue. If not, I will be closing out this git issue by end of day tomorrow. Please reopen this git issue if you have anymore concerns. Thanks

Thanks. We are good to close

From: Frank Hu<EMAIL_ADDRESS>Sent: Monday, August 19, 2019 5:59:23 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Maurizzio, Vinnie<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] When will there be a custom Azure AD RBAC role for managing AAD groups? (#37358)
This email is from an external source - exercise caution regarding links and attachments.
@maxchenzhttps://github.com/maxchenz
Please let us know if there are anymore questions within the scope of this git issue. If not, I will be closing out this git issue by end of day tomorrow. Please reopen this git issue if you have anymore concerns. Thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/37358?email_source=notifications&email_token=ALCYQ2UOSLOPIYWTQSMLGU3QFMJTXA5CNFSM4INEAAO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4UNPNA#issuecomment-522770356, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALCYQ2RUHFDCBAMI2X5FOBLQFMJTXANCNFSM4INEAAOQ.

Hey @maxchenz  alright sounds good thanks for your feedback again!


🚀 Feature: Display Error messages

🔖 Feature description
I noticed when I was trying to log in to the app with a non-existent account, the app just did nothing. It did not display an error message to say what was going wrong.
Display error messages in the form itself or like a toaster could be a nice improvement to the UX
🎤 Pitch
It can be frustrating for those testing out the app when they get no feedback for their actions. Displaying an error message can help the user know exactly what they are doing wrong and make necessary adjustments.
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Thanks for raising this! Would you like to look into fixing this?

Hey, I would love to try this out. Can it still be assigned to me?

Certainly! Thank you! :pray:

Great!

Is this issue open to work on now? If, then can I please work on it?

I want to work on this feature.

#42Dear @timo-tech19  ,
I'm Anuj Varshney, experienced in resolving such issues efficiently. Kindly assign it to me. I'll work diligently and keep you updated. Excited to contribute.


Changing color of circle in programatically

How to change circle indicator color for programmatically?

try this:
val indicator: CircleIndicator3 = binding.indicator
indicator.tintIndicator(mContext.getColor(R.color.red))

@CorzaHorizon Thank you <3


Unable to install httpie via Chocolatey

I tried installing Httpie on my Windows machine but I spits out this error everytime.

I am running Powershell 7 on Windows 11 Version 22H2 (OS Build 22621.1555)

any help with this @jakubroztocil

@ChukwunonsoFrank we’ll look into it 👀  It may take a while, though, so in the meantime, I recommend installing it using pip: https://httpie.io/docs/cli/pypi

hi @jkbrzt same problem of the user. I have also tried with pip, after installing MS build tools, but I got this error:
C:\Users\gianp\AppData\Local\Temp\pip-build-env-avrea4bl\overlay\Lib\site-packages\setuptools\command\build_py.py:204: _Warning: Package 'multidict._multilib' is absent from the packages configuration.
!!
          ********************************************************************************
          ############################
          # Package would be ignored #
          ############################
          Python recognizes 'multidict._multilib' as an importable package[^1],
          but it is absent from setuptools' `packages` configuration.

          This leads to an ambiguous overall configuration. If you want to distribute this
          package, please make sure that 'multidict._multilib' is explicitly added
          to the `packages` configuration field.

          Alternatively, you can also rely on setuptools' discovery methods
          (for example by using `find_namespace_packages(...)`/`find_namespace:`
          instead of `find_packages(...)`/`find:`).

          You can read more about "package discovery" on setuptools documentation page:

          - https://setuptools.pypa.io/en/latest/userguide/package_discovery.html

          If you don't want 'multidict._multilib' to be distributed and are
          already explicitly excluding 'multidict._multilib' via
          `find_namespace_packages(...)/find_namespace` or `find_packages(...)/find`,
          you can try to use `exclude_package_data`, or `include-package-data=False` in
          combination with a more fine grained `package-data` configuration.

          You can read more about "package data files" on setuptools documentation page:

          - https://setuptools.pypa.io/en/latest/userguide/datafiles.html

          [^1]: For Python, any directory (with suitable naming) can be imported,
                even if it does not contain any `.py` files.
                On the other hand, currently there is no concept of package data
                directory, all directories are treated like packages.
          ********************************************************************************

  !!
    !!
    check.warn(importable)
  copying multidict\__init__.pyi -> build\lib.win-amd64-cpython-312\multidict
  copying multidict\py.typed -> build\lib.win-amd64-cpython-312\multidict
  running build_ext
  building 'multidict._multidict' extension
  creating build\temp.win-amd64-cpython-312
  creating build\temp.win-amd64-cpython-312\Release
  creating build\temp.win-amd64-cpython-312\Release\multidict
  "C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Tools\MSVC\14.37.32822\bin\HostX86\x64\cl.exe" /c /nologo /O2 /W3 /GL /DNDEBUG /MD -IC:\Python312\include -IC:\Python312\Include "-IC:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Tools\MSVC\14.37.32822\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Auxiliary\VS\include" /Tcmultidict/_multidict.c /Fobuild\temp.win-amd64-cpython-312\Release\multidict/_multidict.obj -O2
  _multidict.c
  C:\Python312\include\pyconfig.h(59): fatal error C1083: Cannot open include file: 'io.h': No such file or directory
  error: command 'C:\\Program Files (x86)\\Microsoft Visual Studio\\2022\\BuildTools\\VC\\Tools\\MSVC\\14.37.32822\\bin\\HostX86\\x64\\cl.exe' failed with exit code 2
  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for multidict
Failed to build multidict
ERROR: Could not build wheels for multidict, which is required to install pyproject.toml-based projects

Hey! I got the same errors while trying to install using chocolatey and pip through PowerShell.
PS Version:  5.1.22621.2428
Windows 11 Version 22H2

I got the same error while trying to install it using Chocolatey through PowerShell
PS Version: 5.1.22621.2506
Windows 11 Version 22H2

ERROR: The running command stopped because the preference variable "ErrorActionPreference" or common parameter is set to Stop:   error: subprocess-exited-with-error
The install of httpie was NOT successful.

Same issue here
Trying to install httpie with pip didn't work either, the logs are not providing any useful information.
I tried to re-install chocolatey and pip just in case maybe I messed something up on my system but it didn't fix the issue.
PS version: 7.4.0
Windows 11 version 22H2.

Looks like this error is a Windows 11 thing cos I'm getting the same errors to, might have to downgrade back to windows 10 and try installing httpie again.

Looks like this error is a Windows 11 thing cos I'm getting the same errors to, might have to downgrade back to windows 10 and try installing httpie again.

No need to downgrade, I fixed it with the pull request that appears above
https://github.com/httpie/cli/pull/1531

Glad to see the PR helped. Could you give us insight as if you've encountered any issues? Or-- simply put a feedback, positive or negative. Feedbacks can improve the level of confidence for the given PR.
Regards,

The feedback is positive. The issue was fixed with that PR.

The PR does not only "fix" this issue but go way beyond that, with HTTP/2, and HTTP/3 support for example. So that's why we are looking for feedbacks. But, I'll suspect that you encountered none if I am not mistaken. Great!

Sorry about not being able to provide more feedback, since I just used httpie while following a tutorial

Perfect. It's more than enough for a feedback.

So did the fix not get deployed or what? Still getting the exact same error.


FS-8911 Debounce input to allow Mac users to type international characters

Closing this. Meant to submit the PR to a fork.


Add root6.20 and ubuntu 18.04 to CI actions

It is just TEMPORARY version. It is artificially long because
ROOT directory name are not the same in the prepared binaries.
It should be changed.

@alekgajos would it be possible to change the name of the folder to root in the binary package for ROOT6.20?

@alekgajos would it be possible to change the name of the folder to root in the binary package for ROOT6.20?

Done.

@alekgajos ready to be merged


"pool" error with blastn

Hi there,
I'm running the blobtools2 insdc-pipeline on a separate assembly (the first assembly I was mentioning in the first issue & pull requests, actually finished fine).  But this time on this second assembly, I received an error in the blastn step that I didn't see before & seems more like a python / multithreading issue?  See below for logs.  I'm pretty sure the .yaml file is configured correctly, since it was derived from the .yaml that worked already for the other assembly (changing file paths & names of course).
Also worth noting: This was the only insdc-pipeline running & the first one completed some days ago, so not like they're both clobbering the shared database files or something.
WARNING: Use of Snakemake version 5.13.0 is not supported
         INSDC-Pipeline has been developed using version 5.9.1
Building DAG of jobs...
Using shell: /bin/bash
Provided cores: 8
Rules claiming more threads will be scaled down.
Singularity containers: ignored
Job counts:
	count	jobs
	1	all
	1	blobtoolkit_add_hits
	1	extract_nohit_sequences
	1	make_diamond_db
	1	make_masked_lists
	1	run_blastn
	1	run_diamond_blastx
	1	unchunk_blast
	8

[Mon Apr 13 11:52:42 2020]
rule run_blastn:
    input: Ot1p1ph1.chunks.fasta, /lab/weng_scratch/tmp/blobtools2-insdc-pipeline_dbs/ncbi_2020_04/nt.nal, nt.root.1.minus..taxids
    output: Ot1p1ph1.blastn.nt.root.1.minus..nohit, Ot1p1ph1.blastn.nt.root.1.minus..out.raw
    log: logs/Ot1p1ph1/run_blastn/nt.root.1.minus..log
    jobid: 33
    benchmark: logs/Ot1p1ph1/run_blastn/nt.root.1.minus..benchmark.txt
    wildcards: assembly=Ot1p1ph1, name=nt, root=1, masked=.minus.
    threads: 8

python /lab/weng_scratch/Tim/20190410_Ot1.1_gene_structure_annotation/blobtools2/btk_workdir/.snakemake/scripts/tmpchfym7x8.blast_wrapper.py
Activating conda environment: /lab/weng_scratch/Tim/20190410_Ot1.1_gene_structure_annotation/blobtools2/btk_workdir/btk_conda/7f542b78
[Mon Apr 13 11:52:56 2020]
Error in rule run_blastn:
    jobid: 33
    output: Ot1p1ph1.blastn.nt.root.1.minus..nohit, Ot1p1ph1.blastn.nt.root.1.minus..out.raw
    log: logs/Ot1p1ph1/run_blastn/nt.root.1.minus..log (check log file(s) for error message)
    conda-env: /lab/weng_scratch/Tim/20190410_Ot1.1_gene_structure_annotation/blobtools2/btk_workdir/btk_conda/7f542b78

RuleException:
CalledProcessError in line 32 of /lab/weng_scratch/Tim/20190410_Ot1.1_gene_structure_annotation/blobtools2/rules/run_blastn.smk:
Command 'source /lab/solexa_weng/testtube/miniconda3/bin/activate '/lab/weng_scratch/Tim/20190410_Ot1.1_gene_structure_annotation/blobtools2/btk_workdir/btk_conda/7f542b78'; set -euo pipefail;  python /lab/weng_scratch/Tim/20190410_Ot1.1_gene_structure_annotation/blobtools2/btk_workdir/.snakemake/scripts/tmpchfym7x8.blast_wrapper.py' returned non-zero exit status 1.
  File "/lab/weng_scratch/Tim/20190410_Ot1.1_gene_structure_annotation/blobtools2/rules/run_blastn.smk", line 32, in __rule_run_blastn
  File "/lab/solexa_weng/testtube/miniconda3/envs/snake_env/lib/python3.6/concurrent/futures/thread.py", line 56, in run
Shutting down, this might take some time.
Exiting because a job execution failed. Look above for error message
Complete log: /lab/weng_scratch/Tim/20190410_Ot1.1_gene_structure_annotation/blobtools2/btk_workdir/.snakemake/log/2020-04-13T115241.441846.snakemake.log

cat btk_workdir/logs/Ot1p1ph1/run_blastn/nt.root.1.minus..log
2020-04-13 11:52:53,332 [INFO] line 297 Running blast_wrapper
2020-04-13 11:52:55,768 [INFO] line 369 running single BLAST
2020-04-13 11:52:55,768 [INFO] line 239 running BLAST on 26345 sequences in batch 1 of 1
2020-04-13 11:52:56,199 [INFO] line 249 ['blastn', '-lcase_masking', '-outfmt', '6 qseqid staxids bitscore std', '-db', '/lab/weng_scratch/tmp/blobtools2-insdc-pipeline_dbs/ncbi_2020_04/nt', '-evalue', '1e-25', '-max_target_seqs', '10', '-max_hsps', '1', '-taxidlist', 'nt.root.1.minus..taxids', '-num_threads', '8']
2020-04-13 11:52:56,764 [INFO] line 371 calling callback
2020-04-13 11:52:56,764 [INFO] line 345 entering callback
2020-04-13 11:52:56,764 [ERROR] line 339 Command '['blastn', '-lcase_masking', '-outfmt', '6 qseqid staxids bitscore std', '-db', '/lab/weng_scratch/tmp/blobtools2-insdc-pipeline_dbs/ncbi_2020_04/nt', '-evalue', '1e-25', '-max_target_seqs', '10', '-max_hsps', '1', '-taxidlist', 'nt.root.1.minus..taxids', '-num_threads', '8']' returned non-zero exit status 1.
2020-04-13 11:52:56,765 [ERROR] line 384 name 'pool' is not defined
2020-04-13 11:52:56,765 [ERROR] line 385 Unable to process chunks

Here is what my pipeline invocation looks like (from within a symlinked folder w/ symlinks to the insdc-pipeline git repository):
mkdir btk_workdir
CPUS=8
bsub -n ${CPUS} -R "span[hosts=1]" -o snakemake.out.log -e snakemake.err.log "snakemake -p --use-conda --conda-prefix btk_conda --directory btk_workdir/ --configfile btk_workdir/Ot1p1ph1.yaml -j ${CPUS}"

Any ideas?
-Tim

It looks like there is a BLAST error here that isn't getting printed as it is reaching code to close a pool that hasn't been defined. I'll try to catch the call to pool.terminate() then the original BLAST error should get written to the log file and hopefully the cause of the error will be more apparent.

Thanks for the change.  I think this was the probable error:
DESCRIPTION
   Nucleotide-Nucleotide BLAST 2.7.1+

Use '-help' to print detailed descriptions of command line arguments
========================================================================

Error: Unknown argument: "taxidlist"
Error:  (CArgException::eInvalidArg) Unknown argument: "taxidlist"

2020-05-01 11:38:11,978 [ERROR] line 354 Unable to run blastn

There were some path issues where the right blastn wasn't being selected.  Now after a conda install "blast>=2.9.0" "snakemake>=5.13.0" BLAST+ 2.9.0 is being used, which does have the taxidlist parameter.