can't properly iterate children tags and tags named

I have been using goquery to read some simple XML documents successfully so far, until now. The XML in the reference code is a cut-out of a larger XML document that I can work on successfully, except for this particular snippet that deals with a list of images.
OUTPUT: http://pastebin.com/sVqWHiwf
CODE: http://pastebin.com/4TVLCStA

Why can't I properly iterate over the Image tags and their URLs?
Why are the tags changing from Image to img ?

 Goquery relies on those parsed nodes, so that's why you have those issues. Being an html parser, it assumes the input is html and it is probably part of the reference parser doc to replace <image> with <img> (because those are treated as synonyms by browsers, according to http://stackoverflow.com/questions/11928566/img-vs-image-tag-in-html).
And then, because <img> can't have any content (https://developer.mozilla.org/en-US/docs/Web/HTML/Element/img), the content is moved outside the tag.
You'd really want to use an xml parser for this. It would be nice to have goquery support an xml parser instead of the html one for cases like this, and then be able to use the jquery functions to navigate the doc, but xml is a beast with many dark corners, it would probably be a bad idea. I'd suggest something like https://github.com/moovweb/gokogiri or https://godoc.org/gopkg.in/xmlpath.v2 for a pure go xpath implementation.

I see. Thank you for taking the time to explain it. I'll look into using proper XML parsers for it, although I was enjoying using GoQuery for that!


How to set autoexposure limit in python code

Required Info

Camera Model
D435

Firmware Version
<IP_ADDRESS>

Operating System & Version
Ubuntu 16

Kernel Version (Linux Only)
5.4.0-73-generic

Platform
PC

SDK Version
2.45.0

Language
python

Segment
Robot

Issue Description
While I can set the autoexposure limit in the realsense-viewer I have been unable to do this in python. I have tried setting it in the json config file with the 'controls-autoexposure-limit' keyword but get the following error:
RuntimeError: controls-autoexposure-limit key is not supported by the connected device!
I have also tried setting it with the set_option method:
sensor = pipeline.get_active_profile().get_device().query_sensors()[0]
sensor.set_option(rs.option.auto_exposure_limit, 100)

But the error is similar:
RuntimeError: object doesn't support option #85
But since the option works in the realsense-viewer I must be doing something wrong. Could you maybe point out my mistake or give some example code?

Hi @bartonp2  Another RealSense Python user recently experienced the same RuntimeError: object doesn't support option #85 error when attempting to convert the RS2_OPTION_AUTO_EXPOSURE_LIMIT instruction to Python by using sensor.set_option(rs.option.auto_exposure_limit, 100).
https://github.com/IntelRealSense/realsense-ros/issues/1821#issuecomment-827342776
RS2_OPTION_AUTO_EXPOSURE_LIMIT is listed in the official C++ documentation:
https://intelrealsense.github.io/librealsense/doxygen/rs__option_8h.html#a8b9c011f705cfab20c7eaaa7a26040e2ac11001971364b90b542941e51ebb911a
I could not find an equivalent instruction in the official pyrealsense2 documentation though.
https://intelrealsense.github.io/librealsense/python_docs/index.html
The RealSense Viewer is a C++ application, so it makes sense that auto-exposure limit would be accessible.
I attempted to set auto-exposure limit in the Viewer and then export a json file from the Viewer but the auto-exposure limit value was not exported into the json.
I will seek advice from Intel to confirm whether auto-exposure limit is currently exclusive to C++.

After consultation with Intel, an official bug report has been filed with Intel regarding the issues in the above comment.  This case should be kept open whilst the bug report is active.  Thanks again!

Has there been any progress on this issue?

@bartonp2  I have no further information to provide at this time.

Hi @bartonp2   Intel have informed me that that the auto_exposure_limit instruction is already supported in pyrealsense2.  There does not seem to be an entry for it in the pyrealsense2 documentation but a Python script recently made use of it, so I have posted a link to that script reference below.
https://support.intelrealsense.com/hc/en-us/community/posts/1500001050241/comments/1500000948182

Hi @MartyG-RealSense, thanks for the suggestion. But unfortunatly this does not work for me, as noted in the issue description, and results in the runtime error:
RuntimeError: object doesn't support option #85

Intel did also suggest that if you installed with a Python wheel then the problem that you are having could be related to that.

Hi @bartonp2  Do you require further assistance with this case, please?  Thanks!

Hi MartyG,
I am not working on this project at the time being and do not require further assistance for the time being.
Thank you!
On Aug 29 2021, at 8:32 am, MartyG-RealSense @.***> wrote:

Hi @bartonp2 (https://github.com/bartonp2) Do you require further assistance with this case, please? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (https://github.com/IntelRealSense/librealsense/issues/9018#issuecomment-907738209), or unsubscribe (https://github.com/notifications/unsubscribe-auth/AF23LHU6Q7GJUE5WCDRCCMTT7HH7ZANCNFSM442LL7XA).
Triage notifications on the go with GitHub Mobile for iOS (https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675) or Android (https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub).

Thanks very much @bartonp2 for the update!  This case should be kept open for the moment as there is an active Intel bug report associated with it, but you do not need to take any further action.  Thanks again   :)

Hi again @bartonp2   The Intel bug report that was associated with this case is now closed, so this case can be closed too.  Please feel free to re-open this case or create a new one if you return to your project at a future date and require further assistance with this particular question.  Good luck!


[Bug] Deploy Preview - List widget breaks on an error in the text widget inside the list widget

Description
Deploy Preview : It is observed that List widget breaks on an error in the text widget inside the list widget
Expected Result:
If there are any errors the list widget should not break
Actual Error:
Error on text widget breaks list widget
Steps to reproduce the behaviour:
1.1.Launch Appmsith app in the Deploy preview - https://deploy-preview-4189--appsmith-editor-ce.netlify.app/
2.Drag and Drop list widget
3.Place text widget inside any of the item container
4.Open property pane of text widget
5.Click on visible JS
6.Pass the following query - {{currentItem.name === “Bulbasaur”}}
7.Click on visible JS again and observe
videoLink:

Important Details
Version: [14.1 (166<IP_ADDRESS>.3)]
OS: [MacOS]
Browser [Safari]
Environment [Deploy Preview]

@YogeshJayaseelan  The code you pasted includes these quotes ”””””””””
Not sure how text input accepted that, but they should """""".
Please confirm and close this issue. If you are double quotes are posted as apostrophe, please raise another issue.

@SatishGandham This is to confirm you that the quotes are in regular formats """ and not used italic formats ”””””””””. Hence keeping this issue open

Cant reproduce


Function download Range not working

Hi All,
I want download range but it not woking with me
request.setRequestHeader("Range", segment.range);
segment.range is undefined
my file .m3u8 and server support download range
#EXTM3U
#EXT-X-VERSION:4
#EXT-X-TARGETDURATION:11
#EXT-X-MEDIA-SEQUENCE:0
#EXT-X-PLAYLIST-TYPE:VOD
#EXTINF:8.217089,
#EXT-X-BYTERANGE:212440@0
5bd9fd91ce7e1b08435acb87-000.ts
#EXTINF:10.427778,
#EXT-X-BYTERANGE:2814360@212440
5bd9fd91ce7e1b08435acb87-000.ts
#EXTINF:2.794644,
#EXT-X-BYTERANGE:128592@3026800
5bd9fd91ce7e1b08435acb87-000.ts
..........

It is not about failover or partial segments downloads.
It is about Byte-Range Support for Segments in HLS - i.e. when a file contains more than one or all segments (fMP4 for example).


Move internal files to a Core folder (close #1)

Issue #1

Thanks for your pull request. Is this your first contribution to a Snowplow open source project? Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://docs.snowplowanalytics.com/docs/contributing/contributor-license-agreement/ to learn more and sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


Animation does not work well on  Ms Edge browser

Hello,
Do you have a solution to fix the issue with the animation on edge browser?
all content is to the left and then after the animation has finished it goes to the center

This was already dealt with oncr but it seem this bug returned, it has something to do with flex layout (the main layout) of the content and footer.
I'll look into it later but maybe you can try some width 100% or jusy google for IE / Edge flexbox width glitch.

thx :) I will try
And thank you for sharing this project

From what I've seen, is not related to flex main layout
I fixed it by adding a wrapper div in the pages with position relative

Could you please share solution by creating a  PR with the fix ?

Sure,
But I see that I don't have the rights to create a branch for the pull request

Hmm, the way this usualy work is to create a fork of the repository, then create branch with fix and push it to the fork. Github then offers creation of pull request automatically when checking branch on the web.
Hope that helps :)

Hey @rurziceanu  !
Could you please share fix at least here in comment?
Thank you!

Hi,
Sorry, I did not had time for this..
But the fix is in the animation definition
export const routerTransition = trigger('routerTransition', [
transition('* <=> *', [
query(':enter > *', style({ opacity: 0, position: 'absolute', left: 0, right: 0 }), {
optional: true
}),
query(':enter .' + ANIMATE_ON_ROUTE_ENTER, style({ opacity: '0' }), {
optional: true
}),
sequence([
query(
':leave > *',
[
style({ transform: 'translateY(0%)', opacity: 1 }),
animate(
'0.2s ease-in-out',
style({ transform: 'translateY(-3%)', opacity: 0 })
),
style({ position: 'absolute', left: 0, right: 0 })
],
{ optional: true }
),
query(
':enter > *',
[
style({
transform: 'translateY(-3%)',
opacity: '0',
position: 'absolute', left: 0, right: 0
}),
animate(
'0.5s ease-in-out',
style({ transform: 'translateY(0%)', opacity: 1 })
)
],
{ optional: true }
)
]),
query(
':enter .' + ANIMATE_ON_ROUTE_ENTER,
stagger(100, [
style({ transform: 'translateY(15%)', opacity: 0 }),
animate(
'0.5s ease-in-out',
style({ transform: 'translateY(0%)', opacity: 1 })
)
]),
{ optional: true }
)
])
]);
and then make sure that the container has position relative

The stuff from comment unfortunately didn't really work. Partially disabling animation (removing whole page slide-in but still KEEPING the individual element slide-ups) solves this issue.
Preserved animations still look pretty great in IE and Edge.
Chrome / Firefox users can enjoy full experience.


Fixed search error

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
All new code requires tests to ensure against regressions

Description of the Change
Fixed search error
Alternate Designs

Benefits

Possible Drawbacks

Verification Process

Applicable Issues

Please review this bugs.

Hi. I fixed this bug now. @daniil5469


Filter out bad markers on the server and clientside for contributor map

still getting null markers and other tomfoolery, so I've shored it up with yet more null checks and reworked the active users pull request method.
:santa: :christmas_tree: :snowman: :beers:

Lovely work @wadtech, merry christmas!! :snowboarder:


Workaround for bytes.Buffer bug in go 1.11 and earlier

Recent development in Acra has uncovered a bug in CGo which caused Secure Cell calls to panic on CircleCI builds like this:
panic: runtime error: cgo argument has Go pointer to Go pointer [recovered]
	panic: runtime error: cgo argument has Go pointer to Go pointer
goroutine 7 [running]:
testing.tRunner.func1(0xc0000fe300)
	/home/user/go_root_1.11.5/go/src/testing/testing.go:792 +0x387
panic(0x578ca0, 0xc00008ec40)
	/home/user/go_root_1.11.5/go/src/runtime/panic.go:513 +0x1b9
github.com/cossacklabs/themis/gothemis/cell.(*SecureCell).Protect.func1(0xc00000c700, 0xc0000146b0, 0x7, 0x8, 0x8ca438, 0xf, 0xc0000146b0, 0x7, 0xc000104720, 0x3d, ...)
	/home/user/gopath/pkg/mod/github.com/cossacklabs/themis@v0.0.0-20190313112615-419d5a3c2ca8/gothemis/cell/cell.go:195 +0x194
github.com/cossacklabs/themis/gothemis/cell.(*SecureCell).Protect(0xc00000c700, 0xc0000146b0, 0x7, 0x8, 0xc000104720, 0x3d, 0x40, 0xc000104700, 0xc0001046b0, 0x2d, ...)
	/home/user/gopath/pkg/mod/github.com/cossacklabs/themis@v0.0.0-20190313112615-419d5a3c2ca8/gothemis/cell/cell.go:195 +0x1c2

It can be triggered by passing GoThemis byte slices returned by bytes.Buffer in go 1.11 and some earlier versions. It does not seem to be triggered every time and with every CGo call but I have found that “context” argument of Secure Cell calls causes a panic reliably.
Add a workaround for this bug so that we don’t panic randomly. Since we can’t know whether bytes.Buffer has been used, we simply copy each and every byte slice that we operate on before passing it to Themis Core via CGo. The bug has been fixed in go 1.12, so we use conditional compilation to make that workaround a no-op on versions that are not affected.
utils.SanitizeBuffer() is technically exported by GoThemis and available to the users, but this function is considered an implementation detail and may be removed in the future (e.g., once we decide that we no longer wish to support go 1.11 and stop maintaining this workaround).
CircleCI has been running tests only with the version that is shipped with the Docker image. Update the configuration to test GoThemis with go 1.11—1.14. Our policy is to provide full support for the current stable versions of Go (those are 1.14 and 1.13.8 right now), and best effort support for earlier ones.
Checklist

[X] Change is covered by automated tests
[X] The coding guidelines are followed
[X] Public API has proper documentation
[X] Changelog is updated

I’ve just realized that we can handle keys in Secure Messages slightly better, avoiding copying them on every operation and doing it only on Secure Message constructions. Though, this will break an (arguably silly) use case when (*keys.PrivateKey).Value is replaced with a different key without making a new Secure Message.

Okay, everyone! After offline discussion and some prototyping we found out that a proper workaround for this issue is too ugly and hard to maintain, so we will not add it. New users are not expected to use Go 1.11 anymore, and existing users can be given advice to avoid using bytes.Buffer with GoThemis if they see a panic like this.
I have reverted the workaround and removed it from this PR. Other updates are still there, like testing with multiple versions of Go. Regression test also remains, I tweaked it so that it passes with Go 1.11 (expecting a panic in this case).


Module import issue

Unable to import modules while executing the HakeFlowDemo.ipynb.
Note: this was performed after uninstalling the echoflow dependency.

Use sys or os library to import the relevant modules from base path.


J endcarto

so you'll see it's mainly a big cleaning of the console logs and also a global use of endpointGenerator instead of endpointCreator... There are finally some minor changes in the config files in the backend so we could declare whatever link we want for the navbar logo + an extra footer declaration for partners if any...

and now conflicts were resolved....


KeyShared subscription doesn't work on non-persistent topics

Describe the bug
According to https://github.com/apache/pulsar/pull/4462 non-persistent topics should have the support of KeyShared subscriptions. However, it seems it doesn't work (at least in Java client).
To Reproduce
Steps to reproduce the behavior:
0. Using the official pulsar-client 2.7.0

Create a producer for non-persistent topic
Create a consumer for the same topic
Send a new message with key
Observe the fact consumer will never receive the message

Repeating the same exercise with a persistent topic has a different result.
Expected behavior
Consumer should receive the message as it is happening with persistent topic.
Additional context
Using Pulsar image kafkaesqueio/pulsar:2.7.0-kesque-1 and pulsar-client Java library of version 2.7.0.
Code sample (in Scala):
val topic = "non-persistent://public/default/topic-1" // try changing to persistent

val producer = client.newProducer().topic(topic).create()
val consumer = client
  .newConsumer()
  .topic(topic)
  .subscriptionType(SubscriptionType.Key_Shared)
  .subscriptionName("my-sub")
  .subscribe()

val msg = "Hello".getBytes(Charset.defaultCharset())

producer.newMessage().key("key-1").value(msg).send()

val received = consumer.receive()
val result   = new String(received.getData, Charset.defaultCharset())
println(result)

Nvm, seems like a duplicate - https://github.com/apache/pulsar/issues/9703


Animation

Hi there!
I am wondering, what is the best approach to animate this modal?
The problem is – to create robust animation, which is easy to reuse across the whole application I have to create two wrappers to create actual animations and pass correct handlers to close modals.
So, what is the best way to deal with animations, is it possible with a single wrapper?
Maybe I just miss something obvious.
Thanks for any advice!

Hi, you can try to start off by taking a look at this example
Basically you add css for .ReactModal__(Overlay/Content)--(after-open/before-close) css classes

Here's a snippet I came up with:
.ReactModalPortal > * {
    // When the modal is closed, overlay div has no css class
    // This selector should be overridden by the `&--after-open` class below
    opacity: 0;
}
.ReactModal__Overlay {
    transition: opacity 200ms ease-in-out;
    background: rgba(0, 0, 0, 0.15);
    &--after-open {
        opacity: 1;
    }
    &--before-close {
        opacity: 0;
    }
}

You should also provide a closeTimeoutMS prop to the <Modal /> components, otherwise the closing animation will not work

@megapctr oh, thanks a lot, didn't notice this class changing.
It definitely solves my problem, so thanks a lot.
Though it could cause some troubles if need different type of animation.
But anyway, now it'll work. Thanks again!

Avoid using wildcard * selector with these changes.
`.ReactModalPortal > div {
opacity: 0;
}
.ReactModalPortal .ReactModal__Overlay {
transition: opacity 500ms;
&--after-open {
opacity: 1;
}
&--before-close {
opacity: 0;
}
}`

Would it be possible to integrate animating the modal with rc-animate?

Hi! Sorry for reopen this old issue, it's possible to use something similar to @megapctr solution but using Radium for modal styling?

@segulino you can define global classes in radium as well.
So, I personally would separate animation and radium styles here (if they overlap, there is an ugly solution to add !important in css rules).

Hi. Thanks for the animation snippet. The after-open animation works fine for me but the before-close doesn't. No animation is triggered by it, any idea why ?

@clementino36 Have you provided closeTimeoutMS prop to the Modal? It might an issue.

My bad for not reading the docs enough. Thank you!

@Bloomca thanks for your answer, but I don't understand how use this for use in my component, my modal component looks same this:
<Modal
      style={imageModal ? styles.modalImage : styles.modalForm}
      isOpen={isOpen}
      contentLabel={contentLabel}
      onRequestClose={onRequestClose}
    >...

And my Styles.js
const overlayStyle = {
  backgroundColor: modalOverlayColor,
  zIndex: 2
};

export default {
  modalForm: {
    overlay: overlayStyle,
    content: {
      display: 'flex',
      backgroundColor: white,
      overflow: 'auto',
      marginRight: '30%',
      marginLeft: '30%',
      ....

Could you help me with the way that I can include afterOpen / beforeClose selectors on this?

Hi @segulino, yeah, sure -- here is a rough example how it can look like:
<Style
  rules={{
     '.ReactModalPortal > div': {
        opacity: 0,
     },
     '.ReactModalPortal .ReactModal__Overlay': {
         transition: 'opacity 200ms ease-in-out',
         background: 'rgba(0, 0, 0, 0.15)',
     },
     '.ReactModalPortal .ReactModal__Overlay--after-open': {
        opacity: 1,
     },
     '.ReactModalPortal .ReactModal__Overlay--before-close': {
        opacity: 0,
     },
  }}
/>
<Modal
      style={imageModal ? styles.modalImage : styles.modalForm}
      isOpen={isOpen}
      contentLabel={contentLabel}
      onRequestClose={onRequestClose}
    >...

Something like this should work. But I can recommend to put Style component to the top level, so you don't render it again.

I think I understood the behavior of the Radium Style component with this, and your example works great. Thanks for your time and help @Bloomca !

I'm having the opposite issues -- open animation isn't showing, but close animation is. closeTimeOutMS prop is set.

It seems I have to use !important to overwrite the background somehow...

Do people have suggestions on how to animate with direction? I have mine pinned to the bottom of the screen, and want it to animate up from the bottom, and animate down on close. Current content style:
      content: {
        top: 'auto',
        left: '0%',
        right: '0%',
        bottom: '0%',
        borderRadius: '0',
        padding: '0 0',
        overflow: 'visible',
      },

Hi, I followed the instructions and didn't manage to make it work.
In particular, the opening animation works, but the closing one doesn't.
My style looks like this:
.ReactModal__Overlay` {
  transition: opacity 500ms ease-in-out;
  opacity: 0;
}

.ReactModal__Overlay--after-open{
  opacity: 1;
}

.ReactModal__Overlay--before-close{
  opacity: 0;
}

And my component is:
<Modal isOpen={isOpen} contentLabel={"Text"} className="TextPopModal shadow" closeTimeoutMS={500}>
/* STUFF */
</Modal>

I tried getting rid of the classes, but I get the same behavior.
Anyone has any suggestion?
Thanks

@starsoluzioni I have same issue. We missed onRequestClose, hope it helps.
 handleClose = () => {
    this.setState({
      modalIsOpen: !this.state.modalIsOpen
    });
  };

<Modal isOpen={this.state.isOpen} onRequestClose={this.handleClose} contentLabel={"Text"} className="TextPopModal shadow" closeTimeoutMS={500}>
/* STUFF */
</Modal>

The base class and afterOpen class is apply at the same time, it it no animation when open modal, but has animation on close

For those that struggled to get the close animation to work, try boosting closeTimeoutMS to something a fair bit greater than your animation time. My example:
closeTimeoutMS={2000}

	transition: opacity 200ms linear;

Works for me.


Fix CI workflow

Re-enable DocFx build
Suppress NU1507 warning caused by a regression in .NET SDK 6.0.300

Codecov Report

Merging #71 (83bd886) into master (8f08d93) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 83bd886 differs from pull request most recent head 184affe. Consider uploading reports for the commit 184affe to get more accurate results

@@           Coverage Diff           @@
##           master      #71   +/-   ##
=======================================
  Coverage   81.79%   81.79%
=======================================
  Files          18       18
  Lines        1357     1357
  Branches      110      110
=======================================
  Hits         1110     1110
  Misses        219      219
  Partials       28       28

Superseded by #72


Features missing

Are there any plans to upgrade the R# gallery to a newer version, to bring in nuget.org features like organizations? We would like something like this for open source projects with multiple contributors.

Hi Robert!
While we are not planning to upgrade R# Gallery to newer sources of nuget.org we are working to bring one gallery for all JetBrains products, IntelliJ IDEA family products including. https://plugins.jetbrains.com/marketplace


Move visibility operation from search attributes to indexer message

What changed?

Why?

How did you test it?

Potential risks

Release notes

Documentation Changes

@longquanzheng

Pull Request Test Coverage Report for Build 0181c164-5fd5-4ada-ae00-af5372f2b3ef

22 of 24 (91.67%)  changed or added relevant lines in 1 file are covered.
78 unchanged lines in 15 files lost coverage.
Overall coverage decreased (-0.006%) to 57.766%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

common/persistence/elasticsearch/esVisibilityStore.go
22
24
91.67%

Files with Coverage Reduction
New Missed Lines
%

service/history/queue/timer_queue_processor_base.go
1
77.92%

client/history/client.go
2
38.05%

client/history/metricClient.go
2
45.3%

common/persistence/persistenceMetricClients.go
2
55.49%

common/task/weightedRoundRobinTaskScheduler.go
2
89.12%

service/history/execution/mutable_state_builder.go
2
69.03%

service/history/handler.go
2
47.73%

common/task/fifoTaskScheduler.go
4
80.41%

service/frontend/workflowHandler.go
4
60.45%

service/history/task/fetcher.go
4
91.28%

Totals

Change from base Build 0181bd26-606d-4e43-9425-b20785525f7f:
-0.006%

Covered Lines:
83750

Relevant Lines:
144981

💛  - Coveralls


Update binary install docs with new contents of tgz

This is now up to date with contents of 1.12 tgz
Also change usage to dockerd not docker daemon
Signed-off-by: Justin Cormack<EMAIL_ADDRESS>This is the docs adjustment for #23312 - I can't see anywhere else that needs to refer to the docker-proxy other than a mention in the changelog.
cc @thaJeztah

LGTM

LGTM :cat:

thanks!


[V0.9.3]无法保留换行符/Unable to keep line breaks

您好，我是[DeeplxFile]（https://github.com/infrost/DeeplxFile）的开发者，首先非常感谢您的项目，使我的想法成为可能。
在使用您新版本[V0.9.3]的程序后，我发现它会导致我现有的项目出现问题，研究后发现该版本API传入请求后（Free EndPoint），似乎无法正确处理换行符，如下例子：
import httpx, json

deeplx_api = "http://<IP_ADDRESS>:1188/translate"
data = {
	"text": "Hello World \n Thanks for the developer of DeepLX",
	"source_lang": "EN",
	"target_lang": "ZH"
}
post_data = json.dumps(data)
headers = {"Content-Type": "application/json"}
r = httpx.post(url = deeplx_api, data = post_data, headers=headers).text
print(r)

程序返回的data中会直接把该换行符替换成空格：
{"alternatives":["你好，世界 感谢 DeepLX 开发人员","你好，世界 感谢 DeepLX 的开发者","你好世界 感谢 DeepLX 开发人员","你好世界 感谢 DeepLX 的开发者"],"code":200,"data":"你好，世界 感谢 DeepLX 开发人员","id":8388902000,"method":"Free","source_lang":"EN","target_lang":"ZH"}

如果我使用\\n这样的转义换行符，则程序返回的是一个左括号（，这让我感觉到十分困惑。
此外，如果我尝试传入一个字典，如
dic = ["Hello World","Thanks for the developer of DeepLX"]
data = {
	"text": dic,
	"source_lang": "EN",
	"target_lang": "ZH"
}

程序会直接返回400 Empty data错误。
现在我只能使用一种不那么优雅的方法：把换行符替换成
（程序会原封不动地将其传回），然后再把
替换回换行符。
此前的版本似乎没有这个问题，我想问这个是一个可以被修复的issue，还是以后的程序逻辑都会是这样了？

Hello, I am the developer of DeeplxFile and first of all thank you very much for your project that made my idea possible.
After using your new version [V0.9.3] of the project, I found that it causes problems with my existing project, after investigating it, I found out that this version of the API doesn't seem to be able to handle line breaks correctly after passing in the request (Free EndPoint Method), as shown on the above example: The line break is replaced with a space directly in the data returned by the program. If I use an escape newline like \\n, the program returns a left bracket (, which I find very confusing.
Additionally, if I try to pass it in a dictionary, the programme would simply return a 400 Empty data error.
Now I'm stuck with a less elegant method: replacing line breaks with
 (which the program passes back as is) and then replacing
 back with line breaks.
Previous versions don't seem to have this problem, so I'm asking if this is an issue that can be fixed, or if the program logic will be like this from now on?

Thanks for your feedback, I will fix this issue ASAP.

@infrost
I've tested it and the DeepL API returns the original \n as /n , I'm not sure what to do with that at the moment, it's obviously not a good solution if you use a simple string replacement.


How long is "until-revoked"

Hi does the following sentence imply that a user can maintain a valid access token "forever" as long as they remain active and the token is not revoked by an admin or password reset action?
"Refresh tokens are valid for 90 days, and with continuous use, they can be valid until revoked."

Is it technically possible to configure a token to never expire as long as there is ongoing activity?
Is it technically possible for a user to never have to login again as long as there is ongoing activity?

@msbemba Thank you very much for the contribution and sharing this explanation. @adrwh Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Simplifying cmake

Every time I add or delete a file I forget to change cmake, change it to this and there will be no problem.

The explicit listing of files has been done on purpose
https://github.com/paceholder/nodeeditor/pull/142
Besides, your proposed changes with the variable ${QT_VERSION_MAJOR} does not work for some reason. I do not know whether  it depends on cmake, OS or Qt version. In my case (Qt 6.4, MacOS, cmake 3.25) the variable remains empty after finding the Qt libs.

Thank you for your reply

But there is macos qt5.15.2 in the github ci and the compilation passes

You are right, let me pull you branch to my computer and see what exactly happens

Opps looks like you removed your branch completely

Opps looks like you removed your branch completely

i restore it

Hi @paceholder , One more thing that needs to be fixed , Now in cmake you use FATAL_ERRROR with the extra letter R

Sorry for closing this PR immediately, I see that your changes work now. Dunno what was wrong with my attempts before.
Please have a look: https://github.com/paceholder/nodeeditor/pull/355
I took all your proposed changes, move an MSVC flag and kept the whole list of files


⚠ Bofrost.at has degraded performance

In 95cbd20, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1613 ms

Resolved: Bofrost.at performance has improved in c9cc6d5 after 6 minutes.


Not able to run a custom router-based handler on Ubuntu 22.04

As reported by bitcloudo on Discord.
If running this (awesome) project with Lightbug from Dockerfile, some cryptic errors are thrown, even though the packaging step seems to work. This only seems to happen on Ubuntu?
RUN mkdir -p /opt/conda
RUN wget https://repo.anaconda.com/miniconda/Miniconda3-py311_24.1.2-0-Linux-x86_64.sh -O /opt/conda/miniconda.sh \
    && bash /opt/conda/miniconda.sh -b -p /opt/miniconda

ENV PATH=/opt/miniconda/bin:$PATH
RUN conda init

# Install Modular/Mojo
RUN curl -s https://get.modular.com | sh -
RUN modular auth $sometoken
RUN modular install mojo

ARG MODULAR_HOME="/root/.modular"
ENV MODULAR_HOME=$MODULAR_HOME
ENV PATH="$PATH:$MODULAR_HOME/pkg/packages.modular.com_mojo/bin"

# Compile lightbug repo
# Clone the lightbug repository
RUN git clone --depth 1 --branch latest-build https://github.com/saviorand/lightbug_http.git ./lightbug_http

# Optionally copy other necessary directories if they are not in the Git repository
COPY ./external /external
# Run the mojo package command to create the package. Ensure it is placed in a known directory.
RUN cd /lightbug_http && mojo package lightbug_http -o lightbug_http.mojopkg

# Install app
COPY . /usr/app
WORKDIR /usr/app

# Instead of using COPY, use RUN to move or copy the file within the container
RUN cp /lightbug_http/lightbug_http.mojopkg ./lightbug_http.mojopkg
# Run Battlesnake

I updated to the newly released Mojo 24.3.0 (9882e19d) and retried utilizing the the packaged form of Lightbug within my repo. It now works!!! I used the lightbug directories directly in my repo (old work around), the locally packaged version of lightbug, as well as downloading the latest-build package directly from the lightbug_http repo. All three worked for me locally. I'm assuming the issue had something to do with the Mojo compiler. I still need to try this within docker, but i would assume this issue is now resolved. Thanks again for your efforts here.

Ah, great! Yes, quite a lot of changes in 24.3.0 . I also made some updates on the main branch which fixed some compiler bugs. Glad to hear it's working now. Feel free to re-open if anything comes up.


setup_zone fails for a zone with 3 parts

My domain (samueljsb.co.uk) can't be used through this library because it cuts off the front part and tries to connect with .co.uk (https://github.com/ailionx/cloudflare-ddns/blob/05128d498830909792090764e854e789cec79283/cloudflare_ddns/cloudflare.py#L99)
Why does this method chop up the domain like this?

I'm guessing it's to separate subdomains (www.example.com) from the domain+TLD combination (www.example.com). In my use case (where i wanted to update www.subdomain.example.com, and the function chopper was pulling "subdomain.example.com" instead of "example.com", I fixed it by adding an elif clause for a four-part domain name:
            if len(self.domain.split('.')) == 3:
                domain = self.domain.split('.', 1)[1]
            elif len(self.domain.split('.')) == 4:
                domain = self.domain.split('.', 2)[2]
            else:
                domain = self.domain

In your use-case, I'd imagine you'd want it to have a special case for .co.uk, so that .com domains can continue to work normally, so something like this might work:
            if len(self.domain.split('.')) == 3 and self.domain.split('.', 1)[1] != "co.uk"
                domain = self.domain.split('.', 1)[1]
            else:
                domain = self.domain

If I figure out a way to handle two-level subdomains correctly while also handling two-level TLDs, without hardcoding every single double TLD out there, I'll submit a pull request.

This pull request should fix that issue for both of our use cases:
https://github.com/ailionx/cloudflare-ddns/pull/14

This pull request should fix that problem for both use cases:
https://github.com/ailionx/cloudflare-ddns/pull/14

Looks like #14 has indeed solved this, thanks @unicodepepper !


Hide edit payouts when distribution limit is 0 or non-exist

What does this PR do and why?
To be more in line with token allocations, this PR hides the button when a project has no distribution limit.
Effectively, a user cannot really add distributions without a dist limit so I think it makes sense.
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] I have tested this PR in all supported browsers.
[ ] I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #1807   👈

This comment was auto-generated by Graphite.


TextBox overwrites previous characters

Describe the bug
Selecting text in TextBox with Ctrl+a shortcut and then typing letters results in only one letter appearing.
To Reproduce
Steps to reproduce the behavior:

git clone https://github.com/redox-os/orbtk
cargo clean
cargo run --example widgets --release
Type some text into one of the TextBoxes
Press Ctrl+a
Type some text
Only one character appears at a time and replaces previous one

Expected behavior
Text should appear normally
Desktop (please complete the following information):

OS: Windows 10.0.18363
Version: 0.3.1-alpha3 (newest from repo)

Additional context
Pressing backspace fixes it until next Ctrl+a and typing.
Also pressing backspace changes style of text cursor so that might be a hint.

Fixed.


[backport] Switch current directory in Jenkins tests

Backport of #7834

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit db313805cb3b74d464cfca9596fade704a93dbe2, target branch v6) failed with status FAILURE.


additional features added

add heater mode toggle
add circulation pumps (for information)
add ozon generators (for information)

Codecov Report

Merging #10 (1494af8) into main (7811d18) will increase coverage by 0.22%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
+ Coverage   98.49%   98.71%   +0.22%
==========================================
  Files           3        3
  Lines         266      312      +46
==========================================
+ Hits          262      308      +46
  Misses          4        4

Impacted Files
Coverage Δ

controlmyspa.py
100.00% <100.00%> (ø)

tests/test_controlmyspa.py
97.89% <100.00%> (+0.50%)
:arrow_up:

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 7811d18...1494af8. Read the comment docs.

Hello @Hr-Frodo,
Thanks for your first contribution to this project! I renamed the new properties to circulation_pumps and ozone_generators and added some tests. As soon as this is merged I'll release the new version 1.0.8 of the library to pypi.org.
Cheers and happy holidays,
Aarno


Fix private module readme's links

Currently the links in modules/private/README.org are broken. I propose using absolute links, since they work in forks and locally.
Feel free to use relative links if you want.

Thanks!


fileter list bucket objects based on ranger policies

For the recursive list bucket - currently there is checked only the root resource in ranger so it is posible to see all subdirs. The fix checks in ranger all subdirs.

Codecov Report

Merging #43 into master will increase coverage by 1.04%.
The diff coverage is 98.55%.

@@            Coverage Diff             @@
##           master      #43      +/-   ##
==========================================
+ Coverage   84.67%   85.71%   +1.04%
==========================================
  Files          38       38
  Lines         672      721      +49
  Branches       55       67      +12
==========================================
+ Hits          569      618      +49
  Misses        103      103

Impacted Files
Coverage Δ

...la/com/ing/wbaa/airlock/proxy/data/S3Request.scala
100% <ø> (ø)
:arrow_up:

...ock/proxy/api/ProxyServiceWithListAllBuckets.scala
100% <100%> (ø)
:arrow_up:

.../com/ing/wbaa/airlock/proxy/api/ProxyService.scala
100% <100%> (ø)
:arrow_up:

...k/proxy/provider/AuthorizationProviderRanger.scala
95% <100%> (ø)
:arrow_up:

.../wbaa/airlock/proxy/handler/RequestHandlerS3.scala
100% <100%> (ø)
:arrow_up:

.../airlock/proxy/provider/MessageProviderKafka.scala
87.5% <100%> (ø)
:arrow_up:

...ng/wbaa/airlock/proxy/api/PostRequestActions.scala
75% <66.66%> (ø)
:arrow_up:

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 35658c2...391c543. Read the comment docs.


When entering wrong password, it sometimes doesn't show the proper error message

reported by two users (once on Debian once on OS X)

This was fixed: it was a server-side issue


Add location biaising

From @danyyacoub on April 28, 2017 8:3
Hi, is there a way to add location biasing in this plugin ?
The idea is to add to options the bounds objects that contain south west coordinates and north east coordinates, it already exists in google maps api.
https://developers.google.com/maps/documentation/javascript/reference#AutocompleteService
Thanks,
Copied from original issue: psykolm22/ng2-google-place-autocomplete#17

From @dionsteel on August 23, 2017 1:24
this might have already been taken care of, but it seems like you could put this in the options object same as you would for types, etc like
{
    "types": [ "(regions)" ],
    "componentRestrictions": { "country": "au" },
    "bounds": {
        "east": east longitude,
        "north": north latitude,
        "south": south latitude,
        "west": west latitude
    }
}

could be wrong, haven't tested it, but pretty sure.


[FEATURE REQUEST] Support Parameter formats

Is your feature request related to a problem? Please describe.
Currently ws channel bindings allow you to define some query/header parameters: https://github.com/asyncapi/bindings/tree/master/websockets#channel-binding-object
However this solution lacks flexibility and power in comparison to swagger-2 or openapi-3 parameters. Especially in case of non-primitive parameters - there's no way to define how such parameters should be serialized. Swagger-2 spec has a collectionFormat and openapi-3 - style/explode for that.
Can't it be tackled using specification extensions?
IMO this should be part of the core spec.
Describe the solution you'd like
The best would be to replicate openapi-3 approach to describe parameters using in/style/explode (etc.) properties.
Describe alternatives you've considered
Not sure there're some which would allow the same level of flexibility and consistency.
Additional context
I'm writing a codegen tool based on swagger-2/openapi-3/asyncapi-2 specs which automatically handles connections that's why I need a solution to fully handle parameters.

Issue moved to asyncapi/bindings #8 via ZenHub

This issue belongs to the WebSockets binding, so I've moved it there.


merge from arlac77/template-arlac77-github,arlac77/template-esm-only

package.json

chore(deps): add ^7.3.3 remove ^7.3.2 (devDependencies.c8)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


maplibre support for Qt6 and MSVC 2019

Update maplibre-gl-native to compile using Qt 6.1.1 and MSVC 2019
@ntadej should probably have his PR merged first:
https://github.com/maplibre/maplibre-gl-native/pull/118
Once his is merged, I'll work to resolve any conflicts.

Open issues are:

https://github.com/maplibre/maplibre-gl-native/issues/124
Missing zlib dependency, currently must be resolved externally

Thanks @dpaulat!
I have also been playing with this a bit and I have some additional fixes prepared, namely:

One can use icu support and zlib from Qt
I did some cleanup of the CMake configuration (but can wait for this MR to go in)
I fixed various warnings

With #118 merged, I rebased and resolved conflicts.  This PR is ready for review.

This looks like a great activity @dpaulat & @ntadej. 👍  Thank you very much for looking into this.
Is there a first demo app? Or do you develop that for some particular project of yours?
@petr-pokorny-1 Could you please check the code and provide your comments in a review - once you are back from holiday?
BTW We could then eventually update https://www.maptiler.com/news/2019/04/using-maptiler-maps-inside-qt/ or reuse the older sample app at https://github.com/maptiler/maptiler-qml-demo to demo the functionality - esp. if there is no other demo app.

Thank you!  There is a subproject in this repository called mbgl-qt that provides demo functionality.  I can post a screenshot this evening.

Is there a first demo app? Or do you develop that for some particular project of yours?

I have a hobby project where I already had to patch existing Mapbox implementation and having something standalone and independent from Qt releases would be much more useful.
Also I'm happy to develop the revised version of QML plugin under the MapLibre umbrella. I have a prototype ready but I'm still cleaning up and streamlining the build system (some more MRs incoming). https://github.com/ntadej/qt-geoservices-maplibre

Thank you! There is a subproject in this repository called mbgl-qt that provides demo functionality. I can post a screenshot this evening.

@dpaulat it looks like your changes are causing Android and iOS SDKs fails to compile. It is mostly due to changes you made in constructor/methods signatures. Do you have Android/iOS dev environment to fix these issues?

@dpaulat it looks like your changes are causing Android and iOS SDKs fail to compile. It is mostly due to changes you made in constructor/methods signatures. Do you have Android/iOS dev environment to fix these issues?

I think this commit should fix it: https://github.com/ntadej/maplibre-gl-native/commit/5bedf45eceecfd650046a555f852dae1311f0fcb

I will work to spin up an Android environment and see what happened. Hopefully the Android and iOS errors are the same, since I don't have a suitable environment for that.

From: Petr Pokorny @.>
Sent: Sunday, July 25, 2021 12:09:35 PM
To: maplibre/maplibre-gl-native @.>
Cc: Dan Paulat @.>; Mention @.>
Subject: Re: [maplibre/maplibre-gl-native] maplibre support for Qt6 and MSVC 2019 (#125)
@dpaulathttps://github.com/dpaulat it looks like your changes are causing Android and iOS SDKs fails to compile. It is mostly due to changes you made in constructor/methods signatures. Do you have Android/iOS dev environment to fix these issues?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/maplibre/maplibre-gl-native/pull/125#issuecomment-886231326, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAWAXTOR7765CH4OCLD7TWDTZRAM7ANCNFSM5AFUJECA.

Even better. I will get this cherry picked when I get back to my computer.

From: Tadej Novak @.>
Sent: Sunday, July 25, 2021 12:11:29 PM
To: maplibre/maplibre-gl-native @.>
Cc: Dan Paulat @.>; Mention @.>
Subject: Re: [maplibre/maplibre-gl-native] maplibre support for Qt6 and MSVC 2019 (#125)
@dpaulathttps://github.com/dpaulat it looks like your changes are causing Android and iOS SDKs fail to compile. It is mostly due to changes you made in constructor/methods signatures. Do you have Android/iOS dev environment to fix these issues?
I think this commit should fix it: @.***https://github.com/ntadej/maplibre-gl-native/commit/5bedf45eceecfd650046a555f852dae1311f0fcb
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/maplibre/maplibre-gl-native/pull/125#issuecomment-886231558, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAWAXTO2KJ5KPLNAQ4WDSJDTZRAUDANCNFSM5AFUJECA.

@ntadej @dpaulat Great - thanks guys!

Any chance to write CI workflow for QT and Windows (similar to those in /.github/workflows) ?

I have something prepared for Mac, android and windows. Plan to open a MR when my other changes will be merged.
I will be on vacation now for two weeks so after this one is merged, I will probably just open one or two MRs.

Any other actions for this PR prior to merge?  Thanks!

This was a lot of work!  Thanks for contributing to MapLibre & FOSS4G.  Looking forward to the CI workflows for GitHub as well.   I reviewed the iOS build for extra warnings (there were none).


Auto reconnect when internet connection unstable

If internet disconnects on an unstable network, the connection would not resume even after network reconnection. It would be nice to have such 'auto reconnect' feature.

There's an event 'disconnect' on TalkClient
const client: TalkClient = new TalkClient('client');
client.on('disconnect', () => {});

There's an event 'disconnect' as @delta-kor said. From v2.1 #25 relogin method in TalkClient can be used to refresh session if user didn't logout manually.


actually fix tags on Copy sentry.properties so it gets run on update-…

…config this time

gonna go ahead and merge this without review


[discuss] : any suggestions to help me diagnose whether this result is as expected?

As the title says, I suspect there is a memory leak in our database, so I want to diagnose it. but I don't see the stack trace for the database code

grp_and_heap.zip

"finalized" indicates that an object has been set with a finalizer using runtime.SetFinalizer. Since the finalizer cannot determine the type, it cannot record the original type information. If there are a large number of finalized objects, it is possible that the finalizer objects have circular references. You could check if this is the case.
See https://github.com/golang/go/blob/master/src/runtime/mfinal.go#L341

Yes, we do use runtime.SetFinalizer in several places, for example, to close channels or do other detection mechanisms.
I didn't find a direct API (e.g. something like runtime.NumberFinalizer), but I can try to add logs to locate the problem, and it seems to be OK.

Previously we faced memory leaks, where the heap could only be seen as a leak, but not who it was referenced by. This tool would be useful for my work, and I'm also interested in program analysis (but I know nothing about this field right now), is there anything we can do to participate in the development of this library?

Welcome to participate in the development of the tool to make it more user-friendly. You can start by getting a general understanding of the principles and the overall implementation. I have two tasks for you to try, which will help you become more familiar with the development.
The first task is to enhance the current output of the node names. https://github.com/cloudwego/goref/issues/5
The second task is related to the pointer mask. https://github.com/cloudwego/goref/issues/6

Okay, I'll try it when I have time.
I'll try it out when I have time. Also I'd like to suggest that we add some little examples like best practice or quick start 101, I realized that we do have a memory leak. I was using the library the wrong way, running it during runtime.
If I wanted to see the call relationship of the memory leak, I should have waited for the load to run and let it idle for say an hour, and then grabbed the information.
heap :

grp :

As you can see at this point there are indeed some call relationships that belong to our code, but I'm not sure if this information has anything to do with the memory leak.

According to the grf flame graph you provided, the issue primarily arises from memory management of finalized objects. If these objects cannot be released even after being idle for a long time, it is highly likely that runtime.SetFinalizer function has been misused.

I'll try it out when I have time. Also I'd like to suggest that we add some little examples like best practice or quick start 101, I realized that we do have a memory leak.

Good idea, I'll mark it as TODO.


Copied to clipboard message covers the code

Reported by goldmann on 28 Nov 2014 13:36 UTC
When we have many tokens that fill the entire screen on a phone, when we click to get the code generated on the last one on tje screen, the "copied to clipboard" message hides the code for a couple of seconds.
I should be able to read the code immediately.

Comment by tyfox on 8 May 2016 23:59 UTC
This has also been bugging me for a while.
I'm not familiar with android dev nor the codebase of freeotp but I'm proposing a patch (attached).
This should display the "Copied to clipboard" toast on the top half of the screen if the code is in the bottom half.

Closing as outdated, the latest version does not use a Copied to clipboard toast mesage.


allow_module_overrides doesn't work with AsyncBuilder

Tried using the allow_module_overrides with an AsyncBuilder. It sets the param on the builder, but it does not get passed in when the AsyncDriver (and thus the Driver) is initialized:

https://github.com/DAGWorks-Inc/hamilton/blob/main/hamilton/async_driver.py#L490-L495
https://github.com/DAGWorks-Inc/hamilton/blob/main/hamilton/async_driver.py#L232-L246

I think the arg just needs to be passed down from AsyncBuilder -> AsyncDriver -> Driver.
Example:
builder = (
     AsyncBuilder()
    .with_config(config)
    .with_modules(*self.included_modules)
    .allow_module_overrides()
    .with_adapters(self.result_builder)
)
driver = await builder.build()

Working on a PR now


GTMSessionFetcher/Core dependency conflict between Firebase/Auth and GoogleMLKit/LanguageID with Firebase 10.0.0

Description
Attempting a pod update causes Firebase/Analytics and Auth to be downgraded from 9.6.0 to 3.6.0, and GoogleUtilities from 7.8.0 to 1.3.2 leading to a compile error related to a GoogleUtilities header.
I narrowed the problem down to a conflict between Firebase/Auth and GoogleMLKit/LanguageID. If I remove Firebase/Auth the rest of the pods update correctly. If I try to install it after the other pods are updated the errors in 'Relevant Log Output' are shown.
It seems that the conflict is between the accepted versions of GTMSessionFetcher/Core in Firebase and GoogleMLKit
Reproducing the issue
No response
Firebase SDK Version
10.0.0
Xcode Version
14.0.1
Installation Method
CocoaPods
Firebase Product(s)
Analytics, Authentication, Crashlytics, DynamicLinks, ML Model Downloader, Messaging, Remote Config
Targeted Platforms
iOS
Relevant Log Output
[!] CocoaPods could not find compatible versions for pod "Firebase/Analytics":
  In snapshot (Podfile.lock):
    Firebase/Analytics (= 10.0.0)

  In Podfile:
    Firebase/Analytics

    Firebase/Auth was resolved to 3.2.0, which depends on
      Firebase/Analytics (= 3.2.0)

CocoaPods could not find compatible versions for pod "Firebase/Core":
  In snapshot (Podfile.lock):
    Firebase/Core (= 10.0.0)

  In Podfile:
    Firebase/Analytics was resolved to 10.0.0, which depends on
      Firebase/Core (= 10.0.0)

    Firebase/Auth was resolved to 3.7.0, which depends on
      Firebase/Core (= 3.7.0)

CocoaPods could not find compatible versions for pod "Firebase/CoreOnly":
  In snapshot (Podfile.lock):
    Firebase/CoreOnly (= 10.0.0)

  In Podfile:
    Firebase/Auth was resolved to 5.0.0, which depends on
      Firebase/CoreOnly (= 5.0.0)

    Firebase/RemoteConfig was resolved to 10.0.0, which depends on
      Firebase/CoreOnly (= 10.0.0)

CocoaPods could not find compatible versions for pod "GTMSessionFetcher/Core":
  In snapshot (Podfile.lock):
    GTMSessionFetcher/Core (< 3.0, = 1.7.2, >= 1.5, ~> 1.1)

  In Podfile:
    Firebase/Auth was resolved to 10.0.0, which depends on
      FirebaseAuth (~> 10.0.0) was resolved to 10.0.0, which depends on
        GTMSessionFetcher/Core (~> 2.1)

    GoogleAPIClientForREST/Drive was resolved to 2.0.1, which depends on
      GTMSessionFetcher/Full (< 3.0, >= 1.6.1) was resolved to 1.7.2, which depends on
        GTMSessionFetcher/Core (= 1.7.2)

    GoogleMLKit/LanguageID was resolved to 3.2.0, which depends on
      MLKitLanguageID (~> 3.2.0) was resolved to 3.2.0, which depends on
        MLKitNaturalLanguage (~> 4.2) was resolved to 4.2.0, which depends on
          GTMSessionFetcher/Core (~> 1.1)

    GoogleSignIn (~> 5.0.2) was resolved to 5.0.2, which depends on
      GTMAppAuth (~> 1.0) was resolved to 1.3.1, which depends on
        GTMSessionFetcher/Core (< 3.0, >= 1.5)

If using Swift Package Manager, the project's Package.resolved
No response
If using CocoaPods, the project's Podfile.lock
No response

When could we expect the update?

How is this done? I'm very confused

Sorry about the trouble. The compatible MLKit update for Firebase 10 is still in progress.
In the meantime, a workaround would be to add a version specifier to the Podfile like:
pod 'FirebaseAuth', '>= 9.6.0'
This will install at least Firebase 9.6.0 along with a compatible MLKit.

Cannot get this issue fixed

Change all of the Firebase versions to 10.6 and I don't see GoogleMLKit in the Podfile.

Facing the same issue,
I am not well familier with the iOS, so don't really know wha to do and how can we fix it!

See here for workaround


QuotedStringTokenizer to ignore "escaped" delimiters

I am trying to parse a csv that uses double quotes as a delimiter of text fields.
When a column includes a literal double quote, it is escaped with a repetition of the double-quote itself.
For example:
Name, Age, Description
"Robert, Willis", 36, "All-around nice guy who always says hi"
"Michael, Chester", 24, "Also goes by ""Mike"", among friends that is"
The idea is that the parsed description column should include the quotes  in the result
-> Also goes by "Mike", among friends that is"
If using the QuotedStringTokenizer, these text fields will fail to parse correctly.
I've been trying to modify the regex inside the class QST, but I'm afraid that I'm not that good at regexing.
Could you maybe share some tips on how to parse escaped/doubled delimiter symbols?

Just reading more into this, now I see that the escaping double quotes with an additional double quote is what's suggested in the IETF RFC4180
https://tools.ietf.org/html/rfc4180#section-2
So it would definitely be helpful to have this functionality as part of the standard QuotedStringTokenizer behaviour.

Hi Jaume,
Thanks for your valuable informations! :+1:
This week is a little packed with work. I will look into it this weekend and come up with something. :-)
Thanks again, Philipp.

Here is a great discussion on parsing the data:

http://programmers.stackexchange.com/questions/166454/can-the-csv-format-be-defined-by-a-regex

Apache Commons-CSV goes great lengths to support an RFC4180-compliant parser. Probably a Regular Expression is not sufficient, I will investigate it further. The Apache Commons-CSV Lexer is interesting anyway:

https://github.com/apache/commons-csv/blob/trunk/src/main/java/org/apache/commons/csv/Lexer.java

wow, thanks very much for getting on top of this so quickly.
I've come across some of the same SO answers suggesting that the regex is a rather tough battle to fight for this.
I'll check out the Apache Commons-CSV, that might have some interesting insight.

I would start writing a simplified version of the Lexer this weekend. I don't think a Regular Expression is sufficient (plus I am not good with Regular Expression either... the one in the Tokenizer was a mix of 5 SO posts 😳).
We will see how it fits into the existing processing pipeline, I hope it works without too many changes to the core.

I have created a new branch rfc4180, which will deal with this Issue.

I have added an initial version, but it still resembles the Apache Parser too much and I don't think it is fully RFC4180-compliant. The Unit Test with your Test Data works correctly, but I have yet to add some corner cases and I didn't look into UTF8 characters yet.

Added a reworked RFC4180 parser as of commit cdd4fd0, which makes no use of Apache Commons code. The Code is also simpler to read. It will be merged to the trunk as soon as I have written some more test cases.

Hi Philipp, thanks so much for the quick turnaround.
what an effort!
I've pulled the new branch and tried to get my csv samples through the parser, but I'm finding  some issues with columns reportedly being "Out of Range".
Is this how you would expect me to invoke the new functionality?
var csvParserOptions = new CsvParserOptions(skipHeader: true, commentCharacter: string.Empty,
tokenizer: new RFC4180Tokenizer(new Options('"', '\', ',')));
thanks again.
ps: I could also put together a sample test file to help out with debugging.

Hi Jaume,
Could you send me some sample data, that is failing for you? So I can write
unit tests and fix the eventual bugs.
Thanks!
On 27 Jun 2016 12:53, "Jaume Sancho"<EMAIL_ADDRESS>wrote:

Hi Philipp, thanks so much for the quick turnaround.
what an effort!
I've pulled the new branch and tried to get my csv samples through the
parser, but I'm finding some issues with columns reportedly being "Out of
Range".
Is this how you would expect me to invoke the new functionality?
var csvParserOptions = new CsvParserOptions(skipHeader: true,
commentCharacter: string.Empty,
tokenizer: new RFC4180Tokenizer(new Options('"', '', ',')));
thanks again.
ps: I could also put together a sample test file to help out with
debugging.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bytefish/TinyCsvParser/issues/9#issuecomment-228714030,
or mute the thread
https://github.com/notifications/unsubscribe/AAOnDQFpRCF5JArTQ_QoIbcMUTP_7JqAks5qP6usgaJpZM4I75GB
.

Absolutely, let me get a simple project together, so you can test it out yourself.
Thanks a million mate

Hi Philipp,
Finally managed to get to the end of this.
The issue is that the new RFC4180Tokenizer is that doesn't like empty fields.
Whenever a column is empty, the parser will throw a "Column X Out of Range", with "x" being the column following the empty one.
For example.
Name, Age, Description
"Robert, Willis", 36, "All-around nice guy who always says hi"
"Robert, Willliamson", , "All-around nice guy who always says hi"
"Robert, Wilkinson", , "All-around nice guy who always says hi"
Will throw a "Column 2 out of range" when trying to parse the 3rd row, as it has no age value.
This works as expected when using QuotedStringTokenizer, i.e. the value is left empty.
thanks

Thanks for the sample data. I will look into it next weekend.

amazing, thanks again :)

Hi Jaume,
I have fixed the RFC4180Tokenizer, added a Test Case and merged it into the master.
Here is an example for using it:
// Copyright (c) Philipp Wagner. All rights reserved.
// Licensed under the MIT license. See LICENSE file in the project root for full license information.

using NUnit.Framework;
using System;
using System.Linq;
using System.Text;
using TinyCsvParser.Mapping;
using TinyCsvParser.Tokenizer.RFC4180;

namespace TinyCsvParser.Test.Tokenizer
{
	[TestFixture]
	public class Rfc4180TokenizerTest
	{
		private class SampleEntity
		{
			public string Name { get; set; }

			public int? Age { get; set; }

			public string Description { get; set; }
		}

		private class SampleEntityMapping : CsvMapping<SampleEntity>
		{
			public SampleEntityMapping()
			{
				MapProperty(0, x => x.Name);
				MapProperty(1, x => x.Age);
				MapProperty(2, x => x.Description);
			}
		}

		[Test]
		public void RFC4180_CsvParser_Integration_Test()
		{
			// Use a " as Quote Character, a \\ as Escape Character and a , as Delimiter.
			var options = new Options('"', '\\', ',');

			// Initialize the Rfc4180 Tokenizer:
			var tokenizer = new RFC4180Tokenizer(options);

			// Now Build the Parser:
			CsvParserOptions csvParserOptions = new CsvParserOptions(true, tokenizer);
			SampleEntityMapping csvMapper = new SampleEntityMapping();
			CsvParser<SampleEntity> csvParser = new CsvParser<SampleEntity>(csvParserOptions, csvMapper);

			var stringBuilder = new StringBuilder()
				.AppendLine("Name, Age, Description")
				.AppendLine("\"Michael, Chester\",24,\"Also goes by \"\"Mike\"\", among friends that is\"")
				.AppendLine("\"Robert, Willliamson\", , \"All-around nice guy who always says hi\"");

			// Define the NewLine Character to split at:
			CsvReaderOptions csvReaderOptions = new CsvReaderOptions(new[] { Environment.NewLine });

			var result = csvParser
				.ReadFromString(csvReaderOptions, stringBuilder.ToString())
				.ToList();

			Assert.AreEqual(2, result.Count);

			Assert.AreEqual(true, result.All(x => x.IsValid));

			Assert.AreEqual("Michael, Chester", result[0].Result.Name);
			Assert.AreEqual(24, result[0].Result.Age);
			Assert.AreEqual("Also goes by \"Mike\", among friends that is", result[0].Result.Description);

			Assert.AreEqual("Robert, Willliamson", result[1].Result.Name);
			Assert.AreEqual(false, result[1].Result.Age.HasValue);
			Assert.AreEqual("All-around nice guy who always says hi", result[1].Result.Description);
		}
	}
}

Please let me know, if it resolves the problems in your current project.

No, thank you for all your hard work!
We're almost there, but there is still one remaining issue to parse the files that I'm trying to import.
If the last column of a csv row is empty the "column out of range error" will still get thrown.
Said the other way around, empty fields are successfully ignored, unless it's the last column is the empty one.
 I should really flex my skills writing more complex Lexers. :-)

this is it! :dancers:
With this release all our files are getting through without issue.
Thanks again, especially for being such a good sport, and getting everything sorted so quickly.
ps: For now, I'm repackaging TinyCsvParser in my own private nuget feed.
Could you let us know when you are planning to make a new nuget release?

Great! Thanks for the Feedback. 😎 I will create a new NuGet package this
evening and close this issue. 🙌
On 4 Jul 2016 13:46, "Jaume Sancho"<EMAIL_ADDRESS>wrote:

this is it! 👯
With this release all our files are getting through without issue.
Thanks again, especially for being such a good sport, and getting
everything sorted so quickly.
ps: For now, I'm repackaging TinyCsvParser in my own private nuget feed.
Could you let us know when you are planning to make a new nuget release?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bytefish/TinyCsvParser/issues/9#issuecomment-230272395,
or mute the thread
https://github.com/notifications/unsubscribe/AAOnDSrH8C2L2sxFwexmwE2_aOMCPnnlks5qSPKNgaJpZM4I75GB
.

I have uploaded Version 1.3 to NuGet:

https://www.nuget.org/packages/TinyCsvParser/1.3.0

I am closing the ticket. Please let me know and reopen, if there are problems with the release.


fix a typo in routes (case wrong)

And I also updated webconfig files and package.json to use postcss-loader rather than the deprecated autoprefixer. (and fixed the production build issue I left last time.)
It was the capitalization change that prevented me from running on ubuntu 15.10, though, the other was just a warning.
Nice boilerplate, thanks!

@derekja Thanks for providing your improvements Derek. Once i review them and align with the boilerplate's idea, will include your changes.

Integrated just some bits of suggestions.
Don't see the point of integrating subdivide or marktex. I wanna keep my boilerplate lean and clean.

oh! oh no, I hope I haven't been pushing those bits towards you. Sorry, I'm
playing with some other new things in there. I though when I made a pull
request it made a snapshot at the time I requested. Sorry about that! I'll
pull request from a new branch in the future. Thanks,
Derek
On Tue, Mar 22, 2016 at 3:14 PM, Nick S. Plekhanov<EMAIL_ADDRESS>
wrote:

Integrated just some bits of suggestions.
Don't see the point of integrating subdivide or marktex. I wanna keep my
boilerplate lean and clean.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/nicksp/redux-webpack-es6-boilerplate/pull/4#issuecomment-200055154

np Derek


Fix/expansion panel list icon

Added parameter to change expansion icon of expansion panel
Added parameter to show or hide expansion panel icon

Issues

Fixed compulsory icon that comes with ExpansionPaneList

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

Thanks for your contribution. This seems like a reasonable change, but we will need a test before we can land any change.

@funyin Thanks for your contribution. I'm going to close this PR since it doesn't have a test and so can't land in its current state, but please don't hesitate to submit it again when you are ready! If you need help writing tests please don't hesitate to reach out to us on the Discord (link in the contributing guidelines, link to that is below), especially the #hackers-new channel.


express probe is not providing data when express.Router() method is used

acmeair uses express.Router() and the express probe needs to cater for this to work

the same thing happens , some correcion for this problem.
try a hook but I did not work well

Hi,
If I use 'app.get(path, callback)' method for creating route paths, the express probe gets properly created and works just fine. But, in our application code, we are using 'app.route(path).all(callback).get(callback)' method to setup our express route paths. Appmetrics is not able to place express probes on routes created this way. Have you provided any solution yet to setup probes for app.route ?

closing as we have removed the express probe #390 #404


!np profile link includes parenthesis

The !np command's link to the user profile includes a parenthesis in the URL.

it kinda is a twitchchat-problem. There is nothing in the code that says "this is a url". So the only solution would be to not put the url in parenthesis or something...

ok, we could do [ and ] (how are these things called in english again? xd) instead of parenthesis. Just checked and twitchchat doesnt add them to the clickable url


FIX "GET /favicon.ico HTTP/1.1" 404 Not Found

Hi @Segelzwerg
I was wondering wha this issue is about?
Thanks


Is there any descriptive explanation of those examples in uClient repo?

Hi,
I am trying to read through all the examples of uClient to find anything useful for my design. However, I cannot find any descriptive explanation of those examples. Any pointers will be appreciated?
Thank you
V

There are no more explanation for those examples than the code and the XRCE documentation: https://micro-xrce-dds.docs.eprosima.com/en/latest/


[FLINK-18839][docs] Add documentation about how to use catalog in Python Table API

What is the purpose of the change
This pull request add documentation about how to use catalog in Python Table API
Verifying this change
This change is a documentation rework without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

@WeiZhong94 Thanks for the review. I'll address the comments when merging the PR.


Reland "Test child isolates are terminated when root is shutdown" after test fix.

This patch was reverted in https://github.com/flutter/engine/pull/13067 due to LUCI failures.

I'm reworking the test a bit I think the rolled back version wasnt full testing what we wanted it to.


chore: Refactor preview-src/index.ts for better readability and maintainability

@microsoft-github-policy-service agree

Not an improvement


[CON-258] Improve test coverage

Validations

[X] I believe this is a way to improve. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that requests the same enhancement

Problem
Continue could use more tests. It's a piece of software that is front-and-center in the developer experience, so every bit of reliability we can add is a big deal. UI changes often, but core functionality would ideally have high test coverage.
A (not necessarily comprehensive) list of some functionality that should be tested:

[ ] Codebase indexing

[x] walkDir
[ ] getComputeAddDeleteRemove <- here is where we decide which files need to be re-computed, re-labelled, removed, or deleted. It's how we decide to use progress from other branches, or to re-compute. There should be a suite of tests in which 1) a codebase is indexed, 2) the branch is switched with some differing files, 3) we check that only the changed files are actually marked as needing to be re-indexed
[ ] each of the CodebaseIndex classes should have a high-level test that calls its update function with some series of small changes, and checks between each that its databases are in the expected state
[ ] Started for a couple of languages: chunking tests for each of the supported code languages (to validate that we get top-level classes and functions, and that we correctly collapse large classes, and any other sensible situations that we should expect to capture). In every case, we should make sure that it's impossible to generate a chunk with size > MAX_CHUNK_SIZE
[ ] markdown chunker

[ ] Docs crawling: we should select a representative site from the most common 5+ types of docs sites (e.g. docusaurus, README, etc...). These tests should be defined declaratively with a SiteIndexingConfig mapped to a list of expected pages that should all be found by the crawler function
[ ] All of the embeddings providers, rerankers, and LLMs should be tested to return expected responses. This work is already beginning in the openai-adapters package, so the real work to do here is incrementally migrate over all of our LLM providers to use the openai-adapters package. As we do this it would probably be prudent still to test the LLM classes themselves
[ ] Test the renderPrompt function to validate that various .prompt files are rendered as expected
[ ] High-level tests for the HistoryManager class
[ ] Quick validation that logging tokens and then querying the database here returns the expected change
[ ] While not every context provider might have a test, we should at least try to set up a system that makes it easy for the authors of context providers to write tests. This would begin with a function (ContextProviderDescription, query, fullInput) -> Promise<ContextItem[]>, so that it's possible to get the output of getContextItems without needing to repeat all of the set up work of injecting ContextProviderExtras
[ ] Many context providers aren't doing much more than requesting from one of the IDE methods, like getDiff. We should test these fairly thoroughly in each IDE. For VS Code, those tests are started here
[ ] Autocomplete

[ ] Stream modifiers: each of the functions here and here is straightforward to test: the test can be declared as expected (input, output) string pairs, these pairs just being converted to/from async generators
[ ] LLMs often like to repeat themselves, and this function is our last line of defense to filter out these bad completions. This test should just be a collection of input strings that we find in real usage, which we want to improve the implementation to filter out
[ ] Tests to validate the construction of prompt templates

Solution
No response
CON-258

In addition to all these I think a simple E2E test that runs in CI as part of the pre-release/release pipeline would be useful, e.g. something simple using the approaches outline here: https://code.visualstudio.com/api/working-with-extensions/testing-extension

Are these good issues to start doing? I don't want to end up duplicating work.

Hi @nmcostello , we would definitely appreciate some help ticking off items on this list! The only one I'd say to avoid is the docs crawling work since someone else will be picking that up soon.
Let us know which, if any, you end up starting 👍


[Feature Request] Array shuffle and reflect function

Vector Operator node in visual shader editor doesn't have reflect function.
Built-in shuffle function for array would be useful. Something like this:
func ShuffleArray(array):
for i in array:
var index = randi() % (i+1)
var a = array[index]
array[index] = array[i]
array[i] = a

The shuffle part is a duplicate of #2576 which has more input from the devs.
Please open a separate issue for the reflect function if you still think it's needed and if there isn't an issue about it already.


Plaintext mode

This could be useful in a few ways:

pasting in/out as latex
manipulating expressions with fragile functions
conditionally hiding large lists

Supporting the ability to toggle single expressions can help with long list renders slowing down the graph; this could be triggered automatically for massive latex strings
Suggested by @adasba#0178 in the Discord

This could be implemented as a drop-in replacement for the MathQuill editor view. Instead of being opt-in, automatically truncate the latex for long strings, and add an option to opt-out of the plaintext mode.

Resolved by #373


Update IncomingMessage.php

Return in wrong place
Is the pull request based off the lastest version?
What features have you added?
What bugs did you fix?
get attachments bugfix
Is your code valid PSR-2?

Has anything in your pull request already been fixed?
Anything else?
Optional things below
[] Added your name to the credits
[] Add a update name below:

Thanks!!


Add link to "who's using" edit page in README

See https://github.com/unibeautify/website/edit/master/website/data/users.yml and https://unibeautify.com/users.html

@szeck87 false positive from issue-complete bot 😉

I do have an unusual story about this.  About 2 months after I started working at the bank (early fall 2017), my current civilian employer, a coworker scolded me when they saw me using VS Code.  Everybody should be using Atom because Atom Beautify is the best thing since sliced bread.


Add Storage Qiniu

This adds a storage: "Qiniu".
As you kwon S3 is perfect,but unavailable in China, so we only find other solutions instead of it. Qiniu is one of most popular cloud storage services in China.

+1

Hi @tombruijn, I really need this future. I'm happy to help if you have questions.

Looks decent in my opinion, but the regular ::Qiniu.establish_connection looks a bit unnerving. Might be more readable and maintainable to have this "external" class method aliased/proxied inside Storage::Qiniu class.

@tomash Thanks for your advice. I have already extracted #config_credentials method and pushed it.

@tombruijn Any advice ?

Hi @bastengao, thanks for the PR!
The reason I've not answered this in a while is because I wasn't not entirely sure about merging it. I can't really tell what Qiniu does, as I can't read Chinese.
Not entirely sure with the additional secondary dependencies either. I know, it's a bit late to complain about that now 😓  So disregard that
I think that if rebased on the latest master branch we can merge this. So please rebase it so that I can merge it 👍

@tombruijn Thanks for your response. I will do it later. 😄

@tombruijn Rebased it.

Great! Merged it 😄


HTTP client

Codecov Report

Merging #940 (7cf538e) into main (9cd44c8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #940   +/-   ##
=======================================
  Coverage   84.30%   84.30%
=======================================
  Files         388      388
  Lines       40847    40847
=======================================
  Hits        34436    34436
  Misses       6411     6411

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 9cd44c8...7cf538e. Read the comment docs.


Fix AppDesigner/Editors resource classes

Main changes are:

Removed unneeded calls to GetString with "zero args" - this was just returning the same string
Reduced three "resource" classes to 1.

Fixes: #1056
I made sure not to change the name of the resource within the assembly (via the use of the LogicalName metadata on the resource) - to avoid breaking/changing localization.
Found: https://github.com/dotnet/roslyn-project-system/issues/1058.

tag @333fred @basoundr @dotnet/project-system
This is the follow up to https://github.com/dotnet/roslyn-project-system/pull/1052 to give a better name for the resources class.

There should be zero functional changes in this - this is basically a rename of the designer class.


Fix incorrect types passed to Py_BuildValue and PyArg_ParseTupleAndKeywords

Always using i is broken, as that is 4 bytes, but Py_ssize_t and Py_uintptr_t are 8 bytes.
On little endian machines, this is usually fine, since the first 4 bytes (of 4-byte or 8-byte integers) will be the same, but on big-endian machines, a 4-byte integer corresponds with the last 4 bytes of the 8-byte integer
For small numbers on big-endian machines, it is thus likely that the result will be 0. If the value is over 32 bits, then the result will be wrong on little-endian machines as well.
Fixes #142

Here's a downstream build with this PR applied that works on all (little- and big-endian) architectures: https://src.fedoraproject.org/rpms/python-pyahocorasick/pull-request/3

There seems to be one more oddity at:
https://github.com/WojciechMula/pyahocorasick/blob/88256f54d4fb181e28bc9eaf738805b22ffecf0b/src/AutomatonItemsIter.c#L254-L262
vs:
https://github.com/WojciechMula/pyahocorasick/blob/88256f54d4fb181e28bc9eaf738805b22ffecf0b/src/common.h#L49-L66
where if AHOCORASICK_UNICODE is not defined, it states that it is const char * to Python, but the TRIE_LETTER_TYPE is uint16_t. I'm not sure if it's the former or latter that is wrong.


URL Validation is failing for some valid URLs

Examples of packages/URLs that are failing are...

Package: https://chocolatey.org/packages/jtalert/2.15.6
Nuspec: https://github.com/chtof/chocolatey-packages/blob/master/automatic/jtalert/jtalert.nuspec
"In the ReleaseNotes element in the nuspec file an invalid Url is found. Please correct this More...The LicenseUrl element in the nuspec file should be a valid Url. Please correct this More...The ProjectUrl element in the nuspec file should be a valid Url. Please correct this More..."
URLs:
ReleaseNotes: https://hamapps.com/php/notes.php?file=jtalert
LicenseUrl: https://hamapps.com/php/license.php
ProjectUrl: http://hamapps.com/

The validator believes that the bugTrackerUrl isn't a valid url in the package activepresenter
The url specified is valid though (both from web browser, and from powershell): https://talk.atomisystems.com/

Another one that got affected.
The docsUrl on the mpc-hc-clsid2 package is also being detected as invalid.
I assume this is happening because of the cloudflare check that occurs before actually being navigated to the page (fails in powershell, but succeed in web browser).

@AdmiringWorm I am wondering if the mpc-hc-clsid2 is a separate issue.  For that one, the response from CloudFlare is a 503 initially, and then a meta refresh/redirect of the page happens if found to be valid.  That would be tricky to validate in the simple check that we are doing.  However, I am wondering if a 503 response in the context of these validations is actually a valid response.  What are your thoughts?

@gep13 503 = Service Unavailable. This could be temporary or more permanent. There is no way to tell if it is correct.

@mkevenaar said...
503 = Service Unavailable. This could be temporary or more permanent. There is no way to tell if it is correct.

The same could be said about a valid response 😄
The point I was trying to make was that it wasn't a 404 not found, so "something" is there, it just isn't working correctly just now.

@chtof @AdmiringWorm I have found at least "part" of the problem.  Going to raise a PR to address this just now...

@gep13 said...
The point I was trying to make was that it wasn't a 404 not found, so "something" is there, it just isn't working correctly just now.

What if we would allow all but 4xx status codes?

@mkevenaar said...
What if we would allow all but 4xx status codes?

Yip, that is what I am thinking, but again, I think this might be separate issue.  Can I ask you to raise one for discussion?

@chtof @AdmiringWorm @mkevenaar the PR in question is here: https://github.com/chocolatey/package-validator/pull/201
This "fixes" the issue for the URL that @AdmiringWorm provided, i.e. https://talk.atomisystems.com/, however, the URL's that @chtof provided are still causing an error to be returned...

I am wondering if something like the Agent needs to be set in order for the request to go through.  Going to continue to investigate this.

@chtof @AdmiringWorm @mkevenaar I have added another commit to the PR that I created.  This adds the UserAgent header to the requests, which makes the examples that @chtof provided work during testing.

@AdmiringWorm I am wondering if the mpc-hc-clsid2 is a separate issue. For that one, the response from CloudFlare is a 503 initially, and then a meta refresh/redirect of the page happens if found to be valid. That would be tricky to validate in the simple check that we are doing. However, I am wondering if a 503 response in the context of these validations is actually a valid response. What are your thoughts?

Yeah, maybe it should be considered a separate issue, but not completely sure though.
In general 503 errors should not be considered a valid response, but in cases like these it probably would make sense to allow it but make it a Note for maintainers so they can verify that the url is actually working or not.

@AdmiringWorm the way that the validator rules engine works doesn't allow for "something" to be either a requirement or a note.  It either meets the requirement or it doesn't.  Would need to put some thought into how this would work, if we needed to go that route.

Package: https://chocolatey.org/packages/opera-developer/67.0.3554.0
Nuspec: https://github.com/mkevenaar/chocolatey-packages/blob/master/automatic/opera-developer/opera-developer.nuspec
URLs:
ReleaseNotes: https://blogs.opera.com/desktop/changelog-for-67/#b3541.0

@chtof @AdmiringWorm @mkevenaar I am going to go ahead and close this out just now, as I believe the two main problems in this area have been addressed.  I have some other refactoring to do for URL validation in general, and once that is done, I will deploy the new version of package validator, and re-run validation on the packages that I know have been affected.

@gep13 I have read carefully your previous post. So this post shouldn't help and it's just FYI:
https://chocolatey.org/packages/chirp.install/2020.01.03
In the ReleaseNotes element in the nuspec file an invalid Url is found. Please correct this More...
The DocsUrl element in the nuspec file should be a valid Url. Please correct this More...
The BugTrackerUrl element in the nuspec file should be a valid Url. Please correct this More...
The LicenseUrl element in the nuspec file should be a valid Url. Please correct this More...
The ProjectUrl element in the nuspec file should be a valid Url. Please correct this More...
The ProjectSourceUrl element in the nuspec file should be a valid Url. Please correct this More...

@chtof @AdmiringWorm @mkevenaar with the exception of the activepresenter package, all other packages that have been mentioned in this issue have been re-validated, and are now passing correctly.

@gep13 odd that the activepresenter package is still failing the bugtracker url, as this was one of the urls you added to the test library to ensure it succeed.
Is it possible that somehow the old original url is cached somehow and don't use the latest url?

Could you launch a rerun (checbox not displayed, ony reject) on chirp/chirp.install/chirp.portable (I can't and are still in failed status due to invalid URLs).

@AdmiringWorm said...
Is it possible that somehow the old original url is cached somehow and don't use the latest url?

I have included this URL in one of the new unit tests that I have, and it works fine on my machine.  However, when running on the machine where the package-validator is running, it still fails to make a valid TLS/SSL connection to the site, and therefore fails.  It is "something" missing on that machine I think, but I haven't yet figured out what it is.  It is on my list to track down...

@chtof said...
Could you launch a rerun (checbox not displayed, ony reject) on chirp/chirp.install/chirp.portable (I can't and are still in failed status due to invalid URLs).

This should all be taken care of now, right?

It is "something" missing on that machine I think, but I haven't yet figured out what it is. It is on my list to track down...

.NET 4.5 perhaps not installed on that machine?
That version is minimum required to be installed to allow TLS 1.2 connections (.NET 4.8 for TLS 1.3).

@AdmiringWorm nope, it isn't that 😢 That was the first thing that I checked.  There is something else causing it to fail.


Fix Controller.basePath on windows

See endpoints/endpoints-example#23

Sorry for the huge delay--thanks!


Implementing basic functionality of keychain

Need to implement basic functions of keychain declared in #2.

At this moment it is implemented:

sign command for bitshares
key files search by filename and username

Aslo reflection future from lib/fc has been applied for parsing/serialization pipe commands and parsing key files.
Next step:

key encryption implementing

Today I was explored openssl api to find out how key encryption is implemented in the openssl.
Current status: in progress

Question:
Keychain can handle requests from different users in the OS. Сonsequently, one user can remove key owned by another user.
Is it necessary to provide access control system? If necessary we need to discuss how to properly implement.
I assume on linux we can use real UID to identify users. But it may not be possible on other platforms. Another way is to use user password for authenticate user request. However keychain use cases allow to store keys in unencrypted format.
So, it is needed to discuss this question on the focus group.

File encryption (symmetric) has been realized.
Next step: user identification

Creating abstract interface function that will identify user sending request.
Implementing identify function on window and unix, that idenity user by operating system api.
Modify rpc functions and key file format if needed.

Implementing identify function, brief overview:

unix -- using SO_PEERCRED option
http://man7.org/linux/man-pages/man7/socket.7.html
https://stackoverflow.com/questions/9898961/is-there-a-way-to-get-the-uid-of-the-other-end-of-a-unix-socket-connection
windows

get list of ports and process pid - s
get user ID that own process (process owner)

Is this issue solved?


[CARBONDATA-951] create table ddl can specify a bitmap option and add a new Encoding: BITMAP

create table ddl can specify a bitmap option
add a new Encoding: BITMAP

Build Failed  with Spark 1.6.2, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/1877/

Build Failed  with Spark 1.6.2, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/1926/

Build Failed  with Spark 1.6.2, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/1977/

Build Failed  with Spark 1.6.2, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/1983/

Build Failed  with Spark 1.6.2, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/1984/

Build Failed  with Spark 1.6.2, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/1990/

Build Failed  with Spark 1.6.2, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/1993/

Build Failed  with Spark 1.6.2, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/1994/

Build Failed  with Spark 1.6.2, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2046/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2184/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-2.1/80/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-1.6/51/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2185/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-2.1/81/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-1.6/52/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2189/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-2.1/85/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-1.6/56/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2382/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-1.6/261/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-1.6/290/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2411/


[CGP-389] [Docs] Added a writeup on mutual type-level recursion

I'd prefer for Philip to take a loop first. Should have added the "DO NOT MERGE" label. Well, not too important as these are just docs that can be extended/corrected in a future PR if necessary.

I think it's fine to have it in, since I think it's correct, it's just a question of whether or not we will use it.


Support ReadOnlySpan for IsMatch()

Example:
  public bool IsMatch(ReadOnlySpan<char> input)
    {
        return parsedRegex != null && parsedRegex.IsMatch(input);
    }

Just add that signature in addition to the one taking string.
This reduces my time of scanning 200K files from 1.79 --> 1.57 seconds.
It's because the ReadOnlySpan supports an efficient Slice() operation, so I'm not scanning the part of the path above where the .gitignore file resides.
I'm the author of ViLark, a file chooser for vim, sort of like fzf.
And thanks for this library, it helps a lot.  Although I am getting very slow performance with a wildcard like '*.pyc', I might dig into that if I have time.

Thanks for the suggestion. Feel free to raise a PR, otherwise I will try to get to this tomorrow.
Yeah always good to improve performance. The library uses a bunch of regex, which I tried to follow best practices for, but maybe there can be improvements there.