cloud-config option to set NTP time servers

Confirmed with v0.6.0-rc3 that write_files and restart_services can be used to set NTP time servers in /etc/ntp.conf.


There is an empty div even the appearence is interaction-only

The element would be a little shaking when page loading.

same here, seeking solution


Problem w/ leaky_relu

Describe the problem
Having problems using tf.nn.leaky_relu.  Attempted to deal with the problem by trying out tf.nn.elu and tf.nn.relu, but both functions resulted in a fairly significant drop in accuracy.  I was originally building the model in Keras, but switched to TensorFlow after having a problem with the corresponding leaky_relu function in Keras.  The TensorFlow and Keras source code and error logs are below.  Thank you in advance!
Source code / logs
TensorFlow Source Code
def conv2d_w_leakyrelu(x, W, b, s = 1, name = "conlayer_i"):
    x = tf.nn.conv2d(input = x, filter = W, strides = [1, s, s, 1], data_format = "NHWC", padding = "SAME", name = name)
    x = tf.nn.bias_add(value = x, bias = b)
    # return tf.nn.elu(features = x) # TENSORFLOW PROBLEMS WITH LEAKY_RELU, SO USING ELU ACTIVATION FOR NOW
    return tf.nn.leaky_relu(features = x, alpha = lr_alpha)

Keras Source Code
model.add(Conv2D(32, kernel_size = (im_h, im_w), strides = (s, s), padding = "same", kernel_initializer = KI, input_shape = input_shape))
    model.add(LeakyReLU(alpha = leakiness))

Below is the TensorFlow error message

Below is part of the Keras error message

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

**Custom Code? **: No
**TensorFlow Version: ** 1.3.0
CUDA/cuDNN Version: V8.0.61
GPU model and memory: Tesla K80
Exact Command to Reproduce: Call the functions

Your TF version is too old.

@ppwwyyxx Thank you.


the three ... on pod menu are no longer centered

Rancher versions: 2.0 master 5/10
Steps to Reproduce:

Create a workload
Hover over pod

Results:  The three vertical dots are no longer centered

Version - 2.0 master 5/15
Verified fixed


Request: click the timestamp to pop up the exact time

Is your feature request related to any of these? We are not pursuing these at the moment.
[x ] - graphs and data visualizations
[ ] - projections and regression models
[ ] - additional data added to the top of the page
[ ] - changes to the hurdle rate. please see https://github.com/alex/nyt-2020-election-scraper/issues/194; the hurdle rate is not incorrect.
Is your feature request related to an issue? Please cite the issue.
Clearly describe the solution you'd like
It would be wonderful if I could click the timestamp where it says X days ago and it hovers a pop up that tells you the exact day and time that block event occurred. Not too dissimilar to hovering your mouse over the tooltip to get an explanation of each column.

It already definitely does this. (Just hold your mouse over it until it pops up.)

Yeah, hover and it should show up. Please re-open this if it's not working.

Yeah, hover and it should show up. Please re-open this if it's not working.

On iOS / touch interface if you tap the "x ago" nothing happens. There's no way to view the time.


Only renders the initial tab (android only)

On android only the initial tab is visible.  By changing state.index in the constructor I can make any of my three scenes visible, but when I change tabs, I only get a blank screen.  Works perfectly on iOS.
class BusinessScreen extends React.Component {
  constructor(props) {
    super(props)

    this.state = {
      index: 2,
      routes: [
        { key: '1', title: 'Home' },
        { key: '2', title: 'Menu' },
        { key: '3', title: 'Contact' }
      ]
    };
  }

  componentDidMount () {
    this.props.businessRequest(this.props.id)
  }

  _handleChangeTab = (index) => {
    this.setState({ index })
  };

  _renderHeader = (props) => {
    return <TabBar {...props} />
  };

  _renderScene = ({ route }) => {
    console.log('route.key: ', route.key)
    switch (route.key) {
      case '1':
        return <BusinessHome business={this.props.business} />
      case '2':
        return <ProductList business={this.props.business} />
      case '3':
        return <BusinessContact business={this.props.business} />
      default:
        return null
    }
  };

  render () {
    console.log(this.state)
    return (
      <View style={[styles.mainContainer, {
        flex: 1,
        backgroundColor: Colors.blueDark
      }]}>
        <TabViewAnimated
          style={{
            flex: 1,
            backgroundColor: Colors.blueDark
          }}
          navigationState={this.state}
          renderScene={this._renderScene}
          renderHeader={this._renderHeader}
          onRequestChangeTab={this._handleChangeTab}
        />
      </View>
    )
  }
}

I just recently started having this same issue. Not sure what has changed. (Using through ex-navigation)

Hey @jshumakerpruitt, I created a bare minimum example on Sketch based on the example provided by you: https://sketch.expo.io/r1xkUZnsg
Have tested this on Android 4.4.4 on a Nexus 7 and on 6.0.0 on a Nexus 5. I'm unable to reproduce this issue. If you can provide any more details for us to help fix this issue for you, that would be great.
You can also change the code on the Sketch link and give us the updated link if you're able to reproduce your issue over there. Thanks!

@karanjthakkar Could it be due to react-native version?  I do get a yellow box error on the iOS simulator about keyboardShouldPersistTaps wanting a boolean value but being given a string(enum).  I'm using 0.39.x.  I'll try downgrading the version of this package.

Additional info: If I set the tab view screen as my initial screen all tabs work, but if I navigate to the tab view screen from a different screen the problem persists.

I also have encountered this problem on iOS 8.
It seems that tabs that contain ListView don't render if they are not set as initial. It only renders when the user swipes the screen.

I was on<EMAIL_ADDRESS>and an upgrade to<EMAIL_ADDRESS>fixed the issue.

@maxschmeling Worked for me as well. Thanks for the tip!

Thanks @maxschmeling !

Closing #172.

https://github.com/react-native-community/react-native-tab-view/issues/243
I used renderPager={props => <TabViewPagerPan {...props} />} fixed it


workflows: update collections automatically once a day

This github action workflow creates a PR once a day which updates static/collections.json.
From here on out, all you have to do is press merge once a day.
To make it work, you must give Github Actions write access by going to Settings --> Code and automation --> Actions --> General --> Workflow permissions, then check "Read and write permissions" and "Allow Github Actions to create and approve pull requests"

Alright let's give this a try! tnx

I think it could push directly to main as well, if you think that's better.


(PUP-8534) Remove deprecated configtimeout setting

The configtimeout setting has been deprecated since 2014-11-13, in 462d43ecdb, and was replaced by http_connect_timeout and http_read_timeout.
Note: This PR doesn't update the puppet.conf manpage, as that should be automatically taken care of by CI.

CLA signed by all contributors.


1.x Tests refectoring

NetflixOSS » eureka » eureka-pull-requests #486 SUCCESS
This pull request looks good


Functionality Question - Separating Channels

I  want to be able to choose the channel information is streamed to (discord). I attempted to do this by creating multiple bots 1 for scores, standings. matchups etc.. and 2 for adds and drops. The only problem with this is the scores and standings one still does the adds and drops. Is there a way to limit this and/or choose the channel what information is listed. This is with discord in mind.

Unfortunately currently it only just works for the whole server and whatever channel the web hook is setup in.
I can look to do this for the mid season update or get it out sooner. I'll look into this.
Thanks for the request!


Messenger Code API supported

For more info see [1]
[1] https://developers.facebook.com/docs/messenger-platform/messenger-code

Coverage decreased (-2.7%) to 87.982% when pulling 9b28559da5e805f1f6f01235053779352905d238 on anton-nayshtut:master into be4af7eef2505e60c8beba45f2b808e5d58b0838 on conbus:master.

Despite the fact that the patch decreases coverage, I can hardly see any way to unit test it.
Please advise.

Any news with this, guys?

See my comment above (https://github.com/conbus/fbmq/pull/27#issuecomment-321546963)

thanks. I'll check and merge.


Integrity and Privacy guarantee question

Description
In readme.md, it said if we use --blinding and --integrity together, we will guarante privacy and verify computing integrity.
python -m python.slalom.scripts.eval_slalom [vgg_16 | mobilenet | mobilenet_sep] --batch_size=8 --max_num_batches=4 {--blinding}  {--integrity} {--use_sgx}

Question
When I look into the code, I find the blinded operations. But I just couldn't find something like lib.predict_and_verify for integrity check. The code just use tensorflow to get the preds and record the top-k acc. Apparently, in this way, we wouldn't check for integrity.
preds = sess.run(model.outputs[0], feed_dict={model.inputs[0]: images,
                             backend.learning_phase(): 0},
                             options=run_options, run_metadata=run_metadata)

So, where is the integrity check in this situation, am I missing something important?
More
A little confused about this, try to run the code with both blinding and integrity, using CPU and GPU. As far as I know,

script eval.py only run for integrity (with batch_verification and preprocesse).
And  script eval_slalom.py only run for privacy, although readme says it will check for integrity and actually I don't think it will do that for now.

Am I right about this? What do I need to do for getting both privacy and integrity guarantee?
Looking forward to hearing from you. Really need a little help. Thanks very much.

So eval.py is only for integrity, that's correct. But eval_slalom.py can do both integrity and privacy. When running the predictions with tensorflow, the matrix multiplication operations are performed on the GPU and the activations are performed in a trusted enclave. The enclave takes care of blinding inputs to preserve privacy, and it also runs integrity checks at the same time.

If we only need to do integrity checks, then we can simply run the whole model on the GPU, and then send all the intermediate results to the enclave to perform the checks (that's what eval.py does).
However, if we also want privacy, we have to switch between GPU and the enclave at every linear layer (so that the enclave can unblind the data, compute the non-linearity, and reblind).
To do this, we replace the activation layers in our models by a special ActivationQ layer. When this layer is called by TensorFlow, it actually sends the data to the enclave, waits for the response, and then proceeds with the evaluation on the GPU.
(see https://github.com/ftramer/slalom/blob/master/python/slalom/quant_layers.py#L90).
So in the end, we can just call sess.run() and TensorFlow takes care of running the layers in the right order. Every time it hits an ActivationQ layer, it communicates with the enclave.
Does that make sense?

Thanks, that really helped a lot. Absolutely, what you said is very reasonable. I didn't pay much attention to this line in ActivationQ before, and missing so important line, so sad:

outputs = self.sgxutils.relu_slalom(inputs, blind, activation=self.activation)

So,
Q1

This the relu in sgxutils.relu_slalom just stands for activation, right? If we use any other activation, it would always be this function  called. I think relu may couse some confusion.

Q2

However, if we also want privacy, we have to switch between GPU and the enclave at every linear layer (so that the enclave can unblind the data, compute the non-linearity, and reblind).

In my understanding,  if we use privacy and integrity together, then the whole process would be :
the enclave unblind the data ->  integrity check ->  add bias  -> compute the non-linearity ->  reblind -> send back to GPU
Is that right?

Still be confused about the detailed and  accurate process about  integrity check and bias-add  when we use privacy + integrity.
Q3
The GPU only compute for F(x, W) which F stands only for matrix-product, it doesn't include the BiasAdd, Activation.  So the Activation, BiasAdd all do in the tee enclave.Am I right about this?
Thanks again for your reply, really give me a lot of help.

Q1: For now, only ReLU is implemented on the enclave side so I just called this relu everywhere.
Q2: Yes, that's right (except that the bias is added by the GPU). Think of it this way:

the enclave has some input X. It blinds it into X'=X+R and sends that to the GPU.
The GPU computes Z'=W*X' + b and sends Z' back to the enclave.
The enclave computes Z' - WR = WX + b to unblind the results (where W*R was precomputed).
The enclave applies Freivalds' to check that W*(Xr) + (br) = (W*X+b)r. This is the integrity check. Note that br can be precomputed so including the bias in the check is pretty much for free.
The enclave computes X = ReLU(W*X + b), blinds X, and sends it to the GPU, etc.

Q3: As explained above, the GPU does compute the bias addition, but it does not compute the activation.

So clear! Thanks very much.


Add md5 and sha256 hashes to revision and file metadata for osfstorage

Purpose
Allow API to grab md5 + sha256 hashes, and also so that the osf can grab and use them.
This accompanies OSF PR: https://github.com/CenterForOpenScience/osf.io/pull/3517 and needs to be merged together for either to work.
Changes
md5 information is added to the metadata extra section of revision and file classes

@chrisseto This is good to go I believe, although it shouldn't be needed until https://github.com/CenterForOpenScience/osf.io/pull/3517 gets merged.

The two or are a requirement of each other.
Make sure you note that in each PR

LTGM :santa:
Will merge when @sloria merges the PR on the osf side

This appears to be for file checkout rather than hashes. @hmoco Can you clarify?

Probably got extra commits pushed up by accident as this PR is based on @hmoco's develop

But this doesn't include any of the hashes code

welp


Add a general API to get env variables or commands required to activate environment

Deprecate existing Jupyter-specific getActivatedEnvironmentVariables along with this.
Activating environments may be done using:

Activated env variables
Activation commands
A combination of both

We've yet to figure out the exact details on the general extension API. Blocked on https://github.com/microsoft/vscode-python/issues/20492.

Note: I've found that when using globally installed python environments (like HomeBew Python on Mac, Python Org Python on Mac, apt-get install python3 on ubuntu and Windows Store), we still need to update some path variables.
When you install packages like so <python> pip install jupyter notebook, then warning messages are logged in the console indicating the fact that some paths are not added into the PATH variable.
& sure enough things fail to load a result of not updating the PATH variable.
I.e. even for global envs, we need some PATH variables to be updated.

Also, when dealing with pyenv can we ensure the necessary path variables are added
I.e. add the following into the PATH
/Users/mv/.pyenv/shims
/Users/mv/.pyenv

See https://github.com/microsoft/vscode-jupyter/issues/12542

I've found that when using globally installed python environments (like HomeBew Python on Mac, Python Org Python on Mac, apt-get install python3 on ubuntu and Windows Store), we still need to update some path variables.

I've never had a specific issue with e.g. Homebrew with anything like this. But honestly, if you're installing globally you're already asking for trouble and should probably be installing Jupyter via your OS and not Python itself.

why users need to manually update their PATH when installing with
--user flag.

That's only if you are trying to reach a binary that a package installed, not to simply use a library. And it's typically necessary because --user installs to ~/.local/bin on Unix which is not traditionally on PATH. So it's really up to you and/or OS to place that on PATH.

When dealing with pyenv can we ensure the necessary path variables are added

What exactly is in that shim directory? If that's dynamically updated by pyenv then we don't want to add that as we don't control what e.g. python will point to and it can change from underneath us. But I believe we already have support for https://github.com/pyenv/pyenv-virtualenv .

In Jupyter extension when ever we run any python env, we always add the python exec directory into the current path
I.e. PATH=dirname(PythonEnvironment.executable):$PATH

Can I ask why you do that? That's not typical Python behaviour if something is globally installed because it will be found on PATH naturally. And that change to PATH for virtual environments only happens if you activate it, not if you just run Python itself directly. Remember, virtual environment activation is a convenience, not a requirement and people should not be coding as if it's guaranteed to occur.
And can you make those separate asks separate issues? They don't have anything to do with each other and will require their own consideration/solution.

So it's really up to you and/or OS to place that on PATH.

Got it, thanks, looks like we (Jupyter) will do this.
A lot of the packages (i think binaries) are in special directories and when installing via Pip there were a number of warnings indicating the fact that a particular directory is not in path

f you're installing globally you're already asking for trouble and should probably be installing Jupyter via your OS and not Python itself.

Not sure what you mean by installing jupyter via your OS

at? That's not typical Python behaviour if something is globally installed because it will be found on PATH naturally. And that change to PATH for virtual

Aah yes, agreed, this is specific to Jupyter
Basically users run shell commands within a kernel as python -m pip list at this point ptyhon shoudl point to the same executable that points to the kernel. Similar if a user were to run java -h then java executable shoudl be launched form the same directory where python is
& yes this is specifiv to jupyter
I think we'll continue special casing this in Jupyter ext
@brettcannon thanks for the responses, I will hide these comments as resolved.

This now exists in the new environments extension.

You can even get shell specific activation commands. Or activated run command if such exist for the environment type.

This now exists in the new environments extension.

Is this a vscode extension or do you mean environment api in python extension?

In this extension: https://marketplace.visualstudio.com/items?itemName=ms-python.vscode-python-envs


Going to Nether resets you to level 1

Not only do you go from whatever level you were to level 1, when you go back to the Overworld, you remain a level 1. You have lost the use of the hoard.
Work around: Break the hoard and recover it and the gold coins. Place the hoard again and put the coins back into it. You will regain your previous level (you have to open the hoard one more time to see the change).

Added network message to sync client to server for dragon level.


An error occurs if there is a newline in the dictionary of the Vim9 script loaded in the hooks_file.

Description
An error occurs if there is a newline in the dictionary of the Vim9 script loaded in the hooks_file.
Code of Cause in Minimal Config:
g:quickrun_config._ = {
  'runner': 'job',
  'outputter': 'buffer',
  'outputter/buffer/close_on_empty': 1,
}

To Reproduce
Steps to reproduce the behavior:

gvim.exe -N -u ~/vimfiles/vimrc-min-dein-hooks_file-vim9 -U NONE -i NONE
See Minimal Config
:call dein#recache_runtimepath()
:quit
gvim.exe -N -u ~/vimfiles/vimrc-min-dein-hooks_file-vim9 -U NONE -i NONE
:set ft=vim
input echo 'vim'
:QuickRun
Error occured
See Screenshots. Errors encountered are displayed by :messages.

Expected behavior
No error.
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop:
Please complete the following information, when reporting bugs related to the Dein.vim.

OS: Windows11
Vim or Neovim version: Vim 9.0.1491
Commit/Version: 6f40e084

Minimal Config
~/vimfiles/vimrc-min-dein-hooks_file-vim9
set nocompatible
set runtimepath+=C:\Users\hokorobi\_vim\dein\repos\github.com\Shougo\dein.vim
let g:testdic = {}
if dein#min#load_state(expand('~/_vim/dein'))
  call dein#begin('~/_vim/dein')
  call dein#add('thinca/vim-quickrun', #{
        \ on_cmd: 'QuickRun',
        \ hooks_file: '~/vimfiles/vimrc-min-dein-hooks_file-vim9_quickrun.vim',
        \ })
  call dein#end()
  call dein#save_state()
endif
filetype plugin indent on
syntax enable

~/vimfiles/vimrc-min-dein-hooks_file-vim9_quickrun.vim
" hook_source {{{
vim9script
g:quickrun_config = get(g:, 'quickrun_config', {})
g:quickrun_config._ = {
  'runner': 'job',
  'outputter': 'buffer',
  'outputter/buffer/close_on_empty': 1,
}
" }}}

OH no.

https://github.com/vim/vim/pull/12269 is needed.

https://github.com/vim/vim/pull/12269 works.  Please test.

Thank you!
I will try to use it when PR is merged.


Support for API V2

The API V1 will soon be unsupported(2018.06.30). Do you plan to update this to work with the OpsGenie API V2?

Ah good point, need to change this. Will do this in coming days.

Moved up to V2.
Also moved into.net 4.7

It looks live you've used the wrong source for the nugget you pushed. It does not match what's expected from the ReadMe, or more important, from your own unit tests. At least the config class looks like this, should not look like this. Note the ApiUrl. :
namespace OpsGenieApi
{
public class OpsGenieClientConfig
{
public OpsGenieClientConfig();
    public string ApiKey { get; set; }
    public string ApiUrl { get; }
}

}

The config has been changed to not include the url. Need to update the readme.
The url is hardcoded, doesn't need to change ever?
namespace OpsGenieApi
{
    public class OpsGenieClientConfig
    {
        public string ApiUrl => "https://api.opsgenie.com/v2/alerts";
        public string ApiKey { get; set; }
    }
}

I understand that's the idea, but the code in the nugget does not have the URL hardcoded. It's not the same code as in the git repository. Maybe something else is also missing then?

The package downloaded from nuget does show the url being hardcoded:


Test execution time is not correct when tests run in parallel

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: any
OS Version: any

Original issue: https://github.com/microsoft/playwright/issues/20276
Steps to Reproduce:

Run Playwright tests
Observe the total time being the sum of test times, even though those tests ran in parallel

//cc @connor4312

This can be handled by explicitly setting the state on the parent item of the parallel tests and pass in the duration when you do. This will override the duration derived from summation of the child tests.
The summation approach is provided as an easy way for most test runners to work, with the ability to override it like that. I didn't want to just use a wall-clock time, since the time might include setup, network calls, etc. that do not actually reflect test case time. One alternative approach is to record the start and end times for each, and merge those as we go up.
I prototyped that in https://github.com/microsoft/vscode/commit/4c10073ea519295187a2f83f525d4e6a0edad009, but am not sure it's worth the extra complexity and computation time having to track all ranges vs. duration integers.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Spelling mistake in documentation for "snowflake_integratin_grant"

In the documentation the integration grant is missing an "o".
https://registry.terraform.io/providers/chanzuckerberg/snowflake/latest/docs/resources/integration_grant
resource snowflake_integratin_grant grant {
...

@jifeous This issue has already been fixed. https://github.com/chanzuckerberg/terraform-provider-snowflake/pull/378

@jifeous This issue has already been fixed. https://github.com/chanzuckerberg/terraform-provider-snowflake/pull/378

fixed in #378

fixed in #378


unhandled top-level exception: $exn

2020-02-03 18:33:25 UTC [Fatal] Unhandled top-level exception: $exn
Generating crash report
exn: "(monitor.ml.Error\n  (Failure\n    "Failed to connect to Kademlia process: RPC #2 failed: \"initializing helper error: Map log file. Path=/home/dannyjr95/.coda-config/coda_net2/libp2p_helper/libp2p-peerstore-v0/000000.vlog. Error=exec format error\"")\n  ("Raised at file \"stdlib.ml\", line 33, characters 17-33"\n    "Called from file \"src/deferred1.ml\", line 17, characters 40-45"\n    "Called from file \"src/job_queue.ml\" (inlined), line 131, characters 2-5"\n    "Called from file \"src/job_queue.ml\", line 171, characters 6-47"\n    "Caught by monitor coda"))"
☠  Coda daemon crashed. The Coda Protocol developers would like to know why!
Please:
Open an issue:
https://github.com/CodaProtocol/coda/issues/new
Briefly describe what you were doing and attach the crash report /home/dannyjr95/.coda-config/coda_crash_report_2020-02-03_18-33-25.630943.tar.gz

Thank you for the report, and sorry for the long processing delay! I think this is the same as #4254 @simseokhyun, can you confirm that you were running this inside of WSL?

Hi ember,
Yes, I was running this in WSL. I tried to run docker in WSL, officially
made an inquiry to microsoft and got the answer that the feature will be
available in WSL2 which will be released within this year.
I decided to run coda through Google Cloud and all of the problems now
don't apply to me!
Thank you for recognizing my problem.
Me and my fellow peers now use GCP, so I think we won't face the problem :)
Thank you!
Sincerely,
Danny.
2020년 5월 29일 (금) 오전 3:38, ember arlynx<EMAIL_ADDRESS>작성:

Thank you for the report, and sorry for the long processing delay! I think
this is the same as #4254
https://github.com/CodaProtocol/coda/issues/4254 @simseokhyun
https://github.com/simseokhyun, can you confirm that you were running
this inside of WSL?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CodaProtocol/coda/issues/4326#issuecomment-635524923,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOOQSKUJVKVFFOFIWLSOIHDRT2VTJANCNFSM4KPKDRUQ
.

This is the kademlia version of #4254. We're no longer using kademlia. Closing.


[SW-19130] Fix account section for guest login

1. Why is this change necessary?
When logedin as a one time account and going to to the account section (or changing the address in checkout) the user will be logged out with an empty basket.
2. What does this change do, exactly?
Only logout a one time account when the user really wants to by clicking on the appropriate link.
This also fixes the sidebar navigation in the account to display only the logout link.
3. Describe each step to reproduce the issue or behaviour.

Add a product to the basket
Go to checkout and create a one time account
Go to account page or change address in checkout

4. Please link to the relevant issues (if any).
SW-19130
5. Which documentation changes (if any) need to be made because of this PR?
None
6. Checklist

[x] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-22721
Please use this issue to track the state of your pull request.

Hey @sobyte,
i couldn't reproduce this issue anymore in 5.5. I discussed it also with the Ticket Creator, it seems to be fixed since 5.4 Feature "Exit guest mode": But anyway thanks for your Pull Request :blue_heart:


bgpalerter seems to have lost visibility ~ April 8 - RIS issues ?

Took me a few days to open report this as I wanted to make sure it is not some local issue:
Since April 8, ~1200 UTC I am not seeing any monitored events being triggered. RIPE Service status indicates all is well, RIS Live should be functioning.
Running on RHEL 8 as a systemd service. About a year in prod. Worked fine after udpating to v1.27.1.
I checked that notifications work with the -t flag. They do. It spams Email and Telegram when I use the -t flag. I checked that the process has sufficient resources and permissions - all good. I checked bgpalerter's reports.log and it is indeed empty but I know I created plenty of mayhem "events" :P
I then tried creating a new prefixes.yml list and config.yml by stopping the service, renaming the existing ones, executing the binary manually once with  bgpalerter-linux-x64 generate -a ASN-o prefixes.yml -i -m
This completed without errors and created sensible files. I restart service, withdraw a monitored prefix and tail -f reports.log. Nothing.
I hijack my prefix from a lab ASN. Nothing.
Final sanity check before reaching out: I spun up a new Ubuntu server, installed docker and created the bgpalerter docker container. Created config, started it, withdrew a prefix. This instance, too, does not "see" an event.
I do see in error.log of the original prod instance around the thing things stopped working:
2021-04-10T14:34:44+00:00 info: ris connector connected
2021-04-10T14:39:45+00:00 info: ris connector connected
2021-04-10T15:05:15+00:00 error: Error: Unexpected server response: 500
2021-04-10T15:05:15+00:00 error: It was not possible to establish a connection with RIPE RIS
2021-04-10T15:06:20+00:00 error: Error: Unexpected server response: 500
2021-04-10T15:06:20+00:00 error: It was not possible to establish a connection with RIPE RIS
2021-04-10T15:07:25+00:00 info: ris connector connected

But these 500 responses have occured from time to time in the past. Of note is that during my testing of prefix withdrawals the last entry in error.log indicated that we were connected at that time: info: ris connector connected
Did RIPE change anything in RIS Live that could be breaking me ?
Unrelated, I did have some 530 responses from Cloudflare when using cloudflare as my vrpProvider for rpki which broke RPKI detection but I had switched back to ntt since and it worked fine.

Hi @mfld-pub,
I'm checking your ticket.
In the meanwhile, what the uptimeApi says? If any problem occurs in BGPalerter, a warning appears in the api. Was the process monitored at the time of the incident?

I then tried creating a new prefixes.yml list and config.yml by stopping the service, renaming the existing ones, executing the binary manually once with bgpalerter-linux-x64 generate -a ASN-o prefixes.yml -i -m

I'm doing the same with your AS, I'll let you know

I do see in error.log of the original prod instance around the thing things stopped working:
2021-04-10T14:34:44+00:00 info: ris connector connected
2021-04-10T14:39:45+00:00 info: ris connector connected
2021-04-10T15:05:15+00:00 error: Error: Unexpected server response: 500
2021-04-10T15:05:15+00:00 error: It was not possible to establish a connection with RIPE RIS
2021-04-10T15:06:20+00:00 error: Error: Unexpected server response: 500
2021-04-10T15:06:20+00:00 error: It was not possible to establish a connection with RIPE RIS
2021-04-10T15:07:25+00:00 info: ris connector connected

But these 500 responses have occured from time to time in the past. Of note is that during my testing of prefix withdrawals the last entry in error.log indicated that we were connected at that time: info: ris connector connected

This should not matter, connection errors can occur. Everything is ok as long as the process is able to re-establish the connection in the coming seconds.

Unrelated, I did have some 530 responses from Cloudflare when using cloudflare as my vrpProvider for rpki which broke RPKI detection but I had switched back to ntt since and it worked fine.

I'm not sure if you did it already, but I suggest you to use preCacheROAs: true. This precaches a VRP dump instead of using online queries.

It must have been a RIS issue of sorts ?! As of 0914 UTC today it seems to work again both my RHEL 8 prod and my Ubuntu docker container that I set up for this ticket.

Yes. I was able to reproduce your issue and I contacted the main dev behind RIS and he did some digging.
Somebody was flooding the service with connections (now banned), as a result other new legit connections were slow to be served. You spotted this because you were one of those unlucky, we were already connected and we did not.
We are planning some improvements, including a missing/delayed messages monitoring in both BGPalerter and RIS. You will see a PR linked to this issue soon. In the meanwhile a new rule to limit the number of connections per user has been set in RIS (since one connection can have unlimited subscriptions to prefixes, there is no reason at all to open multiple connections...just a lack of reading-the-doc skills).
Thanks for reporting this!!

Thanks for cruising github issues on your Sunday <3
I think they also want us to send a user-agent with our connections in the ?client= parameter of the URI. Something like
wss://ris-live.ripe.net/v1/ws/?client=AS174-basement-lab
Would it make sense to make this a configurable option in BGPAlerter ?

Thanks for cruising github issues on your Sunday <3

You are welcome :)

I think they also want us to send a user-agent with our connections in the ?client= parameter of the URI. Something like
Would it make sense to make this a configurable option in BGPAlerter ?

We agreed with the RIS staff on what user agent to send for all BGPalerter clients, this is already done by BGPalerter and should not be configured per instance. Each user has also an additional random connection id.

As promised, in addition to the fix on the RIS side reported above, in the next release of BGPalerter there will be a check for silent socket sessions.


Enable rule "import/order"

I think it would be nice to add the import/order rule. This is my suggestion for how the rule should be configured:
{
  "alphabetize": {
    "order": "asc"
  },
  "groups": [
    ["builtin", "external"],
    ["internal", "unknown"],
    ["parent"],
    ["sibling", "index"]
  ],
  "newlines-between": "always"
}

This is based on what I've seen mostly when contributing in mostly the Node.js ecosystem. I'm very open to hearing other peoples perspective!
I think that a good strategy could be to start with only the groups option, and then follow up with the other options later.
Just a quick rundown on what it enforces:
import bodyParser from 'body-parser'
import express from 'express'

import projects from './api/projects'
import users from './api/users'

import http from 'http'

import bar from '../lib/bar'
import foo from '../lib/foo'

import info from './'

The benefit of this rule:

Cleaner git diffs and less merge conflicts since there is a predictable place for each import
I know that many people prefers having external dependencies first, and thus modify the file manually after using auto-imports in their IDE, this is now automatic
You can easily see which external dependencies a specific file has
It looks neat and clean

This rule is auto-fixable! 🎉

@LinusU how do you feel about #1517 ?

@mightyiam that specific plugin doesn't support require, that is probably going to be quite a big problem for us? 🤔

Yes. Only where they are mixed. Which should be rare. And ES modules is the future, so maybe that's OK?

Only where they are mixed.

Do you mean that it won't touch requires, and only concern itself with the imports?
I don't have a strong opinion on which plugin we use, although we already include the import plugin. What are the pros of using simple-import-sort instead?

The simple import sort is a one-stop-shop with very little configuration. Probably none that we would use. If we like it — we like it. Nothing more to do. It's kind of similar to standard in this sense.
If the import plugin allows a similar functionality plus it supports require, then it may be preferable. Does it support require?


az storage blob exists Fails

Describe the bug
Command Name
az storage blob exists
Errors:
No module named '_cffi_backend'
Traceback (most recent call last):
python3.6/site-packages/knack/cli.py, ln 215, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 553, in execute
    self.commands_loader.load_arguments(command)
azure/cli/core/__init__.py, ln 478, in load_arguments
    self.command_table[command].load_arguments()  # this loads the arguments via reflection
cli/core/commands/__init__.py, ln 310, in load_arguments
    super(AzCliCommand, self).load_arguments()
...
storage/v2018_11_09/blob/_encryption.py, ln 13, in <module>
    from cryptography.hazmat.primitives.padding import PKCS7
cryptography/hazmat/primitives/padding.py, ln 13, in <module>
    from cryptography.hazmat.bindings._padding import lib
ModuleNotFoundError: No module named '_cffi_backend'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Enter registry.fedoraproject.org/f32/fedora-toolbox:32 container with podman (docker may also work)
Install Azure CLI by following https://docs.microsoft.com/en-us/cli/azure/install-azure-cli-yum
az login
az storage blob exists --container-name {} --name {} --account-name {}

Expected Behavior
A valid response describing if the blob exists
Environment Summary
Linux-5.8.16-200.fc32.x86_64-x86_64-with-glibc2.2.5
Python 3.8.6
Installer: RPM

azure-cli 2.13.0

Additional Context

This is running in fedora-toolbox:32 container (registry.fedoraproject.org/f32/fedora-toolbox:32)
Azure CLI installed by following https://docs.microsoft.com/en-us/cli/azure/install-azure-cli-yum
Other commands (e.g. az group create) work as expected

add to S178

See here https://github.com/Azure/azure-cli/blob/dev/doc/install_troubleshooting.md#errors-on-install-with-cffi-or-cryptography.
sudo yum install -y python3-cffi


[for 2.2.0] merge ledgerstorage with kvledger pkg

Type of change

Improvement (improvement to code)

Description
The ledgerstorage package encapsulates both pvtdata storage and block storage. We needed this package as

we had to have 2-phase commit between pvtdata storage and block storage.
we had a recovery logic that was needed after a commit failure or peer crash between pvtdata commit and blockstore commit.
we treated pvtdata store as a pvt blockstore and blockstore as a public block store.

After introducing the support for peer rollback, we removed the 2-phase commit between the pvtdata storage and block storage as we had to allow the pvtdata storage to have block height higher than the blockstore. Hence, the recovery logic was also removed. Thus, the motivation for having ledgerstorage package became feeble and hence, we remove it.
Further, core/ledger, core/ledger/ledgerstore, core/ledger/ledgermgmt create confusion and provide disinformation. At least in this PR, we remove core/ledger/ledgerstore.
Additional details
Pushing for early review. Yet to fix a data race.
Related issues
FAB-17670

@manish-sethi I am getting two data races
one between

a debug message at lockbased_txmgr (where we invoke multiple listeners after a block commit (and)
the purge pvtdata goroutine at pvtdatastore

another between

a debug message at lockbased_txmgr (where we invoke multiple listeners after a block commit (and)
collElgListener at pvtdatastore

When I remove the debug message, obiviously, I don't get any data races.
While the second data race at least make sense, the first one looks surpising to me. As there are too many moving parts, I need your help to identify the root cause. Let's pair for this when you get time.

Senthil, this happens because the logger tries to print all elements of collElgNotifier which has a map full of pvtdatastorage.store objects and the purgerLock inside is unlocked in that other goroutine which involves an atomic operation increment. Since the logger prints the inner field using reflection, no locking/atomic operations are being used and a data race occurs.
The following piece of code trivially stops the logger from using reflection.
diff --git a/core/ledger/kvledger/coll_elg_notifier.go b/core/ledger/kvledger/coll_elg_notifier.go
index 8ce001e31..ed13e58e1 100644
--- a/core/ledger/kvledger/coll_elg_notifier.go
+++ b/core/ledger/kvledger/coll_elg_notifier.go
@@ -20,6 +20,10 @@ type collElgNotifier struct {
        listeners                     map[string]collElgListener
 }

+func (n *collElgNotifier) String() string {
+       return "foo"
+}
+
 func (n *collElgNotifier) Initialize(ledgerID string, qe ledger.SimpleQueryExecutor) error {
        // Noop
        return nil

Thank you, @yacovm You just taught me how to debug data races especially when it occurs in debug messages. I didn't have this thought process earlier. I appreciate the help.

As it is not justified to add production code to avoid data race while using a debug log, we remove the debug log.

As it is not justified to add production code to avoid data race while using a debug log, we remove the debug log.

Wait, what?? why is it not justified? Adding a stringer is perfectly fine!

My opinion is that it is not justifiable. Similarly, I am not supportive of writing interfaces in the production code so that mock can created.
If @manish-sethi also feels that it is justifiable to change the production code for a debug log, I will do it. Or he might find some other bug in my code :-)

As it is not justified to add production code to avoid data race while using a debug log, we remove the debug log.

Wait, what?? why is it not justified? Adding a stringer is perfectly fine!

Yes, I agree that adding a stringer is good in this case.
Ideally, we should have added a function Name() or ID() in the listener interface (or take a name input while registering the listener). So, it is forced. But, as an easy fix, Stringer seems fine.

please squash to one commit.

@yacovm @manish-sethi as both of you feel that it is completely fine, I keep my philosophy aside.
I will go with adding Name() to the interface as it looks clean.

@ryjones sure. Will do.

Added 20 lines of production code and 64 lines to mock code to retain a debug logger :frowning:


Project Manager endpoint to report execution progress

See design:
https://github.com/enso-org/design/blob/1621fc213937f86e0c37a4f5905a156900fd2129/epics/home-screen/design.md#opening-and-running-projects
The format probably doesn't matter - as long as it is possible to convert the endpoint's output into a percentage for the loading spinner.
I'm assuming this functionality is still wanted, because support for async execution has recently been merged in to the cloud backend.
For example:

a percentage
the count of executed nodes, and the count of total nodes
a list of nodes, with executed state (boolean?) and (arbitrary) weight, or method name

This would likely be a language server's endpoint first as project manager is not really responsible for executing the projects.
This is a larger task of tracking progress (of compilation, execution, setup) and it's unlikely to be picked up soon.


Explorer not showing files

Type: Bug
When I open a folder, the content does not show in Explorer.  I see nothing in the Explorer window.  Is some setting causing this?
VS Code version: Code 1.85.0 (af28b32d7e553898b2a91af498b1fb666fdebe0c, 2023-12-06T20:48:09.019Z)
OS version: Windows_NT x64 10.0.19045
Modes:

System Info

Item
Value

CPUs
13th Gen Intel(R) Core(TM) i7-13800H (20 x 2918)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
31.69GB (14.39GB free)

Process Argv
--folder-uri file:///c%3A/inetpub/wwwroot --crash-reporter-id bc2b09a9-7cd1-427e-85ec-5c5a10fbf0bc

Screen Reader
no

VM
0%

Extensions (8)

Extension
Author (truncated)
Version

adobe-cfml-lsp
com
1.0.469

cflsp
Dav
1.0.48

auto-close-tag
for
0.5.14

coldfusion
ili
1.0.1

vscode-cfml
Kam
0.5.4

remote-wsl
ms-
0.81.8

ColdFusion
Ske
0.0.1

highlight-matching-tag
vin
0.11.0

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vsdfh931:30280409
vshan820:30294714
vstes263:30335439
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
py29gd2263:30899288
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
azure-dev_surveyone:30548225
282f8724:30602487
89544117:30613380
a9j8j154:30646983
showlangstatbar:30737416
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
welcomedialog:30910333
pythonnosmt12:30797651
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
dsvsc013:30795093
dsvsc014:30804076
dsvsc015:30845448
pythontestfixt:30902429
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
pythontbext0:30879054
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
aa_t_chat:30882232

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

I don't know if I inadvertently selected a setting that is preventing the folder from showing in the Explorer panel.

Does the Reset View Locations command from the Command Palette (F1) fix it?

That worked!  I was unaware of that command.  Thank you!


388 clickable links

Addresses https://github.com/cidgoh/DataHarmonizer/issues/388, particularly in light of comment https://github.com/cidgoh/DataHarmonizer/issues/388#issuecomment-1500713438 where @pkalita-lbl considered Markdown support.
(Note: as this is my first contribution I'm setting it up as a draft PR, as it may reflect misunderstandings of the codebase.)
Changelog

Creates a utils called content.js where formatters pertaining to content, rather than data fields, can be placed.
In content.js, adds url2markdown and url2anchor methods to use on arbitrary text.
In content.js, adds support of the Marked parser and exposes one to other modules as renderContent.
Renders the content of the HelpSidebar and toolbarGettingHelp components in Markdown.

Further Investigations

Is this enough to warrant modularizing the content into Markdown documents?
Are there simpler or smaller or more performant parsers to use than Marked?
Should I standardize this code to a non-ECMA6 style?
What would be a more general way to ensure all content is markdown-based on the page?

It looks like this issue also appears downstream?
https://github.com/microbiomedata/nmdc-server/issues/945
https://github.com/microbiomedata/nmdc-server/issues/1041
https://github.com/microbiomedata/nmdc-server/issues/820

I'll be able to help/ do feedback on Wednesday when back from mini vacation.  Right now on gabriola Island, no computer!
Ciao, d.

From: Kenneth Bruskiewicz @.***>
Sent: August 21, 2023 8:55:35 AM
To: cidgoh/DataHarmonizer
Cc: Damion Dooley; Review requested
Subject: Re: [cidgoh/DataHarmonizer] 388 clickable links (PR #405)
It looks like this issue also appears downstream?
microbiomedata/nmdc-server#945https://github.com/microbiomedata/nmdc-server/issues/945
microbiomedata/nmdc-server#1041https://github.com/microbiomedata/nmdc-server/issues/1041
microbiomedata/nmdc-server#820https://github.com/microbiomedata/nmdc-server/issues/820
—
Reply to this email directly, view it on GitHubhttps://github.com/cidgoh/DataHarmonizer/pull/405#issuecomment-1686560993, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AA6QWRTD2PZYPY7U72UBEYLXWN54JANCNFSM6AAAAAA3YRZKBM.
You are receiving this because your review was requested.Message ID: @.***>

Just to give some context on the related NMDC issues. This idea did originally in that project, and it would still be beneficial there. However the biggest impetus in that project was actually a component outside of DataHarmonizer where help information from the schema is also displayed. For that work we were really only concerned about linking bare URLs, and we chose to use https://github.com/markdown-it/linkify-it because basic we found simple regex-based approaches to not be robust enough.
The same people who make that package also make a full-blown markdown parser: https://github.com/markdown-it/markdown-it I might suggest looking into that package because it has bare URL linking and takes care of certain security issues by default.


Duckscript call other task not run dependencies.

[tasks.A]
script_runner = "@duckscript"
script = '''
cm_run_task B
'''

[tasks.B]
command = "echo"
args = ["world"]
dependencies = ["C"]

[tasks.C]
command = "echo"
args = ["hello"]

When run cargo make A, Expect to execute  task C before B

thanks for reporting. i'll check it out.

@umaYnit this is now fixed in the 0.32.13 dev branch.
can you please confirm it's fixed for you?

I will test it right away

Yeah, It's pretty good.
$ cargo make A
[cargo-make] INFO - cargo make 0.32.12
[cargo-make] INFO - Project: rust_test
[cargo-make] INFO - Build File: Makefile.toml
[cargo-make] INFO - Task: A
[cargo-make] INFO - Profile: development
[cargo-make] INFO - Running Task: A
[cargo-make] INFO - Running Task: C
hello
[cargo-make] INFO - Running Task: B
world
[cargo-make] INFO - Build Done in 0.00 seconds.

great thanks for confirming.


Bot is making a weird Spiral

Expected Behavior
Possible to make a more square Spiral?
Can I suggest this algorithm? Link
Use gps_default_altitude for any altitude calculation. If I set gps_default_altitude: 8.0 I should not see 600 in my FollowSpiral log
Actual Behavior
I'm trying to farm a Pokemon nest but the bot is not building a good Spiral..
Moreover, it is changing altitude quite dangerously I'd say.... 90m for a 7m walk is a bit too much of altitude difference..
[2016-08-30 18:28:53] [position_update] Walking from (45.4687152075966, 9.186993983477901,<PHONE_NUMBER>445617) to (45.46873584444358, 9.187011294576443,<PHONE_NUMBER>85426) (2.66181009803 m)
[2016-08-30 18:28:56] [position_update] Walking from (45.4687383679302, 9.187020716537017,<PHONE_NUMBER>983659) to (45.468780613888136, 9.187011294576443,<PHONE_NUMBER>991403) (4.7543554703 m)
[2016-08-30 18:28:57] [position_update] Walking from (45.46879495494637, 9.187018636759099,<PHONE_NUMBER>845956) to (45.468825383332685, 9.187011294576443,<PHONE_NUMBER>500696) (3.4313490297 m)
[2016-08-30 18:29:00] [position_update] Walking from (45.46881126169622, 9.187015417221994,<PHONE_NUMBER>929784) to (45.468870152777235, 9.187011294576443,<PHONE_NUMBER>189738) (6.556251063 m)
[2016-08-30 18:29:02] [position_update] Walking from (45.4688756731702, 9.187019746048506,<PHONE_NUMBER>239292) to (45.46891492222179, 9.187011294576443,<PHONE_NUMBER>009876) (4.41341736023 m)
[2016-08-30 18:29:04] [position_update] Walking from (45.46889605754648, 9.186992948497815,<PHONE_NUMBER>15883) to (45.46895969166634, 9.187011294576443,<PHONE_NUMBER>26875) (7.21896583188 m)
[2016-08-30 18:29:07] [position_update] Walking from (45.468952686820714, 9.187023287636595,<PHONE_NUMBER>019418) to (45.46900446111089, 9.187011294576443,<PHONE_NUMBER>822032) (5.8320628754 m)
[2016-08-30 18:29:09] [position_update] Walking from (45.468990438096775, 9.187004524845825,<PHONE_NUMBER>609743) to (45.46904923055545, 9.187011294576443,<PHONE_NUMBER>644738) (6.55877482672 m)
[2016-08-30 18:29:11] [position_update] Walking from (45.46904749136028, 9.187004515480213,<PHONE_NUMBER>989156) to (45.469094, 9.187011294576443,<PHONE_NUMBER>209363) (5.19807320585 m)
[2016-08-30 18:29:13] [position_update] Walking from (45.46906941041714, 9.187012029752244,<PHONE_NUMBER>379034) to (45.46913876944455, 9.187011294576443,<PHONE_NUMBER>766062) (7.71286742057 m)
[2016-08-30 18:29:15] [position_update] Walking from (45.46913105273312, 9.1869919272824,<PHONE_NUMBER>451162) to (45.469183538889105, 9.187011294576443,<PHONE_NUMBER>862312) (6.02841623377 m)
[2016-08-30 18:29:17] [position_update] Walking from (45.46918151579151, 9.187020262177711,<PHONE_NUMBER>081614) to (45.469228308333655, 9.187011294576443,<PHONE_NUMBER>32414) (5.24992990153 m)
[2016-08-30 18:29:19] [position_update] Walking from (45.469225468161355, 9.186993194737777,<PHONE_NUMBER>989919) to (45.469273077778205, 9.187011294576443,<PHONE_NUMBER>52959) (5.47902759036 m)
[2016-08-30 18:29:22] [position_update] Walking from (45.469286652310146, 9.187013917177774,<PHONE_NUMBER>095186) to (45.46931784722276, 9.187011294576443,<PHONE_NUMBER>975378) (3.47445848281 m)
[2016-08-30 18:29:24] [position_update] Walking from (45.46931099563178, 9.186995313378203,<PHONE_NUMBER>012496) to (45.46936261666731, 9.187011294576443,<PHONE_NUMBER>697509) (5.87382810862 m)
[2016-08-30 18:29:26] [position_update] Walking from (45.46935271217537, 9.186991133827558,<PHONE_NUMBER>828603) to (45.46940738611186, 9.187011294576443,<PHONE_NUMBER>159055) (6.27968754612 m)
[2016-08-30 18:29:29] [position_update] Walking from (45.469427776797694, 9.187027949334329,<PHONE_NUMBER>852292) to (45.46945215555642, 9.187011294576443,<PHONE_NUMBER>768261) (3.00568587124 m)
[2016-08-30 18:29:31] [position_update] Walking from (45.46942929819695, 9.187014258801014,<PHONE_NUMBER>197326) to (45.46949692500097, 9.187011294576443,<PHONE_NUMBER>852857) (7.52322654824 m)
[2016-08-30 18:29:33] [position_update] Walking from (45.469499885079465, 9.187001568026774,<PHONE_NUMBER>234883) to (45.46954169444552, 9.187011294576443,<PHONE_NUMBER>840699) (4.71048891402 m)

Your FULL config.json
		{
			"type": "FollowSpiral",
			"config": {
				"enabled": true,
				"diameter": 100,
				"step_size": 5
			}
		}

maybe diameter is too big. default is 4, i just tried 8 and it's much bigger.

I made various tests, and diameter seems like the number of segments (of step_size length) the bot will walk through, increasing from inside to the outside obviously. The image is showing the Spiral after about 2 hours, so not very big...I'd say 500x100m

In my experience, if followspiral is changing our location, you're IP banned.

Teleporting get you IP banned, not walking from A to B. Isn't it like so?

There a lot of things that might tripper them banning an IP? Are you using a VPN, cloud or proxy? Many networks have been banned.

Do you mean GPX?

@iruy you need to set your altitude range. I think the default is 500 to 1000. Which 600 is within the margin of.

Thank you


removeUnknownsAndDefaults removes known attrs from filter element

it's possible to define x, y among with other attributes on filter element: https://developer.mozilla.org/en-US/docs/Web/SVG/Element/filter#Specific_attributes
SVGO removes x and y attrs with removeUnknownsAndDefaults option

This seems to work fine in version 3.3.2.


Ability to create custom url for note

Would be cool if you had the ability to refer a note to unique url
e.g. https://paste.gg/supernote
It'd become super easy to memoize and share with friends

I don't think this is a good idea - it would make guessing paste links a lot easier.

I think this should be an toggle option for the hoster.

Perhaps it could be done in a similar manner to what git.io does for GitHub.


Hardware>File Signing Service submission automation (testHarness LSA)

Hello team!
Currently, LSA file signing cases are submitted manually via the web service (Hardware>File Signing Service):
https://partner.microsoft.com/en-us/dashboard/hardware/FileSign/NewLSA
Seeing that Attestation.ps1 and HLKx.ps1 are able to automate product related submissions.
May I know if there is a chance to see that the File Sign submission can also be automated?
Much Thanks!

@alastorid, are you asking if LSA signing can be requested with automation?
Currently, LSA signing is only available in the Partner Center dashboard UI. In the Product resource documentation, check out the available options for testHarness.


Fix typo in Symbol(schema_nme) on compiled schemas

Package version
1.7.0
Describe the bug
Change Symbol(schema_nme) to Symbol(schema_name)
Reproduction repo
No response

Hey @arthur-er! 👋🏻
Could you do a PR?

@RomainLanz I just did a PR fixing the typo. #36


Support for parameters of type RegExp

:rocket: Feature Request
Affected Languages

[ ] TypeScript or Javascript
[x] Python
[x] Java
[x] .NET (C#, F#, ...)

General Information

JSII Version: Any

Platform:     any

[ ] I may be able to implement this feature request

[ ] This feature might incur a breaking change

Description
I'd like to use RegExp as input for a method:
public foo(bar: RegExp) {
   ...
}

Currently this seems to be impossible, I get this compile error:
node_modules/typescript/lib/lib.es5.d.ts:970:13 - error TS9999: JSII: Cannot use private type RegExp in exported declarations

970 declare var RegExp: RegExpConstructor;

This could be tricky as there may not be a RegExp type in all programming languages... But I reckon we could vend a standard wrapper in those cases. Serialization to JSON might be another tricky thing (e.g: we need to be careful to include the modifiers, etc...).
The main pitfall however is that RegExp isn't exactly a unified standard - there are many different flavors of regexes with different capabilities, which might make it tricky to interoperate between languages in a way that is not "surprising" to users.

This would be quite a cool thing!
@RomainMuller Yes RegExp aren't a unified standard, but with JSII at least we know that the execution will be done in Node.js. That might confuse Python, Go and Java users but should be easy to document.
Do we have to provide native integration with every language's implementation? Could it mot be as simple as a class that takes a single JS compatible string?

Could it not be as simple as a class that takes a single JS compatible string?

That's effectively the same as modeling your API as accepting a string and doing new RegExp(str) on that. In fact, things become trickier when an API returns a RegExp that the consumer is supposed to make use of (because then we'd have to cycle that to JS to ensure behavior consistency). Ultimately that starts to look oddly similar to implementing a RegExp type fully in user-land and using that (which you don't need compiler support to do!)


What version should I choose

My PHP version is 5.2.17.
What version should I choose?

We don't have a PHP-CPP implementation for such an old PHP version. The oldest version that we support is 5.3. This can be used with PHP-CPP-LEGACY.


Releasing 1.0 and creating a new 2.0 branch

Hi everybody,
The 1.0.0 beta3 version is pretty stable now and used in production on a lot of projects. However there are a lot of BC breaks in dev-master (ADR pattern, immutable metadata system) and I'll add more soon (move Resource and Operation in the metadata system, decoupling from Symfony, using an external library).
A lot of projects (including commercial projets at least at Les-Tilleuls.coop) are based on the beta3 and the migration to dev-master will be hard because of the BC breaks. As I don't want to maintain a compatibility layer now (it's doable but will be a lot of work) I propose the following strategy:
Doing like this will more in the spirit of semver too.

Create a 1.0 branch starting from 1.0.0 beta3 and release a 1.0.0 version (stable).
The master branch will become 2.0 when ready

I plan to that this week. What do you think?
cc @sroze @theofidry @ogizanagi @csarrazi

It sounds great to me. Move fast to make a great library and have more traction.
But you'll have to communicate on this facts on homepage IMO.

Agreed. Regardless, a new release should not introduce BC breaks with beta-flagged releases, whereas it is still possible with alpha releases.
dev-master should be aliased to 2.0.x-dev, and a 1.0 branch created. This will let us work on documenting the upgrade path towards 2.0, by linking to both the stable and unstable versions of the code (see https://github.com/csarrazi/CsaGuzzleBundle#documentation for an example).

Agree too, create a 1.0 branch and made dev-master should be aliased to 2.0.x-dev

Regardless, this doesn't change the fact that we can (and should) change the repository's owner to the @api-platform org.

:+1:

I agree with @csarrazi, it may be better to move dev-master to the @api-platform organisation and keep this repository for 1.0.0 that can be released from 1.0.0 beta3.

For this, we should replace the original package in new repository's composer.json.
See https://getcomposer.org/doc/04-schema.md#replace

Or deprecate the old vendor, to prompt users to use the new one.

See https://github.com/composer/packagist/issues/192 and https://github.com/composer/composer/pull/3325

@dunglas : What do you think of a 1.0.0 beta4 release before doing the big BC break about Resources and Operations into the metadata system ?
The current dev-master brings some great improvements thanks to the query extensions stuff, the ADR pattern and so on.
The small BC breaks in there are quite straightforward and transparent for the classic usage of the bundle, except the disappearance of the ResourceController. The only point is that the NelmioApiDoc support is totally broken and means that something should be done about it.
Or, it might make more sense for it to be part as a alpha release of the 2.0 before the big refactoring of Resources and Operations ?

@ogizanagi Even though there are great improvements in the dev-master branch, if we want to use a tagged version, it should be as an alpha release of the 2.0 version, and not as the fourth beta of release 1.0. It makes more sense, as we should not introduce BC breaks as long as we start using beta releases in a branch.
Regardless, I do no see any issue in having a 2.0.0-alpha1 release before refactoring of Resources and Operations.
Otherwise, if we want to support forward compatibility, we could also have some work done to ensure compatibility between the latest beta in the dev-master branch, and drop the compatibility layer in 2.0. What do you think about it, @ogizanagi @dunglas @sroze @theofidry ?

Totally approved :+1:

:+1:
To sum up:

mark 1.0 here
1.0 stay in that repository
create 2 new repos: api-platform/core (the standalone library) and api-platform/ApiPlatformBundle (the Symfony integration)
Mark api-platform/ApiPlatformBundle as a replacement of dunglas/DunglasApiBundle in its composer.json
When the first release version of both api-platform/ApiPlatformBundle and api-platform/core will be marked as 2.0 to not be mingled with code in the dunglas namespace.

Extra question: how many time do we maintain 1.0?

To sum up:

mark 1.0 here

1.0 stay in that repository
create 2 new repos: api-platform/core (the standalone library) and api-platform/ApiPlatformBundle (the Symfony integration)
Mark api-platform/ApiPlatformBundle as a replacement of dunglas/DunglasApiBundle in its composer.json
When the first release version of both api-platform/ApiPlatformBundle and api-platform/core will be marked as 2.0 to not be mingled with code in the dunglas namespace.

:+1:

Extra question: how many time do we maintain 1.0?

I don't have much experience in that and I guess it also depends of how much traction there is. I guess a few months (3?) after the 2.0 is out (depends of how fast we ship the 2.0).

And @dunglas , when can we start to work on core lib ?

I've already started. I'll try to finish the decoupling this week then we can start to split the bundle and the lib.

I would actually mark 1.0 in the new org, and not here. We should tell people upfront that the repository will be migrated from api-platform, and not dunglas. This way, users will have a deprecation notice directly when running composer install on their local machines.
This is done very easily in packagist's package management webpage.

But what is the advantage to force projects to upgrade to a new namespace for 1.0? When 1.0 will be at its EOL dunglas/DunglasApiBundle can still be marked as deprecated and point to the new org.

Oh, right. I completely omitted the fact that namespaces will have to change, also...

Ok for me, then!

And the 1.0 branch is here: https://github.com/dunglas/DunglasApiBundle/tree/1.0
Can you test it before I mark the first stable release?


Typo error in file - copy-certs-tmpl

Typo error in the file (copy-certs-tmpl.YAML) causing the install script to fail. fixed the typo and now installation is successfull.

Looks good. Thanks.


auto-dispatch to dbt-sqlserver?

v0.20.0 removes the need for this repeated dispatching... right @jtcohen6? I could be overgeneralizing the macro dispatch changes to also include adapter-level dispatch changes. My understanding is that we wouldn't need the below anymore.
https://github.com/dbt-msft/dbt-synapse/blob/2004b7da6560fbe381b020680988f91c137b6029/dbt/include/synapse/macros/adapters.sql#L1-L3

Correct!

@jtcohen6 is there anything needed in this repo to specify that dbt-synapse inheirits from dbt-sqlsever? Or, will dbt-core do this automatically based on the Python class inheritance?


does the vocab.txt  come from the training file "uspto_all_reactions_training.txt"?

does the vocab.txt  come from the training file "uspto_all_reactions_training.txt"?
import rdkit
from rxnmapper.tokenization_smiles import  BasicSmilesTokenizer
basicTok = BasicSmilesTokenizer()
fh = open("uspto_all_reactions_training.txt")
vocabs = set()
for line in fh:
    if line:
        text =line.strip()
        cabs = basicTok.tokenize(text)
        vocabs |= set(cabs)

fh = open("vocab.txt")
vocabs_v1 = set()
for line in fh:
    vocabs_v1.add(line.strip())

print(len(vocabs))
print(len(vocabs_v1))
print(len(vocabs_v1&vocabs))
# print(vocabs_v1- vocabs)
# print(vocabs - vocabs_v1)

output:
>>> print(len(vocabs))
478
>>> print(len(vocabs_v1))
591
>>> print(len(vocabs_v1&vocabs))
430

the vocab.txt come from what reactions?

Good catch! We used the same vocab file as for the BERT models we developed in:
https://chemrxiv.org/articles/Data-Driven_Chemical_Reaction_Classification_with_Attention-Based_Neural_Networks/9897365
It was originally extracted from Pistachio reactions:
https://www.nextmovesoftware.com/pistachio.html


Update Yugabyte info in cockroachdb-in-comparison.md

Updating some minor errors about Yugabyte.

Database horizontal scale: Leaders in yb are automatically balanced and spread across the newly added nodes. https://docs.yugabyte.com/preview/explore/linear-scalability/
Automated repair and RPO(Recovery Point Objective): Yugabyte has 0 RPO, and 3s RTO. https://docs.yugabyte.com/preview/explore/fault-tolerance
SQL: Yugabyte is SQL compatible. https://docs.yugabyte.com/preview/api/ysql/
Data Geo-partitioning: There are CRDB blogs with examples that show geo partitioning in YB. https://www.cockroachlabs.com/blog/postgresql-compatible-database-cockroachdb/
Multi-region: Yugabyte can be deployed across multi regions with writes on both sides. https://docs.yugabyte.com/preview/explore/fault-tolerance/handling-region-failures/
Multi-cloud: Yugabyte can be deployed across multi clouds. https://docs.yugabyte.com/preview/yugabyte-platform/create-deployments/create-universe-multi-cloud/

Just a couple of points from me!

@codingconcepts Thanks for the quick review. I have addressed your comments.


feat: add support for IN queries with documents IDs

I was looking for a way to query several documents by document IDs using IN query.
It turned out that the PHP library doesn't support that at all. Please consider adding this.
Please let me know if any style updates are necessary.

I've added tests and updated code with your suggestions.

Thanks again @rcknr! This will go out with the next release (likely next week).


add more subreddits and add sidebar styling

I have not tested on all of the newly added sites yet; only r/anime, r/manga, r/awwnime, r/animenocontext, r/madokamagica so far.

As for the JSON file for the selector logic, I think we can leave that to be implemented in the future.


Add spring-rabbit explicitly to the dependencies BOM

It actually still has to be managed using a property in
spring-cloud-stream-binders-parent, but this change is for users.
Any user that wants an app using stream and rabbit needs to use the
spring-cloud-dependencies BOM. A gradle user has to do nothing else
with this change. A maven user still has to add an explicit
<spring-aqmp.version/> to the application POM.

Will merge (with indentation fixes). Also, should remove workaround for the NPE bug when declaring queues.

Actually, an additional change is required for the starters, I believe. mvn dependency:tree still brings up 1.5.3.RELEASE there.

Additional change made to starters (and rebased on master).

Plus I fixed the indentation to tabs only


Open project in same window

It would be nice to be able to define that when opening projects from the Project Manager, that the project will open in the current window. Now it opens a new window for each project.
Thanks!

What do you mean?


Allow installing a local .pkg or .tar.gz or .msi file

With rim add <filename> as usual.

For macOS, we would need to extract the R version from the .pkg file, with something like
pkgutil --bom R-4.1.2.pkg
lsbom /tmp/R-4.1.2.pkg.boms.ZOFdZ8/R-fw.pkg/Bom


Upgrade to swift 3 and iOS 10 APIs

Yeah this is going to be painful.. Trying to port my reddift project by hand is turning out to be a entirely frighteningly bad mess. Thank god for version control though. I know you'll likely have zero time however I'm just adding an issue here for future reference so that eventually someone else can try and pick this up.
I'm slowly chugging through this however I'm using a modified code base because I made some bug fixes decided (probably a bad idea at this point) to not commit them or pull new changes from git which could make this merge incompatible. Comand-Option-Control-F is your friend here (fix all issues in scope!)

I've already been your friend....
Muuuuuuu.

...what? I've finished my required changes however I'm not sure how to commit since I'm using pods not a fork.


Orange pi can use ttgo tcall to send message

After completing #6 next step is to update that script so it can utilize ttgo tcall module messaging functionality.

[x] Orange pi should update smsAllowed variable.
[x] Orange pi can send message
[ ] Orange pi can read sms and perform required task.

It can
just need to follow the rules of serial terminal.


az deployment - paramater -c leads to error while without it deployment runs sucessfully

Bicep version
Bicep CLI version 0.13.1 (e3ac80d678)
Windows 10.
Describe the bug
When calling
az deployment sub create -f "C:\src\sw\infra\run\common....\main.bicep" -l westeurope --parameters "C:\src\sw\infra\run\pre..\pre\main.parameters.pre.json" "C:\src\sw\infra\run\pre\main.parameters.extra.pre.json" -c --debug
I get
cli.azure.cli.core.azclierror: MultipleErrorsOccurred - Multiple error occurred: BadRequest,BadRequest,BadRequest,BadRequest. Please see details.
BadRequest - Deployment template validation failed: 'The template parameter 'vnetName' is not found. Please see https://aka.ms/arm-syntax-parameters for usage details.'.
BadRequest - Deployment template validation failed: 'The template parameter 'vnetName' is not found. Please see https://aka.ms/arm-syntax-parameters for usage details.'.
BadRequest - Deployment template validation failed: 'The template parameter 'sofa_appServiceName' is not found. Please see https://aka.ms/arm-syntax-parameters for usage details.'.
BadRequest - Deployment template validation failed: 'The template parameter 'sofa_appServiceName' is not found. Please see https://aka.ms/arm-syntax-parameters for usage details.'.
And when I omit parameter -c everything works.
To Reproduce
Steps to reproduce the behavior:
Additional context
It worked fine until Feb 10. From Monday it has started reporting this error. I've tried to upgrade bicep because the new version annoucment was just displayed. It didn't help.

Closing as dup of #9854


Proposed fix for https://github.com/MikeBrink/python-picnic-api/issues/23

Fixed the search based on the solution of @christof-mathies https://github.com/MRVDH/picnic-api/issues/13 as implemented by https://github.com/MRVDH/picnic-api/pull/14.

Updated headers
New search URL
Filtering the raw results from Picnic search to reproduce the old format

@MikeBrink could you look at this one? I think the Home Assistant Picnic integration is now broken because of this issue.

I can confirm that the HA integration no longer works.


RNFSManager.RNFSFileTypeRegular is undefined

Hi all,
I just downloaded this module. And when I tried to import it into my components, I was given the error "undefined is not an object (evaluating 'RNFSManager.RNFSFileTypeRegular')". I'm using React Native 0.40.0. Tried directly download the module from the current master on github, instead of using npm. Didn't work, either. Any idea why it's not working?
Natasha

RNFS is a NativeModule so it seems like react-native-fs it's not linked to the project. Did you run react-native link react-native-fs? See: https://github.com/johanneslumpe/react-native-fs#usage-ios

@hugihlynsson Using react-native link didn't do it.  I linked it manually and it worked. Thanks!

@natashache Can you please help me with how did you manually linked this module? I am stuck at the same error.

@tkale1 After doing react-native link, I followed the two steps for manual linking here https://facebook.github.io/react-native/docs/linking-libraries-ios.html.
Also, I basically gave up on this module (because of other issues) and I'm using react-native-fetch-blob instead.

there shouldnt be any linking issues if one does this the right way.
react-native init x
cd x
yarn add react-native-fs
react-native link
... add some custom code
react-native run-ios
works.

Don`t forget pod install

On android, after linking, I had to manually add new RNFSPackage() to getPackages in MainApplication.java

still auto linking doesnt work , manually it works like charm!

manually linked it but still getting an issue which isn't really clear CompileC ....

On android, after linking, I had to manually add new RNFSPackage() to getPackages in MainApplication.java

I just ran react-native link react-native-fs which did automatically add it in to the getPackages function in MainApplication.java, but still get this error when building for Android only :/ really at a loss here

This has been fixed for me, you most likely need to update your gradle version on your project

Automatic linking doesnt work.
Manual linking doesnt work following all the steps.
Installing corresponding version of library with respect to your RN version as manual says doesnt help.
What other steps should one try to have this solved?

go work in a farm

This is currently happening on RN 0.60.5


Datagrid filters broken when multiple filters are open

Select one ...  (check one with "x")
[x] bug
[ ] feature request
[ ] enhancement

Expected behavior
When opening multiple filters on a datagrid, each filter should display as "open" and operate independently.
Actual behavior

After opening one filter and leaving it open, opening another filter takes two clicks. Upon the second click, a box appears with only an "x" and no filter contents. This small box cannot be closed. Continuing to click the filter icon causes erratic behavior, such as opening new filter windows on top of the existing ones.

Reproduction of behavior
Clarity's own documentation page exhibits this behavior.
https://vmware.github.io/clarity/documentation/datagrid
Environment details

Angular version: 2.0.X

Clarity version:

OS and version:

Browser: Verified in latest Chrome and Firefox

Hi @jarinwhite: Could you please provide a plunker so that we can replicate this issue? https://plnkr.co/8TwwdL

actually nevermind. I am able to replicate it on the website. Thanks for reporting this.

Happy to help! Thank you @adityarb88 for the quick response.

Sorry to be a pain, but is there an ETA on this by any chance? I saw that it was removed from the 0.9.1 milestone so I was just curious what the status is. Thank you!

Hi @jarinwhite we discovered a couple of issues with the fix, so currently working on addressing them. I expect the fix to go out pretty soon. Best case scenario is 0.9.2 but might slip to 0.9.3.

Sounds great. Thanks @jeeyun!


WIP Improvement: Removed SkPaint Instance in LabelStyleRenderer

Removed SkPaint Instance in LabelStyleRenderer.
Reasons:

It is not ThreadSafe, all render processes access the same instance
It is not disposed.
Cleanup before I replace Obsolete Apis in https://github.com/Mapsui/Mapsui/pull/2603
It seems I fixed a Bug Because Size on Linux and Windows increased by 1 and the size on the Unit Feature Size Tests is now in the range from 40 to 42 instead of 39 to 42.
Fixed the Measurement of Text with a Halo (Added Unit Test)

Yes I agree. I think I can reduce the number of caches to one. Because the principle is always the same add key for identification of object (tuple) add a lambda that creates the object when it isn‘t in the cache. But this is for a seperate pull request. This would later help to for the splitting of rendering where Skia objects would be created and then rendered. Because with caching the same object does not create a new new Skia object.
The problem with caching and nts that an geometry can sometimes be manipulated without being noticeable from outside except comparing a cloned nts object which is quite expensive. Maybe I have to investigate there for a better solution.
I‘ll try in another pull request to remove the global render lock maybe this is fixed now with this pullrequest.


[REQ] Please add more optimisations for low end pc users

Acknowledgement of preliminary instructions

[X] I have read and acknowledged the preliminary instructions.

please add much more fast flags or things to boost fps for low end pc users
im a user of a laptop that uses only 1.7ghz and so slow and has a very old gpu and cpu and im always trynna enjoy roblox but no matter what i always try and try and keep getting :
AVG of 10-7 fps in "Slap Battles"&"Pet Simulator X"
AVG of 6-13 fps in "Arsenal"&"Murder Mystery"
AVG of 11-14 fps in "Ability Wars"&"Hours"
AVG of 5-9 fps in "EVADE"&"Blox Fruits"
and i cant afford a new laptop or something and i really want to feel good once in my life and get a bit of fps
i would really like 25-30 fps i bet its gonna be AMAZING! so yeah.. please add more optimisations and fps boost things into the game i've tried EVERYTHING and i mean EVERYTHING i could..

-Thank you

Bloxstrap is just a bootstrap, so it cannot modify game clients code (other wise Hyperion would block it), but you can take a look at FastFlags channel on discord, there is a list of FFlags that might optimise the game a little bit

We're already planning to add a bunch of options in the next version that can help to improve performance on low end hardware. Though, thank you for letting us know that you'd like for this to happen.


Stripped out the multithreading support to migrate it to a new project.

Removed all of the multithreading work to a new branch called "migrate_multithreading..", it turns out it breaks on micropython (not good!) so alas. We'll create yet another version.

No-fix, we're going to release a different version without mt


[MGDAPI-2729] feat: set prometheus storage configuation in observability

Issue link

https://issues.redhat.com/browse/MGDAPI-2729
What

Add prometheus storage configuration so data with be retained when pod moves between nodes on restarts
The configuration should be the same as the one used previously in AMO:

https://github.com/integr8ly/integreatly-operator/blob/master/pkg/config/monitoring.go#L153-L159

Verification steps

Checkout this branch
Install RHOAM

export INSTALLATION_TYPE=managed-api
make cluster/prepare/local
make code/run

Once observability completes installation

Verify retention and storage is specified in Prometheus CR

 oc get prometheus kafka-prometheus -n redhat-rhoam-observability -o json | jq '.spec.storage'
 oc get prometheus kafka-prometheus -n redhat-rhoam-observability -o json | jq '.spec.retention'

Verify PVC is created for prometheus

oc get pvc -n redhat-rhoam-observability

Reviewed changed files and verified that the retention and storage fields are set in the Prometheus CR to the values used in AMO. Verified that PVC is created for Prometheus.
/lgtm


test_domain.py has one failed test case.

_965,_tcp.example.net is a failing test, not a passing test for domain_fqdn function.
]

Fixed in 0.99.2 dd9fe1acdeb74029fa966b15287ba22062de342c


Merge to production

Issue

Link:
Summary:

How

Screenshots

References

Checklist

[ ] Tests added
[ ] Cleaned up commit history (rebase!)
[ ] Documentation added
[ ] Thought about variable and method names

Do not merge until this is fixed... https://github.com/NebraLtd/hm-pyhelper/issues/270


Added output after successful reset

The user currently just sees an output of the deinstallation which is not statisfying.
PS C:\Users\Joachim> centos reset
This will remove this distro (CentOS) from the filesystem.
Are you sure you would like to proceed? (This cannot be undone)
Type "y" to continue:y
Unregistering...
PS C:\Users\Joachim> centos
[root@DESKTOP-SS5S73V ~]#

Thanks


Magento2 doesn't work with add-on: "Cannot find the main route for Magento. Please add id: magento_route to your routes.yaml"

when trying to use this with platform.sh and magento2 we are getting the following error and it does not fully deploy.  I have redacted the project and branch..
output.txt

I believe the plain-vanilla https://github.com/platformsh-templates/magento2ce results in this problem, could you please confirm that?

correct.

See: https://github.com/platformsh-templates/magento2ce/blob/master/deploy.php#L93
and
https://github.com/platformsh-templates/magento2ce/blob/master/.platform/routes.yaml#L2

IF I remember correctly, the Magento installer requires a base-url. Since any number of routes/domains can be "primary" routes, there needs to be a reliable way to get what is ultimately the default domain.
@tylerssn does that sound correct?
This isn't working with the add-on since we aren't actually parsing the routes.yaml file yet but statically creating the route information for ddev.
The script itself is running because it checks to see if the file app/etc/installed exists, and if not, proceeds to attempt to install Magento. app/etc is a file mount: https://github.com/platformsh-templates/magento2ce/blob/master/.platform.app.yaml#L56
In this case, you'll need to do a ddev pull platform and get the db and mounts before you run ddev start so the files are available locally for when ddev runs the hooks.

IF I remember correctly, the Magento installer requires a base-url. Since any number of routes/domains can be "primary" routes, there needs to be a reliable way to get what is ultimately the default domain.

That's right, we're essentially being explicit about which domain we want to use and we don't have a "give me the default domain" option (that I know of) that could replace this logic.
That said, Magento doesn't technically require that a base URL be set. If it is not set, Magento will answer a request made to it from any domain. I'm not sure what the impact of that would be on this setup script—I haven't looked at it in a while 😄

The "Cannot find the main route" issue is now resolved.
Now we're dependent on getting the template fixed:

https://github.com/platformsh-templates/magento2ce/pull/54


Timeout argument causes problems in sf::TcpSocket::connect()

I tried using sf::TcpSocket recently, and I decided to use the timeout argument.
When I set it to, let's say sf::seconds(5), and if it fails to connect once, the instance of the socket will no longer try to connect, and always fail.
So is this behavior intended or did you just forget to reset timeout state after a connection failed?
Library: SFML 2.4.2-4
OS: ArchLinux x86_64

@Quent42340 Try out PR #1408.


Move "keyboard layout" from VM creation to Settings

Acceptance criteria:

No keyboard layout in VM creation
New section "VM preferences" in Settings (above "Look and Feel"). Put "Keyborard layout" there.
Default - "Standard (US) keyboard"

Also add information in Wiki -> Tags
Connected feature ID: vm_creation_keyboard, settings_keyboard

Failed.

Value in "Keyboard layout" field doesn't translate when change language in settings; (All browser)
An error occurs (screenshot) and value in "Keyboard layout" doesn't save when used on Windows 10 and browser IE (ver. 11.192.16299.0). Value of keybord in GET request is "undefined" when create a new VM;

@rennervo The second one error will be a separated task

It is necessary to show the readiness of changes of the "Keyboard layout" parameter using the value color. (as in the parameters: "First day of the week" and "Time format").
Actual result: New value of keyboard layout field has black color immediately after beginning changed.
Expected result: Gray color of value of keyboard layout field after beginning of parameter change and black color of this value after ending changed of parameter.
Screenshot of actual result:

Screenshot of expected result:

@bwsw Should this parameter be configured in the config or not?

@tamazlykar you can implement that, but not necessary.

Tested on 1220

There is no item "Simplified Chinese keyboard" in VM terminal, therefore, it makes no sense to choose this keyboard value in the settings. It is necessary to discuss this problem with the PM or PO.

Keyboard field is empty in settings for a new user when the first login.
Steps:
1 Create a new user with role Domain Admin or User
2 Login under created user
3 Go to Settings

Actual result: Keyboard field is empty
Expected result: Keyboard field has value "Standard (US) keyboard"

Keyboard field value is reset when refreshing a page
Steps:
1 Log in and go to Settings
2 Select any value in keyboard field
3 Refresh page

Actual result: There is no value in the keyboard field, even after logout and login
Expected result: The keyboard field contains the selected value

The keyboard field value is not translated after changing the language
Steps:
1 Log in and go to Settings
2 Change the language

Actual result: The keyboard field value is not translated
Expected result: The keyboard field value is translated

Lack: 4
Console error with a text "Cannot read property 'lift' of null" appears when VM creating
(This error is not repeated on master)
Steps:

Log in as user or domain admin
Go to VM and click to "Create" button
Click to "CREATE"

Actual result: An error occurs during the creation of the VM, the window of creating is not closed
Expected result: VM is created without errors
FEATURE ID: vm_deploy
Screenshot:


Textfile initializer sharing bug

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
binary
TensorFlow version
2.15
Custom code
Yes
OS platform and distribution
No response
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
The logic here for TextFileInitializer sharing does not consider key/value dtype. If two text file initializers are made for same vocab file but with different dtypes (tf.int64/tf.string), then sharing will cause one of them to fail and crash.
The fix should be small change to add dtypes to shared_name.
Standalone code to reproduce the issue
import os
import tempfile

import tensorflow as tf
from tensorflow.lookup import TextFileIndex

with tempfile.TemporaryDirectory("w") as tmp_dir:
    vocab_file = os.path.join(tmp_dir, "vocab.txt")
    with open(vocab_file, "w") as f:
        f.write("\n".join(["1", "2", "3"]))

    initializer1 = tf.lookup.TextFileInitializer(
        vocab_file, tf.string, TextFileIndex.WHOLE_LINE, tf.int64, TextFileIndex.LINE_NUMBER
    )
    table1 = tf.lookup.StaticHashTable(initializer1, -1)

    initializer2 = tf.lookup.TextFileInitializer(
        vocab_file, tf.int64, TextFileIndex.WHOLE_LINE, tf.int64, TextFileIndex.LINE_NUMBER
    )
    table2 = tf.lookup.StaticHashTable(initializer2, -1)

    _ = table1.lookup(tf.constant(["1", "2", "3"], dtype=tf.string))
    _ = table2.lookup(tf.constant([1, 2, 3], dtype=tf.int64))

Relevant log output
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-12-13 13:50:41.113925: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at lookup_table_op.h:94 : INVALID_ARGUMENT: Conflicting key/value dtypes int64->int64 with string-int64 for table hash_table_/var/folders/3g/0v44vhyd05q6n_77rl_jnh6r0000gn/T/tmpiimi4gc8w/vocab.txt_-2_-1
Traceback (most recent call last):
  File "/Users/pa-loaner/.pyenv/versions/3.9.16/lib/python3.9/pdb.py", line 1726, in main
    pdb._runscript(mainpyfile)
  File "/Users/pa-loaner/.pyenv/versions/3.9.16/lib/python3.9/pdb.py", line 1586, in _runscript
    self.run(statement)
  File "/Users/pa-loaner/.pyenv/versions/3.9.16/lib/python3.9/bdb.py", line 580, in run
    exec(cmd, globals, locals)
  File "<string>", line 1, in <module>
  File "/Users/pa-loaner/Snapchat/Dev/training-platform/scratch/text_file_bug.py", line 1, in <module>
    import os
  File "/Users/pa-loaner/Snapchat/Dev/.venvs/bento/lib/python3.9/site-packages/tensorflow/python/training/tracking/resource.py", line 104, in __call__
    return previous_getter(*args, **kwargs)
  File "/Users/pa-loaner/Snapchat/Dev/.venvs/bento/lib/python3.9/site-packages/tensorflow/python/training/tracking/resource.py", line 99, in <lambda>
    previous_getter = lambda *a, **kw: default_resource_creator(None, *a, **kw)
  File "/Users/pa-loaner/Snapchat/Dev/.venvs/bento/lib/python3.9/site-packages/tensorflow/python/training/tracking/resource.py", line 96, in default_resource_creator
    obj.__init__(*a, **kw)
  File "/Users/pa-loaner/Snapchat/Dev/.venvs/bento/lib/python3.9/site-packages/tensorflow/python/ops/lookup_ops.py", line 347, in __init__
    super(StaticHashTable, self).__init__(default_value, initializer)
  File "/Users/pa-loaner/Snapchat/Dev/.venvs/bento/lib/python3.9/site-packages/tensorflow/python/ops/lookup_ops.py", line 198, in __init__
    self._resource_handle = self._create_resource()
  File "/Users/pa-loaner/Snapchat/Dev/.venvs/bento/lib/python3.9/site-packages/tensorflow/python/ops/lookup_ops.py", line 360, in _create_resource
    table_ref = gen_lookup_ops.hash_table_v2(
  File "/Users/pa-loaner/Snapchat/Dev/.venvs/bento/lib/python3.9/site-packages/tensorflow/python/ops/gen_lookup_ops.py", line 466, in hash_table_v2
    _ops.raise_from_not_ok_status(e, name)
  File "/Users/pa-loaner/Snapchat/Dev/.venvs/bento/lib/python3.9/site-packages/tensorflow/python/framework/ops.py", line 7164, in raise_from_not_ok_status
    raise core._status_to_exception(e) from None  # pylint: disable=protected-access
tensorflow.python.framework.errors_impl.InvalidArgumentError: Conflicting key/value dtypes int64->int64 with string-int64 for table hash_table_/var/folders/3g/0v44vhyd05q6n_77rl_jnh6r0000gn/T/tmpiimi4gc8w/vocab.txt_-2_-1 [Op:HashTableV2] name: hash_table
Uncaught exception. Entering post mortem debugging
Running 'cont' or 'step' will restart the program
> /Users/pa-loaner/Snapchat/Dev/.venvs/bento/lib/python3.9/site-packages/tensorflow/python/framework/ops.py(7164)raise_from_not_ok_status()
-> raise core._status_to_exception(e) from None  # pylint: disable=protected-access

Hello, @hmc-cs-mdrissi!
Thank you for the suggestions!
We need to thoroughly test any modifications to ensure correct functionality and address potential edge cases.
Do you want to modify the logic to include dtypes in the shared_name calculation.
This ensures initializers with different dtypes get distinct shared names, preventing conflicts.
Example: shared_name = f"{vocab_filename}_{key_dtype}_{value_dtype}"
Please let us know the exact changes you are suggesting for documentations which would be helpful to fix it.
Thank you!

Yes I think modifying logic to include dtypes in shared names would be a good fix and is enough. I should have time tomorrow and can make a small PR.

The bug described refers to specific function in TextFileInitializer. That code has not changed and still has same issue. The code sample I gave should still crash on tensorflow master. Nothing has changed for this issue.


Can't copy paste in Explorer

Copy a file in the Explorer
Paste

🐛 It doesn't work.
Found related copy paste issue: https://github.com/microsoft/vscode/issues/200597 cc @alexdima @mjbvz

/duplicate of #200581

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


PTX-1261 Fix a problem when a pod got rescheduled before mount check

tested passed here https://jenkins.portworx.co/job/Torpedo/view/Torpedo Dashboard/job/tp-2-1-0-basic/23/console

I see you are still making changes to the PR. Let me know when it's ready for review.


[ASCII-2435] Hostname resolution for IMDSv2 transition

What does this PR do?
This PR aims to add legacy-resolution-hostname to metadata payload, which will contain the hostname fetch without IMDSv2 scenario.
It also adds ec2_imdsv2_transition, when this config flag is set, IMDSv2 is used by default, if IMDSv2 call fails then it falls back on DMI method to fetch the instance ID.
It also send legacy-resolution-hostname in the hostname metadata payload if it is different from the hostname
Motivation
Some dashboards could be impacted by IMDSv2 transition, this PR aims to minimise this impact by saving the actual hostname with a new field if the hostname is changing
Describe how to test/QA your changes
Done
Possible Drawbacks / Trade-offs
Additional Notes

/trigger-ci --variable RUN_ALL_BUILDS=true --variable RUN_KITCHEN_TESTS=true --variable RUN_E2E_TESTS=on --variable RUN_UNIT_TESTS=on --variable RUN_KMT_TESTS=on

/merge

/merge


[Event Request] - OnBeforeLookupOperationType in Table 36 "Sales Header"

Hello, please consider adding the following event to Table 36 "Sales Header"
    [IntegrationEvent(false, false)]
    local procedure OnBeforeLookupOperationType(var NoSeries: Record "No. Series"; var SalesHeader: record "Sales Header"; var xSalesHeader: record "Sales Header")
    begin
    end;

and call it accordingly inside "OnLookup" trigger of field 12100 "Operation Type":
            trigger OnLookup()
            begin
                **OnBeforeLookupOperationType(NoSeries, Rec, xRec);**

                if PAGE.RunModal(PAGE::"Operation Types", NoSeries) = ACTION::LookupOK then
                    Validate("Operation Type", NoSeries.Code);
            end;

Thank you!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 49950.


Update gradle to the latest release

The template currently uses 4.8.1.
The latest release is 5.1.1.

Hi @tim-hellhake
Thank you for your interest in the project.
Please fill out the whole issue template, this is not optional.
https://github.com/openfaas/templates/blob/master/.github/ISSUE_TEMPLATE.md
Alex

Derek close: fixed, latest is now live.