REF: Refactored Archiver logic

Archiver has become its own subpackage archive. There are now 3 distinct
components: Archiver, _Archive impl, and ArchiveFormat.
The Archiver is responsible for coordinating, it maintains a list of all
current archive formats and archive implementations as well as what the current
version of both of those are.
An _Archive implementation is responsible for the high level structure of an
archive (the uuid root dir and VERSION file). Its job is to recognize if a
given file is valid archive and if so mount/extract it. Additionally it
provides a setup which initializes an existing directory.
An ArchiveFormat specifies the rest of the layout of an archive, this
includes what the data directory is called, metadata, etc...
tldr; Archiver gets ZipArchive to convert an archive to the filesystem and then
gets ArchiveFormat to read the details.
Additionally it is now possible to peek at an archive even if you do not have
the correct plugins installed to load an archive.
Similarly you can extract an archive even if you do not have the latest
ArchiveFormat.

Thanks @ebolyen, nice job on the refactoring! 🎉

🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 🌮 !!!!!!!!!


Create DESIGN.md

Closes
Closes #667

Description
I created the logo using Adobe Illustrator after doing my research about what OS-CODE community does and represents. I feel that this logo will be a great addition to the website since the current logo does not do justice to the amazing community that it represents. The previous Turquoise colour seemed really off and did not match the other colours of the website making it an off addition.
Now I have fixed that using the theme colours. I believe that the design file will be a good place for UI/UX designers to collaborate and add designs so as to make the flow and the UI of the website better overall. It will also make for an interesting read for the people.
Note to reviewers
After this gets merged, I would want to change the assets folder and replace the logo for the website. I am really excited for the revamp.
Checklist:

[x] I have linked the PR to the correct issue.
[x] I have read the Contribution Guidelines
[x] I have built and tested the changes, and they do not break or show any errors.

This looks great @soumizde!!! Thanks a lot, we will get back to u:)


Spell grow and emp

Hi, both spells do not function how they are supposed to be.

What are you expecting them to do? What version of Spigot and Magic are you on?

My apologize, both spells work. It was a bug from our side. I have 2 more questions tho.

Is it possible to change the shop GUI's you created? as example the hat shop?
Dragonball doesn't give SP, is this a bug or supposed to be?

Dragonball should give SP, but only if you hit something with the fireball itself. Mobs or players going into the poison cloud doesn't count.
It is very easy to make your own shops, yes. Just take one of the builtins and put something similar in your spells folder.
https://github.com/elBukkit/MagicPlugin/blob/master/Magic/src/main/resources/examples/survival/spells/shops.yml#L1

Thank you.

Okay, i already see the issue with dragonball. You cannot hit an entity with the dragonball itself. Do you have any solutions for it?

Why can't you? I thought it would hit and explode like normal.
If that is the case though you (or me) could add an AreaOfEffect action to ping everyone around for 1 damage or something.

I've added a 1-damage AOE to Dragonball to fix the SP earn issue.


circular-buffer: Test overwrite on non-full buffer

This test is useful to make sure implementations don't drop items from
non-full buffers when using Overwrite.

Examining other language tracks:
http://x.exercism.io/problems/circular-buffer
Tracks that include this test:

ruby (the first version of the exercise I did, and I failed this exact test in that track, which is why I noticed this difference when compared to Go's)
rust (at my recommendation when the exercise was about to be added to that track)

Tracks that do not include this test:

erlang (but it does test their equivalent of ovewrite on a completely empty buffer, so maybe that is close enough?)
javascript
lfe (same comment as erlang)
lua

So I guess Ruby was the first to actually explicitly test this behavior of Overwrite (what should it do when the buffer isn't full), and I guess now's a good a time as any to ask whether it should be spread to the other tracks with circular-buffer.

Assumption: I should bump testVersion for this (because it might cause previously-passing solutions to fail). I will do so and amend the commit.

Yeah, I think adding the test is a good idea. @soniakeys do you see any gotchas with this?

Very nice.  Thank you, Peter!


Is there a way to highlight a particular string?

This library is a miracle for me, amazing work! I want to use it to display results for a search tool and ideally would want to support results highlighting. Is that possible?

No this feature is not supported.
Just did some quick search and you might find an past example or a discussion on pdf.js highlight text useful.

Thanks very much for the two pointers, very useful - happy to close the ticket unless you want to keep it as a feature request :)


CPU usage when train

hello, thanks your clean code.
But i found a strange phenomenon that is my cpu usage is very high(almost 100% usage per kernel spied by linux system monitor) so that it run smoothly and stop for a while and continue run...
Have you also experienced this phenomenon when training?What may cause this phenomenon in code?

BTW, its GPU usage is not so tight as cpu usage

Different from other implementation on Siamese trackers. This demo directly crop trian image in original GOT10K dataset rather than crop the whole dataset before train. So during the train process, there will be a big burden on CPU. I have tried this demo on I7 9700k CPU RTX2080Ti GPU, it will taken 100% usage of CPU while token no more than 5% usage of GPU. If you don't have a powerful CPU and large RAM, I strongly recommend you to crop the training datasets for the first time.


1-class processor model and stylechecks on all processors.geo.jvm

Started with setEPSG Processor
and validate result
check GeoJvmInit
Imported from Jira STREAMPIPES-642. Original Jira may contain additional context.
Reported by: micklich.
Subtask of issue #775

Hi @flomickl,
was there a specific reason why you changed the method API of Event.addField(String runtimeName, Float value) to Event.addField(String runtimeName, double value)?
The nightly e2e tests failed for the BooleanCounter processor, because the datatype of the counter (an integer) was changed to a double. I changed the API back to Float but wanted to make sure that this does not break any of the other changes you made in the PR.
Cheers,
Philipp
See [1] for my changes and [2] for the e2e test results.
[1] https://github.com/apache/streampipes/commit/7dfe113dc6b30cf20513a4bf69199c50e03bfe79
[2] https://cloud.cypress.io/projects/q1jdu2/runs/401/specs

@tenthe
Hm I did not touch this file at all, not on purpose.
I don't know why this file was changed at all in my commit.
Sorry for the inconvenience! Did you revert my "change"?

Hi @flomickl, no problem at all.
I changed it back, I just wanted to make sure I didn't break anything in the code you wrote.

I stashed all commits before, So I cannot figure it out how it happens and when.
So, I think this ticket can be closed now?


added template support

Any reason why this PR is not addressed?

Hi @israel, sorry for lack of response. To be honest, I saw this PR in november, but because of extreme lack of time I didn't review it then, and later I simply forgot about it. I will try to review it today in the evening

Thanks

Closing #28

Thanks @israel for contribution. This will be included in version 2.5.0, which I will release today


rp: Fix ROM cache ptr() returning the trampoline

When #1384 changed the ROM cache optimization it unfortunately introduced some subtle breakage.  The flash functions assume that a call to ::ptr() returns the actual ROM address (since they need to operate without any calls into XIP), but after the optimization the first call will return the trampoline.  The trampoline itself is (likely) located in flash, so when the flash operation functions try to use it with XIP disabled, they immediately hard fault.
So this just separates out the ::ptr() function into a version that uses the full optimization for the direct calls and one that uses a slightly less efficient version but will always return a ROM pointer.

thanks for the fix! :rocket:


Skips first few lines in PDF.

I'm running the server in docker:
image: ghcr.io/nlmatics/nlm-ingestor:latest
I've only tested with one 300page PDF and it seems to skip the first couple lines of the PDF.  It doesn't seem to be an issue but It makes me wonder if anything else is being skipped. This is the same whether I convert to text, use sections, or convert to html.
What might be the cause?

I'm also seeing this. When using llmsherpa.readers.LayoutPDFReader, the returned output is missing the title line of my PDF--which happens to be one of the first lines.

At least it's not just me! I would be open to fixing it but I'm guessing it's related to the NLP model?

Same here!


Is it remember query string is deprecated??

i'm inertia beginner user.i test ping app and i see query string is remember on url with searched pagination results.
but in my own app it's not working like that. Is it remember query string is deprecated?? or i understand wrong way??
pls help

This is done using my Remember Query Strings library, which is also used by Ping CRM. Follow the instructions on that repo to get it setup in your own app. 👍

oops thanks <3 i thought that was inertia default package


METRON-2161 Ambari client exception occurred: No JSON object could be…

When deploying Metron using the CentOS 7 development image on the feature/METRON-2088-support-hdp-3.1 feature branch, the following error occurs.
TASK [ambari_config : Deploy cluster with Ambari; http://node1:8080] ***********
...
"msg": "Ambari client exception occurred: No JSON object could be decoded"
}
to retry, use: --limit @/Users/nallen/Development/metron/metron-deployment/development/centos7/ansible/playbook.retry

PLAY RECAP *********************************************************************
node1 : ok=83 changed=16 unreachable=0 failed=1

[WARNING]: Module did not set no_log for password

Ansible failed to complete successfully. Any error output should be
visible above. Please fix these errors and try again.

This PR is for the HDP 3.1 feature branch, not master.

Background
I have seen this issue previously and it was caused by the python-requests library.  The CentOS 7 node is running Python Requests 2.6.1, which is the last known working version of the library at least on CentOS 6.  This was not a problem when the original PR went in for METRON-2097 as #1397 . There might have been a change with the underlying CentOS 7 VM image.
Querying Ambari using curl shows Ambari is responding correctly.
[root@node1 ~]# curl -u admin:admin -H "X-Requested-By: ambari" -i -X GET -k http://node1:8080/api/v1/blueprints
HTTP/1.1 200 OK
Date: Tue, 25 Jun 2019 13:56:19 GMT
X-Frame-Options: DENY
X-XSS-Protection: 1; mode=block
X-Content-Type-Options: nosniff
Cache-Control: no-store
Pragma: no-cache
Set-Cookie: AMBARISESSIONID=node0jsb8b6skdylu71zzy5l1qfv2.node0;Path=/;HttpOnly
Expires: Thu, 01 Jan 1970 00:00:00 GMT
User: admin
Content-Type: text/plain;charset=utf-8
X-Content-Type-Options: nosniff
Vary: Accept-Encoding, User-Agent
Transfer-Encoding: chunked

{
"href" : "http://node1:8080/api/v1/blueprints",
"items" : [ ]
}

Querying Ambari using python-requests 2.6.1 on CentOS 7 highlights the problem.
>>> import requests
>>> print requests.__version__
2.6.1
>>> r = requests.get("http://node1:8080/api/v1/blueprints", auth=("admin","admin"))
>>> r
<Response [200]>
>>> r.encoding
'utf-8'
>>> r.content
'\x1f\x8b\x08\x00\x00\x00\x00\x00\x00\x00\xab\xe6RPP\xca(JMSR\xb0\x022JJ\n\xac\xf4\xf5\xf3\xf2SR\r\xad,\x0c,\x0c\xf4\x13\x0b2\xf5\xcb\x0c\xf5\x93rJS\x0b\x8a2\xf3J\x8a\x95t@:2KRs\x8bAZ\xa2\x15b\xb9j\x01OR\xd0\x9bE\x00\x00\x00'
>>> r.text
u'\x1f\ufffd\x08\x00\x00\x00\x00\x00\x00\x00\ufffd\ufffdRPP\ufffd(JMSR\ufffd\x022JJ\n\ufffd\ufffd\ufffd\ufffd\ufffdSR\r\ufffd,\x0c,\x0c\ufffd\x13\x0b2\ufffd\ufffd\x0c\ufffd\ufffdrJS\x0b\ufffd2\ufffdJ\ufffd\ufffdt@:2KRs\ufffdAZ\ufffd\x15b\ufffdj\x01OR\u041bE\x00\x00\x00'
>>> r.json()
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "/usr/lib/python2.7/site-packages/requests/models.py", line 819, in json
return json.loads(self.text, **kwargs)
File "/usr/lib64/python2.7/json/__init__.py", line 338, in loads
return _default_decoder.decode(s)
File "/usr/lib64/python2.7/json/decoder.py", line 366, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/usr/lib64/python2.7/json/decoder.py", line 384, in raw_decode
raise ValueError("No JSON object could be decoded")
ValueError: No JSON object could be decoded

Testing
Deploy Metron on CentOS 7.
cd metron-deployment/development/centos7
vagrant up

Pull Request Checklist

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?
[x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?
[x] Have you included steps or a guide to how the change may be verified and tested manually?
[x] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
[x] Have you written or updated unit tests and or integration tests to verify your changes?
[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[x] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

I tested this in full dev and the issue is resolved.  +1

This has been merged into the feature branch.


a.corner.label .icon, does not use pointer for the cursor.

This seems to be deliberately disabled... but I don't understand why. Definitely isn't intended behaviour on my end. JsFiddle
Suspect source code:
.ui.corner.label .icon {
  cursor: default;
  position: relative;
  top: @cornerIconTopOffset;
  left: @cornerIconLeftOffset;
  font-size: @cornerIconSize;
  margin: 0em;
}

See #4238. I reposted your comment there.


Rename mts

Renamed the multiple time step formulation, fixed issue with lambda_b and the economic mode, added two new sets, minor improvements in documentation, renamed term4 in objective function to revenue in sts and mts

Renamed mts to match the new nomenclature in sts, solved issue with economic mts formulation


Add native image configurations

Purpose
Fixes: https://github.com/ballerina-platform/ballerina-standard-library/issues/3481
Examples
Checklist

[x] Linked to an issue
[ ] Updated the specification
[ ] Updated the changelog
[ ] Added tests

Checked on a sample project with a persist entity. The above configurations are not being generated. Need to check how to handle configs used for test cases.

Closing this PR since as per the discussion, the MySQL driver with the native image configs have not been released yet. Need to check once it's released and the test cases are updated.


pull request for e2e test at 01/30/2017 07:15:12

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


add azure login to push image

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


Add cros ble indication change in weave

Add cros ble indication change in cros since its ble indication args are different from bluez upstream.

Where does BLE_CONFIG_CROS_BLUEZ_FEATURE come from? I believe it should be put into an appropriate config file, documented and provided with a reasonable default value.

Thanks, just set BLE_CONFIG_CROS_BLUEZ_FEATURE as default 0 in bleconfig file.


Trade URL verification bug

So when I'm using this trade URL : http://steamcommunity.com/tradeoffer/new/?partner=89031010&token=BqHjy99N, the offer can't be created because the script doesn't recognize it as a valid trade URL. Though you can clearly see it's working...
Am I getting something wrong or is it the verification process?

I found what was wrong, it only works on HTTPS. Maybe it would be nice to also support HTTP?

I am pretty sure TradeURLs are only generated as HTTPs by Valve, that's why
that check is in place.
2016-08-25 7:46 GMT-04:00 KowalskiThomas<EMAIL_ADDRESS>
I found what was wrong, it only works on HTTPS. Maybe it would be nice to
also support HTTP?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DrBomb/python-steamcommunity/issues/15#issuecomment-242358442,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFvii5hh4Ll7nwchIL_mT8iY2bCUsJz1ks5qjYCWgaJpZM4Js-qB
.


SAK-49400 Portal improvements around SkinnableCharonPortal class

https://sakaiproject.atlassian.net/browse/SAK-49400

SAK-49400 #comment Modernizes the SkinnableCharonPortal class

Adds autowiring of spring components
prefer initialization of config when servlet is initialized vs runtime
add generics where missing
code simplification


is 1.0.4 released?

Hi,
I added 1.0.4 version of this library as a dependency but gradle wasn't able to resolve it. In one of the other issues, I saw a somebody comment that he/she tried 1.0.4 and couldn't find a method. I am not sure whats wrong.
Also, in 1.0.3, I cannot find the following methods:
 kenBurnsView.setSwapMs(...);    kenBurnsView.setFadeInOutMs(...);
but I do see them in KenBurnsView when I download the library itself. This makes me think that 1.0.4 is not released yet. Could you please release it soon because these methods will help me a great deal?
Thanks!
P.S: The effect that you have achieved is awesome :)

Seems that the version 1.0.3 on jcenter is outdated.
Unless @gotokatsuya updates the maven, another approach is either:

fork and upload to another maven repository
bundle the .aar file with the app


enhancement and bug fix for db schema tool

(1) Add schema creation support;
(2) fix the table creation timestamp data type issue.

Reviewed and QA'd with Albert


Yfinance period='Max' option bug

The yfinance version 0.1.70 shows a bug when it comes to download historical data using the period='max' option. In fact if I run the code below I get data starting from 1950 while the data go back to 1928 (colab link: https://colab.research.google.com/drive/1eByqpQGTxKH7KmYUy-FDri6HyYALKzzw#scrollTo=spxN65Qi4EJ4). As further validation if I run the same code using yfinance 0.1.63 I get the proper historical data starting from 1928 (colab link: https://colab.research.google.com/drive/1XvUF6ngMskT-aKduMTi_jgsvsRvSYLz_#scrollTo=iR3GS3f9kU3D)
import yfinance as yf

ticker = '^GSPC'
Stock = yf.Ticker(ticker)
Stock_Data = Stock.history(period="max",auto_adjust = False)

Stock_Data

Thanks in advance for your support and help
Luca

Raised https://github.com/ranaroussi/yfinance/pull/1169 request to fix issue


"Get it on F-Droid" badge broken?

The image for "Get it on F-Droid" on https://github.com/commons-app/apps-android-commons/blob/master/README.md appears to be broken for me. Just checking if this is the case for anyone else or if it's just my connection?

https://f-droid.org/badge/get-it-on.png has become extra-slow.
I reported the problem internally.

Reply:

For quite a while, the entire site is slow. So many users, quite a load! Why not copy the badge to where you want to show it?

So we should use this one instead: https://commons.wikimedia.org/wiki/File:"Get_it_on_F-droid"_Badge.png

I've just created #2896 to fix this. Hope this helps.


Unable to connect Workers with Master node

Describe the bug
The worker node is unable to connect to the master node.
To Reproduce
Follow the steps in quickstart guide.
Logs
anshal@node-1:~/vhive$ sudo kubeadm join <IP_ADDRESS>:6443 --token whir9i.n6txv9nawcf864kh \
> --discovery-token-ca-cert-hash sha256:2f983d50ff42e195deaecc952c8e3c76008c4e3e31347e1f5db4a233834f6898 > >(tee -a /tmp/vhive-logs/kubeadm_join.stdout) 2> >(tee -a /tmp/vhive-logs/kubeadm_join.stderr >&2)
[preflight] Running pre-flight checks
[preflight] Reading configuration from the cluster...
[preflight] FYI: You can look at this config file with 'kubectl -n kube-system get cm kubeadm-config -o yaml'
W0221 03:37:37.312188   32963 utils.go:69] The recommended value for "resolvConf" in "KubeletConfiguration" is: /run/systemd/resolve/resolv.conf; the provided value is: /run/systemd/resolve/resolv.conf
[kubelet-start] Writing kubelet configuration to file "/var/lib/kubelet/config.yaml"
[kubelet-start] Writing kubelet environment file with flags to file "/var/lib/kubelet/kubeadm-flags.env"
[kubelet-start] Starting the kubelet
[kubelet-start] Waiting for the kubelet to perform the TLS Bootstrap...
[kubelet-check] Initial timeout of 40s passed.
[kubelet-check] It seems like the kubelet isn't running or healthy.
[kubelet-check] The HTTP call equal to 'curl -sSL http://localhost:10248/healthz' failed with error: Get "http://localhost:10248/healthz": dial tcp <IP_ADDRESS>:10248: connect: connection refused.
[kubelet-check] It seems like the kubelet isn't running or healthy.
[kubelet-check] The HTTP call equal to 'curl -sSL http://localhost:10248/healthz' failed with error: Get "http://localhost:10248/healthz": dial tcp <IP_ADDRESS>:10248: connect: connection refused.
[kubelet-check] It seems like the kubelet isn't running or healthy.
[kubelet-check] The HTTP call equal to 'curl -sSL http://localhost:10248/healthz' failed with error: Get "http://localhost:10248/healthz": dial tcp <IP_ADDRESS>:10248: connect: connection refused.
[kubelet-check] It seems like the kubelet isn't running or healthy.
[kubelet-check] The HTTP call equal to 'curl -sSL http://localhost:10248/healthz' failed with error: Get "http://localhost:10248/healthz": dial tcp <IP_ADDRESS>:10248: connect: connection refused.
[kubelet-check] It seems like the kubelet isn't running or healthy.
[kubelet-check] The HTTP call equal to 'curl -sSL http://localhost:10248/healthz' failed with error: Get "http://localhost:10248/healthz": dial tcp <IP_ADDRESS>:10248: connect: connection refused.
error execution phase kubelet-start: error uploading crisocket: nodes "node-1.vhive-rs440.ntu-cloud-pg0.cloudlab.umass.edu" not found
To see the stack trace of this error execute with --v=5 or higher

@leokondrashov can you reproduce this behavior?

Yes, I reproduced this for newest main branch.
Looking into journalctl on worker node, I see that kubelet is constantly failing to restart beacuse there is no such thing as /etc/vhive-cri/vhive-cri.sock which should be created by vHive. But vHive is failing as well:
$ sudo bash -c "./vhive > >(tee -a /tmp/vhive-logs/vhive.stdout) 2> >(tee -a /tmp/vhive-logs/vhive.stderr >&2)"
time="2023-02-21T05:16:26.438443983-06:00" level=info msg="Registering bridges for tap manager"
time="2023-02-21T05:16:26.438603409-06:00" level=panic msg="Bridge could not be created" bridge=br0
panic: (*logrus.Entry) 0xc0003f99d0

goroutine 1 [running]:
github.com/sirupsen/logrus.(*Entry).log(0xc0003f9960, 0x0, {0xc000312ac0, 0x1b})
	/users/lkondras/go/pkg/mod/github.com/sirupsen/logrus@v1.9.0/entry.go:260 +0x47e
github.com/sirupsen/logrus.(*Entry).Log(0xc0003f9960, 0x0, {0xc00035fa28?, 0x604?, 0x0?})
	/users/lkondras/go/pkg/mod/github.com/sirupsen/logrus@v1.9.0/entry.go:304 +0x4f
github.com/sirupsen/logrus.(*Entry).Panic(...)
	/users/lkondras/go/pkg/mod/github.com/sirupsen/logrus@v1.9.0/entry.go:342
github.com/vhive-serverless/vhive/taps.createBridge({0xc0002a3878, 0x3}, {0xc0002a3880, 0xb})
	/users/lkondras/vhive/taps/tapManager.go:90 +0x297
github.com/vhive-serverless/vhive/taps.NewTapManager()
	/users/lkondras/vhive/taps/tapManager.go:72 +0x17d
github.com/vhive-serverless/vhive/misc.NewVMPool(...)
	/users/lkondras/vhive/misc/vm_pool.go:34
github.com/vhive-serverless/vhive/ctriface.NewOrchestrator({0x122a852, 0x9}, {0x0, 0x0}, {0xc00035ff40, 0x5, 0x37?})
	/users/lkondras/vhive/ctriface/orch.go:98 +0x85
main.main()
	/users/lkondras/vhive/vhive.go:137 +0x899


alert user on the frontend when they have been banned and they try sending a message

Hasn't this been fixed?https://github.com/benawad/dogehouse/blob/d220885d8f481107ced3e019ad07467b33c36713/kofta/src/vscode-webview/modules/room-chat/RoomChatInput.tsx#L24

if you refresh the page the frontend store will be gone and you can type messages but nothing will happen

I guess currently the data returned when the user connects to the socket doesn't include the banned users map in order to repopulate bannedUserIdMap in useRoomChatStore

yeah this error would be thrown from the server


patch enable RCall can used in embedded julia

https://github.com/armgong/rcall (r package prototype) is R package embedded julia into R through RCall.jl

Rather than create a global variable in Main, it might be better to try to check directly if R is running. There doesn't seem to be an API to querythis, but based on this discussion:
https://stat.ethz.ch/pipermail/r-devel/2015-May/071069.html
you could try
R_inited = unsafe_load(cglobal((:R_NilValue,libR),Ptr{Void})) != C_NULL

thanks,  i will try this tomorrow, and resummit patch.

btw, after this pull request , rjulia can call RCall.jl no need modify,so I delete https://github.com/armgong/rcall

Nice! I'm super excited about having a 2-way interface.


Added tf vars to cloudbuild plan

/gcbrun

Please run the terraform fmt command against the /terraform/cluster_build directory and then resubmit the Pull Request. It is required that terraform fmt be executed against all Terraform configuration files in order to provide consistency across the repo.

/gcbrun

An error was detected executing terraform plan against the /terraform/security directory. Please run terraform plan locally against that directory to resolve any errors and then resubmit the Pull Request.

/gcbrun

An error was detected executing terraform init against the /terraform/security directory. Please run terraform init locally against that directory to resolve any errors and then resubmit the Pull Request.

/gcbrun

An error was detected executing terraform init against the /terraform/security directory. Please run terraform init locally against that directory to resolve any errors and then resubmit the Pull Request.


librade V2 feature requests

A place to jot down notes for features we are considering adding to V2 of the RADE API.  We'll triage these before the coding starts - they may not all make it into V2.

text channel.  Required for FreeDV reporter. There is currently 25 bit/s available for auxiliary data, this is being used for acquisition in V1.  In particular it enables us to trap any false sync states (e.g. a bad freq offset). Allocating some of these bits for txt will affect the robustness of acquisition, which has been a problematic and time consuming area of development.  It will need to be re-tested in simulation and perhaps OTA with a short test campaign to make sure robustness is not affected. We also need build API support, and add some sort of protocol e.g. a high rate (LDPC?) code to mop up errors, a CRC, and some framing (e.g. codec2 reliable txt system).  However this is all DSP work, no ML re-training rqd.

SNR estimation.  This is a tricky, and we haven't found a good solution to date.  In legacy algorithms we measured the variance in the scatter diagram dots based on their expected positions. For RADAE the expected positions are time varying.  We do have a "poor mans" DSP algorithm that works from the pilot symbols - it gives good results for AWGN but is inaccurate for multipath channels.

API Doxygen support.  Add comments to code, Doxygen post processing.

C port of core ML - will speed up RADAE enc/dec, greatly reducing CPU (FARGAN decoder will then dominate), and getting us closer to a pure C implementation.

Cython for dsp.py - will significantly reduce CPU load, and (I think) gives us C code to move us closer to a pure C implementation.

Further ML development - we may be able to get several dB lower, reduce latency, and improve acquisition. R&D required.

Limit 99% power bandwidth to approximately 1500 Hz. Needs to be done carefully to avoid PAPR reduction and carefully tested to ensure no performance degradation. All ctests need to be run with BPF signal, and effort to tweak tests for reliable passes. Initial investigation in #30

doc/radae_intro_waveform suggestions: table comparing other waveforms, e.g. legacy FreeDV, break down enc/dec to include ML and DSP components, figure showing time versus freq, pilots, symbol breakdown to CP for OFDM waveform. References.

Clean up of radae repo, or creation on new "release" repo specifically for production quality code, support of target operating systems.  The radae repo is Davids experimental playground (and we probably need such a repo moving fwd).

I finally got RADE working on my MacBook Pro in freedv-gui. A few things that I noticed so far:

Apparently I need to use NumPy 1.x, not 2.x. This is different from Windows for some unknown reason. Is there a reason why I shouldn't try to make Windows use 1.x as well (or spend additional time making 2.x work on macOS)? Should this be detailed in the documentation if it's not already there?
freedv-gui is happy to use every single core available on my laptop. This makes the rest of the system less responsive. Is there some number of threads that will work reliably for most users that we can pass to torch.set_num_threads() during initialization? (On that note, because of this behavior I can't use RADE on my M1 Mac Mini for this reason since it makes SmartSDR unresponsive, thus no audio being passed in from the radio.)
It looks like I need to apply approximately -4 dB gain to the TX audio or else it's just 3 kHz of noise (at least according to the TX waterfall in SmartSDR). Not sure if it's a configuration issue on my end or a potential RADE bug.

Anyway, I'm going to fix the bugs Walter et al reported and see if they have additional feedback too.

I'd be inclined to stabilise Windows first them come back to this.  I've nervous about a focus on Mac-specific issues when the # users is so small and we are so close to a working Windows version.

I'd be inclined to stabilise Windows first them come back to this. I've nervous about a focus on Mac-specific issues when the # users is so small and we are so close to a working Windows version.

I could see it causing issues on Windows machines as well depending on the machine. We wouldn't need to make this user-configurable, either; maybe something like this when rade_initialize() gets called:
cur_num_cores = torch.get_num_threads()
new_num_cores = cur_num_cores * 3 // 4
if new_num_cores < 1:
    new_num_cores = 1
torch.set_num_threads(new_num_cores)

That all said, we could wait and see how the first few released builds work out before doing such a change.

Suspect this is more a symptom of an overloaded system.  A modern OS shouldn't need manual control of cores to maintain smooth operation.  Suggest we're better off profiling and then addressing total CPU load as we have planned with a C port of selected parts of the Python code as we discussed at PLT.
We'll also need to address end user expectations - not everyone will be able to run the first release.  This should improve over the next few months while we progress optimisation.

Apparently allowing OpenBLAS free reign over the number of threads it can use wreaks havoc if your application also uses threads: https://github.com/OpenMathLib/OpenBLAS/blob/develop/USAGE.md. With this in mind, I updated freedv-gui to force OPENBLAS_NUM_THREADS to 1 and that significantly reduced CPU usage, especially on TX. (This on both macOS and Windows per testing from Walter and others.) That change doesn't appear to have affected the ability to decode in real time, ether.
Anyway, no need to make changes on the library side. Just figured I'd put that info out there since it did seem kinda odd that TX was using so much CPU despite supposedly not per the RADE paper.

Another item to add to the list above: improved error handling in the API (currently Python exceptions cause the application to terminate, for example).

Can you edit the list directly and add it?  More efficient that adding a comment.  We'll triage the list at PLT level at some point.


Add files via upload

Please provide a description. I can't tell what the intended purpose of this pull request is.


Pascal Triangle (Leetcode)

I would like to add this question in C++ language.

Can I work on this issue?

Can I provide my code in c under Lgmsoc'21

Can I try in python under lgmsoc ?

I would like to code this problem in Java. Can you please assign this issue to me?

@harshita9621 mention the language

@harshita9621 mention the language

@Amit366 In C++

@harshita9621 it is already assigned in cpp

@Amit366 In C

Can I work on this using C.

Can I work on this issue in Java?

@Amit-45 Pascal triangle get updated Kindly review.


SouthEastcon cancelled.

From the Official facebook page

Due to the lack of registrations filled out by a majority of schools, the hardware competition has been cancelled for this year's SoutheastCon. We apologize for any inconvenience this may have caused.

Oh well, we had a good run everyone, we'll get them next year. I've already begun scrapping the bot in preparation for the next one.

roflmao

You've all probably figured it out by now, but this was just an April fools day joke.


Release version 0.29.1

I've created the branch 0.29.x with a commit bumping the version and an entry in changelog, it can be used for release.

Any reason https://github.com/rust-bitcoin/rust-bitcoin/pull/1177/commits/94eeabfd6c3de9102a3ba05ac2853683bb8492ad is not on the branch before the release commit?

Because I made it when too tired. It's fixed now. I can change to a different branch if that's desired, I wanted it out ASAP.

Looks good! Tagged, signed and published. Thanks for doing the fiddly branch work.

I don't see 0.29 yanked. Did you forget or is crates.io messed up?

I just forgot. Should be yanked now.


Fix SASS syntax errors from #967

This introduced syntax errors that were not picked up in review, the
:not() psuedo selector can't lead a selector in SASS.
Based on top of #1314, merge that first.

Annnd these CI failures are from the broken Jinja release, I won't fix those in this PR. This PR was only addressing the ci/circleci:build failures.

FYI, this was already on my radar :) https://github.com/readthedocs/sphinx_rtd_theme/pull/1312/commits/5d8fbbaf65a2ac15bd4870fb50a73c6289f26a1d

Sure, but any reason that needs to be in such a large PR? Seems we could merge a few of these and isolate your PR to add a docker development environment.

Sure, but any reason that needs to be in such a large PR?

Nothing more or less than builds not working, so kind of hard as a new contributor to reach the bar of :heavy_check_mark:


tracing(consensus): CL call handlers

Add more logs to CL call handlers

Codecov Report

Merging #1963 (4f880b5) into main (c5f197d) will decrease coverage by 55.69%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##             main    #1963       +/-   ##
===========================================
- Coverage   73.04%   17.35%   -55.69%
===========================================
  Files         424      424
  Lines       51931    51937        +6
===========================================
- Hits        37931     9012    -28919
- Misses      14000    42925    +28925

Flag
Coverage Δ

integration-tests
17.35% <0.00%> (-2.14%)
:arrow_down:

unit-tests
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

crates/consensus/beacon/src/engine/mod.rs
0.00% <0.00%> (-96.88%)
:arrow_down:

... and 304 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[BUG] - Modal scroll is not blocked in mobile Safari

Describe the bug
Modal scroll is not blocked in mobile Safari.
overflow: auto; is a frequently used way, but I think it is better to block it with js.
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue

Open modal in safari
Try scroll outside
Scroll is not blocked

Expected behavior
Scroll need to be blocked
Screenshots or Videos
https://user-images.githubusercontent.com/20078201/175798823-9dc8452c-889f-4acc-97c7-6effb012e6cd.mp4
Operating System Version
iPadOS
Browser
Safari

I was only able to find a library that handled this edge case: React Spectrum.
I don't know if @jrgarciadev would allow a refactor of the component to use useDialog from react-aria instead.

Hey guys, the migration of this component to react-aria is in progress, it should be available in the next release v10, it should fix this issue

@jrgarciadev
It's built-in tool of Safari (https://webkit.org/blog/8993/cpu-timeline-in-web-inspector/)
You can find timelines tab in Safari Web Inspector!


hoisting of functions does not work properly

"use strict";
console.log(f());

function f(){
    return 1
}

gives me an error. In the node / browser environment it outputs "1" correctly. See http://adripofjavascript.com/blog/drips/variable-and-function-hoisting for explanation.

Should be fixed, I switched from Traceur to Babel.


Param INAV_ENABLED not in use after upload

The param INAV_ENABLED does not list as 'in use' when first flashing firmware.
Only after setting it through serial does it register as 'in use' and show up in QGC

/cc @tumbili

@sanderux Will close this since it was indeed not used for VTOL.


No cursor and missalignment with zsh and Meslo LG S for Powerline font

This is how I recorded and rendered the recording:
% node --version
v10.17.0
% terminalizer record -c config.yml myrec
The recording session is started
Press CTRL+D to exit and save the recording
...
% terminalizer render myrec
Rendering frame 6/6 100% [==============================] 0.0s
Merging frame 6/6 100% [==============================] 0.0s

Successfully Rendered
...

Here's the generated myrec.yml:
config:
  command: zsh
  cwd: /path/to/home
  env:
    recording: true
  cols: 87
  rows: 4
  repeat: 0
  quality: 100
  frameDelay: auto
  maxIdleTime: 2000
  frameBox:
    type: solid
    title: null
    style:
      border: 0px black solid
      boxShadow: none
      margin: 2px 5px
  watermark:
    imagePath: null
    style:
      position: absolute
      right: 15px
      bottom: 15px
      width: 100px
      opacity: 0.9
  cursorStyle: block
  fontFamily: "Meslo LG S for Powerline"
  fontSize: 14
  lineHeight: 1
  letterSpacing: 0
  theme:
    background: "#000000"
    foreground: "#c7c7c7"
    cursor: "#c7c7c7"
    black: "#000000"
    red: "#c91b00"
    green: "#00c200"
    yellow: "#c7c400"
    blue: "#0225c7"
    magenta: "#c930c7"
    cyan: "#00c5c7"
    white: "#c7c7c7"
    brightBlack: "#676767"
    brightRed: "#ff6d67"
    brightGreen: "#5ff967"
    brightYellow: "#fefb67"
    brightBlue: "#6871ff"
    brightMagenta: "#ff76ff"
    brightCyan: "#5ffdff"
    brightWhite: "#feffff"
records:
  - delay: 1166
    content: "\e]1337;RemoteHost=user@host.local\a\e]1337;CurrentDir=/path/to/home\a\e]1337;ShellIntegrationVersion=6;shell=zsh\a"
  - delay: 368
    content: "\e[1m\e[7m%\e[27m\e[1m\e[0m                                                                                \r \r"
  - delay: 35
    content: "\e]133;D;0\a\e]1337;RemoteHost=user@host.local\a\e]1337;CurrentDir=/path/to/home\a"
  - delay: 10
    content: "\r\e[0m\e[27m\e[24m\e[J\e]133;A\a\e[7m\e[34m ~ \e[27m\e[34m\e[49m\e[34m\e[39m \e]133;B\a\e[K\e[?1h\e=\e[?2004h"
  - delay: 733
    content: "\e[32mexit\e[39m\b\e[32mt\e[27m\e[39m"
  - delay: 121
    content: "\b\e[32mt\e[39m\e[?1l\e>\e[?2004l\r\r\n\e]133;C;\a"

And this is the render result:

This is what I was expecting, in terms of margins, alignment of the line with the standout mode and the Powerline glyph, and having a cursor (static image):

(Except for the font rendering that looks way much better with terminalizer)  :- )

Try changing the background to "transparent"


Eating a pure fruit sometimes kicks the player

For some reason, fabric claims that a StatusEffect contains its type in the StatusEffect::effectType member, but when run on the server, the actual field is StatusEffect::type.
I need to see why this discrepancy lies.

Fixed in v0.0.2.


Change hash function to SHA-2-256

As per #83.

Don't think there are any. Made sure to abstract the choice of hashing function to just this section 😂


DFI transfer from pool to wallet lost

After updating from 2.1.3 to 2.1.4 backup wallet could not be imported (no access to the button "import wallet" - just in grey colour).
So I restored my wallet by the mnemonic seed. Now the transfer (Txid:
bd9238a47b75995a24eb3ba69387e2bc75140a65d953647c99ba10d2e34d92f7) of 845 DFI is not shown in the wallet.
I tried to put in the old receiving address, but I get the message "Address already exists", I am sure it is mine but I cannot confirm in that app.
How do I get the 844 DFI in my wallet now?
Thanks for your help.
Regards Dirk

Hi @DeFiDirk , thanks for your feedback & screenshot. Could you check your balance with the following commands:
getbalance — utxo DFI

getokenbalances — DFI
The sum of utxo DFI & DFI would be your actual DFI.

Hi @DeFiDirk , I'm unable to see anything for txid bd9238a47b75995a24eb3ba69387e2bc75140a65d953647c99ba10d2e34d92f7, could you double check. You may also try the reindex option in Settings.

In the mean time, update v2.1.6 is out https://defichain.com/downloads/.
Do give a thumbs up to this comment if your issue has been resolved, otherwise, you may reopen this issue with new information (include screenshots) and your app logs. Thank you.

Hi @DeFiDirk , I'm unable to see anything for txid bd9238a47b75995a24eb3ba69387e2bc75140a65d953647c99ba10d2e34d92f7, could you double check. You may also try the reindex option in Settings.

In the mean time, update v2.1.6 is out https://defichain.com/downloads/.
Do give a thumbs up to this comment if your issue has been resolved, otherwise, you may reopen this issue with new information (include screenshots) and your app logs. Thank you.


Get nothing output and has no error message with the same workflow as your example

I use the workflow same as your example but get the nothing output without any error message. Could you help me to find out the cause of the problem?

Are you on Mac by any chance?

yes, I use the mac with apple silicon and the OS version 13.2.1, the python version is 3.12

See if updating to the most recent AnimateDiff-Evolved fixes your issue. The black images are due to a Mac pytorch bug (and only happen with v1 motion models), but in the latest update I've made it automatically turn on the workaround that should fix the issue without having to manually edit code.

See if updating to the most recent AnimateDiff-Evolved fixes your issue. The black images are due to a Mac pytorch bug (and only happen with v1 motion models), but in the latest update I've made it automatically turn on the workaround that should fix the issue without having to manually edit code.

wow! My problem was solved! Thanks very much!

Hey, having the same problem. I am Mac based and I am running latest AnimateDiff-Evolved. Previously I fixed the bug by changing something in Node.py but I think you managed to bypass that in the later versions of AnimateDiff-Evolved. Since updating to the new IPAdapter Advanced, I am running into all sorts of issues but I am not sure what is causing the black image output now. Any thoughts? Thanks


CAMEL-14301: Stomp Component has no option to set custom headers

The org.fusesource.stomp.client.Stomp class has setter method for custom headers, the Camel-Stomp has not exposed that option to be set by the users.
The StompEndpoint and StompConfiguration classes need changes to allow users to set custom headers.

How does this unit test use the custom headers? The test seems more for testing header filter strategy.

Hi @davsclaus
The unit test was for my previous PR. During that time you had asked for Unit test for header filter strategy.
Agree that it is causing confusion.

Should i Split this into two PRs?

Yeah if you have a PR with the header filter unit test. And then another for the new feature

The header filter PR is closed. Thats fine. I will create a new PR for test. And then a separate PR for new feature.
Thank you for your comments

Hi @davsclaus
Incorporated the comments. Please Check

Merged on master. Thanks.


CoCreateInstance returns REGDB_E_CLASSNOTREG

Hi Team,
I'm trying to integrate SDK in my project using CMake. When trying to execute simple program I have a crash in XMLHTTPRequestLoader.cpp. The function CoCreateInstance cannot initialize pRequest. The output of the function is REGDB_E_CLASSNOTREG. I can't find an answer why CLSID_FreeThreadedXMLHTTP60 is not found.  The code starts at  Line 298:
CoCreateInstance(CLSID_FreeThreadedXMLHTTP60,
                             NULL,
                             CLSCTX_INPROC_SERVER,
                             IID_PPV_ARGS(&pRequest));

I 'v also tried to execute your gtest and got the same crash.
Windows 7, VS2015.

Hi rhard,
That is very strange. I periodically run those tests locally, and we have a build server running them nightly. It could be your visual studio install is corrupted. Can you try repairing it? If repair doesn't work, maybe try uninstalling and reinstalling VS completely?
Another thought, are you trying to build in 64 bits of 32 bits? Just occurred to me we default our builds to 32 bits.

Hi David,
just tested with Win10 and VS2017. Everything is OK. However, if I move EXE to Windows 7 and run it, it still crashes. I think there are compatibility problems with older Windows versions. Maybe there is some build definition I need to set? Here is my CMakeLists.txt. I tried to add _WIN32_WINNT=0x600 but it doesn't help.
cmake_minimum_required(VERSION 3.10)
project(BrainCloudTest)

set(CMAKE_CXX_STANDARD 11)

add_definitions(-D_WIN32_WINNT=0x600)

add_subdirectory(braincloud-cpp)

add_executable(BrainCloudTest main.cpp CloudWrapper.cpp CloudWrapper.h)
target_link_libraries(BrainCloudTest brainCloud)

P.S. I tried both x32 and x64 builds.

Hum right I missed the point about windows 7.. that HTTP code might be windows 8+ only.
You might have to use Curl + OpenSSL for that.
The good news is, we have a build config for CURL because that's what we use on Linux. But the bad news that might be tricky to get working on windows because of openssl. You can modifying the cmake file to use CURL instead? You will need to install OpenSSL manually on windows and point to it. That's going to be the painful part.

I think I found why: https://docs.microsoft.com/en-us/windows/desktop/api/msxml6/nn-msxml6-ixmlhttprequest2
IXMLHTTPRequest2

`

Ya... We're going to look into that.. Windows7 is still relevant for gamers. CURL will be the alternative here. You can try our CURL/OpenSSL setup for now and try to make that work. It will take a bit before we get to this. I'll log it now.

Thanks, I will try curl and let you know how it builds on Win.
BTW: Maybe you can add the check of CoCreateInstance return value with some error output. It will save some debugging time.

Agreed, will update the JIRA

Closing this, as it's been marked in our internal JIRA system and will be dealt with in time.

Could you advise me please how to enable CURL on windows? Do I need to disable USE_LIBWEBSOCKETS?
It already has SSL implementation but CURL uses OpenSSL. If I define USE_CURL 1 I have many SSL redefinitions from mbedtls.

hum ya so you can use SSL instead of mbedtls if you go Curl.
comment out that part in the cmake:
    # TLS
    option(ENABLE_TESTING "" OFF)
    option(ENABLE_PROGRAMS "" OFF)
    add_subdirectory(./lib/mbedtls/)
    list(APPEND libs PUBLIC mbedtls mbedx509 mbedcrypto)
    list(APPEND includes
        PUBLIC ${CMAKE_CURRENT_BINARY_DIR}/lib/mbedtls/include/)
    option(LWS_WITH_MBEDTLS "" ON)
    set(LWS_MBEDTLS_INCLUDE_DIRS
        "${CMAKE_CURRENT_BINARY_DIR}/lib/mbedtls/include/")
    if (UNIX AND NOT CMAKE_GENERATOR STREQUAL Xcode)
        set(LWS_MBEDTLS_LIBRARIES
            "${CMAKE_CURRENT_BINARY_DIR}/lib/mbedtls/library/libmbedtls.a"
            "${CMAKE_CURRENT_BINARY_DIR}/lib/mbedtls/library/libmbedcrypto.a"
            "${CMAKE_CURRENT_BINARY_DIR}/lib/mbedtls/library/libmbedx509.a")
    else()
        set(LWS_MBEDTLS_LIBRARIES
            "${CMAKE_CURRENT_BINARY_DIR}/lib/mbedtls/library/")
    endif()

And LWS should pick up the OpenSSL you installed to have curl working.

Thanks! I got it working.

Great news!
Do you mind sharing the cmake changes? I will put them in the ticket as a reference.

I updated libwebsockets reference to my own fork so I can't create pull request directly (there was some issues with OpenSSL Cmake entries). But I just added CURL check in the lines you mentioned above:
if (NOT USE_CURL)
    # TLS
    option(ENABLE_TESTING "" OFF)
    option(ENABLE_PROGRAMS "" OFF)
    if (NOT USE_CURL)
        add_subdirectory(./lib/mbedtls/)
        list(APPEND libs PUBLIC mbedtls mbedx509 mbedcrypto)
        list(APPEND includes
            PUBLIC ${CMAKE_CURRENT_BINARY_DIR}/lib/mbedtls/include/)
        option(LWS_WITH_MBEDTLS "" OFF)
        set(LWS_MBEDTLS_INCLUDE_DIRS
            "${CMAKE_CURRENT_BINARY_DIR}/lib/mbedtls/include/")
        if (UNIX AND NOT CMAKE_GENERATOR STREQUAL Xcode)
            set(LWS_MBEDTLS_LIBRARIES
                "${CMAKE_CURRENT_BINARY_DIR}/lib/mbedtls/library/libmbedtls.a"
                "${CMAKE_CURRENT_BINARY_DIR}/lib/mbedtls/library/libmbedcrypto.a"
                "${CMAKE_CURRENT_BINARY_DIR}/lib/mbedtls/library/libmbedx509.a")
        else()
            set(LWS_MBEDTLS_LIBRARIES
                "${CMAKE_CURRENT_BINARY_DIR}/lib/mbedtls/library/")
        endif()
    endif()
    # LibWebSockets

Update: the condition must be:
if (NOT USE_CURL OR NOT WIN32)

otherwise it will not build on Linux.


example should guard against null string pointer

I was trying out the "hello world" example in rlbox, and wanted to confirm that it guards against the library returning garbage.  When I changed call_cb() in mylib.c to invoke cb(NULL), running hello crashed with a segfault.
As far as I can tell, the example doesn't seem to properly guard against the library function call_cb passing a null string to the cb() callback, in the verifier that hello_cb() passes to copy_and_verify_string.
It would be good if the example showed what kinds of library mis-behavior the validators guard against (maybe I'm confused about NULL being a misbehavior that rlbox is guarding against?), and/or guard against the library returning a null string pointer.

Yep this was a bad validator (on my part) that didn't check for the nullptr case.
We're working on docs + tutorials (with real libraries) that will hopefully illustrate what you're thinking about.

We're (really @shravanrn) also redesigning some of the APIs (e.g., to avoid the C-like API and have convenient APIs like these) so if you have suggestions, happy to hear them!

Yup, as deian mentioned, we are currently redesigning APIs to encourage C++ style usage patterns (which clarify things like ownership etc.)

maybe I'm confused about NULL being a misbehavior that rlbox is guarding against?

This particular case is interesting in that passing a NULL string can be allowed behavior for some programs. So, we need to be careful with automatic RLBox checking here.
In the API redesign, it would be good to pay closer attention to NULLs. One solution could be to explicitly indicate in APIs whether NULL is ok or not. For example, we want to add a version of malloc_in_sandbox, say  malloc_infallible_in_sandbox (still bikeshedding the name), which does NULL checking on the return. This could allow us to stop some denial of service style attacks (due to null pointer access) in addition to memory safety errors.


Divide and Conquer

Added description for Divide and Conquer section.

Checklist

[x] I've looked at the contribution guidelines.
[x] This pull request is complete and ready for review.

Description

Added brief description for Divide and Conquer section.

I like it, if for no other reason than that it looks complete now. I'm going to merge this one. Thanks for contributing!


Compliance with Groovy 2.5's Standard Mechanism for Specifying Extension Modules

Expected Behavior

The Gradle build tool is expected to comply with the standard mechanisms that are specified for correctly configuring the external dependencies it uses. In the context of this issue „compliance“ and „correctly configuring“ means the configuration specified by the upstream external projects that produce the external libraries that the Gradle build tool depends on.
Current Behavior

In Gradle version 5.6.2, the «$GRADLE_USER_HOME»/caches/5.6.2/generated-gradle-jars/gradle-api-5.6.2.jar file contains a service configuration file that — according to Groovy maintainer @blackdrag — does not comply with the Groovy project's most up-to-date standard mechanism for configuring Groovy Extension Modules. The same non-compliant configuration can currently be found in Gradle's version 6+ early access releases too.
Context

As a developer of a library that will be distributed as a JPMS module, I am trying to accomplish unit testing of the classes of my JPMS-modularized library.
During the development and unit testing of my library, I was affected by the JPMS development hurdle reported in Gradle issues #11027 and #10408.
The specific Groovy project-specified standard mechanism that Gradle is not in compliance with, is prescribed in Groovy Issue #8480...

org.codehaus.groovy.runtime.ExtensionModule should move to META-INF/groovy
...
Description
„To avoid confusing Java 9's ModuleFinder. We should fall back to the current location to handle existing jars...“
...

I was made aware of the above issue by Groovy maintainer, @blackdrag, in a jigsaw-dev mailing list discussion...

...
„...No idea what gradle uses here, but the standard mechanism uses now
META-INF/groovy/org.codehaus.groovy.runtime.ExtensionModule
(https://issues.apache.org/jira/browse/GROOVY-8480). So I assume Gradle
is there still on Groovy 2.4...“
...

As far as I have been able to determine, Gradle version 5.6+ uses Groovy version 2.5+ by default:
...
findLatest("groovy", "org.codehaus.groovy:groovy:(2.5,)", versionProperties)
...

GROOVY-8480 was resolved in March, 2018. So I would have expected Gradle to have been in compliance with Groovy's standard mechanisms for that version by now.
The Problem
TL;DR — Even if the hurdle reported in issues #11027 and #10408 is successfully gotten around, «$GRADLE_USER_HOME»/caches/5.6.2/generated-gradle-jars/gradle-api-5.6.2.jar's malformed META-INF/services/org.codehaus.groovy.runtime.ExtensionModule service configuration file entry means any expectation of using that particular Gradle library as an automatic module is still „out of the question“.
A more detailed discussion of the problem can be found in this jigsaw-dev mailing list thread.
Steps to Reproduce

Download and unpack this zip-archived project
In your computer's file explorer, navigate to «$GRADLE_USER_HOME»/caches/5.6.2/generated-gradle-jars/gradle-api-5.6.2.jar
Open the JAR file in an archive management tool
• Do not extract the contents. Actually open the JAR so that you can navigate it as if it were another file system folder
Navigate to the META-INF/services entry of the JAR
Extract the META-INF/services/java.security.Provider service configuration file entry to a folder on your hard drive
Edit the extracted java.security.Provider file so that it contains the single line shown below:

org.gradle.internal.impldep.org.bouncycastle.jce.provider.BouncyCastleProvider

Replace the original, incorrect two-line, META-INF/services/java.security.Provider service configuration file entry in the JAR file, with the corrected one-line one you just edited in Step 6.

In a terminal opened in the root folder of the project downloaded and unpacked in Step 1, run the :test task:

./gradlew :test

Observe the error printed to stdout:

...
...[QUIET] [system.out] Error occurred during initialization of boot layer
...[QUIET] [system.out] java.lang.module.FindException: Unable to derive module descriptor for «$GRADLE_USER_HOME»/caches/5.6.2/generated-gradle-jars/gradle-api-5.6.2.jar
...[QUIET] [system.out] Caused by: java.lang.module.InvalidModuleDescriptorException: Provider class moduleName=model-core not in module
...

Your Environment

------------------------------------------------------------
Gradle 5.6.2
------------------------------------------------------------

Build time:   2019-09-05 16:13:54 UTC
Revision:     55a5e53d855db8fc7b0e494412fc624051a8e781

Kotlin:       1.3.41
Groovy:       2.5.4
Ant:          Apache Ant(TM) version 1.9.14 compiled on March 12 2019
JVM:          12 (Oracle Corporation 12+33)
OS:           Windows amd64

While the fact that Gradle isn't up to date with the location of the groovy extension module file location is correct, this is completely orthogonal to JPMS. Groovy modules have nothing to do with java modules.

Hi @melix,
Thanks for replying.

„...the groovy extension module file location...is completely orthogonal to JPMS...“

And so it should be, melix. Just like the operation of the brakes on the handlebars of my road bike should be orthogonal to the operation of the gearing mechanism on those same handlebars. But if I intentionally Jerry-rig one up to use the cables and levers of the other, there is bound to be trouble at some point down the road.

„...Groovy modules have nothing to do with java modules...“

I agree! That is the reason GROOVY-8480 was raised, after all. Right?
It is true that the goals and intentions of Groovy modules have nothing to do with the goals and intentions of Java modules. But GROOVY-8480 says this...

„...To avoid confusing Java 9's ModuleFinder...“

That issue description acknowledges that Groovy's Extension Modules' co-opting of Java's service provider-configuration file, inappropriately Jerry-rigs the cables and levers of the two together. While such inappropriate coupling might be a benefit to Groovy modules, it breaks Java modules. For that reason, 8480 is advising that using META-INF/services for Groovy modules is the wrong thing to do.
GROOVY-8480 is officially acknowledging, that since the two have nothing to do with one another, it is, therefore, inappropriate and incorrect for Groovy modules to misuse the Java modules' service provider-configuration file. 8480 advises that the correct mechanism from Groovy 2.5 onward is META-INF/groovy/.... That is: META-INF/services/... has been deprecated for Groovy 2.5+. Continuing to use it is a recipe for trouble down the road.
This is where I'm coming from as a humble user, melix...

You know Groovy 2.5+ has META-INF/groovy for its extension modules
You know Gradle 5.6+ uses Groovy 2.5+
You know that the use of META-INF/services is deprecated because it was the wrong thing to do
You know that there is a more up-to-date standard mechanism available
You know the up-to-date standard mechanism is officially advised to be used
You know that using the deprecated mechanism breaks Java modules

So that raises three simple questions:

Why continue to do the wrong thing when you know it breaks Java modules?
What is the reason that not using Groovy's more up-to-date standard mechanism is so essentially required for 5.6+?
Will Gradle ever update its use of Groovy extension modules to use the more up-to-date standard mechanism?
• If so, when?
• If no, why not?

From my perspective as a mere user, it seems to me that Gradle could have simply and easily upgraded to the more up-to-date standard mechanism at anytime since March 2018, and it wouldn't negatively impact Gradle's use of Groovy extension modules. Could you not? That would be an absolute win/win!
Besides, Gradle itself has its own opinions, conventions and standard mechanisms that it is always strongly advising users to comply with. It is also always deprecating things and advising users to upgrade A.S.A.P. No? So wouldn't Gradle want to, at least, be seen to be practicing what it preaches?

Why continue to do the wrong thing when you know it breaks Java modules?

Did I say we shouldn't fix? Where?

What is the reason that not using Groovy's more up-to-date standard mechanism is so essentially required for 5.6+?

Because this was unnoticed because nobody complained so far, as simple as that.

Will Gradle ever update its use of Groovy extension modules to use the more up-to-date standard mechanism?
• If so, when?

PRs accepted, otherwise as soon as we have bandwidth.

Hey thanks for the quick reply, @melix :+1:

Why continue to do the wrong thing when you know it breaks Java modules?

„Did I say we shouldn't fix? Where?“

You're right, melix. You never said that. Sorry if it came off like I was implying that.

What is the reason that not using Groovy's more up-to-date standard mechanism is so essentially required for 5.6+?

„Because this was unnoticed because nobody complained so far, as simple as that.“

That makes sense. Thanks, melix.

Will Gradle ever update its use of Groovy extension modules to use the more up-to-date standard mechanism?
• If so, when?

„PRs accepted, otherwise as soon as we have bandwidth.“

Hey now! That's the best news I've heard all year! :1st_place_medal:
I did consider fixing this and #11027 myself. The other day I repeatedly searched the current remote github.com/gradle/gradle code. When I couldn't find anything there, I also grepped the hell out of my local clone of 5.6.1 I've had on my machine for a few months. But I couldn't find neither the META-INF/services/java.security.Policy file, the org.codehaus.groovy.runtime.ExtensionModule one, nor any reference or mention of either one of them anywhere in either the remote or local code base.
Can you share any pointers, melix, to where people with bandwidth should look to get started working on something like that? TIA :+1:

The other day I repeatedly searched the current remote github.com/gradle/gradle code...

These are a few of the things I bookmarked from my searching the other day...

GradleImplDepsGenerationIntegrationTest.groovy
BaseGradleImplDepsIntegrationTest.groovy
DefaultGeneratedGradleJarCache.java
DefaultCacheRepository.java
CacheBuilder.java

... before I gave up since none of the above references neither of those JAR file service provider-configuration entries.

Look at https://github.com/gradle/gradle/blob/bd45f09e710845b1dca72d57e3048f334412b2e7/subprojects/model-core/src/main/resources/META-INF/services/org.codehaus.groovy.runtime.ExtensionModule
which is the extension module file reported in the original thread on jigsaw-dev.
There are, as far as I can tell, at least 5 of those files in gradle/gradle. It should be as easy to fix as moving them to the right location.

Closing as fixed


could not create haproxy in 3.5.4

Hi, Aidhao, thank you for your greate work, I have seen the demo video on Youtube, but after following your guide in the video: create group, create user, create ssh credential, create server... all of works above have been done,  and installed haproxy which version is 1.5.18, but I can not see the server in the overview page, clicking  the refresh button but the same result, then I want to create a haproxy in the haproxy menu, but I have not a server to choose, where is the server?   I see another server menu in the haproxy menu, how is the difference between this server and the server i have created in the admin area menu?
waiting for your help, thanks

I can't help you without logs and screens
And if you have so many problems at starts how about support? I mean donate and I will do everything

I'd like to donate this project, but I have no paypal account and US$ because I live in China. Maybe I will donate this project if you have a wechat account Alipay account one day.
I am very glad because I have solved this problem because I did not enable this server, now this project is working perfectly, I will also contribute my code to this project.
Thank you very much!

You are welcome)
Отправлено из Mail.ru для Android вторник, 15 октября 2019г., 09:03 +03:00 от genuiner<EMAIL_ADDRESS>:

I'd like to donate this project, but I have no paypal account and US$ because I live in China. Maybe I will donate this project if you have a wechat account Alipay account one day.
I am very glad because I have solved this problem because I did not enable this server, now this project is working perfectly, I will also contribute my code to this project.
Thank you very much!
—
You are receiving this because you commented.
Reply to this email directly,  view it on GitHub , or  unsubscribe .

Can Alipay or WeChat Pay be accessed, otherwise users in China cannot donate  @Aidaho12

Unfortunately no, probably bank transfer? Try to write to<EMAIL_ADDRESS>


Crash in VM after 9b7b91ee662b

The tests

language_2/deep_nesting1_negative_test
language/deep_nesting1_negative_test

crash on ReleaseIA32 on the vm-linux-release-ia32-asan-be bot.
See: log and log.
I'll land status updates as Crash, Pass to avoid figuring out exactly which targets are affected.

These tests are now passing. No marks in status files. Assuming stale issue.


SIP Calculator

Calculator Functions
This Calculator will calculate the Maturity amount after Investing Period

Contribution Event
GSSOC22
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


Show tab identifier in tabline when tabline#show_tabs = 0 is set

This is a follow-up from issue #1108 and issue #639.
People seem very split about whether ALL buffers should be displayed even when there is more than one tab open. If I'm reading the previous issues correctly, a solution that implemented something like:
 | buffer 1 > buffer 2 > buffer 3                  tab 1 < tab 2 < tab 3 |

was implemented, and then backed out. The rationale being there wasn't enough room on the tab line. So now the only way to always see buffers, is to set let g:airline#extensions#tabline#show_tabs = 0. But then it is hard to keep track of open tabs.
When the above config is set, I'd love if there could be a small tab identifier on the right hand side, something like:
| buffer 1 > buffer 2 > buffer 3                      < buffers < tab 2/4 |

This would simultaneously not take up too much space and probably solve the use cases for all the people +1'ing the other two issues. Note, I'm only proposing this happens if show_tabs = 0 is set (or if a new config is created and set).

could we have buffers at the top and tabs at the bottom?

Not possible.

please check #1358


KAFKA-4251: fix test driver not launching in Vagrant 1.8.6

custom ip resolver in test driver makes incorrect assumption when calling vm.communicate.execute, causing driver to fail launching with Vagrant 1.8.6, due to https://github.com/mitchellh/vagrant/pull/7676

LGTM


遇到跨域的问题：No 'Access-Control-Allow-Origin' header

Access to XMLHttpRequest at 'http://<IP_ADDRESS>:1985/rtc/v1/whep/?app=live&stream=livestream' from origin 'http://direct.virtaicloud.com:20410' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource.
查看srs服务器跟本项目都添加了cors的设置，但是还是会出现问题

@aiquanpeng 请问您解决了吗


Add SUSTech-data/neopyter

Repo URL:
https://github.com/SUSTech-data/neopyter
Checklist:

[x] The plugin is specifically built for Neovim, or if it's a colorscheme, it supports treesitter syntax.
[x] The lines end with a .. This is to conform to awesome-list linting and requirements.
[x] The title of the pull request is Add/Update/Remove `username/repo`  (notice the backticks around `username/repo`) when adding a new plugin.
[x] The description doesn't mention that it's a Neovim plugin, it's obvious from the rest of the document. No mentions of the word plugin unless it's related to something else.
[x] The description doesn't contain emojis.
[x] Neovim is spelled as Neovim (not nvim, NeoVim or neovim), Vim is spelled as Vim (capitalized), Lua is spelled as Lua (capitalized), Tree-sitter is spelled as Tree-sitter.
[x] Acronyms should be fully capitalized, for example LSP, TS, YAML, etc.

Thanks for the PR!


How to use EslintFixAll of eslint-lsp to reformat code after prettier has formatted it？

plan to add a hook for this situation.

Can you use the plugin's custom configuration to define one of the cmd parameters as EslintFixAll?

you can simply do it like this
ft('ts'):fmt('prettier'):append({ fn = function() vimc.cmd('EslintFixAll') end })

this way should be work right ? close then

Thanks for your reply


Updates on README.md

Updating all the outdated information to catch up with the recent development.

I think it might be time for 0.2.0 after this gets merged, thoughts?

As for the releasing: I have not done tests with real devices for a bit. Waiting for the docker installation from @easybe before doing that work. Not sure how close we are there though? If it still takes a bit, I can do some tests locally tonight (CET).

As for the releasing: I have not done tests with real devices for a bit. Waiting for the docker installation from @easybe before doing that work. Not sure how close we are there though? If it still takes a bit, I can do some tests locally tonight (CET).

Our internal deployment will be updated today.


Generic type issue after migration from 3.2.0 -> 4.0.0

Hi :)
sry, for coming again regarding the beloved generic type use cases. There is a change with the latest release 4.0.0 when deserializing generic types. Or maybe I have overlooked migration instructions. Please find attached a test script running fine with release 3.2.0 but failed with 4.0.0. and the following result:

test_script.zip
Thank you for your support!

Hi. The special '__type' property is only used when decoding using MapperContainer.globals.fromMap and others. Not the SomeClassMapper.from... methods.
So it works when you replace GenericClassMapper.fromMap(entryMap); with MapperContainer.globals.fromMap(entryMap);.

Ah ok, so this functionality of mapping a class SomeClassMapper<T> with SomeClassMapper.fromMap(entryMap); was removed on purpose from 4.0.0? I'm just asking because it did work with the previous release.
I usually try to avoid to use the whole MapperContainer if I already know the expected (generic) type. But if there is no other way, I can switch from the specific mapper to the global container.

It was unintentionally then. Best to just switch.

Closing this as there is nothing todo.


feat: Accessibility Dashboard Created Confirmation Message #1840

Issue: https://github.com/awslabs/iot-application/issues/1840
Fix:
->Use role="status" for status notifications.

LG but we'll have to figure out what's going on with the playwright tests, it's failing with:
Expected string: "Successfully updated dashboard \"My Dashboard\"."
    Received string: "Successfully updated dashboard \"My Dashboard\".Success Successfully updated dashboard \"My Dashboard\"."

can you please merge your 2 commits into one

can you please merge your 2 commits into one

@mnischay done


Add git tasks

inv git.authors: list authors contribute to this project
inv git.changelog: print changelog

Thanks for the reminding. I've already fixed it.

Thanks for fixing the conflicts.BTW I am aware that my IDE (pycharm) seems to use single quotes by default =.=

xD
Maybe we can change the behavior of black by adding -S.
I have no strong preference on using single or double quotes.


Erro in javascript file

Hi.
I´m trying to use this project in my home, but when I play it I´m receiving the following error
TypeError: a.list is undefined.
d/this.getItemSource http://localhost:7878/js/list.min.js:1:9548
d http://localhost:7878/js/list.min.js:1:9098
d http://localhost:7878/js/list.min.js:1:11489
[8]</b.exports http://localhost:7878/js/list.min.js:1:11523
start http://localhost:7878/js/list.min.js:1:1375
o http://localhost:7878/js/list.min.js:1:3448
 http://localhost:7878/search:117:23
I configured the swagger.config.edn just discommenting the "server-list" parameter.
Could you please check this error?
Tnks

Fixed in 0.5.2.
I also noticed that to make the "server-list" to show something you will need to configure also the "possible-swagger-doc-suffix" parameter as [""].
See https://github.com/IG-Group/swagger-search/commit/82d828e7fe1ea036822c1ffc9f365ada4dc6b4e8


Voting attribution for laws

This PR introduces a source for a new type of swedish legal data: Betänkanden. It provides per-paragraph voting information for laws in SFS when that is possible to deduce from the Betänkande, its Propositioner and Motioner. The match for individual laws is done by name among the laws that references the Betänkande as a Förarbete. For now, only paragraphs mentioned in the proposal point of the Betänkande are included, not paragraphs mentioned inside a Proposition or Motion.

Fantastic! I will take a look as soon as time permits (ie not this week but hopefully soon thereafter). Thank you!

This is very much a draft so far - e.g. no output is generated, only download/parse is implemented.
One question:
Betänkanden can be cross-references to laws in SFS, as published. How are those laws "compressed" into actual laws in lagen.nu? E.g. how / where is "lag om ändring i lagen om ändring i lag om ändring i lag om grisar i ollonskog (2001:47) (2002:48) (2003:56)" compressed into just the new version of "lag om grisar i ollonskog (2001:47)", and how are the URI:s for individual paragraphs in those laws related to each other? Currently, the voter info is collected for paragraph URI:s for the modifying law, (2003:56)...
One note:
I had to modify the serialization of XHTML+RDFa to support arbitrary graphs, esp. graphs with blank nodes referring to other blank nodes. Please check that I didn't break stuff too much!

Some more questions:
<https://lagen.nu/2021:743> <http://rinfo.lagrummet.se/ns/2008/11/rinfo/publ#ersatter> <https://lagen.nu/1998:812#K2P4>
<https://lagen.nu/2021:743> <http://www.w3.org/2002/07/owl#sameAs> <http://rinfo.lagrummet.se/publ/sfs/2021:743>

Right now I have attached the vote-related statements this way
<http://rinfo.lagrummet.se/publ/prop/2020/21:188#P3> <http://lagen.nu/vocab/parliament#approve> <http://rinfo.lagrummet.se/publ/sfs/2021:743>
_:N5e6abbdc58ed4e50a5bbcf619ccd91c0 <http://lagen.nu/vocab/parliament#approve <http://rinfo.lagrummet.se/publ/prop/2020/21:188#P3>
_:N5e6abbdc58ed4e50a5bbcf619ccd91c0 <http://purl.org/dc/terms/creator> <http://lagen.nu/vocab/parliament#acclamation>

Is this correct, or should they reference https://lagen.nu/2021:743 instead? (N5e6abbdc58ed4e50a5bbcf619ccd91c0 is an individual decision point in a Betänkande btw).
I'm in general a bit unsure about the design of the RDF schema / structure, of the vocabularies and of the url patterns, so would be glad to get some help in this regard.


P3 Training Fail: ValueError: num_samples should be a positive integer value, but got num_samples=0

Issue Description
Encountered the following error when attempting to train a Precision Level 3 MaskRCNN model using EPD. This error comes after having integrated the .yaml parser within P3Trainer.py.
Traceback (most recent call last):
  File "tools/train_net.py", line 201, in <module>
    main()
  File "tools/train_net.py", line 194, in main
    model = train(cfg, args.local_rank, args.distributed)
  File "tools/train_net.py", line 72, in train
    start_iter=arguments["iteration"],
  File "/home/cardboardvoice/anaconda3/envs/p3_trainer/lib/python3.6/site-packages/maskrcnn_benchmark-0.1-py3.6-linux-x86_64.egg/maskrcnn_benchmark/data/build.py", line 164, in make_data_loader
    sampler = make_data_sampler(dataset, shuffle, is_distributed)
  File "/home/cardboardvoice/anaconda3/envs/p3_trainer/lib/python3.6/site-packages/maskrcnn_benchmark-0.1-py3.6-linux-x86_64.egg/maskrcnn_benchmark/data/build.py", line 64, in make_data_sampler
    sampler = torch.utils.data.sampler.RandomSampler(dataset)
  File "/home/cardboardvoice/anaconda3/envs/p3_trainer/lib/python3.6/site-packages/torch/utils/data/sampler.py", line 94, in __init__
    "value, but got num_samples={}".format(self.num_samples))
ValueError: num_samples should be a positive integer value, but got num_samples=0

Expected Behaviour
The training is supposed to proceed without any errors.
Actual Behaviour
The training fails the aforementioned error in terminal.
Error Source
Currently, the integration of the .yaml parser in P3Trainer.py seems to be the root cause.
[ Update as of 20220812 ]: The integration of the parser is not the root cause. With the EPD v0.2.2 P3 training workflow failing as well. It can be deduced that the cause should be narrowed to unknown dependency conflicts.

Debug Update
Running the training workflow on the ros-industrial EPD v0.2.2 yields the same error message.
loading annotations into memory...
Done (t=0.00s)
creating index...
index created!
2022-08-12 10:59:03,245 maskrcnn_benchmark.utils.miscellaneous INFO: Saving labels mapping into ./weights/custom/labels.json
Traceback (most recent call last):
  File "tools/train_net.py", line 201, in <module>
    main()
  File "tools/train_net.py", line 194, in main
    model = train(cfg, args.local_rank, args.distributed)
  File "tools/train_net.py", line 72, in train
    start_iter=arguments["iteration"],
  File "/home/cardboardvoice/anaconda3/envs/p3_trainer/lib/python3.6/site-packages/maskrcnn_benchmark-0.1-py3.6-linux-x86_64.egg/maskrcnn_benchmark/data/build.py", line 164, in make_data_loader
    sampler = make_data_sampler(dataset, shuffle, is_distributed)
  File "/home/cardboardvoice/anaconda3/envs/p3_trainer/lib/python3.6/site-packages/maskrcnn_benchmark-0.1-py3.6-linux-x86_64.egg/maskrcnn_benchmark/data/build.py", line 64, in make_data_sampler
    sampler = torch.utils.data.sampler.RandomSampler(dataset)
  File "/home/cardboardvoice/anaconda3/envs/p3_trainer/lib/python3.6/site-packages/torch/utils/data/sampler.py", line 94, in __init__
    "value, but got num_samples={}".format(self.num_samples))
ValueError: num_samples should be a positive integer value, but got num_samples=0

Conclusion
The integration of the .yaml parser is not the root cause.

Debug Update
The conclusion from the previous Debug Update is further reinforced by the following test.
Test
Unreleased dockerized training proceeds without the aforementioned error. Adhering to expected behaviour.
Conclusion
The cause is due to yet another hidden dependency conflict which is prevented once dockerized.

This issue is resolved with https://github.com/ros-industrial/easy_perception_deployment/pull/56. Closing.


Add comment about menu item "context"

I am working on integrating into the Azure Repos view with a menu contribution point. I struggled for a while to figure out how to retrieve the currently selected branch from the main page. Ended up figuring it out and wanted to contribute back with a comment describing that the context parameter can have additional information about the original page the menu was accessed from
(This is my first PR into this repo, said I didn't have push writes so created a fork, let me know if that is incorrect)

Discussed internally. Closing it as it's not required anymore.


improve coverage

Codecov Report

Merging #168 (f45ad4e) into main (a759ba9) will increase coverage by 2.58%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##             main     #168      +/-   ##
==========================================
+ Coverage   78.30%   80.88%   +2.58%
==========================================
  Files         101      101
  Lines       10560    10563       +3
==========================================
+ Hits         8269     8544     +275
+ Misses       1754     1559     -195
+ Partials      537      460      -77

Impacted Files
Coverage Δ

pkg/formatdecenc/rtph264/decoder.go
81.15% <0.00%> (ø)

pkg/formatdecenc/rtph265/decoder.go
76.85% <0.00%> (ø)

pkg/codecs/h264/annexb.go
93.18% <100.00%> (+4.54%)
:arrow_up:

pkg/codecs/h264/avcc.go
93.87% <100.00%> (+8.16%)
:arrow_up:

pkg/codecs/h264/dtsextractor.go
88.00% <100.00%> (+32.26%)
:arrow_up:

client.go
74.43% <0.00%> (+0.28%)
:arrow_up:

pkg/codecs/h264/sps.go
96.25% <0.00%> (+48.89%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


test: cleanup fixedbugs/notinheap.go

After #46731, //go:notinheap is gone, however, there's still test/fixedbugs/notinheap.go which is using it. There's no problem with trybot, because the file is ignored:
https://github.com/golang/go/blob/34f0029a85af054787b279761e89db410621f1d7/test/run.go#L1975
We should revise those test cases:

Writing the equivalent ones, replacing //go:notinheap by cgo.Incomplete.
Remove the file entirely.

cc @mdempsky @randall77

Taking a quick look, not all the cases can be rewrite, for example, there's no thing like this anymore:
//go:notinheap
type t3 byte


make install not try for shells we don't support

This script will never support csh, barring extremely major changes, so remove that.
Comment out kshrc for now - we might be able to support that soon.

@dmd :+1: :+1: :+1:


Search Show from TheMovieDB Not Working

I noticed that when on the 'Manage Content' page, under TV Shows, the search bar does not populate a list of shows from 'TheMovieDB'.

This functionality is working when I search for Movies correctly:

However, searching for the show 'The OA', I get no results when typing the exact string into the search bar:

Now, there is a workaround when I choose 'Create new Show' and search from TheMovieDB, then the show is populated:

I peeked at the source code and could not find any immediate errors.  It does seem like this functionality is setup to be working.
I really enjoy this project, thanks for all the hard work!

Manage Content is meant to look at what you've uploaded, it has no need to populate from the MovieDB

The title of the search bar states "Search Shows from collection or
TheMovieDB", which indicates that it should be doing a lookup from
theMovieDB.  In addition, this functionality works with the Movie search
bar in Manage Content, as pointed out in the issue report.  Also, the
support for doing a MovieDB lookup does seem to be in place in this
location.
If for some reason we didn't want to do a MovieDB search here, than at the
least I believe the search text should read something different (i.e
"Search Shows from collection") to avoid confusion.
Thanks.
On Sun, Feb 26, 2017 at 10:41 AM, mongomay69<EMAIL_ADDRESS>wrote:

Manage Content is meant to look at what you've uploaded, it has no need to
populate from the MovieDB
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dularion/streama/issues/310#issuecomment-282572762,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGmo5MF6KlskgjulbTuKpt5qN4D9qNG7ks5rgbligaJpZM4MMct0
.

yes agreed! this is an oversight on my part


[Bug] Can't focus input when there are nested modals.

Can't focus input when there are nested modals.
If there are nested modals, VFM should disable previous focusTrap of modals

Is this bug fixed already? I have the same issue. Do you have any solutions?

@stonega
Yes, the bug is fixed already. 🤔

@hunterliu1003 Thanks for your reply. I made a mistake, it was a issue related to nested slot but not nested modals, input is not focused inside nested slot, maybe I should open a new issue about it ?
You can close this.


Fix for bug in organism name parsing

Fixes issue #17. In any word in organism_line_split that contains a tab ("\t") the tab is removed.

Thanks for figuring it out :-) With a small code-cleanup utilizing Python's call by reference, I merge this in AutoPACMEN.


firestore not allow the c# code datetime other than UTC

Currently, I need to convert the date to UTC before entering it in Firestore.
for example I write code like this
record.post_date = DateTime.SpecifyKind(record.post_date.GetDateTime(), DateTimeKind.Utc);
it's a MySQL DateTime, if it's have unspecified, it's told to set to UTC, I didn't tried with local.  it must take all kind of DateTime where it's Local, Unspecified or UTC.
it's not working, I write code like this for every column to make it work.

No, I won't be changing this. A Firestore timestamp is UTC, and it's up to you to make sure you provide the right UTC value. A DateTime with a Kind that isn't UTC could mean any number of different things, and I wouldn't want the client library to make an implicit - and quite possibly wrong - choice about that.
You may be able to change how you access MySQL to make it provide you with UTC values, but I'm not going to change the Firestore library in a way that could silently lead to incorrect data being stored.

I am reading the values from MySQL using dapper. I am using Wordpress database, without Zero DateTime and with Zero DateTime both cases are not working.
What could be a better option here, I mean I simply want to put my date to the firestore. This means I need to write some code for every table which have DateTime column

What could be a better option here

Fixing your data. I do sympathize, but the problem isn't at the "inserting data into Firestore" stage, it's at the "fetching data from MySQL" stage. If your database values are meant to represent UTC timestamps, then what you should end up with in your .NET code is DateTime values with a Kind of Utc.
If you really can't get Dapper/MySQL to provide you with that, then I suggest you either just write the per-table code, or write some code that uses reflection to do so in a general way so that you can just call that on any table data.
I will not compromise the integrity of the Firestore library by allowing you to insert ambiguous data.

Thanks for the reply, I will check and let you know if setting UTC Timezone In MySQL make my code work without writing any code for datetime.


[WFLY-14203] Move microprofile-config-smallrye to the microprofile feature-pack

JIRA: https://issues.redhat.com/browse/WFLY-14203
Signed-off-by: Jeff Mesnil<EMAIL_ADDRESS>This PR is built on top of #13756, #13765, #13766 & #13774. Only the last commit for WFLY-14203 should be reviewed.

I made a diff between dist/target/wildfly-22.0.0.Beta1-SNAPSHOT/standalone/configuration  and ee-9/dist/target/wildfly-preview-22.0.0.Beta1-SNAPSHOT/standalone/configuration and there were no differences related to microprofile stuff (differences were about security/elytron, jsr77 and messaging)

@ehsavoie I spotted 2 diffs related to opentracing:
 <extension module="org.wildfly.extension.microprofile.opentracing-smallrye"/>

            <subsystem xmlns="urn:wildfly:microprofile-opentracing-smallrye:3.0" default-tracer="jaeger">
                <jaeger-tracer name="jaeger">
                    <sampler-configuration sampler-type="const" sampler-param="1.0"/>
                </jaeger-tracer>
            </subsystem>

are present in ee-9/dist/target/wildfly-preview-22.0.0.Beta1-SNAPSHOT/domain/configuration/domain.xml while they are not in dist/target/wildfly-22.0.0.Beta1-SNAPSHOT/domain/configuration/domain.xml
Could you check that in #13766 PR please?

@jmesnil I looked at the last commit and it seemed fine.
Comparing the output of build-legacy with and without this, it looked fine. Same with ee-9/build.
Re: the diff of 'build' though there's the same question as was discussed on the MP OT starting at https://github.com/wildfly/wildfly/pull/13766#discussion_r540134348. The MP Config extension/subsystem is removed from all the standard configs except standalone-microprofile[-ha].xml.

mmh, the mixed-domain failures seems genuine. I'm investigating them

The mixed domain tests were genuine because I remove the code that was removing microprofile-config-smallrye from 7.1.0 Domain. However, I realised after that microprofile-config-smallrye remains in domain mode and must be removed when we adjust to 7.1.0. After all tests pass, I've squashed the separate commit


Update L10N resource files with 4.6 strings from Transifex (20151129)

Small update of L10N files before the 4.6.1 release candidate
cc @remibergsma

LGTM, seams to fix linguistic issues, at least solving few for French.

Suggested a small french change, but still LGTM

Thanks for the reviews, I'll merge it to 4.6 like this. The improvement should go via Transiflex website and ends up in a new PR when @milamberspace submits the new translations.

@remibergsma Fine with me


[build-script] Determine HOST_CC in build-script

SR-237 calls for build-script and build-script-impl to be merged. This commit takes another step towards that goal by moving the logic that finds the path to the clang and clang++ executables up into Python-land.
Rather than simply moving all of the logic into utils/build-script, this commit moves relevant functions into a new Python module, named buildcli. This has several benefits:

The logic can be tested. Whereas build-script-impl needed to be run in order to verify its behavior, the logic extracted out of it into buildcli can be tested in isolation.
The logic can be split up into several files without polluting the utils directory, which now contains many different files that are unrelated to build-script.

/cc @gottesmm and @jrose-apple, since they've commented on SR-237 in the past.

@modocache Awesome improvements!

@gribozavr Addressed your comments, except for the one about adding the unit tests to validation-test. That change may end up being a little more involved, so I think it's best to send it in another PR.

@modocache Thank you!  Unfortunately, one more issue with argument handling came up in my testing...

@gribozavr Good point, I'll update utils/build-script as per your suggestions.
I also realized I forgot to add the code headers you mentioned! I added the ones from #762, since it looks like that was just merged. Thanks!

@gribozavr I've added some logic to the argument parser that enables the backwards-compatibility you described. I wanted to make sure we could test the new behavior in isolation, which meant moving the argument parser itself. Hopefully the commits aren't so big that they're unreviewable... :worried:
Let me know if you have any additional thoughts--and thanks for all the reviews!

A proposal to add Python unit tests to the validation tests is up at #778.

@modocache Thanks!  I think your patch will work, but... Sorry for sounding picky, but I think we can use a simpler approach.  Subclassing of argparse.ArgumentParser seems strange to me (I don't feel like it was meant for subclassing), and _move_impl_args mutating the parsed arguments data structure after the fact seems error-prone (for example, because the information about the key in is duplicated).
What do you think about the following approach?  Add a function in the module (so that it is testable) that accepts a command line and a list of options to migrate, and returns the rewritten command line, which is then passed to parse_args.  Rewriting algorithm would be:

Split the argument list on --, to get two lists, one is the build-script arguments, another is the build-script-impl arguments.

Iterate over the build-script-impl argument list, and check if an option is marked for migration.  If it is, append it to the build-script argument list verbatim.

Concatenate two lists with -- in between.

@gribozavr Updated as per your suggestions! :+1:
I think I'd still like to move the argument parser out into its own module someday, though. We could probably discuss this on the mailing lists, in the comments on SR-237, or in another pull request. My reasoning is that it's pretty cool to be able to call parser.parse_args() from within a unit test to verify what values are set. Even with the test_migration.py tests from this pull request, I still had to manually call build-script--the unit tests showed the migrate_impl_args() function was working, but I couldn't be sure the argument parsing was the same as before.
Anyway, thanks for all the dedicated reviews!

@modocache Yes, that would be definitely useful -- I was thinking about a feature in build-script that would validate all presets, for example.  I wouldn't mind if you move it to a module in this PR, but it is the subclassing that looks strange -- I could imagine a function that accepts a list of command line arguments and returns a parsed representation as the API that the module should expose for this purpose.

Ah, gotcha. Well, definitely expect more PRs on this topic, then! :wink:
I feel like this one is in a pretty good state, though. I'm eager to merge it so I can add its tests to the validation tests from #778.

@modocache Thank you!  This patch is much simpler and I like it.  There's one issue though, the C++ compiler name is not correct, see above.

@gribozavr Updated! Thanks.

Also went ahead and added the tests to validation-test. :+1:

@modocache Thank you!  Your PR passes all tests I did.  Merging.


compatibility runHandlers RUN_TIMEOUT setting and performance

Link to PDF file (or attach file here): any large PDF file (300 pages show the effect nicely)
Configuration:

Web browser and its version: IE11
Operating system and its version: Windows 7
PDF.js version: 1.8.188
Is an extension:

Steps to reproduce the problem:

open the file
wait until first page is rendered, scroll to page 2
now wait until page 2 appears

What is the expected behavior? should take no more than a couple of seconds
What went wrong? takes very long
Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):
local install, local file system
I'm having performance issues rendering large documents in IE11 (these issues do not appear in Chrome).
After staring at the f12 tools profiles for a while, I started wondering about all those Timer(runHandlers) invocations, none of these seem to take much time, but there are just so many of them that it fills the seconds.
So I tried changing the RUN_TIMEOUT in runHandlers (I think this is from compatibility.js) and noticed drastic improvements in the waiting time. Are there any disadvantages when setting the value from 1 to 5? Doesn't seem long enough to cause blocking in the UI.

https://github.com/mozilla/pdf.js/blob/master/src/shared/compatibility.js#L928
It is from Promise polyfill. Modern browsers don't use that anymore -- they have native support for promises.

Are there any disadvantages when setting the value from 1 to 5?

We can increase RUN_TIMEOUT value. It will not affect most of the browsers we support, only legacy browsers. A pull request is welcome.

I would like to work on this bug am ready with my dev env

Feel free to work on this. Thanks!

PR #9054 switched PDF.js to use the https://github.com/zloirock/core-js#ecmascript-6-promise polyfill, in order to avoid having to maintain a Promise polyfill ourselves.
Hence I'm closing this issue, since there's no longer anything to address on the PDF.js side.


fix printingUpdateForEvent deadlock (to dev)

version of #1530 targeting the correct branch

replaced with #1531 which includes Walid's mutex fix


Calling enable() does not set the draw button to active

When in edit mode, if you call:
map.pm.Draw.Poly.disable();

...for example, is will update the draw button accordingly.
But when you call:
map.pm.Draw.Poly.enable();

... the map enters draw mode, but the button is not updated.
fiddle: https://jsfiddle.net/qo5kwfww/1/

thanks for the fiddle.
Draw.Poly is for polygon. But the button you have in the Toolbar is for polyline. I can see that this might be confusing.
Here's the fiddle working correctly :-)
https://jsfiddle.net/qo5kwfww/2/


feat: create into convertors rpc transaction into transaction

This change is

#679  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ArniStarkware and the rest of your teammates on  Graphite

This PR is competing with #668.

The brother PR handles it better.


⚠ Bofrost.be has degraded performance

In b133c86, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2345 ms

Resolved: Bofrost.be performance has improved in b0fc796 after 20 minutes.


Disable createScript Functionality

Support running the agent without enabling its createScript functionality, ensuring it only uses pre-created scripts that it can find.

Scripts have been removed from the UI and de-emphasized in general.


fix: add a note to install the webrick gem for Ruby 3.0 users

Why:
Closes #14492

What's being changed:

add a note to install the webrick gem for Ruby 3.0 users under the "Build your site locally" procedural

Check off the following:

[x] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@riteshsp2000
Thanks so much for opening a PR! I'll get this triaged for review :zap:


updated telegram (2.22-50247)

Changes to a cask
Editing an existing cask

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Just ran into this myself. The SHA was incorrect.