[Bug] Fivetran UI data differs from Fivetran Log data

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the issue
I am seeing discrepancies between some of the usage metrics in the Fivetran UI versus the Logs from the dbt package. This includes dollars spent and MARs. The amount in Fivetran is always slightly higher. The plan rate ($/M) is slightly higher in the dbt logs as well. I'm assuming some of these fields are a result of a mathematical formula, so perhas one of the values is off.
I am querying the data from fivetran_log__usage_mar_destination_history and am on the latest version of the package (0.6.3).
Note however that MARs are also off when querying from fivetran_log__mar_table_history
Relevant error log or model output
No response
Expected behavior
Expect values to be identical for same period
dbt Project configurations
/
Package versions
0.6.3
What database are you using dbt with?
snowflake
dbt Version
1.2.1
Additional Context
No response
Are you willing to open a PR to help address this issue?

[X] Yes.
[ ] Yes, but I will need assistance and will schedule time during our office hours for guidance
[ ] No.

Hi @JadAbdallah thanks so much for opening this issue and sorry to hear your totals are off between the package totals and the UI.
Are the totals consistently off, or is this only happening for the current month?

@fivetran-joemarkiewicz It's consistent!

Just curious, do these totals seem to be consistently higher or lower than what is listed on the Fivetran UI?

Consistently lower, by around 100 units (for dollars spent). I looked at whether it was a a consistent extra X, due to perhaps an additional parameter in the calculation, but it fluctuated

@JadAbdallah thanks for these additional details. I have actually been looking into a new table that was rolled out to our Fivetran Log package which may be the reason you are seeing some issue with the end models. A new table incremental_mar has been added to the connector and looks to be replacing the active_volume table (which our package currently uses in the downstream models that calculate your spend).
As such, I have opened Issue #65 to look into this further. I will look into this some more the rest of this week (and into next) and hope to have something for you to try and that could fix this issue.
Before I dig deeper into this new table, can you confirm if you have the incremental_mar table in your Fivetran Log source schema?

I dont' see it actually, but do see the active_volume table.

Hmmm 🤔
Do you know if you have any free MAR or credits? I know the incremental_mar table is intended to help display the paid vs. free MAR. This may be why the active_volume calculation is a bit off on your end. Once the incremenal_mar table is loaded we could then try and switch to using that source.

Do you know why I don't have the incremental_mar table? I would assume that if anything, MAR would be higher on my end as I cannot differentiate free from paid. I would expect the Fivetran UI to display billed amount correctly. That amount is higher than what I am seeing from the package.

I know this incremental_mar table was just recently rolled out. It may not have been rolled out to your account yet.
Would you be open to sharing a screenshot of the usage the Fivetran UI is showing for a period, and the relevant output of the dbt package? If you are uncomfortable sharing in an open GitHub Issue, you can start a private support ticket where we chan discuss off a public forum if you desired.

Dbt package:

Fivetran UI:

Thanks for sharing this @JadAbdallah. I have tried digging into the logic on my end and am having some trouble identifying an issue.
I think this may be easier and more efficient to solve directly on a call. Would you be open to scheduling some time with my team and I using our Calendly link? This way we may debug the variance live on a call.

Done!

Following up on this issue.  I'm having a similar problem, but with a much larger variance.  Our MAR for November was upwards of 12MM, yet the FIVETRAN_LOG__USAGE_MAR_DESTINATION_HISTORY table is showing approximately 600K mar.  I can open a new issue, if necessary.

Hi @pkanter happy to keep the conversation here.
Would you be able to share which version of the package you are using?

I'm using 0.7.0.
On Mon, Feb 6, 2023 at 2:39 PM Joe Markiewicz @.***>
wrote:

Hi @pkanter https://github.com/pkanter happy to keep the conversation
here.
Would you be able to share which version of the package you are using?
—
Reply to this email directly, view it on GitHub
https://github.com/fivetran/dbt_fivetran_log/issues/63#issuecomment-1419643331,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFKLZQ2OOYVKKACFNDADTLTWWFHOJANCNFSM6AAAAAAQ3AJ5AI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks @pkanter. We released a few updates in 0.7.0 that added some more nuance to the MAR calculation (specifically free vs paid MAR).
What field in the package are you using to calculate your MAR? I wonder if you may only be using the paid MAR, but the sum of paid and free may equate to the 12MM you are seeing?

@fivetran-joemarkiewicz , I'm taking the number directly from the Usage MAR Destnation History table in the fivetran_log package.  The dollars are correct, but the rows are off considerably.  The sum of free and paid don't even come to ten percent of the 12MM, at 94k  and 606k respectively.

The dollars are correct, but the rows are off considerably.

This statement jumps out the most to me 🤔. I would assume that if the MAR is off then the dollars spent would be as well. I think it may make the most sense if you schedule some time with my team to investigate this live on a call. You can book time here. In the meantime, I will continue to investigate.

Will do.  Keep me posted.
From: Joe Markiewicz @.>
Sent: Wednesday, February 8, 2023 10:27 AM
To: fivetran/dbt_fivetran_log @.>
Cc: Paul Kanterman @.>; Mention <
@.>
Subject: Re: [fivetran/dbt_fivetran_log] [Bug] Fivetran UI data differs
from Fivetran Log data (Issue #63)
The dollars are correct, but the rows are off considerably.
This statement jumps out the most to me 🤔. I would assume that if the MAR
is off then the dollars spent would be as well. I think it may make the
most sense if you schedule some time with my team to investigate this live
on a call. You can book time here
https://calendly.com/fivetran-solutions-team/fivetran-solutions-team-office-hours.
In the meantime, I will continue to investigate.
—
Reply to this email directly, view it on GitHub
https://github.com/fivetran/dbt_fivetran_log/issues/63#issuecomment-1422782777,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFKLZQ532AREW7HLGNMLX53WWO3NLANCNFSM6AAAAAAQ3AJ5AI
.
You are receiving this because you were mentioned.Message ID: <
@.***>

One thought...the INCREMENTAL_MAR table is only a staging table right now, no final production table is derived from it.  The numbers are correct there.

Thanks for adding that note. The latest version of the package should be leveraging the incremental_mar table through to the final model. See here for how the staging model should flow through to the final usage_mar_destination_history model.
I know you mentioned you are using v0.7.0, but are there any other configs in your dbt_project.yml that may be causing it to not flow to the final downstream models?

The only vars related to the fivetran_log package are "using_transformations" and "using_triggers" which are both false.

Are there any model configs related to the fivetran_log package?

No, forgot to mention that.

Thanks for the additional details. I will continue to look into this. I also appreciate you setting up some time with us for this week. We can go into more debugging then!

Thanks for meeting with our team @pkanter!
Recapping our meeting, we discovered that this line in the fivetran_log__mar_table_history model is erroneously filtering out records from the incremental_mar staging model.
This line was required with the active_volume staging model. However, since that field was deprecated and replaced with incremental_mar, we no longer need to account for the cumulative totals. Instead, we need to include the previous records to accurately total the mar per connector.
I will soon open a PR to address this issue.

Good talking with you and the team today.  Glad I could help squash a bug.
From: Joe Markiewicz @.>
Sent: Thursday, February 9, 2023 2:33 PM
To: fivetran/dbt_fivetran_log @.>
Cc: Paul Kanterman @.>; Mention <
@.>
Subject: Re: [fivetran/dbt_fivetran_log] [Bug] Fivetran UI data differs
from Fivetran Log data (Issue #63)
Thanks for meeting with our team @pkanter https://github.com/pkanter!
Recapping our meeting, we discovered that this line
https://github.com/fivetran/dbt_fivetran_log/blob/226deff6d4f10b19bad5ccff1a6bc6813e809fe7/models/fivetran_log__mar_table_history.sql#L60
in the fivetran_log__mar_table_history model is erroneously filtering out
records from the incremental_mar staging model.
This line was required with the active_volume staging model. However, since
that field was deprecated and replaced with incremental_mar, we no longer
need to account for the cumulative totals. Instead, we need to include the
previous records to accurately total the mar per connector.
I will soon open a PR to address this issue.
—
Reply to this email directly, view it on GitHub
https://github.com/fivetran/dbt_fivetran_log/issues/63#issuecomment-1424707457,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFKLZQ4TH46VABVZJV6LQDDWWVA5DANCNFSM6AAAAAAQ3AJ5AI
.
You are receiving this because you were mentioned.Message ID: <
@.***>

Hi all,
I wanted to post back to let you know that the changes were merged and a new v0.7.1 release has been cut! You should see this live at the top of the hour.
As the issue has been addressed in the latest release, I will close this issue.

@fivetran-joemarkiewicz , I've updated to the 0.7.1 version of the package.  when I ran dbt build, a unique_combination_of_columns test fails.  The full message is below...
14:36:47  Failure in test dbt_utils_unique_combination_of_columns_fivetran_log__mar_table_history_connector_id__destination_id__table_name__measured_month (models\fivetran_log.yml)
14:36:47    Got 1310 results, configured to fail if != 0
14:36:47
14:36:47    compiled Code at target\compiled\fivetran_log\models\fivetran_log.yml\dbt_utils_unique_combination_o_9247891a6fc9fc9b6aa8554805786898.sql
14:36:47

@pkanter cross posting this comment from the other thread if you are available to test before we roll out the patch -
https://github.com/fivetran/dbt_fivetran_log/issues/73#issuecomment-1433349872. If you have a moment, would you be able to test this branch and verify if the test failures are resolved, and your end totals are still accurate.
packages:
  - git: https://github.com/simon-stepper/dbt_fivetran_log.git
    revision: 73-duplicated-months
    warn-unpinned: false


Remote debugging with VSCode fails with Linux Containers on Docker for Windows

Environment data
VS Code version:  1.19.2
Python Extension version:   2018.1.0
Python Version:   3.5.2
OS and version:    (Host) Windows 10 Enterprise, (Container)  Ubuntu 16.04.3 LT
PTVSD version: 3.2.1
Actual behavior
VS Code displays the error when starting python attach "There was an error in starting the debug server. Error = {"code":"ETIMEDOUT","errno":"ETIMEDOUT","syscall":"connect","address":"<IP_ADDRESS>","port":5678}"
Expected  behavior
Expected python to attach to my container.
Steps to reproduce:

Create firewall rule for port 5678 on host.
Start a Linux container under docker for windows with an port exposed.
Use Docker inspect to get IP address and port for container
Run python3

import ptvsd

address=('<IP_ADDRESS>',5678)
ptvsd.enable_attach('secret',address)
ptvsd.wait_for_attach()

Configure python attach in VS Code         {
"name": "Python: Attach",
"type": "python",
"request": "attach",
"localRoot": "${workspaceRoot}",
"remoteRoot": "/project/sawtooth-core",
"port": 5678,
"secret": "secret",
"host": "<IP_ADDRESS>"
},
Run python to attach in VS Code

Logs
Output from Python output panel
There was an error in starting the debug server. Error = {"code":"ETIMEDOUT","errno":"ETIMEDOUT","syscall":"connect","address":"<IP_ADDRESS>","port":5678}
Output from ```Console window``` (Help->Developer Tools menu)
There was an error in starting the debug server. Error = {"code":"ETIMEDOUT","errno":"ETIMEDOUT","syscall":"connect","address":"<IP_ADDRESS>","port":5678}
There was an error in starting the debug server. Error = {"code":"ETIMEDOUT","errno":"ETIMEDOUT","syscall":"connect","address":"<IP_ADDRESS>","port":5678}

The problem was that I did not have the ports exposed correctly in my yaml file.
Add the following to my yaml file.
expose:
- 4004
- 5678
ports:
- 5678:5678
VS Code configuration.
{
"name": "Python: Attach",
"type": "python",
"request": "attach",
"localRoot": "${workspaceRoot}",
"remoteRoot": "/project/sawtooth-core",
"port": 5678,
"secret": "secret",
"host": "localhost"
},
In python3 run the following.

import ptvsd
address=('<IP_ADDRESS>',5678)
ptvsd.enable_attach('secret',address)
ptvsd.wait_for_attach()

Thanks a bunch.

Make sure you install version <IP_ADDRESS> of ptvsd. Later versions hang on ptvsd.wait_fo_attach()


🔴 Ecosystem Project Idea: Smart Contract Code Verifier

What is an Ecosystem Project Idea?
An Ecosystem Project Idea is an idea for a project that someone could build within the Optimism ecosystem. As opposed to an Ecosystem Concept or an Ecosystem Idea, an Ecosystem Project is generally more fleshed out and corresponds to a relatively clear product to be built.

Summary
OP Mainnet and other OP Stack based chains encompass a multitude of smart contracts. It's crucial to have a tool or script that efficiently maps the code deployed on these blockchains to the code found within the corresponding GitHub repositories. This tool should be able to take a smart contract address (or list of addresses) and a commit hash and verify that the deployed code corresponds to the code from the specific commit hash.
Possible Features

Implement a tool or UI that accepts user inputs for L1 network name, contract address, and commit hash.
Allow the tool to download the corresponding GitHub repository at the specified commit hash and initialize it.
Compile contracts and compare the bytecode of the deployed contract with the actual bytecode.
Design the tool to handle proxies if necessary.
Throw alerts if unable to match the deployed code to what's in GitHub.

Impact

Enhances the safety of smart contract deployments by validating the origin of deployed code.
Improves auditability by providing an explicit link between the codebase in the repository and the deployed smart contracts.

Blockscout actually has their contract code verifier as a standalone service you can run,
https://github.com/blockscout/blockscout-rs/blob/main/smart-contract-verifier/README.md

I've been working on something like this - a Github action that can reconcile repository bytecode with on chain bytecode. The action will generate a report on each commit and if the bytecode doesn't match for a contract, then it'll state the previous commit which it does match such that it's easy to see where the deviations start to happen. I'll report back when I'm finished.

@sambacha integrating this into OP stack repos would be a good approach


Allow patchutils to be install through installed_extras

Issue Type

Feature Idea

User story
As a user who uses Drupal VM for contrib development, I would like to be able to install patchutils via installed_extras so that I can run the interdiff command inside the VM.

Thanks @oxyc


Website auth timing out

Is this project dead?
Wondering because I'm trying to authenticate via my Figma account but the connection keeps timing out.
Specifically this route --> https://api.tacos.wtf/chicken/v1/o.

Hi @jamesctucker, the server is not maintained anymore indeed, but you can easily run it locally or on your own server :) Don't hesitate if you need help with that!
Baptiste

Thanks for the reply, @BaptisteAmato! I will try local deployment.


Java 17 Parallel Old/New Parsing has altered from JDK 11, impacting what is reported as promoted bytes and what is reported in young gen

Describe the bug
Censum seems to be reporting on -xmn values as opposed to -xmx values but that is a red herring.  What has actually changed is that after a gc collection record there's no longer always a new gc total heap size record.
To Reproduce
Load the attached log, you'll note that full heap seems to be at 50GB when it should be 56GB, there are other calculations challenges.

java17_parallel_parsing_issue.txt

@kirk-microsoft as discussed on the call.  I'll let you take a look first before we 'fix' and add this to the test data


Investigate common integration testing framework

Currently each project has its own way of performing integration testing. This often involves standing up a Docker stack to simulate the production PASS stack. Unfortunately, various projects will have a different set of components, and may be using versions of these components which are different from each other and from prod.
It might make sense to have a common integration testing framework which each project could access for its integration tests. This should both simplify integration testing and provide greater confidence that the project will behave well in production when integration tests succees.

As part of this, also consider whether integration tests should remain as part of independent projects or be pulled together in an independent project that does nothing but integration testing.

Every Java project that does integration testing with the exception of pass-indexer uses Maven to setup a docker environment. Instead pass-indexer provides a docker-compose.yml. That docker-compose.yml setup could be moved into Maven.

For discussion: https://docs.google.com/document/d/1Yp-cBKfNCHqRLcsZQvn4SNyIu7DrSL183v0_cuDzqsc/edit?usp=sharing

Based on discussion of Mark's document (linked above), the team decided to back off on rebuilding the IT structure and focus instead on larger project architecture considerations, particularly the Fedora replacement work. These changes are likely to have a big impact on how ITs are built and executed. This ticket should be revisited soon after the repository replacement work has completed.

post 1.0

no longer needed


Stim layout 5

Extending to other type of behavioral layout.
Work in progress.

Changed to different branch for commit history.


runner: pass through jobID and taskID

https://jira.twitter.biz/browse/DX-2151

Codecov Report

Merging #242 into master will decrease coverage by 0.05%.
The diff coverage is 41.09%.

@@            Coverage Diff             @@
##           master     #242      +/-   ##
==========================================
- Coverage    59.2%   59.15%   -0.06%
==========================================
  Files          85       85
  Lines        4513     4546      +33
==========================================
+ Hits         2672     2689      +17
- Misses       1594     1610      +16
  Partials      247      247

Impacted Files
Coverage Δ

sched/scheduler/stateful_scheduler.go
74.55% <ø> (ø)
:arrow_up:

sched/generators.go
0% <0%> (ø)
:arrow_up:

runner/runners/queue.go
89.7% <100%> (ø)
:arrow_up:

sched/scheduler/task_runner.go
88.39% <100%> (+0.21%)
:arrow_up:

sched/definitions.go
84.21% <100%> (+0.57%)
:arrow_up:

runner/runner.go
100% <100%> (ø)
:arrow_up:

scootapi/server/run_job.go
74.35% <100%> (+0.67%)
:arrow_up:

workerapi/api.go
83.92% <100%> (+1.23%)
:arrow_up:

runner/status.go
20.48% <15.78%> (-0.07%)
:arrow_down:

runner/runners/invoke.go
62.28% <26.31%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@JeanetteBruno I thought that was another story. Not sure if we want that to be unique to each client, request, or task. Additionally not sure if we want to allow users to provide their own request tag or if it should be generated similar to a JobID.
@dgassaway agreed, I'm wondering if maybe they should be optional fields to mitigate risk of future changes.

I created DX-2156 for the external tag concept.  (This tag would be unique to each job request.)


Support TorchSharp on .NET Framework

CC @NiklasGustafsson @dsyme @michaelgsharp @ericstj

General comment -- having the new syntax and capabilities is most important in the public API, and less so in the implementation of TorchSharp itself.
If we can get rid of some of the #ifdefs by using common capabilities, let's do so unless it in a) and example or tutorial, or b) has some meaningful performance benefit.

@NiklasGustafsson

General comment -- having the new syntax and capabilities is most important in the public API, and less so in the implementation of TorchSharp itself.
If we can get rid of some of the #ifdefs by using common capabilities, let's do so unless it in a) and example or tutorial, or b) has some meaningful performance benefit.

I have addressed that. Let me know if there is any other specific place we need to change.


⚠ Bofrost.be has degraded performance

In 7f775b2, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1198 ms

Resolved: Bofrost.be performance has improved in 7ebf2d9.


multi node hello_world hangs

Hi there,
I'm trying to setup a basic multi node example referencing this stackoverflow thread.
I've this setup
Master:
./bin/hello_world --hpx:localities=2 --hpx:agas=localhost:7910 --hpx:hpx=localhost:7910 --hpx:threads 4

Slave:
./bin/hello_world --hpx:agas=node0:7910 --hpx:hpx=localhost:7910 --hpx:worker

In strace, Master just hangs, while the slave seems to constantly be trying to connect.
It's probable I'm going about this wrong way, any input is appreciated

Please read the stack-overflow answer more carefully. The following should work:
./hello_world --hpx:localities=2 --hpx:agas=node0:7910 --hpx:hpx=node0:7910 --hpx:threads 4
./hello_world --hpx:agas=node0:7910 --hpx:hpx=node1:7910 --hpx:worker

(use the appropriate hostnames instead of node0 and node1). If this does not work, try explicit IP-addresses instead of the hostnames.
The bottom line is that node0 and node1 should resolve to the correct IP address such that the two localities can communicate. If you use localhost this will always be resolved to <IP_ADDRESS> which makes sense in the context of a particular node only. HPX however sends the endpoint information you specify to locality zero where the localhost (which was meant to refer to node1) suddenly refers to node0, thus no connection can be established.

I have tried both with localhost and actual (resolving) hostnames, both hang.

IPs aren't working either.
Both devices are hot spotting from a phone, so I'll try setup a wired connection.

The reason for hanging can only be that the names you specify don't resolve to n IP address the other device can reach. The actual cause could be any of an endless list of possibilities, starting from firewall issues to a misconfigured DNS or simply not reachable IP addresses because of a missing route between the two endpoints. This is very difficult to diagnose from the distance.

the devices were pingable (by name or IP) on the hot spot network, but it was interferrring somehow.
It works with a wired static network


Add e2e simple test

This will be the building step for future e2e tests + profiling targets.
Use ffmpeg to stream a fixed video to FC2.

Launch the watcher.
Run ffmpeg to stream to fc2.
After the video ends, do assertions:

Is the file readable?
Somewhat try to do a image correlation of the X frame of the downloaded video with the frames of original video and check highest percentage.

Closed with: cdf20c969a0fae14b1fc9bae65a3b82945e28378


where to run javac?

where to run : javac -processor org.versly.rest.wsdoc.AnnotationProcessor *.java

If you run javac from the command line, you'll need to make sure the wsdoc jar is in your classpath. Are you setting that via an environment variable?


Using common I2C pins for ESP32 to prevent lockup

Using common I2C pins for ESP32 to prevent lockup when using MPU
Pin 12 cause boot to fail when pull to HIGH (which it does on mpu6050)

Source: https://randomnerdtutorials.com/esp32-pinout-reference-gpios
Pinout: https://lastminuteengineers.com/esp32-arduino-ide-tutorial/#esp32-development-board-pinout

Thanks. Also removed this pin from I2C discovery


[BUG] Issue with Flow Transaction from COA to EOA

Description:
I encountered an issue while attempting to transfer Flow from a COA  to an EOA . The intention was to transfer the balance from the Flow account to the EOA,  it seems that the script used was incorrect.
Steps to Reproduce:
Execute the transfer script designed to move Flow from COA to EOA.
Observe that the transfer actually from Flow account to EOA.
Expected Behavior:
The Flow balance should be successfully transferred from the COA to the specified EOA.
Actual Behavior:
The script used is transfer Flow from Flow account to EOA.
Script Details:
Script Name: callContract
Details: The value being the amount of Flow tokens, and the data should be empty, without a Cadence script.

There two bugs fixed in ios prod version 2.2.9(2).
(1) The Flow token balance should be successfully transferred from the COA to the specified EOA.
txid: https://www.flowscan.io/tx/0261e2cfc2b303087f749406550e903c0ea521f790e9b93b72a186e0d9f66222
(2) The other Ft balance should be successfully transferred from the COA to the specified EOA.
txid: https://www.flowscan.io/tx/3141d07ac2cf9c1f9d88c7ea1da68fa4a2b5046489eeb87b56fca5bd68738cee


DOC fix algorithm section in docstrings

Codecov Report

Merging #32 into develop will increase coverage by 0.04%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop      #32      +/-   ##
===========================================
+ Coverage    98.79%   98.83%   +0.04%
===========================================
  Files           54       54
  Lines         4384     4386       +2
  Branches       485      485
===========================================
+ Hits          4331     4335       +4
+ Misses          28       27       -1
+ Partials        25       24       -1

Impacted Files
Coverage Δ

skhubness/neighbors/graph.py
96.15% <ø> (ø)
:arrow_up:

skhubness/neighbors/tests/test_lof.py
97.95% <100%> (+1.36%)
:arrow_up:

skhubness/reduction/__init__.py
100% <100%> (ø)
:arrow_up:

skhubness/neighbors/tests/test_neighbors.py
99.68% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


refactor(fragmentor): some refactoring done when I tried to understand the code

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
I was trying to understand the underlying logic of the fragmentor. While I read the code, I found some suboptimal code that can be improved. This PR includes these improvements.
I was assigned to rewrite this code, and I think trivial simplifications are good starters.
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Release note
These are all internal changes :-)

Rebased


small typos fixing

Thank you! 👍

If you have other documents that you want me to change them tell me. 👍


ENH: update example to support non-overlapping images

Closes #1556.

/azp run ITK.Linux.Python

This PR is now ready for review. Previously failing were unrelated Python tests.


Dear author, can you provide the code to obtain the data label?

Hi, @mioyeah , for the face label extraction, I use dlib to extract 68 facial landmarks. For the body keypoints extraction, I use OpenPose.

Well, thank you

the < https://github.com/CMU-Perceptual-Computing-Lab/openpose > includes C++ ， how to get body keypoints extraction only includes python ?


(readme): update readme.

Codecov Report

Merging #567 (eea2849) into master (033126e) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head eea2849 differs from pull request most recent head 3bc08df. Consider uploading reports for the commit 3bc08df to get more accurate results

Additional details and impacted files

@@            Coverage Diff            @@
##             master     #567   +/-   ##
=========================================
  Coverage     15.57%   15.57%
  Complexity      441      441
=========================================
  Files           185      185
  Lines          8692     8692
  Branches        748      748
=========================================
  Hits           1354     1354
  Misses         7220     7220
  Partials        118      118


[Improvement] PostMessageIntent able to handle both public and private channel requests

This branch let's PostMessageIntent to be used for both public and private channel requests.
It uses a simple logical block and uses the following helper functions:-

getRoomId (returns roomId of a channel if the argument passed is name of a channel, undefined otherwise)
getGroupId (returns roomId of a group if the argument passed is name of a group, undefined otherwise)

It first checks if the channel is private, if it's not private it checks if it's public and then sends a message using postMessage(roomid, message, headers) method.
Additionally all the post message related requests are achieved using a single helper function postMessage(roomid, message, headers).
postGroupMessage and postDirectMessage functions are removed since they are no longer required.
Affected Intents:

PostMessageIntent
PostEmojiMessageIntent
PostGroupMessageIntent
PostGroupEmojiMessageIntent
PostDirectMessageIntent
PostEmojiDirectMessageIntent

Taken care by #102

Don't close issues unless PRs are merged. Just mention the issue in the PR description.


Make contentLabel optional

According to the ARIA spec, "If the label text is visible on screen, authors SHOULD use aria-labelledby and SHOULD NOT use aria-label." With that said, since the contentLabel uses aria-label, it shouldn't be required since ideally, if the label is visible on the page, then an id to that element should be provided to aria-labelledby.
https://www.w3.org/TR/wai-aria/states_and_properties#aria-label

While a more complex validation, I would support something that's either aria-labelledby OR contentLabel is required. I'm migration some existing modals to use the-modal, and this what I'll be implementing on our wrapper component.


Data Preparation and Analysis of Nigeria Crime Dataset

I created a full preparation and analysis of the Nigeria Crime Dataset

By the way @AiratAroyewun,  you didn't follow the correct naming convention of experimentation_{GitHub_Username}. You named it experiment then combined it with your first name. I have fixed that on my end already though.

Oh my bad, I'm just familiarizing myself with contributing on GitHub. Thank you.

Oh my bad, I'm just familiarizing myself with contributing on GitHub. Thank you.

You're doing great @AiratAroyewun. Weldone.


Fix dry run and  skip pre-release flags

Previously, any value for dry_run and skip_pre_release was considered as true, with this MR it's necessary to set the flag to true, t, yes or 1 (case-insensitive), to enable it.

LGTM otherwise!

LGTM otherwise!

Thank you @igrr, I updated the README


Wrf config to json

Description
At the moment the CMAQ config files are json and the WRF config files are json files with comments. The handling of the configuration file should be unified for WRF and CMAQ.

[x] Remove comments from WRF config files -> the files are now valid json files. All comments are in the config class.
[x] Remove the step that would read the config as a string
[x] Change config access in setup_for_wrf.py from config dict to WRFconfig object

Checklist
Please confirm that this pull request has done the following:

[x] Tests added
[ ] Documentation added (where applicable)

Notes

@lewisjared was there anything else I should change? If not, can you click on approve so I can merge.


File disappears from Dropzone after dragging and dropping file if the Dropzone is specified as a child of a Box

After dragging and dropping a file into the Dropzone, the file disappears from view. Noticed this when specifying the Dropzone as a child of a Box component.
Steps to reproduce:

Create Dropzone as a child of a Box
Drag a file into it
Wait 5 seconds - 2 minutes to see it disappear visually, as well as programmatically.

https://user-images.githubusercontent.com/48930920/133312379-ead98b07-ed1b-45d9-93e5-7d79b09e7b54.mov

Hey @IanKrieger , sorry for the late reply! I tried to reproduce this defect issue but I didn't see that file disappears in 5 s - 2 min. I think this issue has been properly solved, but could you please double check to see if this issue still persists? If this issue is still there, could you please send me code snippet? Thanks!

I'm going to go ahead and close this thread for now. If this is still an issue for you, please feel free to reopen this and provide a code snippet as @MaggieMaSS requested. Thanks!


Add an image/images using Stack?

I'm relatively inexperienced, so not sure if this is possible - but I would like to add some images to the bottom of the candle chart I have created. I thought I would be able to use Stack and Positioned widgets to add them. However putting my chart widget into a Stack breaks the chart. Here are the images that show what happens when I add a Stack only. Is this possible to do?
Container(
      child:  rangeChart(context),
),

Container(
         child: Stack(
                children: <Widget>[
                      rangeChart(context),
                 ]
         )
),

I added a container around my chart widget and set the height and with of the container to solve my problem.


typo

"Before beginning make sure you have a virtual machine and that you have changes the checkpoint type to Production checkpoints."
should be
"Before beginning make sure you have a virtual machine and that you have changed the checkpoint type to Production checkpoints."

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

ID: 4dd8cc6b-2dba-951b-03bc-98180d8b8a9e
Version Independent ID: 6549b0bd-8ea2-d5d3-ab61-e9bc5b2c3505
Content: Using checkpoints
Content Source: virtualization/hyper-v-on-windows/user-guide/checkpoints.md
Service: windows-10-hyperv
Product: windows-10-hyperv
GitHub Login: @scooley

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Contextual Speed Menu functions for Fixture Groups disabled

Describe the bug
With a Fixture Group selected, the right-click Speed Menu shows disabled Fixtures Group functions.
To Reproduce
Steps to reproduce the behavior:

Start QLC+ with a project (QXW file) containing a collection of fixtures
Select Fixtures page using main functions toolbar (bottom left)
Verify the project's fixtures, groups, and Universe(s) are displayed in the Fixture group pane of the main window (left side)
Click on a defined fixture group to select the group
Right-click on selected fixture group. Verify speed menu displays over the group.
Verify "Add Fixture to group" and "Remove Fixture from group" menu selections are greyed out and disabled.

Expected behavior
Problem: "Add Fixture to group" and "Remove Fixture from group" menu selections should be enabled as a context feature for the selected group (currently disabled)
Screenshots

Desktop (please complete the following information):

OS: Windows, Linux Mint, Ubuntu.
QLC+ Version: all versions to date, currently on 4.13.1
hardware setup (mandatory for audio/video issues): N/A - not a hardware problem

Additional context
N/A

Not a bug.
You're clicking a group, not a fixture. Therefore add fixture to group is not enabled.
Please consider reading some documentation and experiment with the software before opening issues here.
Otherwise please use the forums like everybody else. Someone will explain these mechanisms to you.
Thank you.


Support Request for H5086 Smart Plug

Model: H5086
Device Type: Smart plug
Connection Methods: WiFi & Bluetooth. Unsure about LAN mode

I don't use this project, but I needed to communicate with my H5086.
I have documented the changes in the protocol here (note that AA01 uses 01/00 to trun on/off the device)
https://github.com/egold555/Govee-Reverse-Engineering/pull/13
Hopefully it will help


start of work to add identifiers to result

we need to be able to present the "flattened" facts to a human and have something understandable. This first shot will do that -
adding the complete nesting path in the graph for each result. I have added to the changed value and will do added/removed
nodes next. Here is an example:
    "changed_node_value": [
        [
            "A",
            "B",
            "id29",
            "id29",
            "parameter",
            "denylisted_ops",
            "blocklisted_ops",
            "module:tensorflow.lite.python.wrap_toco->function:wrapped_experimental_mlir_quantize->parameter:denylisted_ops",
            "module:tensorflow.lite.python.wrap_toco->function:wrapped_experimental_mlir_quantize->parameter:blocklisted_ops"
        ],
        [
            "A",
            "B",
            "id31",
            "id31",
            "parameter",
            "denylisted_nodes",
            "blocklisted_nodes",
            "module:tensorflow.lite.python.wrap_toco->function:wrapped_experimental_mlir_quantize->parameter:denylisted_nodes",
            "module:tensorflow.lite.python.wrap_toco->function:wrapped_experimental_mlir_quantize->parameter:blocklisted_nodes"
        ]
    ]
}

Where the long paths are newly added, before you didn't have this context.
Signed-off-by: vsoch<EMAIL_ADDRESS>

the bugs in the test are me not being explicit enough in the pararms for nodes/relations - should be easy to fix (and we should be specific when creating them.


Type checking fails for Int32 operators

module NoTcInt32

open FStar.Int32

let add_mod' = FStar.Int.add_mod #32

let add_mod (i j : Int32.t) : Int32.t =
  (add_mod' (i <: FStar.Int.int_t 32) (j <: FStar.Int.int_t 32)) <: Int32.t

let add_mod (i j : Int32.t) : Int32.t =
(FStar.Int.add_mod i j) <: Int32.t

I was encountering this error yesterday on the c_mitls2c branch, and I was able to reproduce it on master.
Both of the bottom definitions fail, the one using the explicitly applied implicit argument #32 provides a better error message, it looks to that the type checker won't solve FStar.Int.int_t 32 <: FStar.Int32.t.
Upon further examination it seems that the code shoud not type check. If one looks in FStar.Int32.fst they would find the exposed integer type to be defined as:
private type t' = | Mk: v:int_t n -> t'
type t = t'

Many operations including add, sub, mul, and div are lifted on to this new type, but not operations such as add_mod (the one I was having trouble with). This means that they are being resolved to operations defined in FStar.Int.fst.
It may be the case that I'm forgetting about a coercion that is intended to make this work, but either way this seems like a confusing UX and FStar.In32 should define these operations for clarity's sake.
In order to rectify this, I propose we modify mk_int.sh to output  wrappers for operations defined in the integer modules.

There is intentionally no signed add_mod since it doesn't have an easy C implementation due to C's undefined behavior for overflow of signed numbers. You can do this with unsigned numbers easily. If you need it for signed numbers you'll have to implement it (and then we can probably add it to the standard library).
There is an op_Plus_Percent_Hat in FStar.IntN.fstp, which is a confusing alias of the Int operation on bounded nats that shouldn't be there.

@tchajed yeah I just read your commits before you commented, I now see what you were doing. Should we remove these aliases?

Which aliases? The op_Plus_Percent_Hats should definitely go.

Aren't all the *_mod operations now invalid? it looks like there are others defined in that file:
see https://github.com/FStarLang/FStar/blob/master/ulib/FStar.IntN.fstp#L100.

I merged a fix for this into master: https://github.com/FStarLang/FStar/commit/3f6b8a3f6d657e7f7ee97b6ba1c0d8f3975ac715
I was a little concerned about making this change without a PR or wider testing, but I think it'll be ok. The earlier removal of the *_mod operations already would have broken other code; this just removes some operators that had the wrong types anyway, as you ran into in this issue.


migrate mono-repo packages to own repositories

I had a conversation with the storybook team this morning and they mentioned an issue with sunsetting packages out of their main monorepo. They still want the package to existing, but need to move the contents and issues into its own repo.
Curious if a script could be made to transfer folders to their own repo. Issues labeled for that package could also be transferred in a similar way.
cc/ @shilman @ndelangen in case more context is needed

Happy to help building that script, sounds like use case that fits @octoherd's current scope. Is there any place I can find out more what you want to do? Could it all be done programmatically with existing data or would you need to prompt to enter e.g. the repository name for each package you want to migrate out of the monorepo?

Happy to help building that script, sounds like use case that fits @octoherd's current scope. Is there any place I can find out more what you want to do? Could it all be done programmatically with existing data or would you need to prompt to enter e.g. the repository name for each package you want to migrate out of the monorepo?

This would make sense to me:
An extract command that would ask:

the directory to extract
where to extract it to on fs
url for repository
tags to move issues to new repository

I assume the script would do the following

iterate through all folders in https://github.com/storybookjs/storybook/tree/next/addons
Create a new repository, e.g. for [/addons/a11y](https://github.com/storybookjs/storybook/tree/next/addons/a11y it would create https://github.com/storybookjs/a11y)
Move all contents to the new repository
Update package.json to point to the new repository
Move all issues with the label addon: a11y

I assume you'd also want to create additional files for testing/building/CI?
Does that sound right? Do all the packages have the 1:1 mapping of folder name to label name?

I assume you'd also want to create additional files for testing/building/CI?

Yeah this is the hard part, but also the part where we can shed some of the monorepo weight.. We should not impose the whole monorepo building & testing setup, on these hopefully leaner repositories.

Does that sound right? Do all the packages have the 1:1 mapping of folder name to label name?

yeah pretty much!


Add react-native-url-polyfill.

Fixes: #4081
I'll follow this up by creating an issue to fix the many uses of regexes and string concatenation; I think #4081 was mainly for settling on a workable solution.

Thanks!
What's the origin of the libdef -- is that from... ah, I see, there are links in comments.
Those links are to libdefs built into Flow, for the types URLSearchParams and URL. Hmm, it sure would be nice to just refer to those, if that's the effect we want, rather than copy them. But then I'm not sure how to actually spell that, because the names are the same.
Nit in commit message:
Many thanks to charpeni for the changes [2] that allow RN v0.60

I think that's meant to be [3].
Otherwise LGTM. I'll fix that one nit and merge.

What's the origin of the libdef -- is that from... ah, I see, there are links in comments.
Those links are to libdefs built into Flow, for the types URLSearchParams and URL. Hmm, it sure would be nice to just refer to those, if that's the effect we want, rather than copy them. But then I'm not sure how to actually spell that, because the names are the same.

Yeah. The first thing I tried was to use those built-ins, but it didn't work.


new GIF method required?

https://telegram.org/blog/gif-revolution
I had my bot doing a search on Giphy,
then returning the direct url for the GIF,
saving the gif with node FS,
sending the gif using bot.sendDocument( fromID, gif);
That was working until they pushed the update today.

I have the same problem right now

I have the same problem right now

Seem its working with sendDocument but current Telegram Apps can't view the files
The message is not supported on your version of Telegram Web. Update the app to view: web.telegram.org.

Seem its working with sendDocument but current Telegram Apps can't view the files
The message is not supported on your version of Telegram Web. Update the app to view: web.telegram.org.

The post was from 2015 by the way. Maybe a problem? Why does no client work with this?

The post was from 2015 by the way. Maybe a problem? Why does no client work with this?

I didn't get any app update on any platform. Why should they update somethin, if its not working?

I didn't get any app update on any platform. Why should they update somethin, if its not working?

In Telegram Android its coming with 3.4.0
Hahahahah didn't realized they set the year wrong

I can confirm in Telegram 3.4.0 its working

how did you download 3.4.0?
on the playstore i see
"What's new
v. 3.3.2"

Its closed beta


Settings itemTemplate values via ajax timing issue

I have the following itemTemplate, the problem is at the time my call returns the incorrect icon is returned and set in the UI, is there a way to rebind to the itemTemplate?  Any suggestions... I'm not an ajax expert.  The only way this works is using async: false but there is a bad ui side effect.
`itemTemplate: function (_, item) {
var isComments;
var yesComments = $("");
var noComments = $("");
                var isComments = $.ajax({
                    //async: false,
                    type: 'GET',
                    contentType: "application/json; charset=utf-8",
                    url: getRequestBaseUrl() + "HasComments?RequestID=" + item.RequestID,
                    dataType: "json"
                }).done(function (data) {
                    if (data) {
                        return yesComments;
                    }
                    else {
                        return noComments;
                    }
                });
                return isComments.responseText === "true" ? yesComments : noComments;

            }`

itemTemplate needs to return HTML markup.

itemTemplate is a function to create cell content. It should return markup as string, DomNode or jQueryElement.
I am returning a jQuery element.

The point is all necessary data to render the grid should be returned by loadData method.
So if you need so data, just return them with all the data on loadData call.
itemTemplate doesn't support deferred rendering. Your example won't work w/o async: false, which is indeed deprecated and provides bad UX.


Have nix-shell work on both stable and unstable nixpkgs

Apparently I incorrectly tested #186 and in fact it breaks running nix-shell on the unstable nixpkgs channel.
I expect this change to work for both the stable and unstable channels.

Running nix-shell --arg coq false --arg c false (what I usually do) gives me the same shell as the one on master. Nix uses the same cache. If this is the expected behavior, then it seems to work on stable.


error test

hello,
when I run the test.py I have this error : (in tflib/inception_score.py)
ValueError: Shape must be rank 2 but is rank 1 for 'MatMul' (op: 'MatMul') with input shapes: [2048], [2048,1008].
can you help me please?

Hi, it seems the input data are not correct. Did you generate the test
results successfully?
On Sat, 31 Oct 2020 at 19:16, Emna B<EMAIL_ADDRESS>wrote:

hello,
when I run the test.py I have this error : (in tflib/inception_score.py)
ValueError: Shape must be rank 2 but is rank 1 for 'MatMul' (op: 'MatMul')
with input shapes: [2048], [2048,1008].
can you help me please?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/charliememory/Disentangled-Person-Image-Generation/issues/12,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPFNDAY6OAHXMKPHI7Y6HTSNRIAJANCNFSM4TGA7JCQ
.


Remove cookies are missing Path and Domain

When a cookie is removed from the Jar, it only records the cookie's name. So when the Set-Cookie header is generated to clear the cookie, the header does not have the Path or Domain parameters, even if the original cookie had them. This is a problem because the new Set-Cookie header only overwrites the previous cookie if the Path and Domain parameters match.

The solution here is to actually find the original cookie in the Jar and store that in the removed_cookies list instead of just storing the cookie name. This would have the side-effect of fixing what I consider to be another bug, which is that empty_jar.remove("foo") produces a Set-Cookie header even though there was no original cookie by that name.

The browser does not pass along any other information besides the name and value of the cookie. As a result, unless cookie is storing cookies, proper deletion is simply impossible with the current API. I believe that the delete method should instead take in a Cookie and specify that it should have all of the original fields set. We can then set the max_age and expires fields properly.

Oh crud, you're right. Or, if we're happy with the current behavior of Max-Age and Expires when removing, we could just have remove() take two more optional parameters for path and domain, e.g.
fn remove<N, P, D>(&self, cookie_name: N, path: Option<P>, domain: Option<D>)
    where N: Into<Cow<'static, str>>,
          P: Into<Cow<'static, str>>,
          D: Into<Cow<'static, str>>

Yeah, we could do that as well.
I'll be revamping CookieJar in the near future, and I'll make sure to fix this along the way.


Introspection query should be updated to include Subscriptions

Pretty sure this is good now -- needed to update to the latest GraphiQL version.

Subscriptions don't seem to work still, in apollo on the default graphiql I am adding:
app.use(
  '/graphiql',
  graphiqlExpress({
    endpointURL: '/graphql',
    subscriptionsEndpoint: 'ws://localhost:8888/subscriptions',
  })
);

Don't see a way to give this app the subscription url?


Fix derivations setting createFindlibDestdir in nix-shell

Motivation for this change
Fix #121991
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

by appending to the preCheck hook maybe?

There is also a preInstall hook. Is there any reason you didn't suggest this one?

Typo, I would suggest preInstall of course.

Here is a new version based on this idea that seems to work as well. I didn't know how to append to the preInstall hook, so I just replaced it.

Result of nixpkgs-review pr 121992 at d0bcd0ea run on aarch64-linux 1

17 packages marked as broken and skipped:

coccinelle
coq_8_5
coq_8_6
cubicle
haxe_3_2
haxe_3_4
libvmi
monotoneViz
obliv-c
opa
...

63 packages skipped due to time constraints:

abella
acgtk
alt-ergo
beluga
coq_8_10
coq_8_12
coq_8_13
coq_8_7
coq_8_8
coq_8_9
...

53 packages built successfully:

coq (coqPackages.coq ,coq_8_11)
coqPackages.Cheerios
coqPackages.CoLoR
coqPackages.ITree
coqPackages.InfSeqExt
coqPackages.QuickChick
coqPackages.StructTact
coqPackages.Verdi
coqPackages.bignums
coqPackages.contribs.zorns-lemma
coqPackages.coq-bits
coqPackages.coq-elpi
coqPackages.coq-ext-lib
coqPackages.coqeal
coqPackages.coqhammer
coqPackages.coqprime
coqPackages.coqtail-math
coqPackages.coquelicot
coqPackages.corn
coqPackages.dpdgraph
coqPackages.equations
coqPackages.flocq
coqPackages.fourcolor
coqPackages.gappalib
coqPackages.hierarchy-builder
coqPackages.interval
coqPackages.iris
coqPackages.math-classes
coqPackages.mathcomp
coqPackages.mathcomp-abel
coqPackages.mathcomp-algebra
coqPackages.mathcomp-analysis
coqPackages.mathcomp-bigenough
coqPackages.mathcomp-character
coqPackages.mathcomp-field
coqPackages.mathcomp-fingroup
coqPackages.mathcomp-finmap
coqPackages.mathcomp-real-closed
coqPackages.mathcomp-solvable
coqPackages.ssreflect (coqPackages.mathcomp-ssreflect)
coqPackages.metalib
coqPackages.multinomials
coqPackages.odd-order
coqPackages.paco
coqPackages.paramcoq
coqPackages.simple-io
coqPackages.stdpp
coqPackages.tlc
dune_2
framac
hevea
opaline
why3

@sternenseemann Does the new version look good to you?

I've thought about it and I think overriding the preInstall hook may gives us trouble. For one I don't really like it because it becomes harder to use preInstall and overriding is probably compromised. More importantly though I think a lot of ocaml packages don't run the preInstall hook due to installPhases` like this:
  installPhase = "ocaml setup.ml -install";

I think the easiest could be to remove createFindlibDestdir altogether and insert the mkdir calls manually everywhere where it is needed. We don't really need to keep this around since the ecosystem seems to be converging on dune anyways. What do you think? I'm somewhat inclined to sacrifice myself and do the chore as well.

I didn't really consider this option because I thought it would be a huge task. Checking now with grep, I see that there are still 116 occurrences. That's not that many compared to the 812 occurrences of buildDunePackage. So if you are up to the task, I'd be happy with the solution you propose.

As @symphorien pointed out, alternatively we could also add a new phase via the setup hook like autoreconfHook does:
https://github.com/NixOS/nixpkgs/blob/1834bc871199fc6583cf270474aacbda9e5127f2/pkgs/build-support/setup-hooks/autoreconf.sh#L1-L7

Given that your PR is ready, and that this feature was becoming less and less useful with the general move toward Dune, I wouldn't bother doing differently now.

@vbgl what do you think?

I acknowledge that there is a problem that should be fixed.
This PR has a QnD fix that looks OK but causes a massive rebuild so we have to double-check before merging…
PR 124504 seems much better (a lot of welcome cleaning, a thorough understanding of how to fix the issue) but the change is large thus tedious to review.

Something in a similar spirit was incidentally implemented in #133008: a3ab43d3a45338f845a0f8a12ed34e84c8b79dae.


How to load custom-trained model and inference?

Thank you for releasing the training code. I have train a model and I am wondering how to load it on the inference stage.
Could you please give me some example scripts or advice?
The checkpoints strcture:
exps/..../000100/
custom_checkpoint_0.pkl  model.safetensors  optimizer.bin  random_states_0.pkl

Hi there,
Thanks for your interest!
Before running the inference script, you should first convert the training checkpoint to huggingface-compatible models by running python scripts/convert_hf.py --config <YOUR_EXACT_TRAINING_CONFIG>". The converted checkpoint will be saved under exps/releases`.
Plz feel free to comment if there is still any trouble doing inference.

Proxy causes error. Wondering how to solve...
Traceback (most recent call last):
  File "OpenLRM/scripts/convert_hf.py", line 80, in <module>
    loaded_step = auto_load_model(cfg, hf_model)
  File "OpenLRM/openlrm/utils/proxy.py", line 38, in wrapper
    os.environ['HTTP_PROXY'] = HTTP_PROXY
  File "/usr/local/lib/python3.10/os.py", line 685, in __setitem__
    value = self.encodevalue(value)
  File "/usr/local/lib/python3.10/os.py", line 757, in encode
    raise TypeError("str expected, not %s" % type(value).__name__)
TypeError: str expected, not NoneType

Hi,
Plz comment out this line as a workaround here.
I'll try fix this problem later by detecting special environ vars and avoid calling no_proxy by default.

Hi,
Fixed in this commit here.
https://github.com/3DTopia/OpenLRM/blob/c2260e0f2a2f16c86f20c3f844d391692f2ae6ea/openlrm/utils/proxy.py#L18
It should work now without manually commenting out no_proxy.

Thank you for releasing the training code. I have train a model and I am wondering how to load it on the inference stage. Could you please give me some example scripts or advice?
The checkpoints strcture: exps/..../000100/ custom_checkpoint_0.pkl model.safetensors optimizer.bin random_states_0.pkl

where did you configure the dataset path? what was the folder structure of the dataset?

Hi, it would be appreciated you can tell me if this is correct after python scripts/convert_hf.py --config /workspace/OpenLRM/configs/train-sample.yaml convert.global_step= 001140:


[x] Need vite electron plugin support
[x] Need ESLintPlugin
[x] Need BundleAnalyzerPlugin support ,use Rollup Plugin Visualizer
[x] Need process.env support
[x]  Need support fs-extra
[x]   Need support extract-zi

Support BundleAnalyzerPlugin there is no rush

what is vite electron plugin?

what is vite electron plugin?

vite not support electron-render target by default, we should conve transform electron.

okey

vite not support electron-render target by default

That's true, but it's maybe not a big problem -- see my experimental repo.

Anyother task I missing?

Anyother task I missing?

none

vite not support electron-render target by default

That's true, but it's maybe not a big problem -- see my experimental repo.
The thing I didn't test is electron-builder -- I don't know if it requires some special code format like not using es modules. Having tracked this here: electron/electron#21457.

I find electron use require to get some file,the problem is more partial,If we use vite ,the building for development env coast time may not less than webpack

If we only want fast bundling time, then we may choose esbuild.
Tree-shaking is even not a significant requirement, since in electron we always bundle all resources.

If we only want fast bundling time, then we may choose esbuild. Tree-shaking is even not a significant requirement, since in electron we always bundle all resources.
In fact, we are using vite because it brings really good dev experience.

As we all know, vite is esm bunlder,we have three way to reslove the commonjs model (electron,nodeBuildInModel,events), first is write commonjs in code . It's easy but ugly.
second we can write vite plugin to transform  the import to require,the last way is we can transform electron node to esm import.
And the last way is my choose, any comments? @hyrious @BlackHole1

we can transform electron node to esm import

What's that mean? Can you provide some code examples?

@hyrious
https://github.com/Cyberhan123/flat/blob/feature/render-vite/desktop/renderer-app/scripts/vite-plugin-electron.ts#L27
It's not enough，It still need to reslove events which in https://github.com/netless-io/flat/blob/main/desktop/renderer-app/src/api-middleware/smart-player.ts#L9

Wow, that sounds promising. I'm upvoting that.
Currently only the electron module needs such hack. We can replace events with eventemitter3 or emittery or even the native EventTarget, that's not a problem.

Wow, that sounds promising. I'm upvoting that.
Currently only the electron module needs such hack. We can replace events with eventemitter3 or emittery or even the native EventTarget, that's not a problem.

okkk

fs-extra is an tricky question，I have no idea to solve this problem. Can you guys give some inspiration？
It at: https://github.com/Cyberhan123/flat/blob/feature/render-vite/desktop/renderer-app/src/utils/courseware-preloader.ts

You can treat it as electron, this is intended to write some node.js related logic at renderer side. In other words, we are violating the context isolation rule of electron. Let me tell you why:

The main reason of using electron, like other guys, is seemingly an easy way to write cross-platform desktop applications. Desktop applications can access the fs or do many other native things. Then why do we have to setup additional IPC to let another process doing some work? That should be a waste of code complexity.

The doc actually says: prevent the website from accessing Electron internals. You can see the purpose: some people are using electron to wrap websites into it. However, our app has different approaches in implementing native and web.

Another main reason for using node modules at renderer side, is agora-electron-sdk currently work in this way.

fs-extra not same as electron .It's not build in model and if we exclude it vite will not handle the import.

Then we may ask @BlackHole1 .

This issue is representative.

As of electron@12, electron no longer recommends the use of nodejs(node / node library) and most electron api's in the renderer process.
So we now have a new task, which is to remove all the node(modules) api and some of the electron api from renderer-app
We can use IPC or contextBridge instead.
cc: @hyrious @Cyberhan123

This issue is representative.

As of electron@12, electron no longer recommends the use of nodejs(node / node library) and most electron api's in the renderer process.
So we now have a new task, which is to remove all the node(modules) api and some of the electron api from renderer-app
We can use IPC or contextBridge instead.
cc: @hyrious @Cyberhan123

I can write an new vite plugin to support fs-extra, and next step we can  remove all the node(modules) api and some of the electron api from renderer-app, It may cost less

good news render app has change to vite  https://github.com/netless-io/flat/pull/1265, but I suggest to keep webpack, we may need reveal all the details


Change "magazines" section to "newsletters" and update resources at library page

Renamed "magazines" title because using "newsletters" title is more appropriate.
Didn't write a description for resources, if needed feel free to make suggestion.
Didn't include revuo-xmr.com because @erciccione pointed some issues at #1976

and it will close #1976

If the moneromoons ccs proposal is not merged to funding , he will not produce the newsletter anymore , something to consider (and is currently away for the next 4 weeks)

Opinions about my comment above?

@HardenedSteel i don't know as i'm not very familiar with any of them, but even just author and structure (published every N) would be enough.

ping @HardenedSteel

we can merge as is OR add/test html scripting with this pull request ive made here https://github.com/HardenedSteel/monero-site/pull/2

I'd merge as it is, since this is already reviewed and i would open a dedicated pr after if you want to propose that patch. That said, personally i don't see the need to increase complexity and add a new file. It's only two entries, but even if it was more, the current structure could still be used without problems.


Crashing when placing turtle

This issue occurs when only Valkyrien Skies and addons are installed and no other mods

[X] I have tested this issue and it occurs when no other mods are installed

Minecraft Version
1.20.1
Mod Loader
Fabric
Issue description
Placing a mining turtle crashes the game.
Issue reproduction
Installed mods:

cc-tweaked-1.20.1-fabric-1.113.1
eureka-1201-1.5.1-beta.3
fabric-api-0.92.2+1.20.1
valkyrienskies-120-2.3.0-beta.5
vs_addition-1.20.1-0.0.8+d92fb8651c

Logs
This seems to be the only relevant part:
[20:28:13] [Render thread/ERROR]: Mixin apply for mod vs_addition failed fabric-vs_addition-common.mixins.json:computercraft.MixinTurtleBrain from mod vs_addition -> dan200.computercraft.shared.turtle.core.TurtleBrain: org.spongepowered.asm.mixin.injection.throwables.InvalidInjectionException Critical injection failure: @ModifyVariable annotation on ValkyrienSkies2$teleportToBlockPos could not find any targets matching 'teleportTo(Lnet/minecraft/world/level/Level;Lnet/minecraft/core/BlockPos;)Z' in dan200/computercraft/shared/turtle/core/TurtleBrain. Using refmap fabric-vs_addition-common-refmap.json [INJECT_PREPARE Applicator Phase -> fabric-vs_addition-common.mixins.json:computercraft.MixinTurtleBrain from mod vs_addition -> Prepare Injections -> localvar$bek000$vs_addition$ValkyrienSkies2$teleportToBlockPos(Lnet/minecraft/class_2338;)Lnet/minecraft/class_2338; -> Parse ->  -> Validate Targets]
org.spongepowered.asm.mixin.injection.throwables.InvalidInjectionException: Critical injection failure: @ModifyVariable annotation on ValkyrienSkies2$teleportToBlockPos could not find any targets matching 'teleportTo(Lnet/minecraft/world/level/Level;Lnet/minecraft/core/BlockPos;)Z' in dan200/computercraft/shared/turtle/core/TurtleBrain. Using refmap fabric-vs_addition-common-refmap.json [INJECT_PREPARE Applicator Phase -> fabric-vs_addition-common.mixins.json:computercraft.MixinTurtleBrain from mod vs_addition -> Prepare Injections -> localvar$bek000$vs_addition$ValkyrienSkies2$teleportToBlockPos(Lnet/minecraft/class_2338;)Lnet/minecraft/class_2338; -> Parse ->  -> Validate Targets]

Will be fixed in the next version.


contrib code is bitrotting

Code under contrib/ is not tested nor even rebuilt regularly.   As a result, things like golang 1.4 breaking static builds and API changes cause it not to work.
As of now, nothing using CGO_ENABLED=0 for static builds works and contrib/for-tests/network-tester does not compile.

FWIW, I plan to delete all of my contrib scripts O(soon) -- maybe today.

There are a lot of things that are built and pushed as docker images one
time, and then left to bit-rot.  Maybe we can extend the build scripts to
run through them and at least ensure they build?
On Wed, Jan 21, 2015 at 7:17 AM, Brian Grant<EMAIL_ADDRESS>wrote:

FWIW, I plan to delete all of my contrib scripts O(soon) -- maybe today.
--
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/3665#issuecomment-70855095
.

+1. We shouldn't have code in the codebase that isn't built, or it will
just bitrot.
On Wed, Jan 21, 2015 at 2:03 PM, Tim Hockin<EMAIL_ADDRESS>wrote:

There are a lot of things that are built and pushed as docker images one
time, and then left to bit-rot. Maybe we can extend the build scripts to
run through them and at least ensure they build?
On Wed, Jan 21, 2015 at 7:17 AM, Brian Grant<EMAIL_ADDRESS>wrote:

FWIW, I plan to delete all of my contrib scripts O(soon) -- maybe today.
--
Reply to this email directly or view it on GitHub
<
https://github.com/GoogleCloudPlatform/kubernetes/issues/3665#issuecomment-70855095

.

—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/3665#issuecomment-70930851
.

Sadly 'go test' is insular and won't run other kinds of tests, so is a poor substitute for a build system unless you are entirely monoglot, which we are not.  We could make our test scripts find *_test.sh and run those, too and then just add a build_test.sh to each dir which just calls 'make'.


remove Binding from file names

Nothing big. Last couple instances of binding in file names. Was looking at something else and was looking for a serviceinstancecredential file and could not find it before I remember renamed from binding.

Let'/ hold off on this until we can discuss as a group tomorrow
On Mon, Sep 25, 2017 at 3:48 PM Morgan Bauer<EMAIL_ADDRESS>wrote:

Nothing big. Last couple instances of binding in file names. Was looking
at something else and was looking for a serviceinstancecredential file
and could not find it before I remember renamed from binding.
You can view, comment on, or merge this pull request online at:
https://github.com/kubernetes-incubator/service-catalog/pull/1268
Commit Summary

rename binding to serviceinstancecredential
binding -> serviceinstancecredential in storage
binding -> serviceinstancecredential in controller file name

File Changes

R pkg/apis/servicecatalog/validation/serviceinstancecredential.go
https://github.com/kubernetes-incubator/service-catalog/pull/1268/files#diff-0
(0)
R
pkg/apis/servicecatalog/validation/serviceinstancecredential_test.go
https://github.com/kubernetes-incubator/service-catalog/pull/1268/files#diff-1
(0)
R pkg/controller/controller_serviceinstancecredential.go
https://github.com/kubernetes-incubator/service-catalog/pull/1268/files#diff-2
(0)
R pkg/controller/controller_serviceinstancecredential_test.go
https://github.com/kubernetes-incubator/service-catalog/pull/1268/files#diff-3
(0)
M pkg/registry/servicecatalog/rest/storage_servicecatalog.go
https://github.com/kubernetes-incubator/service-catalog/pull/1268/files#diff-4
(24)
R pkg/registry/servicecatalog/serviceinstancecredential/storage.go
https://github.com/kubernetes-incubator/service-catalog/pull/1268/files#diff-5
(2)
R
pkg/registry/servicecatalog/serviceinstancecredential/storage_test.go
https://github.com/kubernetes-incubator/service-catalog/pull/1268/files#diff-6
(2)
R pkg/registry/servicecatalog/serviceinstancecredential/strategy.go
https://github.com/kubernetes-incubator/service-catalog/pull/1268/files#diff-7
(2)
R
pkg/registry/servicecatalog/serviceinstancecredential/strategy_test.go
https://github.com/kubernetes-incubator/service-catalog/pull/1268/files#diff-8
(2)

Patch Links:

https://github.com/kubernetes-incubator/service-catalog/pull/1268.patch
https://github.com/kubernetes-incubator/service-catalog/pull/1268.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-incubator/service-catalog/pull/1268, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAWXmJNcplJoOvoKWzIay9QN4HNLVEh0ks5smAOlgaJpZM4PjP0y
.

what's the concern?


serve previous outputs

There is no way to see an image again once it has been sent to the client, although they are saved to disk for later reference.
Add an endpoint to serve the output of previous requests, using a unique identifier created along with the image.
The current txt2img endpoint will probably need to change to return a JSON structure containing the image ID.

Added as /output/:path.


Fix position of webviews and notebooks when the editor grid scrolls

Fixes #116946
This PR makes sure that absolutely positioned webviews and notebooks are correctly moved when the editor widget is scrolled
Background
Changing the parent of a webview (iframe) destroys the iframe content. To work around this, we add the  webview to the workbench root and then absolutely position them in the correct area of the workbench
Currently we rely on calls to layout for this positioning. However layout is not called when the editor area itself is scrolled
Fix
This fix adds listeners for the editor grid's scroll events. When a scroll happens, we reposition the webview/notebook
I've also added clipping rects to the notebooks/webviews. This makes sure the elements do not overflow outside of the editor area. The clipping rect fixes a bug where scrolling would cause the webview/notebook to draw over the sidebar

Merging this to get testing in insiders
I looked at perf here too and don't think we need to throttle onScroll here. The calls to getClientBoundingRect do trigger a restyle but they also all fairly quick. Throttling could also make the webview more stuttery as they are being repositioned


Create index page for application

The index page should give a short description of the provided features:

Preingest Workflow
Search & Retrieve
Semantic Enhancement

Index page is there for a while...


Fixed metadata embed ressource on linux.

Fixes the webeditor.

Need to merge it to test if it works :|


Fix scroll restoration when server props are changed

Description
Fixes #1141
This adds an additional state to only fire the scroll restoration logic when a location has changed, not when pending has changed.

Before submitting the PR, please make sure you do the following:

[x] Read the Contributing Guidelines
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123)
[x] Update docs in this repository according to your change
[x] Run yarn changeset add if this PR cause a version bump based on Keep a Changelog and adheres to Semantic Versioning

Will this trigger the "scroll to top" behaviour when the pathname does not change, but search does? I ask, because in our app, we often use the search to keep track of filters (form inputs) on a page, and it would break the UX if it scrolled to the top whenever a user updates one of the filters. However, on a different page, we use the search for pagination - at which point we do want it to scroll to top. However, we are able to handle this later case in our code with a useEffect and window.scrollTo(0, 0). So my thinking is that for our particular usecase(s), we either need some sort of escape hatch to say: "please don't scroll the top when I change these only the search" OR don't scroll to the top when the only the search changes and we can handle manually scrolling to the top when our intended UX calls for it.

Update: I realized that you can actually accomplish this without us needing a new API:

Manually call setServerProps and use an alternate path, like _search or _filter
Call window.history.pushState() in your client component to update the URL. This does not affect scroll position.
In your server component, look for both params (real search and alternate path). This allows you to present the correct filter values for both initial page loads and realtime RSC updates.


feat(*): Fail the build unless the .template and .yml file are in sync

It's extremely common for people to update spinnaker-dependencies.template but forget to update spinnaker-dependencies.yml (or vice versa). The build task currently updates the .yml to match the .template, but having a 'build' task modify the source code means that CI is testing changes that aren't (and won't be committed).
Add a new 'checkDependencies' task that fails if the .template and .yml are out of sync; make the check task depend on this task to quickly detect when these get out of sync.

I've been meaning to do this for a while, as I seem to always forget to update both files when I'm bumping dependencies.

LGTM


About Observation Centric update paradigm

Hi Noah, I'm wondering if the paradigm of observation centric re-update with online-smoothing can be applied to other domains, such as KF tracking in the BEV plane? For the context, I'm using lidar detections to detect objects but the motion model sucks, so I hope with observation centric update it can smoothen things out?

From a high-level perspective, my guess is - Yes. As whatever the representation is the concept behind observation centric re-update with online-smoothing is the same. I am not sure what stucks means here, can you look into the KF parameters to ensure that they evolve as expected?

Hi Noah, what I mean is the motion model I'm using is a constant velocity one to model nonlinear motion of persons. Pardon me to bring this question up: what is the difference between calling an KF update with something like kf.update(None,None) and just doing nothing, i.e. doing no updates to the KF? also do you mean to eye on the evolution of KF parameters to see it makes sense?

You can refer to the implementation of kf.update here: https://github.com/noahcao/OC_SORT/blob/7a390a5f35dbbb45df6cd584588c216aea527248/trackers/ocsort_tracker/kalmanfilter.py#L465
Simply speaking, by calling kf.update(None,None), you will refresh the observed status of the KF and also the related variables maintained.
You can check the KF parameters maintained to see if the variance (vectorized) of the velocity is smoothened or not.


Remaining 2023 session's by Abhishek Sharma added

Added session's details -

Excel in Google Cloud Career Fair 2023 - Virtual Event in collaboration with GDSC IES College of Technology, GDSC Central University of Haryana GDSC JC Bose & Guys in the Cloud community
Going serverless with Google Cloud - GDG Devfest Raipur(In Person)

Thank you.


Software Catalog won't work with Azure DevOps Server 2020

PR https://github.com/backstage/backstage/pull/11681
@awanlin
The Software Catalog feature should work with Azure DevOps Server 2020 using the url for exemplo (http://vsm.homolog.jm:1111/tfs/projects/)
Expected Behavior
Software catalog is created
Actual Behavior
When you attempt to configure the Software Catalog, the error is show in console 'Azure URL must point to a git repository'
Steps to Reproduce

Install the backstage by @npx

Configure the integrations in catalog-info.yaml
....
backend:
reading:
allow:

host: example.com
host: '*.mozilla.org'
host: '*.homolog.jm:1111'
.....
integrations:
azure:
host: 'vms.homolog.jm:1111'
token: ${AZURE_TOKEN}

Open backstage in browser

Click in "APIs" on menu

Click in "Register Existing API"
4.1 Enter with catalog-info.yaml url

Context
PR https://github.com/backstage/backstage/pull/11681
For solutions we add one new else if in "AzureUrl.ts"
path: packages/integrations/src/azure/
const parts = url.pathname.split('/').map(part => decodeURIComponent(part));
    if (parts[2] === '_git') {
      owner = parts[1];
      project = repo = parts[3];
    } else if (parts[3] === '_git') {
      owner = parts[1];
      project = parts[2];
      repo = parts[4];
   }#  else if (parts[4] === '_git') {
      owner = `${parts[1]}/${parts[2]}`;
      project = parts[3];
      repo = parts[5];
    }#

Your Environment
Browser Information:  Google Chrome 101.0.4951.67 (Versão oficial) 64 bits
Output of yarn backstage-cli info:
OS:   Windows_10 x64
node: v16.14.2
yarn: 1.22.18

For the test i used links (http://vsm.homolog.jm:1111/tfs/projects/) e (https://dev.azure.com/xxxx/xxx)

To be clear the Catalog does work with Azure DevOps Sever 2020, we are using it without any issues. BUT if you come from an older TFS instances that has been upgraded over time then you may have it setup under the /tfs subpath in the URL which is what is the issue here. I'm not sure how common this is anymore but I know for sure it is a setup that is out there and I'm surprised we have not ran into this before.
@jpitapeva  thanks for logging this and submitting the PR, I'll do a review sometime today

Thanks @awanlin


Add missing testRequires(t, Network)

Fixes #12552
Signed-off-by: Sergey Evstifeev<EMAIL_ADDRESS>

hey why did you close, can you please reopen

@jfrazelle I accidentally added wrong issue id in the commit message (see https://github.com/sergeyevstifeev/docker/commit/dd1f7583fcb2a4e3bcd90c4975a5eb87bfa96f53) - #12549 instead of #12552. Created https://github.com/docker/docker/pull/12561 instead.


Use provision from pull request branch

docker-entrypoint.sh used drush dl to download drush, but I don't think that supports feature branches.
We'd need this for https://github.com/aegir-project/provision/pull/2

It only runs if provision is not found.
For the tests, provision is mounted in place before the entrypoint script runs, so the submitted code is what gets tested.
I have to do one last round of testing to ensure pull requests from forked repos test properly, but this is already resolved.


Use SVG icons instead of Font icons

Currently, a mixture of e. g. <Icon>delete</Icon> (Font icon) and <Delete /> (SVG icon) is used. Only SVG icons should be used.

Should I fix that in the vite branch?


Add parameter return-license in build job

https://game.ci/docs/github/returning-a-license
Since Unity only allows returning professional licenses, I added a parameter to switch whether returning licenses or not for other types of licenses.

Thanks, @tadashi0713 🙇
Weird that shellcheck is complaining since you didn't touch any script. Nevertheless, is a valid error that it caught, so I'll go ahead and open a PR with the fix and then we can merge yours.

Speaking as a free Unity user, it'd be kinda nice if the default just worked. Can we always have this orb try to return the license, and fail gracefully and continue if it can't? Or is that a recipe for problems.

Speaking as a free Unity user, it'd be kinda nice if the default just worked. Can we always have this orb try to return the license, and fail gracefully and continue if it can't? Or is that a recipe for problems.

I don't believe it's a problem in most cases. However, It would make your job run time unnecessarily longer. The doc linked by @tadashi0713 says:

Manually returning a license is usually never necessary, unless when running into an unrecoverable error while having a license active.
Also, Unity only allows returning professional licenses.

Meaning that even for Pro and Plus, it's not always necessary to return the license. Perhaps the best approach is to set the new parameter to false as the default. What do you all think @nanodeath and @tadashi0713?

Also, @tadashi0713, can I ask you to include the new parameter in the test job? We will need it there, too, since it also uses the return-license command.

Pushed the change with my suggestions and merged the shellcheck fix, let me know what you all think.

Looks good to me.

@EricRibeiro Thank you for your suggestion/fix, also looks good to me 👍

Looks like this resolves #26


Check if an entry with the same name already exists

Check if an entry with the same name already exists

Thanks for looking into this issue.
There's an ongoing PR handling the same case. https://github.com/sugarforever/chat-ollama/pull/41
We can use that one to address the issue.
Btw, it's better to use count instead of findUnique in this case.

#57


"allowMultipleLines": true

This seems to require "linesForEditing" not equal 0. Sort of obvious but would be nice if stated and / or error messages from graph stated this.

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

ID: 4f7741ee-6dae-6f54-a7b1-3108eaed4343
Version Independent ID: 5be79cf1-f943-3add-b505-59bac868e15e
Content: TextColumn - Microsoft Graph v1.0
Content Source: api-reference/v1.0/resources/textcolumn.md
Product: ****
Technology: microsoft-graph
GitHub Login: @JeremyKelley
Microsoft Alias: MSGraphDocsVteam

As part of a repository clean up effort we are closing older issues. If this issue remains, please: open a new issue, reference this issue, and provide any additional details that may help in resolution. Thank you for your understanding as we work to improve our responsiveness.


Markdown preview not working

VSCode Version: 1.5.1
OS Version: Ubuntu 16.04 64-bit

Every time I try to use the preview for Markdown in Code on Ubuntu 16.04, I get command 'markdown.showPreviewToSide' not found or command 'markdown.showPreview' not found
Steps to Reproduce:

Open Markdown document
Press ctrl+shift+v or ctrl+k, v to preview formatted markdown

I see the same issue.
OS: macOS 10.12.1
VScode: Version 1.8.0-insider

Same +1

Duplicate https://github.com/Microsoft/vscode/issues/15184


Playlist result Not "tab delimited" in JSON ResultMode

Apparently the playlist queries return the json code on a single line.

I will fix this tomorrow

You can add as many indents as you want:
import json
...
result = search.result()
result_json = json.dumps(result, indent=5)

You can add as many indents as you want:
import json
...
result = search.result()
result_json = json.dumps(result, indent=5)

Yeah, but the problem is, they are passing ResultMode.JSON and not getting JSON back. Instead they get back the dictionary. This is a problem that was introduced I think with my PR, since I rewrote a lot of class. I will fix this today, and afterwards release it to PyPI.
BTW thanks for closing a couple of issues for me

I can't reproduce it.
import youtubesearchpython as yt
search = yt.VideosSearch("say it aint so")

rjson = search.result(yt.ResultMode.json)
print(type(rjson) == str)
# True
print("        " in rjson)
# True

rdict = search.result(yt.ResultMode.dict)
print(type(rdict) == dict)
# True
``

I can't reproduce it.
import youtubesearchpython as yt
search = yt.VideosSearch("say it aint so")

rjson = search.result(yt.ResultMode.json)
print(type(rjson) == str)
# True
print("        " in rjson)
# True

rdict = search.result(yt.ResultMode.dict)
print(type(rdict) == dict)
# True

Playlist class, not VideosSearch class

Ok, this is now fixed.


Organise environments

Description
Closes #457. Looks like the problem was the icu 58* pin, which should no longer be necessary as (a) it was set years ago and the current version is 70 and (b) it is no longer present in the test environment.
Locally autosummary is taking a very long time to complete. I want to see if that happens on RTD before I spend too much energy trouble shooting it.
Todos
Notable points that this PR has either accomplished or will accomplish.

[X] Update dependencies in rtd environment
[x] Make sure docs still build on RTD

Status

[x] Ready to go

OK I've updated the environment to match test_env.yaml and fixed the associated issues, which seem to just be to do with mocking dependencies that actually exist.

The PR build looks fine; I'll make sure pymbar stuff is working fine by the next release.
Thanks for doing the hard work here @Yoshanuikabundi - removing icu is definitely not among the first things I was going to try given that all-caps comment.


Bug Report: Unable to Extract Uploader ID

Bug Report: Unable to Extract Uploader ID
Description:
I encountered an issue while using youtube-dl where it was unable to extract the uploader ID. This occurred when attempting to download a video using the following command:
youtube-dl --verbose "https://www.youtube.com/watch?v=kuCsICnLS-0&ab_channel=C4ETechEnglish"
Steps to Reproduce:
Run the command youtube-dl --verbose "https://www.youtube.com/watch?v=kuCsICnLS-0&ab_channel=C4ETechEnglish" in a command-line interface.
Observe the error message: ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see https://yt-dl.org/update on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
The verbose output is also attached to this bug report.
Expected Behavior:
I expected youtube-dl to extract the uploader ID successfully and initiate the video download.
Observed Behavior:
youtube-dl failed to extract the uploader ID and returned an error message.
Environment:
Operating System: [Specify your operating system]
youtube-dl version: [Specify the version you are using]
Python version: [Specify your Python version]
Additional Information:
The issue seems to be related to the uploader ID extraction process.
I have followed the troubleshooting steps mentioned in the error message but the issue persists.
I have also ensured that I'm using the latest version of youtube-dl by following the update instructions provided on the youtube-dl website.
Verbose Output:
[Attach the complete verbose output of the youtube-dl command here]
Please let me know if there's any additional information or details you'd like to include in the bug report.

Quite unnecessary. Instead, read #30839.


🛠 [TASK] : Buildkit needs updating in CI

Summary
Buildkit version in CI not compatible with Earthly
Description
We get this reported now in CI:
buildkitd | Warning: Buildkit version (v0.7.6) is not compatible with Earthly version (v0.8.4)

Need to update buildkit.
I raised this to urgent because we have started getting errors in CI which may be caused by the old buildkit version on the satellite.
/h/r/w/c/c/e/rust+rust-base | *cached* --> COPY --dir /home/runner/work/catalyst-ci/catalyst-ci/earthly/rust+python-scripts/python /scripts/python
/h/r/w/c/c/e/rust+rust-base | *cached* --> COPY --dir stdcfgs /stdcfgs
Error: async force execution for /home/runner/work/catalyst-ci/catalyst-ci/examples/wasm/c+builder: unlazy force execution: failed to get edge: inconsistent graph state
🛰 Reuse cache between CI runs with Earthly Satellites! 2-20X faster than without cache. Generous free tier https://cloud.earthly.dev/
/home/runner/work/_actions/input-output-hk/catalyst-ci/master/actions/run/dist/index.js:2969
                reject(new Error(`child process exited with code: ${code}`));
                       ^

Error: child process exited with code: 1
    at ChildProcess.<anonymous> (/home/runner/work/_actions/input-output-hk/catalyst-ci/master/actions/run/dist/index.js:2969:24)
    at ChildProcess.emit (node:events:514:28)
    at maybeClose (node:internal/child_process:1105:16)
    at ChildProcess._handle.onexit (node:internal/child_process:305:5)

See discussion on this kind of issue here:
https://github.com/earthly/earthly/issues/2454

Thank you Chuck for fixing this.
:+1:


Add Darkbrewery editor theme

Your idea:
Now that the basic CodeMirror theme functionality is in place, we can extend that functionality to add our own themes.
I suggest that we start with the Darkbrewery theme, which has been popular on the subreddit.
DarkBrewery styling is as follows:
@-moz-document domain("homebrewery.naturalcrit.com") {
body {
    ;
}

/* Main BG color and normal text color */
 .CodeMirror {
    background: #293134;
    color: #91A6AA;
}

/* Brew BG */
 .brewRenderer {
    background-color: #29313;
}
/* Blinking cursor */
 .CodeMirror-cursor {
    border-left: 1px solid #e0e2e4;
}
/* Bars at the top */
 .snippetBar {
    background-color: #2F393C;
    color: white;
}
nav {
    background-color: #293134;
}
nav .navItem {
    background-color: #293134;
}
.snippetBar .snippetGroup .dropdown {
    background-color: #2F393C;
}
/* Line number stuff */
 .CodeMirror-gutter-elt {
    color: #81969A;
}
.CodeMirror-linenumber {
    background-color: #293134;
}
.CodeMirror-gutter {
    background-color: #293134;
}
/* column splits */
 .editor .codeEditor .columnSplit {
    font-style: italic;
    color: inherit;
    background-color:#1f5763;
    border-bottom: #299 solid 1px;
}
/* Colors for headings and such */
 .cm-s-default .cm-header {
    color: #c51b1b;
    -webkit-text-stroke-width: 0.1px;
  	-webkit-text-stroke-color: #000;
    /* ###Headings */
}
.cm-header, .cm-strong {
    font-weight: bold;
    color: #309dd2;
    /* bold points */
}
.cm-s-default .cm-link {
    color: #dd6300;
    /* Link headings */
}
.cm-s-default .cm-string {
    color: #aa8261;
    /* links */
}
.cm-s-default .cm-def {
color:#2986cc;
}           /*@import*/
.cm-s-default .cm-variable-2 {
    color: #3cbf30;
    /* Bullets and such */
}
.editor .codeEditor .block {
    color: #e3e3e3;
    /* blocks */
}
.editor .codeEditor .inline-block {
    color: #e3e3e3;
    /* inline blocks */
}
.cm-s-default .cm-tag {
    color: #e3ff00;
    /* Tags (divs) */
}
.cm-s-default .cm-attribute {
    color: #e3ff00;
}
.cm-s-default .cm-atom {
    color:#000;
}
.cm-s-default .cm-qualifier{
    color:#ee1919;
}
.cm-s-default .cm-comment{
    color:#bbc700;
}
.cm-s-default .cm-keyword {
	color:#c302df;
    background-color:#b1b1b1;
}
.cm-s-default .cm-property.cm-error {
	color:#c50202;
}

.CodeMirror-foldmarker {
	color:#f0ff00;
}

.editor .codeEditor .pageLine {
    /* New page  */
  background: #000;
    color:#000;
    border-bottom: 1px solid #fff;
}

.cm-s-default .cm-builtin {
	color:#fff;
}

}

I haven't updated that theme since Dec 2021, it may need some changes, if you need anything just ask.
Aditional info, the @moz-document rule doesnt' work without extra help, in this specific case, stylish uses some script to support it in chrome. It wouldn't be needed anyway.

My thinking was that we could offer users a native option now, rather than needing to install an extension for the custom editor CSS. We could expand on this to include multiple DarkBrewery themes, e.g. DarkBrewery v1 for the original extension CSS, DarkBrewery v3.0.1 for the current version, and DarkBrewery vX.XX for any future revisions.

The color for the BrewRenderer has a typo:
/* Brew BG */
 .brewRenderer {
    background-color: #29313;
}

From some of the adjacent lines, I assume that the #29313 should be #293134.

We could expand on this to include multiple DarkBrewery themes, e.g. DarkBrewery v1 for the original extension CSS, DarkBrewery v3.0.1 for the current version, and DarkBrewery vX.XX for any future revisions.

The reason 3.0.1 was made is because V3 was new, and the old theme did not change those styles, the old theme is no different from the new one, just outdated. I don't think we want variations, just updates.
But yes, the ability to release new themes by ourselves would be great.

Our own icons for snippets do not work like the font-awesome ones, because of that, do not change color with darkbrewery, this should be an easy fix for that:
.snippetBar .fac {
    filter: invert(1);
}

I've applied this fix now - this is the only theme that also targets the Homebrewery nav bars; we may wish to replicate that styling on all dark editor themes.


Add Server Type forge

new variable FORGE_VERSION and default value
new server installation script
enhanced README.md (also with nicer table in the editor using Atom Plugin -> use soft wrap for it)

Looks good! I think I would add a "Forge" chapter in the readme with the info that the forge version has to mach the minecraft version and that currently forge is not compatible with Java 15. Can you do that?

Readme should be fine now. Please review.

Looks good. Merged and released on dockerhub with the tag "3"