nerdctl pull: failed to prepare extraction snapshot, parent snapshot does not exist (containerd bug?)

Description
Looks like in certain circumstances, a simple pull in a clean namespace will trigger a failure from deep inside containerd.
@AkihiroSuda do you have an opinion on this?
=== RUN   TestRemove/Remove_image_with_created_container_-_with_-f
    image_remove_test.go:303: ======================== Pre-test cleanup ========================
    command.go:112: /usr/local/bin/nerdctl --namespace=testremove-private-a0b107b8 rm -f testremove-remove-image-with-created-container-with-f-1cdecc80
    image_remove_test.go:303: ======================== Test setup ========================
    command.go:112: /usr/local/bin/nerdctl --namespace=testremove-private-a0b107b8 pull ghcr.io/stargz-containers/nginx:1.19-alpine-org
    command.go:112: assertion failed: expect.ExitCode is not result.ExitCode: Expected exit code: 0

        Command:  /usr/local/bin/nerdctl --namespace=testremove-private-a0b107b8 pull ghcr.io/stargz-containers/nginx:1.19-alpine-org
        ExitCode: 1
        Error:    exit status 1
        Stdout:
        Stderr:   time="2024-10-15T04:52:41Z" level=fatal msg="failed to prepare extraction snapshot \"extract-549308819-2Mnm sha256:1b4e58f0c6175a24a65c2434758f04fc39b962c9634fca3af65470cfc27eb0bc\": parent snapshot sha256:8ea3b23f387bedc5e3cee574742d748941443c328a75f511eb37b0d8b6164130 does not exist: not found"

        Env:
        HOSTNAME=f807fe8d3cc9
        SYSTEMD_EXEC_PID=47
        container=docker
        HOME=/root
        LANG=C.UTF-8
        INVOCATION_ID=acc9030803844f7d9d1f213be2cb868c
        TERM=xterm
        SHLVL=3
        CGO_ENABLED=0
        _=/usr/local/bin/gotestsum
        PATH=/usr/local/go/bin:/usr/local/go/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
        ***
        DOCKER_CONFIG=/tmp/TestRemoveRemove_image_with_created_container_-_with_-f109564665/001
        NERDCTL_TOML=/tmp/TestRemoveRemove_image_with_created_container_-_with_-f109564665/001/nerdctl.toml
    case.go:164: ======================== Post-test cleanup ========================
    command.go:112: /usr/local/bin/nerdctl --namespace=testremove-private-a0b107b8 rm -f testremove-remove-image-with-created-container-with-f-1cdecc80
--- FAIL: TestRemove/Remove_image_with_created_container_-_with_-f (0.18s)

Steps to reproduce the issue
No response
Describe the results you received and expected
na
What version of nerdctl are you using?
main
Are you using a variant of nerdctl? (e.g., Rancher Desktop)
None
Host information
No response

Maybe? Does it happen with ctr or crictl too?

I think it is racy. Hard to repro on demand for now.
Let's keep an eye on the CI and see if it pops-up again (since we moved to 1.7.23 very recently).


Wip/tree

Added functionality for the tree component to generate a file directory-like tree structure based on dummy data in dummyData.ts

LGTM!


Many bonds are being assigned wildcard parameters in alpha-cyclodextrin

I'm trying to parameterize a host molecule and keep running into many bonds being not matched at all (i.e., getting assigned the default <Bond smirks="[*:1]~[*:2]" id="b1" k="2000.0" length="4.0"/>). It seems to happen for this molecule with either GAFF or SYBYL atom types. The bonds that are mismatched are mostly, but not entirely, between residues.
A short example can be found at this notebook. The notebook takes a few seconds to load because I printed the contents of the mol2 and pdb files at the bottom. You can also get the relevant files by cloning https://github.com/slochower/smirnoff-host-guest and running smirnoff-bond-matching-error.ipynb.
CC @bannanc

The most obvious thing to check as a starting point would be to loop over the atoms and print their elements. I'd guess this is a problem with reading your OEMol... (Though it might be nice to know an example of a bond which is getting assigned this paraemter just to be sure.)

Where did you get your SYBYL types?  (heh, from me?) They look wrong, specifically Os and Ho.
The only remaining link to the Tripos File Format PDF is down, but this site has the types:
http://www.sdsc.edu/CCMS/Packages/cambridge/pluto/atom_types.html#B

Haha. "Os" is what OpenEye gives you if you read in a molecule with GAFF types without using the "Forcefield" flavor reader and write it back out; you tend to get certain oxygen types being interpreted as "Osmium" (Os), which inevitably results in exactly this issue.
(We really need to add the input sanity checks in openforcefield to start giving more informative error messages for this type of thing.)

@nhenriksen I used OpenEye to assign Tripos atom types and write out a mol2 -- so, not from you. But if OpenEye has a problem read in the initial atom types, obviously it's not going to write the correct SYBYL atom types.
@davidlmobley @bannanc Yes, osmium it is. Do you mean OEIFlavor_MOL2_GeneralFFFormat? I did expect that reading GAFF atom types would be trouble without using that, but I didn't expect problems with SYBYL atom types, and so I wasn't using OEIFlavor_MOL2_GeneralFFFormat. The piece that I overlooked is that I had previously (not in this example, but elsewhere) used OpenEye tools to write the SYBYL formatted atom types, and at that time, OpenEye had written osmium for oxygen atom types. Ignoring the SYBYL formatted mol2 and forcing OpenEye to read the GAFF atom types with OEIFlavor_MOL2_GeneralFFFormat seems to fix the issue! Will further test this weekend.
Thanks, all!

I would suggest you use antechamber to convert the original GAFF mol2 to a SYBYL mol2.  I know it's adding another AMBER step, but your already starting with AMBER files, so you might as well use something that OpenEye understands.

@nhenriksen Yep.

Key issue, @slochower , is that if you ever read and/or write an mol2 file with GAFF types using the OpenEye tools without using the forcefield flavor mol2 reader/writers, you'll have exactly this issue (you end up with Osmium or similar in your atom types, and there's no escaping once you have that). As long as you use the forcefield flavor anytime you interact with GAFF or non-SYBYL types (at least in my experience) it behaves, but if you ever don't do that then you end up with incorrect atom types and it's all downhill from there.
Shall I mark this as closed?

Yes!
FWIW, I wrote this quick little function that I'm looping into all my system preparations. Could be a useful sanity check for other people:
def check_bond_lengths(structure, threshold):
    """
    Print out any equilibrium bond lengths above a certain threshold, to make sure all the assigned parameters are sensible.

    Parameters:
    ----------
    structure : pmd.structure
        ParmEd structure with parameters
    threshold : float
        Bond length (`R eq`)
    """
    bad_bonds = False
    for bond in structure.bonds:
        atom1, atom2 = bond.atom1, bond.atom2
        if bond.type is not None:
            if bond.type.req > threshold:
                print(f'{atom1.idx + 1:7d} {atom1.name:4} ({atom1.type:4}) {atom2.idx + 1:7d} '
                    f'{atom2.name:4} ({atom2.type:4}) {bond.type.req:10.4f} {bond.type.k:10.4f}')
                bad_bonds = True
    if not bad_bonds:
        print('Structure looks good.')

Also, I would suggest setting the wildcard to something worse than 4.0 Å, if that is technically possible. That value was not quite bad enough to make my simulations crash -- and I just ended up with a distorted alpha-cyclodextrin -- so I spent a chunk of time thinking the issue was mapping the MD restraints to new atom indices. If the wildcard bond length was something like 10 or 999 Å, I think the simulations would crash immediately and I might have checked the parameters more quickly. Thoughts?

Or, actually, rather than messing with the wildcard length, it would probably be fine to simply print out that a certain bond is matching the wildcard. That would have also gotten my attention and would be less disruptive than setting the wildcard to 999 Å.

We're actually just going to remove the wildcard (see #72 ) so you'll get an Exception in such cases.
Sorry about that.


check for nil testsuite when parsing test results

If a project has no tests and neotrest tries to run tests, no tests are found.
If a project has tests, and the tests are removed, and then neotest tries to run tests, it gets the following error:
neotest-rust: ~/.local/share/nvim/lazy/neotest/lua/nio/init.lua:105: The coroutine failed with this message:
...l/share/nvim/lazy/neotest-rust/lua/neotest-rust/init.lua:362: attempt to get length of field 'testsuite' (a nil value)
stack traceback:
        ...l/share/nvim/lazy/neotest-rust/lua/neotest-rust/init.lua: in function 'results'
        ...al/share/nvim/lazy/neotest/lua/neotest/client/runner.lua:131: in function '_run_spec'
        ...al/share/nvim/lazy/neotest/lua/neotest/client/runner.lua:89: in function <...al/share/nvim/lazy/neotest/lua/neotest/client/runner.lua:88>

This seems reasonable. Please run pre-commit to format the files and I’ll approve it!

ran pre-commit and rebased


feat: custom Gemini api base

Background: Google Generative AI API error: "User location is not supported for the API use." - Stack Overflow
The feature is for the people not in the available regions for now. https://ai.google.dev/available_regions

Why do you need a custom Gemini API base? @cyio

Why do you need a custom Gemini API base? @cyio

Added the background in the top comment.

I think you mixed up https://cloud.google.com/vertex-ai/docs/generative-ai/start/quickstarts/quickstart-multimodal and https://ai.google.dev/docs.
They are not the same. @cyio

I think you mixed up https://cloud.google.com/vertex-ai/docs/generative-ai/start/quickstarts/quickstart-multimodal and https://ai.google.dev/docs.
They are not the same. @cyio

@sigoden I meaning generative-ai.

#308 is what your need.


[Proposal] Web Support for fl_chart

The package is working perfectly for web, but on pub.dev web platform is not showing. I forked the repo and ran flutter create . but it's not creating web folder. So maybe, you can try out the same and enable flutter web support and then push and publish on pub.dev again thereby providing web support!

I think we should write this in our pubspec.yaml.
flutter:
  plugin:
    platforms:
      web:

Okk let me try this and create PR for the same

Oh got it.
Consider supporting multiple platforms:

Package not compatible with runtime flutter-web on Web
Because:

package:fl_chart/fl_chart.dart that imports:
package:fl_chart/src/chart/scatter_chart/scatter_chart.dart that imports:
package:fl_chart/src/chart/scatter_chart/scatter_chart_renderer.dart that imports:
package:fl_chart/src/chart/scatter_chart/scatter_chart_painter.dart that imports:
package:fl_chart/src/chart/base/base_chart/base_chart_painter.dart that imports:
package:fl_chart/src/chart/base/base_chart/base_chart_data.dart that imports:
package:fl_chart/src/chart/base/base_chart/fl_touch_event.dart that imports:
dart:io

Yeah, dart:io isn't supported for Web. Maybe you can use https://pub.dev/packages/universal_io as replacement for dart:io

Yes, we need to check why we needed dart:io there

Did you use Transform or something similar to that?

I will replace the package and see if there's any side effects

No, I just use Platforms.isLinux, ...
Thanks!

BTW It's better to not use a package to solve this (If we can solve it without needing to importing a package)

Agreed.

@imaNNeoFighT So just to check, I removed import 'dart:io'; and to changed the  final isDesktopOrWeb = kIsWeb; but it is still not creating the web folder. So it's not an issue of dart:io

I also tried by adding the following but still no luck

I think we should write this in our pubspec.yaml.
flutter:
  plugin:
    platforms:
      web:

@AbhishekDoshi26 The problem is not the existence of the web directory.
(Actually, there is no other linux, android, or any other directories)
The directory is not an indicator of supporting it. If we can remove dart:io, pub.dev will indicate our library to support web automatically.

@leidig54 Actually the package works perfectly well on web. The only thing is, due to some reason, it is not showing web support on pub.dev
So you can use it for web without any issues.


Dimension Builder only accepts 50K RF/t total

According to the wiki and the in-game documentation, the dimension builder should accept 50K RF/t per side for a total of 300K RF/t.  However, the default configuration seems to be set to 50K RF/t total - DimletConfiguration.java:16 and DimletConfiguration.java:107.
In FTB Infinity 1.8.1, I can only get the dimension builder to accept 50K RF/t total, with one exception: adding tesseracts can increases it another 50K RF/t total (regardless of the number of tesseracts added - 1 tesseract adds 50K total, 5 tesseracts adds 50K total).

Can you show me your setup because I have people using this correclty with cryoducts so I'm interested to see how you set it up.

Just found my problem - user error.  Sorry to bug you!


Parsing error on multiple queries when CREATE RETENTION POLICY doesn't end in DEFAULT

Bug report
When running multiple queries separated by semicolon, when one of the queries (which is not the last query) is a CREATE REPLICATION POLICY query does not end in DEFAULT (for example, CREATE RETENTION POLICY "10m.events" ON somedb DURATION 60m REPLICATION <int>; SELECT a from somedb), the query fails with an error unexpected error: found SELECT, expected ; at line 1, char 61
I've tried a lot of other multiple queries, but the only ones that fail are when it CREATE RETENTION POLICY does not end with DEFAULT.
System info: InfluxDB 0.12.1-1 running on Ubuntu 14.04.4 LTS (GNU/Linux 3.13.0-79-generic x86_64) installed via RPM, within docker image tutum/influxdb:0.12, and installed from source using your build-docker.sh script.
Steps to reproduce:

Using InfluxCLI, HTTP query, client libraries, admin interface, run multiple queries where the first query is CREATE RETENTION POLICY "10m.events" ON somedb DURATION 60m REPLICATION <int> {SHARD DURATION 30m} and the second query is anything, and the two queries are separated by a ; semicolon.
error is returned: ERR: error parsing query: found SELECT, expected ; at line 1, char 86
Retention policy is not created

Expected behavior: Creation of retention policy and results from second query
Actual behavior: Parsing Error ERR: error parsing query: found SELECT, expected ; at line 1, char 86, retention policy not created, no results returned
Additional info:
Sample of multiple queries that work and do not work. If you switch the queries to SELECT is first and CREATE RETENTION POLICY is last, it works as expected. But if you add another query after CREATE RETENTION POLICY, you receive an error.
Also, there are no errors if you use two semicolons ;; after CREATE RETENTION POLICY.
> CREATE RETENTION POLICY rp_90d2 on db duration 90d REPLICATION 2 SHARD DURATION 30m DEFAULT; select * from db;
> show retention policies
ERR: error parsing query: found EOF, expected ON at line 1, char 25
> show retention policies on db
name	duration	shardGroupDuration	replicaN	default
default	0		168h0m0s		1		false
rp_90d2	2160h0m0s	30m0s			2		true

> CREATE RETENTION POLICY rp_90d3 on db duration 90d REPLICATION 2 SHARD DURATION 30m; select * from db;
ERR: error parsing query: found SELECT, expected ; at line 1, char 86
> show retention policies on db
name	duration	shardGroupDuration	replicaN	default
default	0		168h0m0s		1		false
rp_90d2	2160h0m0s	30m0s			2		true

> CREATE RETENTION POLICY rp_90d4 on db duration 90d REPLICATION 2; select * from db;
ERR: error parsing query: found SELECT, expected ; at line 1, char 67
> show retention policies on db
name	duration	shardGroupDuration	replicaN	default
default	0		168h0m0s		1		false
rp_90d2	2160h0m0s	30m0s			2		true

> CREATE RETENTION POLICY rp_90d4 on db duration 90d REPLICATION 2 DEFAULT; select * from db;
> show retention policies on db
name	duration	shardGroupDuration	replicaN	default
default	0		168h0m0s		1		false
rp_90d2	2160h0m0s	30m0s			2		false
rp_90d4	2160h0m0s	24h0m0s			2		true

Additionally, I added a test to the influxql/parser_test.go code, which failed:
func TestParser_ParseQuery_MultipleCreateRP(t *testing.T) {
	s := `CREATE RETENTION POLICY rp on d duration 90d replication 1; select a from d;`
	q, err := influxql.NewParser(strings.NewReader(s)).ParseQuery()
	if err != nil {
		t.Fatalf("unexpected error: %s", err)
	} else if len(q.Statements) != 2 {
		t.Fatalf("unexpected statement count: %d", len(q.Statements))
	}
}

It looks like it's the parse is ignoring the semicolon on these specific queries.


Revert "Send and get data from api- Milestone-2"

This pull request for milestone-2
Here is last TSE review
He said he cant start web and he cant do inline comments. But it was working well. I send another review but now it looks approved in my dashboard. I can't send to another review in dashboard for milestone-2.
Page is working in my side. If it is approved please do request change so ı can able to send request in my dashboard  for milestone-3
In this milestone;

Use leaderboard api
Create function for get data from api
Create function for send data to api
Use async and await features in these functions.

@arslanbisharat  You shouldn't approved it. Now ı can't send another review for milestone-3


Lab2

@nchen269 @SaranshPK I completed Lab2.

Good job! I spotted two minor things from problem 3C:
First sentence: The probability p would be the background produces a signal equal or greater than the signal?
Second sentence: “The probability that the background  produces a signal larger than my candidate signal on at least one pixel in all the 10k pixel
is (1 - p^10000) = 1.44e-09, which is approximately 1/10000 the probability I got from Part A).”
Did you mean “10000 times greater than the probability”?
For 3B, I wrote question as "What is the probablity detecting signal with strength equal to or greater than 7.8 in the 10k pixels CCDs?", but I'm not super sure if that's what the question was asking for.

Good job! I spotted two minor things from problem 3C:
First sentence: The probability p would be the background produces a signal equal or greater than the signal?
Second sentence: “The probability that the background produces a signal larger than my candidate signal on at least one pixel in all the 10k pixel
is (1 - p^10000) = 1.44e-09, which is a3 pproximately 1/10000 the probability I got from Part A).”
Did you mean “10000 times greater than the probability”?
For 3B, I wrote question as "What is the probablity detecting signal with strength equal to or greater than 7.8 in the 10k pixels CCDs?", but I'm not super sure if that's what the question was asking for.

Thanks for your comments.
First sentence: I think I still want to write "not as large as" instead of "equal to or greater than." But I did miss an "as."
Second sentence: you're correct.
3 B): Yeah, I think I'll also add this.

@kentwils I finished Lab2.
Tianhao Jiang


return an async iterator when using streaming and acreate

Describe the feature or improvement you're requesting
I don't know if it is a bug or a missing feature, but I found the return value from acreate is not an async iterator and I'd like to have it.
The code is simple as the following
def sync_stream():
    # Runs totally fine
    response = openai.ChatCompletion.create(
        messages=[{
            "role": "user",
            "content": "hello"
        }],
        temperature=0.2,
        stream=True,
    )

    for trunk in response:
        print(trunk)

async def async_stream():
    response = openai.ChatCompletion.acreate(
        messages=[{
            "role": "user",
            "content": "hello"
        }],
        temperature=0.2,
        stream=True,
    )
    # TypeError: 'async for' requires an object with __aiter__ method, got coroutine
    async for trunk in response:
        print(trunk)

Additional context
No response

You are not await the acreate call...


GUI: Installer: check the network of xpubs when importing descriptor

I can import a descriptor with tpubs on mainnet atm.

@edouardparis actually @kloaec just hit this issue again with v2...


2022092001

Closes #7522
Closes #7493
Closes #7495
Closes #7496
Closes #7497
Closes #7498
Closes #7499
Closes #7501
Closes #7512
Closes #7521
Closes #7533
Closes #7532
Closes #7525
Closes #6562
Closes #6542
Closes #6543
Closes #6544
Closes #6545
Closes #6546
Closes #6547
Closes #6550
Closes #6553
Closes #6554
Closes #6555
Closes #6556
Closes #7552
Closes #7554
Closes #7557
Closes #7558
Closes #7559
Closes #7560
Closes #7562
Closes #7563
Closes #7564
Closes #7565
Closes #7567
Closes #7568
Closes #7569
Closes #7570
Closes #7571
Closes #7535
Closes #7537
Closes #7538
Closes #7539
Closes #7540
Closes #7548
Closes #7549
Closes #7550

Still #7327 - #7335 - #7340 - #7341 - #7342 not included in this pull request? I am wondering why? Why are they jumped?

https://github.com/helium/denylist/issues/7533  为什么我提供了证据还是将我的设备拉黑了@6CornerDefender

@6CornerDefender 请回复我的问题，我已经提供了准确的定位及门牌号及周围环境图

https://github.com/helium/denylist/issues/7533 爲什麼我提供了證據還是將我的設備拉黑了@6CornerDefender

@6CornerDefender 請回復我的問題，我已經提供了準確的定位及門牌號及周圍環境圖

@berserkrrr 我已經回覆了證據爲何還要拉黑我的熱點

#7497 is gone. Will it be included in the denylist?

So can be exchanged to one of those #7327 - #7335 - #7340 - #7341 - #7342
Or am i wrong?

@6CornerDefender @berserkrrr Excuse me, are you pulling for no reason? I have provided my evidence, down to the house number and the deployment location, this cannot be faked, why not reply to me

Still #7327 - #7335 - #7340 - #7341 - #7342 not included in this pull request? I am wondering why? Why are they jumped? They have been added around one month ago, making tons of illegal HNT. Also they are not reported again, because they already have been reported. So why are they favored and not denylisted?

If those are not included in the next pull request, i would say its suspiscious. And looks like those heltec HS are favored by the people who maintain the denylist.

@6CornerDefender  All these legal hotspots have been blacklisted, Can you review carefully? #6527

@6CornerDefender #7522 The issue has been closed by the author. The evidence for the problem description is seriously insufficient. The hotspots in the list have been offline for a long time. Observing recent data shows that two hotspots are witnessing each other, and the rssi and Snr indicators between the two hotspots have not been found.  abnormal.


Setting to set maximum number of reference tokens created?

Trying to find a way to limit the number of active reference tokens per user.

Can you explain a little more?

For a particular user I am trying to have not more than X active reference tokens issued. If a user request a new token, one of the existing tokens is disabled/expired/deleted. The attempt is to limit the devices which can have simultaneous access. Thanks!

We'll look into it. thanks!

I'd suggest this be done in a custom persisted grant store.


Add loadScript

What does this change?

add loadScript, ported/refactored from frontend
removes stub files

Why?

want to use it outside frontend
will remove in frontend once this is published

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


How Porter is used in Azure TRE link not working

Under the section Using the Azure TRE>Customer Template>How to add custom templates:
To add your custom templates follow the next steps:
point number 3
learn more about how it is used in AzureTRE here.
The link returns a 404 File not found error.
How to fix
Replace with the correct link.

Further to the above comment, there are two other links on the same page return 404 error.
Same section, point number 4
Define resource template in the API - follow this readme to register your template.
Use the AzureTRE UI to deploy your resources


APPS-628 Document best practices, limitations and examples of WDL-based global workflows.

Still need to add examples for updating WF metadata after publishing WF (see TODOs), but I think this can be reviewed again re: the other changes. @commandlinegirl  @emiloslavsky

This is ready for review. @emiloslavsky @commandlinegirl @jdidion


Error while running main.py

python main.py
/home/sarthak/anaconda3/lib/python3.7/site-packages/tensorflow/python/framework/dtypes.py:516: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_qint8 = np.dtype([("qint8", np.int8, 1)])
/home/sarthak/anaconda3/lib/python3.7/site-packages/tensorflow/python/framework/dtypes.py:517: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_quint8 = np.dtype([("quint8", np.uint8, 1)])
/home/sarthak/anaconda3/lib/python3.7/site-packages/tensorflow/python/framework/dtypes.py:518: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_qint16 = np.dtype([("qint16", np.int16, 1)])
/home/sarthak/anaconda3/lib/python3.7/site-packages/tensorflow/python/framework/dtypes.py:519: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_quint16 = np.dtype([("quint16", np.uint16, 1)])
/home/sarthak/anaconda3/lib/python3.7/site-packages/tensorflow/python/framework/dtypes.py:520: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_qint32 = np.dtype([("qint32", np.int32, 1)])
/home/sarthak/anaconda3/lib/python3.7/site-packages/tensorflow/python/framework/dtypes.py:525: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
np_resource = np.dtype([("resource", np.ubyte, 1)])
/home/sarthak/anaconda3/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:541: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_qint8 = np.dtype([("qint8", np.int8, 1)])
/home/sarthak/anaconda3/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:542: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_quint8 = np.dtype([("quint8", np.uint8, 1)])
/home/sarthak/anaconda3/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:543: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_qint16 = np.dtype([("qint16", np.int16, 1)])
/home/sarthak/anaconda3/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:544: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_quint16 = np.dtype([("quint16", np.uint16, 1)])
/home/sarthak/anaconda3/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:545: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_qint32 = np.dtype([("qint32", np.int32, 1)])
/home/sarthak/anaconda3/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:550: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
np_resource = np.dtype([("resource", np.ubyte, 1)])
Traceback (most recent call last):
File "main.py", line 6, in
from utils import choose_run_mode, load_pretrain_model, set_video_writer
File "/home/sarthak/Major Project/Online-Realtime-Action-Recognition-based-on-OpenPose/utils.py", line 6, in
from Pose.pose_visualizer import TfPoseVisualizer
File "/home/sarthak/Major Project/Online-Realtime-Action-Recognition-based-on-OpenPose/Pose/pose_visualizer.py", line 6, in
from .pose_estimator import estimate
File "/home/sarthak/Major Project/Online-Realtime-Action-Recognition-based-on-OpenPose/Pose/pose_estimator.py", line 21, in
'coord1', 'coord2', 'score1', 'score2'], verbose=False)
TypeError: namedtuple() got an unexpected keyword argument 'verbose'

"/home/sarthak/Major Project/Online-Realtime-Action-Recognition-based-on-OpenPose/Pose/pose_estimator.py"
Remove the verbose=False parameter in this code. In Python 3.7 this argument was removed

directly remove the verbose and it can work propely


AVAudioPlayer

This converts the alert sound player to use AVAudioPlayer instead of AudioServicesPlaySystemSound().
This change is needed to support volume adjustment in the future, and I'm hoping it also fixes the no-audio issue some people are seeing (#61).

@robinstewart is this change good to be merged?

Yes, this is a simple change and is ready to go.

Merged!


Instructions on how to install Azure PowerShell module (ASM) are outdated and misleading

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

ID: 7aa6c7f7-a61f-5a06-b964-888ee03eb86b
Version Independent ID: a166875b-49f9-ea6d-d7ad-123189ef7b5c
Content: Install and configure the Azure PowerShell Service Management module
Content Source: docs-conceptual/azuresmps-4.0.0/install-azure-ps.md
Service: azure-asm
GitHub Login: @mikefrobbins
Microsoft Alias: mirobb

@omunoz666
I've delegated this to @mikefrobbins, a content author, to review and share their valuable insights.

Switching to tracking this issue via #2594 at @omunoz666's request.


Incompatibility with talib (expected numpy.ndarray, got DataFrame)

Describe bug
talib cannot accept dataframe as input:
Traceback (most recent call last):
  File "/data/1.py", line 7, in <module>
    df['SMA_5'] = ta.SMA(df['Close'], timeperiod=5)
  File "/data/miniconda3/envs/a/lib/python3.10/site-packages/talib/__init__.py", line 64, in wrapper
    result = func(*_args, **_kwds)
TypeError: Argument 'real' has incorrect type (expected numpy.ndarray, got DataFrame)

It works for yf 0.2.44, but not work for 0.2.46+.
Simple code that reproduces your problem
import talib as ta
import yfinance as yf

df = yf.download("MSFT", period='5d')
df['SMA'] = ta.SMA(df['Close'], timeperiod=3)

Debug log
N/A
Bad data proof
No response
yfinance version
0.2.48
Python version
3.10.13
Operating system
Ubuntu 22.04

It works for yf 0.2.44, but not work for 0.2.46+.

Double-check this. I get opposite.

Cause:
After issues #2068 #2100 , yf.download("MSFT") returns by default now a DataFrame with an extra level Ticker
Fix:
Either use
df = yf.download("MSFT", period='5d', multi_level_index=False)

or
df = yf.Ticker("MSFT").history()


Fix Reserved Identifiers

Closes #258

Commit 08a371c conflicts with PR #305. Prefer cherry-picking all commits afterwards and incorporating changes to Signal class directly into #305.

Ok, I did a bit of cleanup. I split off the Delegate change into a new temporary branch removeUnderscoreDelegate. I then rebased this branch to remove that commit so there would be no conflicts with #305. As part of the rebase, I also reordered some of the commits, so the fixup commits came right after the commits they were fixing, and then squashed them together. The result should be a bit cleaner, in that every commit should compile on its own now, and there's no interleaving of changes.
If these changes looks good, I'd like to merge this now.


Set up visual regression tests

Tasks

[ ] Research Storybook-compatible visual regression test frameworks (maybe lost pixel?)
[ ] Select a reasonable one and configure it with this repo
[ ] Write a visual regression test and let us know how it goes.

Option: Chromatic
This is the Storybook-recommended tool and it seems quite promising.
Demo video
I signed up with an email address, as opposed to authenticating via GitHub, which seems to disable a couple of the more advanced (but still free) features:

UI Review: Provides a visual overview of all observable UI changes that team members can review and comment/approve/deny.
TurboSnap: Helps to minimize the number of snapshots generated for each build (free plan provides 5,000 snapshots/month) by focusing only on components that have code changes.

Pros

Chromatic has been pretty easy to setup
Has strong documentation that covers a variety of use cases. Ex: GitHub integration.
Solid free tier - 5K snapshots (visual comparisons) per month
Looks like new users get an extra 75K snapshots included in the first month's limit to provide leeway in getting things setup.

Needs clarification

Would 5K snapshots/month be sufficient for a Bureau-wide tool that multiple teams could be modifying at once?  Seems like 1 story = 1 snapshot, so we currently generate 52 snapshots per run.
Free plan only supports Chrome snapshots.  Do we need IE or Firefox support?
Do all collaborators have Github accounts?

How easy will it be add non-devs as reviewers?

Wise for me to setup Github integration for this tool or should the integration be owned by a CFPB dev?
Should we just pick a tool and run with it until it becomes clear that it's not working for the team?  Or should we continue to evaluate alternatives, poll the team, and move forward with our consensus favorite?

Next Steps

Use Github auth to setup Chromatic project to maximize feature availability and github integration.
Test github integration
Evaluate workflow for identifying and approving/denying UI changes
Ensure all collaborators can access the Chromatic project

Should we just pick a tool and run with it

Yes, go for it!

Would 5K snapshots/month be sufficient for a Bureau-wide tool that multiple teams could be modifying at once?

If we hit this limit and enjoy the tool we can enlist procurement guru @harrisj for advice.

The Guardian has a neat trick to save on Chromatic runs. From their readme:

To run Chromatic (for visual regression testing) on a PR, add the run_chromatic label to the PR in Github. The Chromatic tests are required to pass by CI, so this will need to be done at least once before a PR is merged. We recommend you only add the label once you are happy with any changes made and have checked for visual regressions manually first.


398 quotation marks

fix #398
Two commits:

Replace the »...« by "..."
Replace the "..." by “...” everywhere

Options:

We keep the ASCII quotes "..."

Pros

Easier to type using any keymap
Probably what is already used when pasting text from other sources

Cons

Not typographically accurate
There is already a mix of "..." and “...” in this documentation

We switch everything to “...“

Pros

Typographically accurate
Align the entire documentation to one quotation standard

Cons

Bigger diff (not really a problem)
Hard to maintain (people will probably have the reflex of using "..." in the future).

I did it in 2 different commits, which will be useful if we go with the 1st option (rollback the 2nd).
Let's discuss!

Authors of the guidelines write in markdown and markdown is code. We very much need to stick to the obvious choice of "...". What I could imagine is some post processing that rewrites quotes before publication.

I personally do not have an opinion. Or if one, stick to "...". This task started as getting rid of the German-styled quotes and after a comment from @ePaul , went to apply correct typography.
I would vote for "..." because it is easier to maintain. But then, we need to stick to this styling and not have an hybrid, as it is at the moment.

:+1:

@ddufourboivin-zalando Thanks for the work. However, most of the "..." are in reality for emphasizing terms not for quoting. I would prefer these to use _..._ instead (or *...* in strong cases). When it comes to quoting, it should always use _"..."_ to keep it simple in code, even if I like the German quotes.

@tkrop : I will rework this branch to use "..." everywhere (basically reversing the commit where I change them to “...”). Let's align on the character to use for quoting first and discuss the use of quotation vs emphasis in a next discussion (my proposal).

Next question ...
Which brings us to the next consideration.
Should we align the existing “...” to the decided "..." or leave them be?
Please vote by 👍 or 👎 this comment (smiley in up-right corner).

@whiskeysierra

Authors of the guidelines write in markdown and markdown is code. We very much need to stick to the obvious choice of "...".

I don't really follow that argumentation. Is it »Code should only ever use ASCII«?

I don't really follow that argument. Is it »Code should only ever use ASCII«?
(I'm fine with using ASCII codes, I just want to understand the argument here.)

Yes.

@ePaul not entirely. It is although that this is a English document, where we should use punctuation marks (here quotation marks) as usually used in this language :).

:+1:

Since it is a document, we should be using the right quotation marks “...”, but I also think that it would be difficult to maintain consistency because we'll forever have the reflex of using ASCII/coding quotations "..." when we right. So I am completely good with this decision of using the ASCII ones.
There will be another discussion (initiated by @tkrop ) about using proper emphasis instead of "quotations" in this documentation where it applies (most of the actual quoted text should indeed be emphasised instead). But this is out of scope here.
I will be pushing a last time with the decided "quotation marks" and then we can close this topic.

Ready for re-review.
@whiskeysierra @tkrop @maxim-tschumak @epaul


janus gateway streaming not work

Hi
I am using Janus gateway to streaming RTP to RTCView , but it only works in rn-less-40, like this sample: https://github.com/inspiraluna/JanusReact, but when I switch the newest version (1.58.3), it doesn't work and doesn't show any error message

Because that repo you posted is use RN-less 40.
Master only support higher version of RN.
So all you have to do is upgrade RN on that repo, or downgrade master to support RN less 40 ( maybe rebase or some cherrypicks )
Since there is nothing we can do on this libs.
I'm going to close this now,


Fix the drop down box

The drop-down box has other options, but they aren't updating when selected

Is this the dropdown box in the alert dialog? If so the new alert dialog version should have fixed this.

fixed with new alert dialog

opened accidentally


[detect] Mock interface difficult to use and error prone

Currently, to add a mock, you have to do some odd things:
mbedls -m "C006:VLBUNO51"

mbedls --mock="C006:VLBUNO51"

both add a mock for the target id C006 and the platform name VLBUNO51.
While that my seem reasonable, it can also be done by:
mbedls --mock="+C006:VLBUNO51"

Which is a bit complicated.
Removing mocks are where things get nasty:
mbedls -m "-C006"

is in theory part of the syntax, but -C006 is parsed as a flag.
mbedls --mock="-C006"

is the only working way to actually remove a mock.
The readme also gives the impression that:
mbedls --mock="!C006"

should work, but it does not. Instead, it gives you a lovely trackback:
Traceback (most recent call last):
  File "/usr/bin/mbedls", line 11, in <module>
    load_entry_point('mbed-ls==1.3.2', 'console_scripts', 'mbedls')()
  File "/usr/lib/python3.6/site-packages/mbed_lstools/main.py", line 258, in mbedls_main
    ret_code = args.command(mbeds, args)
  File "/usr/lib/python3.6/site-packages/mbed_lstools/main.py", line 127, in mock_platform
    mbeds.mock_manufacture_id(mid, 'dummy', oper=oper)
  File "/usr/lib/python3.6/site-packages/mbed_lstools/lstools_base.py", line 231, in mock_manufacture_id
    raise ValueError("oper can only be [+-]")
ValueError: oper can only be [+-]

A much better interface would be:
mbedls add-mock C006 VLBUNO51

for add and:
mbedls rm-mock C006

or
mbedls remove-mock C006

for mock removal.
@yennster @bridadan Does this cover our offline discussion?

Thank you for raising this issue. Please note we have updated our policies and
now only defects should be raised directly in GitHub. Going forward questions and
enhancements will be considered in our forums, https://forums.mbed.com/ . If this
issue is still relevant please re-raise it there.
This GitHub issue will now be closed.


"Input string is not in a correct format"

If the value of a "dictionary type Transaction Data" contain a comma, It will response "Input string is not in a correct format" error when sending transaction.
For example:
Data = new Dictionary<string, string>()
 {
    { "owner", "tester112345" },
    { "sym", "4,HVT" }
 }

If the value of "sym" is "4,HVT", the transaction can't be send out, but "4.HVT" can.

Can you post a more complete example? I tryed to send on a memo for a normal transfer and it works.

The comma is not in the memo, It's in the action data.
You can try this:
		public async Task CreateTransactionDict()
		{
			bool success = false;
			try
			{
				var result = await Eos.CreateTransaction(new Transaction()
				{
					Actions = new List<Api.v1.Action>()
					 {
						 new Api.v1.Action()
						 {
							 Account = "zkstokensr4u",
							 Authorization = new List<PermissionLevel>()
							 {
								 new PermissionLevel() {Actor = "youreosaccount", Permission = "active" }
							 },
							 Name = "transfer",
							 Data = new Dictionary<string, string>()
							 {
								{ "owner", "youreosaccount" },
								{ "sym", "0,ZKS" }
							 }
						 }
					 }
				});
				success = true;
			}
			catch (Exception ex)
			{
				Console.WriteLine(JsonConvert.SerializeObject(ex));
			}

			Assert.IsTrue(success);
		}

added a fix https://github.com/GetScatter/eos-sharp/commit/08f78b69f4fcef122a69c6c47e4513cd0ee62a38


incomplete documentation for "Pushing JSON files to a remote WireMock instance"

At https://github.com/tomakehurst/wiremock/blob/master/docs-v2/_docs/running-standalone.md#pushing-json-files-to-a-remote-wiremock-instance it is mentioned it is possible to push JSON files to a remote WireMock instance.
The content of the paragraph starts with "You can push a collection of mappings to a remote" and then it stops in the middle of the sentence.

LOL, I obviously got distracted! Will try and put something useful in there next time I publish the docs.

Could you please share the solution without being elaborate because the documentation will follow?
I am hoping it is possible to HTTP POST the mappings to a remote instance via the admin API.
This one gets close: http://wiremock.org/docs/api/#tag/Stub-Mappings/paths/~1__admin~1mappings/post
But I would like to upload an entire set at once.

Basically, you need to set up a WireMock instance in Java, then call a specific method:
WireMock wm = new WireMock("my.host", 8080);
wm.loadMappingsFrom("/my/root/directory"); // Root dir contains mappings and __files

I was searching for the same and came across this post. Seeing this issue is still open, @ogarling were you able to successfully post the mappings using the above method? Also, could you please share how you have connected to the remote instance? Many thanks.


Support multiple responses based on input match

This patch adds support for different responses based on the input matches. I tried to enable this capability with as small a change as possible (i.e., I didn't want to restructure to much). This has a couple of consequences:

the configuration format is a little clunky as you specify the method name for each rule even though it is the same method with just different input/output.
the patch essentially works by ensuring even though there are multiple handlers generated only a single response is given. to coordinate this some state is maintained. This may not be the cleanest but it seemed to be the most direct approach without significant restructuring.

Updating with additional sparse compare capability


Update actions to mirror changes in g2-sdk-go

coverage and linting

Pull requests:

https://github.com/senzing-garage/template-go/pull/147


Having issue implementing the component

Followed the instructions on npm link,
https://www.npmjs.com/package/react-horizontal-stacked-bar-chart
getting this error:

Hi @Unicity-Pimentel
unfortunately this component doesn't work in React Native because it uses svg tags.
I'm working on a specific component for React Native, but it is not yet finished.


Add label prop to ColumnsLayoutPreview

Description
Add label prop to ColumnsLayoutPreview. For additional customization we add this prop, so in the preview of the FinalFormLayoutSelect there can icons or additional text be added. Fallback is furthermore the width number.
Screenshots/screencasts

@johnnyomair to increase the size of the preview columns, i think we need to make this line more dynamic, do you have an idea, how to do that?
https://github.com/vivid-planet/comet/blob/8b63d6d0546bb7b902d8a07f3594a586d1780d8a/packages/admin/blocks-admin/src/blocks/factories/columnsBlock/FinalFormLayoutSelect.tsx#L118


"Poieni-Solca" is listed in the drop-down menu

Description:
"Poieni-Solca" is listed in the drop-down menu alongside its corresponding county according to the law
Precondition:
The website is up an running.

Step 1
Type "Poieni-Solca" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Poieni-Solca, SUCEAVA".


libspdm_x509_verify_cert_chain fails with same x509 certificate data with openssl 1.1.1q but pass with openssl 1.1.1l

LIBSPDM 2.0.0 tag uses openssl 1.1.1l is able to validate x509 certificate where as LIBSPDM 2.3.0 tag which uses openssl 1.1.1q is not able to validate same x509 certifcate.
Is this a known issue?

It is not a known issue. What exactly is failing?

In this file libspdm/os_stub/openssllib/openssl/crypto/x509/x509_vfy.c file,
static int build_chain(X509_STORE_CTX *ctx);
static int check_chain_extensions(X509_STORE_CTX *ctx);
The above two function returns 0, thus it doesn't proceed to validate chain.

Looks like you might have hit
https://www.openssl.org/news/vulnerabilities.html#CVE-2021-23841
https://git.openssl.org/gitweb/?p=openssl.git;a=commitdiff;h=2a40b7bc7b94dd7de897a74571e7024f0cf0d63b

How to fix this then?

Actually

Disallow explicit curve parameters in verifications chains when X509_V_FLAG_X509_STRICT is used.

was introduced in 1.1.1h, so if that was the issue it should have been present in 1.1.1l. And it says that X509_V_FLAG_X509_STRICT is disabled by default, and I don't think libspdm enables it. (Maybe it should.)

How to fix this then?

Figure out exactly where in check_chain_extensions the failure is.

I found the place where it fails, even after giving the correct buffers in openssllib functions.
static int build_chain(X509_STORE_CTX *ctx)
This line, returns ok = 0 because of that it fails, ok = (depth < num) ? 0 : get_issuer(&xtmp, ctx, x);

 static int check_chain_extensions(X509_STORE_CTX *ctx);
 x = sk_X509_value(ctx->chain, i);
         if (!(ctx->param->flags & X509_V_FLAG_IGNORE_CRITICAL)
             && (x->ex_flags & EXFLAG_CRITICAL)) {
             if (!verify_cb_cert(ctx, x, i,
                                 X509_V_ERR_UNHANDLED_CRITICAL_EXTENSION))
             {
                 return 0;
             }
         }

verify_cb_cert fails in check_chain_extensions function.
But when I updated SPDM responder validator tool to use LIBSPDM 2.0.0 code base, libspdm_x509_verify_cert_chain function passed since it uses openssl 1.1.1l. So, it looks like even afer providing the same certificate data, SPDM responder validator tool fails when it uses LIBSPDM latest code base which uses openssl 1.1.1q.

It seems check is done by openssl. Not sure what libspdm needs to do, unless it is configuration.
Need root-cause at first.

Do you have test certificate chain that has been validated with SPDM responder validator tool which uses openssl 1.1.1q?

Yes, please refer to https://github.com/DMTF/libspdm/tree/main/unit_test/sample_key.

Test certificates are not present in this format right?

Correct, for SPDM you need to append that information.

1/23/2023 - in unit test, we use
uint32_t m_libspdm_use_asym_algo =
    SPDM_ALGORITHMS_BASE_ASYM_ALGO_TPM_ALG_ECDSA_ECC_NIST_P256;
uint16_t m_libspdm_use_req_asym_algo = SPDM_ALGORITHMS_BASE_ASYM_ALGO_TPM_ALG_RSASSA_2048;

Need try ECDSA_ECC_NIST_P384

1/31/2023 update
Recently I tried most algorithms that libspdm supports but I can not reproduce this issue. Latest libspdm with openssl 1.1.1q + test cert chains in UT folder works for me.
BTW there are some issues in libspdm unit test code blocking the change of default asym algo uint32_t m_libspdm_use_asym_algo =     SPDM_ALGORITHMS_BASE_ASYM_ALGO_TPM_ALG_ECDSA_ECC_NIST_P256; for testing.
I will file another ticket. (confirmed the issue is just about UT code itself).


Video stops playing after 30 or 40 min

Without any fixed pattern, randomly but mostly after 30 or 40 min playing, the video stops.
The sound will go on for a few more seconds and then it will shut down.
Here is my log file: https://paste.kodi.tv/bigomumopa
Can you help me please? All of my familly suffer)

Sorry, How can i make that kind of log? BTW my TV android stick shuts down when this issue happens? so i need to reboot it

Shuts down what? Device, kodi, or elementum? Any chance you run out of memory?

The device shuts down. I'm not sure, but i guess it's not memory, because all other apps (even hard ones runs for infinite time with no problems). My device have 4 GB RAM. When I've started debugging mod to make a log, there was a onscreen information, when my issue happens the picture freezes for some time, before the device shuts down and the memory was not full i guess. At least according to that onscreen info

@Insomniac412 try to check your CPU temperature during playback. install some android app that shows temperature, for  example.

Thank you very much, reflashing helped me. In the new firmware version they added the function “CPU and GPU frequency blocking at the maximum value”, and when I turned it on as root, the problem went away. Your addon is great, I will definitely donate


Usage of Cloud Controller V3 APIs

Hey there,
I'm curious what your plans are for adoption of Cloud Foundry's V3 APIs for apps. The CAPI team is beginning to work on deprecation plans for the V2 app APIs

@zrob We don't have any concrete plans as of yet - we are tracking V3 in this issue - https://github.com/cloudfoundry-incubator/stratos/issues/1543.
I'd be interested in the deprecation plans, so we can plan when we need to have V3 support.

@zrob When handling apps, and other entities, we make heavy use of relationships. For instance when we list apps we fetch each app's space, the space's organisation and the app's routes all in one call. When we then drill down into an app we validate the app has routes, space, stack, service binding and the route has domain and space has organisation. If any are missing we then use the _url from the app entity to fetch them. When looking at the v3 apps doc (http://v3-apidocs.cloudfoundry.org/version/3.56.0/index.html#the-app-object) it looks like some of the app relations are available as links. Is there a way to fetch them inline? Also are there any plans to expand the app relations to cover all v2 app relations?
It would be quite tricky having to support both v3 and v2 api's together, so would also be interested in the timeline for expanding v3 to cover all v2 resources

Hey @richard-cox
TLDR; we are looking at ways to have similar functionality to inline-relations-depth, but it will be somewhat more restrictive
We ended up with lots of operating experience in environments that had clients using inline-relations-depth.  That feature ended up causing a very large issue with API performance in most cases, especially when using a depth of 2.    A value of 1 has been less bad, but has still shown some performance issues.  Part of the problem is that it was a generic "get all related things" rather than allowing clients to target the relations they need.
So that being said, the feature still has some obvious value in limiting the number of requests necessary.  Its worth mentioning that we do have a client that is fully V3 for apps and are finding their performance to be better by just paging through each resource they want and using the relationship data to stitch them together.  But that still has some complication, so we have recently introduced what we hope can replace inline-relations-depth in V3.  This is the include parameter here.  It currently only allows including spaces for apps.  We're trying to get some validation from clients before selectively expanding it to other resources.  There will be some limits to the resources we end up supporting as an include, but organization seems like an obvious inclusion.  Routes are a possibility.  The docs aren't super clear on how to use the include property right now, but it is heavily inspired by the JSON API spec for including resources

@zrob Sorry for the incredibly slow response. We've been beavering away getting a release out.
I've created a doc that covers the parts of v2 we use and how they're utilised in Stratos, along with some examples of the requests we make as the user navigates around. I've tried to include a succinct summary too. Myself and @nwmac will both be at Basel in October, heard through the grapevine you're also there? Would it be possible to grab you for an hour and run through our v2 usage, state of v3, deprecation timelines etc? We'd also be interested to understand the use cases that the client on v3 is running with, particularly if they've already solved some of our v2-->v3 challenges.

Hi @nwmac @richard-cox
I am at summit and would love to catch up.  Not sure how best to coordinate. We can try with comments or on slack you can find me ‘@zach’

@zrob great - I’ve pinged you on slack.


flow to image

I run your code and trying to generate an output image. However, my result is very bad. I guess that the conversion from flow tensor to an image is wrong in my implementation, would you mind showing your code or some samples?

I already fix it, thank you for opening code

Hi,
I'm glad that you solved.
Anyway, I used this code to visualize the optical flow: https://github.com/tomrunia/OpticalFlow_Visualization

Thank you for your reply. As your can see below, I obtained a much clear results than other methods before. However, how to incorporate occ_mask and our predicted flow?

I'm not the author of the paper, i only made the pytorch porting.
For any questions related to the approach, you should ask the authors of the paper, the official implementation is here: https://github.com/microsoft/MaskFlownet

@cattaneod I'm sorry to ask you for help. I'm going to build a training pipeline based on your code.
I got a problem when computing the EpeLoss. When executing on this line,
Ideally, it should return a tensor includes a single element.
Actually, it returns a tensor includes four elements on each GPU(my batch_size is 4). So my loss couldn't be backward.
tensor([1.8977, 2.0039, 1.7233, 1.5050], device='cuda:0',
       grad_fn=<AddBackward0>)

how about your training process?
if possible,  any cues would be highly appreciated.

Hi,
the function return the EPE for every sample in the batch. If you want to backward it than you can just take the mean:
loss = EpeLossWithMask()(up_flow, label, mask).mean() loss.backward()
If you want to use the Multi Scale loss, then similarly:
loss = MultiscaleEpe([64, 32, 16, 8, 4], [.005, .01, .02, .08, .32], 'upsampling', eps = 1e-8)(up_flow, label, mask).mean() loss.backward()

hi, yesterday, I already did as you said, I got an error like this
File "/home/fei/maskflownet/models/model.py", line 84, in step
    loss.backward()
  File "/home/fei/anaconda3/envs/flow/lib/python3.6/site-packages/torch/tensor.py", line 198, in backward
    torch.autograd.backward(self, gradient, retain_graph, create_graph)
  File "/home/fei/anaconda3/envs/flow/lib/python3.6/site-packages/torch/autograd/__init__.py", line 100, in backward
    allow_unreachable=True)  # allow_unreachable flag
RuntimeError: function CorrelationFunctionBackward returned an incorrect number of gradients (expected 8, got 2)

I haven't found any useful info on google. If possible, Could you show the key points of your training pipeline?

Hi,
for some reason, the backward of the correlation is broken.
In my training i used a different correlation implementation, but then the results with the original weights are different, I will fix it today or tomorrow.
Thank you for pointing this out.

what's this problem of your correlation package? could you share more details about this? I'd like to fix it. @cattaneod

Ok, i solved the backward problem of the correlation package i was using, just replace line 52 of model/correlation_package/correlation.py with:
return grad_input1, grad_input2, None, None, None, None, None, None


WIP: Pluggable renderers

The Problem
We have a lot of unnecessary dependencies within the library.
For example, ol.layer.Tile depends on both ol.renderer.canvas.TileLayer and ol.renderer.webgl.TileLayer.  Same situation with the rest of the layers.
And ol.Map depends on both ol.renderer.canvas.Map and ol.renderer.webgl.Map.
The map also depends on ol.control.Attribution, ol.control.Zoom, and ol.control.Rotate.  And a whole lot of interactions.
The Compiler and build flags can ameliorate some of this, but we can improve the situation.
Proposed Solution
We can treat map and layer renderers as plugins.  To maintain backwards compatibility (and for convenience), ol.Map can register both map renderers and all the layer renderers.
In addition, ol.Map can continue to use the defaults() functions for controls and interactions (and these will retain their dependencies).
Then, for people who don't need all the extra stuff (and don't mind a bit of inconvenience), we can create a ol.pluggable.Map (and ol.Map will extend this).  People using ol.pluggable.Map will register just the map renderer and layer renderers that they need.  And they can provide controls and interactions directly (by default, ol.pluggable.Map comes with no controls or interactions).  In this way, all dependencies will be explicit.  No build flags will be necessary and other minifiers can be used.

This works, but I need to create some olx types to keep the Compiler from renaming the static methods.

interesting @tschaub thanks for taking this forward

Presumably, this replaces #6303? (And fixes much of #6311.)
A dramatic reduction in the number of unwanted dependencies, and way more flexible than the current setup. Once this passes Travis, I'll pull in the branch and try some tests, and also look at the new dependency tree.
I'm not sure about the PluggableMap name, which doesn't seem very descriptive to me, as the only 'pluggables' are the renderers. It's essentially a map without any defaults, so perhaps BaseMap (or 'raw' or 'bare' or something) would be better?
Something that's missing here is documentation, particularly for those wanting to use PluggableMap. The custom build tutorial has always stated that you list the classes/methods you use in your app in your build config, but the renderers aren't referred to in apps, and do not appear in the docs anywhere, so people have to be told what to include. I notice that api (stable?) has been added to the renderer constructors, but they have no docs text. ISTM there should be some examples of using PluggableMap too, though that could be in a separate PR.

also, it would be good if View could be included in this, as the fit method is currently 'requiring' a lot of vector files, which are only needed if you use a geometry with fit(), probably only used by a small %age of maps. view.js and map.js can remain as is for bc. View is only included in map.js to create a default view if none is provided in the constructor options, but I can use setView() at some later time, so is this actually necessary? This view creation can be omitted from PluggableMap, and if a View is needed at construction time, then those using PluggableMap can be told they have to supply one in the options.
If we then create viewwithoutgeoms.js, and put fit() (or at least the geom bit) in a separate file, then PluggableMap users can use the no-geoms view, and add fit/geoms separately if they wish. This would then enable people to have maps without vectors if they want.

For people who want to explicitly depend on just what they use, there is now a PluggableMap constructor.  To use it, you first need to register the map and layer renderers that you use.  It also doesn't come with any interactions or controls, so you add those explicitly.  For example:
import Map from 'ol/pluggablemap';
import MapRenderer from 'ol/renderer/canvas/map';
import PluginType from 'ol/plugintype';
import TileLayer from 'ol/layer/tile';
import TileLayerRenderer from 'ol/renderer/canvas/tilelayer';
import View from 'ol/view';
import XYZSource from 'ol/source/xyz';
import control from 'ol/control';
import interaction from 'ol/interaction';
import plugins from 'ol/plugins';

plugins.register(PluginType.MAP_RENDERER, MapRenderer);
plugins.register(PluginType.LAYER_RENDERER, TileLayerRenderer);

new Map({
  target: 'map-container',
  interactions: interaction.defaults(),
  controls: control.defaults(),
  renderer: MapRenderer,
  layers: [
    new TileLayer({
      source: new XYZSource({
        url: 'http://tile.stamen.com/terrain/{z}/{x}/{y}.jpg'
      })
    })
  ],
  view: new View({
    center: [0, 0],
    zoom: 2
  })
});

For a bit of convenience, there is now a CanvasMap constructor.  So to get just the canvas renderer (this still depends on all canvas layer renderer types and you get the default controls and interactions), you can do this:
import Map from 'ol/canvasmap';
import View from 'ol/view';
import Tile from 'ol/layer/tile';
import XYZ from 'ol/source/xyz';

new Map({
  target: 'map-container',
  layers: [
    new Tile({
      source: new XYZ({
        url: 'http://tile.stamen.com/terrain/{z}/{x}/{y}.jpg'
      })
    })
  ],
  view: new View({
    center: [0, 0],
    zoom: 2
  })
});

Both examples above will be possible with the next release.

@tschaub Hi, I don't find ol/plugins nor ol/PluginType in OL6. Can you update your code above for OL6 please ? Thanks.

Have a look at the ol/Map code, there is now a createRenderer method instead.
https://github.com/openlayers/openlayers/blob/main/src/ol/Map.js

@MoonE Thanks for the hint but if I load ol/Map instead of ol/PluggableMap, how do I avoid module:ol/interaction~defaults in final bundle?

@nboisteault You can continue using ol/PluggableMap. Just make sure you implement the createRenderer method, e.g. like in https://github.com/openlayers/openlayers/blob/2cac6c4/src/ol/Map.js#L75-L77

Thanks @ahocevar. What I did if it can help someone :
import PluggableMap from 'ol/PluggableMap';
import CompositeMapRenderer from 'ol/renderer/Composite';
import { assign } from 'ol/obj';

class olMap extends PluggableMap {
    /**
     * @param {import("./PluggableMap.js").MapOptions} options Map options.
     */
    constructor(options) {
        options = assign({}, options);

        super(options);
    }

    createRenderer() {
        return new CompositeMapRenderer(this);
    }
}

const myMap = new olMap({...});


Added Pokey Library

@ccallebs - adding this into our community contributed library list. Thanks for sharing!!

Awesome! :+1:


Fix staticfiles clobber

This is simply the static file copy system from the latest jsdoc but converted to ES5.

Forgot to reference issue #15 which I created for this pull request.

Thanks @brendanmig!


Fix git blame on ibm-1047 files

resolves https://github.com/ZOSOpenTools/gitport/issues/76

Tried update and still get error

obviously using zigi to do this

more


wrong select in Persian text

hi!
when I selected a Persian text like first picture and when I want edit it other part of text got change, like second picture, I select and of my text but first part got change
thanks :)

/cc @as-cii

Forgive my cultural ignorance, but is Persian a right-to-left language?

yes :)
[image: Restro Co] http://restro.ir/
Mahdi Majidzadeh / Web developer
<EMAIL_ADDRESS>/<PHONE_NUMBER>0
Restro Co
http://restro.ir
[image: Facebook]  https://facebook.com/mahdi.majidzadeh[image: Twitter]
https://twitter.com/mahdimajidzadeh[image: Google Plus]
https://plus.google.com/mahdimajidzadeh[image: Linkedin]
https://inkedin.com/mahdimajidzadeh[image: Instagram]
http://instagram.com/mahdimajidzadeh
On Fri, Jan 8, 2016 at 2:59 AM, Lee Dohm<EMAIL_ADDRESS>wrote:

Forgive my cultural ignorance, but is Persian a right-to-left language?
—
Reply to this email directly or view it on GitHub
https://github.com/atom/atom/issues/10294#issuecomment-169839807.

There is also a weird problem with mixed rtl and ltr.

If the string begins with a ltr char, you can't just select part of the rtl part. But if it begins with a rtl char that doesn't happen.
Note: in the first one the period is before the rtl part starts.

This is a duplicate of #10729

Or rather #10729 is a duplicate of this (higher number means later created issue)

It was marked as duplicate two days ago.

the issue is not been solved

Kate, Komodo and sublime don't have such problem. What's wrong in Atom's code?

@metemaddar I understand this is frustrating. However, please keep the Atom Code of Conduct in mind and be constructive with your comments.

I can't even mark my text when it's Persian .

After more than 2 years from reporting this issue, the problem still exists.  Please be informed that Ordu and Arabic languages uses Persian font too. It means Atom is unable to cover the language of over 2,000,000,000 people of the world!
Do something for it guys. We used to Atom and wouldn't like to see problems in it.
Thanks!

please fix this

Hi Atom Contributers
When this problem will be fixed?
After more than 4 years from reporting this issue, the problem still exists.
i even cannot select the whole persian text in atom , when i want to select it , it skips to one character before and after text
are you don't want to fix it?

You can contribute to https://github.com/atom/atom/pull/21018 to fix this problem.


Better performance when exploring big files

No matter the size of the file, if it is clearly a binary file, the warning "The file is not displayed in the editor because it is binary" should be shown almost instantly. Currently, the time for this message to show up seems to depend on the size of the file and the editor hangs while this is being determined, making accidental clicks costly (timewise).
I sometimes work with multi-100 MB text fies and want to see what they look like by clicking on them instead of having to run head. I would expect clicking on a large text file to have no worse performance than opening a file with just a few dozen lines.  Unfortunately, vscode hangs for quite a while before it determines it cannot load the file into memory and then applies the strategy of just displaying a few lines without formatting.
Instead, whenever opening a file, the first few lines should be shown almost instantly in every case.
Because of the current slowness of clicking on a large file, exploring just the first few lines of a bunch of huge text files is very costly (timewise).
Interestingly, vscode doesn't hang up completely when you click on a large text file, but when you click on a small one, it will not charge until the big one is ready. It seems there is only one queue for loading the files, so there's definitely some parallelization potential.

No matter the size of the file, if it is clearly a binary file, the warning "The file is not displayed in the editor because it is binary" should be shown almost instantly. Currently, the time for this message to show up seems to depend on the size of the file and the editor hangs while this is being determined, making accidental clicks costly (timewise).

No, we try to detect if a file is binary reading the first 8kb or so.

I think this needs a combination of editor support to stream a model while you scroll and support this in the file layer to only read what you see. We already support reading at position and offset, but it might not be exposed to extension authors.
//cc @hediet @alexdima

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

Hey @bpasero!
I believe you are right when you say that reading at position and offset is not yet exposed to extension authors, see https://github.com/microsoft/vscode/issues/84515#issuecomment-559051830. The proposed API, linked in the comment, is now on its own file here. Is there any plan to make this proposed API available to extension authors?
Thanks!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Adding characters

There is already a huge number of Pull Requests awaiting for merge.
I feel that the more efficient way to add simple characters is rather to open an issue with rationale (please feel free to dismiss this issue if you disagree).
Thus I suggest to add the following ones:

« –  'left-pointing double angle quotation mark' or 'left pointing guillemet (old)' (U+00AB)
» – 'right-pointing double angle quotation mark' or 'right pointing guillemet (old)' (U+00BB)
≠ –  'not equal to' (U+2260)
– – 'en dash' (U+2013)
— – 'em dash' (U+2014)
’ – 'right single quotation mark' or 'single comma quotation mark' or 'apostrophe' (U+2019)

Also not looking to add a new PR as I don't want to create this repo's 100th merge conflict, so I just thought I'd chime in with the characters I have on my streamdeck:
ÉÈÀÇ would nicely complement the existing ones.
I'm a big believer in the zero-width-space symbol  and i feel it could be useful.
And also: ·—↓←→↑🡇🡄🡆🡅
Thank you for this neat little site! Though I suspect you might soon end up with too many characters? Maybe add a little disclaimer saying this is a curated selection and redirect to somewhere like http://xahlee.info/comp/unicode_index.html?q= for a more complete list.


Soft exit that allows current video to end, then optionally play an outro video, before closing program

This would allow for a natural end to the stream without interrupting programming if immediate termination is not needed or desired.

Pressing Ctrl-C to exit the Python program also closes the ffmpeg processes, even if KeyboardInterrupt is caught.

Version 2.2.0 now terminates the stream and adds a restart delay when Ctrl-C is pressed, allowing the stream to be soft-restarted without exiting Mr. OTCS. Allowing the current video to finish before doing this is the next logical step.

This could be expanded to allow limited live control of the playlist, such as restarting or skipping the current video.


REF remove binutils constraint

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

We don't need these since all of the incompatible versions of binutils at 2.34 exclude these versions of gcc.

This should be reverted so that we could remove the ugly constraints in the recipe.

I'm confused by your comment. Which recipe? How does it help?

I want to remove https://github.com/conda-forge/binutils-feedstock/blob/master/recipe/meta.yaml#L42-L44


Different behavior of distinct() from 2.2.37 to 3.0.30

Having a ODatabaseDocument with masters and details:
SELECT expand(distinct(master)) FROM Detail WHERE name LIKE 'foo%'.
Executed via com.orientechnologies.orient.core.db.ODatabase.query(String, Object...).
Using 2.2.37 it returns 1 document.
Using 3.0.30 it returns 3 times the same document, one per each detail, distinct is not working.
Has "distinct()" changed?

Hi @diegomtassis
If you use the legacy API (ie. the old executor) the behaviour should be the same, but if you use the new one (db.query(String, Object...)) then yes, I expect different results.
In general, I'd suggest to use DISTINCT keyword instead, eg.

)

Thanks
Luigi

Thanks @luigidellaquila


Allow URLs

Allow gists.yaml to take URLs inplace of Gist-IDs.

URLs already have the Gist-ID. The simplest solution is to have the user provide only the ID.


 That was probably the case on long blocks too. I have few ideas to fixed it later.

Deserializing very sparse data (a lot of nulls) is slower than before. That was probably the case on long blocks too. I have few ideas to fixed it later.

So this regression was introduced by mine changes? IIRC after my changes serde for longs withs null was similar to serde without nulls before my changes.

Could you add tests like io.prestosql.spi.block.TestLongArrayBlockEncoding for other blocks?

So this regression was introduced by mine changes?

It appears so. But this is not that big of a deal.
Take a look at https://docs.google.com/spreadsheets/d/1kwG8jYMCGqpxJd4wxz9pltPwyNHgzv2JgHAZq6HcMB8/edit#gid=0
Th regression is only in very sparse data sets. And it is lower in bigger type sizes. I'd say there was no or very minor regression on long type. However the byte type shows some problems. Let's stand by with merging this PR until I sort it out.

Th regression is only in very sparse data sets. And it is lower in bigger type sizes. I'd say there was no or very minor regression on long type. However the byte type shows some problems. Let's stand by with merging this PR until I sort it out.

How could there be a regression optimized case with nulls was as fast as non-optimized case without nulls (essentially the best case)?

Can you post the std dev for the benchmark results? BTW, here's a great site for analyzing results of JMH benchmarks: https://jmh.morethan.io/

Full benchmark results: https://drive.google.com/file/d/17BusbXYbd8Y5BVWXvCrNbrQz369RITMH/view?usp=sharing
Errors below 1%

Could you add tests like io.prestosql.spi.block.TestLongArrayBlockEncoding for other blocks?

https://github.com/prestosql/presto/pull/6055

Made changes to deserialize int128. It appears that using System.arraycopy is (!) more performant than copying two longs from array to array. At least in this case.
Anyway, this is the most I can get without doing disassembly. I suggest merging it as it is now (I can throw away the byte and/or short deserialization if we cannot handle that kind of regression) and then immerse into disassembly, branch-prediction, cache lines or whatever is actually going on in here.

Made changes to deserialize int128. It appears that using System.arraycopy is (!) more performant than copying two longs from array to array. At least in this case.

Are you sure it isn't the problem with 2*x multiplications there? There were 4 or more such multiplications in the code

Are you sure it isn't the problem with 2*x multiplications there?

I did the benchmark on this. There are 2 variables position and i. Making position decrease by 2 helped a little bit. Doing the same with i decreased the performance as it needed to be divided by 2 every time anyway.


code is added for a simple calculator

star this repo so more people can reach out

Star this repo so more people can reach out


docs: update readme with rerender explanation

GitHub won't let me comment on lines that aren't close to what you changed, so I'll just drop a couple things here:

the formatting is breaking the "1." MD numbering system (look at the readme on the repos main page rather than its code and you'll see what I mean. We're getting 1, 2, 1, 1, 1, 2). I think it's just the new line to host the snippets not being numbered lines that's confusing it. You might have to put the correct numbers in
line 44 trails off: In Component.test.tsx, import" ... I think this whole section may have gone unfinished as it doesn't get to explaining the mockQueryPayloador usingexpectMutationToBeCalled`
can we run prettier against the whole codebase? I'm not seeing new lines at the eof, so I assume it hasn't been run here

Lastly I assume we have, but if we haven't run that example to make sure it works we need to.
Otherwise their coming together nicely, great job!


Add support for flatmap

Resolves #11

Unfortunately I am no longer able to maintain this project, I apologize. If someone wishes to step up and take leadership of this project please respond to #21

👍 do others like it? @lukesneeringer or @udnisap

Looks good to me.

@davecoates Can you merge master into this branch and push so Travis can pick it up?

👍

@davecoates Still need master merged in before I can merge. As soon as GitHub gives me a green button, I am ready to hit it. :-)

Sorry! Was too early for me... forgot to actually fetch from upstream first

And sadly, since I just merged your other one...now you have to do it again.

... and done!

:tada:


Add non confirming processors

asserts on 400 or 422
moves a test data gen from suite to setup
adds missing normative statement

This should be retitled to Add non-conforming processors. (confirming is an typo.)


chronos REST api server doesn't response

We encountered this issue twice in about 3 months, generally we have about 25K jobs per day and the api server may doesn't response sometime. when this issue happen, curl -X -L <host>:<port>/ping will hang up for ever.

Hi @chengweiv5 , sorry for responding that late.
Did you ever do a stack trace when this happens? You can do that by issuing a kill -3 or using the jstack command. It is helpful if you get multiple stack traces so that we can compare what changed or didn't change in-between.
Maybe, all threads accepting requests are blocked.

thanks for your tips, @kolloch, we'll do that if we lucky to reproduce this issue.


add a list of awesome anime to the entertainment section

Please read the instructions for creating a new list, contributing guidelines and the awesome manifest.
Especially this part:

Wait at least a couple of weeks after creating a list before submitting to give it a chance to mature.

Ping me here in a few weeks and I'll reopen ;)

Cool idea. You should add some commentary for each item on why they're worth watching. Example: https://github.com/sindresorhus/awesome-scifi#childhoods-end-1953-by-arthur-c-clarke-40 And require it for new entries in the contributing guidelines: https://github.com/sindresorhus/awesome-scifi/blob/master/contributing.md#comments


BTTV changed to uppercase

I guess I have autismo or something, but this bothered me for some time, sorry.

With this change it would say BTTV for global emotes, and as before Bttv for channel emotes and `BetterTTV in the settings.
I suggest that it would be best if it is changed to BetterTTV in all instances as that seems to be the official name on the website www.nightdev.com/betterttv

You mean also for variables and stuff or just text (which user can see)?

Just text that the user can see

I guess it should be okay now? 😅


add abs in Cstdlib header

pt., 7 cze 2019, 15:39 użytkownik sinkyoungdeok<EMAIL_ADDRESS>napisał:

You can view, comment on, or merge this pull request online at:
https://github.com/TheAlgorithms/C-Plus-Plus/pull/221
Commit Summary

add abs in Cstdlib header

File Changes

A Cstdlib/abs.cpp
https://github.com/TheAlgorithms/C-Plus-Plus/pull/221/files#diff-0
(16)

Patch Links:

https://github.com/TheAlgorithms/C-Plus-Plus/pull/221.patch
https://github.com/TheAlgorithms/C-Plus-Plus/pull/221.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/TheAlgorithms/C-Plus-Plus/pull/221?email_source=notifications&email_token=AD3N4BJVZTU2YT2FC4AYBQDPZJQJVA5CNFSM4HVXFKL2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GYIE66Q,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD3N4BOWDTNLEMOUPXQYMGLPZJQJVANCNFSM4HVXFKLQ
.

Thank you for contributing, but this method is already defined in the C++ libraries, re-implementing it may be of no use unless there is huge improvement in the code.
Closing PR.


Fix nav bar position after hiding the extension view

When changing the extension view to nil from it previously being set,appropriate changes are made once... but _handleScrolling is then called, leading to incorrect content offsets until the view is scrolled back to the top.  Disabling scroll handling temporarily while laying out the views fixes the problem.

Great! Thanks @evands. I'll have to take a deeper look at what this implies, and hopefully merge. From a glance, I don't see any issue with this. A comment explaining why this fix is necessary would be nice, though.

Added a comment as requested!


Configure Dependabot for npm packages at the template level

Description
Changes to get Dependanbot updates for npm packages:

Update Dependabot config
Remove all the Jinja tags from the package.json in order to make it parsable by Dependanbot
Remove  unwanted dependencies in post-generation hook, to replace what was done by templating

Checklist:

[x] I've made sure that tests/test_cookiecutter_generation.py is updated accordingly (especially if adding or updating a template option)
[x] I've updated the documentation or confirm that my change doesn't require any updates

Rationale
We can't keep npm packages up to date because they are not automatically updated.

Let's give this one a try.


feat: use Xpath as selector

Updated getPageHtml Function: The function is now equipped to discern between XPath and CSS selectors. It properly evaluates XPath expressions, addressing issues where the crawler previously failed to process XPath selectors.
XPath Handling in requestHandler: The request handler now contains logic to distinguish XPath selectors from CSS selectors. This addition ensures the crawler appropriately waits for and handles elements located via XPath.
waitForXPath Function: A new function that waits specifically for XPath elements, improving the crawler's robustness and its ability to handle dynamic content loaded via JavaScript.

looks great, thanks @LeonKohli !


transfer NewBoiler models from API to REopt.jl

done!

I'm not sure why this was marked as done. New boiler is not finished yet.

I'm not sure why this was marked as done. New boiler is not finished yet.


Use stestr for test jobs

Since #229 our test runs take significantly longer. This is mostly a
function of the tests added in that function running over numerous
random graphs and comparing the output against networkx to verify it
works in all cases. However, this validation comes with a runtime cost.
In an attempt to mitigate this partially this commit changes the test
runner from stdlib's unittest runner to use stestr. [1][2][3] stestr
provides a performance improvement by running tests on parallel workers
and also provides an easier to use UI that enables test selection a
history of previous runs, and many other features. The test suite in
retworkx will still need to be strictly unittest compatible so anyone
can run with stestr, stdlib unittest, pytest, or any other test runner
of their choice. But the default in tox and CI will be to use stestr
for the performance benefit.
[1] https://pypi.org/project/stestr/
[2] https://github.com/mtreinish/stestr
[3] https://stestr.readthedocs.io/en/stable/README.html

Pull Request Test Coverage Report for Build 633222624

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

Totals

Change from base Build 618133837:
0.0%

Covered Lines:
5890

Relevant Lines:
6112

💛  - Coveralls


Question about vocabulary_w2v.txt

What is the class of each word2vec in vocabulary_w2v.txt and where to get the word2vec ?

Please refer these two issues: #9  and  #17.

I have already refer i still have some question which class did you choose for vocabulary_w2v.txt and the word2vec source is as same as 48_17 word2vec ?

The vocabulary_w2v.txt contains tags of NUS-WIDE, the MS-COCO class names(such as cat, dog, etc..) and tags that have no word vectors in the pre-trained word2vec model GoogleNews-vectors-negative300.bin has been removed, thus there are 4717 vectors in vocabulary_w2v.txt. In the detection branch, I used all 4717 vectors.
They are from the same source folllowing this, I guess all word vectors are extracted from word2vec project with this pre-trained word2vec model GoogleNews-vectors-negative300.bin

Please feel free to ask if you have any other questions.

Is there has any information about each row of vocabulary_w2v.txt correspond to what class?

I get it from here for fair comparison and I do not know the word order, if you what to know the word of each row, you need to write a parser to analyse the word2vecs GoogleNews-vectors-negative300.bin to match with the vectors in vocabulary_w2v.txt. (In GoogleNews-vectors-negative300.bin, it contains all words and corresponding vectors.)
You might be able to do that:

Download the 4717 tags or words from NUS_WID_Tags.zip, and the Final_Tag_List.txt in it contains the tags
Apply Inverse l2 normalization in vocabulary_w2v.txt to obtain the original word vectors because the vocabulary_w2v.txt was processed by l2 normalization
Download the GoogleNews-vectors-negative300.bin.gz and load it by:

import gensim
model = gensim.models.KeyedVectors.load_word2vec_format('GoogleNews-vectors-negative300.bin.gz', binary=True)

For any word in the Final_Tag_List.txt(which is from NUS_WID_Tags.zip) such as nature, you can get its word vectors through: wv = model.wv('nature')

Search wv in the original word vectors from step 2

Thanks a lot for reply appreciated!

I have some question about zsd performance, does zsd mean train without seg?

The first row ZSD result in Table3 mean train without instance segmentation, while other ZSD results in our paper are with seg.
By the way, ZSI results contain ZSD results, you can train a model with or without seg to perform ZSD.

How do i train without seg in your code?

You can refer the config file in #40.

Thanks a lot I will try it !

What is the different between # dataset_type = 'CocoDatasetSeen48' ,dataset_type = 'CocoDatasetUnseen17' in your config file?
When training which one should i use ?

You can refer #11 for this question.

Thanks a lot !
I train your code without seg, this is my performance below is that close to yours?
Average Precision  (AP) @[ IoU=0.40      | area=   all | maxDets=100 ] = 0.030
Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=100 ] = 0.024
Average Precision  (AP) @[ IoU=0.60      | area=   all | maxDets=100 ] = 0.017
Average Recall     (AR) @[ IoU=0.40      | area=   all | maxDets=100 ] = 0.517
Average Recall     (AR) @[ IoU=0.50      | area=   all | maxDets=100 ] = 0.475
Average Recall     (AR) @[ IoU=0.60      | area=   all | maxDets=100 ] = 0.402

This result is too low compared to my previous results. Is it convenient to provide the config file in the testing phase?

What is your results of ZSI without seg?
This is  my train config
model settings
model = dict(
type='ZeroShotFasterRCNN',
pretrained='torchvision://resnet101',
backbone=dict(
type='ResNet',
depth=101,
num_stages=4,
out_indices=(0, 1, 2, 3),
frozen_stages=1,
style='pytorch'),
neck=dict(
type='FPN',
in_channels=[256, 512, 1024, 2048],
out_channels=256,
num_outs=5),
rpn_head=dict(
type='BackgroundAwareRPNHead',
in_channels=256,
semantic_dims=300,
feat_channels=256,
anchor_scales=[8],
anchor_ratios=[0.5, 1.0, 2.0],
anchor_strides=[4, 8, 16, 32, 64],
target_means=[.0, .0, .0, .0],
target_stds=[1.0, 1.0, 1.0, 1.0],
voc_path=None,
vec_path='data/coco/word_w2v_withbg_48_17.txt',
sync_bg=True,
loss_cls=dict(
type='CrossEntropyLoss', use_sigmoid=False, loss_weight=1.0),
loss_bbox=dict(type='SmoothL1Loss', beta=1.0 / 9.0, loss_weight=1.0)),
bbox_roi_extractor=dict(
type='SingleRoIExtractor',
roi_layer=dict(type='RoIAlign', out_size=7, sample_num=2),
out_channels=256,
featmap_strides=[4, 8, 16, 32]),
bbox_head=dict(
type='SharedFCSemanticBBoxHead',
num_fcs=2,
in_channels=256,
fc_out_channels=1024,
roi_feat_size=7,
num_classes=49,
semantic_dims=300,
seen_class=True,
reg_with_semantic=False,
share_semantic=False,
with_decoder=True,
sync_bg=True,
voc_path='data/coco/vocabulary_w2v.txt',
vec_path='data/coco/word_w2v_withbg_48_17.txt',
target_means=[0., 0., 0., 0.],
target_stds=[0.1, 0.1, 0.2, 0.2],
reg_class_agnostic=False,
loss_semantic=dict(
type='CrossEntropyLoss', use_sigmoid=False, loss_weight=1.0),
loss_bbox=dict(type='SmoothL1Loss', beta=1.0, loss_weight=1.0),
loss_ed=dict(type='MSELoss', loss_weight=0.5)),
bbox_with_decoder=True,
bbox_sync_bg=True)
# mask_roi_extractor=dict(
#     type='SingleRoIExtractor',
#     roi_layer=dict(type='RoIAlign', out_size=14, sample_num=2),
#     out_channels=256,
#     featmap_strides=[4, 8, 16, 32]),
# mask_head=dict(
#     type='SemanticFCNMaskHead',
#     num_convs=4,
#     in_channels=256,
#     conv_out_channels=256,
#     num_classes=49,
#     gzsd=False,
#     semantic_dims=300,
#     seen_class=True,
#     share_semantic=False,
#     voc_path=None,
#     vec_path='data/coco/word_w2v_withbg_48_17.txt',
#     with_learnable_kernel=True,
#     with_decoder=True,
#     loss_mask=dict(
#         type='CrossEntropyLoss', use_mask=True, loss_weight=1.0),
#     loss_ed=dict(type='MSELoss', loss_weight=0.5)),
# mask_with_decoder=True,
# bbox_with_decoder=True,
# bbox_sync_bg=True,
# mask_sync_bg=True)

model training and testing settings
train_cfg = dict(
rpn=dict(
assigner=dict(
type='MaxIoUAssigner',
pos_iou_thr=0.7,
neg_iou_thr=0.3,
min_pos_iou=0.3,
ignore_iof_thr=-1),
sampler=dict(
type='RandomSampler',
num=256,
pos_fraction=0.5,
neg_pos_ub=-1,
add_gt_as_proposals=False),
allowed_border=0,
pos_weight=-1,
debug=False),
rpn_proposal=dict(
nms_across_levels=False,
nms_pre=2000,
nms_post=2000,
max_num=2000,
nms_thr=0.7,
min_bbox_size=0),
rcnn=dict(
assigner=dict(
type='MaxIoUAssigner',
pos_iou_thr=0.5,
neg_iou_thr=0.5,
min_pos_iou=0.5,
ignore_iof_thr=-1),
sampler=dict(
type='RandomSampler',
num=512,
pos_fraction=0.25,
neg_pos_ub=-1,
add_gt_as_proposals=True),
mask_size=28,
pos_weight=-1,
debug=False))
test_cfg = dict(
rpn=dict(
nms_across_levels=False,
nms_pre=1000,
nms_post=1000,
max_num=1000,
nms_thr=0.7,
min_bbox_size=0),
rcnn=dict(
score_thr=0.05,
nms=dict(type='nms', iou_thr=0.5),
max_per_img=100,
))
dataset settings
dataset_type = 'CocoDatasetSeen48'
dataset_type = 'CocoDatasetUnseen17'
data_root = 'data/coco/'
img_norm_cfg = dict(
mean=[123.675, 116.28, 103.53], std=[58.395, 57.12, 57.375], to_rgb=True)
train_pipeline = [
dict(type='LoadImageFromFile'),
dict(type='LoadAnnotations', with_bbox=True, with_mask=True),
dict(type='Resize', img_scale=(1333, 800), keep_ratio=True),
dict(type='RandomFlip', flip_ratio=0.5),
dict(type='Normalize', **img_norm_cfg),
dict(type='Pad', size_divisor=32),
dict(type='DefaultFormatBundle'),
dict(type='Collect', keys=['img', 'gt_bboxes', 'gt_labels', 'gt_masks']),
]
test_pipeline = [
dict(type='LoadImageFromFile'),
dict(
type='MultiScaleFlipAug',
img_scale=(1333, 800),
flip=False,
transforms=[
dict(type='Resize', keep_ratio=True),
dict(type='RandomFlip'),
dict(type='Normalize', **img_norm_cfg),
dict(type='Pad', size_divisor=32),
dict(type='ImageToTensor', keys=['img']),
dict(type='Collect', keys=['img']),
])
]
data = dict(
imgs_per_gpu=1,
workers_per_gpu=2,
train=dict(
type=dataset_type,
ann_file=data_root + 'annotations/instances_train2014_seen_48_17.json',
img_prefix=data_root + 'train2014/',
pipeline=train_pipeline),
val=dict(
type=dataset_type,
ann_file=data_root + 'annotations/instances_val2014_seen_48_17.json',
img_prefix=data_root + 'val2014/',
pipeline=test_pipeline),
test=dict(
type=dataset_type,
ann_file=data_root + 'annotations/instances_val2014_unseen_48_17.json',
img_prefix=data_root + 'val2014/',
pipeline=test_pipeline))
optimizer
optimizer = dict(type='SGD', lr=0.01, momentum=0.9, weight_decay=0.0001)
optimizer_config = dict(grad_clip=dict(max_norm=5, norm_type=2))
learning policy
lr_config = dict(
policy='step',
warmup='linear',
warmup_iters=500,
warmup_ratio=1.0 / 3,
step=[8, 11])
checkpoint_config = dict(interval=4)
yapf:disable
log_config = dict(
interval=50,
hooks=[
dict(type='TextLoggerHook'),
dict(type='TensorboardLoggerHook')
])
yapf:enable
evaluation = dict(interval=1)
runtime settings
total_epochs = 12
dist_params = dict(backend='nccl')
log_level = 'INFO'
work_dir = './work_dirs/zsd/48_17'
load_from = None
resume_from = None
workflow = [('train', 1)]

The detection AP sames lower than mine, I have acheived 9-10 AP50 without seg.
In your test config, you set sync_bg=False but use sync_bg=True in your training config.

I solved it thanks

I have some question about w2v, all these w2v are apply L2 normalize but why there length are not equal to one ?

I guess this is a special l2 normalization. I calculated the length is 0.7666293667273409, indicating that there is an additional constant during normalization.


RFC 31 - Default solo to local mode

This switches the default operation of chef-solo to be equivalent to chef-client -z, but allows for the old style chef-solo by uttering chef-solo --legacy-mode.
cc @chef/client-maintainers

Local mode:
ubuntu@ip-172-31-37-200:~$ sudo chef-solo --recipe-url ./solo.tar.gz -c config.rb -j dna.json
Starting Chef Client, version 12.10.38
resolving cookbooks for run list: ["hypo::default"]
Synchronizing Cookbooks:
  - hypo (0.1.0)
  - chef-solo-search (0.5.1)
Installing Cookbook Gems:
Compiling Cookbooks...
/home/ubuntu/local-mode-cache/cache/cookbooks/chef-solo-search/libraries/vendor/chef/solr_query/lucene_nodes.rb:22: warning: already initialized constant Lucene::SEP
/home/ubuntu/local-mode-cache/cache/cookbooks/chef-solo-search/libraries/vendor/chef/solr_query/lucene_nodes.rb:22: warning: previous definition of SEP was here
Converging 1 resources
Recipe: hypo::default
  * file[/tmp/search] action create (up to date)

Running handlers:
Running handlers complete
Chef Client finished, 0/1 resources updated in 01 seconds

And then legacy mode:
ubuntu@ip-172-31-37-200:~$ sudo chef-solo --recipe-url ./solo.tar.gz -c config.rb -j dna.json --legacy-mode
Starting Chef Client, version 12.10.38
Installing Cookbook Gems:
Compiling Cookbooks...
/var/chef/cookbooks/chef-solo-search/libraries/vendor/chef/solr_query/lucene_nodes.rb:22: warning: already initialized constant Lucene::SEP
/var/chef/cookbooks/chef-solo-search/libraries/vendor/chef/solr_query/lucene_nodes.rb:22: warning: previous definition of SEP was here
Converging 1 resources
Recipe: hypo::default
  * file[/tmp/search] action create (up to date)

Running handlers:
Running handlers complete
Chef Client finished, 0/1 resources updated in 01 seconds

Was there any discussion about having Chef::Config[:solo] be the same value as legacy mode? Lot's of cookbooks check Chef::Config[:solo] and do something special for search or whatever, which Zolo should support now.
Which is more likely to be painful, having a cookbook with solo specific code running in Zolo mode, and still using the cooked in alternatives, or having a cookbook with solo specific code running in Zolo mode and suddenly doing something different?

@btm the discussion has been that Zolo mode needs to still set :solo as before. Yes, in the case of search especially that sucks, but we could provide an updated chef_solo_search cookbook for the users of that. Regardless I think continuity of behaviour for existing customers is the most important thing here.

:+1: besides that i'd be easier to grok #reconfigure if the old code got factored out as I commented on.

you need to preserve the chef-solo --ez flag:
https://github.com/chef/chef/blob/master/lib/chef/application/solo.rb#L178-L181

you need to preserve the chef-solo --ez flag:

it's preserved for legacy mode, which I think is right.

Pending https://github.com/sethvargo/chefspec/pull/711


LongTap not raised on Samsung Galaxy Tab S4

When attempting a LongTap on a Samsung Galaxy Tab S4, the map moves and does not register the LongTap. I looked at the MapControl source and saw the touchSlop=8 added to address this issue (specifically with Samsung devices), so I created my own LongPress event and had to increase the slop to 20! Is there a better approach?

No, not up to now. You could read about the problem in the source code of Android here and here. I assume, that your device has a higher resolution than other devices and has therefore a higher TouchSlop in the configuration file.
Up to now I don't have an idea, where to get this value from. Perhaps it should be configurable by the user.


Update service-fabric-reliable-services-reliable-collections.md

Updating freshness and adding a recommendation

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

#6

#sign-off

@mcoskun : Thanks for your contribution to the Azure documentation! The author, @mcoskun, has been notified to review your proposed change.

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.
It�s easy to access the private repo version of your articles in GitHub. Just add �pr to the URL, as follows:
Public repo URL: https://github.com/Azure/azure-content/blob/master/articles/azure-portal-overview.md
Private repo URL: https://github.com/Azure/azure-content-pr/blob/master/articles/azure-portal-overview.md
We've closed this pull request. Please resubmit the updates through the private repo. If you need perms to the private repo, see the contributor guide instructions.