query probing stuck in presence of value classes

This simple project that uses the new way of creating sources reproduces the problem.
When the classes that represent a cassandra column family contain value classes, the query probien never finishes. The query used to reproduce this behaviour is a simple one:
quote(query[Score])

[info] /Users/luissanchez/development/repositories/github/lvicentesanchez/cassie/src/main/scala/io/github/lvicentesanchez/Main.scala:14: SELECT cust_id, level, attempt, score FROM score
[info]   DB.run(query[Score]).foreach(println)
[info]         ^
[pool-6-thread-4] INFO com.datastax.driver.core.NettyUtil - Did not find Netty's native epoll transport in the classpath, defaulting to NIO.
[pool-6-thread-4] INFO com.datastax.driver.core.policies.DCAwareRoundRobinPolicy - Using data-center name 'Cassandra' for DCAwareRoundRobinPolicy (if this is incorrect, please provide the correct datacenter name with DCAwareRoundRobinPolicy constructor)
[pool-6-thread-4] INFO com.datastax.driver.core.Cluster - New Cassandra host /<IP_ADDRESS>:9042 added

Now... if we replace that query with a wrong query like this one:
quote {
    (playerId: PlayerID) => query[Score].filter(_.custId == playerId)
  }

We get an error as expected:

[info] Compiling 6 Scala sources to /Users/luissanchez/development/repositories/github/lvicentesanchez/cassie/target/scala-2.11/classes...
[info] /Users/luissanchez/development/repositories/github/lvicentesanchez/cassie/src/main/scala/io/github/lvicentesanchez/Main.scala:19: SELECT cust_id, level, attempt, score FROM score WHERE cust_id = ?
[info]   DB.run(some)(PlayerID("1")).foreach(println)
[info]         ^
[pool-6-thread-5] INFO com.datastax.driver.core.NettyUtil - Did not find Netty's native epoll transport in the classpath, defaulting to NIO.
[pool-6-thread-5] INFO com.datastax.driver.core.policies.DCAwareRoundRobinPolicy - Using data-center name 'Cassandra' for DCAwareRoundRobinPolicy (if this is incorrect, please provide the correct datacenter name with DCAwareRoundRobinPolicy constructor)
[pool-6-thread-5] INFO com.datastax.driver.core.Cluster - New Cassandra host /<IP_ADDRESS>:9042 added
[error] /Users/luissanchez/development/repositories/github/lvicentesanchez/cassie/src/main/scala/io/github/lvicentesanchez/Main.scala:19: The sql query probing failed. Reason 'com.google.common.util.concurrent.UncheckedExecutionException: com.datastax.driver.core.exceptions.InvalidQueryException: Partition key part level must be restricted since preceding part is'
[error]   DB.run(some)(PlayerID("1")).foreach(println)
[error]         ^
[error] one error found

The source of the problem seems to be a compiler bug when using Context#eval to probe the query.

Fixed by https://github.com/getquill/quill/pull/417


Throttle functions execution on blob trigger

Hi
Looking at the host file sample, I don't see any configuration parameters to throttle the execution of functions on blob triggers. Image I have a function that reacts to new blobs being uploaded in a storage account. I want to make sure that no more than 30 files are processed at the same time (this is to reduce the load on a backend system). How can I achieve such configuration in a Function?

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

ID: 27150c55-615f-fb44-d5e4-7b5815631c7d
Version Independent ID: 271006dc-afce-99f3-0790-41fa2e35b22b
Content: host.json reference for Azure Functions
Content Source: articles/azure-functions/functions-host-json.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Hi @murdockcrc Thank you for your feedback! We will investigate and get back to you with our findings.

Hey @murdockcrc - The recommendation is to use Event Grid trigger instead as it provides built-in support blob events. Please find more info on this here. You may also use Logic Apps and its connectors in conjunction with Azure Functions to achieve similar result as well.
We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


don't use underscores in Go names

Signed-off-by: yupengzte<EMAIL_ADDRESS>
- What I did
var _dataConfig_schema_v30Json change into _dataConfigSchemaV30Json
func dataConfig_schema_v30JsonBytes change into dataConfigSchemaV30JsonBytes
func dataConfig_schema_v30Json change into dataConfigSchemaV30Json
- How I did it
- How to verify it
- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@AkihiroSuda , thanks.
I think that cli/compose/schema/bindata.go  is to generate new data with data/config_schema_v3.0.json.
The code of cli/compose/schema/bindata.go is not generated by go-bindata.
"DO NOT EDIT!" means “DO NOT CHANGE THE DATA OF LINE 71”.
Is my understanding correct?

no
https://github.com/jteeuwen/go-bindata/blob/a0ff2567cfb70903282db057e799fd826784d41d/convert.go#L71

@AkihiroSuda ,OK, thanks.
Close this PR.


Move HRF utilities into their own package?

The HRF module is very useful, so we've bundled it into pybids (I didn't want to make nistats a dependency just for convolution) I imagine it would be beneficial to other people to have a small utility library that does just HRF convolution. @bthirion how would you feel about moving this into its own package? It would make it easier to maintain and ensure that we stay in sync.

To me the answer is no, for two reasons:

avoid multiplication of packages to maintain (this eventuelly adds work). We had this discussion a couple of years ago about Nilearn, as some people wanted to extract viz utilities from it. I'm glad we did not do it. And not that in the mid term, we plan to merge nistats into nilearn.
Nistats is a light dependence. What is the issue with depending on it ?
Best,
B

That's reasonable. Re: the dependency, nistats itself is light, but it depends on nilearn and sklearn, which are not so light (at least the latter).


Update oauth2-proxy to 7.1.3 & set default redis url if no one is defined in standalone mode

Replaces https://github.com/oauth2-proxy/manifests/pull/18

Well, I am not sure how the redis test could even work before. Trying to improve it...

Pfew, tests are passing. So this would require a new review since this contains a new feature.

Fixes #12

Why is everything frozen here? 😢

@pierluigilenoci because I think we lack maintainers!
I need a review for this one among the maintainers, maybe @NickMeves ?

@desaintmartin now there are two more! 🚀 ❤
@NickMeves @tlawrie @NickMeves

Would you like to help maintaining the chart? ;)

If you see fit, I can help.

@pierluigilenoci if you want, open a PR to add yourself as OWNER!

@desaintmartin https://github.com/oauth2-proxy/manifests/pull/32 done.
Let me know if it is enough.
Side note: the release linked to the changes of this PR is missing.

Fixed in https://github.com/oauth2-proxy/manifests/pull/33.


feat: add copy to clipboard button

✨ Codu Pull Request 💻

👉 Please remove the below and replace with your own values, leaving the headers where they are. 👈
Pull Request details:

Copy to clipboard was removed in the previous PR, I have added it back so that users can copy the article's link to the clipboard.

Any Breaking changes:

None

Associated Screenshots:
( Welcome file extensions include gifs/png screenshots of your feature in action )

@NiallJoeMaher Thanks man and please add the hacktoberfest and hacktoberfest-accepted label to the PR 🙌


Simplify (and fix) settings for transmission module

Why use sed when you can use nix ? :)

What does transmission say if you try to update its configuration from the webUI, now that its config is no longer writeable?
I'm actually a bit ambivalent towards this change. On one side, it is arguably more correct, but on the other, I sort of like the way the current module allows to change certain settings in the webUI.
I guess if the webUI is still useable for quick and dirty configuration changes (which are reset when the service restarts), then +1.

There's this error at startup:
Couldn't save temporary file "/nix/store/4sbdg08sfx8c4j9ybqcx270xsf0pn4w5-settings.json.tmp.GXcVD8": Read-only file system (variant.c:1227)`

I've tried changing some settings through the webUI, and they are correctly taken into account for the current session, but are reset when the service restarts, as you suggested.
This behavior complies with the configuration reproducibility principle of NixOS and I think it should be considered a feature rather than a bug.

There is the option of copying the file instead of symlinking, just for the sake of getting rid of that error message.

Ehm, copying + chmod I guess (chmod +w).

Indeed, copying the file removes the error message. No need to chmod though, from what I've tested.

Sorry to push, but is there any reason this PR is still pending validation ? If something is wrong, please let me know and I'll fix accordingly.

Haven't tested it but seems ok, merging. FYI you can cp --remove-destination.


I would like to know when we can obtain the code for SAM

Thank you for the great work. I am eager to see its performance on SAM. Could you please let me know when I can expect to experience the results? I would love to run it on my edge computing device. Thank you!

Hi,
We just released the first version of EfficientViT SAM models.
Best,
Han


feedback: support for cody and jupyter notebooks inside vscode

Version
any
Areas for Improvement

[X] UI/UX
[ ] Onboarding
[ ] Docs
[ ] Chat
[ ] Commands
[ ] Context
[ ] Response Quality
[ ] Other

What needs to be improved? Please describe how this affects the user experience and include a screenshot.
support for command completion in jupyter notebooks inside vscode, similarly to what is possible with Github Copilot
Describe the solution you'd like to see
same as above
Describe any alternatives that could be considered
No response
Additional context
No response

@faysou I'm sorry for the delayed reply. This issue should be resolved by the PR, which will become available in the next release https://github.com/sourcegraph/cody/pull/5473


Add unit tests for runtime/runtime.go and grpc/api.go

What would you like to be added:
Add unit tests for runtime/runtime.go and grpc/api.go.
These two files are the core engine code of Layotto, responsible for component lifecycle management and grpc request processing respectively.
chinese:
为runtime/runtime.go和grpc/api.go添加单元测试。这两个文件是layotto的核心引擎代码，各自负责组件生命周期管理（启动、注册、初始化）和grpc请求处理，完成此任务可以很好的了解Layotto架构
Why is this needed:
Currently our test coverage is not high :(

Assigned to @tianjipeng
Thanks for your passion!

OK

@tianjipeng  Hi, are you still working on it?

I wiil commit a pr this weekend

I wiil commit a pr this weekend

Fine.If you find some code or module difficult to understand,just tell me and I will add comments and document for that module first.


[Bug]:  Go SDK Quickstart is outdated

What happened?
The Get the SDK and the examples section of Go SDK Quickstart is not accurate, the document is outdated. Here is the link: Get the SDK and the examples
When I followed the instruction and tried to run the command go get -u github.com/apache/beam/sdks/v2/go/pkg/beam on my machine in an empty folder, I got the following error:
go: go.mod file not found in current directory or any parent directory.
        'go get' is no longer supported outside a module.
        To build and install a command, use 'go install' with a version,
        like 'go install example.com/cmd@latest'
        For more information, see https://golang.org/doc/go-get-install-deprecation
        or run 'go help get' or 'go help install'.

However, when I tried to run go install github.com/apache/beam/sdks/v2/go/pkg/beam@latest, I got the following error:
go: downloading github.com/apache/beam/sdks/v2 v2.43.0
go: downloading github.com/apache/beam v2.43.0+incompatible
go: downloading github.com/golang/protobuf v1.5.2
go: downloading google.golang.org/protobuf v1.28.1
go: downloading github.com/google/uuid v1.3.0
go: downloading google.golang.org/grpc v1.50.1
go: downloading google.golang.org/genproto v0.0.0-20221010155953-15ba04fc1c0e
go: downloading golang.org/x/net v0.0.0-20221012135044-0b7e1fb9d458
go: downloading golang.org/x/sys v0.0.0-20220728004956-3c1f35247d10
go: downloading golang.org/x/text v0.4.0
package github.com/apache/beam/sdks/v2/go/pkg/beam is not a main package

I believe this section of documentation is not very friendly for beginners to get started. Could anyone fix this issue?
Go version: go1.18.1 linux/amd64
OS version: Ubuntu 20.04
Issue Priority
Priority: 2 (default / most bugs should be filed as P2)
Issue Components

[ ] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

I think the original command

go get -u github.com/apache/beam/sdks/v2/go/pkg/beam

could work if you have initiated your go module using "go mod init"

I think the original command

go get -u github.com/apache/beam/sdks/v2/go/pkg/beam

could work if you have initiated your go module using "go mod init"

Yes, this works. Thanks for your comment.


Run from zip

@nturinski I assume you can close this in favor of https://github.com/Microsoft/vscode-azurefunctions/pull/320?


Abstract functionality for timedWalk and other researchkit tasks

timedWalk abstraction
:gear: Release Notes

Add functionality for timedWalk ResearchKit task.

:books: Documentation

Alter existing code for timedWalk ResearchKit task.

:white_check_mark: Testing
:pencil: Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

[x] I agree to follow the Code of Conduct and Contributing Guidelines.

@dguo8412 Thank you for all the work here! Feel free to tag me for an other review once you have finished the UI tests and addressed all other red warnings. Great to see the progress in this PR and to have you in the office to work on it! 👍


It seems that the line.spacer() doesn't work correctly

DEMO
https://user-images.githubusercontent.com/24765272/204909458-341235b7-e6af-4207-9f38-ff0684aa918c.mov
Description
Notice that when I toggled between tabs 1 and 2, the position of 1 and 2 changed. I think the culprit is that when the length of the current buffername changed, the spacer seems to still add the same number of spaces. In the demo, since the buffername plugins.lua is longer than [No Name], both 1 and 2 were pushed more to the right.

Can you show me your config?

@nanozuki
  use {
    'nanozuki/tabby.nvim',
    config = function ()
      local theme = {
        fill = 'TabLine',
        head = 'TabLine',
        current_tab = 'TabLineSel',
        tab = 'TabLine',
        win = 'TabLineSel',
        tail = 'TabLine',
      }
      require('tabby.tabline').set(function(line)
        return {
          {
            { '    ', hl = theme.head },
          },
          line.wins_in_tab(line.api.get_current_tab()).foreach(function(win)
            if not vim.api.nvim_buf_get_option(win.buf().id, 'buflisted') then return {} end
            local hl = win.is_current() and theme.win or theme.fill
            local rhl = win.is_current() and theme.fill or theme.win
            return {
              line.sep(win.is_current() and '' or '', hl, rhl),
              win.is_current() and ' ' or '',
              win.buf_name(),
              line.sep(win.is_current() and '' or '', hl, rhl),
              line.sep(' ', theme.fill, theme.fill),
              hl = hl,
              margin = '',
            }
          end),
          line.spacer(),
          line.tabs().foreach(function(tab)
            local hl = tab.is_current() and theme.current_tab or theme.tab
            local rhl = tab.is_current() and theme.tab or theme.current_tab
            return {
              line.sep(tab.is_current() and ' ' or '', hl, rhl),
              tab.number(),
              line.sep(tab.is_current() and ' ' or '', hl, rhl),
              line.sep(' ', theme.fill, theme.fill),
              hl = hl, margin = '',
            }
          end),
          {
            line.sep(' ', theme.tail, theme.tail),
            line.sep(' ', theme.tail, theme.tail),
          },
          hl = theme.fill,
        }
      end)
    end
  }

Sorry for late, I've been a little busy in my daily life lately.
the problem is, the label's width of current win and non-current win is not equal. One is '1', another one is '2'. So, then the current window changed, the label's position is changed.


Wrong result in web_search

Hello again!
I just tryed libvot examples on more bigger image collection. See what i get.

Result isn't even close. But when i try to compare query image and "top ranked" image from result by native opencv functions (same libvot used, but without convertion to SiftData structure, just Flann computing of descriptors), i got false, and true with only same pictures. It looks like something went wrong ))))

Can you share the dataset for me to test it? Also note that we now have a official documentation for your reference.

Yeah, shared it on dropbox https://www.dropbox.com/s/khvc1jibih1en5z/dataset.zip?dl=0

Thanks for this dataset. I have tested it on my machine. It does not performance very well for these unrealistic photos. One reason is that the vocabulary tree algorithm is just based on a bunch of salient features and doesn't go that far into the semantic meaning. It also shows that computer vision is far from mature and count on you to make it better :) If you are interested in the algorithm behind, you can check out http://www-inst.eecs.berkeley.edu/~cs294-6/fa06/papers/nister_stewenius_cvpr2006.pdf

FYI, this is the query result of 254.jpg returned by google image:https://www.google.com/search?newwindow=1&hl=en&tbs=simg:CAESzgEJLmCfkwhyEUgawgELELCMpwgaYgpgCAMSKNsL1gv9FdcV_1BXcC_1gLwwuNHvMU0ijGItMowyL6KtkhpyqtKtgh0CEaMI9dT0mkWBxn1ipBUCkNjQLUfeiUyY7NCvUwBYDzGCO5FIMud3ifFEu_12UxYSNR_1niADDAsQjq7-CBoKCggIARIENNoNJgwLEJ3twQkaOwoICgZuYXR1cmUKCAoGZ2FyZGVuCggKBmZsb3dlcgoSChBib3RhbmljYWwgZ2FyZGVuCgcKBXBsYW50DA&q=egidio+antonaccio+paintings&tbm=isch&sa=X&ved=0ahUKEwiqr-Xok6rNAhXDJZQKHVY_Ai4Qsw4ILw&biw=1440&bih=812&dpr=2
As you can see, the query result is acceptable since the images you give me are already in a very similar style. It just cannot differentiate the fine nuances. It is sort of related to how you define similarity.

So the parameters of vocab tree builder are the point.

Yeah. I just make sure about opencv tools natively doesnt care about pictures'es semantic. Look at this screen: its almost 100% precise recognizing (all non-254.jpg pictures have 0 matches, except 2 or 3 pics wich have 1-3 matches).


Explicit error message when source doc is not an XSpec file

This pull request derives from #532. So needs to be handled after that.

Closes #522
With this cange, XSpec leaves this message when the source document is not an XSpec file:
C:\xspec>bin\xspec.bat test\do-nothing.xsl
Saxon script not found, invoking JVM directly instead.

Creating Test Stylesheet...
Source document is not XSpec. /x:description is missing. Supplied source has /xsl:stylesheet instead.
...

Commits

286af69907c3ea7bc20de7e4ed6f815aed443eb8 adds the message.
a3a0d0ffedf4864ecdc35ad8254f53108a4d5b8c tests the message. Before implementing the message, the test fails as expected: Travis, AppVeyor

@cirulls
Merged this enhancement.


Question: Should this "Note" be a callout?

At the start of the PhoneGap Desktop App Install guide:

The same exists for the PhoneGap CLI Install guide.

Fixed


Text in Object value is not found in some wired cases

Describe the bug
This happened when I tried to search for a word in multiple objects. All of the objects contained the key body which contains markdown text. In some of the field the word is found in other cases it is not. I guess it has to do something with the special characters used in markdown. It can be best seen in my example down below!
Version
Fuse: 6.4.0
Node Version: 12.16.1
Same result in Firefox: 78
Is this a regression?
Stumbled up on this the first time
🔬Minimal Reproduction
const Fuse = require('fuse.js');

const data = [{
    "title": "Title 1",
    "body": "# 111111 \n bla \n \n asdf\n \n asdf\nThis will be found",

}, {
    "title": "Title 2",
    body: "# 22222 2\n- [x] asdf\n- [x] asdf\n- [x] 123ed\n\t- [ ] asdf\n\t- [ ] asdf\n\t\t- [ ] asdf\n\t\t\t- [ ] asdf\n\n- asdf\n- asdf\n- asdf\n\nasdf\n\nasdf\n\nThis will not be found",
}, {
    "title": "Title 3",
    "body": "# 33333 \n bla blupp asdf asdf asdf\n \n asdf\n asdf",
}]

const fuse = new Fuse(data, {
    keys: [
        'title',
        'body',
    ]
});

console.log(fuse.search('found'));

Result:
$ node index.js
[
  {
    item: {
      title: 'Title 1',
      body: '# 111111 \n bla \n \n asdf\n \n asdf\nThis will be found'
    },
    refIndex: 0
  }
]

Expected
At least that the second object is also found since ther word found is also present in its body
Additional context
The content of the body field looks so strange since its markdown. My guess is that this is the cause of the problem but i was not able to narrow it down to a smaller example for the second object!

Not a bug. Please read the scoring theory to understand why this is happening 😄


Drag-and-drop a file with a double-quote char in its name won't open it in Brackets.

Launch Brackets.
Drag a file with double-quote in its name (eg. my"file.js) and drop it inside Brackets window.

Result: The file is not open in Brackets.

I am new to open source development and node.js. I am starting off with starter and low priority bugs to familiarize myself with the codebase. I am looking into this issue and have made some progress. However I am stuck at debugging a callback and would need some help. Could some one ping me on slack for a quick discussion?


Rename file node links

d -> content and 'f->meta` (fixes #715)
Some other changes in here:

Replaces errors.New with fmt.Errorf (it was mixed, go vet likes the former)
Ensures schemas are loaded w/o blocking startup (fixes #733)
Cleans up server profile application
Adds some temp logging to the cafe HTTP API to try and track down those odd 400s
Ensure mobile profile is set when needed (some new settings to try, so need to overwrite the config file) (fixes #735)
Enables blockstore GC (fixes #241)

There were the following issues with your Pull Request

Commit: d3e86e7dd968d8c3781da8e33d12de38d78b3089
Commits must be in the following format: %{scope}: %{description}

This message was auto-generated by https://gitcop.com

Tagged a couple more on here:

Ensure mobile profile is set when needed (some new settings to try, so need to overwrite the config file) (fixes #735)
Enables blockstore GC (fixes #241)


Adding p-o-c UI playlist implementation

Hi, long time no see.
I'm sending here some work I've done lately.
Please let me know if you see any troubles (I'll be fixing them as we go).
This is a lot of code so please take your time to verify this PR.

I took a quick look at it with #v=ZNno63ZO2Lw&t=54s;1m20s+1m33s;1m47s+3m30s;3m46s&v=TM1Jy3qPSCQ&t=2s;16s and I really like the minimalistic slider and nice drag&drop :+1:
I'll test it more thoroughly in the evening, but I think we will merge it for sure. :rocket:

You make my day man, thank you! xD


Implement TLS session resumption via config or add another module to do it

See: https://nodejs.org/api/tls.html#tls_event_resumesession
We may want to implement TLS session resumption in another module (e.g. bedrock-tls-session-storage-foo) ... it may also be that this sort of thing is best handled at another layer of infrastructure for most setups.

Closing since deployment infrastructure typically handles this sort of thing automatically.


[Mangling] Mangle types as generic only when they have generic arguments.

The isSpecialized() check didn’t account for the possibility that a
typealias in a parent of a nominal type could be non-generic when the
parent declaration was generic, leading to incorrect mangling that stated
that they were generic but had no generic arguments.
Fixes SR-8930 / rdar://problem/45216653 and rdar://problem/45813164.

@swift-ci please test and merge


feat: renderSection option to retained data

Description
This PR is in reference to reported issue to resolve #7069, Added a new feature on renders a section's contents using renderSection('page_title', true) to prevent from auto cleans the data after displaying.
Added 2nd argument bool $saveData If is true, saves data for subsequent calls, if false, cleans the data after displaying.
Checklist:

[x] Securely signed commits
[x] Component(s) with PHPDoc blocks, only if necessary or adds value
[x] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

@addngr Thank you for sending this PR!
Can you add the changelog v4.4.0 based on the following?
https://github.com/codeigniter4/CodeIgniter4/blob/develop/admin/next-changelog-minor.rst
and add about this in the Enhancements section?

@kenjis, Thank you for quick response. Please have look changelog v4.4.0 added.

@addngr Thank you!
Generally Okay, but I commented inline on a few points.

Build error occurred.

/github/workspace/user_guide_src/source/changelogs/v4.4.0.rst: WARNING: document isn't included in any toctree
https://github.com/codeigniter4/CodeIgniter4/actions/runs/3932931900/jobs/6729062065

Please add v4.4.0 in the index file:
https://github.com/codeigniter4/CodeIgniter4/blob/develop/user_guide_src/source/changelogs/index.rst

@addngr Thank you!

@kenjis Thank you for your valuable guidance. I have been successfully completed my 1st contribution to open source project. I have learned a lot by doing a contribution to such great work.

@addngr Your PR was no problem. It was very well. 👍
In fact some people can not complete their PRs with lack of skills.
I think you could contribute most of PHP open source projects.


Code refactoring?

Hey @limexp,
First off: awesome work on this! The fact that you were able to translate C# to Python — and carry over all the features from xgbfi — is really impressive. 👏
I was wondering if you'd be cool with major refactoring of the code? I'd love to translate most of it over to PEP 8 and introduce unit tests.
I only ask outright because it'd involve quite a lot of renaming functions/methods/classes.
Zach

Hey @zachwill,
Thank you!
I had a hard year, so didn't pay due attention to this project.
The code base is not big, so there is no problem in refactoring. But one of the difficult questions from the very beginning was the choice of the right interface. it would be great if you share your wishes and remarks.
I'll set up CI this week and start refactoring after this.
Boris

Moving the conversation over to pull request #7.


Ask questions

Localtion attribute in chromium is unforgeable, how to modify it? thank you

This is an old issue, that didn't show much activity recently — closing.
If you have any more information to add, let us know.


'invalid algorithm' when using dat publish with SSL

From @karissa on January 26, 2017 22:42
karissa ~/examples/tweet-data
$ dat publish --server=https://beta.datproject.org/api/v1
name: some-tweet-data
some-tweet-data
Publishing archive with name "some-tweet-data".
invalid algorithm

@mafintosh said it might be an SSL issue because we haven't seen this in the local tests.
Copied from original issue: joehand/dat-next#107

Fixed on the township side, needed to specify the algorithm ES512


fix: code block node view

Fix for code block not using code mirror.
This appears to be regression from recent refactoring.

Hey @ncphillips, @DirtyF  : I just added fix for hybrid controlled behaviour.


chore: preview command enablement

chore: preview command enablement
This PR

Renames dendron.showPreview to dendron.showLegacyPreview
Renames dendron.showPreviewV2 to dendron.showPreview
Adds enablement / when clauses for both dendron.showLegacyPreview and dendron.ShowPreview

dendron.showLegacyPreview will now only appear in the command palette / be able to trigger through keybinding when Dendron is active and the user has Dendron Markdown Preview Enhanced Installed and the current open file extension is .md.
dendron.showPreview will now only appear in the command palette when Dendron is active and the current open file extension is .md.

Unfortunately, VSCode doesn't provide an API for getting context keys or dynamically determine the enablement of contributed features, so I'm resorting to manually testing these with a prerelease build.
The .vsix can be found here: https://github.com/dendronhq/dendron/actions/runs/1401224003
Manual Testing
GIVEN markdown-preview-enhanced installed
  WHEN Dendron is active
    AND markdown file is open
      - [x] THEN Show Preview (legacy) is enabled in command palette
      - [x] THEN Show Preview (legacy) keybinding is enabled
    AND non-markdown file is open
      - [x] THEN Show Preview (legacy) is disabled in command palette
      - [x] THEN Show Preview (legacy) keybinding is disabled
    AND no file is open
      - [x] THEN Show Preview (legacy) is disabled in command palette
      - [x] THEN Show Preview (legacy) keybinding is disabled
  WHEN Dendron is not active
    - [x] THEN Show Preview is disabled in command palette
    - [x] THEN Show Preview keybinding is disabled
    - [x] THEN Show Preview (legacy) is disabled in command palette
    - [x] THEN Show Preview (legacy) keybinding is disabled
GIVEN markdown-preview-enhanced not installed
  WHEN Dendron is active
    - [x] THEN Show Preview (legacy) is disabled in command palette
    - [x] THEN Show Preview (legacy) keybinding is disabled
    AND markdown file is open
      - [x] THEN Show Preview is enabled in command palette
      - [x] THEN Show Preview keybinding is enabled
    AND non-markdown file is open
      - [x] THEN Show Preview is disabled in command palette
      - [x] THEN Show Preview keybinding is disabled
    AND no file is open
      - [x] THEN Show Preview is disabled in command palette
      - [x] THEN Show Preview keybinding is disabled
  WHEN Dendron is not active
    - [x] THEN Show Preview is disabled in command palette
    - [x] THEN Show Preview keybinding is disabled
    - [x] THEN Show Preview (legacy) is disabled in command palette
    - [x] THEN Show Preview (legacy) keybinding is disabled

Pull Request Checklist
You can go to dendron pull requests to see full details for items in this checklist.
General
Quality Assurance

[~] add a test for the new feature
[x] make sure all the existing tests pass
[x] do a spot check by running your feature with our test workspace
[x] after you submit your pull request, check the output of our integration test and make sure all tests pass

NOTE: if you running MacOS or Linux, pay special attention to the Windows output and vice versa if you are developing on Windows

Special Cases

[~] if your tests changes an existing snapshot, make sure that snapshots are updated
[x] if you are adding a new language feature (graphically visible in VS Code/preview/publishing), make sure that it is included in test-workspace. We use this to manually inspect new changes and for auto regression testing

Docs

[x] Make sure that the PR title follows our commit style
[x] Please summarize the feature or impact in 1-2 lines in the PR description
[x] If your change reflects documentation changes, also submit a PR to dendron-site and mention the doc PR link in your current PR

doc changes: https://github.com/dendronhq/dendron-site/pull/258

quick draft for announcement @ScriptAutomate :) feel free to modify as you see fit.
We have updated the enablement of our preview features to reduce clutter in the command palette and prevent confusion.

Dendron: Show Preview is now called Dendron: Show Preview (legacy)
Dendron: Show Preview V2 is now called Dendron: Show Preview
Both preview commands and their keybindings are now disabled if you are in a non-Dendron workspace or if your active open document is not a markdown file. This means they will not be accessible in the command palette or triggered with a keybinding.

related: https://github.com/dendronhq/dendron/issues/1545

I'll include the removal of dev.enablePreviewV2 as part of the upcoming config migration. Currently there's no place that checks for it.


Add WPF lib that works on thread pool except for the reaction

In WPF only the state relevant for the ui should be handeled on the UI thread. Every thing else should be done on workers.
Application.Current.Dispatcher.Invoke(new Action(() => { /* Your code here */ }));

provide dependencies as seperate params so that they can be evaluated independently.


Remove mount beta notification from docs

Remove preview note:
lakeFS DocumentationlakeFS Documentation
Mount
This section covers the Everest feature for mounting a lakeFS path to your local filesystem.

Keren asked me to remove the "beta"from here


Azure File Copy build task not available on hosted Ubuntu build agent

Environment

Server - Azure Pipelines
Account name: chaosbyte
Team project name: D Test
Build definition name: D Test-CI
Build number: 9

Agent - Hosted:
Queue name: Not sure
It is hosted a Ubuntu 16.04 build agent

Issue Description
I got a project building some pulling from an external Git repository, and building it with gcc through a simple bash script - no problem.
However, when I attempt to use the Azure File Copy task to copy over the single output file to a Azure Blob Storage container, I am told that this task won't run on Linux.
Is there any plans to make Azure File Copy work on the hosted Ubuntu 16.04 build agent, or do you have any other suggestion on how to achieve the same result with existing tasks that will run on this hosted build server?
Error logs
2018-11-27T18:03:41.7033273Z ##[section]Starting: AzureBlob File Copy
2018-11-27T18:03:41.7038323Z ==============================================================================
2018-11-27T18:03:41.7038468Z Task         : Azure File Copy
2018-11-27T18:03:41.7038567Z Description  : Copy files to Azure blob or VM(s)
2018-11-27T18:03:41.7038647Z Version      : 2.0.15
2018-11-27T18:03:41.7038738Z Author       : Microsoft Corporation
2018-11-27T18:03:41.7038823Z Help         : [More Information](https://aka.ms/azurefilecopyreadme)
2018-11-27T18:03:41.7038939Z ==============================================================================
2018-11-27T18:03:41.7244544Z ##[error]The current operating system is not capable of running this task. That typically means the task was written for Windows only. For example, written for Windows Desktop PowerShell.
2018-11-27T18:03:41.7258675Z ##[section]Finishing: AzureBlob File Copy

@benne you could use the az cli to do the file copy and that task will work on Linux

@chrisrpatterson az cli might be my best option for now, using a non-interactive service principal. I am gonna experiment a bit with this.
Still interested in if this is gonna make it to the road-map at some point. It would be nice to use the build-in build task for future projects :)

@benne I will leave the roadmap for that task up to the team that owns the task.  The az cli task should work with the same service principal you would have setup for the Azure File Copy task.

@chrisrpatterson I went with a service principal with specific permissions to contribute to a specific storage account. Combined with an Azure CLI build task I was able to upload the assets to the blob storage.
Success!

@benne - Making Azure File copy xPlat is in our backlog. Adding @RoopeshNair . Until then using Azure CLI will be the right way to solve.

azcopy is available on Linux
https://aka.ms/downloadazcopy-v10-linux

if we are using the az cli to achieve the same functionality, we are not getting the same flexibility as the Azure File Copy task in terms of getting the storage uri and sastoken as output. We need to make additional cli calls to achieve the same . So, it would be great if the file copy task works xPlat.

The AzureFileCopy task and the az cli workaround is not at feature parity with each other. I wonder how many teams, like I just did, spent time adding the task to the pipeline, watching it fail and then searching the internet for a couple of hours.  This is how I found this issue. The docs on this are also weak.

Is there any progress to this?
It is very strange that tasks like JavaToolInstaller are able to copy installer file from Azure Storage but we can't use AzureFileCopy to copy files to Azure Storage.

Trying to migrate our pipelines to use self-hosted Linux agent and discovered that its failing due to AzureFileCopy not being supported on Linux. It would be great if this can be made to work with Linux!

Another bump for an update on this.

I've written a YAML task template for Azure Pipelines that fills this gap for now. You can find it on my blog here: https://www.alexangas.com/blog/missing-azurefilecopy-azcopy-task-solved/
Feedback is very welcome! Please raise as an issue on the repo for my blog rather than polluting this thread, thanks.

For the time being, using windows image as a workaround:
  - deployment: deploy_staging
    environment: staging
    pool:
      vmImage: windows-latest
    strategy:
      runOnce:
        deploy:
          steps:
          - download: current
            artifact: $(artifactName)

          - task: AzureFileCopy@3
            inputs:
              sourcePath: $(Pipeline.Workspace)/$(artifactName)
              azureSubscription: $(azureSubscription)
              destination: azureBlob
              storage: memes
              containerName: $web

This also left me very confused.  This obviously should work on linux hosts.

More than a year later, the docs are still stale. I’m in the same boat, anyone found any workarounds for this issue ?

is there an update on this?

I wanted to share my own solution to this that works with vmImage:ubuntu-latest:
  - task: AzureCLI@1
    displayName: Az File Copy to Storage
    inputs:
      azureSubscription: $(azureSubscription)
      scriptLocation: inlineScript
      inlineScript: |
        az storage blob upload-batch \
          --destination \$web \
          --account-name "$(storageAccountName)" \
          --source "$(Agent.BuildDirectory)/$(outputDir)"

Critical thing here is that Azure File Copy will return a SAS Token allowing you to then access these files via ARM deployment.

Thanks for sharing your workarounds! Yet, I am still waiting for the AzureFileCopy task on Linux agents because I can't always remember this issue until I see another failed pipeline. :)

umm, have been hoping to see some kind of update on this for over a year, what gives guys, can we get some support on this? GA opened since azcopy v10 for linux, why couldnt this be integrated?

+1, it would be great to see an update on this

This is a terrible experience for your customers. Why is something so basic as copying files not supported on Linux agents the same as it is supported on Windows?

@vijayma pls triage

+1

+1

+1

+1

+1

+1

+1 (i can already see the headlines in the washington post newspaper in the year 2038 - "Microsoft has finalized file copy for linux pipelines - it is now in the testing stage and soon ready for RC stage") :-)

Switched to Windows hosts for the specific project, but would still love to see a cross-platform solution that just works as intented :)

@jamiehaywood thank you so much! Does anyone know how I can narrow down copying to a repo's subfolder? I'm trying this now with wild guesses and it's either erroring out or copying all the unnecessary files, which ends up taking over 60 minutes and erroring out anyway.

+1, please allow to run on linux

+1

+1

Any chance this is finally being addressed in v5? 🤣🙄

Years & years later :(

I wonder if the "roadmap" is that kind of map of "where do you see yourself in 10 years"... a bunch of tasks work flawlessly in Linux hosts, but this one gets no attention

+1

+1

+1

+1

Four and a half years (including a whole pandemic) and this still isn't working? Is it still on the backlog @kmkumaran ?

We changed our pipelines to use PowerShell (since now PowerShell core works in both OSes) and used az commands to do the copy instead. Working good so far.

@starkmsu @onetocny @DmitriiBobreshev think anyone can possibly look into this issue from 2018?

AzCopy may be used directly from AzureCLI@2 as follows
- task: AzureCLI@2
  displayName: 'AzCopy via Azure CLI'
  inputs:
    azureSubscription: 'AzCopyARM'
    scriptType: 'pscore'
    scriptLocation: 'inlineScript'
    inlineScript: |
      $Date = (Get-Date).AddDays(1).ToString('yyyy-MM-dd')
      $key = $( az storage account keys list --resource-group $(rgName) --account-name $(storageAccount) --query [0].value -o tsv )
      $sas = $( az storage container generate-sas --name $(container) --expiry $Date --permissions "racwdli" --account-name $(storageAccount) --account-key "$key" )
      azcopy copy "./seed_images/*" "https://$(storageAccount).blob.core.windows.net/$(container)/from_az_cli?$(SAS)" --recursive=true

See docs:

https://learn.microsoft.com/en-us/azure/devops/pipelines/tasks/reference/azure-cli-v2?view=azure-pipelines
https://learn.microsoft.com/en-us/azure/storage/common/storage-use-azcopy-v10

The following seems to work to "copy" text files (or shell scripts, or ....) to a VM from a pipeline, using a pipeline variable:
Disclaimer: I must admit I scared myself by doing this. Far from clean, but at least it seems to work.
jobs:
- job: TextCopyToVM

  steps:

  - bash: |
      filePath="$(< path/to/some/text/file/from/git/repo.txt)"
      #
      # Replace \n with %0D%0A, so a multiline pipeline variable is possible
      #
      escapedFile=$(echo "$filePath" | sed ':a;N;$!ba;s/\n/%0D%0A/g')
      echo "##vso[task.setvariable variable=localFileEscaped]$escapedFile"

  - task: AzureCLI@2
    displayName: 'Echo var into file'
    inputs:
      azureSubscription: '<Azure Subscription Name>'
      scriptType: 'bash'
      scriptLocation: 'inlineScript'
      inlineScript: |
        echo "localFileEscaped: $(localFileEscaped)"
        COPY_FILE="$(az vm run-command invoke -g <RESOURCE GROUP> -n <VM NAME> --command-id RunShellScript --scripts 'echo "$(localFileEscaped)" > /some/path/to/file/on/vm.txt')"

Since this is still a thing with a recent comment and since I had same problem landing eventually here, I would like to tell that

@kolosovpetro solution worked great
but only when I changed network access rules with
az storage account update --name $ACCOUNT --resource-group $RESOURCE_GROUP --default-action Allow

otherwise the error you get is just telling you that you have authentication problem. This might save you a lot of hours thinking about your live choices while you are trying to resolve this.

We still want this feature ;)

+1

Yes we want this feature.

Also interested in seeing this given some love from Microsoft.


Remove circuit relay service boot delay config key

The bootDelay config key shouldn't be necessary any more as if the initial DHT self-query hasn't run, it queues any queries for until after it's run.

Closing in favour of #2545


Fix color contrast issues in text

The success button green is too light and the gray for the paths
in the quicklooks is too light of a gray.
Dano mentioned he wanted a different solution for the green but I
think we should fix accessibility issues as soon as possible.
Refs https://github.com/18F/cg-dashboard/issues/921

I'm okay with the grey space text change, but the green color looks too dark. Especially for the icon and the status-pill. I think it's okay to go dark green for contrast regarding text, but I don't think we should make the change across the board.
Generally I don't think any of the green text is critical for accessibility so I would rather hold off on the green change for now and wait until we have a more complete color palette solution.

Even though the green status text isn't that important, I'm still not sure I want to leave it inaccessible. If somebody has trouble reading it, they might not know it's not important. I separate the green text color from the rest of the green.


cmake: Fix cache directory detection

To detect where the cache directory has been located we have been
checking if $HOME is writable, and if it is assuming that $HOME/.cache
must be writable as well.
This is broken in environments where $HOME is owned by users and
$HOME/.cache is owned by admin. To fix this we check $HOME/.cache for
write-ability instead.
Signed-off-by: Sebastian Bøe<EMAIL_ADDRESS>

Why do we cache to home in the first place and not, say, the build directory?

Because we want to store some files persistently between builds to speed up the build.


Problems encountered in using vite plugin PWA in vue3

Use vite plugin PWA in vue3. Every time you update the business code, vite will hot update and refresh the browser. At this time, a series of exceptions will be thrown in the browser (exceptions are as follows)
Basic configuration
// pwa workbox config

// VitePWA

// main.js

The exceptions thrown are similar to the following exceptions

ReferenceError: Cannot access 'router' before initialization
This problem has bothered me for a long time. I want to know how to solve this problem. Thank you

Remove the dev options. This option was included to test your custom sw and the webmanifest. Since we cannot use hmr on sw in any browser, each time the sw is regenerated will trigger a full reload.
You should test the sw with a local build once the app is finished.
Next release will address the errors,  since we're caching the assets, and we should only cache the entry point in the sw precache manifest, check #335

@ZhaoHongzcyh try with latest version 0.12.4, review the navigation fallback option in devOptions, read the dev entry in the docs, has been updated.


implements Buyable and extends OrderItem crashes Dev/Build

Hi everyone, I’ve been trying to use SilverShop in SS4.13, I have no issue with my code until I use “implements Buyable” or “extends OrderItem”.
Basically whenever I use these, I can’t finish the Dev/Build process and I have no idea how to move forward from there. Using pages as products would be way too complicated in that project so I cannot use that solution.
My guess is SilverShop isn’t fully compatible with SS4.13 and that’s why the build keeps crashing.
I leave the code I have even though I don’t think it will be helpful, I also tried creating a class extending OrderItem with no luck.
Any insight is welcome. Thanks in advance.
use SilverShop\Model\Buyable;
use SilverShop\Model\OrderItem\OrderItem;
class RentalReservation extends DataObject implements Buyable {
private static $db = [
    'Title' => 'Varchar(128)',
    'CheckInDate' => 'Date',
    'CheckOutDate' => 'Date',
    'GuestsCount' => 'Int',
    'Price' => 'Currency'
];

private static $has_one = [
    'Client' => Member::class,
    'Rental' => RentalObject::class
];

private static $summary_fields = [
    'Client.Name' => 'Client',
    'CheckInDate.Nice' => 'Reservation Check-in',
    'CheckOutDate.Nice' => 'Reservation Check-out'
];

}

Can you please provide the specific error message you're seeing? You should see it if your site is in dev mode or if you have error logging turned on

Thanks for your fast reply,
Actually been checking the "vendor/silvershop/core/tests/php/Model/Product" folder and with the exemples, I've been able to implement the Buyable and it does go through Dev/Build
Hopefully the module has some good practical exemples.

Anyone struggling with Buyable, just check the folder "vendor/silvershop/core/tests/php/Model/Product" folder and take example on CustomProduct.php and CustomProduct_OrderItem.php


No module named 'elftools'

Test build macOS with gcc-arm-none-eabi
make BOARD=stm32f469i_disco
CHK     include/generated/generated_dts_board.conf
Using /Users/wenchao/zephyr as source for kernel
GEN     ./Makefile
CHK     include/generated/version.h
CHK     include/generated/generated_dts_board.h
CHK     misc/generated/configs.c
CHK     include/generated/offsets.h
Traceback (most recent call last):
File "/Users/wenchao/zephyr/scripts/gen_offset_header.py", line 8, in
from elftools.elf.elffile import ELFFile
ModuleNotFoundError: No module named 'elftools'
make[3]: *** [include/generated/offsets.h] Error 1
make[2]: *** [prepare] Error 2
make[1]: *** [sub-make] Error 2
make: *** [all] Error 2
From master branch

pip install pyelftools

please update wiki and close issues

what wiki were you looking at? This is documented already here: https://www.zephyrproject.org/doc/getting_started/getting_started.html

wiki no add pyelftools

pip3 install -r scripts/requiements.txt，software requied to build zephyr list in requiements.txt

wiki no add pyelftools

what wiki are you referring to?

sorry，i read doc，1.9.0 no release。I mistakenly read the 1.8.0 doc

pip3 install -r scripts/requiements.txt

The getting started page linked above DOES NOT even mention elftools

It does not help to have pyelftools.
pip install pyelftools
Requirement already satisfied: pyelftools in /home/USERNAME/zephyr/venv/lib/python3.10/site-packages (0.28)
make -j
[  1%] Built target version_h
[  3%] Built target parse_syscalls_target
[  6%] Built target syscall_list_h_target
[  6%] Generating include/generated/driver-validation.h
[  6%] Generating include/generated/kobj-types-enum.h, include/generated/otype-to-str.h, include/generated/otype-to-size.h
Traceback (most recent call last):
Traceback (most recent call last):
File "/home/USERNAME/zephyr/zephyr/scripts/gen_kobject_list.py", line 62, in
File "/home/USERNAME/zephyr/zephyr/scripts/gen_kobject_list.py", line 62, in
import elftools
import elftools
ModuleNotFoundError: No module named 'elftools'
ModuleNotFoundError: No module named 'elftools'
make[2]: *** [zephyr/CMakeFiles/driver_validation_h_target.dir/build.make:73: zephyr/include/generated/driver-validation.h] Error 1
make[2]: *** [zephyr/CMakeFiles/kobj_types_h_target.dir/build.make:74: zephyr/include/generated/kobj-types-enum.h] Error 1
make[1]: *** [CMakeFiles/Makefile2:1489: zephyr/CMakeFiles/driver_validation_h_target.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
make[1]: *** [CMakeFiles/Makefile2:1515: zephyr/CMakeFiles/kobj_types_h_target.dir/all] Error 2
make: *** [Makefile:91: all] Error 2

I actually had to install it outside of the virtualenv also for make to find it. This is obviously not how things are supposed to work

I had this problem just now (mid 2022) on a clean MacOS install.   It turned out to be because I had two copies of west present, one of which explicitly loaded the system python in /Library/Developer/CommandLineTools and one of which used /usr/local/the/actual/python/I/wanted.   Check the output of "head which west" and see what python west is running with.
My solution was to get rid of the broken west in my path.

Thanks @unixbigot I had a similar problem, was running inside a virtual environment and had west installed in the system causing this. I uninstalled west from my system and things worked again.

Hi, having same issue on a fresh installation on today, begin of 2024.
west --version
West version: v1.2.0
ModuleNotFoundError: No module named 'elftools'
I have a single copy of west installed.
and nothing changed by "pip3 install pyelftools"

My M1 Mac also has a same issue but solved simply pip install pyelftools outside the venv.

I am using Rocky Linux 8.
The documentation, by default (choosing the option locally), suggests to run pip install -r ~/zephyrproject/zephyr/scripts/requirements.txt inside virtual environment.
https://docs.zephyrproject.org/latest/develop/getting_started/index.html
While the pyelftools don't work from here and needs to be installed globally. May be my environment issue? however worked when I installed it globally using pip3 install pyelftools outside virtenv.

I am using Mac M2 and had the same issue. After pip3 install pyelftools, I can successfully python3 and import elftools. However, it still shows the same error when I build.
Solution: Look at the error message, you'll see a path like: "/Users/<your_user_name>/.local/pipx/venvs/west/bin/python"
This is the Python interpreter west is using. So we need to Install pyelftools for this specific Python environment:    /Users/<your_user_name>/.local/pipx/venvs/west/bin/python -m pip install pyelftools
This helps me solve the problem.

This is the solution when using pipx, thanks a ton!
Not a python dev, so was very confusing. Seems to me like a weird interaction between package managers and python libraries. I use Ubuntu so pip no longer works ("This environment is externally managed"), so you have to use pipx. But pipx creates a separate python virtual environment (venv) for each lib/tool, like west and qmk. So elftools needs to be installed IN THAT venv as well, hence the above instructions.
The confusing part is that when I activate a venv (with elftools installed) and then call west, somehow the pipx/west venv gets used instead of the one I activated.


Features plsql code with END statement.

Which version and edition of Flyway are you using?
Flyway from github, branch master.
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Command-line
Which database are you using (type & version)?
Oracle EE <IP_ADDRESS>
Which operating system are you using?
Linuxs
What did you do?
Deploy plsql below code:
CREATE OR REPLACE PACKAGE WF AS
    FUNCTION SB(VAR VARCHAR2) RETURN VARCHAR2;
    FUNCTION ST RETURN VARCHAR2;
END WF;
/

CREATE OR REPLACE PACKAGE BODY WF AS
    TYPE TBL IS TABLE OF VARCHAR2(30) INDEX BY VARCHAR2(30);
    LTBL_TAB TBL;
    LVAR VARCHAR2(200);

    FUNCTION SB(VAR VARCHAR2) RETURN VARCHAR2 IS
    BEGIN
        EXECUTE IMMEDIATE 'SELECT ''' || VAR || ''' FROM DUAL' INTO LVAR;
        RETURN LVAR;
    END SB;

    FUNCTION ST RETURN VARCHAR2 IS
    BEGIN
        RETURN LTBL_TAB('A') || ' ' || LTBL_TAB('B') || ' ' || LTBL_TAB('C') || ' ' || LTBL_TAB('D');
    END ST;

BEGIN   -- Here called context.increaseBlockDepth() and variable blockDepth incremented.
    LTBL_TAB('A') := 'VALA';
    LTBL_TAB('B') := 'VALB';
    LTBL_TAB('C') := 'VALC';
    LTBL_TAB('D') := 'VALD';
                -- context.decreaseBlockDepth() never called because the END keyword does not occur in PLSQL code
END WF;
/

In sqlplus this plsql code compiles correctly
What did you expect to see?
migration is deployed
What did you see instead?
This plsql code return error: ERROR: Unable to parse statement in ... at line 6 col 1

@codicuz Thanks for reporting this.
I couldn't find any references suggesting that it's valid to have a BEGIN without a matching END, so I'd appreciate if you could share more details on this usage (or some links explaining it). What's the use case for this BEGIN? Is END implied because it's the last block opened before the whole package body is closed?

Hello, this is package body initialization section.
Oracle docs:
https://docs.oracle.com/cd/E11882_01/appdev.112/e25519/packages.htm#LNPLS00907
But to understand how this works, it is better to read here:
https://stackoverflow.com/questions/33818765/initialization-section-of-the-package
Simple, clear and with examples described.

@codicuz Thanks for the links.
It looks like we missed the initialize section when looking at the syntax. This might be a bit tricky to fix given the section has a BEGIN but no END -- probably because it's the last section before the END for the package body.


[feature]: configure interceptdelta in lnd.conf (#6831)

Is your feature request related to a problem? Please describe.

the feature added in #6831 can cause held htlcs where the preimage is not known yet (in some on chain swap for example) to be failed by lnd (when the taker of the swap deliberately wants this to happen). this can cause partial settlement and loss of funds
Describe the solution you'd like

a way to configure this value or turn this feature off
Describe alternatives you've considered

not upgrading to v0.16..., not really an alternative
Additional context

Should be simple enough to make a config flag, the default is DefaultCltvInterceptDelta.
In your example case, is it that the application knows the the pre-image is coming eventually? This new feature was added as a safeguard against unwanted force closures.

In your example case, is it that the application knows the the pre-image is coming eventually?

In my case (the swap), the preimage can come from an on chain spend (not necessarily bitcoin). When both lockups have been done the taker "decides" when the preimage is available for the swap service to settle the htlc with.

Given an htlc that expires at height x, at which height would you no longer attempt to sweep with the preimage? Normally a safety margin is required to ensure that a preimage sweep is confirmed before htlc expiry. Otherwise a race could occur where the peer tries to timeout the htlc too.
The htlc interceptor watchdogs auto-fails at some point, but for safe operation, I don't think there are many more blocks that you could wait longer?
For swaps, the outgoing on-chain htlc should have long been timed out at that point?

Given an htlc that expires at height x, at which height would you no longer attempt to sweep with the preimage?

x - n, where n is the amount of blocks to get my tx confirmed, if someone controls a miner n is 1 (or pays 0.03682719 btc to a miner)

For swaps, the outgoing on-chain htlc should have long been timed out at that point?

It should, but for whatever reason the transaction did not confirm yet, and failing the htlc at that point could cause partial settlement (with help from a miner).
in this case of a swap service, the swap service should fail the htlc whenever it thinks the on chain htlc timeout transaction has enough confirmations to not be reversed. i think for most users this feature helps to get less force closes when forgetting to send a 'fail' to the interceptor, but for this specific purpose lnd doesn't help with failing the htlc, so i like this new feature to be configurable

Ok, got it. In this case it is then probably best to disable auto-fail completely and let lnd never do anything until the counterparty really force-closes the channel.


Problem to get active power series in DERA

Hi
I've been having problems with the get_activepower_series function for the dera model. It gives me the next error:
ERROR: LoadError: MethodError: no method matching compute_output_current(::SimulationResults, ::AggregateDistributedGenerationA, ::Vector{Float64}, ::Vector{Float64}, ::Nothing) Closest candidates are:   compute_output_current(::SimulationResults, ::G, ::Vector{Float64}, ::Vector{Float64}, ::Union{Nothing, Float64}) where G<:DynamicGenerator at C:\Users\jlsan\.julia\packages\PowerSimulationsDynamics\Al2pp\src\post_processing\post_proc_generator.jl:6   compute_output_current(::SimulationResults, ::G, ::Vector{Float64}, ::Vector{Float64}, ::Union{Nothing, Float64}) where G<:DynamicInverter at C:\Users\jlsan\.julia\packages\PowerSimulationsDynamics\Al2pp\src\post_processing\post_proc_inverter.jl:6   compute_output_current(::SimulationResults, ::SingleCageInductionMachine, ::Vector{Float64}, ::Vector{Float64}, ::Union{Nothing, Float64}) at C:\Users\jlsan\.julia\packages\PowerSimulationsDynamics\Al2pp\src\post_processing\post_proc_loads.jl:6
I think the problem is in the DERA definition in PowerSystems.jl, since the AggregateDistributedGenerationA function is a subtype of DynamicInjection and there is no function to compute the output current for devices of that subtype. There is such a function for devices of subtype DynamicInverter and DynamicGenerator (both subtypes of DynamicInjection), but not for DynamicInjection.

Thanks for catching this. You are probably right.
I can fix this ASAP before the next release. Once I pushed this fix to master I'll update this thread.

@rodrigomha, we should also implement a more graceful error while we do the push for the post-processing of results.

closed via #302


closing the modal if pressed outside of it

@mohammad4kh modal should not close when click outside. That same model component will be used to create profile picture cropper component and we don't want the model to close when user is dragging mouse to crop profile pic and mouse cursor get outside model

LGTM.  I verified this locally and it works fine.  Thanks for the work!


add stringtype parameter to entitylinking task

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[X] Other, please clarify
Added missing type in new version of our api

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of “no breaking changes
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@assafi , @ctufts , @josuhazure, @abhahn for awareness too

@assafi , @ctufts , @josuhazure, @abhahn for awareness too

review-requested:@abhahn
review-requested:@assafi
review-requested:@ctuffs

review-requested:@abhahn
review-requested:@assafi
review-requested:@ctuffs


Unable to download Text post.

I'm trying to download all the text post from this translator I sub to to read the stories easier.
The command im using is
.\PatreonDownloader.App.exe --url https://www.patreon.com/c/Emik01/posts --download-directory c:\downloads --descriptions --embeds --campaign-images --json
and it seems to be finding the url for each text post but it fails to download them and it gives
2024-10-31 04:47:11.8876 INFO Preparing to download...
2024-10-31 04:47:14.1029 INFO Crawling...
2024-10-31 04:47:14.6330 INFO -> 115014755
2024-10-31 04:47:14.6602 WARN Post entry doesn't have download url
2024-10-31 04:47:14.6602 INFO -> 115014340
2024-10-31 04:47:14.6602 WARN Post entry doesn't have download url
2024-10-31 04:47:14.6602 INFO -> 115013964
2024-10-31 04:47:14.6602 WARN Post entry doesn't have download url
2024-10-31 04:47:14.6602 INFO -> 115013462
2024-10-31 04:47:14.6602 WARN Post entry doesn't have download url

Accidentally double posted. ignore this one.


Way to print current version

Similar to poetry version, it would be nice if there was a way to print the current version. Especially nice in the case it's using the dynamic version loading machinery to discover the version. This would be useful in a release script, to e.g. create a tag with the current (or newly bumped) version.

Do you mean pdm show --version?

I don't think so. That will only show the installed version of the built package, no? Not the live version from pyproject (or dynamic)

I don't think so. That will only show the installed version of the built package, no? Not the live version from pyproject (or dynamic)

Hmm. I might be wrong about this. At first glance pdm show --version is working how I want it. I'm struggling a little bit to understand how though.

I was indeed wrong. pdm show --version is showing me the info I want already. I didn't realize pdm was so tightly integrated with the pdm.pep517 builder. I wonder what happens with that command if I'm using a different builder...

I wonder what happens with that command if I'm using a different builder...

Just shows None. I guess that is reasonable.


npm install fails unless mocha is installed globally

Because the prepack hook runs the test script, which calls mocha --harmony, unless a dependent package is installed in an environment where mocha is installed globally, this package will fail to install.

@shane-t I am not sure I understand. The prepack script does not run when installing the package only when packing or publishing it (ref.). mocha is not required when installing the package as a dependency from npm.
If you are cloning the repository, make sure to run npm install.
More importantly the ERC820 registry provided in this package, is part of the EIP820 standard which has been superseded by EIP1820. I suggest you work with the EIP1820 standard. You can use the related erc1820 package (source).
The erc1820 package is similar to this one (erc820), it also has a prepack script which calls mocha and mocha is a devDependency which won't be present but you should not need to call any script when installing erc1820 as a dependency.  If you notice any bug, feel free to open an issue on the ERC1820 repo.

@0xjac thank you for your reply and explanation! Please feel free to close this (and the PR) for the above reasons - however I attach a log that shows this issue occurs if I do npm install jbaylina/ERC820 just in case I'm not alone.
I'll admit I'm not 100% clear about npm installing things directly from source control, and whether it somehow executes prepack under unusual conditions.
2019-04-15T17_21_56_363Z-debug.log
I note the standard has been superseded though. Thank you!

@shane-t thanks, I understand what happens, the prepack hook is called when installing from git (which is what npm install jbaylina/ERC820 is. I'll open a corresponding issue in 0xjac/ERC1820. In the mean time, you can get around that by installing from npm: npm install erc820 or preferably npm install erc1820.


The liked heart needs to turn color when you have liked a post

UI Code is implemented but backend is not. We decided not to move forward with this


[BUG] - Azure/Service Fabric/Issues related to the Service Fabric Cluster/My cluster is stuck in updating state when configured with custom ports

Describe the bug
Azure/Service Fabric/Issues related to the Service Fabric Cluster/My cluster is stuck in updating state when configured with custom ports
Area/Component:
Azure/Service Fabric/Issues related to the Service Fabric Cluster/My cluster is stuck in updating state on Portal
To Reproduce
Steps to reproduce the behavior:

Go to 'Azure Portal'
Click on Create 'Service Fabric Cluster'
Create Cluster with Custom Port within the approved range - we have configured a Service Fabric Cluster with a slight change in deployment configs through the Azure Portal - changed Service Fabric Explorer port from default 19080 to 19081, and changed Client Connection Endpoint port from default 19000 to 19001.
See error - Now we observe that Cluster is stuck in Updating state (on Portal), althgouh all the nodes are created and visibile from the related VMSS (VM Scale Set) and Cluster is also accessible through Explorer endpoint (i.e. https://pg-sfcluster-02.northeurope.cloudapp.azure.com:19081/Explorer), although the Explorer is also unable to show details related to Cluster Nodes.

Expected behavior
Azure Portal should show the newly provisioned Service Fabric Cluster with custom ports and all other details of cluster should be visible.
Observed  behavior:
Cluster Creation is stuck in "Updating" state.
Screenshots

Service Fabric Runtime Version:
8.1.321.9590
Environment:

Must be one of these values [Service Fabric Cluster]
OS: [Ubuntu 18.04]
Version [8.1.321.9590]

If this is a regression, which version did it regress from?
Additional context
Not applicable.

Assignees: /cc @microsoft/service-fabric-triage

How did you do step 3 above? During initial creation or post deployment?
Do you have an NSG configured and are the ports allowed?

Please re-open if you are still facing issues here. I only ran in to problems like the above when NSG was not allowing traffic to the non-standard ports.


Docker page contains broken links to documentation

Steps to reproduce
Go to https://hub.docker.com/r/microsoft/powershell/
Click one of the links to documentation, eg 'getting started' or 'Instructions'
Expected behavior
Navigates to corresponding documentation page.
Actual behavior
404 error message
Eg https://hub.docker.com/r/microsoft/powershell/docs/learning-powershell/

The links are relative for github.
Docker exports only README.md
We can replace all relative links by absolute links in README.md
This came up second time already, probably worth to replace them.

Would an update to README.md automatically update hub.docker.com?
I see I can manually edit the "Full Description" manually in hub.docker.com, so I doubt they are connected, more likely somebody just pasted the contents.

Closing - issue seems to be fixed.


Put location input back

As an analyst or Candidate I would like to know in which neighborhoods Respondents live.
Easy: free text input
Good: dropdown or type ahead

NextDoor is a great source: https://nextdoor.com/city/virginia-beach--va/


APT whitelist request for shellcheck

This replaces travis-ci/travis-ci#4105.
The original text by @cep21 follows

Shellcheck is a great tool for integration/unit testing https://github.com/koalaman/shellcheck

Upon further reflection, it appears that there is no real way to use shellcheck on Precise. The test was run here: https://travis-ci.org/travis-ci/apt-whitelist-checker/builds/71822108#L256 but it did not find any source (and the build returned a success, which is wrong).
There is no known source for Precise that has it.
To use it on Trusty, you can:
sudo: required
dist: trusty

before_install:
  - sudo … # add APT source that does have shellcheck
  - sudo apt-get install shellcheck

It was working, I was using shellcheck as you can see here: https://travis-ci.org/D0han/mockmaker/builds/91283587#L211
Can i now still use it with no sudo?

@BanzaiMan it was working fine with precise, as @D0han says, see https://travis-ci.org/Bioconductor/BBS/builds/91467371#L170-L184 for another example.
I think this was removed in error?

@jimhester if you need workaround, there it is:
sudo: required
dist: trusty

before_install:
    - echo "deb http://archive.ubuntu.com/ubuntu/ wily universe" | sudo tee -a /etc/apt/sources.list
    - sudo apt-get update -q
    - sudo apt-get install shellcheck -y

The issue with precise seems to be that shell check has been removed from the package manager. If it doesn't work on Travis-CI container builds that is because it doesn't work on any precise machine, even with sudo, etc.

So there's no way at all to run shellcheck on precise??

You could just build it - > https://github.com/brujoand/sbp/blob/master/bin/bootstrap

I did get v0.3.7 installed with https://github.com/travis-ci/apt-package-whitelist/issues/376#issuecomment-162922005, but I need v0.4.4 :-/

@ljharb the approach I just posted installs v0.4.4

@brujoand thanks, that works, but it's exceedingly slow :-/ it took 9 minutes where the previous one took 49 seconds. Any tips on making it faster?

@ljharb ah, yeah I forgot to mention. You should really cache the .stack folder. First build takes ~10min the rest are down to a minute. It's done in the travis.yml --> https://github.com/brujoand/sbp/blob/development/.travis.yml#L6

Thanks, I'll give that a shot.

An alternative to building Shellcheck on Precise: koalaman/shellcheck#589
addons:
    apt:
        sources:
            - debian-sid
        packages:
            - shellcheck

That didn't work last I tried


Custom link color

Custom link color for AcknowViewController textView

Hi Eirik, and thanks a lot for the pull request!
In my apps, I set a tintColor directly on the main UIWindow, and this value is reflected on the AcknowList links. Are you trying to apply the app global tintColor, or do you specifically want a different color for these links? In the latter case, I don’t think we should modify the library code just to accommodate it. Feel free to subclass this code in your app to customize as you see fit, but I prefer to keep the main library as straightforward as it can (reasonably) be.


ARROW-49: [Python] Add Column and Table wrapper interface

After https://github.com/apache/arrow/pull/52 is merged, I'd like to split Column and Table into separate .pyx files, array.pyx seems a bit overcrowded.

Yep — that can all move to a table.pyx file if that works for you

Thank you, this is a good start. If you like, feel free to go ahead and move this to pyarrow.table

Incorporated the comments and also added the missing ChunkedArray wrapper class. Now we should have wrapper for all Arrow classes used in the basic DataFrame conversions.

+1, thank you, this is excellent


Unexpected parser error when there are multiple comments

Parser should parse the following case without any exceptions:
1 + 2;
// comment
// comment
3 + 4;

but it failed when parsing the second stmt

Seems related to:
https://github.com/Devin-Yeung/loxide/blob/e29e4c0141cb02dee9d5c3c5b7dc5f551532dced/crates/loxide_parser/src/parser.rs#L101
and
https://github.com/Devin-Yeung/loxide/blob/e29e4c0141cb02dee9d5c3c5b7dc5f551532dced/crates/loxide_parser/src/parser.rs#L90
we should consume the comment exhaustively


Investigate removing custom docker build image

If we add apt-get steps directly to ubuntu's azure pipeline file, we could get away from a custom hosted build agent and simply start with a vanilla ubuntu 18.04 image we pull from docker hub

more likely, will just wait until AzDo offers an updated ubuntu image, cosmic or disco - should roll out late 2019.


Add FixedType to uniqueify match statement.

Add a test for cmem and smem with fixed point numbers
Addresses #411.

Can I merge, or are reviewers merging?

You can merge. But I'll just do it for you.

Too late! Hahahahahaha


deis logs doesn't show log with Procfile based deployment

I recently switched from Dockerfile deployment to a Procfile based one. deis logs now only shows deis[api] logs but doesn't show any web logs. For example:
2015-07-01T14:36:05UTC deis[api]: olalonde scaled containers web=1 daemon=1
2015-07-06T12:29:11UTC deis[api]: olalonde deployed 72a8de3

Is it possible to get logs for my web and daemon processes? Viewing the logs with fleetctl works as expected.
deis v1.6.1

Is logspout and deis-logger working on those hosts? If the logs are showing up in fleetctl then they should be carried over to the logger, as long as everything is working.

Yes, all 3 logspout services are active/running and the logger service is also active/running.

it may be worth checking the actual logs to confirm with fleetctl ssh myapp_v2_.web.1 docker logs deis-logspout && deisctl journal logger. We've seen cases where etcd may be unreachable and the containers won't reboot.

Jul 06 17:29:33 ip-10-21-1-252.ec2.internal sh[30638]: 2015/07/06 17:29:33 501: All the given peers are not reachable (Tried to connect to each peer twice and failed) [0]
Jul 06 17:29:34 ip-10-21-1-252.ec2.internal sh[30638]: 2015/07/06 17:29:34 501: All the given peers are not reachable (Tried to connect to each peer twice and failed) [0]
Jul 06 17:29:35 ip-10-21-1-252.ec2.internal sh[30638]: 2015/07/06 17:29:35 501: All the given peers are not reachable (Tried to connect to each peer twice and failed) [0]
Jul 06 17:29:36 ip-10-21-1-252.ec2.internal sh[30638]: 2015/07/06 17:29:36 501: All the given peers are not reachable (Tried to connect to each peer twice and failed) [0]

deisctl restart logger fixed the issue though I'm still wondering what went wrong.

though I'm still wondering what went wrong.

That error indicates that logger was receiving errors from etcd on that host... does journalctl -fu etcd show anything? I think the real fix here is to use etcd2: #3564

Ok thanks.


Can you upload the data set to Google cloud disk? I didn't find their dataset set in paper 3

Or do you know where to find it ? thank you

ok  i see


CLA required?

It would be good to clarify whether members of the AC, TSC, etc. would need to accept the CLA. I'm assuming they would, but that needs to be explicit (here and on the Google form).

That's a good point.
It's implicit for the TSC, are it's "composed of members with significant experience contributing to AMP," but should nonetheless be called out explicitly.
For the AC, I think the requirements are (1) to be able to publish the AC's output under a CC-BY license or similar and, (2) to avoid making patent-encumbered suggestions to the project or to standard bodies. If the CLA covers both of these cases, than asking AC members to accept it seems to be the best solution.

@cramforce should we make this more explicit in the proposal?

Maybe something like.
Code contributions require the acceptance of the project's CLA. At this time, code contributions require consent to Google's CLA.

The risk for the AC is really only limited patents, as any code contribution would have to go through the regular contribution path and require a CLA to be signed.
If patents are a concern, the easiest path forward is to ask all AC members to sign Google's CLA. We can always revisit this once the project is moved to a foundation.
If patents aren't a concern and only copyright is a concern, then the CLA is already required to contribute code, so we don't need to add a specific comment to the governance document.

I don't really see the need to extend CLA coverage beyond what we have. It covers the unit of distribution.
One note: We have a dedicated repo for publishing design documents to get them covered under the CLA. We only use this if the doc doesn't easily fit into GH issue markdown. https://github.com/ampproject/design-docs

This approach seems to be biased towards the folks who are leadership being code contributors. If the project is just producing software, that seems somewhat reasonable, but if you're trying to produce specifications and get buy-in from many parts of the industry, not so much.

@mnot Are you arguing we should extend a CLA requirement for more types of contributions? We'd definitely rather aim in the opposite direction and remove red tape where possible.

I believe @mnot’s concern is around IP protection for specifications. A CLA protects the specification and allows it to be provided royalty free (like other W3C, WHATWG, and Ecma specifications). Sure, it’s a bit of extra red tape, but it’s one of the important elements of keeping the web open.
I must admit that I haven’t put enough thought on the AMP spec itself. Other web-related specs that would originate from AMP should clearly be funelled to WICG or WHATWG, but I’m not east the path forward is for the AMP spec itself. We should definitely talk about this more.

I should also clarify: All contributions that land in a repository are covered by CLA. This does include docs, specs, etc.
For most web standards I'd imagine that the actual specs would live outside of the ampproject GitHub org (the scope of our CLA) such as in WICG repos.

Alright so concretely, would you both be OK with adding the following to the governance document., either split-up between the TSC and AC sections or in a section of its own:

New TSC members which haven't already signed the CLA as part of their prior contributions are required to do so upon joining the TSC.
AC members aren't formally required to sign the CLA, but will be required to do so should they decide to contribute to code, documentation, specifications, design documents, or in any other way where securing IP commitment would be important to keep the projet open source and royalty free.

SGTM

That works.


Version compatibility

Is SwiftyCam now compatible to Swift 4?

Hi,
There is a branch for swift4 on this project that you can choose for cloning/downloading. It will then work with swift4. Let us know any issues you have getting to it.

Thanks

Thanks! 👍

Cocoapods 3.0.0 is now out with Swift 4.2 support


Update arc regions for public preview

Fixes https://github.com/microsoft/azuredatastudio/issues/12348
Also

Updating package version ahead of time in preparation for release
Increasing required azdata version to public preview version

Pull Request Test Coverage Report for Build 258209286

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 38.743%

Totals

Change from base Build 258196070:
0.002%

Covered Lines:
19007

Relevant Lines:
44925

💛  - Coveralls


Night Mode

Assalaamu 'Alaikum,
This is a suggestion. I ask you to put a night mode version, which makes the background dark and the foreground (text) white. This makes it easy for me to read Qur`aan after Fajr.

@AskYous thanks for the suggestion. Need any help implementing this feature?

I wish I had more freetime. I don't plan to implement it myself. Maybe sometime in the future I will.

Okay sounds good.

Rather than having another toggle button, any thoughts on maybe having this based on a timer so say from 9pm it will go dark until 4am?
@mmahalwy Can you assign this to me.

Actually a toggle would be better, it might be annoying for it to automatically happen.

I have an chrome extension that inverts colors on pages for night mode. In case someone wants ideas:

Oh nice! that looks good @AskYous

I can suggest this color scheme

@sabeurthabti Are u working on this ?

@sabeurthabti @mmahalwy I have created this color design for night mode - what are your thoughts ?
Mobile

Tablet

PC

@arshdkhn1 Looks good.  I'd like the background even darker, and I the inner box with that lighter shade of gray to go away.

+1 on @AskYous suggestion

@AskYous okay.

##mob

##tab

##pc

I prefer the previous one with some changes for pc layout

@arshdkhn1 I am not working on this. Last design looks +1.

+1


Re-add XR folder

Overview
Reverts https://github.com/microsoft/MixedRealityToolkit-Unity/pull/8223.
Especially with OpenXR, we now want the repo in a known-good state with specific features enabled, and it feels like the assets have stabilized enough for it to be resonable to check these in again.
This shouldn't have any effect on consumers of the Toolkit, just those who clone the repo and open it in Unity directly.

/azp run


fix: permission denied error for commands "install", "uninstall", "command *"

Error description: After cloning the bash-cli repo the commands failed with "permission denied"
$ ./cli install test-cli
$ ./cli uninstall test-cli
$ ./cli command create fibonacci
$ ./cli command rm fibonacci
This commit sets the executable flags on the files belonging to those commands right.

Thank you for fixing this, I've approved and merged the PR.


setup-guide: Updated with the changes wrt Readme

Signed-off-by: Mohamed Ashiq Liyazudeen<EMAIL_ADDRESS>

merged as part of pr #7


Cache issues

Running:
https://github.com/w3c/secure-payment-confirmation/runs/3648782752
It complains:

Any way to force it to clear the cache?

Also came up here:
https://github.com/w3c/webappsec-secure-contexts/commit/3d8830aa2d440278e5903a237482335b1f71273a

Potential fix or workaround
https://github.com/w3c/webappsec-secure-contexts/pull/96/files
Adding .cache to .gitignore.

Yeah but it's quite surprising.

Any way to force it to clear the cache?

Just rm -rf .cache?

@sidvishnoi is looking at it now. He will let us now what's best.

Found the source of .cache:
https://github.com/w3c/spec-prod/blob/a872d584bb25152e66e418fd947c86c8799c6291/src/validate-webidl.ts#L20-L23
Sending PR...

Fixed and released in v2 ≡ v2.6.1
If your repository has the stray .cache dir in your GitHub pages branch, please remove it manually once. It won't be added in next publishes.


feat: add MFA for WebAuthn

What kind of change does this PR introduce?
Adds MFA for WebAuthn and supporting migrations
What is the current behavior?
MFA for Phone and MFA for TOTP is supported
What is the new behavior?
Developers can use MFA For WebAuthn to sign in

Not quite ready yet, please check back after 12pm CEST tomorrow..

Merging first but let's not release until we do a full test with the updated frontend bindings. I will follow up on that front.


Use string to convert unsafe number instead of bigint

First of all thank you for this library. I'm planning to switch to it until nodejs support BigDecimal natively.
While refactoring, I noticed a small issue when converting large number to BigDecimal.
To give a quick example, Big(Number.MAX_VALUE) would give a different value than expected as bigint is not reliable when converting large numbers:
BigInt(2e80) // will return<PHONE_NUMBER>00000000053219729416734553074804802362401618196263954906979517832626176n
BigInt(Number.MAX_VALUE) // will return<PHONE_NUMBER>86231570814527423731704356798070567525844996598917476803157260780028538760589558632766878171540458953514382464234321326889464182768467546703537516986049910576551282076245490090389328944075868508455133942304583236903222948165808559332123348274797826204144723168738177180919299881250404026184124858368n

For now, I am mitigating the problem by relying on strings for our largest values.

Thanks for the PR, I will review it soon.

@d-mon- Can you also run eslint with npm run lint? It will fix all the auto fixable linting errors. After that you can commit and push.


Integration hangs

Sometimes when integration is unsolvable (even by other more advanced calculators) it just halts.
Possible solutions are:

restricting the computational resources of the process
adding a timeout (e.g. of 10 seconds) for the process
creating an external watchdog for the management of the program flow

For reference:
https://stackoverflow.com/questions/1359383/run-a-process-and-kill-it-if-it-doesnt-end-within-one-hour


Instructions for brew are in the incorrect order

Hi Gildas!
Got an issue today when installing jdev and then Oracle JDK8:
Here the script:
$ brew install jenv
==> Downloading https://homebrew.bintray.com/bottles/jenv-0.4.3.yosemite.bottle.tar.gz
######################################################################## 100,0%
==> Pouring jenv-0.4.3.yosemite.bottle.tar.gz
==> Caveats
To enable shims and autocompletion add to your profile:
  if which jenv > /dev/null; then eval "$(jenv init -)"; fi

To use Homebrew's directories rather than ~/.jenv add to your profile:
  export JENV_ROOT=/usr/local/opt/jenv
==> Summary
🍺  /usr/local/Cellar/jenv/0.4.3: 74 files, 308K

$ if which jenv > /dev/null; then eval "$(jenv init -)"; fi
$ export JENV_ROOT=/usr/local/opt/jenv
$ jenv
jenv 0.4.3

But when running:
jenv add /Library/Java/JavaVirtualMachines/jdk1.8.0_60.jdk/Contents/Home

I get:
ln: /usr/local/opt/jenv/versions/oracle64-<IP_ADDRESS>: No such file or directory

Obviously: ls -l /usr/local/opt/jenv/ gives:
total 32
-rw-r--r--  1 dpilato  admin  6385  7 jui 16:47 README.md
-rw-r--r--  1 dpilato  admin  1060  7 jui 16:47 LICENSE
drwxr-xr-x  7 dpilato  admin   238  7 jui 17:17 libexec
drwxr-xr-x  3 dpilato  admin   102  4 sep 10:00 bin
-rw-r--r--  1 dpilato  admin   286  4 sep 10:00 INSTALL_RECEIPT.json

No versions dir.
If I now run again if which jenv > /dev/null; then eval "$(jenv init -)"; fi, all needed dirs are created.
So everything works fine. I think it's just a question of documentation but I thought it worths a bug report. :p
Best

Indeed,  I am aware of this.
Will fix asap

Same here.
Thanks @dadoonet, as suggested, the following works (I executed from the command line):
if which jenv > /dev/null; then eval "$(jenv init -)"; fi

Thanks

It seems this issues still out there .
Thanks @dadoonet for the suggested fix.
After executing
if which jenv > /dev/null; then eval "$(jenv init -)"; fi
-l /usr/local/opt/jenv/
-rw-r--r--  1 murugull  admin   288 Nov  6 08:13 INSTALL_RECEIPT.json
-rw-r--r--  1 murugull  wheel  1060 Jun  7 10:47 LICENSE
-rw-r--r--  1 murugull  wheel  6385 Jun  7 10:47 README.md
drwxr-xr-x  3 murugull  admin   102 Nov  6 08:13 bin
drwxr-xr-x  7 murugull  admin   238 Nov  6 08:13 libexec
drwxr-xr-x  2 murugull  admin    68 Nov  7 06:50 plugins
drwxr-xr-x  2 murugull  admin    68 Nov  7 06:50 shims
drwxr-xr-x  2 murugull  admin    68 Nov  7 06:50 versions

How about having the caveat recommend to put the export before the if … line?
To use Homebrew's directories rather than ~/.jenv add to your profile:
  export JENV_ROOT=/usr/local/opt/jenv

To enable shims and autocompletion add to your profile:
  if which jenv > /dev/null; then eval "$(jenv init -)"; fi

Can anyone think of a reason not to do this? @gcuisinier I’d gladly do a PR to the caveat if you agree.

I totally agree, I just forget to made the modificiation.
The PR is welcome :-)
Le 2 février 2016 à 10:24:11, Claudia<EMAIL_ADDRESS>a écrit:
How about having the caveat recommend to put the export before the if… line?
To use Homebrew's directories rather than ~/.jenv add to your profile:
export JENV_ROOT=/usr/local/opt/jenv
To enable shims and autocompletion add to your profile:
if which jenv > /dev/null; then eval "$(jenv init -)"; fi
Can anyone think of a reason not to do this? @gcuisinier I’d gladly do a PR to the caveat if you agree.
—
Reply to this email directly or view it on GitHub.


Unresolved dependency on sbt compile

With the current commit (36718cf6040990f2be9ab143adb1d3c519e9d983), I'm getting an unresolved dependency when running sbt compile. Am I missing something locally that isn't covered in the instructions?

The dependency on firrtl 1.1-SNAPSHOT hasn't changed since it was introduced, over 7 months ago. If you're using the master branch of chisel3, you'll need to clone and sbt publish-local firrtl. If you've successfully built chisel3 in from the master branch within the last 7 months, you must have cloned and locally published firrtl. Did you clean out your ~/.ivy2 directory?
The release branch of chisel3 uses the published version of firrtl so the local clone and publish step isn't required.
We'll update the README to reflect this.

That did it. checked out the release branch and it compiled successfully.


fix: remove binary added accidentally

Remove the binary 'hello' that was added by mistake to fix issue #1254.

Waiting for #1258 to merge to fix tests before merging.


Mark ResultStatus labels and DBTaskResult fields for translation (#94)

CI issues are now fixed in main. Rebase and tests should pass.


chore(deps): update common-dev-assets digest to 6565341

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
d25e52d -> 6565341

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline


Fix for issue #331 - missing brackets

missing parenthnesses causing that condition was met also for unit != 'hours'

Hi @qmorek, thanks a lot for filing this pr! It takes small fixes like these to make a library perfect. I've just added some tests. And with that it's ready to merge :partying_face:

Coverage remained the same at 100.0% when pulling 11b8b0f57feb4e360c3af32e5ab03ed04b5ad888 on qmorek:patch-1 into a6fdc61af08da7553ed0d8239444a7976dd39a92 on dbader:master.


Allow parameters to be specified through configuration file

As we begin to add (#65) and consider (#57, #72) new options for the generate command, I think it's likely that the number of options for the generate command will soon exceed what can be reasonably expressed through command-line arguments alone.
Maybe this isn't inevitable. If we took seriously the canon of parsimony in our API design, perhaps we'd find that a small, fixed set of parameters are all that are necessary. This could all be a trap, a slippery slope that projects fall into all too frequently in a rush to add functionality. I'd love to be hear better arguments about limiting feature creep.
But from where I am right now, it's hard to look at, say, the configuration options for jazzy or other comparable tools and think that we'll find a way to solve the same problem without a similar number of settings. Taken altogether in aggregate, you start to get the sense about the inherent complexity of the problem.
So, assuming this is indeed something we'll need, it's probably something we should figure out before 1.0.0. Here's what I'm thinking:

Extend ArgumentParser to make options Codable. Unless this already exists, this is something that would be useful beyond swift-doc and a good candidate for an upstream patch or separate library.
By default, look for a .swift-doc.json (spelling?) file and read those as a base configuration, and add an option for where to look for a configuration file.
Options in the configuration file override the default values. Any options passed from the command line to the executable override ones from a configuration file (similar to how most systems work, including UserDefaults).
We should also think about versioning and validation for these files, as well as documentation.

I wonder what happened with the idea of using .swift files as configuration. It doesn't seem to be getting high adoption outside of Swift Package Manager. Are you considering this as an option?

There are examples of programming languages working really well for configuration (Homebrew formulas), and there are examples where a programming language would have been preferable (pretty much any configuration system that uses YAML). But I can think of many more examples where using a programming language for configuration eventually required support of a serialized form as well. We saw this with CocoaPods and JSON, and are seeing this now with Swift Package Manager (swift package dump-package).
Holding on Swift Package Manager for a moment: I think there are a few interesting benefits to having a Package.swift file, but they come with serious disadvantages, too. I recently decided to write a new library in preparation for supporting multiple modules for swift-doc. The alternative (how SPM reads the manifest itself) requires Package.swift to be built with special compiler flags and introspected. Reading a JSON file seems pretty nice by comparison.
So yeah, the reason we don't see .swift files for configuration is that very few people know how to do it, and the people who do know better than to try.

See also https://github.com/apple/swift-argument-parser/issues/41


Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I can add a personalized logo.
Please assign me under gssoc and hactoberfest.
Use Case
Adding logo will help make your website more professional and asthetic.
Benefits
No response
Priority
High
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

amazing, no need to write word ambuflow, only symbol is sufficient

Okay then I am adding this and creating a pull request.


tauon: fix build, cleanup package

Description of changes
Contains changes made in #334950 that don't have to do with implementing features currently untagged upstream.
libjxl is the wrong package for jpeg xl support, instead we should use the python module in #334948. Do you think we should add the python module as a part of this PR? the issue is that the python package itself hasn't been merged yet.
@jansol @nagisa

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1918