Cannot download habitat test scenes in Qatar

Habitat-Sim version
v0.2.2
🐛 Bug
The following command gives 403 http response:
python -m habitat_sim.utils.datasets_download --uids habitat_test_scenes --data-path habitat_data
more specifically, the below http request returns 403:
wget --continue http://dl.fbaipublicfiles.com/habitat/objects_v0.2.zip
Steps to Reproduce

Create a conda environment and install habitat-sim as suggested in the README file.
Run python -m habitat_sim.utils.datasets_download --uids habitat_test_scenes --data-path habitat_data

Expected behavior
We should be able to download datasets from the public link.
Additional context
For what it's worth, if there is any regional restriction, I am currently in Qatar. I can download the dataset from my computer in Turkey. Is there some sort of boycott?
System Info
ENVIRONMENT INFO:
Platform: Linux-5.15.0-46-generic-x86_64-with-debian-bookworm-sid
Machine: x86_64
Processor: x86_64
Libc version: glibc 2.10
Mac version:
Python version: 3.7.12
Architecture: 64bit ELF
Win version:
System OS: Linux
Release: 5.15.0-46-generic
Version: 49-Ubuntu SMP Thu Aug 4 18:03:25 UTC 2022
Operational System: linux
GCC version: gcc (Ubuntu 11.2.0-19ubuntu1) 11.2.0
CMAKE version: cmake version 3.14.0
NVIDIA-SMI:
Fri Nov 18 15:50:30 2022
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 510.85.02    Driver Version: 510.85.02    CUDA Version: 11.6     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|                               |                      |               MIG M. |
|===============================+======================+======================|
|   0  NVIDIA GeForce ...  Off  | 00000000:01:00.0 Off |                  N/A |
| N/A   51C    P3    18W /  N/A |    661MiB /  6144MiB |      0%      Default |
|                               |                      |                  N/A |
+-------------------------------+----------------------+----------------------+

Pip packages versions:
Conda packages versions:

Hey @alper111, looking into this. Stay tuned.


Fixed #93 Text Overflowing DIV on Post

Fixed issue #93 Text overflowing on post.

Great. Thank you!


Add breadcrumbs for navigation

Overview of the Feature Request
Add breadcrumbs for navigation within the application.
Features:

Add breadcrumb component to be able to navigate Home > Dataset level > File level (use the breadcrumb from the design system). The breadcrumbs will contain links to the corresponding dataset and to the home
Add the breadcrumb component to the Dataset Page
Add breadcrumb component to the File Page

Mock of the File Page with breadcrumbs

What kind of user is the feature intended for?
All users
What inspired the request?
Frontend meeting discussion about the next user workflows that could extend the SPA to cover more features horizontally without too much detail.
What existing behavior do you want changed?
None
Any brand new behavior do you want to add to Dataverse?
None
Any open or closed issues related to this feature request?

Waiting: needs discussion with @GPortas about API design decissions


Create collections specific for ddo versions

Fixes #207  .
Changes proposed in this PR:

ddoSchemas are created dinamically

Should I also modify database/index.ts for ddo database, where we basically retrieve the first schema, into choosing the schema by DDO version? Or this should be done in #205 ?
For the moment, we load the schemas mappings dinamically into typesense db. Let me know if it's needed in this PR to modify also the CRUD methods from database/index.ts. Thank you!
cc @alexcos20 @jamiehewitt15

Or this should be done in https://github.com/oceanprotocol/ocean-node/issues/205

I see what you mean there is some overlap between the two issues. I think it's fine to do it in the next PR, which keeps this one short and easy to review

If an exception is caught on the outer try/catch (have no idea why we need both but anyway) The function returns nothing, it just logs a message, it should return empty array as well to be consistent tkx

I return now empty array in the outer try/catch block.

Thanks for approvals, but @alexcos20 needs to undo the requested changes status in order to merge it 😅


Added support for multiple AVM files.

For generic_unix, AVM files may be specified on the command line.
For ESP32, a main.avm and lib.avm partition can be loaded.
No behavioral change to STM32.
These changes are made under the terms of the LGPL v2.1 (or any later version)
and Apache 2.0 licenses.

I think I got all the style fixes, and I reverted remove the typedef on struct AVMPackData.


Building macOS version of YT-DLP failed - suggestion

Checklist

[X] I'm asking a question and not reporting a bug/feature request
[X] I've looked through the README
[X] I've read the guidelines for opening an issue
[X] I've searched the bugtracker for similar questions including closed ones

Question
The failure to build the macOS version of the 10/11/21 release might be due to PyInstaller v4.6 which was released on 29/10/21.
Version 4.6 changed the way PyInstaller handles ad-hoc code signing. This was done because of a problem for PyInstaller in macOS 10.13.
Can you specify PyInstaller 4.5.1 for the macOS build of YT-DLP ? At least that did work for the 22/10/21 release.

Yes, this has already been identified here. https://github.com/yt-dlp/yt-dlp/pull/1221#issuecomment-964984731
I just haven't had the time to test and push a patch yet

Thanks. Sorry I didn't find your #1221 discussion before asking.


Allow configuring the associated relation

Closes #2

Amazing 👏 can you add an example in the README for this new feature?

Sure thing!


fix: do not overrite playlist invalidation with false

:tada: This PR is included in version 1.4.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Populator is trying to set private static final long serialVersionUID and fails

My questions are:

Any way to prevent this?
Why setting constants in the first place?
Tried exclusion: how to deal with fields of fields of fields i want to exclude?
Tried @Exclude annotation: how to deal with classes that are coming from external jars?

....
Caused by: java.lang.IllegalAccessException: Can not set static final long field java.time.ZonedDateTime.serialVersionUID to java.lang.Long
at sun.reflect.UnsafeFieldAccessorImpl.throwFinalFieldIllegalAccessException(UnsafeFieldAccessorImpl.java:76)
at sun.reflect.UnsafeFieldAccessorImpl.throwFinalFieldIllegalAccessException(UnsafeFieldAccessorImpl.java:80)
at sun.reflect.UnsafeQualifiedStaticLongFieldAccessorImpl.set(UnsafeQualifiedStaticLongFieldAccessorImpl.java:77)
at java.lang.reflect.Field.set(Field.java:764)
at io.github.benas.randombeans.util.ReflectionUtils.setProperty(ReflectionUtils.java:79)
at io.github.benas.randombeans.PopulatorImpl.populateField(PopulatorImpl.java:143)
at io.github.benas.randombeans.PopulatorImpl.doPopulateBean(PopulatorImpl.java:104)
... 44 more

Hi,
Thank you for pointing out this issue!

Populator is trying to set private static final long serialVersionUID and fails. Any way to prevent this?

This is a regression of issue #1 . I just fixed that, it should be fine now.

Why setting constants in the first place?

Constants are not modified. If the class designer has designed the field to be constant, Random Beans does not touch it.

Tried exclusion: how to deal with fields of fields of fields i want to exclude?

You can use the dotted notation for that. See example here. This has been introduced in issue #24 .

Tried @Exclude annotation: how to deal with classes that are coming from external jars?

Random Beans provides two ways to exclude fields:

Either declaratively through the @Exclude annotation
Or programatically through the API

See the wiki page for details. Using the annotation is of course intrusive and is an option only for classes you can modify (see my comment here). For other classes, you can exclude fields programatically.
Kind regards
Mahmoud

Thanks for the quick response.
As you pointed:

Tried exclusion: how to deal with fields of fields of fields i want to exclude?

You can use the dotted notation for that. See example here. This has been introduced in issue #24 .

I tried that. And it did not work. I could try to debug. but I lack a few examples.
As I said: fields of fields of fields?...
And also how to deal with fields residing in object that are inside a collection i'm trying to populate?

I tried that. And it did not work.

Can you give an example please? This will help to figure out what's going on..

how to deal with fields residing in object that are inside a collection, that i'm trying to populate?

Not sure I understand this point, do you mean fields that are of collection type?

Here're the settings.
class A{
String s1
String s2
}
class B {
A a1
A a2
}
class C {
B b1
B b2
List b3
}

I tried that. And it did not work.
Can you give an example please? This will help to figure out what's going on..

populator.populateBean(C.class, "b2") // please, confirm that it works
populator.populateBean(C.class, "b2.a2") // please, confirm that it works
populator.populateBean(C.class, "b2.a2.s2") // please, confirm that it works

how to deal with fields residing in object that are inside a collection, that i'm trying to populate?
Not sure I understand this point, do you mean fields that are of collection type?

populator.populateBean(C.class, "b3") // please, confirm that it works
populator.populateBean(C.class, "b3.a2") // please, confirm that it works
populator.populateBean(C.class, "b3.a2.s2") // please, confirm that it works
And, one more thing:
It might be nice to have a NoRandomizer = another way to Exclude...
Like that:
	return PopulatorBuilder.aNewPopulatorBuilder()
		.registerRandomizer(Serializable.class,
			Long.class,
			"serialVersionUID",
			new NoRandomizer())
		.build();

Hi Pavel,
I created two separate issues to discuss about:

The problem with excluding fields using the dotted notation ( issue #60 )
The new suggested way of excluding fields (issue #61 )

Let's continue the discussion there.
In regards to this issue, as I said , the population of static final fields should not happen and this is a regression. I have fixed it. It would be great if you give it a try and confirm so that I close the issue.
Regards
Mahmoud

How can I try the fixed version?
I pull from Maven repo... 2.0.0-SNAPSHOT

You need to declare the snapshot repository and add the dependency to the snapshot version:
https://github.com/benas/random-beans#current-version

I cloned from git.
Seems to be working.
All "green" ;)

Great, thank you for taking time to test!


Add more Expressions.arrayPath variants

Fixes #1384

This looks okay. I worry that Expressions is getting too large but we can discuss that elsewhere.

@johnktims @Shredder121 Is this ok to merge?

:+1:

I don't know whether bounded wildcards would be more relaxing, but it looks good.
Maybe we could add a few methods that use them later?
But this PR is good to merge.


Gravity Forms Data Incomplete: Missing submitButton Property Causes Errors in next-gravity-forms

I am using the next-gravity-forms plugin to integrate Gravity Forms into my Next.js application. However, I am encountering an error where the form data is incomplete, specifically missing the submitButton property. This causes the application to throw the following error:
Form data is incomplete or missing submitButton: undefined
    at GravityForm (webpack-internal:///./src/components/GravityForm.js:8:11)
    at div
    at div
    at div

I have verified that the form ID is correct and the form exists in Gravity Forms. I have also logged the fetched form data to the console and confirmed that the submitButton property is missing.
Any guidance on how to ensure the submitButton property is included in the fetched form data would be greatly appreciated. Thank you!

Hey @Socipedia! It would be helpful if you could provide as many details as possible. Which version of the package or plugin are you using? Do you use the GraphQL plugin or the REST API? where does this error come from?

Thank you for your response @Gytjarek. Here are the details of my setup and the error I am encountering:
Package/Plugin Versions:
WPGraphQL: 1.27.2
WPGraphQL for Gravity Forms: 0.13.0
next-gravity-forms: 2.1.0
GraphQL or REST API:
I am using the GraphQL plugin.
"use client";

import GravityFormForm from "next-gravity-forms";

const GravityForm = ({ form }) => {
    console.log('GravityForm data:', form);

    if (!form || !form.submitButton) {
        console.warn('Form data is incomplete or missing submitButton:', form);
        return
Form data is incomplete or missing submitButton
;
    }

    return <GravityFormForm data={form} />;
};

export default GravityForm;

log:
Form data is incomplete or missing submitButton: undefined
I have confirmed that the submitButton field is present in the GraphQL query response:
{
  gfForms {
    nodes {
      id
      title
      databaseId
      submitButton {
        imageUrl
        layoutGridColumnSpan
        location
        text
        type
        width
      }
    }
  }
}

@Socipedia try to log form object and debug it. The data you get from query looks like the following:

"data": {
"gfForm": {
"submitButton": {
"text": "Submit1"
}
}
},

So,submitButton prop is part of an inner property gfForm, and it is not located directly in the data object

@Socipedia do you get the right data by getGravityForm  function.  What do you get in your logs? can you provide form object?

@Gytjarek Currently, the issue has shifted to a confirmation error. Below is the log data:
⨯ TypeError: Cannot read properties of undefined (reading 'confirmations')
    at eval (webpack-internal:///./node_modules/next-gravity-forms/cjs/index-Q4P2mYww.cjs.js:8981:486)

@Socipedia most likely, you are not getting the correct data from your form. We destruct confirmations from the data.gfForm object. If it is undefined or something you get the error. This should be handled in the package, and we'll fix it, but I suggest debugging the data you get using the getGravityForm function

@Gytjarek thx for your respond
Form data: {
  gfForm: {
    databaseId: 1,
    description: 'Contact Us For Detail',
    descriptionPlacement: 'BELOW',
    labelPlacement: 'TOP',
    subLabelPlacement: 'BELOW',
    title: 'Contact Us',
    submitButton: {
      conditionalLogic: null,
      imageUrl: null,
      text: 'HIRE ME!',
      type: 'TEXT'
    },
    confirmations: [ [Object] ],
    hasHoneypot: true,
    formFields: { nodes: [Array] },
    pagination: null
  },
  gfSettings: {
  }
}

previously the error occurred due to my PropTypes for confirmation issue.
then another error appeared to replace it. Is there anything I did wrong in implementing it?
error log:
 ⨯ TypeError: Cannot read properties of undefined (reading 'replace')
    at On (webpack-internal:///./node_modules/next-gravity-forms/cjs/index-Q4P2mYww.cjs.js:3095:44)
    at Pn (webpack-internal:///./node_modules/next-gravity-forms/cjs/index-Q4P2mYww.cjs.js:3216:69)

   9 |   return (
  10 |
 11 |       id={`gfield_description_${wrapId.replace(/^field_/, "")}`}
     |                                        ^
  12 |       className={classNames("gfield_description", className)}
  13 |       dangerouslySetInnerHTML={{ __html: description }}
  14 |     />

@Socipedia good catch, most likely, the issue occurs due to the description in the captcha field. As an option, you could temporarily remove the description from the captcha field or create a PR to fix it in the package itself. Otherwise, you should wait until it gets fixed

@Gytjarek I've fixed this, but another issue reappears, namely every time I submit what always appears is:
"There was a problem with your submission. Please review the fields below.",
There is no problem on the client side, the server side too. looks like it's a reCAPTCHA issue again, I'm using v2 invisible

@Socipedia I'll look into this today

@Socipedia I've tried to set up your form, and it works for me without any errors, including the captcha field. I see you also use a honeypot field. Make sure you don't add a value for that fake field; otherwise, you will get errors.

@Gytjarek thanks for the help, now it works


SyntaxError: Unexpected token =

Describe the bug
After Running openapi -h I get an error,
To Reproduce
Steps to reproduce the behavior:
1.npm install -g @redocly/openapi-cli
2.openapi -h
Expected behavior
openapi-cli options
Logs
`openapi -h
/usr/local/lib/node_modules/@redocly/openapi-cli/node_modules/simple-websocket/server.js:45
_handleListening = () => {
^
SyntaxError: Unexpected token =
at new Script (vm.js:79:7)
at createScript (vm.js:251:10)
at Object.runInThisContext (vm.js:303:10)
at Module._compile (internal/modules/cjs/loader.js:657:28)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:700:10)
at Module.load (internal/modules/cjs/loader.js:599:32)
at tryModuleLoad (internal/modules/cjs/loader.js:538:12)
at Function.Module._load (internal/modules/cjs/loader.js:530:3)
at Module.require (internal/modules/cjs/loader.js:637:17)
at require (internal/modules/cjs/helpers.js:22:18)
`
OpenAPI definition
If applicable, add an OpenAPI definition and .redocly.yaml configuration file that helps reproduce the problem.
Version(s)
v1.0.0-beta.12

Could you also tell, which version of Node.js are you using?
On Tue, 8 Sep 2020, 14:01 Diogo<EMAIL_ADDRESS>wrote:

Describe the bug
After Running openapi -h I get an error,
To Reproduce
Steps to reproduce the behavior:
1.npm install -g @redocly/openapi-cli
2.openapi -h
Expected behavior
openapi-cli options
Logs
`openapi -h
/usr/local/lib/node_modules/@redocly/openapi-cli/node_modules/simple-websocket/server.js:45
_handleListening = () => {
^
SyntaxError: Unexpected token =
at new Script (vm.js:79:7)
at createScript (vm.js:251:10)
at Object.runInThisContext (vm.js:303:10)
at Module._compile (internal/modules/cjs/loader.js:657:28)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:700:10)
at Module.load (internal/modules/cjs/loader.js:599:32)
at tryModuleLoad (internal/modules/cjs/loader.js:538:12)
at Function.Module._load (internal/modules/cjs/loader.js:530:3)
at Module.require (internal/modules/cjs/loader.js:637:17)
at require (internal/modules/cjs/helpers.js:22:18)
`
OpenAPI definition
If applicable, add an OpenAPI definition and .redocly.yaml configuration
file that helps reproduce the problem.
Version(s)
v1.0.0-beta.12
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Redocly/openapi-cli/issues/198, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF3VCPDHHI5PU5FJ3KZS4M3SEYFIHANCNFSM4Q72KM5A
.

If we don't support specific node.js versions, can we please restrict them in package.json using engines?

@diogui , thanks for opening an issue. Based on the error it seems that you are using Node.js with version <= 10. Unfortunately, openapi-cli supports Node.js versions >= 12.
If that's not the case, please let us know.
As for the unsupported Node.js versions, I've added restrictions to the package.json in #201

Thanks for the quick reply. Indeed I'm using v10.15.0 . I'm sorry i didn't know that you only support node.js versions >= 12. I've updated node for the latest version (12.18.3) and now it's working. Thanks guys !


Mp/merge pedants

Merge chef-pedant into oc-chef-pedant. As a bonus, enable knife spec tests.  This is still a WIP, as there is additional cleanup to be done.
ping @chef/lob
Note still some failures around upgrading to rspec 3.2, will be looking into those.

I'm concerned there that by just copying over the files that we're losing the history. It makes this PR really hard to review since it's impossible to tell what changed from the original  chef-pedant to the files that  have been merged in here.

LGTM with one HUGE caveat. I can't possibly read through 18000 LOC. What would be easier to parse for review would have been an un-edited copy from chef-pedant and then the commits that fix up the issues. That way anyone reviewing would be able to see what actual changes were made to the tests after they got moved over.

I started out with that - I didn't preserve original history but did have the original files here.  Unfortunately I lost it in overzealous squashing.  I might still be able to rebuild it from reflog, but  I'm not sure - let me see what I can do.

Superceded by https://github.com/chef/oc-chef-pedant/pull/97, which contains a better show of the deltas in files brought in from chef-pedant.


Null-safe equality for unique_key within delete+insert and merge incremental strategies

Null-safe equality for unique_key within delete+insert and merge incremental strategies with an appropriate null comparison.
resolves https://github.com/dbt-labs/dbt-adapters/issues/159
Problem
The current implementation of the incremental merge does not use null-safe equality, meaning if any of the columns are null, duplicate values will be inserted every time the model is run. This is unlikely to be desired behaviour for anyone.
Solution
The SQL for the delete condition is updated to handle a check for null values.
Checklist

[X] I have read the contributing guide and understand what's expected of me
[X] I have run this code in development and it appears to resolve the stated issue
[X] This PR includes tests, or tests are not required/relevant for this PR
[X] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

@dbeatty10
I appologies that reach you directly but is there news for this PR? If there is additional help or actions required? Could you please tell me wether help is needed and if I can I willl try to help. This PR fixed the issue that is really annoying :(
Thanks in advance!

@amardatar
First of all many thanks for such usefull PR! You provided changes for default__get_delete_insert_merge_sql that implements delete+insert strategy. But there is also additional function default__get_merge_sql that responsible for merge strategy and it has it's own {% if unique_key %}.
Could you please add or is null condition for it also?
Thanks in advance!

@colin-rogers-dbt  Reviewed this and thought about it with @dbeatty10 a bit. We think it looks good.

@peterallenwebb and I took a closer look at this today, and we think there's a couple more test cases to add:

Check null comparison when the unique_key has a single column (also with multiple rows with null values)
Check null comparison when the unique_key has a multiple columns (also with multiple rows with null values)

I will take responsibility for trying out these cases and adding these tests.

[!IMPORTANT]
After decoupling dbt-adapters from dbt-core, only the unit tests run in CI; none of the functional tests run in CI in this repo.
So we'll need to run the changes in this PR along within the functional tests in dbt-postgres (and possibly also dbt-snowflake, dbt-bigquery, and dbt-redshift) in order to know that the changes work as intended.

CI tests against various adapters:

https://github.com/dbt-labs/dbt-postgres/actions/runs/12285466964
https://github.com/dbt-labs/dbt-redshift/actions/runs/12285455823
https://github.com/dbt-labs/dbt-bigquery/actions/runs/12285456249
https://github.com/dbt-labs/dbt-snowflake/actions/runs/12285457041
https://github.com/dbt-labs/dbt-spark/actions/runs/12285457699

Short test summary info for dbt-postgres (8 failed tests)

=========================== short test summary info ============================
FAILED tests/functional/adapter/test_incremental.py::TestIncrementalUniqueKey::test__no_unique_keys - AssertionError: assert 8 == 9
 +  where 8 = ResultHolder(seed_count=1, model_count=1, seed_rows=8, inc_test_model_count=1, opt_model_count=None, relation='seed').seed_rows
 +  and   9 = ResultHolder(seed_count=1, model_count=1, seed_rows=9, inc_test_model_count=1, opt_model_count=None, relation='no_unique_key').seed_rows
FAILED tests/functional/adapter/test_incremental.py::TestIncrementalUniqueKey::test__empty_str_unique_key - AssertionError: assert 8 == 9
 +  where 8 = ResultHolder(seed_count=1, model_count=1, seed_rows=8, inc_test_model_count=1, opt_model_count=None, relation='seed').seed_rows
 +  and   9 = ResultHolder(seed_count=1, model_count=1, seed_rows=9, inc_test_model_count=1, opt_model_count=None, relation='empty_str_unique_key').seed_rows
FAILED tests/functional/adapter/test_incremental.py::TestIncrementalUniqueKey::test__one_unique_key - AssertionError: assert 7 == 8
 +  where 7 = ResultHolder(seed_count=1, model_count=1, seed_rows=7, inc_test_model_count=1, opt_model_count=1, relation='one_str__overwrite').seed_rows
 +  and   8 = ResultHolder(seed_count=1, model_count=1, seed_rows=8, inc_test_model_count=1, opt_model_count=1, relation='str_unique_key').seed_rows
FAILED tests/functional/adapter/test_incremental.py::TestIncrementalUniqueKey::test__empty_unique_key_list - AssertionError: assert 8 == 9
 +  where 8 = ResultHolder(seed_count=1, model_count=1, seed_rows=8, inc_test_model_count=1, opt_model_count=None, relation='seed').seed_rows
 +  and   9 = ResultHolder(seed_count=1, model_count=1, seed_rows=9, inc_test_model_count=1, opt_model_count=None, relation='empty_unique_key_list').seed_rows
FAILED tests/functional/adapter/test_incremental.py::TestIncrementalUniqueKey::test__unary_unique_key_list - AssertionError: assert 7 == 8
 +  where 7 = ResultHolder(seed_count=1, model_count=1, seed_rows=7, inc_test_model_count=1, opt_model_count=1, relation='unique_key_list__inplace_overwrite').seed_rows
 +  and   8 = ResultHolder(seed_count=1, model_count=1, seed_rows=8, inc_test_model_count=1, opt_model_count=1, relation='unary_unique_key_list').seed_rows
FAILED tests/functional/adapter/test_incremental.py::TestIncrementalUniqueKey::test__duplicated_unary_unique_key_list - AssertionError: assert 7 == 8
 +  where 7 = ResultHolder(seed_count=1, model_count=1, seed_rows=7, inc_test_model_count=1, opt_model_count=1, relation='unique_key_list__inplace_overwrite').seed_rows
 +  and   8 = ResultHolder(seed_count=1, model_count=1, seed_rows=8, inc_test_model_count=1, opt_model_count=1, relation='duplicated_unary_unique_key_list').seed_rows
FAILED tests/functional/adapter/test_incremental.py::TestIncrementalUniqueKey::test__trinary_unique_key_list - AssertionError: assert 7 == 8
 +  where 7 = ResultHolder(seed_count=1, model_count=1, seed_rows=7, inc_test_model_count=1, opt_model_count=1, relation='unique_key_list__inplace_overwrite').seed_rows
 +  and   8 = ResultHolder(seed_count=1, model_count=1, seed_rows=8, inc_test_model_count=1, opt_model_count=1, relation='trinary_unique_key_list').seed_rows
FAILED tests/functional/adapter/test_incremental.py::TestIncrementalUniqueKey::test__trinary_unique_key_list_no_update - AssertionError: assert 8 == 9
 +  where 8 = ResultHolder(seed_count=1, model_count=1, seed_rows=8, inc_test_model_count=1, opt_model_count=None, relation='seed').seed_rows
 +  and   9 = ResultHolder(seed_count=1, model_count=1, seed_rows=9, inc_test_model_count=1, opt_model_count=None, relation='nontyped_trinary_unique_key_list').seed_rows
============ 8 failed, 569 passed, 12 skipped in 472.15s (0:07:52) =============

It looks like there are similar test failures across all the adapters I tested against.
I'm not sure if these failures are because the tests are making a simplifying assumption that isn't true as it relates to NULLs, of if there is something off about the implementation.
So it looks like either the implementation will need to be updated, or the these test cases will need to be updated.

Hey @dbeatty10 had a look - is it because we've added actual rows to the seeds, and needed to update the seed_rows param? I would've missed that, didn't realise that's how it worked - but have pushed a change.


Public images

Exposes another set of public images under http://downloads.openmicroscopy.org/images/

Do you already have a test run? In general, seems fine, but some files are missing no?
[jamoore@idr0-slot3 curated]$ find . -maxdepth 3 -name public
./olympus-oir/public
./tiff/public
./amira/public
./trestle/public
./hamamatsu-vms/public
./perkinelmer-operetta/public
./vectra-qptiff/public
./nd2/public
./mrc/public
./ecat7/public
./hamamatsu/public
./svs/public
./micromanager/public
./incell3000/public
./cellomics/public
./png/public
./nrrd/public
./imaris/public
./incell/public
./leica-scn/public
./deltavision/public
./nifti/public
./becker-hickl-spc/public
./dicom/public
./leica-lif/public
[jamoore@idr0-slot3 curated]$ find incell/public/ leica-scn/public/ mrc/public/ perkinelmer-operetta/public/ trestle/public/ -iname readme.txt -o -iname COPYING
trestle/public/openslide/readme.txt
trestle/public/openslide/COPYING

Additional readme/licensing added to the various folders
[sbesson@idr0-slot3 curated]$ find incell/public/ leica-scn/public/ mrc/public/ perkinelmer-operetta/public/ trestle/public/ -iname readme.txt -o -iname COPYING
leica-scn/public/openslide/readme.txt
mrc/public/EMDB/readme.txt
trestle/public/openslide/readme.txt
trestle/public/openslide/COPYING

and run the playbook in check-mode
(ansible) sbesson@lifesci-80612:prod (master_merge_trigger) $ ansible-playbook -i ../ansible/inventory/prod-hosts --ask-become -C playbooks/release/public-images.yml
SUDO password:

PLAY [idr0-slot3.openmicroscopy.org] **************************************************************************************************************

TASK [Gathering Facts] ****************************************************************************************************************************
ok: [idr0-slot3.openmicroscopy.org]

TASK [check public images] ************************************************************************************************************************
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'AmiraMesh', u'src': u'../../../repos/curated/amira/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'Cellomics', u'src': u'../../../repos/curated/cellomics/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'DICOM', u'src': u'../../../repos/curated/dicom/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'DV', u'src': u'../../../repos/curated/deltavision/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'ECAT7', u'src': u'../../../repos/curated/ecat7/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'gateway_tests/big.tiff', u'src': u'../../../../repos/curated/samples/carlos/big.tiff'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'gateway_tests/CHOBI_d3d.dv', u'src': u'../../../../repos/curated/samples/ome/CHOBI_d3d.dv'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'gateway_tests/tinyTest.d3d.dv', u'src': u'../../../../repos/curated/samples/ome/tinyTest.d3d.dv'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'Hamamatsu-NDPI', u'src': u'../../../repos/curated/hamamatsu/public'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'Hamamatsu-VMS', u'src': u'../../../repos/curated/hamamatsu-vms/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'HCS/BBBC', u'src': u'../../../../repos/curated/cellomics/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'HCS/INCELL2000', u'src': u'../../../../repos/curated/incell/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'HCS/Operetta', u'src': u'../../../../repos/curated/perkinelmer-operetta/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'Imaris-IMS', u'src': u'../../../repos/curated/imaris/public/'})
changed: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'InCell2000', u'src': u'../../../repos/curated/incell/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'InCell3000', u'src': u'../../../repos/curated/incell3000/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'Leica-LIF', u'src': u'../../../repos/curated/leica-lif/public/'})
changed: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'Leica-SCN', u'src': u'../../../repos/curated/leica-scn/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'Micro-Manager', u'src': u'../../../repos/curated/micromanager/public/'})
changed: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'MRC', u'src': u'../../../repos/curated/mrc/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'ND2', u'src': u'../../../repos/curated/nd2/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'NIfTI', u'src': u'../../../repos/curated/nifti/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'NRRD', u'src': u'../../../repos/curated/nrrd/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'Olympus-OIR', u'src': u'../../../repos/curated/olympus-oir/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'OME-TIFF', u'src': u'../../../repos/curated/ome-tiff/ome-schemas/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'OME-XML', u'src': u'../../../repos/curated/ome-xml/ome-schemas/'})
changed: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'Operetta', u'src': u'../../../repos/curated/perkinelmer-operetta/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'PNG', u'src': u'../../../repos/curated/png/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'SPC-FIFO', u'src': u'../../../repos/curated/becker-hickl-spc/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'SVS', u'src': u'../../../repos/curated/svs/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'TIFF', u'src': u'../../../repos/curated/tiff/public/'})
changed: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'Trestle', u'src': u'../../../repos/curated/trestle/public/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'u-track', u'src': u'../../../repos/curated/zip/u-track/'})
ok: [idr0-slot3.openmicroscopy.org] => (item={u'dest': u'Vectra-QPTIFF', u'src': u'../../../repos/curated/vectra-qptiff/public/'})

PLAY RECAP ****************************************************************************************************************************************
idr0-slot3.openmicroscopy.org : ok=2    changed=1    unreachable=0    failed=0

folder like InCell, PerkinElmer will benefit from a readme
containing info on how to load the file in batch

@jburel agreed although I think such information is really generic for any format with a lot of data. My concern with a per-format readme is that it is not scalable or maintenable. Other suggestions could include putting such a files:

under https://downloads.openmicroscopy.org/images
under https://downloads.openmicroscopy.org/
under https://www.openmicroscopy.org in the context of a page promoting these public samples

In terms of the command, the best I have found with the current structure would be a recursive wget e.g.
wget -r --mirror -np --wait 5 -e --reject="index.html*" -e robots=off http://downloads.openmicroscopy.org/images/PNG/will/WesternBlots/

Briefly following on the conversation above, https://github.com/openmicroscopy/prod-playbooks/pull/72#issuecomment-398745301 certainly needs to be addressed as the burden is on consumers at the moment for large filesets. Options can range from better instructions under downloads.openmicroscopy.org or the main OME website to complete alternative methods (Zenodo, public buckets...).
Solving the exposition and downloading of public images is outwith the initial scope of this PR and most likely an effort that should be scoped by a team. Assuming the last commit addresses the outstanding issues raised https://github.com/openmicroscopy/prod-playbooks/pull/72#issuecomment-395729764, merging and deploying for indidivual MRC, Leica SCN and Trestle samples.


Stream line-numbering is broken

ag doesn't handle stream line-numbering very well, particularly in cases where the pattern matches an empty line:
# From file — works more or less as expected
# (though it seems inconsistent to treat the last new-line as a fifth line)
dreameater:/tmp % printf '\nfoo\n\nbar\n' > tempfile
dreameater:/tmp % cat tempfile

foo

bar
dreameater:/tmp % command ag --nocolor --numbers '.' tempfile
1:
2:foo
3:
4:bar
5:%

# From STDIN — the number of each line is prepended to the number of the next
dreameater:/tmp % printf '\nfoo\n\nbar\n' | command ag --nocolor --numbers '.'
1:
1:2:foo
2:3:
3:4:bar
4:%

# From FIFO — no idea what's going on here
dreameater:/tmp % command ag --nocolor --numbers '.' <( printf '\nfoo\n\nbar\n' )
1:
2:
1:foo
2:
1:
2:
1:bar
2:%

I tried to fix this myself but i can't seem to wrap my mind around it. :/

The first two are fixed in https://github.com/ggreer/the_silver_searcher/commit/08b7b0aa3e6b72e9c8e9d62777cacf6cf3f8507a.
Starting with 7fa435a106859864ad2b1b1e86ff74961ec94f03, ./ag --nocolor --numbers '.' <( printf '\nfoo\n\nbar\n' ) gives:
1:
2:
4:o
5:
7:8:
10:r

The case is that bash passes <(...) as a /dev/fd/N argument;
ag then sees the arg, and sets the opts.search_stream to 0.
Remark: this is to allow several <(...) redirections to be passed to a single command.
As search_stream affects any search, ag would scan character device /dev/fd/N (aka stdin) as ordinary file.
The fix would be to rework the is-stream option to be file-local and handle this case.
However, bothering seems not to be worth it (what would be the use case).

First two are fixed in 08b7b0aa3e6b72e9c8e9d62777cacf6cf3f8507a.
The FIFO case is because ag cannot handle character devices (and for <(...) bash passes one).


[SPARK-21625][DOC] Add incompatible Hive UDF describe to DOC

What changes were proposed in this pull request?
Add incompatible Hive UDF describe to DOC.
How was this patch tested?
N/A

Test build #80200 has started for PR 18833 at commit f346f5b.

Pg does not accept this query;
postgres=# select sqrt(3);
       sqrt
------------------
 1.73205080756888
(1 row)

postgres=# select sqrt(-1);
ERROR:  cannot take square root of a negative number

So, Another solution is to fail in analysis.

Test build #80200 has finished for PR 18833 at commit f346f5b.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
abstract class UnaryNonnegativeExpression(f: Double => Double, name: String)
case class Sqrt(child: Expression) extends UnaryNonnegativeExpression(math.sqrt, \"SQRT\")

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80200/
Test PASSed.

Test build #80214 has started for PR 18833 at commit 3e9ec8c.

Test build #80214 has finished for PR 18833 at commit 3e9ec8c.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Sqrt(child: Expression) extends UnaryMathExpression(math.sqrt, \"SQRT\")

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80214/
Test PASSed.

@maropu that only works for literals. I am sort-of in favor of the Hive default; it seems kinda bad to bring down a job because of negative value.

@hvanhovell yea, both is okay to me. just suggestion.

Can we document this difference in https://spark.apache.org/docs/latest/sql-programming-guide.html#compatibility-with-apache-hive?

Test build #83203 has started for PR 18833 at commit cbbfa5e.

Test build #83203 has finished for PR 18833 at commit cbbfa5e.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Sqrt(child: Expression) extends UnaryMathExpression(math.sqrt, \"SQRT\")

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83203/
Test PASSed.

Merged build finished. Test PASSed.

retest this please

Test build #83461 has started for PR 18833 at commit cbbfa5e.

Test build #83461 has finished for PR 18833 at commit cbbfa5e.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Sqrt(child: Expression) extends UnaryMathExpression(math.sqrt, \"SQRT\")

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83461/
Test PASSed.

Merged build finished. Test PASSed.

Test build #83466 has started for PR 18833 at commit 07cedb5.

Test build #83466 has finished for PR 18833 at commit 07cedb5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83466/
Test PASSed.

Merged build finished. Test PASSed.

Thanks! Merged to master.


Cluster and edit display issues

When performing cluster and edit on a big enough dataset, some items are pushed out of the dialog and squished.
Current Results

Expected Behavior
The contents should fit in the dialog and be laid out properly.

There's a lot of options here.

do we make the right hand side with sliders larger to allow more slider room by default?
Thereby moving the right hand side of table with scrollbars to be a smaller table and let users with long string use the scrollbars more?
A little of both?

This is a long term and very annoying issue, thanks for tackling it @elebitzero, I am sure you will come up with something great :)


cssmin breaks calc() by stripping spaces

Issue is logged in the cssmin project here:
https://github.com/zacharyvoase/cssmin/issues/16

Best solution would probably be to generate the minified version with lessc instead of the python package.

@onyxfish @TylerFisher does this look okay?
https://github.com/nprapps/austin/commit/59043837cbbfa02644098ea239333039408b9892

Looks great!

If you're using cssmin and want to get around this issue without having to switch to another minifier you're able to change your CSS from this:
width: calc(100% + 2px);
to this:
width: calc(100% - -2px);
Not ideal, but it works.


update and enable additional linters and fix the resulting complaints

by either

silencing the linter where appropriate
or fixing an actual issue

Signed-off-by: Josh Gwosdz<EMAIL_ADDRESS>

/retest
another ci flake :(

/hold

agreed to fix the code so we need as few linter silences as possible

/unhold

/lgtm

/lgtm


Uncaught TypeError: Cannot set property 'endIdx' of undefined

steps to reproduce:

highlight a string ("asdf")
try to trigger wrap-around-parens / ctrl-alt-(
=> have to hit ctrl-alt-shift-8 on my keyboard

Atom Version: 1.0.3
System: Mac OS X 10.10.3
Thrown From: lisp-paredit package, v0.4.0
Stack Trace
Uncaught TypeError: Cannot set property 'endIdx' of undefined
At /Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:756

TypeError: Cannot set property 'endIdx' of undefined
  at /Users/jluf/.atom/packages/lisp-paredit/lib/edit-commands.coffee:167:9
  at TextBuffer.module.exports.TextBuffer.transact (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:751:18)
  at TextEditor.module.exports.TextEditor.transact (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:1151:26)
  at edit (/Users/jluf/.atom/packages/lisp-paredit/lib/edit-commands.coffee:159:10)
  at Array.module.exports.wrapAroundParens (/Users/jluf/.atom/packages/lisp-paredit/lib/edit-commands.coffee:48:5)
  at atom-text-editor.<anonymous> (/Users/jluf/.atom/packages/lisp-paredit/lib/utils.coffee:38:70)
  at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:241:29)
  at /Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:61
  at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:524:16)
  at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:347:22)
  at HTMLDocument.module.exports.WindowEventHandler.onKeydown (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:177:20)

Commands
     -0:50.5.0 emmet:next-edit-point (atom-text-editor.editor.is-focused)
  2x -0:49.8.0 core:move-up (atom-text-editor.editor.is-focused)
     -0:48.8.0 emmet:next-edit-point (atom-text-editor.editor.is-focused)
     -0:48.6.0 emmet:prev-edit-point (atom-text-editor.editor.is-focused)
     -0:48.2.0 core:move-down (atom-text-editor.editor.is-focused)
  7x -0:47.9.0 core:move-right (atom-text-editor.editor.is-focused)
     -0:46.4.0 emmet:next-edit-point (atom-text-editor.editor.is-focused)
     -0:45.6.0 emmet:wrap-with-abbreviation (atom-text-editor.editor.is-focused)
     -0:43.5.0 editor:consolidate-selections (atom-text-editor.editor.mini.is-focused)
     -0:43.5.0 core:cancel (atom-text-editor.editor.mini.is-focused)
  2x -0:34.7.0 lisp-paredit:splice (atom-text-editor.editor.is-focused)
  2x -0:26 core:undo (atom-text-editor.editor.is-focused)
     -0:22.5.0 lisp-paredit:splice (atom-text-editor.editor.is-focused)
     -0:15.2.0 core:undo (atom-text-editor.editor.is-focused)
     -0:07.2.0 emmet:wrap-with-abbreviation (atom-text-editor.editor.is-focused)
     -0:01.1.0 lisp-paredit:wrap-around-parens (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "disabledPackages": [
      "copy-config-info",
      "copy-config-info"
    ],
    "themes": [
      "atom-light-ui",
      "atom-light-syntax"
    ]
  },
  "lisp-paredit": {
    "strict": false
  }
}

Installed Packages
# User
color-picker, v2.0.11
editorconfig, v1.0.1
emmet, v2.3.12
lisp-paredit, v0.4.0
minimap, v4.12.1
pigments, v0.9.2

# Dev
No dev packages

seem like it has to do with the highlighting:

set the cursor somewhere and trigger wrap-around-parens => ok
select some text and trigger wrap-around-parens => error

Pushed 0.4.1 with this fix.
Thanks for raising the issue.


Fix #1813: Support calling the super ctor of a JS class with : _*.

The only test for this is currently disabled (commented out), because the ECMAScript 6 output emitted for this code cannot be parsed by io.js at the moment.

Review by @gzm0
I tested locally the ES 5 version before commenting out the test in the second commit.

And with the third commit, I tested locally the ES 6 output on Chrome Canary.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/2134/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2758/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/2136/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2760/
Test PASSed.

Could you reference #1876 in the first commit, please?

That's all. Also, this seems to need a rebase after the objects PR

Could you reference #1876 in the first commit, please?

OK. The issue did not exist yet when I made the commit.

Updated.

LGTM

Thanks :)

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/2140/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2765/
Test PASSed.


nyxxie's portfolio

added personal portfolio

Good work. I am merging your PR.
Do give us a star by hitting the star button of this repo!


BAT tests assume a single tenant

This is bad as

The BAT tests will fail if there are multiple tenants
Some of the BAT tests delete things.

This fix is to modify bat tests to create test tenants. We are dependent on:
https://github.com/gophercloud/gophercloud/issues/157
for this.


input text to phonemize() is str but it must be list of str

Hi, can you help me. I'm trying this code on google colab  and I'm getting this error message.
Your demo code:
c, c_length, phoneme = nix.tokenize("Born to multiply, born to gaze into night skies.")
error:
input text to phonemize() is str but it must be list of str
I change the line like this:
c, c_length, phoneme = nix.tokenize(["hi"])
error:
'list' object has no attribute 'lower'

Hi! Turns out the problem comes from the wrong phonemizer version, phonemizer==2.2.1 is the one that should be installed. I've updated the requirements.txt too.


Haxelib from Homebrew on macOS High Sierra 10.13.6: "Could not link to neko"

On my machine (Haxe 3.4.7 and Haxelib 3.3.0 from Homebrew on macOS High Sierra, Neko 2.2.0 from Homebrew too), Haxelib prefixes the output of every haxelib run command with the text:
Could not link to neko.
I have also observed that it makes it impossible to build projects with OpenFL, as build commands:
$ haxelib run openfl build macos
give the following output:
Could not link to neko.
Warning: Ignoring requested haxelib "openfl" version "3.6.1" (version "8.4.1" was already included)
Could not link plugin to process (CFFILoader.h 203) - with neko

This error is not related to haxelib run, it happens when lime.ndll tries to load libneko.dylib. Similar issue here: https://github.com/openfl/lime/issues/1750


updated emails to gapsummit.com

update lkaa2.cam.ac.uk to<EMAIL_ADDRESS>

would be good to get these merged so we can publicise the trailer. hope youre having a good holiday

Hi Lisa,
I've merged the changes and added the youtube video (you'll have to
uncomment it).
Note, you can't just copy and paste the entire youtube link from your
browser:
src="//www.youtube.com/embed/-IcGvBE5xfg?showinfo=0&rel=0"
You only have to change the highlighted part in the html file.
Cheers,
Croydon
On Sun, May 3, 2015 at 7:54 PM, Lisa Altmann-Richer <
<EMAIL_ADDRESS>wrote:

would be good to get these merged so we can publicise the trailer. hope
youre having a good holiday
—
Reply to this email directly or view it on GitHub
https://github.com/croydondias/gapsummit-site/pull/19#issuecomment-98522770
.

Thanks very much Croydon,
They don’t seem to have gone live yet, but hopefully it’ll update to the live site tonight?
Once that’s done i’ll publish the YouTube video so thanks for setting up a template on the site
Best,
Lisa
On 5 May 2015 at 22:37:08, Croydon Dias<EMAIL_ADDRESS>wrote:
Hi Lisa,
I've merged the changes and added the youtube video (you'll have to
uncomment it).
Note, you can't just copy and paste the entire youtube link from your
browser:
src="//www.youtube.com/embed/-IcGvBE5xfg?showinfo=0&rel=0"
You only have to change the highlighted part in the html file.
Cheers,
Croydon
On Sun, May 3, 2015 at 7:54 PM, Lisa Altmann-Richer <
<EMAIL_ADDRESS>wrote:

would be good to get these merged so we can publicise the trailer. hope
youre having a good holiday
—
Reply to this email directly or view it on GitHub
https://github.com/croydondias/gapsummit-site/pull/19#issuecomment-98522770
.

—
Reply to this email directly or view it on GitHub.

Sorry about that, I missed a step. The latest copy is shown on gapsummit.com
now. :)
On Wed, May 6, 2015 at 7:23 AM, Lisa Altmann-Richer <
<EMAIL_ADDRESS>wrote:

Thanks very much Croydon,
They don’t seem to have gone live yet, but hopefully it’ll update to the
live site tonight?
Once that’s done i’ll publish the YouTube video so thanks for setting up a
template on the site
Best,
Lisa
On 5 May 2015 at 22:37:08, Croydon Dias<EMAIL_ADDRESS>wrote:
Hi Lisa,
I've merged the changes and added the youtube video (you'll have to
uncomment it).
Note, you can't just copy and paste the entire youtube link from your
browser:
src="//www.youtube.com/embed/-IcGvBE5xfg?showinfo=0&rel=0"
You only have to change the highlighted part in the html file.
Cheers,
Croydon
On Sun, May 3, 2015 at 7:54 PM, Lisa Altmann-Richer <
<EMAIL_ADDRESS>wrote:

would be good to get these merged so we can publicise the trailer. hope
youre having a good holiday
—
Reply to this email directly or view it on GitHub
<
https://github.com/croydondias/gapsummit-site/pull/19#issuecomment-98522770

.

—
Reply to this email directly or view it on GitHub.
—
Reply to this email directly or view it on GitHub
https://github.com/croydondias/gapsummit-site/pull/19#issuecomment-99341303
.

thanks very much.
i may spot a couple of other changes that need making and do the updates tonight, but I’m sure you’ll get a pull request
Thanks for doing all this on your holiday. Enjoy the break!
best,
lisa
--
Lisa Altmann-Richer
Sent with Airmail
On 6 May 2015 at 09:22:26, Croydon Dias<EMAIL_ADDRESS>wrote:
Sorry about that, I missed a step. The latest copy is shown on gapsummit.com
now. :)
On Wed, May 6, 2015 at 7:23 AM, Lisa Altmann-Richer <
<EMAIL_ADDRESS>wrote:

Thanks very much Croydon,
They don’t seem to have gone live yet, but hopefully it’ll update to the
live site tonight?
Once that’s done i’ll publish the YouTube video so thanks for setting up a
template on the site
Best,
Lisa
On 5 May 2015 at 22:37:08, Croydon Dias<EMAIL_ADDRESS>wrote:
Hi Lisa,
I've merged the changes and added the youtube video (you'll have to
uncomment it).
Note, you can't just copy and paste the entire youtube link from your
browser:
src="//www.youtube.com/embed/-IcGvBE5xfg?showinfo=0&rel=0"
You only have to change the highlighted part in the html file.
Cheers,
Croydon
On Sun, May 3, 2015 at 7:54 PM, Lisa Altmann-Richer <
<EMAIL_ADDRESS>wrote:

would be good to get these merged so we can publicise the trailer. hope
youre having a good holiday
—
Reply to this email directly or view it on GitHub
<
https://github.com/croydondias/gapsummit-site/pull/19#issuecomment-98522770

.

—
Reply to this email directly or view it on GitHub.
—
Reply to this email directly or view it on GitHub
https://github.com/croydondias/gapsummit-site/pull/19#issuecomment-99341303
.

—
Reply to this email directly or view it on GitHub.


How to parameterize TO clause when creating materialized view?

I'm using<EMAIL_ADDRESS>&<EMAIL_ADDRESS>I'm trying to create a table, then a materialized view. The commands & their params look like this, and this works fine:
{
    createTargetTableSql: `
        CREATE TABLE IF NOT EXISTS {targetTableName: Identifier}
        (
            "domain_name" String,
            "month" Date,
            "sumCountViews" AggregateFunction(sum, Int64)
        )
        ENGINE = AggregatingMergeTree
        ORDER BY (domain_name, month)
    `,
    createMaterializedViewSql: `
        CREATE MATERIALIZED VIEW IF NOT EXISTS {materializedViewName: Identifier}
        TO MonthlyAggregatedData_0_0
        AS SELECT
            toDate(toStartOfMonth(event_time)) AS month,
            domain_name,
            sumState(count_views) AS sumCountViews
        FROM {baseTableName: Identifier}
        GROUP BY
            domain_name,
            month
    `,
    query_params: {
        baseTableName: "HourlyData_0_0",
        targetTableName: "MonthlyAggregatedData_0_0",
        materializedViewName: "MonthlyAggregatedDataMv_0_0"
    }
}

If I change TO MonthlyAggregatedData_0_0 into TO {targetTableName: Identifier}, I get this error:
Both table name and UUID are empty.

If I change TO MonthlyAggregatedData_0_0 into TO {targetTableName: String}, I get this error:
Syntax error: failed at position 110 ('}') (line 2, col 36): }
        AS SELECT
            toDate(toStartOfMonth(event_time)) AS month,
            domain_name,
            sumState(count_views) AS sumCountViews
       . Expected substitution type (identifier).

Is there a specific way to parameterize the TO clause when creating a materialized view?
Please let me know if I can provide more info. Thanks in advance.

I think the minimal repro is this (the client sends the requests similar to the following curls):
curl "http://localhost:8123?param_target=mv_target" \
--data-binary "CREATE OR REPLACE TABLE {target: Identifier} (i Int32) ENGINE MergeTree ORDER BY i;"

(adding an extra src table as it was missing from the OP)
curl "http://localhost:8123?param_target=mv_src" \
--data-binary "CREATE OR REPLACE TABLE {src: Identifier} (i Int32) ENGINE MergeTree ORDER BY i;"

Creating the MV:
curl "http://localhost:8123?param_target=mv_target&param_src=mv_src&param_mv_name=mv" \
--data-binary "CREATE MATERIALIZED VIEW {mv_name: Identifier} TO {mv_target: Identifier} AS SELECT * FROM {mv_src: Identifier};"

Which fails with the same error:
Code: 60. DB::Exception: Both table name and UUID are empty. (UNKNOWN_TABLE) (version <IP_ADDRESS>72 (official build))

I will clarify with the team if it's possible to parametrize the TO ... clause as of now.

It is confirmed that, unfortunately, this is a current limitation of ClickHouse.

@DatGuyJonathan, here's the issue in the main repo: https://github.com/ClickHouse/ClickHouse/issues/62892

Thank you @slvrtrn!


chore(meta): bump sea-orm to 1.0

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Bump sea-orm to 1.0, along with its dependencies.
The motivation is https://github.com/risingwavelabs/risingwave/pull/19156, where the necessary fix (https://github.com/SeaQL/sea-orm/pull/2244) is only included since 1.0.
The only worth-noting breaking change is that, there is no longer a backend-agnostic method to specify longblob for MySQL, blob for SQLite, and bytea for Postgres simultaneously (https://github.com/SeaQL/sea-query/pull/743, a regression to me). As a workaround, add a helper method which dispatches the type based on the backend variant.
Although the latest version of sea-orm is 1.1, we are not upgrading to it in this PR because it depends on a new major version of sqlx, for which we are maintaining our own forks to make it compatible with madsim. We may further bump it in following PRs.
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

#19145 🥵

I think we made similar changes. Shall I include your changes in my PR? I think the trait extension changes are more ergonomic.

#19145 🥵

I think we made similar changes. Shall I include your changes in my PR? I think the trait extension changes are more ergonomic.

I think we can also merge in your PR, then my PR will just contain Cargo.lock and Cargo.toml patches.

#19145 🥵

I think we made similar changes. Shall I include your changes in my PR? I think the trait extension changes are more ergonomic.

I think we can also merge in your PR, then my PR will just contain Cargo.lock and Cargo.toml patches.

Sorry, I just noticed your work. 🥵


Skeleton autoinsert template is active on all .el files

Whenever I open a new .el file emacs offers to "Perform Rational Emacs Lisp Skeleton auto-insertion? (y or n)".
But as I understand the form for autoinsertion, it seems to be offering it only on files wich names starts with "rational-" and ends in ".el", not only the latter.
I still can choose to not accept the offering, but I wonder if it would interfer if some one wants his/hers own auto insertion forms.
Any way — Is this the intended behaviour for this skeleton, or is this a bug?

Autoinsert is on, but there are 2 different skeleton templates for .el files. 1 is in the init file for rational-*.el files, there is another built in for all other *.el files.
If you open a brand new emacs lisp file, and it's name matches the rational-.*\.el regular expression, you see the prompt to insert a description for a rational-emacs module. If you open a brand new file which does not match that regular expression, you see the prompt for a "Short description" which is for any other emacs lisp file.
This is the normal behavior when auto-insert-mode is turned on. If you open a brand new file for another programming language (for example C++) you'll see a different prompt and a different insertion (see auto-insert-alist for the list of default templates). Answering n (or no if you are using yes-or-no-p) for the first prompt will abort the auto insert.
If you wish to turn this off, in your rational-config-path/config.el you can use (auto-insert-mode -1).
In Emacs, C-h d m Autotype RET i autoinserting RET should get you to the info page. Alternatively C-h d m Autotype RET 5 should do approximately the same thing. You can also find information on the Skeleton Language in that same (AutoType) Info node (C-h d m Autotype RET 1)
Hope this helps!

Yea, you are right.
But, still it tries to auto-insert the template on any .el file under my ~/.config/rational-emacs/ directory. I don't see why it should do that. I know that it is matching the regexp, but my cuestion is: Why would my personal configuration be treated as a module for rational-emacs?

Ok, I think I take it back. This might be a bug. Assume you have a path like this: ~/.config/rational-emacs/ and in that path you add the file foo.el. The regular expression matches rational-emacs/foo.el and assumes you mean this is a rational-emacs module. I need to fix the regular expression. I'll work on it.
My understanding was the regular expression was applied against the file name only, however it seems to include the entire path to the file as part of the file name.

@abarocio80 I have pushed a PR to resolve this issue. However, if you want to test it, please pull from my repo.
Thanks for the report!


List unmapped provinces

The converter should have a list of what provinces are still unmapped at the end and if you click them you should start a mapping with them, while the camera goes on the location of the province on both maps.
This will make impossible forgetting provinces.


allow the user to set allowExitOnIdle

I've been using this for a project and I got tired of carrying the patch around.

@dynajoe Love your repo, but man do open handles drive up a lot of issues / unresolved comments on google via stackoverflow etc for test coverage. This update worked for our Node/TypeScript with Express project. Thanks @sorpigal!!!


[spaceship] 403 error clash with Portal and ConnectAPI

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description

When calling Spaceship::ConnectAPI::App.find("dk.dinsen.secret.app") using the API Key on an account that have license agreements waiting to be accepted.
I get this error: undefined method 'status' for nil:NilClass
I dived into the code and found that with Spaceship::Portal the error code 403 is handled here:
https://github.com/fastlane/fastlane/blob/master/spaceship/lib/spaceship/client.rb#L896-L899
But because of this line: raise AccessForbiddenError.new, msg it goes into a loop where it makes 5x5 retries.
And after that it throw this error: undefined method 'status' for nil:NilClass
If I comment out this line raise AccessForbiddenError.new, msg then I'm able to add this into the APIClient.handle_erros and catch the error I'm looking for:
if response.body['errors'].first['code'].eql?("FORBIDDEN.REQUIRED_AGREEMENTS_MISSING_OR_EXPIRED")
  raise ProgramLicenseAgreementUpdated, handle_errors(response) if response && (response.body || {})['errors']
else
  raise AccessForbiddenError, handle_errors(response) if response && (response.body || {})['errors']
end

I have not been able to catch the 403 using Spaceship::Portal since the Developer Portal is showing the AppleIDAndPrivacyAcknowledgementNeeded when trying to login.
So I'm not able to find a solution where we can handle this 403 status for both Portal and ConnectAPI

I guess you are using an Apple ID without 2FA yet?

Yes, I currently don't have access to an account with 2FA enabled and licenses needs to be accepted.

Not sure if it makes sense to solve the problem of "hacking away" 2 error messages. Do you also get the same timeout when trying to do the same with a 2FA-enabled Apple ID?

The 2 error messages is just to catch the error type. Something we could use in the company, so we can avoid using Spaceship::Portal.fetch_program_license_agreement_messages and avoid using session login.
The actually problem is if you use the API Key and the license is not accepted, Spaceship will throw this error: undefined method 'status' for nil:NilClass
It's happening because the Portal client is returning raise AccessForbiddenError.new, msg.
And since the ConnectAPI client is not getting the response from the API, but the AccessForbiddenError error. Spaceship fails with undefined method 'status' for nil:NilClass.
How can we pass the API response to ConnectAPI in case of a 403, and still handle the 403 error for Portal as it is now?
I don't have an account with 2FA enabled and licenses needs to be accepted. So I'm not able to find a solution myself.

Closing this, as https://github.com/fastlane/fastlane/pull/18299 was merged 🚀


Plasma 6: Application Menu Does not Refresh List when Applications added/removed

Describe the bug
When applications are added or removed, the menu stays with the current list from when I originally installed Plasma. Reboots or restarts does not refresh it.
Steps To Reproduce
Steps to reproduce the behavior:

Install Plasma
Add or remove applications (in my case within nixos config)
Menu keeps old list

Expected behavior
Application menu refreshed with current available applications

Add a :+1: reaction to issues you find important.

The problem seems to be that when a new ksycoca6_..... file is created in the ~/.cache, plasma just completely ignores it and happily continues to use whichever one it has been using. A workaround (for now) is to manually run rm ~/.cache/ksycoca* (or add it to your activation scripts, lol) after every nixos-rebuild, and this should take care of things.
Still, it'd be nice to know if this is a NixOS issue, or an upstream issue. Feels like it could be either?

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/enable-plasma-6/40541/7

How would I go about getting the kbuildsyscoca6 binary from inside the "activationScript"? I can't seem to find the correct ${whatever}

A workaround (for now) is to manually run rm ~/.cache/ksycoca* (or add it to your activation scripts, lol) after every nixos-rebuild, and this should take care of things.

When I add it to my activation scripts, it does remove it, but it never comes back. All icons go blank and the only way to get them back is to reboot.

Does a plasmashell replace help?

So there are actually multiple issues here. Most KDE applications pull their knowledge of currently available apps from a shared cache called sycoca (for SYstem COnfiguration CAche), which is maintained by KService. Unfortunately, it does not work correctly on NixOS for multiple reasons:

The cache can be rebuilt in-process, by each application, but the name and contents of the cache depend on XDG_DATA_DIRS, which is different for many processes due to wrappers, so we end up with multiple unrelated copies of the cache, each maintained by a different application, with a different view of things. This isn't necessarily an issue when the cache is updated correctly, except wasting a whole bunch of cycles.
The cache is updated by checking the mtimes on the target files, which for us are always 0, so the cache is effectively never refreshed unless nuked entirely. The solution to that is probably to make KService actually walk up the path until it finds a symlink to the store, and then check the mtime of that, but that's an extremely invasive change.

https://github.com/NixOS/nixpkgs/pull/292748 should unblock us for now by restoring the nuke-all-sycoca activation script from Plasma 5, but we'll definitely want to look into some kind of a better solution going forward, and probably get upstream involved.

Can people affected by this please test https://github.com/NixOS/nixpkgs/pull/292748 ?

For me running xdg-desktop-menu forceupdate fixes the problem as well, so as a workaround I added that to my rebuild script

I had the same issue with NeoChat and was informed, that this fix corrects the issue and indeed, after an update and reboot, it showed up.

Can people affected by this please test #292748 ?

I'm still plagued by this issue, neither installing kcmutils nor the nuke script functions on my system (My lock file has nixpkgs at 73d699a6ff1e83df3fd6c1e60931e13667b8ae14). My flake: https://github.com/NovaViper/NixConfig

I'm still seeing this issue. I run nixos-rebuild switch and now when I click on apps in my KDE "Start Menu", nothing happens. Running latest nixos-stable, updated and rebuilt this morning

Do you have this issue in the "Favorites" part of the menu? (because I even have that issue under Plasma 5)

I can confirm for the Favorites. Normal menu items do work for me in the meantime totally fine.

Ok, I guess then that's a general Plasma5/6 issue on NixOS. 😅
Is this worth a new issue, @K900?

I have not observed this issue on 6, at least.

Strange... I just tested it after a QOwnNotes update in the morning on a plasma 6 Laptop. 🤔
I can search for it or use the normal menu entry from the different categories, but the favorite entry can't be clicked.

I've noticed the same issue in KRunner when doing an app search. Currently on Plasma 6.0.5.

Fedora 41 KDE plasma has this issue. Although restoring it brings the icons. So, I guess nix need to refresh KDE plasma start menu.

I've got this issue and I installed my NixOS within the last week.
Using stable and I get the same issues, the favourites section of plasma doesn't launch anything and I also can't edit them after a nixos-rebuild if the packages get changed.

This should be mostly worked around in 24.11.


training with conditioning t5

I didn’t quite understand how to start training with conditioning and how to arrange a dataset for this.

ditto... some sample code for conditional training would be awesome.

Which part is not clear from the example in the readme?
from audio_diffusion_pytorch import T5Embedder

# Declare the text embedding model
embedder = T5Embedder(model='t5-base', max_length=64)

# Declare the diffusion model
model = AudioDiffusionConditional(
    in_channels=1,
    # other params here...
    embedding_max_length=64,
    embedding_features=768,
)

# Encode text into embedding
embedding = embedder(["First batch item text...", "Second batch item text..."]) # [2, 64, 768]
x = torch.randn(2, 1, 2 ** 18) # Your waveform

# Feed waveform and text into the model
loss = model(x, embedding=embedding)

# Repeat this step multiple times in the training loop
loss.backward()

Sorry... yes, clear enough.

Какая часть непонятна из примера в ридми?
from audio_diffusion_pytorch import T5Embedder

# Declare the text embedding model
embedder = T5Embedder(model='t5-base', max_length=64)

# Declare the diffusion model
model = AudioDiffusionConditional(
    in_channels=1,
    # other params here...
    embedding_max_length=64,
    embedding_features=768,
)

# Encode text into embedding
embedding = embedder(["First batch item text...", "Second batch item text..."]) # [2, 64, 768]

# Your waveform (dummy example)
x = torch.randn(2, 1, 2 ** 18)

# Feed waveform and text embedding into the model
loss = model(x, embedding=embedding)

# Repeat this step multiple times in the training loop
loss.backward()

So where do you put the dataset?

I hadn't realized before, but there are two extra repos to help!
https://github.com/archinetai/audio-data-pytorch
https://github.com/archinetai/audio-diffusion-pytorch-trainer


Filter repo list: only show active repos

Hey,
first of all, big thanks for your extension. It works perfectly, clean, fast and no bloat! Amazing!
One thing that i notices is, that the Repos list is quite long and it wold be nice if i could filter it to only show repos that are active for example.
Thanks and greetings
Leo

Hey, Leo! Thank you! Yeah, nice suggestion! My list is long too, so I have been thinking recently about filtering and sorting repos :-) Will implement it in the next version

I saw that you implemented some filters for the repos. Thats amazing, thanks. The only thing i noticed is, that after the update the repository list was empty because the default filter was falling back to nothing. Only after setting a filter it started working, maybe some wrong default value somewhere?
But since this only happened after an update, its propobly not that big of an issue.

Thank you! Tried to improve default filters behavior.

Improvements work great, much easier to work with now! Amazing work, big thanks!

Glad it helped! Thank you for suggestions and reporting!


Added remove friend logic

We added functionality to the Remove Friend button on profile pages that removes the two users from each other friend lists and delete the friend invite that exists between these two users. Users can not remove friends while logged out or remove other users they are not friends with or users that do not exist.

Closing Statement:  Remove Friend button on profile pages now works. Users can not remove friends while logged out or remove other users they are not friends with or users that do not exist.
@AndrewM131 @henrietta-k  worked on the view function for removing friends
@omarelkashef  worked on on the logic of unfriending users

Closing Statement: Remove Friend button on profile pages now works. Users can now remove friends while logged out or remove other users they are not friends with or users that do not exist.
Team contributions in issue closing statement.

Issue Score: Excellent
Comments:
Same comments as issue - good work!


Add testRectCircle

Hi I would like to help add testRectCircle :)
It is checked by
1 find the nearest point with circle
2 find the distant between the nearest point
3 check the distant if it is less than circle's radius then it is intersect

Nice! Thank you


Only configure git user for CircleCI

I just realized that the changes made to get CI passing for #1617 will also override the git config for anyone running tests locally. 🙀
Since we only needed to make sure CircleCI has a configured git user I moved that into the CircleCI config.
Thanks @dlaynes for bringing it to my attention! Hope this answers your question.

:tada: This PR is included in version 6.5.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


ValueError: Shapes (1, 1, 1024, 255) and (21, 1024, 1, 1) are incompatible

So basically I trained a YOLOv3 Model with ImageAI.
During the load() function it crashes and gives back this traceback:
Traceback (most recent call last):   File "C:/Users/aless/PycharmProjects/Mikulyzer_Locate/firstDetection.py", line 10, in <module>     detector.loadModel()   File "C:\Users\aless\anaconda3\envs\Mikulyzer_Locate\lib\site-packages\imageai\Detection\__init__.py", line 197, in loadModel     model.load_weights(self.modelPath)   File "C:\Users\aless\anaconda3\envs\Mikulyzer_Locate\lib\site-packages\keras\engine\saving.py", line 492, in load_wrapper     return load_function(*args, **kwargs)   File "C:\Users\aless\anaconda3\envs\Mikulyzer_Locate\lib\site-packages\keras\engine\network.py", line 1230, in load_weights     f, self.layers, reshape=reshape)   File "C:\Users\aless\anaconda3\envs\Mikulyzer_Locate\lib\site-packages\keras\engine\saving.py", line 1237, in load_weights_from_hdf5_group     K.batch_set_value(weight_value_tuples)   File "C:\Users\aless\anaconda3\envs\Mikulyzer_Locate\lib\site-packages\keras\backend\tensorflow_backend.py", line 2960, in batch_set_value     tf_keras_backend.batch_set_value(tuples)   File "C:\Users\aless\anaconda3\envs\Mikulyzer_Locate\lib\site-packages\tensorflow\python\keras\backend.py", line 3066, in batch_set_value     assign_op = x.assign(assign_placeholder)   File "C:\Users\aless\anaconda3\envs\Mikulyzer_Locate\lib\site-packages\tensorflow\python\ops\resource_variable_ops.py", line 1141, in assign     self._shape.assert_is_compatible_with(value_tensor.shape)   File "C:\Users\aless\anaconda3\envs\Mikulyzer_Locate\lib\site-packages\tensorflow\python\framework\tensor_shape.py", line 1103, in assert_is_compatible_with     raise ValueError("Shapes %s and %s are incompatible" % (self, other))
My Code is:

execution_path = os.getcwd()
detector = ObjectDetection()
detector.setModelTypeAsYOLOv3()
detector.setModelPath(os.path.join(execution_path, "detection_model-ex-013--loss-0012.906.h5"))
detector.loadModel() # crashes here

Basically, did I not train it right? Or do I not load it correctly?

Hi @aless2003 , You forgot to specify the path to the the detection_config.json generated during the training.
detector.setJsonPath("detection_config.json")
detector.loadModel()

Follow this link to learn more
https://imageai.readthedocs.io/en/latest/customdetection/index.html

Thanks for the fast reply.
Unfortunately Python says ObjectDetection doesn't have something called setJsonPath.
I seem to be on the most recent version though.

Remove from imageai.Detection import ObjectDetection and use the import below instead.
from imageai.Detection.Custom import CustomObjectDetection

Oh! Alright now I understand. Thank you very much! I'm sorry I took your time for that, it works now.
Have a beautiful day!


[Bug]: design-factory dependencies are needed at the root of the workspace

Package name
showcase
Package version
9.4
Reproduction steps
Try to remove the design-factory associated packages from the root package.json (keeping them only in apps/showcase/package.json):
"@design-factory/design-factory",
"ag-grid-angular",
"ag-grid-community",
"bootstrap"
Current result
The build of showcase app fails when trying to @use the scss files from these packages
Expected result
The showcase app should build successfully
Additional comments
No response

The PR #1113 is fixing the SCSS build issue but the removal of the dependency at workspace level is providing this error in server task:
Unchanged output files: 2
Application bundle generation complete. [0.506 seconds]
Reloading client(s)...
Failed to resolve import "@design-factory/design-factory" from ".angular/vite-root/showcase/chunk-S3V3RFK2.js". Does the file exist?
Failed to resolve import "@design-factory/design-factory" from ".angular/vite-root/showcase/chunk-S3V3RFK2.js". Does the file exist? (x2)
1:23:10 PM [vite] Internal server error: Failed to resolve import "@design-factory/design-factory" from ".angular/vite-root/showcase/chunk-S3V3RFK2.js". Does the file exist?
  Plugin: vite:import-analysis
  File: /otter/.angular/vite-root/showcase/chunk-S3V3RFK2.js:3:68
  4713|  import { ChangeDetectionStrategy as ChangeDetectionStrategy5, Component as Component5, forwardRef, Input as Input6, signal, ViewEncapsulation as ViewEncapsulation5 } from "@angular/core";
  4714|  import { FormsModule, NG_VALUE_ACCESSOR } from "@angular/forms";
  4715|  import { CloseInputDatePickerDirective, DfInputIconDirective } from "@design-factory/design-factory";
     |                                                                       ^
  4716|  import { NgbDate, NgbInputDatepicker } from "@ng-bootstrap/ng-bootstrap";
  4717|  import * as i010 from "@angular/core";
      at formatError (file:///otter/.yarn/__virtual__/vite-virtual-6fc7e2660e/0/cache/vite-npm-4.5.0-6fb40946d7-b262ea4880.zip/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:44062:46)
      at TransformContext.error (file:///otter/.yarn/__virtual__/vite-virtual-6fc7e2660e/0/cache/vite-npm-4.5.0-6fb40946d7-b262ea4880.zip/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:44058:19)
      at normalizeUrl (file:///otter/.yarn/__virtual__/vite-virtual-6fc7e2660e/0/cache/vite-npm-4.5.0-6fb40946d7-b262ea4880.zip/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:41844:33)
      at processTicksAndRejections (node:internal/process/task_queues:95:5)
      at async file:///otter/.yarn/__virtual__/vite-virtual-6fc7e2660e/0/cache/vite-npm-4.5.0-6fb40946d7-b262ea4880.zip/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:41998:47
      at async Promise.all (index 70)
      at async TransformContext.transform (file:///Uotter/.yarn/__virtual__/vite-virtual-6fc7e2660e/0/cache/vite-npm-4.5.0-6fb40946d7-b262ea4880.zip/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:41914:13)
      at async Object.transform (file:///otter/.yarn/__virtual__/vite-virtual-6fc7e2660e/0/cache/vite-npm-4.5.0-6fb40946d7-b262ea4880.zip/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:44352:30)
      at async loadAndTransform (file:///otter/.yarn/__virtual__/vite-virtual-6fc7e2660e/0/cache/vite-npm-4.5.0-6fb40946d7-b262ea4880.zip/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:55026:29)
      at async viteTransformMiddleware (file:///otter/.yarn/__virtual__/vite-virtual-6fc7e2660e/0/cache/vite-npm-4.5.0-6fb40946d7-b262ea4880.zip/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:64430:32)

Issue should be fixed in @design-factory/design-factory v16


chore(ci): check if build works for darwin

Description
This PR adds a CI job that build go binaries for darwin/arm64 to prevent merging PR that breaks developer workflow.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
CI
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

@stehessel How about moving stackrox-env into stackrox. Then we don't need to pin.

Problem with that is that stackrox-env actually spans multiple repos, see https://github.com/stackrox/stackrox-env/blob/master/flake.nix. I wouldn't mind splitting it and adding a flake to each of our repos, but I think others have workflows where they want everything in one flake.

I wouldn't mind splitting it and adding a flake to each of our repos, but I think others have workflows where they want everything in one flake.

Good point. As this is a bigger project and I have no idea if we can split flakes per repo and then combine them in stackrox-env let's do it later and now just stop on pinning to the latest commit.

FAIL: TestRateLimitedLogger (0.02s) is it a flake, or have we just discovered something?


Test suite for manifest filtering

Create a test suite for manifest filtering such as the one implemented in iceberg python.
Dependent on https://github.com/apache/iceberg-rust/issues/350.

@Fokko, can you please assign this to me? Thanks!

Assigned, thanks @s-akhtar-baig !


Fix bug in resolve_types with subclasses

Pull Request Check List
This is just a friendly reminder about the most common mistakes.  Please make sure that you tick all boxes.  But please read our contribution guide at least once, it will save you unnecessary review cycles!
If an item doesn't apply to your pull request, check it anyway to make it apparent that there's nothing left to do.  If your pull request is a documentation fix or a trivial typo, feel free to delete the whole thing.
Fixes #842

[x] Added tests for changed code.
[ ] New features have been added to our Hypothesis testing strategy.
[ ] Changes or additions to public APIs are reflected in our type stubs (files ending in .pyi).

[ ] ...and used in the stub test file tests/typing_example.py.

[ ] Updated documentation for changed code.

[ ] New functions/classes have to be added to docs/api.rst by hand.
[ ] Changes to the signature of @attr.s() have to be added by hand too.
[ ] Changed/added classes/methods/functions have appropriate versionadded, versionchanged, or deprecated directives.  Find the appropriate next version in our __init__.py file.

[ ] Documentation in .rst files is written using semantic newlines.
[ ] Changes (and possible deprecations) have news fragments in changelog.d.

If you have any questions to any of the points above, just submit and ask!  This checklist is here to help you, not to deter you from contributing!

Does this type of change need a changelog entry?

Thanks for stepping up real quick as usual!
Yeah I think it’s worth mentioning!
For every bug report there’s 10 people who didn’t bother.
also pls src/attr/_funcs.py:390:80: E501 line too long (82 > 79 characters) :)

Ok.  Gotta go make breakfast but I'll get the changelog in after that.

Oh god why is codecov angry. If you don’t have time for it, I’ll check once I’m back from Italy and off phone-only.

Oh it's because nothing tests the "cached" flow.

Sorry for all the lint failures. I don't have python 3.10 on my machine and you can only run pre-commit or tox -e lint if you have python 3.10.

Ok I have no idea why codecov is complainign about this:

https://app.codecov.io/gh/python-attrs/attrs/compare/843/tree/src/attr/_compat.py

It was complaining about the same thing on #840 and then magically stopped…some codecov flakiness? :-/

It passed!

🤡

Thanks for the quick turnarounds!


Copy over existing content

SUMMARY
This PR is not very interesting.

Copy, without changes, all images from /images to /static/images.
Copy, without changes, all docs from /pages to /docs.
Regenerate all API docs using the experimental Docusaurus plugin for API Documenter.

REVIEWER NOTES
Review and merge #2 before reviewing this PR.

It is tempting to bring over the full Git history for this migration.  However because it is a public repo, GitHub would duplicate the contribution items in everyone's user profiles, which looks bizarre.
Instead, my suggestion is for 152b59feff9108d5ccbe7791a01dc8528a9d1a98 to include a commit comment with a URL pointing to the corresponding commit in the source repo.  This way, if someone is tracing the history of a file ("How did this sentence get here?"), they can easily pick up the bread crumb in the other git log.


scope tells me it's available at URLs that it's not available at

Running scope 0.16.1 on my Mac using docker-machine, I see the following:
$ ./scope launch
c46ebb9bc663573b9ca39000352cba4da5184c5ad61c9450f39ad37757298527
Weave Scope is reachable at the following URL(s):
  * http://<IP_ADDRESS>:4040/
  * http://<IP_ADDRESS>:4040/
  * http://<IP_ADDRESS>:4040/

However, when I click on these URLs, I get "address unreachable" errors for the first and third. Only the second URL is accessible.
Ideally, Scope would only tell me about URLs that I can actually reach. Failing that, the wording should be amended to be more truthful.

Dupe of #1190

Maybe not a dupe actually.  But related.

Ideally, Scope would only tell me about URLs that I can actually reach. Failing that, the wording should be amended to be more truthful.

We should change the wording then, since the URLs that I can actually reach also depend on where you are connecting from (including firewalls and what not) and Scope cannot know about that.

We should change the wording then, since the URLs that I can actually reach also depend on where you are connecting from (including firewalls and what not) and Scope cannot know about that.

I don't think that this a big deal, but sure if we can make a small tweak to the wording (w/o turning it into a novel) then fine.
The main issue jml reported is that even for a purely local setup, scope may show a bunch of addresses that don't work. It would be nice if we were able to filter those out somehow.

IMO fixing this is only worthwhile if we can come up with a fairly clean and short solution.


Using OutlookFileDrag in Xendesktop

Hi Tony,
first of all I thank you for this Addin.
I have installed your software on a Citrix XenDesktop 6.5 Server. The windows Server is 2008 R2 64 Bit. The Microsoft Outlook is 32 bit and I used OutlookFileDragSetup_x64 .zip for installation on this server. I get the following error when an user tries to activate this addin in MS Outlook.
System.Runtime.InteropServices.SEHException (0x80004005): Eine externe Komponente hat eine Ausnahme ausgelöst.
bei System.Deployment.Internal.Isolation.IsolationInterop.GetUserStore(UInt32 Flags, IntPtr hToken, Guid& riid)
bei System.Deployment.Internal.Isolation.IsolationInterop.GetUserStore()
bei System.Deployment.Application.ComponentStore..ctor(ComponentStoreType storeType, SubscriptionStore subStore)
bei System.Deployment.Application.SubscriptionStore..ctor(String deployPath, String tempPath, ComponentStoreType storeType)
bei System.Deployment.Application.SubscriptionStore.get_CurrentUser()
bei System.Deployment.Application.DeploymentManager..ctor(Uri deploymentSource, Boolean isUpdate, Boolean isConfirmed, DownloadOptions downloadOptions, AsyncOperation optionalAsyncOp)
bei System.Deployment.Application.InPlaceHostingManager..ctor(Uri deploymentManifest, Boolean launchInHostProcess)
bei Microsoft.VisualStudio.Tools.Applications.Deployment.IPHMProxy..ctor(Uri uri)
bei Microsoft.VisualStudio.Tools.Applications.Deployment.ClickOnceAddInDeploymentManager.get_Proxy()
bei Microsoft.VisualStudio.Tools.Applications.Deployment.ClickOnceAddInDeploymentManager.GetManifests(TimeSpan timeout)
bei Microsoft.VisualStudio.Tools.Applications.Deployment.ClickOnceAddInDeploymentManager.InstallAddIn()
The first line is in German Language and it means "an external component has triggered an exception".
Do you have any Idea? Should I use the other setup program that you have for only 32 bit systems?
Regards,
Amir

This seems to be a compatibility issue between ClickOnce deployment and Citrxi XenDesktop.  Are you launching Outlook as a published app?  If so, do you have the same problem with a published desktop?
This article may be able to provide some help:
https://support.citrix.com/article/CTX125453?_ga=2.10188021.1175527337.1541824793-706068068.1541730213

Hi Tony,
Thank you for your answer. The users use MS Outlook in a Published Desktop.

Have you been able to resolve this issue?  If so, are there any errors in the Windows event log at the time that the error occurs?

Hi Tony,
No. I got no errors in event viewer. When the user drag an Email and drop it to SAP only the header is transmitted (on Citrix Xendesktop).  In contrary on windows 10 clients after drag and drop the user sees the path of the saved Email in a "temp" folder.

Amir, have you been able to resolve this issue?

Hi Tony,
Thank you for your followup, but my answer is no. We planned to install Microsoft Office 2016 on our server and install your latest version of this software and test it again. I will inform you what the result is later on.

Amir, were you able to resolve this issue?

Hi Tony,
unfortunately no, and we switched to another software that is not free.

Amir, I wanted to reach out and let you know that Chrome 76 that was released yesterday now natively supports drag and drop from Outlook.  If you run this browser, then a plugin should no longer be needed.  You may want to test if this works in your environment.

Hi Tony,
thanks a lot for the news. I will check it and will inform you again.


Fix deployment with octavia load balancer

https://github.com/eimis-ans/eimis-synapse/pull/391 introduces a new way to access the cluster.
So far, there's a workaround to manually get IP nodes on private network but it's not implemented in the workflow

find a better a way to get the cluster IP nodes and configure the load balancer pool members
Make deploy from scratch work again

fixed with #391


CMU Python

https://github.com/norvig/paip-lisp/blob/f589ad5b25854e8fc0ddb126617713669f9faaf8/docs/chapter9.md?plain=1#L97
Shouldn't it be CMU Lisp instead of CMU's Python?

CMU Common Lisp's compiler is called Python.

I didn't know. That's interesting. Thank you.


Custom Name of Parameter not being honored.

When trying to specify a custom name for a parameter we are receiving the following error:
Parameter Declaration:
var wfParams = new MyParam()
{
    Value1 = "value1",
    Value2 = "value2"
};
var rp = new RuleParameter("myValue", wfParams);

var myWorkflow = new Workflow
{
    WorkflowName = "my_workflow",
    Rules = new List<Rule>
    {
        new Rule
        {
            RuleName = "MatchesFabrikam",
            Enabled = true,
            RuleExpressionType = RuleExpressionType.LambdaExpression,
            Expression = "myValue.Value1== \"Fabrikam\""
        }
    }
};

When the rule is evaluated the following error is received:
  Message:
System.Exception : Could not execute rules: [MatchesFabrikam: Exception while parsing expression `myValue.Value1 == "Fabrikam"` - Enum type 'endpoint' not found]`

We're targeting  .net8 and using RulesEngine 5.0.2.

@khPhoenix can you share MyParam class definition?

public class MyParam
{
public string? Value1 { get; set; }
public string? Value2 { get; set; }
}

i added a working example and solution to my fork


[Feature Request] Disable team member list with config.yml setting

Hello.  Which is currently possible by displaying the full Team member list to every member.
config.yml could have a boolean setting that disables generation and display of the Team member list (possibly always overridden if admin=1) and the Activities news stream on the Dashboard.
This could be called "Privacy Mode" or something. This is similar to feature request #1853 (disabling UI entirely), but less severe.
Thank you.

Don't know how to keep it from going stale except bumping.
We could still use the same functionality to prevent blasting entire customer list to every customer.


when are you planning to update on pub.dev

its been 10 months since the last update.

12 months

There are pre-release builds you can use that.

I am planning a new public release soon.
There is still a blocker, I don't have access to the ferry_exec package which is still owned by @smkhalsa, so I can't publish updates for it.
Once this is resolved, I'll publish new stable releases soon.
In the worst case I'll have to fork ferry_exec and publish it under a new name temporarily, but I really would like to avoid that.

Published.


Adding "vscode" to tags for better searchabaility

Also condensing "Visual Studio Code" into one word, since Chocolatey doesn't seem to support spaces in tag names.

Thanks @JVimes!


Add the Amazon EC2 Depolyment docs

For Issue: #31
@smaranjitghose,  Please tell me if this formate is right so that I can make the other docs in the same formate

For Issue: #31
@smaranjitghose,  Please tell me if this formate is right so that I can make the other docs in the same formate

If you do not have experience with Cloud Hosting ignore my above comments except the ones related to markdown syntax. Rest looks fine to me

If you do not have experience with Cloud Hosting ignore my above comments except the ones related to markdown syntax. Rest looks fine to me

@raman08 Updates?

@raman08 Updates?


Flash messages in reactive state

An adaptation of Vulcan's flash message system, but using reactive state instead of React context
Updated withMessages to use reactive state
Added new useMessages hook
Added new useIntl hook
Separated FlashMessages component into its own file
Updated Flash component to use reactive state
Moved getProperties() from the Flash component to normalizeMessage() in withMessages
Added new Material UI implementation of the Flash component
Removed messages from the state of the App component, MessageContext from the component tree, and the methods UNSAFE_componentWillMount, componentWillUnmount, flash, and clear
Removed messages.js, which only defined MessageContext
Updated error.js to report the http status of network errors thrown by Apollo

I'll let you merge as well, sounds cool! My first attempt at Flash message with Apollo client side queries was such a disaster lol it's cool to see it progressing

I'll let you merge as well, sounds cool! My first attempt at Flash message with Apollo client side queries was such a disaster lol it's cool to see it progressing


"Exception of type 'UnityEngine.ExitGUIException' was thrown." error happens whenever you click the circle next to a reference

Doesn't seem to break anything whatsoever, but worth mentioning

Error leads to line 325 of UTEditor.cs

Intresting, not really sure what can be happening here, considering i just draw a regular field 🤔 but i'll take a look, might be a wontfix tho

So i added a try-catch to the gui code to fallback to the default editor instead of fully crashing and stuff, but for some reason - Unity throws an ExitGUI exception when you click on the circle thing, while it hides it when you don't have a try-catch. Aded a check that just silences the ExitGUI as there is seemingly no harm in that, maybe will look into the root of the issue down the line.
Fixed in develop
Here's a package:
UdonToolkit_v0.4.2.zip


VisitorMut example not working

The VisitorMut example from https://docs.rs/sqlparser/latest/sqlparser/ast/trait.VisitorMut.html is not working
use sqlparser::ast::Visit;
use sqlparser::ast::{Expr, Ident, VisitorMut};
use sqlparser::parser::Parser;
use std::ops::ControlFlow;

fn main() {
    struct Replacer;

    // Visit each expression after its children have been visited
    impl VisitorMut for Replacer {
        type Break = ();

        fn post_visit_expr(&mut self, expr: &mut Expr) -> ControlFlow<Self::Break> {
            if let Expr::Identifier(Ident { value, .. }) = expr {
                *value = value.replace("to_replace", "replaced")
            }
            ControlFlow::Continue(())
        }
    }

    let sql = "SELECT to_replace FROM foo where to_replace IN (SELECT to_replace FROM bar)";
    let mut statements = Parser::parse_sql(&GenericDialect {}, sql).unwrap();

    // Drive the visitor through the AST
    statements.visit(&mut Replacer);

    assert_eq!(
        statements[0].to_string(),
        "SELECT replaced FROM foo WHERE replaced IN (SELECT replaced FROM bar)"
    );
}

error[E0277]: the trait bound `Replacer: Visitor` is not satisfied
  --> src/main.rs:26:22
   |
26 |     statements.visit(&mut Replacer);
   |                ----- ^^^^^^^^^^^^^ the trait `Visitor` is not implemented for `Replacer`
   |                |
   |                required by a bound introduced by this call

I had to use VisitMut instead of Visit