[SPARK-37931][SQL] Quote the column name if neededQuote the column name if needed

What changes were proposed in this pull request?
Quote the column name just needed instead of anyway.
Why are the changes needed?
#comments
Does this PR introduce any user-facing change?
Yes， when end-user get the table metadata， if the columns does not need quote. it will be a normal string.
How was this patch tested?
existed testcase.

@cloud-fan Could you take a look ? Thank you very much.

Yes， when end-user get the table metadata， if the columns does not need quote. it will be a normal string.

Let's be specific here. This only affects SHOW CREATE TABLE AFAIK, while "get the table metadata" is low-level operation and it's scarying to change it.

looks fine and I can't think of any breaking cases. cc @HyukjinKwon @viirya as well

thanks, merging to master!

Yeah, I think this is fine.

This is a user-facing breaking change. The third party libraries (esp. SQL clients) might rely on the current format. We should avoid making such a change.

Hm, I think it's less likely that this breaks something because it removes the backquotes when they are unnecessary, meaning that it's still able to read and parse it back with lower versions of Spark.

I think we should clearly document the display-like commands. Their output is not structured and we only expect humans to read it, not programs.


testCase Exception

testCase Exception:com.alibaba.fastjson.JSONException: can't create non-static inner class instance.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.lijiuyi seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #3434 into master will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #3434   +/-   ##
=========================================
  Coverage     80.35%   80.36%
- Complexity     7457     7466    +9
=========================================
  Files           157      157
  Lines         27889    27912   +23
  Branches       6723     6731    +8
=========================================
+ Hits          22411    22431   +20
- Misses         3460     3462    +2
- Partials       2018     2019    +1

Impacted Files
Coverage Δ
Complexity Δ

.../com/alibaba/fastjson/serializer/BeforeFilter.java
94.73% <0.00%> (-5.27%)
5.00% <0.00%> (ø%)

...com/alibaba/fastjson/serializer/SerialContext.java
82.69% <0.00%> (-3.85%)
21.00% <0.00%> (-1.00%)

...stjson/support/jaxrs/FastJsonAutoDiscoverable.java
81.81% <0.00%> (-1.52%)
4.00% <0.00%> (ø%)

...main/java/com/alibaba/fastjson/util/FieldInfo.java
86.62% <0.00%> (ø)
98.00% <0.00%> (-1.00%)

...main/java/com/alibaba/fastjson/util/TypeUtils.java
78.85% <0.00%> (+0.15%)
651.00% <0.00%> (+10.00%)

...ba/fastjson/parser/deserializer/Jdk8DateCodec.java
77.66% <0.00%> (+0.25%)
105.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

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


👑 Support for resource pool and resource pool classifiers in the embedded UI

viewer resource pool parameter: https://github.com/ydb-platform/ydb/pull/10890
Documentation: https://ydb.tech/docs/ru/dev/resource-consumption-management

[ ] pick an icon for resource pool and resource pool classifer – YCDESIGN-8700
[ ] add proper icon when object is listed in schema tree
[ ] add details view: set of tabs, properties on every tab. Right panel and bottom left panel
[ ] add create / alter templates in New SQL menu
[ ] add drop / alter templates to context menu under three dots

It will be usefull to have all the properties on the info tab:

In case of a -1 value, we need to have a proper description that it means that this parameter is disabled.

@dorooleg please let us know when it's enabled on any preprod cluster


[WIP] Do not assume sqlite3_column_text is valid UTF-8.

Fix Statement::value_ref

See #548

Another approach might be to return the &[u8] and let the caller handle it. This API means that if you have non utf8-text it can't be accessed.

Indeed !

Indeed !

Concretely, this code spends around 10% of its time validating UTF8. if getting a &[u8] out of a ValueRef were possible, we could avoid this. (That said, it's not particularly hot code, in practice IO takes most of the time, but still...)
Using ValueRef (outside of rusqlite's internals) already implies you're worried about perf some anyway.


Use the shinning new ResizeObserver

react-container-query to utilize ResizeObserver
The Need to Design API for the Future
The new ResizeObserver API provides a native browser API to listen to element resize event. For unsupported browsers, to listen to resize event, we use element-resize-detector, which requires inserting sentinel elements to the DOM. E.g.

 ...

 ...

In the above markup, .box is our component. .sentinel is inserted for observing resize event. A style="position: relative" is also added to .box for .sentinel to work. It's possible that those added styles and elements can mess up with the component styles.
However, if ResizeObserver API made into production, the markup can be reduced to:

 ...

In order to be future looking and transition to ResizeObserver API without developer changing their code, we have to design react-container-query API around the usage of ResizeObserver.
Limitation of Current API
The current API uses a higher order component pattern:
const HigherOrderComponent = applyContainerQuery(BoxComponent, {
  'width-between-400-and-599': {
    minWidth: 400,
    maxWidth: 599
  }
});

This API is limited because it cannot apply container query to stateless components (see Issue #15). A workaround is to wrap BoxComponent with another element (see PR #38). So the markup will become:

 ...

 ...

.container is a newly added element to wrap BoxComponent. It has several downsides:

.container is redundant if ResizeObserver is available.
It changes the intended HTML structure. If we expect BoxComponent to be responsive, we don't expect another element to wrap it.

The applyContainerQuery API also has some limitations. When to design the BoxComponent, the responsive logic is handled by passing a property from outside. BoxComponent doesn't know the outside logics but it is required to work with the logic. This is not strictly modular code.
New API Design
To make the API easier to reason and be future looking. I propose to create a new component style API:
import ContainerQuery from 'react-container-query';

// Same query structure as before
const query = { ... };

const Box = () => (
  <ContainerQuery query={query} tagName="div" className="box">
    {(matchedQueries) => (

 ...

    )}
  </ContainerQuery>
);

Here we pass a function as the child of ContainerQuery. The function will only be called when matchedQueries changes. It has several advantages:

Work with stateless component naturally while maintaining compatibility with future ResizeObserver API.
Component composing code stays closer to view code. Easier to reason about.

react-container-query to utilize ResizeObserver
The Need to Design API for the Future
The new ResizeObserver API provides a native browser API to listen to element resize event. For unsupported browsers, to listen to resize event, we use element-resize-detector, which requires inserting sentinel elements to the DOM. E.g.

 ...

 ...

In the above markup, .box is our component. .sentinel is inserted for observing resize event. A style="position: relative" is also added to .box for .sentinel to work. It's possible that those added styles and elements can mess up with the component styles.
However, if ResizeObserver API made into production, the markup can be reduced to:

 ...

In order to be future looking and transition to ResizeObserver API without developer changing their code, we have to design react-container-query API around the usage of ResizeObserver.
Limitation of Current API
The current API uses a higher order component pattern:
const HigherOrderComponent = applyContainerQuery(BoxComponent, {
  'width-between-400-and-599': {
    minWidth: 400,
    maxWidth: 599
  }
});

This API is limited because it cannot apply container query to stateless components (see Issue #15). A workaround is to wrap BoxComponent with another element (see PR #38). So the markup will become:

 ...

 ...

.container is a newly added element to wrap BoxComponent. It has several downsides:

.container is redundant if ResizeObserver is available.
It changes the intended HTML structure. If we expect BoxComponent to be responsive, we don't expect another element to wrap it.

The applyContainerQuery API also has some limitations. When to design the BoxComponent, the responsive logic is handled by passing a property from outside. BoxComponent doesn't know the outside logics but it is required to work with the logic. This is not strictly modular code.
New API Design
To make the API easier to reason and be future looking. I propose to create a new component style API:
import ContainerQuery from 'react-container-query';

// Same query structure as before
const query = { ... };

const Box = () => (
  <ContainerQuery query={query} tagName="div" className="box">
    {(matchedQueries) => (

 ...

    )}
  </ContainerQuery>
);

Here we pass a function as the child of ContainerQuery. The function will only be called when matchedQueries changes. It has several advantages:

Work with stateless component naturally while maintaining compatibility with future ResizeObserver API.
Component composing code stays closer to view code. Easier to reason about.

I plan to work on this new API this week if there is new ideas.

An example that uses the API can be found here https://github.com/d6u/react-container-query/blob/e1427ee433bedd2a31dd8b4b8961ae4ab9449335/examples/toggle-sizes/index.js#L40-L48 and there https://github.com/d6u/react-container-query/blob/e1427ee433bedd2a31dd8b4b8961ae4ab9449335/examples/simple/index.js#L16-L20.

Progress please see https://github.com/d6u/react-container-query/pull/40

This is implemented in 0.7 release.

Not sure I understand why there's a need of another element (
) here. Can you please re-explain?
If it's because of the ref, and you need a DOM reference of the element, why not using ReactDOM.findDOMNode(this), and just rendering this.props.children? Calling that from a parent component will get the dom element of the first component that actually renders DOM.
Maybe I got it wrong though.

Thanks @elado! ReactDOM.findDOMNode(this) is exactly what I needed. I hope I know this before. Does that mean this way, I don't have to wrap children with one additional
 parent? Is there any caveat to pay attention to?

Yay. Correct, you need something like:
class ContainerQueryWrapper extends Component {
  componentDidMount() {
    // ...
    // this will get the first DOM element from whatever the children renders.
    const domEl = ReactDOM.findDOMNode(this)
    this.rol.observe(domEl)
  }

  render() {
    return React.Children.only(this.props.children)
  }
}

And you can still keep the HOC/decorator with that.

Hi @elado, thanks for your advise, now 0.8 supports both HOC and Component style API without wrapping element thanks to ReactDOM.findDOMNode(this).


remove redundant log

Signed-off-by: hejianpeng<EMAIL_ADDRESS>What type of PR is this?

/kind bug
What this PR does / why we need it:
remove redundant log which should be replaced by running kurator tool list
Which issue(s) this PR fixes:
Fixes #55
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

/lgtm


Feature Request: Fail test when expect fails in jasmine

Hi There,
Can we fail a test when an expect failes. I have tired with fail() , but having this error ReferenceError: fail is not defined..
Thanks  in advance,
Sri Harsha

+1.
I'm using jasmine 2


Helper for Meteor-CollectionFS

https://github.com/CollectionFS/Meteor-CollectionFS
Uses something like this :
meteor.getUrl()+'/cfs/files/imagesFiles/'+imageFile.$value._id+'?token='+btoa(JSON.stringify({authToken: token}))

Wow, you are really on this. You rock man!
I think this can be the future Native development based on Meteor + RN

Available in 1.0.0-beta15.


Verify compatibility of new Metrics implementation with FT

Latest Metrics specification has introduced a number of backward incompatible changes, including new semantics for counters and some name changes (in particular in the OpenMetrics/Prometheus format).
Since FT depends on Metrics, we need to verify compatibility by at least re-running, and possibly upgrading to, the latest TCKs.

Made part of PR #992.


Avoid duplicate liblib prefix for verilator target.

The default uses of cc_common.create_cc_toolchain_config_info.artifact_name_patterns on unix systems prefix static libs with lib. So by naming a cc_library lib* the output file will get a liblib prefix. E.g.
bazel-out/k8-fastbuild/bin/external/verilator/liblibverilator.a

This change fixes this issue but adds an alias for backward compatibility.

/gcbrun


[$250] Add the modal prompts back to locked toggles in More features

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

Version Number: v9.0.24-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @trjExpensify
Slack conversation: https://expensify.slack.com/archives/C02NK2DQWUX/p1723775264453099?thread_ts=1723724132.692329&cid=C02NK2DQWUX
Action Performed:

Enable QBO
Go to More features
Click on a locked toggle (i.e Tags)

Expected Result:
A modal should appear explaining why it's locked.
Actual Result:
No modal appears, the cursor just turns to 🚫 on hover.
Workaround:
N/A
Platforms:

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

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0126e97d02490a2603
Upwork Job ID:<PHONE_NUMBER>673239304
Last Price Increase: 2024-08-23

Issue OwnerCurrent Issue Owner: @
Issue OwnerCurrent Issue Owner: @thesahindia

PR is here: https://github.com/Expensify/App/pull/47533

PR is here: https://github.com/Expensify/App/pull/47533

Can you add external label so we can get a C+ here? 😄

Thanks for the issue @trjExpensify 🙇

Proposal
Please re-state the problem that we are trying to solve in this issue.
Modal doesn't appear when clicking on the locked toggle switch.
What is the root cause of that problem?
The disabledAction callback prop for the items is missing. For example here:
https://github.com/Expensify/App/blob/f5e63543cbdea0ffcc0e5bf201f8f2e9645582c6/src/pages/workspace/WorkspaceMoreFeaturesPage.tsx#L195-L213
What changes do you think we should make in order to solve the problem?
Adding an appropriate disabledAction callback function to the list will solve this problem. For example:
{
            icon: Illustrations.FolderOpen,
            titleTranslationKey: 'workspace.moreFeatures.categories.title',
            subtitleTranslationKey: 'workspace.moreFeatures.categories.subtitle',
            isActive: policy?.areCategoriesEnabled ?? false,
            disabled: hasAccountingConnection,
            pendingAction: policy?.pendingFields?.areCategoriesEnabled,
            action: (isEnabled: boolean) => {
                if (!policyID) {
                    return;
                }
                if (hasAccountingConnection) {
                    setIsOrganizeWarningModalOpen(true);
                    return;
                }
                Category.enablePolicyCategories(policyID, isEnabled);
            },
            disabledAction: () => {
                setIsOrganizeWarningModalOpen(true);
            },
        }

Result

What alternative solutions did you explore? (Optional)

Updated labels, payout will be due on Sept 7th after hitting prod 3 days ago.

Waiting out the regression period, Melv.

Payment is due tomorrow, so doing this today.

$250 to @hungvu193 for the fix
$250 to @thesahindia for the C+ review

Feel free to request, closing!

$250 approved for @hungvu193

$250 approved for @thesahindia


No specularity from any type of light, material roughness seems like 100% regardless of parameter or texture map

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Godot v3 0fb221b
Tested on Windows 7 Radeon 5500, Windows 7 Gtx 970M and Linux Mint Gtx 970M
Issue description:
Doing a quick visdev and my model was showing great prior to a pull on October 6th 2017. But afterwards it's as if the roughness just scaled up to 100% all the time. Values from parameter and texture channels have little effect.
Changing the roughness only affects reflectivity of environment on the object, but not how the light bounces off the material.
My apologies, I do not know in what specific commit this began to happen.

I am noticing this with other models.
Possibly related to issue : https://github.com/godotengine/godot/issues/10250
Steps to reproduce:
I did an empty scene with a meshinstance->sphere
Material with Roughness parameter at 0
Happens to all lights OmniLight, Directional and spot.
Link to minimal example project:
specularityroughness.zip <- Basic project
The Human character project size is quite big if .imports are included, if you would like the zip just respond here~
SpecularityIssue.zip
Thank you! :)

CC @tagcup @hi-ogawa who did changes around that time + @reduz.

I think I found the reason.
This changes https://github.com/godotengine/godot/pull/11700/files assumed conditional SPECULAR_SCHLICK_GGX will be turned on when render_mode is specular_schlick_ggx, but it seems this conditional SPECULAR_SCHLICK_GGX wasn't even existed before that.
So, at this point now, render_mode specular_schlick_ggx does nothing in shader.
I made up PR for fix. Please take a look at it.

Ah yeah it was in a else statement and just rearranged stuff at last minute. Thanks fir spotting this!

@akien-mga @hi-ogawa
This is maked as fixed, but is still happening on the directional light as of commit 4fb5b1a21186bfc96aaa724e8e19afb44332dd8b

Can also confirm this issue for the directional lights


[SECOAUTH-434] Performance of retrieving access token

Priority: Minor
Original Assignee: Dave Syer
Reporter: Matt Veitas
Created At: Sun, 22 Dec 2013 04:43:52 +0000
Last Updated on Jira: Mon, 30 Dec 2013 10:49:15 +0000

I am using the JdbcClientDetailsService and I have noticed that for every request to retrieve an access token there are about 8 calls to the database (loadClientByClientId).
Wanted to get feedback about possible solutions for this.
One thing that came to mind was to use a ThreadLocal to store the ClientDetails and follow a pattern similar to the RequestContextHolder and have some static methods to get access to the ThreadLocal.
Comments:
david_syer on Mon, 30 Dec 2013 10:49:15 +0000

This seems like it can be solved with a pure cacheing approach (and as such is cross cutting), and should not impact the implementation of the existing business methods. I'm not averse to adding code to the samples to show how this might work - it should be trivial to declare the ClientDetailsService to be cacheable.

we added something like this and the client details queries are gone. is this what you meant dave ?
<cache:advice id="cacheClientsAdvice" cache-manager="cacheManager">
<cache:caching cache="OauthClientDetailsServiceCache">
<cache:cacheable method="loadClientByClientId" key="#clientId"/>
</cache:caching>
</cache:advice>
<cache:advice id="cacheEvictClient" cache-manager="cacheManager">
<cache:caching cache="OauthClientDetailsServiceCache">
        <cache:cache-evict method="removeClientDetails" key="#clientId" before-invocation="false"/>
    </cache:caching>
</cache:advice>

<!-- apply the cacheable behavior to all ClientDetailsService interface methods -->
<aop:config>
    <aop:advisor advice-ref="cacheClientsAdvice" pointcut="execution(* org.springframework.security.oauth2.provider.ClientDetailsService.*(..))"/>
</aop:config>
<aop:config>
    <aop:advisor advice-ref="cacheEvictClient" pointcut="execution(* org.springframework.security.oauth2.provider.ClientRegistrationService.*(..))"/>
</aop:config>

Yes, that's what I mean. I would also recommend using a cache with a configurable TTL.

Caching the clientDetails would greatly improve overall performance. We have implemented our own OAuth server using Spring-security-oauth, and during our load tests, we noticed that for a complete flow in our use case (so user provides credentials, gives his approval, client requests a token using an authorization code)  almost 30% of that time was spent retrieving the client details! It is a very simple query but if your database is on a remote system, the latency adds up.
It seems obvious to cache the clientDetails by default, since they generally won't change often.

Was this implemented? I'm trying to add caching as @shlomicthailand  suggested, but that doesn't solve the issue. It's still the same (one roundtrip about 100 milliseconds what is in case with inmemory 10x slower). If there is some example can someone provide it? Thanks

@dsyer any suggestions? thanks

i realized this too, loadclientdetails is called 6 times from different classes for one client details lookup( implemented mongodb client-details service class, and in memory user details at the moment) , would be really helpful if this is done at framework.

This ticket deserves higher priority as it can be fully fixed only on the framework level (invoke the loader once, then pass the instance to whoever needs it). The caching to be implemented here is too specific, to delegate it to the general cache mechanism. Thread-local cache will be a serious security bug (because of thread pooling in a servlet container), generic cache is tricky to invalidate properly on any change in credentials or permissions, cache based on request-scoped bean does not work (there's simply no request scope defined yet by the moment of the first invocation).


[KOGITO-5490] Escaping pmml path in generated PredictionModels

@danielezonca @kostola  @jiripetrlik
See https://issues.redhat.com/browse/KOGITO-5490

The (build) Runtimes check has failed. Please check the logs.

The (build) Runtimes check has failed. Please check the logs.

The (build) Apps check has failed. Please check the logs.

The (build) Examples check has failed. Please check the logs.

The (build) Apps check has failed. Please check the logs.

The (build) Runtimes check has failed. Please check the logs.


Error saving file ReferenceError: Buffer is not defined

:Error saving file ReferenceError: Buffer is not defined
Description
clone the code and run , try save project to nsp ( from menu [File]->[Save]) , but  failed , report error :"Error saving file ReferenceError: Buffer is not defined "
another issue while running first, console out warning / error : 'WebGL: INVALID_VALUE: viewport: size < 0'
not sure if this warning  make the 'save' feature failed,
Version

0.98.38,

Platform

Mac

"Error saving file ReferenceError: Buffer is not defined "
I noticed that happening in the web version too

A similar problem in the web version, both when saving and publishing :/

Hello
This issue is now fixed on web version.
Thanks a lot for reporting this!


EL7 RPMs for mod_auth and cjose

the releases tab has the rpms for el6 but not for el7. Could you please include the el7 packages as well?

Please let me know when EL7 RPMs will be available for cjose and the module.

the releases do contain el7 rpms


In-project link resolution

Links in Markdown code pointing to in-project modules/headers/exports should be automatically resolved. For example, if we have a link:
[Text](complex/Complex.h#Complex)

then the URL  part should automatically be substituted with the path to the page rendered from the Complex export. Thus, something like this:
[Text](complex/Complex.h/Complex.html)

Deprecating this issue in favor of the new namespaced links feature.


Repo name case sensitivity is inconsistent

We had a problem today which was caused by someone being given a repo name to clone which was the right name but with a casing error in it.
This triggered issued #441 which has now been fixed, but this led me to look into the subject of casing, because even with the fix for #441, you get an authorisation error if you mis-case a repo name, not a 'Not found' error, which IMO would point more quickly to the cause.
It seems that Bonobo is 'a bit case-sensitive' on repo names, but not very.
For example EFRepositoryPermissionService.HasPermission needs a repo to have the right case, because otherwise it can't find it.   However, GitController does something like this:
        var directory = GetDirectoryInfo(project);
        if (Repository.IsValid(directory.FullName))

and this is not case-sensitive - so where this test is performed, it doesn't return a Not Found error, but the auth layer will probably fail you anyway.
My personal feeling is that repo names in URLs should be case-sensitive, if for no other reason than that they are are GitHub, but either way, the whole app should be consistent.  (Update: I'm not actually sure they are case-sensitive at GitHub now - I've read both things now, and perhaps they shouldn't be case-sensitive because Windows isn't, and after all we're only looking at file/directory names really)
I'm happy to produce a PR to fix this in either direction, depending on what people ( @jakubgarfield, @RedX2501  ?)  think is best.

Imho we should be consistently case sensitive on repo names and case insensitive on username and team names.

I'm a bit vague about the management of this project - is there any more opinion/voting needed to decide to do this?

What's the advantage of being case sensitive on Repo names? I Would expect RepoName and reponame to refer to the same repository - As the name is the same? Or said differently would anyone add two repositories where the only difference was one to several cased letters? I don't think so?

The underlying repo storage system (windows) is case insensitive.
Unless you want to support .net core and linux, bonobo should be case insensitive.

@larshg Indeed - one could not add two repositories which differed only in case, at least to the filesystem.  (You might be able to add them to Bonobo's database, I haven't checked.
Is there a consensus for case-insensitive emerging?

@willdean there is not really any voting, if some piece of work will help the project it is gonna get merged :)
usually I try to be consistent with github, but @latop2604 provided good reasons for the repo names being case insensitive and the reason to usernames and being insensitive is smoother AD integration @RedX2501?

OK, I'll do a PR for case-insensitive repo names, and then we can go from there.


i18n(es): Updated middleware.mdx

What kind of changes does this PR include?

Minor content fixes (broken links, typos, etc.)

Description

What does this PR change? Give us a brief description.
Updated middleware.mdx based on this commit 22c58ac6580b01cfaa3005af0f494ced9f2fde8a

Thanks for your review @sasoria 😁


Add Char#+(Char) and fix <=>'s behaviour

This adds the concatenation of two chars and fixes the behaviour of the <=> operator of Char.
I feel like Char#+(Char) was missing and right now in my code I have '"' + char + '"' which I would like to use instead of %("#{char}") or even worse "\"#{char}\"". I personally find the first one the easiest to read.
Also, the first one is much faster:
              +  11.55M ( 86.58ns) (±25.21%)   48 B/op        fastest
percent literal   6.18M ( 161.7ns) (±38.42%)  208 B/op   1.87× slower
       escaping   7.45M (134.16ns) (±33.66%)  208 B/op   1.55× slower

The behaviour of the spaceship operator is currently wrong. It's just an alias of Char#-.
Right now 'a' <=> 'c' returns -2 instead of -1.
This also adds some missing specs.

Seconded. I think we can relax the documentation of <=>.
Regarding Char + String. I would vote for no. But then we actually have String + Char. I would actually remove that as well to favor more the usage of the StringBuilder in that case. If there is a single char the user can do string + char.to_s, but allowing string + char can lead to bad performant code IMO.
Closing to move on that directions. But, thanks @r00ster91 to bring the discussion.


ERROR: Mongoose

Describe the bug
ERROR: Mongoose: Failed to connect to database: bad auth : Authentication failed.
To Reproduce
INFO: Validating config file and environment variables
[2024-01-13 11:01:02] WARN: env: WEATHERSTACK_KEY is missing. Weather command won't work
[2024-01-13 11:01:02] WARN: env: STRANGE_API_KEY is missing. Image commands won't work
[2024-01-13 11:01:02] INFO: Loading commands...
[2024-01-13 11:01:05] INFO: Loaded 54 commands
[2024-01-13 11:01:05] INFO: Loaded 45 slash commands
[2024-01-13 11:01:05] INFO: Loading contexts...
[2024-01-13 11:01:05] INFO: Loaded 0 USER contexts
[2024-01-13 11:01:05] INFO: Loaded 0 MESSAGE contexts
[2024-01-13 11:01:05] INFO: Loading events...
[2024-01-13 11:01:05] INFO: Loaded 15 events. Success (15) Failed (0)
[2024-01-13 11:01:06] INFO: VersionCheck: Your discord bot is up to date
[2024-01-13 11:01:06] INFO: Launching dashboard
[2024-01-13 11:01:07] INFO: Connecting to MongoDb...
[2024-01-13 11:01:08] ERROR: Mongoose: Failed to connect to database: bad auth : Authentication failed.
Expected behavior
I've tried to put it in different ways and it still gives me this error, please help me
Screenshots

If possible

NodeJS version: v 20.11.0
Bot version: discord v14

Additional context
Add any other context about the problem here.

URL must contain your username and password (no special characters)
Check if ip is whitelisted or not on mongo
This is for bugs, not your personal errors, for that join our support server

#366

A URL deve conter seu nome de usuário e senha (sem caracteres especiais)
Verifique se o ip está na lista de permissões ou não no mongo
Isto é para bugs, não para seus erros pessoais, para isso entre no servidor de suporte Bad auth mongodb mas o passe já está correto  #366

But this part was checked, the ip is [<IP_ADDRESS>] and the directory was searched as follows
mongodb+srv://:@bot.e7jno0j.mongodb.net/?retryWrites=true&w=majority

Hi, try this :
mongodb+srv://:@bot.e7jno0j.mongodb.net/?retryWrites=true&w=majority
but without the "<>"
Like that : mongodb+srv://username:password@bot.e7jno0j.mongodb.net/?retryWrites=true&w=majority


ReGaHss: Easier use of other scripting languages within HM programs

With the help of system.Exec it is already possible today to start any scripts (Tcl, Shell, ...) that are located in the file system. But this is very time-consuming or error-prone. Either the script files have to be modified on the console or via SCP, or they are created in advance via additional system.exec's with heavy use escaping.
An optimal solution would be to enter any other script languages into the HM script input window that were previously installed via add-ons or already exist on RaspberryMatic (e.g. Tcl, Shell, node.js). With the following solution, it should be possible to integrate it into the RegaHss with relatively little effort:
Scripts for external interpreters should have a magic first line, similar to the shebang line in shell scripts (#!). If this line is missing the content is considered as standard HM script. This line also contains the path to the executable of the corresponding interpreter (with possible options). The entire script content (after replacing $this$ and $src$) should be passed to the script interpreter via stdin or file. The script interpreter should be started asynchronously and the RegaHss does not wait for return values. The script should read and write values via the web API of the RegaHss. Therefore there is no need to create an interface that allows access to the object model. The RegaHss should write an error to syslog if the interpreter could not be started.
This approach offers further advantages:

The script is part of the project.
The script can be entered without escaping using the existing editor window.
There is no need for the user to create or edit files.

Actually it's an extended system.Exec.

This sounds like a good idea. However, as this might be a major change I woulf have to see how smooth this can be implemented. Ergo it might take quite some time to actually get this in ReGa implemented.


Other Network claim button links look incorrect

Describe the bug
It looks like the 'Incentives on other network' buttons link to '/claims' instead of '/claim'.
To Reproduce
Steps to reproduce the behavior:

Go to Claims page
Click on one of the claim buttons under the 'Incentives on other Networks' section.

Expected behavior
For each network to link to the '/claim' route.
Screenshots

Fixed on branch


Feature: Display movement speed

I used my fitness app today after a long time and noticed that I can view how fast I was on a certain position. I could imagine this as a cool feature for Hauk. Below you'll find a screenshot of the map where green is slow and orange/red is fast.
You can already see in the browser how fast you're currently moving, so with this data you can already paint these colors. All this can happen inside the browser.

Hi! Thanks for the suggestion and sorry for taking so long to reply to this. Your suggestion is interesting and it could possibly be a viable feature for the project, most likely an optional feature, that is easy to enable via the config.
The only problem with doing this for real-time data is that the minimum and maximum speed is not predictable. When loading the page, the user might be going at e.g. 30 km/h, but there'd be no way to know if the user then intends to accelerate to e.g. 50 km/h at its highest. If we decide to make 50 km/h the maximum speed on the color scale, what if the user decides to enter the freeway and go 120 km/h? Then the scale would be wrong. It would be possible to set a defined maximum of e.g. 120 km/h and any faster than that will just always be the "fastest" color, but if the user is just walking instead of driving, they'll never be going faster than maybe 10 km/h, and most of the scale would be unused.
I'm not entirely sure how to best implement this. There could be a scale in the config where colors for specific speeds are defined from walking speed up through highway driving speeds, e.g.
$trail_colors = [
    0 => '#color1',
    4 => '#color2',
    10 => '#color3',
    30 => '#color4',
    70 => '#color5',
    120 => '#color6
];

And the chosen color would be determined based on these on a scale, e.g. 25 km/h would be 75% color4 and 25% color3. An alternative would be to have some kind of sliding window that sets the minimum and maximum speed to whatever is the lowest and highest recorded speed currently displayed on the map. E.g. if the user has always moved between 3 and 8 km/h on the current view, 3 km/h would be 0% and 8 km/h would be 100% until either of those values change. However, that would make the colors of the existing trail change as the user's speed exceeds the current maximum or falls below the current minimum, and would also be more complex to implement.
I'm open to suggestions.

I personally would prefer the second option by calculating this in real-time and to dynamically change the scale.
But a more simpler way would be that the user can choose what he's doing. If the user for example chooses the "Walking mode", the highest value would be 10 km/h. But if the user now begins to drive, he could either change the current mode manually or the server detects that the user exceeded the maximum speed for to long and will change the mode to the next higher one, like "Driving mode".
Like you said, it's hard to determine what the user is currently doing by just looking at the speed. I think we should not over-complicate stuff. It would be even enough to let the user choose what he's doing without all that automatic mode switching. My fitness app for example ask me if I'm walking or running and it will change the color scale (I think that at least).

That's a good point. I'm not really that keen on having the user manually input it, as it will probably be an optional feature; whether it's enabled or disabled by default, there will be people who don't use speed-scaled colors, and they won't really have use of an in-app mode selector. But auto-detecting it on the server would mean that it just works seamlessly to the user. I like your idea about pre-defined modes that work for given speed ranges - maybe the exact ranges could also be configurable. E.g. walking is 0-10 km/h, biking at 0-50 km/h, driving at 0-120 km/h, high speed trains or plane travel or whatever at  higher speeds than that as well. I'll have to think about it and try to figure out advantages and disadvantages to each approach.
Then there's also the issue of multiple people on the map doing multiple modes of transportation. If one person is walking and one is driving on the highway, I'd need some way to maintain resolution in movement speed for both.

That's actually a very good idea and I have installed Hauk on my personal server and my friends and family are impressed and enjoy the application ! I already got two feedback for the speed. What about starting to add the speed on the top right to see the realtime km/h speed first ? I like the idea of colorizing the path as well but starting with a simple number to see the speed would probably give people the wish to provide feedback on what would be the best for mechanism to colorize. (my personal opinion would be to be able to select the range for 3 colors which is the good balance of details & simplicity)

I've started evaluating hauk if I can make some good use of it. For me calculating current speed and displaying it as a text would work well e.g. 3 km/h when walking.

Hey @pvagner, sorry for answering this 25 days later but this feature is already built-in into Hauk. But maybe you already noticed that D:

Yes, thank you, I have noticed it it's great for my needs.


Question about total_ppo_epoch

Hi! I encountered a question about this line of code in ppo trainer:
self.total_ppo_epochs = int(np.ceil(self.steps / self.batch_size))
Can someone explain to me what does steps / batch_size mean and how is it the total_ppo_epoch? The rationale behind this line is not very clear to me. Thanks!

As far as I understand it, it's just a constant for you to use in your training loop so that the total number of gradient updates of your model is equal to steps.
So if you have
for epoch in range(ppo_trainer.total_ppo_epochs):
   for batch in ppo_trainer.dataloader:
      # compute loss and do optimizer step

then this code will run steps many optimizer steps.

If I understand correctly, to make the total number of update steps equal to steps, total_ppo_epochs should be int(np.ceil(self.steps / len(self.dataloader))) instead of dividing by the batch_size. In your code snippet, there are total_ppo_epoch * len(ppo_trainer.dataloader) update steps. Am I right?

Also, I was looking at stack_llama/scripts/rl_training.py. They wrote here.
for epoch, batch in tqdm(enumerate(ppo_trainer.dataloader)):
    if epoch >= config.total_ppo_epochs:
        break

It only runs through the dataset once and calls each PPO step on a batch an epoch. I'm a bit confused here. Could someone also clarify this?

No worries, it's indeed confusing.
@younesbelkada @lvwerra Hi both! Any good advice? Thank you for your time.

Sounds good. Just fyi, I currently use in the outer loop for epoch in range(n_epochs): where n_epochs = args.steps / len(ppo_trainer.dataloader) as a workaround.


cmake: fix OPAEGit.cmake

https://github.com/OPAE/opae-sdk/pull/2051 errantly changed this
opae-libs file in the opae-sdk tree. Bringing the two back into
sync..
The change in 2051 broke our application --version output. This
change corrects it.
Signed-off-by: Tim Whisonant<EMAIL_ADDRESS>

Coverage remained the same at 62.251% when pulling 4acf193dffe86ff417ce6fde55db2ce54149caa4 on tswhison/git_cmake into 162a53e5c05775cc860df781256f467d70773ffa on master.


Support bf16 weight and dynamic shape

Fix crash when weight is not support by numpy(e.g. BFloat16)
Fix crash when the shape is dynamic(not an integer)

Hi @jinjingforever, could you take a look when have time? Thank you!

Thanks @yuanzhedong! I will let @yijie-yang take a look.

Thank you, lgtm!

Thank you! Just fixed the format using
pyink --pyink-use-majority-quotes --pyink-indentation=2 --preview --unstable --line-length 80 --extend-exclude .downloads --extend-exclude "\.pyi" ./


Cut a 1.0.0 release

Things have been working greatly for me over the past 2 years and it is about time to squash some known bugs and lock down the current feature set in prep for a potential big revamp on the project.
This is what I have in mind right now that will make up for 1.0.0:

[ ] Attempt to trim down the image size again
[x] Update to latest heroku/cedar
[ ] Buildpacks updates

[x] Remove bats (only used once in almost 2 years)
[x] Remove phantomjs (used once or twice in almost 2 years)
[x] Update golang and fix GH-94
[x] Update nodejs
[x] Update php
[ ] Fix GH-106
[x] Update python
[x] Update ruby and attempt to implement GH-100
[x] Fix inline's GH-97 / GH-99

[ ] Cover current functionality with some sort of feature tests

If you happen to come across this issue, please LMK if you want something to be kept around or have any suggestions for this release :beers:

I'd like these please :>
https://docs.tsuru.io/stable/using/buildpacks.html

@devekko unfortunately that's not possible given current's devstep state and based on what I've seen so far from official heroku buildpacks. They are very focused on production web apps and I use devstep for CLI tools for example. Some of devstep's (if not most of them) are based on official Heroku buildpacks with some tweaks made by me.
This is something I'd like to improve after 1.0 is out


Improve functionRunner interface

Update items from []*KubeObject to KubeObjects. This aligns the resourceList.items with the latest change in https://github.com/GoogleContainerTools/kpt-functions-sdk/commit/78748f08d9977e08b5cf2dd678d04ed22d209a75#diff-7b055320c11f07c233d6db3fec422e3059d52a0f295a12e02b489aa4a539a50d
functionRunner can parse functionConfig value for ConfigMap.Data and custom functionConfig in two formats

For example, a SetLabel struct that implements Runner interface, have its Owner and Org value automatically parsed from  either 1. SetLabels type with owner and org fields or 2. ConfigMap type with Data fields having keys owner and org
type SetLabels struct {
	Owner string
	Org   string
}

FunctionConfig 1
apiVersion:  fn.kpt.dev/v1alpha1
kind: SetLabel
metadata:
   name: custom
owner: kpt
org: google

FunctionConfig 2
apiVersion: v1
kind: ConfigMap
metadata:
   name: custom
data:
  owner: kpt
  org: google

cc @zyy98


2-7 Medication Card document PDF representation is not correct (Oliver Egger, ahdis)

ch.fhir.ig.ch-emed#0.2.0 /Bundle-2-7-MedicationCard.html
The PDF representation for the FHIR (and also CDA) example contains a dosis per unit of 2.5mg but this is in the source 50 mg / unit for Beloc Zok Metoprolol.
See https://ehealthsuisse.ihe-europe.net/gazelle/testing/test/test/TestInstance.seam?id=39290&cid=354975
PDF can be extracted with https://base64.guru/converter/decode/pdf
Oliver Egger, ahdis

to fix, see also #17 for other correction on PDF example

Fixed in FHIR: see commit on CH EMED
Fixed in CDA: TBD


Inline SQL wrong in SqlAggregateStatsService.cs

The SQL WHERE statement at :
"AND p.PackageDeleteKey IS NULL"
is incorrect. It should be :
"AND p.PackageDelete_Key IS NULL"

Fixed in dev branch.


Tidying

Minor changes to directory structure.

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


Allow requests without Content-Length

Seems unnecessarily strict to me even as a SHOULD. Right now clients working with growing resources would have to wait a batch up a block of data before sending it. Would be nice to support streaming.
What is the rationale of the following?

If a PATCH request does not include a Content-Length header containing an integer value larger than 0, the server SHOULD return a 400 Bad Request status.

You added this statement to the protocol in #104 (see https://github.com/tus/tus-resumable-upload-protocol/pull/131/files#diff-7327496955357b9792bab8720dc507d6R222), so you could probably explain it the best way. In hindsight, I agree that it seems dubious and we should reconsider it.

Hm, I will have to send a message back in time to myself. I'll add a new PR removing it.


.Net: Bug: AssistantChatMessage has no Participant Property

Describe the bug
Kernel.InvokePromptStreamingAsync throws System.MissingMethodException
--> 'Method not found: 'Void OpenAI.Chat.AssistantChatMessage.set_ParticipantName(System.String)'.'
Platform

OS: Windows
IDE: Visual Studio
Language: C#
Source: NuGet Package Microsoft.SemanticKernel 1.18.0-rc

Additional context
Checking the Source Code on Branch dotnet-1.18.0 reveals the following code:
[new AssistantChatMessage(message.Content) { ParticipantName = message.AuthorName }];

switching form .net 6 to .net 8 resolves the issue

@RogerBarreto, could you validate that there's nothing lurking here that we should be concerned about?

@matthewbolanos @method-marco
Please confirm which version of OpenAI package are you using in the solution?
As far as I understand we support only OpenAI-beta.5 any attempt on using a different OpenAI package version may cause conflicts.
This might be related with their recent changes from init to set in this property.
Please provide more context on the versions used and code for reproduction.

@method-marco Small correction from my part the error is expected when you run the .Net 6
We do not support .net 6 for Semantic Kernel since our 1.0 release.


ModuleNotFoundError: No module named 'lm_eval.api'

Hi,
I tried pip install lm_eval, but that package has no api module?
Which lm_eval should I install then?
MODEL_NAME=mamba-370m
MODEL_PATH=/media/hangyu5/Home/Documents/Hugging-Face/$MODEL_NAME
python evals/lm_harness_eval.py \
    --model mamba \
    --model_args pretrained=$MODEL_PATH \
    --tasks lambada_openai,hellaswag,piqa,arc_easy,arc_challenge,winogrande \
    --device cuda \
    --batch_size 64 \
    tee lm_eval.txt

Traceback (most recent call last):
  File "/home/hangyu5/Documents/Git-repoMy/AIResearchVault/repo/Architecture/mamba/evals/lm_harness_eval.py", line 8, in <module>
    from lm_eval.api.model import LM
ModuleNotFoundError: No module named 'lm_eval.api'

Thanks!

Oh, I missed the submodule in 3rdParty

Could you refine README a little bit, I did not find there was a submodule there
git submodule update --init && \
cd 3rdparty/lm-evaluation-harness && \
pip install -r requirements.txt && pip install -e .

This was in the doc for Evaluation, did that not work?


Understanding --minConversionEfficiency argument

Hi,
I wanna keep only cytosines with a methylation coefficient greater than 0.5. Can you confirm that --minConversonEfficiency makes me able to to that ? If I understand well, by fixing --minConversionEfficiency at 0.5, MethylDackel will only keep cytosines in CHH or CHG contexts wich are converted in at least 50% of alignments. Am I wright ?
Thanks for your help !
Thomas.

If you want to keep cytosines that are over 50% methylated then you want to filter the output rather than use --minConversionEfficiency. The --minConversionEfficiency argument is meant to filter individual alignments according to how well the bisulfite conversion worked on them, which is assessed by there being few if any methylated Cs in a CHH or CHG context.


MariaDB 10.3 support

Can you add MariaDB 10.3 support?

https://github.com/lj2007331/lnmp/commit/d5db654047c3852c4cd215ec7ad770c662c8ecd7


Switching to Active Directory Member Service HttpException - Value cannot be null error.

Hi fellow Bonobo Git Server users,
I am trying to get up to the latest version of Bonobo Git Server, unfortunately I'm having some issues doing that.
I have gone through the setup guide/documentation several times and cannot see that I'm doing anything wrong during setup.
When I browse the website I get a runtime error in the browser and the windows event log on the server running Bonobo Git gives me this:

Event code: 3005
Event message: An unhandled exception has occurred.
Event time: 27-01-2016 18:26:36
Event time (UTC): 27-01-2016 17:26:36
Event ID: 2cfdc7d0b3d84a458c980cc6bd01fa9f
Event sequence: 2
Event occurrence: 1
Event detail code: 0
Application information:
Application domain: /LM/W3SVC/1/ROOT/Bonobo.Git.Server-2-130983891948993281
Trust level: Full
Application Virtual Path: /Bonobo.Git.Server
Application Path: C:\inetpub\wwwroot\Bonobo.Git.Server\
Machine name: REVISIONCONTROL
Process information:
Process ID: 4272
Process name: w3wp.exe
Account name: 'DOMAIN\USERNAME'
Exception information:
Exception type: HttpException
Exception message: Value cannot be null.
Parameter name: virtualPath
at System.Web.HttpApplicationFactory.EnsureAppStartCalledForIntegratedMode(HttpContext context, HttpApplication app)
at System.Web.HttpApplication.RegisterEventSubscriptionsWithIIS(IntPtr appContext, HttpContext context, MethodInfo[] handlers)
at System.Web.HttpApplication.InitSpecial(HttpApplicationState state, MethodInfo[] handlers, IntPtr appContext, HttpContext context)
at System.Web.HttpApplicationFactory.GetSpecialApplicationInstance(IntPtr appContext, HttpContext context)
at System.Web.Hosting.PipelineRuntime.InitializeApplication(IntPtr appContext)
Value cannot be null.
Parameter name: virtualPath
at System.Web.VirtualPath.Create(String virtualPath, VirtualPathOptions options)
at Bonobo.Git.Server.Data.ADBackend.<>c.<.ctor>b__18_0()
at System.Lazy1.CreateValue()    at System.Lazy1.LazyInitValue()
at Bonobo.Git.Server.Data.ADRepositoryRepository.GetRepository(String name)
at Bonobo.Git.Server.Data.Update.RepositorySynchronizer.CheckForNewRepositories()
Request information:
Request URL: https://localhost:443/bonobo.git.server/Home/Error?aspxerrorpath=/Bonobo.Git.Server
Request path: /bonobo.git.server/Home/Error
User host address: ::1
User:
Is authenticated: False
Authentication Type:
Thread account name: 'DOMAIN\USERNAME'
Thread information:
Thread ID: 5
Thread account name: 'DOMAIN\USERNAME'
Is impersonating: False
Stack trace:    at System.Web.HttpApplicationFactory.EnsureAppStartCalledForIntegratedMode(HttpContext context, HttpApplication app)
at System.Web.HttpApplication.RegisterEventSubscriptionsWithIIS(IntPtr appContext, HttpContext context, MethodInfo[] handlers)
at System.Web.HttpApplication.InitSpecial(HttpApplicationState state, MethodInfo[] handlers, IntPtr appContext, HttpContext context)
at System.Web.HttpApplicationFactory.GetSpecialApplicationInstance(IntPtr appContext, HttpContext context)
at System.Web.Hosting.PipelineRuntime.InitializeApplication(IntPtr appContext)
Custom event details:

I removed the username from the above event log data.
TROUBLESHOOTING:

I did change the app. pool to an AD user that has 'read' rights on the OU that contains the groups used to define access and team membership.
by viewing the SQLite db with SQLite browser I can't see that the usernames contains '' (this has been mentioned in other forum/issues posts as a culprit.
IIS has been restarted
everything else than App_Data was removed when I installed Bonobo Git server
tried the 'Network Service' acc. on the App. pool - made no diff.
IIS_Users still have 'Modify' rights on the App_data folder
tried adding the app. pool user (the AD user I want to use for this) to the ACL of the App_data folder - no diff.
Initially, before changing to 'AD Member service' I get a login page.
Looking in the web.config folder I saw that the ADBackend folder in the line "      " does not exist. Therefore I tried creating that. Made no diff. So this is properly handled by the code. Anyways, worth the try.
Made sure that the memberOf token included the AD groups used to give access to the Bonobo Git server website.....they where there.

I hope someone can point me in the direction of a solution. Looking forward to hear from you.
Thank you very much.
Regards larssb

In your config on line 26 (perhaps :) ) - I have this entry - what is yours?
<add key="ActiveDirectoryBackendPath" value="~\App_Data\ADBackend" />

Hi @larshg,
I got exactly the same thing.
'  '

But is it correctly understood that no actual folder named 'ADBackend' is necessary under App_Data?

Thank you

It should create an actual folder yes - and inside the ADBackend folder there is User, Repositories, Teams and Roles - and inside these, there should be some json files.

Aha okay - the folder is not created and therefore there is no data beneath
it either. What to do - is there any logs or the like that could help us
troubleshoot this?
/Larsen

Health is wealth, movement is medicine -

On 27. jan. 2016, at 22.57, Lars<EMAIL_ADDRESS>wrote:
It should create an actual folder yes - and inside the ADBackend folder
there is User, Repositories, Teams and Roles - and inside these, there
should be some json files.
—
Reply to this email directly or view it on GitHub
https://github.com/jakubgarfield/Bonobo-Git-Server/issues/433#issuecomment-175879542
.

Please try https://github.com/jakubgarfield/Bonobo-Git-Server/issues/432#issuecomment-175023026

Hi @RedX2501 The rights are correct. Just controlled it again. Made sure to follow the installation instructions. Also, I was already using v4 before so the rights was already there. Made sure to follow the migration steps.
Looking at the error I already have posted, to me it seems like, when Bonobo Git Server is set to use the 'AD member service' it cannot create the ADBackend folder and the items required inside it.
Here:
Value cannot be null.
Parameter name: virtualPath
at System.Web.VirtualPath.Create(String virtualPath, VirtualPathOptions options)
at Bonobo.Git.Server.Data.ADBackend.<>c.<.ctor>b__18_0()
at System.Lazy1.CreateValue()    at System.Lazy1.LazyInitValue()
at Bonobo.Git.Server.Data.ADRepositoryRepository.GetRepository(String name)
at Bonobo.Git.Server.Data.Update.RepositorySynchronizer.CheckForNewRepositories()
Thank you.

I have now also tried building it by downloading master and doing a publish from Visual Studio. The error is exactly the same.

Are you sure that you have write permissions to the folder - maybe it can't create the folders and therefore crash as the virtual path is null.
You can try create the folders manually?
If not - you properly have to try remote debugging on the IIS directly. Publish a debug version on the server and in VS attach a debug to a process where you enter the servername - you have to have admin rights on the server though, I think.

Hm.. Maybe it doesn't get that far to even try creating the folders... So creating them manually is properly not helping :(

Can you perhaps disable custom error page and provide are more full debug stack trace... I tried manually to set the backendpath to null and I got a similar error as you, but more detailed in the browser...

Hi @larshg,
I just checked again, the IIS_IUSRS group have [MODIFY] permissions. Manually creating the ADBackend dir. does not help. Hmm I'll do the remote debug if it is essential, as I have never tried that before :-).
This is the error after having disabled the custom error page.
Server Error in '/Bonobo.Git.Server' Application.
Value cannot be null.
Parameter name: virtualPath
Description: An unhandled exception occurred during the execution of the current web request. Please review the stack trace for more information about the error and where it originated in the code.
Exception Details: System.ArgumentNullException: Value cannot be null.
Parameter name: virtualPath
Source Error:
An unhandled exception was generated during the execution of the current web request. Information regarding the origin and location of the exception can be identified using the exception stack trace below.
Stack Trace:
[ArgumentNullException: Value cannot be null.
Parameter name: virtualPath]
System.Web.VirtualPath.Create(String virtualPath, VirtualPathOptions options) +12538514
Bonobo.Git.Server.Data.ADBackend.<.ctor>b__1() +113
System.Lazy1.CreateValue() +14337348    System.Lazy1.LazyInitValue() +524
Bonobo.Git.Server.Data.ADRepositoryRepository.GetRepository(String name) +32
Bonobo.Git.Server.Data.Update.RepositorySynchronizer.CheckForNewRepositories() +157
[HttpException (0x80004005): Value cannot be null.
Parameter name: virtualPath]
System.Web.HttpApplicationFactory.EnsureAppStartCalledForIntegratedMode(HttpContext context, HttpApplication app) +12601933
System.Web.HttpApplication.RegisterEventSubscriptionsWithIIS(IntPtr appContext, HttpContext context, MethodInfo[] handlers) +175
System.Web.HttpApplication.InitSpecial(HttpApplicationState state, MethodInfo[] handlers, IntPtr appContext, HttpContext context) +304
System.Web.HttpApplicationFactory.GetSpecialApplicationInstance(IntPtr appContext, HttpContext context) +404
System.Web.Hosting.PipelineRuntime.InitializeApplication(IntPtr appContext) +475
[HttpException (0x80004005): Value cannot be null.
Parameter name: virtualPath]
System.Web.HttpRuntime.FirstRequestInit(HttpContext context) +12618980
System.Web.HttpRuntime.EnsureFirstRequestInit(HttpContext context) +159
System.Web.HttpRuntime.ProcessRequestNotificationPrivate(IIS7WorkerRequest wr, HttpContext context) +12458597
Version Information: Microsoft .NET Framework Version:4.0.30319; ASP.NET Version:4.0.30319.34274
To me it seems like it is pretty much the same error as the one I've found in the eventlog on the Windows server.
Thank you.

Can you check the ActiveDirectoryBackendPath value in your web.config (in AppSettings block) ?

Hmmm yes I did that. Larshg already asked me that and I have also mentioned
it before in this issue.....however, it is ~\App_Data\ADBackend
/Larsen

Health is wealth, movement is medicine -

On 28. jan. 2016, at 23.41, Rémy de Sérésin<EMAIL_ADDRESS>wrote:
Can you check the ActiveDirectoryBackendPath value in your web.config (in
AppSettings block) ?
—
Reply to this email directly or view it on GitHub
https://github.com/jakubgarfield/Bonobo-Git-Server/issues/433#issuecomment-176460234
.

Arf sorry. My bad

It's okay :-)
/Larsen

Health is wealth, movement is medicine -

On 30. jan. 2016, at 00.52, Rémy de Sérésin<EMAIL_ADDRESS>wrote:
Arf sorry. My bad
—
Reply to this email directly or view it on GitHub
https://github.com/jakubgarfield/Bonobo-Git-Server/issues/433#issuecomment-177022779
.

Anyone with an idea on what could be the culprit in this case? Kind of anxious to get going with the new features and especially the AD integration. Yeah in general get back to a working Bonobo Git install (without rolling back is preferred ;-).

@larssb I did have a similar problem once, where the fields in in Repository were missing while discovering existing repos. The problem was here https://github.com/jakubgarfield/Bonobo-Git-Server/blob/master/Bonobo.Git.Server/Data/Update/RepositorySynchronizer.cs#L37 maye adding users, teams and admins might help.

Hi @RedX2501 - Thank you for responding :-) - and I would have to do that through sqlite? Or is there some other way?

@RedX2501
Went through the SQLite db with SQLiteBrowser and found a couple of teams and such that did not have values in some fields. The entries where either deleted or updated with valid data. Still the exact same error.
Any ideas?

You have to add them in the code.
The location that i posted.
teams = new string[0];

and so for users and admins.

Hi @RedX2501 - still get the same error.
For good measure the error is
Server Error in '/Bonobo.Git.Server' Application.
Value cannot be null.
Parameter name: virtualPath
Description: An unhandled exception occurred during the execution of the current web request. Please review the stack trace for more information about the error and where it originated in the code.
Exception Details: System.ArgumentNullException: Value cannot be null.
Parameter name: virtualPath
Source Error:
An unhandled exception was generated during the execution of the current web request. Information regarding the origin and location of the exception can be identified using the exception stack trace below.
Stack Trace:
[ArgumentNullException: Value cannot be null.
Parameter name: virtualPath]
System.Web.VirtualPath.Create(String virtualPath, VirtualPathOptions options) +12538514
Bonobo.Git.Server.Data.ADBackend.<.ctor>b__1() +113
System.Lazy1.CreateValue() +14337348    System.Lazy1.LazyInitValue() +524
Bonobo.Git.Server.Data.ADRepositoryRepository.GetRepository(String name) +32
Bonobo.Git.Server.Data.Update.RepositorySynchronizer.CheckForNewRepositories() +157
[HttpException (0x80004005): Value cannot be null.
Parameter name: virtualPath]
System.Web.HttpApplicationFactory.EnsureAppStartCalledForIntegratedMode(HttpContext context, HttpApplication app) +12601933
System.Web.HttpApplication.RegisterEventSubscriptionsWithIIS(IntPtr appContext, HttpContext context, MethodInfo[] handlers) +175
System.Web.HttpApplication.InitSpecial(HttpApplicationState state, MethodInfo[] handlers, IntPtr appContext, HttpContext context) +304
System.Web.HttpApplicationFactory.GetSpecialApplicationInstance(IntPtr appContext, HttpContext context) +404
System.Web.Hosting.PipelineRuntime.InitializeApplication(IntPtr appContext) +475
[HttpException (0x80004005): Value cannot be null.
Parameter name: virtualPath]
System.Web.HttpRuntime.FirstRequestInit(HttpContext context) +12618980
System.Web.HttpRuntime.EnsureFirstRequestInit(HttpContext context) +159
System.Web.HttpRuntime.ProcessRequestNotificationPrivate(IIS7WorkerRequest wr, HttpContext context) +12458597
Version Information: Microsoft .NET Framework Version:4.0.30319; ASP.NET Version:4.0.30319.34274
====================
Dang! :-(

@RedX2501 - it seems to have something to do with the repos. I moved them out of App_Data\Repositories. After that Bonobo seems to be starting and prompts for a user to login with. However, I cannot login. I have added the user to the Bonobo git admin AD group and the app. pool user has read rights to the OU where the group is located....
This is my web.config file, the AD service part.

Thank you.

You need to uncomment these tags. (remove ).
This is why the value is null at runtime

Hi @latop2604
Sorry for the confusion. There is two lines in the web.config file regarding the MembershipService. Default it is set to Internal as far as I remember. Just changed that line instead of removing the comment from the other line. In short.....the line is without
But thank you for replying.

Hmmm anyone else with an idea?

@larssb The only way I can repro this is to comment out the <add key="ActiveDirectoryBackendPath"...> line in web.config.
It would be good if you could post the actual appSettings part of webconfig that you're using at the time of the crash, because the earlier comments on this leave room for doubt, and the symptoms are exactly that of having the line <add key="ActiveDirectoryBackendPath" value="~\App_Data\ADBackend" /> commented-out.
Anyway, I've added some diagnostics to a branch on my fork - feel free to checkout https://github.com/willdean/Bonobo-Git-Server/tree/Bug433 and see if you get a line which looks a bit like this written into the error log:

iisexpress.exe Error: 0 : CONFIGURATION ERROR: The 'ActiveDirectoryBackendPath' element in web.config appSettings does not appear to be set

Ooooooh noooooo. I have a confession. Yesterday when I troubleshooted this I saw that I needed to remove the commenting from the appSettings part correctly. I had not done that. I can't say sorry enough for wasting your precious time.
Other comments:

I would recommend that the documentation mentions that the AppPool should also have 'Read' rights to the 'Users' and 'Computers' system AD containers. This also gave me some grey hair during troubleshooting.

Thank you so much for your time....! Highly appreciated and I pretty embarrassed right now :-D

@larssb Don't worry, I think it just points to the value of building in some configuration validation.  Lots of bits are harder to troubleshoot than they could be.

Hey Will,
Thank you. Yes agree some bits are a bit hard to troubleshoot.
/Larsen

Health is wealth, movement is medicine -

On 16. feb. 2016, at 10.58, Will Dean<EMAIL_ADDRESS>wrote:
@larssb https://github.com/larssb Don't worry, I think it just points to
the value of building in some configuration validation. Lots of bits are
harder to troubleshoot than they could be.
—
Reply to this email directly or view it on GitHub
https://github.com/jakubgarfield/Bonobo-Git-Server/issues/433#issuecomment-184604512
.

Hi @larssb
It seems you already resolve this issue.
Currently I'm facing this issue too.
I tried to configure the IIS and Bonobo many times carefully.
But the Bonobo did not start after all.
Here is my system:
Windows 7 - IIS 7.5 (Default)
Bonobo version 6.1.0
.NET framework 4.5.2 (I think it's OK)
Prerequisites
Install IIS - OK
Install ASP.NET MVC 4 - OK
Register MVC framework with IIS - OK
Installation
Set permission for IIS_IUSRS - OK
Convert Bonobo.Git.Server to Application in IIS - OK
Configure Authentication - OK
Here is the result when I start Bonobo
http://localhost/bonobo.git.server/Home/Error?aspxerrorpath=/bonobo.git.server/Home/Error
Could you please show me how to fix this issue
Many thanks

@buvatu This issue was about a problem with 'Active Directory' - before you try to do that you should make sure Bonobo works with its default configuration.

You need to upgrade to .net 4.6 at least !

Thank you so much @larshg
I follow the instruction of main page and it did not work at all.
After upgrade my .NET 4.6, everything works like a charm.
Thanks again.


Upgraded composer to v1.8.

composer v1.2.4 was released on 6 December 2016.
This patch upgrades it to the latest v1.8, last released on 11 February 2019.
It is still compatible with PHP 5.6.

Thanks for your contribution, however we do not merge anything to master on this project. Please open your PRs against the 1.0 or 2.0 branch. Thanks.


transactionReceipt deprecated since iOS 7.0

Should we be worried about transactionReceipt being deprecated?
....pub-cache/hosted/pub.dartlang.org/flutter_inapp_purchase-0.6.4/ios/Classes/FlutterInappPurchasePlugin.m:257:36: 'transactionReceipt' is deprecated: first deprecated in iOS 7.0 - Use -[NSBundle appStoreReceiptURL]

@holyspidoo Currently no! But could be updated and some PR will be welcomed!


Remove anchor from last crumb

I have removed the anchor from the last crumb, this is not only better for SEO, but also you don't need to set parameters if needed so the crumb can be generated just using annotations

Looks awesome, thanks a lot :)


Maybe Update the Usage for browser

The following HTML (which is according to the README) will throw error of "Uncaught TypeError: Cannot read properties of undefined (reading 'renderToString')"
<!doctype html>
<html>
  <body>
    <link rel="stylesheet"<EMAIL_ADDRESS>crossorigin="anonymous">
    <script defer<EMAIL_ADDRESS>crossorigin="anonymous"></script>
    <script<EMAIL_ADDRESS>    <script<EMAIL_ADDRESS>
    <script type="module">
      (function () {
        marked.use(markedKatex({ throwOnError: false }));
        let a = marked.parse("katex: $c = \\pm\\sqrt{a^2 + b^2}$");
        console.log(a);
      })();
    </script>
  </body>
</html>

And I found a solution at #315 so maybe update the README to give a better guide?
BTW, following works fine.
<!doctype html>
<html>
  <body>
    <link rel="stylesheet"<EMAIL_ADDRESS>/>
    <script<EMAIL_ADDRESS>    <script<EMAIL_ADDRESS>    <script src="https://cdn.jsdelivr.net/npm/marked-katex-extension/lib/index.umd.js"></script>
    <script src="https://cdn.jsdelivr.net/npm/marked/marked.min.js"></script>

    <script type="module">
      (function () {
        marked.use(markedKatex({ throwOnError: false }));
        let a = marked.parse("katex: $c = \\pm\\sqrt{a^2 + b^2}$");
        console.log(a);
      })();
    </script>
  </body>
</html>

Is it just because you removed defer from the katex script tag?
We can change that in the readme. Do you want to create a PR?


Show bundle loading progress on Android

This implements a loading banner like on iOS that shows the progress of the packager.

Test plan

Tested that it displays similar messages as it does on iOS and also that is show the right message when waiting for the remote debugger.
Tested that errors are still shown properly.
Tested that it works with packagers that don't support multipart response (add && false in https://github.com/facebook/react-native/blob/master/packager/src/Server/MultipartResponse.js#L81).
Run new unit tests.
Tested that backgrounding / foregrounding the app hides / show the banner properly.

cc @mkonicek

This is awesome! 💯  🎉

Just a few nits and questions.

Cool idea from @tomocchino: "Can we please cache the total number of modules from the last run and start the denominator at that number?".

@janicduplessis awesome job!

Cool idea from @tomocchino:

Can we please cache the total number of modules from the last run and start the denominator at that number?

Caching should happen on the server side, we should keep the client as dumb as possible.

This should be possible, I could have a look in a follow up to keep this PR simpler.

I agree - followup is easier. Some discussion at fb:

@mkonicek @cheeseok Updated with your comments, thanks for reviewing! Let me know if there is anything else.

Awesome, thank you!
Merging.

Needs some internal changes because this PR adds a method to an interface implemented by some closed source code (I didn't know about that code until now). Will try to change internal code and merge tomorrow.

It's merged \o/


Fix #657: Float being interpreted as integer

Fixes #657
Changes proposed in this PR:

Improve upstream ccxt & ohclv checks such that no floats can be entered as timestamps
Improve tests that show that floats are handled correctly before being pushed into a pl.DataFrame

I fixed the test issue


@conda-forge-admin please add user @aymondebroglie

It looks likes the package is not kept up to date anymore and I'd like to help adding the new versions !

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added user @aymondebroglie in conda-forge/ddtrace-feedstock#11.

@conda-forge/core would you mind adding me to the list of maintainers ? I would like to help keep it up to date ! Thanks !

If you open a PR adding yourself as a co-maintainer, I'd be happy to merge it.

Oops, didn't see the auto-generated PR. I've merged that.

Thanks !


Adding Endpoint called /user-role

The user-role endpoint encountered an HTTP 400 Bad Request error with the message "Invalid or expired verification token." This issue typically arises when the token provided for authentication or authorization is either incorrect, expired, or otherwise invalid. It prevents the update of user roles due to failed token verification.

No issues


STYLE: Remove ViolaWellsMutualInformation from Metrics components

This component appears obsolete, and may be removed from the repository, as was agreed upon during our internal Elastix sprint (with Stefan Klein and Marius Staring).
Note that this component was OFF by default already.

@mstaring @stefanklein Is this note from https://github.com/SuperElastix/elastix/blob/5.0.0/Components/Registrations/MultiMetricMultiResolutionRegistration/elxMultiMetricMultiResolutionRegistration.h#L56-L57 still relevant?
Note that some metrics (ViolaWellsMutualInformation for example)
do not support the image sampler functionality (yet).

agreed on the removal.
the remark I am not sure. I think you would need to check all metrics if they inherit from the advancedimagetoimagemetric.

the remark I am not sure. I think you would need to check all metrics if they inherit from the advancedimagetoimagemetric.

It appears that (at least) elastix::CorrespondingPointsEuclideanDistanceMetric<TElastix> does not inherit from itk::AdvancedImageToImageMetric<TFixedImage, TMovingImage>.
Does that mean that CorrespondingPointsEuclideanDistanceMetric does not support the image sampler functionality ?
I could just remove "(ViolaWellsMutualInformation for example)" from the comment at  https://github.com/SuperElastix/elastix/blob/5.0.0/Components/Registrations/MultiMetricMultiResolutionRegistration/elxMultiMetricMultiResolutionRegistration.h#L56-L57  OK?

CorrespondingPointsEuclideanDistanceMetric  does not need samples, so that one is fine. If that is the only one then all metrics now support samplers (when needed). So the entire comment can go then.

If that is the only one then all metrics now support samplers (when needed). So the entire comment can go then.

Thanks, but I do not know if CorrespondingPointsEuclideanDistanceMetric is the only one. It's just the first one I encountered, that does not inherit from AdvancedImageToImageMetric.


Drain collector buffer on Close

Update the Close() logic of the HTTP and Scribe Collectors to drain any
pending spans before returning. This provides a mechanism to prevent
loss of buffered trace data when shutting down.
This is of particular use for short lived processes such as cli clients.

Thanks for the contribution Richard. I will check the code on Friday when done with meetings and traveling. I quickly glanced at it and it looks good but want to give it a tiny bit more attention.

Thanks for the update. I can quickly add the CLI example change to this PR. Didn't notice it when I was updating the tests to make use of the new Close logic.

Thanks for the contribution Richard


Ambiguous source

Hello,
please check this source string: "You will use this name when writing a query the uses the snippet."
and confirm if it  should read: "You will use this name when writing a query that uses the snippet."
Kind regards,
Anne H., QC

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

ID: 84b188d2-c21c-c402-f69c-a0a158eaf2a4
Version Independent ID: e1313e0e-33b2-2d0f-4f25-382096344bf6
Content: Query snippets - Azure Databricks - Databricks SQL
Content Source: databricks/sql/user/queries/query-snippets.md
Service: azure-databricks
Sub-service: databricks-sql-analytics
GitHub Login: @mssaperla
Microsoft Alias: saperla

@QueryCoordinator
Thanks for reporting this! We have created a PR for this issue, and the changes should go live soon.


[Icon Request] rpm and archlinux folder

[x] I'm sure this issue is not a duplicate.

[x] I'm submitting an icon request:

Type: folder
Icon Name: rpm, archlinux
Sample original Icon:

rpm: https://simpleicons.org/?q=redhat or https://simpleicons.org/icons/redhat.svg
archlinux: https://simpleicons.org/?q=archlinux or https://simpleicons.org/icons/archlinux.svg

More info:
This folder is used to store packaging resources or scripts, like deb folder. #2418

see #650


Feature Request: Add Calendar To Visualize Upcoming Transactions

It would be nice to be able have a calendar view to visually see the upcoming transactions. Right now I have this in a Google Calendar, however the days do not always line up with non-standard recurring bills. A calendar would help visualize this better and provide a way to plan a bit easier.
Expected Behavior

A calendar that shows up coming transactions due on their respective dates.
A way to edit those transactions from that calendar.
Multiple views, such as 1 month, semi-monthly, bi-weekly, and weekly.

This feature request is being tracked in Trello.


Clipboard not working on macOS

Seems like a wkwebview issue since upstream winit is working fine.

Just realized we are using private APIs. We should remove them too.
https://github.com/webview/webview/pull/405/files#r456947213

we might need to add init acripts to interact with DOM
https://developer.apple.com/library/archive/documentation/AppleApplications/Conceptual/SafariJSProgTopics/CopyAndPaste.html


refine: seperate parquet reader and arrow convert

This PR separates out the parquet reader from the arrow module. And make the arrow module a dir so that we can separate the from_arow and to_arrow.

cc @liurenjie1024 @Xuanwo @Fokko

cc @viirya Would you also take a look?


Fix discourse urls

The current discourse embedding snippet was not working for me. I found two problems

the discourseUrl variable hardcoded  a couple of forward slashes at the beginning. The correct path should be set only with the site.comments.discourse.server configuration variable.

the canonical url assigned to discourseEmbedUrl was not computed correctly.

This commit attempts to fix both.

My two cents:

I use currently still a very old version of jekyll+this theme, i.e. I have no experience here.
I could imagine that the discourse canonical url must match the url in the RSS feed that is read in by Discourse. So shouldn't we create the canoncical url in the same manner like in feed.xml ?

Yes looking at the canonical url in xml feed makes sense. Where is that
constructed?
On Aug 16, 2017 14:21, "Robert Riemann"<EMAIL_ADDRESS>wrote:

My two cents:

I use currently still a very old version of jekyll+this theme, i.e.
I have no experience here.
I could imagine that the discourse canonical url must match the url
in the RSS feed that is read in by Discourse. So shouldn't we create the
canoncical url in the same manner like in feed.xml ?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mmistakes/minimal-mistakes/pull/1181#issuecomment-322753241,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABAmNHnyZf4jiIws3TpAIbm4DjbRF5qSks5sYt7ZgaJpZM4O4w2n
.

@tendon That's handled not by the theme but by jekyll/jekyll-feed plugin.

It seems that feed.xml uses simply
{{ post.url | absolute_url }}

On Wednesday, 16 August 2017 15:10:58 CEST Michael Rose wrote:

@tendon That's handled not by the theme but by [jekyll/jekyll-feed
plugin](https://github.com/jekyll/jekyll-feed/blob/master/lib/jekyll-feed/f
eed.xml).

Which is pretty much what we're using now, just with a few filters to deal with index.html in paths.


Allow to specify generated additional interfaces via configuration

I would like to add interfaces to all my generated classes without adding javaInterfaces to my schema definitions. Would you be willing to take a PR that adds a GenerationConfig option additionalInterfaces (or something like that), which are added to all generated root types?

I guess this won't happen, so closing it.


adds artists and tracks components with artist and track component be…

…ing rendered based on response from api

Creates the rendering for artists and tracks and sets it up for future querying of each specific track/artist


Issues with next version of ggplot2

Hi
We preparing the next release of ggplot2 and our reverse dependency checks show that your package is failing with the new version. Looking into it we see that the example for plot_gmm() errors because the input to stat_function() returns NULL.
You can install the release candidate of ggplot2 using<EMAIL_ADDRESS>to test this out.
We plan to submit ggplot2 by the end of October and hope you can have a fix ready before then
Kind regards
Thomas

Should be fixed with latest release of plotmm (v0.1.1) currently on its way to CRAN. Thanks for the heads up!


MIDI Program Change broken in TinyUSB used by Pico SDK 2.0.0?

See https://github.com/hathach/tinyusb/issues/2775 .

The develop branch (and soon 2.1.0) have moved to TinyUSB 0.17.0.

moving to 2.2.0 - just because (whilst a TinyUSB issue) it may still be in play


Add TypeScript definitions for tcomb-form

This adds TypeScript definitions for tcomb-form, based on the Guide.
A devDependency on @types/react was added for the Form component and the other Component classes.
Similar to the type definition for tcomb-validation, https://github.com/gcanti/tcomb-validation/pull/63, this extends the tcomb module, adding getTcombFormFactory and getTcombFormOptions to t.Type<T>.
I am, however, not sure if the type definitions are 100% complete or accurate. I'd appreciate your feedback to improve them.

@RedRoserade this is great, thanks for taking the time to write this definitions

Why it's still doesn't merged?

My bad, there was a pending review which I didn't submit at the time, sorry @RedRoserade

Can the above change request please be acted on? It would be really helpful to have Typescript definitions enabled.


Request NPM has been deprecated

$ npm install request --save
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142

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

ID: 06ea8157-ae16-1905-1638-185ae47ebbaa
Version Independent ID: ca831fc5-ed09-3e3a-1da9-8b1a38448214
Content: Quickstart: Detect faces in an image with the Azure REST API and Node.js - Azure Cognitive Services
Content Source: articles/cognitive-services/Face/QuickStarts/node.md
Service: cognitive-services
Sub-service: face-api
GitHub Login: @PatrickFarley
Microsoft Alias: pafarley

@fadyanwar Thanks for the question. We  will update you shortly.

@PatrickFarley Could you please check on this.

I'll ask the product team to update their sample code

Code has been updated
#please-close


Explain gotchas related to testing and @Tracing

When adding Lambda Powertools Java to my project and I have annotated my handler method with @Tracing.
When a basic unit test is run I get an error "SegmentNotFound".
I think you may be able to create a segment in a test like AWSXRay.beginSegment("handleRequest") but that is not what I want to be doing.
It would be helpful if the documentation could explain what to do in this testing scenario.

Definitely, AWSXRay.beginSegment is needed for test cases. Fair point regarding documentation, we can update docs to make it but more clear

Hey @GuyJackson , Docs are now updated to with recommendations for testing. https://awslabs.github.io/aws-lambda-powertools-java/core/tracing/#testing-your-code

Great thank you @pankajagrawal16 will the recommendations a go.


[$1000] Chat - The green line is not displayed on new messages.

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

Action Performed:

Open app.
In the main testing device (as account A) Navigate away from the conversation with account B.
Send some messages to Account A (as account B).
Verify that the username on the left hand navigation is displayed bold (bold username indicates it has unread messages).
Enter the conversation that has unread messages and Verify there is a green line indicating the New Messages.

Expected Result:
Verify there is a green line indicating the New Messages.
Actual Result:
The green line is not displayed on new messages.
Workaround:
Unknown
Platforms:

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

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

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/230368220-76548e12-264e-4241-9ebd-c8b2915e38fd.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01131b1df9e7b7f9fd
Upwork Job ID:<PHONE_NUMBER>034382336
Last Price Increase: 2023-04-07

Triggered auto assignment to @slafortune (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Looks good!

Job added to Upwork: https://www.upwork.com/jobs/~01131b1df9e7b7f9fd

Triggered auto assignment to @bfitzexpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @fedirjh (External)

Triggered auto assignment to @Beamanator (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Dupe https://github.com/Expensify/App/issues/15019

Sweet, closing


Null Pointer Exception when Hook hits certain blocks

Due Date: 01/11/2021
It seems that the code is producing a null pointer exception when the Frozen Anchor hook passes through breakable tiles (like thorns in crimson/corruption)
-- @alecrm

Been testing this some and I can't actually reproduce the bug. May have been resolved in some other work previously done on the Rognir mod and never closed. @benjazzy, can you try testing this some, maybe, and see if you can get it to trigger?

Been testing this some and I can't actually reproduce the bug. May have been resolved in some other work previously done on the Rognir mod and never closed. @benjazzy, can you try testing this some, maybe, and see if you can get it to trigger?

After testing, neither @benjazzy nor @alecrm were able to reproduce it. Closing now until it maybe pops up again

After testing, neither @benjazzy nor @alecrm were able to reproduce it. Closing now until it maybe pops up again