[BugFix] Fix create user and create role log compatibility bug

Fixes #issue
In #25364, we used CollectionV2 to replace the original data structure, but missed the compatibility issue in Create user log
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.1
[ ] 3.0
[ ] 2.5
[ ] 2.4

[FE PR Coverage Check]
:disappointed: fail : 6 / 41 (14.63%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/persist/RolePrivilegeCollectionInfo.java
3
30
10.00%
[61, 62, 63, 66, 67, 68, 71, 72, 73, 75, 76, 77, 80, 81, 84, 85, 86, 88, 89, 90, 92, 93, 94, 95, 97, 98, 99]

:large_blue_circle:
com/starrocks/persist/CreateUserInfo.java
3
11
27.27%
[88, 89, 90, 92, 93, 94, 95, 96]

@Mergifyio backport branch-3.1

https://github.com/Mergifyio backport branch-3.1

https://github.com/Mergifyio backport branch-3.0

https://github.com/Mergifyio backport branch-3.1


CLT Lightrail app

Created an app that show real-time schedule for Charlotte Lightrail Trains (w/ All Stations)

I have read the CLA Document and I hereby sign the CLA

recheck

I have read the CLA Document and I hereby sign the CLA

recheck


Convert resource usage to string

Improve error message by converting resource usage values to a human-readable string.

LGTM

Thanks! Congratulations on your first merged contribution! 🎉


XGBoostError: Labels must be in sorted order

I am experiencing this error when training my model with a dataset:
raise XGBoostError(py_str(_LIB.XGBGetLastError()))
xgboost.core.XGBoostError: [12:36:33] /home/conda/feedstock_root/build_artifacts/xgboost_1588600955503/work/src/objective/regression_obj.cu:327: Check failed: last_abs_y <= abs_y: CoxRegression: labels must be in sorted order, MetaInfo::LabelArgsort failed!

Environment:
OS - centos 7
Python - 3.6.10
The dataset summary is as follows: -
Observations - 2919
Features - 1107
Positive labels - 168

Error occurs during training:
bst = xgb.train(params, dtrain, evals=[(dtest, 'test')], evals_result=evals_result,
                        num_boost_round=n_estimators, verbose_eval=False)

My params are as below:
hyperopt_evals = 5
n_splits = 2
n_estimators = 200

I scaled the parameters down so I can get a quicker turn-around for troubleshooting.
Any help to point to where this could be wrong will be highly appreciated.

As the error suggests, you should re-order your training data such that the labels are ordered in the ascending (increasing) order.


Release prep 2.0.0-alpha.5

Initial Object Store implementation (#150)
TLS first connection (#156)
Consume clean exit fixes (#161)
Add NatsMemoryOwner (#162)

You want to publish object store? I think it's csproj still has packable false

You want to publish object store? I think its csproj still has packable false

Yes it was! Thank you.


how to optimize federated index?

torvalds.dev is analyzing the ticket

The issue "how to optimize federated index?null" seems to be related to optimizing the performance of a federated index in a distributed system. A federated index is a type of index that spans multiple databases or systems, allowing for more efficient querying across these systems.
The file you provided, TestDynamicFieldNamesIndexCorrectly.java, is a test file from the Apache Solr project. Apache Solr is a powerful, open-source search platform built on Apache Lucene. It provides distributed searching and indexing, among other features.
This file tests the correct indexing of dynamic field names in a distributed Solr environment. It seems to be related to the issue you're facing as it deals with indexing in a distributed system, which is a key aspect of a federated index.
Here are some actions that could be taken to optimize a federated index:

Index Optimization: Regularly optimize the index to ensure it is as compact as possible. This can be done in Solr using the Optimize command.

Sharding: Distribute the index across multiple servers. This can improve search performance by spreading the load across multiple machines.

Caching: Use caching to store frequently accessed data in memory, reducing the need for expensive database queries.

Query Optimization: Optimize your queries to retrieve only the necessary data. This can be done by using filters, limiting the number of returned results, etc.

Hardware Optimization: Ensure that the servers hosting the index have sufficient resources (CPU, memory, disk I/O) to handle the load.

Load Balancing: Distribute search requests evenly across the servers hosting the index to prevent any one server from becoming a bottleneck.

Relevant files to this issue would be any files related to indexing and distributed searching in your Solr project. This would include configuration files for your Solr cores, schema definitions, and any custom code related to indexing or searching.


🐛 BUG: value of stdout.lastframe() is undefined is returned with some Wrangler commands

Which Cloudflare product(s) does this pertain to?
D1, Wrangler
What version of Wrangler are you using?
2.12.3
What operating system are you using?
Windows
Describe the Bug
The following steps both print out value of stdout.lastframe() is undefined:
Running wrangler d1 execute DBNAME --file="./schema.sql where schema.sql is:
DROP TABLE IF EXISTS cooltable;
CREATE TABLE cooltable (some_data TEXT NOT NULL, some_other_data TEXT NOT NULL);

Result:
🌀 Mapping SQL input into an array of statements
🌀 Parsing 2 statements
🌀 Executing on DBNAME (DBID):
🚣 Executed 2 commands in 8.666151999495924ms
value of stdout.lastframe() is undefined

Running a wrangler d1 execute DBNAME --command that returns no data.
For example, running wrangler d1 execute DBNAME --command "SELECT * FROM cooltable" or wrangler d1 execute DBNAME --command "DROP TABLE cooltable"
Result:
🌀 Mapping SQL input into an array of statements
🌀 Parsing 1 statements
🌀 Executing on DBNAME (DBID):
🚣 Executed 1 command in 4.784738999791443ms
value of stdout.lastframe() is undefined

It seems to come from here: https://github.com/cloudflare/workers-sdk/blob/6fd062419c6b66c2c5beb7c45ec84a32dfa89e01/packages/wrangler/src/utils/render.ts#L86

The output value comes from https://github.com/cloudflare/workers-sdk/blob/6fd062419c6b66c2c5beb7c45ec84a32dfa89e01/packages/wrangler/src/utils/render.ts#L46

I think the problem is that if there is no rendered output from the Ink.js component then there will be no last frame.
I'll look into this in case tomorrow.

I don't know if it stops statements excution. I have three slq commands in .sql file but only one statement is excuted.
🌀 Mapping SQL input into an array of statements
🌀 Parsing 3 statements
🌀 Executing on DB (b5823f1d-d731-475c-a869-b51d965842cd):
🚣 Executed 1 command in 0.01684001088142395ms
value of stdout.lastframe() is undefined


Improving similarity measure

Hi,
I actually did something similar to this a while back at https://channel-similarity.johnpyp.com/ but it didn't generate much interest, presumably due to among other things a lack of visualization which is a core part of this project and which makes it cool and interesting to look at and is well executed.
However, I do think that the similarity measure that I used is better in the sense of capturing similarity between channel communities, and could be implemented here without too much issue. Mathematically what I did is outlined at https://channel-similarity.johnpyp.com/details but it essentially boils down to a couple of differences from what you currently have going on right now:

The weight of a viewer should be normalized according to the number of channels that they're in. The reason behind this is that we want the relative weight of that user to be determined by how much of the percent of their viewing is dedicated to that channel.
For example: channel A and channel B sharing a viewer that ONLY views these two channels on the entire site should account for more than if channel A and channel B happen to share Nightbot that's present on a large chunk of channels on the site. Currently their relative weight for similarity is the same.
(this should be fairly simple to implement, doesn't require any scraping changes, and can also be used for the realtime channel page view)

The weight of a viewer should be determined not only by if they happened to be in that channel during the time period collected, but by the amount of time spent there (i.e the number of scrapes they appeared in).
An example of the shortcoming of the current approach: if channel A happens to host a channel B during the period collected, then all those chatters appearing momentarily in channel B's chat currently provide as much weight to similarity as chatters that spend long periods of time in both of these channels.
(this could require scraping changes to store these values)

Thanks for the great feedback here, and also wanted to say that your site is very cool.
The original goal of the project was to implement what you have there, to measure actual similarity between different channels. However I wasn't comfortable with the math behind it and chose the easier route, where I simply check the overlap at a specific time, without taking into account previous occurrences.
For the first point you have, I'm not exactly sure how to do the actual calculation. Currently, I take all the channel pairs and compare them individually, which is pretty inefficient. If I were to do what you said there, I would need create a set that contains all the viewers captured in that time, which I guess I can union each of the individual channels together into one massive set. And then for each of the viewers in the big set, check if they belong in any of the channels. And I'm unsure where to proceed from there.
For the Nightbot problem, I did try to accommodate that by filtering out any users that had "bot" in their name.
For the second point, we would have to store each viewer and the channels they are present in over time. And again, I'm not sure how to proceed.
The short coming you explained shouldn't really be an issue, since I only retrieve the top live channels, which when A hosts B, A wouldn't appear as a live channel anymore, and thus wouldn't be updated, only B would. B would see a boost in chatter count, which might influence the number overlaps in other channels, but it won't affect A's numbers.

Regarding the first point, the data structure that we're dealing with is a sparse matrix (a very large number of possible viewer-channel pairs, where only very few of them are actually populated) and in python there are modules (eg. sklearn) to efficiently perform calculations on it without doing operations on individual pairs. Once this matrix is created the calculation to find the channel x channel similarity scores is very short. However, in C# I'd have no idea how to do this, so this is something for maybe further down the line or maybe if I have time to integrate/port my analysis code into here.
For the second point, as far as implementation everything said above still applies, and though you're right about the host thing I do think there is still an advantage over the current approach: currently if channel A and channel B happen to do one singular collab stream that month and their unique viewers filter through each other, then that skews the similarity score almost as much as if they had streamed together every day for that month and their respective viewers were on each other's channel for a lot of the time (I say almost because obviously there are different viewers each time, so more collabs would marginally increase similarity). Intuitively, it seems to me that when people think of channel communities being similar they'd place importance on the amount of time spent shared rather than just a presence at one point.

Yep, except it I think of it as n x m

How would I calculate the weight of an individual chatter?
I've constructed the matrix with the weight of 1 for being apart of a channel, and that results in the n x n matrix. Since I didn't do anything with the weights, the resulting matrix is exactly the same as my method for calculating intersection so far.
Maybe I'm constructing the matrix incorrectly or I'm missing a step.
Here is the code I have so far.
https://gist.github.com/snoww/39063edd9bac1824f0685640944754c9

Yeah no problem, I've been busy as well. I have a couple of more questions.
There's a problem with the pseudocode you gave, product is an n x n matrix, and matrix is an n x m matrix (m x n after transpose), so you can't multiply them, perhaps you meant something else?
Secondly, what do you mean by Mean(sim)? Do you mean Mean(sim[c_1]) excluding the self referenced sim[c_1][c_1]?
Also to track presence over time, I'd need to have a new method of data collection to keep track of how many times a user is present during collection. Since I assign each index j to a unique viewer during that collection, I'd have no way of knowing what the index based of a user. Do you have any recommendations as to how this could be done? Currently I'm thinking a dictionary to store every user, and the value of each key would be the channel they are present in. This dictionary is saved during each collection in a database or something, and then during the n'th collection, we tally everything up and construct a matrix using that data. I'm don't think the time/space complexity will be good at all.

I fixed the matrix issue with an edit, I just mean normed transposed and multiplied by itself and typoed.
For Mean(sim) you're right in that if we exclude the diagonal self-similarities (which are all 1) it makes slightly more sense, though in practice when the number of channels is large the difference will be small. So you can replace Mean(sim) with (Sum(sim)-n)/(n^2-n) where Sum(sim) is just the sum of all of the matrix entries.
For data collection, I'd recommend looking at ClickhouseDB which I've used to good effect. It has a bit of a learning curve if you don't know SQL, but it's especially designed for usecases such as this. At a basic level you could just have a table with columns of timestamp,username,channel which you'd write to when scraping, and your analysis would be based on aggregate queries from this table. It has various compression techniques to make storing this not as big a deal as you'd think, and also you can overcome the time queries take by using materialized views (which basically means precalculating) for timeframes that you'd want to make available like last hour, last day, last week, last month.

Also for the database, I'm currently using postgresql for overlap data, maybe I can figure out an implementation that works for postgres.

The diagonal of the sim matrix should be all 1's, so something's wrong.
I think what could be the issue is that I assumed normed.TransposeAndMultiply(normed) meant normed * normed^T but it could mean normed^T*normed. You want to do normed * normed^T so that the normalized rows of normed are multiplied by each other.

No, normed.TransposeAndMultiply(normed) does mean normed * normed^T.


Findbugs low reporting and fixes

What
Enabled the findbugs low level reporting and fixed or excluded existing medium/low issues.
How
Renamed JsonToObject to jsonToObject to meet Java naming conventions 56d79ef.
Fixed potential NPE in Changes 76c504c.
Fixed inefficient boxing/toString of integers 91af7f4.
Fixed inefficient concatenation in some toString methods bac3fc9.
Fixed inefficient iterators where more efficient Map.entrySet()could be used b568c19.
Removed some anonymous inner classes in favour of statics 64d421d.
Removed dead local store from shutdown() 82251ec.
Fixed toLowerCase() calls that failed to specify a  Locale internationalisation issue 8ec2ce9.
Fixed equals/hashcode for Document and ReplicatorDocument 5303880.
Fixed risk of arrays being modified when used by reference 64d9f30.
Moved error stream close into finally block where it is known to be non-null 8432ae8.
Fixed missing throw from setPermissions() b04dc55.
Enabled "low" findbugs level and added excludes 	ebdc589.
Testing
Fixed designDocCompare test to correctly set revision f03591e.
Improved assertion message in assertConflictsNotZero 884b2f4.
Existing tests pass against CouchDB and Cloudant service.
Reviewers
reviewer: @alfinkel
reviewer @tomblench

Other than the formatting comment, +1.

+1


Restart Workload feature

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)

Closes #735

/test pre-busola-integration-test-k3s

/test pre-busola-integration-test-k3s


Problem extending from eshint (eslint-config-hapi)

I want to use hapijs eshint rules in a typescript project. The hapijs esrules are made available by the eslint-config-hapi npm module. Unfortunately, I can't extend from the eslint rules.
Bug Report

TSLint version: 3.14.0
TypeScript version: 1.8.10
Running TSLint via:  Cli (rpm)

tslint.json
{
"extends": "eslint-config-hapi",
"rules": {
"no-constant-condition": true
}
}
Actual behavior
Unfortunately, I get this error when ruinng eslint (after installing the eslint-config-hapi module):
Error:
Could not find implementations for the following rules specified in the configuration:
camelcase
consistent-return
vars-on-top
new-cap
no-constant-condition
no-native-reassign
no-underscore-dangle
no-undef
no-process-exit
no-unused-expressions
no-regex-spaces
no-catch-shadow
no-lonely-if
brace-style
no-shadow
no-unused-vars
one-var
handle-callback-err
hapi/hapi-capitalize-modules
hapi/hapi-for-you
hapi/hapi-scope-start
array-bracket-spacing
dot-notation
eol-last
no-trailing-spaces
no-eq-null
no-extend-native
no-redeclare
no-loop-func
yoda
sort-vars
arrow-spacing
hapi/no-arrowception
quotes
consistent-this
no-array-constructor
no-confusing-arrow
no-new-object
no-spaced-func
no-mixed-spaces-and-tabs
keyword-spacing
semi
semi-spacing
space-infix-ops
space-unary-ops
strict
eqeqeq
no-else-return
no-return-assign
no-new-wrappers
comma-dangle
no-sparse-arrays
no-ex-assign
no-var
prefer-arrow-callback
prefer-const
space-before-function-paren
func-style
object-curly-spacing
Try upgrading TSLint and/or ensuring that you have all necessary custom rules installed.
If TSLint was recently upgraded, you may have old rules configured which need to be cleaned up.
at Object.loadRules (/usr/local/lib/node_modules/tslint/lib/ruleLoader.js:29:15)
at Linter.lint (/usr/local/lib/node_modules/tslint/lib/tslint.js:56:44)
at processFile (/usr/local/lib/node_modules/tslint/lib/tslint-cli.js:138:29)
at /usr/local/lib/node_modules/tslint/lib/tslint-cli.js:177:74
at Array.forEach (native)
at Object. (/usr/local/lib/node_modules/tslint/lib/tslint-cli.js:177:41)
at Module._compile (module.js:541:32)
at Object.Module._extensions..js (module.js:550:10)
at Module.load (module.js:458:32)
at tryModuleLoad (module.js:417:12)
Expected behavior
I expected the eshint rules to work.

These are two different projects. Both projects have a different API that is not compatible.

yeah, basically what @mrmlnc said. eslint rules are not compatible out-of-the-box with tslint -- the AST APIs are completely different.

Thanks for the explaination. I tried tslint-eslint-rules also but it did not make a difference (same errors messages). Do any of you know of any other way - f.x. a tool to convert rules from eslint to tslint?
P.S. I did notice a very recent eslint tweet of upcomming typescript support. That might be a way also?

Unfortunately, there's no way to convert between the two. You could give the eslint-typescript thing a try, although I don't know how well it works at the moment.


Fix item render offsets

The only reason I can come up with that the items are too big when holding them is because render offsets aren't properly set for the item. I'd say render offsets should be able to be controlled with a RenderOffset() method that returns the correct render offset for the item. For example, if we want to create a new sword, we should be able to use render offsets of a Minecraft sword.
The workaround I use: I have to create attachables, animations and render controllers for each custom item as they don't show up properly without them.
Maybe there's already a fix which I didn't notice at all, but that doesn't seem the case, at least for me.

This happens only in first person by the way, not in third person IIRC. The calculateItemScale method could be the issue custom items are big in first person. Why are the correct base values used only when the item's HandEquipped() method returns true?

After some research, I found out attachables are the way to prevent items to be so big. Render offsets won't make it out of experiments, so it's just better to create attachables for each custom item.


Tidy up command line in tests

introduce dirname
proper CLI options
indent code
support Windows

Fixes #681

@xitox97 Please run php path\to\composer run-script test:unit

Hi,
Thank you very much for digging into this issue!
Using the dirname function is for supporting Windows? If so can you also change the usages in tests/Rules folder? For example here: https://github.com/nunomaduro/larastan/blob/master/tests/Rules/ModelPropertyRuleTest.php#L13 I believe this will fix the rest of the errors @xitox97 getting

Using the dirname function is for supporting Windows?

No, there are several forward slashes in  several paths causing no problems.
Using real PHPStan path /vendor/phpstan/phpstan/phpstan makes Windows work.
/vendor/bin/phpstan/ is a shell script on Windows thus php.exe cannot execute it.

Thank you!


Refinery emitting multiple metric events with same timestamp, only one contains data

Refinery's Honeycomb Metrics are sending two or three events with the same exact timestamp (to the millisecond) but only one of the events has fields populated (the other one or two events are empty)

Version: Refinery 1.2.1 (AMD64)

Closing as events are not empty (though appear so at first blush) but are just different metrics goroutines running on the same instance.


🐛 Fixed member importer crash for failed imports

refs https://github.com/TryGhost/Team/issues/2004

for imports, members are created inside a transaction, which causes the member created events to be dispatched.
its possible that transactions for import can be rolled back if for some reason there is an error down the line while inserting other member properties. The rollback doesn't commit the member to DB, but the event dispatched earlier will still try to create the member created event which fails due to missing member id.
knex transactions resolve the executionPromise both in case of explicit commit or rollback from the user, so just the transaction end check will not be good enough to make sure the member exists in DB
adds explicit config to knex to reject transaction in case of explicit rollback, which is then caught and event is not dispatched

We still do:
DomainEvents.dispatch(MemberSubscribeEvent.create({
      memberId: member.id,
      source: source
  }, eventData.created_at));

But I couldn't find any listeners for this event. This isn't a big problem now, but can we delete this or also wrap it with the execution promise?


Create light.json

#194 is also implementing this

This change is planned to be merged as part of the abovementioned PR.


Choosing alternative opens the prompt, changing does not hide the prompt again

When you click "Stem" in the app, you get an "Are you sure?" verification. That verification doesn't disappear if you change the selected option.

Fixed in #1489


[SMALLFIX]Use static imports for standard test utilities

change import org.junit.Assert to import static org.junit.Assert.assertEquals, import static org.junit.Assert.assertFalse, import static org.junit.Assert.assertTrue, import static org.junit.Assert.assertNull;
change Assert.{{method}}* to {{method}}*

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3540/
Test PASSed.

can you reset the change to webui/npm-shrinkwrap.json which is unrelated to this PR

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3543/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3546/
Test PASSed.

alluxio-bot, merge this please


Fix: cross-version placeholder support

Closes https://github.com/ramda/ramda/issues/3458
The strict equality operator means that _isPlacerholder works for only the R.__ from that specific install. If your project has multiple versions of ramda (which can easily happen due to npm semver resolution and ^ versus ~ versus exact in package.json), you'll have problems.
https://github.com/ramda/ramda/pull/3388/files#diff-cfb81eb9592aa5597e5aa32c6e449214f5b7eabeb172ad2447e2e537232454d2L2-R4
Reverting the logic back to what it was before is a looser check that will work cross-version

I believe we can remove this file as well.

I know this PR is merged but still we can do one more thing that is too keep both the check. So that if anyone using the latest version he gets the benifit of === and for backward compatibility lets use already existing.


consider hyperdrive-next for blob storage

the diff, downloadDiff, and downloadRange methods seem potentially helpful for selective sync of blobs.
https://github.com/hypercore-protocol/hyperdrive-next

Great, yes, this is pretty much how I was proposing to implement our blob storage (based on conversations with Mafintosh, so it's not surprising it's the same approach!). What would be helpful though is to store arbitrary metadata with each file entry (which is perfectly possible, because it's just a value in hyperbee). Maybe the Hypercore team would be willing to add this as an option - we could store contentType at least, and maybe other data like geolocation.
I didn't no an update to Hyperdrive was on the cards, and I would say let's use this rather than implement something almost identical ourselves.

it looks like there's an undocumented ability to store metadata by passing it in the put options: https://github.com/hypercore-protocol/hyperdrive-next/blob/master/index.js#L187
closing this in favor of #52


[DO NOT MERGE] Add a new Benchmarking A/B test for the child maintenance page title

This isn't live yet. It's waiting on some decisions on wording for the test itself (smart-answers PR coming shortly, hopefully), the cdn-configs PR (https://github.digital.cabinet-office.gov.uk/gds/cdn-configs/pull/129), and Tuesday when the test is scheduled to start. I'm just readying this now so it's not a mess of PRs on Tuesday.

And the A/B test name for this might change between now and then... 🙈

Name aside 😛 , LGTM 🚀

I just updated the name: BenchmarkCmTitle1.


Align versions to spring boot 2.7.4

Align versions to versions in spring-boot-dependencies 2.7.4 : https://repo1.maven.org/maven2/org/springframework/boot/spring-boot-dependencies/2.7.4/spring-boot-dependencies-2.7.4.pom

yes this will be good to get into 3.19.0 before its released

@oscerd I tried building/testing apache/camel-karaf locally against this branch, and I didn't get any failures.    The spring-batch version here is different than what is in apache/camel-karaf/pom.xml, and the jboss-logging version is different, but I don't see a couchbase/hibernate version in camel-karaf.    Is there something else I should test this against?    I'm happy to do whatever testing is necessary but I'm not all that familiar with karaf and what to test there.

@cunningt this is fine, the test on karaf is usually to test the platform/karaf/features that tests that the components can be installed in karaf, which is often where things can break if versions are upgraded or other dependency chagnes.

@cunningt this PR now has merge conflicts


VLD is invalid library on Alpine

Strange things happen on Alpine Linux (Docker) :confused:
It works fine on Windows and Debian.
PHP: from source/package.
VLD: from source (VLD_0_14_0).
How can I help (debug) for you?
# uname -a
Linux 3.16.0-4-amd64 #1 SMP Debian 3.16.39-1 (2016-12-30) x86_64 Linux

# php -v
PHP 7.1.6 (cli) (built: Jun 12 2017 21:23:32) ( NTS )

# php -dextension=vld.so
PHP Warning:  PHP Startup: Invalid library (maybe not a PHP library) 'vld.so' in Unknown on line 0

How did you build vld?
What is in vld.so? Try "file vld.so" (in the right path) and "head vld.so" (in the right path).

Thanks for the quick reply.
git clone -o VLD_0_14_0 --depth 1 https://github.com/derickr/vld.git /tmp/vld
cd /tmp/vld
phpize
./configure
make
make install

# /usr/local/lib/php/extensions/no-debug-non-zts-20160303

# file vld.so
sh: file: not found

# head vld.so > /vld_head.log

vld_head.log (sorry for Dropbox)
Btw, Alpine uses musl as its libc.

# file vld.so
sh: file: not found

You do need to install "file" for that to work :-)
But at least the "head" looks like that it is a shared object (ELF) file.

Alpine uses musl as its libc; musl isn't 100% compatible with glibc, hence the missing symbols.

You didn't include these in your report, and they are important to see.

Oh, Alpine and me. Sorry :blush:
# file ./vld.so
./vld.so: ELF 64-bit LSB shared object, x86-64, version 1 (SYSV), dynamically linked, not stripped

I meant the missing symbols! I need to figure out what musl misses.


Hi, you are missing a few key cartographic icons such as:

I noticed you have icons like: Magic Portal and have more than one castle, as a serious mapplication developer, I would love to see:

Cloud
smoke
weather

and for moving map.....

helicopter (top down)
small aircraft (top down)
large aircraft (top down)

Thanks for your great work!

Yes, a couple more icons would be nice. Also icons like "meeting point" or "coffee" would be nice.

Hey great work. I miss an icon for vending machines. Is this project still in active development?


Added index on condition code

Added index on conditions.code.coding.code which is essential for searching conditions by code. In practice this comes into play when using the new condition-code parameter on Patient resources, allowing you to search patients by conditions they have.

We may want to consider adding a few more indexes on other codes we currently use. For example, the coding for medication orders? Open to feedback here.

Closing this for now. We can reopen it later if it's still an issue.


moveFinishedBackup: EPERM: operation not permitted, rename...

Joplin 2.2.1 (prod, win32)
Windows 10 21H1
Simple Backup v1.0.1
I have this error message popping up regularly.

It doesn't seem to have any impact on the backup itself. As far as I can tell, despite this message, backups are fine.
I found out a pattern when the message pops up. It happens :

when I start my computer in the morning
or
when the computer comes out of sleep mode

These are the only times when this message appears as soon as I click the Joplin icon in the notification area. Other thant that, the rest of  the time,  the plugin run silently as expected without any error message.
Here are my plugin settings and the backup log file (verbose mode).
backup.log

Ah I see your problem, there are two running backuptasks, so one job cannot rename the folder

Do you use a portable version of Joplin?

Yes, I do.

Could you please test the v1.0.2

Just installed. I'll let you know if the message pops up again.
Thanks !

So, a few days later... everything's fine. No more error messages.
Thanks again .


reduce top-margin space at phone screens

Fixes #1146
Screenshots of the changes

Don't change it for every screen size, use media queries.

Coverage remained the same at 84.266% when pulling 62a5085997b41280adc4d7800d8486a079be3df6 on mahmodHammad:whitesp into 4872e6f5d4355054c383207a8ed9e10b6ff1db7e on CircuitVerse:master.

Don't change it for every screen size, use media queries.
@tachyons
Could you please review the changes

@nitin10s
i used media query for making this changes on Phone screens only

i will check it again

https://github.com/CircuitVerse/CircuitVerse/pull/1147/#issuecomment-599006671
https://github.com/CircuitVerse/CircuitVerse/pull/1147/#pullrequestreview-376255574
@nitin10s
i think it's better now , Please Check it again !


Update flax.optim.rst

What does this PR do?
Update documentation for AdaBelief optimizer, which was recently added to the codebase in commit https://github.com/google/flax/commit/bb1f807322f1b5c86edfb59ce72d001f850130b0.
Checklist

[x] This PR fixes a minor issue (e.g.: typo or small bug) or improves the docs (you can dismiss the other
checks if that's the case).
[ ] This change is discussed in a Github issue/
discussion (please add a
link).
[x] The documentation and docstrings adhere to the
documentation guidelines.
[ ] This change includes necessary high-coverage tests.
(No quality testing = no merge!)

Thanks for the PR!


Features/1191 adding plantuml releases

Fixes #1191 by implementing a (very simple) bridge that simply read the releases page

Damn, I used the wrong branch as base !


[BEAM-9146] Integrate GCP Video Intelligence functionality for Python SDK

This PR relates to https://issues.apache.org/jira/browse/BEAM-9146 and integrates GCP Video Intelligence functionality as part of a PTransform that implicitly accepts a PCollection of GCS URIs or raw bytes.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

R: @aaltay @kamilwu

Thank you @EDjur.
I was hoping that we can implement most of the functionality in tfx_bsl, and add cloud AI services by defining new endpoints. We had a discussion on dev list [1] related to this.
[1] https://lists.apache.org/thread.html/r3d9b0e3270668fda989f363eab1f79e4ef0e7fa3e0fcbe9c26344f14%40<dev.beam.apache.org>

I see. I signed up to the dev mailing list just last week so must've missed that discussion.
What does that mean in regards to this ticket?
I agree that it would be nice to avoid duplicated behaviour across libraries.

@EDjur - Sorry for the miscommunication. These types of decisions should have reflected on JIRA as well. I feel bad that you spent time on this and we are changing direction after your PR is out.
I think what it means for this ticket is that:

If tfx_bsl has ability to call into services (which we need to add and TFX team agreed to support us with reviews)
We can add thin transforms for different things (e.g. AnnotateVideo) and each of these transform will have quite a bit of shared code and behavior.

@kamilwu may have other thoughts, since he was also working on this. I would like to hear what is his opinion.
For this PR, let's try to re-use as much as possible.

@aaltay No worries - I've learnt a lot by creating this PR anyway! Curious to hear @kamilwu's thoughts on it too.
I'm open to modify / work on another PR related to tfx_bsl if needed.

Thanks @EDjur.
tfx_bsl seems to be useful only in some cases. One of these cases is AI Platform Prediction, which, by the way, we also would like to support in Beam in the nearest future.
Video Intelligence API is pretty much different. We should treat it as a fully-managed machine learning service. There is no machine learning model which could be trained and deployed. Instead, videos are annotated using pre-trained custom model provided by Google. All a user has to do is to enable API and make a request (using REST API or client library). The only input is a GCS path to a video file. As a result, there is little behavior that could be shared.
If you have any questions regarding tfx_bsl and how it connects with Beam, just let me know, as I'm currently working to provide support for remote inference (an ability to call into services) in tfx_bsl.

@EDjur Because you've added a new dependency, we have to specify it in setup.py file (sdks/python/setup.py)

Thank you @kamilwu for the comment. That makes sense to me.
Related question, what is a good location for these family of transforms? io/... does not sounds right in this case. How about a top level ml/ folder similar to io. These transform could be put under ml/gcp?

Related question, what is a good location for these family of transforms? io/... does not sounds right in this case. How about a top level ml/ folder similar to io. These transform could be put under ml/gcp?

I agree io/... can be somewhat misleading. ml/gcp sounds good.

Thanks for the feedback! I've adjusted the code to conform to py23 standards as well as added the extra arguments to annotate_video.
I also added the google-cloud-videointelligence dependency which I've tested from version 1.8.0 up until 1.12.1.

Got one question regarding the naming. Is video_intelligence a good name for the module? I'm slightly worried it might cause confusion with the google.cloud.videointelligence.

Got one question regarding the naming. Is video_intelligence a good name for the module? I'm slightly worried it might cause confusion with the google.cloud.videointelligence.

I think it is a good name reflects the underlying service as it is. Not very different from other gcp io with the similar names to the gcp services.

LGTM.

Got one question regarding the naming. Is video_intelligence a good name for the module? I'm slightly worried it might cause confusion with the google.cloud.videointelligence.

I think it is a good name reflects the underlying service as it is. Not very different from other gcp io with the similar names to the gcp services.

I started work on https://issues.apache.org/jira/browse/BEAM-9247 to integrate the vision API in a similar PTransform. However, here the naming gets more complex. Consider e.g.
  from google.cloud import vision
  from apache_beam.ml.gcp import vision

One solution for the vision transforms is to rename the module vision_api instead. But I think we should be consistent across the different GCP ML APIs, so perhaps video_intelligence will need renaming too.
What's your take?

Got one question regarding the naming. Is video_intelligence a good name for the module? I'm slightly worried it might cause confusion with the google.cloud.videointelligence.

I think it is a good name reflects the underlying service as it is. Not very different from other gcp io with the similar names to the gcp services.

I started work on https://issues.apache.org/jira/browse/BEAM-9247 to integrate the vision API in a similar PTransform. However, here the naming gets more complex. Consider e.g.
  from google.cloud import vision
  from apache_beam.ml.gcp import vision

One solution for the vision transforms is to rename the module vision_api instead. But I think we should be consistent across the different GCP ML APIs, so perhaps video_intelligence will need renaming too.
What's your take? @aaltay

I do not have a good idea. Users could solve this with something like the following:
from google.cloud import vision as gcp_vision_api
from apache_beam.ml.gcp import vision

So, maybe it is not a big issue. I agree, it would be good if we can avoid this state.
Related, in the io folder we have modules named gcsio, bigtableio etc. but we also have modules named bigquery, pubsub. So it is not very consistent.
We can take the io example and change the names by appending ml, your example will look like:
from google.cloud import vision
from apache_beam.ml.gcp import visionml
from apache_beam.ml.gcp import videointelligenceml (Notice I also dropped _ between the words.)

What do you think?

I like the idea of appending ml to the modules. Dropping the _ also seems prudent so it's consistent with most of io modules.

LGTM. Triggering tests.

Run Python PreCommit

Seems tests aren't running properly in Jenkins. But they all pass on my local machine. I'll let you re-trigger them and hope that it works.

Seems tests aren't running properly in Jenkins.

At the moment, only Beam commiters have permissions to trigger Jenkins jobs. But you can always ask someone on devlist to trigger them for you.

Could you also add a new feature note in https://github.com/apache/beam/blob/master/CHANGES.md

/cc @markflyhigh - Tests seems to be not triggering?

tests, will you trigger?

@EDjur - This LGTM. I would like to merge it but I cannot seem to trigger the tests
@markflyhigh is there a manual way to run the tests?

retest this please

retest this please

Finally, tests are running.

retest this please

Some minor lint issues should be fixed with that last commit. But it seemed Jenkins was failing on something else: failed to import pycodestyle when running the tests. Not sure that is related to this PR.

Some minor lint issues should be fixed with that last commit. But it seemed Jenkins was failing on something else: ModuleNotFoundError: No module named 'pycodestyle' when running the tests. Not sure if that is related to this PR.

This is fixed in (https://github.com/apache/beam/pull/10844). You may need to rebase.

retest this please

retest this please

retest this please

I cannot seem to make the tests fail locally so this is proving to be quite a struggle 🙄 But hopefully this should fix it.

retest this please

retest this please

retest this please

retest this please

retest this please

@EDjur your phrase triggers unfortunately won't trigger the tests - there are Jenkins restrictions that won't let you do it

@EDjur your phrase triggers unfortunately won't trigger the tests - there are Jenkins restrictions that won't let you do it

Gotcha. Thanks for triggering them!

retest this please

I'm having a small discussion regarding the video_context with @kamilwu here: https://issues.apache.org/jira/browse/BEAM-9247
Perhaps this is something we should look at integrating to this PR before (or after considering the use-case seems small) merging too.

retest this please

Test failures look unrelated to this PR.
11:04:57 self = <apache_beam.runners.interactive.caching.streaming_cache_test.InMemoryReader object at 0x7fdcc6b742d0>
11:04:57 watermark = 0, processing_time = 1
11:04:57
11:04:57     def advance_watermark(self, watermark, processing_time):
11:04:57       record = TestStreamFileRecord(
11:04:57           watermark=Timestamp.of(watermark).to_proto(),
11:04:57 >         processing_time=Timestamp.of(processing_time).to_proto())
11:04:57 E     ValueError: Protocol message TestStreamFileRecord has no "processing_time" field.

Test error seems to be related to: https://github.com/apache/beam/pull/10856/files

retest this please

retest this please

retest this please

retest this please

:sdks:python:test-suites:tox:pycommon:docs  task is failing -- There might be pydocs issues in the change.

Seems yapf doesn't format docstrings 🤷♂  Should be fixed now.

retest this please

retest this please

retest this please

@EDjur -- Thank you, merged this.
Could we update 'google-cloud-videointelligence>=1.8.0<=1.12.1', to use the recently released 1.13.0 version as well? Do we need anything special?

Looks fine to update! The 1.13.0 essentially added two new enums to the types.Feature, but since these are specified by the user, all should be well.
I also manually reran all the tests without mocking the videointelligence client and they ran as expected.
Cheers!


policy-check should verify dependencies are not duplicated

There have been a few cases where someone adds something to  dependencies that was already present in devDepenendencies.  This isn't a huge problem by itself, but the version management/release tooling assuming entries are only present once, and in cases where this is a fluidframework package, it will bump the version of the entry in dependencies and not devDependencies, then later in the process will complain that the versions are mismatched.  policy-check should warn about this, and when run with fix should remove the duplicate entry from devDependencies

This is addressed by the no-repeated-dependencies rule of npm-package-json-lint, which is run as part of policy-check.
https://github.com/microsoft/FluidFramework/blob/e8e22ab7a6b9a03c8d5d8c0de54ffc21d7d90c87/tools/build-tools/src/repoPolicyCheck/handlers/npmPackages.ts#L524-L536


Removed region lock-in, used store for sharing variables, and added a cleanup notebook.

Issue #, if available:
Description of changes:

Removed hard coded region links from previous notebooks.
Created a shared set of variables for the Arn using the store feature in IPython preventing the need to re-define variables.
Created a cleanup notebook to tear down after completion of demo

Previous notebooks were hard coded to region locking
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@yangcheng any changes you'd like to see made here?

@rohitmenon83 your feedback would work also. Really just looking for a thumbs up for the merge.


BitmapMask isn't applied when using a canvas renderer

Version

Phaser Version: 3.14.0
Operating system: MacOS High Sierra

Description
For certain devices / browsers I need to use the Canvas rather than WebGL renderer. I'm finding that bitmap masks aren't being applied when falling back to Canvas.
Example Test Code
Use the Phaser Labs BitmapMask Example
http://labs.phaser.io/edit.html?src=src\display\masks\BitmapMask Example.js
and just change the renderer type to Canvas.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Canvas cannot support bitmap masking. This is a WebGL only feature (it does say as much in the docs, but it's easy to miss)


Where Do I Belong Challenge Test

Challenge Where do I belong has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.87 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code returns correct answers, but they are reported as failures.I tracked down the problem to be with the "for (var x in arr)" statement. The statement iterates through an array similar to
"for (var x=0;x<arr.length;x++)" which when used causes the tests to succeed. I suspect it's because the the first version of the "for" creates the variable "x" as a character type for some reason. The tests are likely comparing the results using the "===" operator.
If you change the return to "return x-0;" it will work.
My code:
function where(arr, num) {
  // Find my place in this sorted array.
  arr.sort(function (a, b) {return a-b;});
  for (var x in arr) {
    if (num <= arr[x]) {
      return x;
    }
  }
  return arr.length;
}

where([20,3,5], 19);

This is not an error with our site. As you've identified your code is returning a string instead of a number. This is one of many reasons why for ... in is a bad and uncommon choice for iterating through an array. Thanks and happy coding.

@ltegman Thank you ,your answer is useful!


minimum connectionTimeout value has changed, readme.md out of sync

Similar to https://github.com/brettwooldridge/HikariCP/pull/140 it seems the readme.md file in master indicates a minimum connectionTimeout of 100ms, while the code (after moving this logic from HikariConfig to AbstractHikariConfig) throws an exception if this value is under 1000ms.
Was the shift to 1s intentional or a typo during porting? Version 2.1.0 has 100ms in HikariConfig while 2.3.1 has 1000ms in AbstractHikariConfig.

@drobert The shift was intentional, but I will consider whether there is a way to enable shorter timeouts reliably.  The basic issue comes from the fact that HikariCP introduced the concept of validationTimeout.  We had a user (possibly several) that want an infinite connectionTimeout (they're willing to wait in getConnection() until one is available), but the fact that HikariCP was also using connectionTimeout for connection alive validation causes a problem in that case.  The expectation is that a connection can be validated as alive within some smaller amount of time (for example, several seconds).
We come then to connection validation.  Whether the user is using a validation query (eg. SELECT 1) or JDBC4 validation, the minimum timeout for setQueryTimeout() or isValid() is denominated in seconds, not milliseconds.  So, 1 second is the lower-limit on validation.  If that is the case (as per JDBC), there is no way to guarantee a connectionTimeout of 100ms when, after obtaining a Connection from the internal bag, it might take a minimum of 1 second to validate the Connection.
HikariCP treats the connectionTimeout as a hard contract.  The user will either receive a Connection within connectionTimeout or they will get a SQLException.  As it stood, there was no way to satisfy this guarantee.
Having said that, there is one "out" that may allow HikariCP to support a shorter timeout.  If the JDBC driver supports the setNetworkTimeout() method, which is denominated in milliseconds, it is possible to uphold a connectionTimeout (and validationTimeout) of less than 1000ms.
The question then becomes, is it actually reliable.  For example, if connectionTimeout is 100ms then validationTimeout must necessarily be 100ms or less.  If a database server is busy (loaded), can it reliably satisfy a connection validation in 100ms or less?  A connection failing validation is closed and ejected from the pool.  If a DB server under even moderate load cannot satisfy validations with 90% or better success, the pool will end up churning Connections severely, resulting is even more load on the DB and downward spiral of failure.


Web - Remove user messages on #admin is broken on removing multiple users together and also on deleting workspace

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open the app
Open settings
Open workspaces
For this test, create new workspace
Open #admin room report of workspace
Again open settings->workspaces->workspace created on step 4->members
Invite multiple members, observe in #admin, messages are correct for adding members
Again visit members page of the workspace and delete all the invited members, now observe in #admin, removed messages are not displayed for all users and repeated for some
Repeat invite and remove process few time and delete workspace, observe that it again changes all removed user messages and even now, they have random repeats

Expected Result:
App should display added and removed users messages as per our action properly in #admin room of workspace
Actual Result:
On removing multiple users at once and deleting the workspace, app randomly repeats removed user messages and does not display removed user message for all users that were removed
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/10992766-ac4b-41dd-ac5c-2fd1ae018f3b
https://github.com/Expensify/App/assets/93399543/813eaeef-7ac9-4db6-8075-bb6069b51a81
Expensify/Expensify Issue URL:
Issue reported by: @dhanashree-sawant
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1690168747969629
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01cf458e83d1c75708
Upwork Job ID:<PHONE_NUMBER>666582016
Last Price Increase: 2023-07-25

This looks like it's a back-end issue, so will need to be handled Internally.

Moving to weekly. Will look into this soon.

Actually, @yuwenmemon I believe you implemented workspace change logs, so I'm going to assign this over to you. Feel free to re-assign if I got that wrong. Thanks!

@MonilBhavsar Can you take a look at this once since it's related to employees?

Looking this week

Looking today

Issue is identified. Doing some testing before submitting PR for review

@kbecciv can we please check if this issue is fixed

Hey @MonilBhavsar Checking in latest build <IP_ADDRESS>, will update you shortly

Unable to reproduce on build <IP_ADDRESS>, please attach the video.
https://github.com/Expensify/App/assets/93399543/dd365f5c-c2aa-4452-be0b-2b66e13d9fcc

Thanks, I am also not able to reproduce anymore.
So going to close it
Feel free to reopen if issue persists.


Handle MIDI messages to control effects

Fixes #14.

I'm going to merge this for now and work on the re-architecture separately.


Update ext-lang-pl.js

Add translations button close.

1. Why is this change necessary?
Because it adds a translation in Polish.
2. What does this change do, exactly?
Adds a translation to the close button in Polish.
3. Describe each step to reproduce the issue or behaviour.
Log in to the Shopware backend in Polish in which the shopware will not be in the latest version.
4. Please link to the relevant issues (if any).
No needed changes
5. Which documentation changes (if any) need to be made because of this PR?
No needed changes.
6. Checklist

[*] I have written tests and verified that they fail without my change
[*] I have squashed any insignificant commits
[*] This change has comments for package types, values, functions, and non-obvious lines of code
[*] I have read the contribution requirements and fulfil them.

 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.

@michubortek Can you accept the cla?

@michubortek Can you accept the cla?

accepted

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-25540
Please use this issue to track the state of your pull request.

Thanks for your Contribution! :blue_heart:


Invariant Violation: Module RCTDeviceEventEmitter is not a registered callable module

Description
I'm trying to create a fresh react-native project with a fresh native module without any new code. I installed the MyLibrary into MyApp using yarn add file:/path/to/module, I don't know if it could be the problem :/

React Native version:
System:
OS: macOS Mojave 10.14.6
CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
Memory: 452.77 MB / 16.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 12.16.3 - /usr/local/bin/node
Yarn: 1.12.3 - /usr/local/bin/yarn
npm: 6.14.4 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
Managers:
CocoaPods: 1.9.1 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 13.2, DriverKit 19.0, macOS 10.15, tvOS 13.2, watchOS 6.1
Android SDK:
API Levels: 22, 23, 25, 27, 28
Build Tools: 25.0.3, 26.0.2, 28.0.3
System Images: android-28 | Intel x86 Atom_64
Android NDK: 19.2.5345600
IDEs:
Android Studio: 3.6 AI-192.7<IP_ADDRESS>92135
Xcode: 11.3.1/11C504 - /usr/bin/xcodebuild
Languages:
Java: 1.8.0_252 - /usr/bin/javac
Python: 2.7.16 - /usr/local/bin/python
npmPackages:
@react-native-community/cli: Not Found
react: 16.11.0 => 16.11.0
react-native: 0.62.2 => 0.62.2
npmGlobalPackages:
react-native: Not Found
I'm using react-native CLI from npx.
Steps To Reproduce
I just did exactly what is in this guide https://reactnative.dev/docs/native-modules-setup and when I try to run my app on my phone I got the screenshot error.

I'm using react-native CLI from npx.

I'm having this issue as well

+1

+1

I experienced this problem as well, and after a whole day of trying to fix the problem, what worked for me was changing the dependencies in the package.json of MyLibrary so that all that was left was:
"peerDependencies": {
  "react-native": "^0.62.2"
}

without devDependencies nor dependencies. Make sure you also remove node_modules after making this change in MyLibrary. This is what the directory structure of MyLibrary (I called it MusicKit) looked like for me after I got it working:

And here is a screenshot of the package.json:

I'm having this issue as well

Hi @piiih thank you for the issue. Could you provide a repro that consistently demonstrates this error?

I have same issue , in my condition , it cause by i update my node version .
solution : delete all node_modules which already downloaded ,and run "yarn install" command for re-install them .
then they will be work .

I have same issue:

Invariant Violation: Module RCTLog is not a registered callable module (calling logIfNoNativeHook)

It only occur on remote debugging mode, if you disable remote debugging in dev mode, it's still work

Make sure you have one version of react-native installed. npm ls react-native should help.

If it helps anyone:
metro.config.js
const path = require("path");
const watchFolders = [
  //Relative path to packages directory because I'm in yarn workpspaces
  path.resolve(__dirname + "/../.."),
];
module.exports = {
  resolver: {
    extraNodeModules: {},
  },
  transformer: {
    getTransformOptions: async () => ({
      transform: {
        // this defeats the RCTDeviceEventEmitter is not a registered callable module
        inlineRequires: true,
      },
    }),
  },
  watchFolders,
};

react-native start --reset-cache

This helped me to fix it!
Many many many thanks! 🙏🙏🙏

Surely this cannot be the fix right?


Proposal - Insiders Cloud

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[x] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[x] I attest that the connector works and I verified by deploying and testing all the operations.
[x] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[x] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[x] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[ ] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[ ] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[ ] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[ ] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Hello @tillbier,
Thank you for submitting your connector, kindly remove proposal and follow the instructions provided in the below link and accept the license/cla agreement.
https://learn.microsoft.com/en-us/connectors/custom-connectors/certification-submission-ip

@microsoft-github-policy-service agree company="Insiders Technologies GmbH"


Non Approved Users Show Up In Contacts List

fixed with https://github.com/Free-Republic-of-Verdis/verdis-communications/commit/454e6498b9fc44211cdb7eb568351d92029e8389


How to decode spectra.obj files?

Thanks for your sharing! It is a wonderful paper. However, i was wondering how to open the raw data of RRUFF which you upload at this link.
Actually ,i used scrpit as follow:
with open('spectra.obj', 'r') as f: data = f.read()
and i got this error UnicodeDecodeError: 'gbk' codec can't decode byte 0x80 in position 0: illegal multibyte sequence

Hi, Thanks for your interest. : )
You are almost there. It is import pickle; data = pickle.load(open("spectra_obj", "rb))

It works. Thanks for your patience.


Form subscription are ignored

Current behavior
I specify "submitting: true" and "pristine: true" to useForm subscription and form is render all the time
Expected Behavior
Form is render only when submitting and pristine change
CodeSanbox

See README. This is unavailable with hooks.

Will it eventually be possible to use subscriptions or will the hook version never support this?

By default, 🏁 React Final Form Hooks subscribes to all changes, but if you want to fine tune your form, you may specify only the form state that you care about for rendering your gorgeous UI.

Can you point me to an example that shows how to do this?

useForm({subscriptions: []})

Like this?


Feature + Tests: i64 operations

Pull Request Overview
This pull request adds support for i64 operations.
Testing Strategy
This pull request was tested locally using the provided test cases.
TODO or Help Wanted
This pull request still needs reviews over the refactoring of the WasmReader structure. This was modified because I have started working on it based on the branch related to the magic bytes converted to slices (#23).
Formatting

[x] Ran cargo fmt
[x] Ran cargo check
[x] Ran cargo build
[ ] Ran nix fmt
[ ] Ran treefmt

Github Issue
This pull request partially implements #2.
Author
Signed-off-by: Popescu Adrian<EMAIL_ADDRESS>

Otherwise this looks good to me, good job @nerodesu017!
When the conflicts (due to the merge of https://github.com/DLR-FT/wasm-interpreter/pull/41) are resolved, we are ready for a merge👍

I inferred that only the feat: i64 ops is the relevant one. I cherry picked that one onto a fresh main branch over https://github.com/DLR-FT/wasm-interpreter/tree/feature/i64-ops-wucke13-fix to solve the merge conflict. Please take a look at it, if you like it, I will force-push that over this PR's branch so that we can eventually merge this PR.


Rake task to upload the parenting and childcare taxonomy

This rake task takes a CSV file containing taxon titles, descriptions
and parents, and creates them in content tagger.
The reason for this task is so we can upload the new taxonomy to staging
and perform a data dump for the prototype. It will also let us bulk tag
all pieces of content related to the parenting and childcare branch, so
they are available in search. This way, we can point the prototype to
staging and be able to show both scoped search results and related
links.
NOTE: this rake task is temporary and only needed until we can
create draft taxons live. Since the prototype will be tested before
that's available, we need it as a temporary step.
There is no need to merge this branch, as it can be deployed to integration and staging as it is.

Rebased on https://github.com/alphagov/content-tagger/pull/316. Merging now, so that we can update https://github.com/alphagov/content-tagger/pull/318 and have this work with the draft workflow.


Pinning Same Version Recursively Across Monorepo?

Clear and concise description of the problem
Quick question - is there a suggested way of pinning versions across a monorepo to be the same?
This is generally not recommended, please feel free to close.

However, I can imagine this might not be a good idea if the user might desire to keep some packages, like React 18 vs React 19, separately pinned in different repos.
Suggested solution
For example taze -r --pin --include typescript would update the package.jsons
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

I would suggest you use https://pnpm.io/catalogs, where Taze also supports that. Otherwise I think it would be a bit out-of-scope of taze to handle this.


[USM] Fix documentation for http2 plain and TLS

What does this PR do?
Corrects HTTP/2 decoding documentation to reflect recent changes made for both HTTP/2 plain and TLS.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


CSS and JS files paths are not correct if website is installed in server subdirectory

This is most likely related to Twigpack plugin. There is console errors like this:
bootstrap:108 Uncaught (in promise) Error: Loading CSS chunk 1 failed.
(/assets/dist/css/styles.6f82a377c329418c8101.css)
    at r.<computed>.r.<computed>.<computed>.t.push.r.<computed>.d.onerror (bootstrap:108:17)

It seems that javascript is trying to load files with address like http://localhost/assets/dist/js/styles.6f82a377c329418c8101.js, ignoring subdirectory. File links in Twig files are outputted correctly and contain subdirectory in which Craft is installed.

Open the .env file and change DEFAULT_SITE_URL= to your desired url


CIP-???? | SpendMany script purpose (replace Spend)

This CIP proposes to replace the current Spend purpose with another one called SpendMany, removing the need of computational waste and higher fees when a transaction validates multiple inputs coming from the same script.
If adopted, Cardano smart contract development would be more efficient, less prone to security issues and easier from a Dev experience.

(rendered proposal in branch)

@matteocoppola following up from my https://github.com/cardano-foundation/CIPs/pull/858#pullrequestreview-2183768383 from CIP meeting today (re: does SpendMany "replace" Spend, add to it, modify it, supersede it and/or deprecate it...):
Consensus is that Ledger input (cc @lehins @WhatisRT) is needed to decide how to go ahead with this.  Everyone agrees the newly defined purpose is an improvement on the old one, with the old use cases being definable with the new script purpose, but can't be sure if there aren't "edge cases" that would break backwards compatibility: and therefore a requirement for versioning and/or supporting both the old & new script purpose.
Because of this question of scope, as well as feasibility of doing this in the Ledger, we've decided to wait for Ledger's initial feedback with some kind of endorsement before assigning this a candidate CIP number.  Once we get to this stage I'll be sure it's on the next CIP meeting agenda.

I'll need to think a bit more about this, but my first thoughts are that this is a great idea. The issue with the redeemer is a bit bigger though, since Ledger also needs to match redeemers to inputs. One idea would be to allow an input to be a list of inputs locked by the same script. Then one can still run the same script with different redeemers while the logic that matches redeemers could be unchanged.

This is definitely a sensible proposal. That being said, we can't do this change for PlutusV1..PlutusV3, since we can't change the context for those versions and addition of a new script purpose is a change to the plutus context. So, we can't "replace Spend", as the title suggests. But, that doesn't mean we can't implement it for PlutusV4. In the matter of fact, I just made a comment on another CIP that we could solve those two proposals with the same solution.
New script purpose could look something like this for plutus:
PV4.SpendingMany [(Integer, TxIn)]

where Integer would refer to the index in the inputs
While for Ledger's redeemer pointer/plutus purpose (i.e. Ledger's script purpose) would look like this:
SpendingMany (Set Word32)

There are a few issues that come with this approach off top my head, neither of which might be an actual problem, but something developers and users need to be aware of. In all of the examples below I assume all inputs are locked by the same script, which would be enforced by ledger:

Same redeemer is expected for multiple inputs. Naturally, datum supplied and execution units must be the same for an identical execution of a script, so this is something to be expected:

Map.fromList [(SpendingMany [1,2,3], (_ :: Data, ExUnits 123 456))]

When different behavior is desired it would still be possible to split it up into multiple purposes:

Map.fromList
  [ (SpendingMany [1,2], (a :: Data, ExUnits 123 456)
  , (SpendingMany [3], (b :: Data, ExUnits 1230 4560)
  ]

Incorrect usage could lead to execution of the same script for the same input multiple times:

Map.fromList
  [ (SpendingMany [1,2], (a :: Data, ExUnits 123 456)
  , (SpendingMany [1,3], (b :: Data, ExUnits 1230 4560)

In this example the script that locks input 1 would be executed twice: once with redeemer (a :: Data, ExUnits 123 456) and the other time with redeemer (b :: Data, ExUnits 1230 4560). That being said, there could be some use cases for this and people actually want this behavior or we could add some ledger rules that prevent duplicate execution. Something that would have to be decided.

@matteocoppola it's noted by @fallen-icarus (https://github.com/cardano-foundation/CIPs/pull/418#issuecomment-2299286201) that this should be considered a solution for candiate CPS-0004.  By convention that CIP & CPS would be linked in the header so please keep that in mind for future revisions as these documents evolve.


<IP_ADDRESS>

@ohld Great job! Can i merge?

It should be tested. Can we make CI build successfully? And add tests?


Remove Deprecated Course Materials

These courses are being replaced by the on-demand modules and have gone a bit stale. This PR removes the no-longer-maintained course materials for GitHub for Everyone and GitHub for Developers. For more up-to-date course materials, keep an 👁  on the on-demand courses at: https://services.github.com/on-demand/

@github/services-training please review that everything looks ok locally!

@brianamarie and @hectorsector thank you for testing these changes - I am going to go ahead and :ship: it!


Remove joda and switch to use java.time (requires java 8)

Obviously a bit contentious. Also, I'm not 100% sure about some of the time zone stuff. Anyway here is the code if anyone wants it

Codecov Report

Merging #620 into master will increase coverage by 0.05%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #620      +/-   ##
============================================
+ Coverage     10.43%   10.49%   +0.05%
- Complexity        0      472     +472
============================================
  Files           550      550
  Lines         20436    20446      +10
  Branches       2626     2018     -608
============================================
+ Hits           2132     2145      +13
+ Misses        18151    18148       -3
  Partials        153      153

Impacted Files
Coverage Δ
Complexity Δ

...t/exchange/webservices/data/core/EwsUtilities.java
19.57% <100%> (ø)
33 <0> (+33)
:arrow_up:

.../exchange/webservices/data/util/DateTimeUtils.java
100% <100%> (ø)
13 <0> (+13)
:arrow_up:

...nge/webservices/data/core/EwsX509TrustManager.java
42.1% <0%> (-2.9%)
1% <0%> (+1%)

...rvices/data/core/request/HttpClientWebRequest.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...ft/exchange/webservices/data/core/PropertySet.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...core/response/ExecuteDiagnosticMethodResponse.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

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 0e1ba52...a67f832. Read the comment docs.


Fixing the linter, first commit of <IP_ADDRESS>

Chained dependencies should be built successfully on a staging channel before review.

Building out dependency chains
PR guidelines

These docs are news to me, thanks!

@SC426 was already working on this, I will retract my effort in favor of his.


Detect first run of different versions

This fixes #1 (except the reset case, since that was already dealt with).

Hi @pteixeira.
I think we can improve by taking a step back and re-thinking this feature: what we want is firstRun to be true per package, per version.
My suggestions is to add a perVersion: true|false (defaults to false) to toggle this behavior. With this in mind:

Default options assignment should be done here: https://github.com/sindresorhus/first-run/blob/master/index.js#L7; you can actually leave it like that because it isn't necessary to initialize opts.perVersion to false explicitly, since undefined is a falsy value.
If opts.perVersion is true, you should re-use the I/O being done here https://github.com/sindresorhus/first-run/blob/master/index.js#L13 to avoid calling readPkgUp two times. You may need to refactor that code to only call readPkgUp once.
We can improve how you store things in the Configstore. Your current implementation will store different keys as new versions are released (e.g.: 1.0.0.firstRun, 1.0.1.firstRun, etc); this is not ideal, since this is actual data that is stored in files. As a consequence, clearState gets more complicated. We can simplify that:

If options.perVersion is false, we store firstRun as true|false as it is now.
If options.perVersion is true, we store firstRun as { version: <version>, run: true|false }; we then compare the stored version with the current one to know if it changed (ofc we also compare if it already ran).
The implementation should work when changing from options.perVersion true to false and from false to true. You may decide if we store firstRun as { version: null, run: true|false }instead oftrue|false` to simplify code.
We should consider if  firstRun() should return true only if the version was incremented?

Hope that you understand my view and try to come up with a new PR that follow my suggestions.

Hi @satazor !
Thank you for the feedback, it helped me see this feature in a different perspective.
Your suggestion of storing firstRun always as { version: <version>, ran: true | false } makes sense and it simplifies things.
Could you clarify one thing though: the only time this program returns true is at the absolute first time the program is executed, regardless of the version and the mode it's running on, right? Or am I missing a case here? That's more or less the reason for my initial approach on this, since the output would change depending on which version we were checking for a previous run.
As for returning true when the version is incremented (checking this could be done using https://github.com/npm/node-semver), it seems interesting and perfectly doable, if you think this won't change the program scope.

I appreciate the pull request, but this is just too far away from what I would like and I unfortunately don't have time to help get it there, so closing this. Sorry.


Fetch aggregated proofs in Sovereign integration test

Summary
Sovereign SDK has implemented JSON RPC APIs for aggregated proofs, such as ledger_getAggregatedProof. The first step to integrating this is to try and fetch the proofs in a simple integration test.

It is unclear whether the aggregated proof service has been fully implemented. This requires study of the Sovereign SDK source code, or consult the Sovereign team.
We need to check whether the prover service is enabled for the rollup node by default. If not, the prover service needs to be enabled when bootstrapping the rollup node.
The initial integration test would submit only a local bank transfer transaction, and attempt to fetch an aggregated proof that captures the state transition.
The fetched aggregated proof would be printed using println!, or written to a file in the test directory.
We should manually inspect the fetched aggregated proof, and ensure that the format matches what is expected by sovereign-ibc.
It is unclear how long it will take to generate the proofs. We should take note of the time taken to run this simple test locally and on the CI.

Things to take note:

We do not use the aggregated proof to implement UpdateClient in this issue. This is more of an initial discovery phase.
For testing, we do want to run the prover service locally or on the CI. i.e. we should not rely on external services to generate the proofs, even if it is expensive.

We would only consider non-local alternatives, if this experiment shows that it is too expensive to generate the proof locally.

This won't be done in the near future.


Wrong size in geometry edition numerical text fields

Context
When editing a geometry, numerical text fields have a wrong size
Steps to Reproduce

Edit a point in a table with a numerical column

Current Result
Those text fields have a wrong width

deployed


Why don't you submit this package to npm

Hi,
Thank you for the great library that makes every JS application easier to handle the file uploads.
Anyway, recently there few people using bower to solve their dependencies but use npm (or yarn) directly. So could you submit this library to npm?
That makes us able to describe the dependence on this library directly on package.json.

Hi @sotarok. I'm working on it in the https://github.com/Rovak/InlineAttachment/tree/dev-3.x branch where i've rewritten the library to ES6 classes. When it's done i will add it to NPM.

@Rovak Thank you for reply me.
That is a good news for me! I can't wait for v3.

looking forward to v3 :)

Indeed, please publish to npm.
Consider doing it with the current version to upload it now

hmm will do, haven't had much time to work on v3

In my personal circumstance, I vendorized the file in my own repo.
Will v3 offer a pre-built, ES5 compatible version in dist/?

yes it will

version 2.0.3 should be available on npm now


ContentFilteredTopic Support

part of https://github.com/ros2/rmw/issues/241

we will re-open the new PR since this is old.


[bug][production] Semicolon on profile pages

Steps to recreate:

Go to any user profile page

Expected:
There to not be a semicolon on the upper left corner of the page.
Actual:
There is a semicolon on the upper left corner of the page.

Fixed in #3595


[Bug] Fix issue6386 and specific windows starter script bug

Purpose of this pull request
This PR is to fix seatunnel-issue-6386 and specific windows starter script bug
Does this PR introduce any user-facing change?
No
How was this patch tested?
Yes,  added unit test cases.
Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

Hi @GangLiCN , thanks for your first contribution, the CI is not pass
It is possible your branch is based on the old dev branch in Apache SeaTunnel, please sync your branch to the latest dev branch. For example as below:
git fetch upstream
git rebase upstream/dev
git push origin YOUR_BRANCH --force

I've submitted a new PR:
https://github.com/apache/seatunnel/pull/6532


add tracker to interpreter context

Proposed changes:

Add the tracker to the interpreter context so that it can be used by custom components in the NLU pipeline.

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

 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.Daniel Popescu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi. I had this problem while trying to do coreference. The tracker (especially the events) isn't available in any way in the interpreter. This is the cleanest way, cause it simply puts it in the interpreter's context variable, which as far as I can see, it's exactly that, a 'context' memory area.

Hi @pdan93 we are currently in the process of revamping the entire rasa architecture for the 3.0 release scheduled in Q4-21. These changes will allow for much more flexible custom models, I would suggest waiting for this release and checking if the new changes solve your use-case?

Hi @pdan93 we are currently in the process of revamping the entire rasa architecture for the 3.0 release scheduled in Q4-21. These changes will allow for much more flexible custom models, I would suggest waiting for this release and checking if the new changes solve your use-case?

Hello. Sounds good. I kind of expected that you would redo the arhitecture in 3.0. It's good to hear it's scheduled for Q4 :)
Thanks

@pdan93 We are currently working on a big architecture revamp which will unfortunately invalidate the given change. We will hopefully merge our changes to main soon (somewhen next week). Happy to discuss how we can apply such a change then (would be great if you could open an issue to discuss this first as this change has quite a lot of implications).


Fill in November demo script

See https://app.zenhub.com/workspace/o/humancellatlas/data-store/issues/1511
And see the script doc I've started: https://docs.google.com/document/d/1EAulvcSrOrT1Z58lYquyZuz1Zg2iWqGo3qiD3zL7nBc/edit#
For status see https://app.zenhub.com/workspace/o/humancellatlas/dcp/boards?repos=106337233&showPRs=false

Successful demo given on Nov 1.


SAMOA-36: Update Apache Flink Adapter to 0.9 stable release

This patch upgrades the Flink adapter to 0.9.0 stable release. It includes several bug fixes and performance improvements along with compatibility with the new operator interfaces of Flink.
I got a x3 speedup on VHT compared to the previous version using a local cluster with 8 cores.

Looks good to me, this is an important update. :+1:

+1

One small detail. The copyright in the pom.xml needs to be updated to the ASF and to the right years (see the main pom.xml for an example).

@gdfm  Indeed, thanks for noticing. I just updated it and rebased

Thanks. I'll commit this as soon as I get another +1.
In the meanwhile, given the new merge policy, could you prepend "SAMOA-36: " to the commit log?

+1


Dependencies miss index

Dependencies do not have an index set in Pipfile.lock
$ python -m pipenv.help output
Pipenv version: '11.10.1'
Pipenv location: '/home/florian/.local/venvs/pipenv/lib/python3.5/site-packages/pipenv'
Python location: '/home/florian/.local/venvs/pipenv/bin/python'
Other Python installations in PATH:

2.5: /usr/local/bin/python2.5

2.5: /usr/local/bin/python2.5

2.7: /usr/bin/python2.7

2.7: /usr/bin/python2.7

3.5: /usr/bin/python3.5m

3.5: /usr/bin/python3.5

3.6: /usr/bin/python3.6m

3.6: /usr/bin/python3.6

2.7.15: /usr/bin/python

2.7.15: /usr/bin/python2

3.6.5: /usr/bin/python3

PEP 508 Information:
{'implementation_name': 'cpython',
 'implementation_version': '3.5.4',
 'os_name': 'posix',
 'platform_machine': 'x86_64',
 'platform_python_implementation': 'CPython',
 'platform_release': '4.15.0-3-amd64',
 'platform_system': 'Linux',
 'platform_version': '#1 SMP Debian 4.15.17-1 (2018-04-19)',
 'python_full_version': '3.5.4',
 'python_version': '3.5',
 'sys_platform': 'linux'}

System environment variables:

XDG_VTNR
LC_MONETARY
GDMSESSION
EDITOR
XDG_SESSION_ID
VTE_VERSION
GIO_LAUNCHED_DESKTOP_FILE_PID
COLORTERM
GTK_MODULES
GIO_LAUNCHED_DESKTOP_FILE
XDG_SESSION_TYPE
LC_CTYPE
XDG_DATA_DIRS
GJS_DEBUG_OUTPUT
USER
PAGER
PROJECT_HOME
SSH_AUTH_SOCK
SHELL
LC_TIME
TERMINATOR_DBUS_PATH
XDG_SESSION_COOKIE
XDG_CURRENT_DESKTOP
XDG_SESSION_DESKTOP
LANG
WINDOWPATH
OLDPWD
XDG_SEAT
SSH_AGENT_PID
PYTHONDONTWRITEBYTECODE
_
QT_ACCESSIBILITY
DISPLAY
DBUS_SESSION_BUS_ADDRESS
TERMINATOR_UUID
GJS_DEBUG_TOPICS
LSCOLORS
PEX_INHERIT_PATH
LESS
XDG_MENU_PREFIX
LC_MEASUREMENT
S_COLORS
ZSH
ANSIBLE_NOCOWS
LC_PAPER
LS_COLORS
DESKTOP_SESSION
SESSION_MANAGER
PWD
LOGNAME
QT_SELECT
TERMINATOR_DBUS_NAME
HOME
XDG_RUNTIME_DIR
XAUTHORITY
LC_NUMERIC
ELECTRON_TRASH
GDM_LANG
SHLVL
GPG_AGENT_INFO
PIP_PYTHON_PATH
USERNAME
GNOME_DESKTOP_SESSION_ID
TERM
PATH

Pipenv–specific environment variables:
Debug–specific environment variables:

PATH: /home/florian/.yarn/bin:/usr/local/heroku/bin:/home/florian/.local/bin:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games
SHELL: /usr/bin/zsh
EDITOR: vim
LANG: en_US.utf8
PWD: /home/florian/.local/venvs/pipenv

Expected result
Pipfile.lock should contain an index key for every package.
Actual result
This is what Pipfile.lock looks like after pipenv install Django:
{
    "_meta": {
        "hash": {
            "sha256": "68309cd71a258c30a39567fce09a09ad5c4ff0bdc85b6fba22b47598c985c883"
        },
        "pipfile-spec": 6,
        "requires": {
            "python_version": "3.6"
        },
        "sources": [
            {
                "name": "pypi",
                "url": "https://pypi.org/simple",
                "verify_ssl": true
            }
        ]
    },
    "default": {
        "django": {
            "hashes": [
                "sha256:2d8b9eed8815f172a8e898678ae4289a5e9176bc08295676eff4228dd638ea61",
                "sha256:d81a1652963c81488e709729a80b510394050e312f386037f26b54912a3a10d0"
            ],
            "index": "pypi",
            "version": "==2.0.4"
        },
        "pytz": {
            "hashes": [
                "sha256:65ae0c8101309c45772196b21b74c46b2e5d11b6275c45d251b150d5da334555",
                "sha256:c06425302f2cf668f1bba7a0a03f3c1d34d4ebeef2c72003da308b3947c7f749"
            ],
            "version": "==2018.4"
        }
    },
    "develop": {}
}

Which index did pytz come from?
Steps to replicate
Run pipenv install Django in an empty pipfile and look for the index key for pytz in the lockfile.

Hi, thanks for reporting this. Did you notice something wrong while using pipenv? Was there a bug or an issue you came across?
For context, this is not a required key and currently its presence depends on 1) whether you have multiple indexes specified and it came from a non-primary index, or 2) it was resolved via the fallback resolver and not a cached lookup or a json query (I might have the last one backwards). Given that you have only once source specified, suffice to say it came from pypi.

Hi, yeah I noticed it while reporting #2098 and thought I'd open a bugreport in case it was an oversight. Thanks for clearing that up!
That said, it is still weird that Django would have the index set, since it clearly also can come only from pypi ;)

Yeah there is a reason but it was an unintentional side effect or something like that


style(notification-bar): edit directory name

To Reviewers

[ ] Skipping review is not a problem (style, chore ONLY)
[x] Not that difficult

Type of Change

[ ] New feature
[ ] Bug fixes
[ ] Feature improvement

[x] These changes require a usage change

[x] Others (performance improvement, refactoring, CI/CD, etc.)

Checklist

[ ] Updated Storybook documents for component
[ ] Wrote test codes for composable, utils and etc.
[ ] Tested with console(if usages are changed)

Description

edit directory name (notification > notification-bar)

:tada: This PR is included in version 2.0.0-beta.34 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Change the entities collection to each side of Many to Many relationship

Q
A

Version
2.5.11

Symfony Version
4.1.7

hi!
I created a Many to Many relationship with two entities convention <-> profil.
Entity Convention
/**
     * Many convention have Many profil.
     * @ORM\ManyToMany(targetEntity="Profil", mappedBy="conventions")

     */
    private $profils;

Entity Profil
    /**
     * Many profil have many convention
     * @ORM\ManyToMany(targetEntity="Convention", inversedBy="profils")
     * @ORM\JoinTable(name="asso_profil_convention",
     *      joinColumns={@ORM\JoinColumn(name="profil_id", referencedColumnName="profil_id")},
     *      inverseJoinColumns={@ORM\JoinColumn(name="convention_id", referencedColumnName="convention_id")})
     */
    private $conventions;

My problem when i try to set profil in the convention the insert in my association table does'nt work but the invers yes.
I have tried many thing and I solved my problem with that:
    /**
     * Many convention have Many profil.
     * @ORM\ManyToMany(targetEntity="Profil", inversedBy="conventions")
     * @ORM\JoinTable(name="asso_profil_convention",
     *      joinColumns={@ORM\JoinColumn(name="convention_id", referencedColumnName="convention_id")},
     *      inverseJoinColumns={@ORM\JoinColumn(name="profil_id", referencedColumnName="profil_id")})
     */
    private $profils;

But I get this error "The field App\Entity\Acces\Convention#profils is on the owning side of a bi-directional relationship, but the specified mappedBy association on the target-entity App\Entity\Acces\Profil# does not contain the required 'inversedBy' attribute."

You need something like this:
class Profil
{
    // ...
    /** @ORM\ManyToMany(targetEntity=Convention::class, inversedBy="profils") */
    private $conventions;
    // ...
}

class Convention
{
   // ...
    /** @ORM\ManyToMany(targetEntity=Profil::class, mappedBy="conventions") */
    private $profils;
    // ...
}

The @JoinTable (if needed) should be on the association that has the inversedBy attribute

Thanks for your answer,
I need the @JoinTable  without I have one error who specify  write the @JoinTable,
For define the targetEntity the  : : class doesn't expected in Symfony 4
I suposed the problem doesn't come from this.
If you need more element for uderstand my problem ask me.
Thanks to take your time to answer me

You can either use ::class or the FQCN (fully qualified class name) of the referenced class between quotes. If you use ::class, then it must be imported or in the same namespace.

Hi
I try your tow method but I have the same problem when I try to modify profil in my CRUD Convention
no change is operated
And when I look the trace of sql,  I saw the select of all profil but no one insert in my association table
My entitys are in the same namespace


docs(connectivity): mention mtu changes as a troubleshooting step

Lowering the MTU can fix many subtle connectivity issues, so I thought it should be mentioned in the docs.

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

GitHub release
v1.4.3

Your semantic-release bot :package::rocket:


API not working for push..

hi it's a bit urgent as
the api for mongolabs have changed can you please look into it, and we are not able to push even a single notification although the devices are getting added properly..
can you please reconfirm that the issue is due to the change from mogolab to mlabs
or is it due to my internal mistakes..
awaiting your response for the same....

Not sure to understand what the issue is ?
Can you be a little mote specific ?

hi tried using another demo project was working. sorry for the inconvenience..


chore: add text and html for coverage reporter

To check the reports in the local machine.
The author should do the following, if applicable

[ ] Add tests
[ ] Run tests
[ ] bun run format:fix && bun run lint:fix to format the code
[ ] Add TSDoc/JSDoc to document the code

This will be helpful. Merging now.


fix typo: on-premises missing s

Microsoft Style Guide nominates On-premises (plural) as correct here


add postgis support to schema_dumper

I was getting WARNING: generating adapter-specific definition and no candidate create_trigger statement could be found when switching from the PostgreSQL to the PostGIS ActiveRecord adapter. This should fix that.

Would be nice to have this merged. Is there anything I can do to help @jenseng ?

released in v0.2.24


Literal on-disk index with index write path and on-disk index query path

This patch includes the literal on-disk index along with the index write path and on-disk index query path. The patch is organised as follows:

The literal on-disk index is based on an on-disk trie terms dictionary that uses an trie structure to map ByteComparable terms to long postings. The trie structure is managed on disk by the TrieTermsDictionaryWriter and TrieTermsDictionaryReader and the postings are managed on disk by the PostingsWriter and PostingsReader. Control of the writers is handled by the InvertedIndexWriter.
The index build is handled handled by StorageAttachedIndexBuilder. An index build relates to a build invoked by the SecondaryIndexManager when a new a new index is created or when the SIM is asked to rebuild an index by NodeTool.
The index is also written by the StorageAttachedIndexWriter when a Memtable is flushed or SSTables are compacted or loaded by repair or SSTable loading. A StorageAttachedIndexWriter is created by the StorageAttachedIndexGroup when an IFlushObserver is requested by the SSTableWriter.
The on-disk indexes are queried by an implementation of SearchableIndex. This implementation is responsible for coordinating the query response from a single SSTable. The QueryController is responsible for mapping query expressions to specific SSTable indexes.

The feature branch has been rebased to trunk at 3eb605b4db0fa6b1ab67b85724a9cfbf00aae7de


Cu 21prutc enter search terms that will be used to search the elements in the regulatory domain matthew clements

Both task C-21prutc and CU-21prutm are included in this PR

Task linked: CU-21prutc Enter search terms that will be used to search the elements in the regulatory domain

Task linked: CU-21prutm Meaningful title for each node at the top of the information page


Add outline of Smsdia: SD interrupt-file assignment

Add outline of Smsdia: SD interrupt-file assignment

@ved-rivos Please take a look at this outline - if it looks ok, please merge it and then add additional edits.

Should we define the addresses of the new CSRs (msdeip and msdeie) with this PR (Smsdia)?
Do you assume they are indirect CSRs (Smcsrind/Sscsrind)

I dont see need for them to be indirect CSRs. Usually CSR numbers are provided by AR later.


use requirements.txt format for python dependencies

along with instructions on copy-pasting to a file requirements.txt and using the appropriate conda or pip commands with --file  option

I remamed the old binder-environment repo to software and testing here:
https://github.com/coderefinery/software/tree/rkdarst/update-conda
so far just environment.yml, though.

It seems that requirement.txt was made by @rkdarst in https://github.com/coderefinery/software in the master branch on the same date as the above comment is made. Should this issue be closed?

Can be closed. There is another issue open to check that they are still up to date.


SDK: test exporters and processors

Empty tests:

[ ] https://github.com/open-telemetry/opentelemetry-ruby/blob/master/sdk/test/opentelemetry/sdk/trace/noop_span_processor_test.rb
[ ] https://github.com/open-telemetry/opentelemetry-ruby/blob/master/sdk/test/opentelemetry/sdk/trace/multi_span_processor.rb (file misnamed)
[ ] https://github.com/open-telemetry/opentelemetry-ruby/blob/master/sdk/test/opentelemetry/sdk/trace/export/in_memory_span_exporter_test.rb
[ ] https://github.com/open-telemetry/opentelemetry-ruby/blob/master/sdk/test/opentelemetry/sdk/trace/export/multi_span_exporter_test.rb
[ ] https://github.com/open-telemetry/opentelemetry-ruby/blob/master/sdk/test/opentelemetry/sdk/trace/export/noop_span_exporter_test.rb
[ ] https://github.com/open-telemetry/opentelemetry-ruby/blob/master/sdk/test/opentelemetry/sdk/trace/export/simple_sampled_span_processor_test.rb

Note that https://github.com/open-telemetry/opentelemetry-ruby/blob/master/sdk/test/opentelemetry/sdk/trace/export/batch_sampled_span_processor_test.rb will be implemented in #74

I'm going to make a start on this today.


Migrate to imgui-rs 0.10.0

Migration is currently blocked by michaelfairley/rust-imgui-opengl-renderer/pull/22.
I have decided to drop support for OpenGL3 for the time being as it's the only way we can move forward on our end. Once we find a way of having our own renderer, re-adding support will be trivial.
@Jakobzs would you be interested in trying to tackle writing an OpenGL rendering engine for hudhook, as opposed to using the external one? Sadly I have little to no knowledge of OpenGL myself and wouldn't know where to begin, though I feel confident I could easily manage incremental updates over time (especially if we end up implementing #42 as a form of regression testing).

We talked it over on Discord. I decided to fork the imgui-opengl-renderer crate and create one called imgui-opengl that I'll be maintaining, primarily also cause it looks to be very low maintenance.
I have PR'ed a fix for OpenGL3 that includes the adding of the fork which contains the fixes for imgui 0.9.0.

0.10.0 came out already, looking at the changelog it shouldn't have anything breaking on our side but it'll require a bump from all dependencies again. While 0.4 isn't released yet, might as well just bump that too.

do you have an ETA of when this would be migrated to 0.10.0? @veeenu

Probably not before April as this is a very very busy time of the year for me.
There are also other issues scheduled for 0.4 that I'd like to get sorted. But I'll gladly accept and review PRs, and once that's in main one could use the GitHub link as dependency until 0.4 hits.
This particular one should be rather simple, essentially just bump deps and change stuff until it works again.

Closed by #88


This makes the headline of the post list page customizable.

(Closes #166)

This PR is updated to make use of "posts-headline" and "posts_headline" now.

Looks better, I add a little comment to be ready to merge :-)

Okay ... it's updated.