YouTube

Need to connect to the YouTube API and automate the process of getting details on new videos uploaded to my channel and save it in videos table.

I moved this to https://github.com/drewroberts/media


Pico freezes due to hello_sleep.c and hello_dormant.c

Entering sleep/dormant mode will work a couple of times, but then the Pico gets frozen. There must be configurations which are not recovered after wake-up. Saving and then restoring the clock registers solves only  some issues, but does not prevent the system from getting stuck. Reenabling the ROSC does not help either: rosc_write(&rosc_hw->ctrl, ROSC_CTRL_ENABLE_BITS);

I have the same problem. I can enter dormant mode a handful of times but then the Pico freezes up. Not sure what's causing this issue.
I need to wake up and perform actions every few minutes but the majority of the time I need to be in dormant mode to conserve power since I am running off battery.


Updated Dockerfile to solve the wget certificate error

The certificate has expired
Issue #, if available: Not available
Description of changes:
The above PR solve the certification error while building docker file. It was certification expired error. and by adding --no-check-certificate flag able to build image.
[build 4/6] RUN pushd /tmp && wget http://msa.sbc.su.se/downloads/kalign/current.tar.gz && mkdir -p /tmp/kalign2/build && tar -xvzf current.tar.gz -C /tmp/kalign2 && pushd /tmp/kalign2 && ./configure && make && make install && popd && rm -rf /tmp/kalign2 && popd:
0.192 /tmp /
0.195 --2024-07-09 10:26:50--  http://msa.sbc.su.se/downloads/kalign/current.tar.gz                                                                         0.195 Resolving msa.sbc.su.se (msa.sbc.su.se)... <IP_ADDRESS>
0.443 Connecting to msa.sbc.su.se (msa.sbc.su.se)|<IP_ADDRESS>|:80... connected.
0.565 HTTP request sent, awaiting response... 301 Moved Permanently
0.687 Location: https://msa.sbc.su.se/downloads/kalign/current.tar.gz [following]
0.687 --2024-07-09 10:26:50--  https://msa.sbc.su.se/downloads/kalign/current.tar.gz
0.724 Connecting to msa.sbc.su.se (msa.sbc.su.se)|<IP_ADDRESS>|:443... connected.
1.031 ERROR: The certificate of 'msa.sbc.su.se' is not trusted.
1.031 ERROR: The certificate of 'msa.sbc.su.se' has expired.
1.031 The certificate has expired

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@ajfriedman18 or @brianloyal can you review and merge this???


Mixing different module types

Is there any way to mix different types of modules into one module?
I would like to use the functionality of, for example, module/network, but also add extras with module/script.
I am able to reproduce the functionality of the network module with custom scripting, so this is in no way an pressing issue, however I still would like to know if something like this is possible.

No, this is not possible.

Ok, thats good to hear.
So I suppose full custom shell scripting via module/script would be the correct option, right?

Yeah, that is the only way to do it.

Great, thanks for the quick replies!


google play store [ICON]

Icon Request

Icon name:
Use case:
Screenshots of similar icons:

Please fill out the report completely. See also https://github.com/iconoir-icons/iconoir/blob/main/CONTRIBUTING.md#icon-requests.
Closing this for now. Thank you for your understanding!


add command to deploy in local and give parameters to serverless invoke

In order to deploy on local machine with serveless I need to give some paremeters to the invoke function

Thanks, I have built upon this PR (the code wasn't working) and opened #11. I have kept your commit, I'll be closing this one and merging #11.


stm32: Don't perform any side effects in irq_update().

The purpose of irq_update() is to cache value of UART IRQ status
register for devices which needs such caching. No other driver
performs any other side effects in this call. For STM32, clearing
TC (tx complete) bit was introduced in 8c079e91c90f94ece822e3be06
which is otherwise titled as a conversion to STM32Cube HAL. Thus,
there does not seem to be specific reasons why this code was added.
On the other hand, it leads to behaviorial artifacts when dealing
with interrup-driven UART code (specific issue seen was delaying
of transmitting every other character).
Change-Id: Id20bf214b36eeb6c09e29cc2e6bfca4f7221a1a4
Signed-off-by: Paul Sokolovsky<EMAIL_ADDRESS>

This change is

Migrated from https://gerrit.zephyrproject.org/r/#/c/12917/ (which has a lot of discussion!)

This patch depends on https://github.com/zephyrproject-rtos/zephyr/pull/42

@erwango , @galak , @daniel-thompson

@lag-linaro : please review

You require a review from the author of the original snip-it.

@lag-linaro : Thanks. If you don't have concerns with this patch and find explanations in the commit messages convincing, (and of course if you have time) can you please approve this PR (either with github's native approval, or by just posting +1 comment)? Without this patch, #49 doesn't work as expected.

Btw, with #42 merged, nothing blocks this patch.

I had understood the underlying problem was that the (broken) test "forced" the STM32 driver into using the latched TC (tx-complete) rather than the level sensitive TE (tx-double-buffer-empty) interrupt as a source of tx interrupt.
That change avoids acking the TC interrupt but still uses TC rather than TE as the source of interrupt.
@erwango: I think this means using the TC interrupt means that the interrupt will always run too late (e.g. after the UART has gone idle) leading to gaps related to interrupt latency between characters. I admit I'm doing this mostly from memory (i.e. without reference manual). Do I have the role of TC and TE correct?

@daniel-thompson:  Indeed TC is Transmition Complete, while TE is TXEmpty. I'm not sure about the double buffer though. One possible issue is that in some system using TE can lead to not transmitting the last byte of the data you want to transmit, as you can enter sleep mode just after for instance.
In the other  hand, TE will enable faster transmition (only in specific cases I think). But I'm not sure data rate is what we want from UART.

I think this means using the TC interrupt means that the interrupt will always run too late (e.g. after the UART has gone idle) leading to gaps related to interrupt latency between characters. I admit I'm doing this mostly from memory (i.e. without reference manual).

So, from my memory either, but the reason I'm after this patch is because with this code in, #49 worked following way: you press a key, nothing output. Then you press second key, both chars appear.

but still uses TC rather than TE as the source of interrupt.

Handling that would be subject of the next patch, requires cross-arch research (I remember @daniel-thompson that you did one, I appreciate and trust it, but I'd like to do it formally and fill in this table: https://drive.google.com/open?id=1e4hx9iVmAp5pBSTd2namF_VdsiMBS-OmlJUI4UPBMN4)

That change avoids acking the TC interrupt

So, @erwango reports breakage with #42 in the "arm" branch (details TBC). I think that's good sign: test works and detects that something's wrong ;-). But I start to wonder if we really should standardize of "completion" interrupt to behave like edge-triggered, i.e. give user UART ISR a chance to handle it, but not spam ISR with it repeatedly. But we'd need to do that smarter that the code being removed in this patch.
From IRC:

 danielt: Hi, so in our previous discussions, did you want to say (or said, but myself concentrating on other things) that UART "tx complete" IRQ should be naturally an edge one (so we may need to emulate that edge-ness for platforms where it isn't)?

using TE can lead to not transmitting the last byte of the data you want to transmit, as you can enter sleep mode just after for instance.

So, that's the point: to transmit well (especially in the case of real FIFO), you need to use what STM32 calls TE, but to got to sleep, you need to watch for TC first. They're both needed and can't replace one another, and we need to handle them well - for STM32, and in general.

My +1 would be meaningless here, since I don't know if it's the right thing to do.
This is why I suggested that you require either the original author or another SME to +1.

Per https://docs.google.com/spreadsheets/d/1e4hx9iVmAp5pBSTd2namF_VdsiMBS-OmlJUI4UPBMN4/edit#gid=2127015340, this patch works as expected on 96b_carbon on the current master, and on current arm branch, being rebased on master, to pick up #42 fix.

 Tested ok on nucleo_f411re on both current master and arm branch

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

TC versus TE serve different purposes don't they? TE is used to manage data flow (FIFO or double buffer refill) whilst TC is used to manage power (and perhaps to manage half duplex serial links).
Using TC to simulate TE simply results in risk of needless buffer backlogs emerging (since we cannot transmit as fast as we can receive).


Remove the special [U+e0000] "whitespace" character used by the Chatterino chat client.

Hello! I have recently discovered your library, and it looks great, good job on that.
Unfortunately, I have noticed that your library doesn't account for the special [U+e0000] or \U000e0000 Unicode character.
And it is sorta important because there is a Twitch chat client called Chatterino, and in the communities I happen to be in, many people use it. And so, it has a feature which allows you to post the same message multiple times, by adding the special Unicode character I've mentioned above.
Due to it being considered as a different character, if you directly handle the message, the character blocks the command from executing.
I could have created a PR, but I'm not that familiar with Elixir, and don't happen to know the best way to implement it.
This should be the character itself →
I kindly ask you to consider implementing this, in case you have any more questions you can ask them right away.
Thank you for your great work!

I think you could do it yourself for now.
Using the README example:
defmodule MyBot do
  use TMI

  @impl TMI.Handler
  def handle_message("!" <> command, sender, chat) do
    case command do
      "dice" ->
        say(chat, Enum.random(~w(⚀ ⚁ ⚂ ⚃ ⚄ ⚅)))

      "echo " <> rest ->
        say(chat, rest)

      "dance" ->
        me(chat, "dances for #{sender}")

      _ ->
        say(chat, "unrecognized command")
    end
  end

  # Strip out the Chatterino special character and call `handle_message/3` again.
  def handle_message("\u000e0000" <> command, sender, chat) do
    handle_message(command, sender, chat)
  end

  def handle_message(message, sender, chat) do
    Logger.debug("Message in #{chat} from #{sender}: #{message}")
  end
end

I think you didn't entirely understand me, so I'll provide you with a better reference.
If you send a message the first time, the tags should be like this.
https://paste.ivr.fi/raw/kegequfuro
If you send it a second time, then they're like this.
https://paste.ivr.fi/raw/guxawugewi
If you look carefully at the tags, then it adds a small unicode character in the beginning of the message. So it would be good if you would provide me with a nice, elixir like way to strip the character(s? potentially) without much to none overhead

Can you see what this logs
defmodule MyBot do
  use TMI

  @impl TMI.Handler
  def handle_message(message, _sender, _chat, tags) do
    Logger.debug("Message: #{inspect(message)}")
    Logger.debug("Tags:\n#{inspect(tags, pretty: true)}")
  end

   def handle_message(message, _sender, _chat) do
     Logger.warning("If you see this tags are not enabled")
   end
end

Oh it actually tells me that tags are not enabled
18:51:44.370 [warning] If you see this tags are not enabled

18:51:46.204 [warning] If you see this tags are not enabled

Albeit it's weird, because i have this in my secret file
import Config

config :treuksbotv2,
  bots: [
    [
      bot: Treuksbotv2,
      user: "treuks",
      pass: "oauth:redacted",
      channels: ["treuks"],
      capabilities: ["tags", "membership", "commands"],
      debug: false
    ]
  ]

Just for fun try using charlists: ['membership', 'tags', 'commands'] (single quote instead of double)

uhh i have this if it might serve useful for you

18:51:31.804 [debug] [Elixir.Treuksbotv2] UNRECOGNIZED: {:unrecognized, "421", %ExIRC.Message{args: ["treuksbot", "TAGS", "Unknown command"], cmd: "421", ctcp: false, host: [], nick: [], server: "tmi.twitch.tv", user: []}}

18:51:31.987 [debug] [Elixir.Treuksbotv2] UNRECOGNIZED: {:unrecognized, "421", %ExIRC.Message{args: ["treuksbot", "MEMBERSHIP", "Unknown command"], cmd: "421", ctcp: false, host: [], nick: [], server: "tmi.twitch.tv", user: []}}

18:51:31.987 [debug] [Elixir.Treuksbotv2] UNRECOGNIZED: {:unrecognized, "421", %ExIRC.Message{args: ["treuksbot", "COMMANDS", "Unknown command"], cmd: "421", ctcp: false, host: [], nick: [], server: "tmi.twitch.tv", user: []}}

Oh, actually, it seems like using single quotes actually made it work

Here you go
19:22:28.014 [debug] Message: "test"

19:22:28.014 [debug] Tags:
%{
  "@badge-info" => "",
  "badges" => "",
  "color" => "#9ACD32",
  "display-name" => "danks2555",
  "emotes" => "",
  "first-msg" => "0",
  "flags" => "",
  "id" => "d179a3ee-6858-47d9-b054-41dbb8cdab0c",
  "mod" => "0",
  "room-id" => "212793593",
  "subscriber" => "0",
  "tmi-sent-ts" => "1650817347829",
  "turbo" => "0",
  "user-id" => "707918309",
  "user-type" => ""
}

19:22:29.846 [debug] Message: "test "

19:22:29.846 [debug] Tags:
%{
  "@badge-info" => "",
  "badges" => "",
  "color" => "#9ACD32",
  "display-name" => "danks2555",
  "emotes" => "",
  "first-msg" => "0",
  "flags" => "",
  "id" => "c8f4c0f5-3b1e-43c9-a214-8db827ff38e2",
  "mod" => "0",
  "room-id" => "212793593",
  "subscriber" => "0",
  "tmi-sent-ts" => "1650817349661",
  "turbo" => "0",
  "user-id" => "707918309",
  "user-type" => ""
}

So, for now it looks like you could use this?
String.trim_trailing(message, "\u000e0000")

Actually, it looks like you have a space and that special character at the end `"test "

32 is the codepoint for space.
So, you might want:
String.replace_suffix(message, <<0x20, 0xF3, 0xA0, 0x80, 0x80>>, "")

Okay, @treuks
What is supposed to happen when you have this whitespace character? I don't get it.

yep


add easily master cubeFS object storage life cycle management strategies

https://github.com/cubefs/cubefs_blog/issues/33

I find it still need to update a file: blog/README.md，otherwise it can't be display. @fengshunli


Token cache optimization

Follow up to #5792, addressing the same problem but for token lookups.
Today on every lookup of tokens in the cache we iterate through every entry in the cache (including cache entries not belonging to MSAL), attempt to parse it into JSON and then validate that it matches our credential shape. This is incredibly inefficient and becomes more inefficient with each additional thing stored in browser storage.
This PR adds a new cache entry which contains an object of keys to all of the tokens known to MSAL for a given clientId. Now when the tokens need to be enumerated the worst case number of cache entries touched and parsed is 1 + number of tokens of the given type.
Example:
key: msal.token.keys.<clientId>
value: {
  idToken: ["idToken-key-1", "idToken-key-2"],
  accessToken: ["accessToken-key-1", "accessToken-key-2"],
  refreshToken: ["refreshToken-key-1"]
}

Codecov Report

Merging #5806 (1b796d8) into account-cache-optimization (fb4418a) will decrease coverage by 0.10%.
The diff coverage is n/a.

Flag
Coverage Δ

*Carryforward flag

msal-angular
96.63% <ø> (ø)

msal-browser
86.20% <ø> (+0.02%)
:arrow_up:
Carriedforward from fb4418a

msal-common
84.05% <ø> (-0.22%)
:arrow_down:
Carriedforward from fb4418a

msal-node
80.57% <ø> (-0.05%)
:arrow_down:
Carriedforward from fb4418a

msal-node-extensions
52.08% <ø> (ø)

Carriedforward from fb4418a

msal-react
94.68% <ø> (ø)

Carriedforward from fb4418a

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files
Coverage Δ

lib/msal-browser/src/cache/BrowserCacheManager.ts
91.32% <ø> (+0.35%)
:arrow_up:

lib/msal-browser/src/utils/BrowserConstants.ts
100.00% <ø> (ø)

lib/msal-common/src/cache/CacheManager.ts
74.31% <ø> (-4.21%)
:arrow_down:

...msal-common/src/cache/entities/CredentialEntity.ts
94.44% <ø> (+2.28%)
:arrow_up:

...b/msal-common/src/client/ClientCredentialClient.ts
89.61% <ø> (-0.27%)
:arrow_down:

lib/msal-common/src/client/OnBehalfOfClient.ts
65.16% <ø> (-0.10%)
:arrow_down:

lib/msal-common/src/client/RefreshTokenClient.ts
52.71% <ø> (-0.73%)
:arrow_down:

lib/msal-common/src/client/SilentFlowClient.ts
88.46% <ø> (-0.22%)
:arrow_down:

lib/msal-common/src/config/ClientConfiguration.ts
100.00% <ø> (ø)

lib/msal-common/src/index.ts
100.00% <ø> (ø)

... and 3 more


OCR base implementation

In order to digitalize sudoku board, we should implement an OCR library.

Implementation was completed


Adding ability to handle logging plugin logs on debug flag

fixes konveyor/crane-lib#73

Must wait on https://github.com/konveyor/crane-lib/pull/79


"master" --> "main" in WORKSPACE (Bazel)

the current Bazel (even 3.7.2) requires "main" instead of "master" for rules_cc and rules_proto

@googlebot I signed it!


What does embedding mean in "... array type when embedding is important"?

See title

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

ID: b83658b4-2e2f-f617-174d-6741575beca3
Version Independent ID: 09fd3e8d-d0e2-bfe9-ebdf-24b7c2b5db9e
Content: Welcome back to C++ - Modern C++
Content Source: docs/cpp/welcome-back-to-cpp-modern-cpp.md
Service: visual-cpp
Sub-service: cpp-lang
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

That's a good question. I'm not sure what the original author was thinking there. Perhaps they were thinking that there is some performance benefit (memory consumption perhaps?) to using an array when it is a member of a class or struct and you don't need the benefits of a vector and you anticipate many instances of the class/struct? Whatever the reason, that's overly broad guidance that seems destined to miss many subtleties. I'm going to remove that from the article. The update will be live by Monday afternoon.


"--offline" not respected when used in conjunction with "--use"

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (cce3a44 2022-07-20T00:15:31.034352Z)
What command(s) is the bug in?
forge build --offline --use "0.8.15"
Operating System
macOS (Apple Silicon)
Describe the bug
When running this command:

forge build --offline --use "0.8.15"

On a computer on which I don't have v0.8.15 of solc installed, I would expect Forge not to install the compiler version for me when I add the --offline flag. But that is not the behavior that I'm observing. Forge proceeds with pulling the compiler from remote.
This is in contradiction to what the docs for forge build --offline say:

Do not access the network. Missing solc versions will not be installed.

nice catch, I believe use, and potential installation is handled separately, but should take --offline into account


Availability On Ollama

Dear Team,
Is there anyway we could see this model availanle via Ollama?
Thank you in advance,
Jeffrey

I think when the gguf format is ready

Thank you for the response.

@jeffreylm Apparently somebody has made gguf format of this model, I have never tested it. If you want to give it a try, here is the link: https://huggingface.co/sroecker/moondream2-GGUF/tree/main

How should the Modelfile be set up to work with ollama?

It is available now! You may need to use the latest pre-release version of ollama. https://ollama.com/library/moondream


Add CodeCov Private Repository Support

This PR adds the ability for CodeCov badges to be generated for private repositories. CodeCov supports a badge token which can be passed via a the token query param.
Also added documentation to try.html

This would fix https://github.com/badges/shields/issues/368.

@espadrine updated the code per your comments. Also see comments I've made on getting the token from the url path rather than query string in #368.

Updated to match regex suggested in ##368.

Great work here! :+1:

Thank you for adding this support :) :+1:


Usage reporting enhacement

…ters

Update usage.proto
Add clusterDefinitionName as the blueprint name
This is used to track the number of datamart and dataengineering datahub clusters.
In future usage of different flavours of datalake clusters.

./gradlew clean build

recheck

recheck

Branch should be 2.16 instead of 1.16

@keyki thank you!

@keyki Could you please merge this?

@keyki thanks again!


Transposer Issues

Missing connections between valves on control layer

The control layer given in 3DuF does not have channels connecting the valves to a port for pressure control.

@maniwu This has been partially fixed (the inside channels are done). It would be good if you all parameterized component so that this does not remain to be a problem.


"Address Book" feature for sending BTC / DFI / others

What would you like to be added:
I'd like to have a feature that offers the possibility to save common address I often use for sending DFI or something.
Why is this needed:
Currently I use a document for keeping the addresses and I don't want to have an external document for stuff like this.

Hi @Brownie89 you can manually add your address in the app. Turn off the Automatically generate a new address.

I'm pretty sure my English is not that bad you could misunderstand my concern in this way.
I know how to insert my (old) receive addresses into the app.
Maybe it get's clearer by giving an example:
By LM I receive DFI and I want to send them to cake. Now I need to login into cake and copy my receive address there or I open up my local document and copy it. At this point I'd like to have the possibility to choose from something like an address book inside the app.

Excellent suggestion @Brownie89, just wanted to create that feature request!

BTW @izzycsy, that button most likely should be an address book launcher is apparently malfunctioning, it just displays an empty box...

Hi @Brownie89 , thanks for you feedback. Will forward it to the team.
Hi @tame81 , thanks for the feedback, the team is aware of the blank QR code issue.

Hi @Brownie89 , team has accepted your feature request to be considered in future builds, I can’t give you a timeline on it right now as we’re prioritising other issues at the moment, thank you for your understanding and patience.

Hi @tame81 @Brownie89 , update v2.3.2 is out, this is a mandatory update.
Please read through the release notes before updating.
The above feature is no included in this update yet, this is comment to notify you about the update.

Hi @tame81 @Brownie89 , your feedback has been conveyed to the team, however I do not have a timeline on whether and when this would be implemented. Just wanted to update you before this thread becomes stale. Rest assured that it'll be considered by the team. For future enhancement related ideas, you can post it on DeFiChain Discussions, garner some support from the community, and the team can decide whether to implement the feature in the future. Thank you!


Titles and parent IDs or types don't play nicely together

This is due to https://github.com/open-contracting/flattening-ocds/blob/master/flattening_ocds/input.py#L31 which replaces all colons with slashes.
A temporary fix to allow users to supply a non-title header for parent-ids or typed columns would be to not perform the conversion if we find a slash.
A longer term solution might be to use a different separator for this purpose when titles are used.

If possible I think it would be useful to re-examine the way we're handling the sub-sheet to parent sheet referencing... as not sure we've hit the right spot yet in terms of maximum understandability to the basic user.
Worth a quick discussion when we next meet?

Yes, definitely worth discussing,

We discussed this briefly, deciding to stick with the current approach.
I'm fairly sure that there are some implications in terms of Cove, and ability to use type annotations in the headers. Current task is for me to write this up.

With https://github.com/OpenDataServices/flatten-tool/pull/89 we've removed types annotations from the main sheet. We hope to extend this to all other sheets https://github.com/OpenDataServices/flatten-tool/issues/90 -> this will also have implications for how sub-sheet to main-sheet referencing works; the task in that issue to write up what the possible approaches might be, and what the impacts of those are.
Closing this issue as I think it's been superseded.


[Question] How can I get width/height of a cell ?

Thanks for great package. I read through README but found no way to get width/height of a cell ?
So is this possible to get width/height of a cell by sheetjs ?
Or how can I get width of a specific column and height of specific row ?
var width = cell.w;
var height = cell.h;

Width/height are properties of the column / row respectively.   You need to inspect those properties:

Row Height https://docs.sheetjs.com/#row-properties
Column Width https://docs.sheetjs.com/#column-properties

@SheetJSDev Thanks for the quick response. But I have another problem that how I can get the RowInfo and ColInfo object from a cell ? (because I want to get width/height of a cell)
var address = "A1";

/* Get worksheet */
var worksheet = workbook.Sheets[sheetName];

/* Find desired cell */
var cell = worksheet[address];

var rowInfo = cell.rowInfo(); // how can I do that

I try to access RowInfo and ColInfo from worksheet same as on README, but it returns undefined
var rows = worksheet[!"rows"]

What I am missing?
Hope to hear from you soon.

@SheetJSDev Could you take a look at my question? Many thanks!

var addr = XLSX.utils.decode_cell(address);
if(worksheet["!rows"] && worksheet["!rows"][addr.r]) row_height = worksheet["!rows"][addr.r].hpx;
if(worksheet["!cols"] && worksheet["!cols"][addr.c]) column_width = worksheet["!cols"][addr.r].wpx;

@SheetJSDev Thanks for the quick response again. But worksheet["!rows"] and worksheet["! cols"] always return undefined .
Do I need to change anything in my excel file ?
This is my excel file: test.xlsx

@SheetJSDev Could you take a look at my comment ? Thanks in advance!

@SheetJSDev I finally figured out cellStyles option but the column's width I get by sheetjs is totally different from the actual width of column in excel file
https://github.com/SheetJS/sheetjs/issues/591#issuecomment-289469196


tableOfContents breaks when headings have nested tags (after webpack bundling)

Minimal reproduction here: https://github.com/bensmithett/mdx-error-repro
Example markdown: https://github.com/bensmithett/mdx-error-repro/blob/master/some_markdown.mdx
Example of what I'm trying to do with it: https://github.com/bensmithett/mdx-error-repro/blob/master/index.js#L19
If a heading other than a h1 has nested tags (e.g. code via backticks, italics via *) the tableOfContents expects a global MDXTag to exist.

I don't really know enough about how MDX works to make any intelligent suggestions, but...
Could this have something to do with it?
Also noticed the error actually makes it into the output bundle code so it's not just a runtime thing?

Thanks in advance 🙂

The issue here appears to be that mdx-table-of-contents is using an outdated version of the mdx-js package.
It's possible that a fix could be as simple as upgrading the version of mdxjs, but given how much changed between the two versions, I think it would require a bit of work.
Unfortunately I don't have time to implement a fix right now, but would definitely accept a PR if you can get it working.


Training a chatterbot from the start

Hey!
I really like your library, and I want to use this library with discord.py, in order to make a chatbot. But, I want to make it so that, it would learn from every single message sent in every channel. How can I accomplish it? Anyone who can guide me, do this? Would be highly grateful to that... I want to so it so that it learns from the messages and the replies to those messages from different users....
Thanks in advance!

https://stackoverflow.com/questions/67516244/how-can-i-train-my-chatterbot-in-python-with-chatterbot-module/67516520#67516520
This is the Stackoverflow link of this question, if anyone is willing to answer...


Information Out Of Date

This page is out of date.
The documentation for Azure Maps clearly shows that the feature is not currently available for GCC, but it is not documented as an exclusion here.
Microsoft Fabric is not currently available in GCC.
There have been many new features introduced to Power BI that are not currently available in GCC, and they should be properly reflected here.  The items noted here are just a couple of concrete examples.

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

ID: d017bc96-49d4-7b63-9394-1739cdf7528d
Version Independent ID: 1e70d724-121d-feea-471e-307c2f27dd24
Content: Power BI for US government customers - Overview - Power BI
Content Source: powerbi-docs/enterprise/service-govus-overview.md
Service: powerbi
Sub-service: powerbi-admin
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for your comment, @qwerty2582. This issue has been assigned to the doc author for review.

Thanks for your comments. The table on that page is not meant to be a comprehensive list, and is reviewed by engineering periodically. I have added Azure Maps to the table.


[New PowerToy] PowerAccent

Summary of the Pull Request
This PR add a new module which have the possibility to manage accents with key combination. Issue #2668

Features Checklist

[x] PowerAccent module
[x] Settings view
[x] Oobe view
[x] Hide taskbar icon
[ ] Docs
[x] Check if someone else reproduce this behavior

Detailed Description of the Pull Request / Additional comments
This is a draft, first is to know if the current implementation is correct. Next implement and improve all the missing points.

That looks great, sorry I was busy yesterday. Did you make your fork of this repository private? I would like to contribute to this PR, but it seems like I would have to create a seperate PR unless your fork is public.
Extra info of what I looked up if it helps.

Oh nevermind. I just looked at your screenshot and it looks like you never made a fork. I guess if crutkas had to give you permission to push, he'll probably have to let me too.

Sorry, I'm not at my pc at the moment because I'm going on holiday. I do remember having an issue launching the settings menu of poweraccent in one of the projects but the other project, which I think was named "poweraccent UI", could launch. The error message was very unhelpful, something about the windows11 image path being incorrect, but when I changed the path or removed the line altogether it still referenced that line of code (maybe it was caching the build).
Anyway, maybe try using the project called poweraccent UI and check it to see what is different and if you can get the settings to launch, because it seems like it won't even launch from the taskbar icon. Sorry that I couldn't look into this myself.

What I'm saying is that your implementation for launching the settings menu is likely correct. The only problem is that the settings dialogue itself will always crash.

Hello, yeah indeed I seen that. It already fixed on my side (though it was pushed in this PR, but clearly it is not the case..).
So I guess you launched it with the right click on the task bar icon.
Originally I wanted to hide this icon, and let PowerToys have the capability to disable/access settings, to create some logical with the other modules.
But I guess I will abandonned this idea for now. Too much complicated, and it can be updated later, in a future update.
Will go to holidays for one week too. Have fun and see you! :)

Please hotlink the related issue in the frist comment? :)

@damienleroy @Snowworm3000 Any help needed here to move this forward?

@stefansjfw If you have some clue about how create a communication inter-process (here, ask from PowerToys asking to PowerAccent to open Settings view), could be great :)
Anyway, I don't think it's urgent. I guess we can do this without it for a first release. A changing it after that. So I will let display the task bar icon for now to be able to access Settings.
@Jay-o-Way no issue for this as now. And I will push the fix quickly with the other changes. Do you think it is better to create one in any case?

no issue for this as now.

You can use #2668

Oh, ok. thought you mean an issue for the problem in the conversation. Sorry not use to this yet...

@stefansjfw If you have some clue about how create a communication inter-process (here, ask from PowerToys asking to PowerAccent to open Settings view), could be great :) Anyway, I don't think it's urgent. I guess we can do this without it for a first release. A changing it after that. So I will let display the task bar icon for now to be able to access Settings.

I see you were looking into FancyZones Editor button in Settings already. For that to work, you need to pass IPC callback to PowerAccent ViewModel (last argument here https://github.com/microsoft/PowerToys/blob/main/src/settings-ui/Settings.UI/Views/FancyZonesPage.xaml.cs#L19) which will be triggered by button press like this https://github.com/microsoft/PowerToys/blob/main/src/settings-ui/Settings.UI.Library/ViewModels/FancyZonesViewModel.cs#L864.
Last step is to implemment call_custom_action for your module (PowerAccent) which is called on IPC message triggered by clicking the button in Settings app, see here:
https://github.com/microsoft/PowerToys/blob/main/src/runner/settings_window.cpp#L111
Let me know if anything else is needed :)

Oh nevermind. I just looked at your screenshot and it looks like you never made a fork. I guess if crutkas had to give you permission to push, he'll probably have to let me too.

@crutkas

So far this is very nice! Great work! Few comments for now:

we need better icon in for dark theme. @niels9001 can help here

I think it would be better to eliminate separate settings window and include PowerAccent settings in PowerToys settings

@damienleroy @stefansjfw Ooh, this is exciting stuff 😊! I can definitly help out with anything related to UI.
I'll have a look at what we can do with the icon. Would be nice to have the UI that show the different characters be inline with the Windows/Fluent design guidelines as well! Is there a .GIF or videoclip to see this in action?

e.g. For position setting you can just use DropDownButton like VCM does:

@damienleroy I don't know how to reproduce this, but from time to time, while typing I get glitches in the top left corner of my screen. PowerAccent toolbar appears for ~0.1 second and dissapears

@damienleroy I don't know how to reproduce this, but from time to time, while typing I get glitches in the top left corner of my screen. PowerAccent toolbar appears for ~0.1 second and dissapears

got it. press 'a' 'e' and space together and it appears

@stefansjfw I got it when I type too quickly. But normally it should not type any accent if space key released enough quickly.
About integrate PowerAccent Settings in PowerToys, this keys settings is what I am afraid of. And this will change several time in the future (integrate others type of characters / accents from different countries, ...). It's why I thought it is better to use the original view instead of creating one another.

@damienleroy
Please don't forget to add an entry for the settings page here: https://github.com/microsoft/PowerToys/blob/main/src/settings-ui/Settings.UI/App.xaml.cs#L114
This is needed to open the page directly from an external process.

@damienleroy when do you want us to start doing some deeper reviews. Would love to get this in if possible for this release

@Clint @.***> can you please tell me when the release is planned ?
I implemented Oobe view, but it crashing...
So need to resolve it and after this I guess it should be ok for the review.
Also I notice something odd with visual studio. I have to see what is happening.
I use to type 'alt' + '<' to back the cursor to the last selection.
When powertoys & poweraccent are running together it create an icon in the position of the cursor..
Would be great to know if you have the same issue.
Damien

From: Clint Rutkas @.>
Sent: Saturday, August 20, 2022 8:40:42 PM
To: microsoft/PowerToys @.>
Cc: damienleroy @.>; Mention @.>
Subject: Re: [microsoft/PowerToys] add poweraccent (draft) for PR (PR #19212)
@damienleroyhttps://github.com/damienleroy when do you want us to start doing some deeper reviews. Would love to get this in if possible for this release
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/pull/19212#issuecomment-1221384021, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADDGPWNU6VGZVD3SEC7TDTTV2ERCVANCNFSM5264YVTA.
You are receiving this because you were mentioned.Message ID: @.***>

Are you ok if we take this across the finish line?
we try to release beginning of the month and would like to start putting some stress against the new utilities:)

It ok for me.
The entry page has been added. And the Oobe issue has been fixed.
I also removed the icon in the taskbar. I can rollback it if needed.
What about the Settings page? Do you plan to do it before the release? Or use the current one for now? (in this case, rollback the icon in the taskbar will be better).
Se you!
Damien

De : Clint Rutkas @.>
Envoyé : samedi 20 août 2022 23:26
À : microsoft/PowerToys @.>
Cc : damienleroy @.>; Mention @.>
Objet : Re: [microsoft/PowerToys] add poweraccent (draft) for PR (PR #19212)
Are you ok if we take this across the finish line?
we try to release beginning of the month and would like to start putting some stress against the new utilities:)
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/pull/19212#issuecomment-1221412110, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADDGPWMQMGHXWCVHU2L3VVDV2FEPBANCNFSM5264YVTA.
You are receiving this because you were mentioned.Message ID: @.***>

I did the installer and signing part.
Few things yet to be done here:

add settings

I wouldn't add a custom hotkey here as it doesn't make much sense. I'd add 2 separate toggle switches for enabling space activation and arrows activation
add UI position setting. Not necessarily as a part of this UI, this can wait, IMHO

cleanup needed code when all is done

@damienleroy Why does poweraccent intercept WM_SYSKEYDOWN and WM_SYSKEYUP?

Please don't forget to add this to the issue templates.

Please don't forget to add this to the issue templates.

done

Hi @damienleroy,
We've been avoiding releasing new PowerToys with names started by "Power".
Are you OK if we change the name to something like Quick Accent? Other ideas are welcome, as well.

I've added a commit for the å, ø and ñ characters.
;)

Hi @damienleroy,
We've been avoiding releasing new PowerToys with names started by "Power".
Are you OK if we change the name to something like Quick Accent? Other ideas are welcome, as well.

What about "Accent Board", "Accent Toolbar" or "Accent Menu"?

I just tested it and I think Œ is missing. And I also think ß should be added to the s character.
Also I encouter a weird bug that when PowerAccent is running I can't use the shift key on dead keys. So when I want to input a \`` char I normally have to press shift + ^, but now it always inserts ^`, also when pressing shift.

I just tested it and I think Œ is missing. And I also think ß should be added to the s character.
Also I encouter a weird bug that when PowerAccent is running I can't use the shift key on dead keys. So when I want to input a char I normally have to press shift +^, but now it always inserts ^`, also when pressing shift.

You're right. Accents that need to press shift to get to will just assume the accent if you hadn't pressed shift.

@damienleroy and @stefansjfw ,
I've added a proper ToUnicodeEx call that doesn't change the dead key state of the keyboard.
Also removed the hacks that were added for it. Please test to check if those hacks are still needed (not sure what cases they were trying to handle or how to replicate them).
@Aaron-Junker , that should have fixed your ^ issues. Please verify.

I just tested it and I think Œ is missing. And I also think ß should be added to the s character.

I don't know those characters. Œ is what character accentuated? Not sure where to add it.

I just tested it and I think Œ is missing. And I also think ß should be added to the s character.

I don't know those characters. Œ is what character accentuated? Not sure where to add it.
You've got any reference we can use?

ß is a german variant of s and typed using altgr. It is available is lower case and upper case.

I just tested it and I think Œ is missing. And I also think ß should be added to the s character.

I don't know those characters. Œ is what character accentuated? Not sure where to add it.
You've got any reference we can use?

ß is used in the German used in Germany and Austria (Not in the swiss hogh german though) and represents two times the letter s.
https://en.wikipedia.org/wiki/ß?wprov=sfla1
Œ is used in french and is a combination of o and e
https://en.m.wikipedia.org/wiki/Œ

@Aaron-Junker , that should have fixed your ^ issues. Please verify.

@jaimecbernando I can't very this, because I can no longer open PowerAccent. It shows up in Task Managerm but not in the taskbar.

For what is win11desktop.jpg needed? And why two times?

@jaimecbernardo
Quick Accent is of for me. Fun fact: we can flip the icon, so it look like a Q :)
@Aaron-Junker
Agree for Œ. It is not an accentuated character. But I think you can adding it to both (e & o)?
In any case In PowerAccent I would like in the settings be able to choose the country, to have default special characters depending of them (in this case we can manage ß and Œ differently).
I would recommand first to limit characters to accents. And next try to find a way to include others special characters.
I'm not sure implement all of them in the current implementation is a good idea, the list is long :) .
About the image win11desktop.jpg it is for settings display. Is it here twice because I had trouble with path when I wanted to launch the application from PowerAccent and not PowerAccent.UI project. The quickfix was duplicate the image.
Also I removed PowerAccent from the taskbar. If you need it I can add it again.
Sorry for the delay... I thought answering through emails worked. Obviously not. I think you didn't get a lot of answers from me...

@jaimecbernando I can't very this, because I can no longer open PowerAccent. It shows up in Task Managerm but not in the taskbar.

@Aaron-Junker , what do you mean?

Can someone please explain what's the (Powershell doesn't take back issue)?
I'm trying to add some logging like in the other modules, but I keep getting those System.Diagnostics.Trace.WriteLine calls in the logs.

@jaimecbernando I can't very this, because I can no longer open PowerAccent. It shows up in Task Managerm but not in the taskbar.

@Aaron-Junker , what do you mean?

When I build PowerAccent and I open "PowerToys.PowerAccent.exe" it does nothing. The process appears in the task manager, But there's no notification icon.

It is because I removed it in a previous release...
As the settings will be in PowerToys I though it was good to remove it. But it is easy to rollback if you need it again.

It is because I removed it in a previous release... As the settings will be in PowerToys I though it was good to remove it. But it is easy to rollback if you need it again.

I just saw your comment and edited mine. I think it doesn't need one.

@jaimecbernardo
For (Powershell doesn't take back issue), I remember got an issue about PowerAccent didn't replace the e by é, but adding it giving eé in the console. But I just checked, finally it is not inside Powershell, but inside Windows Terminal. I've got the issue.

@Aaron-Junker , that should have fixed your ^ issues. Please verify.

Yes this is fixed

I've removed the Command Line dependency, since that was adding quite some localization dlls to the installed folder and replaced it with code like in other PowerToys.
Also removed debug writelines, but left the logger code commented because of the Trace.WriteLine hack.
I'll need to verify that, but if it's not fixing anything, I'll remove it to get logger in.

@jaimecbernardo
Don't forget the github label.

feedback
In app + user docs:
Once you understand how to invoke, super easy, if you don't understand how to invoke, this is frustrating. We really need to land description + in-app clear invoke rules on top of just user docs.
Crash:
Experienced one crash, unsure why, event viewer didn't show a crash, logs don't show a crash.  With how this was being invoked, had to use task manager to see if it was still running.
  If i do the combo together, it works. "ääää" was my output.
If I use 'spaces' or 'both', when normal typing, I see a lot of false UI flashes and i think it is due to the bug above.
My compile was from: cbd6c89acdd92ddd5b997ef7e4ac205c68d03296

@crutkas

you see the dialog flash but then a space is added.
I think I can do something about that. I will try.

@damienleroy , already did, I see why the space was there. It was to not eat the space on false starts.
https://github.com/microsoft/PowerToys/pull/19212/commits/d0e79b3a36df6dfdd188ceded506b5cfc094b02f

Now for the flashing UI. What happens here is that when some users type fast, they'll press the new key before fully releasing the last one.
I think what can be done here is just show the UI after InputTime instead of showing it right away.

Exactly. I implemented a "stopwatch" (here 200 ms), to avoid type accent if it too quick.
But the display is still there.

@jaimecbernardo I will try to take care of this case

@damienleroy , sorry, was already working on a potential solution with a timer.

Your solution seems simpler than min, though :) Will give it a try.

Not sure it is the best way to do it. Let me know if you are ok with that :)

For reference, this was my attempt here:
    private System.Timers.Timer _timer;

    private void PowerAccent_OnChangeDisplay(bool isActive, char[] chars, int showDelayMs = 0)
    {
        if (_timer != null)
        {
            // Received a new request for changing the display status. Dispose of the current timer.
            _timer.Enabled = false;
            _timer.Dispose();
            _timer = null;
        }

        if (isActive && showDelayMs > 0)
        {
            // When trying to show the display, only show it after a certain delay, to account for fast typers.
            SynchronizationContext context = SynchronizationContext.Current;
            _timer = new System.Timers.Timer(showDelayMs);
            _timer.Elapsed += (object source, System.Timers.ElapsedEventArgs e) =>
            {
                context.Post(
                    (_) =>
                    {
                        PowerAccent_OnChangeDisplay(isActive, chars);
                    }, null);
            };
            _timer.AutoReset = false;
            _timer.Enabled = true;
            return;
        }

        this.Visibility = isActive ? Visibility.Visible : Visibility.Collapsed;
        if (isActive)
        {
            CenterWindow();
            characters.ItemsSource = chars;
        }
    }

@damienleroy,
Just tested your solution to the UI flashing, I think we can go with it :)
Thank you

Reistated the logs.
Removed the Trace WriteLine hack and replaced it with a 1 ms delay instead. That seems to fix it consistently for PowerShell and Terminal.

Added some better explanations to Settings and OOBE:

@crutkas , can you please take another look at it? We added logs, fixed the issues presented and added descriptions.
Thank you!

I think we may want to default to "both".  Why: I'm trying to do "O" and near impossible to press and hold O and do the arrows.  I can do space in all scenarios however.

@Aaron-Junker
On which letter is the variant Œ?

I've set "Both" as the default activation method and added Activation Key (space key or left and right arrow keys) to the module description in Settings.

I've set "Both" as the default activation method and added Activation Key (space key or left and right arrow keys) to the module description in Settings.

Nice! I initially set the default to arrows because of that UI glitch when typing fast and hitting space. Now with that fixed, Both is definitely a better option 👍

@Aaron-Junker
I would say on the O

Thank you very much for your help!
I am really excited about how the community will use this feature. :)

Agree for Œ. It is not an accentuated character. But I think you can adding it to both (e & o)?
In any case In PowerAccent I would like in the settings be able to choose the country, to have default special characters depending of them (in this case we can manage ß and Œ differently).
I would recommand first to limit characters to accents. And next try to find a way to include others special characters.
I'm not sure implement all of them in the current implementation is a good idea, the list is long :) .
Damien

De : Aaron Junker @.>
Envoyé : mercredi 24 août 2022 16:46
À : microsoft/PowerToys @.>
Cc : damienleroy @.>; Mention @.>
Objet : Re: [microsoft/PowerToys] [New PowerToy] PowerAccent (PR #19212)
I just tested it and I think Œ is missing. And I also think ß should be added to the s character.
I don't know those characters. Œ is what character accentuated? Not sure where to add it.
You've got any reference we can use?
ß is used in the German used in Germany and Austria (Not in the swiss hogh german though) and represents two times the letter s.
https://en.wikipedia.org/wiki/ß?wprov=sfla1
Œ is used in french and is a combination of o and e
https://en.m.wikipedia.org/wiki/Œ
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/pull/19212#issuecomment-1225828337, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADDGPWPHNLN3GAQVA5W2QRDV2YYVRANCNFSM5264YVTA.
You are receiving this because you were mentioned.Message ID: @.***>

I was still underwhelmed with the available alternatives for typing accented/special characters on Windows, and ended up developing windows-accent-layers. It is completely customizable and works like typing diacritics on macOS with the layers provided by the Option key (Option-e, Option-i, etc.).
For those not familiar with overpriced hardware with good battery life, here you can find an official description of how you can input diacritics Apple devices:
https://support.apple.com/en-is/guide/mac-help/mh27474/mac
There are two ways to type diacritics there (other than layout switching):

By holding a letter until a pop-up appears, then selecting one of the options by pressing 1 through 9. Quick Accent is based on this, although not exactly like this. It does not work with every application (e.g. inside vim). It's also pretty slow if you're typing a lot of text daily.
By using modifier/dead keys, which are unfortunate/legacy names for what I call layers. For instance, to type é you would press Option+e to activate the acute layer, then e.

The second solution is simply the best one I think there is (in terms of reliability and speed). Nothing really comes close to it if you're typing a lot of code/text and constantly switching between languages (other than Linux's Compose key ports for Windows, which either don't work properly or are ridiculously bloated, and you'd also lose an entire key's functionality).
On Windows, you can use the US-International layout to type é by pressing ' then e, but this comes with the side effect of turning ' into a modifier key (which is pretty bad if you write code with quotes). Also, if you are switching between OSes for some reason, you will lose the muscle memory of the arguably better solution when coming back to Windows.
Here are my criticisms of Quick Accent/Power Accent:

Using Space as a activation key is just a bad idea. In general, you want to avoid having Space as a modifier to anything since it can trigger confirmation pop-ups/dialogues the same way as Enter. I don't know how you would emulate macOS's hold activation if you wanted to, but I don't think you should since it is bad/slow by itself. I can't believe anyone would write a considerable amount of text daily with this input method.
The current activation mechanism is dependent on the user's repeat delay setting. If you're used to lower repeat delays, this entire thing becomes unusable (see #30252 for instance). This is even worse if you are one of the unfortunate vim users who are using something to reduce the repeat delay to even lower what your OS allows by default (on Windows, there is a slider somewhere in the Keyboard settings, for those that didn't know).
I don't understand why Left and Right are available as activation keys. You shouldn't ever make the user want to take his hands out of the home row when typing text. It is bad ergonomics and can make you rotate your wrist, which is arguably borderline evil design if the target user is prone to CTS or some other physical condition. With Apple's  pop-up, you are at least doing a vertical movement from the home row to the number row, instead of a wrist rotation to the arrow keys. I'm aware you can still use arrows instead of numbers on macOS, but that doesn't make Apple's design less awful. Plus, for ergonomics, there you don't have to hold any key once it pops up, which makes using the number or arrow keys not as bad as it could be.
Why are the character sets hard-coded? I did a 1-minute issue search here, and there are multiple issues where a user wants to have a few characters added. Why is the character set not available as an input field on the settings window? Just for comparison, you could write the entire 700+ line hard-coded C# character definition file in maybe about 30 lines if you just use a better input format.

In summary, I think Apple's 1st solution was bad design to begin with, and Quick Accent is trying to emulate it with additional limitations imposed by the OS. For instance, if you tried to implement all of its features, you would probably find out that placing the accent bar right near the caret (which is what actually happens on macOS) might be difficult/tricky on Windows for some applications such as Chrome or anything Electron-based, due to the way the OS handles carets depending on the application.
I think people could benefit from a PowerToy based on Apple's 2nd solution, in case anyone wants to (re)write it in some flavor of C and add it as a PowerToy. Personally, I'm perfectly happy with the layered solution.


timerDirective now takes an optional custom reservoir

I did this because I wanted to use a custom reservoir such as this when collecting my metrics

Cool! I see what you're trying to do here. I cherry-picked the bulk of it over to https://github.com/Backline/akka-http-metrics/pull/7 and am building that now.
Scala is a bit weird with zero-arity methods and defaulted parameter lists. I think the only real way to do it is just to have differently named methods.

If you want this change just ping me. We likely need to re-do the work to support custom metric reservoirs after https://github.com/Backline/akka-http-metrics/pull/18.
https://github.com/Backline/akka-http-metrics/pull/7


Port to FStar.Tactics.CanonCommSemiring in FStar@santiago_ring

This should be merged at the same time as https://github.com/FStarLang/FStar/pull/1838

The Linux build timed out waiting for an FStar build to finish.
I got a green in a later build I launched manually:
https://msr-project-everest.visualstudio.com/Everest/_build/results?buildId=26312 and on an everest build.


Unify all docs on developer.tbd.website to respect one dependency

Introduce a Bill of Materials.
This will give us a single place to update, for instance, a Web5 or SSI dependency. All downstream efforts with the Web5 dependency: TODO App Guide, TODO app starter template, TODO app finished example, Quickstart, Quickstart index.js, Quickstart front-end widget - respect and use that one declared dependency.  This gives us one place to update across the entire docs suite - no more cascading updates in several places and issuing new releases across different repos.
@dayhaysoos to dig in and provide an RFC for stakeholder review.

Proposal: Centralization of Examples in developer.tbd.website Repo
First of all, I just want to say that this is one step of many towards making a smoother process when it comes to going from release to docs being updated.
I suggest we add a dedicated examples folder in the developer.tbd.website repository to consolidate the separate repositories we currently maintain for examples.
Currently, the following examples are maintained in separate repositories:

TODO App Guide
TODO App starter template
TODO app finished examples
Quickstart frontend Widget

Managing these separate repositories has been cumbersome and lead to inconsistencies and tracking issues. I think a good first step would be to move all these repos under an examples folder. By making this folder, it would help with:

Improved Efficiency: Having a centralized location for all examples will streamline our update process and eliminate the need to switch between repositories.
Ease of Tracking: We can track changes and updates across all examples more effectively when they are in one location.
Consistency: It will help to ensure that changes made are consistent across all examples.

I also propose that we should make it a requirement that each example should work "out of the box". If the example is in an index.html, we should be able to see it working from copying/pasting into a local html file, for example This would help avoid any path or dependency issues and improve the experience for developers who are trying out or learning from our examples.
The scope of this step is pretty small, but I would love to talk about expanding from here. Here are some things that we should talk about:

Reactifying the Quickstart to make it easier to reason about and edit
Ways to synchronize between examples and tutorials (are there certain tools we can use to handle this, ways we can template it?)
Editing the examples/tutorials so they are 1:1 copies of each other (basically, the same exact code being written in the example is also being used for the tutorial, I noticed we had to refactor things to fit the appropriate context)

I'll leave it here for now, feel free to share any thoughts or concerns around this task.

1000% yes. Thank you for writing this all out. And apologies I didn't see it right away, so sorry.
So yes this would make our lives so much simpler.
So I see it that these need to be moved into examples:

Incubating Labs TODO app
Quickstart Frontend Widget (where is this? I haven't been updating this)
Everything in web5-tutorials repo needs to be squashed and moved

Sounds appropriate @dayhaysoos - and will unlock a lot for us in subsequent stages reducing maintenance burden.
Questions:

Where do we put the single dependency declaration for Web5, for SSI, etc? How do the things under the examples directory you propose honor that single declaration?
How are you thinking about this; is the example folder you propose something that's under the existing root of developer.tbd.website? Or is it a move into a monorepo-like approach like you'd mentioned a couple of times prior?
Is examples a term that covers the full suite of things we're looking to align under one roof? We existing assets are: 1) completed examples, showing what the expected "done" state is if the user follows a guide/tutorial 2) The starting point "template" for devs to pull and build atop 3) Playgrounds intended to be deployed, or followed as examples
And also to be included there - the QuickStart index.js, yes? That's the "completed example" if the QuickStart is done right, and what we should be pulling from to assemble the code snippits in the QuickStart.

I have hesitation about this part:

Editing the examples/tutorials so they are 1:1 copies of each other (basically, the same exact code being written in the example is also being used for the tutorial, I noticed we had to refactor things to fit the appropriate context)

In most cases, it's healthy to have the examples vary to show the flexibility of the system. There's so many optional parameters and combinations of calls. We don't want to just show one flavor. With proper context, we're showing developers how to customize their code for specific use cases

@dayhaysoos, looking for more info on how the dependency will be set up and incorporated in the examples

Hey @angiejones !!
About your hesitation:
I agree that it's healthy to show variations of how to use something, however, I also think it can go both ways, where seeing more than one example done in different ways can be really confusing at the same time. I think this might be something that comes down to how it's executed.
I think we can come to a happy medium where we mention that we offer optional params and configurations and link off to those specifically in case they're interested in that.
Relocating the TODO App
This is a good question, I sort of see 2 approaches here.
Stripe has a repo dedicated to examples, snippets, demos. I think the key takeaway here is that everything is downloadable and runnable out the box, which could be really nice to have:
https://github.com/stripe-samples
I think they're scoped down to really small features, like just doing a simple payment for a specific tech stack or something.
Another approach is how Vercel does it. They have a massive directory full of examples:
https://github.com/vercel/vercel/tree/main/examples
I personally like this because it's all under one place. When I need to see how something is set up or implemented I don't have to search for a specific repo. You can explore this directory to see if what you need is in here. As far as how people could run it locally, we'd have to defer to external tooling for that but we can go over that soon.
Example Dependency concern
cc @ALRubinger
I think the best move would be to switch to a monorepo and use pnpm instead of npm. pnpm is way more efficient at managing dependencies in a performative way, and all versions are scoped to the specific examples. You won't end up with colliding dependency versions between examples like you possibly could with npm. It's also easier to test these examples locally with workspaces before we ship.
I use pnpm
With use-shopping-cart:
Config:
https://github.com/dayhaysoos/use-shopping-cart/blob/master/pnpm-workspace.yaml
The scripts in the package.json are reflective on pnpm as well:
https://github.com/dayhaysoos/use-shopping-cart/blob/master/package.json#L2
Switching to pnpm won't feel like such a huge change, it'll bring us a step closer to feeling like we have more control over more things under one repo, though. Would love to know y'all thoughts!

On its bones, sounds pretty great to me.
So the new structure you're envisioning is:
developer.tbd.website (repo)
- site/ (contains a full lift of the existing repo; the Docusaurus site)
- examples/ (or another name, to contain contents of tutorial starter templates, finished examples, playgrounds, etc. Each app in here has its own README and build script, honoring the top-line dependencies declared in the root folder)
- package.json (contains top-level dependencies respected by both the site and all "examples"

...?
If so, TL;DR of my thoughts:

I do think it makes sense to unify all examples as you've defined them into a monorepo, with top-level dependencies (BOM)
I may have evolved my opinion - I don't feel it makes natural sense to couple the examples monorepo with the dev site.

I think the natural dependency tree looks like:
Web5 | SSI upstream projects > examples monorepo > Dev Site
In that way the examples could be uniformly versioned together. We could cut a release for that monorepo, and the Dev Site could declare a single dependency on that monorepo's tag to build the published docs.
Either way, sounds to me like @dayhaysoos has laid out a great proposal and unified our thinking; I'm eager to bring the opinions into a room on tbd-stage and drive to a conclusion by end of week so we can take action and start unifying things to ease particularly @bobbilee19's maintenance burden and that of the Web5 team.

For me, examples are part of our docs. We will be referring to these examples from our docs site and that's the main reason why I'm proposing we put it all under one monorepo.
I would definitely like to learn more about examples being uniformly versioned together. While it sounds nice to have examples that are also versioned with the docs, I can't help but feel like examples should always reflect the most recent version while the docs themselves are versioned in a way that they can be referred to for previous releases.
With that being said, I bet we can figure out a way to version the examples if we need to.


pkg/sdk: add support for per-Informer Handlers

Today using the SDK it is not possible to set up multiple handlers (e.g. a handler per resource type) in the same binary because all Informers use the global Handler set by sdk.Handle. With this change, a user can create an Informer that will invoke a specific handler. We also add a sdk.WatchWith method which mirrors sdk.Watch but allows the user to provide a Handler for the underlying Informer.

As discussed offline we'll hold off on this change and instead wait until the SDK uses the controller-runtime. #382
With the controller-runtime it's easier to define distinct controllers within the same operator which gives the same functionality as having distinct handler per informer.


Editing properties

Hi Karim!
First of all. Thank you for your work. :)
I can't edit the properties of the features:

print testfile[0].properties
{u'ALTA_DB': u'2014/10/15', u'ID_CP':<PHONE_NUMBER>1.0, u'COD_POSTAL': u'08516'}
testfile[0].properties={u'zipcode':u'08021'}
print testfile[0].properties
{u'ALTA_DB': u'2014/10/15', u'ID_CP':<PHONE_NUMBER>1.0, u'COD_POSTAL': u'08516'}

while I can do other edits

print testfile.bbox
[0.159181151588882, 0.310456711008385, 42.82497667423411, 42.861370527827546]
testfile.bbox=[0,0,0,0]
print testfile.bbox
[0,0,0,0]

Any idea?
Thanks!
Diego

Hi Diego,
Thanks for pointing out this problem.
It appears that editing features in place doesnt work in the current version. This is because all features are actually stored as dicts, and are only copied into a feature instance upon indexing or looping. Changing the feature only changes the copy, not the original. You will have to replace the orig feat with the changed one to make it work.
This is not ideal of course, so i will try making a new version where features are stored as instances so they can be edited.
If you use pygeoj alot, feel free to tell me of any other changes or suggestions to make it easier to use, eg how to delete features etc.
Karim
Sent from my iPhone

On 08 Jun 2015, at 14:09, Diego Villuendas Pellicero<EMAIL_ADDRESS>wrote:
Hi Karim!
First of all. Thank you for your work. :)
I can't edit the properties of the features:
print testfile[0].properties
{u'ALTA_DB': u'2014/10/15', u'ID_CP':<PHONE_NUMBER>1.0, u'COD_POSTAL': u'08516'}
testfile[0].properties={u'zipcode':u'08021'}
print testfile[0].properties
{u'ALTA_DB': u'2014/10/15', u'ID_CP':<PHONE_NUMBER>1.0, u'COD_POSTAL': u'08516'}
while I can do other edits
print testfile.bbox
[0.159181151588882, 0.310456711008385, 42.82497667423411, 42.861370527827546]
testfile.bbox=[0,0,0,0]
print testfile.bbox
[0,0,0,0]
Any idea?
Thanks!
Diego
—
Reply to this email directly or view it on GitHub.

Hi Karim,
I can add items to the properties dict:

for feature in testfile:
feature.properties[u'zipcode']=feature.properties['COD_POSTAL']

I can delete items from the properties dict:

for feature in testfile:
del feature.properties['COD_POSTAL']

I.e., in my case it works doing this:

for feature in testfile:
del feature.properties['ID_CP']
del feature.properties['ALTA_DB']
feature.properties['zipcode']=feature.properties['COD_POSTAL']
del feature.properties['COD_POSTAL']

But I can't assign a new dictionary directly

for feature in testfile:
feature.properties={u'zipcode': feature.properties['COD_POSTAL']}

Best,
Diego

I see. I will have to clean up the internals and switch to storing each feature as a feature instance. With a better overview and cleaner approach it should be straightforward to assign a new dictionary and do any other changes. Will let you know.

Hi Karim,
I was using the old version of the module (awesome stuff!) and just updated to the new release.  I noticed that the .getfeature method is not working.
Thanks,
Sarah

Hi @Tcboo , sorry for the delayed response.
I just released a new bugfix version 0.22 on pypi that clarifies in the readme which methods to use. Specifically I have changed the getfeature, addfeature, replacefeature, and removefeature to have underscore between the words, such as get_feature, replace_feature, etc. I don't want to break yours or others old scripts, so I have also brought back the original syntax purely for legacy use, and added a "will be depreceated" warning.
@dieguico I also fixed the internal bug in the new version, so that setting with feat.geometry = ... or feat.property = ... should work. Though I kept the dictionary data structure, meaning that Feature and Geometry instances are simply wrappers around the underlying data.
Karim

👍🏼


package: update engines

Update the required version of Node.js to 4.2 (the current LTS release, "Argon") as per discussion in #470.
I also removed the engine-strict because 1) it's called engineStrict so it wasn't used and 2) it's deprecated and isn't respected anymore.
@rwaldron: In the other issue you wrote 4.2.1, but I couldn't see anything in the changelog between 4.2.0 and 4.2.1 that would indicate that we really don't work on 4.2.0. I think that we should either say "the latest LTS" (4.3.1) or "any version of the current LTS" (4.2.0). Did I miss anything?

The Travis failure is bogus

Thanks for the quick pull :+1:

😬


about compile error

Hi.
I'm using 14.04 LTS version of ubuntu.
and I'm trying to install this program
but when I 'make' in the 'build' directory, it shows compiling error like below.
kev8305@cropgenomics-PowerEdge-R910:/data/program/bamtools/build$ make
Scanning dependencies of target SharedHeaders
[  1%] Exporting SharedHeaders
[  1%] Built target SharedHeaders
Scanning dependencies of target APIHeaders
[  2%] Exporting APIHeaders
[  2%] Built target APIHeaders
Scanning dependencies of target AlgorithmsHeaders
[  3%] Exporting AlgorithmsHeaders
[  3%] Built target AlgorithmsHeaders
Scanning dependencies of target BamTools
[  4%] Building CXX object src/api/CMakeFiles/BamTools.dir/BamAlignment.cpp.o
[  5%] Building CXX object src/api/CMakeFiles/BamTools.dir/BamMultiReader.cpp.o
[  6%] Building CXX object src/api/CMakeFiles/BamTools.dir/BamReader.cpp.o
[  7%] Building CXX object src/api/CMakeFiles/BamTools.dir/BamWriter.cpp.o
[  8%] Building CXX object src/api/CMakeFiles/BamTools.dir/SamHeader.cpp.o
[  9%] Building CXX object src/api/CMakeFiles/BamTools.dir/SamProgram.cpp.o
[ 10%] Building CXX object src/api/CMakeFiles/BamTools.dir/SamProgramChain.cpp.o
[ 11%] Building CXX object src/api/CMakeFiles/BamTools.dir/SamReadGroup.cpp.o
[ 12%] Building CXX object src/api/CMakeFiles/BamTools.dir/SamReadGroupDictionary.cpp.o
[ 13%] Building CXX object src/api/CMakeFiles/BamTools.dir/SamSequence.cpp.o
[ 14%] Building CXX object src/api/CMakeFiles/BamTools.dir/SamSequenceDictionary.cpp.o
[ 15%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/bam/BamHeader_p.cpp.o
[ 16%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/bam/BamMultiReader_p.cpp.o
[ 17%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/bam/BamRandomAccessController_p.cpp.o
[ 18%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/bam/BamReader_p.cpp.o
[ 19%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/bam/BamWriter_p.cpp.o
[ 20%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/index/BamIndexFactory_p.cpp.o
[ 21%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/index/BamStandardIndex_p.cpp.o
/data/program/bamtools/src/api/internal/index/BamStandardIndex_p.cpp: In member function ??void BamTools::Internal::BamStandardIndex::WriteLinearOffsets(const int&, BamTools::Internal::BaiLinearOffsetVector&)??:
/data/program/bamtools/src/api/internal/index/BamStandardIndex_p.cpp:958:89: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
if ( numBytesWritten != (sizeof(offsetCount) + linearOffsets.size()*sizeof(uint64_t)) )
^
[ 22%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/index/BamToolsIndex_p.cpp.o
[ 23%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/io/BamDeviceFactory_p.cpp.o
[ 24%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/io/BamFile_p.cpp.o
[ 25%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/io/BamFtp_p.cpp.o
[ 26%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/io/BamHttp_p.cpp.o
/data/program/bamtools/src/api/internal/io/BamHttp_p.cpp: In member function ??bool BamTools::Internal::BamHttp::SendGetRequest(size_t)??:
/data/program/bamtools/src/api/internal/io/BamHttp_p.cpp:409:62: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
if ( WriteToSocket(requestHeader.c_str(), headerSize) != headerSize ) {
^
/data/program/bamtools/src/api/internal/io/BamHttp_p.cpp: In member function ??bool BamTools::Internal::BamHttp::SendHeadRequest()??:
/data/program/bamtools/src/api/internal/io/BamHttp_p.cpp:501:62: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
if ( WriteToSocket(requestHeader.c_str(), headerSize) != headerSize ) {
^
[ 27%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/io/BamPipe_p.cpp.o
[ 28%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/io/BgzfStream_p.cpp.o
[ 29%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/io/ByteArray_p.cpp.o
[ 30%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/io/HostAddress_p.cpp.o
[ 31%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/io/HostInfo_p.cpp.o
[ 32%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/io/HttpHeader_p.cpp.o
[ 33%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/io/ILocalIODevice_p.cpp.o
[ 34%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/io/RollingBuffer_p.cpp.o
[ 35%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/io/TcpSocket_p.cpp.o
[ 36%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/io/TcpSocketEngine_p.cpp.o
[ 37%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/io/TcpSocketEngine_unix_p.cpp.o
[ 38%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/sam/SamFormatParser_p.cpp.o
[ 39%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/sam/SamFormatPrinter_p.cpp.o
[ 40%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/sam/SamHeaderValidator_p.cpp.o
[ 41%] Building CXX object src/api/CMakeFiles/BamTools.dir/internal/utils/BamException_p.cpp.o
Linking CXX shared library ../../../lib/libbamtools.so
[ 41%] Built target BamTools
Scanning dependencies of target BamTools-static
[ 42%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/BamAlignment.cpp.o
[ 43%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/BamMultiReader.cpp.o
[ 44%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/BamReader.cpp.o
[ 45%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/BamWriter.cpp.o
[ 46%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/SamHeader.cpp.o
[ 47%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/SamProgram.cpp.o
[ 48%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/SamProgramChain.cpp.o
[ 49%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/SamReadGroup.cpp.o
[ 50%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/SamReadGroupDictionary.cpp.o
[ 51%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/SamSequence.cpp.o
[ 52%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/SamSequenceDictionary.cpp.o
[ 53%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/bam/BamHeader_p.cpp.o
[ 54%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/bam/BamMultiReader_p.cpp.o
[ 55%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/bam/BamRandomAccessController_p.cpp.o
[ 56%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/bam/BamReader_p.cpp.o
[ 57%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/bam/BamWriter_p.cpp.o
[ 58%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/index/BamIndexFactory_p.cpp.o
[ 59%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/index/BamStandardIndex_p.cpp.o
/data/program/bamtools/src/api/internal/index/BamStandardIndex_p.cpp: In member function ??void BamTools::Internal::BamStandardIndex::WriteLinearOffsets(const int&, BamTools::Internal::BaiLinearOffsetVector&)??:
/data/program/bamtools/src/api/internal/index/BamStandardIndex_p.cpp:958:89: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
if ( numBytesWritten != (sizeof(offsetCount) + linearOffsets.size()*sizeof(uint64_t)) )
^
[ 60%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/index/BamToolsIndex_p.cpp.o
[ 61%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/io/BamDeviceFactory_p.cpp.o
[ 62%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/io/BamFile_p.cpp.o
[ 63%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/io/BamFtp_p.cpp.o
[ 64%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/io/BamHttp_p.cpp.o
/data/program/bamtools/src/api/internal/io/BamHttp_p.cpp: In member function ??bool BamTools::Internal::BamHttp::SendGetRequest(size_t)??:
/data/program/bamtools/src/api/internal/io/BamHttp_p.cpp:409:62: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
if ( WriteToSocket(requestHeader.c_str(), headerSize) != headerSize ) {
^
/data/program/bamtools/src/api/internal/io/BamHttp_p.cpp: In member function ??bool BamTools::Internal::BamHttp::SendHeadRequest()??:
/data/program/bamtools/src/api/internal/io/BamHttp_p.cpp:501:62: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
if ( WriteToSocket(requestHeader.c_str(), headerSize) != headerSize ) {
^
[ 65%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/io/BamPipe_p.cpp.o
[ 66%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/io/BgzfStream_p.cpp.o
[ 67%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/io/ByteArray_p.cpp.o
[ 68%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/io/HostAddress_p.cpp.o
[ 69%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/io/HostInfo_p.cpp.o
[ 70%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/io/HttpHeader_p.cpp.o
[ 71%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/io/ILocalIODevice_p.cpp.o
[ 72%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/io/RollingBuffer_p.cpp.o
[ 73%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/io/TcpSocket_p.cpp.o
[ 74%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/io/TcpSocketEngine_p.cpp.o
[ 75%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/io/TcpSocketEngine_unix_p.cpp.o
[ 76%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/sam/SamFormatParser_p.cpp.o
[ 77%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/sam/SamFormatPrinter_p.cpp.o
[ 78%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/sam/SamHeaderValidator_p.cpp.o
[ 79%] Building CXX object src/api/CMakeFiles/BamTools-static.dir/internal/utils/BamException_p.cpp.o
Linking CXX static library ../../../lib/libbamtools.a
[ 79%] Built target BamTools-static
Scanning dependencies of target jsoncpp
[ 80%] Building CXX object src/third_party/jsoncpp/CMakeFiles/jsoncpp.dir/json_reader.cpp.o
[ 81%] Building CXX object src/third_party/jsoncpp/CMakeFiles/jsoncpp.dir/json_value.cpp.o
[ 82%] Building CXX object src/third_party/jsoncpp/CMakeFiles/jsoncpp.dir/json_writer.cpp.o
Linking CXX static library ../../../../lib/libjsoncpp.a
[ 82%] Built target jsoncpp
Scanning dependencies of target BamTools-utils
[ 83%] Building CXX object src/utils/CMakeFiles/BamTools-utils.dir/bamtools_fasta.cpp.o
[ 84%] Building CXX object src/utils/CMakeFiles/BamTools-utils.dir/bamtools_options.cpp.o
[ 85%] Building CXX object src/utils/CMakeFiles/BamTools-utils.dir/bamtools_pileup_engine.cpp.o
[ 86%] Building CXX object src/utils/CMakeFiles/BamTools-utils.dir/bamtools_utilities.cpp.o
Linking CXX static library ../../../lib/libbamtools-utils.a
[ 86%] Built target BamTools-utils
Scanning dependencies of target bamtools_cmd
[ 87%] Building CXX object src/toolkit/CMakeFiles/bamtools_cmd.dir/bamtools_convert.cpp.o
[ 88%] Building CXX object src/toolkit/CMakeFiles/bamtools_cmd.dir/bamtools_count.cpp.o
[ 89%] Building CXX object src/toolkit/CMakeFiles/bamtools_cmd.dir/bamtools_coverage.cpp.o
[ 90%] Building CXX object src/toolkit/CMakeFiles/bamtools_cmd.dir/bamtools_filter.cpp.o
[ 91%] Building CXX object src/toolkit/CMakeFiles/bamtools_cmd.dir/bamtools_header.cpp.o
[ 92%] Building CXX object src/toolkit/CMakeFiles/bamtools_cmd.dir/bamtools_index.cpp.o
[ 93%] Building CXX object src/toolkit/CMakeFiles/bamtools_cmd.dir/bamtools_merge.cpp.o
[ 94%] Building CXX object src/toolkit/CMakeFiles/bamtools_cmd.dir/bamtools_random.cpp.o
[ 95%] Building CXX object src/toolkit/CMakeFiles/bamtools_cmd.dir/bamtools_resolve.cpp.o
[ 96%] Building CXX object src/toolkit/CMakeFiles/bamtools_cmd.dir/bamtools_revert.cpp.o
[ 97%] Building CXX object src/toolkit/CMakeFiles/bamtools_cmd.dir/bamtools_sort.cpp.o
[ 98%] Building CXX object src/toolkit/CMakeFiles/bamtools_cmd.dir/bamtools_split.cpp.o
/data/program/bamtools/src/toolkit/bamtools_split.cpp: In member function ??bool BamTools::SplitTool::SplitToolPrivate::SplitListTagImpl(BamTools::BamAlignment&)??:
/data/program/bamtools/src/toolkit/bamtools_split.cpp:517:39: error: ??WriterMap?? has no member named ??cend??
if (writerIter == outputFiles.cend()) {
^
/data/program/bamtools/src/toolkit/bamtools_split.cpp: In instantiation of ??bool BamTools::SplitTool::SplitToolPrivate::SplitListTagImpl(BamTools::BamAlignment&) [with T = signed char]??:
/data/program/bamtools/src/toolkit/bamtools_split.cpp:415:95:   required from here
/data/program/bamtools/src/toolkit/bamtools_split.cpp:504:86: error: ??TagValueType?? has no member named ??cbegin??
typename TagValueType::const_iterator tagValueIter = currentValue.cbegin();
^
/data/program/bamtools/src/toolkit/bamtools_split.cpp:505:84: error: ??TagValueType?? has no member named ??cend??
typename TagValueType::const_iterator tagValueEnd  = currentValue.cend();
^
/data/program/bamtools/src/toolkit/bamtools_split.cpp: In instantiation of ??bool BamTools::SplitTool::SplitToolPrivate::SplitListTagImpl(BamTools::BamAlignment&) [with T = short int]??:
/data/program/bamtools/src/toolkit/bamtools_split.cpp:416:96:   required from here
/data/program/bamtools/src/toolkit/bamtools_split.cpp:504:86: error: ??TagValueType?? has no member named ??cbegin??
typename TagValueType::const_iterator tagValueIter = currentValue.cbegin();
^
/data/program/bamtools/src/toolkit/bamtools_split.cpp:505:84: error: ??TagValueType?? has no member named ??cend??
typename TagValueType::const_iterator tagValueEnd  = currentValue.cend();
^
/data/program/bamtools/src/toolkit/bamtools_split.cpp: In instantiation of ??bool BamTools::SplitTool::SplitToolPrivate::SplitListTagImpl(BamTools::BamAlignment&) [with T = int]??:
/data/program/bamtools/src/toolkit/bamtools_split.cpp:417:96:   required from here
/data/program/bamtools/src/toolkit/bamtools_split.cpp:504:86: error: ??TagValueType?? has no member named ??cbegin??
typename TagValueType::const_iterator tagValueIter = currentValue.cbegin();
^
/data/program/bamtools/src/toolkit/bamtools_split.cpp:505:84: error: ??TagValueType?? has no member named ??cend??
typename TagValueType::const_iterator tagValueEnd  = currentValue.cend();
^
/data/program/bamtools/src/toolkit/bamtools_split.cpp: In instantiation of ??bool BamTools::SplitTool::SplitToolPrivate::SplitListTagImpl(BamTools::BamAlignment&) [with T = unsigned char]??:
/data/program/bamtools/src/toolkit/bamtools_split.cpp:418:96:   required from here
/data/program/bamtools/src/toolkit/bamtools_split.cpp:504:86: error: ??TagValueType?? has no member named ??cbegin??
typename TagValueType::const_iterator tagValueIter = currentValue.cbegin();
^
/data/program/bamtools/src/toolkit/bamtools_split.cpp:505:84: error: ??TagValueType?? has no member named ??cend??
typename TagValueType::const_iterator tagValueEnd  = currentValue.cend();
^
/data/program/bamtools/src/toolkit/bamtools_split.cpp: In instantiation of ??bool BamTools::SplitTool::SplitToolPrivate::SplitListTagImpl(BamTools::BamAlignment&) [with T = short unsigned int]??:
/data/program/bamtools/src/toolkit/bamtools_split.cpp:419:97:   required from here
/data/program/bamtools/src/toolkit/bamtools_split.cpp:504:86: error: ??TagValueType?? has no member named ??cbegin??
typename TagValueType::const_iterator tagValueIter = currentValue.cbegin();
^
/data/program/bamtools/src/toolkit/bamtools_split.cpp:505:84: error: ??TagValueType?? has no member named ??cend??
typename TagValueType::const_iterator tagValueEnd  = currentValue.cend();
^
/data/program/bamtools/src/toolkit/bamtools_split.cpp: In instantiation of ??bool BamTools::SplitTool::SplitToolPrivate::SplitListTagImpl(BamTools::BamAlignment&) [with T = unsigned int]??:
/data/program/bamtools/src/toolkit/bamtools_split.cpp:420:97:   required from here
/data/program/bamtools/src/toolkit/bamtools_split.cpp:504:86: error: ??TagValueType?? has no member named ??cbegin??
typename TagValueType::const_iterator tagValueIter = currentValue.cbegin();
^
/data/program/bamtools/src/toolkit/bamtools_split.cpp:505:84: error: ??TagValueType?? has no member named ??cend??
typename TagValueType::const_iterator tagValueEnd  = currentValue.cend();
^
/data/program/bamtools/src/toolkit/bamtools_split.cpp: In instantiation of ??bool BamTools::SplitTool::SplitToolPrivate::SplitListTagImpl(BamTools::BamAlignment&) [with T = float]??:
/data/program/bamtools/src/toolkit/bamtools_split.cpp:421:94:   required from here
/data/program/bamtools/src/toolkit/bamtools_split.cpp:504:86: error: ??TagValueType?? has no member named ??cbegin??
typename TagValueType::const_iterator tagValueIter = currentValue.cbegin();
^
/data/program/bamtools/src/toolkit/bamtools_split.cpp:505:84: error: ??TagValueType?? has no member named ??cend??
typename TagValueType::const_iterator tagValueEnd  = currentValue.cend();
^
make[2]: *** [src/toolkit/CMakeFiles/bamtools_cmd.dir/bamtools_split.cpp.o] Error 1
make[1]: *** [src/toolkit/CMakeFiles/bamtools_cmd.dir/all] Error 2
make: *** [all] Error 2
would you like to let me know what is the problem?

I have had a similar problem. Just about everything installs, but then I get these exact errors. None of the EXEs appear in the ./bin directory.


Mesh doesn't load on the Vercel demo

No errors or anything in my console or Network tab.
Works on CodeSandbox though!

Ahh I'm sorry. I was just finishing the demo scene before publishing it ahah. I got me in the middle ;).

haha no worries! It's working now! Thanks again for the cool material! <3


turn on organic fract option in VIC 5

In VIC 4, @tbohn added an ORGANIC_FRACT option in as part of his dissertation work, which, when set to true in the global parameter file, read in the bulk and soil density of organic matter as well as the fraction of organic matter in each layer of soil. This was then used in the frozen soils routine. We need to turn this on in VIC 5 to incorporate organic content of the soil into our RASM simulations.
Additionally, we need to account for the new option BULK_DENSITY_COMB that enables us to read the combined bulk density (mineral and organic) in from the parameter file. If this is set to True, we want to specify that we read in the organic fraction and soil density of the organic matter, not the bulk density of the organic matter separately.

Description of feature
Description of the problem the feature addresses

See above.

Will the proposed feature be backward compatible?

Yes.

Will the proposed feature change the science results of VIC?

If the ORGANIC_FRACT option is set to True, yes - as the organic fraction of the soil is used extensively in the frozen soils routine and will result in different soil temperatures in areas where the organic content of the soil is greater than 0.

closed via #837


Add Momina-Rezai to Before I Die

Issue Addressed

Proposed Changes and Benefits

Screenshots

Notes to Reviewers

Checklist

[x] I have thoroughly read and understand The Contribution Guidelines.
[x] The title of this PR follows the keyword: brief description of change format, using one of the following keywords:

Feature - adds new or amends existing user-facing behavior.
Chore - changes that have no user-facing value, refactors, dependency bumps, etc.
Fix - bug fixes.
Docs - fixing typos and adding new content.

[x] I have listed my proposed changes and their benefits.
[x] I have linked the corresponding issue in the Issues section.

@Mominarez  Your pull request has been successfully merged!!! Congratulations
If you haven't already, please go ahead and star this repo and you are more than welcome to continually contribute to this project by assisting with our open issues, or adding your own new open issues, and additionally you can get an idea of ideas we have to continually grow this project further as technology and tech stack continually evolve by looking at the ROADMAP for the project. Have a great day, and thank you for taking the time to contribute, Momina! You can view your contribution now on the live link. Have a great weekend!
Xander


farmr error in the logs. Been finding it on all the chains.

error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory

same here and fullnode go not synced (machinaris 0.6.7)

After restarting the container blockchain start to sync.
2 hours later again stop syncing and get in logs :
reply: 'HTTP/1.1 200 OK\r\n'
header: Server: gunicorn
header: Connection: close
header: Content-Type: text/html; charset=utf-8
header: Content-Length: 17
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory
error: stat of /root/.chia/farmr/log*txt failed: No such file or directory

now list in that directory (/root/.chia/farmr/) is :
4.0K drwxr-sr-x 2 root users 4.0K Dec  2 20:22 blockchain
4.0K drwxr-sr-x 2 root users 4.0K Dec  5 12:20 cache
4.0K drwxr-sr-x 2 root users 4.0K Dec  2 20:22 config
4.0K -rw-r--r-- 1 root users   48 Dec  2 20:22 id.json
 48K -rw-r--r-- 1 root users  47K Dec  5 12:20 log-xch.txt.1
 68K -rw-r--r-- 1 root users  68K Dec  2 20:26 log-xch.txt.2
4.0K -rw-r--r-- 1 root users   60 Dec  5 12:15 log.txt.1
4.0K -rw-r--r-- 1 root users   60 Dec  2 20:25 log.txt.2
4.0K -rw-r--r-- 1 root users   31 Dec  5 11:04 override-xch-binary.txt

There is missing log*txt files.
I really hope that help to resolve the issue.

Thanks.  Please try upgrading all your containers to :test as this spurious log message has already been addressed there.

I have the same message, but I also get CPU use under 2%. It was up to 80%, but no longer seems to be plotting.

Marking as fixed in v0.6.8.  Based on user feedback after using Farmr for a month, I have set farmr as DISABLED by default.  One must specifically enable farmr if one really wants such additional monitoring. Thanks again!


Stopping/and starting VerneMQ on centos doesn't work after upgrade

Environment

VerneMQ Version: 1.2.1/1.2.3 (upgrading from 1.2.1 to 1.2.3)
OS: Centos7/RedHat7
Erlang/OTP version (if building from source):

Expected behavior
If VerneMQ is installed and running, then after doing yum upgrade ./vernemq-1.2.3-1.el7.centos.x86_64.rpm doing systemctl stop vernemq.service should work.
Actual behaviour
Instead of the expected behaviour, the systemctl stop vernemq.service command hangs.
Likely the VerneMQ service should be automatically stopped when the package is installed and started again afterwards. Everything works if the service is stopped and started manually before the upgrade. This is likely an issue with the rpm package itself.

I think this was fixed in 1.8.0


Change Http404 to an appropriate HttpResponseForbidden

Reference: https://en.wikipedia.org/wiki/HTTP_403

Tests are failing, which means this won't be merged in.
But I'm also not keen on changing this here.
The "resource", or what we're calling the resource here, isn't forbidden, it doesn't exist. The resource here being a valid registration invitation. If anything, replacing it with HTTP 410 would make more sense, but I don't see a compelling reason to change this.


fix: memory setter bug

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

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fish count of zeros

Note that there are fish counts of zero in older data (all with survey status = "MIGRA")  and dplyr::uncount() drops those rows. Question for Corey - are those mistakes in an import (and if so should be 1 or possibly another number?) or are they in there to indicate zero catch?

Currently the SURVEY_STATUS field is in the garbage bin. Consider elevating to preserve if useful to future merges/work/anything. Otherwise, if it is only useful as a diagnostic to fix a contributed file, it can stay in the garbage bin (those things aren't dropped, just sorta tagged as "not needed to us").

From a GH slack message: "FYI the issue I raised where TOTAL_CATCH=0 in raw data is not an issue -at least, not the kind I thought it was. In fact, these are duplicates, so we do NOT need to add in the data as I was doing. They can just be dropped because all the fish counted in the CPUE data have lengths in the raw data. At least for the GN surveys."

We need to look at the total count column in the individual fish file to verify whether or not there is a need for an uncount of those indiv fish data.


Static files leads to False positives

Describe the bug
Currently this rule test on static files like (CSS, JS) which leads to false positive.
Expected behavior
Skip the test on static files
Would you like to help fix this issue?
Yes

@itsdivyanshjain thanks for reporting the issue. Is it possible to add the false positive report or little bit more details so that we can understand more?
Also, please go ahead with implementation.
thanks,
Karan


vscode terminal maximum, no responding for a long time

VSCode Version:1.23.1
OS Version:Win10

Steps to Reproduce:
1.open vscode , the terminal on the bottom
2. Click "Maximize Panel Size", the window no responding while doing any input

Can you try the insiders build and add this setting to see if it's fixed?
"terminal.integrated.rendererType": "dom"

It works well according to your desc.

Great! Thats the current workaround for this class of issues, this setting will hit stable this week.


chore(master): release 2.0.1

:robot: I have created a release beep boop
2.0.1 (2024-07-11)
Bug Fixes

literal: fix bin_number parse (c072507)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v2.0.1
:sunflower: