[BUG] Coins are stuck on the blockchain

Hello everybody,
it's about this topic:
https://www.reddit.com/r/defiblockchain/comments/ky4qz2/bug_coins_are_stuck_on_the_blockchain/
I've tried the reindex option before. Additionally on another PC. The 64 DFI are no longer displayed.
Can someone help me there? If log files are required, I will be happy to provide them.
Version 2.1.10 (But I also tried older versions.)
Thank you!

I have now deducted everything from the address, but the 64 dfi cannot be moved. I can't move it. There are several more with the problem, according to Reddit. Blockchain bug or app bug?
Please check it out.
Address: https://explorer.defichain.io/#/DFI/mainnet/address/dZaotNHvsJhwcUdvsHdg1q4Xfee1cSnCQP
Tansaction: https://explorer.defichain.io/#/DFI/mainnet/tx/7e6efe9cc0bed864e89df737d9b00de5d556a6777a22c3a75ff684f4fb3eeb6e

I have now deducted everything from the address, but the 64 dfi cannot be moved. I can't move it. There are several more with the problem, according to Reddit. Blockchain bug or app bug?
Please check it out.
Address: https://explorer.defichain.io/#/DFI/mainnet/address/dZaotNHvsJhwcUdvsHdg1q4Xfee1cSnCQP
Tansaction: https://explorer.defichain.io/#/DFI/mainnet/tx/7e6efe9cc0bed864e89df737d9b00de5d556a6777a22c3a75ff684f4fb3eeb6e

Hi @gutenmorgen2 , thanks for your feedback. Could you type the following via the CLI:
getbalance — utxo DFI
gettokenbalances — DFI
The sum of utxo DFI & DFI would be your total tokens.
Next, please provide the output of:
listaccounts {} true true true — breakdown of your coin(s) / token(s)
Thank you.

Hi @gutenmorgen2 , thanks for your feedback. Could you type the following via the CLI:
getbalance — utxo DFI
gettokenbalances — DFI
The sum of utxo DFI & DFI would be your total tokens.
Next, please provide the output of:
listaccounts {} true true true — breakdown of your coin(s) / token(s)
Thank you.

Thanks for answer.
just do this transaction: I can only move the 0.1 dfi. The 64 dfi frozen on blockchain.
https://explorer.defichain.io/#/DFI/mainnet/tx/ba76e3b1f54f6887dc97862872f2a848b47dddb24ffdf5b058051309520a9c80
I can no longer reach the 64!
https://explorer.defichain.io/#/DFI/mainnet/tx/7e6efe9cc0bed864e89df737d9b00de5d556a6777a22c3a75ff684f4fb3eeb6e
The 64 are not displayed in the defi app. If I send 5 dfi to the address now, I can also use these 5 dfi, but the 64 dfi are stuck. (See first transaction with the 64 dfi).
Tokens are no longer on the address.
gettokenbalances
[
"1.67388290@0",
"0.00012457@2",
"0.01862148@3",
"7.07296830@5",
"1309.34849962@6"
]
getbalance
0.99932357
And this balance is from this Address:
https://explorer.defichain.io/#/DFI/mainnet/address/dXU7fg5nk2t4hRGd3c6XM86FdPJAXaCUJk
See this transaction from just:
https://explorer.defichain.io/#/DFI/mainnet/tx/ba76e3b1f54f6887dc97862872f2a848b47dddb24ffdf5b058051309520a9c80

Thanks for answer.
just do this transaction: I can only move the 0.1 dfi. The 64 dfi frozen on blockchain.
https://explorer.defichain.io/#/DFI/mainnet/tx/ba76e3b1f54f6887dc97862872f2a848b47dddb24ffdf5b058051309520a9c80
I can no longer reach the 64!
https://explorer.defichain.io/#/DFI/mainnet/tx/7e6efe9cc0bed864e89df737d9b00de5d556a6777a22c3a75ff684f4fb3eeb6e
The 64 are not displayed in the defi app. If I send 5 dfi to the address now, I can also use these 5 dfi, but the 64 dfi are stuck. (See first transaction with the 64 dfi).
Tokens are no longer on the address.
gettokenbalances
[
"1.67388290@0",
"0.00012457@2",
"0.01862148@3",
"7.07296830@5",
"1309.34849962@6"
]
getbalance
0.99932357
And this balance is from this Address:
https://explorer.defichain.io/#/DFI/mainnet/address/dXU7fg5nk2t4hRGd3c6XM86FdPJAXaCUJk
See this transaction from just:
https://explorer.defichain.io/#/DFI/mainnet/tx/ba76e3b1f54f6887dc97862872f2a848b47dddb24ffdf5b058051309520a9c80

Now I will explain it in a little more detail.
So, after the incident I created a new address and all coins / tokens / liquidy mining to this address:
https://explorer.defichain.io/#/DFI/mainnet/address/dXU7fg5nk2t4hRGd3c6XM86FdPJAXaCUJk
I have now prepared a little for clarification.  Look:
https://explorer.defichain.io/#/DFI/mainnet/address/dZaotNHvsJhwcUdvsHdg1q4Xfee1cSnCQP
Blockchain says: 65.34034314 DFI
As a result of the transaction, a small residue came to this address:
https://explorer.defichain.io/#/DFI/mainnet/address/dTVknCTeFrSiSaUVmNQQQwRKSY2WJDDAvk
Blockchain says: 65.34034314 DFI
0.02005788 DFI
Now I type in "getbalance":
getbalance
1.37999963
Address 1 + Address 2 = 65,37999963 DFI Subtract the 64 = 1.37999963
The DFI are NOT recognized. I've also completely reloaded the blockchain. Tried on another PC. I hope you understand better what I mean now. Either there is a bug in the blockchain or the Defi app has a bug.
I hope you will have a look at it soon, as the values are constantly changing (use DeFi liquidity mining productively). I won't move anything today so that you can understand it better.

Now I will explain it in a little more detail.
So, after the incident I created a new address and all coins / tokens / liquidy mining to this address:
https://explorer.defichain.io/#/DFI/mainnet/address/dXU7fg5nk2t4hRGd3c6XM86FdPJAXaCUJk
I have now prepared a little for clarification.  Look:
https://explorer.defichain.io/#/DFI/mainnet/address/dZaotNHvsJhwcUdvsHdg1q4Xfee1cSnCQP
Blockchain says: 65.34034314 DFI
As a result of the transaction, a small residue came to this address:
https://explorer.defichain.io/#/DFI/mainnet/address/dTVknCTeFrSiSaUVmNQQQwRKSY2WJDDAvk
Blockchain says: 65.34034314 DFI
0.02005788 DFI
Now I type in "getbalance":
getbalance
1.37999963
Address 1 + Address 2 = 65,37999963 DFI Subtract the 64 = 1.37999963
The DFI are NOT recognized. I've also completely reloaded the blockchain. Tried on another PC. I hope you understand better what I mean now. Either there is a bug in the blockchain or the Defi app has a bug.
I hope you will have a look at it soon, as the values are constantly changing (use DeFi liquidity mining productively). I won't move anything today so that you can understand it better.

After this transaction (conversion of tokens to coins 2 days ago) the 64 dfi "disappeared".
https://explorer.defichain.io/#/DFI/mainnet/tx/7e6efe9cc0bed864e89df737d9b00de5d556a6777a22c3a75ff684f4fb3eeb6e

After this transaction (conversion of tokens to coins 2 days ago) the 64 dfi "disappeared".
https://explorer.defichain.io/#/DFI/mainnet/tx/7e6efe9cc0bed864e89df737d9b00de5d556a6777a22c3a75ff684f4fb3eeb6e

Solved. I imported an old wallet backup. Then it worked. But that's risky when you think about it. The seed alone is apparently not enough. A wallet backup is essential.

Solved. I imported an old wallet backup. Then it worked. But that's risky when you think about it. The seed alone is apparently not enough. A wallet backup is essential.

Hi @gutenmorgen2 , thanks for your feedback. I'll forward this to the team for improvement. Agreed, however the mnemonic seed & wallet backup is store in your own local PC. Therefore not accessible but external parties unless you send / lose it. Please keep it safe. Thank you.

Hi @gutenmorgen2 , thanks for your feedback. I'll forward this to the team for improvement. Agreed, however the mnemonic seed & wallet backup is store in your own local PC. Therefore not accessible but external parties unless you send / lose it. Please keep it safe. Thank you.


Add runner selector like project/build selector to status bar

Is there a way to add additional runners to the list?
I'm using openocd on a disco_l475_iot1 board, and the only option is default. I'd like to be able to add openocd as an option so I can select it, even if I then have to play with launch.json to set up the configuration file, etc.
Right now I'm selecting default, then going into zephyr-ide.json and modifying runnerConfigs by hand.


error: unexpected keyword argument 'root_name' (nbtlib version mismatch)

C:\Users\akrrk\Documents\Galaxy_Jukebox-main>python main.py 1.nbs 1.schem
Traceback (most recent call last):
File "main.py", line 65, in
convert(argv[1], argv[2])
File "main.py", line 60, in convert
schem.save(out_path)
File "C:\Users\akrrk\Documents\Galaxy_Jukebox-main\sponge_schematic.py", line 75, in save
}, gzipped=True, root_name="schem")
TypeError: init() got an unexpected keyword argument 'root_name'
fix please....

hmmm what version of nbtlib do you have? pip show nbtlib
I just updated ( this commit, a week ago ) the code to work with nbtlib 2.0.4 instead of 1.12.1

ah.... Thank you ^^

I'll close this seeing as it seems to be resolved, but if you have any problems be sure to reopen / open another issue ;)


Refactor: Project Samples Test

What does this PR do and why?
This PR refactors system/projects/samples_test.rb.
Upon revision, there were many aspects of the testing file that needed cleanup and/or fixing.
This is including but not limited to:

Ensuring assertions are actually testing what the test title states
Removing redundancies between tests (eg: no need to check for a header every time we visit a page unless we're testing a redirect). Note: There are cases where tests are flaky, and the resolution is to add additional asserts (such as anything with select2). These cases will have repetitive assertions and should be commented on.
Split larger tests into multiple tests so tests are shorter and more focused.
Use of ID selectors as much as possible

Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other pull requests.
How to set up and validate locally

Verify tests pass locally

PR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

[x] I have evaluated the PR acceptance checklist for this PR.

This is looking good. I left some comments, but in general can you assert for something that will ensure the samples table has fully loaded in your setup before proceeding with actions. This strategy should reduce flakes.

Thanks for the suggestions. I believe I've captured all the instances where we would like an assertion to verify the page state prior to taking any actions/additional assertions.


Proposal: SonikDrop

Description

A seamless cross-chain open-source tool that streamlines airdrop creation, management, and claim for ERC20 tokens and POAPs. The goal is to help communities distribute rewards and POAPs (for events) in a fast, reliable, and cheap means. Deploying this tool on Kaia as the pilot chain.

The current UI sneak peek (https://sonikdrop.vercel.app/)  Demo will be fully ready shortly as we are currently building.
We will be making it a telegram mini app also to reach most TG crypto users.
Bounty track

Surf Szn (Consumer)

interesting. Approved. i think telegram mini app is cursed tho because of very poor EVM support but think of something

Oh, we will figure something out, thanks!


Use our standard ruby-cleanup definition

This looks like it will shrink our install by ~5% on disk by cleaning up
more files. We'll need a real build before standing by that number
though.
Signed-off-by: Tim Smith<EMAIL_ADDRESS>

Testing build in ad-hoc # 168


Fix ManagedResource example CRD

How to categorize this PR?

/area quality
/kind bug
What this PR does / why we need it:
Earlier the generated ManagedResource CRD in the /example dir missed the status subresource, shortNames, as well as the additionalPrinterColumns (see pkg/operation/botanist/component/extensions/crds/templates/crd-managedresources.tpl.yaml for a reference)
Also, example/resource-manager/10-crd-managedresource.yaml in favor of 10-crd-resources.gardener.cloud_managedresources.yaml.
Release note:

The `ManagedResource` CRD in the `/example` dir was fixed.

@timuthy Can you check the failing verify step?


Salination Tower Heating

Please use the search functionality before reporting an issue. Also take a look at the closed issues!
Issue description:
I am Trying to get more heat in the Tower. Using Solar panels and Placed in a Desert. For Some reason i cant conect heat pipes to the Valve / Fuelwood Heater and Resistive Heater both dont seem to work.
im new to the pack and couldent find any fix for this.
Steps to reproduce:

Grab any mecanism heater and try to connect it to a salination tower.

Version (make sure you are on the latest version before reporting):
Forge:
1.12.2
Mekanism:
1.12.2
Other relevant version:
If a (crash)log is relevant for this issue, link it here: (It's almost always relevant)
[gist/pastebin/etc link here]

screenshots please.

![e4cfce4823e1e65e97946b1e8e8a5958](https://user-images.githubusercontent.com/42297017/44313408-8736a880-a408-11e8-9752-f4b4fb3b12ac.png
Keeps switching fast between Having a bit of water and no Fluid.

Remove solar panels from salination plant if you are going to use external heat source. You may have one or the other, but not both. This is a common error...
From your screenshot, your pump may not be pumping fast enough. Upgrades!
After using heat pipes, I find they are really only good for 6-8 blocks before they become pointless. You are better off putting your resistive heater right next to the valve. Resistive heaters MUST be powered to the green square sides. The green square side does not output heat!
Fuelwood heaters must be placed directly on valve. Orientation is of no importance.

Fuelwood heaters must be placed directly on valve

I am 99% sure that is not true.
@Moebius777 if you want help with the heat situation you need to show the heat connections as screenshots. Pump is likely not enough power / not enough upgrades.


[Archery] Use gh cli for artifact management

Describe the enhancement requested
The gh cli has functionality that allows the direct download of artifacts:
$ gh run download --help
Download artifacts generated by a GitHub Actions workflow run.

The contents of each artifact will be extracted under separate directories based on
the artifact name. If only a single artifact is specified, it will be extracted into
the current directory.

USAGE
  gh run download [<run-id>] [flags]

FLAGS
  -D, --dir string            The directory to download artifacts into (default ".")
  -n, --name stringArray      Download artifacts that match any of the given names
  -p, --pattern stringArray   Download artifacts that match a glob pattern

This would make it unnecessary to upload the artifacts as releases to make them easily available outside of the workflow run. (Except nightlies we don't publish anywhere else)
Component(s)
Archery

cc @raulcd

This is tagged as Archery, is there a Python library that offers similar functionality as the gh CLI? Keeping Python-only dependencies would generally make it easier to use Archery.


Is it possible to pre-train for specified amount of styles?

Hi!
My neural networks knowledge is not so strong, so the question might be stupid :)
Is it possible to pre-train network with some amount of style images to speed up it?

Try to find on GitHub repo "chainer-fast-neuralstyle"


Files .wasm

How were the bit files obtained: cycle.wasm and hub.wasm?
Thanks!

We upload the wasm files through the agent manually.


Fixes #220 - Avoid inifinite recursion while getting wizard form list

This change will mask the implementation issues in the custom conditional form logic that have caused the recursion, silently returning form_list instead of whatever the developer intends which would be confusing. It would be better to raise an exception with a useful message when recursion is encountered so the developer knows what they have to fix.
It would be better still to avoid this infinite recursion problem altogether while fixing the behaviour that was broken in #209 🙂

I understand your feeling. Basically, there are two different uses of get_form/get_form_list. One to really get the forms depending on some conditionals, and one just to access a specific form to get data or some other information (typically from some of the conditions). We could have added a kwarg to these methods to be more explicit about those two use cases, but then there are backwards compatibility issues due to method signature changes. I'm clearly not 100% satisfied with that proposed solution, but it's a way to make things going forward. I'll probably merge this soon to prepare a new release, but if you have any ideas about a better fix for this, any contribution is welcome.

I understand it's an awkward situation and you need to move it forward somehow!
I think this change is okay to merge as an improvement on the current situation if it raises an exception instead of returning self.form_list.  If I'm upgrading and something fails I want it to fail loudly so it doesn't slip into production. Or if I'm implementing a new wizard with conditional forms, I want the success/failure of my new code to be clear while I test and debug. Returning self.forms would be unexpected and leave it to me to look into the wizard code and find out why. I think raising an exception would communicate the problem better.
What do you think about also adding a note to the documentation at https://django-formtools.readthedocs.io/en/latest/wizard.html#conditionally-view-skip-specific-steps.  I guess the example provided there will trigger infinite recursion.
Regarding you comment that "Basically, there are two different uses of get_form/get_form_list.", well, there are now but there weren't before 209. Looking at the code prior to 209, the only use for get_form() was to naively instantiate a form from form_list, while the purpose of get_form_list() was to switch those forms on and off but not add new forms. The design limited how dynamic the form list could be (all possible forms needed to be listed in form_list) but at least it was a consistent and cohesive design.
Regarding a better fix, I'm going to spend some time on it today, hopefully I can add something more practical to the discussion. If the two form list generation strategies can't be merged coherently without further breaking changes, then I'm inclined toward a solution that keeps the two strategies separate and simpler, but is a breaking change for implementations using the more dynamic approach given that it is newer and presumably has fewer implementations to update.

In terms of my workflow: the default is form_list, and as such a fallback to that if get_form_list() is indeed the desired behaviour.

Coming back to this after attempting my own fix I've realised I was looking at this wrong way - apologies for the misguided commentary @claudep. I can see that this change allows adding forms in get_form_list(), restores previous behaviour, and is backwards compatible. It would be a valuable improvement.
I note that it doesn't support calling get_cleaned_data_for_step() on a step/form that has been added in get_form_list() during condition evaluation. I realise this is intended to be a small step forward so perhaps this is not the change to achieve such things; perhaps this is not even a bigger picture goal.

I understand your feeling. Basically, there are two different uses of get_form/get_form_list. One to really get the forms depending on some conditionals, and one just to access a specific form to get data or some other information (typically from some of the conditions). We could have added a kwarg to these methods to be more explicit about those two use cases, but then there are backwards compatibility issues due to method signature changes. I'm clearly not 100% satisfied with that proposed solution, but it's a way to make things going forward. I'll probably merge this soon to prepare a new release, but if you have any ideas about a better fix for this, any contribution is welcome.

The time for exercising caution about backwards compatibility was before #209 was merged. I do not understand the continued efforts to double down on the problems in 2.4 with even more hacks to make it "work" when reverting #209 is the cleanest and clearest path forward and could have been done half a year ago. No one would accept PRs like these in Django proper, which is where django-formtools originated.
get_form_list has exactly one documented and tested use case and that is to process condition_dict to prune steps/forms. It is unfortunately named since it does not actually operate on nor return a list (form_list is an OrderedDict at this point and that is what is returned). It should have been called process_conditions(), since that is all it does. Nevertheless, it seems that some have become fixated on this method since that was the only place they could figure out to hack into things in their own projects, but that does not mean this project needs to lower its standards of code quality and make these hacks "official".
Those that need this new feature need to make a serious and thought through proposal to add the appropriate hooks for their use case, in a way that either preserves backwards compatibility (which I don't think will be possible) or at least has a clear deprecation timeline for any internals that need to be refactored along the way.

I will nevertheless go forward with this patch, even if not the optimal solution. This package is blocked with this recursion regression for too long now. We can always improve later, and as usual patches welcome!


OCR0001: Creating Training data from Hand written Chuyik (8)

Description:
We have given two sets of worksheets, portrait mode and landscape mode. The annotators are instructed to write the Tibetan Cursive or Chuyik in the work sheet paper given. Now after reviewing and collecting the reviewed papers we need to do following steps:

Scan all the paper ( annotators work )
Write a script to crop all the reviewer accepted lines from each image
Name the line image as per its page or paper name and line number
and then create a csv for the line_name and its text.
Scaned pages potrait, landscape
transcripts

Completion Criteria:

accepted line images and a csv containing the line image and text

Implementation Workflow:

Work items:

[x] script to split pdf to jpg
[x] check if the page ordering is consistent
[ ] find bounding positions for potrait
[ ] crop the images with positions with mapping
[ ] classifying line image (good or bad / black dot or no black dot)
[ ] map good line image with transcript
[ ] find bounding positions for potrait
[ ] run the pipeline for landscape

For the second work item "check if the page ordering is consistent", i split the pdf P000010_v001_00001 - 00500.pdf into images and 500 images were in the output. So the number count matches, as for the number in each pdf page, i manually scrolled for a minute to check if it ordering is in proper manner. and so far is does.

yeah it sounds like they are in order, they were told to put them in order. you can go ahead and start with it. and as for getting the line images bounding box, maybe you can use transkribus to manually annotate one or two images and then use the bounding box output in xml from the transkribus to use for cropping out the line images. if you want my help with line annotation in the transkribus let me know. trranskribus download. create an account and download the application and open it in your computer. we can meet tomorrow to annotate the lines.

Thank you @ta4tsering , would look into that.

To classifying the line image quality,
Good Image

Bad Image

Black to white ration for 3500 image (Potrait)

Setting threshold of 0.125 such that
>0.125 -> Good image,
<=0.125 ->Bad image,
is greate result

Problem

Reason to the problem
The reason is due to the some difference in gaps in the starting of the scanned pages, ~(8 out of 500 images) so 7*8 = 56 images

Solution
Bigger bounding position for the line image label


Don't manage frontmatter in templates

🚀 The feature, motivation and pitch
When the frontmatter of is managed by obsidian.nvim it is near impossible to edit template files. I usually writte these in markdown as well so obsidian.nvim treats them as notes inserting a frontmatter before saving the file.
I want the plugin to manage the frontmatter in most of my notes, but not in the templates because some of those aren't supposed to have a frontmatter.
I would suggest disabling the frontmatter in templates, by matching the path against the template path in the BufWritePre autocommand.
Alternatives
Another more flexible solution might be allowing the user to optionally specify disable_frontmatter as a function. This can cover more cases where someone might want to opt out of the automatically managed frontmatter, but having access to obsidian.nvim's internal normalization of dir might be beneficial for my use case.
Additional context
The first option could be implemented by adding an early return to the callback in the BufWritePre autocommand or by extending the existing check.
if self.opts.templates ~= nil and vim.fn.matchstr(args.match, '^' .. tostring(self.dir / self.opts.templates.subdir / ".*")) then
  return
end

This could possibly be extended by an additional option in the opts.templates table, to give users the ability to opt out if they want the frontmatter of templates to be managed.

Hey @Puerling, you're absolutely right. I like your first suggestion. Interested in making a PR?


javascript issue in Golang template code

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.64.0 (universal)
OS Version: MACOS 10.15.7 Catalina

This part of the code generates errors: '(' expected.javascript and ')' expected.javascript
  {{with .Error}}
    notify("{{.}}", "error")
  {{end}}
</script>

  {{end}}
In {{with .Error}} the errors occur on the DOT and on the first closing }
The effect is that JS refuses to run.

/extGo


Support for Producer using deepseq

I want to use yield with Deepseq to prevent THUNK leaks.

@junjihashimoto: Sorry for the delayed reply.
I would prefer not add these, mainly because an end user can achieve the same thing using the ($!!) operator from the deepseq package, like this:
yield $!! x

each $!! xs

@Gabriel439
Thank you for letting me know.


Kubeconfig file for the cluster

Is your request related to a new offering from AWS?
No
Is your request related to a problem? Please describe.
A kubeconfig was being generated previously with a template existing in the templates folder but the feature was removed?
Describe the solution you'd like.
A kubeconfig file generated for the cluster post creation
Additional context

you can generate the config using aws eks update-kubeconfig --region <region-code> --name <cluster-name> - see https://docs.aws.amazon.com/eks/latest/userguide/create-kubeconfig.html


video_player minimum Android version support?

After attempting to run the video_player example code on my Moto E4, the debug console outputs a long series of errors, most of which stem from the very first line:
D8: Invoke-customs are only supported starting with Android O (--min-api 26)

Moto E4 only supports officially up to Android 7.x. Does the error above imply all Android phones < O cannot run the video_player plugin? If it can, how do I skirt around the issue above?
FAILURE: Build failed with an exception.
* What went wrong:
Execution failed for task ':app:transformClassesWithDexBuilderForDebug'.
> com.android.build.api.transform.TransformException: com.android.builder.dexing.DexArchiveBuilderException: com.android.builder.dexing.DexArchiveBuilderException:`

flutter doctor -v
[√] Flutter (Channel stable, v1.0.0, on Microsoft Windows [Version 6.1.7601], locale en-US)
    • Flutter version 1.0.0 at C:\Mobile App\flutter
    • Framework revision 5391447fae (2 weeks ago), 2018-11-29 19:41:26 -0800
    • Engine revision 7375a0f414
    • Dart version 2.1.0 (build 2.1.0-dev.9.4 f9ebf21297)

[√] Android toolchain - develop for Android devices (Android SDK 28.0.3)
    • Android SDK at C:\Users\j\AppData\Local\Android\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    • All Android licenses accepted.

[√] Android Studio (version 3.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 31.2.1
    • Dart plugin version 181.5656
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

[√] VS Code (version 1.30.0)
    • VS Code at C:\Users\j\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 2.21.1

[√] VS Code, 64-bit edition (version 1.23.1)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 2.21.1

[√] Connected device (1 available)
    • Moto E 4 • ZY2245BRZN • android-arm • Android 7.1.1 (API 25)

• No issues found!

@JesseRMeyer
The issue at https://github.com/flutter/flutter/issues/25434 has been closed and moved here. Future collaboration on this issue will be done here.

I don't see any connection with FlutterFire,
reopening https://github.com/flutter/flutter/issues/25434


LibC::ESHUTDOWN is not defined

https://github.com/crystal-lang/crystal/search?q=ESHUTDOWN shows it being used but not defined
https://github.com/crystal-lang/crystal/commit/27b790f380fc3bc12882840d06fe0c55b99b1757#diff-b67ae37b7956e573f1a5e19006b2615aR78 changed by @ysbaddaden
Possibly related: https://github.com/python/cpython/blob/2ebc5ce42a8a9e047e790aefbf9a94811569b2b6/Objects/exceptions.c#L2653

Same for:

ENOTBLK
ESOCKTNOSUPPORT
EPFNOSUPPORT
ESHUTDOWN
ETOOMANYREFS

Is this still relevant after #8885?

Fixed by https://github.com/crystal-lang/crystal/commit/4164afe5d9844c419e59c6fdb01998a2ceea81e2#diff-84f879f7cc7be3862a99405ba54b77eb26a54d5a7f96ddfafe00f166416ac87dL137
Thanks


Spyder3.exe crashes on launch.

Description of your problem
When trying to run Spyder3.exe in Python36-32/Scripts, the Spyder console opens for a few seconds then crashes with following error report.
Problem signature:
Problem Event Name:	APPCRASH
Application Name:	pythonw.exe
Application Version:	3.6.1150.1013
Application Timestamp:	58d169c2
Fault Module Name:	Qt5WebEngineCore.dll
Fault Module Version:	<IP_ADDRESS>
Fault Module Timestamp:	588148ea
Exception Code:	80000003
Exception Offset:	000a8384
OS Version:	6.1.76<IP_ADDRESS>.768.11
Locale ID:	1033
Additional Information 1:	0a9e
Additional Information 2:	0a9e372d3b4ad19135b953a78882e789
Additional Information 3:	0a9e
Additional Information 4:	0a9e372d3b4ad19135b953a78882e789
What steps will reproduce the problem?

Installed Python directly from website. Installed QtPy and Spyder using cmdline.
Double clicked Spyder3.exe in Scripts folder.

What is the expected output? What do you see instead?
The IDE doesn't start, although a screen appears for a few seconds resulting in a crash.

Please provide any additional information below
Versions and main components

Spyder Version:3.1.4
Python Version:3.6.1
Qt Version: Not sure, Used pip to install PyQt through wheels so it installed all the dependencies automatically.
PyQt Version:5.8.2
Operating system:Win7 Starter

Dependencies
Please go to the menu entry Help > Optional Dependencies (or
Help > Dependencies), press the button Copy to clipboard
and paste the contents below:

Please follow this https://github.com/spyder-ide/spyder/issues/3736#issuecomment-262682824
If that doesn't work, please download and install Anaconda. Pip installations are for experts only, so if you want to continue towards that path, it's up to you to solve your problem, sorry.


How to load xib for image sticker view?

Can you please provide a way to load xib for imahgeStickerViewContainer? As for now its a only view with contraint added programatically.

xib just puts the layout of the control into the xib file. The usage method is the same as implementing the layout directly with code.


example on docker-compose

how could write this in compose file?

Here is the docker-compose file I used, you can take it as a reference:
version: '3.9'

x-logging:
  &default-logging
  driver: json-file
  options:
    max-size: "100m"
    max-file: "3"

services:
  gitlab:
    image: zengxs/gitlab:16.10.3-ee  # You can choose ce (community edition) or ee (enterprise edition) depending on your needs.
    container_name: gitlab
    restart: unless-stopped
    ports:
      - 9002:80
      - 2222:22
    depends_on:
      - redis
      - postgres
    links:
      - redis:redis.local
      - postgres:postgres.local
    extra_hosts:
      - customers.gitlab.com:<IP_ADDRESS>
    environment:
      TZ: Asia/Singapore
      GITLAB_OMNIBUS_CONFIG: from_file('/etc/gitlab.rb')
    volumes:
      # Your gitlab configurations file
      - ./gitlab.rb:/etc/gitlab.rb:ro
      # GitLab data dirs
      - ./gl-config:/etc/gitlab
      - ./gl-log:/var/log/gitlab
      - ./gl-data:/var/opt/gitlab
    logging: *default-logging

  # gitlab-runner (if needed)
  runner:
    image: gitlab/gitlab-runner:alpine
    container_name: gitlab-runner
    restart: unless-stopped
    environment:
      TZ: Asia/Singapore
    volumes:
      - ./glr-config:/etc/gitlab-runner
    logging: *default-logging

  # I'm using an external Redis service (you can also use the one inside the
  # GitLab image, which would make this section unnecessary).
  redis:
    image: redis:7.0.14
    restart: unless-stopped
    volumes:
      - gitlab_redis:/data
    command: redis-server --save 60 1 --appendonly yes
    logging: *default-logging

  # External postgres service (similar to Redis, you can also use the
  # internal postgres service inside the GitLab image, If you opt for
  # an external service, make sure the version is consistent with what
  # GitLab requires).
  postgres:
    image: postgres:13.13
    restart: unless-stopped
    volumes:
      - gitlab_postgres:/var/lib/postgresql/data
    environment:
      TZ: Asia/Singapore
      POSTGRES_USER: gitlab
      POSTGRES_DB: gitlab_db
      POSTGRES_PASSWORD: your_database_password
    logging: *default-logging

volumes:
  gitlab_redis:
    driver: local
  gitlab_postgres:
    driver: local

Below is the configuration file for GitLab, which should be named gitlab.rb and placed in the same directory as the docker-compose.yml:
########## GitLab Configurations ##########
# More configurations see: https://gitlab.com/gitlab-org/omnibus-gitlab/blob/master/files/gitlab-config-template/gitlab.rb.template

external_url "https://your-domain.com"
gitlab_rails['gitlab_default_theme'] = 3
gitlab_rails['gitlab_username_changing_enabled'] = false

# postgresql settings (if you use external postgresql service)
postgresql['enable'] = false
postgresql['version'] = 13
gitlab_rails['db_adapter'] = "postgresql"
gitlab_rails['db_database'] = "gitlab_db"
gitlab_rails['db_username'] = "gitlab"
gitlab_rails['db_password'] = "your_database_password"
gitlab_rails['db_host'] = "postgres.local"
gitlab_rails['db_port'] = 5432

# redis settings (if you use external redis service)
redis['enable'] = false
gitlab_rails['redis_host'] = "redis.local"
gitlab_rails['redis_port'] = 6379

# puma settings
puma['enable'] = true

# bundled nginx settings
# This is the internal nginx configuration for GitLab. I'm using an additional
# external nginx to handle SSL. If you want to use the internal nginx to handle
# SSL directly, or if you prefer not to use SSL at all, you can modify it according to your needs.
nginx['redirect_http_to_https'] = false
nginx['listen_addresses'] = ['*']
nginx['listen_port'] = 80
nginx['listen_https'] = false
nginx['gzip_enabled'] = false
# Add custom robots.txt (if needed)
nginx['custom_gitlab_server_config'] = <<EOS
  location = /robots.txt {
    default_type text/html;
    return 200 'User-Agent: *\nDisallow: /\n';
  }
EOS
nginx['worker_processes'] = 1
nginx['worker_connections'] = 768

# misc settings
prometheus['enable'] = false
prometheus_monitoring['enable'] = true
gitlab_exporter['listen_address'] = '<IP_ADDRESS>'
gitlab_exporter['listen_port'] = '9168'
registry['enable'] = false

# omniauth settings
# I use SAML for implementing SSO. If it's not needed, you can delete the following section.
gitlab_rails['omniauth_allow_single_sign_on'] = ['saml']
gitlab_rails['omniauth_auto_sign_in_with_provider'] = 'saml'
gitlab_rails['omniauth_block_auto_created_users'] = false
gitlab_rails['omniauth_auto_link_saml_user'] = true
gitlab_rails['omniauth_providers'] = [
  {
    name: 'saml',
    label: 'Keycloak',
    groups_attribute: 'groups',
    required_groups: ['gitlab-users', 'gitlab-administrators'],
    admin_groups: ['gitlab-administrators'],
    args: {
      assertion_consumer_service_url: "https://REDACTED",
      idp_cert_fingerprint: "REDACTED",
      idp_sso_target_url: "https://REDACTED",
      issuer: "REDACTED",
      name_identifier_format: "urn:oasis:names:tc:SAML:2.0:nameid-format:username",
      attribute_statements: { nickname: ['username'] },
    },
  }
]

# email settings
gitlab_rails['gitlab_email_enabled'] = true
gitlab_rails['gitlab_email_from'] = 'REDACTED'
gitlab_rails['gitlab_email_display_name'] = 'GitLab'
gitlab_rails['gitlab_email_subject_suffix'] = 'GitLab Instance'
gitlab_rails['smtp_enable'] = true
gitlab_rails['smtp_address'] = "email-smtp.us-west-2.amazonaws.com"
gitlab_rails['smtp_port'] = 587
gitlab_rails['smtp_user_name'] = "REDACTED"
gitlab_rails['smtp_password'] = "REDACTED"
gitlab_rails['smtp_domain'] = "REDACTED"
gitlab_rails['smtp_authentication'] = "login"
gitlab_rails['smtp_enable_starttls_auto'] = true

I hope this is helpful to you.

Is it normal that gitlab takes an absurd amount of time to start every time? It looks like it's re-running all the rails install every time...

Is it normal that gitlab takes an absurd amount of time to start every time? It looks like it's re-running all the rails install every time...

@kubrickfr It is indeed normal for the GitLab Docker container to take some time to start as it reconfigures itself. This behavior is by design. The GitLab container goes through a series of initialization steps every time it starts to ensure that the configuration and environment are properly set up.
This includes steps like database migrations, configuration setup, and other necessary initializations to ensure the system runs correctly and consistently.


Chrome runs 52 test while FF runs 36

I've not dug into why, most likely a shim difference that accounts for this but still wanted to file an issue to remind me (or someone else) to really determine why.

if you're seeing this on travis, can you give links to the runs?
alternatively the command lines you're using, with the BROWSER and BVER envars?

32 for firefox beta - https://travis-ci.org/webrtc/adapter/jobs/67002072
38 for Firefox stable - https://travis-ci.org/webrtc/adapter/jobs/67002071

The difference between the two firefoxen is that stable returns this for enumerateDevices:
ok 36 valid enumerateDevices output: [{"kind":"audioinput","deviceId":"default","label":"","groupId":""},{"kind":"videoinput","deviceId":"default","label":"","groupId":""}]
beta returns this:
ok 30 valid enumerateDevices output: []
The extra tests are in a loop running over the device enumeration result.
I think beta's output is more correct, since the travis machines (presumably) don't have audio or video input.
Chrome (stable and beta) also runs 32 tests in that iteration, and has enumerateDevices output of [].
Running the tests locally SHOULD produce quite different results, due to the presence of devices.
Later runs also run more tests for Chrome, since they include the tests for "originalChromeGetStats".
Closing this bug as "investigated enough".

We could take out the conditional tests, since conditional tests are arguably not very useful. Thoughts?

I think conditional tests are much more useful than no tests - for instance, the Chrome stats-massage tests are extremely important for Chrome users, but Firefox users won't be affected if they break, so it makes sense to only run them on Chrome.


feat: add reactGA4 and uiSetting to use it

Description [STORY LINK]

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] Tested changes locally.
[ ] Updated documentation.

Screenshots

This looks fine, but I can't approve it now because we're about to release everything on the beta branch to support Tap integration.


unclear error message

Consider the following query:
SELECT * {
  ?s ?p ?o.
  BIND (?o as ?s)
}

The error message is not very informative
error at 4:2: expected [_]

Contrast this with sparql.org's error message:
BIND: Variable used when already in-scope: ?s in BIND(?o AS ?s)

NB: in this minimal example, it is quite obvious what the error is. But I ran into this problem with a long query where it was much less obvious.

Indeed SPARQL parser errors are awful at the moment. I have opened #129 about it.


cmd/go: test -c fails to rebuild when test changes

go test -c correctly avoids the rebuild if the test or its dependencies have not changed.
Unfortunately, it also incorrectly avoids the rebuild when the test has changed.
I ran
cd go/src/path/filepath
go test -c
add a test case to cleantests in path_test.go
go test -c

and the second go test -c did not rebuild the filepath.test in the directory.

Actually I can't reproduce this anymore. Modifying comments correctly prevents the rebuild; modifying the test or xtest packages or the package itself does trigger a rebuild. I must have gotten confused trying different kinds of edits.


Include ChangeLogClass in ChangeEntry search query

In order to support splitting change log classes up (i.e. one change log class per collection) it is useful to include the change log class name in both the ChangeEntry search query and the in the unique index on dbchangelog.
This change adds the required functionality

My concerns after the review:
Why do you need such functionality?
If we merge and release this, we loose backward compatibility. It is very danger because after upgrading the lib in a project all existing ChangeSets will become "new" and will be executed again...
Second thing: if you want to refactor your very long, old ChangeLog and split it into few smaller ones (move ChangeSet methods in different ChangeLog class) then mongobee will execute changesets again.
When I wrote this ChangeEntry class I was thinking about including class name into ChangeEntry. That time the against argument it was that class is "easy refactorable" and ID value strings are not. My first concern above was one of the against arguments as well.
What do you think?

Hi
The intent of the change was to ensure that duplicate change set ids in different change logs files are treated as being different, and are not skipped over. i.e. if I have change log A with a change set with an id of 'createInitialIndexes', and a change log B with a change set id of 'createInitialIndexes', the second change set would be skipped, even though it has not been run (assuming they were authored by the same person).
As mentioned we tend to split our change logs into a class per collection, and as such it becomes harder to ensure that you have duplicate change set ids across the various files.
I do however take your point about backwards compatibility. Could we add a flag to indicate whether or not to include the change log name?
Sorry for the slow reply.
Regards
Brian


[CE1.x] replace CA3 with CA4, refactoring/improving version of CA3

Vulnerable to malicious nodes
Inherits un-smart specifications of CA2


onents

Closing since no description of the problem has been provided.

Did I butt dial a github issue?! Don't know where this came from. Thanks for closing!


Zig does not correctly determine runtime value in nested scopes

fn get_foo() fn(&u8)usize {
    comptime {
        return struct {
            fn func(ptr: &u8) usize {
                // Triggers: ' error: cannot store runtime value in compile time variable'
                var u = @ptrToInt(ptr);
                return u;
            }
        }.func;
    }
}

test "foo" {
    const foo = get_foo();
}

The compiler should determine whether a variable lives in runtime or compile time through the nearest function scope.

This could also possibly help with nested functions, when they are to be implemented.


How can I create a cert for example.com & www.example.com?

Here is the command I tried:
AWS_ACCESS_KEY_ID="" AWS_SECRET_ACCESS_KEY="" certbot --agree-tos -a certbot-s3front:auth --certbot-s3front:auth-s3-bucket example.com --certbot-s3front:auth-s3-region us-west-1 -i certbot-s3front:installer --certbot-s3front:installer-cf-distribution-id <ID> --config-dir ./config --work-dir ./work --logs-dir ./logs -d www.example.com -d example.com

When I add the www.example.com, it fails. If I just run this with example.com, it works. It logs the following:
Starting new HTTPS connection (1): acme-v01.api.letsencrypt.org
Obtaining a new certificate
Performing the following challenges:
http-01 challenge for example.com
http-01 challenge for www.example.com
Found credentials in environment variables.
Calling s3:put_object with {'Body': u'<id>', u'Bucket': 'example.com', 'Key': u'.well-known/acme-challenge/<id>', 'ACL': 'public-read'}
Starting new HTTPS connection (1): s3-us-west-1.amazonaws.com
Starting new HTTP connection (1): example.com
Starting new HTTPS connection (1): example.com
Calling s3:put_object with {'Body': u'<id>', u'Bucket': 'example.com', 'Key': u'.well-known/acme-challenge/<id>', 'ACL': 'public-read'}
Starting new HTTPS connection (1): s3-us-west-1.amazonaws.com
Starting new HTTP connection (1): www.example.com
Starting new HTTPS connection (1): www.example.com
Unable to reach http://www.example.com/.well-known/acme-challenge/<id>: hostname 'www.example.com' doesn't match 'example.com'
Self-verify of challenge failed, authorization abandoned!
Waiting for verification...
Cleaning up challenges
Starting new HTTPS connection (1): s3-us-west-1.amazonaws.com
Incomplete authorizations

Note that I added www.example.com AFTER initially setting up the cert with example.com.

Probably a bit late but I came here looking for the answer and figured it out eventually:
Firstly, you specify additional domains by adding extra -d options to the original commandline. So in your example you'd have -d example.com -d www.example.com. As you've already run it certbot will ask you if you want to expand the existing certificate. Just say yes.


issue/6731:dict path: allow matching float with corresponding int (#6734)

Pull request opened by the merge tool on behalf of #6734

Merged into branches iso6 in b3a25522b06671118f6acdeac3f56cb4ca30efba


How to Recover Data from Samsung S3/S4/S5 with Broken Screen

"Is it possible to extract my photos and contacts on my Samsung Galaxy S4 which with the broken screen? I want to buy a new S7 edge and don't want to change a new screen for it, how can I do?"
Are you also met the similar situation and looking for a effective way to recover data from the cracked Samsung S4, as well as Galaxy S3/S5 and other Samsung phone?
As a matter of fact, to Recover Data from Samsung with Cracked Screen, what you need is just the Samsung Data Extraction, which can restore any lost and deleted from Samsung Galaxy device in a click.
Related Articles:
How to Fix Samsung Galaxy Sudden Death
How to Fix Samsung Galaxy Black Screen of Death

Samsung Data Recovery - The World's 1st Android Data Recovery for Smartphones & Tablets, which enables to retrieve lost or deleted contacts, text messages, photos, WhatsApp messages, audio files, videos, documents and more even from the broken & black screen & bricked devices.
So, you can easily restore data from Samsung with broken screen by using this tool.
Read the video guide here:
Read more:
Unlock Samsung Phone with Broken Screen
Recover Data from Samsung Sudden Death

This Coolmuster Data Recovery software not only recover deleted data from Samsung , but also retrieve data from others android phone. With only several clicks, you can recover phone deleted contacts, text messages, photos, audio, video and documents .
How to retrieve deleted text messages


tool doesn't seem to be working for java9 jstack output

Hi guys,
Thanks for the wonderful tool. I was trying it out for the java9 jstack output (attached) and the tool doesn't seem to be doing anything, I tried it for a java8 jstack as well and it still doesn't respond. Can you please see if I am missing something obvious here?
Thanks!
sample-jstack.txt

Seems to work for me with your example thread dump, can you post a screenshot of what you're getting?

Thanks for the reply @walles - it was my adblocker that was being (over)smart here and blocking the load of analyze.js. So it was not a problem per-se. Thanks and apologies for the trouble!


fix for eslint bug #15261

Details
Only the last restricted import from the same path is checked and others are ignored when there are multiple imports due to the eslint bug
Fixed Issues
$ https://github.com/Expensify/App/issues/22459
PROPOSAL: https://github.com/Expensify/App/issues/22459#issuecomment-1627305981
Tests

open a random component file such as src/components/AddressSearch/index.js
at the top of the file, import one of the following packages from react-native

useWindowDimensions
TouchableOpacity
TouchableWithoutFeedback
TouchableNativeFeedback
TouchableHighlight
StatusBar

for each imported component above, see an eslint error appears in VS Code if ESLint extension is added (a hint for error is shown when hovering mouse over the imported component.)

[x] Verify that no errors appear in the JS console

Offline tests
Same steps above
QA Steps
Same steps above

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos
Since it is not a UI issue, no ui screenshots are added. Eslint gives console error when a restricted component is used as shown in the following:

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

I have read the CLA Document and I hereby sign the CLA

looks like there's a prettier diff that needs to be committed

@roryabraham, @allroundexperts: sorry guys

FWIW I have my editor configured to run prettier whenever I save. It's pretty easy to set up that way and then I never think about formatting my code. Just type and press CMD+S and everything's formatted correctly.

🚀 Deployed to staging by https://github.com/roryabraham in version: 1.3.40-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@HLEDYA @allroundexperts @roryabraham Can you pls QA this internally?

@mvtglobally this should have been marked as No QA, we're good to go here.

FWIW I have my editor configured to run prettier whenever I save. It's pretty easy to set up that way and then I never think about formatting my code. Just type and press CMD+S and everything's formatted correctly.

Thank you @roryabraham, I also enabled format on save for my VS code ide.

🚀 Deployed to production by https://github.com/Julesssss in version: 1.3.40-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Fixed bug in  trivial_inf_flexes

Bug reported by Sean Dewar: the computation of inf_flexes was wrong, and in particular it depended on the order in which the vertices appeared in the construction of the graph.
This PR fixes the bug by making uniform between the methods rigidity_matrix, trivial_inf_flexes, and inf_flexes the choice of vertex and edge order.

Could you add the descriptions of edge_order and vertex_order to the docstrings? And fix the description of edge_order in rigidity_matrix as well?
Please, add Sean's example to the tests.


nixos/release-combined: disable i686 iso_minimal

Motivation for this change
It is blocking the channel.
https://github.com/NixOS/nixpkgs/issues/82435

Wait a while, please; I'm testing a hacky fix of the build at least.


bug: test flake with tcpingress proxy with dbless

It's been observed that the integration tests will flake on the TCPIngress integration tests, particularly waiting for the teardown to complete.
This so far only happens in DBLESS mode, and does not appear to occur with Postgres mode nor in the legacy integration tests.
Modulating the resolution intervals for the proxy cache has so far appeared to be ineffective, the purpose of this issue is to continue the investigation and get this resolved.

https://github.com/Kong/kubernetes-ingress-controller/pull/1397 appears to have resolved this.


Don't Merge! PR real time test for webhook case

Docs Build status updates of commit c1ef6fa:
:clock10: Full build: the average full build time is 6 min(s) 5 sec(s), this is based on the last 483 build(s) for this repository.

Docs Build status updates of commit c1ef6fa:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

docs-ref-conceptual/testing.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Any version after 2.0.0 crashes on clean fabric server

A 1.20.1 fabric server with only Thermite (version 2.1.0 - 4.1.0) and its dependencies will crash with the error:
[16:41:04] [main/ERROR]: Failed to start the minecraft server
java.lang.RuntimeException: Could not execute entrypoint stage 'main' due to errors, provided by 'therm'!
at net.fabricmc.loader.impl.FabricLoaderImpl.lambda$invokeEntrypoints$2(FabricLoaderImpl.java:388) ~[fabric-loader-0.15.0.jar:?]
at net.fabricmc.loader.impl.util.ExceptionUtil.gatherExceptions(ExceptionUtil.java:33) ~[fabric-loader-0.15.0.jar:?]
at net.fabricmc.loader.impl.FabricLoaderImpl.invokeEntrypoints(FabricLoaderImpl.java:386) ~[fabric-loader-0.15.0.jar:?]
at net.fabricmc.loader.impl.game.minecraft.Hooks.startServer(Hooks.java:63) ~[fabric-loader-0.15.0.jar:?]
at net.minecraft.server.Main.main(Main.java:112) ~[server-intermediary.jar:?]
at net.fabricmc.loader.impl.game.minecraft.MinecraftGameProvider.launch(MinecraftGameProvider.java:470) ~[fabric-loader-0.15.0.jar:?]
at net.fabricmc.loader.impl.launch.knot.Knot.launch(Knot.java:74) ~[fabric-loader-0.15.0.jar:?]
at net.fabricmc.loader.impl.launch.knot.KnotServer.main(KnotServer.java:23) ~[fabric-loader-0.15.0.jar:?]
at net.fabricmc.loader.impl.launch.server.FabricServerLauncher.main(FabricServerLauncher.java:69) ~[fabric-loader-0.15.0.jar:?]
Caused by: java.lang.NoSuchFieldError: checkbox
at me.lortseam.completeconfig.data.BooleanEntry.(BooleanEntry.java:26) ~[completeconfig-base-2.4.0-41ce8135a410a76.jar:?]
at me.lortseam.completeconfig.data.Entry.create(Entry.java:42) ~[completeconfig-base-2.4.0-41ce8135a410a76.jar:?]
at me.lortseam.completeconfig.data.EntrySet.lambda$resolve$1(EntrySet.java:31) ~[completeconfig-base-2.4.0-41ce8135a410a76.jar:?]
at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:197) ~[?:?]
at java.util.stream.ReferencePipeline$2$1.accept(ReferencePipeline.java:179) ~[?:?]
at java.util.Spliterators$ArraySpliterator.forEachRemaining(Spliterators.java:992) ~[?:?]
at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:509) ~[?:?]
at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:499) ~[?:?]
at java.util.stream.ForEachOps$ForEachOp.evaluateSequential(ForEachOps.java:150) ~[?:?]
at java.util.stream.ForEachOps$ForEachOp$OfRef.evaluateSequential(ForEachOps.java:173) ~[?:?]
at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234) ~[?:?]
at java.util.stream.ReferencePipeline.forEach(ReferencePipeline.java:596) ~[?:?]
at me.lortseam.completeconfig.data.EntrySet.resolve(EntrySet.java:32) ~[completeconfig-base-2.4.0-41ce8135a410a76.jar:?]
at me.lortseam.completeconfig.data.Parent.resolveContainer(Parent.java:58) ~[completeconfig-base-2.4.0-41ce8135a410a76.jar:?]
at me.lortseam.completeconfig.data.Parent.resolve(Parent.java:110) ~[completeconfig-base-2.4.0-41ce8135a410a76.jar:?]
at me.lortseam.completeconfig.data.Config.lambda$new$0(Config.java:51) ~[completeconfig-base-2.4.0-41ce8135a410a76.jar:?]
at me.lortseam.completeconfig.data.Config.deserialize(Config.java:94) ~[completeconfig-base-2.4.0-41ce8135a410a76.jar:?]
at me.lortseam.completeconfig.data.Config.load(Config.java:121) ~[completeconfig-base-2.4.0-41ce8135a410a76.jar:?]
at thermite.therm.ThermMod.onInitialize(ThermMod.java:73) ~[thermite-4.1.0_mc-1.20.1.jar:?]
at net.fabricmc.loader.impl.FabricLoaderImpl.invokeEntrypoints(FabricLoaderImpl.java:384) ~[fabric-loader-0.15.0.jar:?]
... 6 more
Weirdly, it works fine on quilt, though it may have corrupted certain items or block, but that could have been caused by something else

I am also getting the same issue. My bet is it has something to do with the fact that quilt uses an older version of fabric api (version 0.90.0 instead of 0.91.3)

Try using version 0.14.21 of the fabric loader on the server. I tested it and it worked on that version, but same error on anything above it.
It seems to be something to do with CompleteConfig. Versions 2.1.0-4.1.0 all use CompleteConfig, but below 2.1.0 it isn't used.
Also I just found out that CompleteConfig is deprecated and won't update to Minecraft 1.21 so I will be using a different config library in the future.
Also I don't know anything about quilt.

How come this doesn’t crash on clients then?

I cannot use an older version of fabric loader as I have several mods with dependencies on fabric 0.14.24


Volumes: broken links

The links for "azureDisk volume plugin" and "azureFile volume plugin" are broken.
Current: https://github.com/kubernetes/examples/tree/main/staging/volumes/azure_disk/README.md
Should be: https://github.com/kubernetes/examples/tree/master/staging/volumes/azure_disk/README.md

#29311 will fix it

Fixed in #29311
/close


Map controls css

Main Changes:

Moved control buttons (CSV download, tour, link/embed map) to map view
Changed order of rendering (search box in sync with controls)
Moved the search box to the right
Added Material Icons and FontAwesome stylesheets
Changed calendar icon for "Time Sort" from png to font

Rebased to upstream/dev

Please rebase to dev so we can merge this. If this was already merged to master, please close because we merged master to dev already.

It was rebased to dev ("Rebased to upstream/dev") and was not merged to master, that's why it's on queue for the merge.

Please rebase again, as dev has changed since.

Sure, but before I push it I want to do a little QA and since both up-to-date master and dev give out the error below, then I can't check it until it is fixed.
"Traceback (most recent call last):
File "main.py", line 14, in
from clusters_calculator import retrieve_clusters
File "/Users/lironshalev/anyway/clusters_calculator.py", line 5, in
import concurrent.futures
ImportError: No module named concurrent.futures"

Did you run 'pip install -r requirements.txt' after updating?
On Tue, Jun 30, 2015 at 4:54 PM LironRS<EMAIL_ADDRESS>wrote:

Sure, but before I push it I want to do a little QA and since both
up-to-date master and dev give out the error below, then I can't check it
until it is fixed.
"Traceback (most recent call last):
File "main.py", line 14, in
from clusters_calculator import retrieve_clusters
File "/Users/lironshalev/anyway/clusters_calculator.py", line 5, in
import concurrent.futures
ImportError: No module named concurrent.futures"
—
Reply to this email directly or view it on GitHub
https://github.com/hasadna/anyway/pull/341#issuecomment-117192441.

Needs rebasing to dev again, sorry.
Also, could you squash your commits so that each commit is one whole coherent change?


[feature] Publish buildinfo in a project specific artifactory repo

Provide an option to the conan_build_info --v2 publish subcommand to be able to specify an artifactory project with a dedicated buildinfo repository where the buildinfo should be stored.
Currently one can only perform this with the jfrog CLI build upload command (see https://www.jfrog.com/confluence/display/JFROG/Artifactory+REST+API#ArtifactoryRESTAPI-BuildUpload) but this requires an additional tool in the CI pipeline (including setup and authentication) though the principal functionality is already contained in the conan_build_info tool (see also https://github.com/czoido/conan-build-info-example/issues/1).

[*] I've read the CONTRIBUTING guide.

this ticket has been moved to the conan-extensions repository, where the new conan art:commands extension commands for Artifactory, including commands to generate the BuildInfo are.
@czoido any update regarding this ticket?

Closing this as solved by https://github.com/conan-io/conan-extensions/pull/56 that introduced support for projects in build info. Thanks @fg68 for reporting.


factory: set guest time after resuming

We might have paused a guest for a long time so we need to sync its time.
depends on https://github.com/kata-containers/agent/pull/425

Assuming you will be adding the commit list in your vendor commit, this
lgtm
As @jodh-intel pointed, will be useful to have an integration test for this.

Sure @jodh-intel . Please see https://github.com/kata-containers/tests/pull/969

/test


Update @nx/workspace to 19.6.5

Update Nx from 19.6.4 to 19.6.5

Auto-generated by create-pull-request

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

Project coverage is 97.63%. Comparing base (2864f9f) to head (332bafb).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #160   +/-   ##
=======================================
  Coverage   97.63%   97.63%
=======================================
  Files          17       17
  Lines         211      211
  Branches       30       30
=======================================
  Hits          206      206
  Misses          5        5

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


Add actual version number to pestle

We need to add an actual version number to pestle, along with a version command that reports it.  Might make build process more formal while we're at it.

Added pestle.phar version, and added some checks to our build scripts to ensure latest git tag and latest version match.


Datascripts works with clojure 1.10.2

With graalvm-ce-java8-20.2.0
$ target/datascript
([:red [30 40 50] [10 20 30]] [:blue [7 8] [7 8]])
Hello GraalVM.

Output
Compiling simple.main
Created /Users/jeroen/Projects/Github/graalvm-clojure/datascript/target/datascript-0.1.0-SNAPSHOT.jar
Created /Users/jeroen/Projects/Github/graalvm-clojure/datascript/target/simple-main.jar
[./target/datascript:42257]    classlist:   2,662.68 ms,  1.49 GB
[./target/datascript:42257]        (cap):   2,739.74 ms,  1.49 GB
[./target/datascript:42257]        setup:   8,772.37 ms,  1.49 GB
[./target/datascript:42257]     (clinit):     166.56 ms,  2.18 GB
[./target/datascript:42257]   (typeflow):   8,450.98 ms,  2.18 GB
[./target/datascript:42257]    (objects):   7,303.24 ms,  2.18 GB
[./target/datascript:42257]   (features):     328.84 ms,  2.18 GB
[./target/datascript:42257]     analysis:  37,140.81 ms,  2.18 GB
[./target/datascript:42257]     universe:     628.34 ms,  2.18 GB
[./target/datascript:42257]      (parse):   2,419.50 ms,  2.19 GB
[./target/datascript:42257]     (inline):   1,962.76 ms,  2.43 GB
[./target/datascript:42257]    (compile):  14,987.94 ms,  3.41 GB
[./target/datascript:42257]      compile:  35,348.16 ms,  3.41 GB
[./target/datascript:42257]        image:   2,185.15 ms,  3.44 GB
[./target/datascript:42257]        write:     718.46 ms,  3.44 GB
[./target/datascript:42257]      [total]: 122,739.52 ms,  3.44 GB

I was misled by the comment in the README:
Datascript does not compile at all when using clojure 1.10

It does work with 1.10.2 now.

I was misled by the comment in the README:
It does work with 1.10.2 now.

could you please update the README in that case?

Sure! I removed the comment from the README and specified the clojure version explicitly

thank you


Update Developer Guide

Update user authentication implementation.
Signed-off-by: jjlee050<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build 582

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

Totals

Change from base Build 577:
0.0%

Covered Lines:
2064

Relevant Lines:
2748

💛  - Coveralls

Hang on. I work on the about this guide.


Virtual Component Groups in the Hierarchical Action List (1/2)

Pull Request Description

Parse the Engine's response containing Virtual Component Groups and store the results in a field of the Execution Context type.
https://www.pivotaltracker.com/story/show/181865548
Important Notes

This PR implements the subtask 1 of 2 in the "Virtual Component Groups in the Hierarchical Action List" task.

[ci no changelog needed]
Checklist
Please include the following checklist in your PR:

[x] The documentation has been updated if necessary.
[x] All code conforms to the
Scala,
Java,
and
Rust
style guides.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed: Enso GUI was tested when built using BOTH
./run.sh ide dist and ./run.sh ide watch.

QA approval granted, have a nice day!


{😀/😐/🙁 short headline describing the feedback}

Feedback
Leave your feedback here.
Details
Supplemental data gives context for the feedback and makes it easier to act on, where appropriate.
Provide as many details as you feel comfortable sharing publicly.
Alternatively, email data to<EMAIL_ADDRESS>while referencing the
created issue, or opt to send the feedback entirely by email.
Output
Optionally include the tool's output. You can add a screenshot, or copy and paste the text output
either in its entirety or only the relevant section(s). Inline the output as a code block here or
attach a text file.
example output

File(s) scanned
Optionally attach the Bazel profile(s) that were analyzed by the Bazel Invocation Analyzer.
Ideally, do not attach files directly. Bazel profiles can include sensitive data, such as exposing
which dependencies your project includes.
Instead, upload files to the cloud storage provider of your choice and grant appropriate access.
This allows you to adjust who can access them now and in the future. For example, you may want to
remove access to the files once the issue has been resolved.
Further details
Provide other data related to this issue that may prove helpful.

This seems to be the issue template without any edits / data added. Closing.


typos in doc

In docs of python API, https://carla.readthedocs.io/en/latest/python_api/#carlaworldsnapshot, acces should be access, recieve should be receive

Thank you for pointing this out, this will be corrected in an internal pull request, closing this PR.


Add usage docs in-tree

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

Thanks @steveklabnik! I prefer to not recommend "*" as a version requirement, but I will update that part when merging all these PRs.

Yeah, I mostly copied what was in each project's README.


Use the GitHub API to render Markdown

It would look a lot better, but need to watch the request limits.

As long as users are logged in, the request limits are pretty high. As an alternative, I would consider CommonMark.NET. You can parse the markdown to an AST, and then use a custom printer (modeled after the one in CommonMark.NET) which handles the GFM syntax for referencing issues.


brew install subversion18 --universal --with-java fails due to dependency failing to be located

Please follow the general troubleshooting steps first:

[x] Ran brew update and retried your prior step?
[x] Ran brew doctor, fixed as many issues as possible and retried your prior step?

Bug reports:
The install of subversion18 fails because the serf-1.3.8.tar.bz2 package has moved from the googlecode.com repo to the following URL:
https://archive.apache.org/dist/serf/serf-1.3.8.tar.bz2
https://serf.googlecode.com redirects to https://code.google.com/archive/p/serf/ where the following is shown:

With the closing of Google Code, the serf project has now become Apache Serf!!
Please see our new home page for information, downloads, and issue tracking for the project.
The error in from brew is listed below:
$ brew install subversion18 --universal --with-java
==> Installing subversion18 from homebrew/versions
==> Downloading https://www.apache.org/dyn/closer.cgi?path=subversion/subversion-1.8.16.tar.bz2
Already downloaded: /Library/Caches/Homebrew/subversion18-1.8.16.tar.bz2
==> Patching
patching file configure
Hunk #1 succeeded at 25433 (offset 67 lines).
patching file subversion/bindings/swig/perl/native/Makefile.PL.in
patching file build/get-py-info.py
==> Downloading https://serf.googlecode.com/svn/src_releases/serf-1.3.8.tar.bz2
curl: (22) The requested URL returned error: 404 Not Found
Error: Failed to download resource "subversion18--serf"
Download failed: https://serf.googlecode.com/svn/src_releases/serf-1.3.8.tar.bz2

Oh, I did try to manually download the file and move it into the /Library/Caches/Homebrew/ folder, but it is still not picking up the dependency from that location

Fixed via https://github.com/Homebrew/homebrew-versions/commit/516908f1edff0362d6f1004007c3aff1505062b2. Please brew update and try again.
Note reporting issues that apply to homebrew/versions formulae should be done in that tap, Thanks!


Allow Metrics.NET to export directly to Prometheus.io

Since Prometheus has been blessed as "the" way to monitor by the CNCF, it's likely worth adding to Metrics.NET
This is clearly just a WIP, but I was wondering if that's the approach you'd want for this.

I agree, it would be nice to have Prometheus integration with Metrics.NET.
In my opinion the ideal way to add new reporters would be as separate entities from the core metrics library (separate codebase & nuget). This way they can have their own dependencies and lifecycles, and people that don't want to use a certain reporter won't get it when they install the core library.
For Prometheus, I see that you need to have an endpoint that is periodically pulled. Currently there isn't any way of adding a new endpoint directly to the MetricsHttpListener without directly modifying the class. So I'm thinking that in order to implement a decoupled Prometheus reporter we'd have to find a way to add pull-model reporters via config extension methods.
We'll investigate how this can be done (it would also be future-proofing the integration with other pull-model services). Let us know what you think.

I'm all for decoupling the two, in the meantime, since this is basically
"just another format" I'd dearly appreciate not to have to maintain my own
branch until the future-proofing is ready.
Happy to rebuild this once the plugin reporters are available though.
Thoughts?
On Tue, May 31, 2016 at 10:21 AM, Paul Parau<EMAIL_ADDRESS>wrote:

I agree, it would be nice to have Prometheus integration with Metrics.NET.
In my opinion the ideal way to add new reporters would be as separate
entities from the core metrics library (separate codebase & nuget). This
way they can have their own dependencies and lifecycles, and people that
don't want to use a certain reporter won't get it when they install the
core library.
For Prometheus, I see that you need to have an endpoint that is
periodically pulled. Currently there isn't any way of adding a new endpoint
directly to the MetricsHttpListener without directly modifying the class.
So I'm thinking that in order to implement a decoupled Prometheus reporter
we'd have to find a way to add pull-model reporters via config extension
methods.
We'll investigate how this can be done (it would also be future-proofing
the integration with other pull-model services). Let us know what you think.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Recognos/Metrics.NET/pull/27#issuecomment-222703646,
or mute the thread
https://github.com/notifications/unsubscribe/AStJnk_grnVYklxPqjwcgoKXn_pZ5KYFks5qHEPvgaJpZM4Io0G3
.

OK, I understand your point.
Let's do it this way: since we didn't get a chance to look at the PR in detail we will do that first. If there's an easy/quick way of decoupling the reporter we'll do it, otherwise we'll merge the current approach.

Sounds good. There's a few more things I want to add to that PR, I just
wanted to surface those issues before I spent too much time on a path you
guys would reject.
In particular, tags should convert to prometheus labels, types need to
convert properly (which is to say, mostly strip them because they have
subtly different semantics :( ) and naming needs to be cleaned up a bit.
Lemme know about decoupling the reporter, and I'll fix the rest before the
weekend. That works for you?
On Tue, May 31, 2016 at 6:47 PM, Paul Parau<EMAIL_ADDRESS>wrote:

OK, I understand your point.
Let's do it this way: since we didn't get a chance to look at the PR in
detail we will do that first. If there's an easy/quick way of decoupling
the reporter we'll do it, otherwise we'll merge the current approach.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Recognos/Metrics.NET/pull/27#issuecomment-222844273,
or mute the thread
https://github.com/notifications/unsubscribe/AStJnnTy2OEMZzYj-5IJAYe-K_Kiui8mks5qHLqCgaJpZM4Io0G3
.

I was looking through your code out of curiosity, and I noticed a couple minor things I just wanted to point out. Firstly, there's a typo in the PromotheusReport.cs filename. :)
Secondly, in the reporter, the current timestamp is provided by the BaseReport, so you don't need to (and probably shouldn't?) get it using DateTime.UtcNow. The values are base.ReportTimestamp and base.CurrentContextTimestamp which contain the timestamp that the report was started and the timestamp that the MetricsData was captured respectively. If you want to use one of those values, the ReportTimestamp is set immediately before StartReport() is called, and the CurrentContextTimestamp is set immediately before StartContext() is called. Judging from your code, it looks like you'll likely want to use ReportTimestamp.
Additionally, I am also in favor of adding potential support for decoupling pull-model reporters. I had considered implementing this previously but didn't have a strong enough use-case (and available time) to actually do it. However, I think it can be fairly easily and elegantly done using a few new config extensions without making many code changes to the MetricsHttpListener.

On Wed, Jun 1, 2016 at 12:51 AM, Paccc<EMAIL_ADDRESS>wrote:

I was looking through your code out of curiosity, and I noticed a couple
minor things I just wanted to point out. Firstly, there's a typo in the
PromotheusReport.cs filename. :)

Bah. Fixed.

Secondly, in the reporter, the current timestamp is provided by the
BaseReport, so you don't need to (and probably shouldn't?) get it using
DateTime.UtcNow. The values are base.ReportTimestamp and
base.CurrentContextTimestamp which contain the timestamp that the report
was started and the timestamp that the MetricsData was captured
respectively. If you want to use one of those values, the ReportTimestamp
is set immediately before StartReport() is called, and the
CurrentContextTimestamp is set immediately before StartContext() is
called. Judging from your code, it looks like you'll likely want to use
ReportTimestamp.

Actually, I really wanted CurrentContextTimestamp. Fixed.

Additionally, I am also in favor of adding potential support for
decoupling pull-model reporters. I had considered implementing this
previously but didn't have a strong enough use-case (and available time) to
actually do it. However, I think it can be fairly easily and elegantly done
using a few new config extensions without making many code changes to the
MetricsHttpListener.

Happy to do it once the Metrics.NET side is in place.
Peace, Love and Metrics,
Christian Lavoie

I built an initial implementation of support for pull-model reports on the dev branch. It's basically a way to add endpoints to the MetricsHttpListener at config time by using the .WithEndpointReporting method. Individual endpoints can be configured using MetricsEndpointReports.WithEndpointReport.
Please have a look and tell me what you think.

Looks good at first blush, any reason why you didn't convert the existing
reporters? It<s pretty much the final test to see if it's a good API or not.
On Sat, Jun 4, 2016 at 11:39 AM, Paul Parau<EMAIL_ADDRESS>wrote:

I built an initial implementation of support for pull-model reports on the
dev branch. It's basically a way to add endpoints to the
MetricsHttpListener at config time by using the .WithEndpointReporting
method. Individual endpoints can be configured using
MetricsEndpointReports.WithEndpointReport.
Please have a look and tell me what you think.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Recognos/Metrics.NET/pull/27#issuecomment-223762180,
or mute the thread
https://github.com/notifications/unsubscribe/AStJnrS6cY4tWYeTnr22nDSnntRw9YgBks5qIZwngaJpZM4Io0G3
.

--
Peace, Love and Unfinished Signatures,
Christian "Camembert" Lavoie

any reason why you didn't convert the existing reporters?

Didn't get a chance :) I'll do that soon. I'm also thinking that these changes (new reporter type + prometheus) should be rolled out as prerelease first, just to make sure everything is ironed out before releasing a stable build.

Just let me know when the first one is converted, I'll port my stuff.
On Jun 6, 2016 17:34, "Paul Parau"<EMAIL_ADDRESS>wrote:

any reason why you didn't convert the existing reporters?
Didn't get a chance :) I'll do that soon. I'm also thinking that these
changes (new reporter type + prometheus) should be rolled out as prerelease
first, just to make sure everything is ironed out before releasing a stable
build.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Recognos/Metrics.NET/pull/27#issuecomment-224095670,
or mute the thread
https://github.com/notifications/unsubscribe/AStJnpyOyFxzgzAc4cgLrnBGCtuPZxTiks5qJJJxgaJpZM4Io0G3
.

I just finished another iteration, you can have a look on the dev branch. Among other changes, configuration is now done at HTTP endpoint level. I've also converted the existing endpoints to the new model.

How do you expect non-default endpoints to be registered?
Is it worth it to think through registering new endpoints without having
access to the source? For the record, it's not a concern I have, but
figured I'd throw it out there.
Otherwise, LGTM.
On Thu, Jun 9, 2016 at 1:50 PM, Paul Parau<EMAIL_ADDRESS>wrote:

I just finished another iteration, you can have a look on the dev
branch. Among other changes, configuration is now done at HTTP endpoint
level. I've also converted the existing endpoints to the new model.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Recognos/Metrics.NET/pull/27#issuecomment-224973735,
or mute the thread
https://github.com/notifications/unsubscribe/AStJngbLI8qu6NYvbw-8OvS2B8B9Y7tYks5qKFJQgaJpZM4Io0G3
.

--
Peace, Love and Unfinished Signatures,
Christian "Camembert" Lavoie

I think a good way to register new endpoints is to do it similarly to the default ones: create an extension method for MetricsEndpointReports (e.g. WithPrometheusReport etc.) where you call the MetricsEndpointReports.WithEndpointReport (esentially providing a way to generate a MetricsEndpointResponse). Then you can call the extension method in the configuration action parameter for every HTTP listener you want (WithHttpEndpoint - you can look at the SamplesConsole project for an example which calls MetricsEndpointReports.WithEndpointReport directly).

Is it worth it to think through registering new endpoints without having access to the source?

I think it's best to keep it simple for the moment :) we'll extend the mechanisms for defining and new registering endpoints when/as the need arises.

I've released v0.4.0-pre with support for endpoint reports. I've also documented it here https://github.com/Recognos/Metrics.NET/wiki/Http-Endpoint-Reports.
What do you think about making your own repository for the Prometheus reporter?  (e.g. like the Google Analytics reporter https://github.com/hinteadan/Metrics.NET.GAReporting)
I know I mentioned this before but new reporters should be separate from the core Metrics.NET (I know that elastic search and influxdb reporters are already in there, we'll try to separate them in the future). As a separate project (both repository and nuget package), a custom reporter would have it's own lifecycle and it would be much easier to maintain issues and releases. Adding reporters to the core just bloats it unnecessarily (just imagine having 10 different reporters inside the core, each with its own dependencies).
Let me know what you think.

Would you be able to provide a sample implementation of the EndpointReport?

With apologies for disappearing, that would make my life easier too.
On Mon, Jul 25, 2016 at 10:17 AM, Keith Andrew Hill <
<EMAIL_ADDRESS>wrote:

Would you be able to provide a sample implementation of the EndpointReport?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Recognos/Metrics.NET/pull/27#issuecomment-234965903,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AStJnorPlVRg_N1wwqj_Mupq6fIIGTaQks5qZMVrgaJpZM4Io0G3
.

--
Peace, Love and Unfinished Signatures,
Christian "Camembert" Lavoie

Looks like doing something like this works.
config.WithEndpointReport("/prometheus", (d,h,c) => new MetricsEndpointResponse(PrometheusReport.RenderMetrics(d,h), "text/plain"))

Problem is that it only works with HttpEndpoints. The Owin adapter is still using older logic. :(

I was curious on this PR. Do we think this will be merged at some point? What will it take to merge it?
I am super excited to try this feature out.

Problem is that it only works with HttpEndpoints. The Owin adapter is still using older logic. :(

@vic10us Indeed, the Owin adapter does not support this new model, nor does the Nancy adapter for that matter. It's something I overlooked & I intend to look into it as soon as I can.
The example provided by @vic10us is correct. Support for endpoint reports is still in beta because I wanted to see an actual implementation of such a report (besides the built-in reports which were already ported), in case changes were still needed to the API (they might still be needed - re: adapters).
In the end, to merge the PR, it should be moved to a separate csproj & made to use the new endpoint reporter model (which was build specifically to allow such separation). @clavoie-acquisio you can try to port it now, you can wait until after I settle the issues with the adapter or if you want I can do it myself when I get some time. Or, if you want to, you could make & maintain a separate repo & nuget package (which would have the advantages I stated in my post above). It is up to you :)

It's a question of finding the time. I aim to get a new repo "ASAP", but
I've been beyond swamped. First to get to it? ;)
On Fri, Jul 29, 2016 at 11:20 AM, Paul Parau<EMAIL_ADDRESS>wrote:

Problem is that it only works with HttpEndpoints. The Owin adapter is
still using older logic. :(
@vic10us https://github.com/vic10us Indeed, the Owin adapter does not
support this new model, nor does the Nancy adapter for that matter. It's
something I overlooked & I intend to look into it as soon as I can.
The example provided by @vic10us https://github.com/vic10us is correct.
Support for endpoint reports is still in beta because I wanted to see an
actual implementation of such a report (besides the built-in reports which
were already ported), in case changes were still needed to the API (they
might still be needed - re: adapters).
In the end, to merge the PR, it should be moved to a separate csproj &
made to use the new endpoint reporter model (which was build specifically
to allow such separation). @clavoie-acquisio
https://github.com/clavoie-acquisio you can try to port it now, you can
wait until after I settle the issues with the adapter or if you want I can
do it myself when I get some time. Or, if you want to, you could make &
maintain a separate repo & nuget package (which would have the advantages I
stated in my post above). It is up to you :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Recognos/Metrics.NET/pull/27#issuecomment-236209442,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AStJnp7nY_zvoSz4fnGQcaW8ukjULgaPks5qahoigaJpZM4Io0G3
.

--
Peace, Love and Unfinished Signatures,
Christian "Camembert" Lavoie

I'm ready to do it. Gimme a little time. :)

No rush :) I also don't know when I'll get to fixing the adapter issues, I hope soon. Hopefully there won't be any more major API changes.

I've decided to push this to my own repo https://github.com/DEXSinc/Metrics.NET.Prometheus and created another Nuget Package Metrics.NET.Prometheus it's built against the pre-release and has the same numbering scheme. currently 0.4.1-pre

Wow, that was quick. Great job!

Thanks Keith.
On Sat, Jul 30, 2016 at 6:36 AM, Paul Parau<EMAIL_ADDRESS>wrote:

Wow, that was quick. Great job!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Recognos/Metrics.NET/pull/27#issuecomment-236358184,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AStJnrHi5gcFGg5d60q83diTiN6hW6VBks5qaykUgaJpZM4Io0G3
.

--
Peace, Love and Unfinished Signatures,
Christian "Camembert" Lavoie

I created a page in the wiki (also linked from the readme) with a list of external reporters and added a link to the Prometheus reporter. Since it was implemented as a separate repository, I'm closing this PR.
I've created a new issue #37 for adding adapter support.
Thanks again @clavoie-acquisio & @vic10us !


Add useCurrentScope to Resource effect

Supersedes https://github.com/arybczak/effectful/pull/28
As first suggested here: https://github.com/snoyberg/conduit/issues/436#issuecomment-616345570

Of course there might be a more general way of accessing "shadowed" effects further up the list, or it might not even make sense at all for this package, I haven't put any thought into it :)

I'm reluctant to accept this based on Michael's comments in the original PR. What's the use case?

I've been using that for some time as part of a rendering engine which requires prompt destruction of resources (and not just resources, actions which require begin/end pairs) over many (nested) scopes, it's very handy to be able to do this destruction automatically, while still being able to utilize the global scope.
I don't think that Michael's comments about it being hard to get right are super-valid, after all the (eventual) implementation for ResourceT is 2 very simple lines: https://github.com/snoyberg/conduit/issues/436#issuecomment-616345570, and this one is pretty succinct too (I hope I've got the Eff side of it down).
This can certainly get misused, but I think that's more an argument for clear documentation (happy to add to this PR) rather than total exclusion.
If this won't make it in, would it at least be possible to expose the Resource constructor so that I might publish this in a separate package?

It looks like getInternalState and runInternalState are enough and if they are in the public API (in the "low level API" section or something), you can write useCurrentScope yourself.
Would that be satisfactory?

Yeah, that'd be great.
On Mon, Jul 26, 2021 at 9:51 PM Andrzej Rybczak @.***>
wrote:

It looks like getInternalState and runInternalState are enough and if
they are in the public API (in the "low level API" section or something),
you can write useCurrentScope yourself.
Would that be satisfactory?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/arybczak/effectful/pull/29#issuecomment-886720991,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGRJXATD5WB5GBIDX76TILTZVR3DANCNFSM5A7XIB6A
.

30a28558429bbb20660c17a36d5680afe49f2300 should do the trick :slightly_smiling_face:

Thanks!
On Tue, 27 Jul 2021, 2:17 am Andrzej Rybczak, @.***>
wrote:

30a2855
https://github.com/arybczak/effectful/commit/30a28558429bbb20660c17a36d5680afe49f2300
should do the trick 🙂
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/arybczak/effectful/pull/29#issuecomment-886922074,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGRJXEWEXHXIPUHHCSGLKLTZWRBXANCNFSM5A7XIB6A
.


Models with same name can be loaded but not stored in the database

If model has same name than an existing one, print an error message.
Also we need to handle the reloading of models, possibly just by giving the same error message, thus forcing to delete the old one first.

Resolved: Prevents loading if same name already exists


Need solution - top_posts and getMediasByTag result

For now, this code return 4 images for 'bybakery_ru' hashtag:
$Instagram = new \InstagramScraper\Instagram();
$media = $Instagram->getMediasByTag('bybakery_ru');
foreach ($media as $key=>$item) {
	echo 'getLink().'">';
	echo $item->getId();
	echo '
';
}

But there are at least 7 images with this hashtag. Proof:
https://www.instagram.com/explore/tags/bybakery_ru/
So why InstagramScraper doesn't return all images?
I investigate this issue:
getMediasByTag function use this code to get images:
$nodes = $arr['tag']['media']['nodes'];
But $arr['tag'] also contained 'top_posts' key. This key has "nodes" too with all 7 images.
I don't understand logic, why ['media']['nodes'] contained 4 images, why ['top_posts']['nodes'] contained 7 and how getMediasByTag must be modificated to return all images in all situations.
Do you have any solution?

Just add
´´´
$first = false;
if ($maxId == '') {
$first = true;
}
//...
$nodes = $arr['tag']['media']['nodes'];
if ($first == true) {
$nodesTop = $arr['tag'['top_posts']['nodes'];
$nodes = array_merge($nodes,$nodesTop);
}
´´´
.. if you need top posts. But be aware that some top_posts may be duplicates of media-posts.

@Sieboldianus, thanks for code.


Support Cisco ACS

Same as other sourcetypes.  I will check with my management if we can share logs out.  Also, according to my network admins ACS is EOL but is replaced by Cisco ISE so they would be ISE logs.

Yes this is certainly EOL but its going to hold on for quite some time for a number of reasons. I see plenty of customers with ACS and ISE that moved to ISE for wifi/wan network auth but continue to use ACS for admin auth to hardware

ACS sample logs.txt
Here are some sample syslogs for ACS 5.8

This is a great start do you have any that show an authentication activity?

Here are some more samples.

CRLF are newlines. ACS logs can arrive as either UDP or TCP based syslogs.