WARNING: ASCII isn't correctly supported

I use windows explorer to open ftp://<IP_ADDRESS>:2121/

Can we have an explanation why the "ASCII" is not properly supported?
Are there any corrections planned for this?

Well, it doesn't really respect the standard:
https://www.ietf.org/rfc/rfc959.txt
         <IP_ADDRESS>.  ASCII TYPE

            This is the default type and must be accepted by all FTP
            implementations.  It is intended primarily for the transfer
            of text files, except when both hosts would find the EBCDIC
            type more convenient.

            The sender converts the data from an internal character
            representation to the standard 8-bit NVT-ASCII
            representation (see the Telnet specification).  The receiver
            will convert the data from the standard form to his own
            internal form.

            In accordance with the NVT standard, the <CRLF> sequence
            should be used where necessary to denote the end of a line
            of text.  (See the discussion of file structure at the end
            of the Section on Data Representation and Storage.)

            Using the standard NVT-ASCII representation means that data
            must be interpreted as 8-bit bytes.

            The Format parameter for ASCII and EBCDIC types is discussed
            below.

But this standard makes very little sense for any current use of a FTP server. If someone has a use-case where the lack of proper ASCII support is an issue, I'll fix it.

My use case:
ftp> open ftp.example.com
Connected to ftp.example.com.
220 example FTP
200 I'm in UTF8 only anyway
User (ftp.example.com:(none)): USER
331 OK
Password:
230 Password ok, continue
ftp>
ftp> Cd /Your/orders
250 CD worked on /Your/orders
ftp> lcd \\test.com\zf\c\alb\Here(W)\WebOrdre
Local directory now \\test.com\zf\c\alb\Here(W)\WebOrdre.
ftp>
ftp> mget *.txt
Could not list: failure
200 ASCII isn't properly supported: https://github.com/fclairamb/ftpserverlib/issues/86
Cannot find list of remote files.

Stopped working 2-3 days ago :-/

Hi,
I added support for ASCII mode, so you should not see this warning anymore if you update to the latest version.
However I think this isn't the issue here, mget should send a NLST command and we don't support wildcards while listing directories. I think this should work:
ftp> Cd /Your
250 CD worked on /Your
ftp> lcd \\test.com\zf\c\alb\Here(W)\WebOrdre
Local directory now \\test.com\zf\c\alb\Here(W)\WebOrdre.
ftp>
ftp> mget orders

are you saying wildcards worked before?

I have used wildcard for years.
With no issue,
issues started between the 21/01-2021 and today as I can confirm, on windows 10
Yes it works if I call file with full name.
But provider are doing random strings on the orders, so not sure how I should fix without wildcard without rebuilding. and not use this anymore.

May be my provider that changed FTP server.. not sure here.

ah ok, so you are not using this library yourself. Then you have to ask to your provider. The library simply passes "* .txt" to the implementer, so the wildcard logic could be handled downstream.

No Sorry just relished, that this was a server lib :facepalm:
Thanks for your input  you'll have a great weekend :-)


fish shell completion

Would you consider adding support for fish shell completion?
It is much less painful than writing files for other shells, but I couldn't figure out how to port the current completion scripts.
Here is a link to the fish docs: http://fishshell.com/docs/current/commands.html#complete

I'd take a patch for this, but I won't be adding this myself.

I'd like to understand how fish completion works I'm going to read the docs and take a stab at this

I guess this has to be submitted upstream instead: https://github.com/fish-shell/fish-shell/issues?q=is%3Aopen+is%3Aissue+label%3Acompletions


Feature: Resolve rebase conflicts

[ ] Unittests
[ ] All TODOs removed
[ ] Toolbar buttons for aborting and continuing the rebase

@exactly-one-kas I implemented it rawly now. What do you think about the toolbuttons or do you see a better location?

I think that aborting and continuing rebases should be handled like merges; otherwise the merge operations would also habe to be placed on the toolbar for consistency

I think that aborting and continuing rebases should be handled like merges; otherwise the merge operations would also habe to be placed on the toolbar for consistency

The second image is good. But maybe we can then also move the merge out. I think it is quite hidden. I did not know ^^

I moved the rebase buttons.
I added also the merge abort button there or should I let it in the menubar only, or should I delete it from the menubar then?

@exactly-one-kas do you understand why the unittest fails with that exact repository? For some reason I have a nullptr exception

Does the unit test only fail with that exact repository? Considering the stack strace it seems to me that it should fail with any repository.
I'd guess that this happens because the unit tests are run with an offscreen / dummy QT backend that doesn't create actual windows and therefore has no window handles.
Try the following:

Rename w_handler to w_handle
Use int size = kSize * (w_handle ? w_handle->devicePixelRation() : 1);

I moved the rebase buttons. I added also the merge abort button there or should I let it in the menubar only, or should I delete it from the menubar then?

I'd say that all buttons regarding rebasing and merging should be with the commit button (after all, the "commit merge" button is the merge button), so that new users have an easier time finding these options. Personally, I think that this location is the more logical one than the toolbar or menu bar
However, "abort merge" and "abort rebase" should also be present in the "branch" menu as that's where users currently expect them to be. We should keep them there for a while and phase them out for v2 / the next major redesign

Yes exactly it fails only with this repository, If I replace the zip filename by another in that location it runs through.

I moved the rebase buttons. I added also the merge abort button there or should I let it in the menubar only, or should I delete it from the menubar then?

I'd say that all buttons regarding rebasing and merging should be with the commit button (after all, the "commit merge" button is the merge button), so that new users have an easier time finding these options. Personally, I think that this location is the more logical one than the toolbar or menu bar
However, "abort merge" and "abort rebase" should also be present in the "branch" menu as that's where users currently expect them to be. We should keep them there for a while and phase them out for v2 / the next major redesign

Good idea, I will mark it so we don't forgett to remove it at some point

This is how the stacktraces look like

What I have seen is that for rebaseConflicts Repository, the number of commits is not zero as for the other

Watching .git/HEAD would be an option

hmm yes. So we need another application always looking there. I think this goes beyond this scope.

hmm yes. So we need another application always looking there. I think this goes beyond this scope.

Why would this need to be in another application? Change detection happens in the same process from another thread (see src/watcher/RepositoryWatcher.cpp)

hmm yes. So we need another application always looking there. I think this goes beyond this scope.

Why would this need to be in another application? Change detection happens in the same process from another thread (see src/watcher/RepositoryWatcher.cpp)

You are right sorry. This will work so we will react on .git/HEAD changes and then we are doing a refresh? Would you like to implement ^^ I am completely full next weeks. Maybe I can go on a little bit with the libgit2 update ....

@exactly-one-kas can we merge this? I think implementing watching .git/HEAD can be done separately because it affects all commands executed in command line not only rebase

Yep, works for me


Footer offer pages

@sunny0625
[Feature] : Footers Link are dummy we can add pages for that #855
close #855
description
i created and added footer offer pages section
also added links in every pages

@sunny0625
kindly please check this and merge it
also add level

@sunny0625
kindly please check this and merge it
also add level

@sunny0625
please check this and merge it
so i can make another pr


Conflict {maybe?} with Mount Journal Enhanced

Reported an issue to MJE's author, and did the testing he asked me to help him track down the problem. Even though the error report is showing MJE as the responsible party, it's having DressUp enabled that actually makes the error come up. See pasted response below...
MJE ticket:
With just MJE and DressUp installed, got this error. In all combinations I tested, it was loading/unloading DressUp that changed whether an error was caught by BugSack. However, the error message has not fundamentally changed. I know that loading order can do wonky things to what is being reported as the problem, so it could be that your addon is unfairly getting the blame. Will cross-post to it's page.
153x ...aceMountJournalEnhanced\UI\DressUpButton.lua:27: Usage: hooksecurefunc([table,] "function", hookfunc)
[C]: in function hooksecurefunc' ...aceMountJournalEnhanced\UI\DressUpButton.lua:27: in function <...aceMountJournalEnhanced\UI\DressUpButton.lua:3> ...aceMountJournalEnhanced\UI\DressUpButton.lua:41: in function ApplyPreviewButton'
...ddOns\MountJournalEnhanced\Settings\OptionsPanel.lua:111: in function <...ddOns\MountJournalEnhanced\Settings\OptionsPanel.lua:77>
[C]: ?
Masque\Libs\AceGUI-3.0\AceGUI-3.0-40.lua:72: in function <Masque\Libs\AceGUI-3.0\AceGUI-3.0.lua:70>
Masque\Libs\AceGUI-3.0\AceGUI-3.0-40.lua:290: in function Fire' ...GUI-3.0\widgets\AceGUIContainer-BlizOptionsGroup.lua:32: in function <...GUI-3.0\widgets\AceGUIContainer-BlizOptionsGroup.lua:31> [C]: in function pcall'
FrameXML\InterfaceOptionsFrame.lua:206: in function <FrameXML\InterfaceOptionsFrame.lua:205>
[C]: in function `securecall'
FrameXML\InterfaceOptionsFrame.lua:229: in function <FrameXML\InterfaceOptionsFrame.lua:221>

Hey there. I'm going to fix this in MJE.
It's basically the same issue as in #16. I'm also trying to access DressUpFrame.ModelScene. That key was added with 8.2.5 into the Blizzard DressUp frames. It's really not nice to replace some frames entirelly ;)

Yes it's a little problematic to override so much but I didn't have the patience or the time to start learning how the model scene thing works or if it can even be used to implement all the features this addon has.

Maybe you should find the time then. Or you could find another solution which doesn't rely on overwriting DressUpFrame. Because there is a similar error, when you just preview a mount from a vendor or link. ;)

I'm not actively playing or even subscribed at the moment, and that's not even counting the fact addon developers aren't even paid for the work so I don't think I should really do anything.
Sorry it's causing problems but I at this moment don't care enough to fix anything in my addon.

This also happens when you try to sheathe/unsheathe weapons in the dressing room, when you preview a mount, and basically all things that are referring to ModelScene.


KeepAlive with GoPro Fusion

KeepAlive does not seem to be working for the GoPro Fusion.
I have successfully connected to the Fusion using goprocam - I can capture, query media, etc.
Can anyone confirm if they've managed to keep their Fusion from shutting down after ~7 minutes using the KeepAlive method?

I don't have a fusion anymore but when I had it KeepAlive worked fine...

Thanks for the response, Konrad. I've just run Wireshark and see that the Fusion responds with "Destination/port unreachable" when sending the keep-alive message on port 8554. Maybe they've since changed the port for this feature.
I will poke around to see if I can find the right port for this. If all else fails, I will just trigger capture periodically as that also keeps it alive.

Is there a suggested method to verify that the KeepAlive port is still 8554? The results of 'nmap -sU -p 8554 <IP_ADDRESS>' show that the port is closed. Maybe that's not indicative of much, considering UDP scan results on other ports give inconsistent results, but given that it takes ~7 minutes to verify whether the Fusion responds to my modified version of the KeepAlive function, a brute force approach is impractical and I'd really appreciate any guidance!


Bootstrap material design ripples fires multiple change events

It seems that using bootstrap material design ripples.js triggers multiple clicks on ngFileSelect directive

I have done some changes to the way the click is being handled, This may or may not be fixed in version 3.0.0.


syz-fuzzer: fold fuzzer into manager process on host?

Currently we run syz-fuzzer process inside of the target VM. The original motivation for this was performance and scalability: generation/mutation of programs happens locally and is distributed (manager is not a bottleneck).
This issue is to consider changing the current scheme to the following:

fuzzer is folded into manager and runs on the host
the only binary that runs on target is executor

This has a number of advantages:

We won't require target Go toolchain (Akaros and Fuchsia don't have them and use HostFuzzer mode, which is a hacky approximation of this proposal)
This will reduce resource consumption in the VM, fuzzer is a beefy Go process that holds whole corpus. Consequently currently we have N copies of the corpus, in the new scheme we will have just one copy on host.
Presumably it will simplify lots of things: multiple fuzzers won't need to exchange inputs, etc. Manager will be in total control of the corpus.
This may increase fuzzing efficiency because manager will be able to do more optimal global decisions.

We may also consider how/if we can support VMM/user-space library testing in this mode. For these things there is even no "target machine", so it's reasonable to have corpus on host in 1 copy.
The overall idea is that executor will need a bit of fuzzer functionality: mandate fork server, probably do first step of coverage processing for performance (otherwise we would always need to send all coverage back to host for each program). Manager will send serialized programs to executors, receive results back and do the rest of processing.
The main question is performance of this scheme. It's just different, hard to say if it will be worse, or maybe significantly better. Currently we send each program before execution to manager in text form (in the log); in the new scheme we will need to send each program in binary form to the machine (back we don't need to log it back b/c manager will already know what programs VM is running). We could use some buffering: send up to N programs to executor and then wait for results. Program triage may get slower because we will need to do lots of round-trips and send coverage back most of the time.
We will also need to re-implement all of pkg/host in executor in C. But as a good side-effect it will fix pkg/host for akaros/fuchsia and will also allow to properly account for sandboxing restrictions. E.g. "is a file accessible under a particular sandbox?", currently we can also guess or approximate. If we have this in executor, we just do the checks under the actual sandbox.
Another potential application is UEFI fuzzing (#3132).
Another potential use case that can be incorporated is support for deterministic VMs/emulators that fork for each test case.

This is an interesting problem. This could likely eliminate the need of having an RPC server and the fuzzer could be a thread on the program. The bulkiness of VM images would go down, and it might have a positive effect on vm start up time.

Hi.
Do you know how difficult would be to do this in the future? What would be the advantages and disadvantages of having this approach?

I would assume it will be quite difficult to this since, since it a major architecture redesign.
The advantages and disadvantages listed in the issue descirption.

What would be the estimated effort to decouple sys-fuzzer from syz-executor?

Hard to say w/o doing it :)
Will also depend a lot on who will do it (syzkaller/Go knowledge) and also on exact scope (remove all Go code from target, only move corpus/fuzzing to host, keep/remove pkg/host, etc).

For the record:
After https://github.com/google/syzkaller/pull/4579, the transition of the fuzzing engine to the host has been completed. Almost all of the aforementioned use cases should be unblocked now.
The only remaining limitation is that we still need to be able to run some Go code inside the fuzzed VM. For the fuzzing of non-typical syzkaller targets one now needs to write the code similar to syz-manager/rpc.go that would invoke fuzzer.NextInput() to get new inputs and report the results to fuzzer.Done() -- it's already a much much smaller amount of necesary changes.

This is 95% done, what's left is:

the only binary that runs on target is executor (still have syz-execprog)
We won't require target Go toolchain (Akaros and Fuchsia don't have them and use HostFuzzer mode, which is a hacky approximation of this proposal)


Handle dynamic vuex stores

When combined with a dynamic store registration as described by the vuex docs, the decorators fail as they try use the store before it exists.

In your component write:
public created() {
if (!this.$store.state.yourModuleName) this.$store.registerModule('[yourModuleName', yourModuleName)
}

Yes that's to register. But when I create a decorator binding to something in that store I get an error that the module name space is undefined.

@Getter('yourModuleName/yourGetterName') VariableName

I have registered dynamic every order in the store and using their id as a namespace.
I am doing something like:
const orderStore = namespace("orderStore");
@Component
export default class COrder extends Vue {
@Prop({ type: Number, default: 0 }) private orderId!: number;
@orderStore.State(function(this: COrder , s) { return s[this.orderId]; }) private order!: IOrder;
}
is there any better way to do it?

The binding to the Getters is not done properly when registering vuex stores dynamically with registerModule. The Getters throw an error: [vuex] module namespace not found in mapGetters(): person-lookup/.
Registering the vuex store in the beforeCreate lifecycle hook works just fine:
  beforeCreate () {
    if (!this.$store.state['person-lookup']) {
      this.$store.registerModule('person-lookup', PersonLookupModule)
    }
  }

The problem isn't that the store isn't registered, it just doesn't register properly. I'm still not sure exactly what is happening but my Vue DevTools plugin shows the getters are mapped to the root state but the state is properly namespaced.

@oppianmatt the work around is to register your module sometime before the mount lifecycle (i.e. created, beforeCreate) and try to use the getters using @Getter instead of @myNamespace.Getter. That mostly works... There are a few caveats.

@ktsn I'm not seeing much activity on this repo since last year. Are you planning to maintain this code base?

You can call getModule or registerModule to register vuex module dynamically when the route is loaded
use 'vuex-module-decorators' dynamic: true
@Module({ name: 'user', namespaced: true, store, dynamic: true }) export class User extends VuexModule implements IUserState {   public userProfile = {}   public roles: string[] = []   public loading: boolean = false }
router config:
{     path: '/login',     component: async () => {       // 动态加载vuex模块和路由对应的组件       getModule((await import('../store/modules/user')).User)       return import(/* webpackChunkName: "login" */ '../views/login/index.vue')     }   }


Make ILB count configurable and increase default to 2.

What type of PR is this?

What this PR does / why we need it:
This PR makes the number of ILB services in the scale test to be configurable.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

Eventually all tests will start using the template being added in https://github.com/kubernetes/perf-tests/pull/1785
Until we maintain the current config file, it will be useful to configure the ILBs being deployed.
cc @freehan
/assign @tosi3k @jkaniuk

/lgtm
/assign @jkaniuk

/approve


permute error ??

https://github.com/jwyang/faster-rcnn.pytorch/blob/0797f6290e104e7d63cd487af759840d4a36985b/lib/model/rpn/proposal_layer.py#L98
I am not sure but, I think this permutation is not correct.
Initially, we set the shifts as a link below,
https://github.com/jwyang/faster-rcnn.pytorch/blob/0797f6290e104e7d63cd487af759840d4a36985b/lib/model/rpn/proposal_layer.py#L83
The permutation should be modified as
bbox_deltas = bbox_deltas.permute(0, 3, 2, 1).contiguous() such that finally, bbox_deltas also follows the order [x, y] (not [y, x]) which is the same order in anchors([x_min, y_min, x_max, y_max)
I currently modify the code to operate 3-dims region proposal networks for my research, I suddenly encounter this problem.
I am not sure but hope to get any feedback.

For my RPN-3D that is modified from the code here, I achieve the correct results after I modify the permute orders. (Specifically, I set (x_min, y_min, z_min, x_max, y_max, z_max) and permute the 4-dim features.permute(0, 4, 3, 2, 1). If I follow the origin code, I should permute the 4-dim features.permute(0, 2,3,4,1).)


Added @Override annotation in sample code

+1 looks great to me.

@charleswhchan I merged this into master thanks for the update.


ValueError in datamodel.Data

Describe the bug
Hey, great job for this package. Is it still being maintained ? I'd like to know before using it in prod.
Also, I believe there is a bug in datamodel.Data. If you pass a dataset_2 that does not contain id_column_name_1 (which seems like a valid case) you will get a ValueError.
To Reproduce
Steps to reproduce the behavior:
import pandas as pd
from pyjedai.datamodel import Data

df1 = pd.DataFrame({'id': [1, 2], 'data': ['a', 'b']})
df2 = pd.DataFrame({'other_id': [1, 2], 'data': ['a', 'b']})
Data(
    dataset_1=df1,
    id_column_name_1="id",
    dataset_2=df2,
    id_column_name_2="other_id",
)

--> ValueError
Expected behavior
No ValueError
Additional context
I believe here, it should be self.attributes_2.remove(self.id_column_name_2) instead of self.attributes_2.remove(self.id_column_name_1)

Hello! Yes it is maintained and will be expanded even more. As for the bug, you're right. Thanks for your trace and detailed report. I fixed it and will be updated asap in the next release.
Feel free to contact me if you need anything else.
Cheers,
Konstantinos


Can't update cfg file org.apache.karaf.command.acl.bundle.cfg

Hi there,
I saw the following log when starting the container. Should this (Can't update cfg file) be fixed?
[Cause] The org.apache.karaf.command.acl.bundle.cfg file belongs to the root user, the java process runs as nexus user and doesn't have permission to modify it.
======= LOG ======
2016-09-26 08:22:42,575+0000 WARN  [pool-15-thread-1]  org.apache.karaf.features.internal.service.FeaturesServiceImpl - Can't update cfg file
java.io.FileNotFoundException: /opt/sonatype/nexus/etc/org.apache.karaf.command.acl.bundle.cfg (Permission denied)
at java.io.FileOutputStream.open0(Native Method) [na:1.8.0_102]
at java.io.FileOutputStream.open(FileOutputStream.java:270) [na:1.8.0_102]
at java.io.FileOutputStream.(FileOutputStream.java:213) [na:1.8.0_102]
at java.io.FileOutputStream.(FileOutputStream.java:162) [na:1.8.0_102]
at org.apache.felix.utils.properties.Properties.save(Properties.java:152) [org.apache.karaf.features.core:4.0.4]
at org.apache.felix.utils.properties.Properties.save(Properties.java:148) [org.apache.karaf.features.core:4.0.4]
at org.apache.karaf.features.internal.service.FeatureConfigInstaller.updateStorage(FeatureConfigInstaller.java:292) [org.apache.karaf.features.core:4.0.4]
at org.apache.karaf.features.internal.service.FeatureConfigInstaller.installFeatureConfigs(FeatureConfigInstaller.java:107) [org.apache.karaf.features.core:4.0.4]
at org.apache.karaf.features.internal.service.FeaturesServiceImpl.installFeature(FeaturesServiceImpl.java:1166) [org.apache.karaf.features.core:4.0.4]
at org.apache.karaf.features.internal.service.Deployer.deploy(Deployer.java:778) [org.apache.karaf.features.core:4.0.4]
at org.apache.karaf.features.internal.service.FeaturesServiceImpl.doProvision(FeaturesServiceImpl.java:1089) [org.apache.karaf.features.core:4.0.4]
at org.apache.karaf.features.internal.service.FeaturesServiceImpl$1.call(FeaturesServiceImpl.java:985) [org.apache.karaf.features.core:4.0.4]
at java.util.concurrent.FutureTask.run(FutureTask.java:266) [na:1.8.0_102]
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142) [na:1.8.0_102]
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617) [na:1.8.0_102]
at java.lang.Thread.run(Thread.java:745) [na:1.8.0_102]

I am pretty sure this is https://issues.sonatype.org/browse/NEXUS-10049 which is slated to be fixed already.  Feel free to watch/vote for that issue there.  If I am misunderstanding and this is distinct, please feel free to comment here.


WI optimisation

Modularise/granularise the component being encrypted and backed up such that they can be updated instantaneously and asynchronously post user journeys (ex: TC initiations and approval, New accounts addition, etc.)
The WI updation should be done from a screen that is instrumental to every Hexa session(ex: home screen) while the non-critical components can be updated at the end of their respective flow(such as account states in terms of bal/tx/utxo/indexes).
Should be done once AC management is in place to avoid database and restoration related re-work.

Would require database changes(migration to Realm DB) in order to implement WI sync granularisation.

Would require database changes(migration to Realm DB) in order to implement WI sync granularisation.

@Parsh is this still relevant or can the issue be closed?

Completed


Python.h not found on WSL Ubuntu 20.04 installation

Hey, I was trying to install the package but I get an error saying that it couldn't find Python.h while installing. I'm using Ubuntu 20 on WSL and I've installed gcc python-dev libgmp3-dev before both installing the package using pip3 & installing directly from the source.
Also my Python version is 3.8, if it's useful
Here's the full log
> sudo python3 setup.py install
running install
running bdist_egg
running egg_info
creating fastecdsa.egg-info
writing fastecdsa.egg-info/PKG-INFO
writing dependency_links to fastecdsa.egg-info/dependency_links.txt
writing top-level names to fastecdsa.egg-info/top_level.txt
writing manifest file 'fastecdsa.egg-info/SOURCES.txt'
reading manifest file 'fastecdsa.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
writing manifest file 'fastecdsa.egg-info/SOURCES.txt'
installing library code to build/bdist.linux-x86_64/egg
running install_lib
running build_py
creating build
creating build/lib.linux-x86_64-3.8
creating build/lib.linux-x86_64-3.8/fastecdsa
copying fastecdsa/benchmark.py -> build/lib.linux-x86_64-3.8/fastecdsa
copying fastecdsa/ecdsa.py -> build/lib.linux-x86_64-3.8/fastecdsa
copying fastecdsa/point.py -> build/lib.linux-x86_64-3.8/fastecdsa
copying fastecdsa/curve.py -> build/lib.linux-x86_64-3.8/fastecdsa
copying fastecdsa/util.py -> build/lib.linux-x86_64-3.8/fastecdsa
copying fastecdsa/keys.py -> build/lib.linux-x86_64-3.8/fastecdsa
copying fastecdsa/__init__.py -> build/lib.linux-x86_64-3.8/fastecdsa
creating build/lib.linux-x86_64-3.8/fastecdsa/tests
copying fastecdsa/tests/test_prehashed.py -> build/lib.linux-x86_64-3.8/fastecdsa/tests
copying fastecdsa/tests/test_key_export_import.py -> build/lib.linux-x86_64-3.8/fastecdsa/tests
copying fastecdsa/tests/test_rfc6979_ecdsa.py -> build/lib.linux-x86_64-3.8/fastecdsa/tests
copying fastecdsa/tests/test_prime_field_curve_math.py -> build/lib.linux-x86_64-3.8/fastecdsa/tests
copying fastecdsa/tests/test_point.py -> build/lib.linux-x86_64-3.8/fastecdsa/tests
copying fastecdsa/tests/test_whycheproof_vectors.py -> build/lib.linux-x86_64-3.8/fastecdsa/tests
copying fastecdsa/tests/test_p256_ecdsa.py -> build/lib.linux-x86_64-3.8/fastecdsa/tests
copying fastecdsa/tests/test_key_recovery.py -> build/lib.linux-x86_64-3.8/fastecdsa/tests
copying fastecdsa/tests/test_keygen.py -> build/lib.linux-x86_64-3.8/fastecdsa/tests
copying fastecdsa/tests/test_nonce_generation.py -> build/lib.linux-x86_64-3.8/fastecdsa/tests
copying fastecdsa/tests/test_brainpool_ecdh.py -> build/lib.linux-x86_64-3.8/fastecdsa/tests
copying fastecdsa/tests/__init__.py -> build/lib.linux-x86_64-3.8/fastecdsa/tests
copying fastecdsa/tests/test_whitespace_parsing.py -> build/lib.linux-x86_64-3.8/fastecdsa/tests
creating build/lib.linux-x86_64-3.8/fastecdsa/encoding
copying fastecdsa/encoding/sec1.py -> build/lib.linux-x86_64-3.8/fastecdsa/encoding
copying fastecdsa/encoding/asn1.py -> build/lib.linux-x86_64-3.8/fastecdsa/encoding
copying fastecdsa/encoding/util.py -> build/lib.linux-x86_64-3.8/fastecdsa/encoding
copying fastecdsa/encoding/der.py -> build/lib.linux-x86_64-3.8/fastecdsa/encoding
copying fastecdsa/encoding/pem.py -> build/lib.linux-x86_64-3.8/fastecdsa/encoding
copying fastecdsa/encoding/__init__.py -> build/lib.linux-x86_64-3.8/fastecdsa/encoding
creating build/lib.linux-x86_64-3.8/fastecdsa/tests/encoding
copying fastecdsa/tests/encoding/test_der.py -> build/lib.linux-x86_64-3.8/fastecdsa/tests/encoding
copying fastecdsa/tests/encoding/test_sec1.py -> build/lib.linux-x86_64-3.8/fastecdsa/tests/encoding
copying fastecdsa/tests/encoding/__init__.py -> build/lib.linux-x86_64-3.8/fastecdsa/tests/encoding
running build_ext
building 'fastecdsa.curvemath' extension
creating build/temp.linux-x86_64-3.8
creating build/temp.linux-x86_64-3.8/src
x86_64-linux-gnu-gcc -pthread -Wno-unused-result -Wsign-compare -DNDEBUG -g -fwrapv -O2 -Wall -g -fstack-protector-strong -Wformat -Werror=format-security -g -fwrapv -O2 -g -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2 -fPIC -Isrc/ -I/usr/include/python3.8 -c src/curveMath.c -o build/temp.linux-x86_64-3.8/src/curveMath.o -O2
In file included from src/curveMath.c:1:
src/curveMath.h:4:10: fatal error: Python.h: No such file or directory
    4 | #include <Python.h>
      |          ^~~~~~~~~~
compilation terminated.
error: command 'x86_64-linux-gnu-gcc' failed with exit status 1

Have a good day and thanks for your time

Turns out problem was not about this specific package but the fact that I haven't installed python3.8-dev dependency. Didn't know each version had its own dev package.


Downscaling using responsive prop results in new image downloads

I don't know if you guys use the official cloudinary javascript library for the resizing behavior, but cloudinary's media delivery docs for responsive images says that browser downscaling is used when the images already downloaded are larger.
Yet the network tab looks like this when the browser scales down:
(I couldn't make a screenshot of the 2 widths of the downloaded images, but the order and file sizes indicate the same ;))

Is the current behavior intended?
If yes: how can I tell <cld-image> only to download new images when we do not already have a larger one?

@katerlouis we are using the official SDK from Cloudinary, both Node SDK, JS SDK and Vue SDK.
There is a bug of cld-image when you use responsive image together with placeholder, it will download the first original version rather than the optimized one. A new version of the Vue SDK is developed but yet to be released, hence we couldn't update this module yet.
Can you provide me the code example (Like codepen?) so I can see exactly what the use case is.
Thank you.
Maya

@katerlouis with the latest version of cloudinary-vue sdk (0.1.2), this issue should be solved. Please install the latest module version (beta-4).


Async support

Just to give a little bit of context, Rust's async functions return Futures that need an executor to drive them to completion. The usual way to achieve this is to let the async runtime, e.g. tokio, control the "main" thread. This is not a viable option in our case since it would block the UI thread, i.e. the one Neovim runs in, defeating the whole point of concurrent code.
However it should be possible (emphasis on the should) to use libuv, which Neovim uses for the event loop, as the executor. We could expose something like a Task object that wraps an async {} block and drives it to completion by scheduling it via libuv.

can't you just run tokio on another thread ?

@momofor the lua state is thread local and gets initialized on the main thread when the plugin is required.

Looking through this it looks like there is an existing libuv wrapper for Rust called libuv-rs. If I am understanding this correctly, a Task would be something that would except an async function and we could schedule using something like a spwan to execute this in a new thread, is this what you have in mind?

we could schedule the function using something like a spawn to execute this in a new thread

Not in a new thread, in the main thread. That's what's tricky about it. If we could use other threads we could just use tokio, but those futures have to run on the same thread that Neovim runs in for the reason I described above.

Does this mean it is fine to run stuff asynchronously with Tokio as executor in another thread, as long as no interaction with Neovim is done in that thread? Or am I misunderstanding something?

@nik-x yes basically. Btw I'll probably close this issue as completed once I've polished a bit abf70cbf2f5df2e4450f7578daf9008ec2548bd0. You can check out this example to see how it works.

Nice! Thank you

closed by 7d2e85a92a9b9fbe66dc72ac5af0e1bed0fcf28b


add labels to PV

Hi,
This is just a question. Is there a possibility to add labels to a PV upon creation? I love the way the local provisioner works but I got this question asked.
Kind regards,
Eric V.

This is currently not a feature we have built in but it should only be a small change in the code base to make it work!

Hey Maartje,
Hartelijk bedankt voor je snelle reactie!
So what would you look at to use to set the labels? File system labels? We
use XFS but label on a filesystem works on any type.
Kind regards,
Eric Van Steenbergen
E-mail<EMAIL_ADDRESS><EMAIL_ADDRESS>Skype: ericvs2014
LinkedIn: Eric Van Steenbergen
http://es.linkedin.com/pub/eric-van-steenbergen/b/8a4/51b
On Mon, Aug 19, 2019 at 5:32 PM Maartje Eyskens<EMAIL_ADDRESS>wrote:

This is currently not a feature we have built in but it should only be a
small change in the code base to make it work!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/src-d/kubernetes-local-pv-provisioner/issues/9?email_source=notifications&email_token=AACCAP24X3FUFOPDLSW52HTQFJSBDA5CNFSM4IM2ZQHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4SJIEI#issuecomment-522490897,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACCAP6WJGK6ELTG5UM7IKLQFJSBDANCNFSM4IM2ZQHA
.


Reference to unknown VM fedora02 in 03_ocpv_basics.adoc

The Optional module "Controlling virtual machine state" states to use VM fedora02 which doesn't exist if you've been following the lab.

There's also a reference to fedora02 in the screen shot in the 4. Live migrate a virtual machine section of the page.

Also getting confused by this on my dry run.


Outdated recommendation for "logs"

❯ bacalhau job get j-a8654281-8234-4499-a128-243170c54d17
Fetching results of job 'j-a8654281-8234-4499-a128-243170c54d17'...
No results found
You can check the logged output of the job using the logs command.

  bacalhau logs j-a8654281-8234-4499-a128-243170c54d17

❯ bacalhau logs j-a8654281-8234-4499-a128-243170c54d17
Command "logs" is deprecated, Please use `bacalhau job logs` to follow job logs.
See the migration guide at https://docs.bacalhau.org/references/cli-reference/command-migration for more information.

Closed by https://github.com/bacalhau-project/bacalhau/pull/4253


Fix assertion message to show the missing file.

When the coverage file is missing the exception that is thrown in CoverageData::__construct() uses the output from realpath() to create the exception message. However if the file is missing realpath() returns FALSE  so the filename appears empty.

Thanks!


🐛 Bug Report: Cloudbuild Plugin limited to global scoped builds

📜 Description
As discussed in #19514 , cloudbuild only displays builds in the global scope.
A potential fix might be use the https://cloud.google.com/build/docs/api/reference/rest/v1/projects.locations.builds/list endpoint to allow support for regional scoped builds as well.
cc: @PurseChicken
👍 Expected behavior
Cloudbuild plugin should list builds in a regional scope.
👎 Actual Behavior with Screenshots
Request is completed, however, the response is empty due to the location of builds.

👟 Reproduction steps
Follow the plugin documentation.
📃 Provide the context for the Bug.

[x] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

@vedantthapa Thanks!
I'll work on a PR for this. We will likely need a new annotation to specify the region for the component.

@vedantthapa
Could you please let me know if the following method returns information about one of your builds outside of the global scope?
https://cloud.google.com/build/docs/api/reference/rest/v1/projects.builds/get?
Use your projectID and buildID for the parameters of "Try this Method".
Thanks!

@PurseChicken
It returns an error:
{
  "error": {
    "code": 404,
    "message": "Requested entity was not found.",
    "status": "NOT_FOUND"
  }
}

@PurseChicken
It returns an error:
{
  "error": {
    "code": 404,
    "message": "Requested entity was not found.",
    "status": "NOT_FOUND"
  }
}

@vedantthapa
Crud, I was afraid of that.
Can you use the following method instead?
https://cloud.google.com/build/docs/api/reference/rest/v1/projects.locations.builds/get?
In the name parameter use the following:
projects/,your-project./locations/us-central1/builds/<your-build-id>

@PurseChicken, already did and it returns the relevant build :)

@vedantthapa
Thanks so much for confirming. In order to support builds outside the global scope we will need to modify all the Cloud Build queries in the plugin. Its going to take me a bit longer to get that done. It will likely need to wait until after my existing PR is approved for Cloud Build.

Hi @PurseChicken 👋🏻 , could you link this issue with the existing PR?

@camilaibs Sure. Ill update the existing PR and link this issue. I'll do that early next week due to the US holiday.
Thanks!

Closed by #21397


[Spaces] Doc about setting custom sleep time 😴

Sorry about the few unrelated changes, hopefully it's ok

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

okok i'm about to merge this in 5-10 mins unless objection!


Feat/migrate to esm

Fixes #89
Migrates all packages from commonjs formats to module packages with import and export maps. This has the following consequences:

all source files have been renamed from.ts to .mts
import and export maps are added, while retaining compatibility with commonjs
to support compilation to commonjs, tsup is used to handle the conversion
due to some other benefits tests have been migrated from jest to vitest
deno conversion scripts needed to be adapted to support import maps
conversion scripts were added to generate bun compatible exports
all dependencies are updated to the latest versions

As promised, I wish to report that both my projects currently using Rimbu - that is, Hermes and OmniCourse-core - seem to be working fine when referencing the new branch, with the only exception of the missing List.sort() method.

Thanks for checking, that helps a lot! I thought I added the sort method, but perhaps I forgot. I will re-check.

🥳Looking forward to the release of this brand-new version! Keep up the 🌟excellent work! 🤗🦋


What does this mean? Dropdown is now created as a child of its parent?

What does "Dropdown is now created as a child of its parent" mean?
What should i change ?
Dropdown now works weird...

I second this.  Dropdowns now only show 2 items and no scrolling available.

Also, dropdown items are no longer wrapping.

+1. It would help to have clearer documentation on dropdowns. I have to override default css on z-index and overflow to make it work properly.

+1 same here. overriding z-index and overflow to make it work.

In the most recent release, Dropdowns are created as a sibling to its button


Add name properties

It cannot be used into standard form because of sending value don't have name property
Example:
"": "+7 999 999-999"

Can you please elaborate ?

@ilyasf I'm not sure we understand what your question is, can you please explain what is the issue?

Closing as the OP hasn't replied.


Hide the time

I don't want my players to know exactly what time it is, so only having the sky showing for them would more adequately translate what their characters see and know (there are basically no clocks in the world).

Hmmm, this is a good suggestion, I can see where it'd be useful to have the ambiguity. Also I think it probably makes more sense to have the "can players see the sun and/or time" control be scene-specific, to account for underground scenes etc. Gonna think this through, might rework things a bit.

Implemented in 1.7.0 :)


docs: make target for PDF and HTML

This adds a docs make target for producing a PDF and HTML.
The output of tree in ./serialization.md does not jive with pandoc.
Not sure what better visual layout of a directory tree we can use.
This also pins the version of code-of-conduct, instead of just master.
Fixes #16
Signed-off-by: Vincent Batts<EMAIL_ADDRESS>

Thanks for doing this @vbatts.  As a consumer of the HTML and PDF versions.  I'm excited for the OCI Image Format spec to be available in these formats as well

LGTM overall


Removing +255 arguments to function in order to allow <python37

This seems to fix the Python <3.7 incompatability seen in #44.
Is this supported by the maintainers? If so, I'll change it across the board (currently only fixed the one I need).

Hi @WilliamWCYoung ,
Thanks for taking time and making effort for this PR. We have an internal ticket raised for this particular issue(#44) DCM-3701. It would be good to drive the change from swagger + code.
As this change effects the swagger generated files, will get back to you about the possibility of PR in a day or two.
-Harsha

Hi @WilliamWCYoung ,
We will not be able to merge this PR, as this is a swagger related, we will handle this from our side and address in the future releases.
If you have no further comments, I will be closing this PR.
-Harsha


How to get title and url from the return of search object

I have a simple json file :
[

{

    "url": "http://localhost:1313/actualites/",
    "title": "Actualités"

}

,
{

    "url": "http://localhost:1313/feature2/",
    "title": "Rapide"

}

,
{

    "url": "http://localhost:1313/feature3/",
    "title": "Spring"

}

How do I create a serialise index from it that Lunr will understand and parse ?
    var index, store;
    $.getJSON('/index.json', function (response) {
        index = lunr(function(){
          var that = this;
          this.field('id');
          this.field('url');
          this.field('title', { boost: 50 });

          $.each(response, function( index, value ) {
            that.add({
              "id": index,
              "title": value.title,
              "url": value.url
            })
          });
        });

    });

That us my code so far but all I get is a returned object with a ref. I'm kinda lost how to get the title, and url I added in the $.each
Any help will be appreciated!

Lunr isn't a general purpose store for the documents that you are indexing, it can only return you the ref of the document that you indexed. You should then use this to retrieve the full document from some where else. In your case it would probably make sense to keep the parsed JSON you receive around as a lookup for the ref returned in the search results.
Also, if it was me, I'd consider using the url as the ref rather than trying to index it, e.g.
index = lunr(function (builder) {
  builder.ref('url')
  builder.field('title')
})

Finally, if you are using Lunr 2.x, boosts are not applied at indexing time, but at search time. Though if you only index the title it will make no difference.

Thank you so much for the answer, yeah I figured it out little bit after posting this, I use the ref returned and it worked great.
Good to know for the boost, thank you!


Add emission_finished signal to Particles2D

(Also remove some Particles2D::testee method, which was unused)
Closes #3963

my lord this is awesome, can we please get this in 3.0? can't find the signals

Well, it is removed by reduz in 95560e02c5eed3561996ed2b1d1e47e26e8bb81c, when he added GPU particles. Guess someone should redo it :)

really wish particle attractor  2d node could make a return someday as well. loved the suction (black hole) effect you could create with that stuff. however, the performance with the new particle system is absolutely way better so not complaining there. just trying to think of more ways to make it even better

@girng you can do something similar with a custom particles shader.. not that it is as simple as before..

@bojidar-bg thanks, will look into it


Remove global.yaml

This fully removes the global.yaml

Commit 1 and 2: regen operator tests, to ensure we are properly testing this change
Commit 3:

Sync values.yaml to istiod-remote chart
Fix a couple missing fields in values.yaml of istiod
Remove global.yaml

End result of generate config is identical as seen by the diffs in testing
I highly recommend you review only commit (3), as commits 1 and 2 are auto generated from ./operator/scripts/run_update_golden_snapshots.sh and will be a distraction/

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
d619301ff2efd6dd348ba76f4dd6c0ce35d1dbb6
link
/test release-notes_istio

lint_istio
d619301ff2efd6dd348ba76f4dd6c0ce35d1dbb6
link
/test lint_istio

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.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
d619301ff2efd6dd348ba76f4dd6c0ce35d1dbb6
link
/test release-notes_istio

lint_istio
d619301ff2efd6dd348ba76f4dd6c0ce35d1dbb6
link
/test lint_istio

release-test_istio
008a5c2e426a77deb55452331de92e5a9258294d
link
/test release-test_istio

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.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-test_istio
008a5c2e426a77deb55452331de92e5a9258294d
link
/test release-test_istio

lint_istio
008a5c2e426a77deb55452331de92e5a9258294d
link
/test lint_istio

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.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-test_istio
008a5c2e426a77deb55452331de92e5a9258294d
link
/test release-test_istio

lint_istio
008a5c2e426a77deb55452331de92e5a9258294d
link
/test lint_istio

integ-security-k8s-tests_istio
e2cff5a2515a8926748086729811f9ea6c82301f
link
/test integ-security-k8s-tests_istio

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

@howardjohn: PR needs rebase.

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.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
c69fc41647a8583f95eb548a41a1888b02657466
link
/test integ-multicluster-k8s-tests_istio

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.

@howardjohn: PR needs rebase.

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.

@howardjohn lets get this merged as without it in 1.8 Helm support will be fragile with globals.yaml around, thanks!

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
c69fc41647a8583f95eb548a41a1888b02657466
link
/test integ-multicluster-k8s-tests_istio

lint_istio
216f718f449898329b935cd6ffd4bd752c176f0c
link
/test lint_istio

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.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-helm-tests_istio
a390c41a8ad118abb4f14382683f81ec29587209
link
/test integ-helm-tests_istio

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.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
a4d2190841270b3601ef1b2eaa81be445d4c6c77
link
/test integ-pilot-multicluster-tests_istio

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.

@howardjohn can you cherry-pick for 1.8 or is it too late now?

/retest

@howardjohn can you cherry-pick for 1.8 or is it too late now?

For 1.8.0 its too late I think. For patch release.. I think this would be a breaking change to some extent? Perhaps we can just make it so global.yaml is there but optional? Which would involve a partial backport of just the few lines that changed in the charts?

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-mc-k8s-tests_istio
a4d2190841270b3601ef1b2eaa81be445d4c6c77
link
/test integ-telemetry-mc-k8s-tests_istio

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.

@howardjohn can you cherry-pick for 1.8 or is it too late now?

For 1.8.0 its too late I think. For patch release.. I think this would be a breaking change to some extent? Perhaps we can just make it so global.yaml is there but optional? Which would involve a partial backport of just the few lines that changed in the charts?

That works for me. We don’t document using `globals.yaml’ anywhere afaik for users so I’m not sure if it’s even a breaking change for patch releases but if you want to conservative I’m good with it.


certs generation issue

Hi guys!
I've followed manual for generating self signed certs https://opendistro.github.io/for-elasticsearch-docs/old/0.9.0/docs/security/generate-certificates/#generate-admin-certificate
Then in a month admin and node certs become expired. Some time was spent for error solving.
Consider to add key -days ... in manual for generating certs step, plz. For all who will use the manual further
Thanks for the opendistro elasticsearch

Hi @wildc4t, thanks a bunch for the suggestion -- totally fair, and an oversight on my part. I only mention -days in the root cert, but of course it applies to the others, as well. I've tried to clarify things here: https://github.com/opendistro/for-elasticsearch-docs/commit/6e54a28132add3f224e22f33d14392e011246678
Thanks again!


Fix setNftMetadata params & base response

add collectionId params to setNftMetadata.

Users can put values in the desired param. Since ERC721 usually uses contractAddress and Native NFT uses CollectionId, we have modified it to flexibly accommodate both.

Fix sendRequest response.

When the response data is empty, there is a case where the user does not receive the intended message, so it is modified.

Thanks for the review!


Fix run_generation test commands for TRL out usage example

In DPO pipeline section, it shows run_generation.py command which it runs with DPO output.
The DPO output (./dpo) is generated from llama2-70b-hf, but the examples (merging/run_generation) are basing llama2-7b-hf.
It fails with wrong dimension during merging and OOM when it runs run_generation test.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


docs: add fswatch guide

As discussed, this is a neat workaround for end users who use screenshot managers that don’t support third-party integrations. I will add more to it in the near future.

@CovertDuck other than the title change, I think it's ok to merge. Would love for you to fill the other sections too though, either now or in the near future


Auto restart

When I'm using steam or steam-query and the server goes down for say a minute the bot quits and needs a manual restart. I have found a work around using nodemon but thought it might be a nice addition to add👍

Hmm, it should automatically reconnect already 🤔 Do you have any error messages or crash logs you could share? :)

Hey thanks for the reply :)
yeah here is the error i get running "steam-query" when the arma server goes offline.
`Starting Discord Bot...
Logged in as Arma 3#8498!
App setup done...
/home/m0edev/bots/arma/node_modules/rxjs/dist/cjs/internal/util/reportUnhandledError.js:13
throw err;
^
Error: Steam did not respond with a single server, aborting the game status extracting.
at SteamProvider. (/home/m0edev/bots/arma/dist/adapter/steam/steam-provider.js:78:35)
at step (/home/m0edev/bots/arma/dist/adapter/steam/steam-provider.js:48:23)
at Object.next (/home/m0edev/bots/arma/dist/adapter/steam/steam-provider.js:29:53)
at fulfilled (/home/m0edev/bots/arma/dist/adapter/steam/steam-provider.js:20:58)
at processTicksAndRejections (node:internal/process/task_queues:96:5)`

Sorry for the late replay here. I changed the behavior and the tool will now try to take these things into consideration. It will now retry to query the server after some time (basically starting with 2 seconds between the call, with up to 30 seconds). All about the tool will retry querying the server or around 3 minutes and 45 seconds. If the server is not getting available again in this timeframe, the tool stops working as before.
Let me know if we need to tweak the timing here a bit or something 👍


Error trying to use PyVISA-py on Windows

I've been using the Agilent (now Keysight) VISA implementation with PyVISA for some time on my Windows machine.
I just tried installing PyVISA-py using:
pip install -U https://github.com/hgrecco/pyvisa-py/zipball/master

However I get:
Python 2.7.7 |Anaconda 2.0.1 (32-bit)| (default, Jun 11 2014, 10:41:43) [MSC v.1
500 32 bit (Intel)] on win32
Type "help", "copyright", "credits" or "license" for more information.
Anaconda is brought to you by Continuum Analytics.
Please check out: http://continuum.io/thanks and https://binstar.org
>>> import visa
>>> rm = visa.ResourceManager('@py')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Anaconda\lib\site-packages\pyvisa\highlevel.py", line 1471, in __new_
_
    visa_library = open_visa_library(visa_library)
  File "C:\Anaconda\lib\site-packages\pyvisa\highlevel.py", line 1440, in open_v
isa_library
    cls = get_wrapper_class(wrapper)
  File "C:\Anaconda\lib\site-packages\pyvisa\highlevel.py", line 1420, in get_wr
apper_class
    pkg = __import__(pkgname)
  File "C:\Anaconda\lib\site-packages\pyvisa-py\__init__.py", line 26, in <modul
e>
    from .highlevel import PyVisaLibrary
  File "C:\Anaconda\lib\site-packages\pyvisa-py\highlevel.py", line 20, in <modu
le>
    from pyvisa.compat import integer_types, OrderedDict
ImportError: cannot import name OrderedDict

This is the debug info output:
C:\Users\marnj>python -c "from pyvisa import util; util.get_debug_info()"
Machine Details:
   Platform ID:    Windows-7-6.1.7601-SP1
   Processor:      Intel64 Family 6 Model 58 Stepping 9, GenuineIntel

Python:
   Implementation: CPython
   Executable:     C:\Anaconda\python.exe
   Version:        2.7.7
   Compiler:       MSC v.1500 32 bit (Intel)
   Bits:           32bit
   Build:          Jun 11 2014 10:41:43 (#default)
   Unicode:        UCS2

VISA:
   PyVISA Version: 1.6.1.dev0

   #1: C:\windows\system32\visa32.dll
      found by: auto
      bitness: 32
   #2: C:\windows\system32\visa32.dll
      found by: auto
      bitness: 32

I am closing this for now. Feel free to reopen if necessary.

This should perhaps be a separate issue but I'm now getting this as my debug info:

C:\Anaconda>python -m visa info
Machine Details:
   Platform ID:    Windows-7-6.1.7601-SP1
   Processor:      Intel64 Family 6 Model 58 Stepping 9, GenuineIntel

Python:
   Implementation: CPython
   Executable:     C:\Anaconda\python.exe
   Version:        2.7.7
   Compiler:       MSC v.1500 32 bit (Intel)
   Bits:           32bit
   Build:          Jun 11 2014 10:41:43 (#default)
   Unicode:        UCS2

PyVISA Version: 1.6.3.dev0

Backends:
   ni:
      Version: 1.6.3.dev0 (bundled with PyVISA)
      #1: C:\windows\system32\visa32.dll:
         found by: auto
         bitness: 32
         Vendor: Keysight Technologies
         Impl. Version:<PHONE_NUMBER>
         Spec. Version: 5243136
      #2: C:\windows\system32\visa32.dll:
         found by: auto
         bitness: 32
         Vendor: Keysight Technologies
         Impl. Version:<PHONE_NUMBER>
         Spec. Version: 5243136
   py:
      Could not obtain debug info
      -> global name 'serial' is not defined

I just updated both PyVISA & PyVISA-py from the master on github about an hour ago. I don't have PySerial installed.
Also:

Why does the VISA32.dll show up twice?
Would it be possible to change the name of the first backend category from "ni" to that pulled from the Vendor? Or perhaps change it to "Proprietary" or something.


Deformation Of Mesh

Hi,
thank you for your project that it is so wonderful !
But I am wondering if the mesh generated could be deformed like the youtube video below:
https://www.youtube.com/watch?v=MMa2oT1wMIs
like the empty eyes, or the contour fitting to 2D face image.
Thanks a lot !

Hi,
Thanks!
Making the eyes empty should be easy by just removing the associated vertices and triangles.
As for the mesh, you can definitely deform a face mesh obtained with eos in a way like that. What they do is beyond the fitting that's available in eos (we fit a PCA shape model and expression blendshapes).

But how can I know which vertices are associated ?
Just try and error ?
Thanks.

Just write out the mean face as obj and open it in MeshLab or Blender to select the vertices.
You can either use the provided low-res reference.obj, or write out the mean using draw_sample and write_obj, or use the example app generate-obj.

Is it possible to modify sfm_shape_3448.bin and expression_blendshapes_3448.bin like adding some new expression, or adding new face ?
Thanks a lot ! Sorry for many questions , I am a newbie , and I am trying to finish my school project with eos.

I suggest you to read our VISAPP paper (and possibly more) to get a better understanding of the PCA shape model.
Adding new expressions is easy and could simply be done by adding new blendshapes (either from 3D data or manually created ones using for example Blender).

I'll close this, feel free to re-open if you have further questions.


MsBuild: Allow overwriting the version information in the AssemblyInfo files without MsBuild.Projects

We should be able to overwrite the version information in an AssemblyInfo file without needing to integrate with the project files.

Also allow adding attributes etc. that don't exist yet, e.g. build information attributes


Question: why shaka cannot play the content having different stream types between periods

What version of Shaka Player are you using?
v2.0.6

Can you reproduce the issue with our latest release version?
yes

Can you reproduce the issue with the latest code from master?
yes

Are you using the demo app or your own custom app?
my custom app

If custom app, can you reproduce the issue using our demo app?
not tried

What browser and OS are you using?
electron v1.6.1

What did you do?

What content did you load?
Load the MPD manifest written with some periods.
One period has a video and audio. But next one has a video only.

What did you expect to happen?
can play the content having different stream types between periods.

What actually happened?
shaka raised INVALID_STREAMS_CHOSEN error when I played between period.

I found that shaka prevent different stream in StreamingEngine.
https://github.com/google/shaka-player/blob/fc46b29ed8b0781bf60165682f125470e2c04b22/lib/media/streaming_engine.js#L1687-L1708
Why shaka implemented such behaviour?

We use MSE to play media.  We create a buffer for audio and video data and append data as we get it.  The browser pulls data from the buffers to play it in the <video> element.  If we switch from video+audio to just video, the browser will pause at the transition waiting for the content to buffer, but it doesn't know that the audio is never coming.
What we could do is reset MSE once we get to the transition, but then the transition won't be smooth since we need to delete everything and re-download it when the playhead gets there.  It would also be a lot more complicated to implement.  To make things simpler, we don't allow changing from video+audio to video-only.

@TheModMaker Okay, I see.
Does shaka have the policy to prevent different stream types, or does it play if possible in the future?

I don't understand your question, can you rephrase?  Are you asking if we support things other than audio/video/text, or are you asking about what the purpose of that code is?

I should note that although we won't support changing video+audio to video-only, we should support changing between text and no text.  I was told that we don't support changing between Periods that do and do not have text streams.  That is something we should fix.

@TheModMaker I understand. Thanks.

Renamed and reopened to track switching between periods with and without text.

This was just released in v2.0.7.


pennylane>=0.15 for amazon-braket-pennylane-plugin

Describe the bug
pennylane-lightning 0.15.1 requires pennylane>=0.15, but you have pennylane 0.14.1 which is incompatible.
amazon-braket-pennylane-plugin 1.1.1 requires pennylane==0.14.1, but you have pennylane 0.15.1 which is incompatible.
To reproduce
pip install pennylane-lightning
pip install amazon-braket-pennylane-plugin
Expected behavior
Both needs to be version compatible
Additional context
Since some of the features are incompatible between pennylane>=0.15 and pennylane==0.14., it seems like one definitely needs to have Braket plugin support pennylane==0.15

Currently being worked on in #62

Thanks for that info :D


Remove wrapper div tag

Describe the problem
Svelte kit auto wrap the dom tree with a div tag which make semantic tags meaningless.

Describe the proposed solution
Add an option to ignore wrapping dom tree with div tag
Alternatives considered
Completely remove the wrapping step.
Importance
nice to have
Additional Information
No response

You have control over this by editing src/app.html.

You have control over this by editing src/app.html.

Right solution!


Request SmartCash (SMART) Icon

To request icons for your coin, please fill info below:

SmartCash (SMART)

link to logo: https://smartcash.cc/wp-content/uploads/2018/07/Sigma-Original-New.png
official website: https://smartcash.cc/

Optional info (not mandatory, but saves me time):

logo color (HEX): #FEC60D
bitcointalk.org thread: https://bitcointalk.org/index.php?topic=2560430.0

Please note: This is all completely voluntary work, so if you want to support my efforts and speed up the addition of your coin, please donate. Donation addresses can be found on my website

This is not the correct logo.

SVG colored round logo here
https://drive.google.com/file/d/1D3HH4J2mXu2YgRQ55n4bdbqQjRipawt_/view


options.error not honored for suite

suite.js has this.options = {error: true} in the constructor, ignoring the value you have specified in vows.options.error.
This error feature I would almost consider broken, as not every callback takes an error as first argument...
for example http.get({/**/, this.callback);
will blow up because it only has 1 argument, the response, making you have to wrap this.callback to pass null instead...
So I did
var vows = require('vows');
vows.options.error = false;
but noticed it still did not work due to Suite ignoring your setting.
The options.error stuff really needs to be removed from context.js line 20~ event emitter.
Have it only apply to an ACTUAL event emitter if it triggers error, not on callbacks.
What is passed to this.callback should be exactly what is passed to your test cases.

vows 1.x doesn't support vows.options, nor does it support this argument to the suite constructor, so I'm going to close this


Update README.md

Was wondering if it was ok to add UIFontComplete to the list.
Thanks!
Project URL
https://github.com/Nirma/UIFontComplete
Description

Why it should be included to awesome-ios (optional)
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 or later
[x] Supports Swift 3
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

I was wondering if I could add my micro library for working with UIFont.
Thanks!

          1 Error

:no_entry_sign:
Found 3 link issues

          1 Warning

:warning:
Please update the Pull Request title to contain the library name

Link issues by awesome_bot

Line
Status
Link

2408
503
https://www.amazon.com/Programming-Objective-C-6th-Developers-Library/dp/0321967607

2411
503
https://www.amazon.com/Core-Data-iOS-Data-Driven-Applications/dp/0321670426

2412
503
https://www.amazon.com/Cocoa-Design-Patterns-Erik-Buck/dp/0321535022

  Generated by :no_entry_sign: danger

Hey @Nirma, thanks for contributing! 👍


fix: reduce items on menu if elsewhere in the UI

closes #1241

Thank you for attending to this 👍
However I can't open the "Import IEDs", "Settings" or "Extensions" menu items with this PR when I did some local testing so I think perhaps other modifications are required elsewhere otherwise the menu is not in synch with some other data source?

Thank you for checking it out, I'll look at it and let you know when Ive got a solution for this!

@danyill fixed the problem. It still tried to render menu items that weren't created any more. Should work fine now!


Migrate to Helm 3.0

Helm v3.0 GA was released in Nov 2019 https://helm.sh/blog/helm-3-released/
While at the moment we'd prefer to pin Helm 2.x, consider migrating to 3.x when it'll stabilize a little bit and see what involves for us to rely on latest Helm.

Hi @armab , hope you are doing well ? As you have added migrate to helm 3.0 feature for stackstorm . Do you have any initial thoughts how you are going to take it forward ? If i am correct i have made change in https://github.com/StackStorm/stackstorm-ha/blob/master/.circleci/config.yml#L20 file to point helm version 3.0.0 . but still getting issue as
helm install stackstorm-ha/ --generate-name
Error: found in Chart.yaml, but missing in charts/ directory: rabbitmq-ha, mongodb-replicaset, external-dns, etcd-operator  Do you have any points for this?

These are 3rd party Helm charts from https://github.com/helm/charts/blob/master/stable/
From the message above, looks like in order to switch to Helm 3.0 those upstream dependencies needs to be migrated first.
See https://github.com/helm/charts/blob/master/stable/rabbitmq-ha/Chart.yaml#L2 which still references previous Helm API Version.

Hi @armab, I installed rabbitmq-ha-1.36.4 first with helm 3.0 and it succeeded:

NAMESPACE     NAME                                  READY   STATUS             RESTARTS   AGE     IP
default       my-rabbit-rabbitmq-ha-0               1/1     Running            1          2d8h    <IP_ADDRESS>
default       my-rabbit-rabbitmq-ha-1               1/1     Running            1          2d8h    <IP_ADDRESS>
default       my-rabbit-rabbitmq-ha-2               1/1     Running            1          2d8h    <IP_ADDRESS>

So does it mean this chart works in helm 3.
But if install ss ha with :
helm install stackstorm -n st2 -f values.yaml .
The error is:

manifest_sorter.go:175: info: skipping unknown hook: "crd-install"
Error: unable to build kubernetes objects from release manifest: unable to recognize "": no matches for kind "EtcdCluster" in version "etcd.database.coreos.com/v1beta2"

That part is coming from the "etcd-operator" Helm chart dependency from the official Helm repository.
Relevant answer from similar https://github.com/kubernetes-sigs/kubefed/issues/1143#issuecomment-528992390:

@armab  Thanks. So the left is etcd-operator's issue. Let's wait for it to be updated to support helm 3.

These are 3rd party Helm charts stackstorm-ha relies on https://github.com/helm/charts/blob/master/stable/
From the message above, looks like in order to switch to Helm 3.0 those upstream dependencies needs to be migrated first.
See https://github.com/helm/charts/blob/master/stable/rabbitmq-ha/Chart.yaml#L2 which still references previous Helm API Version.

Thanks @armab and @wantdrink for replying back . have tried to make changes in https://github.com/rahulparihar270992/charts  repo as belows https://github.com/rahulparihar270992/charts/blob/master/stable/rabbitmq-ha/Chart.yaml#L2 . and tried to changed requirements.yml as https://github.com/rahulparihar270992/stackstorm-ha/blob/feature/stackstome-v1/requirements.yaml#L4 . Not sure this is the right pointing to stable dependencies ? Do you have any suggestion how it going to work?  Still facing similar issue.
Error: found in Chart.yaml, but missing in charts/ directory: rabbitmq-ha, mongodb-replicaset, external-dns, etcd-operator

I hadn't chance to really try Helm 3.0 yet, so google is your better friend than me here.
However looks like the first big visible difference between v2 and v3 is that requirements should be moved to Chart.yml instead of requirements.yml file.

Hi @armab, I installed rabbitmq-ha-1.36.4 first with helm 3.0 and it succeeded:

NAMESPACE     NAME                                  READY   STATUS             RESTARTS   AGE     IP
default       my-rabbit-rabbitmq-ha-0               1/1     Running            1          2d8h    <IP_ADDRESS>
default       my-rabbit-rabbitmq-ha-1               1/1     Running            1          2d8h    <IP_ADDRESS>
default       my-rabbit-rabbitmq-ha-2               1/1     Running            1          2d8h    <IP_ADDRESS>

So does it mean this chart works in helm 3.
But if install ss ha with :
helm install stackstorm -n st2 -f values.yaml .
The error is:

manifest_sorter.go:175: info: skipping unknown hook: "crd-install"
Error: unable to build kubernetes objects from release manifest: unable to recognize "": no matches for kind "EtcdCluster" in version "etcd.database.coreos.com/v1beta2"

@wantdrink , @armab  I am also facing similar issue . Have you updated helm chart for etcd-operator ?
Can you please give me some pointer here, what next step have you taken?

@rahulparihar270992 SS chart has a set of dependencies chart. Fortunately some of them already supports Helm3.
unfortunately etcd-operator as one mandatory dependency cannot be omitted and must be installed. But its chart hasn't been updated yet for Helm3.0 (version 0.10.0 in requirement).
Let me try 0.10.2 today and see if it works.

@armab  Thanks, you are right.

0.10.2 can be installed with helm3. It looks like we can move forward.

Sounds promising.
Can you guys create a PR to address that?

Hi @armab I'd like to do that. I'm still testing today with outside rabibtmq & mongo instead of installing them with ss chart.
Now I pods/services/statefulset/ep are created successfully, login portal/cli/default pack action running all good.
But I still have some questions:

There is still "crd-install" warning but it seems ss works well. Could you please guide us for how to validate if etcd cluster works well together with ss? Maybe some simple testing is needed.

I want to put ss images into my private repository, in that way I can deploy my own pack image to test, also to faster the deployment. I added repository in values.yml as below but it doesn't work.

image:
  repository: repository.test.com:8443/stackstorm
  pullPolicy: IfNotPresent

Then I changed templates/_helpers.tpl as:
# Generate Docker image repository: Private 'docker.stackstorm.com' for Enterprise vs Public Docker Hub 'stackstorm' for FOSS version
{{- define "imageRepository" -}}
{{- if required "Missing context '.Values.enterprise.enabled'!" .Values.enterprise.enabled -}}
docker.stackstorm.com
{{- else -}}
repository.test.com:8443/stackstorm
{{- end -}}
{{- end -}}

It didn't work either and images were stilled pulled from docker hub.

The same with item 2 how can I change the default repository of etcd-operator?
I changed the stackstorm-ha/charts/etcd-operator/values.yaml and it didn't work.
E.G.

# etcdOperator
etcdOperator:
  name: etcd-operator
  replicaCount: 1
  image:
    repository: repository.test.com:8443/quay.io/coreos/etcd-operator
    tag: v0.9.4
    pullPolicy: IfNotPresent
  resources:
    cpu: 100m
    memory: 128Mi

Some of StackStorm services will just fail if they couldn't contact etcd for coordination.
For using custom st2 packs, see: https://github.com/StackStorm/stackstorm-ha#install-custom-st2-packs-in-the-cluster and https://docs.stackstorm.com/install/k8s_ha.html#custom-st2-packs.
You can ask other questions in StackStorm Community: https://stackstorm.com/#community Slack or Forum.
Let's please keep this issue dedicated to Helm v3.0.

Thanks @armab
Find the root cause. I used tgz file for command helm install and the image repository of ss/etcd cannot be changed by passing the values.yaml.
To solve this just modify the _helper and charts/etcdxxx/values.yml of tgz file then rerun and finally the helm pull images from my private repository.

@armab  I've tested the installation of etcd-operator 0.10.2. There is an error when executing helm install but everything can be installed successfully.
Also I tested ss feature and it looks good.
Finally I rebooted all k8s nodes and there is an issue: the etcd cluster pods cannot be restarted automatically:
default       etcd-cluster-6255djfdkq                              0/1     Completed          0          17m     <none>
default       etcd-cluster-k5lmchwjp5                              0/1     Completed          0          16m     <none>
default       etcd-cluster-nlzjt5qmdn                              0/1     Completed          0          17m     <none>

I have to restart them manually and after that ss works well.

Good to hear.
BTW Etcd issue you mentioning is not really related to Helm 3.0. We have it documented here: https://github.com/StackStorm/stackstorm-ha/issues/94

@armab  hum......maybe single etcd instance is better.
Thanks.

Hi @armab, I installed rabbitmq-ha-1.36.4 first with helm 3.0 and it succeeded:

NAMESPACE     NAME                                  READY   STATUS             RESTARTS   AGE     IP
default       my-rabbit-rabbitmq-ha-0               1/1     Running            1          2d8h    <IP_ADDRESS>
default       my-rabbit-rabbitmq-ha-1               1/1     Running            1          2d8h    <IP_ADDRESS>
default       my-rabbit-rabbitmq-ha-2               1/1     Running            1          2d8h    <IP_ADDRESS>

So does it mean this chart works in helm 3.
But if install ss ha with :
helm install stackstorm -n st2 -f values.yaml .
The error is:

manifest_sorter.go:175: info: skipping unknown hook: "crd-install"
Error: unable to build kubernetes objects from release manifest: unable to recognize "": no matches for kind "EtcdCluster" in version "etcd.database.coreos.com/v1beta2"

It seems like I am late to this conversation but I have managed to find a workaround to this problem which was to install etcd operator manually before installing stackstorm-ha. Currently I can run Stackstorm without any issues.
But I must say that, Helm's error traces are just awful. It says there is an issue but does not say about where the problem is...

Do you have an eta for the migration of the charts to helm 3?
Or is there some repo available that has the adopted files?
@wantdrink your example with external components would be very nice and helpful.
Thanks a lot!

There is no ETA for migrating to Helm 3, but we’ll be happy to accept PR contributions.

Is there some progress to build upon available or does one have to start from the helm 2 charts?

You can read this Issue history with the pointers regarding Helm 2 migration.

Version 0.10.2 no longer exists for some reason (0.10.1 doesn't exist either). But version 0.10.0 can still be downloaded. I haven't gotten it to install yet locally though with helm 3...not sure if it has something to do with Docker Desktop (which is how I'm running K8s) or something else...

have people had success with helm v3 ?

@Fauli  @c1505
The simplest way to install with helm3(currently) is to clean the dependencies of requirements.yaml and then define the db/message/coordinator url etc in the values.yaml. Of course these dependencies should be installed separately before installing ss ha.
@armab another simple question: if we want to specify the ssl key/cert file like below, where should we put these files?
.......
st2:
  config: |
    ......

    [database]
    host = <IP_ADDRESS>
    ssl = True
#    ssl_keyfile = <Path to key file>
#    ssl_certfile = <Path to certificate>
#    ssl_cert_reqs = <One of none, optional or required>
#    ssl_ca_certs = <Path to certificate form mongod>
#    ssl_match_hostname = <True or False>

Thanks.

@wantdrink You need to find the way to share these files with all the Pods.
Existing SSH key in Helm values and following that in Helm templates/source code can give some pointers as an example where to look at.

Thanks a lot @armab

https://helm.sh/blog/helm-v2-deprecation-timeline/

Hi all, I know I am a bit late to this thread, but I have been running StackStorm-HA deployment using Helm 3 from the day 1 in our production and have not come across any issue that's blocker.
NAME                                                              READY   STATUS      RESTARTS   AGE
etcd-cluster-6qkcrbvm2r                                           1/1     Running     0          76d
etcd-cluster-xzkpsw74rx                                           1/1     Running     0          76d
etcd-cluster-zh4gjj2skm                                           1/1     Running     0          76d
etcd-operator-etcd-operator-etcd-backup-operator-7766cf9488dmxn   1/1     Running     2          76d
etcd-operator-etcd-operator-etcd-operator-56dfc86b9d-pqrd4        1/1     Running     4          76d
etcd-operator-etcd-operator-etcd-restore-operator-c564466bbbpbj   1/1     Running     2          76d
st2ha-etcd-operator-etcd-operator-5fc86f779d-6jh5q                1/1     Running     0          76d
st2ha-job-st2-apikey-load-29mh5                                   0/1     Completed   0          6d15h
st2ha-job-st2-key-load-2mp95                                      0/1     Completed   0          6d15h
st2ha-job-st2-register-content-bpqfw                              0/1     Completed   0          6d15h
st2ha-mongodb-ha-0                                                1/1     Running     0          76d
st2ha-mongodb-ha-1                                                1/1     Running     0          76d
st2ha-mongodb-ha-2                                                1/1     Running     0          76d
st2ha-rabbitmq-ha-0                                               1/1     Running     1          39d
st2ha-rabbitmq-ha-1                                               1/1     Running     4          76d
st2ha-rabbitmq-ha-2                                               1/1     Running     3          76d
st2ha-st2actionrunner-58c5bb6946-8b6g2                            1/1     Running     0          6d15h
st2ha-st2actionrunner-58c5bb6946-czpt9                            1/1     Running     0          6d15h
st2ha-st2actionrunner-58c5bb6946-g56cg                            1/1     Running     0          6d15h
st2ha-st2actionrunner-58c5bb6946-m7hxr                            1/1     Running     0          6d15h
st2ha-st2actionrunner-58c5bb6946-v9975                            1/1     Running     0          6d15h
st2ha-st2api-789578f8f5-cqbxx                                     1/1     Running     0          6d15h
st2ha-st2api-789578f8f5-dl5bn                                     1/1     Running     0          6d15h
st2ha-st2auth-76b46895b-hp9w6                                     1/1     Running     0          6d15h
st2ha-st2auth-76b46895b-qp48j                                     1/1     Running     0          6d15h
st2ha-st2client-6467866ddc-57td8                                  1/1     Running     0          6d15h
st2ha-st2garbagecollector-5599b58cb6-4qmxx                        1/1     Running     0          18d
st2ha-st2notifier-55b9c595c8-2glxh                                1/1     Running     0          18d
st2ha-st2notifier-55b9c595c8-7zz8m                                1/1     Running     0          18d
st2ha-st2rulesengine-7d858458bf-cw92d                             1/1     Running     0          18d
st2ha-st2rulesengine-7d858458bf-r5t7g                             1/1     Running     0          18d
st2ha-st2scheduler-6985f84c9c-84sdr                               1/1     Running     0          18d
st2ha-st2scheduler-6985f84c9c-hq6rq                               1/1     Running     0          18d
st2ha-st2sensorcontainer-6855744bf-tcxf7                          1/1     Running     0          6d15h
st2ha-st2stream-7867fc988f-bhk5p                                  1/1     Running     0          18d
st2ha-st2stream-7867fc988f-nwpfm                                  1/1     Running     0          18d
st2ha-st2timersengine-69d47748b8-ppmt2                            1/1     Running     0          18d
st2ha-st2web-74b7548b66-txjbn                                     1/1     Running     0          76d
st2ha-st2web-74b7548b66-wsvhl                                     1/1     Running     0          76d
st2ha-st2workflowengine-6bc99758fd-r2cnt                          1/1     Running     0          18d
st2ha-st2workflowengine-6bc99758fd-vpsd8                          1/1     Running     0          18d```

Below is the helm client version running on my machine from where I deployed the app. The same was present in our Bamboo agent.
```helm version
version.BuildInfo{Version:"v3.3.4", GitCommit:"a61ce5633af99708171414353ed49547cf05013d", GitTreeState:"clean", GoVersion:"go1.14.9"}```

I am not sure about any specifics but so far this has been running fine for 2+ months. I have 2 different clusters in AWS using EKS.

Hi everyone, I have raised PR https://github.com/StackStorm/stackstorm-ha/pull/163 in an attempt to move this issue along to completion. I have tested it in a few environments with helm v3.4.1 seems to work.


Add evaluation for 10kGnad

Dataset: https://huggingface.co/datasets/gnad10
Document classification: https://www.deepset.ai/german-bert

Already done in https://github.com/OpenGPTX/lm-evaluation-harness/tree/mlqa


Get-SmaReportingDefinition can be run without an exclusive parameter set

one of 3 exclusive parameters must be set or an error is thrown. function should require one exclusive parameter to execute query.

resolved in 1.0.6


Refactor app_main

Refactored app_main to improve code readibility by using separate functions.

Hi @ehoseini, firstly thanks contributing!
This example is associated with the Tutorial Explore the LED Blink example (Specifically Section 2: Understand the code). As such we want to keep it all contained in a single function so the behavior is easier to read and understand for a new developer.


[9.x] Add ability to define "with" relations as a nested array

Purpose
This PR aims to add the ability to specify relationship "withs" as a PHP array, rather than dot notation.
Usage
// before...

User::with([
    'avatar',
    'posts.tags',
    'posts.author',
    'posts.featureImage',
    'posts.comments.tags' => fn ($q) => $q->latest(),
])->get();

// after...

User::with([
    'avatar',
    'posts' => [
        'tags',
        'author',
        'featureImage',
        'comments' => [
            'tags' => fn ($q) => $q->latest(),
        ],
    ],
])->get();

Advanced
// supports chaining...

User::with([
    'posts' => fn ($q) => $q->withCount('comments')->with([
        'comments' => [
            'tags',
        ],
    ]),
])->get();

// supports specifying attributes inline, although I think with this notation you
// should probably be specifying selects via a constraint.

User::with([
    'posts:id,title,user_id' => [
        'comments:id,content,post_id' => [
            'tags',
        ],
    ],
])->get();

// supports mixing notation. Not saying you should, just saying you can 🙈

User::with([
    'posts' => [
        'comments',
    ],
    'posts.image',
])->get();

// merges constraints from different notation - again, not saying you should. just saying you can.

User::with([
    'posts.comments' => fn ($q) => $q->with('tags'),
    'posts:id,title,user_id' => [
        'comments' => fn ($q) => $q->withCount('tags'),
    ],
])->get();

What isn't supported
I should note that merging of selects from different notation is not supported. I don't see this as a problem, as the current dot notation suffers from the same constraint...
// does not select "id" and "name"
User::query()
    ->with(['users:id,name'])
    ->with(['users:email'])
    ->get();

// nor does this...

User::query()
    ->with(['users' => fn ($q) => $q->select(['id', 'name'])])
    ->with(['users:email'])
    ->get();

and so the following is explicitly not supported...
User::with([
    'posts.comments' => fn ($q) => $q->addSelect(['id']),
    'posts' => [
        'comments' => fn ($q) => $q->addSelect(['post_id'])
    ],
])->get();

User::with([
    'posts' => [
        'comments:post_id,title'
    ],
    'posts.comments:id',
])->get();

or any other mixture of notation with duplicate selects.
Notes
I'd love to get some eyes on this from a range of people who are deep in Eloquent land.

Does this also work on ->load()?


getindex(::Column) performance

Starts to address #250 . Entirely fixes the bug as described in that issue (Float64s, each call to getindex allocates nothing).
The changes include:

introducing a layer of indirection (thanks @tpgillam for this suggestion) in the PQValue type to prevent the need to allocate when result (which is a mutable struct) is used to construct a PQValue.
moving a lot of type information into the type of PQValue and Column so that it can be statically resolved.
Replacing the parse_type closure with the ParseType{T} callable to force the compiler to specialise.

I need advice as to how to proceed (@iamed2 I would like to take you up on your offer in #250 :) ) :

Have I gone about this in a sensible way -- I don't have the overview of the package's architecture that you do @iamed2, so I might be missing something obvious.
how should I write performance tests for this? Is there a good place in runtests into which I should hook? It's a rather large file, so I'm not completely sure what a good entrance point would be.
What types should I be testing for performance here? Clearly floats / ints / dates etc are good targets -- do we have a collection of examples columns instantiated somewhere in the tests that I could attach some performance tests to?

edit: interestingly, it actually looks like the indirection wasn't necessary in order to remove the allocations. It appears to be the case that everything is sorted by being slightly more assertive in the specialisations that the compiler is made / allowed to perform. Even more interestingly, it doesn't even look like we need to specialise on the typeof jl_result in PQValue.

Have I gone about this in a sensible way -- I don't have the overview of the package's architecture that you do @iamed2, so I might be missing something obvious.

This is excellent, I love how small and targeted it is now.

What types should I be testing for performance here? Clearly floats / ints / dates etc are good targets -- do we have a collection of examples columns instantiated somewhere in the tests that I could attach some performance tests to?

We should test with binary format and with text format, and with custom conversion functions (the conversions argument to execute) and custom types (type_map and column_types arguments to execute). We should also check with not_null and without. Testing with those types are good, as well as strings, and it might make sense to test with array types or interval types since those are a bit more complicated type-wise.
What you could do is create a single row with the data of types you care about (in text), then create a temporary table and insert that row N times. You can then select the contents of the table (several times for different args to execute) and measure parsing on each column or together. The test_data variables in the test file have examples of PostgreSQL literals and what they parse to.
Given that the benefits are so very huge, it could be worth just recording the memory estimate and time from a benchmark generated now and just testing that we haven't exceeded it by ~2-3x. Long term it would be good to set up a proper benchmark suite using https://juliaci.github.io/PkgBenchmark.jl/stable/.

What you could do is create a single row with the data of types you care about (in text), then create a temporary table and insert that row N times. You can then select the contents of the table (several times for different args to execute) and measure parsing on each column or together. The test_data variables in the test file have examples of PostgreSQL literals and what they parse to.

I'm actually not sure how to proceed with this. Could you provide an example where we already do this?

There are examples of all the pieces in the nebulous runtests.jl so it's just a matter of pulling them together in the right way. Collecting them:
This creates a temporary table that will be destroyed when you close the connection. This particular table has one identity column (you can ignore it) and then one string column, for other types you can just add items to the column list replace varchar with e.g. timestamptz.
        conn = LibPQ.Connection("dbname=postgres user=$DATABASE_USER")

        result = execute(conn, """
            CREATE TEMPORARY TABLE libpqjl_test (
                id    bigint PRIMARY KEY GENERATED BY DEFAULT AS IDENTITY,
                my_string_column   varchar
            );
        """)

This code inserts a row (you can repeat it to insert multiple rows; they will get a different id):
    result = execute(conn, "INSERT INTO libpqjl_test (my_string_column) VALUES (\$1)", ["foo"])

or
    result = execute(conn, "INSERT INTO libpqjl_test (my_string_column) VALUES ('foo')")

In the latter version, anything in this variable test_data in the first tuple element can be substituted for 'foo'. When SELECTing, it should parse as the second tuple element by default.
In this other test_data, we have a middle column. When you specify that type in a type_map or column_types argument, you should expect the third tuple element as the result when SELECTing.
I think that's a good place to start, I can explain the other bits once you've made it through the above, so that we have that shared context.
Happy to have a call or a pair-programming session to help you get started. But it's worth saying I don't think performance tests should prevent this MR from going in.

Thanks for the advice @iamed2 . I've added some regression tests (wound up only using one of the two test data sets). Let me know if you think I've missed anything!

Note: we should definitely squash + merge this one when it goes in, because I accidentally committed a bunch of larger files that we don't want in the history.

Thanks @iamed2 . Will merge when CI passes

Having said that, I don't have merge right.
@iamed2 pls squash + merge when you get a minute :) Thanks again for the review and helping me get this across the line!


feat(berlin_summit_23): new blogpost [#98]

Closes #98
Open Issues:

[x] Let the blog post sit for a day
[x] Optimize the used markdown images (experimental feature https://docs.astro.build/en/guides/images/)
[x] Squooshify the images

https://docs.astro.build/en/guides/images/#srcassets-experimental
we are already using it:

https://docs.astro.build/en/guides/images/#srcassets-experimental
we are already using it:

ok then i guess only the preview is broken.. should be working according to the docs 👍🏻
https://docs.astro.build/en/guides/assets/#update-your-markdown-mdx-and-markdoc-files

Sorry for not running a spellchecker first 💩 finally did it


dev(CI): add deno fmt

Describe the enhancement request
Add formatting check using Deno in the CI. This job would simply call deno fmt --check indexer/ and fail if the formatting of any of the deno files is incorrect.

Gm! Can I take this?

Gm! Can I take this?

assigned