forcefieldGenerator issue with Python 3.x?

I've run into this issue, which seems to be a problem with the forcefield generators.
> nosetests -vv perses/tests/test_topology_proposal.py:test_small_molecule_proposals
======================================================================
ERROR: Make sure the small molecule proposal engine generates molecules
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/choderaj/miniconda3/lib/python3.5/site-packages/nose/case.py", line 198, in runTest
    self.test(*self.arg)
  File "/Users/choderaj/github/choderalab/perses/perses.choderalab/perses/tests/test_topology_proposal.py", line 89, in test_small_molecule_proposals
    initial_system, initial_positions, initial_topology = oemol_to_omm_ff(initial_molecule, "MOL")
  File "/Users/choderaj/github/choderalab/perses/perses.choderalab/perses/tests/test_topology_proposal.py", line 70, in oemol_to_omm_ff
    system = system_generator.build_system(topology)
  File "/Users/choderaj/github/choderalab/perses/perses.choderalab/perses/rjmc/topology_proposal.py", line 1216, in build_system
    system = self._forcefield.createSystem(new_topology, **self._forcefield_kwargs)
  File "/Users/choderaj/miniconda3/lib/python3.5/site-packages/simtk/openmm/app/forcefield.py", line 1113, in createSystem
    if generator(self, res):
  File "/Users/choderaj/miniconda3/lib/python3.5/site-packages/openmoltools/forcefield_generators.py", line 504, in gaffTemplateGenerator
    molecule = generateOEMolFromTopologyResidue(residue)
  File "/Users/choderaj/miniconda3/lib/python3.5/site-packages/openmoltools/forcefield_generators.py", line 139, in generateOEMolFromTopologyResidue
    [status, output] = getstatusoutput(command)
  File "/Users/choderaj/miniconda3/lib/python3.5/subprocess.py", line 495, in getstatusoutput
    data = check_output(cmd, shell=True, universal_newlines=True, stderr=STDOUT)
  File "/Users/choderaj/miniconda3/lib/python3.5/subprocess.py", line 316, in check_output
    **kwargs).stdout
  File "/Users/choderaj/miniconda3/lib/python3.5/subprocess.py", line 385, in run
    stdout, stderr = process.communicate(input, timeout=timeout)
  File "/Users/choderaj/miniconda3/lib/python3.5/subprocess.py", line 788, in communicate
    stdout = self.stdout.read()
  File "/Users/choderaj/miniconda3/lib/python3.5/codecs.py", line 321, in decode
    (result, consumed) = self._buffer_decode(data, self.errors, final)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xb0 in position 34: invalid start byte

I'm not sure why we haven't run into this before. I don't believe the test was disabled.
I'll do more testing to see if this might be a peculiar misconfiguration of my environment.

This must have been a peculiarity with my installation, since I can't replicate this on travis or another machine.


Add options needed for GDEM029T94 initialization

GDEM029T94 is used in 2.9inch_e-Paper_Module_V2.

Made second try to merge code in #49377


[WIP] Plotting improvements

See history.rst for details of the changes. This PR should be ready, let it stage a bit before merging.

Coverage decreased (-0.06%) to 81.022% when pulling f8c221d346663e814333e2ba7899f4bbe82bd3fa on plotting into 361f0258a210193f482c6197ea879765a9041e91 on master.


Update issue templates to assign labels by default

Changes automatically generated via GH UI, following instructions here.

Codecov Report

Merging #1328 (09387e7) into main (c5ee164) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main    #1328   +/-   ##
=======================================
  Coverage   91.79%   91.79%
=======================================
  Files          96       96
  Lines       10387    10387
=======================================
  Hits         9535     9535
  Misses        852      852

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


PROMPT_COMMAND environment variable is set incorrectly

The prompt_command environment variable can be incorrectly set to contain two consecutive semi-colons. This causes the following error:
-bash: PROMPT_COMMAND: line 0: syntax error near unexpected token `;;'
-bash: PROMPT_COMMAND: line 0: `direnv_hook;prompt_command;;preexec_invoke_cmd`

It seems related to #767
This occurs on OSX El Capitan.

Hi Samze..
I've sent a issue ticket about this and already made a temporary workaround..
See here: https://github.com/Bash-it/bash-it/issues/767


Manifest merger failed / ActivityNotFoundException on CameraActivity

Hi,
I get

Error:Execution failed for task ':hospitalitix:processDebugManifest'.
Manifest merger failed : Attribute application@theme value=(@style/AppBaseTheme) from AndroidManifest.xml:17:9-44
is also present at [com.github.boxme:squarecamera:1.1.0] AndroidManifest.xml:11:18-76 value=(@style/squarecamera__CameraFullScreenTheme).
Suggestion: add 'tools:replace="android:theme"' to  element at AndroidManifest.xml:14:5-33:19 to override.

when trying to use your lib. If I override as suggested I get

android.content.ActivityNotFoundException: Unable to find explicit activity class {com.assaabloyhospitality.hospitalitix/com.desmond.squarecamera.CameraActivity}; have you declared this activity in your AndroidManifest.xml?

when trying to start your activity via Intent. Not sure the problems are related though.
Is there a reason you have a theme set on application level? Since you only have one Activity, and it has a theme set, the application level theme is never used.
Br.
Jonas

I got the same problem...please provide us a solution for it

"android.content.ActivityNotFoundException: Unable to find explicit activity class {com.assaabloyhospitality.hospitalitix/com.desmond.squarecamera.CameraActivity}; have you declared this activity in your AndroidManifest.xml?"
 Seems pretty basic..As for the first error, you may have conflicting themes. Try updating your project to the latest build tools 25 e.t.c

"Seems pretty basic.."
Okay, I guess I can declare your activity in my Manifest. One could argue that you could declare your own activity in your own Manifest in your own module. Seems pretty basic.. :)
As for the theme squarecamera__CameraFullScreenTheme declared on application level, it seems pointless.
Thank you for your reply.

Actually I had this issue sometime back and took me a while to remember how I solved this but as the log says all you need to do is add "tools:replace="android:theme" to your PROJECT level manifest file..under application...and it's not my module :). Just removed this line in my code and got the same error. This assumes you are using the module and not the .jar

may I know is it resolved?

@finalight just in your AndroidManifest file add
tools:replace="android:theme"
in the application tag


Improve Navbar 21

Need to improve this component on Navbar/src/navbar-21.html
Improvements required :

Navbar is not responsvie

@Rajdip019 Please assign this to me

Done #893


Include PowerShell and TypeScript in examples

It would be helpful to include PowerShell (and TypeScript) in the examples as we are not all C# experts.  But we do know PowerShell (and TypeScript).

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

ID: 926ab97b-0406-116a-9b44-59df02beee7d
Version Independent ID: 3094c3cf-cd16-518e-48fc-095fb14036dd
Content: Customizing
Content Source: docs/solution-guidance/modern-experience-customizations-customize-pages.md
Service: unspecified
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

Agreed!

Good suggestion, but it's not exactly actionable at this time & this issue has been open for over a year with no activity. Happy to accept a PR if someone wants to contribute time to write up alternatives.


CTRL-C with dotnet watch in docker takes 20 to 30 seconds

Copied from https://github.com/dotnet/dotnet-docker/issues/431 by @richlander
I am running a Docker container per the instructions in #420. CTRL-C works for ASP.NET locally and for console apps in a container but not for ASP.NET apps in a container. I get this same behavior on Windows and macOS.
Command line:
docker run --rm -it -p 8000:80 -v c:\git\dotnet-docker\samples\aspnetapp:/app/ -w /app/aspnetapp microsoft/dotnet-nightly:2.1-sdk dotnet watch run

Result:
Hosting environment: Production
Content root path: /app/aspnetapp
Now listening on: http://[::]:80
Application started. Press Ctrl+C to shut down.
^CApplication is shutting down...
watch : Shutdown requested. Press Ctrl+C again to force exit.
^C
^C^C^C^C^C^C
^C^C^C^C^C^C^C^C
^C^C^C^C^C

/cc @natemcmaster @MichaelSimons @glennc
Copied from https://github.com/dotnet/dotnet-docker/issues/431#issuecomment-374237841 by @MichaelSimons
This appears to be tied to using dotnet watch in conjunction with ASP.NET.  CTRL-C works as expected when running an ASP.NET app directly within a container.
Copied from https://github.com/dotnet/dotnet-docker/issues/431#issuecomment-374247175 by @MichaelSimons
Upon further inspection, CTRL-C is working. It is just extremely slow.
Copied from https://github.com/dotnet/dotnet-docker/issues/431#issuecomment-374384509 by @richlander
I just repro'd that. It takes about 20s for me, on both macOS and Windows. Surprisingly, Windows containers don't seem to have this problem.
Copied from https://github.com/dotnet/dotnet-docker/issues/431#issuecomment-374390354 by @natemcmaster
I can repro, too. You can move this to aspnet/DotNetTools. I have a hunch it may be something in the "kill process tree" code, but I'm not sure.

Ugh. cc @glennc

Moving this out due to scheduling constraints.

Ok, I've narrowed down the cause to this code: https://github.com/aspnet/Common/blob/e79302c419482487aa26d136d23abc944c5972bf/shared/Microsoft.Extensions.Process.Sources/ProcessHelper.cs#L88-L111. This is the code that attempts to kill a process tree.
From what I can tell, there appears to be some kind of deadlock on the thread pool. On macOS and Linux, ProcessHelper.KillTree attempts to use Process.Start + pgrep to discover the tree of processes. The thread invoking this path deadlocks on Process.WaitForExit(int timeout). The 30 second hang when killing the aspnet process exactly matches the timeout given to WaitForExit.
We should re-examine how we determine the process tree. We may be able to use procfs on Linux, and p/invoke to sysctl on macOS to get the process tree. Ideally, .NET Core would provide an API for this (see https://github.com/dotnet/corefx/issues/26234), but until then, we need to workaround.

Thanks Nate! So this is not for Docker containers, then?

It doesn't appear to affect Docker anymore. Also, it doesn't affect Windows.
More details: CTRL+C kills the parent process (dotnet.dll) but leaves dotnet-watch.dll running as an oprhaned process. Docker kills any orphaned process when the container exits. The issue still appears when running in the shell though. You'll still get some console output 30 seconds later from the orphaned dotnet-watch.dll process if you run dotnet watch --verbose.

Got it! Thanks and I updated the title.

This actually turns out to be a regression in Process.WaitForExit. https://github.com/dotnet/corefx/issues/29699. But I've added a workaround to dotnet-watch for now that should still work when/if https://github.com/dotnet/corefx/issues/29699 is resolved.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

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

Comparison is base (595c70c) 95.69% compared to head (9be89b7) 95.69%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #867   +/-   ##
=======================================
  Coverage   95.69%   95.69%
=======================================
  Files           1        1
  Lines         395      395
  Branches       53       53
=======================================
  Hits          378      378
  Misses         17       17

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


[smoke-tests] skip overriding version for dependency not managed in this repo

@azure/msal-node is not managed in this repo, so we should keep the version
specified in package.json.

@ckairen please review

/check-enforcer overrice


Rename QueryM to Contract and newtype

Create a newtype of QueryM to Contract and newtype instances like MonadEffect, MonadError etc. keeping QueryM for internal usage.

Including this as part of PR https://github.com/Plutonomicon/cardano-browser-tx/issues/154


Linux 64-bit build required

Working release builds for Linux (64-bit and 32-bit) need to be created and added to the releases.

A burn-in test is not needed for these builds as I'll build them against the last-used version of the libraries for now.

A burn test will be needed as this will be build with FPC 3 rather than the previous 2.x series.

Build created using the previous library versions but new compiler version for 64-bit Linux.


Raspberry pi not displaying

I'm accessing it over xrdp from Win10, but I also tried tightVNC (I couldn't get any xservers to work).
Anyway, I run the install then try to run the pictures demo and I get this, it looks like it's running but there is no display.
root@raspberrypi:/usr/local/share/kivy-examples/demo/pictures# python main.py
[INFO   ] [Logger      ] Record log in /root/.kivy/logs/kivy_16-07-24_9.txt
[INFO   ] [Kivy        ] v1.9.2-dev0
[INFO   ] [Python      ] v2.7.9 (default, Mar  8 2015, 00:52:26)
[GCC 4.9.2]
[INFO   ] [Factory     ] 193 symbols loaded
[INFO   ] [Image       ] Providers: img_tex, img_dds, img_gif, img_pygame, img_pil (img_ffpyplayer ignored)
[INFO   ] [Window      ] Provider: egl_rpi
[INFO   ] [GL          ] OpenGL version <OpenGL ES 2.0>
[INFO   ] [GL          ] OpenGL vendor
[INFO   ] [GL          ] OpenGL renderer
[INFO   ] [GL          ] OpenGL parsed version: 2, 0
[INFO   ] [GL          ] Shading version <OpenGL ES GLSL ES 1.00>
[INFO   ] [GL          ] Texture max size <2048>
[INFO   ] [GL          ] Texture max units <8>
[INFO   ] [Shader      ] fragment shader:
[INFO   ] [Shader      ] vertex shader:
[INFO   ] [Window      ] virtual keyboard not allowed, single mode, not docked
[INFO   ] [GL          ] NPOT texture support is available
[INFO   ] [Text        ] Provider: pygame
[INFO   ] [OSC         ] using  for socket
[INFO   ] [Base        ] Start application main loop
Any clues what I'm doing wrong?
I also tried setting KIVY_BCM_DISPMANX_ID=2, made no difference.
I don't have the 7" LCD installed yet (It'll be ordered tomorrow).

Hi, xrdp does not support openGL, so it cannot show the kivy app


cleanup(sinsp)!: enforce some runtime checks and remove an extra space from evt.args

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area driver-modern-bpf
/area libsinsp
/area tests
Does this PR require a change in the driver versions?
No
What this PR does / why we need it:
This PR does the followings:

add a test for UNIX sockets in sinsp
add some tests for evt.arg/evt.rawarg/avt.args fields
enforce some runtime checks check_param_name_exists/check_param_id_exists on evt.arg this should prevent our users from using non-existent fields with evt.arg.* . I noticed it in one of our default rules Non sudo setuid, see it here https://github.com/falcosecurity/falco/issues/2715#issuecomment-1771226612. we use exe_flags=%evt.arg.flags when the event is setuid but setuid doesn't have a flags arg. Please note this is a BREAKING CHANGE since now some broken rules will stop Falco at runtime... it would be amazing to stop it when parsing the filter-checks names but at the moment we don't have any event information at that point so we need to do that during extraction :/ BTW there is room for improvements in the future

Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
cleanup(sinsp)!: enforce some runtime checks and remove an extra space from `evt.args`

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Oh this is a real bummer, I wasn't aware Falco would just crash in such cases, re specifically the exe_flags=%evt.arg.flags besides renaming it to flags we should adjust all upstream rules to not cause crashes and only use fields that are valid. We should still fix it here, but at least that way we fix things for Falco 0.37 and the next rules release. I can take care of that PR.

at the moment Falco doesn't crash if a field evt.arg.* doesn't exist for an event, so we are fine. The 2 checks implemented here check_param_name_exists/check_param_id_exists cannot work with actual Falco because we have several rules in which we use evt.arg.* but the field is not defined for all the events in the condition (see https://github.com/falcosecurity/rules/issues/214) so it would cause random crashes at runtime :/
I will detach the tests in another PR and probably close this one! until we solve the above issue (if we want to solve it) we cannot implement this strict validation unfortunately
/hold

Thanks for the clarification and also thanks for looking into it!

@Andreagit97 could you rebase? I think we could start getting this over the finish line if you would like, I'll help as reviewer, thank you!

close in favor of https://github.com/falcosecurity/libs/pull/1791


Fix missing backslash in Dockerfile.rpi

Please check the following:

Make sure you are targeting the master branch, pull requests on release branches are only allowed for bug fixes.
Read contributing guidelines: https://github.com/go-gitea/gitea/blob/master/CONTRIBUTING.md
Describe what your pull request does and which issue you're targeting (if any)

You MUST delete the content above including this line before posting, otherwise your pull request will be invalid.

Good catch. LGTM

LGTM


len(dataloader) in distributed setting is different with datapipes and with map-style datasets

In a distributed setting, len(dataloader) will return:

len(dataset) // (batch_size * num_GPUs) if dataset is a map-style dataset
len(dataset) // batch_size if dataset is a datapipe

This discrepancy makes it a bit difficult to work with torchvision's training recipes, where we often need the size of the dataloader.
Below is an illustration of this discrepancy - you can run the snippet (even without a GPU) with torchrun --nproc_per_node 4 script.py
# Run this with e.g. `torchrun --nproc_per_node 4 script.py`
import torch.utils.data as data
import torch.distributed as dist
import torchdata

def replace_print():
    import builtins as __builtin__
    builtin_print = __builtin__.print
    def print(*args, **kwargs):
        if dist.get_rank() == 0:
            builtin_print(f"[GPU 0]", *args, **kwargs)

    __builtin__.print = print

# Setting up DDP - you can ignore this
dist.init_process_group(backend="gloo")
replace_print()
dist.barrier()

size = 800
dp = torchdata.datapipes.iter.IterableWrapper(range(size)).sharding_filter()
dl = data.DataLoader(dp, batch_size=10, num_workers=4, drop_last=True)
print(f"with dp, {len(dl) = }")
# Gives : 80

ds = list(range(size))
dl = data.DataLoader(ds, batch_size=10, num_workers=4, drop_last=True, sampler=data.DistributedSampler(ds, shuffle=False))
print(f"with mapstyle, {len(dl) = }")
# Gives: 20

Thank you for opening the issue. It's kind easy to fix but need to consider all the use cases.
If users don't specify sharding_filter in the pipeline, the length should be len(dataset) * num_GPUs // batch_size.
I do want to understand when you need the size of dataloader? is this related to the meta data for each Dataset?

If users don't specify sharding_filter in the pipeline, the length should be len(dataset) * num_GPUs // batch_size.

I agree. Interestingly, with map-style datasets, len(dataset) is equal to len(dataset) // batch_size if users don't pass sampler=DistributedSampler(), which is equivalent to not calling .sharding_filter(). But I think len(dataset) * num_GPUs // batch_size as you proposed makes more sense.

I do want to understand when you need the size of dataloader?

We rely on the size for our logger, which is how I found out about the discrepancy:
https://github.com/pytorch/vision/blob/59c4de9123eb1d39bb700f7ae7780fb9c7217910/references/classification/train.py#L25
https://github.com/pytorch/vision/blob/59c4de9123eb1d39bb700f7ae7780fb9c7217910/references/classification/utils.py#L109

is this related to the meta data for each Dataset?

No, not directly. But I'm still looking into convenient ways  to specify the length of the torchvision datapipes. I'll definitely come back to you on this when this is clearer for me.


More fine-grained exception hierarchy

At the moment we basically only use SSLException, SSLHandshakeException and IOException for QUIC. It would be better to be a bit more fine-grained here to make it easier for people to understand why something failed.

Should the structure of exception follow the same structure quiche has for error codes? Or do you have something different on mind?

Yeah maybe we could just include the exact error as an enum int the message.

I'll take a stab on this one tomorrow.


Add server-side rendering error page

See the #46 PR I think @zanemountcastle  actually already added some code there that may be able to be stolen.

This should be similar to the PR @emilgoldsmith mentioned above and the 404 error page. More of a copy/paste exercise here.

This is a duplicate for #122 I just realized, also it seems I was plain wrong on the PR stuff, I'm not quite sure what I was referring to even though it's only a month ago tbh. But yeah I'll close this for the moment and we can work on the other one.


openhashtab-np: Add version 3.0.2

[x] I have read the Contributing Guide.

/verify

--> Installing the Manifest openhashtab-np.json

Installing 'openhashtab-np' (3.0.2) [64bit]
OpenHashTab_setup.exe (3.0 MB) [==================================================================================] 100%
Checking hash of OpenHashTab_setup.exe ... ok.
Linking ~\scoop\apps\openhashtab-np\current => ~\scoop\apps\openhashtab-np\3.0.2
Running post_install script...
'openhashtab-np' (3.0.2) was installed successfully!

--> Refreshing environment variables

--> Comparing ARP Entries

DisplayName                DisplayVersion Publisher ProductCode
-----------                -------------- --------- -----------
OpenHashTab version v3.0.2 v3.0.2         namazso   {C0EEE3CD-665D-4E4E-B3BC-ADCD0FE73C0F}_is1

PS C:\Users\WDAGUtilityAccount\Desktop\bucket> scoop uninstall openhashtab-np
Uninstalling 'openhashtab-np' (3.0.2).
Running pre_uninstall script...
Unlinking ~\scoop\apps\openhashtab-np\current
'openhashtab-np' was uninstalled.
PS C:\Users\WDAGUtilityAccount\Desktop\bucket>


SWARM-1472: Arquillian fraction has test scope in all boms.

Motivation
Arquillian fraction should have "test" scope in all boms so that its
dependencies are not added to the resulting artifact during compilation.
Modifications
Added scope to FractionMetadata. Arquillian fraction is always added
with test scope. It is also possible to specify a default scope using
"swarm.fraction.scope" property in a fraction pom.
Result
Arquillian fraction has test scope in all boms.

I'm also going to send a PR to https://github.com/wildfly-swarm/wildfly-swarm with <swarm.fraction.scope>test</swarm.fraction.scope> added to the arquillian fraction. It's true the property will be ignored but I think it's good to be consistent, or at least pretend consistency ;-).


Azure AD: fix my dumb mistake when refreshing user delegation credentials

We must refresh whenever the credential expires before 1 minute from now, not after.

This change is

Now that I think about it, I don't think the 1 minute buffer is actually useful here in the first place: AFAICT, the SAS url generated by SignWithUserDelegation() can exceed the UDC (user delegation credential)'s expiration, so long as it was created before the UDC expires. Since we always call getUserDelegationCredential to get the UDC... I think we can simplify the check to a.udcExp.Before(time.Now()). The only concern is if we're so close to the expiration that it expires in between the check and the call to SignWithUserDelegation()... and if that happens, then we generate one bad URL, re-try, refresh, and get a good url. @jgraettinger thoughts?

I take back what I said in my comment. According to the docs I found:

The SAS is invalid after the user delegation key expires

That's correct: Signed URL Exp <= UDC Exp


Shadow seems to be clipped

Hello... I got it working on Qt. Now I played around with some examples. In the following example the shadow seems to be clipped.
main.cpp.txt

You likely need to increase the viewport of the underlying OrthographicCamera used by the DirectionalShadow. I think the behaviour is similar to three.js.
This part of the API is not great and you likely ned to #include "threepp/lights/LightShadow.hpp"
To get the camera:
 auto shadowCamera = light->shadow->camera->as<OrthographicCamera>();

Thank you very much. I try to check it, but it might take some time. But I will return here and tell you my result :)


Unnecessary update

I just updated juliaup, and it found and installed a new 1.7.3 release. I was not aware of a new version of 1.7.3. Perhaps this is a bug?
shell> juliaup status
 Default  Channel  Version          Update
------------------------------------------------------------------------
          beta     1.8.0-rc1+0~x64  Update to 1.8.0-rc3+0.x64 available
          lts      1.6.6+0~x64      Update to 1.6.7+0.x64 available
       *  release  1.7.3+0~x64      Update to 1.7.3+0.x64 available

shell> juliaup update
Installing Julia 1.8.0-rc3+0.x64 (x64).
Installing Julia 1.6.7+0.x64 (x64).
Installing Julia 1.7.3+0.x64 (x64).

I changed the version string format we use internally to be semver compatible (see https://github.com/JuliaLang/juliaup/issues/140) and that triggered this. Should be a one-time thing.


Installing duckdb through rust literally crashes entire PC due to compile times

If you run
cargo add duckdb --features bundled

It will crash your pc unless you have extremely high end hardware. This is not feasible if you're looking to replace something like sqlite where you install it precompiled and it's done. Ik you can install duckdb like that but it doesn't integrate with the rust library.

Ik you can install duckdb like that but it doesn't integrate with the rust library.

It's perfectly possible to link in a precompiled duckdb library if you want to avoid the compile time issue. Have actually tried doing this?


[post-1.20] Fake Dynamic Client requires scheme registration with listType, breaks duck typing.

we in Knative attempted to update to k8s client-go 1.20 and found that the fake dynamic client no longer works with duck typing because of this addition: https://github.com/kubernetes/client-go/commit/20c034c1785b34a893f7f96d91e5b849ac52ff31#diff-8ba8eeecc835702a1f7fbb6d06b268cb17453f82c5ef5c8294f25fcdf0f53dacR316-R318 @deads2k I will keep looking at the impl, but I fail to see how the dynamic client is useful if scheme registration is required, it means you can not use the dynamic client for unknown types?
it does not look like ListType is even required to list in the dynamic client: https://github.com/kubernetes/client-go/blob/f6ce18ae578c8cca64d14ab9687824d9e1305a67/dynamic/simple.go#L253-L275
unknown list types should be *unstructured.UnstructuredList and I think the change to fake dynamic client is an error/bug.
It does not make sense to require scheme registration with the dynamic client, the whole point is it is dynamic right?
Knative issue: https://github.com/knative/eventing/issues/5185

A few points

this is the fake client
This isn't requiring a scheme registration.
Since the listkind is provided by the server in the real client, it provides data that is missing in the fake client.  Missing this data causes problems of consistency when testing with fake client
A fake client used in a unit test is a constrained view of reality that must be predicted in order to write an effective test.  Thus asking for this mapping when creating a fake client is reasonable.

I'm open to allowing this to be specified via function if you like, but the data needs to be returned by the fake client to have parity with the real client, so I don't see an effective way to simply stop requiring this mapping.

I'm open to allowing this to be specified via function if you like, but the data needs to be returned by the fake client to have parity with the real client, so I don't see an effective way to simply stop requiring this mapping.

I'm open to a PR to make the dynamic client take a mapping function instead of a static mapping.  But we have to return the objectmeta on the list  values returned from the fake dynamic client.

oh, I see. There is a new method I should be using, the old method will not work with List.
func NewSimpleDynamicClientWithCustomListKinds(scheme *runtime.Scheme, gvrToListKind map[schema.GroupVersionResource]string, objects ...runtime.Object) *FakeDynamicClient {

OK, followup, there is code in the original NewSumpleDynamicClient that will look at the scheme registered does explode out the ListKinds. So if you need to work with duck types with the Fake Dynamic Client, you have to register the List Kind:
scheme.AddKnownTypeWithName(schema.GroupVersionKind{Group: "example.com", Version: "v1", Kind: "FooBarList"}, &unstructured.UnstructuredList{})

/close


pH_better

I have added an ion visualizer to the pH sensor code, as well as some improvements on the UI.

Don't forget to update the NPM package.


check what duct is "busy with"

ATM running a singularity container through duct shows 2-3% CPU of duct.  I wonder what it is doing.        Ssl  10:58   0:00             Singularity runtime parent
yoh      1623076  0.0  0.0  19740 16584 ?        S    10:58   0:00               python /opt/dsst-defacing-pipeline/run.py -p 0001 -s 01 -- . derivatives/dss-defacing-0.0.5-1
yoh      1623098  0.0  0.0   4040  3056 ?        S    10:58   0:00                 /bin/tcsh /usr/bin/@afni_refacer_run -input /mnt/DATA/data/yoh/1076_spacetop/sub-0001/ses-01/anat/sub-0001_ses-01_acq-MPRAGEXp3X08mm_T1w.nii.gz -mode_deface -no_clean -prefix /mnt/DATA/data/yoh/1076_spacetop/derivatives/dss-defacing-0.0.5-1/bids_defaced/sub-0001/ses-01/anat/sub-0001_ses-01_acq-MPRAGEXp3X08mm_T1w
yoh      1674377  100  0.0<PHONE_NUMBER>64 ?       R    11:10   0:01                   3dcalc -a tmp.00.INPUT.nii -c tmp.05.sh_t2a_thr.nii[1] -expr a*not(bool(c)) -prefix tmp.99.result.deface.nii -datum float -ISOLA

but I could be wrong.

This bugfix helps https://github.com/con/duct/commit/b6c9219bc3a20f781f321f906396ba0a25bc5250
Here's some fun dogfooding :)
$ duct duct sleep 100
duct is executing duct sleep 100...
Log files will be written to .duct/logs/2024.06.21T13.06.49-1223368_
duct is executing sleep 100...
Log files will be written to .duct/logs/2024.06.21T13.06.49-1223371_
Exit Code: 0
Command: sleep 100
Log files location: .duct/logs/2024.06.21T13.06.49-1223371_
Wall Clock Time: 100.003 sec sec
Memory Peak Usage (RSS): 1920 KiB
Memory Average Usage (RSS): 1920.000 KiB
Virtual Memory Peak Usage (VSZ): 221584 KiB
Virtual Memory Average Usage (VSZ): 221584.000 KiB
Memory Peak Percentage: 0.0%
Memory Average Percentage: 0.000%
CPU Peak Usage: 0.0%
Average CPU Usage: 0.000%
Samples Collected: 98
Reports Written: 2

Exit Code: 0
Command: duct sleep 100
Log files location: .duct/logs/2024.06.21T13.06.49-1223368_
Wall Clock Time: 100.073 sec sec
Memory Peak Usage (RSS): 17920 KiB
Memory Average Usage (RSS): 17196.735 KiB
Virtual Memory Peak Usage (VSZ): 682564 KiB
Virtual Memory Average Usage (VSZ): 627538.367 KiB
Memory Peak Percentage: 0.0%
Memory Average Percentage: 0.000%
CPU Peak Usage: 4.0%
Average CPU Usage: 1.658%
Samples Collected: 98
Reports Written: 2

Average CPU Usage: 1.658% is an improvement, but still using a lot more power than we probably need.

that was a very cute use of duct -- to monitor duct ;-)

on my attempt -- we sit at 3.5%
❯ duct duct sleep 100
2024-08-20T09:40:21-0400 [INFO    ] con-duct: duct is executing 'duct sleep 100'...
2024-08-20T09:40:21-0400 [INFO    ] con-duct: Log files will be written to .duct/logs/2024.08.20T09.40.21-3232070_
2024-08-20T09:40:21-0400 [INFO    ] con-duct: duct is executing 'sleep 100'...
2024-08-20T09:40:21-0400 [INFO    ] con-duct: Log files will be written to .duct/logs/2024.08.20T09.40.21-3232074_
2024-08-20T09:42:03-0400 [INFO    ] con-duct: Summary:
Exit Code: 0
Command: sleep 100
Log files location: .duct/logs/2024.08.20T09.40.21-3232074_
Wall Clock Time: 100.003 sec
Memory Peak Usage (RSS): 1966080 bytes
Memory Average Usage (RSS): 1966080.0 bytes
Virtual Memory Peak Usage (VSZ): 5652480 bytes
Virtual Memory Average Usage (VSZ): 5652480.0 bytes
Memory Peak Percentage: 5652480
Memory Average Percentage: 5652480.0
CPU Peak Usage: 0
Average CPU Usage: 0.0
Samples Collected: 98
Reports Written: 2

2024-08-20T09:42:03-0400 [INFO    ] con-duct: Summary:
Exit Code: 0
Command: duct sleep 100
Log files location: .duct/logs/2024.08.20T09.40.21-3232070_
Wall Clock Time: 101.877 sec
Memory Peak Usage (RSS): 14909440 bytes
Memory Average Usage (RSS): 14888140.799999997 bytes
Virtual Memory Peak Usage (VSZ): 250023936 bytes
Virtual Memory Average Usage (VSZ): 247739432.95999998 bytes
Memory Peak Percentage: 250023936
Memory Average Percentage: 247739432.95999998
CPU Peak Usage: 200.0
Average CPU Usage: 3.5020000000000002
Samples Collected: 100
Reports Written: 2

duct duct sleep 100  3.65s user 4.92s system 8% cpu 1:42.05 total

that is not great but not too  bad either, let's just consider it ok for now, although might reconsider if we see it up


Feature: Dynamic Schemas / Columns

Feature: Dynamic Schemas / Columns
Problem:

unable to eliminate similar schema.yml's by abstracting their contents into a callable file/macro.
this issue originally raised in dbt Slack

To illustrate, here's a artificial example. Assume I have a models already called dog and cat.
Also assume that the SQL required to query any animal type is very complex and is worthy of needing a macro for DRY reasons. Then, we would have a macro like:
macros/pet.sql:
{% macro pet(type) %}
  SELECT * FROM {{ref(type)}}
  WHERE im_actually_a_super_complex_macro_needing_sql_statement IS TRUE
{% endmacro %}

And we would express our models like:
models/dog.sql:
{{ pet("dog") }}

models/cat.sql:
{{ pet("cat")

And finally, our schemas like:
models/dog.yml:
version: 2
models:
  - name: dog
    columns:
        - name: name
          description: 'the name of the dog'
        - name: age
          description: 'the age of the dog'
        - name: breed
          description: 'the breed of the dog'

models/cat.yml:
version: 2
models:
  - name: cat
    columns:
        - name: name
          description: 'the name of the cat'
        - name: age
          description: 'the age of the cat'
        - name: breed
          description: 'the breed of the cat'

Again, this is a contrived example, but the problem is magnified in larger projects. i.e. Both dog.yml and cat.yml files are nearly identical except the animal type. Leading to WET code.

Some Possible Solutions:

Allow schema.yml files to be macroable:

models/dog.yml:
{{ schema_pet("dog") }}

models/cat.yml:
{{ schema_pet("cat") }}

macros/schema_pet.yml
{% macro schema_pet(type) %}
version: 2
models:
  - name: {{ type }}
    columns:
        - name: name
          description: 'the name of the {{ type }}'
        - name: age
          description: 'the age of the {{ type }}'
        - name: breed
          description: 'the breed of the {{ type }}'
{% endmacro %}

In this situation, the model may not know which schema belonged to it (barring file name parsing), so the models would also need to specify, I believe:
models/dog.sql:
{{ config(schema = 'dog') }}
{{ pet("dog") }}

models/cat.sql:
{{ config(schema = 'cat') }}
{{ pet("cat") }}

Actually, the majority of the duplication occurs in the columns section of the yml file. I believe introducing a dynamic models.name parameter inside the schema.yml will cause a lot of complications, so even if this was static, but the rest of the file could be dynamic, this would be great.

macros/pet_columns.yml
{% macro pet_columns(type) %}
- name: name
  description: 'the name of the {{ type }}'
- name: age
  description: 'the age of the {{ type }}'
- name: breed
  description: 'the breed of the {{ type }}'
{% endmacro %}

models/dog.yml
version: 2
models:
  - name: dog
    columns: {{ pet_columns('dog') }}

models/cat.yml
version: 2
models:
  - name: cat
    columns: {{ pet_columns('cat') }}

The doc macro would be great for this too, however it's not possible to provide args to the docs/doc macro.
Forgive me if some of my suggestions are incompatible with the current dbt structure. I am new, and not familiar with how the internals work. These are only suggestions, and I welcome the feedback from the dbt masters!

Hey @jordanarseno - cool ideas!
I'm picturing something that looks a lot like these suggestions, but may be a more tractable given how dbt parses schema.yml files. Some options:

Provide a method to register model/column definitions

We can provide a method in the manifest that accepts a schema spec and incorporates it into the manifest. You could call this method from model code (or via a macro):
-- models/base/cat.sql

{% do add_pet_schema_columns('cat') %}

select ...

coupled with:
{% macro add_pet_schema_columns(type) %}

{% do manifest.set_column_schema(this, column='name', description='the name of the ' ~ type) %}
{% do manifest.set_column_schema(this, column='age', description='the age of the ' ~ type) %}
{% do manifest.set_column_schema(this, column='breed', description='the breed of the ' ~ type) %}

{% endmacro %}

We could also provide a higher-level method that configures many columns at once, eg:
{% macro add_pet_schema_columns(type) %}

{% do manifest.update_column_schemas(this,
  {
    "name": {"description": "...."},
    "age": {"description": "...."},
    "breed": {"description": "...."}
  }) %}

{% endmacro %}

Provide arguments to docs blocks

This is a really good idea, and something that I think we're going to tackle implicitly via https://github.com/fishtown-analytics/dbt/issues/1334. While you'll still need to enumerate all of the columns, you'll be able to pull in descriptions from templates docs blocks for sure.

So, this is a solvable problem, and the question becomes: do we want to do this? I think a feature like this would be valuable, but it will also add a bunch of complexity to the parsing code in dbt. Further, it will become more difficult to understand where certain model attributes are being configured from. This would be problematic if we eg. wanted to link to a schema.yml spec from dbt docs. While some descriptions (or tests) may be set in schema.yml files, others might be contributed from macros! I think the lack of provenance for these specifications is something that we should think really hard about when considering to prioritize a change like this.
Let me know what you think1

I agree that schema.yml files can be verbose to create and information is often repeated across schema specs. I don't think we'll want to leverage jinja to solve this problem, but it is very much an issue worth solving.
Closing this in favor of a more actionable issue, but I'm certainly very happy to re-open it for discussion if anyone feels strongly about this :)


Flexibility

Dear sirs,
thanks for giving me the opportunity to evaluate your AI.
AI can find many original and efficient solutions to problems, even different from those initially designed by the programmer.
The problem is that these solutions are not always the best because the programmer has a different view from that of the bot.
This is because every programmer as a human being has difficulty being schematic and translating the initial project he has in mind into consequential, complete and precise instructions.
The human being has a different approach, he thinks by sensations: the programmer usually arranges the project “in itinere” (in progress) according to new needs and new objectives that arise during the development of the project.
While the programmer is flexible and easily changes the parameters of the project and its objectives, the AI proves to be inflexible and rigid: once the initial path is set, it does not accept changes and forces the developer to remain within the initial parameters.
In this way, the initial saving of time is resolved in an increase in time because the AI forces the programmer to start the project again from the beginning, as if it were a new project, rather than making changes in progress.
This could lead the developer to a distrust and resistance in the use of this AI.
I think we need to give the developer the possibility to make changes that are going on, and AI needs to be more flexible.
The risk of an error-generating tool could be overcome with thorough check procedures after changes are made to follow the plan-do-act-check cycle.
Best regards
Gabriele Farina

Can you give concrete examples of where it has been inflexible in responding to your requests?

Dear sir,
Please find below a concrete example of the inflexibility of the AI.
My project starts: “Program that moves photos and videos to folders by year and month.  The program takes as input, as a command line argument, the path where the photos and videos are saved.”
AI quickly implemented the solution.
After this, I wanted to add the following feature: “Program that takes in input moves photos and videos in folders by year and month, taking in input, as a command line argument, the source and destination path.
In addition to the year and month, photos and videos will be classified into the following folders according to the format of their name: "Camera" folder, "Whatsapp" folder, "Instagram" folder, "Screenshot" folder and "Others" folder.”
The AI did not adapt to the new solution: I tried to implement the changes manually and from time to time the AI always gave me the same suggestion "go back to the basic version of the project by deleting the changes I wrote".
As if the AI deemed my edits useless, inefficient, or even wrong.
To solve this problem, I had to rewrite the software from the beginning in a separate file, causing me a considerable waste of time.
The same problem occurs when I wanted every new features i.e. “in addition to search photos and videos in the source path, search them also in each sub-folders”, “once a sub-folders of the source path is empty delete it” and so on.
Anyway, when the project restarts the AI made a very good job.
Best regards
Gabriele Farina
From: Logan Ramos @.>
Sent: Monday, July 24, 2023 4:52 PM
To: microsoft/vscode-copilot-release @.>
Cc: Gabriele Farina @.>; Author @.>
Subject: Re: [microsoft/vscode-copilot-release] Flexibility (Issue #332)
Can you give concrete examples of where it has been inflexible in responding to your requests?
—
Reply to this email directly, view it on GitHub https://github.com/microsoft/vscode-copilot-release/issues/332#issuecomment-1648065830 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AZDQS4LC2UJLPNYBWPZ7XETXR2DYHANCNFSM6AAAAAA2VJPGBI .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AZDQS4MQJ4O2UHNVCWQHDBDXR2DYHA5CNFSM6AAAAAA2VJPGBKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTCHN6SM.gif Message ID: @.*** @.***> >


Add error collections

if err != nil {
	return nil, err
}

Seeing this block over and over again inside the score submission handler really bothered me, so I've tried to come up with a solution for that. Not sure how well this will work, or if this is even a good way to handle this issue.
Here is a quick example:
collection := common.NewErrorCollection()

result, err := SomeFunctionWithError()
collection.Add(err)

result2, err := SomeFunctionWithError2()
collection.Add(err)

if collection.HasErrors() {
    return nil, collection.FirstError()
}

@ascenttree what do you think about this approach


Memory leak, too many connections to SQL

In a bigger application, this cause fatal error Too many SQL connections opened. Also slows down api generation by alot and doesn't seem to do anything after the application is bootstrapped.

This will also fix the Unauthenticated bug (https://github.com/mpociot/laravel-apidoc-generator/issues/188#issuecomment-316810677).


Why last row not rendering in Sugiyama Algorithm?

`
add.setOnClickListener{
if (clicked_nodes.size == 1) {
graphView.adapter.graph.addEdge(Node(clicked_nodes[0]), Node(nodeCount))
graphView.adapter.notifyInvalidated()
            nodesData.add(0)
            nodeCount++
        } else {
            try {
                graphView.adapter.graph.addEdge(searchNodeByData(0, graphView.adapter.graph.nodes), Node(nodeCount))
            } catch (e: ArrayIndexOutOfBoundsException) {
                graphView.adapter.graph.addEdge(Node(0), Node(nodeCount))
            }
            graphView.adapter.notifyInvalidated()

            nodesData.add(0)
            nodeCount++
        }
    }

`

After debug, I found out that other Nodes fully not exists as Views

for some reason nodes that not have children not rendering

Thats odd. Could you create a repo with a small executable sample project which shows this issue, so we can look at it?

I created new project and copy my code, then problem solves...

Problem solves, in original project I added bottom padding and this breaks last row, hah

Yes we should include the padding in the view calculation. Thanks for that.

This should be fixed in the new version! Try v0.7.0


Lower LightGBM test AUC base value

Description

The new LightGBM from SynapseML gives lower AUC value which makes smoke tests failed.
This PR lowers the test base value.
Related Issues

See https://github.com/microsoft/recommenders/issues/1615
Checklist:

[X] I have followed the contribution guidelines and code style for this project.
[X] I have added tests covering my contributions.
[ ] I have updated the documentation accordingly.
[X] This PR is being made to staging branch and not to main branch.

@anargyri the change made than Ilia requires to also change the result of AUC, meaning, do we still need to merge this PR?

Codecov Report

Merging #1619 (73b6a0d) into staging (6a20936) will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           staging    #1619      +/-   ##
===========================================
+ Coverage    59.12%   59.20%   +0.07%
===========================================
  Files           88       88
  Lines         8942     8956      +14
===========================================
+ Hits          5287     5302      +15
+ Misses        3655     3654       -1

Flag
Coverage Δ

pr-gate
59.20% <100.00%> (+0.07%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

recommenders/models/rbm/rbm.py
83.60% <100.00%> (+1.35%)
:arrow_up:

recommenders/evaluation/spark_evaluation.py
87.05% <0.00%> (+0.44%)
:arrow_up:

Continue to review full report at Codecov.

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

@anargyri the change made than Ilia requires to also change the result of AUC, meaning, do we still need to merge this PR?

Yes, I think we need to merge this PR. The other PR (early stopping parameter) does not affect AUC at all.


Handler not converting 404 in Laravel 5.5

It shows JSON for just regular browser requests. It should show JSON on api requests & for HTTP browser requests the regular 404 view.
If I use https://github.com/flugger/laravel-responder#option-2-use-convertsexceptions-trait it returns the 404 view but not a 404 JSON response just a error with no message.
Right now i'm just using this with Option 2:
if($request->wantsJson()) {
       $this->convertDefaultException($exception);

       if ($exception instanceof HttpException) {
             return $this->renderResponse($exception);
       }
}

Hi - I agree the handler should check for this. Just tagged v2.0.9 which makes the change. I've also added a little paragraph about it to the Option 2 section of the handler docs.


SQL: counters for rows read and written

In a durable object, state.storage.sql now has two new properties: rowsRead and rowsWritten. These count the number of rows read and written by the underlying SQLite database.
This is more than just the number of rows returned from queries or the number of records inserted/updated; it counts all rows read by the database engine in the processing of a query. For example, SELECT MAX(unindexed_column) FROM a_table returns a single result but reads every row from disk, and rowsRead reflects that.

Does this patch come directly from libSQL? If so we need to include license and copyright information, probably in the description header for each patch file. (Looks like it's MIT license, so no issues using it, but still need to include the copyright.)

The patch now contains an appropriate copyright header pointing at libsql.


Updating Querqy documentation for OS 2.3

Signed-off-by: ariamarble<EMAIL_ADDRESS>Description
Updates Querqy documentation for OS 2.3
Issues Resolved
#1054
Checklist

[X] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

tech review complete


MSSQL issue to backup

Summary

Hi Currently having issue to backup MSSQL on windows,

the compression is not working, when activated does not backup to the s3 bucket, but if deactivated works correctly

its not deleting the .bak from the temp route

Steps to reproduce

Very important because the MSSQL is a windows i had to cifs mount the shared route if not it wont work
mount -t cifs //<IP_ADDRESS>/tmp /root/bakssql/temporal --verbose -o vers=2.1,user=administrador,password=mypasss,domain=mydomain

This is the docker compose, using S3 minio bucket which gets error on the backup when having GZ

services:
  example-mssql-db-backup:
    container_name: example-mssql-db-backup
    image: tiredofit/db-backup:latest
    volumes:
      - ./backups:/backup
      - ./temporal/backups/:/tmp/backups    # shared tmp backup directory
      #- ./post-script.sh:/assets/custom-scripts/post-script.sh
    environment:
      - TIMEZONE=America/Bogota
      - CONTAINER_ENABLE_MONITORING=FALSE
      - CONTAINER_NAME=example-mssql-db-backup
      - TEMP_PATH=/tmp/backups
      - DEFAULT_COMPRESSION=GZ
      - DEFAULT_CHECKSUM=NONE
      - DEBUG_MODE=TRUE
      - DEBUG_BACKUP_MSSQL=TRUE
      - DEFAULT_BACKUP_LOCATION=S3
      - DEFAULT_S3_BUCKET=mssqlbucket
      - DEFAULT_S3_KEY_ID=miniouser
      - DEFAULT_S3_KEY_SECRET=mypass
      - DEFAULT_S3_HOST=<IP_ADDRESS>:9000
      - DEFAULT_S3_PROTOCOL=http
      - DEFAULT_S3_PATH=backup
      - DB01_TYPE=mssql
      - DB01_HOST=<IP_ADDRESS>
      - DB_PORT=2866
      - DB01_NAME=test
      - DB01_USER=sa
      - DB01_PASS=mypass
      - DB01_DUMP_INTERVAL=1400           # backup every minute
      - DB01_CLEANUP_TIME=1400         # clean backups they are older than 5 minute

it does not delete the .bak from the temp routes

root@dockers:~/bakssql/temporal/backups# ls -lah
total 6.4M
drwxr-xr-x 2 root root 8.0K Nov 29 23:42 .
drwxr-xr-x 2 root root    0 Nov 13 21:57 ..
-rwxr-xr-x 1 root root 3.2M Nov 29 23:39 mssql_test_<IP_ADDRESS>_20231129-233944.bak
-rwxr-xr-x 1 root root 3.2M Nov 29 23:42 mssql_test_<IP_ADDRESS>_20231129-234232.bak

What is the expected correct behavior?

It should in theory backup with compress the .bak

and should delete .bak in the temp routes

Relevant logs and/or screenshots

was to long to add it here
https://pastebin.com/6mHjkscd)

Possible fixes

reading the log it shows that .bak is created correctly in the temp

+ write_log info 'DB Backup of '\''mssql_test_<IP_ADDRESS>_20231129-234704.bak.gz'\'' completed successfully'
+ var_true ''
+ '[' '' = true ']'
+ '[' '' = yes ']'
+ output_off
+ '[' true = true ']'
+ set +x

but later in the log it says cannot find the .gz
++ run_as_user stat -c%s /tmp/backups/mssql_test_<IP_ADDRESS>_20231129-234704.bak.gz
++ sudo -Eu dbbackup stat -c%s /tmp/backups/mssql_test_<IP_ADDRESS>_20231129-234704.bak.gz
stat: cannot stat '/tmp/backups/mssql_test_<IP_ADDRESS>_20231129-234704.bak.gz': No such file or directory
+ dbbackup_size=
++ run_as_user date -r /tmp/backups/mssql_test_<IP_ADDRESS>_20231129-234704.bak.gz +%s
++ sudo -Eu dbbackup date -r /tmp/backups/mssql_test_<IP_ADDRESS>_20231129-234704.bak.gz +%s
date: /tmp/backups/mssql_test_<IP_ADDRESS>_20231129-234704.bak.gz: No such file or directory
+ dbbackup_date=
+ case "${backup_job_size_value,,}" in
+ backup_job_size_value=1
+ '[' 1 = 1 ']'
++ run_as_user stat -c%s /tmp/backups/mssql_test_<IP_ADDRESS>_20231129-234704.bak.gz
++ sudo -Eu dbbackup stat -c%s /tmp/backups/mssql_test_<IP_ADDRESS>_20231129-234704.bak.gz
stat: cannot stat '/tmp/backups/mssql_test_<IP_ADDRESS>_20231129-234704.bak.gz': No such file or directory
+ filesize=

which i assume thats why it does not delete it

The user-provided path /tmp/backups/*. does not exist.
+ run_as_user rm -rf /tmp/backups/mssql_test_<IP_ADDRESS>_20231129-234704.bak.gz.
+ sudo -Eu dbbackup rm -rf /tmp/backups/mssql_test_<IP_ADDRESS>_20231129-234704.bak.gz.
+ run_as_user rm -rf /tmp/backups/mssql_test_<IP_ADDRESS>_20231129-234704.bak.gz
+ sudo -Eu dbbackup rm -rf /tmp/backups/mssql_test_<IP_ADDRESS>_20231129-234704.bak.gz
+ run_as_user rm -rf /tmp/backups/mssql_test_<IP_ADDRESS>_20231129-234704.bak.gz
+ sudo -Eu dbbackup rm -rf /tmp/backups/mssql_test_<IP_ADDRESS>_20231129-234704.bak.gz
+ var_true ''
+ '[' '' = true ']'
+ '[' '' = yes ']'

but in the container i can see the .bak but not the gz
[tiredofit/db-backup:4.0.25 23:53:05 /] $ ls -lah /tmp/backups/
total 9.5M
drwxr-xr-x 2 root root 8.0K Nov 29 23:47 .
drwxrwxrwt 1 root root 4.0K Nov 29 23:47 ..
-rwxr-xr-x 1 root root 3.2M Nov 29 23:39 mssql_test_<IP_ADDRESS>_20231129-233944.bak
-rwxr-xr-x 1 root root 3.2M Nov 29 23:42 mssql_test_<IP_ADDRESS>_20231129-234232.bak
-rwxr-xr-x 1 root root 3.2M Nov 29 23:47 mssql_test_<IP_ADDRESS>_20231129-234704.bak

Thank you

Interesting, I am stumped on this one. The logs are very useful from the pastebin:
Specifically lines 1249.
What would happen if from inside the container you ran this?
sudo -Eu dbbackup pigz -q -3 -p 2 /tmp/backups/mssql_test_<IP_ADDRESS>_20231129-234704.bak
Note, your timestamp may be different from other backups, please adjust the 20231129-234704.bak accordingly.
With a successful compression the .bak should disappear and be replaced with a .gz extension.

Just to add to this, can I also get the output of image_version from inside container. I'm working on another issue related to MSSQL and they have similar settings yet compression is working so this is where things get quite interesting.

Hi @tiredofit thanks for the reply, sure thing the version is tiredofit/db-backup:4.0.25 18:13:50 /
This is what i got
[tiredofit/db-backup:4.0.25 18:20:46 /tmp/backups] $ sudo -Eu dbbackup pigz -q -3 -p 2 /tmp/backups/mssql_test_<IP_ADDRESS>_20231129-234704.bak
[tiredofit/db-backup:4.0.25 18:20:50 /tmp/backups] $ ls -lah
total 32M
drwxr-xr-x 2 root root 8.0K Dec  4 18:16 .
drwxrwxrwt 1 root root 4.0K Dec  4 18:16 ..
-rwxr-xr-x 1 root root 3.2M Nov 29 23:39 mssql_test_<IP_ADDRESS>_20231129-233944.bak
-rwxr-xr-x 1 root root 3.2M Nov 29 23:42 mssql_test_<IP_ADDRESS>_20231129-234232.bak
-rwxr-xr-x 1 root root 3.2M Nov 29 23:47 mssql_test_<IP_ADDRESS>_20231129-234704.bak
-rwxr-xr-x 1 root root 3.2M Nov 30 00:01 mssql_test_<IP_ADDRESS>_20231130-000125.bak
-rwxr-xr-x 1 root root 3.3M Dec  1 00:01 mssql_test_<IP_ADDRESS>_20231201-000126.bak
-rwxr-xr-x 1 root root 3.3M Dec  2 00:01 mssql_test_<IP_ADDRESS>_20231202-000126.bak
-rwxr-xr-x 1 root root 3.3M Dec  3 00:01 mssql_test_<IP_ADDRESS>_20231203-000125.bak
-rwxr-xr-x 1 root root 3.3M Dec  4 00:01 mssql_test_<IP_ADDRESS>_20231204-000125.bak
-rwxr-xr-x 1 root root 3.3M Dec  4 18:13 mssql_test_<IP_ADDRESS>_20231204-181259.bak
-rwxr-xr-x 1 root root 3.3M Dec  4 18:16 mssql_test_<IP_ADDRESS>_20231204-181628.bak
[tiredofit/db-backup:4.0.25 18:20:52 /tmp/backups] $

thank you

So very strange. Could this be because of a mount somewhere? It should compress in place and then remove the old file post that pigz command.
Now that I think about it, I see the problem here. All of the files are owned as root and I bet /tmp/backups is also owned as root due to the way the temp folder is mapping.
This will reduce security IMO, but if you try DBBACKUP_USER=root do backups then start to work?

hi @tiredofit
Thank you so much that did the trick DBBACKUP_USER=root
yeah it seems because if your going to mix the windows MSSQL with the linux container it has to mount to see it


WIP: Enable windows CI on travis

Travis now adds support for Windows environment. This PR adds initial support for Windows building on travis.

CI failed because of no enough space on disk. Seems this is a common issue of travis. refer https://travis-ci.community/t/there-is-not-enough-space-on-the-disk/338/3.

cc @jterry75 - FYI this is coming

/assign @Random-Liu

/lgtm


zenburn theme overrides for hints

@stevemessick, ptal, css to theme codemirror's hints popup.

fix https://github.com/dart-lang/dartpad_ui/issues/46

LGTM!
On Fri, Jan 30, 2015 at 12:12 PM, Devon Carew<EMAIL_ADDRESS>wrote:

fix #46 https://github.com/dart-lang/dartpad_ui/issues/46
—
Reply to this email directly or view it on GitHub
https://github.com/dart-lang/dartpad_ui/pull/66#issuecomment-72263073.


Error about intel-PCM in VMware virtual system

Hello developers, when I build the code in a Ubuntu 18.04 virtual machine and run, there is one question troubled me for a long time.
**yang@Pluto:~$ pcm
Intel(r) Performance Counter Monitor (2024-11-05 15:03:02 +0100 ID=a83f3125)
Detected a hypervisor/virtualization technology. Some metrics might not be available due to configuration or availability of virtual hardware features.
=====  Processor information  =====
Linux arch_perfmon flag  : yes
Hybrid processor         : no
IBRS and IBPB supported  : yes
STIBP supported          : yes
Spec arch caps supported : yes
Max CPUID level          : 32
CPU family               : 6
CPU model number         : 183
pcm: /usr/src/packages/BUILD/src/types.h:1561: pcm::uint32 pcm::build_bit_ui(pcm::uint32, pcm::uint32): Assertion `end <= 31' failed.
DEBUG: caught signal to interrupt (Aborted).**
What can I do to solve the bug?

Hi, thanks for reporting the issue. Unfortunately it is not reproducible in our VMWare environment. Would it be possible to compile pcm in debug mode (sh scripts/debug-build.sh) and then run "gdb ./pcm" and then "bt" to show the assert callstack. This should be helpful to understand what is going on.


Create context menu on tiles

Need to add context menu on tiles to add functions to apply to the tiles

was not able to get into 2.3.3

Not going to add context menu at this time may re-open when ready


Scheduled messages

How I can send scheduled messages? I am sorry for my carelessness, if method is on the surface.

I'd think that implementing this in this library would be out of scope.
Depending on your needs you could utilise the following:

postTimer := time.NewTicker(<duration>)
https://github.com/robfig/cron

In the first instance, have a goroutine running that receives from <-postTimer.C and do the telebot.Send() when timer ticks, i.e. every several hours.
In the second instance, using the cron library, you define a function that sends the message with telebot.Send(), i.e.
c := cron.New()
c.AddFunc("30 * * * *", func() { telebot.Send(<chat>, "sends this message every 30 min") })

Hope that answers your question.

Or simply:
time.AfterFunc(time.Minute, func() {
  b.Send(m.Sender, "I'm back!")
})


chore(ci): use label so Helm stops replacing secrets

Thanks for the PR!
Deployments, as required, will be available below:

Frontend
Backend

Please create PRs in draft mode.  Mark as ready to enable:

Analysis Workflow

After merge, new images are deployed in:

Merge Workflow

lets put that as conditionals, as we don't want to persist pr deployment secret objects


Added link to Blog in Header and Footer

Fixes #2268
Describe the changes you have made in this PR -
There is no direct link of CircuitVerse blog site available in the navbar as well as footer in the main website of CircuitVerse
Added link to Blog in the nav and footer, which will redirect users to https://blog.circuitverse.org

Coverage remained the same at 84.751% when pulling ac05c27526c6e055febbd330dbda8ffe71aac840 on garimasingh128:add-blog#2268 into 79d3836ee3453f13557395d96e0e7f4bf2d16714 on CircuitVerse:master.

Can you add screenshots for the changes you made ?

Fixed in #https://github.com/CircuitVerse/CircuitVerse/pull/2334


Beaglebone Black AIO Fix

@arfoll Here is my attempt at fixing the AIO logic for the BBB. Sorry, my other fixes are lumped in with the other BBB stuff, but check out the latest commit.
Commit message
Beaglebone AIO pins seem to be a little different than most boards, so this is my attempt to work with that without impacting other boards. I added a new flag in mraa_board_t to indicate whether or not the aio pins are sequential. One the beaglebone, they are not. To go along with this, I added a new device mraa_aio_dev_t, that will map each aio to a physical pin.
In the main aio logic, if aio_non_seq is true for the board, the manual mapping is used, otherwise the old mathematical mapping is used.

This fix does assume that all of the board initialize the mraa_board_t structure correctly. Might be worth having a function that allocates an mraa_board_t, initializes it properly to 0's/NULL as appropriate, and returns a pointer. I didn't check the other boards to see if they did calloc to allocate the struct like the beaglebone module does.

Looks good to me. Although I think we should update the documentation for the AIO functions/class and mention analog pins are always 0 indexed regardless of their position on the I/O header.

@Propanu I agree with that, for sure. Before looking at the source, I thought that the mraa_aio_init function took the raw pin number as an argument. Looking at the docs again, they do say channel number, but the variable is called pin, so it's a little confusing.

Merged, this is awesome. Thanks alot

Updated the docs as @Propanu suggested


Fixes for rustc 1.0.0-nightly (3ef8ff1f8 2015-02-12 00:38:24 +0000)

Even though this is a effective duplicate of #152, I use different syntax for the env::args().collect() calls.
Also there are some bugfixes, without which racer didn't work properly for me and RustAutoComplete (Sublime Text Plugin)

It seems besides the obvious fixes, there is another issue though, as simple completions now fail ungracefully:
RUST_BACKTRACE=1 target/racer complete 3 14 /Users/byron/Documents/dev/depot/src/keynote/res/rula/src/bin/reversor.rs
PREFIX 14,14,
thread '<main>' panicked at 'assertion failed: index < len', /Users/rustbuild/src/rust-buildbot/slave/nightly-dist-rustc-mac/build/src/libcollections/vec.rs:577
stack backtrace:
   1:        0x110a430a7 - sys::backtrace::write::h1864de2e29cfcfa3jWA
   2:        0x110a6abe3 - failure::on_fail::hb779450ca026c246YwJ
   3:        0x1109b5708 - rt::unwind::begin_unwind_inner::h252a44c757d4dd84eeJ
   4:        0x1109b5e71 - rt::unwind::begin_unwind_fmt::h6b3ebabbe58c4ea9KcJ
   5:        0x110a6a50e - rust_begin_unwind
   6:        0x110ab8757 - panicking::panic_fmt::haa2c8b833452fc7bSvw
   7:        0x110ab29f2 - panicking::panic::hdc8095395c2cbfb7Vtw
   8:        0x10fea2b42 - vec::Vec<T>::remove::h12763448109877615233
   9:        0x10fe810fd - racer::nameres::resolve_path::h5d65255f3a1f3cd6YNe
  10:        0x10fec376e - racer::complete_from_file::hc5d64468514f4bceZ2h
  11:        0x10fec7fce - complete::ha528e44216b55ed7lji
  12:        0x10fecb476 - main::h22bfb46b6f99fa0fBwi
  13:        0x110ae21a9 - rust_try_inner
  14:        0x110ae2196 - rust_try
  15:        0x110a6b62d - rt::lang_start::ha42b32fa3423fe20ZqJ
  16:        0x10fecb7bf - main

The problem above is a plain old out-of-bounds issue, as the commandline parameter for the right column is not checked or clamped into range.
I think all that's needed is to either clamp it, or to fail with a an improve error message.

Even though the latest commit fixes issues, there are plenty of tests missing to prevent regression. Also I don't know how things are supposed to be running, which is why it might be best if the maintainers would have a look (or let me know how it should be done).
Thank you

Hi @Byron ,
Thanks for this patch. I've applied because it fixes so much, and will take a look at the TODOs
Cheers,
Phil


LOW: Kubernetes Ingress status field not updating

Do you want to request a feature or report a bug?

Bug (I think)

What did you do?

Migrated Ingress controller from NGINX to Traefik (We need h2c)
Had existing Ingress rules that were working in every sense of the word

What did you expect to see?
The Ingress rules should have their status fields updated if Traefik has decided to use them.
In Rancher, they use this status field to mark an Ingress as active or inactive.
Looks something like this:
# Ingres YAML
...
status:
  loadBalancer:
    ingress:
    - hostname: internal-a31ddaefe4dec11ea897802728661815-1703866244.us-west-2.elb.amazonaws.com

I'm actually not sure how NGINX found that LB as I wasn't using service type LoadBalancer, but using NodePort. Maybe it was resolving the DNS CNAME?
What did you see instead?
Status fields are left empty so in Rancher even though the Ingress' are active they aren't flagged as such.

Defaults from the containous Traefik helm chart plus an extra --metric.prometheus=true flag.

If applicable, please paste the log output in DEBUG level (--log.level=DEBUG switch)
N/A

Hello @iridian-ks,
By default, the Traefik v2 helm chart does not load ingress objects, only CRD objects.
If you enable the kubernetesingress provider as mentioned in the values.yaml file, you will need to add the ingress endpoint to tell traefik what endpoint to use to populate the status field:
https://docs.traefik.io/v2.2/providers/kubernetes-ingress/#ingressendpoint
Hope that helps.

Hi @dtomcej ,
Thanks for the swift response.
I apologize for not showing all my values:
---

deployment:
  replicas: 2
  podAnnotations:
    "prometheus.io/scrape": "true"
    "prometheus.io/port": "9000"
    "prometheus.io/path": "/metrics"

service:
  type: NodePort

globalArguments: []
additionalArguments:
  - "--providers.kubernetescrd.ingressclass=traefik"
  - "--providers.kubernetesingress=true"
  - "--providers.kubernetesingress.ingressclass=traefik"
  - "--metrics.prometheus=true"

My Ingress rules also have the kubernetes.io/ingress.class: traefik annotation.
I do want to mention - everything is working perfectly from a user perspective. As in, Ingress rules are working great. It's just the status field in the Ingress that isn't getting updated.

Hello @iridian-ks,
thanks a lot for your interest :-)
It's because you did not configure the ingress provider to do so.
You need to set the publishedService https://docs.traefik.io/providers/kubernetes-ingress/#publishedservice to traefik, in order to have the status updated with the external IP.
Does it make sense?
As it's more a question rather than a bug, I'll close that issue.
Do not hesitate to look out for our community forum https://community.containo.us/c/traefik/traefik-v2/10 and raise a topic there!


chore(cargo): inherit repository in identity_verification

Description of change
Inherit repository key for correctly labeling/pointing to the GitHub repo.
This makes location of the source code easier to see Crates.io. 🙂
This change only affects identity_verification crate. All other modules already correctly inherit repository.
Links to any relevant issues
None
Type of change
Add an x to the boxes that are relevant to your changes.

[ ] Bug fix (a non-breaking change which fixes an issue)
[ ] Enhancement (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation Fix

How the change has been tested
None
Change checklist
Add an x to the boxes that are relevant to your changes.

[x] I have followed the contribution guidelines for this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Thank you for your contribution @Foorack :)


feat: initial work on i18n

This should be ready for an initial round of feedback.
The biggest blocker at the moment I think it's that the routes are /docs/latest/es-ES. It will probably be better for them to be /es-ES/docs/latest/ and all be lower case.
This will touch docusaurus and CDN configurations. And we might want to consider the scenario for versioned docs as well.


Update rootfs/api/serializers.py

Changed the regexp that checks the memory limits (prohibit KB, MB or GB and allow only K, M or G)

@eudalov as per my closing comment on #165, let's close this PR.

Closing according to comments in #165


Remove shell escaping from buffer text extraction

This extra escaping was ending up posted in the body of comments
directly to GitHub, showing up in the final review output. Given the way
that the variables are passed to the gh CLI, this escaping is not
necessary.
Fixes: #118

Very nice.


Encapsulation of expression in HPTIME conversion macros

The HTPTIME2EPOCH() and EPOCH2HPTIME() macros do not encapsulate the provided expression, this means a call such as:
MS_EPOCH2HPTIME (a+b)

does not work as expected since the conversion is applied only to b.
The expression is easily encapsulated with something like:
#define MS_EPOCH2HPTIME(X) (X) * (hptime_t) HPTMODULUS

but perhaps the entire thing should also be encapsulated as well.
Before committing this change, we need to check the internal usage to make sure any casting is not being done that relies on the current usage.

Fixed.  Will be in next 2.x and 3.x releases.


Move website dir in its own repo

What should be done?
website dir contains 520 unnececery files and fetching these files everytime is realy time-consuming ,storage-consuming and bandwidth-consuming .. It would be nice to move this dir in its own repo and make developers happy
Thanks
Where?
website dir
Definition of Done

handler updated;
all backends updated;
unit tests added/updated;
integration/compatibility tests added/updated;
spot refactorings done;
user documentation updated or an issue to create documentation created;
something else?

Well, we (developers) update documentation quite often, too, and making code changes and documentation changes in the same PR to a single repo is much easier.
What is your use case that is time-consuming, storage-consuming, and bandwidth-consuming? Are you cloning the entire repository with LFS each time you do something? What is that "something" and why?

The only thing that could be fixed I can think of should be fixed by e98188452d531817bc9fc3eca9383e60a720c03b


MdisPlugin properly loads rotation angles from Isd.

Added intersect and non-intersect test to MdisNacSensorModelTest. Updated imageToGround typo in collinearity eqs.

Good work around for testing private methods :+1: :heavy_plus_sign: :a:


Add preflight check for required programs

Fixes #47

Oh of course, this breaks the travis tests because travis doesn't have qsub yet.
Not sure why this breaks pep8, it passed when I tested it here.

Argggh... travis!

I wonder what we should do... It wouldn't be hard to just configure travis to apt-get install gridengine-client even if we don't configure it for now...

Will add to Travis config tomorrow and see what it does.


Update minimum_required_os for Xcode >= 11.4

This pull request makes the xcode resource aware of the minimum OS requirements for Xcode 11.4 and later.

Thanks @nickdowell, much appreciated!


Microservices and exception handling

:+1: shipit
https://images.duckduckgo.com/iu/?u=http%3A%2F%2Fmedia.riffsy.com%2Fimages%2F51b124323761ef98a388ba5b1a335256%2Fraw&f=1


Editor van Night werkt niet

Employee: Editor van Night werkt niet.

The form has a text box to display the contents of the boolean night property of ShiftType.
The getter in TabShiftDatePanel tries to equal to a boolean value instead of a string and therefore always returns false and the value is not stored.
wrong:
public Boolean getNight(){
return jNightTextField.getText().equals(true);
}
right:
public Boolean getNight(){
return jNightTextField.getText().equals("true");
}

Was fixed in #4

solved in v0.36


Updated Nation.cs

Added new Nations options and commented out removed/reassigned nations.

@trydis this is my first pull request so let me know if there's anything I have done incorrectly!

Thanks @CoverJ, would be great if you could delete the removed nations 👍 Just push another commit to the same branch and it will show up here.
Leagues would be appreciated too 😄

You also need to update clubs.

@trydis sorry if this has become a bit of a mess but nations, leagues, and clubs should all be updated. I did it by scraping all the data from the FIFA 17 FUT DB. (https://www.easports.com/au/fifa/ultimate-team/fut/database) and then made a little program to go through and get all the unique nations/leagues/clubs, stripped out accented characters and replaced with just the english chars and then gave it a quick browse to make sure it all made sense, like i removed the "1. " from the beggining of some club names, changed cote d'ivoire to ivory coast etc.
I also added the little update from @kauefelipe in #305 so can search via filters rather than just resource id
Let me know if this needs modification!

Like @A-VORONKIN mentioned, the teams (which league they belong in) must be updated as well.
Right now the build has 409 errors because of the changed league values:
https://ci.appveyor.com/project/trydis/fifa-ultimate-team-toolkit/build/4.0.1-PullRequest.289+6.build.70#L1123

fixed in #309


prune unused imports

What does this PR do?

Fixes # (issue)
Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests? [not needed for typos/docs]
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review

[ ] Is this pull request ready for review? (if not, please submit in draft mode)

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Hello @Borda! Thanks for updating this PR.

In the file flash/core/data/data_module.py:

Line 324:112: W504 line break after binary operator


use pkg/wellknown

Please provide a description of this PR:
fixes: https://github.com/istio/istio/pull/46823

@zirain: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
ac9879df9e7aac24321cef87f736fb0486050ae3
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-istiodremote-mc_istio
3778fc44ae018fc1ce3f32dbff557507304b79ab
link
true
/test integ-pilot-istiodremote-mc

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-istiodremote-mc_istio
3778fc44ae018fc1ce3f32dbff557507304b79ab
link
true
/test integ-pilot-istiodremote-mc

integ-pilot_istio
3778fc44ae018fc1ce3f32dbff557507304b79ab
link
true
/test integ-pilot

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

why? Is this removed from go-control-plane

maked as deprecated.


feat: Add user notes

Added user notes to the user page
Allowed staff only to add notes
Users can only see Public notes and User notes
Staff can only see their Private notes and all other notes which are not Private of other staff
Private note can only be deleted by its creator
Added a unittest
Added translations to babel

Awesome job, gave you some comments, will probably return to this PR tomorrow

This is problematic for future use as API. Consider changing this behavior.
On Sun, Oct 11, 2020, 12:18 orronai<EMAIL_ADDRESS>wrote:

@orronai commented on this pull request.
In lms/lmsweb/views.py
https://github.com/PythonFreeCourse/lms/pull/240#discussion_r502889241:

@@ -208,6 +208,50 @@ def share():
return fail(400, f'Unknown or unset act value "{act}".')

+@webapp.route('/notes/int:user_id', methods=['GET', 'POST'])
+@login_required
+def note(user_id: int):

if not current_user.role.is_manager:

This function is only for creating and deleting notes.
A user can't do any of that.
The fetching is directly in the user template file.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/PythonFreeCourse/lms/pull/240#discussion_r502889241,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALMPSZEEFDHAJ3IO7CAOATSKFZ7VANCNFSM4SKBRRLA
.


Cut/Paste not working

First of all, congrats for the work you've been doing.
However, I encounter the same problem when cutting and then pasting a file from one folder to another.
Specifically, I cut an .srt file and then try to paste it in another folder so as to rename it after the video's name.
Each and every time, although it says that the operation has been performed, it has not.
It is rather irritating and, to be honest, I uninstalled Files since I cannot perform this simple task reliably.
Please fix the bug.
My system runs on Win11 and the Files version used is 2.3.1 and 2.3.3.

You will need to upload the logs files directly to github. You can get them in files settings > about > open log folder.

debug_fulltrust.log
debug.log

Where when you copying from and too. A local drive, external device like a phone, network drive, FTP server?

@yaichenbaum

I can't reproduce this, is it posible the item is open?

If you mean if it's open in another application when I try to move it, no, it is not.

Merging with #8972


allow inclusion in C programs

Including "c_api.h" from programs compiled with C compilers errors due to required inclusion of C++ headers. These edits allow c_api.h to be included directly in C programs.

Thanks for this @drewmiller !
Is it possible to share a link to the C project you're working on that needs to link against LightGBM? No worries if not, we'll understand if it's proprietary.
Also would you be willing to write up a feature request at https://github.com/microsoft/LightGBM/issues explaining why it would be valuable for C projects to be able to link against LightGBM directly, with specific examples of such projects if you have them? If this is a feature that people care about, we'll want to add a continuous integration job that tests that linking such a project works, to ensure that compatibility issues aren't accidentally introduced in the future.

Thanks for this @drewmiller !
Is it possible to share a link to the C project you're working on that needs to link against LightGBM? No worries if not, we'll understand if it's proprietary.

Unfortunately, it's proprietary.

Also would you be willing to write up a feature request at https://github.com/microsoft/LightGBM/issues explaining why it would be valuable for C projects to be able to link against LightGBM directly, with specific examples of such projects if you have them? If this is a feature that people care about, we'll want to add a continuous integration job that tests that linking such a project works, to ensure that compatibility issues aren't accidentally introduced in the future.

Sure.

Thanks for the great product, MS team. I've added a simple feature request, and I'm happy to help write a toy example for inclusion in a unit testing pipeline if helpful. I'm not super familiar with the code base yet, so I recognize my "trivial" commit may have cascading effects. However, this software would be more useful to me with this feature, so I thought I'd open the door with some proposed changes, and I'm happy to help bring them into existence however I can. That includes bulking out the feature request as necessary. Warm regards.

Thanks very much for opening #4609. I think it's something we should consider supporting and should having  tests for, but that work doesn't need to be done on this PR.
Could you please merge the latest master into this branch? We recently had to turn off the CUDA CI jobs due to some infrastructure problems (#4611). Sorry for the inconvenience.

Thanks very much for opening #4609. I think it's something we should consider supporting and should having tests for, but that work doesn't need to be done on this PR.
Could you please merge the latest master into this branch? We recently had to turn off the CUDA CI jobs due to some infrastructure problems (#4611). Sorry for the inconvenience.

Done; although the CI seems to be hanging at all-successful

although the CI seems to be hanging at all-successful.

This is a GitHub protection from miners: https://github.com/microsoft/LightGBM/pull/4266#issuecomment-837346143. We have to click Approve and run button for PRs from a first-time contributors after each their commit.

@shiyu1994  can you take a look on this?

/gha run r-valgrind

/gha run r-solaris


Setting up a basic multilingual content management system with goHugo

Hello !
The aim was to be able to easily adapt this page to several languages, so I've used the basic language management system offered by goHugo, which works very well with Netlify.
At the moment we only have 2 languages, English and French.
Light mode :

Dark mode :

It's pretty simple :

If you want to modify an existing language, simply go to the corresponding file in the i18n folder, e.g. i18n/en.yml (for english) and modify the contents. If a translation key does not exist, it automatically switches to English content.

If you want to add a new language, it's only a 2-step process

In the hugo.toml file, add the new language (in this case, Spanish in 3rd position)

In the i18n folder, add a yml file with the language shortname, for example es.yml (if spanish), and translate each key in this file.

That's it :-)

Great idea. Do you need help with something?

Great idea. Do you need help with something?

I'm sure there are a few translations to improve, but otherwise it's fine.


pgrep docker daemon

Issue #124
Signed-off-by: Thomas Sjögren<EMAIL_ADDRESS>

Closed due to #153


[feature] figure out how to list packages from remarkable sections

we should have a quick way of only listing packages from toltec repo, this is first step towards UI for package management

#solved


Support flashing recovery image

https://dl.google.com/dl/edgedl/chromeos/recovery/recovery.conf have enough info to determine an image to recover from a model name (first part of hwid), which is a best publicly available way.

This has already been impremented.


Reconfigure getliveevents for frontend

Coverage increased (+0.8%) to 64.564% when pulling 99641f47256acde0a32f6c5c98bb26cf9a7b7bd3 on getliveevents-for-frontend into 66c9871cf8dfc132d5f5a4bb9b1144e573bc030c on development.


Re-enable (and fix) test_toggled_outlet_does_not_trigger_fencing_update in ssi test suite

test_toggled_outlet_does_not_trigger_fencing_update (tests.integration.shared_storage_configuration.test_power_control.TestHostFencingConfig) disabled/skipped in ssi test suite in order to get #244 landed. Investigate this failure and find a fix so it can be re-enabled.

Is there a ticket opened on the actual failure that was being seen with this test?

http://jenkins.lotus.hpdd.lab.intel.com/job/integration-tests-shared-storage-configuration/1877/arch=x86_64,distro=el7/testReport/tests.integration.shared_storage_configuration.test_power_control/TestHostFencingConfig/test_toggled_outlet_does_not_trigger_fencing_update/

Covered in #315

Reopening to track the fixing of #315 and reverting of #341.

#341 has been reverted. At this point this issue is a dup of #315 I think.


Visualisation issue

First, my system is based on the current anaconda stable version.
I had two issues when calling the visualisation module:
svmsmote = SVM_SMOTE(random_state=1, svm_args={'class_weight' : 'auto'})

I am not really sure, but **svm_args should be passed as kwargs and not svm_args:
svm_args={'class_weight' : 'auto'}
svmsmote = SVM_SMOTE(random_state=1, **svm_args)

The second issue should be just a matter of the matplotlib version.
I went and changed a few things in the code a couple of weeks ago, but never updated the visualization.py. I also noticed that the line you mentioned had to be commented out for the current version of matplotlib.
Thanks for pointing it out, I'll make the appropriate changes now.


Copyright not properly adjusting to page size

Copyright content is overlapping table with fileshare content. If browser window is resized, copyright floats upward

@mickdigrazia Thanks for reporting. I am looking into it.

@mickdigrazia I believe this issue has been addressed pending review of pull request #54

@mickdigrazia I believe this issue has been addressed pending review of pull request #54

Great! Thanks @SvenAelterman


parse more of numpy

Codecov Report

Merging #16 (34b342d) into master (2baa24b) will decrease coverage by 0.23%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
- Coverage   69.92%   69.68%   -0.24%
==========================================
  Files          14       14
  Lines        3308     3332      +24
==========================================
+ Hits         2313     2322       +9
- Misses        995     1010      +15

Impacted Files
Coverage Δ

papyri/gen.py
76.93% <0.00%> (ø)

papyri/ts.py
71.29% <11.76%> (-3.22%)
:arrow_down:

papyri/miniserde.py
87.20% <20.00%> (-3.04%)
:arrow_down:

papyri/take2.py
71.20% <80.00%> (+0.03%)
:arrow_up:

Continue to review full report at Codecov.

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


ensure workflow scripts point at the base siliconcompiler directory

Closes #1338
We should trigger the surelog build before merging, probably next week

BTW, the macOS half of this change has effectively been tested by https://github.com/siliconcompiler/siliconcompiler/actions/runs/4309851810, which has the caching logic removed. The Linux build doesn't actually cache Surelog (for reasons...), so I'm running a workflow to test that the Linux build still works: https://github.com/siliconcompiler/siliconcompiler/actions/runs/4315760176.
I think if that passes we're pretty safe to merge this, although I'll leave that up to you.

We'll merge this and the other PR assuming it passes the wheels build today.


Make possible functions constant

Make many simple functions constant. Functions that are harder to be constant haven't been included in this work.
Some lints had to be ignored due to false-positives in Clippy relating to Drop implementations.

Rebased

Rebased

Rebased and made two new functions const.


Add search feature

It would be great if users could search for channels they want to save to favorites via some sort of autocomplete search bar

Maybe add a search field in the vicinity of the add favorites input that, when it has text, renders search results like Favorite Streamers cards but with a Follow button, or Following if you're already following them that turns red with Unfollow text on hover

https://dev.twitch.tv/docs/v5/reference/search/#search-channels


Buyable interface

Great bit of work with LaravelShoppingcart! I'm interested in implementing the Buyable interface on my product Model. Can you please provide some pointers on how to do this?
Thanks

class Product implements Buyable

See above

Err. Thanks, but anyone asking this question is unlikely to know how to deal with half an answer.
I've added class Product implements Buyable to my model, and now I get an error saying:
Class App\Product contains 3 abstract methods and must therefore be declared abstract or implement the remaining methods (Gloudemans\Shoppingcart\Contracts\Buyable::getBuyableIdentifier, Gloudemans\Shoppingcart\Contracts\Buyable::getBuyableDescription, Gloudemans\Shoppingcart\Contracts\Buyable::getBuyablePrice)
I suspect it's something simple, but I'm pretty new to Laravel and haven't come across anything like this before.

@matbeard Sorry if it came across as a harsh answer. Just thought that would be enough information. In the end, if you're using Laravel and doing OO programming, it seems to me it's pretty trivial how to "implement an interface".
But again, didn't mean to sound harsh, and glad you figured it out! Good luck with your website!

To add on to what @matbeard mentioned, It now throws an error saying:
Declaration of App\Book::getBuyableDescription() must be compatible with Gloudemans\Shoppingcart\Contracts\Buyable::getBuyableDescription($options = NULL)

Also the full declaration in your Model is :
`
use Gloudemans\Shoppingcart\Contracts\Buyable;
class Food extends Model implements Buyable
{
`

shouldn't adding those 3 functions be in the doc?


Spaces as far as the eye can see

This introduces a Pipfile for the root of the repository, which is where common linters and testing dependencies can be installed that are completely independent of the actual applications. It includes flynt and black as well.
flynt converts older style string formatting into f-strings wherever possible. f-strings are awesome and we like them.
black is an uncompromising code formatter. It's serious about being uncompromising.
To setup, use pipenv to install them. I place the venv in the project in .venv using the env variable below. You can forego this if you prefer to use a centralized location for pipenv virtualenvs.
$ pwd
/home/dade/natlas
$ PIPENV_VENV_IN_PROJECT=1 pipenv install

Then you can go into a pipenv shell and run the things that have been installed:
$ pipenv shell
$ black .
$ flynt -e node_modules .
$ exit

This PR also batches in a couple markdown linting fixes for issue templates and hopefully fixes the pull request template to actually work...

Github released the super-linter which does a bunch of cool stuff.


AFNetworking won't build in a WatchKit Extension

AFNetworking references a couple unavailable methods when included in a WatchKit Extension.
AFNetworkActivityIndicatorManager.m references [UIApplication sharedApplication].
UIAlertView+AFNetworking.m references UIAlertView's initWithTitle:message:delegate:cancelButtonTitle:otherButtonTitles:
These should probably be wrapped in #ifndef AF_APP_EXTENSIONS.

Fixed by #2589.

Thanks, Mattt.


ngf-change event not fires in IE8

For ngf-select directive ngf-change event not fires.
Tested in virtual windows machine for IE8 on the demo page for current version ( 7.0.4).
It works fine for IE9.

Fixed at version 7.0.8

:+1:


Add a --version flag to fossil

It would be nice to know the version of a given standalone fossil binary.
I'm thinking we can use https://pkg.go.dev/embed to build this into the CI build process.

There is also a build flag method that lets you set the value of a variable
main.go
var version string

I didn't look up the actual flag, but it is something like the following
go build -ldflag "main.version=v1.0.0"


fixed some typos

Just a couple typos, no biggy

Awesome. Thank you!

Thanks for this package, it saved me a lot of time!
On Sun, Apr 10, 2016 at 7:57 PM, Joshua Poehls<EMAIL_ADDRESS>wrote:

Awesome. Thank you!
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/jpoehls/gophermail/pull/28#issuecomment-208096288


switch-to-configuration: handle /proc and /sys specially

With this fsoptions implementation should work fine.

Wait a bit, I discovered that this won't work for the first activation. I'll submit updated patch later today.

Done!

Oh, sorry, I had disabled watching on my own nixpkgs repo and so didn't see this before now!

The vm test fails for me, does it work for you?

Ouch, this is because we override fileSystems in VMs: https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/virtualisation/qemu-vm.nix#L420
This is unfortunate. Of course we can just copy needed fileSystems entry to the test but then this won't be the module test -- it would rather test this mount option. I'm unsure what to do...

I would rather go ahead with my original implementation, sure it's a bit ugly, but it will work. This change can still go onto the enable systemd-remount-fs in containers PR or does that whole thing no longer matter, given that the main use-case for enabling it seems to have evaporated?

Hm, I don't see much value in it without this (and some added complexity). Let's go ahead with yours, then -- at least it works indeed!

(BTW, please cherry-pick my documentation fix)

Oh, I see, you've already fixed that in https://github.com/NixOS/nixpkgs/pull/14372/commits/732ee7d6f16d9aa7551e6dc54d78eeae01fa655e -- good!


Input image masks on Image2Image does not seem to do anything

I get the exact same results when using the draw mask on image-to-image. Is this a bug, or is it not supported? And if it's not supported, maybe the draw feature should be disabled for these?
I hope it is supposed to work, because it would be really nice to be able to ignore some areas from img2img, since some models don't support inpainting.

we can't just get inpainting for free, unfortunately. It will be less confusing after we overhaul the SD UI.


Correct log rotation and upload

These are the available log files:
elife@prod--annotations:~$ ls -lh /srv/annotations/var/logs/
total 838M
-rw-rwS--- 1 www-data www-data 831M Jul 30 11:45 all.json
-rw-rwS--- 1 www-data www-data 6.6M Jul 30 11:34 error.json

all contains all entries, while error is a copy of the ones with ERROR level or higher. We upload the first to Loggly, and rotate them both every day.

Quiet time so let's go with this.


support nginx-1.15.2

added new patch(support REWRITE phase) for nginx-1.15.2
note that wont support patch that disables REWRITE phase

Test Case

$ MOD_DYCONF_PATH=/home/xiaochen.wxc/work/github/nginx-1.15.2 TEST_NGINX_BINARY=/home/xiaochen.wxc/work/github/nginx-1.15.2/objs/nginx prove -v -I /home/xiaochen.wxc/work/github/nginx-tests/lib /home/xiaochen.wxc/work/github/ngx_http_proxy_connect_module/t
/home/xiaochen.wxc/work/github/ngx_http_proxy_connect_module/t/http_proxy_connect.t ..

test_enable_rewrite_phase: 1
DNS server: try to bind server port: 18085
DNS server: daemon pid: 12750
DNS server: working
ok 1 - 200 Connection Established
ok 2 - 200 Connection Established server name
ok 3 - 200 Connection Established server name
ok 4 - 200 Connection Established server name
ok 5 - 200 Connection Established not allowed port
ok 6 - Get method: proxy_pass
ok 7 - Get method: return 200
ok 8 - forbid CONNECT request without proxy_connect command enabled
ok 9 - set remote address
ok 10 - set local address and remote address
ok 11 - $connect_host, $connect_port, $connect_addr
ok 12 - dns resolver fail
ok 13 - skip proxy connect module without rewrite phase enabled
ok 14 - skip proxy connect module without rewrite phase enabled: if/return
DNS server: stop
1..14
ok 15 - no alerts
ok 16 - no sanitizer errors
All 14 subtests passed


Please mention that Lookup may not return all columns like it leads you to believe

https://powerusers.microsoft.com/t5/Building-Power-Apps/Does-Lookup-function-return-all-fields/td-p/1216164

Thank you for reaching out. Is the feedback specific to this topic? https://docs.microsoft.com/en-us/power-apps/maker/canvas-apps/functions/function-filter-lookup