Vehicle Exit Controllers aren't saving linked seats when duped

What it says on the tin, dupes aren't saving the linked seats and require relinking every paste. Curiously, older vehicles I have with exit controllers still retain their links.

Might be caused by https://github.com/wiremod/wire/commit/b457bc15c95471e9cc842644ceba4184c9bed3a6 will have to test

@Divran

I can look tomorrow

Got it

sweet


question: can this be used for react native?

I'm new to react/native/semantic-ui.
Probably question is dumb. but can this be used for react native?

Unfortunately not.  Our components only generate valid HTML, whereas React native components would generate native UI elements.

@pcompassion would you be interested in creating a react native library for semantic-ui?

@tarang9211 I would

I took a jump to semantic-ui and not looking back. The react-native capability would be something I would like to see as well. However, I wouldn't know where to start. Is there any organised project around this where I would take few pieces to work on ?

@HintikkaKimmo I can start working on it. Give me a few days

@HintikkaKimmo @mmahalwy Keep us posted with the development! It would certainly streamline my development process from web to mobile.

Hey folks, love to see continued interest around this.  Let's move it over to the more updated issue #1000.


Changing loop name doesn't trigger save

Doesn't look like changing the song's name changes the save invalidation in the music tool. Doesn't look like changing the tracks does either.

This has been fixed.


An Op version of #400

Coverage decreased (-0.07%) to 64.085% when pulling 5a5fbea3fdba6314006e857fc5a4eb78a46e3243 on glycerine-auroc into 5fb5944d4a1f0e599e10850235bcdc936cf05460 on master.

Coverage decreased (-0.07%) to 64.085% when pulling 5a5fbea3fdba6314006e857fc5a4eb78a46e3243 on glycerine-auroc into 5fb5944d4a1f0e599e10850235bcdc936cf05460 on master.


Add an async iterater for paging through media items

Make it easier to page through items using an async iterator.

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

Project coverage is 94.20%. Comparing base (6a5d3ea) to head (abaa460).
Report is 27 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #18      +/-   ##
==========================================
+ Coverage   93.72%   94.20%   +0.47%
==========================================
  Files           6        6
  Lines         303      328      +25
==========================================
+ Hits          284      309      +25
  Misses         19       19

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


Field location now saved for quote and project

Quote will now serialize and save field location geometry (just back into json) and on quote approval the field locations will now be written to SQL Server's geography type.  Quote map stage is still just using static values but otherwise the quote can be modified, the map shows in the right way and sql is getting the values.

I'm guessing this geography value is right :)
0xE61000000104050000007C3FF8DD67444340A3A9B7D61E6F5EC02AD9FA9A67444340B0E65EB4126F5EC03C4C8E6D6C444340C7DCE120126F5EC038AA00F46B444340332EA6441E6F5EC07C3FF8DD67444340A3A9B7D61E6F5EC001000000020000000001000000FFFFFFFF0000000003

🤖 Looks right to me 🤖


BUG: Buffer delete command closes Neovim when neo-tree is open

Did you check docs and existing issues?

[X] I have read all the docs.
[X] I have searched the existing issues.
[X] I have searched the existing discussions.

Neovim Version (nvim -v)
NVIM v0.10.1 Build type: Release LuaJIT 2.1.1713484068 Run "nvim -V1 -v" for more info
Operating System / Version
Windows 11 Pro x64
Describe the Bug
I tend to toggle neo-tree open with a keyboard shortcut (F3 from my old CUA days is a hard habit to break). Here's the procedure I follow to evince what seems to me like a bug:

Open nvim with a single file specified on the command line.
Hit F3 to toggle neo-tree (it appears in a new left pane). Open a second file using neo-tree.
Issue the :bd command.

With two files open like that, I would think deleting the buffer that's open in the right pane would leave neo-tree open on the left and the original file from the command line open on the right. What actually happens with close_if_last_window set to true is that Neovim exits completely. And I believe it's neo-tree doing that for two reasons: (1) changing close_if_last_window to false stops it from happening, as does (2) if I toggle the neo-tree pane closed before the :bd command. Serious question: am I wrong in that initial expectation? Seems like this might be a bug.
Screenshots, Traceback
No response
Steps to Reproduce

Open nvim with a single file specified on the command line.
Hit F3 to toggle neo-tree (it appears in a new left pane). Open a second file using neo-tree.
Issue the :bd command.

Expected Behavior
I expected my :bd command to delete the buffer I'm seeing, leaving neo-tree open in the left pane but now showing the second file open in the right pane.
nvim-config-folder-2024-10-11_2012.zip
Your Configuration
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  -- add any other plugins here
}

local neotree_config = {
  "nvim-neo-tree/neo-tree.nvim",
  dependencies = { "MunifTanjim/nui.nvim", "nvim-tree/nvim-web-devicons", "nvim-lua/plenary.nvim" },
  cmd = { "Neotree" },
  keys = {
    { "<Leader>e", "<Cmd>Neotree<CR>" }, -- change or remove this line if relevant.
  },
  opts = {
    -- Your config here
    -- ...
  },
}

table.insert(plugins, neotree_config)
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

-- Sorry, gang, I don't know how to produce a "minimal init.lua to reproduce this issue". I'm attaching an archive of my entire %LOCALAPPDATA%\nvim folder instead.

I am experiencing the same issue. I will continue to investigate and update this thread if I find a solution

Did anyone manage to find a resolution to this?

Not really. I've left close_if_last_window set to false to avoid having Neovim quit unexpectedly. Then I largely stopped using the tree plugin in favor of the Oil plugin and Telescope for opening files instead.

I've tried setting close_if_last_window to false as well, and I still see a problem - albeit a slightly different one. Instead of exiting neovim, closing my current buffer focuses the window on neo-tree instead of other open buffers. I suspect the root cause might be neovim not being able to return to the other buffers when neo-tree is enabled.


Setup instructions missing?

Hi there,
I've installed and set up etherpad locally in the same VM as the one humhub is running on but I can't get it to work. I've followed the instructions but this is what I get so far:

I can't seem to edit the note, all I see is its empty value :-/ What am I missing here?

OK, I think it has to do with having humhub in https and etherpad in regular http. However, I am unable to get the SSL to work. If I try to use https in the url field, I no longer get the connection working:
Current Status: Could not connect to API!

Help?

Nevermind, the problem was that I was using <IP_ADDRESS> and it started working as soon as I changed it to the real IP. Noob.

Where did you change this?

In humhub configuration page, if I remember correctly.

Hi, I've tried everything to make it work but this module is too complicated. Can you help me please.


500 error when add new issue with picture

Gogs version (or commit ref):   <IP_ADDRESS>3
Git version: 2.11.0
Operating system:  Ubuntu 14.04
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[x] SQLite

Can you reproduce the bug at https://try.gogs.io:

[ ] Yes (provide example URL)
[ ] No
[x ] Not relevant

Log gist:
2017/01/10 10:59:05 [D] Session ID: 570a423c60ca3af3
2017/01/10 10:59:05 [D] CSRF Token: IOCcoAPdcvNZRCWdhOAsXGtpStc6MTQ4Mzk1NjYxMDY3MzAxMTMyNA==
2017/01/10 10:59:05 [T] New attachment uploaded: c5e1b80b-b137-4330-879e-3178036ca4d7
[Macaron] 2017-01-10 10:59:05: Completed /issues/attachments 200 OK in 47.389833ms
[Macaron] 2017-01-10 10:59:10: Started POST /emugroup/v6emuwithportal/issues/new for <IP_ADDRESS>
2017/01/10 10:59:10 [D] Session ID: 570a423c60ca3af3
2017/01/10 10:59:10 [D] CSRF Token: IOCcoAPdcvNZRCWdhOAsXGtpStc6MTQ4Mzk1NjYxMDY3MzAxMTMyNA==
2017/01/10 10:59:10 [...outers/repo/issue.go:436 NewIssuePost()] [E] NewIssue: newIssue: getAttachmentsByUUIDs [uuids: [c5e1b80b-b137-4330-879e-3178036ca4d7]]: no such column: uuid
2017/01/10 10:59:10 [D] Template: status/500
[Macaron] 2017-01-10 10:59:10: Completed /emugroup/v6emuwithportal/issues/new 500 Internal Server Error in 29.139349ms

Description
When I create a new issue with a picture (drag or add), get a 500 error from gogs.
Logs above.
...

This is fixed issue in master, make sure you have go get -u -v github.com/go-xorm/xorm when you install from source code.


Update tutorial-archive.md

Minor syntax changes for the Kusto queries tutorial.

@Artur-Mo - we can proceed with this pull request after you've followed the CLA instructions: https://github.com/MicrosoftDocs/dataexplorer-docs/pull/2162#issuecomment-1930514212
#assign: @shsagir
@shsagir

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@Artur-Mo - we can proceed with this pull request after you've followed the CLA instructions: #2162 (comment)
#assign: @shsagir @shsagir

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged" @MicrosoftDocs/public-repo-pr-review-team

Hi, this is a Pull Request for candidate evaluation. Please disregard it. We will close it once the evaluation is complete. Thank you.


Improve debugging output for HTTP2Handler.

Motivation:
Sometimes it's useful to be able to print out objects and see their
internal state. Somehow, HTTP2Handler can't do this.
Modifications:
Add a conformance to HTTP2Handler.
Result:
Awesome debugging output.
 Any objection to me just merging past them>

Fine by me; looks like a warning raised to an error anyway. Can you fix/file an issue against NIO as well?

See apple/swift-nio#1492.


Add GitHub actions for CI and Release processes

Ok, this pull request was created as a draft on on purpose due to:

When a contributor adds a github action, I recall the maintainer must approve it first, berfore it start working.
I need to discuss some details about how do you (@yusukebe) imagine the release process should look like.

I think the fastest approach would be to let gorelease to create the release on GitHub by itself, this is supported by it out of the box. The relase will be triggered when a git tag is pushed, and this will be created as a draft or pre-release so you can manually change anything you like aftewards.
What do you think? Should we go that way or do you have other idea?
And yes, you will need to test the process with a dummy release (ie: v1.1.1 or v1.2.0-pre)
PS: I will clean up the branch history after we agree on how to proceed.

Hey @yusukebe , sorry for the delay. Everything has been sorted out. I tested it the goreleaser configurations against my own fork and it worked as expected; now it is only matter you test it by doing a git tag and pushing it on this repository.

Than you @jossemarGT!I fixed some issues ( 1b67ba235365c69b80538a9d88c1d38fbd737058, #22  ) and

Oh my! :man_facepalming: @yusukebe those 2 configurations totally slip my mind, I was sure those were there. Anyhow, I am happy you were able to sort it out.


[run the installer] TASK failing when attempting to spin up a 4 cluster

fatal: [clientvm.ibb0701ocp4b.internal]: FAILED! => {"ansible_job_id": "494777713481.17175", "changed": true, "cmd": ["openshift-install", "create", "cluster", "--dir=/home/ec2-user/cluster-ibb0701ocp4b"], "delta": "0:36:23.382917", "end": "2019-07-17 19:26:02.715785", "finished": 1, "msg": "non-zero return code", "rc": 1, "start": "2019-07-17 18:49:39.332868", "stderr": "level=info msg=\"Consuming \\\"Install Config\\\" from target directory\"\nlevel=info msg=\"Creating infrastructure resources...\"\nlevel=info msg=\"Waiting up to 30m0s for the Kubernetes API at https://api.cluster-ibb0701ocp4b.ibb0701ocp4b.mg.dog8code.com:6443...\"\nlevel=info msg=\"Make sure ssh-agent is running, env SSH_AUTH_SOCK is set to the ssh-agent's UNIX socket and your private key is added to the agent.\"\nlevel=info msg=\"Use the following commands to gather logs from the cluster\"\nlevel=info msg=\"ssh -A<EMAIL_ADDRESS>'/usr/local/bin/installer-gather.sh <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS>'\"\nlevel=info msg=\"scp core@<IP_ADDRESS>:~/log-bundle.tar.gz .\"\nlevel=fatal msg=\"waiting for Kubernetes API: context deadline exceeded\"", "stderr_lines": ["level=info msg=\"Consuming \\\"Install Config\\\" from target directory\"", "level=info msg=\"Creating infrastructure resources...\"", "level=info msg=\"Waiting up to 30m0s for the Kubernetes API at https://api.cluster-ibb0701ocp4b.ibb0701ocp4b.mg.dog8code.com:6443...\"", "level=info msg=\"Make sure ssh-agent is running, env SSH_AUTH_SOCK is set to the ssh-agent's UNIX socket and your private key is added to the agent.\"", "level=info msg=\"Use the following commands to gather logs from the cluster\"", "level=info msg=\"ssh -A<EMAIL_ADDRESS>'/usr/local/bin/installer-gather.sh <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS>'\"", "level=info msg=\"scp core@<IP_ADDRESS>:~/log-bundle.tar.gz .\"", "level=fatal msg=\"waiting for Kubernetes API: context deadline exceeded\""], "stdout": "", "stdout_lines": []}

@ibolton336 can you please share the .openshift_install.log file on your clientvm instance.
Connecting to clientvm instance :
ssh -i <YOUR_OUTPUT_DIR>/agnostic_<YOUR_GUID>/<GUID>key

Once logged in, find /home/ec2-user/cluster-<YOUR_GUID>/.openshift_install.log file.
YOUR_GUID : guid variable from my_vars.yml file
YOUR_OUTPUT_DIR: output_dir variable from my_vars.yml file

Sorry @pranavgaikwad I already ran the cleanup/destroy script

This looks like an intermittent issue. He was able to launch the cluster the second time he tried. I'm keeping the issue open until further occurrence.

A note that @Danil-Grigorev has been seeing this issue many times when he tries to use agnosticd for 4 envs.

This issue exists, and from initial observation, the issue is that the 4.x deployment does not create compute nodes. No matter which number of nodes is specified in configuration. Tried to deploy an ocp4 many times, (10+ at least with different OCP4 releases) and I'm not sure if I got any success at this point. Probably the problem is in the agnosticd itself, but the issue needs more attention, because some configurations now doesn't work as expected.
Same error message:

I just want to add I have hit this problem 3 times in a row launching a 4.x cluster. The deadline exceeds and the openshift install logs do not provide any useful debugging information it appears.

@Danil-Grigorev @dymurray please let me know if changing the ocp4_token solved the issue for you.

Identified Cause :
Wrong ocp4_token in secret.yml file leads to this issue. The OpenShift Installer downloads images from quay.io during installation. It uses the access token for quay.io present in secret.yml file. That token was either wrong or completely missing. This resulted in the Installer receiving "access denied" errors when trying to download images. This could be confirmed by gathering installer logs as specified in the last two lines of the error message.
To generate installer logs, run this from clientvm instance :
ssh -A core@<ip> '/usr/local/bin/installer-gather.sh <ip1> <ip2> <ip3>'
Secondly, copy those logs to clientvm instance :
scp core@<ip>:~/log-bundle.tar.gz .
See last three lines of error message to get the actual commands.
Extract the log bundle, and read logs from log-bundle/bootstrap/journals/bootkube.log file.
The error message in the log looks like this :
Jul 23 14:05:19 ip-10-0-15-218 bootkube.sh[1416]: unable to pull quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:e63937b00e: unable to pull image: Error determining manifest MIME type for docker://quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:e63937b00e: Error reading manifest sha256:e63937b00e in quay.io/openshift-release-dev/ocp-v4.0-art-dev: unauthorized: access to the requested resource is not authorized

Solution :
Make sure you use the correct ocp4_token in secret.yml file. The token should look like this :
ocp4_token: '{"auths":{"cloud.openshift.com":{"auth":"<SOME_VALUE>","email":"john@doe.com"},"quay.io":{"auth":"<SOME_VALUE>","email":"john@doe.com"},"registry.connect.redhat.com":{"auth":"<SOME_OTHER_VALUE>","email":"john<EMAIL_ADDRESS>

Updating the pull-secret from official page and replacing the full secret in configuration worked for me. Thanks @pranavgaikwad I would say that secret.yml.example is a bit misleading in this specification https://github.com/fusor/mig-agnosticd/blob/master/secret.yml.sample#L15 It would be better to change the line to ocp4_token: 'REPLACE_WITH_PULL_SECRET' to avoid confusion.

@jwmatthews this issue can be closed now.


Research frontend handling reorgs

The frontend might need to adapt for reorgs.
Especially when sending the request tx and when waiting on events like RequestFilled, RequestClaimed, etc.
Timeboxed issue for researching what needs to be done

Based on my research, the current frontend code is prone to errors due to re-org events after we land on Ethereum.
I propose adding the following:

Define a confirmation time (in blocks) per chain which the app consumes in order to determine the "finality" of a transaction
Consume the defined confirmation time by waiting on each transaction execution until the transaction reaches "finality"
When listening on events via filters, make sure to check for the event transaction finality before acting upon.


fix: add Meet Mulik

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[X] All new and existing tests passed.
[X] This PR does not contain plagiarized content.
[X] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Thank you so much! @Panquesito7 @emmalearnscode @Vyvy-vi


BasicFormat doesn't indent statement in if expression correctly

The output of the code
let expr = """
    if a {
        print(1)
    }
    """
let meth = """
    func f(a: Bool) {
        \(raw: expr)
    }
    """ as DeclSyntax
print(meth.formatted())

is
func f(a: Bool) {
    if a {
    print(1)
    }
}

Notice that the print(1) statement is not indented correctly.

Tracked in Apple’s issue tracker as rdar://125290309

This behaves as expected. Using the raw interpolation style, the effective string produced for meth is
func f(a: Bool) {
    if a {
    print(1)
}
}

This matches how raw string interpolation works for normal strings and you can see this if you print meth without formatting it.
BasicFormat now detects that print(1) is already manually indented and will thus not adjust its indentation (the idea here is that you are able to manually indent parts of your source code and BasicFormat should only touch those that aren’t already formatted. It will thus not add another level of indentation to print(1).
To fix this, you can change expr to be an ExprSyntax and not use the raw string interpolation style inside meth. With that you are in the smarter syntax-based string interpolation world, which will add the indentation of the line that contains the string interpolation anchor to the entire node you are inserting, adding 4 spaces to every line in expr.
let expr: ExprSyntax = """
  if a {
      print(1)
  }
  """
let meth = """
  func f(a: Bool) {
      \(expr)
  }
  """ as DeclSyntax
print(meth.formatted())

Good to know. Thank you for the explanation! I guess the general advise then is to work more with syntax objects and syntax builders instead of raw strings to get the full and proper support of the SwiftSyntax infrastructure.


adds badge for zulip chat

I was struggling to find the link to the Zulip chat... I only found it by going to the (also undocumented?) deprecated slack channel. This PR adds a badge to the Zulip chat to the README

This has actually been a somewhat intentional choice @neuromusic, but maybe one we should revisit. There is a reference to the zulip in the core developer guide and I really see the zulip as a place for core developers to do basic coordination around things like agendas and scheduling. I would like to keep as much technical discussion / bug reports / feature requests on the GitHub, and as many user help and support requests on image.sc. I worry having a link to the zulip on the top of the repo only invites more users to use the zulip.
For me, there are two main reasons for this

Posts on image.sc and GitHub are indexed by search engines and so much more discoverable - helping one person on zulip helps one person, but helping one person on image.sc can help many people.
Responding to people in realtime chat applications is a much larger burden on the responder and can lead to many more unpleasant interactions, where there are mismatched expectations on how long a core developer can carry on a conversation for, and what is feasible. I've seen some real burnout and time syncs for maintainers in this context that I want to avoid. Maybe when our community is much larger having more realtime chat support options makes sense, but I think we are far from that right now.

How do @jni @royerloic @tlambert03 feel about this?

I agree with @sofroniewn ...

Oh that's fair to not to distract from user support forums.
The other place I looked was under contributor docs... I didn't realize
there was yet another "core" developer guide. So if you are trying to hide
it, you're most definitely doing a good job 🤣.
I was looking for it to ask about the "hackathon" but ended up getting the
info through other backchannels.
On Thu, Feb 27, 2020, 14:03 Loic Royer<EMAIL_ADDRESS>wrote:

I agree with @sofroniewn https://github.com/sofroniewn ...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/napari/napari/pull/1007?email_source=notifications&email_token=AAJQDL6QBS5MEWXNZJ67SOLRFA2BRA5CNFSM4K5AJENKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENGD7DI#issuecomment-592199565,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJQDL5XRPHS2RWSBDA44HDRFA2BRANCNFSM4K5AJENA
.


Multiple '+' or '-' prefix operators in a row should be disallowed

Currently, using two '+'s or '-'s in a row causes rock to produce a postfix increment/decrement operator in the resulting C code.
Semantically, there is no reason in particular why it should be allowed anyways, it just causes confusion since ooc is a C family language but we have no increment/decrement operators.

I agree. If I remember correctly, ++variable works, but variable++ does not, which always confused me.

Yeah, it works accidentally, it shouldn't.
I would highly advise you to go through your code and replace all instances it appears in, I will make it an error soon.

++variable is just semantically +(+(variable)), but is generating C code "++variable" because of no special-casing. Whereas I gather "variable++" simply doesn't parse?

I would highly advise you to go through your code and replace all instances it appears in, I will make it an error soon.

I was just about to. :)

@vendethiel
Exactly but I do think disallowing multiple prefix + and - operators is the way to go, as I said it would be really confusing otherwise, even if the generated code did not result in C's increment/decrement operators.

change the rule of unary operator  to '+' !'+' works for me.
https://github.com/zhaihj/nagaqueen/commit/d14f181111f41481bc24c18470a99b78cf75db44
I'm trying to make the error message more clear

Could be done with something like:
UNARY_PLUS  = '+' ('+' {  rewindWhiteSpace; throwError(...); })?

@shamanas
I'm wondering why this doesn't work:
UNARY_PLUS = '+' (!'+') ~{ throwError( ); }

I'm not too sure how ~ is implemented in greg, perhaps it can only be used after a match, so the ! here makes it useless?
I don't know, so I'm probably wrong though :)

I think @shamanas is right: (!'+') doesn't mean "Match something that isn't a plus", it means "stop matching if the character ahead is a plus" - so the rule probably doesn't match at all.
I think this was the right idea:

UNARY_PLUS  = '+' ('+' {  rewindWhiteSpace; throwError(...); })?

..except why can't it live in a separate PRE_INCREMENT / POST_INCREMENT rule? It might feel silly to actively look out for an operator we don't support, but that's the price to pay for having clear error messages, I guess.

If it lived in a PRE_INCREMENT rule, we should do something like:
UnaryOp = (UNARY_PLUS | PRE_INCREMENT | .... ) expr: Access

Anyways right?
So we still actively look for the operator we don't support, just in the child rule.
To me, UNARY_PLUS = '+' ('+' { rewindWhiteSpace; throwError(...); })? reads like: match unary + operator exactly one time and error out if we find it more than once, feels natural.
Anyway, it's just a little trick for better error reporting, this can be fixed in better ways (but with bad error reporting :P)

If it lived in a PRE_INCREMENT rule, we should do something like (...)

not necessarily, you can have several rules that start with '+', as long as unary plus has a !'+' after the first plus is matches, the double-plus rules should kick in next.
(then again I haven't touched nagaqueen in a while)

Sure, I don't disagree.
The point is, at some point or another, you will need to 'call' the double-plus rule so that error checking is done.
That means that no matter what you do, you will have some weird looking rule.
I personally prefer having the error checking in the UNARY_PLUS rule because I feel it makes it explicit that a chain of +s is not legal.
Another way would be to have a separate DISALLOW_PREFIX_INCREMENT, that is fine too imo.
Anyway, I think it's just a question of style, I don't lean in any direction too much :P


Add cloud credential validation

This method containing these changes is only called on cluster create. Should we consider checking the cloud credential on create, import, and update? It is possible that a user could change the cloud credential on a cluster after it is created.

@thedadams this method is run before moving to the importing phase so I think it's fine there https://github.com/rancher/aks-operator/blob/ff05137c5e97d4d3e26b7f61bb894c193bc829d2/controller/aks-cluster-config-handler.go#L197-L201
I think the point about validating for updates is valid though, @mjura what do you think?

@thedadams @cmurphy yes, it is safe to do this in validateConfig, because it will be checked before importing and creating phase. I have tested it and works fine

@mjura It is possible to change the cloud credential on a cluster. If a user creates an AKS cluster with valid cloud credential and then changes the cloud credential on the cluster to an invalid one, then the updates may fail.
Is this a concern we should address?

@mjura It is possible to change the cloud credential on a cluster. If a user creates an AKS cluster with valid cloud credential and then changes the cloud credential on the cluster to an invalid one, then the updates may fail.
Is this a concern we should address?

We can add validateConfig to function during update phase. I will test it

It is not needed any more. It is validated in https://github.com/rancher/aks-operator/blob/master/pkg/aks/client.go#L97
and in Rancher master it is validated also by UI.

@mjura was there an issue tracking this, if not please create one and send to test, write up how QA can test.


Support managing admin spec of mesh controller

Fixed https://github.com/megaease/easemesh/issues/65 Based on #70

Codecov Report

Merging #71 (f71ca4e) into main (98abfc0) will decrease coverage by 0.13%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##             main      #71      +/-   ##
==========================================
- Coverage   68.64%   68.51%   -0.14%
==========================================
  Files           2        2
  Lines         236      235       -1
==========================================
- Hits          162      161       -1
  Misses         58       58
  Partials       16       16

Impacted Files
Coverage Δ

operator/pkg/sidecarinjector/sidecarinjector.go
69.56% <85.71%> (ø)

...rator/pkg/controllers/meshdeployment_controller.go
64.70% <100.00%> (ø)

Continue to review full report at Codecov.

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


Minor Oculus changes

I have a few minor tweaks to Oculus support you may be interested in.

:+1:


Corrupt file when downloading screenshot in MakeCode

When I try to download screenshot in MakeCode, I get a corrupt file. Link added.
When working on bigger project the generated screenshot-file could be pretty large if it contains all code. Maybe an option for downloading screenshot of highlighted function or a single block, or what is visible on the screen.
Browser: Chrome version 65.0.3325.181 (64-bit)
OS: Windows 10 Enterprise
Link to project: https://makecode.com/_PpAMzTgADh28

This is likely fixed.


Bonfire: Symmetric Difference (difference length)

Challenge Bonfire: Symmetric Difference has an issue.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
function sym(args) {
  var symDif = [];
  //#1
  symDif.push(3,4,5,4,5,5,5,5);
  //#2
  //symDif.push(1,4,5,1,5,5,5,5);
  //#3
  //symDif.push(7,4,6,2,3,3,3,4);
  return symDif;
}
//#1
sym([1, 2, 3], [5, 2, 1, 4]);
//#2
//sym([1, 2, 5], [2, 3, 5], [3, 4, 5]);
//sym([1, 1, 2, 5], [2, 2, 3, 5], [3, 4, 5, 5]);
//#3
//sym([3, 3, 3, 2, 5], [2, 1, 5, 7], [3, 4, 6, 6], [1, 2, 3]);

Result checker response for cases 1, 2 and 3:
#1 (return is: [3,4,5,4,5,5,5,5])
ok > sym([1, 2, 3], [5, 2, 1, 4]) should return [3, 5, 4].
not ok > sym([1, 2, 5], [2, 3, 5], [3, 4, 5]) should return [1, 4, 5]
not ok > sym([1, 1, 2, 5], [2, 2, 3, 5], [3, 4, 5, 5]) should return [1, 4, 5].
not ok > sym([3, 3, 3, 2, 5], [2, 1, 5, 7], [3, 4, 6, 6], [1, 2, 3]) should return [ 7, 4, 6, 2, 3 ].

#2 (return is: [1,4,5,1,5,5,5,5])
not ok > sym([1, 2, 3], [5, 2, 1, 4]) should return [3, 5, 4].
ok > sym([1, 2, 5], [2, 3, 5], [3, 4, 5]) should return [1, 4, 5]
ok > sym([1, 1, 2, 5], [2, 2, 3, 5], [3, 4, 5, 5]) should return [1, 4, 5].
not ok > sym([3, 3, 3, 2, 5], [2, 1, 5, 7], [3, 4, 6, 6], [1, 2, 3]) should return [ 7, 4, 6, 2, 3 ].

#3 (return is: [7,4,6,2,3,3,3,4])
not ok > sym([1, 2, 3], [5, 2, 1, 4]) should return [3, 5, 4].
not ok > sym([1, 2, 5], [2, 3, 5], [3, 4, 5]) should return [1, 4, 5]
not ok > sym([1, 1, 2, 5], [2, 2, 3, 5], [3, 4, 5, 5]) should return [1, 4, 5].
ok > sym([3, 3, 3, 2, 5], [2, 1, 5, 7], [3, 4, 6, 6], [1, 2, 3]) should return [ 7, 4, 6, 2, 3 ].

Not sure if symmetric difference can be of any length as long as it includes only right digits.

I don't know what you are saying here.  Are you saying that so long as the first elements match that it passes the test, regardless of if it's much longer?

yes, if I understand it right [3,4,5,4,5,5,5,5] is invalid answer if result must be [3, 5, 4] (not depending on the order).

Confirmed.
Looks like the test cases need some additional work.


Compatibility Issue

Bootstrap Datepicker and Bootstrap Markdown is not working together.
When using the datepicker and markdown together Markdown is not working.
Thanks,
Vikash

@vkshrivastva can you provide a jsfiddle example showing the problem? Also which datepicker you use?

I got it solved.
Thanks,
Vikash


Unable to package WinRT library

Hello,
I'm trying to package my application which uses Windows runtime API (also known as WinRT) and there is an official Python wrapper provided by Microsoft. More info is available here  and here.
However this library is not packaged correctly because when I run my distributable I get an AttributeError and if you look into the winrt library under winrt.windows.devices.__init__.py on line 5 lies the root cause and 'Windows.Service' somehow don't get packaged in the final output.
Here is a repo with minimal reproduction of the issue https://github.com/nemwiz/winrt-nuitka-build
How do I include this library? I'm a bit stuck.
Also I couldn't find examples on how to include external libraries e.g. .pyd files. I just want my executable to find the needed libraries even if I have to copy/paste them from somewhere.
Thanks

After some trial and error I managed to find the issue.
WinRT ships a binary _winrt.pyd and when the library initializes they try to load it with the following code.
loader = importlib.machinery.ExtensionFileLoader(module_name, _winrt.__file__)

The key part here is _winrt.__file__ and after Nuitka compilation this throws an error
'winrt._winrt' has no attribute '__file__'
I saw the option --file-reference-choice and I've tried both runtime and original values, but same error appeared.
Is this a bug in Nuitka or the library uses some non standard way of importing binaries? Any guidance on this would be welcome.
At the moment I've found 2 workarounds and I don't like any of them. :)

Edit the library source code and replace __file__ with a correct path to the binary
Exclude the library with --nofollow-import-to=winrt and copy/paste the winrt library from venv into the dist folder

Closing for lack of feedback.

Hello,
I have the exact same issue with Bleak WinRT (Python 3.9 and Python 3.10 on Windows 10).
The __init__.py file generated by WinRT looks like this:
# WARNING: Please don't edit this file. It was generated by Python/WinRT v1.0.0-alpha.6

from . import _winrt

def _import_ns_module(ns):
    import importlib.machinery
    import importlib.util

    try:
        module_name = "_bleak_winrt_" + ns.replace('.', '_')

        loader = importlib.machinery.ExtensionFileLoader(module_name, _winrt.__file__)
        spec = importlib.util.spec_from_loader(module_name, loader)
        module = importlib.util.module_from_spec(spec)
        loader.exec_module(module)
        return module
    except Exception:
        return None

as @nemwiz said,  _winrt has no __file__ attribute.
I currently resolved my issue by patching the __init__.py file like this:
# WARNING: Please don't edit this file. It was generated by Python/WinRT v1.0.0-alpha.6

from . import _winrt

import os
_winrt.__file__ = os.path.join(os.path.dirname(__file__), "_winrt.pyd")

def _import_ns_module(ns):
    import importlib.machinery
    import importlib.util

    try:
        module_name = "_bleak_winrt_" + ns.replace('.', '_')

        loader = importlib.machinery.ExtensionFileLoader(module_name, _winrt.__file__)
        spec = importlib.util.spec_from_loader(module_name, loader)
        module = importlib.util.module_from_spec(spec)
        loader.exec_module(module)
        return module
    except Exception:
        return None

I hope this additional information can help. :)

@flozz it does, can you however create a new issue, where I would have a reproducer even. Nuitka should be setting __file__ since last release, but I have zero information and context in order to reproduce this.


Yes, it is preferred.


Suggest moving kind profile from this repo into community hub

Hi all,
The first version of the Greenfield Installation Project is ready for use. Any feedback and/or suggestions are welcome!
As part of the greenfield project, we also re-organized the zeebe-helm-profiles project (since the greenfield project depends on zeebe-helm-profiles).
One of the refactors we made was to copy the kind/camunda-values.yaml file from this repo and moved to zeebe-helm-profiles/development/camunda-values.yaml. We also changed the file slightly to make it run more consistently.
What do you think about removing the kind directory from this project, and referring folks to the greenfield kind instructions instead?
If you agree, here are the steps we would need to take:

merge the pull request to refactor zeebe-helm-profiles
Rename the zeebe-helm-profiles repository to camunda-8-helm-profiles
Accept this pull request to remove the kind directory out of this project

Thanks,
Dave

Hm not sure about this. I would like to keep that close to the charts to not get this outdated (again), Furthermore, I actually want to have some more tests also for the provided kind values file. This means currently I see no real value in removing it from the repo. Does this make sense?
Maybe @aabouzaid has a different opinion.

imo if we provide something as an example, it's 100% better if we can guarantee it works (e.g. with tests, for example). this is a net benefit for users as they will always have a nice getting started experience :+1:

IMO, I'm in favor of having KIND as one of the supported platforms/tools. It will make it much much easier to start with the Camunda Platform and also for the local development.
I really like the idea of having camunda8-greenfield-installation but I see it as an extension for what we officially support in the C8 Helm charts repo.

Thanks all! It sounds like the consensus is to keep the KIND profile in this repository and sounds like there is good reasons to do so.
We also have the zeebe-helm-profiles repo. And, now, in addition, we also have camunda8-greenfield-installation.
I'm a little confused about what should be maintained in this repo, vs what should be put in zeebe-helm-profiles, vs what should go in the camunda8-greenfield-installation project?
I think @falko has some thoughts on this as well, so maybe we can leave this open and hear his thoughts when he returns from vacation?
Here are some more notes to consider:
I think we should try to avoid maintaining KIND values files in multiple repos. If the KIND values files are maintained here, then what about other values files intended for similar audiences? For example:

ingress controller config values files
recommendations for small/medium/large hardware sizing
values files enabling prometheus metrics
etc.

Any opinions? Should these types of values files be maintained here as well?
Maybe we can consolidate these repos? Or maybe we just need to more clearly define which artifacts should go where?
Looking forward to organizing all this as best as possible, thanks again!

@upgradingdave personally I don't have an opinion about this topic as it's too early to decide it (we are still in the early stages and many things will be different when we have official support for EKS and AKS).
It's also not clear to me what would be the difference between zeebe-helm-profiles (which should be renamed to something like camunda8-self-managed-helm-profiles) and camunda8-greenfield-installation?
Isn't better to combine both of them in a single repo?

Hi @Zelldon, @aabouzaid, @npepinpe,
sorry for my late response. I just returned from vacation.
Running Camunda 8 Self-Managed on K8s requires three different artifacts:

A K8s cluster (which can be created using the scripts in camunda8-greenfield-installation in AWS, Azure, Google, and KIND. As a convenience they can also install an opinionated "greenfield" Helm profile.)
A Helm chart (which is officially supported and maintained in this repo)
Chart values for the desired usage profile (which are opinionated, not supported, and currently spread across this repo (KIND, OpenShift) and the updated camunda-8-helm-profiles see also this proposed PR)

Example profiles currently include:

Generic Nginx Ingress profile
Azure Ingress profile
Small development environment profile (primarily for KIND but can be used in any other K8s environment and should be similar to docker-compose-core.yml)
Maybe in the future: Google Load Balancer Ingress?, AWS load balancer ingress?, Identity with ldap?

I welcome any ideas that help us maintaining consistent chart value profiles and test them. One option for that could certainly be to use a monorepo approach and have all profiles inside this repo. I just have the feeling that it would make it less clear what is supported product, i.e. the Helm chart itself, and what are just community contributed examples that may or may not work. I'd also be concerned about how much "noise" of community contributions to the profiles we would want inside this repository . That's why I suggest to keep the profiles in the separate repo that we already have and that just needs get this general overhaul.
The Community Hub signals the invitation for community collaboration and clearly indicates that the profiles are examples and not by any means the ONLY supported configurations. There should be technical solutions for us to still run Q&A tests on the profiles and nothing stops us to also update/refactor them together with the Helm charts, right? Or would a monorepo make these things so much easier?
Cheers,
Falko

@falko Sorry, I don't get your question :sweat_smile:
So the current PR is about KIND, and we would like to keep it part of the official repo/docs.
However, I'm not sure what the mono repo is about? :thinking:
Do you mean camunda-8-helm-profiles and camunda8-greenfield-installation?
If yes, then mono repo could be a good idea otherwise things like enforcing practices and style could be done via Github Actions.
In all cases I don't see a problem of mentioning those 2 repos when they are ready.
We just need to make a new section called community or something like that and mention them there.

Hi @aabouzaid,
the entire discussion should have been in the PR over at zeebe-helm-profiles and this PR would have been a natural follow-up on that. Now @upgradingdave already started discussing here and thus I gave me response here as well. Please, read it in the larger context of all the Helm profiles that are available or in the making. Another set of profiles are going to be the SaaS t-shirt sizes, which @berndruecker and I a planning to build.
So when I talk about a monorepo, I mean having the Helm chart sources together with all these chart value profiles in a single repo and retiring the zeebe-helm-profiles repo completely. The K8s setup scripts in the camunda8-greenfield-installation repo are IMHO addressing a separate concern and should not be mixed with our chart value collection even if they use some of the profiles for the convenience installing Camunda together with K8s.
Cheers,
Falko

Thanks again for taking the time to meet today, since we decided to leave the KIND profile here, no longer need this pull request.


[JSDoc] Add missing documentation for botframework-config

Addresses # 2602
Description
This PR introduces the change to jsdoc/require-jsdoc in the .eslintrc.json file from warn to error and adds all the missing JSDoc documentation for botframework-config based on the errors thrown. Some documentation is ported and adapted from botbuilder-dotnet
Specific Changes

Updated jsdoc/require-jsdoc in .eslintrc.json from warn to error.
Private methods' documentation that does not exist on botbuilder-dotnet we added the @private tag in the JSDoc comment.
Added JSDoc comments in 11 files.

Testing
In the following image there is a sneak peek about the ESLint Report with no JSDoc errors.

Promoted


Support -- in cc-wrapper

Description of changes
Closes #228136.
cc @expipiplus1
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

@ofborg build tests.cc-wrapper-clang tests.cc-wrapper-gcc

LGTM


:wrench: README language reworked

Thank you for the contribution! However, I am a bit undecided on what to do with this one. Half of the changes I appreciate, the other half is not about language but rather about style and this is a very subjective matter. For example:

Why did you truncate most sentences so that they are no sentences anymore? Most of your sentences either miss a verb or a subject. This makes it pretty hard to read and somewhat less fluent...
Why did you put the gif to the bottom?
I like the idea of using emoji for the different modes, but why 🐁, 🔲, and 🖱? Why different mice? What's the square supposed to be?
Why did you add the link to Weblate at this position? It does not make much sense in the table of contents and one cannot see that there are actually two links in one line.
Sorry, I do not get the "grumpie menu ♜-ie". Is "tower-ie"? "fort-ie"? "castle-ie"? :smile:

At this point, I would consider this to be a set of well-intentioned suggestions of which I will happily adopt some, but I would refrain from merging them all in this state.


Add Status Bar Api

What does the pull request do?
Adds Status Bar api through ITopLevelWithPlatformStatusBar, with color and visibility properties. The Api can be accessed on any Control instance, but is only applied when using the TopLevel's Content, ie the MainView. This Api is targeted toward mobile platforms.
New Properties in Control;
Color StatusBarColor { get; set; }

bool? IsStatusBarVisible { get; set; }

What is the current behavior?

What is the updated/expected behavior with this PR?

How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues

You can test this PR using the following package version. 11.0.999-cibuild0025421-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

If this only applies to the TopLevel why are we defining the property on Control level?

You can test this PR using the following package version. 11.0.999-cibuild0025556-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

Replaced StatusBarColor with StatusBarTheme

You can test this PR using the following package version. 11.0.999-cibuild0025566-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 11.0.999-cibuild0025772-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 11.0.999-cibuild0025964-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]


Hammer_add command no longer works

Using the hammer_add command no longer works (I have both infinity hammer and infinity tools) and all my previously added prefabs are no longer there.

There is now a new file for them infinity_tools.yaml. You might be able to copy over your old ones from infinity_hammer.yaml but the format has changed quite a bit so might need tweaking.
Unfortunately hammer_add is no longer supported since there are lot more options now.


Bot Emulator 3.5.27 not able to connect to bot getting 301 response

So i followed the ChatConnector example for Node Js.
This is the log i see in bot emulator
[20:34:29] -> POST 301 [conversationUpdate]
Please help in solving this issue
This issue is on Windows 7

Are you certain is was 301? I'm wondering if it could have been 401.
If 301:

Was your bot running locally or remotely?

If 401:

Was your bot configured with MSA appId and password, or running without credentials?

It was 301 and it was running locally.
In fact from the emulator logs I got the payload which it tried to POST to
the bot.
I did the same using Postman and it worked.
Is there something which I am missing here ?
But when I used console connector to try out a hello world bot it worked.
So only the chat connector is causing issues.
Please help me to resolve this
Thank you
On 8 May 2017 22:31, "Eric Anderson"<EMAIL_ADDRESS>wrote:

Are you certain is was 301? I'm wondering if it could have been 401.
If 301:

Was your bot running locally or remotely?

If 401:

Was your bot configured with MSA appId and password, or running
without credentials?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotFramework-Emulator/issues/180#issuecomment-299926598,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJO3_VlycJ5HwOEXCQLcl11zf7hKRanSks5r30qBgaJpZM4NTLea
.

Could you please copy/paste a few lines from your emulator log showing the error?

This is all I see in red colour
POST 301 [conversation update]
Nothing else , I don't know why is it redirecting .
On 9 May 2017 02:05, "Eric Anderson"<EMAIL_ADDRESS>wrote:

Could you please copy/paste a few lines from your emulator log showing the
error?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotFramework-Emulator/issues/180#issuecomment-299982991,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJO3_VwTlpV-BsktGsG83v9_wmhSxLjFks5r33x9gaJpZM4NTLea
.

A new version of the emulator is available that may be able to provide more information about the 301. Direct download link: https://github.com/Microsoft/BotFramework-Emulator/releases

Nope still the same issue. Any idea on why this might be returning back 301?
I am on the latest 3.5.29
On windows 7
On 9 May 2017 22:54, "Eric Anderson"<EMAIL_ADDRESS>wrote:

A new version of the emulator is available that may be able to provide
more information about the 301. Direct download link:
https://github.com/Microsoft/BotFramework-Emulator/releases
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotFramework-Emulator/issues/180#issuecomment-300239919,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJO3_ctfB-xltXzoNgw9Ok0PHBS2e5xEks5r4KFPgaJpZM4NTLea
.

The 301 would have to be coming from something external as there's nothing in the Node SDK that does any sort of redirect.  Is it possible that you have a firewall or proxy of some sort running locally?  Out of curiosity can you see where it's redirecting you too???

How can I see where it is redirecting me to ? I clicked on that log and it
shows me the body of the post which the emulator sent as the request but
where can I see the response ?
Thanks for the quick reply
On 10 May 2017 23:40, "Steven Ickman"<EMAIL_ADDRESS>wrote:

The 301 would have to be coming from something external as there's nothing
in the Node SDK that does any sort of redirect. Is it possible that you
have a firewall or proxy of some sort running locally? Out of curiosity can
you see where it's redirecting you too???
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotFramework-Emulator/issues/180#issuecomment-300567369,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJO3_QNQieqqq2_TW13SjkFyHH5i3fRZks5r4f2ZgaJpZM4NTLea
.

If the 301 is clickable in the emulator log, clicking it will show the content of the response in the detail view. If the 301 is not hyperlinked, then you'll need to inspect the network traffic using a network debugging tool like fiddler. If you use fiddler, be sure to set your machine's proxy settings to not bypass proxy for local addresses, otherwise you won't see localhost traffic in the fiddler capture.

So on clicking the 301 response all I can see is a HTML like this
 \r\n
 redirection <\title>
<\head>

Redirect <\body>\r\n
But no redirect URL anywhere
What do I do now ?
Thanks
On 11 May 2017 10:23, "Eric Anderson"<EMAIL_ADDRESS>wrote:

If the 301 is clickable in the emulator log, clicking it will show the
content of the response in the detail view. If the 301 is not hyperlinked,
then you'll need to inspect the network traffic using a network debugging
tool like fiddler https://www.telerik.com/download/fiddler. If you use
fiddler, be sure to set your machine's proxy settings to not bypass
proxy for local addresses, otherwise you won't see localhost traffic in the
fiddler capture.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotFramework-Emulator/issues/180#issuecomment-300681913,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJO3_Qt6PXP6k8fz-q6VKK1_5cRVDlvZks5r4pQ3gaJpZM4NTLea
.

Try using fiddler (as described above) to look at the Location header of the 301 response. This header contains the redirect URL.

Going ahead and closing this now. If you'd like further assistance, reply here and we'll reopen it.


Display "Name" in provider tooltips

This will help users understand that they're actually looking at health care providers or doctors.
In some datasets, the "Name" field might be the name of a Facility or might be the First + Last Name of a doctor. We will have to adjust per pre-loaded dataset.

@ericboucher Is this taking into account Providers' First and Last names? If so, I'm not seeing that in any of FKM datasets.


UI freeze when a link is added.

The javascript console report:
Uncaught TypeError: a.position is not a function
at i. (almost-joint.min.js:formatted:167)
at v (backbone.min.js:1)
at p (backbone.min.js:1)
at c (backbone.min.js:1)
at i.o.trigger (backbone.min.js:1)
at i.notify (joint.min.js:1)
at i.pointerup (joint.min.js:1)
at i.pointerup (joint.min.js:1)
at i.dragMagnetEnd (joint.min.js:1)
at i.pointerup (joint.min.js:1)
It seems to be a problem with almost-joint

Cloded by: https://github.com/B3rn475/almostjs-joint/pull/8


Authentication branch

Added devise for user authentication
Added method that requires user authentication before accessing
Added views for signing up and signing out
Added styling to the signup form

Thank you.


Fix long cable timeout also on RPI2

See title, simply fixed long cables timeout also in RPI2.
Please marge and add a new tag.
Thank you.

It's correct, needs to merge more quickly

I support this merge, tested on a RPi3 and a DHT11

hi this repo is going to be deprecated soon, please try
https://github.com/adafruit/Adafruit_CircuitPython_DHT
which will replace this library


Dust protection for Sentinels

Snipe them at low gas prices.

The core protection feature in this PR.
As an optional feature, operators can supply a threshold file that specifies token addresses and corresponding amounts, which will be used to determine liquidation inclusion.
In cases where the threshold file is not provided, the system will fall back to its default behavior. Additionally, if a threshold file is supplied but does not include information for all tokens, any missing tokens will also revert to the default behavior.
Please note that we are still testing and this can be changed before merging.


Notebook output scrolling is not working when output contains images

Running the following code in an .ipynb notebook results in a long output that is not scrollable, despite me having set "notebook.output.scrolling": true.
import matplotlib.pyplot as plt
import numpy as np

for i in range(10):
    print("this is some text\n" * 10)
    plt.plot(np.linspace(0, 10, 100), np.sin(np.linspace(0, 10, 100)))
    plt.show()

Commenting out the plt.show() line makes it scrollable, as expected.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

We currently keep the outputs independent since they each have their own output menu to be able to select the presentation type. Grouping them all in one scrollable region would mean moving logic outside of the individual output renderer and into the workbench - a pretty significant change to make.

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


do you have special code for sparse data?

is your code running faster of sparse data?

Hi @Sandy4321, my code has not been optimized to run on sparse data, although sparse data is supported as long as it's represented by a dense numpy matrix with 0s.
There is a fork of the project that adds support for sparse matrices. However, that is currently 41 commits behind this project.

Hello
It is exactly how I found your code by using this link
https://github.com/srome/PyFactorizationMachines
The problem with link is that there is no advanced optimization like Adam and GPU use
But problem with your code is, that sparse data representation is not used, so big matrices can not be used.
For example data from this link
https://github.com/srome/PyFactorizationMachines
can not be represented in dense matrix , since it uses huge matrix , that can not be placed in RAM
Can you try to use sparse matrix ?

@Sandy4321, pyfms version 0.2.4 now has support for sparse data. Please see the documentation and example for usage details.


Draft: State artifact specification

As described in #6, this describes Singer State artifacts which can be split between its constituent streams and combined again.

https://www.jsonschemavalidator.net/ is nice to test the proposed schema against various state formats


Performance issues

Ever since we moved to https://render.com to host our backend, I feel like the performance went down quite a bit. Especially cold starts (when there was no incoming request for a while, probably 30-60 min), a simple GET request to obtain package data might take several seconds. Maybe we can do something about it.

I believe this has been solved.
Closing this.


Error: Unpacking boot image failed!

This error occurs on my Galaxy S6 and I assume also in further Galaxy devices. However the following zip patches correctly: https://androidfilehost.com/?fid=24459283995297893

with latest commit boot patcher exe? probably libc mismatch, might need to recompile all the binaries static, i have considered trying to distribute libc for each arch inside lazyflasher but it's been a mess and i don't have the time to figure it out

I assume the problem lies in not finding the correct boot_block. In this patch it finds it in /dev/sda8 while in the working zip is found in /dev/block/platform/15570000.ufs/by-name/BOOT


set logger as static final

use of StringBuilder instead of StringBuffer
call dispose() on graphics to avoid memory leak
use logger instead of calling e.printStackTrace()
thanks for taking time to review

I'd prefer to not use static for loggers, if there is no obvious gain.

Oh, if it is OK for you @Nowheresly , can you rool them back ?

@lefou thanks for your feedback
Well using static logger or not is a matter of taste.
From the official documentation
http://www.slf4j.org/faq.html#declared_static
'We no longer recommend one approach over the other '
Previously there were some logger declared as static, some other not.
We should at least be consistent and document it.


Fix 0.4.1

Closes #8

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

requirements: run: fluidsim-core >= 0.4.1 should not contain a space between relational operator and the version, i.e. fluidsim-core >=0.4.1

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

@conda-forge-admin, please rerender

Impressive! Fluidsim should be usable on Windows and MacOS! With all Pythran extensions available!


Vulkan SDK release feature cut-off Friday August 27

LunarG is planning the next Vulkan SDK release in the week of Monday August 30.
MoltenVK PRs approved by Friday August 27 will be included, and a MoltenVK SDK release will be made Monday August 30.
@cdavis5e @HansKristian-Work @richard-lunarg

MoltenVK 1.1.5 is now released to cover Vulkan SDK 1.2.189.


Match queries with integer properties constraints return 0 results after adding gin index

Describe the bug
When a GIN index is added to the properties of a vertex, MATCH queries with a constraint on an integer property return 0 results.
How are you accessing AGE (Command line, driver, etc.)?

psql

What data setup do we need to do?
SELECT * FROM cypher('cypher_index', $$CREATE (p:Person {id: 1, name: 'John'}) return p$$) as (p agtype);
CREATE INDEX props on cypher_index."Person" USING GIN (properties);
SET enable_seqscan=false;

What is the necessary configuration info needed?

/

What is the command that caused the error?
SELECT * FROM cypher('cypher_index', $$MATCH (p:Person {id: 1}) return p$$) as (p agtype);
 p
---
(0 rows)

Expected behavior
One row should be returned
Environment (please complete the following information):

Version: https://github.com/apache/incubator-age/commit/d5cd56883da75bcf6d85f85b71f169cc5421a325

Additional context
This bug is a minimal reproduction of the issue described in https://github.com/apache/incubator-age/issues/45#issuecomment-1121484835
The same query wih a constraint on a string property gives correct results:
SELECT * FROM cypher('cypher_index', $$MATCH (p:Person {name: 'John'}) return p$$) as (p agtype);
                                              p
---------------------------------------------------------------------------------------------
 {"id":<PHONE_NUMBER>31969, "label": "Person", "properties": {"id": 1, "name": "John"}}::vertex
(1 row)

Explain for the integer property constraint:
SELECT * FROM cypher('cypher_index', $$EXPLAIN MATCH (p:Person {id: 1}) return p$$) as (p agtype);
                                  QUERY PLAN
-------------------------------------------------------------------------------
 Bitmap Heap Scan on "Person" p  (cost=12.00..16.02 rows=1 width=32)
   Recheck Cond: (properties @> agtype_build_map('id'::text, '1'::agtype))
   ->  Bitmap Index Scan on props  (cost=0.00..12.00 rows=1 width=0)
         Index Cond: (properties @> agtype_build_map('id'::text, '1'::agtype))
(4 rows)

Explain for the string property constraint:
SELECT * FROM cypher('cypher_index', $$EXPLAIN MATCH (p:Person {name: 'John'}) return p$$) as (p agtype);
                                      QUERY PLAN
--------------------------------------------------------------------------------------
 Bitmap Heap Scan on "Person" p  (cost=12.00..16.02 rows=1 width=32)
   Recheck Cond: (properties @> agtype_build_map('name'::text, '"John"'::agtype))
   ->  Bitmap Index Scan on props  (cost=0.00..12.00 rows=1 width=0)
         Index Cond: (properties @> agtype_build_map('name'::text, '"John"'::agtype))
(4 rows)

This issue has been fixed in https://github.com/apache/age/commit/392a937bdfa7c7a869798d7dca93f250c94d42e9.


Add announcement style

Follow up on yesterday's OME meeting: proposing a style for announcements on the home page that would get people's attention.
Copy used is for placeholder only. This requires real copy/content, please :)
For your review, @joshmoore.

Thanks, @lunson. I was too slow in responding to your trello notification. Let's continue the conversation here.

Thanks, @lunson !

V
V
V

Now deployed live to https://www.openmicroscopy.org/


fix(types): correct the type of isScrolling property

fix(types): correct the type of isScrolling property

Pull Request Test Coverage Report for Build 911321069

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 5.774%

Totals

Change from base Build 909810979:
0.0%

Covered Lines:
17

Relevant Lines:
260

💛  - Coveralls


Some changes to sim info and maxr fix

Includes two features:

Move proc --> startCode
maxr fix

lots of. diffs, I guess I need to rebase to the dev branch, its not an immediate need anyway

Thanks, Sophie. I think it's just because I haven't merged main into v05-dev yet. This PR might look cleaner once I do that. By the way, does the maxr fix not need to go into v4 too?

yes, it should go into that too I guess, I'm not sure if anyone but me is using it but I could put in two PR's if you'd prefer?

Thanks, Sophie. Two PRs would be great :)


Has a question about receiver address when solana as destination chain

Suppose I want to bridge USDC from EVM-chain, then, upon create the ‘burn' Transaction，what destination address (solana account) should I put in? Is it normal user wallet address or ATA under that wallet address?
If the destination account do not exist pre-hand, will solana-cctp-contract create this account automatically? If yes, this will be done at 'receive_message'  method, right?
Of course, It would be better if your guys can provide some examples Tx url.

what destination address (solana account) should I put in? Is it normal user wallet address or ATA under that wallet address?

It needs to be a hex encoded USDC token account address. Please see this section in our developer docs for more information on that. Here is an example transaction transferring USDC from Ethereum to Solana: https://etherscan.io/tx/0x58158c70beab799c85c7e4c6faf8b2c453d54d85e7aee79bc35ed38d5f11cc9d

If the destination account do not exist pre-hand, will solana-cctp-contract create this account automatically? If yes, this will be done at 'receive_message' method, right?

No, the CCTP solana contracts do not create the ATA, this would need to be done wither by the user or integrator.


The issue of accuracy with DDQ and DINO models.

The results of training DDQ and DINO models on our own dataset are very poor, with a mean average precision (map) of only 0.03. In contrast, using the same dataset with Faster R-CNN achieves a level of detection comparable to normal. I would like to inquire about what might not have been set up successfully.

hello, have you meet this bug?
KeyError: 'DDQDETR is not in the mmdet::model registry. Please check whether the value of DDQDETR is correct or it was registered as expected. More details can be found at https://mmengine.readthedocs.io/en/latest/advanced_tutorials/config.html#import-the-custom-module'


[Request-For-External] Table 472 "Job Queue Entry"  - procedure SetDefaultValues

Describe the request
Hello, we need the following currently local/internal procedure to be set to global, so we can call it from a subscriber. Can you please check this? Thank you!
procedure SetDefaultValues(SetupUserId: Boolean)

Additional context
we need to call the procedure from tableextension
Internal work item: AB#539831

Hi @MH1612, what is the scenario and reason behind needing to call this procedure?

Closing due to inactivity, reopen a new issue with the information if this is still required.


Strange Windows notification shows every minute continuously

Evento Version: 2.0.74-0-ge3f654d
Windows Version: 23H2 22631.4317
As I mentioned in title , after closing the evento window , this notiification shows up every minute .
How to repelicate the bug

Disable "Minimize to the tray after closing the window".
Close the window. Now there's still a functional icon in the tray.
Wait for a minute then the notification will show.

看起来有些程序正在尝试向我们的随机端口发送奇怪的数据
For C++ Team：也许我们需要添加固定包头，并忽略缺乏包头的无效数据

Adding some info:


Redesign "Edit lecture" page

⚠ Merge #609 beforehand.
Current preview:

closed in favor of #628


Better Homepage for Sources

Describe your suggested feature
A browse menu like neko or a "feed" in tachiyomi sy that displays latest updates of popular content for simpler view and search for new content.

Other details
I like tachiyomi sy feed tab but it has a lot of bugs and nekos browse menu is just awesome but unfortunately it's only available for mangadex and not other sources. I know it might not be possible coz it's a lot of work.
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an request for a feature in the original Tachiyomi, I am opening because multiple versions of Yōkai have been released without this feature.
[X] If this is an issue with an extension, or a request for an extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to version 1.7.10.
[X] I have checked through the app settings for my feature.
[ ] I have admitted that I am a clown by having checked this box, as I have not read these acknowledgements.
[X] I will fill out all of the requested information in this form.

This is planned actually, just not sure how to implement it just yet...

This is planned actually, just not sure how to implement it just yet...

A feed like that in TachiyomiSY would be a great addition.


Storage Class Update is not possible on Postgres-Flex

Currently the provider has no stringplanmodifier.RequiresReplace() for the change of the storage class and the STACKIT API returns: update functionalitynot supported for field 'class'
This ends in a fail since the change can't be handled, but the provider suggests that it can be changed.
That's likely due bad design on the Postgres-flex API which suggests it can be updated.

This applies like also to the MongoDB.

Thank you for the report @roberth1988!
I will be working on a fix for this today and provide an update once this issue is addressed.

I just merged some changes to support this for Postgres-Flex. Regarding MongoDB, we are experiencing some issues that require further investigation.
Thank you again for reporting, this issue will remain open until we can produce a fix for MongoDB as well. Will provide updates on that soon.


translated terms of reference file for issue #146

Added french translation to terms of reference file.

@JulieBB23 are you ok with this translation:
https://github.com/gc-da11yn/gc-da11yn.github.io/blob/48ebe415e59915df10d014760535930225aabc35/.github/TERMS_OF_REFERENCE.md


feat: viewer blocks api

feat: viewer blocks api
https://github.com/bestchains/bc-explorer/issues/3

Update module name. github.com/bjwswang/bc-explorer -> github.com/bestchains/bc-explorer


Migration to clean up bogus rooms

These rooms have invalid state snapshots, leading to correctness problems. The existence of such snapshots leaves us open to performance problems due to gappy state updates.
We have prevented invalid state snapshots from being formed (#255, #266) in the future. The remaining hole is to discard existing invalid snapshots.
Closes #256.

Can we get some data on how many rooms were affected? It would be nice to have as a sanity check when running this.

I was thinking the same. I guess I will have to write the migration in go for this?

I'll need to test this somehow.

Tested manually in a staging environment and then with a gradual rollout.


Add jvm backend check for UTF-8 Constant length

Fix #15850

On scala 2, I'll try intercepting the error and only then doing a post-mortem.
I wonder if other error messaging would benefit from doing no analysis except after the fact; or maybe "interactive mode" drags apparatus along, which is turned off in batch mode.


[Reminder Feature] - Send reminders for subscriptions

Pivotal Tracker

https://www.pivotaltracker.com/story/show/180846802

Description / PR Commit Message

added ActionMailer to send reminder emails
implemented sidekiq as enqueuing system
set up basic reminder CRUD

Guidance for Review

Need to write tests
reminder database needs to be polished

Author's Checklist

[ ] Wrote tests?
[x] Linked to issue in Github?
[ ] Tests passed?
[ ] Test deployment to gatekeeper-tamu-test?

Also remember to export the new env variables and test on gatekeeper-tamu-test heroku app to verify functionality


clang: error: the clang compiler does not support '-march=native' (Apple M1)

Thanks for your effort, but when I try to do a "npm install word2vec", I get an error, as mentioned in the title of the issue:
"clang: error: the clang compiler does not support '-march=native'". I also have a computer with an Apple M1 chip, a MacBook Air (M1, 2020). I was trying to figure it out, but it ended up not being so straightforward for me. Could you please give me a hint to help me solve this? Thanks!

I am closing this issue because the problem was solved.


Update AbsoluteLayoutFlags Functionality

This PR adds .LayoutFlags() funcitonality to the docs as introduced in https://github.com/CommunityToolkit/Maui.Markup/issues/215

I almost merged this one... I guess this is still a draft until we merge the feature?

Yeah let's hold off until we release the feature


Add syntax highlights

Hi, thanks for the theme. Unfortunately, it doesn't have a very popular feature: Syntax highlighting.
Other themes usually use Prism for this. What your thoughts on this?

I agree it's a popular feature but still there are people that don't need it, I don't want them to load some resources if it's not necessary. I feel more confortable if this feature works as something optional, something that you can include only if you really need it.
Since you will need to make some changes to the template anyways to load the theme you want and to target the languages you want, might be better if I create a page in the Wiki with the instructions to do this and adapt the theme a bit to let people add the Prism scripts easily.
What do you think?

Yeah, that would work, but requires users to manually edit the source code.
Right now users can just grab the zip file and upload it through ghost admin panel but now, they should clone this repo, make changes and publish it again( Besides they need to setup docker, docker-compose to test if it works correctly.)
I personally edited the source code and set it up manually, but I think users may find it hard to accomplish this task.
What if we make it config based? The user just set a variable to true, and we do the rest

Oh no, you don't have to do all that work to use Prism with this theme. To install Prism you have two main options:

Download the zip file, modify the file default.hbs to include the stylesheet and the scripts of Prism, zip it and upload it to your Ghost instance.

Open the Ghost admin panel, head to the Code injection section and then include the stylesheet and the scripts of Prism.

I don't think Prism should be included by default, I think is better if users install it by their own because one way or another, you will end up editing a little the source code to include the theme you want and to include the styles of languages you want. I can't include all the 199 languages and all the 8 themes that Prism support.

Can I offer an alternative option?
I came across your theme (love it by the way!) and am currently going through the process of adding syntax highlighting to it. I agree that having people put it in the Code Injection section is feasible, but it's obviously much better to serve those files locally and not having to use a CDN.
Perhaps two zip's can be produced, one with prism and the other without?

@eddiesigner Since I'm a developer myself, I've already forked this repo, included prism by default in it.
It gets built with other assets.
To note my pov, syntax highlighting is something that should be integrated in theme itself (Sorry your soloutions seems hacky to me)
Like every time you update the theme, the poor client should first download the zip, modify defaults.hbs and then zip it again?
Besides, the whole thing talking about is 20KB

@mehrdad-shokri not every person that uses ghost will use code samples, so no it's not an integral part of a theme.
The author is completely justified in his decision

This should be not included in the theme as default.
The easiest method would be just including the prismjs files via Code Injection as so:
<link href="https://cdnjs.cloudflare.com/ajax/libs/prism/1.17.1/themes/prism.min.css" rel="stylesheet" />
<script src="https://cdnjs.cloudflare.com/ajax/libs/prism/1.17.1/components/prism-core.min.js"></script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/prism/1.17.1/plugins/autoloader/prism-autoloader.min.js"></script>

If you want to serve the files yourself you'll have to repack the release zip file with the necessary files.
And if you want to do more custom stuff you can create a fork of this repo and use it as an upstream branch from which you can pull updates.

Like I said before, the most optimal solution would be to create two zips if it's that much of an issue.
But those who are using code samples are probably skilled enough just to include it themselves.
It should not be added via code injection.
This is such a non-issue and should just be closed.

I've forked this repo and added prism as an internal part of the theme.

I've created a new page in the Wiki to let users know how to install Prism in order to add syntax highlighting: https://github.com/eddiesigner/liebling/wiki/Add-syntax-highlighting-with-Prism
The reasons because I think this is the best way to install Prism are the following:

Theme agnostic, that means you can keep it even if you change the theme.
Most of the time is better to serve this kind of files from a cdn instead of locally.
You don't have to touch the source code.
You can add the Prism theme you want and you can give support to the languages that you actually need.

I'll close this issue for now, please feel free to reopen it if you think is necessary.


[BOT RULES] Adding Mention

Adding Mention for Container Registry bot rules to coincide with CODEOWNERS file change here: #30634

/check-enforcer override

/check-enforcer override


(Event Streams) Support initial-request in RPC-based protocols

Effort is to create a document which provides sufficient context to easily pick up this issue in the future.


files program does not save dark mode

In the program settings, "appearance" section, the dark mode is not saved when the program is closed and when it is reopened it appears in normal mode and it is necessary to put it back in dark mode.
I would beg you to fix this bug
Thank you<!--
🚨🚨🚨🚨🚨🚨🚨🚨🚨🚨
I ACKNOWLEDGE THE FOLLOWING BEFORE PROCEEDING:

If I delete this entire template and go my own path, the core team may close my issue without further explanation or engagement.
If I list multiple bugs/concerns in this one issue, the core team may close my issue without further explanation or engagement.
If I write an issue that has many duplicates, the core team may close my issue without further explanation or engagement (and without necessarily spending time to find the exact duplicate ID number).
If I leave the title incomplete when filing the issue, the core team may close my issue without further explanation or engagement.
If I file something completely blank in the body, the core team may close my issue without further explanation or engagement.

All good? Then proceed!
-->
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS Version: [e.g. Windows 10 1903 18362.175]
App version: [v0.11.0]

Additional context
Add any other context about the problem here. Does this problem occur again after restarting the app?
Log file
Please upload the log file here so that we can understand your issue better. You can access it from Settings->About->Open log location. You can also find this file by going to %localappdata%\Packages\49306atecsolution.FilesUWP_et10x9a9vyk8t\LocalState, you should see a file in this directory called debug.txt or/and debug_fulltrust.txt.

Duplicate of #3291


Rename to_str(array) and to_array(str)

Rename to_str(array) and to_array(str) to str_split and `array_join
Fixes #1401

Also we need you to accept the CLA.

Closing in favor of #1415


hotfix/issue-744

fixes #744 : logo on OSCM portal now displays version when clicked

This change is

Reviewed 6 of 6 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

still need to integrate changes with ESS, please do not merge yet.

Fix confirmed working on OSCMaaS. Thanks, @digitalfishpond.


Bakerloo line is not announced (Elephant bound)

On the National Rail section of the Bakerloo line (Queens Park - Harrow & Wealdstone), all trains to Elephant & Castle are not announced. They are shown on the departure board for live announcements, but do not actually become announced. (if it is because it is marked as underground, I do not believe there is a separate announcement for this, I think it's just the regular elephant & castle, just Celia & Phil are missing the calling points, but I might be wrong, your could also try to bodge it in the way that this has it, but it is a different announcer https://youtu.be/oieUusLTk5k?t=127).

It'd likely need to be a custom case for Live Trains which allows me to play something like "Calling at all stations to Elephant & Castle".
TfL don't put all of the calling points in the data feeds, and the reason this particular case is broken is that they also use a different station (CRS) code to Elephant & Castle National Rail station, too.

Could you not add a line of code so that if it Elephant & Castle Underground, it just plays the announcement for regular Elephant & Castle (it wont deal with the calling at, but will deal with it not being announced)?

Same issue with district line to Upminster lol when Announcing from Richmond/Gunnersbury/Kew Gardens and that's something I was really looking forward to hearing. It is announced for the direction to Richmond (LUL).