User_agent library returning incorrect browser on my andriod chrome

My user agent: Mozilla/5.0 (Linux; Android 8.0.0; Mi A1 Build/OPR1.170623.026) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/66.0.3359.126 Mobile Safari/537.36
The _set_browser function at https://github.com/bcit-ci/CodeIgniter/blob/3.1.8/system/libraries/User_agent.php#L297 actually compares strings from $browsers array in the config file (https://github.com/bcit-ci/CodeIgniter/blob/3.1.8/application/config/user_agents.php#L61) of the user agent. In my case the phone build number has the OPR (Oreo build, primary platform branch, built from code branch r1) string in it which gets matched with the OPR key of the $browsers array returning browser as opera. Maybe you can make the preg_match stricter by not comparing the $browsers array keys to the build number number

Do you have an idea how to do that?

https://developer.chrome.com/multidevice/user-agent#chrome_for_android_user_agent this could help (check there the "Phone pattern" and "Tablet pattern")

@narfbg @hamzawain7
I have the same problem in Asus zenfone 4:
User Agent: MyMozilla/5.0 (Linux; Android 8.0.0; ASUS_Z01KD Build/OPR6.170623.013) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/67.0.3396.87 Mobile Safari/537.36
Browser Chrome 67 / Android 8
Detected Browser  Opera 6

Ok, I'll be blunt about this ...
The UA library is legacy crap that (depending on the use case) a) doesn't belong on the server-side; b) you probably don't need; and (not depending on the use case) c) I am not interested in supporting.
So when I asked for an idea about this, I was really asking for a concrete solution, preferrably in a pull request. Sorry to disappoint, but if you wait on me to fix this, you're probably never going to see it ...

I know this is an old question and @narfbg  stated he would only "fix" this if someone submitted a pull request, but as I'm not sure where the best way would be to implement this, I'll to explain it here as best as I can.
The regex that check the useragent looks like this:
$key.*?([0-9\.]+) where the $key variable is from the list of browsers in the config file.
And while checking the user agents for opera browser you can see that they all have OPR/{version} or OPERA/{version} or something similar to this patter.
So what if in the list of browsers where you have the key OPR we just added OPR\/ so it would force regex to check for / (forward slash) as well when looking for opera browsers.
This way it will match the user agents properly with the one specified in the comments above and using some sample agents for opera matches them correctly as well.
Do point out if something is wrong with this approach :)

Sounds fine. The problem with altering the library logic itself is that a lot of patterns are already build to rely on it and that may break them; just another pattern entry isn't a risk.


Update index.d.ts

Hey everyone,
The names ScrollViewProperties, FlatListProperties and SectionListProperties are deprecated and replaced with ScrollViewProps, FlatListProps and SectionListProps respectively.
thanks

Nice fix! When will this be merged?


Substandard performance without postprocessing.

Good Evening,
There seems to be lot of rule based post processing being done after getting the model outputs. (Forcing prediction to be snowboarding if snowboard is present etc.). Can you report the numbers without such post processing being applied? Evaluating the model after removing post-processing is giving me very bad results, so I'm not sure if I'm doing it properly. I couldn't find any mention of such numbers in the BMVC paper.
Thank you!

Thanks for the question.
We follow the processing steps by prior work (Visual Semantic Role Labeling, Detecting and Recognizing Human-Object Interactions) to remove prediction scores for certain action-object pairs. You can see in the eq2 of Visual Semantic Role Labeling, C is predefined. Thus, we know the object categories that are related to a specific action class.
You can simply add a --prior_flag 0 flag to the Train_ResNet_VCOCO.py, i.e. using the following command
python tools/Train_ResNet_VCOCO.py --model iCAN_ResNet50_VCOCO --num_iteration 300000

to obtain the mAP without any post-processing.

Thank you for the quick response!
I ran the testing pipeline with the above command using prior_flag = 0.
On the pre-trained model provided, the mAP values obtained with prior flag = 0 are 2.38(Scenario 1) and 4.81(Scenario 2).  Is such a huge drop expected? Compared to the reported mAP of 45, these are very surprising numbers.
Is the contribution of this prior knowledge 'C'  is so significant?
Thank you!

Good Morning,
Any information on this?
Thank you!

As I recall, we will lose around 3mAP without processing. The mAP should be around 42.
I will check and see where is the problem.

Good Morning,
Any information on this?
Thank you!

hi, man,do you know the metric for hico-det

Good Morning,
Any information on this?
Thank you!

hi, man did you test the vcoco or the hico-det datasets, and how did you test the model, I know just
needs to test hico-det model, if you know, can you help me?
oh, thank you very much.


Bug - Backspace triggers validateEmail handler

Hello 👋
I have a usecase where I only want the "Enter" key to trigger submission/validateEmail handler. I've tried:

Setting 13 as the delimiter
Not setting a delimiter

But it looks like the "Enter" and "Backspace" key both always trigger submission/validation. Is this a bug?
It would also be really nice if we could somehow pass the key that was pressed to the validateEmail handler. But I guess this wouldn't be as useful if we delimiter would work as intended.
Thank you for looking into this!

You can use on 1.0.14

Thanks sooooo much!!
One other thing I've noticed is that when I click outside of the input, the validateEmail handler also triggers. Is there a way you know of that I can prevent this?

It is probably validating as the blur event occurs.
Let's see if exception handling for blur event is possible.


Speedup logging

Codecov Report
Attention: Patch coverage is 97.75281% with 2 lines in your changes missing coverage. Please review.

Project coverage is 65.77%. Comparing base (2df20a8) to head (455ad4b).

Files with missing lines
Patch %
Lines

wirebson/logging.go
97.75%
2 Missing :warning:

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #42      +/-   ##
==========================================
+ Coverage   65.08%   65.77%   +0.68%
==========================================
  Files          41       41
  Lines        1753     1791      +38
==========================================
+ Hits         1141     1178      +37
- Misses        442      443       +1
  Partials      170      170

Files with missing lines
Coverage Δ

wirebson/logging.go
93.29% <97.75%> (+0.99%)
:arrow_up:

Flag
Coverage Δ

test
65.77% <97.75%> (+0.68%)
:arrow_up:

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


Generate mock for pkg/injection/codemodule/installer interface using vectra/mockery

Description
Removed old manual mock impl and added the auto generated one via Mockery.
How can this be tested?
make go/gen_mocks to generate mocks
make go/test to see everything is working fine
Checklist

[x] Unit tests have been updated/added
[x] PR is labeled accordingly with a single label
[x] I have read and understood the contribution guidelines

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (1fb2753) 64.79% compared to head (98e7a97) 64.79%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2385   +/-   ##
=======================================
  Coverage   64.79%   64.79%
=======================================
  Files         245      245
  Lines       15917    15917
=======================================
  Hits        10313    10313
  Misses       4847     4847
  Partials      757      757

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[REEF-567] Remove deprecated WakeConfiguration

This addressed the issue by

Deleting WakeConfiguration.java

JIRA
REEF-567

I'll merge this.


Pagination active page-link lacks noticeable focus styling

Operating system and version : Any
Browser and version: Any
Reduced test case: https://codepen.io/tmorehouse/pen/MOQXyd

The current CSS for pagination .page-item.active .page-link does not include any noticeable focus styling, which makes it difficult for keyboard only users to know if the active link is focused or not.
Tab through the page links in the codepen to see the issue (and one possible solution)
I've used the following workaround CSS to provide focus styling for the active page-link:
.pagination .page-item.active .page-link:focus {
    box-shadow: 0 0 0 3px rgba(0,123,255,.5);
    z-index: 2;
}

Currently the active page link looks like this with focused and not focused (the change in outline is negligible):

With the proposed focus styling it looks as follows (which uses similar styling as the primary button focus styling):

Another quick workaround is to place the class .btn-primary (without the .btn class) on the active page-link (the <a> tag), which produces the same effect when focused.

Hi @tmorehouse, thanks for reporting this.
The focus on anchors is the default focus of the browser, for example: on a mac on chrome it looks like this:

I see what you mean about the focus ring not been consistent with .btn and I am for adding the shadow on pagination. @patrickhlauke What do you think?

I should have mentioned that I was also using Firefox, which has a very subtle focus ring

@tmorehouse I agree, different browsers will show less noticeable focus rings. Let's wait for @patrickhlauke comment. As I mention before I think we should add the same focus ring as in btns for consistency.


Improve error message when clock period/part num cannot be determined.

Currently, the error message just shows tapac: error: either a valid --platform or a valid --clock-period is required. Ideally it should:

show more specific instructions to check /opt/xilinx/platform if --platform is specified, or
show more specific instructions to explain the clock period and part num needs to be either extracted from --platform, or specified directly from --clock-period, when none of the arguments are specified.

Fixed by 36e272dac06bcbe35b97a6e80f20066b3d0f340c. @linghaosong Can you check if this has been fixed in the 0.0.20220227.3 version? You can update by running the install.sh script again. Feel free to reopen if there still are issues.


Small changes for URLs to work on localhost

I have made tiny fixes so that users working on localhost/canvas/public style domains would not have troubles. Laravel has great helpers for that purpose

Thanks for the help! Looking forward to anything more you can contribute to the project!


Add orb pack option

Checklist

[x] All new jobs, commands, executors, parameters have descriptions
[x] Examples have been added for any significant new features
[x] README has been updated, if necessary

Motivation, issues
This will let users use "include" in their syntax
Description
Adds an option to use the be orb pack command instead of the config pack command

@gmemstr i know that v10 addresses this, but would you be willing to publish this PR as a patch, as it seems like the number of updates that one has to make their workflows to use v10 might be high? and just this will sort of unblock a few folks who want to use the pack command from orb alone

@gmemstr updated :)


expect-playwright not working with jest-playwright

While trying out jest-playwright i've experienced issue with expect-playwright. My project GitHub repo. I cannot use any assertions from expect-playwright, when running tests from terminal the following error occurs:
TypeError: expect(...).toHaveTitle is not a function

  1 | export async function testGoogle(){
  2 |     await page.screenshot({path: './target/sampleTest/sampleTest.png'})
> 3 |     await expect(page).toHaveTitle('Google');
    |                        ^
  4 | }

  at Object.testGoogle (src/util/sampleTest.js:3:24)
  at Object.<anonymous> (src/spec/sample.spec.js:8:5)

This also happens for toHaveURL()


fatal error: Index out of range

https://github.com/Zewo/WebSocket/blob/master/Sources/Socket.swift#L395
The index here can be -1
Don't know what this code do, but checking for != -1  solves a runtime error
let ind = frames.endIndex.predecessor()
if ind != -1 {
    frames[ind] = frame
}

Also it was impossible to understand where bug was.
Stack trace from SPM:
fatal error: Index out of range
Current stack trace:
[1]    93505 illegal hardware instruction  .build/debug/Broq

Xcode showed random line of code each time.  Looks like it's swift debugger error.

Can you please send a PR?


Super admin users are affected by location restrictions OR Single location store location select BUG

Expected behavior:
As a super admin user I should NOT being affected by restrictions of any type.
Actual behavior:
Super admin users are affected by restrictions at least on location select process and related actions.
Reproduce steps:
In single store mode I am NOT able as a super admin to select location on Table creation:

Or category creation:

And so on...
If I enable multi-location mode I am able to  select all location from user drop-down menu and this made everything works (BUT is clearly a workaround and NOT a proper solution)

Version:
v3.0.4-beta.25.2
Additional Information:
After git clone of flame nothing changed (Just to underlying is NOT a flame version/corruption problem)

Suggestions:
Also location should be mandatory in table creation process and so on. Which is the point of a table without location?
In single store mode the location should be automatically set and ideally not displayed at all or selected and greyed out.

When you select a location it defaults to that location - thats why the location field does not work

Can you explain what you mean by select a location? In single store mode I shouldn't take care of that, am I right?

In single store mode there is only one location so it defaults to that.
In multi store mode you need to select all locations from the drop down for it to be selectable, otherwise it defaults to the location you are on.

In single store mode there is only one location so it defaults to that.
This do not happen to me. I have none selected and the field blocked

Enabling multi location was a workaround to this behaviour

When you save do you get an error?

On 22 Dec 2020, at 11:49, BreakSecurity<EMAIL_ADDRESS>wrote:
In single store mode there is only one location so it defaults to that.
This do not happen to me. I have none selected and the field blocked
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/tastyigniter/TastyIgniter/issues/615#issuecomment-749502043, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAMVOYKS7LVVU6WYEFQ7CLSWCBURANCNFSM4UWYCQ3A.

Nope it create let's say the table without location assigned to it

I'll try and get some time to investigate single location mode - never really used it

Might be related: https://github.com/tastyigniter/TastyIgniter/commit/bfc652262292751a1a0e487e8b7289e819193e5e

Might be related: bfc6522

Still having this bug in single location mode

Might be related: bfc6522

Unfortunately this did NOT fix the problem :(

In single store mode there is only one location so it defaults to that.
This do not happen to me. I have none selected and the field blocked

That's the expected behaviour for single location mode. There's no point enabling the dropdown for super admin in a single location mode since only one location is expected in the system.
The dropdown remains blank until the locationable entity is created then the default location id is assigned to the entity.

In single store mode there is only one location so it defaults to that.
This do not happen to me. I have none selected and the field blocked

That's the expected behaviour for single location mode. There's no point enabling the dropdown for super admin in a single location mode since only one location is expected in the system.
The dropdown remains blank until the locationable entity is created then the default location id is assigned to the entity.

Related to #649

Related to #649

@sampoyigi now the location field is always displayed and it's not grayed out, right?

@sampoyigi now the location field is always displayed and it's not grayed out, right?

Right

Right

Closing since will be "fixed" with #649

Closing since will be "fixed" with #649


fix(v2): allow swizzling of component even if case doesn't match

Motivation
Fixes  #3148.
Have you read the Contributing Guidelines on pull requests?
yes
Test Plan

Thanks
@anshulrgoyal the swizzle.ts file starts to become bigger and bigger, and not so easy to understand. What about refactoring it a bit in smalller pure functions and adding some tests? As there's currently no test, it does not incite to add some when we touch this code, but all this should rather be tested ;p

Thanks
@anshulrgoyal the swizzle.ts file starts to become bigger and bigger, and not so easy to understand. What about refactoring it a bit in smalller pure functions and adding some tests? As there's currently no test, it does not incite to add some when we touch this code, but all this should rather be tested ;p

Yeah, I think same.

thanks :)


validate build-arg

fixes #24101
- What I did
fixes #24101
Build-arg is in the form of =

<varname> is required, but <value> not
Shouldn't use env arg as a build-arg.
- How I did it
Add validateArg
- How to verify it
Some tests in runconfig/opts/opts_test.go
- Description for the changelog

Signed-off-by: sakeven<EMAIL_ADDRESS>

@sakeven I found that in opts.go, ValidateEnv never returns a non-nil error. Is it reasonable? Just one thought on this.

@allencloud it's used as a ValidatorFctType for NewListOpts so the signature of the method is a little imposed :angel:

@allencloud @vdemeester  ValidatorFctType is magic that it not only validate passed values but also modify values. According to TestValidateEnv, almost any value are legal.

@sakeven true :sweat_smile:

@vdemeester My test failed, but it seems not my fault?

The win2lin is sometimes.. flakey.. I'll restart the build ;-)

@vdemeester It failed again...

Design OK.

Actually, what we're really saying here is that a null value is not acceptable. Does this make sense?

Sorry, what means a null value? The value passes to ValidateArg, or value of build-arg?
At least, I think build-arg should be in the form of =, and the  is legal to be empty.

@sakeven The build-arg <value> being empty.

@cpuguy83 I think it's make sense that the build-arg <value> being empty.
Someone maybe define a non-empty ARG in dockerfile, so we should support pass an empty ARG value to cover the default one.

I am afraid you had better squash these three commits into a single one, my friend. @sakeven

@allencloud Thanks. 😀

@allencloud @vdemeester @cpuguy83 Could you please review my code, or give me any suggestion? 👻👻👻

Small comment, but LGTM 🐸

@vdemeester Thanks.👻

LGTM


More precise PA and F1?

Here is the current definition (quote from our AGI-2019 paper):
"The first perspective is extent to which reference corpus is parsed at all – it is called “parse-ability” (PA) and it computes the average percentage of words in a sen - tence recognized by grammar tester: PA = (Σ(ki/ni))/N, where: PA – parse ability, N – total number of sentences, ki – number of words in i-th sentence recognized by the grammar tester, ni – total number of words in i-th sentence.
For the second metric we use conventionally defined F-measure or F-score (F1) metric computed on basis of recall and precision, averaged across all sentences in the corpus as Recall = (Σ(ci/ei))/N and Precision = (Σ(ci/li))/N, where ci – number of cor- rectly identified links in i-th sentence, ei – number of expected links and li – number of identified links, including false positives. That is, for recall we take average per- sentence number of overlapping links in test and reference parses divided by the total number of links in reference parses. Respectively, for precision we take the overlap - ping number divided by the total number of links in test parses."
The problem is that if we have two sentences of 100 and 10 words/links with matches 90 and 1, the assessment will be average = (90/100 + 1/10)/2 = 0.5 - without of account to sentence length. However if we consider individual word/links or average with account to umber of them in the sentence the assessment would be 91/110 = 0.83 which is more "fair".
Here alternative is discussed:
https://docs.google.com/document/d/1YtN0-hvGWHJy1_KzXSfGE8w_m3kU5m0LcMmOw4KHT3Q/edit#heading=h.twoiv52o0tou
see appendices H and J in the bottom.
We should decide if we want to move to this metric for parses evaluation and when to do that if we decide so.
This issue extends #198

"Alternative" F1 estimation -- Alternative_F1_for_ALE_ILE%20clustering_2019-04-12.html

Writing down what I said in previous video calls:
I don't think this is a good idea for two reasons:

The example explained above with a 100-word sentence having 90 correct links and a 10-word sentence having only 1 correct link, although possible, is not realistic. Normally, longer sentences will be more difficult to parse correctly. The alternative method proposed gives a lot more weight to long sentences, which is detrimental to quality scores, and also unfair because longer sentences are harder to parse.
The community calculates F1 for each sentence and then averages among all sentences, like our current code does. E.g. see the compute_metrics() function of the AdaGram evaluation code: https://github.com/sbos/AdaGram.jl/blob/332d80e448c83734d05715040d632e51fdfc3f58/test-all.py#L61

Because everybody seemed to agree to this arguments during the call, I am closing this issue


Show on content page if a file is part of a playlist already

PR Checklist

Please check all that apply to this PR using "x":

[X] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[ ] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[X] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

Fixes
Issue Number: Closes #6206

Just indicates that the file is currently already saved in a List, the button is still the same and clicking allows to remove/edit
Other information
[Requires lbryio/lbry-redux/pull/419]

Thanks for the PR, this is a cool feature not even YT has! We'll get a review on it soon.

Ok, testing this out.
I merged the redux pr, so this one will need a redux bump / yarn.

Something going on here with icon color.

60bd918d5ee95ea4e597d1e5c6bccf992ef83d28 is the new redux hash. rebump?


facet_grid failing with just the y parameter

I see a similar issue to #183. However, with only the x parameter the facet_grid command works, although with a vertical instead of horizontal layout, as stated in #182. Oh the other hand, with just the y parameter, the command fails with an error:
AttributeError: 'NoneType' object has no attribute 'unique'

ggplot version is 0.6.5. Tested on both Windows and OS X, with both Python 2.7 and 3.4, and both console and IPython Notebook.
A reproducible example is:
import pandas as pd
import numpy as np
import matplotlib.pyplot as plt
from ggplot import *

column1 = np.random.randint(1, 21, 100)
column2 = np.random.randint(1, 21, 100)
column3 = np.random.randint(1, 6, 100)
column4 = np.random.randint(1, 6, 100)
data = list(zip(column1, column2, column3, column4))
data_raw = pd.DataFrame(data = data, columns = ['a', 'b', 'c', 'd'])

p1 = ggplot(data_raw, aes(x = 'a', y = 'b')) + geom_line()
p1 = p1 + facet_grid(x = 'c', y = 'd')
print(p1)

p2 = ggplot(data_raw, aes(x = 'a', y = 'b', color = 'c')) + geom_line()
p2 = p2 + facet_grid(x = 'd')
print(p2)

p3 = ggplot(data_raw, aes(x = 'a', y = 'b', color = 'c')) + geom_line()
p3 = p3 + facet_grid(y = 'd')
print(p3)

fixed in 0.9.3


About running the x-decoder demo

Thank you for your great work.
I am trying to run the X-Decoder demo in a local environment.
I want to run run_demo.sh but it doesn't work because torch and triton versions conflict.
Also, there is no demo.py in python demo/seem/demo.py.
I also get the error No module named 'modeling' when I try to run app.py. Is there a solution to this?
I look forward to hearing from you.

I tested my local environment, triton is not needed for X-Decoder/SEEM, as I didn't install it.

Which demo do you want to run? The current run_demo.sh are referring to SEEM (I highly suggest you run from SEEM repo that is better supported for now). If you want to run X-Decoder, please temporarily refer to https://huggingface.co/spaces/xdecoder/Demo/tree/main

Thanks for your references, I just fixed demo.py to app.py, that is a typo.

This is very strange, are you running python3 demo/seem/app.py? You could try to add at the very top of app.py
pth = '/'.join(sys.path[0].split('/')[:-2]) sys.path.insert(0, pth)


add 'eigen3-nnls' to hunter-packages

https://github.com/hmatuschek/eigen3-nnls
This is required for an eos update.
I have a hunter patch.
Thanks.
UPDATE: https://github.com/headupinclouds/eigen3-nnls/tree/hunter

https://github.com/hunter-packages/eigen3-nnls

#823


Bump inflections to get extra argument to define-obsolete-function-alias

Similar to https://github.com/hlissner/doom-emacs/issues/4534. I could build doom with the current Emacs 28.0.50 native-comp branch after this without problems.

Sorry for the delay. A much larger effort to bump :lang ruby (and the rest of our :lang modules) was already underway. What's more, bump commits must follow strict formatting so they remain machine readable for future CI. In any case, inflections was bumped in a79275a, which should resolve your issue. Thanks for bringing it to my attention!


Update Page “workPortfolios”

Automatically generated by Netlify CMS

Deploy preview for dds-mil ready!
Built with commit b6954185328eaebe7317a9f93b9afd8a6884e07f
https://deploy-preview-288--dds-mil.netlify.app

Deploy preview for dds-mil ready!
Built with commit eccece7ad3f9f7b7b311194a95b2542053f41711
https://deploy-preview-288--dds-mil.netlify.app


Auto Update Enums

Is there a way to make it so that the enums or at least the way i have my enums setup, to auto update every time i update a default value for a variable in a tuple?
Right now if i make a change to a value i have to also make the change in the doc string ... but what i want it to do is grab the default value like it does if it is a function.
Here is example of my code
class OrderTypeT_QF(tp_qf.NamedTuple):
    """
    Define if you want to do an entry or exit trade.

    !!! tip
        A quick exmaple would be something like this `order_type=OrderType.LongTP`.

        This is the same as saying `order_type=3.` and would set your order type to a long take profit.

    LongSL: float = 1.
    LongTSL: float = 2.
    LongTP: float = 3.
    LongLiq: float = 4.
    ShortEntry: float = 5.
    ShortSL: float = 6.
    ShortTSL: float = 7.
    ShortTP: float = 8.
    ShortLiq: float = 9.

but this guy that i am forking my project from does it in a way that grabs the values automatically and he doesn't have to type any of the changes out
class CallSeqTypeT(tp.NamedTuple):
    Default: int = 0
    Reversed: int = 1
    Random: int = 2
    Auto: int = 3

CallSeqType = CallSeqTypeT()
"""_"""

__pdoc__['CallSeqType'] = f"""Call sequence type.
```json
{to_doc(CallSeqType)}
```json
Attributes:
    Default: Place calls from left to right.
    Reversed: Place calls from right to left.
    Random: Place calls randomly.
    Auto: Place calls dynamically based on order value.
"""

https://vectorbt.dev/api/portfolio/enums/#vectorbt.portfolio.enums.CallSeqType
Is there some type of way of doing this is mkdocstrings and if not is there some type of function i could create that would do something like vectorbt ... because it is going to be a nightmare every time i add and remove things from these tuples

same thing with this numpy data type
https://vectorbt.dev/api/portfolio/enums/#vectorbt.portfolio.enums.log_dt
https://github.com/polakowo/vectorbt/blob/9155bddebc634846b419e261f9058cbda708bbdc/vectorbt/portfolio/enums.py#L1645-L1694
i am doing the same thing and there is no way i want to have to keep typing all of this stuff out lol

Also i am noticing that the mkdocstrings doesn't seem to allow you to do something like
f"""
Hey
"""

If i do that then it doesn't read anything ... so is it even possible to do a f""" ?

@pawamoy  .... wanted to check back in on this and see if you had any ideas?

Hey @QuantFreedom1022, that's a good suggestion yes. I think other users asked for the "default" column for attributes tables, and I believe it's possible to have it (though I'd probably call it "value" more than "default").
Now, the vectorbt library you mentioned seems to use pdoc, and pdoc runs the code (inspects/introspects it) to get the data. This is why they can use some magic __pdoc__ variable, and modify docstrings dynamically, and this is why they support f-strings for docstrings and we don't: f-strings are dynamic, they need to be executed to get their final value, which we do not want to do within our static parser/visitor.
But there's a workaround: Griffe (the tool that collects info about your Python code), is also able to inspect (not visit) your code, and it provides a builtin extension that lets you choose which object to inspect, additionally to visiting it. It's the "hybrid" extension (and it's experimental). I'll try to come up with an example on how to use it a bit later :slightly_smiling_face:

@pawamoy hey man thanks for getting back to me on this ... and yeah I agree with u about the default column ... I am sure tons and tons of users would love this and I also agree that it could be called value instead of default
And I kind of feel bad having you have to look up all this information yourself and figure it out and then get back to me. I would be open to making this a collaborative effort where maybe we could jump on a screen sharing call and talk about possible solutions and then work on the solution together so that way you don't have to do this by yourself

That's very kind @QuantFreedom1022, I appreciate your willingness to help, but don't worry, this is my project, I enjoy helping people using it :smile: I have started playing with Griffe extensions more, and I want them to work well before I advertise them, so it's good that I try them myself!
I also much prefer asynchronous collaboration rather than getting on a screen sharing call. I believe everything can be accomplished with async messages, even if it can sometimes takes more time.
Thank you again for such kindness, this is a great thing to have in open source communities :heart:

@pawamoy yeah no problem ... and when you think you have it done i would be more than happy to be the test dummy

@pawamoy is there any update on this? do you need me to do anything? or how do i get setup with some type of notification that you completed this or are looking for it to be tested?

You'll be notified of any progress on this ticket if you've subscribed to it (which is the case by default when you create a ticket) :wink:

@pawamoy ... hey i have a question ... was wondering if you were able to implement this yet ... and also was wondering if it would be possible to get some one on one lessons for like an hour or two on how to use mkdocstrings ... i have a strong enough understanding to use it but i just want to skip all the learning curve and be able to ask you questions in real time ... willing to pay you for your time

@QuantFreedom1022 I did not have a lot of time to work a lot on mkdocstrings these last months. Development was very slow. But my professional situation changed very recently and I'll be able to dedicate a lot more time to mkdocstrings. Keep watching the organization and its repositories and you should see much more activity :wink:
I'm not ready yet to offer one-on-one support calls, but since you mention financial compensation, please know that the project now relies more than ever onto sponsors. We can also chat in real time on the Gitter channel.


Burn can be called by anyone

burn function is vulnerable. Anyone can call this function and burn tokens from any other account.
Remediation- Use approapriate access control modifiers such as Openzeppelin onlyOwner or Roles

Can we check @vishaldantuluri this part with rari and OpenSea

@techpandora i have changed it to the way rarible is doing it

#24 follow up on this issue after observing another vulnerability.
This issue can be closed @techpandora


Add many more error locations

This adds real error locations anywhere you invoke a free variable.
Closes #239

This probably fixes 99% of the type error location complaints.
Error clarity is a separate problem

💯

before:

after:


Remove bold from speaker tags in semi-diplomatic texts

We don't want speaker tags to be bolded in semi-diplomatic transcriptions. Please change the font-weight to be the same as the main text. Style is currently being inherited from span[data-el="speaker"], body[data-catrefs*="Primary"] article div[data-el="label"][data-type="heading"] where font weight is 500.

Also check that the font colour matches that of the main text.


Use new Maven plugins, add Gradle support

Okay, rad. What do we need to get out of the way to merge this?


Unable to open a new tab in chrome(v54) instance using CTRL+T.

Hi there!
I recently updated my chrome version to the latest (v54) and then I am unable to open a new tab using CTRL+T through automation. It is working fine with the lower versions of Chrome.
Here is the snippet:
browser.actions().keyDown( protractor.Key.CONTROL ).perform().then( function () {    browser.actions().sendKeys( 't' ).perform();
} );
Can someone please help me out.
Regards,
Charvaka.

Hi there!
Your question is better suited for StackOverflow or Gitter. Please ask a question there with the 'protractor' tag or post in the Gitter Channel to get help.
From the the getting help section of the README:

Please ask usage and debugging questions on StackOverflow (use the "protractor" tag) or in the Angular discussion group. (Please do not ask support questions here on Github.)

Thanks!


[Hub Generated] Review request for Microsoft.StorageSync to add version 2019-02-01

If you are a MSFT employee you can view your work branch via this link.
Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-js

Automation for azure-sdk-for-python

Automation for azure-sdk-for-java

Automation for azure-sdk-for-node

Automation for azure-sdk-for-go

Can one of the admins verify this patch?

@anpint thanks for using OpenAPI Hub to create this PR, are you expecting to make more changes on the specs? or is it an api-version update with the same content?

@anpint thanks for using OpenAPI Hub to create this PR, are you expecting to make more changes on the specs? or is it an api-version update with the same content?

We will be making additional changes to the specs. I was advised by my teammate to first create an API version update with the same content, and follow-up with another PR containing changes to the protocol.

@anpint in this case, would you want to work against a branch for your team in the repo? We can create one for you and we could review PRs against it, if you'd like. You can put all your work there, until you're ready to send it to master.
This PR is currently against master branch and we should not merge partial work to master.
If you're going to be the only person working on the changes, you can also keep adding commits to this PR and add [Do not merge] or [Work in Progress] to the title of the PR.
If you have more questions, feel free to let me know.
Thanks!

@anpint in this case, would you want to work against a branch for your team in the repo? We can create one for you and we could review PRs against it, if you'd like. You can put all your work there, until you're ready to send it to master.

OpenApi Hub created this branch for us: dev-storagesync-Microsoft.StorageSync-2019-02-01. Are you suggesting that we could create PRs against that branch instead of master for incremental work?

This PR is currently against master branch and we should not merge partial work to master.
If you're going to be the only person working on the changes, you can also keep adding commits to this PR and add [Do not merge] or [Work in Progress] to the title of the PR.

This is my first time going through this process, and my team member who did this before (ankushbindlish2) suggested that we follow this procedure:

Create a PR with the new API version as a copy of the previous API version. (This is the current PR)
Create a second PR with the actual protocol changes.

The intention of this approach is to make it easier to review the actual protocol changes, because in this current PR, all files for the 2019-02-01 API version are being treated as new files, even though it's just a copy of the previous API version, so it will be harder to see the incremental change between the new API version, and the previous API version.

@anpint I believe we clarified this offline. If you create a branch with the updates to the api-version, that's great, then send PRs to that branch for review, so once your work is completed you can have 1 PR against master approved and merged. If you have more questions, feel free to let me know.

@anpint When you're done with changes in this PR, just let me know, thanks!

@veronicagg The changes for this protocol have been finalized.

@anpint thanks, could you please review the CI failures for linter? Thanks!

@anpint thanks, could you please review the CI failures for linter? Thanks!

Sorry for not noticing the failures before. I wanted to define some properties in the protocol as both readonly and required, and it looks like that's not supported. I kept them as readonly.

Change looks good, this PR is composed of the new api-version with an already reviewed commit plus a few changes to the properties.
Do you still need to wait on afedyashov for review?
Are the APIs deployed? If so, I can approve and merge this PR.
Thanks!

Change looks good, this PR is composed of the new api-version with an already reviewed commit plus a few changes to the properties.
Yes, that's correct.

Do you still need to wait on afedyashov for review?
No, we don't need to wait. Initially he was going to add some changes to the protocol, but we dicided not to make further changes.

Are the APIs deployed? If so, I can approve and merge this PR.
Can you clarify this? Do you require having the 2019-02-01 API version already supported by our RP in production? This API version is currently making its way through our release pipeline, so it will be a few weeks before it hits production. As a general process, I would expect that swagger should be the first step in creating a new API version (so that the changes go through proper approvals before we implement them in our service). Is that not the case?

@anpint It's correct that the swagger is the first step, but we should have the APIs deployed in production before merging the PR. The reason is that once a swagger is merged in master, an SDK can get generated right away, and if the APIs are not available the SDKs won't work for customers. Hope that makes sense.
Let me know when the APIs are deployed and I can merge this PR. Thanks!

@anpint It's correct that the swagger is the first step, but we should have the APIs deployed in production before merging the PR. The reason is that once a swagger is merged in master, an SDK can get generated right away, and if the APIs are not available the SDKs won't work for customers. Hope that makes sense.
Let me know when the APIs are deployed and I can merge this PR. Thanks!

Ok, I understand now. I'll tag this PR as "Awaiting Deployment" and update it again once we have the new API version in production.

Automation for azure-sdk-for-net
A PR has been created for you:
https://github.com/azure/azure-sdk-for-net/pull/5596
.NET SDK Commits:
https://github.com/adxsdknet/azure-sdk-for-net/pull/5596/commits/007d15146f362eee3b38029e6723bf2a8c808693

The service code that supports this new API version is now in our Canary regions, so we would like to go ahead and complete the PR. We will be rolling out the service to all production regions in the next few weeks.

@anpint ok, great. Due to my explanation above, we should not merge and release SDKs until the APIs are in production, please let me know when that's the case, for PR merge.

@veronicagg  We are already LIVE in FLIGHT regions. Please merge this change.  All SDKS would be publish only after ALL regions are LIVE.


Fix total values when ignore regex used

Expected Behavior:
The report should display the total values (stmts, miss, and cover), calculated for all files excluding those specified by the ignore_regex.
Actual Behavior:
Previously, when using ignore_regex to exclude files, the report's total values were incorrect. The values remained the same whether the ignore_regex argument was passed or not.
Steps to Reproduce:
~/pycobertura >  pycobertura show --ignore-regex ".*Main|.*ISorted" --format markdown tests/cobertura.xml
Report:
| Filename                 |   Stmts |   Miss | Cover   | Missing   |
|--------------------------|---------|--------|---------|-----------|
| search/BinarySearch.java |      12 |      1 | 91.67%  | 24        |
| search/LinearSearch.java |       7 |      2 | 71.43%  | 19-24     |
| TOTAL                    |      34 |      3 | 90.00%  |           |

Changes:

Added ignore_regex as an input to the functions responsible for calculating total values.
Introduced a test to ensure correct total value calculation when using ignore_regex.

Result:
After the fix, the total values are now calculated correctly, reflected by the following report, generated with the same command.
Report:
| Filename                 |   Stmts |   Miss | Cover   | Missing   |
|--------------------------|---------|--------|---------|-----------|
| search/BinarySearch.java |      12 |      1 | 91.67%  | 24        |
| search/LinearSearch.java |       7 |      2 | 71.43%  | 19-24     |
| TOTAL                    |      19 |      3 | 84.21%  |           |

Thanks @danctorres, much appreciated! I'll take a look at it soon.
Can you update the PR description to show the before/after to better understand what you observed that looked wrong?

Thanks @danctorres, much appreciated! I'll take a look at it soon.
Can you update the PR description to show the before/after to better understand what you observed that looked wrong?

Updated the description, hope it helps

v3.3.1


Getting TypeError: Converting circular structure to JSON,  when doing JSON.stringify(currentState)

Bug or feature request?
Bug
Description:
Getting TypeError: Converting circular structure to JSON,
when doing JSON.stringify(currentState)
(Bug) Expected result:
it should stringify without error
(Bug) Actual result:
serializedState = JSON.stringify(currentState);  // Machine had nested  machines, used invoke
^
TypeError: Converting circular structure to JSON
at JSON.stringify ()
at Object. (\xs_test\index.js:55:25)
at Module._compile (module.js:652:30)
at Object.Module._extensions..js (module.js:663:10)
at Module.load (module.js:565:32)
at tryModuleLoad (module.js:505:12)
at Function.Module._load (module.js:497:3)
at Function.Module.runMain (module.js:693:10)
at startup (bootstrap_node.js:188:16)
at bootstrap_node.js:609:3
(Bug) Potential fix:
Required! If you found a bug and can consistently reproduce it, please take a minute to think about what the potential fix might be.
Even if the fix is incorrect, this can help diagnose and find the solution to the issue as quickly as possible.
I also strongly encourage you to create a pull request if you are confident in the potential fix.
(Feature) Potential implementation:
Not sure
Required! If you would like to see a feature or enhancement make its way to xstate, please describe:

What the API would look like
If this is a breaking change
If this is part of the existing SCXML specification

Link to reproduction or proof-of-concept:
You can use CodePen or Code Sandbox to create a reproduction/POC.

hey @davidkpiano, thanks for doing all the work on this, working with XState is a charm ♥
Just as a note: unfortunately the release of this fix was a breaking change, in a minor release and broke our service invocation since we referenced service functions directly. Also, the documentation hasn't been updated yet, and still shows that functions can be used as a src. I think while this fix is important, the release might've been a bit too quick. If I find time, I'll look into creating a MR for an documentation update 😄

Hey @deniaz, can you please share the code that is breaking? I'll try to mitigate the issue in a patch.

Hey @davidkpiano, first thanks a lot for your great work - like @deniaz already stated working with XState is really a charm. I've looked into our problem and I suppose the problem occurred because of this change
https://github.com/davidkpiano/xstate/commit/0041c94f36d4ebff2e8f715eff8feb756d0ecb67#diff-a22679c0e1167fd703b79102f0b220adL531 since we initially used the possibility to directly call a function which returns a promise and this is not possible anymore.

Thanks @dwirz! If you can post a reproduction (via CodeSandbox or otherwise) that'd be helpful - hopefully it's not a breaking change (all the previous tests still passed).

Hey @davidkpiano i just played around a little in CodeSandbox and it seems in the newest version (4.3.3) the problem is fixed. 🎉  Thanks and sorry for the inconvenience. Keep up the great work! 🙂

Hi @davidkpiano , thanks for all the hardwork on this! However I'm still getting this error on 4.10.0, and I'm not using nested machines.
Here is the xstate definition that I used -> https://codesandbox.io/s/romantic-herschel-0mmkd. The error only occurs when the serialisation happens after state transition in line 139 and 147.
Please let me know

@myarjunar Have you tried 4.11?

@davidkpiano is there a bugfix or changes regarding this issue on 4.11? I will try that anyway, thanks!

Also, the CodeSandbox you linked is currently invalid @myarjunar. If you're still experiencing the same issue in 4.11, please open a new issue Thanks!


empty dataset throws exception on fold-by

Tablecloth 4.04.1
Exception is thrown by fold-by if dataset has no rows.
It should just return the original empty dataset.
tablecloth> DS
_unnamed [3 2]:

| :a | :b |
|----|----|
|  1 | :a |
|  2 | :b |
|  9 | :c |
tablecloth> (api/select-rows DS (comp #(< % 1) :a))
_unnamed [0 2]:

| :a | :b |
|----|----|
tablecloth> (api/fold-by (api/select-rows DS (comp #(< % 1) :a)) :a)
Execution error (ArityException) at tablecloth.api.group-by/ungroup (group_by.clj:203).
Wrong number of args (0) passed to: tech.ml.dataset.base/concat

confirmed, fixing

fixed in 4.04.2 and 5.0-SNAPSHOT


V1.4 - Backports 12-02-2019

Backports
#7030
#7029

This change is

test-me-please

Do not merge this. This is not critical for 1.4.0

test-docs-please

test-missed-k8s


Fixes Pass-through captures in Directed Moves Generator

Fixes #33

Current coverage is 82.00%

Merging #34 into master will increase coverage by +0.06% as of aa138cb

@@            master    #34   diff @@
=====================================
  Files           24     24
  Stmts          349    350     +1
  Branches        55     55
  Methods          0      0
=====================================
+ Hit            286    287     +1
  Partial         13     13
  Missed          50     50

Review entire Coverage Diff as of aa138cb

Powered by Codecov. Updated on successful CI builds.


Add closing codes and reasons

The webocket protocol states that a reason can be given by the endpoint, when sending a closing frame. It should be a two byte code (1000 for "everything is fine") followed by an optional utf-8 encoded message. Python's websockets server generates an error, when no code is sent, assuming something went wrong.
It would be good to send a confirmation that closing was expected.
I could fix my issue by changing the "close frame.vi" as follows:

Download (valid until 2020-01-31)
I would recommend to make the Ring Menu into a Control and use it as an optional input. Then one can program a client that automatically closes the connection with the correct error code, when e.g. a binary payload instead of an expected text payload is received.

Hi Lucas,
The WebSockets specification states:

The Close frame MAY contain a body (the "Application data" portion of
the frame) that indicates a reason for closing

The closing body and status code is not required by the WebSockets specification so if the Python library is returning an error then that is an issue with the Python library.

True. I also wrote that a reason CAN be given.
Not sure how well it is handled on the python side. The error is printed but the server keeps running happily. Maybe it is intended as a strong suggestion...
Anyways, I thought it is an easy thing to add and it could have sensibel default values (maybe empty by default, but "OK/end of use" when used in the example client at program stop). And it allows the server to know what went wrong in case something is not OK.

Yes, that's OK - I will leave the issue open and have no issue with implementing it at some stage but if the Python side is generating an exception/error then that's not an issue with the LabVIEW library.


PR for base-repo v1.2.1

Codecov Report

Merging #57 (f4ba662) into main (c73480d) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head f4ba662 differs from pull request most recent head 87c5e6b. Consider uploading reports for the commit 87c5e6b to get more accurate results

@@            Coverage Diff            @@
##               main      #57   +/-   ##
=========================================
  Coverage     81.89%   81.89%
  Complexity       81       81
=========================================
  Files            12       12
  Lines           359      359
  Branches         25       25
=========================================
  Hits            294      294
  Misses           49       49
  Partials         16       16

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@sonatype-lift ignoreall


Emoji Switcher Game Added to JS_Games

Title and Issue number

Want to add Emoji Switching Game to JS_Games | #880
Close #880

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md

Screenshots

@pranjay-poddar Kindly review my PR. Thank you.😊

Closing as part of Hacktoberfest 2022 and now outdated.


[ENH] Update and implement new unit test for limit/offset

Description of changes
Summarize the changes made by this PR.

Improvements & Bug fixes

Updates test_filtering.py. The new implementation sorts the result by internal offset_id instead of user provided id, thus the unit tests need to be updated accordingly.

In addition, a simple unit test is added to test get() with only offset and limit

Test plan
Run the existing and new tests locally

[ ] Tests pass locally with pytest for python, yarn test for js, cargo test for rust

Documentation Changes
Add documentation to explain the order of results return by get()

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#2796  👈
#2790
main

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

Join @Sicheng-Pan and the rest of your teammates on  Graphite

Merge activity

Sep 18, 5:29 PM EDT: @Sicheng-Pan started a stack merge that includes this pull request via Graphite.


Invalid keyboard layout passed

That is the error I started getting with my first boot of my day today. Everything has been working for months up until now. I haven't played around or even touched my settings as well.

Steps to reproduce

input {
    kb_layout = dvorak
    kb_options = caps:backspace
}

localectl set-keymap dvorak and loadkeys dvorak do not do anything either.
It is also worth noting that I tried a few other generic layouts. The only two that worked were 'us' and 'de'??? 'Azerty' and 'uk' do not work for example...

Noted outcome

Hyprland defaults back to (us) qwerty. Consequently, this issue took me 15 minutes to create because I forgot most of my qwerty muscle memory. Free me from this pain :(

dvorak is a variant no?

dvorak is a variant no?

Would you look at that... problem fixed.
But why has it consistently worked for all these months, until it decided enough is enough today? :D Oh well, thanks!

dvorak is a variant no?

Would you look at that... problem fixed.
But why has it consistently worked for all these months, until it decided enough is enough today? :D Oh well, thanks!


add space-vim-dark colors for easymotion hints

First off, thank you for the creating these beautiful color schemes!
I've added colors for vim-easymotion movement hints to space-vim-dark.vim; feel free to merge or not depending on whether you think this would be helpful for others.
Cheers.

Merged, thanks!


I would advocate adding ruff for our linting.

Based on running ruff over the current codebase, we're touching almost all the files and so we should postpone doing this until we have a lull in PRs

To clarify, black and ruff as precommits wouldn't just be catching errors, they would be fixing the errors for you

Doing some experiments with this in pyproject.toml
line-length = 100

[tool.ruff]
line-length = 100```

Obnoxiously, `black` seems to consider this line to be 92 characters long and thus safe.

                [len(v.vehicle_state.trip_plan) for v in veh_dispatch_pooling_in_fleet]  # type: ignore

Ruff considers the line to include its inline comment `# type: ignore` to be part of it, and yells about it. Black ignores comments

nrel/hive/reporting/handler/time_step_stats_handler.py:321:101: E501 Line too long (107 > 100 characters)


Added Primalbase Token (PBT) contract ABI

@shoreward For verification purposes, can you show the contract source code?

Also it's moved #519

@gamalielhere please specify where to send the contract code to you

@shoreward you can email it at<EMAIL_ADDRESS>

@shoreward Closing this PR now. It's moved here #519

Hey. We sent the contract code for a verification from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>

@shoreward yes. Its been merged. The token will be there on the next release.

Oh, thanks! We really looking forward to this :)


UNet Segmentation Network + Example (road segmentation)

Everything should work out of the box. If not please let me know.
Downloading the dataset take quite a while. Is there any way to store the images as a .zip archive somewhere (~8GB)? (tiffs are uncompressed and download is slow because they are individual files)
If something needs changing just let me know!
Cheers,
Fabian

I just checked my layer activations and noticed that the Deconv Layers look weird. Could you check if that is the case for you as well?
For my own experiments i switched to Upscale2DLayer

Wow thanks for your suggestions! This is really helpful (and I am still learning myself)!
I will implement your changes tomorrow =)

Hey Fabien, looks quite nice! I had a couple of suggestions for possible improvements, but it looks good in general.

Is there any way to store the images as a .zip archive somewhere (~8GB)?

Is this the size of the compressed .npy arrays? We have some S3 storage, but I'm not sure if we want to host downloads that large -- so far we only use it for hosting pre-trained model weights. I can see that the download process is quite cumbersome, though.

Thanks for this, it looks like a great example.
As @f0k said, we can store the data in S3 if necessary, but I'd like to bring the size down if possible. Compressed .npy may work, or maybe converting to .jpg. Would you get a decent result on a subset of the data? I guess probably not if you're doing data augmentation already...
Also, it seems like this would work well as an IPython notebook, many people would probably be satisfied viewing the results online and not need to download and train it themselves.

Hi,
thank you for your feedback! So far the .npy files are not compressed, let me check tomorrow by how much I can reduce the file size.
Using less data would probably not work very well. Already now the results are worse than I would like them to be (96% accuracy sounds great but most of the image is background so that does not mean very much). As we speak my PC at work is running some more experiments in order to check if the results can get a bit better when using different hyperparameters and by tweaking some other stuff. It would be great if we could provide a model that works as well as possible so that people are motivated to dive into it a bit more!
Using an IPython notebook is a great idea (btw one of the reasons I started out in lasagne is because of your Neural networks with Theano and Lasagne talk on youtube ;-) ). However my IPython notebook does not find my Python Kernel. Since I myself mostly use plain IPython I did not put too much effort into fixing this. Let me have a look at it tomorrow. If I get it to run I will include a notebook as well.
So far my pretrained weights are hosted on my personal dropbox. I don't mind the space it takes (its just some 25MB) but I am not sure how much traffic dropbox allows. Once we have a final version it would be great if you could host the pretrained weights.
Cheers,
Fabian

Uncompressed numpy arrays have a total size of 10.5GB, compressed npz file has 6.2GB. I guess that ist still too large for your S3 storage. I think the best solution is to let people download the image files themselves.
How many workers can I use do download the image files without having the server refuse the connection? I just downloaded the dataset with 4 workers and that took quite a long time, especially considering that we have >100Mbyte/s downstream speed...

Most suggestions should be included by now. I have to leave for a family week end. Will continue working on everything on Monday.
Have a nice week-end!

How many workers can I use do download the image files without having the server refuse the connection?

I guess I'd just use 4 to be polite.

I just downloaded the dataset with 4 workers and that took quite a long time

This shouldn't be a problem. If people know, they can let it download in the background and do something else. They'll also have to find something to do while training the network. But, out of curiosity: How long did it take?
Enjoy the weekend!

This looks great, thanks for contributing! I have one comment regarding evaluation: as you already pointed out measuring the accuracy can be a bit misleading in this setting. Due to the imbalance of the problem it might make more sense to report AUC scores or some form of precision / recall scores instead. When I used this dataset in a paper I reported the AUC, but the original work using this dataset (Vlad Mnih's PhD work) uses a more elaborate evaluation protocol.

I think AUC will suffice for this educational example. It has been added with the last commit

Alright I think all of your suggestions are now incorporated into the pull request. If anything else needs changing please just let me know!
I added a new set of pretrained weights for the net:
val accuracy:  0.963513  val loss:  0.114994  val AUC score:  0.978996643458
https://www.dropbox.com/s/0vasqq491skf9iz/UNet_mass_road_segm_params.zip?dl=0
The .zip archive is only 9 MB large. I will leave it up to you whether you want to host it yourself or just leave it on my Dropbox. The dataset is simply too large to host it. Users will have to download it themselves.
Thank you very much for your constructive criticism so far! It really helped not just to make this pull request more mature but also in teaching me a lot of new things!

Hi,
what is the status of the pull request? What do you think about providing both implementations?
I can understand if this fits more in the papers section. If you want I can rearrange my code accordingly.
Cheers,
Fabian

Hey, sorry for the late response. I'm happy to merge as is or if you want to reformat for /papers/ that's fine too.
I uploaded the weights to https://s3.amazonaws.com/lasagne/recipes/pretrained/UNet_mass_road_segm_params.zip, if you want to change it to point to there. I think you're right that the dataset it too large to rehost, we can revisit that if the original goes down for some reason.
Thanks again for your contribution.

Hi @FabianIsensee - I'd like to get this merged, if you have a chance could you update the url for the weights and squash your history to a single commit? If you're too busy I'll do it, just need to figure out how to retain your authorship.

Hi,
sorry for keeping you waiting this long. I will do it within the next days and let you know once everything is completed.
Cheers,
Fabian

Hi,
I am almost finished with the PR.
My original implementation of UNet was not very close to the paper. I adapted it to be almost identical (except for pad 'same' instead of valid and my use of batch_norm). This adaptation requires new pretrained weights. It would be great if you could update the weights on aws with these: https://www.dropbox.com/s/i9sara0ds90d79b/pretrained_weights.zip?dl=0
I will post again once my commits are squashed.
Cheers,
Fabian

Mhm I am not much of a git ninja. I just attempted to squash my commits but am unsure whether this actually worked. This is the commit where I squashed everything: d25461731732487ae8a6dc9ea6c338ef5986018b
Please let me know whether this is wat you need ;-) If so then the PR is ready to go.
(Please remember to update the weights on aws)
Cheers,
Fabian

Hmm, apparently github now lets me squash directly from the merge button... I've updated the weights and I'll merge now.
Thank you again for the great example!

Alright then! Glad you like it!

@FabianIsensee You mentioned in a code comment that the paper does not describe the dropout configuration.
This can be found in the authors' implementation. In particular, one dropout layer with p=0.5 is added after net['contr_3_2'] and another one after net['contr_4_2'].
Source: http://lmb.informatik.uni-freiburg.de/people/ronneber/u-net/


Update documentation for lolex 2

Fixes https://github.com/sinonjs/lolex/issues/109

Thanks! Clear and sufficient


Hash URL's not working

Hi guys
I'm using redux-router 2.0.0.
When implementing the router I was expecting hash url's. I can see that react router supports them. I can also safely say that the routing with redux-router works as it should, just no state in the url is updated.
Could it be because browser history is enabled by default?
How can I solve this?
Sincerely, Jan

What history are you importing? For HashHistory you need the following:
import { createHashHistory } from 'history';

And are you adding the reducer to your store?
If you continue to have issues, providing a small example would be helpful. Also taking a look at some of the examples and seeing how these differ from your setup.


identity libs not found with version 8.4.2

Describe the bug
I use maven to load the 8.4.2 version, but the identity libs
io.camunda:identity-spring-boot-starter:jar:8.4.3
io.camunda:identity-sdk:jar:8.4.3
couldn´t find.
I use a intern artifactory, but this is just a proxy and will normaly find every common lib.
To Reproduce
<spring-zeebe.version>8.4.2</spring-zeebe.version>
...

io.camunda
spring-zeebe-starter
${spring-zeebe.version}

Expected behavior
No exceptions if I load the libs.
Log/Stacktrace
[WARNING] Failed to download identity-sdk-8.4.3.pom [https://xxxx/artifactory/libs-release/]
[WARNING] Failed to download identity-sdk-8.4.3.pom [https://xxxx/artifactory/libs-snapshot/]
[WARNING] The POM for io.camunda:identity-sdk:jar:8.4.3 is missing, no dependency information available
[WARNING] Failed to download identity-spring-boot-starter-8.4.3.pom [https://xxxx/artifactory/libs-release/]
[WARNING] Failed to download identity-spring-boot-starter-8.4.3.pom [https://xxxx/artifactory/libs-snapshot/]
[WARNING] The POM for io.camunda:identity-spring-boot-starter:jar:8.4.3 is missing, no dependency information available
[WARNING] The artifact io.camunda:spring-zeebe-starter:jar:8.4.2 has been relocated to io.camunda.spring:spring-boot-starter-camunda:jar:8.4.2
[WARNING] Failed to download identity-spring-boot-starter-8.4.3.jar [https://xxxx/artifactory/libs-release/]
[WARNING] Failed to download identity-spring-boot-starter-8.4.3.jar [https://xxxx/artifactory/libs-snapshot/]
[WARNING] Failed to download identity-sdk-8.4.3.jar [https://xxxx/artifactory/libs-release/]
[WARNING] Failed to download identity-sdk-8.4.3.jar [https://xxxx/artifactory/libs-snapshot/]
Cannot resolve io.camunda:identity-spring-boot-starter:8.4.3
Cannot resolve io.camunda:identity-sdk:8.4.3
Environment:

OS: Windows
Deployment: -
Spring-Zeebe Version: 8.4.2
SpringBoot Version: 3.2.2
Configuration: -

We have same problem.

Same. Can someone from Camunda take a look at this? Nobody can update to 8.4.2. Is this going to be fixed with 8.5.0?

This is a community-supported project, and while we do have some Camundi that work in this repo, there is no maintainer(s) at this time.
For the official Camunda-supported version, please refer to the Zeebe repository. You can also read more about the changes in a recent blog.

Thanks. Do you mean this? https://mvnrepository.com/artifact/io.camunda/spring-boot-starter-camunda-sdk/8.5.0
It has the same issue, the identity SDK is not published so it can't be directly used.

@namero999 can you confirm you are still running into this after following https://docs.camunda.io/docs/apis-tools/spring-zeebe-sdk/getting-started/#add-the-spring-zeebe-sdk-to-your-project?

Thank you for the reply.
I think the question that everybody here is asking, is why is that artifact not published on Maven, together with the other dozens of artifact that Camunda (community or otherwise) already publishes?
For one transitive dependency (which is not even used by most people), developers have now to add a new repository to their project. While not a big deal, it is certainly surprising and time consuming as opposed to a smooth experience as it used to be.
What is preventing the identity sdk to be published to maven?

Any news on this?

Also, I can confirm that the issue is still present when using the custom maven repository @akeller
Could not find io.camunda:identity-sdk:8.5.0.
Required by:
    project :CamundaWorkerDemo > io.camunda:spring-boot-starter-camunda-sdk:8.5.0

Hi @namero999, this SDK is not part of my team, so I'm having discussions internally with no public facing issues I can reference.
Currently, the Identity team uploads their artifacts only to Camunda Artifactory. This is in progress on their backlog, but I can't give an estimate. I can tell you that this includes uploading the SDK and starter to Maven central.

Fantastic this is great to hear, thank you!


Add functionality for updating station url via www.radio-browser.info

When user adds to his database new station from the stations listed by radio-browser after some time its URL may get out-dated i.e. the station may change to stream under different URL. radio-browser keeps track of such changes which is why such feature for automatic update could be implemented.
The problems:

only stations that are listed on www.radio-browser.info could make use of such feature
is the name of station enough to resolve to correct station? what if station name has changed in www.radio-browser.info database
www.radio-browser.info is kept updated by people, if no one updates the station with new working URL it will stay broken

I second this. I don't see a way to provide maintenance to stations.

It would be a big feature I suppose. There are a couple of things that stop me from trying to implement it.

Although now only www.radio-browser.info is used when searching for stations originally I was planning that there would be possibility to search stations within multiple databases. Right now I can think of https://directory.shoutcast.com/ as a possible addition.
This would complicate things a little bit. Stations would probably need to be associated with source database to avoid conflicts
What about user created stations (those added manually)? I imagine we would just be unable to update those.
How should we keep track of station info. Should we just do it by station name (is it enough)? Can we expect that it will not change within the external database? What if there are duplicates of a name (www.radio-browser.info allows that)?

These are questions that are hard for me to answer alone. Nonetheless I'll try to analyze it little more so maybe I will be able to get to some conclusions.


Test Windows Phone

Pinging @lazaronixon to see if he can test the live deployment of this project here https://rails-push-notifications-test.herokuapp.com/ . Specially the Windows Phone push notifications.

Ok.. I will test soon as posible.. not is required Pem encoded certificate for test.

Really? So under development you can push notifications just by knowing the 'push url' of the destination device?


[Vue] Don't show NotFound component while data is not loaded

Description / Motivation

Prevent Vue app to show NotFound component while we don't have layoutData result
Testing Details

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation update (non-breaking change; modified files are limited to the /docs directory)

Closing as target was switched to dev and fix is already in place.


Update metadata processor test

Follow-on resolution to: https://github.com/harvard-lts/DSpace/issues/9


[#415] bump up webdriver-manager 3.5.4->3.7.0

also fix tests
update CHANGELOG.md 2.0.0b4 version

Closes #415

Reason: already in master


Playlist m3u

It would be great if the app could also open and run m3u Playlists
m3u list
https://github.com/iptv-org/iptv
In order to play Playlist m3u, you need to install other player VLC kodi.
Their players also have problems with format decoding, consume more resources on the phone and have a complicated interface.

Duplicate of #167


Macro-generated defines aren't recognized by module-macroexpand

When generating a define with a macro, such as this:
(define-syntax test
  (syntax-rules ()
    ((_ a)
     (define asdf (lambda () 'aaa)))))

(test 2)
(asdf)

You get this error: These variables are undefined: (asdf) #<module-reference #3 loader: #<loader #4 name: local> path...
However, when you execute the same code in Blackhole REPL, it does work.
Commenting out the checks done at module-macroexpand (around line 482: undefined-names) makes it work, but of course you loose the undefined variables check, which is a nice thing to have.

Interesting.
Actually, as I understand it, it is the REPL's behavior that is incorrect. syntax-rules macros are hygienic, with no option for opting out of the hygiene. That also implies that you can't export a name from a syntax-rules macro unless you actually put the name as a parameter to the macro. For instance,
(define-syntax test
  (syntax-rules ()
    ((_ a)
     (define a (lambda () 'aaa)))))

(test asdf)
(asdf)

ought to work as intended. (I haven't tested this though.)

The reason it works in the REPL is because

To allow for mutually recursive top level functions, it allows usage of undefined names, assuming that they might be defined in the future
The original syntax-rules macro that you presented does generate a top level binding, even though it ought to be hidden.

The second point above is where the error is: since it is a top level REPL binding it simply calls it ~#asdf. This is wrong, since you should be able to create your own top level asdf REPL binding, and it should even be possible to invoke the macro several times and have it generate unique top level bindings.
Simply expanding the define into something equivalent to a let would be correct in this particular case, but not in general:
(define-syntax test2
  (syntax-rules ()
    ((test2 name value)
     (define tmp value)
     (define (name) (tmp)))))

(test2 one 1)
(test2 two 2)

(pp (+ (one) (two)))

The previous code snippet should print 3, and in the top level there should be 5 bindings:

The test2 macro
The tmp name generated by the first invocation of test2, with the value 1
The tmp name generated by the second invocation of test2, with the value 2
The one function, which accesses the first tmp name
The two function, which accesses the second tmp name

Black Hole sure doesn't do the right thing in this case; it lacks the logic of creating several unique top level names with the same name. It would (I think) do the right thing if you wrapped the whole thing in a let though.


Cache not used if installed via homebrew (macOS)

Caching is not used (if explicitly enabled) when php-cs-fixer is installed via homebrew on macOS. I guess the detection if installed via composer or phar fails.

Feel free to debug it and contribute.
We do not support homebrew directly.

ping @pascal-hofmann do you have any more input for us to work with?

This is the homebrew formular for php-cs-fixer: https://github.com/Homebrew/homebrew-php/blob/master/Formula/php-cs-fixer.rb
As far as I understand it downloads the .tar.gz and extracts it in $HOMEBREW_CELLAR/php-cs-fixer/$VERSION/. Afterwards a symlink /usr/local/bin/php-cs-fixer is created, which points to $HOMEBREW_CELLAR/php-cs-fixer/$VERSION/php-cs-fixer
So php-cs-fixer is not installed via composer, and also not as a phar. But IMHO still caching should be enabled in this case. But it's disabled. (I guess the detection is there, to prevent caching in a php-cs-fixer development environment.)

Do you want to propose a solution?
We don't have anyone working with brew i think

Since homebrew-php indeed installs the tool from source in a tar caching is not available.
Wonder why they install like this, as it also make the selfupdate command not available.
@keradus what is the reason caching is only on for phar builds?

Closing as obsolete, not enough details and fact that we do not support homebrew.


I use the pdf.js plugin in the project, it will clear my website cookie information, how I want to operate

When I use the pdf.js plugin in the project, it will clear the cookie information from my website, and how do I do it?
Users at the site of the original login, then refresh the page, is out of the state, how should I do?

This is not caused by PDF.js as PDF.js does not clear or overwrite cookies. It's most likely a problem with your own implementation. Closing as invalid unless a working example demonstrating the problem can be put online.


Initial attempt at extracting two person chat out of conversable agent

Why are these changes needed?

Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Codecov Report
Attention: Patch coverage is 0% with 53 lines in your changes are missing coverage. Please review.

Project coverage is 27.62%. Comparing base (78aa0eb) to head (42d4a26).

Files
Patch %
Lines

autogen/agentchat/two_person_chat.py
0.00%
51 Missing and 2 partials :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2173      +/-   ##
==========================================
- Coverage   37.36%   27.62%   -9.74%
==========================================
  Files          74       75       +1
  Lines        7481     7534      +53
  Branches     1617     1762     +145
==========================================
- Hits         2795     2081     -714
- Misses       4445     5203     +758
- Partials      241      250       +9

Flag
Coverage Δ

unittests
27.54% <0.00%> (-9.81%)
:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


About the code "images = torch.cat(images, dim=0)"

hi~ the "images" here is a tensor, but the first parameter in cat() is a tensor sequence. I don't understand what's going on here. Could you please explain it, thank you very much.

hi I have the same question. Did you figure out why?


Building for Android by Android NDK.

Support for building OpenSSL for four 32-bit architectures supported by Android NDK.
Included:

Changes to "config" and "Configure" scripts.
New file: setenv-android.sh script, based on
http://wiki.openssl.org/images/7/70/Setenv-android.sh script, mentioned at
http://wiki.openssl.org/index.php/Android .

The compiler options in "Configure" script are based on:

Existing options from previous versions of this script.
Options that are supplied by Android NDK r10 for linux-x86_64, toolchains:
arm-linux-androideabi-4.6, mipsel-linux-android-4.6, x86-4.6.
As can be seen, NDK supplies quite different options are for different
architectures.
Considerations about footprint vs speed of the resulting code on mobile
devices.

Hi, Patch Reviewers.
This pull request is in queue for almost 4 months already. I'd like to note that:

The main part of the patch ("Configure" and "config" files) is short and simple.
The added set-env script is based on the script that already resides on OpenSSL Wiki.
Patches from me were accepted before (in pre-github era), for what it's worth.

Can I please get some attention to my pull request?

in general RT is the place we look first. four months isn't long :) we're working on improving but ...

@levitte isn't this done now as part of the config revamp?

Uhmmmm, not at all sure

Hi guys, you can add support to generate objs files in /obj/local/<CPU_ABI>/objs/ crypto/, openssl/, ssl/ and ssltest/
Sample as generated by https://github.com/yixia/OpenSSL-Vitamio
I only get this files in ``/obj/local/<CPU_ABI>/ libcrypto.soandlibssl.so`
Thank You :+1:

Not sure what we do with this one, but my guess is part of the problem is no one on the dev team has access to this platform to test this out.

I'd like to leave some thoughts here.

During 19 months lifetime of this pull request, nobody found an issue with it, nor proposed a better patch. I don't know how many people on the Internets tested the patch, but this patch works for our company for all those 19 months.

"my guess is ... no one on the dev team has access to this platform". It is pretty easy to get access to the Android platform. One can buy an Android phone (or download an emulator!), compile OpenSSL using this patch, write a sample program or get one of the existing OpenSSL tools compiling, and run that program on the phone. It can be a pure-C command-line program, compiled by standalone NDK toolchain and be ran from "adb shell". You don't have to make an Android project with Java, XML, Manifest, Activities or JNI in order to test.

I see support for many rare platforms in the "Configure" script, much rarer than Android. Does it mean that every platform was actually tested by an OpenSSL dev team member? I doubt it.

What can go wrong?  Going from "unsupported" state to "badly supported"? Hidden exploit? Look at the patch, the only changes in "Configure" and "config" scripts are compilation flags.  No C code is changed.

Google is regularly releasing newer Android NDK versions and sometimes even adding new architectures (64-bit ones). If my patch gets merged - I will feel more motivated to update the patch for newer NDKs. FWIW.

Clover Network is using this patch to deploy OpenSSL on dev devices. We have found no issues.

This clearly doesn't apply to 1.1.0 or master.

Sure it doesn't, after 2+ years. I am willing to update the patch, but not sure how to do it. GitHub says "Add more commits by pushing to the building-for-android branch on alexeikh-promon/openssl." Just adding commits to the existing branch will not help, because the current branch is branched off too old commit. What to do then? Rebase the branch to the latest master? Or make a new branch and new pull request?

Rebasing and forcing a push should do it.

An idea: NDK now recommends clang over gcc, and Google is planning drop gcc. It might be better to add clang support instead of fix gcc.
FYI: I have a patch that uses clang and unified headers introduced in NDK r14 beta1 at https://github.com/yan12125/python3-android/blob/master/mk/openssl/fix-cflags.patch.

Thanks for the help, kroecx.
Build scripts on OpenSSL_1_1_0-stable or master are indeed very different from OpenSSL_1_0_2-stable branch or 2-years-old master. I don't know when I will have time to rework this patch for master.
In the meantime, for those who still use OpenSSL 1.0.2 or earlier, I've updated this patch to the tip of OpenSSL_1_0_2-stable branch. The new patch can be found on this branch:
https://github.com/alexeikh-promon/openssl/commits/OpenSSL_1_0_2-stable.Building_for_Android.2017-01
The new patch consists of several commits, starting from commit edcc244 "Remove generated assembly files during "make clean"."
yan12125, your idea totally makes sense. I tried to introduce clang and unified headers support (still on OpenSSL_1_0_2-stable, but it wasn't very easy). Particularly, when trying clang instead of gcc, the build kept failed on "makedepend" and it wasn't obvious to me how to fix it. Unified headers support is currently only available in NDK r14 beta 1, but this NDK version somewhy ignores NDK_TOOLCHAIN_VERSION parameter (though respects APP_UNIFIED_HEADERS parameter) and forces clang even if I ask for gcc. So it was a bit difficult to try unified headers on that branch too. Well, unified headers for an older OpenSSL branch are not so relevant.

Build with clang is not in scope of this issue and all my following comments are only for gcc.
I cannot see any reason to update build configuration for 1.0* branches.
The current configure targets are enough to me to build OpenSSL for all six supported by Android NDK (abiFilters) : armeabi, armeabi-v7a, x86, mips, arm64-v8a, x86_64 and mips64.

"android" is generic(!) configure options and must not be touched. It could be used for 64-android builds.

OpenSSL builds system allow  compiler or linked flags to be passed on command line. As for other openssl build system for android Configure should contain minimum of flags.
It is end user responsibility to choose those flags. For instance build engineer may decide to use "hardening" flags.

It is user responsibility to define environment variables that help him easy to switch(upgrade) to another ndk.

In conclusion:
Modification that remove "android" from Configure is wrong.
Rest of the patch belongs to build guide.
I would like to propose issue to be closed.

Here I'd like to proposed my pull request at #2229. It implements GCC & Clang dual support with Android NDK. Unified headers support is not included - I'd like to postpone it until NDK r14 stable is released.

Particularly, when trying clang instead of gcc, the build kept failed on "makedepend"

I guess either the latest OpenSSL or the latest NDK has fixed it.

Unified headers support is currently only available in NDK r14 beta 1, but this NDK version somewhy ignores NDK_TOOLCHAIN_VERSION parameter (though respects APP_UNIFIED_HEADERS parameter) and forces clang even if I ask for gcc.

Sounds an NDK bug. Anyway OpenSSL doesn't use ndk-build or Android.mk at all, so those problems are irrelevant.

This is outdated; see also #2229

Hi all, i want to make a simple SSL connection with server in android NDK.  Could anyone tell me how to setup SSL lib in android NDK step by step?. Any help would be appreciated.


standardize exceptions

some exceptions are in subfolders, some has a exceptions suffix and some don't have it. We should standardize it.

No Subfolders
No Suffix, only for abstract exceptions if no better name can be found


Suggestion - Integrate MobileSAM into the pipeline for lightweight and faster inference

Reference: https://github.com/ChaoningZhang/MobileSAM
Our project performs on par with the original SAM and keeps exactly the same pipeline as the original SAM except for a change on the image encode, therefore, it is easy to Integrate into any project.
MobileSAM is around 60 times smaller and around 50 times faster than original SAM, and it is around 7 times smaller and around 5 times faster than the concurrent FastSAM. The comparison of the whole pipeline is summarzed as follows:

Best Wishes,
Qiao

Thanks for your great tool, salt.
We can add a comment through the experience implementing MobileSAM to our image annotation tool RectLabel.
MobileSAM was trained on auto generated annotations in Segment Anything 1 Billion (SA-1B) dataset.
So that the mask area when you click a foreground point tends to be large.
But, if you click multiple foreground/background points, the accuracy is almost the same as ViT-Base SAM.
The model size of MobileSAM is smaller so that the calculation speed is faster.
We used the quantized ViT-Base SAM model as comparison.
28.1MB mobile_sam_preprocess.onnx
16.5MB mobile_sam.onnx
108.9MB sam_vit_b_01ec64_preprocess.onnx
8.8MB sam_vit_b_01ec64.onnx
When click a foreground point using MobileSAM.

When click a foreground point using ViT-Base SAM.

Please let us know your opinion.


No clue how to compile this correctly to work for both simulator and device

So I've tried to compile a fat file using:
lipo -create ${CURRENTPATH}/build/${LIB}/i386/lib/libsybdb.a ${CURRENTPATH}/build/${LIB}/armv7/lib/libsybdb.a ${CURRENTPATH}/build/${LIB}/armv7s/lib/libsybdb.a ${CURRENTPATH}/build/${LIB}/arm64/lib/libsybdb.a ${CURRENTPATH}/build/${LIB}/x86_64/lib/libsybdb.a -output ${CURRENTPATH}/build/${LIB}/Fat/lib/libsybdb.a
after compiling freetds for all of those architectures.
This ends up making a fat binary file that works for devices but fails on simulators.  Any idea what I'm doing wrong?

I'd like to add that the reason I'm trying to recompile the freetds binary is because the binary uploaded here is not compatible with ipv6.  After recompiling with a newer freetds version it is compatible with ipv6, but as I said it does not work on the simulator.

I'm pretty busy for the next two weeks, but I can try to look at this after that.
If I get it figured out, I'll leave a comment explaining what I did to get it working as well as update the repo with the new binary.  Otherwise, if you get it working in the meantime, can you make a PR with the working binary?

I actually found a working binary from the project you were inspired by, SQLClient.  I didn't see anywhere where he explained how he compiled from but if you look at his page he has the precompiled binary with the most recent version of freetds.  Thanks for responding, though.  Also this is Jake, I'm the one working on 585 mobile now lol.


Purchase the item: ItemID doesn't exist

The documentation says the call body for Add/Purchase requires an ItemID, which makes sense.
The API reference and Unity SDK don't have a field for the ItemID though. It's not possible to actually purchase an item, because the ItemID can't be specified.

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

ID: 7e87cee8-7304-5fbc-7626-0d59fa1b1684
Version Independent ID: 7e87cee8-7304-5fbc-7626-0d59fa1b1684
Content: Player Inventory Quickstart - PlayFab
Content Source: playfab-docs/features/economy-v2/inventory/quickstart.md
Product: playfab
GitHub Login: @wesjong
Microsoft Alias: wesjong

@drallcom3 Can you give a link of what you're referring to re: "The API reference and Unity SDK don't have a field for ItemID"?
Here's the PlayFabSDK Client from the Unity Example project in the playfab repo
https://github.com/PlayFab/UnitySDK/blob/fe27c714b14e3bcd30abbd482f4aaf6b4cdaabc5/ExampleTestProject/Assets/PlayFabSDK/Client/PlayFabClientInstanceAPI.cs#L1405
And the REST API reference
https://docs.microsoft.com/en-us/rest/api/playfab/client/player-item-management/purchase-item?view=playfab-rest
You may have better luck asking your question on the playfab forum https://community.playfab.com/index.html

I got confused by InventoryItemReference, which is actually a
CatalogItemReference.
I don't have inventory items before I buy any.
On Wed, Sep 14, 2022 at 10:36 PM Elliot DeMatteis @.***>
wrote:

@drallcom3 https://github.com/drallcom3 Can you give a link of what
you're referring to re: "The API reference and Unity SDK don't have a field
for ItemID"?
Here's the PlayFabSDK Client from the Unity Example project in the playfab
repo
https://github.com/PlayFab/UnitySDK/blob/fe27c714b14e3bcd30abbd482f4aaf6b4cdaabc5/ExampleTestProject/Assets/PlayFabSDK/Client/PlayFabClientInstanceAPI.cs#L1405
And the REST API reference
https://docs.microsoft.com/en-us/rest/api/playfab/client/player-item-management/purchase-item?view=playfab-rest
You may have better luck asking your question on the playfab forum
https://community.playfab.com/index.html
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/playfab-docs/issues/351#issuecomment-1247326211,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF7FO6ZNXXLP34ZAJFIZVRDV6JAMVANCNFSM6AAAAAAQJ7HULQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

We've since improved the item field names and examples. ItemId should now be properly seen as a parameter for the Inventory operation APIs


Malonso/durable objects

[x] use class and script name for DO bindings instead of namespace id
[x] adhoc create/delete/unused migration support
[ ] tagged create/delete/unused migration support

[ ] fetch current migration tag from API
[ ] coalesce migrations from wrangler.toml into single API migration

[ ] support for rename/transfer migrations

Pre-merge tasks:

[ ] help text for migration directives
[ ] change modules support to only treat .bin as an arraybuffer -- unknown extensions are an error
[ ] rename deploy output helper to run_deploy


socket.gaierror: [Errno -2] Name or service not known

installed: python 2.7.6, boto
i am having this error:
Traceback (most recent call last):
File "/s3-parallel-put", line 410, in
sys.exit(main(sys.argv))
File "/s3-parallel-put", line 381, in main
bucket = connection.get_bucket(options.bucket)
File "/usr/local/lib/python2.7/dist-packages/boto/s3/connection.py", line 503, in get_bucket
return self.head_bucket(bucket_name, headers=headers)
File "/usr/local/lib/python2.7/dist-packages/boto/s3/connection.py", line 522, in head_bucket
response = self.make_request('HEAD', bucket_name, headers=headers)
File "/usr/local/lib/python2.7/dist-packages/boto/s3/connection.py", line 665, in make_request
retry_handler=retry_handler
File "/usr/local/lib/python2.7/dist-packages/boto/connection.py", line 1071, in make_request
retry_handler=retry_handler)
File "/usr/local/lib/python2.7/dist-packages/boto/connection.py", line 1030, in _mexe
raise ex
socket.gaierror: [Errno -2] Name or service not known
May i know how to resolve it?
Thanks.

I am not in a position to provide technical support for s3-parallel-put.

Is that a joke reply @twpayne ?  Please re-open; I have the same problem, my bucket happens to not reside in US-Standard, most likely you simply need to support the bucket's region if you're using boto under the hood.

Perhaps author means they don't have time to support their code anymore. Which is why forking is great.
https://github.com/weftio/s3-parallel-put/blob/d0aaf13e1ec9213877679c08813469b1678353d7/s3-parallel-put#L382
Here's a quick change I made to support my bucket in us-west-2 region @alkemrebecca, feel free to use that script instead and make changes to your bucket's region.
I personally don't work much with Python and have little interest or time myself in creating a proper fork, library, wrapper, or supporting this (awesomely powerful) library just so that it can handle buckets in another region correctly and easily, but I'm probably not the first fork to do so on what appears to be abandonware (and mostly uncommented abandonware).
Here's to hoping someone else comes along and fixes it up the right way.

PR ^ above to support fixing this issue.

Hi mlkeatlas,
thanks so much. It seems to work.
May I know if I am able to update to a specific folder in s3?
It seems like for the current it uploads the same folder structure as the origin.

I'm sure it's possible, but for me this was a 1-time task, so I just set my personal s3-parallel-put source path to mimic the structure of the bucket keys I wanted on s3:
python s3-parallel-put --bucket=weft-wind-data --secure --put=update --processes=50 --content-type=guess --verbose --log-filename=/tmp/s3pp.log /your/local/ec2/path
where every directory and file in /your/local/ec2/path gets mimicked into the bucket path.
I challenge you to find out this parameter on the command line and figure out how the tool creates bucket paths that mirror the local file system, and then maybe you find a way to add a new bucket path prefix to upload to. (Although... isn't there a "prefix" parameter here?" Either way, I don't have additional time to help customize and support this script right now.

Yeah, you should be able to just use --prefix=PREFIX for specific bucket prefix key (technically there are no folders in s3, just a bucket + bucketKey. The bucketKey includes the path and file name all-in-one.

Hi mlkeatlas, i've tested --prefix and it seems like there is no response to the parameter.
I realised that it uploads as standard.
May I also know if it is possible to upload as reduced redundancy instead?

@alkemrebecca again this is not my library to support, please consider if you need to extend functionality that you make an effort to debug and make PRs yourself. Or read the documentation of S3 and this script:

--header=HEADER:VALUE adds an arbitrary header to the S3 file. This option can be specified multiple times.`

See: http://docs.aws.amazon.com/AmazonS3/latest/API/RESTObjectPUT.html

To store an object using reduced redundancy, set the x-amz-storage-class request header to REDUCED_REDUNDANCY.

How about you try: --header=x-amz-storage-class:REDUCED_REDUNDANCY with this script, see if it works?

Hi mlkeatlas, the reduced redundancy works!
Thanks so much!
Guess I just have to figure out the folder part.

Lucky you :) You're welcome :) The "folder" part you must remember is that on s3, there is only "Bucket Name" and "Bucket Key". There concept of "folders" is simply bucket keys with (/slashes/in/the/keys/file/name).

Sure. I will take note of that.
Thanks soooo much!

Hi mlkeatlas, just to share with you and the others who are looking for the same.
I've figured how to do the folder part.
I have created the exact directory which I wanted in s3 and created a symbolic link to the original folder.
This way, when running the script, it will follow the path which I indicated. =)
Though i spotted that the final child files do not follow the permission that is set while indicating the command. Might want to upload to another folder and change the permission first before moving to the actual folder.

I wrote s3-parallel-put for a project I was working on three years ago, and have not used it since. If someone would like to take over maintainership then I would be very happy - I can add you as a contributor here so you can push and merge pull requests.

@twpayne Appreciate the offer and I wish I had a reason to maintain it myself. Alas, I'm not a daily Pythonista, nor do I have daily needs to s3-parallel-put, but so long as you're willing to accept PRs from time to time, that's good enough for most of us.

Some kind of test suite an probably a pip package would be nice, but I don't really even know what goes into making those. Most of my time are in ruby/node/angular projects these days (gems, npm and bower) with a sprinkling of Clojure/Java (maven)...


Support multiple main cameras

Any ideas?

Use RenderLayers? Each Camera having a different one, and you duplicate the effect for every one of them?
That's how I plan to handle it in bevy_basic_portals but I didn't test that yet and RenderLayers are ignored by the shadow system so that create problems.


Create flow diagrams for PR and propose-downstream workflows

make them simple and include sources here in the repo so the diagrams can be updated in the future
show what's happening where: build, test, code, github, copr, koji, dist-git...

Closing in favour of https://packit.dev/docs/fedora-releases-guide/ and #480


update dacfx to 18.8

Description
Updates DacFx to 18.8

Related issue:
https://github.com/actions/virtual-environments/issues/4265
Check list

[x] Related issue / work item is attached
[x] Tests are written (if applicable)
[x] Documentation is updated (if applicable)
[x] Changes are tested and related VM images are successfully generated

@miketimofeev done, thanks!

/azp run windows2016, windows2019, windows2022


added version checker

You should fix the test for version command to use versionChecker instead ....

@progrium it's fixed now


add credits to finishtodo files

Coverage remained the same at 82.01% when pulling c745c6a52936066e6a5c702dd2ea3f5f33df9689 on linnnruoo:linruo/todotask into 0353c5df0939cdf3f46f6ea2e616881a34a9ce62 on CS2113-AY1819S1-W12-1:master.


create vm

nice document to create VM -simplified

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

ID: 2e58109c-c8d3-6693-8e1d-03a89ca26168
Version Independent ID: f20a7fc4-328a-e90d-29db-92a91aa1b2bd
Content: Quickstart - Create a Windows VM in the Azure portal
Content Source: articles/virtual-machines/windows/quick-create-portal.md
Service: virtual-machines-windows
GitHub Login: @cynthn
Microsoft Alias: cynthn

Thanks for the feedback! We are currently investigating and will update you shortly.

@naveez-alagarsamy Glad to know the doc was helpful to you. :)
We will now close this issue. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.