[Question] Can CenterTrack model be compiled using TorchScript?

I want to load model using TorchScript scripting/tracing. However, I unsuccessfully tried:

scripted_model = torch.jit.script(model) and got error:

RuntimeError:
module has no attribute 'pre_hm_layer':
at model/networks/dla.py:311:20
    def forward(self, x, pre_img=None, pre_hm=None):
        y = []
        x = self.base_layer(x)
        if pre_img is not None:
            x = x + self.pre_img_layer(pre_img)
        if pre_hm is not None:
            x = x + self.pre_hm_layer(pre_hm)
                    ~~~~~~~~~~~~~~~~~ <--- HERE
        for i in range(6):
            x = getattr(self, 'level{}'.format(i))(x)
            y.append(x)

        return y
'__torch__.tracker.center_track_tracker.model.networks.dla.DLA.forward' is being compiled since it was called from '__torch__.tracker.center_track_tracker.model.networks.dla.DLASeg.imgpre2feats'
<...>

traced_model = torch.jit.trace(self.model, torch.rand(1, 3, 512, 512)) and also got error:

model/networks/DCNv2/dcn_v2.py:31: TracerWarning: Converting a tensor to a Python integer might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
  ctx.deformable_groups)
<...>
module._c._create_method_from_trace(method_name, func, example_inputs, var_lookup_fn, _force_outplace)
RuntimeError: Only tensors or tuples of tensors can be output from traced functions (getOutput at /pytorch/torch/csrc/jit/tracer.cpp:209)
frame #0: c10::Error::Error(c10::SourceLocation, std::string const&) + 0x33 (0x7f72f3f82813 in venv/lib/python3.6/site-packages/torch/lib/libc10.so)
frame #1: torch::jit::tracer::TracingState::getOutput(c10::IValue const&) + 0x365 (0x7f72925de2a5 in venv/lib/python3.6/site-packages/torch/lib/libtorch.so)
frame #2: torch::jit::tracer::exit(std::vector<c10::IValue, std::allocator<c10::IValue> > const&) + 0x3c (0x7f72925de5ac in venv/lib/python3.6/site-packages/torch/lib/libtorch.so)
frame #3: <unknown function> + 0x5705c2 (0x7f72f49455c2 in venv/lib/python3.6/site-packages/torch/lib/libtorch_python.so)

Is it possible in general to save/load CenterTrack using TorchScript scripting/tracing? Maybe I just lack some knowledge of PyTorch.

Sorry for the delayed reply. CenterTrack uses two frames and a prior heatmap as input, you probably want to try something like
traced_model = torch.jit.trace(self.model, torch.rand(1, 3, 512, 512), torch.rand(1, 3, 512, 512), torch.rand(1, 1, 512, 512)).

Hey @ievbu and @xingyizhou , I got it to work with jit.trace. The main issue is that the model forward function packages the model output within a list. You got to undo that by changing the BaseModel.py, line 91, to return out[-1]. Then, go to the Detector.py process function and remove the [-1] right after the model call. Finally, mark your jit trace as strict=False
Here is the trace call.
    dummy_inputs = torch.rand((1, 3, 512, 512)), torch.rand((1, 3, 512, 512)), torch.rand((1, 1, 512, 512))
    self.model = torch.jit.trace(self.model, dummy_inputs, strict=False)

It is also fairly easy to update the project to the current PyTorch version and compile to half precision AND optimized_for_inference. I got a 10-15% speedup from this on a GTX 2070.


Add chmod, fchmod, fchmodat functions

Done.

@asomers Done the proposed changes.

@asomers Fixed.

Thanks, @antage.  It all looks good now.
bors r+


Using --node-external-ip leads to https://<IP_ADDRESS>:443 connection refused

Environmental Info:
K3s Version:

k3s version v1.21.4+k3s1 (3e250fdb)
Node(s) CPU architecture, OS, and Version:

Linux htpc 5.4.0-84-generic #94-Ubuntu SMP Thu Aug 26 20:27:37 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Cluster Configuration:

Single node cluster, running on an Intel NUC with ubuntu
Describe the bug:

I got k3s working fine, but ran into some trouble when trying to access my services via the internet.
I simply installed via the script, with no flags or extra configuration.
I’m using external-dns to automatically create DNS records. Problem was I could only get the DNS working from my local network, not via the internet.
I looked up the DNS rules, and saw that they are referencing my node’s local ip address.
I look in kube-system, and see that the external ip of the traefik load balancer is also referencing my node’s local ip address.
I attempt to resolve this by creating /etc/rancher/k3s/config.yaml with the following entries:
node-ip: <my node's local ip address>
node-external-ip: <my public ip address>
I run k3s-killall.sh and restart with systemctl restart k3s.
Now im seeing multiple pods in kube-system failing. I check their logs, and find that the problem with all of them is that they are getting connection refused on https://<IP_ADDRESS>:443.
So, the config options above breaks the pods, and from what I understand it has something to do with containerd and iptables.
Wondering how to resolve this.
Steps To Reproduce:

Install k3s with script on ubuntu machine. Set node-external-ip to match public ip, restart k3s.
Expected behavior:

Should work.
Actual behavior:

Pods fail because https://<IP_ADDRESS>:443 gives connection refused.
Additional context / logs:

Never got it working with node-external-ip.
The solution for me was to use annotations with external-dns to override the local ip from the loadbalancer.
If you run into the same problem, simply add the following annotations to your apps ingress:
external-dns.alpha.kubernetes.io/target: <public ip>


Tuning contextual embeddings with hierarchical relations

I have a masked LM  pretrained with bert.
The embeddings are poor on the sentence level, but do well for base tokens.
There is a natural tree structure to my corpus that I believe stands to gain from something like on-lstm.
Do you think swapping out the embedding layer of the on-lstm with pretrained bert embeddings could be fruitful?

If you have a sentence-level task, I believe it's worth to try.


Re-rendering QueryResult when sidebar width changes

Before this PR, re-sizing the sidebar doesn't cause QueryResult to re-render:

After this, it'll re-render when we stop dragging:

I'm experimenting with ways to re-render as we drag but changing query-browser's onResizeStop to onResize was really laggy. I tried debouncing it but it just didn't fire at that point until I stopped dragging.
Open to ideas, but in the mean time this small change is better than leaving it as is IMO.

Great, thanks @s-kem ! I agree, no need for throttle/debounce.


Feature | Add ML libraries

There is no ML Pack page. Add some useful ML libraries.

Go for it @wasimreja

@imniladri bhaiya could you assign the label as medium? ML Pack page isn't created previously. I have to create the page, integrate it with other pages and then add the libraries.

Ok sure, I was too thinking about it that it may take time.

@imniladri bhaiya could you assign the label as medium? ML Pack page isn't created previously. I have to create the page, integrate it with other pages and then add the libraries.


More brazilian holidays ibge

superseedes and potentially closes #409
Important items:

Added 27 Brazil calendars (thanks a lot to @luismalta & @mileo),
Fixes and additions to some calendars (again, thanks to @luismalta & @mileo),
Added an IGBE_REGISTER to reference IGBE (brazilian) calendars with related tests.

[x] Tests with a significant number of years to be tested for your calendar.
[ ] Changelog amended with a mention describing your changes.

could you review this, please @luismalta and @mileo?

Thanks!

Thank you!


Allow to publish multiple ports for accessories

Closes https://github.com/mrsked/mrsk/issues/347
I'm not a ruby expert, but, luckily, there are 'host' and 'hosts' directives already, so I just followed the approach.

Thanks for your contribution @bohdan-shulha - it'll be useful to publish multiple ports for an accessory.
I disagree with the style changes proposed by @Zeko369 - I think it would be better to maintian the existing code style, and keep the implementation in the same vein as the existing #hosts_from_host and #hosts_from_hosts.
Otherwise, this looks good to me.

Hi @lewispb ,
Thanks for review. ;)
Would it be enough to revert to this commit then?
https://github.com/mrsked/mrsk/pull/348/commits/f6f34dea7914bde7e1ee4633656ba06cf839eea5


unknown pongo2.Error field 'ErrorMsg' in struct literal

I am all of a sudden starting to get this error. What could be causing this? Below is the offending output.
.openshift/g/src/github.com/flosch/pongo2-addons/filters.go:225: unknown pongo2.Error field 'ErrorMsg' in struct literal
.openshift/g/src/github.com/flosch/pongo2-addons/filters.go:234: unknown pongo2.Error field 'ErrorMsg' in struct literal
.openshift/g/src/github.com/flosch/pongo2-addons/filters.go:257: unknown pongo2.Error field 'ErrorMsg' in struct literal
.openshift/g/src/github.com/flosch/pongo2-addons/filters.go:267: unknown pongo2.Error field 'ErrorMsg' in struct literal

I have checked pongo2.Error and it doesn't have that field. Should it be OrigError?

Will be fixed very soon in https://github.com/flosch/pongo2-addons/pull/3

Fixed in tip.


Error on Installation

I'm getting this error on installation of this plugin. The error is below.
Parse error: syntax error, unexpected '[' in --pathtofolder-/craft/plugins/pimpmymatrix/services/PimpMyMatrixService.php on line 224

I'm using 2.1.2, but tried all of the 2.5 releases. On Craft Pro 2.5.2759

Hmm that’s weird - what version of php are you using?

Looks like I'm on PHP Version 5.3.29

Aha, that may well be the issue. Try upgrading to at least v 5.5: http://plugins.supercooldesign.co.uk/plugin/pimp-my-matrix/docs#requirements


Major issue

Hey, It can't accept any custom viewholder which extends RecycleView.ViewHolder.
My class is like that and it is showing following error
class MyHolder extends RecyclerView.ViewHolder

@AAPBD Hey,
Actually why do you declare your SimpleItemRecyclerViewAdapter extending RecyclerView.Adapter<CustomViewHolder> ? I can see that SimpleItemRecyclerViewAdapter is yours from the package com.aapbd.leatnbanglaeasily... then I suggest you to extend it from the base abstract type which allows you use it with different components/libs and usually is a good code style... Please take a look at small example class which I wrote for you
CustomViewHolder class
public class CustomViewHolder extends RecyclerView.ViewHolder {
    public CustomViewHolder(View itemView) {
        super(itemView);
    }
}

SimpleItemRecyclerViewAdapter class
public class SimpleItemRecyclerViewAdapter extends RecyclerView.Adapter<RecyclerView.ViewHolder> {
    private List<String> items = new ArrayList<String>();
    private Context mContext;
    public SimpleItemRecyclerViewAdapter(Context context){
        mContext = context;
    }

    //you can declare a return type as **RecyclerView.ViewHolder** but actually you return your **CustomViewHolder**
    @Override
    public RecyclerView.ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
        //Here a **RecyclerViewExampleItem** is any class extending **GroupView**.
        return new CustomViewHolder(new RecyclerViewExampleItem(mContext));
    }

    //declare a **RecyclerView.ViewHolder** as a parameter and you can convert it to
    your **CustomViewHolder** if you wish
   @Override
    public void onBindViewHolder(RecyclerView.ViewHolder viewHolder, int position) {
        CustomViewHolder cvholder = (CustomViewHolder)viewHolder;
        // Do here something specific to the CustomViewHolder type...
        RecyclerViewExampleItem rvei = (RecyclerViewExampleItem) cvholder .itemView;
        String str = getItem(position);
        rvei.bind(str);
    }

    @Override
    public int getItemCount() {
        return items.size();
    }

    public String getItem(int position) {
        return items.get(position);
    }

    public void addAll(List<String> lst){
        items.addAll(lst);
    }
}


PR: Add --no-sandbox argument for QtApplication

Description of Changes
Add --no-sandbox argument when instantiating QtApplication object, in order to avoid a bug linked to Ubuntu 22.04 (see https://discourse.slicer.org/t/extensions-not-showing-anything/23432/5 and https://doc.qt.io/qt-6/qtwebengine-platform-notes.html)
Note sure if this is needed in manager.py tho, put it here just for consistency.
Issue(s) Resolved
Fixes #20358
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: tlunet

I can confirm that this fixes the issue for me when running from source using python bootstrap.py.
Good work!

Thanks @jitseniesen, you did pointed me already in the good direction with your comment on PyQt :smiley:

@tlunet Nice, to me it indicated that it was too hard a problem for me to solve, so I gave up!

Btw, should I make this parameter conditional to Linux so the MacOS test can pass ?

I don't see evidence that the MacOS failure has anything to do with your change; in fact, it's not clear at all why the test timed out. I am running it again, see whether it makes a difference.
Say hello to Daniel Ruprecht (I saw SDC and TUHH on your GitHub profile, so I assume you work with him).

Seems not linked to this PR, if I'm not mistaken ? See : https://github.com/spyder-ide/spyder/actions/runs/4088046116/jobs/7049275728

Seems not linked to this PR, if I'm not mistaken ?

Yeah, I don't think it is. I restarted that job to see if it passes a second time.

I had this issue and the PR solved it.
Spyder version:

Spyder version: 5.4.2  (conda)
Python version: 3.11.0 64-bit
Qt version: 5.15.6
PyQt5 version: 5.15.7
Operating System: Linux 6.0.12-76060006-generic

Linux Pop!_OS 22.04 LTS


Long strings on character sheet can throw the position calculation of the status bar

Need to look at the sheet CSS to see if there's a better way of doing the layout.

Test Webhook


[ServiceBus] Fix bug in converting ServiceBusMessage to RheaMessage

when the ServiceBusMessage has timeToLive property set.
The problem is that we are adding a number to a Date object, resulting in an
invalid date value for absolute_expiry_time. This PR corrects it by calling
getTime() on the Date object first before adding the ttl number.
A unit test is added and the sendMessages sample is updated to show how to set
timeToLive.
Packages impacted by this PR
@azure/service-bus

API change check
API changes are not detected in this pull request.


Add missing header guard for Interface.hh

🦟 Bug fix
Summary

Noticed it was missing during #248
Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[x] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.
🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸

CI error
libcurl: (51) SSL: no alternative certificate subject name matches target host name 'staging-fuel.ignitionrobotics.org'
  [Wrn] [FuelClient.cc:309] Failed to fetch models from server, returning cached models.
  URL: https://staging-fuel.ignitionrobotics.org/
  Version: 1.0
  API key:

CI error

I think this is due to changes in fuel.  In the future, it would be nice to decouple a lot of these tests from having a fuel server actually running.


After Update to 0.8.22 - no messages there

Hello,
after Update to 0.8.22 had the issue that old and new messages not show...
got notification that a new message is there... but can not see it...
here some screenshots, hope this helps to understand what I mean
https://www.bilder-upload.eu/bild-ce345f-1549609360.png.html
https://www.bilder-upload.eu/bild-f74993-1549609372.png.html
https://www.bilder-upload.eu/bild-478289-1549609387.png.html
https://www.bilder-upload.eu/bild-f9b0a6-1549609401.png.html
https://www.bilder-upload.eu/bild-76c360-1549609414.png.html

Sailfish OS <IP_ADDRESS> with Andorid-Support

Ok. I don't know what it means in term of Android version...
If you have the opportunity to use the last version of riot from Jenkins, the bug should be fixed.
Or please wait for official release.

Or please wait for official release.

Thats what in "Store" is offer
OK will ask Sailfish Support.
Thanks

@bmarty, @ds2k5 seems to be using a phone with SailfishOS, which is not Android but includes an Android compatibility layer. The Android API level supported by the layer depends on the device SailfishOS is running on. According to this answer (from some developer I guess, though I found other answers in other threads stating similar info), the Jolla 1 phone seems to support Android 4.1.2 (Jelly Bean), the Jolla C and Xperia X support Android 4.4.4 (Kit Kat).
I'm also using a Jolla 1 and experienced the same "blank message list" as @ds2k5 when I upgraded to Riot 0.8.22 from F-Droid. I was able to downgrade 0.8.21 and I got the messages back (and nothing seemed to break in the operation).

Ok, thanks @ivilata so this is well a duplicate of #2917


Can't install dm-robotics-moma and dm-robotics-manipulation

Hi, I got this message when I attempt to install dm-robotics-moma/dm-robotics-manipulation.
ERROR: Could not find a version that satisfies the requirement dm-robotics-controllers (from dm-robotics-moma) (from versions: none)
ERROR: No matching distribution found for dm-robotics-controllers
I think the issue comes from the fact that dm-robotics-controllers are not fully in python mode. If I drag the whole dm_robotics package to my local directory, where I also extract the controllers folder and put it in the dm_robotics folder, it still doesn't run since files such as cartesian_6d_to_joint_velocity_mapper are written in C++. Is it possible to have a complete python version of the dm_robotics package as soon as possible since I'm working on a robotic project and the deadline is really close. Thanks for your understanding!
FYI I use macOS.

Hello,
We don't currently support MacOS. This is likely to change in the future, but probably not on the timescales you're focused on.

Hello, again. My peers and I are really confused about extracting the action space information from the environment i.e.  env = environment.rgb_stacking(observation_set=environment.ObservationSet.VISION_ONLY, object_triplet='rgb_test_triplet1'). Can you provide any guidance on that? env.action_space is not the right way to do it obviously.

A bit sorry to hear about that, but fortunately it works pretty well in ubuntu. Thx for your reply anyways!

I have the same error on Windows 10. Do you have any plans to publish Win10 wheels in the future?

When dm_control support Windows, then we will be able to give a timeframe for us to also support Windows.
I can't make any promises about when that will happen, though.

Hi! Thanks for sharing a great repo! Any updates regarding the mac os support?

Sorry not yet, but we are discussing this.

Hi! When I tried to do pip install dm-robotics-controllers on my win11 machine, almost the same error message appeared. I was able to install the other dm-robotics libraries like geometry and transformations as well as dm_control. Is this controllers library not available on windows yet?

Hi, the controllers library is not available in Windows. Unfortunately, we cannot give you a timeline of when it will be available in Windows.


[WIP] Modularization of PBstat.py

This pull request aims to modularize the code of PBstat.py and create an API (as proposed in #25) to use it as a library.
This is a Work In Progress so don't merge it until the work is done.
The goal is to create visualization functions (neq, map and weblogo) that use as an input the output of count_matrix in PBlib.py so an user can :
   # An example
    import PBlib as PB
    chains = PDB.chains_from_files(["demo2_tmp/psi_md_traj_1.pdb"])
    seqs = []
    for comment, chain in chains:
        dihedrals = chain.get_phi_psi_angles()
        sequence = PB.assign(dihedrals)
        seqs.append(sequence)
    pb_count = PB.count_matrix(seqs)
    neq = PB.compute_neq(pb_count)
    PB.plot_neq(neq, "psi_md_neq.png")

What do you think ?
So far, I have only split the module into functions to avoid floating code.
ping @jbarnoud

Seems good to me @HubLot !

Quite a huge commit here. The details are in the commit message but here a few more explanations:
Now, all the  visualization functions (neq, map and weblogo) have been moved to PBlib.py and use as an input the output of count_matrix in PBlib.py so an user can (as state previously):
   # An example
    import PBlib as PB
    chains = PDB.chains_from_files(["demo2_tmp/psi_md_traj_1.pdb"])
    seqs = []
    for comment, chain in chains
        dihedrals = chain.get_phi_psi_angles()
        sequence = PB.assign(dihedrals)
        seqs.append(sequence)
    pb_count = PB.count_matrix(seqs)
    neq = PB.compute_neq(pb_count)
   # Modification of the parameter's order
    PB.plot_neq("psi_md_neq.png", neq)

I have also added to functions to the library :
compute_freq_matrix which compute a frequency matrix needed for neq and map from an occurrence matrix. It's called from the visualization functions to be transparent for the API user.
The main issue I encountered is how to deal with the residue-min and residue-max options from the PBstat.py CLI and how to slice correctly the values for the visualization functions. Because from an API user, the PB.count_matrix() doesn't deal with residues indexes. The new function _slice_matrix(matrix, residue_min=1, residue_max=None) aims to resolve this problem by slicing a given matrix with the boundaries given in parameters. It ensures the boundaries is good and returned the sliced matrix.
This function is only called by the visualizations one (write_neq, plot_neq, plot_map) because it's not worth it to handle sliced matrices (with correct offset and max boundaries) for the whole API.
Now, the PBstat.py is shorter and doesn't differ from the API.
However, the creation of the weblogo image is still done with the old way (that's why there is a check_residue_range function inside the module. The reason is that I would like to use the weblogo API instead of the binary to generate images (see #78)

:+1:
At one point we should split PBlib into several files. I do not like to require matplotlib to do a PB assignation. This is out of the scope of this PR, though.

The last commit solve the issue #78. Now, the weblogo images are generated from the API of weblogo.
The call to the function generate_weblogo is no different from the API.
This function and read_occurence_matrix have been moved to PBlib.py
With this last commit, the PR can now be reviewed and merged if it's ok.
I agree with @jbarnoud, now that almost all functions are in PBlib.py, we need to split it in several files as proposed in #25.

Thanks guys!
Indeed PBlib is becoming bigger and bigger...


Add user parameter to vagrant_plugin resource

Added user parameter to resource to be able to install for specified users since plugins are stored in ~/.vagrant.d. The shell_out hacks are needed because $HOME is not set correctly otherwise.

I merged #14.


reorganized docs superstaq and deleted unnecessary application examples

reorganizing the files in our apps directory

This file simply moves things around in anticipation of https://github.com/Infleqtion/client-superstaq/pull/558

There will be an odd instance when building the html where when clicking into the max sharpe ratio example, the page seems to expand a little bit. I will be editing that file in #558 to alleviate this issue.


Fix locale and language-dependent code in the product

The language pack recommender, especially on Windows, relies on the app.getPreferredSystemLanguages() API in order to properly recommend language packs.
The usage of that API has been temporarily reverted to fix the regression https://github.com/microsoft/vscode/issues/166936. The revert should also incidentally fix https://github.com/microsoft/vscode/issues/168583 and https://github.com/microsoft/vscode/issues/167871.
This issue looks into using that API again and the processing needed on the output to ensure that the results do not break any areas outside of the language recommender.
TODO

[ ] Determine which users of platform.locale actually require the system locale, and which ones merely require the language variable.
[ ] Determine how the language variable is actually set. platform.locale is set either through app.getLocale() or app.getPreferredSystemLanguages()[0].
[ ] Modify the code as needed

Determine how the language variable is actually set. platform.locale is set either through app.getLocale() or app.getPreferredSystemLanguages()[0].

language is set when you install a language pack or it's set to en by default.


[#5619] feat(api): supports credential api for Gravitino

What changes were proposed in this pull request?
This is the first PR of credential vending for Gravitino server to add credential API , please refer to #5682 for the overall workflow.
Why are the changes needed?
Fix: #5619
Does this PR introduce any user-facing change?
no
How was this patch tested?
using draft PR to do E2E test

@jerryshao @yuqi1129 please help to review this, thanks

@jerryshao  could you help to review this, thanks

I'm fine with the current implementation.

@jerryshao  do you have time to review this PR?

@FANNG1 can you please update the PR description to reflect your updated API?

@FANNG1 can you please update the PR description to reflect your updated API?

done


Fix wrong handling of java and slave option arguments in powershell script

In the powershell init script there is a wrong if statement.
The configured JAVA_OPTS and SLAVE_OPTS are not added to the Invoke-Expression call

Codecov Report

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

@@            Coverage Diff            @@
##             master     #235   +/-   ##
=========================================
  Coverage     20.39%   20.39%
  Complexity      245      245
=========================================
  Files            72       72
  Lines          2902     2902
  Branches        273      273
=========================================
  Hits            592      592
  Misses         2246     2246
  Partials         64       64

Continue to review full report at Codecov.

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


Implement Math.round using floor on GPU

Spark and libcudf round are more like rould half away from zero
Reimplementing Math.round(x) for Orc Timestamp-from-Double conversion using floor(x+0.5)
Fixes #10017
Signed-off-by: Gera Shegalov<EMAIL_ADDRESS>

build


Licensing issues

in this comment @briansmith mentions that docs.rs isn't following licenses properly.
https://github.com/briansmith/ring/blob/master/LICENSE is *ring*'s license.
I would imagine that it'd be a good idea to turn off generation for any crate that has something like this in its Cargo.toml.

I've been reading LICENSE of ring since @briansmith mentioned.
I am not sure how docs.rs is breaking license terms of ring. docs.rs is also hosting a copy of LICENSE. AFAIK ISC license is similar to MIT and simplified BSD licenses.

I'm not sure either, but I still think this is a good idea overall.

docs.rs is also hosting a copy of LICENSE.

I see. For some reason I'd overlooked it.

I'm not sure either, but I still think this is a good idea overall.

I think, at a minimum, it's a good idea to more prominently display or link to the license, ideally near where "Author" is displayed.
I think there's a broader issue of whether MIT, Apache, ISC, and other licenses require the copyright notice and license in each .rs file to be copied into the HTML, especially when the copyright notices in the .rs files aren't all uniform (e.g. different copyright holders) and/or when they are different from what's in the LICENSE file. Note, in particular, the top line of the ring LICENSE file.

Triage: This could be part of #740 as templates are passed license files so a License tab could be made that renders the given crate's license


Tool crashes during 'Processing'

Hi! We stumbled across this tool and are excited to try and implement it, so thank you so much for creating this resource.
I am working on getting it working in our Canvas instance, but the tool crashes right after the 'Processing' pop-up with the gif shows. I can confirm...

No pages are being created
I have-double checked with BlackBoard that our Ally ID and keys are correct.
I'm editing the action.php and js file within Brackets for Mac
Adding the appropriate file locations of these files into the config section of the js file.
Files are hosted on our web server, which does have PHP. Because the issue starts once it needs to use PHP to process the script, I surmise this might be it. I am investigating the possibility of a hosting error but at the moment will assume it works. (Updated: PHP issues resolved and left with code issue)

I've attached the code snippet appended to our Canvas JavaScript file, as well as a brief video of the error in question. Any help would be appreciated.
https://user-images.githubusercontent.com/121257531/209209172-dafebd0c-b3ee-49de-b68d-c49158a623de.mov
Code snippet:
$.getScript("https://my.uiw.edu/_dev/ally-api/js/convert.js", function () {
console.log('convert.js loaded');
});
// Convert Ally PDF HTML to Canvas page
$(window).on('load', function () {
if (($.inArray('admin', ENV.current_user_roles) !== -1 || $.inArray('teacher', ENV.current_user_roles) !== -1) && document.location.href.indexOf('/files') > -1) {
$.getScript("https://my.uiw.edu/_dev/ally-api/js/convert.js", function () {
console.log('PDF to Canvas Page Ran');
});
}
});

Update: I have worked with our network team to fix some of the errors and view the logs, so now, the only error I receive is the one below.

Thank you for reporting this issue - we will look into it, but it may take us a few weeks. I will report back if we can identity a solution.

Our developer has reviewed the issue and he indicated that the issue was that for some reason the JavaScript was not finding the Dom element that we use titled "ef-item-row". I don't know why that would be working in our instance, but not in yours, but perhaps that is helpful in looking at it. At this point we have limited resources to do much work on this - but hope to revisit it and potentially spend some more time on it in the future. Sorry we can't be more helpful.


Update harness-platform-faqs.md

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

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

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

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


Cannot Load MLKR & MMC, everything else is fine

python2.7 64bit, windows7
metric_learn v 0.3.0
This does not work for me:
from metric_learn import *
mod = MLKR()
error:
NameError: name 'MLKR' is not defined
Any ideas?

MLKR and MMC are new since version 0.3.0.
I hope to release v0.4 sometime soon, but until then you can try upgrading to the version currently on the master branch:
pip install https://github.com/all-umass/metric-learn/archive/master.zip


Warn developers if they forgot to include css file

I recently found myself using leaflet and it not working the way I wanted. I figured out pretty quickly that this was because I forgot the css file.
I figured that I could add a simple check to see if the .leaflet-map-pane-CSS class is defined and if not warn the developer on the console that they forgot to include the file. Would you accept such a feature or do you think that it would bloat the code base too much?

Yeah, that could be useful if it can be done in just a couple lines of code.

As far as I can tell this should do the trick:
function isLeafletCssLoaded() {
  for (var i = 0; i < document.styleSheets.length; i++) {
    var rules = document.styleSheets[i].rules || document.styleSheets[i].cssRules;
    for (var x in rules) {
      if (typeof rules[x].selectorText === 'string'  && rules[x].selectorText.indexOf('.leaflet-map-pane') !== -1) {
        return true;
      }
    }
  }
  return false;
}

if (!isLeafletCssLoaded()) {
  throw new Error('Leaflet css class not found. Did you forget ton include the css?');
}

Not the prettiest/shortest function in the world but it does the right thing. One could of course also just check if  leaflet.css is in the loaded stylesheets but people could always change the name. So I found that checking for the CSS class is the best option and this seems to be the way to do it.
I'm just not sure where to include it. Directly in the initialize function in  Map.js just before everything else is executed? Or where would be the appropriate place to put this?

@k-nut does that mean it would be ran everytime you load the page? What happens when there is like a ton of CSS stylesheets included? Wouldn't it slow you down a bit? What if someone decides to use @import statement instead of linking leaflet css directly?

Thats right. It would of course have to be loaded every time. And you are also right that this could become a performance issue. I guess we could also wait until the map has loaded and then check for something that we know. Something like:
document.querySelectorAll(".leaflet-container")[0].style.length > 0
Do you think that could work? It should be a lot better on the performance side I guess.

I'm not really sure that Leaflet should deal with missing stylesheet. It might be better to tell people (via the docs) that if they don't include the stylesheet, something really ugly is going to happen.

I don't really think we need this in leaflet.
If you miss the CSS it's going to be really broken and discovered at development time.

I tend to agree. Not worth the extra code IMO.

Same here.


PDI Plugin OSGi Bundles are not loaded

PDI Plugins such as

pentaho-marketplace-di
repositories-plugin (7.0 or higher)

are not loaded.
This is because Karaf is not launched.

Resolved by 1822bc33d1ceb575246e3e9d183016eb29aa4665.


Users are re-added to gather-users plugin the week after they remove themselves.

Two of my users have reported unsubscribing from the gather-users plugin using /gather-plugin off, I have confirmed this with each of them using /gather-plugin info, which showed that they were no longer on the list.
However, when monday rolled around, we found that these two users had again been matched with someone for a meeting, and appeared as subscribed in the /gather-plugin info dialogue.

I really don't know how to reproduce it, I try different instances/configurations/user actions, let me know if you find any clues


Google Drive support?

Hi,
Thanks for the really useful app!
Is it possible to do this with Google Drive as well?

Ah, I figured out a way. I just pointed to drive.google.com instead of docs.google.com. However, creating a new doc seems to want to open a new tab that opens up in the browser.

This is actually a good idea and would help me a LOT with dealing with Drive since I refuse to sync it to my computer for HD space reasons and find it clunky in FireFox.
Also encountered the issue you had with the new document thing but don't have the time to fix it now.  Probably something around forcing targets to open within that window.
Closing this as per below and maybe we can pick up at the link below?
Made a new repo here


Update result-cluster.md

Updating documentation for clarifying common asks from partners/customers on the behavior of the result cluster relevance.

@jeffkizn request to review and approve the PR


CLIMATE-891 Make (non-ESGF and pydap) examples script python 3 compat…

CLIMATE-891 Make (non-ESGF and pydap) examples script python 3 compatible

xrange to range
urlretrieve
print()
raw_data()
lots of pylint

@MichaelArthurAnderson can you please update PR to resolve conflicts? Thanks

@lewismc conflict merged.  What an odd conflict as it seemed to be coughing on a blank line!


Error: During saving

I have this error during saving a PHP file :
Error detected while processing function phpcomplete_extended#checkUpdates..<SNR>52_updateLocalCache..<SNR>52_setClassData: line    5: E121: Undefined variable: g:phpcomplete_index

Hi @sducamp
As I know @m2mdas doesn't maintain this plugin anymore. You can try my one https://github.com/mkusher/padawan.vim

Thank you @mkusher I'm going to try yours!


add placeholder for handling image

This PR adds a placeholder to handle the image API.

@annsts I'm adding this for now so the front-end website does not break from the new image API.


Fixed removal of core policy bug, added support for existing lambdas and devices

As per option 2 in dzimine's response to issue #21 , added a way to distinguish between lambdas to be created and existing lambdas. Also stipulated that existing lambdas will not be removed.
In the main greengo.yaml, added an existing lambda called "HWBeta" (hello world beta). You must create this lambda before attempting to run greengo create.
You might want to remove my changes to the main greengo.yaml file for clarity or put them in separate directory.
Also added an example directory that contains an example for using greengo with a lambda using local resources (see the readme) and an example that creates devices. Also added a shadows example which makes use of device support to automate most of AWS greengrass getting started module 5.
Fixed a major bug where greengo failed to remove AWS's core policy. The previous version would not correctly store core information and errored out when trying to access this info to remove it. While core definitions were removed, the core IoT thing itself was not. The core information is now correctly stored in state.

Hi @QuinnCiccoretti thank you for the PR and apologies for not watching for Greengo closely recently.

No worries. I'm excited for V2!


components directory in Angular 1.4

It looks like the new Angular router calls controller/template pairings "components". Your current guide has  a components directory containing directives, how do you see that changing with 1.4?

The specific folder that @shock99 is talking about perhaps should have been called something else anyway. Maybe "WebControls", "Widgets", "Gadgets" or something that denotes their purpose.
There is a larger problem: the Angular team has appropriated the word "component" for a very specific usage. The guide in all other cases uses "component" in a generic sense to refer to anything registered with Angular's IoC.
Should the guide change that terminology? I looked at synonyms and couldn't find anything i liked better. I'm really stuck.
At a minimum we might make a note that distinguishes "component" (the everyday word) from "Component", the ng 2.0 specific term.
Thoughts?

My feeling: a style guide should recommend staying clear of words that have a meaning in the framework. I think most developer already avoid naming their stuff "module", "controllers", etc.
Sadly, I don't have a better name idea to use instead of component, though.
On the plus side, some of the other Angular words are quite easy to avoid, I have never once wanted to name anything in my software "transclude"  :-)

This guide is not about Angular 2.0, so I don't see any reason why this should be changed. It's clear that an Angular component before 2.0 is a thing specific to Angular like controller, module, factory, ...

agreed.

@johnpapa so this repo is only for Angular 1.x style guide? Is there a 2.0 style guide available or is it in the works? Thanks for all your work!


Commit

Describe your change:

[ ] Add an algorithm?
[ ] Fix a bug or typo in an existing algorithm?
[ ] Documentation change?

References

Checklist:

[ ] I have read CONTRIBUTING.md.
[ ] This pull request is all my own work -- I have not plagiarized.
[ ] I know that pull requests will not be merged if they fail the automated tests.
[ ] This PR only changes one algorithm file.  To ease review, please open separate PRs for separate algorithms.
[ ] All new Java files are placed inside an existing directory.
[ ] All filenames are in all uppercase characters with no spaces or dashes.
[ ] All functions and variable names follow Java naming conventions.
[ ] All new algorithms have a URL in its comments that points to Wikipedia or other similar explanation.
[ ] If this pull request resolves one or more open issues then the commit message contains Fixes: #{$ISSUE_NO}.

CodeScene PR Check

Warnings
Degrades in Code HealthRootPrecision.java degrades from a Code Health of 10.0 -> 9.66BubbleSort.java degrades from a Code Health of 10.0 -> 6.85

Code Health Delta Descriptions
RootPrecision.java Degradations: Primitive Obsession - new issueBubbleSort.java Degradations: String Heavy Function Arguments - new issueBrain Method (Change with 45 lines of code and increase with 13 conditional complexity (cc) in getPagedService: cc = 18, cc threshold = 9) - new issueExcess Number of Function Arguments (getPagedService has 7 arguments, threshold = 5 arguments) - new issueBumpy Road Ahead (getPagedService has 4 logical blocks with deeply nested code. Threshold is 1 (one) single block per function) - new issueDeep, Nested Complexity (Increasing with 2 nesting levels in getPagedService: depth  = 5 levels, threshold = 4 levels) - new issuePrimitive Obsession - new issue

Review of New Files
1 new files where the code health is above the lower threshold for new code (8.1). 0 new files below the threshold.


Missing type check for JSX inside const

I've been facing an issue recently where JSX inside a const isn't properly type checked as expected.
type TCounter = (props: {
  value: number
}) => ReactElement
const Counter: TCounter = ({ value }) => (
  <div>{ value }</div>
)

const SomeOtherComponent = () => (
  Counter({ value: '4' })
)

const SomeOtherComponent2 = () => (
  <Counter value='4' />
)

Here, SomeOtherComponent correctly tells me that there is an issue with the type of value, whereas SomeOtherComponent2 incorrectly passes. Do I need some additional configuration to have Flow support this use case, is it a bug, or is this not supported by design?

The issue here is that Flow doesn't currently support functional components in JSX. This is a new feature in React 0.14 and we simply haven't caught up with them yet. Closing this in favor of the more clearly described issue in #1081. Thanks!


Error on parsing SNMP message

I was working with a TCW241
TCW241-Ethernet-I-O-module-user-manual-R4.2.pdf
If i try to interrogate the states of the sensors using ".getAll" I get the following error:
Woops! An error occurred while parsing an SNMP message. :(
To have this problem corrected, please report the information below verbatim
via email to<EMAIL_ADDRESS>or by creating a GitHub issue at
https://github.com/calmh/node-snmp-native/issues

Thanks!

AssertionError [ERR_ASSERTION]: 48 == undefined
    at parse (C:\Sviluppo\GitLab\Clevermind\Cloud\services\Technology-SNMP\node_modules\snmp-native\lib\snmp.js:260:12)
    at Session.msgReceived (C:\Sviluppo\GitLab\Clevermind\Cloud\services\Technology-SNMP\node_modules\snmp-native\lib\snmp.js:411:15)
    at emitTwo (events.js:126:13)
    at Socket.emit (events.js:214:7)
    at UDP.onMessage [as onmessage] (dgram.js:658:8)

Message data:
    30 82 00 1b 02 01 01 04 06 70 75 62 6c 69 63 a2
    82 00 0c 02 04 5c 2c 8c 01 02 01 01 02 01 00
events.js:183
      throw er; // Unhandled 'error' event
      ^

AssertionError [ERR_ASSERTION]: 48 == undefined
    at parse (C:\Sviluppo\GitLab\Clevermind\Cloud\services\Technology-SNMP\node_modules\snmp-native\lib\snmp.js:260:12)
    at Session.msgReceived (C:\Sviluppo\GitLab\Clevermind\Cloud\services\Technology-SNMP\node_modules\snmp-native\lib\snmp.js:411:15)
    at emitTwo (events.js:126:13)
    at Socket.emit (events.js:214:7)
    at UDP.onMessage [as onmessage] (dgram.js:658:8)

Just followed the instructions and shared the problem.
Sensors are not present and problably this is the reason of such situation (maybe when missing the devide doesn't really handle correctly the SNMP standard )
Everything else works correctly.
I would like to be able to handle the error and ignore the OIDs which trigger it, instead of seeing the entire "getAll" explode ( the "abortOnError" options doesn't seem to work ).
Any help to understand it would be nice!
EDIT:
If I use ".get" instead of ".getAll" the problem doesn't occur.
The only difference is that with ".getAll" I use an array of OIDs; if I use ".get" I need to cycle over the array and do the single call for every entry (it's far from perfection but it works)

hhm, this sounds like a buggy SNMP implementation, given that get() works while getAll() fails. It would not be the first time that a SNMP implementation chokes on getNext()...
you should be able to catch the AssertionError and then you can take a look at the nolog branch, which is less noisy about these errors...

Thanks,
I can confirm it's a problem of the SNMP integration on the device.


[UNR-4795] Reset server movement data on local client possession.

SimPlayer are doing the hacky thing of replacing their PlayerController with an AIController. When that happens, the client's movement data is reset(timestamp goes back to 0), but the server data is still there, not knowing that anything happened client side. So to get them back in sync, I added a server RPC to reset the movement data on the server.
Changes to BP_SimulatedPlayerCharacter, with comments.

You may be able to remove the component enables and disabled now I think.

You may be able to remove the component enables and disabled now I think.

Unfortunately no, this is used to purge the buffer of outgoing moves.

Unfortunately no, this is used to purge the buffer of outgoing moves.


Is it possible to style labels container?

I would like to add some border-top to the labels container on the X axis. Is it possible to style such element?

That's not possible with styling on SVG elements. However, you could use draw events and draw this line using Chartist.SVG.


Task 1

@GOKULTVSN
Task -1
Code verified. provide a annotations and don’t access others folders in github its arasie PR conflcits


Error "address already in use" when trying to connect to google cloud SQL

Question
I'm connecting my NextJS app to my Cloud SQL instance. I've followed the documentation and I'm stuck in the following error.
Error: listen EADDRINUSE: address already in use /Users/josechavez/projectName/projectName/.s.PGSQL.5432
at Server.setupListenHandle [as _listen2] (node:net:1812:21)
at listenInCluster (node:net:1877:12)
at Server.listen (node:net:1976:5)
at node:internal/util:410:7
at new Promise (<anonymous>)
at Server.<anonymous> (node:internal/util:396:12)
at Connector.startLocalProxy (webpack-internal:///(rsc)/./node_modules/@google-cloud/cloud-sql-connector/dist/mjs/connector.js:207:22)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async connect (webpack-internal:///(rsc)/./src/app/lib/connect.ts:16:5)
at async prismaClientSingleton (webpack-internal:///(rsc)/./src/app/lib/prisma.ts:17:16) {
code: 'EADDRINUSE',

   errno: -48,
   syscall: 'listen',
   address: '/Users/josechavez/projectName/projectName/.s.PGSQL.5432',
   port: -1
}

Basically, the error is saying that the address is already in use, and I have no clue why. In the console, my instance seems to be correctly configured (with the basics configs, like public IP, etc.)
I've followed your examples on how to connect to the Cloud SQL
This is what my connection code looks like.
import { PrismaAdapter } from '@lucia-auth/adapter-prisma';
import { connect } from './connect';

const instanceConnectionName = process.env.GOOGLE_DB_CONNECTION_NAME ?? '';
const user = process.env.GOOGLE_DB_USERNAME ?? '';
const database = process.env.GOOGLE_DB_NAME ?? '';

const prismaClientSingleton = async () => {
  try {
    return await connect({ instanceConnectionName, user, database });
  } catch (error) {
    console.error(error);
  }
};

const globalForPrisma = globalThis as unknown as {
  prisma: any | undefined;
};

const { prisma, close } = await (globalForPrisma.prisma ??
  prismaClientSingleton());
const adapter = new PrismaAdapter(prisma.session, prisma.user);

export default prisma;
export { adapter };

if (process.env.NODE_ENV !== 'production') globalForPrisma.prisma = prisma;

Additional Context
I've tried with 5432, 5433, ... and so on. but nothing still.
I've also tried to use this whole path that is mentioned in the docs /cloudsql/INSTANCE_CONNECTION_NAME/.s.PGSQL.5432
But this time I get an error that says no access/view permissions, but I'm not sure if this is the way.
Someone please help me, Thanks in advance!

Hi @JoseChavez98,
Here's my best guess at what is wrong: The startLocalProxy() function opens and listens on a unix socket on the local host. If startLocalProxy() was called by multiple node processes running at the same time, then it would attempt to open same unix socket for listening more than once, resulting in the error Error: listen EADDRINUSE: address already in use for all but the first node process.
Is this a possible cause?
-Jonathan

Hi @JoseChavez98, I wasn't able to reproduce this behavior. Feel free to comment back if want more help.

Hello Jonathan, thanks for the help.
I'm not sure if your above assumption is happening. I'm just running the library as in the example you guys provided.
This is how I connect to the cloud.
import { PrismaAdapter } from '@lucia-auth/adapter-prisma';
import { connect } from './connect';

const instanceConnectionName = process.env.GOOGLE_DB_CONNECTION_NAME ?? '';
const user = process.env.GOOGLE_DB_USERNAME ?? '';
const database = process.env.GOOGLE_DB_NAME ?? '';

const prismaClientSingleton = async () => {
  try {
    return await connect({ instanceConnectionName, user, database });
  } catch (error) {
    console.error(error);
  }
};

const globalForPrisma = globalThis as unknown as {
  prisma: any | undefined;
};

const { prisma, close } = await (globalForPrisma.prisma ??
  prismaClientSingleton());
const adapter = new PrismaAdapter(prisma.session, prisma.user);

export default prisma;
export { adapter };

if (process.env.NODE_ENV !== 'production') globalForPrisma.prisma = prisma;

and the connect function is exactly like in your examples.
import { resolve } from 'node:path';
import {
  AuthTypes,
  Connector,
  IpAddressTypes,
} from '@google-cloud/cloud-sql-connector';
import { PrismaClient } from '@prisma/client';

export async function connect({
  instanceConnectionName,
  user,
  database,
}: {
  instanceConnectionName: string;
  user: string;
  database: string;
}) {
  const path = resolve('.s.PGSQL.5432'); // postgres-required socket filename
  const connector = new Connector();
  await connector.startLocalProxy({
    instanceConnectionName,
    ipType: IpAddressTypes.PUBLIC,
    authType: AuthTypes.IAM,
    listenOptions: { path },
  });

  // note that the host parameter needs to point to the parent folder of
  // the socket provided in the `path` Connector option, in this example
  // that is going to be the current working directory
  const datasourceUrl = `postgresql://${user}@localhost/${database}?host=${process.cwd()}`;
  const prisma = new PrismaClient({ datasourceUrl });

  return {
    prisma,
    async close() {
      await prisma.$disconnect();
      connector.close();
    },
  };
}

This is currently stopping me from moving forward. I will be happy if you let me add you to my project for testing purposes.
Thanks in advance!

I wonder if the cause of this error is maybe related to https://github.com/GoogleCloudPlatform/cloud-sql-nodejs-connector/issues/349

@jackwotherspoon I'm gonna say  maybe. I made some tests and I was able to successfully connect to my DB using the traditional connection (without the proxy). So the problem is related to that localProxy implementation/connection. Is there a chance We can have this looked up? Its a real deal breaker for me .
Thanks!

@jackwotherspoon I'm gonna say maybe. I made some tests and I was able to successfully connect to my DB using the traditional connection (without the proxy). So the problem is related to that localProxy implementation/connection. Is there a chance We can have this looked up? Its a real deal breaker for me .

@JoseChavez98 Yes something isn't working the way we would like it to. I will digging into this over the next day or so and fixing these issues 😄

thanks @jackwotherspoon, let me know if I can help with anything.

thanks @jackwotherspoon, let me know if I can help with anything.

@JoseChavez98 thanks! Will keep you in the loop if I have any additional questions 😄

@JoseChavez98 I was able to reproduce this error and now the cause of it makes sense to me. So let me try and explain what is happening to cause the address already in use error.
TLDR; The close() returned from const { prisma, close } = await connect({...}); must be called to  properly delete the unix socket file ending in s.PGSQL.5432. (we can probably update our docs/sample to have a try/catch to make sure the close() is being called)
Let me give a detailed explanation using the sample you used as reference:
https://github.com/GoogleCloudPlatform/cloud-sql-nodejs-connector/blob/dc7e0d0a23a419f539ce971350bf76694abcc4c8/examples/prisma/postgresql/connect.ts#L15-L46
In the sample connector.startLocalProxy(...) creates a local unix socket file at the location specified by path in the listenOptions. So in this example and yours it will create the .s.PGSQL.5432 file in the current working directory.
The connect function returns the Prisma client, along with a close() function that when called will disconnect the client as well as closes the Cloud SQL Connector object by callingconnector.close(). This last part is the critical piece that is resulting in your error.
If the close() is not called, the connector is not properly cleaned up. Part of the Connector's cleanup is to close and delete all local unix socket files. @JoseChavez98 from what I can see, your code never calls close() thus the local unix domain socket file is not being deleted, hence why on the next call/time your program is run you are seeing the error address already in use, because the file is still there from the previous run.
Steps to fix the error:

Delete the stale local unix domain socket file located at /Users/josechavez/projectName/projectName/.s.PGSQL.5432
Add the call await close() to your code. You will want to call it whenever you are done interacting with the Prisma client to disconnect the client  and close the connector cleanly. (Note: Probably will want to add a try/catch that also calls await close() when an error is thrown. This way the client and connector are also cleaned up when errors are thrown from the client and before you program exits.

I'll leave this issue open and take an action item for myself of updating our README, examples to clearly point out the close() must be called.

I confirm that that was the problem, thanks for the help @jackwotherspoon .
I know this might be unrelated, but I'm having another issue.
It looks like It starts the local proxy but didn't quite make it through the PrismaClient.
It feels like it's in a loop or if It is waiting for something.
Below is my modest way of debugging and the result in the console.

Logs

As you can see, It doesn't make it through the construction of PrismaClient object.
I'll appreciate your input again.
Thanks for all the help!

Thanks for all the help @jackwotherspoon .
To respond to my last question, There was a mismatch between prisma and prismaClient library versions. That was causing the weird behavior.

@jackwotherspoon thanks for the help.
I'd appreciate it if you could take a look at this question since it is slightly related to this issue. Or maybe you know someone who could help me.
Thanks in advance

I'd appreciate it if you could take a look at this question since it is slightly related to this issue.

I will take a look at this question sometime today or tomorrow at the latest 😄


Fix error InvalidMavenPublicationException

Description
Fix error org.gradle.api.publish.maven.InvalidMavenPublicationException:
Coming from the comment https://github.com/opensearch-project/notifications/pull/549#issuecomment-1278186297
Issues Resolved
https://github.com/opensearch-project/notifications/issues/501
Check List

[x] Commits are signed per the DCO using --signoff

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

Build pass on my local with this change
./build.sh manifests/2.4.0/opensearch-2.4.0.yml --component notifications-core, ./build.sh manifests/2.4.0/opensearch-2.4.0.yml --component notifications
maven folder with org.opensearch.plugin
pwd
/usr/share/opensearch/git/opensearch-build/tar/builds/opensearch/maven/org/opensearch/plugin
[opensearch@ee8276647b38 plugin]$ ls -ltr
total 8
drwxrwxr-x 3 opensearch opensearch 4096 Oct 13 23:49 notifications
drwxrwxr-x 3 opensearch opensearch 4096 Oct 13 23:50 opensearch-notifications-core

@prudhvigodithi This is interesting.
Right now I am not sure why this is happening. But I noticed one interesting this, without your modification from this PR I am still able to build both plugins individually like this:
# Let's get to root notification folder which hosts subprojects (the plugins)
$ cd notifications

# Start with the core plugin
$ cd core
$ ../gradlew clean publishToMavenLocal

# Switch to notification plugin and build it
$ cd ../notifications
$ ../gradlew clean publishToMavenLocal

And now I can see the following in my local mvn repo
$ pwd
/Users/lukas.vlcek/.m2/repository/org/opensearch/plugin

$ ls -ltr
drwxr-xr-x  4 lukas.vlcek  staff  128 Oct 14 20:57 opensearch-notifications-core
drwxr-xr-x  4 lukas.vlcek  staff  128 Oct 14 20:59 notifications

So both the plugins were build correctly (at least it seems to me).
Except I am not just sure why the notifications plugin is deployed into notifications folder and not into opensearch-notifications folder, but this might be different issue.
It feels to me that something is stepping on each other's toes when gradle is run from the parent folder (which I think is the case of the opensearch-build script).

Hey @lukas-vlcek thanks, if so we should consider modifying the notifications-core and notifications build scripts and execute the build workflow
Example:
./build.sh manifests/2.4.0/opensearch-2.4.0.yml --component notifications-core
./build.sh manifests/2.4.0/opensearch-2.4.0.yml --component notifications
Once this build successfully we can then remove the added setting startParameter.excludedTaskNames=["publishPluginZipPublicationToMavenLocal"] wdyt?
Thank you

The is also a nebule plugin upgrade PR going on https://github.com/opensearch-project/OpenSearch/pull/4810 (it might be relevant, or not... I do not know atm). Let's keep a reference this PR here (even if it is for main).

I had a short chat with @prudhvigodithi about this yesterday and the following is a summary:
TL;DR: We are ok for now to move forward with this PR as it is.
For now we are ok with the workaround provided in this PR as it is unblocking the build process. The side-effect of this PR is that OpenSearch plugins (opensearch.opensearchplugin) are not built for the publishToMavenLocal task but they are not needed for the local repo anyway (unlike to the staging repo), so this is acceptable compromise.
It would make sense to try to run the build for each individual plugin as suggested and demonstrated above because this does not lead to the exception and all plugins are built as expected. But that would require changes to the build scripts itself and that can be a risk that we do not want to take now given we are approaching 2.4 code freeze soon.


Add basic tag position to fire_hydrant

Disclaimer: I am new to this project.
This will add the basic tag fire_hydrant:position to emergency=fire_hydrant (source).
I am not sure if I should set "customValues": to true or false though.

Excellent, thanks! :+1:


Added CI tests for torcharrow

Fixes #301
Changes
-Added torcharrow config to domain_ci.yml

@ejguan sorry for the delay.Been a bit occupied for the past weeks. I have made the changes as you have requested.


Can change the synthethizer voice ?

i'm using your library, and is awesome, but i have a little question, is posible to change the voice?
Thanks for your reply.
Best regards.

Thanks!!!!


Validating client credentials against a "non current" AD.

I am able to authenticate and validate username / password against the current AD (I am authenticated to). But only for testing purpose, I'd like to force use another AD (not the one am authenticated against) with other credentials. Is that route possible in the current library?
From code perspective, I don't want to use a ServerCurrentCredential object because that will get populated with the current server (which I don't want to use). I have tried PasswordCredential too but that does require username and password which is not what we want from a server perspective. Ideally I would like to only point to a server (no password needed).
If that is not possible, I'll have to perform the test the right way: logon into another machine that is part of the test AD and run the code from there instead of running it from my dev machine.
Thanks,
Hisham

You want to be able to pretend to be another user, and convince other network services that you're another user, without ever logging in as that user?
If that were possible, then I could put my laptop on your network and pretend to be your CEO and take all of your company's money.
Do you not see the inherent mistake you're making?

I never said "without ever logging in as that user (either by... or by..)". I actually said the opposite and clearly stated: "I'd like to force use another AD with OTHER CREDENTIALS" (valid ones of course so there is definitely username and password here!).
So back to your reply: as per my understanding you cannot put your laptop on our network and pretend to be our CEO or any other employee because you will have NO valid username / password to validate against our AD.
Scenario (again):

Am logged in against my company's AD
I want to ignore the current server (and Kerberos token) and instead use my TEST username and password against our TEST AD (not the company's and of course we are authenticating)
When I get a Kerberos token from the TEST AD, I'll know the TEST username and password are valid (again not the company's so this is totally isolated)
As stated, this is temporarily so at least I can continue developing on my dev machine without having to deploy to a VM each time I have to test against the TEST AD. It'll save us a lot of time.

Microsoft's sspi api does not allow for authenticating against arbitrary AD servers, at least to the best of my knowledge. Feel free to browse the SSPI api and tell me how you'd expect to accomplish what you're trying to do natively.
You can call PasswordCredential providing any domain you want, so long as that domain is either the domain your computer is joined to, or is a domain trusted by the domain your computer is joined to.
https://social.technet.microsoft.com/Forums/en-US/48b6ecb5-2715-4af5-a7aa-d67148cdff1d/single-computer-login-to-multiple-domains?forum=winserverDS
A computer can be joined to only one domain at a time. Sspi operates within the context of the domain that your computer is joined against.

Thanks for your reply Kevin.
Yes of course a computer can be part of one domain only.
I think the second option (making sure the domain is trusted then switching to PasswordCredential) should do the job. In the meantime, I've switched to using a VM joined to the dev domain.
I have another question though which I am raising in a separate ticket.
Thanks.


OnBotTradeOffer() System.TypeLoadException: Could not load type 'EType' from assembly

ERROR ASF > OnBotTradeOffer() System.TypeLoadException: Could not load type 'EType' from assembly 'ArchiSteamFarm, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0'.
   at ItemDispenser.ItemDispenser.OnBotTradeOffer(Bot bot, TradeOffer tradeOffer)
   at System.Runtime.CompilerServices.AsyncMethodBuilderCore.Start[TStateMachine](TStateMachine& stateMachine)
   at ItemDispenser.ItemDispenser.OnBotTradeOffer(Bot bot, TradeOffer tradeOffer)
   at ArchiSteamFarm.Plugins.PluginsCore.<>c__DisplayClass30_0.<OnBotTradeOffer>b__1(IBotTradeOffer plugin)
   at System.Linq.Enumerable.SelectEnumerableIterator`2.MoveNext()
   at System.Threading.Tasks.Task.WhenAll[TResult](IEnumerable`1 tasks)
   at ArchiSteamFarm.Core.Utilities.InParallel[T](IEnumerable`1 tasks)
   at ArchiSteamFarm.Plugins.PluginsCore.OnBotTradeOffer(Bot bot, TradeOffer tradeOffer)

ASF <IP_ADDRESS>
ItemDispenser <IP_ADDRESS>
linux x64 build

Thank you for report! I'll update is ASAP.

Please check that v.<IP_ADDRESS> fixes this issue for you.

Hello, it's working as intended now.
Does this plugin have the "PluginsUpdateMode" feature enabled? Unsure how to implement it or if it's even supported.

Thank you for confirmation.
No, none of my plugins have auto-updates implemented yet. I plan to do it at least for some plugins, but don't have much time now to do that.


Improve CLI

Currently, there are some steps involved to generate proof in toy project setup done using cargo prove new

cd program
cargo prove build
cd ../script
cargo run --release

It would be great if a command like cargo prove done in top-level projects would do the above tasks, hereby improving the dev ex.
Related https://github.com/succinctlabs/sp1/issues/248

Hey @zemse! Thanks for raising this issue, we added a feature a few months ago automatically builds the program when you call cargo run --release in the script directory using a build file that wasn't documented in the quickstart. The above PR updates that in the documentation, but you can use that already today.


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Correct calculation of the erase sector size

According to the documentation (page 95) you should add one to the value extracted from the CSD to obtain the real erase sector size.

@LemonBoy - Thanks for the PR. Sector size should be incremented by 1. But seems erase size is still incorrect for the CSD Version 1 cards.
Erase size should be _erase_size =(BLOCK_SIZE_HC * (ext_bits(csd, 45, 39) + 1)). For explanation please refer to SD card specification Pg 94
We do not have CSD version 1 card, Can you please update and verify.

Alright, I've been fooled by the use of the byte offsets.
I think that's why the erase+program sequence ends up trashing the whole filesystem, there should be a way to ask the FS driver not to do so when the single-block erase isn't supported.


Preliminary Couchbase QScript planner

Stated and unstated ToDos remain. A few QScript terms are unimplemented due to recent arrival or unclear mapping to N1QL.
Addresses: #1321 and part of #1322

Can one of the admins verify this patch?

Looks like the build failure is related to a post merge to master issue since it compiled successfully for a few in the matrix over here. I'll merge in master.

Current coverage is 63.88% (diff: 40.71%)

Merging #1563 into master will increase coverage by 0.51%

@@             master      #1563   diff @@
==========================================
  Files           326        340    +14
  Lines         15369      15624   +255
  Methods       14860      15341   +481
  Messages          0          0
  Branches        151        144     -7
==========================================
+ Hits           9740       9982   +242
- Misses         5629       5642    +13
  Partials          0          0

Powered by Codecov. Last update fcf59f7...8b8da05

Current comments have been addressed with commits or issues.

I'll merge when the build passes.


HumanTaskServicesBaseTest: use null jurnal for bitronix to avoid inte…

…rruptep thread destroying transaction manager when using default disc journal

Please cherry-pick this into 6.3.x as well, if accepted.

Looks fine -- although I don't know anything about the problem. Could you explain what the problem is?
If you could also add it to the persistence test classes in drools-persistence-jpa, that would also be nice -- if it's applicable there.
Thanks.

TBH I don't really know that much about this. I just copied the changes @mswiderski did here https://github.com/droolsjbpm/jbpm/commit/ae6c2de28c231893fe5c270d72b584d624338fad (after checking with him). So this or may not be applicable to other tests (don't know).

in general for test purpose we can switch all tests to use NULL journal instead of disc one which is fragile to thread interruption. @psiroky would you mind extend this PR to include others as well?

@mswiderski what others do you mean? All the top-level/abstract test classes?

yes, all base classes for modules starting at jbpm-persistence-jpa and up in maven module hierarchy

Ouch. That will likely take more than few minutes (at least to me). I can take a look at that, but now sure when.

no worries, whoever gets into this earlier then will do it

closing this one as #314 has been merged, thanks @psiroky


Add LICENSE info to the README.md

Add LICENSE info to the README.md

@leggetter Can you please review it?

We have the license file, this is not required


Date picker breaks on Mozilla firefox 70.0.1 macOS

Browser: Mozilla firefox version: 70.0.1
OS: macOS 10.13.6

Date is not selected
Calendar incorrect

Working fine on my end (same browser/OS). View the console to see what errors are being thrown (if any).

I upgraded my Firefox version to 71.0. I can still reproduce this

Also I don't see any console errors but warnings

This could be one of the extensions that you're using. 😄

Closing this issue, this was due to an add on that I installed "Change Timezone(Time Shift)"


Block drag outside not firing when below the toolbox

Expected Behavior
Please describe what should happen
The "drag outside" event should fire when the dragged block leaves the blocks area, including below the toolbox.
Actual Behavior
Describe what actually happens
Dragging over the toolbox and down off the workspace, specifically for dragging into the backpack, does not fire the drag outside event.
Steps to Reproduce
Explain what someone needs to do in order to see what's described in Actual behavior above

Turn on "log events" in the vertical playground, and watch the events when dragging below the toolbox vs. below the main workspace.

The cause is in this function, WorkspaceSvg#isInsideBlocksArea which returns true if the event is over the delete area OR over the blocks area. The delete area is registering as "infinitely" tall, instead of its normal height. This seems like it was clearly a conscious choice, so I don't want to mess with that without knowing the history. But we do need to be able to drag a block off the workspace into a spot directly below the toolbox, so this logic may need to change.
/cc @picklesrus or @rachel-fenichel, maybe one of you could shed some historical light on the infinite size of the delete area, and what to consider when fixing this?

Sounds like you've found all of the relevant code.  Blockly makes the delete area extend up and down to make it easier for users to find, but in this context it's the wrong behaviour.  Changing the delete area for the toolbox/flyout will work fine.
Note that the delete area also extends pretty far to the left of the actual flyout and toolbox.  If you're putting the workspace in a context with more drag targets to the left, you'll also want to change that.


Duplicated internal and external

Devices that share an internal and external storage show the same value for both fields
http://images.tapatalk-cdn.com/15/04/21/95a0a0e516fa5adada7a71c6c7040995.jpg

Since newer devices share on storage system, I'll change the options of external and internal to just storage, and will show both if I detect the values different

Had to go to class, see description of 8035d3398cdce73ce0ff0331efecab249752eb39 to know how to finish up and test. also, Might I suggest creating an another view that says Used storage if we cannot simply change titles of our views

This has been completed as of 99bcee2c017f6eb021a6ec274c4136e7b1494a45


Migrate geometry intersection to dask geopandas

After tests, there is a problem with newest dask version (https://github.com/geopandas/dask-geopandas/issues/284) as well as problems with compatibility with python 3.11 and 3.12.
Monitor changes to dask geopandas repository.
Things to do: reuse points_in_polygon example from https://github.com/geopandas/dask-geopandas/pull/165

Replaced by STRtree with multiprocessing


Packages status?

I am now using this package to mange Heralds server side media :)
I would love some of the tools in your TODO. What is your timing looking like? Are you still actively working on this?

I've added instructions on the README to include how to use houston:admin to run/edit tasks in the queue.  Didn't think it merited a version change, but thought you might be interested.

That is a fun trick. I will play around with it soon, thanks for the tip.

I also hoped to a put a "purge all" button, at least on the log collection, but I see that that is a feature request on houston:admin, so will probably work towards that upstream.


Settings: changes needed

The new Settings tab needs some help. There are a few issues that make its behavior problematics:

Parameters are saved both in the session file and a .vapor3_settings file. Its not clear how they interact

Some of parameters only take effect after a restart. The current mechanism for notifying the user of this does not make this clear (changing text to yellow and adding a Qt tooltip that may or may not be seen)

There is no way to restore the system defaults, and thus get back reasonable values.

I'm not sure I understand issue 1.  2 and 3 should be fixed now though.

I'm going to close this since I think all of the issues have been addressed.
Regarding item #1, there should be no session file modifications done from the Settings tab.


Add compact variant to Textfield

Ready for review @lucasveil .
I ended up not doing a compact version of the multiline field, felt it didn't make much sense...

@BirteThornquist Looks like the compact version is using the wrong color on "input text" ( placeholder color text). It has default on comfortable enabled and tertiary on compact enabled.

I cannot see that @mimarz? The input text for both default and compact in Figma is tertiary...?

I cannot see that @mimarz? The input text for both default and compact in Figma is tertiary...?
ahh, yeah. I was looking at the colored ones; error, warning & success. They all seem to have the same issue...

Great job @BirteThornquist ! Variant are implemented well. My only concern is actually with the multiline. In compact, if you select multiline, the first line looses it initial height:

That's a good point @lucasveil. I'll create compact variants of the multiline as well 👍

I cannot see that @mimarz? The input text for both default and compact in Figma is tertiary...?

ahh, yeah. I was looking at the colored ones; error, warning & success. Only these seem to have this issue...

Good catch @mimarz. Fixed. There was also a mix between dark/light mode where secondary color was used in dark mode while default in light mode for both default/compact. Changed to primary color for both.

Ready for review here @pomfrida @mimarz

Ready for review here @pomfrida @mimarz

@BirteThornquist Looks good. Only the height was adjusted? Should be ok.
With icon the field is very compact, but I guess we have to live with that 🙈

Yes @mimarz I agree about the icon. We decided to keep the icon size for now, but it could be we need to look into the icon sizes again...


Use explicit scheme names

When built with Carthage, the scheme building step lists it as Building Scheme "iOS" in ISO8601.xcodeproj. Most of the other projects in my Cartfile use the pattern of ProjectName-Platform (including your own Crypto framework) or ProjectName Platform. To that end, I've renamed the schemes to
+ISO8601-iOS
-iOS

+ISO8601-OSX
-OSX

Closing due to age

The only place that name shows up is in that log for what it's worth.


Bug: uncaught RuntimeException in deletes even when a session exists

I cannot remember the reason why UniqieList#remove requires equals equality. If it is not needed, then we may fix this issue by updating the UniqueList#setElement to compare by isSame.

The bug turned out harder than expected to fix because the issue was on ModuleClass becoming immutable, so removeSessions calling removeSession multiple times on the same ModuleClass object does not work.


Added support for multiple streamdecks

Added support for multiple streamdecks

Looks good.
One thought I had was that you could pass StreamDeckInfo instead the string serial, but that's just a style thing I suppose.

Yeah I just wanted to make it easy to use from the command line or any tool anyone else wants to use, so just passing the serial is a bit easier


Im  getting "Error from server: admission webhook "mutate.kyverno.svc-fail" denied the request: mutation policy check-images error: failed to apply policy check-images rules [call-aws-signer-extension: failed to evaluate list response.results: failed to fetch data for APICall: HTTP 406 Not Acceptable: failed to verify container" when I try to run this on playground namespace on eks fargate.

Im trying to implement this to our eks cluster which is run on fargate. Could you please help me to find the issue here ?
Error:
kubectl -n playground run test --image=1234567899.dkr.ecr.ap-southeast-1.amazonaws.com/mailbox-processor:latest-dev Error from server: admission webhook "mutate.kyverno.svc-fail" denied the request: mutation policy check-images error: failed to apply policy check-images rules [call-aws-signer-extension: failed to evaluate list response.results: failed to fetch data for APICall: HTTP 406 Not Acceptable: failed to verify container nphc-claimsflow-mailbox-processor: failed to verify image {{1234567899..dkr.ecr.ap-southeast-1.amazonaws.com mailbox-processor mailbox-processor latest-dev } /spec/containers/0/image}: signature verification failed : failed to fetch data for APICall: HTTP 406 Not Acceptable: failed to verify container nphc-claimsflow-mailbox-processor: failed to verify image {{1234567899..dkr.ecr.ap-southeast-1.amazonaws.com/mailbox-processor latest-dev } /spec/containers/0/image}: signature verification failed
Trust Policy:
`apiVersion: notation.nirmata.io/v1alpha1
kind: TrustPolicy
metadata:
name: trustpolicy-sample
spec:
version: '1.0'
trustPolicyName: tp-playground
trustPolicies:

name: aws-signer-tp
registryScopes:

"*"
signatureVerification:
level: strict
override: {}
trustStores:
signingAuthority:aws-signer-ts
trustedIdentities:
"arn:aws:signer:ap-southeast-1:1234567899:/signing-profiles/1234_signer"

apiVersion: notation.nirmata.io/v1alpha1
kind: TrustPolicy
metadata:
name: trustpolicy-sample-fail
spec:
version: '1.0'
trustPolicyName: tp-playground-fail
trustPolicies:

name: aws-signer-tp
registryScopes:

"*"
signatureVerification:
level: strict
override: {}
trustStores:
signingAuthority:aws-signer-ts-fail
trustedIdentities:
"arn:aws:signer:ap-southeast-1:1234567899:/signing-profiles/1234_signer"`

Kyverno-policy
`apiVersion: kyverno.io/v1
kind: ClusterPolicy
metadata:
name: check-images
spec:
validationFailureAction: Enforce
failurePolicy: Fail
webhookTimeoutSeconds: 30
schemaValidation: false
rules:

name: call-aws-signer-extension
match:
any:

resources:
namespaces:

playground
kinds:
Pod
operations:

CREATE
UPDATE
context:

name: tlscerts
apiCall:
urlPath: "/api/v1/namespaces/kyverno-notation-aws/secrets/svc.kyverno-notation-aws.svc.tls-pair"
jmesPath: "base64_decode( data."tls.crt" )"
name: response
apiCall:
method: POST
data:

key: images
value: "{{images}}"
key: trustPolicy
value: "tp-{{request.namespace}}"

- key: attestations
value:
- imageReference: "*"
type:
- name: sbom/example
conditions:
all:
- key: {{creationInfo.licenseListVersion}}
operator: Equals
value: "3.17"
message: invalid license version
service:
url: https://svc.kyverno-notation-aws/checkimages
caBundle: '{{ tlscerts }}'
mutate:
foreach:

list: "response.results"
patchesJson6902: |-
- path: {{ element.path }}
op: replace
value: {{ element.image }}`
Service Account Permissions

    {             "Action": [                 "ecr:UploadLayerPart",                 "ecr:PutImage",                 "ecr:InitiateLayerUpload",                 "ecr:GetDownloadUrlForLayer",                 "ecr:GetAuthorizationToken",                 "ecr:DescribeImages",                 "ecr:CompleteLayerUpload",                 "ecr:BatchGetImage",                 "ecr:BatchDeleteImage",                 "ecr:BatchCheckLayerAvailability"             ],             "Effect": "Allow",             "Resource": "*",             "Sid": "ECR"         },    {             "Action": [                 "signer:SignPayload",                 "signer:ListTagsForResource",                 "signer:ListProfilePermissions",                 "signer:GetSigningProfile",                 "signer:GetRevocationStatus"             ],             "Effect": "Allow",             "Resource": [                 "arn:aws:signer:ap-southeast-1:1234567899:/signing-profiles/1234_signer",                 "*"             ],             "Sid": "signerpermission"         }

Sorry for the late reply @lahirushanaka
The error here seems to be that signature verification has failed

Is the image signed?
Are you using the right certificate
Are the trust policies and trust store configured properly?

@lahirushanaka - any updates on this?

Hi @JimBugwadia and @vishal-chdhry , Sorry for the delay.
This can be ignore as we have discussed this with AWS support. as per them AWS signer is not supporting for the private clusters. which mean it need public access to verify the images.

@lahirushanaka - by "private clusters" do you mean your cluster has a private endpoint?

Confirmed that Signer currently does not currently support a VPC endpoint.

yeah, aws Signer dosen't have vpc end points. Since my cluster has been deployed on private subnet, nodes cannot connect to public singer endpoint.


Scaling to zero using the faas-idler

Hi,
What is the timeline of adding faas-idler to faas-nomad? if this is going to take so long, is there a workaround solution for scale to zero when for inactivity when deploy OpenFaaS to Nomad ?

Hi @mabuaisha, thanks for reaching out:
The faas-idler should work with the faas-nomad provider as is since the provider itself only deals with CRUD operations to the containers.
Looking at: https://github.com/openfaas/faas-netes/blob/master/yaml/faas-idler-dep.yml we could port this over to Nomad HCL and since the ideler only needs access to the gateway and provometheus, the gateway will invoke the nomad-provider's delete handler etc.
In short, no changes are required on the provider to get this working.

@acornies Thanks for reply, Glad to hear that, If I understand you correctly then all requried is try to build a Nomad HCL from https://github.com/openfaas/faas-netes/blob/master/yaml/faas-idler-dep.yml  and deploy it to Nomad cluster and it will be fine
Actually I'm new to Nomad and will try to port this it HCL, but if you could guide me here that would be helpful

Take a look at: https://github.com/acornies/faas-nomad-demos. I have a few HCLs for the OpenFaaS project and this would be a good addition!

@acornies Thanks, sure will take a look there


multiple languages support?

I think it is a very useful add-on. However, This add-on seems to be limited to single-byte characters.

Is it possible to support multiple languages?

It would be useful to be able to open a new page without closing the search results with a middle click.
Thanks!

Almost solved,

I can use double-byte character in "PRECISE" mode
Ctrl+Enter works well in this situation

Hi @ojarux : Could you try this option?
# make CJK chars work for fuzzy search
uFuzzyOptions:
  interSplit: (p{Unified_Ideograph=yes})+

See also this issue: https://github.com/Fannon/search-bookmarks-history-and-tabs/issues/87

I've seen that you use the hybrid search mode. Does it give you any benefits?
It was added at a time when the fuzzy search was not giving as relevant results as nowadays, so combining it with precise seemed a good workaround. But now fuzzy search got better and I'm not sure whether it makes sense to keep hybrid around.

Thank you for your prompt reply.
As I already mentioned, I was able to avoid the problem by setting the search mode to "PRECISE".
I also tried uFuzzyOptions and it works, however the result with or without fuzzy search are the same.
I think it is the best of the similar add-ons!
If I may make a request, it would be great if the display space could be widened and the font size could be changed to improve the listability.

Thanks for the kind words and feedback!
Good point about increasing size in general. I'll think about that how that can be done best, as a browser extensions popup are not really "responsive" like you can make a website responsive. Right now it's a very fixed layout and it would be some effort to customize that. But I can think about making it generally a bit bigger.

I've reopened this issue because for the average user, it's unrealistic they'll look for and apply a custom setting to fix this.
But I also didn't want to apply the ideagraph setting generally, so I created a PR where this is only added when necessary.


MacOS: Accent popup position is still wrong

This should have been resolved by https://github.com/flutter/flutter/pull/77608, but for some reason FlutterTextInputPlugin doesn't store reference to the view controller, and the rectangle is not flipped correctly (needs to adjust for height).

Closed via https://github.com/flutter/engine/pull/25524


Fix broken links in README.md

Admin REST API & Documentation links were broken
Issue tracker link pointed to old repository

@maximRNBack thanks


fix phosphor fonts

Fixes #27
You can see an example of the broken font in #29

thanks! but I chose to go with bumping the priority down by one instead, since it'll avoid breaking the other icons again


[BUG] Management interface with a VLAN ID fails to do DHCP configuration in the interactive installation

Describe the bug

Management interface with a VLAN ID fails to configure in the interactive installation.

To Reproduce
Steps to reproduce the behavior:

Boot an ISO in a network environment that only supports tagged VLAN. No DHCPD in vlan1 or non-vlan network.
Create a management network with a VLAN ID, and set the method to DHCP.
The installation can't continue.

Expected behavior

The installation should continue.
Support bundle

Environment

Harvester ISO version: v1.4.0-rc5
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630):

Additional context

The function always sends testing DHCP packets without a VLAN ID, even the user configure a VLAN to the management network.
Supposed to be the cause of https://github.com/harvester/harvester/issues/3428, I'll leave https://github.com/harvester/harvester/issues/3428 open because Windows DHCPd is involved.

Test plan:

As per issue description, install harvester via ISO, and set a proper VLAN ID for the management network and use DHCP to get IP, the installer should get an IP successfully and continue the following steps.

When TCPDUMP is used to detect the network traffic, it will show that the DHCP related frames are encapsulated with the VLAN ID.

When VLAN ID is not input, the installation should also be successful.

Verifying on v1.4-head (24/11/11).
Result
$\color{green}{\textsf{PASS}}$ Use ISO install with proper VLAN ID for the management network to get DHCP IP   $~~$

ISO install and set VLAN ID on the Management NIC, it can use DHCP to get VIP IP Address correctly

https://github.com/user-attachments/assets/6b9b4d7d-b72e-4f7a-a889-0ea3ded5bf38

And finish all the configuration setting, it can trigger the installation well
https://github.com/user-attachments/assets/4bd7db89-5d76-441b-a385-43994c292ea8

$\color{blue}{\textsf{Ongoing}}$ Use ISO install with no VLAN ID for the management network to get DHCP IP   $~~$

ISO install and do not set VLAN ID on the Management NIC, it can use DHCP to get VIP IP Address correctly

And finish all the configuration setting, it can trigger the installation well

Test Information

Test Environment: Single node harvester on bare-metal machine and local kvm
Harvester version: v1.4-head (24/11/11)

Verify Steps
 ISO install with VLAN ID on management NIC

Under the environment where the network switch can provide valid VLAN ID
Use ISO install Harvester
On the Configure Network page
Provide the valid VLAN ID
Use the default bond mode active-backup
Use DHCP as the IPv4 method
Check can get VIP IP address through VLAN ID
Continue with the configuration
Check can complete all the configuration to trigger Harvester installation process

 ISO install without VLAN ID on management NIC

Under the environment where the network switch can provide valid VLAN ID
Use ISO install Harvester
On the Configure Network page
Do not provide VLAN ID, leave it emtpy
Use the default bond mode active-backup
Use DHCP as the IPv4 method
Check can get VIP IP address correctly
Continue with the configuration
Check can complete all the configuration to trigger Harvester installation process


Svelte dependancy

I have hard  time figuring out why my code referencing from local repo is not working.
"dependencies": {
....
    "svelte": "^1.46.0"
  }

Is this necessary to be there? or can this can be behind? last was 1.25.0 then jump to 1.46.0

I'm not sure I understand what problem you're experiencing?
One thing I've thought about doing is moving svelte to peerDependencies. That way you would have to install it separately, but you'd have total control over which version was used.

My problem was this https://github.com/Kiho/svelte-redux-zero/blob/context-destroy/package.json#L28
was not working, this plugin is overriding that and installs 1.46.0.
I had to change "rollup-plugin-svelte": "^3.1.0" to "rollup-plugin-svelte": "~3.1.0"

I see. That would probably be fixed by using peerDependencies, I think

I've just released version 4 of this plugin — Svelte is now in peerDependencies, so hopefully this won't arise again.


Localization and collaboration with Transifex

I'm interested in starting a Finnish translation for Teslamate, but the effort of translating everything alone is quite daunting. Would it make sense to centralize translation efforts with proper collaboration tools? Transifex is free for Open Source projects.
https://docs.transifex.com/projects/open-source-project
Edit: For example Signal Messenger uses Transifex.

I haven't heard of Transifex. I'm going to check it out.
To start a new translation, generally these two files would have to be translated (the first file is much more important):

default.pot
errors.pot

Cool! Transifex, or some other similar tool, would make it super easy for non-programmers to contribute.


Typescript files are ignored as non parsable

for every typescript file i get a
[codehawk-cli] Unable to parse "C:/dev.tail/interspot/2022/web/js/_jmx/jmx.ts", skipping
message. According to the readme, typescript should work. What am I missing?

After installing @babel/plugin-transform-typescript it works.


Test commit to check workflow. WARNING: DON'T MERGE

IGNORE

Looks like it works ;)

Yes! It does!


Retro Roms cover arts are not being downloaded/shown

so my roms for the GBC/GBA and etc..are launching with no issues however, whenever i try to download the covers it doesnt appear. even though some of the games are not being detected and those that gets detected the cover arts gets downloaded but still doesnt show up on the flow launcher

Try rescanning.
Covers should show if they have been downloaded.

Try rescanning.
Covers should show if they have been downloaded.

thank you very much for your reply. i have figured out the issue i was having. the name format of the game is not as no intro as you have mentioned in the FAQ it was missing the region code at the end of each rom. you may close this incident as its not considered as an issue

No problem, glad you figured out the issue.


Is using BatchNormalization even worth the huge additional computation time?

While I observed that BatchNormalization is speeding up the convergence, it also adds A LOT to the training time. When I use BatchNormalization after each of my Conv layers for example, my epoch time rises from ~10 seconds to multiple minutes. Given that I could train dozens of non-BN-epochs for each BN-epoch, I'm sceptical if using BatchNormalization in Keras is worth the additional training time.
Thoughts? Is there any other benefit?

There may be something odd with your setup. I've never had BN add that dramatic an amount of time. Perhaps try playing around with the theano settings.

Thanks for your input. These are my current theano settings:
[global]
floatX = float32
device = gpu

[lib]
cnmem = 0.5

[nvcc]
optimizer_including=cudnn
fastmath=true

Do you have any suggestions what I could change that could have an impact on the BatchNormalization speed?

Make sure that Keras use the new Theano cudnn wrapper of batch-norm. There
was other discussion on git about this. I forgot if the change was merge in
keras or not.
This will give compuation and compilation time speed up.
I saw report from different people that in some use case, the old BN didn't
make computation slower, but in other it was doing this. It is in those
case that there was good speed up by using cudnn batch norm.
On Mon, Aug 1, 2016 at 7:54 AM, madeofwin<EMAIL_ADDRESS>wrote:

Thanks for your input. These are my current theano settings:
[global]
floatX = float32
device = gpu
[lib]
cnmem = 0.5
[nvcc]
optimizer_including=cudnn
fastmath=true
Do you have any suggestions what I could change that could have an impact
on the BatchNormalization speed?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fchollet/keras/issues/3352#issuecomment-236560627,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALC-4Y7a-03-3AIJ48yJ88aGABFkrxeks5qbd6KgaJpZM4JYyzo
.

Keras is using cudnn's BatchNormalization now in it's new version, huge speedup - problem solved!


sqlcipher: 4.2.0 -> 4.3.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/sqlcipher/versions.
meta.description for sqlcipher is: '"Full Database Encryption for SQLite"'.
meta.homepage for sqlcipher is: '"http://sqlcipher.net/"
Compare changes on GitHub

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 4.3.0 with grep in /nix/store/z47bq8isqq0kqhs4q0rpmg8nv86i1wyl-sqlcipher-4.3.0

directory tree listing: https://gist.github.com/b7bda0cfc3e18b1f8b472ba7ee5a5d8a

du listing: https://gist.github.com/26adbb192e6d5d42ae052d4cac4b1183

Rebuild report (if merged into master) (click to expand)

18 total rebuild path(s)
6 package rebuild(s)
6 x86_64-linux rebuild(s)
5 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
5 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
kmymoney
qtox
retroshare
retroshare06
skrooge
sqlcipher

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/z47bq8isqq0kqhs4q0rpmg8nv86i1wyl-sqlcipher-4.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A sqlcipher https://github.com/r-ryantm/nixpkgs/archive/4acce5c18cebee2b674e099adf0e8a23f2c1d1c0.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/z47bq8isqq0kqhs4q0rpmg8nv86i1wyl-sqlcipher-4.3.0
ls -la /nix/store/z47bq8isqq0kqhs4q0rpmg8nv86i1wyl-sqlcipher-4.3.0/bin

https://github.com/sqlcipher/sqlcipher/blob/master/CHANGELOG.md