Revamp eslint/typescript config

Our eslint config is messy as a result of manually combining all of the rules from various create-react-app libraries. Most of them likely can be replaced in favor of extending from a recommended preset from an appropriate library. Our eslint config also needs to updated to properly lint backend-specifc code (ex: cypress, .storybook, root configs). Our typescript may also need to be updated so that typescript checking in IDEs work appropriately between backend and frontend code (specifically cypress types colliding with our unit testing library).
Acceptance Criteria:

[ ] all recommended rule presets available in our currently use libraries applied
[ ] team-approved manual changes from recommended baseline applied
[ ] code updated to be compliant with any new rules
[ ] linting appropriately applied to both backend and frontend code
[ ] typescript checking works correctly on both frontend and backend code

Accomplished.


Update and cleanup dependencies

Just trying to keep the dependencies footstep at minimal. So:

wdm gem removed. As the readme states Windows is unsupported either way.
ruby18_source_location gem removed. Well, it's 2k15.
middleman-livereload gem removed. It comes by default with Middleman Gemfile but it's not even activated in the config.rb.
All gems updated to their latest versions.

:+1: Just tested it, and it looks great to me. Thanks so much for the contribution! :smile:


[Feature Request]: Favorite system for tools

What would you like to happen?
Hello,
I would like to have a favorite category when you can choose a tool and add it to your favorites category. The thing is I use 80% of the time less than 10 tools and that would save time.
Best regards,
Simon
Issue Priority
Priority: 3
Issue Component
Component: Actions, Component: Hop Gui

The request is unclear regarding which kind of tools are you referring to and what do you intend by favorite category. Can you elaborate a bit more maybe with examples to help the dev team understand better?

Hello @sramazzina and thanks for your answer
By tools, I mean the item in this window :

and here how it works on Alteryx :
You can click on a little star on a tool to make it a favorite

And then a new category Favorite appears with all your favorite tools

Best regards,
Simon


Error starting server

Issue Description
sudo python /opt/king-phisher/KingPhisherServer -L INFO -f server_config.yml
WARNING  python 2.7 support is deprecated, see https://github.com/securestate/king-phisher/wiki/Python-3 for details
INFO     listening on <IP_ADDRESS>:80
INFO     serving files has been enabled
INFO     initializing database connection with driver sqlite
Traceback (most recent call last):
File "/opt/king-phisher/KingPhisherServer", line 238, in
sys.exit(main())
File "/opt/king-phisher/KingPhisherServer", line 232, in main
status_code = build_and_run(arguments, config, plugin_manager)
File "/opt/king-phisher/KingPhisherServer", line 91, in build_and_run
king_phisher_server = build.server_from_config(config, plugin_manager=plugin_manager)
File "/opt/king-phisher/king_phisher/server/build.py", line 159, in server_from_config
server = KingPhisherServer(config, plugin_manager, handler_klass, addresses=addresses, ssl_certfile=ssl_certfile, ssl_keyfile=ssl_keyfile)
File "/opt/king-phisher/king_phisher/server/server.py", line 746, in init
self.database_engine = db_manager.init_database(config.get('server.database'), extra_init=True)
File "/opt/king-phisher/king_phisher/server/database/manager.py", line 258, in init_database
inspector = sqlalchemy.inspect(engine)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/inspection.py", line 63, in inspect
ret = reg(subject)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/reflection.py", line 139, in _insp
return Inspector.from_engine(bind)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/reflection.py", line 135, in from_engine
return Inspector(bind)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/reflection.py", line 109, in init
bind.connect().close()
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/base.py", line 2082, in connect
return self._connection_cls(self, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/base.py", line 90, in init
if connection is not None else engine.raw_connection()
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/base.py", line 2168, in raw_connection
self.pool.unique_connection, _connection)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/base.py", line 2142, in _wrap_pool_connect
e, dialect, self)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/base.py", line 1456, in _handle_dbapi_exception_noconnection
exc_info
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/util/compat.py", line 202, in raise_from_cause
reraise(type(exception), exception, tb=exc_tb, cause=cause)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/base.py", line 2138, in _wrap_pool_connect
return fn()
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/pool.py", line 328, in unique_connection
return _ConnectionFairy._checkout(self)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/pool.py", line 766, in _checkout
fairy = _ConnectionRecord.checkout(pool)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/pool.py", line 516, in checkout
rec = pool._do_get()
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/pool.py", line 1292, in _do_get
return self.connection
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/util/langhelpers.py", line 754, in get
obj.dict[self.name] = result = self.fget(obj)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/pool.py", line 1264, in connection
return _ConnectionRecord(self)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/pool.py", line 461, in init
self.__connect(first_connect_check=True)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/pool.py", line 651, in __connect
connection = pool._invoke_creator(self)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/strategies.py", line 105, in connect
return dialect.connect(*cargs, **cparams)
File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/engine/default.py", line 392, in connect
return self.dbapi.connect(*cargs, **cparams)
sqlalchemy.exc.OperationalError: (pysqlite2.dbapi2.OperationalError) unable to open database file
.
Environment Details

KALI 2016

It's complaining that your database connection is not properly configured in the servers configuration file. It is highly recommended that you do not use SQLite like you are and instead use PostgreSQL. Instructions on how to set this up are available in the wiki on the Database page.

Haven't gotten a response in a week so I'm closing this out.


[cxx-interop] Reenable exporting Foreign Reference Types to C++

This feature worked prior 5.10 but the semantics was undefined. This PR restores the behavior with the old semantics, and a separate PR will update the documentation to describe the behavior.
rdar://129420670

@swift-ci please smoke test

Looks like clang format messed up the test file just before committing it. I need to rerun the tests.
@swift-ci please test


Update GVF spec Column 3: "type" to allow "parent of sequence_alteration"

There is a need to be able to specify in GVF when a locus has been sequenced and explicitly found NOT to differ from the reference sequence. In #334 , this term was determined to be SO:0002073: no_variation, and is a sibling of sequence_alteration.
GVF spec v1.07 for Column 3 reads:

Column 3: "type"
The type of the feature. This is constrained to be either: (a) the SO term sequence_alteration (SO:0001059), (b) a child term of sequence_alteration, (c) the SO term gap (SO:0000730), or (d) the SO accession number for any of the previous terms. The gap feature, while not a sequence_alteration, provides a way to annotate gaps in the individuals genome assembly where sequence_alteration information is unknown (low-coverage, no-call regions).

Since no_variation is a sibling of sequence_alteration, to allow its use the spec must be updated to allow "parent of sequence_alteration" and any child term thereof, instead of sequence_alteration and any child thereof.

sequence_alteration is mentioned several times in the blue box near the beginning of the spec, labeled, "A Quick Explanation and Example of GVF Content." The spec needs to be evaluated in this box, and throughout, to determine what text should be updated to accommodate the change discussed here.

Tim, we have added the specs to git, using mark-down
https://github.com/The-Sequence-Ontology/Specifications

I think you should be able to go in and make these changes now Tim.

Hi @keilbeck ,
I made a pull request for the GVF spec on May 19:
https://github.com/The-Sequence-Ontology/Specifications/pull/1

Did it work?
—K
Karen Eilbeck
Associate Professor
Biomedical Informatics
University of Utah
From: Tim Hefferon <notifications<EMAIL_ADDRESS>Reply-To: The-Sequence-Ontology/SO-Ontologies <reply<EMAIL_ADDRESS>Date: Tuesday, May 31, 2016 at 9:17 AM
To: The-Sequence-Ontology/SO-Ontologies <SO-Ontologies<EMAIL_ADDRESS>Cc: Karen Eilbeck <keilbeck<EMAIL_ADDRESS>Mention <mention<EMAIL_ADDRESS>Subject: Re: [The-Sequence-Ontology/SO-Ontologies] Update GVF spec Column 3: "type" to allow "parent of sequence_alteration" (#351)
Hi @keilbeckhttps://github.com/keilbeck ,
I made a pull request for the GVF spec on May 19:
The-Sequence-Ontology/Specifications#1https://github.com/The-Sequence-Ontology/Specifications/pull/1
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/The-Sequence-Ontology/SO-Ontologies/issues/351#issuecomment-222721797, or mute the threadhttps://github.com/notifications/unsubscribe/AKXgFyWC7mJWWLanwLw7Mr4SusXAxHnJks5qHFEegaJpZM4IAQEO.

Hi @keilbeck ,
It worked, however the link to 'no_variation' is broken because the term is in the current SVN but not the current release.
Shall I update the link from:
http://www.sequenceontology.org/browser/current_release/term/SO:0002073
to:
http://www.sequenceontology.org/browser/current_svn/term/SO:0002073
or just wait for no_variation to become part of the current release?
Thanks,
Tim

Where are you getting the release from?
Karen Eilbeck
Associate Professor
Biomedical Informatics
University of Utah
From: Tim Hefferon <notifications<EMAIL_ADDRESS>Reply-To: The-Sequence-Ontology/SO-Ontologies <reply<EMAIL_ADDRESS>Date: Tuesday, June 21, 2016 at 10:35 AM
To: The-Sequence-Ontology/SO-Ontologies <SO-Ontologies<EMAIL_ADDRESS>Cc: Karen Eilbeck <keilbeck<EMAIL_ADDRESS>Mention <mention<EMAIL_ADDRESS>Subject: Re: [The-Sequence-Ontology/SO-Ontologies] Update GVF spec Column 3: "type" to allow "parent of sequence_alteration" (#351)
Hi @keilbeckhttps://github.com/keilbeck ,
It worked, however the link to 'no_variation' is broken because the term is in the current SVN but not the current release.
Shall I update the link from:
http://www.sequenceontology.org/browser/current_release/term/SO:0002073
to:
http://www.sequenceontology.org/browser/current_svn/term/SO:0002073
or just wait for no_variation to become part of the current release?
Thanks,
Tim
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/The-Sequence-Ontology/SO-Ontologies/issues/351#issuecomment-227497171, or mute the threadhttps://github.com/notifications/unsubscribe/AKXgF-INRjExwlYfHoIIsPCdEkEkEidTks5qOBLtgaJpZM4IAQEO.

Per #334 I will update the gvf spec to reflect the changes made in the relevant portion of the SO.


Packet Commitment verification fails  on transfer with different account prefix

Packet commitment verification fails when transferring to a destination chain that uses a different prefix .
I[2020-03-23|12:25:29.051] ✔ [ibc0]@{286} - msg(transfer) hash(CB9A024A586095C64540B3970026DA203BB0CD386305EE6D93DAE6786E7AB321)
I[2020-03-23|12:25:30.307] - [ibc1] -> sending transaction:
TRANSACTION INFO
I[2020-03-23|12:25:30.308] ✘ [ibc1]@{0} - msg(ics20/transfer) err(client: packet commitment verification failed)
{
  "height": "0",
  "txhash": "E1D8903BB782104761074433A2CA0495FD00293CB525309E3039265B04A21E02",
  "codespace": "client",
  "code": 16,
  "raw_log": "packet commitment verification failed: verifying value: leaf value hash not same: invalid proof: couldn't verify counterparty packet commitment",
  "gas_wanted": "200000",
  "gas_used": "98142"
}

We should check how the destination address is being serialised, it may need to take the account prefix into account somehow.

Should we move this issue over to Gaia?

Closing this in favor of https://github.com/iqlusioninc/relayer/issues/43 as that was the first report.


Reusing SecretRegistry from the previous deployment file

This PR implements the feature of reusing SecretRegistry form a previous deployment file.
After this PR, users are able to use the same SecretRegistry instance for multiple deployments of TokenNetworkRegistrys.  Then a payment spanning the multiple TokenNetworks become possible.
This fixes #963 .

I'll add the documentation, and then ask for review.

Thank you!


Fix microsoft/vscode#77827

This PR is for https://github.com/microsoft/vscode/issues/77827.

Thanks!


feat: skip login screen when logged in

Closes #84
active tab was not set in init(). Fixed it.

flutter: 2021-02-05T10:40:36.558239	D	CodeModel.init:	 read code: null, null, null
flutter: 2021-02-05T10:40:39.327024	E	AuthModel.init:	 prefs getAccount failed
NoSuchMethodError: The getter 'platform' was called on null.
Receiver: null
Tried calling: platform
#5      main
package:git_touch/main.dart:41
#6      _runMainZoned.<anonymous closure>.<anonymous closure> (dart:ui/hooks.dart:231:25)
#7      _rootRun (dart:async/zone.dart:1190:13)
#8      _CustomZone.run (dart:async/zone.dart:1093:19)
#9      _runZoned (dart:async/zone.dart:1630:10)
#10     runZonedGuarded (dart:async/zone.dart:1618:12)
#11     _runMainZoned.<anonymous closure> (dart:ui/hooks.dart:223:5)
#12     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:301:19)
#13     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168:12)

There are some errors in the first launch (without any shared_preferences values). Not sure if it's introduced in this PR, or an existing bug. Could you please take a look?

Alright. I'll take a look.

Done. Error was in setting _activeTab on init()


Create new release to include pyside fix on windows and any other patches worked on the week

@jaimergp @goanpeca created a v0.1.2 release:

GitHub release: https://github.com/napari/napari-plugin-manager/releases/tag/v0.1.2
PyPI: https://pypi.org/project/napari-plugin-manager/0.1.2/
conda-forge related issue/PR to update recipe (needs approval and merge): https://github.com/conda-forge/napari-plugin-manager-feedstock/issues/6 and https://github.com/conda-forge/napari-plugin-manager-feedstock/pull/7

Let me know if something else is needed!

Due to #106 and after #107 gets merged, could it be worthy to release v0.1.3?

Let's do it!

v0.1.3 released! (pending conda-forge recipe PR merge):

GitHub release: https://github.com/napari/napari-plugin-manager/releases/tag/v0.1.3
PyPI: https://pypi.org/project/napari-plugin-manager/0.1.3
conda-forge issue and PR (PR needs approval/merge): https://github.com/conda-forge/napari-plugin-manager-feedstock/issues/8, https://github.com/conda-forge/napari-plugin-manager-feedstock/pull/9

I approved, but might be worth just adding the auto-merge?
https://conda-forge.org/docs/maintainer/infrastructure/#conda-forge-admin-please-add-bot-automerge

Merged! Thanks @dalthviz!


Atomic types as enums and php::xxx factory

What I propose
use ExtendedTypeSystem\php;

/**
 * array{
 *     a:  string,
 *     b?: int,
 *     c:  Traversable<string, false>,
 *     d:  callable(float, T:Generator): void,
 *     ...
 * }
 */
$type = php::unsealedShape([
    'a' => php::string,
    'b' => php::optionalKey(php::int),
    'c' => php::objectOf(Traversable::class, php::string, php::false),
    'd' => php::callable([php::float, php::classTemplate('TSend', Generator::class)], php::void),
]);

All atomic types are now enums with a single case self. This should greatly optimize memory. This also allows to check for atomic types using ===.
Changed Type implementations' suffixes to Type as proposed by many developers.
Added lovely php:: factory. This approach allows to name all built-in types without any modifications and use types as defaults in functions and constants. Alternative names: T, t.
Marked all *Type::__construct as @internal to force usage of php:: factory.

Alternative factory ideas I tried, but didn't like
t->int, t->array() like expressions
This can be done by declaring const t = new types() as proposed here.
Merits:

It looks a bit cleaner.
Wow effect.

Demerits:

Non-static expressions cannot be used as defaults.
Requires to explicitly autoload a file in composer.json.

Сonstants and functions without a class: intT, listT(objectT)
Merits:

The cleanest way to express types in code.

Demerits:

Requires to explicitly autoload a file in composer.json.
No autocomplete of all available options as in php:: f.e.
Requires to import a lot of functions and constants in userland.
list, callable, array functions require a prefix/suffix.
true, false type constants require a suffix/prefix.

Closing in favor of #8. We'll see, when we are ready to think about a beautiful factory/facade.


feat: close connection after responding the short-connection request

Change-Id: I22d17642339ece5f198936bf30bd50768808f46a
What type of PR is this?
feat

What this PR does / why we need it (English/Chinese):
EN: close connection after responding the short-connection request
ZH: 短连接响应写回后主动关闭连接

Which issue(s) this PR fixes:
Fixes #30

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


adding to other projects

updated traffic stop
added:

FirstModular
next gen pub safety uis
remote surgery
eda device

👷 Deploy Preview for interactive-realities processing.

Name
Link

🔨 Latest commit
984ab5605abfc9ba7548ded8084ff246124ed223

🔍 Latest deploy log
https://app.netlify.com/sites/interactive-realities/deploys/636bf12d4b1e870008df1386


Highlight located resources

Summary of changes
Display an arrow on the virtualized tree pointing to located resources.

This will highlight resources satisfying one of the following conditions:

The resource is located
The resource contains located children, and the resource is not expanded.

Context and reason for change
This allows a user to find located resources, which satisfy the filters chosen in the locator popup (Ctrl+L).
How can the changes be tested
See unit tests.
The locator popup does not yet set any state, so testing it normally is not possible (See #1944). To try it in the app, I hardcoded some located resources in ResourceBrowser.tsx:
  // const locatedResources = locateSignalActive
  //   ? resourcesWithLocatedAttributions.locatedResources
  //   : undefined;
  // const resourcesWithLocatedChildren = locateSignalActive
  //   ? resourcesWithLocatedAttributions.resourcesWithLocatedChildren.paths
  //   : undefined;
  const locatedResources = new Set<string>(['/Frontend/shared-styles.ts', '/index.tsx']);
  const resourcesWithLocatedChildren = ['/', '/Frontend/'];

As discussed earlier with Benedikt I realy do not like the solution with arrows. However, I am fine with fixing this in a follow up ticket.

I was only trying to match the PoC! lol
We have a finalize styling ticket, so lets leave it for that ticket.


Update readme with forceConsole info

Adds a section to the readme about the forceConsole flag added in https://github.com/winstonjs/winston/pull/2276, at the request of @DABH.
Note that I haven't yet tried the new flag so opening as a draft. Will flip this to ready once there's a Winston build out with the new flag that I can test locally to confirm I have the sample code correct.

(P.S, #2276 was released in winston 3.14.0, so we should merge this asap)


Updated DefaultMultifactorAuthenticationProviderBypass locateMatchingAttributeBasedOnPrincipalAttributes to use correct property

The class org.apereo.cas.services.DefaultMultifactorAuthenticationProviderBypass was updated to read the correct property value when checking for a principal attribute bypass.
Previously setting the bypass.principalAttributeValue would be ignored with the side effect of any principal with only the bypass.principalAttributeName being able to bypass MFA security.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

backported to 5.1.x


336 | Palindrome Pairs | C++

Title - Adding code for 336. Palindrome Pairs
What will change - Adding C++ code for leetcode problem 336
Type of Issue -

[x] Adding New Code

Programming Language

[x] C++

Please assign this to me.

Please assign this to me


BatchAI: Updated banner

Updating deprecation banner

OPS Build status updates of commit 9199a7a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/overview/azure/batchai.md
:white_check_mark:Succeeded
View

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

#sign-off

#hold-off - Need to fix link

OPS Build status updates of commit 406330c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/overview/azure/batchai.md
:white_check_mark:Succeeded
View

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

#sign-off


Added troubleshooting information to GitHub Apps docs

Hey, I just made a Pull Request!

'@backstage/docs': patch
Added some simple documentation regarding troubleshooting a particularly obtuse error regarding GitHub Apps integrations. Related to multiple open issues:
https://github.com/backstage/backstage/issues/13282
https://github.com/backstage/backstage/issues/19562
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] All your commits have a Signed-off-by line in the message. (more info)

Unclear why docs quality is failing. Line length maybe?

Just a note for future contributions: changesets are created using a command line tool which will create a file you would commit with your PR, but in the case of changes that are only documentation you don't need one. More information on changesets here: https://github.com/backstage/backstage/blob/master/CONTRIBUTING.md#creating-changesets

Just a note for future contributions: changesets are created using a command line tool which will create a file you would commit with your PR, but in the case of changes that are only documentation you don't need one. More information on changesets here: https://github.com/backstage/backstage/blob/master/CONTRIBUTING.md#creating-changesets

Thanks for that info! I'll update my other PR
https://github.com/backstage/backstage/pull/22109
If you wouldn't mind taking a look as well :)

Thanks for this @jrtaylorJH, just a typo to fix. Also, is there any other GitHub documentation we could point to for further details on this?

Unfortunately this is one of those that is incredibly obtuse. Backstage actually generates the NotFoundError by making inferences on the attributes of the app itself, namely "is len of installations list greater than 0", then in turn wraps that error in this "HttpError", so at the end of the day none of the errors actually come from GitHub.
There is a guide for installing a GitHub app seen here: https://docs.github.com/en/apps/using-github-apps/installing-your-own-github-app
However, this makes no mention of the "Install" field not appearing due to the lack of "Owner" permissions, which in my opinion is much of the issue.

@awanlin Just a ping if you're in the business of reviewing PRs today :)

@awanlin Just a ping if you're in the business of reviewing PRs today :)

Still looks good, I can't merge without another approval from the maintainers so just waiting on that 👍


⚠ SceneTime has degraded performance

In 9df9b0e, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 15483 ms

Resolved: SceneTime performance has improved in e501e25 after 18 minutes.


Option for using data-uri instead of file name for "Export for PixiJS Movie".

Add an option to put the data-uri in place of the of the file name for the "Export for PixiJS Movie" option. This allows for images to be kept with their JSON animation and to also be packed inline easily using tools like Webpack.
Current:
"meta": {
    "image": "spritesheet.png"
}

With option enabled:
"meta": {
    "image": "data:image/png;base64,iVBORw0KGgo..."
}

Steps to reproduce the bug

Export for PixiJS Movie.
Need additional "replace file name with data-uri" option.

Environment details

operating system: Ubuntu 17.10
offline application version: 0.14.0

I can see how this would be useful, would happily accept PRs for this :)


Adding rule 12-8-2

Oh, my dear - the results of this automatic merge thingy look disgusting. Gonna rebase next time.


The SQL execution generated by the export Oracle table data function will report an error

System information:

Operating system (distribution) and version: Windows 10 Enterprise Edition (Simplified Chinese), 21H1
DBeaver version: <IP_ADDRESS>202051448

Connection specification:

Database name and version: Oracle <IP_ADDRESS>.0
Driver name: Oracle (oracle.jdbc.OracleDriver)

Describe the problem you're observing:
When I used DBeaver to export Oracle table data, I found that the generated SQL statement used the feature of inserting multiple rows in a single statement that Oracle does not support. An example of the generated SQL statement is as follows:
INSERT INTO ISC_USR_PRIPWD (ID,NAME,LOGIN_NAME,PASSWORD) VALUES
	 ('40286c007aebcb8d017aebf4d4fe0004','zjb11','zjb11','C769CF2B9C97D9AD211D60D6ADAB962F'),
	 ('40286c007aefed1f017af5505e2e0044','zjb730','zjb730','7F7E23A043F0090AE8603472CE061B6D'),
	 ('8a9680817cc6286f017cc64182630004','fsir_002','fsir_003','7F7E23A043F0090AE8603472CE061B6D');

@bianyun1981
the number of the data rows per statement could be changed in Data Export preferences

@LonwoLonwo
Thank you for your reply. I didn't know that Oracle supports this form of syntax for inserting multiple rows with a single statement, so this configuration (Data rows per statement) makes sense for Oracle.
As for the fact that DBeaver saves preferences for different export settings based on different database types can be confusing, I agree that it is.
However, if a configuration (such as a configuration in export table data: "Upsert keyword") needs to be set to different values for different types of databases, then the saving of preferences for this configuration should not be saved globally. Even if this configuration value is not saved by database type, it should at least save one configuration value per connection, so that each connection only needs to set this configuration once. Otherwise, when switching between different types of databases, you may need to modify this configuration every time you switch.


Marketplace Contract Implementation

Objective: Develop a marketplace smart contract with fundamental methods to facilitate asset listing, buying, updating, and cancellation.
Methods:

init(admin: Address, percentage: i128):
Initialize the contract with the admin's address and a percentage for the admin fee.
list_asset(seller: Address, asset: Address, price: i128):
Allow sellers to list assets by specifying the seller's address, the asset's address, and the asset's price.
buy_asset(buyer: Address, asset: Address, price: i128):
Enable buyers to purchase assets by providing the buyer's address, the asset's address, and the agreed-upon price.
update_listing(seller: Address, asset: Address, old_price: i128, new_price: i128):
Permit sellers to update the price of a listed asset, ensuring they provide the correct seller and asset addresses, as well as the old and new prices.
cancel_listing(seller: Address, asset: Address, price: i128):
Allow sellers to remove a listing by specifying their address, the asset's address, and the price at which it was listed.

Checklist:

[x] Scope
[x] Implementation
[x] Integration Tests
[ ] Documentation

@mariopil @eloylp Let me hear any thoughts and suggestions.

#82 Was reviewed and approved - with pending impl at #90 - closing this - re-open if necessary.


Possible Memory Leak

Hi,
I'm trying to test Goose with 11M urls and I always have an out of memory due to jsoup after few pages. In the stack I can always see that jsoup accumulate lot of data. Is there any way to clean the extractor after a page has been processed?

I found where could be the problem. The jsoup ParseError accumulate tons of data coming from wrong html files (for examples images: http://stackoverflow.com/questions/8030303/how-do-you-disable-error-tracking-in-jsoups-tokeniser).
As you can see the only way to solve quickly the problem is to change some code in jsoup 1.6.1. But the real problem is not in jsoup but in the way data are provided to jsoup. I'll try to investigate how to solve the problem changing your code.

Do not want to bother you just to say that patching jsoup disabling error tracking solved the memory leak issue.
In the file org.jsoup.parser.Tokenizer.java just put to false:
private boolean trackErrors = false;
In jsoup 1.6.2 there's a new set/get trackErrors to disable this programmatically.
Hope It is useful.

ahhh this is great info, we just started hitting this as well, all our crawlers starting maxing out the heap. We're going to be digging into it today.

confirmed on our system as well
ParseError was using 4.67Gigs of RAM
num     #instances         #bytes  class name
1:     125474242<PHONE_NUMBER>  org.jsoup.parser.ParseError
2:        346511      532368208  [Ljava.lang.Object;
3:        279172      436142624  [C
4:         25886       69010632  [B
5:        821313       26282016  java.lang.String
6:        635160       25406400  java.util.LinkedHashMap$Entry
7:        112376       15290312
8:        251069       14059864  java.util.LinkedHashMap
9:        112376       13495952
10:        264109       10817536  [Ljava.util.HashMap$Entry;
11:          9220       10611888
12:        150087        9589640

I just added an issue to the jsoup repo: https://github.com/jhy/jsoup/issues/159
Until then, we will be going back to jsoup 1.5.2.


Choose a libc

It's become apparent to me since before the development reset that writing a C library and a kernel simultaneously can be a bit of a pain. I've decided that it might be better to integrate a third party libc, but I'm not sure which:

Newlib - is partially GPL'd, and while it is just parts of the build system, it still probably causes compatibility issues with Spinel's MIT license. It is not an option
uClibc-ng - is LGPL'd. With static linking, this applies: "If you statically link against an LGPLed library, you must also provide your application in an object (not necessarily source) format, so that a user has the opportunity to modify the library and relink the application." Since Spinel is open source, this shouldn't matter, but I imagine this does partially conflict with the MIT license used (which allows for binary distributions).
Musl - MIT license, so definitely compatible with Spinel's license. However, it requires either Linux's syscalls or a compatibility layer. This might be a good option.
PDCLib - Public domain is pretty cool! Doesn't support locales;
which means that's something I'd have to implement myself (or perhaps that someone else would have to implement).
FreeBSD's libc - This is licensed under the BSD license (unsurprisingly), which is compatible with Spinel's MIT license. Being built as part of FreeBSD, it is inherently tied into the operating system, whether that be via syscalls or via direct kernel integration. Still, this might be an option worth investigation.
Write my own - This will be the most permissive option and is intrinsically compatible with Spinel... but I'd have to write it. Honestly, I'm starting to feel like this might be a possibility.

If anyone has any thoughts, please let me know!

Now that I think about it, writing my own libc is starting to sound fun again, so I might just go with that. Leaving this open for a bit for any last minute comments


[opendistro-1.1][backport] Add validation for null elements in JSON array (#1157)

(cherry picked from commit 960d4209282027adedc0b37b28d74df8913dfafe)
opensearch-security pull request intake form
Please provide as much details as possible to get feedback/acceptance on your PR quickly

Category: (Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation)

Github Issue # or road-map entry, if available:

Description of changes:

Why these changes are required?

What is the old behavior before changes and new behavior after changes? (Please add any example/logs/screen-shot if available)

Testing done: (Please provide details of testing done: Unit testing, integration testing and manual testing)

TO-DOs, if any: (Please describe pending items and provide Github issues# for each of them)

Is it backport from main branch? (If yes, please add backport PR # and commits #)

By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1349 (2d7f4a8) into opendistro-1.1 (80e3670) will increase coverage by 0.06%.
The diff coverage is 100.00%.

@@                 Coverage Diff                  @@
##             opendistro-1.1    #1349      +/-   ##
====================================================
+ Coverage             63.53%   63.59%   +0.06%
- Complexity             3023     3035      +12
====================================================
  Files                   237      237
  Lines                 16567    16587      +20
  Branches               2965     2972       +7
====================================================
+ Hits                  10526    10549      +23
+ Misses                 4520     4518       -2
+ Partials               1521     1520       -1

Impacted Files
Coverage Δ

...est/validation/AbstractConfigurationValidator.java
83.78% <100.00%> (+2.53%)
:arrow_up:

...csearch/security/auditlog/sink/InternalESSink.java
69.23% <0.00%> (-11.54%)
:arrow_down:

...arch/security/filter/OpenDistroSecurityFilter.java
67.07% <0.00%> (-0.61%)
:arrow_down:

...search/security/tools/OpenDistroSecurityAdmin.java
46.91% <0.00%> (-0.27%)
:arrow_down:

...earch/security/auditlog/impl/AbstractAuditLog.java
75.00% <0.00%> (+1.07%)
:arrow_up:

...transport/OpenDistroSecuritySSLRequestHandler.java
59.03% <0.00%> (+6.02%)
:arrow_up:

Continue to review full report at Codecov.

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


Failed to switch to print dialogue in chrome to get element and press print button

First of all Thank you so much for the great framework which saved me lots of time.
my code automated chrome to open and print a pdf embedded on a internal web page(it is hard to get the link) and I succeeded to open the print dialogue of chrome.
I would like to then change the printer option and press the print button. I inspected the elements of the print dialogue (chrome://print/)
I used the demo code provided to switch because it seems to be on a different tab as the parent page but I still failed to get any elements on the print dialogue. May I know how could I do so?
    Dim targetTab As New CDPBrowser     Set targetTab = chrome.getTab("chrome://print/")     targetTab.wait

Hi, print dialogue is usually not an html dialogue but a windows dialogue. You can only automate such with UIA framework.


Define starts and ends for integrated stress response signaling (ISR)

currently there are 36 genes in human annotated to this which seems high
The definitions are inconsistent in how they are phrased, and whether they talk about starts or ends:

[] GO:0140467 ! integrated stress response signaling "The series of molecular signals generated in response to diverse stress stimuli required to restore cellular homeostasis. The core event in this pathway is the phosphorylation of eIF2 alpha by one of four members of the eIF2a kinase family (EIF2AK1/HRI, EIF2AK2/PKR, EIF2AK3/PERK and EIF2AK4/GCN2), which leads to a decrease in global protein synthesis and the induction of selected genes, including the transcription factor ATF4, that together promote cellular recovery."

[i] GO:0140469 ! GCN2-mediated signaling "A series of reactions in which a signal is passed on to downstream proteins within the cell via GCN2 (also known as EIF2AK4), an intracellular protein kinase that is activated by stress signals, such as amino acid starvation."
[i] GO:0140468 ! HRI-mediated signaling "A series of reactions in which a signal is passed on to downstream proteins within the cell via HRI (also known as EIF2AK1), an intracellular protein kinase that is activated by stress signals, such as heme deficiency, oxidative stress, osmotic shock, mitochondrial dysfunction and heat shock."
[i] GO:0039585 ! PKR/eIFalpha signaling "An intracellular signaling cassette that starts with activation and autophosphorylation of PKR (also known as EIF2AK2), which phosphorylates proteins including the translation initiation factor eIF2 to inhibit translation. PKR is activated by stress signals and during the antiviral response, activated by binding to viral double-stranded RNA (dsRNA) leading to inhibition of protein synthesis during viral infection."
[i] GO:0036499 ! PERK-mediated unfolded protein response "The series of molecular signals mediated by the endoplasmic reticulum membrane stress sensor PERK (PKR-like ER kinase). Begins with activation of PERK** in response to endoplasmic reticulum (ER) stress and ends with regulation of a downstream cellular process, e.g. transcription. The main substrate of PERK is the translation initiation factor eIF2alpha. Serine-phosphorylation of eIF2alpha by PERK inactivates eIF2alpha and inhibits general protein translation. In addition, eIF2alpha phosphorylation preferentially increases the translation of selective mRNAs such as ATF4 (activating transcription factor 4), which up regulates a subset of UPR genes required to restore folding capacity."

I propose we model ISR as starting with the activity of the member of the eIF2a kinase family.
Note this would rule out annotations to e.g DELE1
https://amigo.geneontology.org/amigo/reference/PMID:32132706
However, an argument could be made for including this (the def for the PKR form starts "starts with the activation of..."). But we should be clear and consistent!
The end point is trickier. The PERK form is the only one to talk about the end and it says a generic ends with regulation of a downstream cellular process, e.g. transcription. In this case, this could be anything. It could be argued that the endpoint is the transcription regulated by ATF-4 but some might consider it to be extended to include GADD34.
Tangentially related:
https://github.com/geneontology/noctua-alliance-pathway-preview/issues/16

@PCarme we can put integrated stress response on our target pathway list since fission yeast is used as a complementary
model our PI works on this ! https://pubmed.ncbi.nlm.nih.gov/33313903/
Also tagging @sylvainpoux  who has done a lot of work on the human annotation for this area.
So we have 2 experts.


add non-x86_64 architecture libfuzzer target support using qemu-user

This provides support to run non-x86_64 based targets using user mode QEMU.
This is done by wrapping the target_exe in a shell script that executes the target underneath qemu-(ARCH) as part of the task template.
Given these limitations, I've added warnings sent to the user in the CLI to indicate this is a preview feature and may change in the future.
Limitations:

Does not support live-repro.   This is actually simpler, rather than launching it in our own gdbserver, we can just use qemu's built in GDB.
Does not support coverage extraction.  We would need to replace the wrapper script with functionality in the agent that does the wrapping, move to using QEMU's gdb server support, and then fix up our gdb scripts to use target remote.

Note crash reporting is extremely slow.  Our default timeout of 5 seconds is far too short.  As such, this sets the timeout for significantly longer.

On windows, using --target_exe ./foo sends the target_exe to ./foo-wrapper.sh.  Due to a service issue, this is an invalid blob store path which causes a failure in the blob store library in the service.
The CLI shouldn't pass ./ here.  (and a follow-on, the service could attempt to catch ./ in the path to catch this as an error, rather than a 500 error when the blob store rejects it)

Follow-up PRs addressed the service issue.


Upload and display pdf files

Hi there !
I am discovering the ContentTools library, it is very impressive.
Is it possible to add a tool to upload a file to a server (pdf in my case) and then to generate the html code to display it (pdf.js in my case) ?
Thank in advance

Hey @ziguane ,
There's nothing built in for uploading files and creating a link to them but this is very possible. You could either look to implement something similar to the existing image uploader (see the tutorial http://getcontenttools.com/tutorials/handling-image-uploads) but instead upload a file and from the returned file URL create a link. Or use an existing third-party file upload like KCFinder to handle the file uploads and again generate a link in your document once you get a return URL.


Question about the feasibility of CSI data migration (receiver to Cloud platform) (AEGHB-514)

Hi there,
I'd like to address a question if it is possible to transfer the collected CSI data from the receiver part to the Cloud platform.
It aims to apply the CSI data packet for further algorithm analysis like machine learning.
I would expect to use the sub-project of "csi_recv" equipped with the wifi function listening to another ESP device (as a transmitter) and then migrate the collected CSI data to the Cloud platform from the open-source.
If it is feasible how I can achieve that and avoid the interruption from the CSI data collection (the receiver part)?
Just found a decent demonstration like this:

This approach is feasible. Have you encountered any specific issues during the implementation process? For example, did any interruptions occur while receiving CSI data?

Since my project terminated, abort this trial currently.

Since my project terminated, aborted this trial currently.


Added a missing word

Description of changes:
Added a missing word
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

thanks!


Rails bootsnap Cannot load such file -- tiktoken_ruby

Describe the bug
Cannot load such file -- tiktoken_ruby
File: https://github.com/patterns-ai-core/langchainrb/blob/main/lib/langchain/utils/token_length/openai_validator.rb has a require for require "tiktoken_ruby".
This happen to me in Travis build running specs, launching rails server or running specs doesn't fail in my localhost.
To Reproduce
Langchain::Utils::TokenLength::OpenAIValidator
Expected behavior
An error about a missing dependency should not happen.
Terminal commands & output
Failure/Error: require_relative '../config/environment'
LoadError:
  cannot load such file -- tiktoken_ruby
# ./vendor/bundle/ruby/3.2.0/gems/bootsnap-1.18.3/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:27:in `require'
# ./vendor/bundle/ruby/3.2.0/gems/zeitwerk-2.6.15/lib/zeitwerk/kernel.rb:34:in `require'
# ./vendor/bundle/ruby/3.2.0/gems/langchainrb-0.13.3/lib/langchain/utils/token_length/openai_validator.rb:3:in `<main>'
# ./vendor/bundle/ruby/3.2.0/gems/bootsnap-1.18.3/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:30:in `require'
# ./vendor/bundle/ruby/3.2.0/gems/zeitwerk-2.6.15/lib/zeitwerk/kernel.rb:26:in `require'
# ./vendor/bundle/ruby/3.2.0/gems/zeitwerk-2.6.15/lib/zeitwerk/loader/helpers.rb:148:in `const_get'
# ./vendor/bundle/ruby/3.2.0/gems/zeitwerk-2.6.15/lib/zeitwerk/loader/helpers.rb:148:in `cget'
# ./vendor/bundle/ruby/3.2.0/gems/zeitwerk-2.6.15/lib/zeitwerk/loader/eager_load.rb:175:in `block in actual_eager_load_dir'
# ./vendor/bundle/ruby/3.2.0/gems/zeitwerk-2.6.15/lib/zeitwerk/loader/helpers.rb:42:in `block in ls'
# ./vendor/bundle/ruby/3.2.0/gems/zeitwerk-2.6.15/lib/zeitwerk/loader/helpers.rb:25:in `each'
# ./vendor/bundle/ruby/3.2.0/gems/zeitwerk-2.6.15/lib/zeitwerk/loader/helpers.rb:25:in `ls'
# ./vendor/bundle/ruby/3.2.0/gems/zeitwerk-2.6.15/lib/zeitwerk/loader/eager_load.rb:170:in `actual_eager_load_dir'
# ./vendor/bundle/ruby/3.2.0/gems/zeitwerk-2.6.15/lib/zeitwerk/loader/eager_load.rb:17:in `block (2 levels) in eager_load'
# ./vendor/bundle/ruby/3.2.0/gems/zeitwerk-2.6.15/lib/zeitwerk/loader/eager_load.rb:16:in `each'
# ./vendor/bundle/ruby/3.2.0/gems/zeitwerk-2.6.15/lib/zeitwerk/loader/eager_load.rb:16:in `block in eager_load'
# ./vendor/bundle/ruby/3.2.0/gems/zeitwerk-2.6.15/lib/zeitwerk/loader/eager_load.rb:10:in `synchronize'
# ./vendor/bundle/ruby/3.2.0/gems/zeitwerk-2.6.15/lib/zeitwerk/loader/eager_load.rb:10:in `eager_load'
# ./vendor/bundle/ruby/3.2.0/gems/zeitwerk-2.6.15/lib/zeitwerk/loader.rb:413:in `block in eager_load_all'
# ./vendor/bundle/ruby/3.2.0/gems/zeitwerk-2.6.15/lib/zeitwerk/loader.rb:411:in `each'
# ./vendor/bundle/ruby/3.2.0/gems/zeitwerk-2.6.15/lib/zeitwerk/loader.rb:411:in `eager_load_all'
# ./vendor/bundle/ruby/3.2.0/gems/railties-<IP_ADDRESS>/lib/rails/application/finisher.rb:80:in `block in <module:Finisher>'
# ./vendor/bundle/ruby/3.2.0/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:32:in `instance_exec'
# ./vendor/bundle/ruby/3.2.0/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:32:in `run'
# ./vendor/bundle/ruby/3.2.0/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:61:in `block in run_initializers'
# ./vendor/bundle/ruby/3.2.0/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:60:in `run_initializers'
# ./vendor/bundle/ruby/3.2.0/gems/railties-<IP_ADDRESS>/lib/rails/application.rb:426:in `initialize!'
# ./config/environment.rb:5:in `<top (required)>'
# ./spec/rails_helper.rb:4:in `require_relative'
# ./spec/rails_helper.rb:4:in `<top (required)>'
# ./spec/components/meta_title/component_spec.rb:3:in `<top (required)>'

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: OS X
Ruby version 3.2.2
Langchain.rb version 0.13.3

Additional context
Add any other context about the problem here.

I ran into this yesterday within Github Actions. The gem specifies ruby_tiktoken as an optional dependency, but it looks like if your code is using the OpenAI adapter, ruby_tiktoken is required. The workaround/fix for me was just to explicitly add it to my Gemfile: bundle add tiktoken

@bknoles That would do it, buy then it should be in the docs.

Yeah, I wasn't suggesting it as a long term fix. Just wanted to document the workaround I used to keep my side project moving!
https://xkcd.com/979/

I've gotten rid of the token length validators altogether here: https://github.com/patterns-ai-core/langchainrb/pull/704. I don't think they were super useful. I'm open to adding them later if we ever figure out how to do it in a much cleaner way.


[ENG-432] Update Links to Point to New Knowledge Base

Purpose
Update links to point to new knowledge base on figshare
Changes
Update links in emails
Other minor changes to email copy (standardize email signature, etc.)
QA Notes
This shouldn't need QA
Documentation
Side Effects
Ticket
https://openscience.atlassian.net/browse/PRODUCT-936

Merging into feature branch feature/raticate for testing on staging3


Change the cookie policy links

What's to be corrected:
The link we provide in the cookie banner takes us to the Squarespace cookie page. It needs to take us to our cookie page, here in the platform.
How should it look like (screenshot or sketch):
Our cookies explanation is included in our Privacy Policy, so we should point our visitors to this link: https://www.whichost.com/legal/privacy-policy.
How should it work (write use case):
N/A. It's a change of link for the cookie policy.
Priority (tag accordingly): Urgent
Domain (tag accordingly): Frontend
Tested on: (platform, OS, browser, version) Mac, latest, Chrome, latest
Branch: Master
Work on it

Work on staging or master (if it's urgent)
Tag with ongoing and move the card to ongoing.
Correct the default
Commit your work
Create a PR (if it's urgent) and ask for validation. Move the card to staged.
Close the issue if all the work have been done

** Validate **

Check on mobile and on desktop if the default have been correctly corrected.
If everything is working fine, move the card toaccepted column.
If the feature isn't completely implemented, re-open the issue with a comment clearly what the problem is and what is the desired result. Move the card to have issues column.

Please follow https://github.com/Whichost/frontend/wiki/Git-workflow

What's to be corrected:
The link we provide in the cookie banner takes us to the Squarespace cookie page. It needs to take us to our cookie page, here in the platform.
How should it look like (screenshot or sketch):
Our cookies explanation is included in our Privacy Policy, so we should point our visitors to this link: https://www.whichost.com/legal/privacy-policy.
How should it work (write use case):
N/A. It's a change of link for the cookie policy.
Priority (tag accordingly): Urgent
Domain (tag accordingly): Frontend
Tested on: (platform, OS, browser, version) Mac, latest, Chrome, latest
Branch: Master
Work on it

Work on staging or master (if it's urgent)
Tag with ongoing and move the card to ongoing.
Correct the default
Commit your work
Create a PR (if it's urgent) and ask for validation. Move the card to staged.
Close the issue if all the work have been done

** Validate **

Check on mobile and on desktop if the default have been correctly corrected.
If everything is working fine, move the card toaccepted column.
If the feature isn't completely implemented, re-open the issue with a comment clearly what the problem is and what is the desired result. Move the card to have issues column.

Please follow https://github.com/Whichost/frontend/wiki/Git-workflow

@lcoenen @Shane-Walsh
Issue has been updated.

@lcoenen @Shane-Walsh
This issue was created 12 days ago! It is marked as urgent. Where is the urgency of solving this then?
This is what our visitors see today:

Why?

What have been done: Correct the link
Pushed to: master
Tag somebody: @Shane-Walsh
Asking for validation:

Check on mobile and on desktop if the issue have been correctly fixed.
If everything is correct, move the card to accepted column.
If the issue isn't fixed, re-open the comment clearly what the problem is and what is the desired result.
Move the card to have issues column.
Merge (or express approval in) the corresponding PR if needed. Move the card to the launch column.


Make Gradle 8 the default version

What?
This PR makes Gradle 8 the default version. The first release for Gradle 8 was made on Feb 13, 2023, so it should be safe to make it default (coming from version 7).

[x] Tests are added/updated if needed
[x] Docs are updated if needed

@aleksrutins, somewhat.
But I think it is better to have these changes in different pull requests, though. As far as I know, supporting Java 21 does not require making Gradle 8 the default.

Just supporting Java 21 doesn't necessarily; but, since it's the new LTS version, #1081 makes it the default, which does require changing the default Gradle to 8.

This is not blocked by #1020, but it would be nice to get that done first.

not stale

not stale

@coffee-cup @aleksrutins, reviving this. :-)
This is good to go (I've solved a few linting/formatting issues unrelated to my original changes).


Naming of NH3 flux components when post-processing with the combine utility

CMAQv5.3.1-i6
Date: 2020-03-09
Contact: Christian Hogrefe<EMAIL_ADDRESS>Description: In CMAQv5.3.1, there is an inconsistency in the naming convention of the bi-directional NH3 flux components contained in the CCTM_DRYDEP file between the STAGE and M3DRY dry deposition options. When running CMAQ with the NH3 bi-directional flux option enabled (CTM_ABFLUX set to Y in the run script), the model calculates and stores three NH3 flux components in the CCTM_DRYDEP file: the downward deposition flux, the upward emissions flux, and the net flux representing the difference between the deposition and emissions fluxes. The table below shows how these flux components are named in the CCTM_DRYDEP file depending on whether the STAGE or M3DRY dry deposition option is used.

STAGE
M3DRY

Downward Deposition Flux (always positive)
NH3
NH3_DDEP

Upward Emissions Flux (always positive)
NH3_Emis
NH3_EMIS

Net Flux (positive if downward and negative if upward)
NH3_Flux
NH3

When using the STAGE dry deposition option with bi-directional NH3 flux enabled, the CCTM_DRYDEP file contains additional diagnostic deposition values (NH3_Stom, NH3_Cut, NH3_Soil, NH3_Ag, NH3_Nat, and NH3_Wat) that are described in the CMAQv5.3.1 Release Notes.
Note that when the model is run without the bi-directional NH3 flux option enabled (CTM_ABFLUX set to N in the run script), the variable NH3 in the CCTM_DRYDEP file represents the unidirectional NH3 dry deposition flux in both STAGE and M3DRY.
Scope and Impact: These differences in naming conventions need to be considered when performing post-processing of CCTM_DRYDEP outputs. For example, the “combine” example species definition file for deposition fields utilizes the CCTM_DRYDEP variable “NH3” when defining variables “DDEP_NH3” and “DDEP_NHX” (see for example see the CB6r3_AE7 species definition file under CCTM/src/MECHS/cb6r3_ae7_aq).
Solution: If the intent of these derived fields is to represent downward deposition fluxes, no changes are needed if CMAQ was run without the bi-directional NH3 flux option with either STAGE or M3DRY, or if CMAQ was run with the bi-directional NH3 flux option and the STAGE dry deposition option. However, if CMAQ was run with the bi-directional NH3 flux option and the M3DRY dry deposition option, these definitions in the species definition file should be changed from “NH3” to “NH3_DDEP”.

Resolved with release of v5.3.2.
See v5.3.2 Release Notes


Really fix always run

These patches revert the original always_run fix and use check_mode: no instead.

This PR should fix the errant logic from #2037.

LGTM.


Emojis Emotion Prediction

Submitting task for top emotion intensive emojis and willing to extend it for all 150 emojis https://github.com/abushoeb/emotag

@googlebot I fixed it.

@googlebot I fixed it.

@googlebot I fixed it.

@googlebot I fixed it.

@googlebot I consent.

Hi, I am one of the reviewers assigned to review this task.
Thank you for submitting this interesting task.
Feedback according to the review criteria:
Correctness: Good, passed the automatic tests
Formatting: The keywords listed in the Readme file are not the same as in the task.json
Specificity and thoroughness: I am a bit confused whether this task is testing if the language model is able to estimate the strength of association that humans have between certain emojis and eight human emotions (in this case it is a multiple choice task), or it is testing if the model is able to select one emotion with the strongest association (in this case it is not multiple choice).  Please clarify in the Readme and amend your task.json if the task is not multiple choice.
Difficulty: This task is challenging and most probably does not reach the perfect agreement among multiple human raters. My suggestion would be to add a level of inter-rater agreement to the description of the task, to represent the level of its difficulty (optional).
Not solvable by memorizing the Internet: The data for this task might appear in training corpora as the dataset was previously published. I would suggest testing the task on existing language models (using e.g. this example code) to provide the baseline scores .
Novelty: Good.
Justification: Good.
Size: Good. The task includes 131 input-output pairs of examples.
Compute resources: Feasible.
Additionally, please add the dataset licensing information to the content of Readme.

“@chiafullo  requires licensing review "

Hello, I am your second reviewer. I agree with the first jeknov's points and add just one:

Formatting: the inputs being given the model seems unclear without the additional context that this is a categorization problem. If someone texted me " Option:" I would be hard pressed to understand how to respond. I think you should format your inputs like "What emotion corresponds to ".

@abushoeb We are now requesting that task authors please include an explicit section in the README.md file called "data source", listing exactly where you got your data from (or explicitly state that you made it up yourself). Many thanks!

Hello, I am your second reviewer. I agree with the first jeknov's points and add just one:

Formatting: the inputs being given the model seems unclear without the additional context that this is a categorization problem. If someone texted me " Option:" I would be hard pressed to understand how to respond. I think you should format your inputs like "What emotion corresponds to ".

Good point to replace "Option" with "What emotion corresponds to ". It's updated now, thank you!

@abushoeb We are now requesting that task authors please include an explicit section in the README.md file called "data source", listing exactly where you got your data from (or explicitly state that you made it up yourself). Many thanks!

Done, thanks!

Hi, I am one of the reviewers assigned to review this task.
Thank you for submitting this interesting task.
Feedback according to the review criteria:
Correctness: Good, passed the automatic tests
Formatting: The keywords listed in the Readme file are not the same as in the task.json
Specificity and thoroughness: I am a bit confused whether this task is testing if the language model is able to estimate the strength of association that humans have between certain emojis and eight human emotions (in this case it is a multiple choice task), or it is testing if the model is able to select one emotion with the strongest association (in this case it is not multiple choice). Please clarify in the Readme and amend your task.json if the task is not multiple choice.
Difficulty: This task is challenging and most probably does not reach the perfect agreement among multiple human raters. My suggestion would be to add a level of inter-rater agreement to the description of the task, to represent the level of its difficulty (optional).
Not solvable by memorizing the Internet: The data for this task might appear in training corpora as the dataset was previously published. I would suggest testing the task on existing language models (using e.g. this example code) to provide the baseline scores .
Novelty: Good.
Justification: Good.
Size: Good. The task includes 131 input-output pairs of examples.
Compute resources: Feasible.
Additionally, please add the dataset licensing information to the content of Readme.

Thanks for your detailed review. I really appreciate it.
The keywords in readme are part of the header which we are instructed to modify it. Can you please clarify it? If this not the case then I can certainly update the Readme to reflect the same keywords from task.json file. Besides that, the majority of the issues have been addressed in the new commit.

@ErrWare are one of the assigned reviewers for this task.  Please provide feedback and be sure to notify me with a formal accept or does not meet criteria by Tuesday June 22 (if you have not already done so).

@jeknov are one of the assigned reviewers for this task.  Please provide feedback and be sure to notify me with a formal accept or does not meet criteria by Tuesday June 22 (if you have not already done so).

@abushoeb you are right, the header seems to be overwritten on merge anyway, so probably no need to update it manually. Thanks for making the other changes!

@jeknov Thanks for your valuable feedback! Regarding your question about the task setup, in all multiple choice tasks in BIG-Bench, a model is given multiple choices and will choose a single one of them as its prediction (the one with the highest LM probability).

@gdemelo That's true, in the multiple choice tasks a model chooses a single output as a prediction. This is why the target_scores should be either 0 or 1 in order for the multiple_choice_grade to work properly. Please have a look at the examples of json files of the other multiple choice tasks in Big-Bench. Please also note that the section Behavior of existing language models on task should be in the Readme.

@jeknov BIG-bench allows multiple choice tasks to have non-zero scores for multiple choices per example. To maximize its score, a model still needs to pick the ideal answer for each example. If the model occasionally chooses an answer that is not the ideal one, but still a reasonably good one, it makes sense to give it a score that is lower than that of the best answer but greater than 0. This enables fine-grained comparisons between models. However, if an all-or-nothing scoring scheme is deemed preferable by the reviewers, the task can easily be modified to adopt such a scheme.
Regarding previous language models, the regular "gpt2" model obtains 0.2101526717557252 in a zero-shot evaluation, and 0.2715267175572518 in a 1-shot evaluation.

@gdemelo Thank you for the explanation. It makes sense to use a more granular scoring for the purpose of fine-grained comparisons. Please add these details to the Design Considerations in the Readme. Thanks for reporting the results on the GPT2, please also add this to the Readme.

The amount of tasks we received this round is tremendous.  With so many tasks to review we have decided to extend the review period to Tuesday, June 29.
Reviewers: if the submitter has made revisions please be sure to notify me with a formal ”accept” or ”does not meet criteria” by Tuesday, June 29  (if you haven't already done so).
this is an automated message

I think this task is in a good state. I agree with the benefit of finer-grained scoring.

@chiafullo  Accept

Your task has been accepted in the initial review stage.  The next stage of this process is for us to assign your task a meta-reviewer for final review and merge.  An assigned meta-reviewer will follow-up by commenting on the PR should it need further revisions.

Thanks for the update. I look forward to the next steps!

Hi @abushoeb, thank you for gathering this dataset, and for your BIG-Bench contribution! I'm the meta reviewer for this task.
It looks like there might be some issues with the formatting of the task. What I see when I run the tests is:
Q: [emoji]What emotion corresponds to [choice1]What emotion corresponds to[choice2]...

A: joy

It appears you have "What emotion corresponds to" as your choice_prefix at the moment. You should instead use this, or something like it, as your task_prefix, and your choice_prefix, can be removed or replaced. As an example for how I might imagine posing this question to a human:
Select the emotion that most closely corresponds to the presented emoji.

Q: [emoji]
A: joy
...

Could you please make the necessary adjustments?

Hi @vedant, thanks for providing your valuable feedback. I’m certainly going to make the changes.

@googlebot I consent


pkg_resources.ResolutionError: No script named 'django-admin.py'

Hi;
I installed graphite in redhat 7.4, I use the Django.1.11.13 like requirement
so when I make this commande ligne to create the databases :
PYTHONPATH=/usr/share/graphite/webapp django-admin syncdb --settings=graphite.settings
I got this error:
Traceback (most recent call last):
File "/usr/bin/django-admin", line 5, in
management.execute_from_command_line()
File "/usr/lib/python2.7/site-packages/django/core/management/init.py", line 364, in execute_from_command_line
utility.execute()
File "/usr/lib/python2.7/site-packages/django/core/management/init.py", line 308, in execute
settings.INSTALLED_APPS
File "/usr/lib/python2.7/site-packages/django/conf/init.py", line 56, in getattr
self._setup(name)
File "/usr/lib/python2.7/site-packages/django/conf/init.py", line 41, in _setup
self._wrapped = Settings(settings_module)
File "/usr/lib/python2.7/site-packages/django/conf/init.py", line 110, in init
mod = importlib.import_module(self.SETTINGS_MODULE)
File "/usr/lib64/python2.7/importlib/init.py", line 37, in import_module
import(name)
ImportError: No module named graphite.settings
So I read some issues in git
I make this commande to fix my problem :
PYTHONPATH=/opt/graphite/webapp django-admin.py migrate --settings=graphite.settings --run-syncdb
so I got another error that's :
Traceback (most recent call last):
File "/usr/bin/django-admin.py", line 4, in
import('pkg_resources').run_script('Django==1.11.13', 'django-admin.py')
File "/usr/lib/python2.7/site-packages/pkg_resources.py", line 540, in run_script
self.require(requires)[0].run_script(script_name, ns)
File "/usr/lib/python2.7/site-packages/pkg_resources.py", line 1449, in run_script
raise ResolutionError("No script named %r" % script_name)
pkg_resources.ResolutionError: No script named 'django-admin.py'
someone can help me?
Thank you for your help.

Where did you find that doc? It should be updated, it now is manage.py instead of graphite-admin.py

According to Google search if django-admin.py saying pkg_resources.ResolutionError: No script named 'django-admin.py' that means some Django installation error, e.g. mixing different version of Django installed with different ways (e.g. through pip and easy_instyall). I didn't find any solution for that case, though.
But you can try what @piotr1212 said - use manage.py or python -m django instead - https://docs.djangoproject.com/en/2.1/ref/django-admin/

I Make this commande :
]# PYTHONPATH=/usr/share/graphite/webapp manage.py syncdb --settings=graphite.settings
-bash: manage.py: command not found

]# PYTHONPATH=/usr/share/graphite/webapp manage.py syncdb --settings=graphite.settings
-bash: manage.py: command not found

PYTHONPATH=/usr/share/graphite/webapp manage.py syncdb --settings=graphite.settings
-bash: manage.py: command not found

I tried django-admin (without .py) but then it dies with "ImportError: cannot import name generic" which stackoverflow answer with "The django.contrib.contenttypes module has been reorganized in Django 1.7"


Supporting Python3.7

Python 3.7.0 had been shipped in June 2018, PEP537, I think it's better to pre-empt the end-of-life for Python 3.6 than to wait for user uptake and rush for a build.
I think we're safe to have a tentative support for 3.7 later this Dec 2018 when 3.6.8 is shipped. End of life for Python 3.6 is Dec 2012 =)
This should help when porting code https://docs.python.org/3/whatsnew/3.7.html
It would be good if NLTK supports Python3.7 and the CI should be updated accordingly.
Any brave soul?

Now that CI is fully running and passing on 3.7 is there anything else that needs to be done in order to 'officially' support 3.7 for the library? Other than updating docs and specifying versions for pypi.


DOC: hyperlink registry

I test locally later.
You can add a rtd build check using the site (click button at botton of image)

couldn't get the hyperlink to work. I belive because https://filesystem-spec.readthedocs.io/en/latest/_modules/fsspec/registry.html isn't part of the API.

Feel free to add it. The registry could probably do with more work to include searching for entrypoints anyway.

bit unsure where to put it. Dropped it in Built-in Implementations but not sure if autofunction is appropriate. The hyperlink works but it's not very useful. Ideally want to show the dict when you click on the hyperlink from the home page
"file": {"class": "fsspec.implementations.local.LocalFileSystem"},
    "memory": {"class": "fsspec.implementations.memory.MemoryFileSystem"},
    "dropbox": {
        "class": "dropboxdrivefs.DropboxDriveFileSystem",
        "err": (
            'DropboxFileSystem requires "dropboxdrivefs",'
            '"requests" and "dropbox" to be installed'
        ),
    },
...

A couple of possible solutions for directly including the dictionary: https://stackoverflow.com/questions/27875455/displaying-dictionary-data-in-sphinx-documentation/29789910

(ping)

Sorry, this seems to have got lost. Still interested?


Simple API for receiving external metrics from agent

:+1:
On Thursday, January 8, 2015, Artyom Keydunov<EMAIL_ADDRESS>wrote:

You can merge this Pull Request by running
git pull https://github.com/keydunov/snowman-io master
Or view, comment on, or merge it at:
https://github.com/snowman-io/snowman-io/pull/65
Commit Summary

Simple API for receiving external metrics from agent

File Changes

M lib/snowman-io/api.rb
https://github.com/snowman-io/snowman-io/pull/65/files#diff-0 (10)

Patch Links:

https://github.com/snowman-io/snowman-io/pull/65.patch
https://github.com/snowman-io/snowman-io/pull/65.diff

—
Reply to this email directly or view it on GitHub
https://github.com/snowman-io/snowman-io/pull/65.

--
Алексей Вахов


Home Screen UX issues

Note: These issues only relate to the default (no credentials) state of the Home Screen, such as if just installed.

[ ] Header label should be "BC Wallet"
[x] "You have no...." line should end in a period
[x] 'No notifications' text should be regular weight, not bold
[ ] More space needed below notification (and before heading / "You have" line)
[x] There is no settings icon in top-right of header
[x] Font is not BC Sans (see separate ticket)
[ ] Margin on right side of notifications blue panel should be the same as on left side
[ ] Right margin between notification text box (e.g. containing “You have no new notifications”) and right edge of blue box should be wider, to match margin between info icon and left edge of blue box.
[ ] Top header area needs shadow onto body area

For >Header label should be "BC Wallet" this needs to be added at build time.

Right margin between notification text box (e.g. containing “You have no new notifications”) and right edge of blue box should be wider, to match margin between info icon and left edge of blue box.

The designs show the info boxes as left aligned. Should they be centered?


New permission lvl: specific people only

This would be permission lvl 4 (currently owner only)
Will need a new command override dict where a whitelist of people can be stored per command of people to grant perm lvl 4 for this command
currently lvl 4 and 5 will become 5 and 6, needs automatic config conversions to not screw up current configrations

Will specific people, AND a trusted role (for example) be a setting?

yes, as mentioned above trusted would be a new one that is above admin, the idea is that you can have a server with like entry verification, or not have the bot public, so coinflip and such can be locked down to only vetted members, but not give them mod powers
specific people would be more for people who want to lock some commands to specific people, this would be per command, so giving a single person access to the configure command, or other commands (like announce or such later), that you may not want moderators able to use


AN-186 Clean up pools from Google Batch enablement

This is a follow-on to #367 now that the non-prod pools there have been (mostly) put into use.

Update test utils to use new pool version
Remove the old versions of pools from pool_schema.yml files
Remove ALL resource-config files for unused pools

Questions for reviewers:

Do we want to remove resource-config files for older pools? Is this desirable cleanup or am I deleting a useful record?
In #367 I added new versions of all non-TDR pools in environments I didn't know to be defunct (perf, alpha), because this change should apply to ALL new Terra workspace Google projects. In a couple cases, though, I couldn't find where the pool was used, so I couldn't update that usage to the new pool version. This leaves us with a newer pool with the correct settings, and an older pool that may or may not be in use somewhere, and so can't be removed. What's the best way to handle this? The two pools in this state are tools/cwb_ws_fiab_v7 and toolsalpha/resource_toolsalpha_v8.

I can't find any reference to tools/cwb_ws_fiab_v7, or v8 for that matter. Maybe their use died with FIABs. The comment here says resource_toolsalpha is for local testing. Maybe you want another PR for it, but I think you can drop cwb_ws_fiab all together and drop the old version of resource_toolsalpha


Don't report duplicate folding ranges

I’m not entirely sure when this started occurring but when retrieving folding ranges e.g. for
func foo() {
    print("someText")
}

We report the folding range of "someText" twice: Once as the body of the call to print and once as the body of the first argument. Each folding range on its own makes sense because you might want to collapse either all arguments or only a single argument. But it doesn’t make sense to report the same range twice.
To fix this, remove duplicate ranges from the folding ranges.

@swift-ci Please test

@swift-ci Please test


hybrid store caching misses docker media types but ChildrenHandler expects them

The caching code is here:
	if isAllowedMediaType(wOpts.Desc.MediaType, ocispec.MediaTypeImageManifest, ocispec.MediaTypeImageIndex) || s.ingester == nil {
		return s.cache.Writer(ctx, opts...)
	}

However, the containerd Children handler here:
	switch desc.MediaType {
	case MediaTypeDockerSchema2Manifest, ocispec.MediaTypeImageManifest:
		p, err := content.ReadBlob(ctx, provider, desc)
		if err != nil {
			return nil, err
		}

		// TODO(stevvooe): We just assume oci manifest, for now. There may be
		// subtle differences from the docker version.
		var manifest ocispec.Manifest
		if err := json.Unmarshal(p, &manifest); err != nil {
			return nil, err
		}

		descs = append(descs, manifest.Config)
		descs = append(descs, manifest.Layers...)
	case MediaTypeDockerSchema2ManifestList, ocispec.MediaTypeImageIndex:
		p, err := content.ReadBlob(ctx, provider, desc)
		if err != nil {
			return nil, err
		}

		var index ocispec.Index
		if err := json.Unmarshal(p, &index); err != nil {
			return nil, err
		}

		descs = append(descs, index.Manifests...)
	default:
		if IsLayerType(desc.MediaType) || IsKnownConfig(desc.MediaType) {
			// childless data types.
			return nil, nil
		}
		log.G(ctx).Debugf("encountered unknown type %v; children may not be fetched", desc.MediaType)
	}

expects the ability to read from the provider (in our case, the cache) for all four media types (OCI manifest and index, docker manifest and manifest list). This means that if we actually do pull down some such from the docker types, it will try to read it, but it isn't cached, and will return a "not found" error.
The simplest solution appears to be to add the two media types - I have a PR about to submit for it - but I am concerned that the lack of passthrough (see #225) might mean this would break something else.

It is by design that oras works only with OCI manifests and index since oras stands for OCI Registry As Storage.
If you are interested in docker manifests for private uses, you may take a look at https://github.com/shizhMSFT/oras/tree/shizh/docker_manifest.

Actually, I agree with you, in principle. I far prefer to use OCI media types.  It is entirely possible to have to deal with a registry that only supports legacy formats. ORAS mostly handles that ok, but every now and then runs into an issue, like here.

I'd love technical purity on this, but we should be able to handle it if it isn't that way.

Why even have it in a supported media type if we will break?


Scala Hadoop Shred: finish tests

There are some incomplete specs in the test suite.

If the only incomplete test is #919, then we can de-schedule and close this one.

No more incomplete tests #919 removed and #1960 restored.


[WIP] Add support for different registries for Kaniko publisher

Possible solution for issue: https://github.com/apache/camel-k/issues/613
Please use this PR only as an example.

Thanks for your PR. I think there should be something more to make it work, but the direction seems good.
One thing I don't understand in Kaniko doc is how it recognizes which secret to use for pushing. In the example you've linked, it mounts the AWS secret into /root/.aws. Do we have to do something like this for custom registries?
What tells kaniko to look into /root/.aws? Is it something in the json config file?

@nicolaferraro my understanding is that kaniko provides support for some credential helpers by default that are activated by docker's config.json.
To support pushing to different registries from the kaniko build, I think we need to support the definition of arbitrary configmap/secrets to be mounted in the build configuration spec.

This PR is half year old. What is the status? Or should it be closed?

This has been implemented in #1084


Update developer guide formatting (Dev Ops and User Stories)

Pull Request Test Coverage Report for Build 27

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 91.546%

Totals

Change from base Build 26:
0.0%

Covered Lines:
1516

Relevant Lines:
1656

💛  - Coveralls

Any reason why dependencies is in italics in section 7.6?

What have you done...


encase example vars as {variable}

Instead of creating the hcl with values and this note, such as:
 region = "us-west-2"
and
Note: Make sure to replace the ami, key_name, and regionvalues with your own values.
How about defining variables:
 region = "{region1}"
(The number in the variable is when multiple regions are defined, which is typical of production hcl. Some would prefer {primary_region}. So a post-processor could make those changes if the values inserted are consistent.

Hi there. Sorry for taking long to respond. I think trying to post-process responses in that way will be error prone, and it's also questionable whether modifying the responses from the provider is a good idea strategically.
We could, theoretically, add wording to the prompts we send to the provider that instruct it to use variables in its responses, but this will use up more tokens. For now, I think it's better to use the chat feature and instruct the provider to return variables.
I'm closing this, as we will not be moving along with this feature.


len() used with inducing variables does not work with tf.function

Bug / performance issue / build issue
I create my own inducing variable class which has variable shape, rewriting the len function, but when I optimize the model I got an error calling that "'Tensor' object cannot be interpreted as an integer" on len function. I think is an unfortunate interaction between how len() is handled in python and tf.function.
To reproduce
Minimal, reproducible example

import numpy as np
import gpflow
import tensorflow as tf

class VariableInducingPoints(gpflow.inducing_variables.InducingPoints):
     def __init__(self, Z, name=None):
         super().__init__(Z, name=name)
         # overwrite with Variable with None as first element in shape so
         # we can assign arrays with arbitrary length along this dimension:
         self.Z = tf.Variable(Z, trainable=False, dtype=gpflow.default_float(),
             shape=(None, Z.shape[1])
         )

     def __len__(self):
         return tf.shape(self.Z)[0]  # dynamic shape
         # instead of the static shape returned by the InducingPoints parent class

X, Y = np.random.randn(50, 2), np.random.randn(50, 1)
Z1 = np.random.randn(13, 2)

k = gpflow.kernels.SquaredExponential()
m = gpflow.models.SGPR(data=(X, Y), kernel=k, inducing_variable=VariableInducingPoints(Z1))

Z2 = np.random.randn(29, 2)
m.inducing_variable.Z.assign(Z2)

opt = tf.optimizers.Adam()

@tf.function
def optimization_step():
    opt.minimize(m.training_loss, m.trainable_variables)

for _ in range(iter):
    optimization_step()

Stack trace, or error message
TypeError                                 Traceback (most recent call last)
<ipython-input-24-9a082736eedc> in <module>
     38
     39
---> 40     optimization_step()
     41
     42

~/anaconda3/lib/python3.8/site-packages/tensorflow/python/eager/def_function.py in __call__(self, *args, **kwds)
    778       else:
    779         compiler = "nonXla"
--> 780         result = self._call(*args, **kwds)
    781
    782       new_tracing_count = self._get_tracing_count()

~/anaconda3/lib/python3.8/site-packages/tensorflow/python/eager/def_function.py in _call(self, *args, **kwds)
    821       # This is the first call of __call__, so we have to initialize.
    822       initializers = []
--> 823       self._initialize(args, kwds, add_initializers_to=initializers)
    824     finally:
    825       # At this point we know that the initialization is complete (or less

~/anaconda3/lib/python3.8/site-packages/tensorflow/python/eager/def_function.py in _initialize(self, args, kwds, add_initializers_to)
    694     self._graph_deleter = FunctionDeleter(self._lifted_initializer_graph)
    695     self._concrete_stateful_fn = (
--> 696         self._stateful_fn._get_concrete_function_internal_garbage_collected(  # pylint: disable=protected-access
    697             *args, **kwds))
    698

~/anaconda3/lib/python3.8/site-packages/tensorflow/python/eager/function.py in _get_concrete_function_internal_garbage_collected(self, *args, **kwargs)
   2853       args, kwargs = None, None
   2854     with self._lock:
-> 2855       graph_function, _, _ = self._maybe_define_function(args, kwargs)
   2856     return graph_function
   2857

~/anaconda3/lib/python3.8/site-packages/tensorflow/python/eager/function.py in _maybe_define_function(self, args, kwargs)
   3211
   3212       self._function_cache.missed.add(call_context_key)
-> 3213       graph_function = self._create_graph_function(args, kwargs)
   3214       self._function_cache.primary[cache_key] = graph_function
   3215       return graph_function, args, kwargs

~/anaconda3/lib/python3.8/site-packages/tensorflow/python/eager/function.py in _create_graph_function(self, args, kwargs, override_flat_arg_shapes)
   3063     arg_names = base_arg_names + missing_arg_names
   3064     graph_function = ConcreteFunction(
-> 3065         func_graph_module.func_graph_from_py_func(
   3066             self._name,
   3067             self._python_function,

~/anaconda3/lib/python3.8/site-packages/tensorflow/python/framework/func_graph.py in func_graph_from_py_func(name, python_func, args, kwargs, signature, func_graph, autograph, autograph_options, add_control_dependencies, arg_names, op_return_value, collections, capture_by_value, override_flat_arg_shapes)
    984         _, original_func = tf_decorator.unwrap(python_func)
    985
--> 986       func_outputs = python_func(*func_args, **func_kwargs)
    987
    988       # invariant: `func_outputs` contains only Tensors, CompositeTensors,

~/anaconda3/lib/python3.8/site-packages/tensorflow/python/eager/def_function.py in wrapped_fn(*args, **kwds)
    598         # __wrapped__ allows AutoGraph to swap in a converted function. We give
    599         # the function a weak reference to itself to avoid a reference cycle.
--> 600         return weak_wrapped_fn().__wrapped__(*args, **kwds)
    601     weak_wrapped_fn = weakref.ref(wrapped_fn)
    602

~/anaconda3/lib/python3.8/site-packages/tensorflow/python/framework/func_graph.py in wrapper(*args, **kwargs)
    971           except Exception as e:  # pylint:disable=broad-except
    972             if hasattr(e, "ag_error_metadata"):
--> 973               raise e.ag_error_metadata.to_exception(e)
    974             else:
    975               raise

TypeError: in user code:

    <ipython-input-24-9a082736eedc>:32 optimization_step  *
        optimizer.minimize(m.training_loss, m.trainable_variables)
    /home/maltamirano/anaconda3/lib/python3.8/site-packages/tensorflow/python/keras/optimizer_v2/optimizer_v2.py:374 minimize  **
        grads_and_vars = self._compute_gradients(
    /home/maltamirano/anaconda3/lib/python3.8/site-packages/tensorflow/python/keras/optimizer_v2/optimizer_v2.py:429 _compute_gradients
        loss_value = loss()
    /home/maltamirano/anaconda3/lib/python3.8/site-packages/gpflow/models/training_mixins.py:64 training_loss
        return self._training_loss()
    /home/maltamirano/anaconda3/lib/python3.8/site-packages/gpflow/models/model.py:57 _training_loss
        return -(self.maximum_log_likelihood_objective(*args, **kwargs) + self.log_prior_density())
    /home/maltamirano/anaconda3/lib/python3.8/site-packages/gpflow/models/sgpr.py:154 maximum_log_likelihood_objective
        return self.elbo()
    /home/maltamirano/anaconda3/lib/python3.8/site-packages/gpflow/models/sgpr.py:164 elbo
        num_inducing = len(self.inducing_variable)

    TypeError: 'Tensor' object cannot be interpreted as an integer

Expected behavior
Optimize without problem
System information

GPflow version: 2.1.1
GPflow installed from: pip install gpflow
TensorFlow version: 2.3.1
Python version: 3.8.3
Operating system: Elementary OS 5.1.7 Hera

Additional context
Instead of optimizing the model with the inducing variables, I would like to optimize the model without optimizing the inducing variables, manually reassigning the inducing variables at each step of the optimization.

Thanks @maltamiranomontero for the thorough issue report. The underlying issue is that TensorFlow's function tracer doesn't know that len(inducing_variable) will be mapped to inducing_variable.__len__().
To support this use-case, we need to change the inducing variable interface to a plain method call (or maybe property). inducing_variable.length()? inducing_variable.num_inducing? Any thoughts?

More minimal example of the len() vs tf.function() issue:
import tensorflow as tf
import numpy as np

class A(tf.Module):
    def __init__(self, a):
        self.a = tf.Variable(a)

    def __len__(self):
        return tf.shape(self.a)[0]

    def length(self):
        return self.__len__()

a = A(np.zeros((5,3,2)))

def test():
    return len(a)

test_fn = tf.function(test)

def test2():
    return a.length()

test2_fn = tf.function(test2)

print(test2())  # works
print(test2_fn())  # works

print(test())  # works
print(test_fn())  # fails

@maltamiranomontero Fixed! I've updated the answer in https://stackoverflow.com/questions/64145766/reshape-of-inducing-variables-gpflow/64146443#64146443 accordingly


💭 Add linter rules to detect non-inclusive terminology

As stated in Project Philosophy, one of the goals of the project is to Use inclusive terminology. Since this project is a linter, it would be good to create linter rules for this, this way Rome code will be checked for inclusive terminology by itself.

Looks like it's already fixed in https://github.com/romefrontend/rome/pull/267, closing

Note that it's currently disabled while we figure out a more sustainable approach to term management. ie. Non-English, custom words etc. See #793 for additional conversation.


Deleting a line

how to delete a line content.

You can run Cmd+Shift+K or search for this command in Command Palette.


Bridge request for Pokémon News

Bridge request

General information

Host URI for the bridge https://www.pokemon.com/us/pokemon-news:

Which information would you like to see?
Recent posts on Pokemon News

How should the information be displayed/formatted?

An image, date, title and small description. If it's possible, being able to go the article link and put the whole blog.

Which of the following parameters do you expect?

[X] Title
[X] URI (link to the original article)
[ ] Author
[ ] Timestamp
[X] Content (the content of the article)
[X] Enclosures (pictures, videos, etc...)
[ ] Categories (categories, tags, etc...)

Options

[ ] Limit number of returned items

Default limit: 5
Pokemon News seems to only give 9 articles on load.

[ ] Load full articles

Cache articles (articles are stored in a local cache on first request): yes
Cache timeout (max = 24 hours): 24 hours

[X] Balance requests (RSS-Bridge uses cached versions to reduce bandwith usage)

Timeout (default = 5 minutes, max = 24 hours): 5 minutes

Additional notes

I only know how to code a bit of python, not PHP. So I tried to make this rss generation/scraping on my own: https://codeberg.org/zortazert/Python-Projects/src/branch/main/rss/pokemon-news.py
It's missing:

Automatically updating. I don't have a server/website.
Being able to see the full article.

Thank you for doing this! Looking at the todo comment in the source code how did you find that api?


Add option to compile salva2d with wasm-bindgen

Right now, salva2d only compiles with stdweb. Future momentum seems to be moving towards wasm-bindgen, so adding support like e.g. nphysics has seems natural.
I followed the same strategy used in nphysics (see their Cargo.toml). By default, the salva crate uses stdweb to preserve compatibility. A user of salva can switch to wasm-bindgen like so:
salva2d = { version = "whatever", default-features = false, features = ["use-wasm-bindgen"] }

If y'all are ok with this strategy, I can copy it over to salva3d as well.

Adding to the default features seems unnecessary at first glance for a change to a specific feature set enabling wasm - for my curiosity, why do you propose adding to the default feature set in this case?


Maybe semantic PR titles won't work

I was thinking pull request titles were used as the default merge and squash commit message, but I was wrong. They are the default body of the commit, not the subject 🤔
So, it seems a semantic PR title will not suffice for a triggering a semantic release:

Thoughts on working around this, @groundwater and @gr2m?

semantic-release is quite modular now, it has analyzeNotes and generateNotes extension points that you could customize to look for the prefixes in the first line of the commit body if the commit subject starts with Merge pull request #
https://semantic-release.gitbooks.io/semantic-release/content/docs/usage/configuration.html#analyzecommits

I think it works if you force squashing of merges.

force squashing of merges.

Yeah.. the Electron maintainers group wants to leave our PR merging options open. Everyone seems to have a use case for which they need squashing, rebasing, or merging. So it would be ideal to make this thing work for any pull-request-landing strategy.


core update - 3.15.26

core update - 3.15.26

closing this PR since the core version not supported


fix extracting multipart/alternative parts

When extracting multipart/alternative part, it is currently assumed that the mail contains at least one text/html part. While that's the case most of the time, it doesn't have to be always true. There are cases when multipart message contains just a single single text/plain part. (Fixes #72).
The proposed fix conforms to RFC 2046:
"The use of the "multipart" media type with only a single body part may be useful in certain contexts, and is explicitly permitted."
"In general, the best choice is the LAST part of a type supported by the recipient system's local environment."
I would be willing to write some tests, but was unable to run them for client-sync package. Is there currently a way to execute them?

@MattSkala are you participating in Hacktoberfest (https://hacktoberfest.digitalocean.com/)? If so, we could close this PR and open it again tomorrow to have it count?

I've been looking at this, and it may be related to #82; can you confirm @simonft?

@seesemichaelj this appears to be for RX. What @simonft is working on is for TX.

@seesemichaelj That's OK, you can keep it open.

I do wonder what we see our supported parts as being. I would assume text & or HTML. The rfc allows for other format as well, does blindly choosing the last one make sense? Or do we explicitly check for HTML and then fall back to text? I'm not sure what the other possible formats may be. Also by always choosing the last part, we are trusting all clients out there to do the right thing.
Just thinking out loud, let me know what you think.

I'll look at this tomorrow

I'm also afraid of ignoring the warning of complexness that Nylas put in the comments and just going for it. I'm wondering if there are email clients that screw stuff up we're not aware of currently

After more thought on it, and looking over the RFC a bit more, the strategy in my head is:

Prefer text/HTML

Fall back to text/enriched

Fall back to text/plain

Settle for whatever type may be available (not the last version, but the first, as we don't recognize the types, but the first should be the "plain" format)

My reasoning: there are a ton of mime types, and we can't display the majority of them in the fashion intended, so we try to do the right thing, and then fall back to whatever is left. Perhaps some day we could pretty print things like text/yaml etc.

If I understand the code correctly, until now, we supported only text/html type in multipart/alternative messages. That resulted in a blank content displayed when that type was not present.
This commit supports both text/plain and text/html, preferring the type whichever is positioned later in the message.
Please note that to get alternative parts, we are calling _getDesiredMIMEParts recursively. This function filters out unsupported parts and returns only parts with types present in desiredTypes set, currently text/plain and text/html. Therefore, it shouldn't happen that we select a part with an unsupported type. (https://github.com/MattSkala/nylas-mail/blob/47b6c95c162423f2868b026691a7b7ab3726d3e3/packages/client-sync/src/local-sync-worker/sync-tasks/fetch-messages-in-folder.imap.es6#L188)
Are you suggesting to support also text/enriched and fallback to any type available?

So, it appears that I missed that bit when I reviewed this before. In that case, we can safely ignore my request, and choose the last supported part. I will go ahead and merge this in. Thanks for the great work!


[One Thing] Adding marquee feature to text

Extension
https://www.raycast.com/pernielsentikaer/one-thing
Description
Hello, is it possible to add marquee (scroll) feature to the text input? For example, if I input "physics simulations 101", the label will continuously scroll from right to left?
Who will benefit from this feature?
This is particularly beneficial for people who need to focus on just one item at a time, and to be reminded of to-do tasks constantly through out the day. Any text label in motion drives eye gaze towards it––as a reminder mechanism.
Anything else?
No response

@sai-educ This isn't possible with the Raycast API right now. I'll suggest that improvement to the team and see what we can come up with.

So, it's something that makes sense, and we may implement that further down the road. We're not sure when yet as it's a relatively low-priority feature 🙂


Switch default branch to master

Merging before switching the default branch from develop to master.

Switched at https://github.com/digiaonline/redux-fetch-data/settings/branches, deleted develop, and updated some docs in https://github.com/digiaonline/redux-fetch-data/commit/631a5dddcb3fa24b34afb10fa8143c6308ac3872.


opt forbid some bad pattern aggregate in AggregateStrategy

Proposed changes
Issue Number: close #xxx
Problem summary
since we cannot do stats derive and cost estimate on agg very good.
this PR remove some aggregate pattern that usually not good.

one stage agg after exchange. this pattern is good only when process very few rows.
three stage distinct agg with gather middle merge.

Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.03 seconds
stream load tsv:          427 seconds loaded<PHONE_NUMBER>9 Bytes, about 167 MB/s
stream load json:         26 seconds loaded<PHONE_NUMBER> Bytes, about 86 MB/s
stream load orc:          59 seconds loaded<PHONE_NUMBER> Bytes, about 17 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230424065836_clickbench_pr_133843.html

run buildall

run buildall

run buildall