expfactory fails after installing psiturk

I installed and set-up psiturk (using pip install psiturk, pip installed with conda) and now expfactory won't open. Even after I pip uninstall psiturk I get the below error when calling expfactory. Further, even if I uninstall and reinstall expfactory, the error remains. The only solution I've found is to complete remove and reinstall anaconda python. Is this a anaconda issue or an expfactory/psiturk issue.
psyc308-01:~ congalab$ expfactory
Traceback (most recent call last):
File "/opt/anaconda/anaconda2/bin/expfactory", line 11, in
load_entry_point('expfactory==2.5.46', 'console_scripts', 'expfactory')()
File "/opt/anaconda/anaconda2/lib/python2.7/site-packages/expfactory/scripts.py", line 141, in main
from expfactory.interface import start
File "/opt/anaconda/anaconda2/lib/python2.7/site-packages/expfactory/interface.py", line 44, in
app = EFServer(name)
File "/opt/anaconda/anaconda2/lib/python2.7/site-packages/expfactory/interface.py", line 23, in init
custom_battery_download(tmpdir=self.tmpdir)
File "/opt/anaconda/anaconda2/lib/python2.7/site-packages/expfactory/vm.py", line 37, in custom_battery_download
download_repo(repo,"%s/%s/" %(tmpdir,repo))
File "/opt/anaconda/anaconda2/lib/python2.7/site-packages/expfactory/vm.py", line 23, in download_repo
return Repo.clone_from("https://github.com/expfactory/expfactory-%s" %(repo_type), destination)
AttributeError: type object 'Repo' has no attribute 'clone_from'

hey @oricon ! This looks like a version error with the github module in Python, which likely was updated / changed when you updated psiturk. Could you verify the version? The one that I have that should work is 2.1.1.  Here is how you check:
 ipython
 from git import Repo
 Repo. [TAB][TAB] to see if there is a function clone_from

To check your version:
  import git
  git.__version__

To fix the version:
 pip install gitpython==2.1.1

And it's my fault for not specifying the version in requirements.txt, and also my fault for having you need to think about dependencies, period!  u I wasn't super experienced with containers when I designed the client, and a much better approach would have all dependencies tightly packaged to avoid these issues.

Thanks! That helped. I had git 2.11.1 installed originally, so I
uninstalled that and installed 2.1.1. I then installed psiturk and it looks
like psiturk changed git to 0.1.7. But after installing 2.1.1 both psiturk
and expfactory worked.
-Joe
--
Joseph M. Orr, Ph.D.
Assistant Professor
Department of Psychology &
Texas A&M Institute for Neuroscience
Texas A&M University
College Station, TX
On Fri, Mar 31, 2017 at 3:07 PM, Vanessa Sochat<EMAIL_ADDRESS>wrote:

hey @oricon
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_oricon&d=DwMFaQ&c=ODFT-G5SujMiGrKuoJJjVg&r=ZKy1VO33u0kvO-PqY1gpb9Ld-AGhtT8c9PAcpsEyp70&m=ep_7BDFYUL6REoqdIjhQ7oqbE4jpPrRFxVS3e_sS34w&s=o_ysWZE7zVdyL-CkwxTsz0laBAjIimmjK4wIXnRgAtY&e=
! This looks like a version error with the github module in Python, which
likely was updated / changed when you updated psiturk. Could you verify the
version? The one that I have that should work is 2.1.1. Here is how you
check:
ipython
from git import Repo
Repo. [TAB][TAB] to see if there is a function clone_from
To check your version:
import git
git.version
To fix the version:
pip install gitpython==2.1.1
And it's my fault for not specifying the version in requirements.txt, and
also my fault for having you need to think about dependencies, period! u I
wasn't super experienced with containers when I designed the client, and a
much better approach would have all dependencies tightly packaged to avoid
these issues.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_expfactory_expfactory-2Dbattery_issues_67-23issuecomment-2D290816750&d=DwMFaQ&c=ODFT-G5SujMiGrKuoJJjVg&r=ZKy1VO33u0kvO-PqY1gpb9Ld-AGhtT8c9PAcpsEyp70&m=ep_7BDFYUL6REoqdIjhQ7oqbE4jpPrRFxVS3e_sS34w&s=e3JtTLmFIHcODHjBIdtQq77XA1q3qgnJqLALMxsyajY&e=,
or mute the thread
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AILZLwxSyX5C2NbtNJiM75AMSKYAQ1wuks5rrV0dgaJpZM4Mv23g&d=DwMFaQ&c=ODFT-G5SujMiGrKuoJJjVg&r=ZKy1VO33u0kvO-PqY1gpb9Ld-AGhtT8c9PAcpsEyp70&m=ep_7BDFYUL6REoqdIjhQ7oqbE4jpPrRFxVS3e_sS34w&s=g5YrZ44B2Ohyd-WvTY1TRvv73mvehZetznoBUMWn2xk&e=
.

no problem! I'm going to close the issue, please ping if you have any more trouble.


Show completions in the UI. r=vporof. Fixes Issue #200

r? @victorporof

Coverage decreased (-0.3%) to 86.486% when pulling b12758d7f0a49a7c8f5d45adfdac5c7490fa5bfd on bgrins:completions into 34fca6eef5a8034e72ec971d341e4db9b2c07dc0 on mozilla:master.

This was merged


Update the zcl and template packages during generation

Create a flag called --upgradeZapFile which can be passed during generation to update the package paths during generation. add this to apack.json
During generation use the flag to determine if packages should be updated in the .zap file. If the .zap file does not already have the packages mentioned in generation(uc_generate) then the .zap file is updated with those packages and saved.
JIRA: ZAPP-1456

I'll say this: I don't like this. We're "on-the-fly"overwriting the input file. Someone, somewhere will say: "Why do I have a diff in this file? I didn't change anything!"
So I don't like this functionality in general, because I think it has a potential of causing more damage than not.
But it's hidden behind the CLI switch, so if someone doesn't want to use it, they can simply not pass in that switch. So I'm ok with merging this, because this is all optional, I just don't want this to me missused as some kind of silver bullet.

Our chat for reference https://silabsiot.slack.com/archives/D67QEA77C/p1729626363562279


Tp 14 create station

Tests Implemented and passing:

Used MockMvc with Rest Assured to test Admin Controller;
Used Selenium with Firefox Driver to run Cucumber Tests.

looks good to me


[Docs] Add documentation

This project needs good documentation. Refer to the previous projects of hG.

kindly assign it to me

Take reference from houseofgeeks/hg#2


Version 2.0

This issue is a reminder to release a version 2.0 of this template prior to fall courses & workshops.

our prerelease was version 0.1, so this will actually be v.1.0

@Peter-Kedron : this is the last call for a final proofread of the template before releasing version 1.0

I think this looks good. My one suggestion is that we move the citation reference to the top of the top-level readme. If we want people to cite, I think we have a much better chance if that is right in their face when they land on the page.

Yes, exactly. Close the issue if you are satisfied.


Bundling restructure

Replaces https://github.com/googleapis/gax-java/pull/214

Codecov Report

Merging #226 into bundling-flowcontrol-branch will decrease coverage by -0.21%.
The diff coverage is 83.83%.

@@                       Coverage Diff                       @@
##             bundling-flowcontrol-branch   #226      +/-   ##
===============================================================
- Coverage                           70.2%    70%   -0.21%
- Complexity                           483    485       +2
===============================================================
  Files                                 67     68       +1
  Lines                               2541   2557      +16
  Branches                             267    267
===============================================================
+ Hits                                1784   1790       +6
- Misses                               659    669      +10
  Partials                              98     98

Impacted Files
Coverage Δ
Complexity Δ

...n/java/com/google/api/gax/grpc/BundleExecutor.java
100% <100%> (ø)
5 <2> (-1)
:x:

.../com/google/api/gax/bundling/ThresholdBundler.java
96.55% <100%> (-0.03%)
27 <0> (-1)

...java/com/google/api/gax/grpc/BundlingCallable.java
88.88% <100%> (ø)
3 <0> (ø)
:x:

...e/api/gax/bundling/AccumulatingBundleReceiver.java
83.33% <100%> (ø)
3 <1> (ø)
:x:

src/main/java/com/google/api/gax/grpc/Bundle.java
55% <55%> (ø)
4 <4> (?)

...n/java/com/google/api/gax/grpc/BundlerFactory.java
78.46% <73.68%> (-1.87%)
12 <9> (+1)

.../com/google/api/gax/grpc/BundledRequestIssuer.java
78.94% <75%> (ø)
6 <2> (?)

...e/api/gax/bundling/ThresholdBundlingForwarder.java
87.5% <83.33%> (ø)
4 <1> (ø)
:x:

Continue to review full report at Codecov.

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


toos/testbuild.sh: Remove the below warnning in nightly build

====================================================================================
Configuration/Tool: lm3s6432-s2e/nsh,CONFIG_ARMV7M_TOOLCHAIN_GNU_EABIL
Change-Id: I95cd6f0c00f68c68f9175574d44e75e24c41bc12
Cleaning...
Configuring...
Copy files
Select CONFIG_HOST_LINUX=y
Refreshing...
Disabling CONFIG_ARMV7M_TOOLCHAIN_CODESOURCERYL
Enabling CONFIG_ARMV7M_TOOLCHAIN_GNU_EABIL
Refreshing...
.config:1542:warning: override: reassigning to symbol ARMV7M_TOOLCHAIN_GNU_EABIL
Building NuttX...
Signed-off-by: Xiang Xiao<EMAIL_ADDRESS>

Related to Issue #386 ?

These warnings are the consequence of replacing kconfig-treak with sed.  kconfig_tweak is slower, but takes are of everything.


Trouble decoding file

Hello,
I am having trouble parsing a Hebrew XLSX file
attached here: http://goo.gl/kPOfNr
Some of the cells data comes out empty.
For example for the first sheet: A1, A2, A3, A5, C6 etc..
Any help would be appreciated,
TIA,
Shakhal

I apparently already fixed it in my local copy, https://github.com/SheetJS/js-xlsx/blob/master/xlsx.js#L2053 should read
			var o = parse_si(ss[i].trim(), opts);

Keeping open until next commit lands

Awesome!
Thank you so much :)

Any idea when the next commit lands? ;)

+1 waiting for next realease, much appreciated! TIA!

Hey,
Thanks for your efforts on merging xls & xlsx,
Seems like this issue is back again...


Input[number] ng-pattern validation overrides the ng-maxlength validation

I have a input[number] field with ng-maxlength and ng-pattern set on it. The input structure is as follows:
                <label class="control-label">Amount
                </label>
                <input type="number"
                       class="form-control"
                       placeholder="Amount"
                       ng-maxlength="10"
                       required
                       ng-pattern="/^[0-9]+(\.[0-9]{1,2})?$/"
                       validation-msg-pattern="Field with 2 decimals allowed"
                       ng-model="formData.amount">
            </div>```

Here, if I input more than 10 characters it validates the ng-maxlength validation but then further if I increases the characters it gets overrides with the ng-pattern validation.

Please help

The problem being most likely that the number input is still buggy in multiple browsers. Thanks for reporting though. I will look into this.

Check out this plunkr:
http://plnkr.co/edit/NQzCas?p=preview
When you assign the value from the controller:
$scope.formData={
          amount:<PHONE_NUMBER>123456789012.34
};

It gets converted to 1.2345678901234568e+21
That might be why the max-length acts weird at some point (and so does the pattern check).

Ok Thanks @johannesjo for the explanation. But how can we avoid this to work as required. Can you give some suggestion to solve this issue of number getting converted into decimals.

You have 3 options:

Use an input[type=text] and build the logic yourself
Trust your users to not enter such ridiculus numbers
Try using min and max instead

If you're still unsure, I recommend going to Stackoverflow. Your problem is really beyond the scope of this repo.

Thanks for the suggestions :+1:

You're welcome!


Black Morass recruitment is appearing under the Black Temple section

This is simply because "black" is a tag for black temple, so "black morass" is picked up. Similarly, "bm" is a german tag for hellfire ramparts, which is also picked up even though my game is in english.

I'm actually seeing BM under Hellfire Ramparts rather than Black Morass. Checked the settings and it clearly says BM should fall under Black Morass but for whatever reason it's not.


𝐔𝐑𝐆𝐄𝐍𝐓: Fix update password

The API has changed to require who is updating a user: alphagov/digitalmarketplace-api@7f578ea
This change was reflected in utils for updating a user's information: 8a4bd48
…but wasn't changed for updating a user's password. This is currently causing a "Could not update password due to an error" error on production.
This commit allows the email address of the current user to be passed in, fixing the bug.
--
Addresses: https://www.pivotaltracker.com/story/show/104342384

:+1:


Various bugs

Fix most of the bugs. Most that i can find anyway

PoolSnapshot needs the block for our pool graph to work, see Graph.tsx changes (but might be we are not needing the mechanism in there which is a chunked fetching for performance reasons I think)

In this query all the fees are 0. Using did:op:433c71ffc85604727da7e272ea1f63b083c6f8597086fd9d37acdc7d32373976 as test asset so might be it was published with older contracts

Fees are a todo, because at that time they were constantly in works , you can check this list https://github.com/oceanprotocol/ocean-subgraph/issues/208

cool, thanks for info. Already excited about all those fee methods in there allowing us to output accumulated fees.
Another thing I couldn't get to work is pool creator liquidity or more precisely NFT owner liquidity, using this query and the shares are always []. But again might be because of test asset

keep them coming  , i'll add them in the list in the main description.


PDF/EPUB generation doesn't seem to be happening.

Details

Read the Docs project URL: https://bids-specificaion.readthedocs.io/en/latest/
Read the Docs username (if applicable): choldgraf

Expected Result
The documentation is built as PDF/EPUB (since those boxes are checked on my advanced settings menu). Also, I expect to see the little floating window in the bottom-right since I'm hosting on readthedocs.
Actual Result
There is no built file in /downloads, nor in the downloads tab on the RTD config page. There is also no floating box.

PDF and epub formats aren't supported by MkDocs. The floating box is an issue, though.

Looks like jquery is missing when injecting the rtd's version selector (floating box)

ahh - it'd be helpful if it were made clear that PDF generation only works for Sphinx builds and not mkdocs...perhaps here :-)
re: the floating box, do we need to specify jquery as a dependency somehow?

We mention that here https://docs.readthedocs.io/en/latest/builds.html#mkdocs, I'm not sure if we need to mention that on the features pages, as it's a MkDocs limitation, not rtd.

re: the floating box, do we need to specify jquery as a dependency somehow?

I think we inject those dependencies somewhere (or maybe we assume that everyone is using jquery?) I need to check the code.

Sure - I'm just giving you some data from my personal path through that documentation...I didn't notice that statement about PDF on that page

So, I looked into the code, we inject the code for the floating footer https://github.com/rtfd/readthedocs.org/blob/a4291d17728b8df6b7a9ce57537c5d620416e485/readthedocs/doc_builder/backends/mkdocs.py#L106-L106, but we don't have jquery as an explicit dependency https://github.com/rtfd/readthedocs.org/blob/master/readthedocs/core/static-src/core/js/readthedocs-doc-embed.js
Not sure if we should add jquery or rely on themes to added it. Going to mark this as a design decision.


Backport wrapinstance fix

Fixes https://github.com/conda-forge/pyside-feedstock/issues/32

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


BUG, inflated number of annotations

See npr2
GO:0034198 | cellular response to amino acid starvation | ISO with S000000778 | GO_REF:0000024 | 603
says 603, but there are  only 8

I think it's because it's counting the genes from all annotations of the gene where GO:0034198 appears in an extension.  Not helpful!  I'm working on a fix.

I've fixed this in time for the nightly load.  I'll check everything is OK in the morning.


Admire comment

https://user-images.githubusercontent.com/6754223/195221350-c8d4265f-f68c-4518-8ac3-a02d626bc260.mov
https://user-images.githubusercontent.com/6754223/195222550-058ab5ae-65a3-4b7f-b388-df0f97b6492e.MOV

@kaitoo1 @Robinnnnn I was having some trouble logging in on mobile ios in the vercel preview. hoping y'all can help check this out tomorrow.


Fix DelimitedTextFile offset.

Use a sequence file to manage offset.
fix #101

@yyejun would you mind rebasing?

I rebased, but not sure what caused tests exceed 50 minutes.

This PR would need to sync with the latest code.


[Question] Has anyone got a fork for adapter.rooms

I'm talking about storing rooms and the clients they contain in Redis as well.
213 forks... :)

Figured it out: Its already supported.
io.in('room').clients(function (error, clients) {
                if (error)
                    throw error;

                res.send(clients);
            });

Thanks SIO Redis team.


RT-1.7: Local BGP Test- OTG: Adding BGP-PEER_GROUP configs to local_bgp_test.go

Adding BGP-PEER_GROUP configs since bgp neighborship would not come up without those changes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
580

Relevant Lines:
1173

💛  - Coveralls

@stthummuri, ping to address above comments
@bstoll, @shidzhan pinging for review

Closing pull request since the changes got merged as part of https://github.com/openconfig/featureprofiles/pull/730/


HubConnection lacks state property

Is there a way to check the status of the HubConnection similar to the HttpConnection:
private enum State: String {
    case initial = "initial"
    case connecting = "connecting"
    case connected = "connected"
    case stopped = "stopped"
}

I saw there is a connectionId which is nil when is not connected.
Best regards,
Hugo Robles

These states are private to HttpConnection and cannot be checked from outside. The lifetime of the HubConnection is different from the lifetime of HttpConnection and HubConnection itself is not actually aware of the state of the HttpConnection. You can track the state of the connection via the delegate that contains callbacks invoked for different lifecycle events.

Considering the question answered.

Ho can I check the connection state when app return from the background? Ho can I understand if iOS kill the SignalR connection?
I'm trying to implement the reconnection to restore the connection at willEnterForegroundNotification.
Thanks

You should be able to use lifecycle events for this.

With lifecycle I can only understand when the app enter background or enter foreground. I can't get the real state of the connection: I notice that websocket connection survive for some seconds/minutes when the app is in background.
I don't want to force a connection close when app enter background, this may cause a bad UX.
I think that the use of the lifecycle is limiting and is a workaround, not a real solution.
The only real solution that can ensure consistency is to expose the connection state.
Anyway, thanks.

I mean connection life cycle events (i.e. https://github.com/moozzyk/SignalR-Client-Swift/blob/c8bf32badfeee5d002621018f3625eb7cdd18741/Examples/HubSample/AppDelegate.swift#L168-L193)
I actually think stopping the connection when application is backgrounded and starting when it goes to the foreground might be better than relying on the observed iOS behavior you don't control over and that can change any time. iOS will stop network connections when the application is backgrounded. Sometimes they survive for a random period of time. The client will likely not have the correct information anyways when the app is foregrounded and it may take some time for it to figure out the state of the connection (e.g. timeout, maybe reconnect logic will kick in etc.). If you stop the connection yourself when the app goes to background then you know for sure that that connection is stopped when it is foregrounded and starting the connection can be much quicker that some timeouts that can kick in leaving the application in an unusable state in the meantime and forcing the user to experience this.

The only real solution that can ensure consistency is to expose the connection state.
I don't even think this is technically possible. My understanding is that if the application goes to the background, the connection is killed and execution is suspended. As a result when you foreground the app the state will reflect the value it had when it was backgrounded but the connection was already killed. Also, the connection can be dropped immediately after you check the state.

Ok thanks!!!


Object references

Thank you for creating Remfile - it makes opening lots of large NWB files for small metadata actually tolerable!
I've noticed that using object references results in eager reads of unnecessary amounts of data:
import time
import h5py
import remfile

LARGE_HDF5_URL = 'https://dandiarchive.s3.amazonaws.com/blobs/f78/fe2/f78fe2a6-3dc9-4c12-a288-fbf31ce6fc1c'
SMALL_HDF5_URL = 'https://dandiarchive.s3.amazonaws.com/blobs/56c/31a/56c31a1f-a6fb-4b73-ab7d-98fb5ef9a553' # useful for testing quickly

url = LARGE_HDF5_URL

nwb = h5py.File(remfile.File(url), mode="r")

# this is an instance of <HDF5 object reference>:
object_reference = nwb['units/electrode_group'][0]

# the location `object_reference` points to (which can't be determined from the object reference itself)
url_to_actual_location = {
    LARGE_HDF5_URL: '/general/extracellular_ephys/17216703352 1-281',
    SMALL_HDF5_URL: '/general/extracellular_ephys/18005110031 1-281',
}

# 1. accessing the location directly and reading metadata is fast:
t0 = time.time()
_ = nwb[url_to_actual_location[url]].name
print(f"1. Time to get referenced object data directly: {time.time() - t0:.2f} s")

# 2. when using the object reference, a lazy accessor seems to be returned initially
# (which is fast):
t0 = time.time()
lazy_object_data = nwb[object_reference]
print(f"2. Time to get lazy object reference: {time.time() - t0:.2f} s")

# 3. when the same component is accessed, it is much slower than in 1. - suggests
#    more data than necessary is being read
t0 = time.time()
reference_path = lazy_object_data.name
print(f"3. Time to get referenced object data: {time.time() - t0:.2f} s")
assert reference_path == url_to_actual_location[url]

# 4. subsequent access of a different component is fast - supporting the idea that
#    more data than necessary is being read (and cached) in 3.
t0 = time.time()
second_object_reference = nwb['units/electrode_group'][-1]
second_reference_path = nwb[second_object_reference].name
print(f"4. Time to get second referenced object data: {time.time() - t0:.2f} s")
assert second_reference_path != url_to_actual_location[url]

output:
1. Time to get referenced object data directly: 0.07 s
2.  Time to get referenced object data: 119.33 s
4. Time to get second referenced object data: 0.01 s

I don't think this is an issue with Remfile itself: times were similar with @martindurant's suggestion of using fsspec and cache_type="first" (though the initial opening of the file was much improved from cache_type="readahead" - almost as fast as Remfile). I was just hoping that you might have some insight into what's going on here, and a way to go faster!

I think this would be related to the inner-workings of h5py and Hdf5. Hdf5 was created with the assumption that you would have the data locally, so some of the lookups and references iterate through things and it becomes very slow when reading remotely.
To get a better insight, you could try verbose=True on remfile.File constructor. That will tell you about where it is reading from the file, and how much data.
I'm guessing that there's nothing that can be done.
Although to give some good news, we are working on ways to make loading these things much faster by using tools like kerchunk. And maybe these will get integrated into dandi at some point.

Ah the verbose option is very useful, thanks (for some reason I'm not able to put breakpoints in the methods of Remfile.File and step through the process..). Interestingly, with _max_threads > 1 the threadpool isn't being activated, even though the total downloaded should warrant it. I wonder whether making these read requests with an async library like httpx could speed this up. I may have a go.
kerchunk'd files on dandi sounds like a great idea. Thanks for your efforts!

Note that fsspec is async (via aiohttp), but the file-like API is not (because the python file-like object is not). If you want to fetch multiple byte ranges from one or more files concurrently, you can use the cat_ranges method.


No handler reference for Directives

Information about the syntax of the handlers in a Directive is scattered across various pages of the Atmo docs. It would be nice to collect information about the various syntactic elements (steps, group, as, with...) on a single page.

Relevant Atmo types for handler steps.

fn is explained in Usage → Creating handlers
group is only briefly mentioned in Concepts → State
as and with is discussed in Usage → Managing state
onErr is discussed in Usage → Error handling

The meaning of other properties (besides steps) is briefly mentioned in Usage → Creating handlers, but there doesn't seem to be much comprehensive documentation about their possible values and usage.


Save Data with Pointer

Is it possible to save data in a class with Pointer (relational data)?

I'm working on it.

Do you need help?

Do you need help?

The current version does not allow saving data with Pointer.
I'm already working on solution for this functionality. I hope to launch in the next few days.


Install client into LOCALAPPDATA

In preparation to https://github.com/FAForever/downlords-faf-client/issues/1753, change the installation directory to local AppData.

Should be coupled with #1753


[IRGen] Remove SetBitEnumerator from ClusteredBitVector

The change replaces 'set bit enumeration' with arithmetic
and bitwise operations. For example, the formula
'(((x & -x) + x) & x) ^ x' can be used to find the rightmost
contiguous bit mask. This is essentially the operation that
SetBitEnumerator.findNext() performed.
Removing this functionality reduces the complexity of the
ClusteredBitVector (a.k.a. SpareBitVector) implementation and,
more importantly, API which will make it easier to modify
the implementation of spare bit masks going forward. My end
goal being to make spare bit operations work more reliably on
big endian systems.
Side note:
This change modifies the emit gather/scatter functions so that
they work with an APInt, rather than a SpareBitVector, which
makes these functions a bit more generic. These functions emit
instructions that are essentially equivalent to the parallel bit
extract/deposit (PEXT and PDEP) instructions in BMI2 on x86_64
(although we don't emit those directly currently). They also map
well to bitwise manipulation instructions on other platforms (e.g.
RISBG on IBM Z). So we might find uses for them outside spare bit
manipulation in the future.

We could probably do without ClusteredBitVector altogether and just use APInts (or else turn ClusteredBitVector into a wrapper around an APInt). This looks like good progress though.

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - d3262ec10d7e41b9403f83f2f89474795f9eed3a

Build failed
Swift Test OS X Platform
Git Sha - d3262ec10d7e41b9403f83f2f89474795f9eed3a

Looks like lldb also has to be updated; do you have a PR for lldb as well?
/home/buildnode/jenkins/workspace/swift-PR-Linux/branch-master/lldb/source/Symbol/SwiftASTContext.cpp:593:34: error: no member named 'enumerateSetBits' in 'swift::ClusteredBitVector'
13:31:56     auto enumerator = m_tag_bits.enumerateSetBits();
13:31:56                       ~~~~~~~~~~ ^
13:31:56 1 error generated.

Thanks @jckarter. I made a pull request against swift-lldb here: https://github.com/apple/swift-lldb/pull/1607. Alternatively I could keep the API around for now and delete it later on (along with, potentially, ClusteredBitVector itself).

Please test with following pull request:
https://github.com/apple/swift-lldb/pull/1607
@swift-ci Please test

Please test with following pull request:
apple/swift-lldb#1607
@swift-ci Please test


Check presence of rsync in Linux install

https://stackoverflow.com/questions/592620/how-can-i-check-if-a-program-exists-from-a-bash-script

Done

Done


Create speculative decode dynamic parallel strategy

Motivation
Create new speculative decode dynamic parallel strategy for our team needs
Features
Here we briefly describe features that we will implement in order to implement speculative decode dynamic parallel strategy. Each feature has high level description as a part of request for change with more description provided inside pull request for particular feature
Save speculative decoding states #7358
Allow users to optionally receive speculative decoding artifacts such as history of draft token indices for each step of speculative decode algorithm
Create draft from random tokens from promt #7359
Implement speculative proposers that enrich previous draft with tokens randomly sampled from current prompt
Allow model executor to return many next tokens #7361
Current implementation of model executor and model runner produce one last next token in decode stage. This feature would allow inner model runners to return next tokens for a range of tokens
Create parallel scorer #7362
Implement scorer that uses feature that allows to get next tokens in single forward request. Runs target model and returns scores for next tokens
Create speculative decode dynamic parallel strategy #7363
Combine all features such that users can use this speculative decode strategy

Why are all of your PRs empty?


csi-vmware-cloud-director: overwrite clusterId with param

What this PR does / why we need it:
This PR allows to overwrite and customize the clusterid value in the embedded csi-vmware-cloud-director addon. ClusterId can then be set to different values, most notably to include the prefix NO_RDE_, to disable RDE in the csi-driver when interacting with the vCloud API.
See https://github.com/vmware/cloud-director-named-disk-csi-driver/issues/117 and https://github.com/vmware/cloud-director-named-disk-csi-driver/pull/22 for reference.
What type of PR is this?
/kind feature
Does this PR introduce a user-facing change? Then add your Release Note here:
NONE

Documentation:
NONE

/ok-to-test


model: tensorflow: dnnr: Fixing regressor

Fixes #374

Awesome!!!! Just the one spelling issue in the CHANGELOG.md. What ended up being the issue?

It was due to the change in default values of optimizers in tf.estimator module. When I used tf.compat.v1 the original behaviour was restored. facepalm!


Fix bug in SelectionSetNode.AddSelections() and update test snap

3 character bugfix 😮

Can you add another test that ads a selection to a field that already has a selection?

And thanks for your help

Can you add another test that ads a selection to a field that already has a selection?

Unless I am misunderstanding you, there is already an existing test for this, which was succeeding because the snapshot file contained output from the buggy method. That was fixed in this PR. But let me know if you meant a different test.

I see ... misread the snapshot... you are right.


Check Object documentation for Groups

Be sure that style is right and that all pieces are documented as things have been added since the documentation was written.

Completed in #793.


Add a button to go to an installed service from its marketplace listing

I was able to get this added pretty easily. I'm not sure if there was a specific location or style in mind but adding "View Service Details" down near "Other Versions" if the app is installed seemed like the simplest solution:

Below is another solution I explored but doesn't feel as clean:

Let me know what you all think and I will put up a PR @dr-bonez @afg419 @MattDHill

I like the presentation of the former better but the latter placement is definitely more convenient. The trouble is going to be around whether you can compress the second solution into a space where the action is still clear.

A couple other placement options/variations if you think it's worth the convenience of having it at the top of the view rather than the bottom:

"View Installed Service" may be a more clear button label than "View Service Details", any thoughts there?

@MattDHill

Please use "Go to Service" and this image:

Also, this can go out in 0.2.9 if it's ready

@MattDHill  I opened up a PR against integration/0.2.9 branch


Issue on the Laravel 9.2

Problem 1
- luilliarcec/laravel-table[3.2.0, ..., 3.2.1] require laravel/framework ^7.0|^8.0 -> found laravel/framework[v7.0.0, ..., 7.x-dev, v8.0.0, ..., 8.x-dev] but the package is fixed to v9.17.0 (lock file version) by a partial updat
e and that version does not match. Make sure you list it as an argument for the update command.
- Root composer.json requires luilliarcec/laravel-table ^3.2 -> satisfiable by luilliarcec/laravel-table[3.2.0, 3.2.1].
is there any chance to update to support laravel 9.2^?

Added support for L9

thanks


resources are always added by "OCW Studio Bot" in the git commit log

Steps to Reproduce

Log into Studio with your user
Create a site and sync some resources from gdrive, for example: https://ocw-studio-rc.odl.mit.edu/sites/peters-test-site-for-unpbulishing/type/resource/
Take a look at the commit history in github, for example: https://github.mit.edu/ocw-content-rc/test101-spring-2023/commits/main

Expected Behavior

The commit that adds the resources should have the same user as the commit that created the course

Actual Behavior

All commits that a result of a gdrive sync are made by "OCW Studio Bot"

Screenshot or Screencast

Thanks for the context, but I don't see why these commits that add resources should be "anonymous" since there is a logged in user who clicks the "sync w/Google drive" button.


Update aws-java-sdk-core to 1.12.545

About this PR
📦 Updates com.amazonaws:aws-java-sdk-core from 1.12.459 to 1.12.545
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.amazonaws", artifactId = "aws-java-sdk-core" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #612.


atsamd-rs:owners should own crates published by atsamd-bot

@TDHolmes noticed that some of the crates recently published through github actions were owned by only atsamd-bot, and not atsamd-rs:owners. We should update the github action script (atsamd/.github/workflows/release-crates.yml) to include a command such as cargo owner add github:atsamd-rs:owners (not sure if this is the exact syntax, please double check)

Having looked through here, I can confirm that the atsamd-rs team members are owners of all crates we currently maintain. Not sure if we should try to add the team members as owners every time a crate is published, or as needed as new crates are created.

Cool, so I guess after we have #757 merged, we can see what happens when we release a crate then close this if it is owned by the atsamd-rs team?

Can confirm that all crates released successfully.


overvaluation of "bait"

so I finally figured out why bait is being overvalued.
		// number of ticks needed to repopulate an army
		let timer = log(1 + soldiers() * breed() / add(Bait, 100)) / log(1 + breed());

The truth is, trapping doesn't have any value later in the game. The speed that you can open a trap maxes out at once per tick. So the base breeding speed completely overwhelms the bait/trapping speed.
My HCE is 60. I spent most of my time playing decay at the moment. Even before I had decay unlocked, I was still noticing this whole thing that auto traps were useless, because using trapping to try to repopulate my army did not have a significant effect (less than one second, even with yellow 12 repopulation time).
In other words, bait/trapping has value in the early game, when 10 or 20 starting traps are being used to populate my initial army, or when my initial flood of mansions causes a large need of more trimps. But not late game, which seems to be what perky is based on.

Why do you quote that line of code? Do you believe it incorrectly computes the time needed to repopulate an army?
You’re correct that trapping doesn’t have any significant effect past the very early game (z40 or so). Perky also gets to the same conclusion, which is why it recommends very little Bait if you set your target zone high enough.

I quote that line of code because it looks like it is describing the amount of time that it takes to repopulate the Army, and because it is using the bait value as part of the calculation.
It is dividing by the value of bait, without taking any limit of reproduction into account.
In other words, the effectiveness of bait is not determined by "add (bait, 100)", but by a smaller number that represents the behavior that your trapping speed hits a limit even as player effectiveness should continue to increaseJ Is okay

The player can’t open more than 10 traps per second regardless of efficiency, and this cap is reached fairly quickly. Perky assumes that you’re constantly at the cap, and so the overall trapping speed is 10 * (Bait level + 1). Given this assumption, the calculation is correct.


Custom PATH environment

Maybe this is too specialised, but it would be great to be able to set a custom PATH (don't know what that would be on OSX) environment variable for execution.

You can install atom-fix-path plugin or something like that.
It's recommended to write it into script's README until Atom team fixes it.


I tried to install the Visual Studio Keymap but I don't think it worked.

I tried to install the Visual Studio Keymap expecting to get the same shortcuts as I'm used on Visual Studio 2015.
I don't know if I did something wrong but it's been really hard for me using the editor and learning all over again the shortcuts.

If this extensions is not for that. Does anybody have already mapped the shortcuts to be similar to the ones from VS2015?
Thanks

Hi @bvpmiranda - the list of keybindings are here. https://github.com/rebornix/vscode-vs-keybindings/blob/master/package.json#L26
Which ones are working for you?

Apparently the bindings are still the default ones. Like, to fold everything on VS2015 I use ctrl+M ctrl+L.
In VSCode it's ctrl+k ctrl+0.
When I installed the extension, I thought It'd change to ctrl+M ctrl+L.

This extension is in preview phase knowing that we missed some. Can you open a PR and add that keybinding?

@bvpmiranda I've fixed the fold all issue. It would be really awesome if you can point out what other keybindings are bothering you, they should be easy to add if the command it's already there in Code.
@waderyan I've bumped the version. Feel free to have a new release.


Accessing dataset

Hi, I'm trying to run the code.
However, the dataset link that you attached does not work at all.
Can you please fix this?
Thank you!

Thank you for your kind reply.
I will look at the new repo instead.
Thank you again!


Add Missing Perms, Set Node 14 as Minimal Ver, Update Deps, Fix Commando Message Methods

The Package was Tested after updating the Dependencies and looks like there are no conflicts and works as it should be

This is little fix for Commando Message Methods, not ready for v13 (Need to Update to v13 sending methods), its ready to get Merged as quick fix.


hide and seek

well done on your project  you did very well✔🎆🎉🎉🎉🎉🎉🎉👍

how ever I notice on your JavaScript file there is the syntax error on line 338

syntax error fixed

🙏🙏🙏🙏🙏🙏🙏🙏


Higher resolutions than 720p

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm requesting a feature unrelated to a specific site
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
When i'm using yt-dlp, the max resolution is always 720p, even if the video's max resolution is higher. Is it possible to add support for higher resolutions ?
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

https://github.com/yt-dlp/yt-dlp/wiki/FAQ#do-i-need-any-other-programs

I  agree with OP. FFMPEG is available from the yt-dlp directory, but I only get 720p by default. Currently looking for things I'm missing.


Support for React 0.15 - PR submitted by popular request

Please note I did not update the test suite due to lack of documentation

Hey @hilkeheremans
Thanks for the PR, it's so nice to discover that more people is interested in react-json. I thought I was the only one.
I have a deeper look at it this afternoon and merge it.


for updating tagnames is not selected in the SelectizeTextInput..please help me to solve this

and manually put  ** 'value' => 'munnar,kerala',**   for checking if its selected in the selectizetextinput
     <?= $form->field($model, 'tagNames')->widget(SelectizeTextInput::className(), [
    'loadUrl' => ['tag/list'],
    'options' => ['class' => 'form-control'],
        'value' => 'munnar,kerala',
    'clientOptions' => [
        'plugins' => ['remove_button'],
        'valueField' => 'name',
        'labelField' => 'name',
        'searchField' => ['name'],
        'create' => true,
    ],
])->hint('Use commas to separate tags') ?>

@vc7deo I do not have those issues at all. Have you make sure 'tagNames' is set in rules as 'safe' ? is it set? You should not set the 'value' but ensure the 'tagNames' is actually filled with imploded values when you display the form.
On another note, i see that you have not put the 'delimiter'. Do you mind updating your code with it?
<?= $form->field($model, 'zip')->widget(
    SelectizeTextInput::className(),
    [
        'options' => ['class' => 'form-control'],
        'clientOptions' =>
            [
                'plugins' => ['remove_button'],
                'create' => true,
                'delimiter' => ',',
                'persist' => false,
                'createOnBlur' => true
            ]
    ]
)->hint('Use commas to delimit zip codes') ?>


Remove the removal of newlines

Summary
The JSON serialization was removing newlines from the output. This caused issues when there were literal newlines in the JSON string and cause errors with HTML or XML content present in a JSON field value.
Other Information
Would like to know why this included in the first place as it seems arbitrary to remove on the toString
Thanks for contributing to the Watson Developer Cloud!

We may want to parse and stringify the json if we can't remove new lines.

@germanattanasio I'm confused by your comment. This code is generating JSON into a String. What is it that you are suggesting should be parsed?
Without this commit, this code also strips properly escaped newlines (that is \n) out of the data. This is damaging the data. I do not see why this is being done. Do you know what the goal is here?


plz help volume24H is 0 ? is it fix?

plz help volume24H is 0 ? is it fix?

Coinpaprika may not have the data of all the cryptocurrencies on the market. Also, I do know the context of your question.
Anyway, taking bitcoin for example, it does work fine.
from coinpaprika import client as Coinpaprika

client = Coinpaprika.Client()

candles = client.candles("btc-bitcoin", start="2022-01-01", limit=30, interval="24h")

for candle in candles:
    print(candle)

Here is a screenshot of the results.

This is just a wrapper. The data is provided by Coinpaprika. If they provide no data, feel free to contact their team.
I hope it answers your question. :sunglasses:

thank for answer
i use client.tickers() at 15 min
<EMAIL_ADDRESS>

i get all tickers at 15 min and filters by volume24h

I will repeat myself, but as I said, Coinpaprika is doing something wrong that is not something I can fix. It is Coinpaprika's responsibility at this point.
You can also test and see the experimentation for yourself in the JavaScript library made by someone else.
( async () => {
    const util = require('util');
    const cp = require( 'coinpaprika-js' );

    // Get tickers for all coins (USD,BTC,ETH)
    console.log(JSON.stringify(await cp.tickers(), null, 4));

} )();

Nice catch though! :)

thank you very much

Hi guys, thank you for the ticket. We heavily optimized our back-end infrastructure and introduced this bug, which is fixed now. Please feel free to contact us at<EMAIL_ADDRESS>in case of any further problems. Thank you for using API.

@r--w Awesome! Ticket closed.


[Question] Does the package support App Open Ads?

Hi, Id like to implement Open Ads in Flutter. I'm beginner in Flutter and Android dev as well. I would really appreciate it if you could provide some simple example.

This plugin only provides support for NativeAds, but you can try mine: https://pub.dev/packages/native_admob_flutter.
It has support for:

[x] Customizable Native Ad
[x] Banner Ad
[x] Interstitial Ad
[x] Rewarded Ad
[x] App Open Ads

How to create App Open Ads?


when ::new() is called error "No such file or directory" sets

Hi I'm seeing an issue where any time I call new() (in regards to registering a GPIO pin or I2C) I get the error "No such file or directory"  Im using the latest version of rrpal (0.11.3) and a Raspberry pi 2B.  Is this a known issue or am I missing files?  I dont think its the permissions issue since I'm not getting that error.
Thanks
Fred

My bad....it was a permissions thing no issues just me

Hi @fmc5046
I'm facing the same issue, can you share your solution?

Hi @fmc5046
I'm facing the same issue, can you share your solution?


Docs don't match latest release

I wanted to play with the project, and it seems to me the docs are built from the latest commit which contains classes and methods that are not yet available in the latest release.
For example, runtime.Store was added Aug 30, 2024, but 0.12 is from 19 July.
If the intent is to use jitpack snapshots, I think this could be mentioned in the readme.

Hi @abelsromero and thanks a lot for your interest in the project!
You are correct that docs and the latest release are quite different, sorry for that.
We are working hard to polish the public API to be able to release a 1.0.
For 0.0.12 you can refer to the docs at the tag: https://github.com/dylibso/chicory/tree/0.0.12
If you want to add instructions to the docs on how to use Jitpack, please 🙏 go ahead and I'll be happy to merge a PR.
That said, we plan to release a Milestone or RC next week, hopefully the situation will be better after that.

I'll look forward to the release.

@abelsromero we released 1.0.0-M1, I consider this fixed, let me know if something is missing.


"AFK"

Hello,
The text mentions: Starting with Windows 10, version 1809, Windows PE is released separately from the AFK.
Is "AFK" a typo for "ADK"?
Thanks,
t

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

ID: 7ab08370-69ab-d025-cea2-c41c3aa4b908
Version Independent ID: 9f5603f8-10e6-2651-3120-6d5627217cc9
Content: Prepare for deployment with MDT (Windows 10)
Content Source: windows/deployment/deploy-windows-mdt/prepare-for-windows-deployment-with-mdt.md
Product: w10
Technology: windows
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

Good catch. I have submitted a pull request.
Thanks


Chore/upgrade backend deps

Upgrade ruby, bundler and vulnerable dependencies.

I was annoyed by failing security job emails :D, maybe it should just write something on slack and not email me :P


Not show all elements with MapView in layout and in App

When added MapView in layout with any parent not show all elements in layout and in application too.
Android Studio v.2.2.2
. . .
compile 'org.osmdroid:osmdroid-android:5.5:release@aar'
. . .
How to fix ?
Thanks!

related #439

working ok over here with 2.2.1 using the api 24 renderer. Not sure. The preview thing has always been flaky. I've learned to live without it. It is nice to have but never something i've relied on

https://www.reddit.com/r/androiddev/comments/58ykvo/android_layout_editor_99_of_the_time/

works fine here. reopen if its still a problem

Thanks.
I to reinstalled Android Studio, updated her and this helped me.


Questions about training my own datasets

Hi! Thank you for your contributions to this repo!
Here, I have a question about training a model for a self-collected dataset. When I used this code, I modified the argument  self.num_classes=21 to self.num_classes=5 and other parts which are relative to #num_classes, remaining the rest unchanged. However, some of the classes' IoU turns to be 0 even in the train set. Also I turned down the ''semi'' setting and used a "supervised" mode and it turned out that there was no improvement on the previous mentioned classes' IoU.
There are two interesting facts: firstly, I trained with DeepLab-V3+ before and each of the class has non-zero IoU. Secondly, when I trained  the CCT model with self.num_classes=21 and other hyper-parameters remain the same as in this repo, each of the class has non-zero IoU, too. That is, if I simply changed self.num_classes(whatever the mode is), the performance of some classes collapsed.
Can you comment on this or give some advices on how to train with a different dataset? Thx!

Hi @WingsleyLui, thank you for your interest in our work;
Hmmm, weird behavior, to be honest, did you visualize/print the targets to make sure that there is no problem in the pre-processing, and the classes are present in the ground truth, and from 0 - 5. In general, make sure you have the expected results along the whole pipeline. In the beginning, only use supervised mode, if the mIoU of all classes > 0 then go to the semi model.
You can also try a adjusting the LR & more training epochs. You can also try a focal loss instead of a normal cross entropy.

@yassouali Thank you for your reply!
Actually I visualized the ground-truth and the prediction using Tensorboard and as indicated by the printed info, there was no such prediction in the class of zero IoU, although there exists some annotations of that class(which means the correct pre-processing). This was showed by tensorboard in visualization of supervised images, semi-supervised images and  validation images. It also happened when I used supervised mode only. However, when the number of classes was set to 21, all the classes have non-zero IoU and the visualization of predictions looks closer to the ground-truths.
Thank you for your suggestions and I'll try different loss function. Since a smaller learning rate didn't work and this collapse happened in the very beginning of the training process, it might be not helpful to increase the number of training epochs. Apart from this, do you have other thoughts like tuning other hyper-parameters?

Hi @WingsleyLui, thank you for your interest in our work;
Hmmm, weird behavior, to be honest, did you visualize/print the targets to make sure that there is no problem in the pre-processing, and the classes are present in the ground truth, and from 0 - 5. In general, make sure you have the expected results along the whole pipeline. In the beginning, only use supervised mode, if the mIoU of all classes > 0 then go to the semi model.
You can also try a adjusting the LR & more training epochs. You can also try a focal loss instead of a normal cross entropy.

Hi @WingsleyLui, thank you for your interest in our work;
Hmmm, weird behavior, to be honest, did you visualize/print the targets to make sure that there is no problem in the pre-processing, and the classes are present in the ground truth, and from 0 - 5. In general, make sure you have the expected results along the whole pipeline. In the beginning, only use supervised mode, if the mIoU of all classes > 0 then go to the semi model.
You can also try a adjusting the LR & more training epochs. You can also try a focal loss instead of a normal cross entropy.
@yassouali  Thank you for your reply!

Actually I visualized the ground-truth and the prediction using Tensorboard and as indicated by the printed info, there was no such prediction in the class of zero IoU, although there exists some annotations of that class(which means the correct pre-processing). This was showed by tensorboard in visualization of supervised images, semi-supervised images and validation images. It also happened when I used supervised mode only. However, when the number of classes was set to 21, all the classes have non-zero IoU and the visualization of predictions looks closer to the ground-truths.
Thank you for your suggestions and I'll try different loss function. Since a smaller learning rate didn't work and this collapse happened in the very beginning of the training process, it might be not helpful to increase the number of training epochs. Apart from this, do you have other thoughts like tuning other hyper-parameters?

@WingsleyLui
If you still don't have good results, I'd suggest trying to use deeplabv3+ as the base model instead of pspnet, given that deeplabv3+ is better, just make sure to follow the same procedure, by using multiple decoders and a shared encoder, the choice of the decoders / encoder parts is up to you as a design choice.

I have the same issue using my own data set. I ran the framework fully supervised and still have 0.0 IoU for my classes.

I only adjusted self.num_classes = 4 in voc.py
Is there more I should adjust?
However, when I leave self.num_classes = 21 and train on my own data set with 4 classes. The results seem good.

Of course, the mean IoU looks low because of the 0.0 classes, so I would need to calculate mIoU myself.


Enhancement: Wait feature in ccs

Maybe if there was a wait feature in ccs we could do more advanced things and make more commands work 100% of the time since I currently have one that mutes people for mentioning me (lmao)

Maybe like a {{wait:1s}}

I feel like necroposting in order to bump this up, this is now a thing with the delete trigger and response options yet is not yet its own function

Doable, closing


More linters

Integrate additional code formatters and liners into our pre-commit hooks and CI more generally.

Had to do remove a couple of the python moderinizing changes, since Pydantic doesn't play nice with
from __futures__ import annotations

See #1598


Lost one of the 12-word seed on multi sig

I have a multi sig wallet and lost one of the devices, which remained one of the combinations of 12 words, please help to restore the ability to work with a wallet, becuse I have an unspent funds

Hi there,
If your multi-sig wallet is 2-2 there is nothing to do.
If your multi-sig wallet is x-2 with x >= 3, all you have to do is use the recovery tool and use all the exported backup format available for you. In this case the 12 words will not work because the exported format have the public key ring for all of the copayers.
Hope that helps.

@Azverin
I can recover the lost words of your wallet if you want need help send me email<EMAIL_ADDRESS>


Add iperf3 to the performance test image

Add iperf3 to the performance test image for bandwidth benchmark implementation
Rename the image name apache-bench to a more generic name perftool

@antoninbas, Could you please upload this image to the antrea repo on docker hub?

/test-all

/test-all

/test-all


Add instructions for recovery after unexpected RabbitMQ shutdown

[x] This contribution adheres to CONTRIBUTING.md.
[ ]

What does this Pull Request accomplish?
RabbitMQ clusters in k8s have a known issue where the cluster can fail to restart after an unexpected shutdown. We are seeing this at customer sites. Adding documentation to our template on the recommended workaround for this issue.
Why should this Pull Request be merged?
Will help customers who encounter this issue in the future.
What testing has been done?
N/A

@prestwick @MelissaHilliard Pinging for attention on this review.


Advice and Question

Dear @bummoblizard,
What language do you use to program this application? If it is Swift, I can help, as well as transferring the app to Android, with Java. Just let me know if you need/want any extra help from me. Also, I noticed there are a lot of bugs in a short amount of time since the most recent update. I would suggest using Apple' bug testing program, which I don’t remember the app name, if iirc it is Test (something). Then I would send out testing invitations to certain people who you know would use it and give constructive feedback. If you need someone to help start bug testing/beta testing, just let me know. Try to get almost every bug squashed (more than this most recent update), and then release the update, even if it goes against your timeframe. Just some feedback on how the app versions work and happen, and how to prevent this many bugs. No offense meant, as well. Thanks for understanding!
Disclaimer: This was put here as a separate issue, just in case you didn’t see it in #32.
Best Wishes,
Lukas Batema

Hello Lukas,
The app is written in Swift and TypeScript. However, I don’t have a plan to open source this project or port it to Android as it’s still in its early stage. Thanks for the offer though!
Regarding the bugs, I have submitted an update to the App Store. It should be released in a day or two. TestFlight is also a great idea. I will post a link to it in the future.
Ken

Closing this for now.


Do not reuse SourceManager future if it failed

Previously, a failure to fetch the source would prevent the archive from ever being fetched since the the manager would just yield the failed future on subsequent requests.

https://ocd.appscale.com:8080/job/Daily Build/4890/


2561 signup working

Spinner looks good!  When I register, I get these errors in the console:
Uncaught TypeError: Cannot read property 'style' of null
    at PortfolioNavs.updatePortfolioNavHeight (eval at <anonymous> (main.957a588….js:5733), <anonymous>:63:24)
    at PortfolioNavs.updatePortfolioNavHeight (eval at <anonymous> (main.957a588….js:7948), <anonymous>:44:30)
    at later (eval at <anonymous> (main.957a588….js:738), <anonymous>:17:12)

(The signup completes successfully in spite of that error though.)

Ok, taking a look into those errors now.

OK, that error should be fixed up now.

Looks good!


Build failure: mkdocs-simple-hooks

Steps To Reproduce
Steps to reproduce the behavior:

build python311.packages.mkdocs-simple-hooks

Build log
https://hydra.nixos.org/build/247802209/nixlog/1
Additional context
An update of the nixos-23.11 channel broke mkdocs-simple-hooks and by extension ntfy-sh can no longer build.
Notify maintainers
@arjan-s
Metadata
Channel used :
    name: nixos-23.11
    url: https://releases.nixos.org/nixos/23.11/nixos-23.11.3920.25e3d4c0d359/nixexprs.tar.xz
    hash: 1wx3qxnz0pm3ld7k6gfvlxs7cv8dvj26917c3136ihvg7znfj1i8

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

It builds just fine on the release-23.11 and staging-next-23.11 branches.


[Feature request]: Support empty captions

I believe it can be beneficial to let Flux train on empty captions. But as it is, this is not allowed.
https://github.com/kijai/ComfyUI-FluxTrainer/blob/main/library/train_util.py#L1708
This line can be changed to:
caption = lines[0].strip() if lines else ""  # handle empty files
And now it allows empty caption files.
I don't know if this is good or bad for this training script.

Ah, okay.
Well, some setups use empty files. I have used this for training before.
I guess supporting both types of setups would just help more users?


Fail with error 'SafeERC20: low-level call failed'

I made some changes to the code (but not the smart contract) and once it found an arbitrage opportunity and it called the smart contract, it failed with the above error message.
Is there something wrong in the contract, or did I do something wrong on my side?
Any help would be much appreciated.
https://polygonscan.com/tx/0xa2685e50e596df610877ddf1f871b5443e232133f02a51310552f5c6d9e5b588

Not sure.  Some lending pools of dodo seem to be not working, so ..
Well it's hard to debug. .

I am a bit confused since it seems to be hardcoded what dodo pool to use:
https://github.com/yuichiroaoki/poly-flashloan-bot/blob/5e1cad274f9e94c36bef3e4c9b962292c15ecb5f/src/flashloan.ts#L33
Or am I wrong?

Yeah that's something needs to be changed. I haven't tested the code with other dodo pools.


Unable to do upload_folder

Describe the bug
api.upload_folder doesn't work and fail with an error related to either wrong token or invalid repo name.
The repo is a public one and I verified that the token is a writable token, so I'm not very sure where has gone wrong. My request is to upload a folder to the repo's root directory.
Reproduction
import huggingface_hub as hf

def upload_folder_to_hf(repo_id, repo_type, token, folder):
    try:
        hf.hf_api.repo_info(repo_id, repo_type=repo_type)
        print("Repo found")
    except hf.hf_api.RepositoryNotFoundError:
        hf.create_repo(repo_id, repo_type=repo_type)

    api = hf.HfApi()
    api.upload_folder(
        folder_path=folder,
        path_in_repo=".",
        repo_id=repo_id,
        repo_type=repo_type,
        ignore_patterns="**/logs/*.txt",
    )

upload_folder_to_hf("xianbao/test-upload", "dataset", "hf_xxxx",
                    "work")

Logs
Repo found
---------------------------------------------------------------------------
HTTPError                                 Traceback (most recent call last)
File ~/.local/lib/python3.10/site-packages/huggingface_hub/utils/_errors.py:264, in hf_raise_for_status(response, endpoint_name)
    263 try:
--> 264     response.raise_for_status()
    265 except HTTPError as e:

File /usr/local/lib/python3.10/dist-packages/requests/models.py:1021, in Response.raise_for_status(self)
   1020 if http_error_msg:
-> 1021     raise HTTPError(http_error_msg, response=self)

HTTPError: 401 Client Error: Unauthorized for url: https://huggingface.co/api/datasets/xianbao/test-upload/preupload/main

The above exception was the direct cause of the following exception:

RepositoryNotFoundError                   Traceback (most recent call last)
Cell In[4], line 19
     10     api = hf.HfApi()
     11     api.upload_folder(
     12         folder_path=folder,
     13         path_in_repo=".",
   (...)
     16         ignore_patterns="**/logs/*.txt",
     17     )
---> 19 upload_folder_to_hf("xianbao/test-upload", "dataset", "hf_xxxx",
     20                     "work")

Cell In[4], line 11, in upload_folder_to_hf(repo_id, repo_type, token, folder)
      8     hf.create_repo(repo_id, repo_type=repo_type)
     10 api = hf.HfApi()
---> 11 api.upload_folder(
     12     folder_path=folder,
     13     path_in_repo=".",
     14     repo_id=repo_id,
     15     repo_type=repo_type,
     16     ignore_patterns="**/logs/*.txt",
     17 )

File ~/.local/lib/python3.10/site-packages/huggingface_hub/utils/_validators.py:124, in validate_hf_hub_args.<locals>._inner_fn(*args, **kwargs)
    119 if check_use_auth_token:
    120     kwargs = smoothly_deprecate_use_auth_token(
    121         fn_name=fn.__name__, has_token=has_token, kwargs=kwargs
    122     )
--> 124 return fn(*args, **kwargs)

File ~/.local/lib/python3.10/site-packages/huggingface_hub/hf_api.py:2699, in HfApi.upload_folder(self, repo_id, folder_path, path_in_repo, commit_message, commit_description, token, repo_type, revision, create_pr, parent_commit, allow_patterns, ignore_patterns)
   2686 commit_message = (
   2687     commit_message
   2688     if commit_message is not None
   2689     else f"Upload {path_in_repo} with huggingface_hub"
   2690 )
   2692 files_to_add = _prepare_upload_folder_commit(
   2693     folder_path,
   2694     path_in_repo,
   2695     allow_patterns=allow_patterns,
   2696     ignore_patterns=ignore_patterns,
   2697 )
-> 2699 commit_info = self.create_commit(
   2700     repo_type=repo_type,
   2701     repo_id=repo_id,
   2702     operations=files_to_add,
   2703     commit_message=commit_message,
   2704     commit_description=commit_description,
   2705     token=token,
   2706     revision=revision,
   2707     create_pr=create_pr,
   2708     parent_commit=parent_commit,
   2709 )
   2711 if commit_info.pr_url is not None:
   2712     revision = quote(_parse_revision_from_pr_url(commit_info.pr_url), safe="")

File ~/.local/lib/python3.10/site-packages/huggingface_hub/utils/_validators.py:124, in validate_hf_hub_args.<locals>._inner_fn(*args, **kwargs)
    119 if check_use_auth_token:
    120     kwargs = smoothly_deprecate_use_auth_token(
    121         fn_name=fn.__name__, has_token=has_token, kwargs=kwargs
    122     )
--> 124 return fn(*args, **kwargs)

File ~/.local/lib/python3.10/site-packages/huggingface_hub/hf_api.py:2333, in HfApi.create_commit(self, repo_id, operations, commit_message, commit_description, token, repo_type, revision, create_pr, num_threads, parent_commit)
   2330 warn_on_overwriting_operations(operations)
   2332 try:
-> 2333     upload_modes = fetch_upload_modes(
   2334         additions=additions,
   2335         repo_type=repo_type,
   2336         repo_id=repo_id,
   2337         token=token or self.token,
   2338         revision=revision,
   2339         endpoint=self.endpoint,
   2340         create_pr=create_pr,
   2341     )
   2342 except RepositoryNotFoundError as e:
   2343     e.append_to_message(_CREATE_COMMIT_NO_REPO_ERROR_MESSAGE)

File ~/.local/lib/python3.10/site-packages/huggingface_hub/utils/_validators.py:124, in validate_hf_hub_args.<locals>._inner_fn(*args, **kwargs)
    119 if check_use_auth_token:
    120     kwargs = smoothly_deprecate_use_auth_token(
    121         fn_name=fn.__name__, has_token=has_token, kwargs=kwargs
    122     )
--> 124 return fn(*args, **kwargs)

File ~/.local/lib/python3.10/site-packages/huggingface_hub/_commit_api.py:477, in fetch_upload_modes(additions, repo_type, repo_id, token, revision, endpoint, create_pr)
    459 payload = {
    460     "files": [
    461         {
   (...)
    468     ]
    469 }
    471 resp = requests.post(
    472     f"{endpoint}/api/{repo_type}s/{repo_id}/preupload/{revision}",
    473     json=payload,
    474     headers=headers,
    475     params={"create_pr": "1"} if create_pr else None,
    476 )
--> 477 hf_raise_for_status(resp)
    478 preupload_info = _validate_preupload_info(resp.json())
    479 upload_modes.update(
    480     **{file["path"]: file["uploadMode"] for file in preupload_info["files"]}
    481 )

File ~/.local/lib/python3.10/site-packages/huggingface_hub/utils/_errors.py:306, in hf_raise_for_status(response, endpoint_name)
    292 elif error_code == "RepoNotFound" or response.status_code == 401:
    293     # 401 is misleading as it is returned for:
    294     #    - private and gated repos if user is not authenticated
    295     #    - missing repos
    296     # => for now, we process them as `RepoNotFound` anyway.
    297     # See https://gist.github.com/Wauplin/46c27ad266b15998ce56a6603796f0b9
    298     message = (
    299         f"{response.status_code} Client Error."
    300         + "\n\n"
   (...)
    304         " make sure you are authenticated."
    305     )
--> 306     raise RepositoryNotFoundError(message, response) from e
    308 elif response.status_code == 400:
    309     message = (
    310         f"\n\nBad request for {endpoint_name} endpoint:"
    311         if endpoint_name is not None
    312         else "\n\nBad request:"
    313     )

RepositoryNotFoundError: 401 Client Error. (Request ID: Root=1-63d7debc-5294510e30afd2464273197e)

Repository Not Found for url: https://huggingface.co/api/datasets/xianbao/test-upload/preupload/main.
Please make sure you specified the correct `repo_id` and `repo_type`.
If you are trying to access a private or gated repo, make sure you are authenticated.
Invalid username or password.
Note: Creating a commit assumes that the repo already exists on the Huggingface Hub. Please use `create_repo` if it's not the case.

System info
Copy-and-paste the text below in your GitHub issue.

- huggingface_hub version: 0.12.0
- Platform: Linux-5.4.209-116.367.amzn2.x86_64-x86_64-with-glibc2.35
- Python version: 3.10.6
- Running in iPython ?: No
- Running in notebook ?: No
- Running in Google Colab ?: No
- Token path ?: /home/admin/.cache/huggingface/token
- Has saved token ?: False
- Configured git credential helpers:
- FastAI: N/A
- Tensorflow: N/A
- Torch: 1.13.1
- Jinja2: 3.1.2
- Graphviz: N/A
- Pydot: N/A
- Pillow: 9.4.0
- hf_transfer: N/A
- ENDPOINT: https://huggingface.co
- HUGGINGFACE_HUB_CACHE: /home/admin/.cache/huggingface/hub
- HUGGINGFACE_ASSETS_CACHE: /home/admin/.cache/huggingface/assets
- HF_HUB_OFFLINE: False
- HF_TOKEN_PATH: /home/admin/.cache/huggingface/token
- HF_HUB_DISABLE_PROGRESS_BARS: None
- HF_HUB_DISABLE_SYMLINKS_WARNING: False
- HF_HUB_DISABLE_IMPLICIT_TOKEN: False
- HF_HUB_ENABLE_HF_TRANSFER: False

Hi @xianbaoqian :wave: I'm not what is the root problem of your issue but it might be related to the token not been passed to HfApi. In your code, you have a token parameter in upload_folder_to_hf that is never used. By default, repo_info and create_folder will look at the token saved on your machine (either using huggingface-cli login or from huggingface_hub import login; login()). Could it be that the token you are passing to the function has write-access but not the one on your machine ?

Ah @Wauplin  you're awesome! Sry I forgot that I did a copy and was using the wrong version!  Thanks for catching that.

Good, you're welcome :)


STORY: As a user, I can send a personal message to the admins so that they'll contact me back.

This mission will be broken down into three parts:

frontend- The app will include a "contact us" page. the form will contain fields such as; Name, Email, Subject, and content.
backend- form will be fully functional (the contact details will be forwarded/saved)
integration- the contact details will be sent to the app's SQL server and be contained within a table.

I'll deal with parts 1 (#51 ) and 2
I and @BarakMerdler will collaborate on part 3


[BUG] Can Firelord run on firebase functions?

Version?
"firebase-admin": "^11.5.0",
"firebase-functions": "^4.2.1",
"firelord": "^2.3.6",

What is the Meta Type?
not relevant

What operations are you trying to run?
not relevant

What do you expect?
I would like to use firelord in a firebase function

What do you see?
I receive an error when deploying the function code to firebase (see: 👇)
This import seems to be the problem: (https://github.com/tylim88/Firelord/blob/f4eb307ae0df0503dc5085566f8367aed05cd378/src/index.ts#L62)
Actually I have never used firebase/firestore never before.

is it a possible to run firelord on firebase functions?
Why do get this error? Do I need to install firebase/firestore in my firebase functions?

ah damn

please use 2.3.7

👍 ...deployment works. Thank you very much


Using PortableDid to import did to sign VC, removing kid as default value

test vector changes for https://github.com/TBD54566975/web5-kt/pull/262

ty for testing this in both JS and KT

@kirahsapong are you aware of this PR? Will the changes here impact web5-swift?


Bug in your MyFragment class

You have the following code:
mBottomSheetContentView.setOnClickListener(new OnClickListener()
  {
  @Override
  public void onClick(final View v)
    {
        mBottomSheetLayout.expandSheet();
    }
  });

The correct for me is this one:
mBottomSheetContentView.setOnClickListener(new OnClickListener()
  {
  @Override
  public void onClick(final View v)
    {
        if (mBottomSheetLayout.getState() == State.PEEKED) mBottomSheetLayout.expandSheet();
    }
  });

Because if you click the mBottomSheetContentView (in the imageView or in the textView) when the state is State.EXPANDED, you have a bug that resizes the image and puts the other textview element in a wrong position

As I remember, the imageView isn't visible when it's expanded, because it only appears at the bottom, when it's already peeked.
Anyway, Google has provided a new official API for bottom sheet:
http://android-developers.blogspot.co.il/2016/02/android-support-library-232.html
I hope it's possible to do what I did, using their API.

I will try to make a video for explain better, sorry for my english.
Thanks for the response! I know the api it's only a little bug in your code.


Does it support the automatic calculation of position when x and y have no value?

X and Y values are set to 0 as default.But a layout definition should be correct. I can not handle all possible variations but I have to assume that the layout given is correct.


fix(javadoc): removed erronous tag

What this PR changes/adds
removes an {@param ....} tag from the javadoc.
Why it does that
Gradle's javadoc task complained, causing the CI build to fail.
https://ci.eclipse.org/edc/job/Playground/job/Build-Component-Template/19/console
Checklist

[ ] added appropriate tests?
[ ] performed checkstyle check locally?
[ ] added/updated copyright headers?
[ ] documented public classes/methods?
[ ] added/updated relevant documentation?
[ ] assigned appropriate label? (exclude from changelog with label no-changelog)
[x] formatted title correctly? (take a look at the CONTRIBUTING and Etiquette for pull requests for details)

Codecov Report
Base: 64.16% // Head: 64.16% // No change to project coverage :thumbsup:

Coverage data is based on head (ebed005) compared to base (3cc3101).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2264   +/-   ##
=======================================
  Coverage   64.16%   64.16%
=======================================
  Files         823      823
  Lines       17326    17326
  Branches     1082     1082
=======================================
  Hits        11117    11117
  Misses       5766     5766
  Partials      443      443

Impacted Files
Coverage Δ

...ava/org/eclipse/edc/spi/result/AbstractResult.java
82.60% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Realpath error with 1.64.0  OSX, ZSH

Describe the bug
looks like this is the same as this issue here about real_path
https://github.com/antonbabenko/pre-commit-terraform/issues/336

How can we reproduce it?
zsh --version == zsh 5.8 (x86_64-apple-darwin21.0)
OSX macOS Monterey  12.0.1
terraform --version == 13.7
Help us to reproduce your bug as succinctly and precisely as possible. Any and
all steps or script that triggers the issue are highly appreciated!
Do you have long logs to share? Please use collapsible sections, that can be created via:
Make Terraform Changes that require terraform fmt to be performed

 2022-03-14 14:33:16 ⌚  NYC3-0606 in ~/Code/codebase
± |stanleyz/branch-readd {11} S:9 U:2 ?:8 ✗| → git commit -m 'readding foo'
[WARNING] Unstaged files detected.
[INFO] Stashing unstaged files to /Users/stanleyzheng/.cache/pre-commit/patch1647282801-9301.
Check python ast.........................................................Passed
Terraform fmt............................................................Failed
- hook id: terraform_fmt
- exit code: 1

/Users/stanleyzheng/.cache/pre-commit/repoccjwtpbf/hooks/terraform_fmt.sh: line 9: realpath: command not found
/Users/stanleyzheng/.cache/pre-commit/repoccjwtpbf/hooks/terraform_fmt.sh: line 11: ./_common.sh: No such file or directory
/Users/stanleyzheng/.cache/pre-commit/repoccjwtpbf/hooks/terraform_fmt.sh: line 9: realpath: command not found
/Users/stanleyzheng/.cache/pre-commit/repoccjwtpbf/hooks/terraform_fmt.sh: line 11: ./_common.sh: No such file or directory

pytest-check.............................................................Passed
[INFO] Restored changes from /Users/stanleyzheng/.cache/pre-commit/patch1647282801-9301.

## Pre-commit file

repos:
-   repo: https://github.com/psf/black
    rev: 22.1.0
    hooks:
    - id: black
      language_version: python3
-   repo: https://github.com/pre-commit/pre-commit-hooks
    rev: v1.2.3
    hooks:
    - id: check-ast
    - id: flake8
- repo: https://github.com/antonbabenko/pre-commit-terraform
  rev: v1.63.0
  hooks:
    - id: terraform_fmt

-->
Environment information

OS:

zsh --version == zsh 5.8 (x86_64-apple-darwin21.0)
OSX macOS Monterey  12.0.1
terraform --version == v0.13.7`

* `.pre-commit-config.yaml`:

sorry about the reformatting.. should be good now but yes
Mirroring my comment
I tried digging into the source of the error but im confused at where to start. I had pre-commit terraform fmt work correctly sometime so this was confusing when I ran into this issue.
My coworker has also run into this issue so it is reproducible.

saw issue was closed, i was able to get this to work.
As mentioned in the other thread, this will likely be fixed with v1.65
@MaxymVlasov I think it still should be called out that coreutils need to be installed as a tool.
i did not have it so its a silent dependency?

https://github.com/antonbabenko/pre-commit-terraform#1-install-dependencies -> MacOS

Ah, OK, need to update README, because now it used by all hooks


fixes #671 and add pass exception to after_request hook callback when it it occured.

First, before_request and after_request is very useful hooks for resource management, but after_request must know has any exception is raised while request be processing, for decide how to release resource, commit or rollback database transaction for example.
Then, I read @vascop 's PR https://github.com/bottlepy/bottle/pull/702/files and got some inspire. But his code still have some bugs, such as when RouteReset is raise, after_request hook callback will be called twice.
So, I extracted some code from _handle() to _inner_handle(). Collected exception and pass it to after_request hook callback, if not error, the argument exc will be None.

(for inter-issue linking: #671)

before_request and after_request is very useful hooks for resource management

Arguably, that role can also be achieved with app.install(plugin), although there are problems with that too (#663)

Yes, Eric, I agree with you.
As you said, Plugin is not so good, let's provide more choices to users. :)

Is inner_handle used more than once? If not I's inline it again. Function calls and dot lookups are expensive.


[SQL][DOC][MINOR] Add (Scala-specific) and (Java-specific)

What changes were proposed in this pull request?
Adds (Scala-specific) and (Java-specific) to Scaladoc.
How was this patch tested?
local build

Test build #64708 has started for PR 14891 at commit a927c83.

I think this kind of thing is fine to call out, but we haven't used this convention, quite, elsewhere. The APIs are all "Scala-specific" unless otherwise noted. Even then they're callable from Java, just not very conveniently. There are some methods that exist specifically to make calling from Java simpler. In other cases I've just seen this described with a sentence of prose that says this is a convenience method for Java callers. That might be more explanatory than just writing "Java-specific", because indeed even those are callable from Scala and aren't that specific to Java.
I'd just note the methods that are Java-friendly, and borrow a sentence from existing doc of similar methods.

Test build #64708 has finished for PR 14891 at commit a927c83.

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

Merged build finished. Test PASSed.

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

Frankly, I'd not have bothered with the changes if I had not seen them elsewhere. See http://spark.apache.org/docs/latest/api/scala/index.html#org.apache.spark.sql.Dataset and look for foreach and foreachPartition for (Java-specific) while http://spark.apache.org/docs/latest/api/scala/index.html#org.apache.spark.sql.Encoders$ and javaSerialization /  kryo for (Scala-specific). I have seen examples with both in one source like http://spark.apache.org/docs/latest/api/scala/index.html#org.apache.spark.sql.RelationalGroupedDataset. That was the main reason while the other was to understand why there are two different variants of the same method (and then it clicked but only when I'd seen the other examples).

OK fair enough LGTM

@srowen, anyone you'd recommend to accept the PR (after you accepted)?

Merged to master


Slack module upgrade

The version 3.x of slack module was breaking things in my end, so I upgraded to v4.x. See https://github.com/slackapi/node-slack-sdk/wiki/Migration-Guide-for-v4 for further details.

Some basic testing added.

Thank you for this! Looked over the code and everything looks good; I’ll get it published on NPM tomorrow.


Missing copyright info about data publisher in map charts

Describe the bug
In the map chart, the copyright info about the data publisher is missing.
To Reproduce
Steps to reproduce the behavior:

Create a map chart...
or go to this exemple
notice the missing "SFOE" copyright notice (see screenshot below)

Expected behavior
The publishing body should be integrated into the Copyright line of the map
Screenshots

Environment (please complete the following information):

Visualize environment and version: TEST v3.4.8 (25036c0)
Browser and version: Edge

Additional context
None

Dear @AFoletti
Thank you for reporting the issue. At the moment, based on the current concept, we show the information about the data publisher below the chart (see attached image). The copy right data shown in the map relates to the map data itself.

Dear @sosiology
many thanks for your feedback!
Yes, I am aware of the "souce" below the map.
Nevertheless, I allow myself to disagree when you make a distinction between the "map data" (as in: the base layer) and "the data" (as in: my blue dots on the map). My blue dots are 100% integral part of the map and actually the reason why the map exist.
The swiss geoportal uses the same approach, dynamically adding infos in the map while you add data from different publishers

This remains a very low prio issue for us, and we can very well live with the actual state of things. I'd be however happy if you could spend some time considering my points. 😄
Best,
A,


[UWP] Add event for "Height changed, host should re-layout card"

When the user does something that causes the card's height to change (like clicking on a ShowCard action), hosts (like Action Center) need to know that the height of the card has changed so that they can re-position the card.
This was previously added to HTML version so bots could scroll down to ensure the shown card is visible. Similar is needed for UWP.
This event needs to fire AFTER the layout has actually updated. It can't be fired immediately when the ShowCard click occurred, since the layout hasn't actually changed (XAML waits for the current UI thread to exit before processing layout changes).

Removing toasts tag as Action Center will just use SizeChanged event, so we don't need this feature.


Added binary support. https://github.com/hmalphettes/redisdown/issues/17

Hey,
as we discussed, I added binary support.
Please review my changes and test if it doesn't break things.
I did some tests and it seems like it works fine.
Another thing I'd recommend for your next version, is to take out all JSON serialization. JSON serialization made in LevelUp.
Cheers.

I fixed more stuff...
Basically the problem with storing buffers as values happens because you perform json serialization on values.
I deleted all the instances of JSON serialization. Levelup is responsible for serializing keys/values into strings or buffers.
Redisdown should be responsible only for persisting it.
I can't think of a usecase where levelup instance would request persisting a JSON value from a leveldown instance.
Anyway, I deleted all the json serialization stuff.
It passes all tests.

Many thanks for your contribution @tounano much appreciated.


Async unit tests

Since we now have excellent Promise and Future support, we should start thinking about how to write async unit tests.
@koute you said you had some ideas about this, what are they?

Yes, well, it shouldn't be super hard to get async tests working today, it'd just involve quite a bit of legwork.
The biggest issue currently with having async tests is that Rust itself doesn't support custom test harnesses, so we'd have to get around that.
Something like this:

Write a procedural attribute macro, in a fashion similar to the current #[js_export] - call it #[async_test] or something like that. For every function marked with it something like this would be generated: (would require #![feature(linkage)])

#[linkage = "external"]
#[no_mangle]
#[allow(dead_code)]
extern fn test_foobar() {
    // ...
}

Then you could just slap an #[async_test] anywhere you want in your project just as you'd do with #[test].

When compiling tests have cargo-web extract all of the async tests, generate a custom test harness for it and then run them.

I don't think we should involve promises and futures into this though, at least on the lowest level; having fn test_foobar(done: fn()) would be enough (then you could test promises and futures themselves like this, and most of our async APIs don't need full power of promises/futures to test them anyway)

I don't think we should involve promises and futures into this though

I disagree on the futures point - I think we should push towards ubiquitous use of futures and streams for async programming within stdweb - I imagine we'll get some form of await usable on stable this year, and I'd like to be able to take full advantage of that with stdweb.
I want to be able to slap #[async_test] directly on an async function and have it behave as expected. I think we should even push for this async test harness to be supported upstream of stdweb, and I think there will be wide support for this amongst all rust users.
The way I see it, stdweb should be the gateway between the javascript/browser/node world and the rust world, providing interfaces and compatibility layers where appropriate, but on the rust side of that boundary we should have "normal" rust code, where the best practices for rust on stdweb are the same as the best practices for any rust project.

@koute I don't think we should involve promises and futures into this though, at least on the lowest level

I agree that the lowest level should be very simple, but testing Futures will be very common, so we need some sugar on top which makes it easy.
Perhaps something like this:
#[async_callback_test]
fn foo(done: fn()) {
    // ...
}

#[async_future_test]
fn foo() -> impl Future<Item = (), Error = Error> {
    // ...
}

@Diggsey I think you've misunderstood me. (:
We should definitely support promises in async tests for user code! What I'm saying is that:

the very basic mechanism of async tests should not depend on futures, and that it should be possible to write async tests without requiring futures,
async tests in stdweb itself should not use futures - I'd like to keep the futures feature optional, and I don't think we actually need the power of futures anywhere.

@Pauan I don't think two attributes are necessary; the procedural attribute macro has access to the function's signature, so it can figure out which type of test to generate based on that (although, as I've said - the future based tests should be implemented on top of done: fn() ones, so the generated code would be mostly the same, with the difference that the futures one would desugar to the done: fn() one)

Also, one more thing - it'd be nice to have access to #[linkage = "external"] on stable (to have the ability to export functions from anywhere, not only from lib.rs or main.rs), and I don't believe that it's possible to do that in any other way except #![feature(linkage)]. Sooner or later I'll probably whip up an RFC with a proposal to stabilize this functionality in some way.

@koute Yup, we're in agreement: futures should desugar to done: fn(), and you make a good point about being able to dispatch based on the return type, I like it.

Also, I'm currently busy with some other things, so I won't be able to tackle this anytime soon, I just wanted to get the ball rolling, since I think this is a very important issue for stdweb (both within stdweb and also for users of stdweb).

I've just added experimental support for async tests to cargo-web. (It works across every --target, both under node.js and under headless Chromium.) I'll push a procedural attribute macro soon to stdweb.

And the #[async_test] procedural macro is here.
Example test: (not yet commited since I want to possibly refactor the Promise while I'm at it)
#[cfg(all(test, rust_1_30_or_newer, rust_nightly))]
mod async_tests {
    use super::*;
    use webapi::global::set_timeout;
    use async_test;

    fn new_promise() -> Promise {
        js!(
            var resolve_fn;
            var reject_fn;
            var promise = new Promise( function( resolve, reject ) {
                resolve_fn = resolve;
                reject_fn = reject;
            });
            promise.call_resolve = resolve_fn;
            promise.call_reject = reject_fn;

            return promise;
        ).try_into().unwrap()
    }

    #[async_test]
    fn test_promise_done< F: FnOnce( Result< (), String > ) >( done: F ) {
        let promise = new_promise();
        promise.done( |result: Result< i32, () >| {
            match result {
                Ok( 123 ) => done( Ok(()) ),
                result => done( Err( format!( "unexpected value: {:?}", result ) ) )
            }
        }).leak();

        set_timeout( || {
            js!( @{promise}.call_resolve( 123 ); );
        }, 0 );
    }
}


Input (clicks) not working in current release

I'm unable to activate any menu items in the functional test app.

Altspace: Current release on Steam
Account: Logged in as eanders-ms
Functional Tests: master branch running locally

When I click on a functional test menu item, nothing happens. Looks like there's an associated error in the Altspace client log:
ArgumentNullException: Value cannot be null.
Parameter name: User cannot be null when performing an action
  at MixedRealityExtension.Behaviors.Actions.MWAction.UpdateAction (MixedRealityExtension.Core.Interfaces.IUser user, MixedRealityExtension.Behaviors.Actions.ActionState newState) [0x00003] in <9333852053d24b18b052d53eeb1ab451>:0
  at MixedRealityExtension.Behaviors.Actions.MWAction.StopAction (MixedRealityExtension.Core.Interfaces.IUser user) [0x00000] in <9333852053d24b18b052d53eeb1ab451>:0
  at Assets.Altspace.Features.MREApps.Scripts.Behaviors.MREButtonBehavior.<MREButtonBehavior>m__1 (System.Boolean _, System.Boolean isPressed) [0x0003c] in <ea54653d4d5e4e75a3234405367d3e95>:0
  at AbstractBaseValue`1[T].FireValueChanged (T oldValue, T newValue) [0x00022] in <b63311e65b1949cc838842546badadf8>:0
UnityEngine.DebugLogHandler:Internal_LogException(Exception, Object)
UnityEngine.DebugLogHandler:LogException(Exception, Object)
UnityEngine.Logger:LogException(Exception, Object)
UnityEngine.Debug:LogException(Exception)
AbstractBaseValue`1:FireValueChanged(Boolean, Boolean)
ValueEventHelper`1:FireIfValueChanged()
ComputedValue`1:FireIfChanged()
TargetableButtonBehaviour:@??@?@A@AAA?@A?@BB@@?AA(TargetableButtonTool, Boolean)
TargetableButtonTool:?B@A?A?BBB?AB@BBA??A@B@(TargetableButtonBehaviour, TargetableButtonBehaviour)
AbstractBaseValue`1:FireValueChanged(TargetableButtonBehaviour, TargetableButtonBehaviour)
MutableValue`1:set_Value(TargetableButtonBehaviour)
BaseMutableValue`1:Set(TargetableButtonBehaviour)
TargetableButtonTool:AA@BB@BAB@A??A@?AA?@ABA(TargetableButtonBehaviour)
TargetableButtonTool:Update()

More info: The repro seems to involve re-entering a space. Could it be that after re-entering, MRE click behaviors no longer work?

I've never been able to reliably repro this bug, but it seems to be related to any unhandled exception in MRE-land. Can also repro this bug by attempting to load an invalid library object.

Yeah.  I don't think this bug is accurately titled to be input.  Input is just one of the most common places we see this.  I believe this is related to the frozen app state we get in to because of some unhandled exception.

Regarding the bug title: the callstack in the client log is related to input, so at least in this instance it is input-specific.
Will see if I can repro again later today. I was hitting it frequently last night, but not 100%.

It looks like I have a reliable repro:

Launch Altspace and go to your home space
Spawn the funcitonal test app in your homespace
Try to click on a test menu button - it should work
Re-enter the space
Try again to click on a test menu button - if the bug repros, it should not work this time

I think I tracked it down (see associated Altspace PR)


Apertium RDF - more metadata in lexicons and translations sets

I would add dc:source, rdrf:label and rdfs:comment in every lime:Lexicon and vartrans:TranslationSet individual
e.g. :
dc:source <https://github.com/apertium/apertium-eng-spa  > ;
 rdfs:label "Apertium EN Lexicon"@en ;
 rdfs:comment "This is an RDF OntoLex-lemon lexicon in English that comes from the original  Apertium bilingual dictionaries."@en .

apparently for TranslationSet the metadata is there, pending for Lexicon