feat: Characterization of proper maps for Hausdorff and compactly generated spaces

A continuous map whose codomain is a Hausdorff and compactly generated space is proper if and only if the preimages of compact sets are compact.
A Hausdorff and weakly locally compact space is compactly generated.
A Hausdorff and sequential space is compactly generated.

I'll let you a week to review and then I'll review myself

I'll let you a week to review and then I'll review myself

This was a month ago, so I'm going to review it now.

@ADedecker Is there any progress with reviewing different definitions? I think that we should fix one of them, add as a typeclass, and use it here. We may need to add all three in the future. If Definition 1 is enough for our immediate needs (i.e., implies all the properties we're going to prove about these spaces in the near future), then I suggest to go with it, because this will make our theorems that assume [CompactlyGeneratedTopology X] stronger.

Do you need help with the git?

Yes I am completely lost. I wanted to solve the merge conflict but I messed up the solving because I added something which in fact had been moved to another file. Anyway I've been trying to revert to commit bd60edd but I can't find a way to do it.

Is there anything of value in that branch after bd60edd? If not you can do git reset HEAD~N where N is the number of commits you want to revert, then do git push --force. WARNING: You will lose the content of the commits you've reset (well, not quite lost but much more hardly accessible).

Thank you very much, that's exactly what I needed! I had tried reset but then I could not push, I didn't know about the force option.

With great power comes great responsibility. You should use --force sparingly as it can overwrite other people's work if you collaborate on the same branch.

Yes I looked it up and saw this could happen. Thank you for your help!

UPD: it looks like we're going to use Definition 2 in another series of PRs. Not sure if we need more than one definition right now.

Thanks for the review!
bors r+


List the PackageIdentifier of the package to be added to automation
Chocolatey.ChocolateyGUI
Details about the application
https://github.com/chocolatey/ChocolateyGUI/releases

https://github.com/chocolatey/ChocolateyGUI/issues/943

Added to https://docs.google.com/spreadsheets/d/1dnTRsRI8JdTyHibbBy8b1-Odd1gHJRjScm_KWUQ-jHk/edit?usp=sharing


ninjaben/matlab-support:latest

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

...
...

What is the Problem?


Check all uploaded data is ingested into SM

We get a LOT of files uploaded into SM - and later changes do check that all files in a search path are covered by a sample map - but a script to double check this would bring a lot more confidence to make sure nothing is slipping through the cracks.

Another awesome thing would be to generate a list of files to delete once they've been reprocessed.


WIPP Sample Integration

Waste Isolation Pilot Plan (WIPP) Sample Integration

Hey @JAG-UK  receiving message file reformatting under integrity checks . . .
The above says 2 files reformatted then states 2 files must be reformatted. . . not sure which two files or why this message is being received.  Wondering if you have additional insight.
Run export PYTHONPATH=samples:${PYTHONPATH}
export PYTHONPATH=samples:${PYTHONPATH}
pycodestyle --format=pylint archivist_samples
python3 -m pylint --rcfile=pylintrc archivist_samples
black archivist_samples
modified=$(git status -s | wc -l)
if [ $modified -gt 0 ]
then
echo "there are $modified files that must be reformatted"
exit 1
fi
shell: /usr/bin/bash -e {0}
env:
pythonLocation: /opt/hostedtoolcache/Python/3.9.6/x64
LD_LIBRARY_PATH: /opt/hostedtoolcache/Python/3.9.6/x64/lib

Your code has been rated at 10.00/10
reformatted archivist_samples/wipp/main.py
reformatted archivist_samples/wipp/run.py
All done! ✨ 🍰 ✨
2 files reformatted, 51 files left unchanged.
there are 2 files that must be reformatted
Error: Process completed with exit code 1.

I took this locally and tried running format, got the same issue, I ran it with -v to get more details and it then simply worked, checked the diff and only very limited changes occured with little to no integrity issues - I have submitted the changes
Everything seems okay but please do verify as well

Thanks for the info. . . I make the format changes locally to test on my end and go from there.


Add pygraphviz

Thanks!
Looks like appveyor is failing by not being able to find a header:
C:\Users\appveyor\AppData\Local\Programs\Common\Microsoft\Visual C++ for Python\9.0\VC\Bin\cl.exe /c /nologo /Ox /MD /W3 /GS- /DNDEBUG -INone -IC:\Miniconda\envs\_build\include -IC:\Miniconda\envs\_build\PC /Tcpygraphviz/graphviz_wrap.c /Fobuild\temp.win32-2.7\Release\pygraphviz/graphviz_wrap.obj
graphviz_wrap.c
pygraphviz/graphviz_wrap.c(2954) : fatal error C1083: Cannot open include file: 'graphviz/cgraph.h': No such file or directory
error: command 'C:\\Users\\appveyor\\AppData\\Local\\Programs\\Common\\Microsoft\\Visual C++ for Python\\9.0\\VC\\Bin\\cl.exe' failed with exit status 2

C:\Miniconda\conda-bld\work\pygraphviz-1.3.1>if errorlevel 1 exit 1

I imagine @nividic doesn't need this to work on win right away, so what about skipping the win platform?

Nice!


Enhanced documentation

Please specify for the config file(s) involved what user, group and permission should be;

wrong user/group or too strict mode and it won't work
wrong user/group or too loose mode and it is insecure
E.g. what should be the settings for /etc/pam-weblogin.conf?

Is it documented somewhere how the system gets uids and other attributes (GECOS?) from the identity store? Have searched around and not finding this.

it seems like it should be in scope? if all you are doing is validating passwords you're leaving a large chunk of the identity question unaddressable. it's not like there's some separate mechanism for attaching users created when they are logged in for the first time to consistent UIDs and GIDs. or is there? what are users of this PAM module doing for that?

This module doesn't create users, it just authenticates. The users must already be present in any available backend e.g. LDAP. We (the SURF SRAM project) has means to provision user identities into LDAP (https://github.com/SURFscz/plsc) so that the relying parties have a way to do OIDC authentication for those users without knowing a password. I must admit, this provisioning tool does not create uid and gids. That is totally up to te relying party when consuming and grooming the directory.

okay, but when a user is logged into a machine for the first time via LDAP via SSSD, some data structures are created to match that user. SSSD's identity store (I admit I do not understand this well) does not create an entry in /etc/passwd, but it does know things about the user from directory services such as uidNumber and gidNumber.
suppose you have a "fresh" system with no users created in it beyond root. you install pam-weblogin to permit users to authenticate into the machine. user "bob" attempts to log in with OIDC credentials. what happens after the OIDC provider verifies that "bob's" password is correct?

We should probably create some documentation on how to set up a system using pam-weblogin beyond just the module itself.

@jdkruzr Please open a separate bug/support request for your question.  This bug is about @pneerincx original issue (not clear what the file permissions of the config file shiould be)


sync api version

What changed?

Why?

How did you test it?

Potential risks

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I'm going to try to fix the automatic thing now, please don't merge this, let's use the automated one


Chinese font occlusion problem

When editing a book in Chinese, there will be a problem of font blocking.

For the record, regardless of the font, the original is what it is


Doesn't work with x.com

Twitter is now redirecting to x.com instead of twitter.com and the current extension does not work with x.com added as a pwa

Fixed, version should be available soon. I'm still going to call it twitter though ;p


Update report-paginated-data-retrieval.md

correcting a typo in 'parameter' in the note, under Data source types

@HeidiHasting : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@HeidiHasting Thanks for this fix


support new actions during interactive learning

It would be great to support creating new actions during interactive learning. The user would come up with the new action on the spot, but would have to back later and ensure there is a template defined (for utter actions) or there is a custom action registered at the appropriate endpoint.

whenever i tell interactive learning for a new actions it does this:
? What is the next action of the bot?  0.02 family_form
? Do you want to stop?  Continue
? Export stories to (if file exists, this will append the stories)  data/stories
.md
? Export NLU data to (if file exists, this will merge learned data with previous
training examples)  data/nlu.md
? Export domain file to (if file exists, this will be overwritten)  domain.yml
? Could not load existing NLU data, please specify where to store NLU data learn
ed in this session (this will overwrite any existing file)  data/nlu_interactive
.md
then the same happens again and again
How should i train it for a new action

I'm not sure what's happening in your case. Can you please create an issue? Please include the version you are running, your domain file, and steps to reproduce.

? Export NLU data to (if file exists, this will merge learned data with previous
training examples) data/nlu.md
? Export domain file to (if file exists, this will be overwritten) domain.yml
? Could not load existing NLU data, please specify where to store NLU data learn
ed in this session (this will overwrite any existing file) data/nlu_interactive
.md

Same thing happens to us. What is the purpose of saving the NLU data twice, to different files? What is the difference between "learned data" and "data learned in this session"? Why in the first save does it save an empty file every time (rather than merging)? Why can't the agent find the nlu.md file that it's been using all along? Why does the first save create an empty file, and the second save create a file consisting solely of the new session?


953: do not stop chain processing

@essepuig Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@essepuig Thank you for signing the Contributor License Agreement!

Congrats @essepuig on your first contribution :)

Codecov Report

Merging #954 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #954   +/-   ##
=========================================
  Coverage     71.24%   71.24%
  Complexity      628      628
=========================================
  Files           121      121
  Lines          2793     2793
  Branches        289      289
=========================================
  Hits           1990     1990
  Misses          626      626
  Partials        177      177

Continue to review full report at Codecov.

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


Basic static type inference

While full blown type inference may not be feasible, it should be quite doable to catch at least some basic things, like:
selectattr | length

Where Ansible will complain with something like:
The conditional check 'dbms_postgres_reconfig.results | selectattr('changed'
) | selectattr('restart_required') | length' failed.

The error was: Unexpected templating type error occurred on ({% if dbms_postgres_reconfig.results | selectattr('changed') | selectattr('restart_required') | length %} True {% else %} False {% endif %}
):
object of type 'generator' has no len()

The error appears to be in '/home/vagrant/ansible/roles/dbms/tasks/main.yml': line 124, column
 3, but may\nbe elsewhere in the file depending on the exact syntax problem.

The offending line appears to be:
- name: restart postgres if our changes necessitate it
  ^ here"

Basically we can infer that the selectattr filter is a generator, and that length will not work on a generator, but needs a list.

Another blunder that we could spot with this would be
- name: do something
  when: "ansible_distribution == 'Debian' and ansible_distribution_version >= 11"
  ...

which yields
 AnsibleError: Unexpected templating type error occurred on ........ '>=' not supported between instances of 'AnsibleUnsafeText' and 'int'


Run lint using python2 instead of python.

On some distros (such as Arch Linux) python points to python3 rather than python2. This causes issues with the lint command, as cpplint.py doesn't seem to work on Python 3.
Simply replacing python with python2 won't work, as the default Windows installation doesn't have python2.exe, just python.exe, so a special case is needed.
Running plain python should work on macOS as well.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks!


prepare internals for robinhood integration

working on robinhood integration on the backend. if someone can do the frontend that'd be dope. I was thinking of a section below RECOMMENDATIONS called INVESTMENTS and under that two labels (Portfolio and Buying Power) which I would fill in on the backend (also the ability to open in new window like FUNDS). Possibly also an interchangable login/logout button below. I do understand this is intended to be a primarily offline but I think Robinhood integration could really make it an all-in-one application.

Hey apologies for the late response there, I've been pulled into other projects. This is an awesome initiative but I'm trying to keep Fortunæ agnostic from investment platforms for the time being.


Missing class in the dagger example ?

I cannot find TasksRepositoryModule.class in the dagger example...is it there and I'm not seeing it ? also the search doesn't return anything

You need to build it.


Missing dependency

Hi,
I am trying to use this package and am getting the following error:
Module not found: Error: Can't resolve 'fs' in...

from this import in the index.js file

It doesn't seem to be listing in package.json either. Am I missing something?
Thanks!

The library is developed to be used with node.


Update acknowledgement link

The original dribbble is down. Use web archive?: https://web.archive.org/web/20170711185308/https://dribbble.com/shots/1509889-Free-Social-Media-Icons

Fixed in 6895d570b05efe55257cd020d5029714dfd161c8


[kilt] RPC node didn't properly restart when upgraded

Migrated from: https://github.com/dwellir-public/ops/issues/143
When Jibril upgraded, the node started, but the Relaychain didn't import.
A first restart via ssh didn't get it to start either.
Only after a second attempt, the Relaychain started imports.
I think this might be due to the node starting to accept remote connections very early, serving nearly 2000 connections in a short moment.
Possibly we should have an action that raises a firewall letting a node come back into sync.
We need to have a super robust situation for our ops staff. This is why we should think extra much on how we can ensure upgrades work well and that restarts succeed.

Has not been seen again in quite a while. Closing this issue.


'@microsoft/file-browser' package not providing share/move/search functionality.

'@microsoft/file-browser' package is not updated from last two year. I want the share, move and search functionality there. Do i can customize this package to extend the package functionality? Can you help me on this?

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


Implementation of case function.

Description
Implementation of the case function
Issues Resolved
https://github.com/opensearch-project/opensearch-spark/issues/645
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@lukasz-soszynski-eliatra  please ping me when this is ready for review
thanks

I'll do my best to make corrections today.

@YANG-DB
Please take a look at the commit. I added more tests and documentation.

@YANG-DB does this syntax match what we expected? From the issue description, the valid synatx is:
CASE(<predicate1>, <value1>, [<predicate2>, <value2>, ...] [ELSE] <default>)

An valid example:
source = my_index
| eval status_category =
    case(status_code >= 200 AND status_code < 300, 'Success',
         status_code >= 300 AND status_code < 400, 'Redirection'
         status_code >= 400 AND status_code < 500, 'Client Error'
         status_code >= 500, 'Server Error',
         'Unknown')
| stats count() by status_category

But the implementation we merged is totally different as:
CASE(<predicate1>, <value1>[, <predicate2>, <value2>, ...] [ELSE <default>])

The example above needs rewritten to
source = my_index
| eval status_category =
    case(status_code >= 200 AND status_code < 300, 'Success',
         status_code >= 300 AND status_code < 400, 'Redirection'
         status_code >= 400 AND status_code < 500, 'Client Error'
         status_code >= 500, 'Server Error'
         ELSE 'Unknown')
| stats count() by status_category

Note, the , after 'Server Error' is removed and ELSE keyword is required for this case.
Please double confirm the current syntax is by designed.


What does safe-mix use for?

I can't find any usage of this crate, but it in many module's deps.

It was used by frame_system but not anymore. And copy & paste by all other pallets. https://github.com/paritytech/substrate/pull/4656


.NET core is incorrectly assumed when building a solution targeting .NET framework.

As much as I like something like this it expects a project.json file when it should really check if any projects in the solution targets .NET core to begin with. If none are found then it should assume .NET framework and not require that file to exist just to build. I hate having to start up VS2017 which is a memory hog just to build my stuff.

This extension doesn't support .Net Framework builds. The "build" command should appears only when you are working with dotnet core projects. It also doesn't load "project.json" projects. So, if you can see the build option, and when you click it ask for the project.json file, it seems a problem with the dotnet core SDK you have installed in your machine.
In order to be able to analyze if the error is actually in the extension, Would you mind to give me more details about your solution an project file?

Sure, the project files and solution files are here to clone to test: https://github.com/Elskom/Els_kom_new/ at branch add-crc-xml-converter

Solved in next release: Now the solution context menu doesn't show the dotnet core commands if the solution doesn't have any CPS (dotnet core) project.

@fernandoescolar is there a way to extend a vscode extension though to add a .NET framework, and a Mono build option? Although I think it would be nice if all 3 build options was added to this extension and have "Build/Build Solution (.NET Core)", "Build/Build Solution (Mono)", and then "Build/Build Solution (.NET Framework)" then disable the options that do not apply to the solution.


Simplify Operator usage

The benefits of this simplified Operator usage are

simpler instance control through enums
less code for declaring Operator instances

The downsides are

No generic type for operation return type anymore

@Shredder121 What do you think of this solution approach?
In addition to Date/Time types also Spatial types (JTS vs Geolatte) make Operator type usage difficult. It seems to be useful for String, Boolean and Number.

I think it is at least not as bad as I thought, although I know of some improvements.
Since there is no generic return type anymore, I think it is important to realize there must be some validation.
Maybe there can be at creation time (in the static factory methods sounds good enough) a validation that the type of the operator matches (roughly for Date/Time and Spatial types) the type the expression wants.
This roughly translates into an interface like this:
public interface Operator extends Serializable{

    String name();

    Class<?> getType();

}

This allows us to query the Operator like this:
public static <D extends Number & Comparable<?>> NumberExpression<D> create(Class<? extends D> type, Operator op, Expression<?> one) {
    Preconditions.checkArgument(op.getType().isAssignableFrom(type));
    return new NumberOperation<D>(type, op, ImmutableList.<Expression<?>>of(one));
}

This prevents operations like this:
NumberExpression<Integer> num = Expressions.numberOperation(Integer.class, Ops.UPPER, one);

I like fail-fast behavior better than undefined behavior at runtime, since this only fails at AbstractNumberType.getValue():40, whereas it could have failed with any ClassCastException elsewhere.
Of course, as for third-party types, this is really difficult, but at least this way we don't lose all safety that generics 'promised' us.
What do you think?

Sounds good, I will do the necessary changes.

@Shredder121 How about now?

Does com.querydsl.scala.Operations.number() declare a new operator? Or how do I read that?

Operations contains factory methods for Operation creation.

I fixed now the remaining issue on the Scala side. Expressions on the Scala side use also primitive types, since Scala supports primitive types as generic type parameters.
The fix normalizes the type of OperationImpl to the wrapper type before comparison.

Ah I see, that indeed sounds like the problem.

For consistency it would be nice to have the same constraints for expression types for both Java and Scala based Querydsl usage, but on the other hand they are different languages and this way Querydsl matches better the expressivity of Scala with a little added complexity in some places.

Since there is no type parameter anymore, we now have a few SuppressWarnings less.

@Shredder121 Do you think this is good to merge now, or do you still see other improvements that could be applied?

I think we are there. I don't know of any further (important) improvements.
So then I will merge it, if you didn't come up with anything in the meantime?
You can merge it as well for once, if you think it's good.

I might still make some changes to the Scala operation usage, but that can be independent of this PR.


Add Alacritty

Alacritty from https://github.com/jwilm/alacritty
This is my first scoop script so if I'm doing anything wrong please do point out.

Śhould be moved to extras since it's GUI application. Also why to use working directory parameter when it's by default?? Based on their documentation:

On Windows the config file is located at:
%UserProfile%\alacritty.yml

But maybe these applications have some exlusions, since cmder is in main too.

Opened in scoop-extras: https://github.com/lukesampson/scoop-extras/pull/1514


DPG, investigation, CADL feature to M4 schema

Check features in https://github.com/microsoft/cadl/blob/main/docs/tutorial.md, and see if current M4 schema supports that. If not, how we might do to augment the M4 (sketch would do).
Note that some feature is only for compiler, e.g. "type alias". One can ignore them.
But some is uncertain, like "interface".
Discuss and have a write-up.

Later, we might want to add some CADL file as tests (e.g. see how "interface" would affect emitter).

one may use https://github.com/Azure/autorest.java/tree/cadl-code-model/cadl-extension

https://microsoftapc-my.sharepoint.com/:x:/g/personal/haolingdong_microsoft_com/EWafV27KftVFl7_dSOUcDb4BcnVqDPKEgwb2IyuWP4E8zA?e=TxHFSJ

Done P0 / P1.


Address vhtml#11: Add ability to directly set innerHTML of a component using the dangerouslySetInnerHTML attribute.

Should take care of #11.
New Size: 630B.
50B is kinda significant :cry: but I consider this a useful feature.

+1 for this

I think we can slim this down a bit without losing much functionality, but inlining the property name and skipping one of the conditionals:
if (attrs && attrs.dangerouslySetInnerHTML) {
    s += attrs.dangerouslySetInnerHTML.__html;
}

Good call on dropping the extra check, no need for __html to be valid to add it. I didn't inline the property because that seems to cost 1B more than declaring let setInnerHTMLAttr = 'dangerouslySetInnerHTML'.
However I did find 4B that can be cut out regardless (b8199d36) which should helps a little.

@ithinkihaveacat - I'm going to merge Peter's here, since it's substantially smaller. I personally don't care what we return for the case of a boolean dangerouslySetInnerHTML value - since that actually throws in React most folks wouldn't be using it.


active_users.py:23: Перенести классы в отдельные файлы 53

The puzzle 899-adc26f6b from #899 has to be resolved:
https://github.com/blablatdinov/tg-quranbot/blob/befd7a82bfe71bd69d99ed14f929dfb2dfae4157/src/srv/users/active_users.py#L23-L23
The puzzle was created by Ilaletdinov Almaz on 20-Jun-24.
role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 899-adc26f6b has disappeared from the source code, that's why I closed this issue.


catch block arguments

In a catch block, if a thrown type has a non-trivial copy constructor, cpy isn't passively invoked for the user. See: https://godbolt.org/z/vfxPx5cWb
In the case of throwing a constructed object, the static constraint on references prevents this code from compiling as well: https://godbolt.org/z/T5o6TbWPz

Yes, need to specify how to support both cpy and other bindincs in the [safety] feature.
An inaccurate error message but not a borrow checker bug. The exception object doesn't have static storage duration, so we can't bind static borrows to the exception object. I don't know what to do here. Maybe there needs to be a /throw lifetime? The exception object doesn't live on the stack, and it's not static... Is it sufficient to treat it as if it has automatic storage duration? I'd have to do some experiments to see if the __cxa_free_exception can occur prior to the closing brace of a catch-handler.


Addition of cycle detection in linked list

💥 Proposal
Addition of hare and tortoise algorithm for cycle detection in linked list
Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
YES

I would like to do it as part of GSSOC 2021

I am a GSSOC'2021 participant. Please assign me this work.I am good at C++,C,Java and python.Also good at algorithm and data structures.I will write programmes with detailed explanation and proper commenting.

I would like to implement the above question in Python, I will write neat and clean  commented code
please assign me this issue under GSSoC 2021

I would like to implement the above problem in Java under GSSoC 2021. I would add comments at regular intervals and write clean code. Please assign this issue to me.


Scope app token to only this repo for security

https://github.com/dependabot/fetch-metadata/pull/442 bumped to a new version of this action which now supports a "repositories" key that scopes the token to the designated repositories.

I don't think this is a big deal either way, but it can't hurt and since I saw the note about it while merging #442, thought I'd throw this up there as well.

CI won't check this, we'll only find out if it's working when we try to build/do a release... but again, not a big risk because easy to revert if we run into problems.


fix: git:// breaks OSM

Adds "git://" to the chain of .replace() calls in the GitHub URL normalizer. This is a kludge, but I think it should cover 100% of cases, as I can think of no circumstances under which a repository can be installed as git://github.com/foobar, but not as https://github.com/foobar.

not really a kudge, makes all sense in the normalize step :)


Mockito extension is ignored when running with Spek

Hi.
There is an incubating feature of mockito framework that allows to mock previously unmockable entities like final classes.
Here is a sample project that demonstrates the issue: https://github.com/vokilam/test-spec-mockito/tree/master/app/src/test/java/com/vokilam/test_spec_mockito.
The problem is that this feature is ignored when running with Spek plugin (see ExampleUnitTestKotlinSpek.kt). Both java junit test (ExampleUnitTest.java) and kotlin junit test (ExampleUnitTestKotlin.kt) works as expected (final class is mocked).
In case of Spek test/resources/mockito-extensions/org.mockito.plugins.MockMaker file is ignored and I got the following output:
Testing started at 18:56 ...
May 07, 2017 6:56:50 PM org.junit.platform.launcher.core.ServiceLoaderTestEngineRegistry loadTestEngines
INFO: Discovered TestEngines with IDs: [spek]
org.mockito.exceptions.base.MockitoException:
Cannot mock/spy class com.vokilam.test_spec_mockito.MainActivity
Mockito cannot mock/spy because :
 - final class

I'm not 100% sure this problem is connected with Spek plugin, however I wanted to identify a problem and to direct it to a corresponding tracker.

In your kotlin unit test please remove @Throws(Exception::class), it should fail as well. Can you try adding :assembleDebugUnitTest as a before run task for the run configuration?

Removed @Throws annotation – same behavior as before: kotlin unit test is passing, final class is mocked (no error messages from mockito).
Added :assembleDebugUnitTest task to default configuration for Spek (as described here). Before that I did it manually. Same behavior for spek test – mokito extension is not working.

That's weird, I'll have to test with an android project. It's working with a non-android one. I'll get back to you if I have leads.

My environment is
Android Studio 2.3.1
Build #AI-162.3871768, built on April 1, 2017
JRE: 1.8.0_112-release-b06 x86_64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
macOS Sierra 10.12.4

Anything I can help you with?

@vokilam Sorry, I haven't really have started the investigation - I'm a bit busy with work stuff right now.

@vokilam i'm setting up my android environment, by any chance is the test passing when running from the command line?

Cannot check until tomorrow, sorry. Which task should I run?
On Sun, May 21, 2017, 15:32 Ranie Jade Ramiso<EMAIL_ADDRESS>wrote:

@vokilam https://github.com/vokilam i'm setting up my android
environment, by any chance is the test passing when running from the
command line?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/raniejade/spek-idea-plugin/issues/36#issuecomment-302933991,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AApvSRs9_W7imk1bVaixJMY8LCyma9Gwks5r8C71gaJpZM4NTMun
.

I'm not sure - not an android guy, probably the task that runs the unit tests :)

Let me check, I will get back

./gradlew test – completes successfully, reports are in app/build/reports/tests folder. I had to annotate test class with @RunWith(JUnitPlatform::class) so it can be picked by gradle.

So most likely the problem is within the IDE. While using @RunWith(JUnitPlatform::class) is the test passing when running from the IDE? If it passes, the problem is with the spek plugin, if not most likely it's an IDE issue.

I'm a bit confused with results I get. I will check in IDEA also and get back :)

Okay, here are my results.
Set up IDEA
I set up a gradle based java project in IDEA with junit 5.
I created a classic java test and spek test. Each test mocks a public final class MyClass.
MyClass test = Mockito.mock(MyClass.class);

I created a file test/resources/mockito-extensions/org.mockito.plugins.MockMaker with following line (configure Mockito extension – now it should mock final classes):
mock-maker-inline

Results

Run ./gradlew junitPlatformTest – two tests succeeded
Run java test from IDE – success
Run spek test from IDE (spek plugin) – success
Comment mock-maker-inline line and rerun previous tests – all failed with following exception (as expected):
org.mockito.exceptions.base.MockitoException:
Cannot mock/spy class MyClass
Mockito cannot mock/spy because: final class

Set up Android Studio
I set up a gradle based Android project with junit 4.
Same setup for test classes. Spek test class has @RunWith(JUnitPlatform::class) so it can be visible when testing from console.
Same setup for mockito extension.
Results

Run ./gradlew testDebugUnitTest – results are in build/reports/tests/testDebugUnitTest, two tests succeeded
Run java test from IDE – success
Run spek test from IDE – failed with MockitoException: Cannot mock/spy class MainActivity – that was not expected
Comment mock-maker-inline line and rerun previous tests – all failed with exception.

Set up Android Studio JUnit 5
I set up a gradle based Android project with junit 5.
Same setup for test classes. @RunWith(JUnitPlatform::class) annotation is removed because test is already visible for console command.
Same setup for mockito extension.

Run ./gradlew junitPlatformTestDebug – two tests succeeded
Run java test from IDE – success
Run spek test from IDE – failed with MockitoException: Cannot mock/spy class MainActivity – that was not expected
Comment mock-maker-inline line and rerun previous tests – all failed with exception.

Summing up: Mockito extension is not applied when test is run from Android Studio via Spek plugin (junit4 and junit5). Same setup for IDEA (junit5) works fine.

Hi, sorry for the very long delay. I still cant replicate it, can you try moving test/resources/mockito-extensions/org.mockito.plugins.MockMaker to main/resources/mockito-extensions/org.mockito.plugins.MockMaker and see if it passes?

Hi, I'm facing the same issue...
I tried put this extension in main/resources, but still get the same error...


New rule: Avoid offensive or profane words

I swear at the computer but not in my code (usually). But a rule would make sure of it 😉
Suggest putting a list like on https://www.cs.cmu.edu/~biglou/resources/ into abaplint with an option in the rule to add other words.

this could probably be solved via a simple grep, but it would not become a real annotation that way
I would like to avoid defining what is considered bad words, so perhaps a feature like
https://docs.abapopenchecks.org/checks/70/
which searches the code for different regex'es, I think it would have multiple usecases including looking for bad words

Or be able to provide a link to a word list in abaplint.json?
I guess the linter would be faster checking a word list than ABAP

https://github.com/marketplace/actions/profanity-filter or custom ATC check will do


npm - version not found error

npm install videojs-ima or npm update returns "No matching version found for<EMAIL_ADDRESS>Thanks,
D

Hi @dilburt ,
Just tested npm install videojs-ima and v1.7.2 successfully installed.
Closing the issue, but please reopen if the problem persists.


Bump wasmtime to v20.x

New wasmtime version was released a few days ago - https://github.com/bytecodealliance/wasmtime/releases/tag/v20.0.0
I'll take a crack at doing this myself if it's something that would be accepted.

I'm interested into the component model with this update, does wasmtime-dotnet support anything related to component ?

@xylobol PR's are always welcome, I did open #314 but seems we have a few things to catch up on the project which I am working through in #315

https://github.com/bytecodealliance/wasmtime-dotnet/releases/tag/v20.0.2
Will be release newer version (v21 and v22 pretty quickly after this)


AVP panics on reference to deleted secret version in Vault

Description
The current version of AVP panics when referencing a deleted version of a secret. AVP can handles the case where a path or a version doesn't exist, but Vault return nil in the body of the secret when an old version of the secret has been deleted.
Checklist
Please make sure that your PR fulfills the following requirements:

[x] Reviewed the guidelines for contributing to this repository
[x] The commit message follows the Conventional Commits Guidelines.
[ ] Tests for the changes have been updated
[ ] Are you adding dependencies? If so, please run go mod tidy -compat=1.17 to ensure only the minimum is pulled in.
[ ] Docs have been added / updated
[ ] Optional. My organization is added to USERS.md.

Type of Change

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

Other information

Running the current version of AVP
~/Documents/Playground/argocd-vault-plugin on main *1 ?1 ❯ go run main.go generate -c failures/config.yaml failures/ex1.yaml
panic: interface conversion: interface {} is nil, not map[string]interface {}

goroutine 1 [running]:
github.com/argoproj-labs/argocd-vault-plugin/pkg/backends.(*Vault).GetSecrets(0x14000a066f0, {0x1400099e6e6, 0x10}, {0x1400099e6fc, 0x1}, 0x140008ba400?)
        /Users/alexander/Documents/Playground/argocd-vault-plugin/pkg/backends/vault.go:76 +0x544
github.com/argoproj-labs/argocd-vault-plugin/pkg/backends.(*Vault).GetIndividualSecret(0x1400099e6f7?, {0x1400099e6e6?, 0x140005c9208?}, {0x1400099e6f7, 0x4}, {0x1400099e6fc?, 0x0?}, 0x0?)
        /Users/alexander/Documents/Playground/argocd-vault-plugin/pkg/backends/vault.go:95 +0x3c
github.com/argoproj-labs/argocd-vault-plugin/pkg/kube.genericReplacement.func1({0x1400099e6c0, 0x1e, 0x20})
        /Users/alexander/Documents/Playground/argocd-vault-plugin/pkg/kube/util.go:137 +0xa54
regexp.(*Regexp).ReplaceAllFunc.func1({0x0, 0x0, 0x0}, {0x14000129950?, 0x1e?, 0x20?})
        /opt/homebrew/Cellar/go/1.18.3/libexec/src/regexp/regexp.go:705 +0x80
regexp.(*Regexp).replaceAll(0x1400014b180, {0x1400099e6c0, 0x1e, 0x20}, {0x0, 0x0}, 0x2, 0x140005c9460)
        /opt/homebrew/Cellar/go/1.18.3/libexec/src/regexp/regexp.go:640 +0x2c0
regexp.(*Regexp).ReplaceAllFunc(0x140005c94f8?, {0x1400099e6c0?, 0x1e?, 0x16?}, 0x140005c94f8?)
        /opt/homebrew/Cellar/go/1.18.3/libexec/src/regexp/regexp.go:704 +0x5c
github.com/argoproj-labs/argocd-vault-plugin/pkg/kube.genericReplacement({0x14000129350, 0x8}, {0x1400099e540, 0x1e}, {{0x14000129380, 0x6}, 0x14000a06150, {0x102b455d0, 0x14000a066f0}, {0x0, ...}, ...})
        /Users/alexander/Documents/Playground/argocd-vault-plugin/pkg/kube/util.go:117 +0x144
github.com/argoproj-labs/argocd-vault-plugin/pkg/kube.secretReplacement({0x14000129350, 0x8}, {0x1400099e540, 0x1e}, {{0x14000129380, 0x6}, 0x14000a06150, {0x102b455d0, 0x14000a066f0}, {0x0, ...}, ...})
        /Users/alexander/Documents/Playground/argocd-vault-plugin/pkg/kube/util.go:221 +0x10c
github.com/argoproj-labs/argocd-vault-plugin/pkg/kube.replaceInner(0x140000703c0, 0x14000881af0, 0x102b15868)
        /Users/alexander/Documents/Playground/argocd-vault-plugin/pkg/kube/util.go:77 +0x4a0
github.com/argoproj-labs/argocd-vault-plugin/pkg/kube.replaceInner(0x140000703c0, 0x140000703d0, 0x102b15868)
        /Users/alexander/Documents/Playground/argocd-vault-plugin/pkg/kube/util.go:53 +0x558
github.com/argoproj-labs/argocd-vault-plugin/pkg/kube.(*Template).Replace(0x140000703c0)
        /Users/alexander/Documents/Playground/argocd-vault-plugin/pkg/kube/template.go:73 +0xb4
github.com/argoproj-labs/argocd-vault-plugin/cmd.NewGenerateCommand.func2(0x14000911180?, {0x14000c7de00?, 0x3?, 0x3?})
        /Users/alexander/Documents/Playground/argocd-vault-plugin/cmd/generate.go:88 +0x584
github.com/spf13/cobra.(*Command).execute(0x14000911180, {0x14000c7ddd0, 0x3, 0x3})
        /Users/alexander/go/pkg/mod/github.com/spf13/cobra@v1.4.0/command.go:856 +0x4c4
github.com/spf13/cobra.(*Command).ExecuteC(0x14000910f00)
        /Users/alexander/go/pkg/mod/github.com/spf13/cobra@v1.4.0/command.go:974 +0x354
github.com/spf13/cobra.(*Command).Execute(...)
        /Users/alexander/go/pkg/mod/github.com/spf13/cobra@v1.4.0/command.go:902
main.main()
        /Users/alexander/Documents/Playground/argocd-vault-plugin/main.go:10 +0x24
exit status 2

With fix:
~/Documents/Playground/argocd-vault-plugin on main !1 ?1 ❯ go run main.go generate -c failures/config.yaml failures/ex1.yaml
Error: Replace: could not replace all placeholders in Template:
The secret version 1 for Vault path secret/data/test is nil - is this version of the secret deleted?
Usage:
  argocd-vault-plugin generate <path> [flags]

Flags:
  -c, --config-path string         path to a file containing Vault configuration (YAML, JSON, envfile) to use
  -h, --help                       help for generate
  -s, --secret-name string         name of a Kubernetes Secret in the argocd namespace containing Vault configuration data in the argocd namespace of your ArgoCD host (Only available when used in ArgoCD). The namespace can be overridden by using the format <namespace>:<name>
      --verbose-sensitive-output   enable verbose mode for detailed info to help with debugging. Includes sensitive data (credentials), logged to stderr

exit status 1

Codecov Report

Merging #411 (e9bfa88) into main (6396995) will decrease coverage by 0.05%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main     #411      +/-   ##
==========================================
- Coverage   77.47%   77.42%   -0.06%
==========================================
  Files          23       23
  Lines        1030     1032       +2
==========================================
+ Hits          798      799       +1
- Misses        145      146       +1
  Partials       87       87

Impacted Files
Coverage Δ

pkg/backends/vault.go
77.27% <66.66%> (-1.30%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

This is quite an important fix. @werne2j Would you please consider releasing a new version with this fix? That would be 🥇

https://github.com/argoproj-labs/argocd-vault-plugin/releases/tag/v1.13.1


Refactor/incoming tab params

improvements to the incoming tab

it will open to the correct page and selection after a refresh

changes to the url are done with browserHistory

params comes fro withRouter instead of being passed as a prop

[ ] I wrote tests for the feature

[ ] I documented the feature if needed

If the feature is a refactor, please explain why your way is better

the botfront-api build will need to be updated for the conversations.spec test to pass

ready to be merged

@pheel the project import request is still being refused


Fix JSON validation. Missing commas

Drat, sorry I missed that Nick. I'll have to rig up a simple test to confirm the JSON is valid.

No worries at all :) Thanks!


Question about starting up 1000 leshan clients

Hi, I was using the leshan client demo, and was tring to boot up 1000 clients.
I made a change in the config file like this:
leshan.thread.num = 1000
leshan.imei=865067040010001
leshan.flag=0
leshan.serverip=nbiotbt.heclouds.com
leshan.port=33334
leshan.cellId = 300
leshan.cellId.num = 100
leshan.currentVersion = 1.0.0
leshan.targetVersion = v1.0
leshan.linkUtilization = 200
leshan.linkUtilization.num = 10

and then ran the .jar file on 2 different linux machines.
on one of the linux machines, I found that the 1000 clients boot up very quickly and looks that they were boot up simultaneously
2018-08-03 20:03:02,901 INFO LeshanClient - Starting Leshan client ...
2018-08-03 20:03:02,901 INFO LeshanClient - Starting Leshan client ...
2018-08-03 20:03:02,901 INFO LeshanClient - Starting Leshan client ...
2018-08-03 20:03:02,901 INFO LeshanClient - Starting Leshan client ...
2018-08-03 20:03:02,901 INFO LeshanClient - Starting Leshan client ...
2018-08-03 20:03:02,901 INFO LeshanClient - Starting Leshan client ...
2018-08-03 20:03:02,902 INFO LeshanClient - Starting Leshan client ...
2018-08-03 20:03:02,901 INFO LeshanClient - Starting Leshan client ...
2018-08-03 20:03:02,901 INFO LeshanClient - Starting Leshan client ...
2018-08-03 20:03:02,901 INFO LeshanClient - Starting Leshan client ...
2018-08-03 20:03:02,901 INFO LeshanClient - Starting Leshan client ...
2018-08-03 20:03:02,901 INFO LeshanClient - Starting Leshan client ...
2018-08-03 20:03:02,901 INFO LeshanClient - Starting Leshan client ...
2018-08-03 20:03:02,900 INFO LeshanClient - Starting Leshan client ...

on the other linux machine, I found that the 1000 clients boot up one by one:
2018-08-03 17:20:27,614 INFO LeshanClient - Starting Leshan client ...
2018-08-03 17:20:27,627 INFO LeshanClient - Leshan client[endpoint:865067040010008;865067040010008] started at coap://<IP_ADDRESS>/<IP_ADDRESS>:33341 coaps://<IP_ADDRESS>/<IP_ADDRESS>:59139
2018-08-03 17:20:27,628 INFO RegistrationEngine - Trying to start bootstrap session to coap://nbiotbt.heclouds.com:5683 ...
2018-08-03 17:20:33,374 INFO RegistrationEngine - Bootstrap started
2018-08-03 17:20:33,419 INFO LeshanClientDemo - Press 'w','a','s','d' to change reported Location (-28.0,-69.0).
2018-08-03 17:20:33,419 INFO LeshanClient - Starting Leshan client ...
2018-08-03 17:20:33,431 INFO LeshanClient - Leshan client[endpoint:865067040010009;865067040010009] started at coap://<IP_ADDRESS>/<IP_ADDRESS>:33342 coaps://<IP_ADDRESS>/<IP_ADDRESS>:51136
2018-08-03 17:20:33,432 INFO RegistrationEngine - Trying to start bootstrap session to coap://nbiotbt.heclouds.com:5683 ...

Can someone kindly help me and tell me why?
I was using the same jar file with the same boot up command, where is the differences?
Thanks very much

With so little information, it's hard to help.
You just say "I ran exactly the same thing on 2 different machines and it seems to not behave the same."
What is this config file you talking about ? leshan.thread.num = 1000 ..., this is not leshan-client-demo config ?
How do you run your 1000 clients ?
What are the differences between the 2 linux machines ?

ping ?

I close it because of inactivity (Feel free to reopen it, if needed)


[Feature Request] Download m3u8/dash live video natively

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

When I try to download m3u8 live video it only downloads 3 segments and then it stops downloading?
Can I download live m3u8 video with --downloader "Dash,m3u8:native" or can you add this feature for live m3u8 video if it is not supported?
Command I use:
yt-dlp 'https://mn-nl.mncdn.com:443/blutv_trt1/smil:trt1_sd.smil/chunklist_b2064000.m3u8' --fragment-retries 'infinite' -f 'best' -o 'liveVid.%(ext)s' --downloader "Dash,m3u8:native"

No, live videos are not supported by the native downloader. It will be handed over to ffmpeg

No, live videos are not supported by the native downloader. It will (normally) be handed over to ffmpeg.

But with ffmpeg segments will be skipped if there a problem occurs right? İt would be nice if
--fragment-retries 'infinite' could also work with ffmpeg then?
When you have time for it. I would be appreciated if you could add this feature. if it is possible?

Yes, it is technically possible, but also quite difficult to implement

But with ffmpeg segments will be skipped if there a problem occurs right? İt would be nice if
--fragment-retries 'infinite' could also work with ffmpeg then?

ffmpeg doesnt support retrying afaik. So it is not possible

Yes, it is technically possible, but also quite difficult to implement

Could you maybe think adding this feauture when you have time for it please?

Yes, it is technically possible, but also quite difficult to implement

https://live.corusdigitaldev.com/groupd/live/49a91e7f-1023-430f-8d66-561055f3d0f7/live.isml/.mpd this is livestream, please

Does yt-dlp now have the ability to download live streams natively or did I found a workaround?
Cuz with this command on termux I'm able to download live streams?
yt-dlp $(youtube-dl -g "URL") --downloader "dash,m3u8:native" -o vid.ts -R infinite --fragment-retries infinite --no-config

Don't think it works. It will just download the stream from start to the present, but wont wait for and download future segments

Don't think it works. It will just download the stream from start to the present, but wont wait for and download future segments

yes you're right.
it did just as you said.
sorry to bother you was excited it might work :(

I would also appreciate if this could get added as a feature since ffmpeg relatively often hangs when capturing livestreams on twitch, it will just hang indefinitely.
it's pretty well explained here: http://michael-peeters.blogspot.com/2019/03/hanging-m3u8-downloads-in-ffmpeg.html?m=1
since from ffmpegs side there seems to be no interest in fixing this or providing a way to set a timeout, the only way to not miss out on hours of recording sometimes is to not use ffmpeg.
all -timeout as well as -reconnect options that ffmpeg has do not work for this issue

Is --downloader 'dash,m3u8:native' better than using aria2?

Is --downloader 'dash,m3u8:native' better than using aria2?

@batbone If you use the -N option and `--extractor-args "youtube:formats=dashy", then yes, it is better IMO
This is off-topic though


Ability to customize gc-docker rules

At the moment the hourly cleaning of Docker images is a bummer for caches… especially on our builder machine. I'd be neat to support a S3 file containing the gc-docker rules… or perhaps just paste them in as a stack parameter?

I'd suggest we ditch docker-gc entirely, or at least only run it if disk space is an issue.

On 5 Jul 2016, at 10:03 AM, Tim Lucas<EMAIL_ADDRESS>wrote:
At the moment the hourly cleaning of Docker images is a bummer for caches… especially on our builder machine. I'd be neat to support a S3 file containing the gc-docker rules… or perhaps just paste them in as a stack parameter?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I'm fine with that! I'll just axe it.

I think we should probably keep it for long-lived persistent machines, because Docker will fill things up pretty quick, but increase the TTL of images to 3 days, or 1 week, or something more than 30 minutes. And also enable the FORCE_IMAGE_REMOVAL option (https://github.com/spotify/docker-gc#forcing-deletion-of-images-that-have-multiple-tags) whilst we're there.


docs: [IAC-2491]: Add TF Auth approaches

Description

Add OpenTofu and Terraform authentication approaches depending on use case.
JIRA

PRs must meet these requirements to be merged:

[x] Successful preview build.
[x] Code owner review.
[x] No merge conflicts.
[x] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6703c60cb153109b943f5c0f--harness-developer.netlify.app


Python Program to Convert Kilometers to Miles

Taking kilometers input from the user
kilometers = float(input("Enter value in kilometers: "))
conversion factor
conv_fac = 0.621371
calculate miles
miles = kilometers * conv_fac
print('%0.2f kilometers is equal to %0.2f miles' %(kilometers,miles))

hey, i can do this one . can you please assign  to me ?

I've made a PR regarding this issue.Happy to help.

is it still open for contribution

i want it bro....

Hey i can do this in min no of lines
I would like to contribute
assign this issue to me.
I will fix it within 24 hours


perf: min max buffer

This PR attemps to reduce the number of re-renders that happens because data is fetched.

Moved the area logic from fetch-pages.ts into get-column-pages.ts and get-row-pages.ts
Added max buffer logic in get-column-pages.ts and get-row-pages.ts

Before:
Every time a new row or column that had cells without data was scrolled into "view", data was fetched for that row/column (+ min buffer). This means that scrolling 100 rows/column would at most trigger 100 re-renders.
After:
Every time a new row or column that has cells without data is scrolled into "view", data is fetched for that row/column (+ min buffer + max buffer). This means that scrolling 100 rows/column will at most only trigger 4 re-renders.

I'm actually having a bit trouble understanding the concept of this PR, but i think this part here:

This means that scrolling 100 rows/column would at most trigger 100 re-renders.

does not happens really, because if you scroll into a row/col which does not has the data yet, it will try to fetch from that starting point(what ever row/col you landed on, at least as far as i know about the data fetching flow).
and here is the current behaviour on main, which shows that even scrolling into a completely empty area, does not re-render 100 or n times (n is the row counts in that empty area):
https://github.com/qlik-oss/sn-pivot-table/assets/29652890/2ca8d2da-993b-41c0-9d8e-db40556149bb
So even if we fetch the max buffer area, potentially we are fetching more data that expected maybe?
Also regarding this part:

This means that scrolling 100 rows/column will at most only trigger 4 re-renders

I think the min buffer's purpose is exactly what we want to achieve with max buffer! why not instead, choose a bigger minBuffer? since the scenario of fetching new data will always happen with both min and max buffer?
and in that case, what would be the main diff between min and max buffer in essence?

@a-m-dev I've been struggling to describe how and why this works as it does. So I'll give another try here for some of your concerns.

I'm actually having a bit trouble understanding the concept of this PR, but i think this part here:

This means that scrolling 100 rows/column would at most trigger 100 re-renders.

does not happens really, because if you scroll into a row/col which does not has the data yet, it will try to fetch from that starting point(what ever row/col you landed on, at least as far as i know about the data fetching flow).

The "at most" of what I said is important here. To simply, a user can do 2 kinds of scrolling actions that will trigger data to be fetched - 1. User scrolls slowly so that "every" row or column is triggering a data fetch. 2. User scrolls quickly (what you did) to a new position, this means that "throttled data fetching logic" will get triggered and skip some of the scroll events that could have trigger a data fetch.
So when I said 100 re-renders I meant the slow scrolling example where you actual get 100 data fetches, thus 100 re-renders. In you example the number of re-renders will likely barely improve since the "throttled data fetching logic" is already handling that fairly well.

Also regarding this part:

This means that scrolling 100 rows/column will at most only trigger 4 re-renders

I think the min buffer's purpose is exactly what we want to achieve with max buffer! why not instead, choose a bigger minBuffer? since the scenario of fetching new data will always happen with both min and max buffer? and in that case, what would be the main diff between min and max buffer in essence?

The MIN buffer and the MAX buffer have different purposes. The MIN buffer is there to improve the user experience when the user scroll slowly (see the 1. description above), it reduces the chance of rendering empty cells when no data have been fetched yet. That is why the min buffer extends the rendered area a little bit, it triggers data to be fetched for an area that is yet not visible to the user. So when they scroll that area into view, the data have already been fetched.
The MAX buffer is there to re-reduce the number of times data is fetched. For this to make some sense it important to understand when data is fetched. The flow for fetching data looks something like this: Scroll Event -> Scroll react-window components -> react-window's onItemsRendered callback checks if any rendered cells are missing data -> Fetch data for missing rows or columns -> Re-render "all" components. This is why scrolling slowly can trigger data to be fetched for every "empty" row or column. The MAX buffer will make it so that the react-window's onItemsRendered callback checks if any rendered cells are missing data step less often finds cells with missing data. Resuling in fewer re-renders.
Not sure if I can explain this any better. If there are still some uncertainties we can have a chat about it?

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


godeb: no downloads available

What version of acs-engine?
master at commit d11ad5b98af7041a55a11e83fbb320ae471fd017
godeb from s3 looks like having problem fetching go version.
`
acs-engine$ ./scripts/devenv.sh

docker build --pull -t acs-engine .
Sending build context to Docker daemon  227.2MB
Step 1/18 : FROM buildpack-deps:xenial
xenial: Pulling from library/buildpack-deps
Digest: sha256:07d3330a162edc027a589df0ec6ac4cce91ae426f6dc490f49b3b119e9e6e8d8
Status: Image is up to date for buildpack-deps:xenial
---> c063789e4027
Step 2/18 : RUN apt-get update     && apt-get -y upgrade     && apt-get -y install python-pip make build-essential curl openssl vim jq gettext     && rm -rf /var/lib/apt/lists/*
---> Using cache
---> 7ef442837204
Step 3/18 : ENV GO_VERSION 1.8
---> Using cache
---> 0468e8c7c54b
Step 4/18 : RUN mkdir /tmp/godeb     && curl "https://godeb.s3.amazonaws.com/godeb-amd64.tar.gz" > /tmp/godeb/godeb.tar.gz     && (cd /tmp/godeb; tar zvxf godeb.tar.gz; ./godeb install "${GO_VERSION}")     && rm -rf /tmp/godeb
---> Running in 55c152feca2c
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100 2002k  100 2002k    0     0   422k      0  0:00:04  0:00:04 --:--:--  422k
godeb
error: no downloads available at http://golang.org/dl/
`

Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm)
What happened:
go download crashed
What you expected to happen:
docker running
How to reproduce it (as minimally and precisely as possible):
run ./scripts/devenv.sh
Anything else we need to know:
looks like godeb.s3 tarball is bad

same happens with v0.8.0 on Ubuntu 16.04 server
Downloading the godeb tarball manually works:
curl "https://godeb.s3.amazonaws.com/godeb-amd64.tar.gz" > /tmp/godeb/godeb.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 2002k  100 2002k    0     0  7251k      0 --:--:-- --:--:-- --:--:-- 7254k

running godeb fails:
./godeb install 1.8
error: no downloads available at http://golang.org/dl/

known issue with godeb: https://github.com/niemeyer/godeb/issues/31

Can you guys try this again? I think we might have fixed this issue last week.

yup, it's working for me now, thank you


The method 'listen' isn't defined for the type 'VxState'.

The method 'listen' isn't defined for the type 'VxState'.
Try correcting the name to the name of an existing method, or defining a method named 'listen'.
from your flutter video https://youtu.be/j-LOab_PzzU
day 27 when we put listen in add_to_cart.dart file it was displaying the above issue.
and may this error is in the new version or vxstate

.

Use .watch()


conformance: pull tests shouldn't rely on tag listing

I don't have a fully-implemented container registry yet. I am implementing
endpoints incrementally in order to work toward a fully-functioning
implementation of the spec. This PR removes call to the tag listing endpoint
that was erroneously being relied on to determine the name of a valid tag to
pull.
Instead, I replaced a tag := ... with a tag = ... in an earlier ginkgo case
in order to set the outer scopoed tag variable used in subsequent tagged
manifest HEAD call. I suspect the tag listing endpoint was added here to
address this apparent := vs = bug.

@sudo-bmitch thanks for the heads up, i pushed a commit to fix it.

The overall change LGTM. Would you squash the commits and add the DCO signoff?
https://github.com/opencontainers/distribution-spec/pull/476/checks?check_run_id=17356808821
https://github.com/opencontainers/distribution-spec/blob/main/CONTRIBUTING.md#sign-your-work

@sudo-bmitch done, thanks!


Stabilization of EmailAddress.length property

Description
We would like to stabilize the EmailAddress.length property.
Dependencies
This issue is blocked by the following ones:

[x] #357

Checklist

[ ] Remove its experimental annotation, update its versioning annotation, update its tests, update the public API binaries and the unreleased changelog.
[ ] Close this issue as completed and update tracking ones if present.

We are not ready to implement this change.


android-studio-full: fix changelog ordering

Description of changes

Followup to #315233 fixing the changelog ordering and moving the relnote to 24.11.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

gtest is documented at https://github.com/NixOS/nixpkgs/pull/320194/files#diff-9c253a17d297f3da51e6ef73a3cb898d879ccf8b0c8613199770917e9387d92cL311 and became part of the android-studio-full PR due to a bad merge.

^^

Oh.


unit test test_slice_default_axes_cpu from onnx is failing for the python bindings

Describe the bug
Error is:
RuntimeError: [ONNXRuntimeError] : 10 : INVALID_GRAPH : Node: Node () has input size 3 not in range [min=1, max=1].
unit test to run: onnx_backend_test_series.py
It was passing before.
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows and Debian
ONNX Runtime installed from (source or binary): master branch 03/14
ONNX Runtime version: master branch 03/14
Python version: 3.7.2
CPU

To Reproduce
Build the python python and run onnx_backend_test_series.py.
Expected behavior
No error.

What is your ONNX version?

ONNX from master branch cloned today, this is how I build onnxruntime. The behaviour seems different based on the setting used to compile.
python3.7 ./onnxruntime/tools/ci_build/build.py --build_dir ./onnxruntime/build/linux --config Release --build_wheel --use_mkldnn --use_openmp --use_llvm --numpy_version= --skip-keras-test

That's the problem. You must use the same one in onnxruntime's submodule.

Hi @xadupre
Please reinstall your onnx python package and let me know if the error is gone.

I did not see any improvment and two others tests were failing:     '^test_averagepool_2d_ceil_cpu.',
'^test_maxpool_2d_ceil_cpu.',. Maybe that's some tests which were renamed. I pull onnx and onnxruntime fomr github this morning.

Hi @xadupre , you can't install onnx from your onnx repo, you should install it from onnxruntime's sub folder, so that we can ensure it's the same version used by onnxruntime.

That's not a common scenario, what if onnxruntime requires a specific runtime and torch another one. What's the difference between onnx from onnxruntime and official onnx?

@xadupre
I'm sorry, I have to say. it's not supported.  It won't be supported unless onnxruntime only picks released onnx versions(not a random git commit).

What's the difference between onnx from onnxruntime and official onnx
We extended the protobuf definition.(aka, model format) . see: https://github.com/Microsoft/onnxruntime/blob/master/onnxruntime/core/protobuf/onnx.in.proto

Ok let's close the issue then. The failure is not tested and out of scope.


Revert incorrect change to LICENSE

Brackets were in section on re-applying the license, so was correct.

This change is

Sorry about that (^^;)


ISPN-11023 Indexless query fails with missing component in the registry

https://issues.jboss.org/browse/ISPN-11023

@danberindei were you able to reproduce the problem?

@danberindei Split into 3 commits with a minimal reproducer

Integrated, thanks Gustavo!
I'll open a separate PR to remove the random and always pick the same server.


Adding default fade for toast

Adding a default fade for a toast

You've stated that you don't want to impose logic with the library.  Maybe new var(s) for the pop and fade?

What does this animation do?
Is it for when you hide/remove an item? Because that won't work. Hiding or removing is instant and it happens before the CSS animation.

When a toast is triggered, your pop animation runs.  It then waits 2.8 seconds, and then automatically fades it with the next animation (the added code).

First problem, It doesn't seem to work as you explained. It shows up with the toast-pop animation, then after 2.8s it hides with the new toast-fade animation but immediately (as soon as animation ends - 0.25s) it becomes visible again.
Second problem, what if someone doesn't want the toast to auto hide?
Maybe they want it to be closed on click or using a JS timer.
I think if we're adding something like this, it should be with new class like auto-hide

Second problem, what if someone doesn't want the toast to auto hide? Maybe they want it to be closed on click or using a JS timer. I think if we're adding something like this, it should be with new class like auto-hide

The code was pulled from a working implementation, that handles toasts via a svelte store, so apologies I didn't forsee the issue.  I'll close this, because you're right, this type of request is better left to project implementors.
Thank you for the feedback!


Update ticket decode logic

Currently Negotiator validate tickets(embedded inside MagicLink) with src/Attestation/SignedDevonTicket.ts
but tickets can have multiple schemas. for example - new ticket format is different, commitment moved inside ticket.
so would be good to use "asn1js" directly and avoid use "src/Attestation/SignedDevonTicket.ts"
we can use ticket reader from https://github.com/TokenScript/attestation/blob/js_pr212/src/main/javascript/crypto/src/Ticket.ts , its compatible with both ticket formats.

resolved in https://github.com/TokenScript/token-negotiator/pull/173


Code Release on evaluate_imitate.py

Hi, thanks a lot for your incredible framework and open-sourced codes!
I was wondering if you have any plans to release the evaluation framework for the "Language-Conditioned Imitation" task, namely, evaluate_imitate.py (currently a blank file)
It would be quite interesting to port the language-condition imitation code to the simulation frameworks (e.g. coppeliasim), and I hope further research could be facilitated from your work by doing so.
Hope this issue could reach you, and thanks again for your outstanding work!

Hey @jobin2725 - sorry for the delayed response! The plan is to add the simulation evaluate_imitate.py following the new standardized environments from RoboHive (https://github.com/vikashplus/robohive/tree/main). Unfortunately, I am a bit swamped with a couple final threads this month, but I will get to it as soon as I can!
In the meantime, you can use the R3M Evaluation Code (https://github.com/facebookresearch/r3m/tree/eval/evaluation) which is what we initially forked / used to run the evaluations in the paper!


Performance issues with Docker SQL 2019-latest image

After SQL2019 image was updated, tests on 2019-latest docker container performance is poor, I started using to 2019-CU27-ubuntu-20.04 and performance of tests is normal again.
Is it known what causes performance issues?

Image
Ubuntu v
Created date
Last updated

2019-latest
Ubuntu 20.04
11/07/2018
08/01/2024

2019-CU27-ubuntu-20.04
Ubuntu 20.04
06/13/2024
06/13/2024

Docker SQL container tags
Tested with versions 1.4.0 and 1.4.1

Seems that problem is the way file group is deployed:
Creating Filegroup [TAB_ADJ_2010]...
Creating File [TAB_ADJ_2010_1]...
EXEC : error SQL72014: Core Microsoft SqlClient Data Provider: Msg 5009, Level 16, State 1, Line 1 One or more files listed in the statement could not be found or could not be initialized. [C:\[path]\DBPROJECT.Tests.csproj]
EXEC : error SQL72045: Script execution error.  The executed script: [C:\[path]\DBPROJECTB.Tests.csproj]
  ALTER DATABASE [$(DatabaseName)]
      ADD FILE (NAME = [TAB_ADJ_2010_1], FILENAME = '$(DefaultDataPath)FILENAME_TAB_1.ndf', SIZE = 5 MB, FILEGROWTH = 10 %) TO FILEGROUP [TAB_ADJ_2010];

EXEC : error SQL72014: Core Microsoft SqlClient Data Provider: Msg 5170, Level 16, State 1, Line 1 Cannot create file 'C:\Program Files\Microsoft SQL Server\MSSQL16.MSSQLSERVER\MSSQL\DATA\FILENAME_TAB_1.ndf' because it already exists. Change the file path or the file name, and retry the operation. [C:\[path]\DBPROJECTB.Tests.csproj]

These new (and latest) SQL Server images do not start at dotnet-sqltest at all:

2019-CU28-ubuntu-20.04
2022-CU14-ubuntu-20.04
2022-CU14-ubuntu-22.04

It is cause by major changes at the path to sqlcmd tool inside the container. It was reported at testcontainer-dotnet: https://github.com/testcontainers/testcontainers-dotnet/issues/1220
I need to update couple of packages and it will handle new and old images without any issues. Additionally sqltest must have a better and faster error handling on that and similar situations.

Interesting that command dotnet publish (MSBuild.Sdk.SqlProj) tries 20 times to start the connection with SQL Server and then fails:

These new (and latest) SQL Server images do not start at dotnet-sqltest at all:

2019-CU28-ubuntu-20.04
2022-CU14-ubuntu-20.04
2022-CU14-ubuntu-22.04

It is cause by major changes at the path to sqlcmd tool inside the container. It was reported at testcontainer-dotnet: testcontainers/testcontainers-dotnet#1220
I need to update couple of packages and it will handle new and old images without any issues. Additionally sqltest must have a better and faster error handling on that and similar situations.

I reported this as well https://github.com/microsoft/mssql-docker/issues/898 as we noticed issues when running our docker containers locally. Wondering what would be their suggestion or next version release, change path to executable binary or revert to previous path as they did with CU14

Note Starting with SQL Server 2022 CU 14, we are updating SQL Server 2022 container images to include the new mssql-tools18 package. With the introduction of SQL Server 2022 CU 14, and in all future container images, the previous directory /opt/mssql-tools/bin will be phased out. The new directory for Microsoft ODBC 18 tools will be '/opt/mssql-tools18/bin', aligning with the latest tools offering. For additional details on all the changes and security enhancements, please refer to the blog post [ODBC Driver 18.0 for SQL Server Released - Microsoft Community Hub](https://techcommunity.microsoft.com/t5/sql-server-blog/odbc-driver-18-0-for-sql-server-released/ba-p/3169228). This new iteration, ODBC driver version 18, is designed with an ‘encryption-first’ approach, ensuring that utilities like sqlcmd and bcp that utilize the Microsoft ODBC driver will operate under the ‘secure by default’ principle. Users who wish to disable encryption will need to do so explicitly.

It seems that's the future CU14 for SQL2022 was release on the same date . From docker.com

I think test containers already have fix for this (implemented 3 weeks ago):
https://github.com/testcontainers/testcontainers-dotnet/blame/7174786e4803f66158d6cd753ec55352f46953b7/src/Testcontainers.MsSql/MsSqlContainer.cs#L72

Yes, testcontainers-dotnet 3.10.0 has fix for this issue, so does new version of dotnet-sqltest 1.4.2.

Thank you. This resolved issue on pipeline and locally


AttributeError: I'm having a hard time solving this issue.

Hopefully I'm not going to bother you guys with a really mundane, simple, "totally my fault error"; but, at the same time, perhaps those are the best type of issues! :)
So, I first ran my code in the google collab notebook to do an enrichment study . Long story short, it worked, but I had a hard time getting the print_summary function to work, but got the print_results function to work instead, and went with that. Turns out, I like the additional statistics it provides, so I wish to continue using that function.
I then run the script through terminal, and I get the following error:
AttributeError: 'GOEnrichmentStudy' object has no attribute 'print_results'
I'm curious as to what I am doing wrong? A cut out of my code is below, let me know if you need anymore information/code/files. Of note, when running from terminal, I got the print_summary function to perform (I'm going crazy). Again I hope this is a simple issue to resolve!
Thanks in advance,
Curtis
study = tca_dict.keys()
methods = ['fdr']

g = GOEnrichmentStudy(ecoli_keys, matches_reference, obodag,
                      propagate_counts = True, alpha = 0.05,
                      methods = methods)

g_res = g.run_study(study)

print('\n\n#----------------------------------------------------------------------#')
print('#----------------------------------------------------------------------#')
print('#--------------------------------#')
print('#----------------------------------------------------------------------#')
print('#----------------------------------------------------------------------#\n\n')

g.print_results(g_res) #these ticks are not on actual code```

Hello Curtis,
Thank you for your interest in GOATOOLS and taking the time to write us.
Did you get the GOATOOLS package from github (eg, "git clone …GOATOOLS..." or from PyPI (eg, "pip install GOATOOLS")? Can include the full error messages?
What version of the GOATOOLS are you using?
import goatools
print(goatools.__version__)

I did the easy_install I believe (honestly, don't quite remember).
I'm running the 0.8.4 version:
And here's the full error result from terminal:
  File "GOATOOLS_survival.py", line 83, in <module>
    g.print_results(g_res)
AttributeError: 'GOEnrichmentStudy' object has no attribute 'print_results'

And from Spyder:
Traceback (most recent call last):

  File "<ipython-input-7-0d638d4d78c9>", line 1, in <module>
    runfile('/home/cross12tamu/.config/spyder-py3/GOATOOLS_survival.py', wdir='/home/cross12tamu/.config/spyder-py3')

  File "/home/cross12tamu/Research/anaconda3/lib/python3.6/site-packages/spyder_kernels/customize/spydercustomize.py", line 678, in runfile
    execfile(filename, namespace)

  File "/home/cross12tamu/Research/anaconda3/lib/python3.6/site-packages/spyder_kernels/customize/spydercustomize.py", line 106, in execfile
    exec(compile(f.read(), filename, 'exec'), namespace)

  File "/home/cross12tamu/.config/spyder-py3/GOATOOLS_survival.py", line 83, in <module>
    g.print_results(g_res)

AttributeError: 'GOEnrichmentStudy' object has no attribute 'print_results'

Thank you so much for the prompt reply.
Curtis

The latest GOATOOLS version is 0.8.9.
Try updating your GOATOOLS and then re-running.
pip install --upgrade goatools

Also, if you comment out the line, "g.print_results(g_res)", does your script work?

Can you include a copy of your import statements?

The upgrade actually fixed the function for me. Now the print_results works.
And, when I ran this before, the script would run and just get that error when it hit g.print_results at the end of the script.
SO, for now, the print_results is solved, however, now the print_summary is giving me...odd results? I'll link results in just a sec (as well as do the other commands from your most recent post).

1st, the dir prints
print(dir(GOEnrichmentStudy))
['__class__', '__delattr__', '__dict__', '__dir__', '__doc__', '__eq__', '__format__', '__ge__', '__getattribute__', '__gt__', '__hash__', '__init__', '__init_subclass__', '__le__', '__lt__', '__module__', '__ne__', '__new__', '__reduce__', '__reduce_ex__', '__repr__', '__setattr__', '__sizeof__', '__str__', '__subclasshook__', '__weakref__', '_log_multitest_corr', '_run_multitest_corr', '_run_multitest_local', '_run_multitest_statsmodels', '_update_pvalcorr', 'get_adj_records', 'get_item_cnt', 'get_ns2nts', 'get_prtflds_default', 'get_pval_uncorr', 'get_results_msg', 'print_results', 'print_results_adj', 'print_summary', 'prt_tsv', 'prt_txt', 'run_study', 'run_study_nts', 'wr_py_goea_results', 'wr_tsv', 'wr_txt', 'wr_xlsx']

print(dir(g))
['__class__', '__delattr__', '__dict__', '__dir__', '__doc__', '__eq__', '__format__', '__ge__', '__getattribute__', '__gt__', '__hash__', '__init__', '__init_subclass__', '__le__', '__lt__', '__module__', '__ne__', '__new__', '__reduce__', '__reduce_ex__', '__repr__', '__setattr__', '__sizeof__', '__str__', '__subclasshook__', '__weakref__', '_log_multitest_corr', '_run_multitest', '_run_multitest_corr', '_run_multitest_local', '_run_multitest_statsmodels', '_update_pvalcorr', 'alpha', 'assoc', 'get_adj_records', 'get_item_cnt', 'get_ns2nts', 'get_prtflds_default', 'get_pval_uncorr', 'get_results_msg', 'go2popitems', 'log', 'methods', 'obo_dag', 'pop', 'pop_n', 'print_results', 'print_results_adj', 'print_summary', 'prt_tsv', 'prt_txt', 'pval_obj', 'run_study', 'run_study_nts', 'wr_py_goea_results', 'wr_tsv', 'wr_txt', 'wr_xlsx']

2nd, print_results v. print_summary
Here is the first "result" from g.print_results:
GO      NS      enrichment      name    ratio_in_study  ratio_in_pop    p_uncorrected   depth   study_count     p_fdr   study_items
GO:0006101      BP      e       citrate metabolic process       9/16    10/3623 5.25e-08        n.a.    9       0       P05042, P0A836, P0AC33, P0AC41, P0AFG3, P14407, P25516, P36683, P61889

Here is are the results from g.print_summary:
# Generated by GOATOOLS v0.8.9 (2018-10-04)
# min_ratio=[GOEnrichmentRecord(GO:0006101), GOEnrichmentRecord(GO:0006099), GOEnrichmentRecord(GO:0072350), GOEnrichmentRecord(GO:0006091), GOEnrichmentRecord(GO:0017144), GOEnrichmentRecord(GO:0045333), GOEnrichmentRecord(GO:0015980), GOEnrichmentRecord(GO:0016999), GOEnrichmentRecord(GO:0009060), GOEnrichmentRecord(GO:0019752), GOEnrichmentRecord(GO:0043436), GOEnrichmentRecord(GO:0006082), GOEnrichmentRecord(GO:0055114), GOEnrichmentRecord(GO:0044237), GOEnrichmentRecord(GO:0008152), GOEnrichmentRecord(GO:0006097), GOEnrichmentRecord(GO:0006108), GOEnrichmentRecord(GO:0044281), GOEnrichmentRecord(GO:0017004), GOEnrichmentRecord(GO:0046487), GOEnrichmentRecord(GO:0009061), GOEnrichmentRecord(GO:0009987), GOEnrichmentRecord(GO:0006106), GOEnrichmentRecord(GO:0022900),

...and this continues for many more GO terms.
So, I had g.summary working yesterday, with g.print_results NOT working. Today (after update), g.summary works but not in the same way, with g.print_results working.
¯_(ツ)_/¯

I am so glad that print_results is working for you. I am going to close this issue.
Please open a new issue for the summary issues. We can track things better if we do this.
Thank you again for your interest in GOATOOLS. I will take a look at the summary that you are seeing.

I will do so later this afternoon, and certainly by tomorrow morning!
Thank you again for the prompt replies and assistance.
Curtis


Feature: cross-service commands

For service orchestration, I’m pondering the best way for one service’s process manager to call another service’s command. Perhaps each service could generate and serve an auto-generated sdk that other services could consume?  Any other ideas?
Obviously the decision to use orchestration for certain processes creates a tighter coupling but is required for certain business requirements.

I try to avoid this pattern, but when necessary I just use an axios helper function to call commands, something like the HTTP client found in the eventually-express package.
This is needed though when policies read from projections, I use the axios approach too.
If you build the ESML model of your services, you will clearly see the blue, green, and orange arrows showing inter-context coupling... find your compromise, but try to avoid blue ones
I just deployed a new feature in the ESML create-eventually-app generator to deal with something like this... generating vscode tasks and docker-compose config to run all services in your dev env.


Fix "DS-n_v2-series" VM Sizes

DS-n_v2-series, the VMs in this table were missing the "S" is "DS", and there was a slight error in the VM SKU Family name missing a hyphen. Creating a PR to suggest the fix, rather than reporting this minor issue to the content developers.

@NeilBird : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

#sign-off


Update SVG sprite task to delete the output folder

Removing SVG's leaves orphans in the sub-folders which get added to the sprite on compile.

Closed via #18


receive "PHP Warnings" since last upgrade

Describe the bug
PHP Warning:  Module "igbinary" is already loaded in Unknown on line 0
PHP Warning:  Module "imap" is already loaded in Unknown on line 0
PHP Warning:  Module "msgpack" is already loaded in Unknown on line 0

PHP versions
At least with 8.1, didn't check the lower versions
To Reproduce
run this file:
#!/bin/bash

# This is based on:
# * https://github.com/shivammathur/homebrew-php
# * https://github.com/shivammathur/homebrew-extensions

PHP_VERSIONS="5.6 7.0 7.1 7.2 7.3 7.4 8.0 8.1" # last one is used as default PHP version
EXTENSIONS="amqp imagick imap pcov redis"

echo "Add taps (--force-auto-update)"
HOMEBREW_DEVELOPER=1 brew untap shivammathur/php
brew tap shivammathur/php --force-auto-update
HOMEBREW_DEVELOPER=1 brew untap shivammathur/extensions
brew tap shivammathur/extensions --force-auto-update

echo "Remove all available PHP configs"
rm -rf /usr/local/etc/php/*

echo "Uninstall imap-uw"
brew uninstall --ignore-dependencies imap-uw

for version in $PHP_VERSIONS
do
    echo "Install PHP $version with: $EXTENSIONS"

    brew reinstall shivammathur/php/php@$version
    brew link --overwrite --force php@$version

    for extension in $EXTENSIONS
    do
        if [ "$extension" = "pcov" ] && { [ "$version" = "5.6" ] || [ "$version" = "7.0" ]; }; then break; fi

        echo "Extension: $extension"

        if [ "$extension" = "redis" ]; then
            brew remove redis@$version || true
            brew remove igbinary@$version || true
            brew remove msgpack@$version || true
        fi

        brew reinstall shivammathur/extensions/$extension@$version
    done

    php -v
    for extension in $EXTENSIONS
    do
        php -m | grep $extension
    done
done
``

**Expected behavior**
No warnings

**Screenshots/Logs**
<img width="699" alt="CleanShot 2022-07-12 at 19 55 24@2x" src="https://user-images.githubusercontent.com/995707/178560543-5107e89d-1c14-4177-af47-945521bcfb5a.png">

**Additional context**
-

**Are you willing to submit a PR?**
-

Sorry for the breaking change.
Please run php --ini.
If you have both igbinary.ini and 20-igbinary.ini, delete the igbinary.ini. Same for msgpack and imap.


Dev test: Graphing Quadratics 1.2.0-dev.2

@KatieWoe, @kathy-phet, @amanda-phet, graphing-quadratics/1.2.0-dev.2 is ready for dev testing.
This is a full dev test for PhET and PhET-iO brands.
Document issues in https://github.com/phetsims/graphing-quadratics/issues and link to this issue.
Assigning @kathy-phet and @KatieWoe for prioritization.

General Dev Test

What to Test

Click every single button.
If there is sound, make sure it works.
Make sure you can't lose anything.
Play with the sim normally.
Try to break the sim.
Try to include browser version numbers
If there is a console available, check for errors and include them in the Problem Description.
Run through the string tests on at least one platform, especially if it is about to go to rc.
Check the Game Up harness on one platform.

General Dev Test Platforms

[x] Latest macOS, Safari (Time = 2.75)
[x] Windows 10, Chrome (Time = 0.5)
[x] Windows 10, Firefox (Time = 1.5)
[x] Latest iOS, Safari (Time = 0.25)

Light testing, or optionally skip if time crunch:

[x] Latest macOS, Chrome (Time = 1)
[x] Latest Chrome OS, Chrome (Time = 0.5)

Link(s)

Simulation

PhET-iO Dev Test

What to Test

Make sure that public files do not have password protection. Use a private browser for this.
Make sure that private files do have password protection. Use a private browser for this.
Make sure standalone sim is working properly.
Make sure the wrapper index is working properly.
Make sure each wrapper is working properly.
Launch the simulation in Studio with ?stringTest=xss and make sure the sim doesn't navigate to youtube
For newer PhET-iO wrapper indices, save the "basic example of a functional wrapper" as a .html file and open it. Make
sure the simulation loads without crashing or throwing errors.

General Dev Test Platforms

[x] Latest macOS, Safari (Time = 4.25)
[x] Latest macOS, Chrome(Time = 0.5)
[ ] Latest iOS, Safari (Time = )
[ ] Windows 10, Chrome (Time = 2)
[ ] Windows 10, Firefox (Time = )
[ ] Latest Chrome OS, Chrome (Time = )

Link(s)

Wrapper Index

FAQs for QA Members

There are multiple tests in this issue... Which test should I do first?
Test in order! Test the first thing first, the second thing second, and so on.

How should I format my issue?

We typically assign the developer who opened the issue in the QA repository.

My question isn't in here... What should I do?
You should:

Consult the QA Book.
Google it.
Ask Katie.
Ask a developer.
Google it again.
Cry.

Calling QA here.

Thanks @KatieWoe and QA team.
I'll leave this open until the 1.2 release branch has been created.
There is one issue that has not been addressed, https://github.com/phetsims/graphing-quadratics/issues/154. Since that's a change to a client guide, I could proceed with creating the release branch, then patch any changes into that branch later.

Release branch has been created. Closing.


[FLINK-26687][Connectors][NiFi] Remove Apache NiFi connector

What is the purpose of the change

Remove Apache NiFi connector

Brief change log

Removed connector code
Removed documentation
Removed from build/tests

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): yes
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@flinkbot run azure

Tests passed in https://dev.azure.com/apache-flink/apache-flink/_build/results?buildId=33240&view=results


add practice

This pull request has been deployed to Vercel.

Latest commit:
de6ab9f

✅ Preview:
https://sui-bootcamp-2024-jundqg9ke-vbi-academy.vercel.app

🔍 Inspect:
https://vercel.com/vbi-academy/sui-bootcamp-2024/DE1eYS9jTjii7ogQpNK9kB7q58Hz

View Workflow Logs


Env var for using specific Python version

It'd be great to have something like CONDA_PYTHON_VERSION or so to be able to explicitly request for example 3.6 instead of 3.7. In my case, one of my dependencies doesn't build on 3.7 yet, and conda upgraded from 3.6 to 3.7, so now my build is broken with seemingly nothing I can do.

You can use conda create -n envname anaconda python=3.6 and instead of using activate use activate envname. Would probably only be a very small change actually.

Actually, I think anaconda shouldn't be in that command. Documentation is a bit weird.

I see. I will test a bit.
Shall we recognize and support Python 2 as well?

Well, that'll make things slightly more complex I suppose, but it's a logical next step letting people use 2 for "latest python 2"

My idea was to download the Python 2 miniconda script if the version begins with 2, and if it's exactly 2 just use the latest, otherwise install a version explicitly.

Sounds good yeah

Let's first focus on the Python 3 aspects.

Tested adding a call like conda install python=<version>. This doesn't "just work" with older versions like 3.4:
Warning: 'conda-forge' already in 'channels' list, moving to the top
Solving environment: failed

UnsatisfiableError: The following specifications were found to be in conflict:
  - conda[version='>=4.5.11'] -> pycosat[version='>=0.6.3'] -> python[version='>=3.6,<3.7.0a0']
  - python=3.4
Use "conda info <package>" to see the dependencies for each package.

I don't know how to map "conda script version" to "works for python version". Ideas welcome.

My idea was more based around #5.


geting "no meshes to process" error

Hi
I just downloaded v-hacd blender add-on with testVHACD.exe for 64 bit windows.
I load an obj file to blender then select it in blender object mode then in tools--> V-HACD
i set testVHACD.exe path and click V-HACD button but I got error saying "no meshes to process".
Can you please help?

Have you tried the different binaries? bin/win32, bin-no-ocl/win32(and 64)?
(you might not have opencl support for instance).
There is a chance that you are not getting any output from the underlying .exe of the script.
If you in blender go to Window->"Toggle System Console" and then run VHACD as before.
In it it should perform a couple of different steps Convex Decomposition, Merge Convex Hulls for example and report back progress.
If you get an error message (which I suspect you will) it might be more informative.

Thanks for fast reply
yes i did test with bin/win32, win64 , bin-no-ocl/win32 , win64
here is what i get in blender console:
Exporting mesh for V-HACD: C:\Users\IDEA\AppData\Local\Temp\Suzanne.off...
Running V-HACD...
C:\Program Files\Blender Foundation\Blender\2.77\testVHACD.exe --input "C:\Users\IDEA\AppData\Local\Temp\Suzanne.off" --resolution 100000 --depth 20 --concavity 0.0025 --planeDownsampling 4 --convexhullDownsampling 4 --alpha 0.05 --beta 0.05 --gamma 0.00125 --pca 0 --mode 0 --maxNumVerticesPerCH 32 --minVolumePerCH 0.0001 --output "C:\Users\IDEA\AppData\Local\Temp\Suzanne.wrl" --log "C:\Users\IDEA\AppData\Local\Temp\Suzanne_log.txt"

'C:\Program' is not recognized as an internal or external command,
operable program or batch file.

C:\Users\IDEA\AppData\Local\Temp\Suzanne_log.txt  not created for me!!

Yeah okay, so if we look at the output. The software (blender or python) seem to have an issue with the space in the C:\Program Files\ ... and thinks the C:\Program part is a program to be executed. For you, moving the testVHACD.exe to somewhere else, where the filepath does not contain a space will fix it for you in the short run.
In otherwords move testVHACD.exe to somewhere else (like make a new folder under C called VHACD and place the exe there instead, so that the filepath becomes C:\VHACD\testVHACD.exe )
Hopefully that will fix your problem for now.

thanks it fixed that error. but now getting another error.
I close this issue and will open a new issue for that.


add missing slash to path for llvm-cov bin

Fixing missing slash in path to llvm-cov binary, so that slather can successfully invoke llvm-cov.
In case it matters, these is the environment under which slather is executed:

bash-3.2$ xcodebuild -version
Xcode 7.1.1
Build version 7B1005
bash-3.2$ ruby -v
ruby 2.0.0p481 (2014-05-08 revision 45883) [universal.x86_64-darwin14]

As it has already been fixed in branch feature-profdata, I will close this PR now.


Testmyapp v0.0.65

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@stephengillie what could cause and is it a blocker?
Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

https://github.com/microsoft/winget-pkgs/pull/148685#issuecomment-2052198311