[Request] CFG/CFP GUI like flags GUI, or show entries set for player/group in the flags GUI

Would be nice if these either showed in the flags GUI via context hover, or had their own GUI entirely.
Currently the wiki states Gets/Sets flag permission for a group in claim you are standing in. but passing only /gd flag group [group] or /cfg [group] only shows usage.
Sure you can check it in Luckperms, but it would be nicer to see things in-game in the context of GD usage.

Fixed in commit https://github.com/bloodmc/GriefDefender/commit/61a9d8efd1d03a7a3010828ef4b6992414cc73ae


Cmake build error on windows 10 (fatal error C1001)

System information

Have I written custom code: No
OS Platform and Distribution: Windows 10
TensorFlow installed from:source
TensorFlow version:master
Bazel version:NA
CUDA/cuDNN version:8.0/5.1
GPU model and memory:GTX 1080
Exact command to reproduce:MSBuild /p:Configuration=Release tf_python_build_pip_package.vcxproj

Describe the problem
I was following cmake build instruction
https://github.com/tensorflow/tensorflow/tree/master/tensorflow/contrib/cmake
to build python package with GPU support using visual studio 2015 Update 3. It took a few hours until I hit fatal error C1001.
logs
c:\work\libraries_local\tensorflow\tensorflow\core\kernels\reduction_ops_gpu.cu.cc(64): fatal error C1001: An internal error has occurred in the compiler. [C:\work\libraries_local\tensorflow\tensorflow\contrib\cmake\build\tf_core_gpu_kernels.vcxproj]

  (compiler file 'f:\dd\vctools\compiler\utc\src\p2\main.c', line 255)

   To work around this problem, try simplifying or changing the program near the locations listed above.

Ah, an internal compiler error in the main Eigen statement.  Not sure what can be done here other than using a newer version of Visual Studio.
@mrry: We don't seem to have documentation on which Visual Studio versions work.  I realize none of them are supported, but is there a place I can refer to about unsupported versions?

https://github.com/tensorflow/tensorflow/blob/master/tensorflow/contrib/cmake/README.md talks about known-good configurations, but I'm pretty sure we build with VS 2015 Update 3 regularly.
Perhaps the introduction of DT_COMPLEX support in https://github.com/tensorflow/tensorflow/commit/88a6cdeb5ee79765462932a611d4d16dd715007c has broken the GPU build on Windows?

@guileryu01 Can you disable the complex instantiation and see if it goes through?

Thank you. Sure I will give it a try. How do I disable complex instantiation? Is that one of the options when generating solution and project files with cmake?
Below is the command I was using
cmake ..  Many thanks!

@girving
I observed the same thing as @guileryu01 and have a similar setup.
Have I written custom code: No
OS Platform and Distribution: Windows 10
TensorFlow installed from:source
TensorFlow version:master
Bazel version:NA - (Using Cmake and Visual Studio 2015)
CUDA/cuDNN version:8.0/5.1
GPU model and memory:GTX 1060 6Gb

Exact command to reproduce:
c:>cmake .. -A x64 -DCMAKE_BUILD_TYPE=Release ^
-DSWIG_EXECUTABLE=C:/swigwin-3.0.12/swig.exe ^
-DPYTHON_EXECUTABLE=C:/Users/tknight/AppData/Local/Programs/Python/Python35/python.exe ^
-DPYTHON_LIBRARIES=C:/Users/tknight/AppData/Local/Programs/Python/Python35/libs/python35.lib ^
-Dtensorflow_ENABLE_GPU=ON ^
-DCUDNN_HOME="C:\cuDNN" ^
-Dtensorflow_WIN_CPU_SIMD_OPTIONS=/arch:AVX

c:>MSBuild /p:Configuration=Release tf_python_build_pip_package.vcxproj

results in:
\tensorflow\core\kernels\reduction_ops_gpu.cu.cc(64): fatal error C1001: An internal error h as occurred

after about 3 hours of building.
If I remove the line regarding arch:AVX in the cmake command
-Dtensorflow_WIN_CPU_SIMD_OPTIONS=/arch:AVX

The python wheel builds ok.
I want to help getting the builds with CPU optimizations to work for Windows -
I wasn't exactly clear what you were asking to be done.  I tried commenting out the six lines you referenced, and this resulted in 31 linking errors instead. - I have clipped them from the console output and put them in build_errors.txt.
Let me know if/how to continue..

I ran a build with
cmake .. -A x64 -DCMAKE_BUILD_TYPE=Release ^
-DSWIG_EXECUTABLE=C:/swigwin-3.0.12/swig.exe ^
-DPYTHON_EXECUTABLE=C:/Users/tknight/AppData/Local/Programs/Python/Python35/python.exe ^
-DPYTHON_LIBRARIES=C:/Users/tknight/AppData/Local/Programs/Python/Python35/libs/python35.lib ^
-Dtensorflow_WIN_CPU_SIMD_OPTIONS=/arch:AVX

MSBuild /p:Configuration=Release tf_python_build_pip_package.vcxproj /fl1 /fl2 /fl3 /flp2:logfile=JustErrors.log;errorsonly /flp3:logfile=JustWarnings.log;warningsonly

and it built ok (no errors), so the problem appears to be limited to the combination of tensorflow with GPU and arch:AVX.

@mrry , After commenting those lines out it built ok,
"C:\tools\msys64\home\tknight\tensorflow\tensorflow\contrib\cmake\build\tf_python_build_pip_package.vcxproj" (default target) (1) ->
(PostBuildEvent target) ->
  EXEC : warning : no files found matching '*.so' under directory '*' [C:\tools\msys64\home\tknight\tensorflow\tensorflow\contrib\cmake\build\tf_python_build_pip_package.vcxproj]

    4632 Warning(s)
    0 Error(s)

Time Elapsed 03:50:15.38

 Directory of C:\tools\msys64\home\tknight\tensorflow\tensorflow\contrib\cmake\build\tf_python\dist

05/01/2017  03:51 PM    <DIR>          .
05/01/2017  03:51 PM    <DIR>          ..
05/01/2017  03:51 PM        51,751,621 tensorflow_gpu-1.1.0rc2-cp35-cp35m-win_amd64.whl
               1 File(s)     51,751,621 bytes

installed wheel and it is functional.   next steps...

Looks resolved. Please re-open if not...


Pathfinding doesn't account for entities being able to step up blocks

In PreciseGroundNodeGenerator.java, blocks such as slabs, trapdoors, and carpets are not considered for path nodes. I believe this is an important feature, especially with the addition of the step height attribute.
This issue is firstly due to this method used to check if an entity can move from one position to another:
@Override
public boolean canMoveTowards(@NotNull Instance instance, @NotNull Point startOrg, @NotNull Point endOrg, @NotNull BoundingBox boundingBox) {
    var end = endOrg.add(0, Vec.EPSILON, 0);
    var start = startOrg.add(0, Vec.EPSILON, 0);

    Point diff = end.sub(start);
    PhysicsResult res = CollisionUtils.handlePhysics(instance, instance.getChunkAt(start), boundingBox, Pos.fromPoint(start), Vec.fromPoint(diff), null, false);
    return !res.collisionZ() && !res.collisionY() && !res.collisionX();
}

I do not know how to deal with the registry appropriately to fix this or the maths required to calculate if it can move when stepping is included (some annoying things to consider: you can't step if there's a block above your head, for example)
Additionally, the gravitySnap method used by lines 35-41 does not step upwards (and the lines here do not account for that:)
double floorPointX = current.blockX() + 0.5 + x;
double floorPointY = current.y();
double floorPointZ = current.blockZ() + 0.5 + z;

var optionalFloorPointY = gravitySnap(instance, floorPointX, floorPointY, floorPointZ, boundingBox, MAX_FALL_DISTANCE);
if (optionalFloorPointY.isEmpty()) continue;
floorPointY = optionalFloorPointY.getAsDouble();

Line 36 should probably be changed to double floorPointY = current.y() + maxStepHeight; or something similar to account for this.
And finally the GroundNodeFollower should be changed to implement this:
if (dy <= maxStepHeight) entity.refreshPosition(position.add(0, dy, 0));
final var physicsResult = CollisionUtils.handlePhysics(entity, new Vec(speedX, 0, speedZ));
this.entity.refreshPosition(Pos.fromPoint(physicsResult.newPosition()).withView(yaw, pitch));

It may be more appropriate to change the physics handler, however, but this issue is mainly from the perspective of a pathfinding issue.

I am adding a comment here because I found a related issue.
Blocks with different height such DIRT_PATH or the SOUL_SAND block break the Navigator's pathfinding too.


add community repo

UOR Requires a central hub for organizing community building resources, discussions, and projects.
This PR will allow for instantiating an organization level GH Discussions board amongst facilitating other community activity as well.

/cc @christianh814

I concur with this change, but do not know how to validate this configuration.

@usrbinkat with the uor-community org now in use, can we close this?

@afflom we still need this. We can box in the scope more narrowly but see kubernetes/community for examples of types of content we may place here that does not belong directly in uor-community

@usrbinkat Any thoughts on converting the emporous-docs repo to community instead? I know that we are not 100% sure that the function of the repo is and it seems like the PR against it are something that could go into the community repo. Thoughts?

@usrbinkat, Can you please rebase this and use the previously(https://github.com/uwu-tools/peribolos/blob/241265ba26e46bd1285f170b7a36c664364f56d8/README.md?plain=1#L43) flag to denote the emporous-docs to be changed to community? Please reach out if you have any questions.


Adding native build support to client in maven archetype

🆕🐥:dog: First Timers Only
This issue is reserved for people who never contributed to Open Source before. We know that the process of creating a pull request is the biggest barrier for new contributors. This issue is for you 💝
:space_invader: Description of the issue
Currently the pom of the client module do not provide a native build support. This can easily be done by adding the javafx-maven-plugin plugin to the build. Here is an example how the plugin can be used:
<plugin>
                <groupId>com.zenjava</groupId>
                <artifactId>javafx-maven-plugin</artifactId>
                <version>8.7.0</version>
                <configuration>
                    <mainClass>PATH.TO.MAIN.CLASS</mainClass>
                    <vendor>Canoo Engineering AG</vendor>
                    <appName>JavaFX-Application</appName>
                    <needShortcut>true</needShortcut>
                    <needMenu>true</needMenu>
                </configuration>
                <executions>
                    <execution>
                        <id>create-jfxjar</id>
                        <phase>package</phase>
                        <goals>
                            <goal>build-jar</goal>
                        </goals>
                    </execution>
                    <execution>
                        <id>create-native</id>
                        <phase>package</phase>
                        <goals>
                            <goal>build-native</goal>
                        </goals>
                    </execution>
                </executions>
            </plugin>

This issue is related to the maven archetype for the Dolphin Platform that can be found here: https://github.com/canoo/dolphin-platform-spring-boot-archetype
📋 Step by Step

🙋 Claim this issue: Comment below.
🔄 replace the up for grabs label with in progress.
:fork_and_knife: fork the repository in github by simply clicking the 'fork' button.
:arrow_heading_down: check out the forked repository
:twisted_rightwards_arrows: create a feature branch for the issue. We do not have any naming definition for branches.
💾 Commit your changes.
🔀 Start a Pull Request.
🏁 Done 👍 Ask in comments for a review :)
🔬 If the reviewer find some missing peaces or a problem he will start a discussion with you and describe the next steps how the problem can be solved.
:boom: you did it! We will merge the fix in the master of the Dolphin Platform project.
:bouquet: Thanks, thanks, thanks for being part as an open source contributor for Dolphin Platform

🤔❓ Questions
You can join our community chat at Slack

can I work on this issue @hendrikebbers

@codedsun sure :)

@hendrikebbers  Guide me for the pom file and where the following code is to be added.
Thanks.

@codedsun have a look at this repository https://github.com/canoo/dolphin-platform-spring-boot-archetype
It is a maven archetype. If you are not familiar with archetypes check this doc: http://maven.apache.org/guides/mini/guide-creating-archetypes.html

@hendrikebbers, hello I need to ask where to add this code?

@codedsun I already added the link to the repo? What exactly is your problem? Never worked on an maven archetype?

nopes :(

I have to add this whole code as it is to pom.xml?


nested for loop in f-string

Snakemake version
Bug in latest versions. Tested versions 8.7.0 and 8.11.3.
Works in version 7.18.2
Describe the bug
I often use nested 'for' loops to create a list (outside of a rule). This works fine, but if the loop is placed WITHIN an f-string it does not work with the latest versions of snakemake. Before version 8 it was working.
Logs
$ snakemake -j1
SyntaxError in file Snakefile, line 6:
invalid syntax. Perhaps you forgot a comma? (Snakefile, line 6)
Minimal example
works for both versions
a = [(x, y) for x in [1,2,3] for y in [3,1,4]]
print(f"works for both versions: {a}")
works for older versions only
print(f"works only in old version: {[(x, y) for x in [1,2,3] for y in [3,1,4]]}")

Snakemake states an error in your Snakefile. Please provide a minimal example to produce your error using the current version of Snakemake.

Yes... This is indeed a bug that caused by f-string.
In brief, python will split each 'word' in fstring such as f"works only in old version: {, [, (, x, ,, y, ), for, x, in ...
however, the space between each word are missed. Snakemake read smk file using python parser, and this is difficult to keep it.
This bug will be fixed in the future, but it is also better to use f-string in a more gentle way :)


Added ServiceEndpoint for Microsoft.Storage to the generator for development

Which issue this PR addresses:
https://dev.azure.com/msazure/AzureRedHatOpenShift/_workitems/edit/14178476
What this PR does / why we need it:
This gets us passed this error when deploying the shared RP env and gets the template updated to match production.
{"statusCode":"BadRequest","serviceRequestId":null,"statusMessage":"{"error":{"code":"SubnetsHaveNoServiceEndpointsConfigured","message":"Subnets rp-subnet of virtual network /subscriptions/26c7e39e-2dfa-4854-90f0-6bc88f7a0fb8/resourceGroups/v5-eastus/providers/Microsoft.Network/virtualNetworks/rp-vnet do not have ServiceEndpoints for Microsoft.Storage resources configured...
Test plan for issue:
This has been tested multiple times doing shared RP setups from localhost.
Ran 'make generate'.
Is there any documentation that needs to be updated for this PR?
Documentation for this and other items related to the shared RP will be included in a separate PR.
@karanmagdani -- we ran into this a while back, please advise. Thanks!
Replaces PR: https://github.com/Azure/ARO-RP/pull/2067

Added folks from the old PR for reference: @m1kola @bennerv

/azp run ci


Gui labels

based on #605

i've build + run, it seems that adding a label on outgoing tx not confirmed will not save the label.

I've built + run, it seems that adding a label on outgoing tx not confirmed will not save the label. edit: only from the transaction menu

I dig more into that:

the label is well saved but the displayed label in transaction menu is not updated, i need to go to another menu and then come back to transaction menu to update the label.
I wonder why we got a payment label and a transaction label? can a payment contain several tx? it's not really clear to me...


Update zb.py

new api url / via @abit

api.zb.com still works.
May be we should first try .com and then .cn, or make the url configurable.

interesting.. i think it was down for a while, as was reported by @abit, hence my update / pull :)

On 30 Jul 2019, at 22:16, Zapata<EMAIL_ADDRESS>wrote:
api.zb.com still works.
May be we should first try .com and then .cn, or make the url configurable.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/Zapata/bitshares-pricefeed/pull/2?email_source=notifications&email_token=AAB4SAEZHW7VSZAJJYHDVVDQCCOTHA5CNFSM4HNPZQF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3FF4QY#issuecomment-516578883, or mute the thread https://github.com/notifications/unsubscribe-auth/AAB4SADPMYBKEOQFWOWJ4C3QCCOTHANCNFSM4HNPZQFQ.

I have improved your PR a little to make the base URL configurable:
zb:
    klass: Zb
    url: 'http://api.zb.cn'
    quotes:
        - BTS
    bases:
        - QC

See https://github.com/Zapata/bitshares-pricefeed/commit/b90fc3db6ab114be0e7c01ca1b3e1aba79d0a9f2


Simplify aggregate_tests.py

Description of the changes being introduced by the pull request:
There is a simpler way to skip modules or particular tests
built-in into the unittest module.
That's why it doesn't make sense for us to manually filter
modules based on the python version we are running.
Signed-off-by: Martin Vrachev<EMAIL_ADDRESS>
[x] The code follows the Code Style Guidelines
[ ] Tests have been added for the bug fix or new feature
[ ] Docs have been added for the bug fix or new feature

Cool stuff! I'd even go so far and and remove the glob+ for loop. unittest has built-in test loader discovery function for that.

Btw. here's said test loader in action: https://github.com/in-toto/in-toto/blob/develop/tests/runtests.py
@MVrachev, do you have capacities to finalize this PR?

Yes, sorry. I was busy with my other pr Delegate port generation to the OS.
I used a discover unittest.TestLoader().discover() successfully, I just wanted to find a way how to randomize the unit tests order.

TBH I'm not convinced about randomizing unit test modules invocation order. I have the feeling that, if anything, it might only randomly reveal errors in the test code and not in the tested code, and its ability to do so is very limited, because test cases and functions in a module are still executed in a non-random order. And all that at the cost of determinism. 🤷
What do others think, @joshuagl, @jku, @sechkova?

mixed feelings: finding issues in tests earlier rather than later is usually better (and randomization can help with that) but ... flaky tests are also the worst. :shrug: indeed

Okay, for now, I will just upload the version with discover until we wait for the opinion of Joshua and Tedi.
I don't have strong feelings about the order of the tests.

I don't have a strong opinion as long as we don't end up randomly executing test cases from different modules which is discouraged by the docs.

TBH I'm not convinced about randomizing unit test modules invocation order. I have the feeling that, if anything, it might only randomly reveal errors in the test code and not in the tested code, and its ability to do so is very limited, because test cases and functions in a module are still executed in a non-random order. And all that at the cost of determinism. 🤷

Agreed, I don't think the way we randomise test invocation order adds any value.

Let's drop randomization then.


Windows builds on CI are failing with EmitC enabled

Windows builds on our GitHub Actions CI are failing after enabling EmitC by default (https://github.com/google/iree/pull/6717): https://github.com/google/iree/runs/3299480891?check_suite_focus=true
Logs are a bunch of this:
FAILED: iree/vm/test/emitc/CMakeFiles/iree_vm_test_emitc_module_test.dir/module_test.cc.obj
  C:\PROGRA~2\MICROS~1\2019\ENTERP~1\VC\Tools\MSVC\1429~1.300\bin\Hostx64\x64\cl.exe  /nologo /TP -DIREE_HAVE_EMITC_DIALECT -ID:\a\iree\iree\main_checkout -ID:\a\iree\iree\iree-build -ID:\a\iree\iree\main_checkout\third_party\googletest\googlemock\include -ID:\a\iree\iree\main_checkout\third_party\googletest\googlemock -ID:\a\iree\iree\main_checkout\third_party\googletest\googletest\include -ID:\a\iree\iree\main_checkout\third_party\googletest\googletest /DWIN32 /D_WINDOWS  /EHsc /MD /O2 /Ob2 /DNDEBUG /W3 /DWIN32_LEAN_AND_MEAN /DNOMINMAX /D_USE_MATH_DEFINES /D_CRT_SECURE_NO_WARNINGS /D_CRT_SECURE_CPP_OVERLOAD_STANDARD_NAMES /GR- /bigobj /wd4200 /wd4018 /wd4146 /wd4244 /wd4267 /wd4005 /wd4065 /wd4141 /wd4624 /wd5105 -std:c++14 /showIncludes /Foiree\vm\test\emitc\CMakeFiles\iree_vm_test_emitc_module_test.dir\module_test.cc.obj /Fdiree\vm\test\emitc\CMakeFiles\iree_vm_test_emitc_module_test.dir\ /FS -c D:\a\iree\iree\main_checkout\iree\vm\test\emitc\module_test.cc
  D:\a\iree\iree\iree-build\iree/vm/test/emitc/arithmetic_ops.h(16): error C2229: struct 'arithmetic_ops_state_t' has an illegal zero-sized array
  D:\a\iree\iree\iree-build\iree/vm/test/emitc/arithmetic_ops.h(17): error C2229: struct 'arithmetic_ops_state_t' has an illegal zero-sized array
  D:\a\iree\iree\iree-build\iree/vm/test/emitc/arithmetic_ops.h(18): error C2229: struct 'arithmetic_ops_state_t' has an illegal zero-sized array
  D:\a\iree\iree\iree-build\iree/vm/test/emitc/arithmetic_ops.h(736): error C2466: cannot allocate an array of constant size 0
  D:\a\iree\iree\iree-build\iree/vm/test/emitc/arithmetic_ops_f32.h(16): error C2229: struct 'arithmetic_ops_f32_state_t' has an illegal zero-sized array
  D:\a\iree\iree\iree-build\iree/vm/test/emitc/arithmetic_ops_f32.h(17): error C2229: struct 'arithmetic_ops_f32_state_t' has an illegal zero-sized array
  D:\a\iree\iree\iree-build\iree/vm/test/emitc/arithmetic_ops_f32.h(18): error C2229: struct 'arithmetic_ops_f32_state_t' has an illegal zero-sized array
  D:\a\iree\iree\iree-build\iree/vm/test/emitc/arithmetic_ops_f32.h(1473): error C2466: cannot allocate an array of constant size 0

cc @marbre

Ah yes - in those cases we need to emit a dummy field (when there is no global state)

Here  is the Code generating these arrays.
std::max(...global_bytes(), 1) should work. (Simply skipping arrays of size 0 is another Option)


Support for #region folding

Hello, I would really like to see support for #region folding for Puppet manifests, as described here:
https://github.com/Microsoft/vscode-python/issues/33
This is supported out-of-box by VSCode and only a minor change is required to make it work it with Puppet manifests:
"folding": {
	"offSide": true,
	"markers": {
		"start": "^\\s*#region\\b",
		"end": "^\\s*#endregion\\b"
	}
}

The code is taken from aforementioned issue and tested locally.

There is also an option to add syntax aware folding (as opposed to the default indentation and maker style)
https://code.visualstudio.com/docs/editor/codebasics#_folding
https://code.visualstudio.com/docs/extensionAPI/vscode-api#FoldingRangeProvider
This would require upping the minimum VSCode version to 1.22
Although I'm yet to find a compelling reason to go down this route.  Something to think about.

Fixed and will be available next release!

Great, thank you :+1:
I think we can close this.


Test cleanup so playback and live tests properly work.

/azp run go - azeventgrid

/azp run go - azeventgrid

/azp run go - azeventgrid - basic


A pub package autoroll caused CocoaPods resolution(?) failures

The roll here https://github.com/flutter/flutter/pull/156440
Caused failures like https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8734597920760676097/+/u/run_platform_views_scroll_perf_ad_banners__timeline_summary/stdout
[2024-10-08 18:00:22.743431] [STDOUT] stdout:                Resolving dependencies of `Podfile`
[2024-10-08 18:00:22.743457] [STDOUT] stdout:                  CDN: trunk Relative path: CocoaPods-version.yml exists! Returning local because checking is only performed in repo update
[2024-10-08 18:00:22.743485] [STDOUT] stdout:                  CDN: trunk Relative path: all_pods_versions_5_9_a.txt exists! Returning local because checking is only performed in repo update
[2024-10-08 18:00:22.743515] [STDOUT] stdout:                  CDN: trunk Relative path: Specs/5/9/a/Google-Mobile-Ads-SDK/11.2.0/Google-Mobile-Ads-SDK.podspec.json exists! Returning local because checking is only performed in repo update
[2024-10-08 18:00:22.743591] [STDOUT] stdout:                  CDN: trunk Relative path: all_pods_versions_4_2_c.txt exists! Returning local because checking is only performed in repo update
[2024-10-08 18:00:22.743630] [STDOUT] stdout:                  CDN: trunk Relative path: Specs/4/2/c/FlutterMacOS/3.16.0/FlutterMacOS.podspec.json exists! Returning local because checking is only performed in repo update
[2024-10-08 18:00:22.743647] [STDOUT] stdout:                [!] CocoaPods could not find compatible versions for pod "Google-Mobile-Ads-SDK":
[2024-10-08 18:00:22.743695] [STDOUT] stdout:                  In Podfile:
[2024-10-08 18:00:22.743718] [STDOUT] stdout:                    google_mobile_ads (from `.symlinks/plugins/google_mobile_ads/ios`) was resolved to 5.2.0, which depends on
[2024-10-08 18:00:22.743859] [STDOUT] stdout:                      Google-Mobile-Ads-SDK (~> 11.10.0)

Looks like the pub package roll updated google_mobile_ads from 5.1.0 to 5.2.0

cc @bparrishMines @christopherfujino

IIRC we've had issues before with CI machines not running pod repo update regularly. Cocoapods unfortunately uses a machine-level cache that require explicit commands to update, and the resolution is against that cache, so stale machine caches can cause problems like this.

@jmagman Do you remember how this has been addressed before? Did they get manually updated? Or did we have to add ad-hoc calls to repo update to tests?

Or did we have to add ad-hoc calls to repo update to tests?

Unfortunately yes, that's what we did.

Put out https://flutter-review.googlesource.com/c/recipes/+/60340 to fix this. When that lands, we can follow up and delete all pod repo update references from tests.

The latest roll attempt got further, but also had failing iOS builds: https://github.com/flutter/flutter/issues/156606

When that lands, we can follow up and delete all pod repo update references from tests.

@christopherfujino can it be removed from flutter/packages as well, does that recipe module run for both repos?


Update to Gatling 3.5 and Scala 2.13

Update dependencies to Gatling 3.5 and Scala 2.13, and update code as required to sync with these changes.
(I'm not sure if this project is still actively maintained---it would be good to clarify if it's abandoned so that someone else can take it over with your agreement.)

Merged. Thanks.

I am not able to find a release / published artifacts after this merge.
Any plans around an official release? Is there something i could help with?
@mnogu


Test failure

Describe the bug
A test fails on 1.27.63 and up. Currently using Fedora 36-38, Python3.10 and 3.11.
Expected Behavior
Test passes
Current Behavior
tests/unit/test_client.py:1650: AssertionError
________________ TestClientErrors.test_BOTO_DISABLE_COMMONNAME _________________
self = <tests.unit.test_client.TestClientErrors testMethod=test_BOTO_DISABLE_COMMONNAME>
@mock.patch.dict(os.environ, {'BOTO_DISABLE_COMMONNAME': 'true'})
def test_BOTO_DISABLE_COMMONNAME(self):
    creator = self.create_client_creator()
    self.endpoint_data['sslCommonName'] = 'bar'

    with pytest.warns(None) as warning:
        creator.create_client(
            'myservice', 'us-west-2', credentials=self.credentials
        )

      self.assertEqual(len(warning), 0)

E           AssertionError: 1 != 0
tests/unit/test_client.py:1650: AssertionError
Reproduction Steps
pytest
Possible Solution
No response
Additional Information/Context
No response
SDK version used
1.27.63-65
Environment details (OS name and version, etc.)
Fedora 36-37-38

Hi @limburgher thanks for reaching out. The unit test succeeded in all of the test builds so there may be something in your environment causing the test to fail. There may be another environment variable causing the issue. If you could share any relevant environment variables (with sensitive ones redacted) then that is something we could look more into.

Hi @limburgher thanks for reaching out. The unit test succeeded in all of the test builds so there is likely something in your environment causing this. You could be getting an unrelated warning that is making the test fail. We’d need to know what the warning this which would require a further inspection of your code. Are you able to print the warning value?

2507.patch.txt
build-1.27.65-1.fc38.log
Attaching the only patch we carry and the complete build log.
FYI this is failing during the RPM build, I'm one of the Fedora package maintainers.

Tests are failing in openSUSE as well:
[  354s] =================================== FAILURES ===================================
[  354s] _____________ TestAutoGeneratedClient.test_BOTO_DISABLE_COMMONNAME _____________
[  354s]
[  354s] self = <tests.unit.test_client.TestAutoGeneratedClient testMethod=test_BOTO_DISABLE_COMMONNAME>
[  354s]
[  354s]     @mock.patch.dict(os.environ, {'BOTO_DISABLE_COMMONNAME': 'true'})
[  354s]     def test_BOTO_DISABLE_COMMONNAME(self):
[  354s]         creator = self.create_client_creator()
[  354s]         self.endpoint_data['sslCommonName'] = 'bar'
[  354s]
[  354s]         with warnings.catch_warnings(record=True) as captured_warnings:
[  354s]             creator.create_client(
[  354s]                 'myservice', 'us-west-2', credentials=self.credentials
[  354s]             )
[  354s]
[  354s]         deprecated_endpoint_warnings = [
[  354s]             w
[  354s]             for w in captured_warnings
[  354s]             if w.category == FutureWarning
[  354s]             and 'deprecated endpoint' in str(w.message)
[  354s]         ]
[  354s] >       self.assertEqual(len(deprecated_endpoint_warnings), 0)
[  354s] E       AssertionError: 1 != 0
[  354s]
[  354s] tests/unit/test_client.py:1656: AssertionError
[  354s] ________________ TestClientErrors.test_BOTO_DISABLE_COMMONNAME _________________
[  354s]
[  354s] self = <tests.unit.test_client.TestClientErrors testMethod=test_BOTO_DISABLE_COMMONNAME>
[  354s]
[  354s]     @mock.patch.dict(os.environ, {'BOTO_DISABLE_COMMONNAME': 'true'})
[  354s]     def test_BOTO_DISABLE_COMMONNAME(self):
[  354s]         creator = self.create_client_creator()
[  354s]         self.endpoint_data['sslCommonName'] = 'bar'
[  354s]
[  354s]         with warnings.catch_warnings(record=True) as captured_warnings:
[  354s]             creator.create_client(
[  354s]                 'myservice', 'us-west-2', credentials=self.credentials
[  354s]             )
[  354s]
[  354s]         deprecated_endpoint_warnings = [
[  354s]             w
[  354s]             for w in captured_warnings
[  354s]             if w.category == FutureWarning
[  354s]             and 'deprecated endpoint' in str(w.message)
[  354s]         ]
[  354s] >       self.assertEqual(len(deprecated_endpoint_warnings), 0)
[  354s] E       AssertionError: 1 != 0
[  354s]
[  354s] tests/unit/test_client.py:1656: AssertionError

@glaubitz could you share a bit more information on how you're running the test? I'm unable to reproduce using openSUSE and python in a docker container.

@glaubitz could you share a bit more information on how you're running the test? I'm unable to reproduce using openSUSE and python in a docker container.

You need to run a package build.
To reproduce on openSUSE:
# zypper in osc

and
$ osc co devel:languages:python:aws python-botocore
$ cd python-botocore
$ osc build

Hi @glaubitz, thank you for providing those details. I attempted to reproduce the test failure with your instructions above on a fresh SUSE instance (using the "SUSE Linux Enterprise Server 15 SP4 (HVM)" AMI available on EC2). I am very new to these tools and may be misunderstand the process, but it looks to me as if the build was a success. Here are the last few lines of the output of osc build:
[  194s] ip-172-31-63-218 finished "build python-botocore.spec" at Mon Sep 12 16:57:56 UTC 2022.
[  194s]

/var/tmp/build-root/openSUSE_Tumbleweed-x86_64/home/abuild/rpmbuild/SRPMS/python-botocore-1.27.64-0.src.rpm

/var/tmp/build-root/openSUSE_Tumbleweed-x86_64/home/abuild/rpmbuild/RPMS/noarch/python38-botocore-1.27.64-0.noarch.rpm
/var/tmp/build-root/openSUSE_Tumbleweed-x86_64/home/abuild/rpmbuild/RPMS/noarch/python39-botocore-1.27.64-0.noarch.rpm
/var/tmp/build-root/openSUSE_Tumbleweed-x86_64/home/abuild/rpmbuild/RPMS/noarch/python310-botocore-1.27.64-0.noarch.rpm

I did notice that the command osc co devel:languages:python:aws python-botocore downloads the file botocore-1.27.64.tar.gz. Version 1.27.64 is newer than the version reported as failing tests by @limburgher but older than the version that generated your test output above.
Do you have any suggestions for what I can do to get closer to reproducing the error?
Notes for anyone else attempting to reproduce this:

An account for openSUSE is required as documented on the openSUSE Wiki here.
I had to place this file in my ~/.osc-plugins directory and then used it to osc in obs-service-format_spec_file. I'm sure there is an easier way to achieve this, but this is the one I figured out...

For completeness, all versions since the one I reported have this issue for me.

Hi @glaubitz, thank you for providing those details. I attempted to reproduce the test failure with your instructions above on a fresh SUSE instance (using the "SUSE Linux Enterprise Server 15 SP4 (HVM)" AMI available on EC2). I am very new to these tools and may be misunderstand the process, but it looks to me as if the build was a success.

Apologies, my mistake. I forgot that python-botocore is a multibuild package where the testsuite is a separate package build.
Try building with:
$ osc build -M test

and to ensure it really builds inside Tumblweed, build with:
$ osc build -M test openSUSE_Tumbleweed

@glaubitz Could you please also advise on how I can get osc to use the latest version of botocore instead of 1.27.64?

@glaubitz Could you please also advise on how I can get osc to use the latest version of botocore instead of 1.27.64?

Install py2pack:
# zypper in python3-py2pack

then run:
$ py2pack fetch botocore
$ osc addremove

Edit the spec file to update the version field to the version that you just downloaded. Then try building again.
The current version of python-botocore has not been updated due to this bug report.

The root cause turned out to be something unrelated to the build environments: There had been leakage of test setup between tests. The new test surfaced this for the first time, but only when functional tests are run before unit tests and in the same test session.
#2761 addresses that and, barring further surprises, should fix these test failures once released. I also audited the use of similar code patterns in the botocore test code and made some improvements to reduce the likelihood of something similar happening again.
Note that this test failure was due to testing code only, this was not an issue with botocore functionality itself.
Thank you @limburgher and @glaubitz for reporting the problem and sharing detailed information!

Thank you, applyng 2761 to 1.27.71 works!

Good to hear the issue has been resolved. Could you release a new version with the patch included soonish? Then I can just update the package in openSUSE and don't need to cherry-pick the patch.

Version 1.27.72 was. released approximately an hour ago and contains this change.
Out of curiosity: How often does this process of making new botocore versions in openSUSE and Fedora happen? (In times when there is no test failure that is blocking the process.)

I typically update Fedora rawhide and occasionally rawhide-1 within 48 hours of each release.


Added SpecularGlossiness export

This PR handles importing a material using the KRH_materials_pbrSpecularGlossiness extension.
The node tree generated follows the setup needed for exporting a specular/glossiness material to gltf.

 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.

The addon seems to import the specular material just fine, but upon export the material generated just has the occlusion and emission channels.
Sample model:
After import and immediate export:

Guessing, I think SpecularGlossiness is replaced by https://github.com/KhronosGroup/glTF/blob/main/extensions/2.0/Khronos/KHR_materials_specular/README.md spec.

As said by fire, KHR_materials_pbrSpecularGlossiness is set as archived. (see here )
So, creating new files with this extension is not recommanded (See here )
This is Superseded by KHR_materials_specular, that is currently under implementation in #1646
I so suggest to close this PR

Especially with the upcoming development of Principled BSDF v2, I'd be inclined to agree we should probably not ship support for KHR_materials_pbrSpecularGlossiness here. This feature could (I expect?) be implemented through this add-ons extension API as well.
I would also just flag that KHR_materials_pbrSpecularGlossiness is mutually exclusive with other material-related glTF extensions. Features like clearcoat, iridescence, IOR, volume, transmission, etc. are supported on the metal/rough workflow only. With the addition of KHR_materials_specular, there is no longer need for KHR_materials_pbrSpecularGlossiness.

So ... We agree that this will not be included in official exporter.
Closing.
Thanks!


Option somehow to change server within magisk?

would it be feasible to have a method to change the supl and ntp server within magisk, the only reason being that if theres an update to this repo, magisk's updater would update using the repo's default ntp server in europe which is far from where I am. so all config changes currently wont persist in updates.

I think the easiest solution here would be to add a selection when installing the module (like other modules do it with the volume keys). Do you have any suggestions on which servers to include?

I would say allowing to select just generic pool.ntp.org for quicker ntp time (allow ntp to decide which region the user is in) and the alternative supl providers you have listed (tmobile, telstra, orange, etc) as in my case the tmobile supl responds way faster than the graphene provider does (tmo is my carrier)

I would say allowing to select just generic pool.ntp.org for quicker ntp time (allow ntp to decide which region the user is in) and the alternative supl providers you have listed (tmobile, telstra, orange, etc) as in my case the tmobile supl responds way faster than the graphene provider does (tmo is my carrier and i dont mind sending my imsi to them as they already have it)

Changed the NTP pool to the global one on this pr.
About the SUPL_PORT, it looks the same in my phone, but a quick nmap scan shows the 1234 port is not open on supl.geo.t-mobile.com (but 7275 seems to be).


Add Python and R versions to README

Mambaforge version is unnecessary but R version is - changing

Done but documentation could be clearer

done


Show exception widget for promise rejection

I set different StoppedEvent reasons for exceptions vs promise rejections, and we should show the exception widget for promise rejections too. From #21905
It seems like it's currently only shown when reason === 'exception', but it should be shown either whenever the exception text is set, or when reason === 'promise rejection'

Let's do this after I've added support for a "robust reason attribute" to the DAP (currently the "reason" attribute is localized).

@roblourens I'm a bit reluctant to hardcode an additional StoppedEvent reason into the VS Code frontend. Could you try to use "exception" reason for promise rejections and use the new "description" attribute for conveying the "Promise Rejection" to the UI?
With the "robust reason attribute" work "reason" is no longer shown in the UI if a "description" attribute is present.

@roblourens @weinand While this seems to work the information seems to be pretty useless. I understand that in this case rejection happens with null but at least the error message should talk about 'Rejected Promise' instead of 'Exception has occurred: null'

@michelkaporin we do not seem to show the "description" attribute. Its value would be "Promise Rejection" in this case (and this already shows in the CALL STACK view).

@weinand we do surface description attribute in exception widget. I can see that 'null' is coming from DA for description, and it is surfaced in UI as-is.

Ok, I've tweaked the use of the "description" attribute in node-debug.
Here is how it looks now for different values passed to reject():

BTW, the missing stacktrace bug is tracked by https://github.com/Microsoft/vscode/issues/25555

To align better with the "inspector" debugger, I've changed the wording from "Rejected Promise" to "Promise Rejection":


Typo fixes

Sorry I merge previous branch too fast

They were existing issues with your phpdoc which StyleCI made me notice, so I've sorted them out. :)

:smile: thanks

If you ever have any questions about StyleCI further down the line, feel free to get in touch. :)

Ok, @GrahamCampbell :+1:


How to update label text?

What are the rules around how Label text gets updated in the console? I have a class that extends View, and every time AddValue() is called, it is supposed to update a _divider Label based on the new width of the View. However no matter what I try, including putting the update inside an Application.MainLoop.AddTimer(), the _divider label text doesn't seem to update.
private readonly Label _divider;
public Constructor(string name)
{
    _divider = new Label(string.Empty);
    Add(_divider);
}
public void AddValue(string value)
{
    if (value.Length > _width)
    {
        _width = value.Length;
        Width = _width;
        _divider.Text = string.Empty.PadLeft(value.Length, '-');
    }
    Add(new Label(value) { Y = Pos.Y(this) + this.Subviews.Count });
}

It's better you put all the code to look at it, but it's seems that you are adding another label too. I think this is running after Application.Init and Application.Run of course.

Is it a problem if App, my TopLevel view, is constructed before being passed into Application.Run()?
public class Program
{
    private static Container _serviceProvider;

    public static void Main()
    {
        _serviceProvider = ConfigureServices();
        Application.Init();
        Application.Run(_serviceProvider.GetService<App>());
    }

    private static Container ConfigureServices()
    {
        var container = new Container();
        container.Options.DefaultLifestyle = Lifestyle.Singleton;

        container.Register<App>();
        container.Register<MainView>();

        return container;
    }
}

It's better you put all the code to look at it, but it's seems that you are adding another label too. I think this is running after Application.Init and Application.Run of course.

That is not what I intended to say. I was just saying that you can only update the Label when the application is actually running, that is, after Application.Run. If the issue really is after this instruction, I will have to analyze it with a more or less real code of the problem you are facing. Thanks.

Okay, that may actually answer my question. Because the flow of execution means that the label gets updated while the TopLevel view is still being constructed.

It's possible but it won't be shown, but if it the text Label is setted it will display the new text. But if it doesn't  display try to remove the readonly on the declaration of the Label.

Okay, I figured out my issue. While the Label text was actually changing. I was unaware that the width of the label was set during its construction. And because all I was really doing was attempting to make the _divider longer with additional - characters the issue was hidden.

I'm glad you managed to resolve it.


Added support for .Net Standard 2.0

LOG4NET-572: Add target for netstandard-2.0

Needed log4net for atleast .net standard 2.0.
Problem in publishing app of .net core 3.0 where log4net 2.0,8 is referenced

Needed log4net for atleast .net standard 2.0.
Problem in publishing app of .net core 3.0 where log4net 2.0,8 is referenced

My change is to get a new build for .net standard 2.0 but the current build system doesn't support it. Need a new build system setup.

I am afraid nant is no longer able to cope with this task. We would have to replace all what nant does with something else. This could be msbuild, paired with custom inline build tasks written with c# code. Feel free to go for it! Contributions welcome!

@dpsenner
I don't know whether you got my emails about i've shown my interest in contributing to Log4Net.
I have set up a sample build for Log4Net.standard solution using azure devops. Let me know if its okay to proceed then we can migrate all builds to azure devops.
https://dev.azure.com/logging-log4net/Log4Net/_build/results?buildId=20&view=artifacts

Seems like #59 added .NET Standard 2.0 targets.


Description of Field and Topics

Many section titles are not completely self-explained. In order to improve the ordering of the list every section should be discriped with at least one sentence.
Here an example:
Life-cycle assessment

a methodology for assessing environmental impacts associated with all the stages of the life-cycle of a commercial product, process, or service

Pls close this issue.

Great idea - I would like to build out something like CNCF does with it's landscape to capture this...
https://landscape.cncf.io/guide


vertex groups functionality

Hi!
I would very much like to know if abnormal will have connection with vertex groups in the future?
It would be really wonderful if it had. Any chance that the 'Point at target' function implements an object as a target for good and not only in edit time? Thx!

It already has vertex group support. You can set the vertex group in the blender UI panel below the start modal button and change it through there while in the modal.
Point at target can be done with the Normal Edit modifier. Abnormal is only a temporary mode.


fix: improved = and != with val null for HANA and Postgres

In the case of Postgres all queries doing x = null where being translated to x is $1 which causes trouble as the operator name is is null unlike the SQLite operator which is just is.
The Postgres error was already resolved in the HANA Service, but the SQL Execution plan cache optimization caused trouble when comparing with LOB type columns. As HANA does not support compare operation on LOB columns. There for the only allowed operators on the LOB columns is is (not) null. Which also causes the issue that SQLite and Postgres are both allowed to execute null = (LOB), but for HANA this causes a SQL error. As null = ... always returns null it will always filter everything out. Therefor the HANA Service translate it into null = null as the behavior is the same and does not throw a SQL error.

makes sense to me


[SPARK-36210][SQL] Preserve column insertion order in Dataset.withColumns

What changes were proposed in this pull request?
Preserve the insertion order of columns in Dataset.withColumns
Why are the changes needed?
It is the expected behavior. We preserve insertion order in all other places.
Does this PR introduce any user-facing change?
No. Currently Dataset.withColumns is not actually used anywhere to insert more than one column. This change is to make sure it behaves as expected when it is used for that purpose in future.
How was this patch tested?
Added test in DatasetSuite

Test build #141265 has started for PR 33423 at commit 3b69021.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45779/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45779/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/45779/

Test build #141265 has finished for PR 33423 at commit 3b69021.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/141265/

cc @viirya

@HyukjinKwon Any more comments? If no, I will merge this tomorrow. Thanks.

nope lgtm too!

Seems OK to me. I'm very slightly concerned about the behavior change - column order is different - but seems worth it as a 'fix'.

Thanks. Merging to master/3.2/3.1

I'm not super against back-porting, but is more questionable given the behavior change

Hmm, withColumns is not a public API yet. It is used by a few ML classes to add columns. Although column order is different, I guess that change is less severe? I'm totally okay to revert backported patches.

Oh I see, OK


[Storage] Fix #27052: az storage blob delete-batch: Use utc as default timezone to remove "Datetime with no tzinfo will be considered UTC." warning

Related command

Description

Fix #27052
Use utc as default timezone to remove "Datetime with no tzinfo will be considered UTC." warning
Testing Guide

History Notes

[Storage] Fix #27052: az storage blob delete-batch: Use utc as default timezone to remove "Datetime with no tzinfo will be considered UTC." warning

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Storage


Create a node-exporter rock

          Do we do rocks for sidecars?

Originally posted by @sed-i in https://github.com/canonical/loki-k8s-operator/pull/373#discussion_r1567537522

We have to add update-ca-certificates package in order to be able to have TLS: https://github.com/canonical/loki-k8s-operator/issues/389


There is a bug with one of the dependencies that was updated on November 25, 2020

Describe the bug
I ran composer update, "lcobucci/jwt" has been updated from 3.3.3 to 3.4.0, then authentication doesn't work anymore
To Reproduce
install fresh or update current install and try to verify a token
app('firebase.auth')->verifyIdToken($request->get("jwt"));

Expected behavior
it should auth correctly
Environment (please complete the following information):

OS: Windows 10
PHP version: 7.4.0
kreait/firebase-tokens Version: 1.11.0
firebase/php-jwt Version : v5.2.0
kreait/firebase-php Version: 5.11.0

Additional context
Error message
The token has an invalid signature: Implicit conversion of keys from strings is deprecated. Please use InMemory or LocalFileReference classes.

Stacktrace
\vendor\kreait\firebase-tokens\src\Firebase\Auth\Token\Verifier.php:145
\vendor\kreait\firebase-tokens\src\Firebase\Auth\Token\Verifier.php:68
\vendor\kreait\firebase-php\src\Firebase\Auth\IdTokenVerifier.php:53
\vendor\kreait\firebase-php\src\Firebase\Auth.php:581

I'll try to create a new release in the next few days. In the meantime please see https://github.com/lcobucci/jwt/issues/550 - there are some approaches in the thread on how to handle the deprecation messages that explain them better than I could :)

Fixed with release 1.12 (already since last night, but I forgot to notify you here) 😅
Let me know if there are problems that I missed 🤞

Thanks, it work well 😃


No way to login with a token

There currently doesn't appear to be a way to login to discord with a token on an API application account. :/

Difference is basically calling bot.loginWithToken("token"); instead of bot.login("email", "password");

Set discordEmail to bot ID & discordPassword to bot token, tested this and it works.

Incoming PR fixing this.


Use Phoenix.Components.used_input?/1

Use Phoenix.Components.used_input?/1 to only render errors when the form fields are used.
See https://hexdocs.pm/phoenix_live_view/1.0.0-rc.6/Phoenix.Component.html#used_input?/1
The commit is not backwards compatible with earlier versions of live_view. Any suggestions for this?

Petal Components will be upgraded to 1.0 when it's released. We could merge this PR at that point in time. Is that what you're getting at?

I'm just worried about the backwards compatibility. It's up to what the maintainers want to support. You could do something like (not checked):
  if Code.ensure_loaded?(Phoenix.Component) and function_exported?(Phoenix.Component, :used_input?, 1) do
    defp field_has_errors?(%{form: form, field: field}) when is_map(form) do
      case Keyword.get_values(form.errors || [], field) do
        [] -> false
        _ -> Phoenix.Component.used_input?(form[field])
      end
    end
  else
    defp field_has_errors?(%{form: form, field: field}) when is_map(form) do
      case Keyword.get_values(form.errors || [], field) do
        [] -> false
        _ -> true
      end
    end
  end

  defp field_has_errors?(_), do: false

Or just clearly state it won't work with earlier versions of live_view, which would force people to update.

I think it makes sense that once Live View 1.0 is released, it'll become the base line for future development (which would include used_input?/1). Once there is a release that refers to Live View 1.0, then users will either need to upgrade their project to Live View 1.0 or use an older version of Petal Components.
When the time comes, we'll update the change log and the upgrade notes (with regards to the change to Live View 1.0). However, if this causes a pain point for people, then we can we can certainly re-evaluate what our options are at that point.

I'm just going to close this for now. If you want to add anything or re-raise the issue around the release for 1.0, feel free to do so

@mitkins @joepstender Now that Phoenix LiveView 1.0 has been officially released, I think it’s a good time to revisit it.

I will see if I can make a PR.

PR would be welcome! I've reopened this for now.

I'm going through Petal Components right now - with regards to phx-feedback-for and used_input?/1

I updated my PR, don't know how to fix the mix.lock conflict?

Hi @joepstender. I've used your updates in the related PR to update Forms V1 and V2 to LiveView 1.0


Defensive abilities named Resist ____ cause issues.

Jeslyn
CR 6
XP 2,400
Female half-elf oracle 7
CE Medium humanoid (elf, human)
Init +1; Senses low-light vision; Perception +11

DEFENSE
AC 22, touch 15, flat-footed 20 (+6 armor, +3 deflection, +1 Dex, +1 dodge, +1 natural)
hp 80 (7d8+45)
Fort +8, Ref +5, Will +6; +4 vs. disease, +2 vs. enchantments
Defensive Abilities resist life; Immune disease, sickness, sleep
Weaknesses sickened

OFFENSE
Spd 30 ft.
Melee +1 scythe +9 (2d4+5/x4)
Space 5 ft.; Reach 5 ft.
Special Attacks soul siphon 1/day

Oracle Spells Known (CL 7th; concentration +11)
3rd (5/day)—animate dead, bestow curse (DC 17), inflict serious wounds (DC 17), summon monster III
2nd (7/day)—bull's strength, bear's endurance, false life, hold person (DC 16), inflict moderate wounds (DC 16)
1st (7/day)—cause fear (DC 15), command (DC 15), divine favor, inflict light wounds (DC 15), obscuring mist, sanctuary (DC 15), shield of faith
0 (at will)—bleed (DC 14), detect magic, detect poison, guidance, read magic, resistance, stabilize
Mystery bones

TACTICS
Before Combat Jeslyn casts bull's strength, bear's endurance, false life, and shield of faith once she fears the PCs are drawing near.
During Combat Jeslyn casts divine favor on the first round of combat, then attacks with her scythe. If she's having trouble hitting foes, she abandons the scythe and instead uses her inflict spells. She uses bestow curse to lower ability scores against foes-she's particularly fond of reducing Wisdom scores for clerics, but doesn't bother using spells with saving throws against paladins.
Morale Jeslyn fights to the death.

STATISTICS
Str 16, Dex 13, Con 18, Int 10, Wis 8, Cha 18
Base Atk +5; CMB +8; CMD 23
Feats Craft Magic Arms and Armor, Craft Wondrous Item, Dodge, Martial Weapon Proficiency (scythe), Skill Focus (Perception)
Skills Intimidate +13, Knowledge (planes) +9, Knowledge (religion) +9, Linguistics +3, Perception +11
Languages Abyssal, Common, Elven, Hallit, Necril
SQ oracle's curse (wasting), revelations (armor of bones, resist life, soul siphon)
Combat Gear scroll of dispel magic (3); Other Gear +1 scythe, amulet of natural armor +1, cloak of resistance +2, keys to areas P15 and P18, 350 gp in onyx gems for animate dead, 32 gp

Results in

Rather than when the name was changed:

Not sure if/how much you can do about this.

Yeah, i can fix that by ignoring "resist" (the keyword for resistances) in the defensive abilities line. Fix will be included in 3.3.1

Fixed in 3.3.1


LitElement prevents form submit event from bubbling

Description
On the parent element, it is not possible to listen for the 'submit' event fired by the child element.
Live Demo
lit-element example: https://jsfiddle.net/ao9u3tsw/2/
Pure js example: https://jsfiddle.net/arjv9kt1/
Steps to Reproduce

Create 'child-element' rendering 'form' element
Create 'parent-element' rendering 'child-element'
Add 'submit' event listener to 'child-element' in the template of 'parent-element'

Expected Results
The event listener function to be called.
Actual Results
The event listener function is not called.
Browsers Affected

[x] Chrome

Did not check for the rest.

[ ] Firefox
[ ] Edge
[ ] Safari 11
[ ] Safari 10
[ ] IE 11

Versions

lit-element: v2.2.1

This isn't a LitElement issue. The problem is that the input and the event listener are in different shadow roots, but submit is not a composed event.
You'll need to re-fire the event from the child component, use the form participation API, or take the input and submit button as slotted children.

or take the input and submit button as slotted children

Excuse me, can you elaborate this with a concrete example? Because even then it didn't work for me. I actually have this exact use case from the beginning on, using slotted children, but I'm still facing the issue of the submit event apparently not hitting the form element itself when triggered by a slotted input or button. Here's the reproducer: https://jsfiddle.net/nhxuLpc6/

Potentially related: https://github.com/WICG/webcomponents/issues/1075

I mean slotting the input and button into the custom element, from where the form is:
<form>
  <my-element>
    <input slot="input" type="text">
    <button slot="button" type="submit">Submit</button>
  </my-element>
</form>

Right. Thanks.


Example code: user => users  ?

Hi.
Thanks for this helpful client software.
When working through the examples I noticed that I only got it to work when I changed this:
get(client, "/user/" <> login <> "/repos")
into this
get(client, "/users/" <> login <> "/repos")
Also the API now seems to require setting a user agent:
plug Tesla.Middleware.Headers, %{"User-Agent" => "tesla"}

Hey @DerKastellan
You're right! The README example was never meant to be working, but I'm fine with updating it :)
Can you make z PR with required changes? Please not the new syntax for headers since 1.0 (and current master)

Sorry for the late reply, very busy work week.
Would you be okay with a merge request that actually changes the examples to httpbin.org? I used them to gain more experience with your API and REST and it's a good training ground with less hoops to go through for the novice.

The reason I chose Github was to show the primary use case of Tesla - writing client libraries for APIs, where one would define functions like “users” and “repos”. If you can think of a way to use httpbin while keeping that concept I’m fine with that :)
On 13 Apr 2018, 19:49 +0200, DerKastellan<EMAIL_ADDRESS>wrote:

Sorry for the late reply, very busy work week.
Would you be okay with a merge request that actually changes the examples to httpbin.org? I used them to gain more experience with your API and REST and it's a good training ground with less hoops to go through for the novice.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

@DerKastellan 👋


fix: PR status layout on mobile

This PR fixes the layout of PR status layouts on mobile. For longer status context names or on very small screens the text would overflow and push the "Details" and "Required" badges out of the container.
Before:

After:

How about moving the .pr-status into the Less file's root? This change alone will probably introduce 1-2kB of additional CSS, mostly because of all the useless parent selectors that come from the nesting.

Makes sense, changed!

please send backport


dev docs refer to the shell as "fsh"

The docs here will need to be updated https://github.com/IBM/kui/tree/master/docs/dev for example under quick start
git clone git@github.com:ibm-functions/shell.git
$ cd shell
$ ./tools/setup-npmrc.sh && npm install
$ ./fsh.js

fixed by #118


Update PackageName for WindowsAppRuntime 1.3.2

Related to https://github.com/microsoft/winget-cli/issues/3696

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Exception during executable launch operation System.AggregateException: One or more errors occurred. (Access is denied.)  ---
System.ComponentModel.Win32Exception (5): Access is denied.
--- End of stack trace from previous location where exception was thrown ---    --- End of inner exception stack trace ---    at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)

@wingetbot waivers Add Validation-Executable-Error


Update com.fasterxml.jackson in lavaplayer to 2.9.1

Because in actual version it throws:
Exception in thread "main" java.lang.NoClassDefFoundError: com/fasterxml/jackson/databind/deser/SettableBeanProperty$Delegating

Updated all dependency versions.


Database tags seem isolated from main system tags

Describe the bug
When entering tags into a tag column, added tags do not appear in Obsidian's tag pane. Furthermore, when using Obsidian to search the vault for a specific tag, Database rows using the tag are not reported.
To Reproduce

Designate a column a type "Tags"
Enter a new tag that doesn't exist in Obsidian's tag tree
Observe issue no. 1: the tag added to the database column is not added to Obsidian's tag tree.
In another row of the Tags column, add a tag that DOES exist in Obsidian's tag tree.
Use Tag Browser to locate the tag. Click the tag to initiate a vault search for that tag.
Observe that the search results do not include the Database row tagged with the matching tag.

Expected behavior
The effect of entering a tag into a Database should be the same as adding one to a document. That tag should be searchable and findable using the tag tree.
Screenshots
n/a
Desktop (please complete the following information):

OS: All
Version 2.2.2

hi @pm64
to link the tags to obsidian there are 2 ways:

change your column name to "tags" if you are using a yaml field type
add # to your values if you are using inline field type

As soon as autocomplete issue is finished, this behaviour will be added to the docs

From what I'm observing, the tags remain unlinked even when either of the above conditions are met.

https://user-images.githubusercontent.com/11924043/187052239-96303237-b88d-4d37-8de8-ad1a08d6b5d2.mov
Should be this way

May I close this issue? was your use case resolved?


Problem with h5py while trying to run eval with pretrained model:

cuda 10.1
python 3.6
All the libraries mentioned in the requirements. I have downloaded the model but struggling with the h5py python module while creating the model:
Please note I have added an argument in line 98 in 'model.py'. The error is as follows:
File "eval.py", line 98, in
model = ERC_xlnet(args, n_classes, use_cls = True)
File "/home/ahsanyleg/emotion_proj/DialogXL/DialogXL/model.py", line 98, in init
self.xlnet = XLNetModel.from_pretrained(args.bert_model_dir, from_tf=True)
File "/home/ahsanyleg/anaconda3/envs/dialogxl/lib/python3.6/site-packages/transformers/modeling_utils.py", line 1059, in from_pretrained
model = load_tf2_checkpoint_in_pytorch_model(model, resolved_archive_file, allow_missing_keys=True)
File "/home/ahsanyleg/anaconda3/envs/dialogxl/lib/python3.6/site-packages/transformers/modeling_tf_pytorch_utils.py", line 292, in load_tf2_checkpoint_in_pytorch_model
load_tf_weights(tf_model, tf_checkpoint_path)
File "/home/ahsanyleg/anaconda3/envs/dialogxl/lib/python3.6/site-packages/transformers/modeling_tf_utils.py", line 467, in load_tf_weights
with h5py.File(resolved_archive_file, "r") as f:
File "/home/ahsanyleg/anaconda3/envs/dialogxl/lib/python3.6/site-packages/h5py/_hl/files.py", line 408, in init
swmr=swmr)
File "/home/ahsanyleg/anaconda3/envs/dialogxl/lib/python3.6/site-packages/h5py/_hl/files.py", line 173, in make_fid
fid = h5f.open(name, flags, fapl=fapl)
File "h5py/_objects.pyx", line 54, in h5py._objects.with_phil.wrapper
File "h5py/_objects.pyx", line 55, in h5py._objects.with_phil.wrapper
File "h5py/h5f.pyx", line 88, in h5py.h5f.open
OSError: Unable to open file (file signature not found)

The transformer model was not downloaded properly. Solved the problem by downloading the model with 'git lfs' instead of 'git'.


bug(forge test): --fail-fast flag does not work as it is not applied across multiple test suites

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

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (13af418 2023-12-05T00:21:30.568484000Z)
What command(s) is the bug in?
forge test --fail-fast
Operating System
None
Describe the bug
Running FOUNDRY_FUZZ_RUNS=100000 forge test --fail-fast on the below code should run test_Increment and testFuzz_SetNumber in parallel, and fail test_Increment should fail immediately. However, forge test does not fail and exit until all 100,000 fuzz runs complete
// SPDX-License-Identifier: UNLICENSED
pragma solidity ^0.8.13;

import {Test, console2} from "forge-std/Test.sol";
import {Counter} from "../src/Counter.sol";

contract CounterTest is Test {
    Counter public counter;

    function setUp() public {
        counter = new Counter();
        counter.setNumber(0);
    }

    function test_Increment() public {
        assertTrue(false);
    }

    function testFuzz_SetNumber(uint256 x) public {
        counter.setNumber(x);
        assertEq(counter.number(), x);
    }
}

looking at code it looks like --fail-fast applies for different test suites and not to tests within same suite, considering a test contract like
// SPDX-License-Identifier: UNLICENSED
pragma solidity ^0.8.13;

import {Test, console2} from "forge-std/Test.sol";
import {Counter} from "../src/Counter.sol";

contract ACounterTest is Test {
    Counter public counter;

    function setUp() public {
        counter = new Counter();
        counter.setNumber(0);
    }

    function test_Increment() public {
        assertTrue(false);
    }

    function testFuzz_AnotherSetNumber(uint256 x) public {
        counter.setNumber(x);
        assertEq(counter.number(), x);
    }
}

contract BCounterTest is Test {
    Counter public counter;

    function setUp() public {
        counter = new Counter();
        counter.setNumber(0);
    }

    function testFuzz_SetNumber(uint256 x) public {
        counter.setNumber(x);
        assertEq(counter.number(), x);
    }
}

if running without flag then both contracts are tested (even though test_Increment in first contract fails)
FOUNDRY_FUZZ_RUNS=2000 forge test

Ran 2 tests for test/Counter.t.sol:ACounterTest
[PASS] testFuzz_AnotherSetNumber(uint256) (runs: 2000, μ: 27500, ~: 28366)
[FAIL. Reason: assertion failed] test_Increment() (gas: 10766)
Test result: FAILED. 1 passed; 1 failed; 0 skipped; finished in 97.51ms

Ran 1 test for test/Counter.t.sol:BCounterTest
[PASS] testFuzz_SetNumber(uint256) (runs: 2000, μ: 27630, ~: 28387)
Test result: ok. 1 passed; 0 failed; 0 skipped; finished in 99.50ms

Ran 2 test suites in 197.01ms: 2 tests passed, 1 failed, 0 skipped (3 total tests)

Failing tests:
Encountered 1 failing test in test/Counter.t.sol:ACounterTest
[FAIL. Reason: assertion failed] test_Increment() (gas: 10766)

Encountered a total of 1 failing tests, 2 tests succeeded

when run with  fail fast the 2nd suite is not executed anymore
FOUNDRY_FUZZ_RUNS=2000 forge test --fail-fast

Ran 2 tests for test/Counter.t.sol:ACounterTest
[PASS] testFuzz_AnotherSetNumber(uint256) (runs: 2000, μ: 27540, ~: 28366)
[FAIL. Reason: assertion failed] test_Increment() (gas: 10766)
Test result: FAILED. 1 passed; 1 failed; 0 skipped; finished in 99.20ms

Ran 1 test suite in 99.20ms: 1 tests passed, 1 failed, 0 skipped (2 total tests)

Failing tests:
Encountered 1 failing test in test/Counter.t.sol:ACounterTest
[FAIL. Reason: assertion failed] test_Increment() (gas: 10766)

Encountered a total of 1 failing tests, 1 tests succeeded

looks like a design choice not a bug?

proposed way to interrupt runs for fuzz / invariant tests https://github.com/proptest-rs/proptest/issues/460 to be applied in addition to using rayon collect / short-circuit on Err

Just confirming this is still an active issue, setup as described still yields:
FOUNDRY_FUZZ_RUNS=1000000 forge test --fail-fast
[⠊] Compiling...
No files changed, compilation skipped

Ran 2 tests for test/Counter.t.sol:CounterTest
[PASS] testFuzz_SetNumber(uint256) (runs: 1000000, μ: 31045, ~: 31288)
[FAIL. Reason: assertion failed] test_Increment() (gas: 3071)
Suite result: FAILED. 1 passed; 1 failed; 0 skipped; finished in 22.34s (22.34s CPU time)

Ran 1 test suite in 22.34s (22.34s CPU time): 1 tests passed, 1 failed, 0 skipped (2 total tests)

Failing tests:
Encountered 1 failing test in test/Counter.t.sol:CounterTest
[FAIL. Reason: assertion failed] test_Increment() (gas: 3071)

Encountered a total of 1 failing tests, 1 tests succeeded

This is very annoying, would love for this to not be the case :/


Proxy-Server setup

Hello, how can I setup the proxy-server in order to intercept packets from Pro-Tank client and view them?
I have tried the following config
cargo run --bin proxy-server -- --bind <IP_ADDRESS>:8080 --target (Redacted):1337
I have then changed the proxy settings of my PC to connect to port 8080, and then loaded up the client. However, the client stops loading, can you please assist me on this?

Hey @ScraperWizard,
yes you'll be able to monitor all send and received packets in a parsed manner.
As a little hint, you may have to provide the --log-protocol cmd argument.
In order to connect via the proxy I recommend using a custom chainloader.
The systems proxy settings are ignored by adobe flash or air.
I personally use a heavily modified version of https://github.com/protanki-re/client-chainloader.git


Fix time bug in San Diego Trolley app

The San Diego Trolley app has not been updating live arrival times as expected and instead arrival times keep increasing 🤦♂ I suspect this is because time.now was previously being called at setup time rather than inside the main() function. This change moves the call to time.now inside one of the functions called from main() to resolve the bug.
It's worth noting that I'm not able to reproduce this bug locally, but based on the observed symptoms I'm pretty confident about this explanation. It makes sense that I wouldn't encounter this behavior with pixlet render but it's a little surprising that pixlet serve doesn't run the application code in the same way as the production environment. Maybe this is just a configuration issue on my end 🤷♂

@serriere That is a bit weird yeah. Feels like pixlet serve shouldn't "reload" the app on every run, unless the starlark has changed. If you can repo, please open an issue on tidbyt/pixlet!


Full ingest of seqcol

Full ingest of seqCol level1 and level2 objects

Insert level1 seqCol in the db
Insert extended seqCol attributes data in the db
Retrieve level1 seqCol (with 1 db lookup)
Retrieve level2 seqCol (with 2 db lookups)

Note: some of the tests are activated with the profile 'seqcol', so it requries local run (local db).

I think everything is good now.
And the reason why disabled the test in this commit is because the method getSeqColByDigestAndLevel(RESULT_DIGEST, 2) is, for some reason, throwing a org.hibernate.LazyInitializationException when executed with JUnit test. Although it works perfectly fine when invoked from the endpoint by the controller.
I'll try to figure out why did this happen.


optional return types for jodd-json

Hello,
do you think about adding optional return types for jodd-json?
Instead of using these nullable return
String test = jsonObject.getString("test");
if(test != null) {
    //do something
}

using optional return is more modern way to handle null
Optional<String> test = jsonObject.getOptionalString("test");
if(test.isPresent()) {
    //do something
}

what about implement these methods?

Yeah, Jodd is old :)
I personally prefer Consumer, e.g.:
json.withStringValue("test", value -> {});

but again, I could add Optional as well. Just not sure about the method name... getStringValue maybe?

more modern

That is not necessarily better and - at least in your example - there is no actual advantage over it. It will only require more memory.

After some thinking, I believe its better not to migrate anything now. Otherwise, the change should be applied all over the code, and that would break the API.


Readme: How it works addition

Disclaimer: I'm not a coder/scripter and can only try to grasp what some code does, so bear with me.
I love how someone decided to make a script like this, even though I manually check how likely a mod was made using Mcreator. Despite that, I have some questions about the script you've made.
In the first part, your script looks at directory and class names to determine if something's a mcreator mod or not. That's how I do it by opening the mod with 7zip and just browsing it.
However, the second part about character size is a bit more vague. Are you looking for the length of the modfile name, or perhaps just the amount of whitespaces in the modfile name? I'm wondering about this, because in my fair share of mcreator mods I've seen had very long names with plenty of "+" signs connecting the words together.
So if you could elaborate on the second part of the script in the readme as well, that would be great.

The second part with the + - = are for the console gui system I created. The console gui uses a simple little engine I created.  I'll make sure to include more comments in the next commit, they're kinda lacking in this project atm

Just finished adding some more comments, as well as a line about the console GUI in the readme. not too much to say about it in the readme


doc: Add systemd-resolved description to our DNS reference

This will need to be iterated on after getting some Kevin's feedback, something similar would be needed for the remote configuration, ...

/assign @robin-owen

I've made a few more changes than what was requested to align this with https://github.com/code-ready/crc/pull/1819

I've made a few more changes than what was requested to align this with https://github.com/code-ready/crc/pull/1819


FR: Showing or hiding metadata fields based on certain criteria

In Tana you are given the ability to specify when a field is shown to edit based on if it's empty, not empty, always, etc.

Then it looks like this when hidden:

Might be out of the scope of this plugin but would be a really awesome feature!!

I'm in the waiting list for a Tana account. Can't wait to get my hands on it :)
I'm working on a metadata form view to group all metadata fields and facilitate their update. this capability could be included in such a form (see #131 )

now that Metadata Menu modal is in place, I could add options in the fileClass definition to hide some fields from the modal:

hide : [fieldA, fieldB, ...]
hideIfEmpty : [fieldC, fieldD, ...]
hideIfNotEmpty : [fieldE, fieldF; ...]

WDYT?

I think this is now covered with the blocks and table view :)
Don't hesitate to fill a new issue if you have specific needs on top of the current features


Explicitly table name for Mapper class

Hello,
In my company we using cassandra enginee with datastax csharp-driver. We have the problem with mapping the one model to more tables. This change solves our problem.
Please, consider adding it to the main branch.
Regards,
Kamil Myśliwiec

I agree that we should have support in the Mapper for mapping 1 entity to n tables, but I'm not sure if this is the best way to provide it.
Mapper methods use the poco type as a generics parameter (not the Mapper instance itself), so it will be best to add it by method.
@LukeTillman What do you think?

I agree that doing this at the method level is definitely more appropriate than at the Mapper level (for the reasons Jorge pointed out). I have some doubts though. I feel like the most common use case for this is to update some base table and the denormalized copies of that data living in other tables.  Those denormalized tables are likely to have a different PRIMARY KEY definition than the "base" table (i.e. the table that the POCO is mapped to by default).
For an INSERT, just specifying the table name would be enough (since we don't need to do any sort of WHERE clause there). But for UPDATE, DELETE, and many of the SELECT cases, we do generate a WHERE clause based on the PRIMARY KEY columns. So for those cases, just specifying a different table name probably isn't enough information, we'd probably need to know the key columns as well to generate the appropriate WHERE clause. And that probably means allowing users to specify the key columns somehow at the method level along with the table name.
I wonder if rather than adding these at the driver level, the Global Indexes in CASSANDRA-6477 that are coming in C* 3.0 take care of the need for this instead? Does that make any sense?

I also use switching table source for the same entity like map to a materialized view with same identical columns and leveraging same fluent mapping settings. You can just implement this on your repository class. The table class already have overload or optional parameters for table name.
public CqlQuery CreateQuery(Expression<Func<T, bool>> expression = null, string tableName = null)
{
var query = string.IsNullOrEmpty(tableName)
? Table
: new Table(Session, MappingConfiguration.Global, tableName);
        if (expression != null)
            return query.Where(expression);

        return query;
    }

Closing this pull request for no activity since the last comments.
Feel free to create a new ticket or submitting a new pull request if needed.


HTML

##User Story
As a bootcamp student
I want all the prework notes to be structured on a webpage
so that I can easily read and find information
##Acceptance Criteria
GIVEN a prework study guide website
WHEN I visit the website in my browser
THEN I see four boxes titled HTML, CSS, Git, and JavaScript with the associated notes listed

Do you want to create a website that contains all codes/notes for HTML?

Well I’m doing so for a boot camp but ultimately yes!
On Sun, Jan 28, 2024 at 8:17 AM Suvro Bhowmick @.***>
wrote:

Do you want to create a website that contains all codes/notes for HTML?
—
Reply to this email directly, view it on GitHub
https://github.com/ianhara/prework-study-guide/issues/1#issuecomment-1913593460,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFU5JKPK6NRQ5PI2QPUX5LTYQZFWJAVCNFSM6AAAAABCODBIKCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJTGU4TGNBWGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

great lets see i can contribute in this project or not!

Did you find me on GitHub? I’m a junior in computer science and know Java
and OOP but this full stack dev is new
On Sun, Jan 28, 2024 at 8:41 AM Suvro Bhowmick @.***>
wrote:

great lets see i can contribute in this project or not!
—
Reply to this email directly, view it on GitHub
https://github.com/ianhara/prework-study-guide/issues/1#issuecomment-1913600707,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFU5JKPWFEQUU6YOYDACVGLYQZIP5AVCNFSM6AAAAABCODBIKCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJTGYYDANZQG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

Github

I am going to build my own separate website to learn HTML
On Sun, Jan 28, 2024 at 8:57 AM Suvro Bhowmick @.***>
wrote:

Github
—
Reply to this email directly, view it on GitHub
https://github.com/ianhara/prework-study-guide/issues/1#issuecomment-1913605643,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFU5JKOJBAX7BPLXT4MIGCLYQZKK7AVCNFSM6AAAAABCODBIKCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJTGYYDKNRUGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

JavaScript and CSS as well
On Sun, Jan 28, 2024 at 9:51 AM Ian H @.***> wrote:

I am going to build my own separate website to learn HTML
On Sun, Jan 28, 2024 at 8:57 AM Suvro Bhowmick @.***>
wrote:

Github
—
Reply to this email directly, view it on GitHub
https://github.com/ianhara/prework-study-guide/issues/1#issuecomment-1913605643,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFU5JKOJBAX7BPLXT4MIGCLYQZKK7AVCNFSM6AAAAABCODBIKCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJTGYYDKNRUGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

cool


Show C# Riders's suggestions, while computing TabNine suggestions

Hi there,
I noticed, that sometimes TabNine takes some extra time to compute suggestions for certain, especially context free (after closing a statement with ;) or context heavy (lambda operator, for example LINQ statements: .Select(x => x.SomeProperty)) situations.
These are situations, where the standard IDE suggestions usually suffice. I also really enjoy TabNines suggestions and coding with an AI partner. :-)
But while the TabNine plugin also delivers standard autocompletion suggestions, it fails to show them on time, while still calculating its own suggestions. Does anyone experience the same on other IDEs/languages?
I would be very happy, if someone knew how to resolve this. Thank you for having a blast coding sofar!

This should be fixed, if not please tell us.


Added groups to history page

Addressed Issue #32 by modifying the update_history function and history.html.theme to show the services and their history in the history page by their groups.

Hmm, I’m still getting the same error. Just wanted to check, have you tested it on your end and is everything working fine for you?

@harsxv yes it's working perfectly fine on my end.

Err weirds then, I am still unable to get it works on my other machine. Need someone else to confirm and let me double check.

@harsxv I have checked both for Windows and Ubuntu now. It is working fine for both of them.

@harsxv are you still getting the same errors ?

Hey @aarya-16, unfortunately yes. I just deployed a brand new server, but I'm still facing the same issue. I'll double check once I get some time later.


[fix] no-unused-modules: handle ClassDeclaration

Fixes #1368
This commit finds all codes related to FunctionDeclaration, and double it up with ClassDeclaration.

Coverage increased (+0.001%) to 97.803% when pulling 8b55e8f86b282f408d0310f478213bcd9b2c0be8 on golopot:no-unused-modules-handles-class into ca4bf952be88d534262906d645c25f9e148be44d on benmosher:master.

Coverage increased (+0.001%) to 97.803% when pulling 8b55e8f86b282f408d0310f478213bcd9b2c0be8 on golopot:no-unused-modules-handles-class into ca4bf952be88d534262906d645c25f9e148be44d on benmosher:master.

Coverage increased (+0.001%) to 97.803% when pulling 8b55e8f86b282f408d0310f478213bcd9b2c0be8 on golopot:no-unused-modules-handles-class into ca4bf952be88d534262906d645c25f9e148be44d on benmosher:master.


null/undefined type shorthands

Ref: microsoft/TypeScript#7426
I'm suggesting ?type for type | null and type? for type | undefined. The null prefix would be consistent with JSDoc's nullable type and the undefined suffix would be similar to TypeScript's existing optional parameters. Something else to consider is possibly merging the two to avoid ambiguity à la Flow's maybe type.
type T0 = ?number // => number | null
type T1 = number? // => number | undefined

function foo(a: ?number, b: string?): void
// => a: number | null
// => b: string | undefined

Note that a?: T and a: T? are not the same; the former doesn't require a to present while the latter does.

Thanks for opening this issue it's definitely something we'd want to add to Civet. Here's a link to a related discussion: https://github.com/DanielXMoore/Civet/discussions/144

I would lean towards Flow semantics:
// .civet
function foo(a: ?number, b: string?): void
// ↓↓↓ .ts
function foo(a?: number? | null, b: string?)

I think the code will be harder to read if both prefixes and suffixes are used. I'd rather have a different symbol in the same position than the same symbol in a different position, especially when the types are long.
Also, there's a difference in readability between prefix and suffix.
Preffix:
x: ?{
  foo: ALotOf
  bar: ?VisualNoiseThat<Your, Eyes>
  baz: WillTry<ToSkip>
  qux: ?WhenQuickly<SkimmingThrough, TheCode>
}

Suffix:
x: {
  foo: ALotOf
  bar: VisualNoiseThat<Your, Eyes>?
  baz: WillTry<ToSkip>
  qux: WhenQuickly<SkimmingThrough, TheCode>?
}?

IMO, the suffix is easier to miss than the prefix.
But going a step backwards, how many cases is it even useful to distinguish between null and undefined? I do it in some rare cases, but I'd be fine to have ?T mean T | null | undefined for most cases and then in the specific cases where I actually need to distinguish, I can be more verbose and explicit with T | null and T | undefined so nobody reading the code misunderstands it.

Agreed with @gustavopch. If ?T and T? meant different things where one is null and one is undefined, I doubt I would ever remember which. I think | null | undefined is what needs the shorthand the most, and I like that it matches Flow.

The other nuance about Flow semantics is that object properties are required and function parameters are truly optional. Since I don't know enough about the reasoning why Flow chose to do it that way I'd defer to their semantics (especially if we use the same symbol in the same position they do). Maybe if someone can find a link to the original discussion in the Flow project it could add some information to expand/modify the design further.

Civet could use nothing or nil as a shorthand for null | undefined.

Civet could use nothing or nil as a shorthand for null | undefined.
nothing/nil isn't quite as compact as the ? shorthand, but it would offer some improvement. Plus, this avoids further overloading the meaning of ?. 'nil' or 'nothing' would be also more distinctly searchable in the docs for newcomers.

nil to represent both null | undefined seems neat. void would be nice too.

void already has a different meaning in TypeScript. I personally prefer the ? operator but shorthand for null | undefined also seems interesting.

Something to note, it's already possible to do type nil = null | undefined in userspace TS: https://www.typescriptlang.org/play?#code/C4TwDgpgBAdglgGygXlgVwUgPlNMAmEAZnDBPgFAWiRQAaKsi140AmowM7ABOpA5lBzwEFIA
My personal feeling is that most of the value of this proposal would be extending TS syntax with the prefix ? and matching Flow semantics.

This is now (mostly) solved; see https://civet.dev/reference#optional-types
In particular, T? means T | undefined (for compatibility for most other ? types), and T?? means T | undefined | null. We didn't define a meaning for ?T but we could in the future.


s3authserver service does not start when restarting CORTX

Steps: Follow proper shutdown and restart of CORTX.
https://github.com/Seagate/cortx/blob/main/doc/CORTX_on_Open_Virtual_Appliance.rst
Issue: s3authserver service did not start

@justinzw, yes I too observed the same as,
[root@myvm ~]# systemctl is-active s3authserver
failed
[root@myvm ~]# hctl status
Cluster is not running.

hi friends,
this workaround has been working for me (3 files)  :
cat /etc/systemd/system/cortx_start.service
[Unit]
Description=cortx hctl start
After=network.target
[Service]
Type=idle
RemainAfterExit=yes
ExecStart=/var/tmp/cortx_start
TimeoutStartSec=30
[Install]
WantedBy=default.target
[root@koren-cortx ~]# cat /var/tmp/cortx_start
#!/bin/bash
hctl start
sleep 10s
/var/tmp/services-run-cmd start
[root@koren-cortx ~]# cat /var/tmp/services-run-cmd
systemctl $1 rabbitmq-server
systemctl $1 elasticsearch
systemctl $1 haproxy
systemctl $1 s3authserver
systemctl $1 sspl-ll
systemctl $1 csm_agent
systemctl $1 csm_web

Issue was s3authserver was trying to start before its dependent service (slapd) hence s3authserver was failing with LDAPConnectionException error after reboot.
Expected behavior is dependent service should start first and then s3authserver should start.
We are updating our OVA documentation to provide the correct steps to start these services in sequence.
PR: https://github.com/Seagate/cortx/pull/445

Hi @justinzw @mukul-seagate11 since we have merged this PR with required changes, Can we close this issue?

Closing the PR as the issues has already been taken care & addressed in doc

guys, https://github.com/Seagate/cortx/pull/445 is a good solution/doc for a proper manual shutdown and restart, however users might expect VM restart to work automatically without ever touching any CLI...
where do i contribute my 2 scripts above to be included (if approved) in the next OVA release ?

reopening the ticket and tagging Justin and John.

This discussion is already been discussed in https://github.com/Seagate/cortx/discussions/473 so closing the issue


Show desktop notification on song change

It would be cool to get a nifty notification on song change.
Using html5 notifications: https://developer.mozilla.org/en/docs/Web/API/notification

Implemented! I'm going to push in a bit.
Screenshot: http://ctrlv.in/529197

Looking good!


feat(sqllab): TRINO_EXPAND_ROWS: expand columns from ROWs

SUMMARY

Reimplement, in a simpler way, part of the PRESTO_EXPAND_DATA feature for trino.
Make use of the trino library's type expansion logic added since the original feature was introduced, as we now have sqla ROW types parsed out recursively in a way we can analyse.
Analyse those ROWs and expand our definition of get_columns out to include dotted.path references to all fields it's possible to query by dotted path (i.e. all ROWs, nested arbitrarily deep).
Add an extra optional query_as field to the column definition so that we can override the way it's queried: otherwise sqlalchemy is going to quote the whole thing as a single column name, which isn't correct, it should actually be quoted per dotted segment, and we'll want to alias it to the full dotted string.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before / without feature flag
Schema

Preview

After
Schema

Preview

TESTING INSTRUCTIONS

Turn on TRINO_EXPAND_ROWS feature flag
Query tables in trino containing rows and/or nested rows
Observe that dotted paths are expanded out on the schema preview, and queried in the table preview

To easily construct some data containing nested rows, you can use the following instructions:

Run trinodb/trino:latest in docker
docker exec -it trino trino to cli into it
Create a table with nested fields like this:

USE memory.default;
CREATE TABLE foo AS (
  WITH
    data AS (
      SELECT
        (
          SELECT
            ARRAY[1, 2, 3] arr1,
            ARRAY[4, 5, 6] arr2,
            (
              SELECT
                (SELECT 'foo' foo, 'bar' bar) nest1a,
                (SELECT 'hodor' foo, 'HODOR HODOR HODOR' bar) nest1b
            ) nest1,
            (
              SELECT
                (SELECT 'baz' baz, 'buzz' buzz) nest2a,
                (SELECT 'bzzzzzzt' buzzing, 'BZZZZZT!' buzzsaw) nest2b
            ) nest2
        ) field1,
        'toplevel' field2
    )
  SELECT * FROM data

);

Connect to the dockerised trino via superset; I did this by simply adding my trino container into the standard docker-compose definiton, so I could connect to it via trino://root@trino:8080/memory/default

ADDITIONAL INFORMATION

[x] Has associated discussion: https://github.com/apache/superset/discussions/25713#discussioncomment-7421689
[x] Required feature flags: TRINO_EXPAND_ROWS
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[x] Introduces new feature or API
[ ] Removes existing feature or API

@john-bodley there are some difficulties in expanding out the data to be more readable which I thought about while working on this:

we can't really do much about arrays; the PRESTO_EXPAND_DATA thing tried to spread them across multiple rows in the output but this seemed very confusing for a few reasons (I went into some detail in the linked discussion)
the original way that was being done was too fragile; it was actually parsing the junk you see there and manually expanding it out, without awareness of types and with too much reliance on being able to parse the syntax we see here. I considered a part 2 to this work being to do that in a more sensible way, either wrapping the user's query with an exhaustive SELECT on the fields to expand out the ROWs for you, or giving the user more control by providing some macros to just make it easier to do exactly that.

There's more, I left a fair few notes on some challenges I found on the related discussion if you're interested.
But I think this fixes a good chunk of the problem and I have no prejudice on whether we also add on some display improvements as well; it's fairly well-encapsulated piece so hopefully it'll work fine with what @justinpark is working on.
Currently from our end we consider it acceptable that querying a ROW or querying SELECT * directly will produce an unreadable blob since users can see the field breakdown in the schema, see more readable data in the table preview, and autocomplete foo.bar.baz to let them query fields more directly. We'd rather users selected a handful of relevant fields than build datasets with hundreds of selected fields anyway. We did notice that when we query via the trino cli we actually get a slightly different and nicer format though, like this:
{foo=123, bar="something"}

which is a lot easier to understand than [123, "something"] with no labels. So we intended to try to get that same format into superset as our next step, but if there's already work being done to make it a tree we can expand out then sounds like a better solution is already in progress. I'll take a look at the SIP, we're definitely be interested in what's being done there and this stuff is very fresh in my mind right now.

Hi @john-bodley @villebro @michael-s-molina gentle nudge about this one when you have some time.

The backend changes LGTM. @michael-s-molina or @villebro would you mind taking a look at the frontend changes?

@justinpark Should we test this PR with our database to see how the data is displayed?

Hi @michael-s-molina, @justinpark have you had chance to test this one? Happy to merge this one?

@giftig LGTM with the current UI design.
One thing I would like to revise the response structure to accommodate a future tree view user interface (UI) that is organized hierarchically. This UI will consist of columns, with each column containing child elements.

@giftig LGTM with the current UI design.
One thing I would like to revise the response structure to accommodate a future tree view user interface (UI) that is organized hierarchically. This UI will consist of columns, with each column containing child elements.


added travis-ci test against cockroach 2.1.0 in ssl mode

Just tried to add a test against an SSL mode of the latest version of Cockroach.

You're welcome. I did nothing if you didn't start. Thanks to you.