[Metrics] Supplement extra metrics for monitoring the amount of recycled logs.

Description
This pr supplements the necessary metrics for monitoring the amount of recycled logs.

@tabokie PTAL, thx.


Default font changed for results grid

SQL Operations Studio Version: 0.27.2

Steps to Reproduce:

Run a query

Expected: Default font is same as 0.27.1
Actual: Default font is different

@anthonydresser I think this is likely related to the grid options fix we merged yesterday.  Is it easy to get the default font to be the same as before, or should I revert that change?

Also, it looks like the Grid font settings are also changing the tab font settings.

It uses the same default font as the editor.

If we plan to change the default font from what we've previously shipped then we'll need to go through a UX review process.  This isn't the type of change we can slip in through a bug fix, but needs to be an explicit design change.
I thought we already went through a similar process with the vscode-mssql extension, and we got it right there IIRC.  I don't remember the exact details, but I think we ended up with separate font settings for editor and result grid, similar to what we support in SSMS.
If there is an easy fix then let's do that.  Otherwise, we should rollback this change until the UX changes are reviewed, or we confirm we aren't significantly changing the UX we previously shipped.

...moving to April since I'll revert the grid options change in March release branch.


Examples/readme

Coverage increased (+0.005%) to 63.089% when pulling bf39af6c0ea4935946dfa378d9fd66d785659048 on examples/readme into e6bc7dd8951410b733bb85091d0e4506c25e6f70 on master.

Coverage increased (+0.005%) to 63.089% when pulling bf39af6c0ea4935946dfa378d9fd66d785659048 on examples/readme into e6bc7dd8951410b733bb85091d0e4506c25e6f70 on master.


Add the fabric8-tenant-che-migration job.

This PR is to setup the CI build of the single-tenant to multi-tenant migration tool.

Can one of the admins verify this patch?

[test]

👍


New Fitbit Connect version - <IP_ADDRESS>89

Bump fitbit-connect to the latest version - <IP_ADDRESS>89

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Thanks for the PR!


Copy sample code issue

Describe the bug
A few weeks ago, I told Rick that I was having a problem when I copied the sample course code into my project, but couldn't quite get a grip on what was happening.
I think that I have eventually tracked it down do a Visual Studio!
If I copy the sample code (from ANY tutorial) and paste it DIRECTLY into VS, I get problems. (especially when I copy the code for a MVC VIEW.)
However, If I first save the text in notepad++ (or simply Notepad) and then put the file into the project, there are no issues.
To Reproduce
Steps to reproduce the behavior:

Using Visual Studio (Ver 15.8.5) and any version of ASP.NET Core from V2.0 (I haven't tried older versions)

Copy the sample code from a tutorial and paste DIRECTLY into the project in Visual Studio.
(I have experiences a problem with all of the following tutorials.)
https://docs.microsoft.com/en-us/aspnet/core/data/ef-mvc/intro?view=aspnetcore-2.0
https://docs.microsoft.com/en-us/aspnet/core/data/ef-rp/intro?view=aspnetcore-2.1&tabs=visual-studio
https://docs.microsoft.com/en-us/aspnet/core/tutorials/first-mvc-app/?view=aspnetcore-2.0
https://docs.microsoft.com/en-us/aspnet/core/tutorials/razor-pages/?view=aspnetcore-2.0

With these arguments '....'

See attached screenshot for errors

Expected behavior
Visual Studio reports errors (particularly when copying the code for the VIEWS.)
Screenshots
In the attached image, the files surrounded by the red line, were straight copy/pastes, whereas the files in the blue area were 'washed' in notepad++, before being saved in my project.
(sorry about the squiggly line....I am trying out a RollerMouse)
Additional context
Add any other context about the problem here.
Include the output of dotnet --info

Thanks for contacting us, @lm28.
@NTaylorMullen, is this something you're familiar with? Thanks!

@lm28 i'm unable to reproduce your issue. It looks like you have some other Visual Studio extension installed however, normal VS doesn't red curly underline files for instance. My bet is that you're seeing issues due to an installed Visual Studio extension you have.


[Bug] No height overlays in textarea and images

Branch 1.2
https://github.com/user-attachments/assets/4ac844ff-8fc4-4da0-b63e-0ecfa4e1a42b

This is fixed!


Handling of anchors/deprecated anchors

It seems that now we can't edit or remove anchors added to the text block components. Therefore we can't copy a text block component containing an anchor and reuse it either, since we then will have identical anchors after copying the component (in the copied text component).
Is it decided that we only will have anchor links in the separate anchor link component, and not e.g. in the text block components - where we have added anchors?
Is the idea that we need to recreate all anchor links?
Is it possible to have both a separate anchor component and have it as an option in some components

Check to see if we can add new anchor with the same value as a deprecated anchor

Made the deprecated anchor writable only to clear the old value.

Changes were applied only to the anchor component inside Accordion. There are a lot more places where the anchorReferenceField is used. I believe what you have to do is to add the rule in the anchorReferenceField component itself?
I am also not sure if I agree with the logic, did you check if adding a new anchor with the same value as the old one works? If it does we probably don't need to change anything...
On the other hand, if the plan is to allow people to delete it, but not to add new ones, we could also disable the readOnly when the field is empty.
readOnly: ({ value }: { value?: string }) => !value,
Maybe discuss with @meols to understand what is the best scenario here?

I like this suggestion

On the other hand, if the plan is to allow people to delete it, but not to add new ones, we could also disable the readOnly when the field is empty.

Editors can modify the existing value in the deprecated anchor field. The field is disabled if there is no value, to prevent use if that deprecated field.
They can also modify an existing value of deprecated field. ( A warning will be shown not to use the field).

@meols The deprecated anchor fields in the TextBlock and Accordion components are now made editable with a warning. However if the anchor field is cleared.. the field becomes read only ( simply it is not possible to use deprecated field if its empty )

Thanks @padms  - this was a good solution :thumbsup:


[DOC]: Building docs for the first time creates empty python api docs

How would you describe the priority of this documentation request
Medium
Please provide a link or source to the relevant docs
https://github.com/nv-morpheus/Morpheus/blob/branch-23.03/docs/README.md
Describe the problems in the documentation
Looks like the docs need the python modules to be installed in the current env prior to building.
(Optional) Propose a correction
Remove the documentation target from the default install, similar to MRC where we first build the code, then build the docs.
This would also be a developer workflow improvement to be able to configure morpheus with -DMORPHEUS_BUILD_DOCS=ON but not need to build the docs for each build iteration.
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I have searched the open documentation issues and have found no duplicates for this bug report

No longer relevant, as docs are no longer in the default target.


Textcomplete suggestions not working when applied ckeditor plugin

Hi , I applied ckeditor plugin to a textarea , then when i tried to apply textcomplete suggestions its not working , even the ajax request is not going ,
$('#edit-email-template').textcomplete([{
match: /\B:([-+\w]*)$/,
search: function (term, callback) {
$.getJSON('/notification/findAllUserVariables')
.done(function (resp) {
variables = resp.variable_name;
alternates = resp.content;
search = resp.term;
callback($.map(variables, function (n, i) {
return  i;
}));
})
.fail(function () {
callback([]);
});
},
template: function (value) {
return  variables[value];
},
replace: function (word) {
return ' ' + alternates[word] + ' ';
}
}]);
And  i given 'ckeditor' class  for the textarea , thats all .

#90 duplicated

But its not actually resolved my problem . can you give a snapshot on how to do it

+1


Race condition with babel-plugin-transform-export-extensions

using a react app if I have something setup similar to this:
app/containers/FilterLink.js
import { connect } from 'react-redux'
// not doing a import from ../components because of
// race condition of link not being in the app/components/index.js yet
import Link from '../components/Link'
import { setVisibilityFilter, toggleTodo } from '../actions'

const mapStateToLinkProps = (
  state,
  ownProps
) => {
  return {
    active:
      ownProps.filter ===
      state.visibilityFilter
  };
};

const mapDispatchToLinkProps = (
  dispatch,
  ownProps
) => {
  return {
    onClick: () => {
      dispatch(
        setVisibilityFilter(ownProps.filter)
      );
    }
  };
}
export default connect(
  mapStateToLinkProps,
  mapDispatchToLinkProps
)(Link);

app/components/index.js
export Link from './Link'
export Footer from './Footer'
export Todo from './Todo'

index.js
import React, { Component } from 'react';
import ReactDOM from 'react-dom';
import Redux, { combineReducers, createStore } from 'redux';
import { Provider, connect } from 'react-redux';
import { render } from 'react-dom';
import todoApp from './app/reducers'
import { addTodo, toggleTodo, setVisibilityFilter } from './app/actions'
import { Link, Footer, Todo } from './app/components'
// if I was to do the import as I stated in app/containers/FilterLink.js
// I found that Footer would be undefined because of the race condition

Is this expected, or a bug?
If you think it's a bug. I can try to recreate the code

Thanks for reporting! However repo is for reporting issues/suggestions on our website, http://babeljs.io/, not for bugs in babel or support.
If you have an an actual bug/suggestion for babel itself - free feel to make an issue on our phabricator. If you just have an usage issue or want help, I would recommend checking out our slack community or StackOverflow.


Attachments: Implement batch upload and annotation

This needs a design... stop me before I wing this mutha


duration_segment : using PS0 instaed of  'trap DEBUG'

Both pureline 'duration_segment' and vscode 'shell integration' install a DEBUG trap to enhance the shell experience
But both work by sourcing a script, and by default the DEBUG trap is not inherited in that case.
Solution 1 (rejected)
We could tell the user to add those lines in its .bashrc :
if [[ $(trap -p DEBUG) =~ ^trap ]]; then
    set -o functrace # same as 'set -T'
fi

... but this may have side-effects since it does MUCH MORE than what we wants :
set -o functrace
set -T
       If set, any traps on DEBUG and RETURN are inherited by shell functions,
       command substitutions, and commands executed in a subshell environment.
       The DEBUG and RETURN traps are normally not inherited in such cases.

Thus this solution is rejected.
Solution 2 (adopted)
In Bash 4+, the value of PS0 is expanded and displayed by interactive shells after reading a command and before the command is executed.
PS0 is not evaluated for empty commands, which leaves a blank line.
We can use "Arithmetic expansion" to run __timer_start=$(timer_now) inside PS0 :

It runs in the current shell instead of a sub-shell and can assign to variables.
It also produces output, which we consume with ${0:0:$((...,0))}, to output nothing.

The solution is to replace the DEBUG trap with this line:
PS0+='${0:0:$((__timer_start=$(timer_now),0))}'

Example :
Using pureline inside a vscode terminal where 'shell integration' is enabled.
Before (using the DEBUG trap) :

After (using PS0) :

Note: PS0 is not evaluated for empty commands, which leaves a blank line.

I solved conflicts with the base branch


Scripts for Deployment Process

@stefanw @fin we've me & @robinharms have been e-mailing about issues on how to get FROIDE up and running.
Could you please make config, Ansible Script accessible or share with us?
@robinharms please add any extra script requirements that would help you make it work.

Just a list of dependencies (like libmagic) and needed services. Example configs would be really useful.

Currently FragDenStaat.de deployment is still a fab script, but it only takes care of deployment (= updating code, dependencies, running migrations etc. and restarting services), not of provisioning (which is the hard part.
I just updated froide-theme and pushed my current (not yet fully working) version of a dockerisation of froide and fragdenstaat. This should contain some hints on how to set it up.
I'd be happy to continue to develop this with you guys, if it makes sense to you. Ansible may be another good way to get things started.

Aloha! :D We're thinking about deploying Froide in Hawaii. Awesome work, btw.  I have a set of (fairly opinionated) Ansible scripts that currently let us deploy this - http://beta.uipa.org using Terraform  + Ansible to Digital Ocean. They may have to be tweaked a little, but let me know the best way to share them.

@ryankanno do you think you could share it as a gist?
i'm in the process of looking over my scripts to make them publish-able, but i need to be sure there are no keyfiles in there, etc.

@fin Absolutely. I'm in the process of cleaning them up right now.  I'll link the gist here once I get everything cleaned up. I also have a topic branch that I'll link to see the commits, etc. What's the best strategy to update the Froide settings? Update settings.py with a new class re: class Vagrant(Base):

@ryankanno re froide settings: i'd suggest creating a repo based on froide-theme and adding custom config there, as i did here: https://github.com/fin/fragdenstaat.at/blob/master/froide_theme/settings.py
(and as stefan does with fragdenstaat.de as well)

@fin Thanks for that tip. Definitely going down that route. There were 22 files changed so I wasn't sure how that would look as a gist. I did check all my changes into a branch. I tried to document what I did in a Vagrantfile readme.

@fin As I was going through setting up the application and what not, I also realized that we didn't set up elasticsearch. Will be updating the branch to take care of that.

@robinharms @mattiasaxell i'll be (and have been) hard to reach for a week or so due to the death of a dear friend. sorry for the delay.
i was 95% done with the deploy script, but it seems like @ryankanno has you mostly covered.
(my deployment even runs with whoosh instead of elasticsearch - which should currently be the default - without performance issues for now)

I'm sorry for your loss @fin, sad to hear that! Thanks a lot for taking time to answer.
@ryankanno is your custom config working properly in production stage? In that case we'd love to have a look over it. Is it a little cleaned up now and ready for release?

@fin My condolences to you and yours.
@mattiasaxell @robinharms I haven't had time to configure the search index yet in this branch. I'll be getting to it shortly. With that said, I've set it up with Vagrant in that branch, so you should be able to play around / customize it in isolation fairly easily. Let me know if you need anything else from me.

@fin beautiful gift at the online memorial website!
@ryankanno alright, so it would be soon ready to be up and running in production mode? We would love to have your list of dependencies, is that already set up in the branch? To me it looks like it, but I'm new to this area. Also, would it be much work to change it to Debian? :)

@mattiasaxell If you want to ping me off this github issue, I can try to help you get it setup.  I'm in the process myself of trying to get it productionized. As a note, that Vagrantfile doesn't deal with securing the machine since that's a bit out of scope (and a lot of personal idealogy). :)

@ryankanno I'd love to, how can I ping you outside the issue? I'm thiking maybe it is wise if you @robinharms if you can help. He will be looking at your script when he tries to set it up for production purpose on Tue 6th via Debian-based VPS. Do you think you can have it ready by then?
Securing the machine, do you mean as in firewall or stability? I'm new to system deployments. :)


Modified Readme

In reference to #2 . @Marchrius Tell me if I need to add some more things

Thanks for the help, but I want a descriptive README.md that descript the functionality of the plugin

@Marchrius Can you elaborate on functionality so that I can include it in my commits ?


Added virtual SettingInterface default destructor

Issue #, if available:
https://github.com/alexa/avs-device-sdk/issues/1245
https://github.com/alexa/avs-device-sdk/issues/1136
https://github.com/alexa/avs-device-sdk/issues/1161
https://github.com/alexa/avs-device-sdk/issues/1165
Description of changes:
Added virtual destructor for SettingInterface
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hi @jbfitbit,
Thanks for the pull request. This change had already been fixed in 1.12.


Feature: Register commands with update options

Description
Allow the third argument to editor.registerCommand() to take an object that includes the priority as well as the update options. registerCommand() could then be called as follows:
editor.registerCommand(COMMAND_NAME, commandFunction, COMMAND_PRIORITY_LEVEL);

or
editor.registerCommand(COMMAND_NAME, commandFunction, {
  discrete: true,
  onUpdate: // post-update function,
  priority: COMMAND_PRIORITY_LEVEL,
  skipTransforms: true
});

This would let us manage the command with more finesse, in situations where we need to use these options to narrow the scope of the command's changes.
For my current use case, I have an update listener that submits user input to the server. I also have a command that updates the editor with data from the server. While the command is operating, I don't want the listener to be active, or else it creates a feedback loop.
Currently I'm setting a flag in the command, and the update listener knows not to send to the server while that flag is set. I want to be able to unset the flag in an onUpdate callback function, but can't do so directly in the command. Right now I queue a microtask to do so, but that seems "unLexical."

I think the biggest issue here is that you can register multiple listeners for the same command and it doesn't always make sense to combine these sorts of options (skipTransforms and discrete aren't very composable, if any update has them then its effect happens for all updates in that batch).
Having it be possible to add an onUpdate from within an update makes sense, I think a better solution for that would be to have a dollar function to do that. The implementation would be basically function $addOnUpdate(fn) { $getEditor()._deferred.push(fn); }

That makes sense. Should this be closed and the $addOnUpdate proposal be introduced in a new ticket?

I don't think a new issue needs to be created, if you wanted to create a PR for it with a test then it would probably happen quickly. I didn't think too much about the name, maybe $onUpdate or $afterUpdate would be more clear.


'pydantic.error_wrappers.ValidationError: 12 validation errors for GranuleFeed' error when download the MOD13Q1.061 product

Describe the bug
When donwload the MOD13Q1.061  product using the API (code is attached below), the   pydantic.error_wrappers.ValidationError: 12 validation errors for GranuleFeed comes out.
To Reproduce
Steps to reproduce the behavior:
snwe = [34.5, 34.74, -91.5, -91.26]
start_date = "2022-01-01"
end_date = "2022-02-01"

S, N, W, E = snwe

# Reusable session
session = ModisSession(username=username, password=password)
collection_client = CollectionApi(session=session)

collections = collection_client.query(short_name="MOD13Q1", version="061")
granule_client = GranuleApi.from_collection(collections[0], session=session)

granules = granule_client.query(
    start_date=start_date, end_date=end_date, bounding_box=[W, S, E, N]
)

print(list(granules))

Expected behavior and actual behavior
I expected to download MOD13Q1.061  tiles. However, the following exception occurred:
Traceback (most recent call last):
File "/Users/xeno/Documents/carbon/carbon/rsinputs/modis/modis.py", line 90, in
print(list(granules))
File "/Users/xeno/miniconda3/envs/co2/lib/python3.9/site-packages/modis_tools/resources.py", line 119, in query
granule_feed = GranuleFeed(**feed)
File "pydantic/main.py", line 341, in pydantic.main.BaseModel.init
pydantic.error_wrappers.ValidationError: 12 validation errors for GranuleFeed
entry -> 0 -> links -> 1 -> href
URL scheme not permitted (type=value_error.url.scheme; allowed_schemes={'https', 'http'})
entry -> 0 -> links -> 4 -> href
URL scheme not permitted (type=value_error.url.scheme; allowed_schemes={'https', 'http'})
entry -> 0 -> links -> 7 -> href
URL scheme not permitted (type=value_error.url.scheme; allowed_schemes={'https', 'http'})
entry -> 0 -> links -> 9 -> href
URL scheme not permitted (type=value_error.url.scheme; allowed_schemes={'https', 'http'})
entry -> 1 -> links -> 1 -> href
URL scheme not permitted (type=value_error.url.scheme; allowed_schemes={'https', 'http'})
entry -> 1 -> links -> 4 -> href
URL scheme not permitted (type=value_error.url.scheme; allowed_schemes={'https', 'http'})
entry -> 1 -> links -> 7 -> href
URL scheme not permitted (type=value_error.url.scheme; allowed_schemes={'https', 'http'})
entry -> 1 -> links -> 9 -> href
URL scheme not permitted (type=value_error.url.scheme; allowed_schemes={'https', 'http'})
entry -> 2 -> links -> 1 -> href
URL scheme not permitted (type=value_error.url.scheme; allowed_schemes={'https', 'http'})
entry -> 2 -> links -> 4 -> href
URL scheme not permitted (type=value_error.url.scheme; allowed_schemes={'https', 'http'})
entry -> 2 -> links -> 7 -> href
URL scheme not permitted (type=value_error.url.scheme; allowed_schemes={'https', 'http'})
entry -> 2 -> links -> 9 -> href
URL scheme not permitted (type=value_error.url.scheme; allowed_schemes={'https', 'http'})
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: MacOS Ventura 13.0

Your Environment:

modis_tools version used: 1.1.0
Other dependencies and packages installed in environment you are using (e.g. output of pip freeze or conda list or poetry show):

Any Additional context
Add any other context about the problem here.

Hi @xeno0, thanks for submitting this issue. Our team has received it and loaded it into our internal issue tracking system. We hope to address it in the coming weeks

Hi @xeno0 ! The modis-tools team recently updated our pydantic validation to use anAnyUrl format (a more flexible validation than the previous allowed_schemes={'https', 'http'}) that you were receiving as an error (see https://github.com/fraymio/modis-tools/commit/c2d8c6fb3c37a530affc7b910dc03c0d419fe908).
I was able to replicate your code just now and downloaded the requested granules.
After pulling the latest code, let us know if you have any similar problems! Thanks for raising this issue

Hi @ilsep93 I tested this commit, it seems that there are still some errors on my end, please see below, thank you
Downloading:   0%|                                                                                | 0/2 [00:00<?, ?file/s]
concurrent.futures.process._RemoteTraceback:
"""
Traceback (most recent call last):
  File "/Users/xeno/miniconda3/envs/co2/lib/python3.9/concurrent/futures/process.py", line 246, in _process_worker
    r = call_item.fn(*call_item.args, **call_item.kwargs)
  File "/Users/xeno/miniconda3/envs/co2/lib/python3.9/concurrent/futures/process.py", line 205, in _process_chunk
    return [fn(*args) for args in chunk]
  File "/Users/xeno/miniconda3/envs/co2/lib/python3.9/concurrent/futures/process.py", line 205, in <listcomp>
    return [fn(*args) for args in chunk]
  File "/Users/xeno/Documents/modis-tools/modis_tools/granule_handler.py", line 80, in wrapper_download_from_urls
    return cls.download_from_urls(*args)
  File "/Users/xeno/Documents/modis-tools/modis_tools/granule_handler.py", line 139, in download_from_urls
    req = cls._get(url, modis_session)
  File "/Users/xeno/Documents/modis-tools/modis_tools/granule_handler.py", line 167, in _get
    location = cls._get_location(url, modis_session)
  File "/Users/xeno/Documents/modis-tools/modis_tools/granule_handler.py", line 182, in _get_location
    location_resp = session.get(https_url, allow_redirects=False)
  File "/Users/xeno/miniconda3/envs/co2/lib/python3.9/site-packages/requests/sessions.py", line 600, in get
    return self.request("GET", url, **kwargs)
  File "/Users/xeno/miniconda3/envs/co2/lib/python3.9/site-packages/requests/sessions.py", line 573, in request
    prep = self.prepare_request(req)
  File "/Users/xeno/miniconda3/envs/co2/lib/python3.9/site-packages/requests/sessions.py", line 484, in prepare_request
    p.prepare(
  File "/Users/xeno/miniconda3/envs/co2/lib/python3.9/site-packages/requests/models.py", line 368, in prepare
    self.prepare_url(url, params)
  File "/Users/xeno/miniconda3/envs/co2/lib/python3.9/site-packages/requests/models.py", line 445, in prepare_url
    raise InvalidURL(f"Invalid URL {url!r}: No host supplied")
requests.exceptions.InvalidURL: Invalid URL 'https:///h': No host supplied
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/Users/xeno/Documents/carbon/carbon/rsinputs/modis/modis.py", line 176, in <module>
    modis_lists = download_modis250m_ndvi(
  File "/Users/xeno/Documents/carbon/carbon/rsinputs/modis/modis.py", line 136, in download_modis250m_ndvi
    modis_lists = GranuleHandler.download_from_granules(
  File "/Users/xeno/Documents/modis-tools/modis_tools/granule_handler.py", line 62, in download_from_granules
    result = process_map(
  File "/Users/xeno/miniconda3/envs/co2/lib/python3.9/site-packages/tqdm/contrib/concurrent.py", line 105, in process_map
    return _executor_map(ProcessPoolExecutor, fn, *iterables, **tqdm_kwargs)
  File "/Users/xeno/miniconda3/envs/co2/lib/python3.9/site-packages/tqdm/contrib/concurrent.py", line 51, in _executor_map
    return list(tqdm_class(ex.map(fn, *iterables, chunksize=chunksize), **kwargs))
  File "/Users/xeno/miniconda3/envs/co2/lib/python3.9/site-packages/tqdm/std.py", line 1178, in __iter__
    for obj in iterable:
  File "/Users/xeno/miniconda3/envs/co2/lib/python3.9/concurrent/futures/process.py", line 562, in _chain_from_iterable_of_lists
    for element in iterable:
  File "/Users/xeno/miniconda3/envs/co2/lib/python3.9/concurrent/futures/_base.py", line 609, in result_iterator
    yield fs.pop().result()
  File "/Users/xeno/miniconda3/envs/co2/lib/python3.9/concurrent/futures/_base.py", line 446, in result
    return self.__get_result()
  File "/Users/xeno/miniconda3/envs/co2/lib/python3.9/concurrent/futures/_base.py", line 391, in __get_result
    raise self._exception
requests.exceptions.InvalidURL: Invalid URL 'https:///h': No host supplied

@xeno0 let's troubleshoot a bit more. Would you mind confirming if you pulled the most recent version of main (git pull main)? I ask because there was another recent update to the repo to use MODIS 061, which I noticed you were using in the reproducible example

Hi @ilsep93  thank you for quick reply! I first run the git pull origin main  and then pip install -e ., after that, I run this code,
   snwe = [34.5, 34.74, -91.5, -91.26]
   start_date = "2022-01-01"
   end_date = "2022-02-01"

    S, N, W, E = snwe

    # Reusable session
    session = ModisSession(username=username, password=password)
    collection_client = CollectionApi(session=session)

    collections = collection_client.query(short_name="MOD13Q1", version="061")
    granule_client = GranuleApi.from_collection(collections[0], session=session)

    granules = granule_client.query(
        start_date=start_date, end_date=end_date, bounding_box=[W, S, E, N]
    )

    # Download the MODIS file
    modis_lists = GranuleHandler.download_from_granules(
        granules, threads=4, modis_session=session, path=output_dir
    )

Hi @xeno0, a few of us have not been able to replicate the error with the code you provided. The downloads work fine on our end. I'm closing this for now, but let us know if you keep running into issues


Darkmode

Description
This PR fixes #26
Added Dark Mode For Website
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Good work! If you haven't joined the Layer5 Community Slack, yet, I invite you to jump in.

Also, please follow the steps mentioned here to fix the DCO check.

Ready for another review, @thisiskaransgit?

I have made all the asked changes and followed the for rebasing the branch but the DCO check still shows error

@thisiskaransgit will you be incorporating this feedback?

Hey @leecalcote, @Nikhil-Ladha,  I have added the required changes, kindly review it

It wasn't available in the resources section so I recreated the light logo, I overlooked the font style sorry for that, I have fixed the issue

Logos need to be edited at source. The edited version here has unwanted artifacts. I'll see about getting a dark mode version created.

@thisiskaransgit Here's a white version of the Meshery Learn logo.

@thisiskaransgit there are a few items remaining to address:

The mode toggle isn't responsive. [@leecalcote]

Incorporation of the white version of the logo. [@leecalcote]

The subtitle has too low of contrast in dark mode. Text needs to be lighter. [@thisiskaransgit]

Let's change the text color of this "button"'s hyperlink to #1E2117 in dark mode. [@thisiskaransgit]

I'll take care of #1 and #2 in this PR, if you take care of #3 and #4 in a separate PR.


Infrastructure as code

Create a CD pipeline, deploying & smoke testing the artifacts created by the CI pipeline, using AWS resources (S3, ECR, EKS).

Ah @ArjaanBuijk, looks like Github is hiding 23 of my added comments. Could you go through the hidden comments as well?

Ah @ArjaanBuijk, looks like Github is hiding 23 of my added comments. Could you go through the hidden comments as well?

@b-quachtran ,
Thanks for pointing that out. I now reviewed & fixed/commented-on all your conversations.
Please check them out again.


Can't add Connector Incoming Webhook

Steps to reproduce
I was following this tutorial to add the Incoming Webhook connector: https://learn.microsoft.com/en-us/microsoftteams/platform/webhooks-and-connectors/how-to/add-incoming-webhook?tabs=newteams%2Cdotnet#create-an-incoming-webhook

Select a channel within a team
Select the three dots > manage
Conectors > Edit
Search for incoming webhook
Add > Add again
When click on add for the second time, it goes back to the search page and does not add the connector.

Expected behavior
Add the connector
Actual behavior
When click on add for the second time, it goes back to the search page and does not add the connector.
Error details
No response

@anateotonio, thanks for raising your issue. We are also able to repro this in Microsoft Teams version 24187.501.2981.3768. The client version is 49/24053101421.

We are checking this with engineering team and after confirmation will raise bug accordingly.

Hi @anateotonio - O365 Connectors Substrate team is aware of this issue and there is an ICM raised to fix this. Once this is fixed, we will notify you in this thread. Thanks!

Hi @Prasad-MSFT any updates on this issue ?

Hi @anateotonio , @dorukgunes , could you please share your respective tenant Ids where you are facing this issue, so that we can share those details with engineering team to fix ?

According to latest update from engineering team, you can start using teams web to create Incoming Webhook & RSS connectors. However, the fix for Teams Desktop client is under investigation.

According to latest update from engineering team, the bug has been fixed in both Teams web and desktop client. Thanks!


Cluster scoped resources need to be prefixed

Bug description:
Cluster scoped resources are cluster scoped and share a naming scope with all other components on the cluster. All cluster scoped resources need to have a prefix to avoid collisions.
A ValidatingWebhookConfiguration named validating-webhook-configuration has a high risk of colliding with another package.
https://github.com/vmware-tanzu/cartographer/blob/2ee79e68041b06c62d2326673fb2376a0b22e380/config/webhook/manifests.yaml#L20
Steps to reproduce:
Please provide a set of steps to reproduce the behavior (example):

Install the RabbitMQ cluster operator 1.3.0
Try to install cartographer using kapp

Expected behavior:
Both packages should be installable and run side-by-side
Actual behavior:
kapp refuses to install Cartographer because the validating-webhook-configuration resource already exists and isn't owned by Cartographer.


slave broker frequently lose connection to master when no msg producing

when the master broker is idle and no msg is producing,  the slave frequently disconnect to the master.
2019-03-27 15:13:10 INFO HAClient - HAClient, processReadEvent read socket < 0
2019-03-27 15:13:10 WARN HAClient - HAClient, housekeeping, found this connection[<IP_ADDRESS>:10912] expired,<PHONE_NUMBER>646
2019-03-27 15:13:10 WARN HAClient - HAClient, master not response some time, so close connection

I think it's a bug. Someone has already submitted a PR, but it has not been accepted.


Postgres Container doesn't run

i used this command to create postgres container
docker run -d -p 5432:5432 --name posgres1 postgres -v postgres-data:/var/lib/postgresql/data
but when try to connect to the container the container stopped and When run docker logs i got this logs
The files belonging to this database system will be owned by user "postgres".
This user must also own the server process.
The database cluster will be initialized with locale "en_US.utf8".
The default database encoding has accordingly been set to "UTF8".
The default text search configuration will be set to "english".
Data page checksums are disabled.
fixing permissions on existing directory /var/lib/postgresql/data ... ok
creating subdirectories ... ok
selecting default max_connections ... 100
selecting default shared_buffers ... 128MB
selecting dynamic shared memory implementation ... posix
creating configuration files ... ok
running bootstrap script ... ok
performing post-bootstrap initialization ... ok
WARNING: enabling "trust" authentication for local connections
You can change this by editing pg_hba.conf or using the option -A, or
--auth-local and --auth-host, the next time you run initdb.
syncing data to disk ... ok
Success. You can now start the database server using:
pg_ctl -D /var/lib/postgresql/data -l logfile start

WARNING: No password has been set for the database.
This will allow anyone with access to the
Postgres port to access your database. In
Docker's default configuration, this is
effectively any other container on the same
system.
     Use "-e POSTGRES_PASSWORD=password" to set
     it in "docker run".

waiting for server to start....2019-06-21 19:06:12.680 UTC [48] LOG:  listening on Unix socket "/var/run/postgresql/.s.PGSQL.5432"
2019-06-21 19:06:12.783 UTC [48] LOG:  could not resolve "localhost": Name or service not known
2019-06-21 19:06:12.783 UTC [48] LOG:  disabling statistics collector for lack of working socket
2019-06-21 19:06:12.783 UTC [48] WARNING:  autovacuum not started because of misconfiguration
2019-06-21 19:06:12.783 UTC [48] HINT:  Enable the "track_counts" option.
2019-06-21 19:06:12.794 UTC [49] LOG:  database system was shut down at 2019-06-21 19:06:08 UTC
2019-06-21 19:06:12.826 UTC [48] LOG:  database system is ready to accept connections
done
server started
/usr/local/bin/docker-entrypoint.sh: ignoring /docker-entrypoint-initdb.d/*
2019-06-21 19:06:12.862 UTC [48] LOG:  received fast shutdown request
waiting for server to shut down....2019-06-21 19:06:12.866 UTC [48] LOG:  aborting any active transactions
2019-06-21 19:06:12.868 UTC [48] LOG:  background worker "logical replication launcher" (PID 53) exited with exit code 1
2019-06-21 19:06:12.869 UTC [50] LOG:  shutting down
2019-06-21 19:06:12.917 UTC [48] LOG:  database system is shut down
done
server stopped
PostgreSQL init process complete; ready for start up.
postgres: invalid option -- 'v'
Try "postgres --help" for more information.

docker run -d -p 5432:5432 --name posgres1 postgres -v postgres-data:/var/lib/postgresql/data

Your syntax is off; the -v option to docker run is after the image, so the command is treating that as an argument for the image
$ docker run
"docker run" requires at least 1 argument.
See 'docker run --help'.

Usage:  docker run [OPTIONS] IMAGE [COMMAND] [ARG...]

So move any options to docker run before you give it the image:
docker run -d -p 5432:5432 --name posgres1 -v postgres-data:/var/lib/postgresql/data postgres

It is working thanks alot @wglambert

docker run -d --name=db -e POSTGRES_USER=postgres -e POSTGRES_PASSWORD=postgres postgres:9.4


revise Definition CCR4-NOT complex

current def
The evolutionarily conserved CCR4-NOT complex is involved in several aspects of mRNA metabolism, including repression and activation of mRNA initiation, control of mRNA elongation, and the deadenylation and subsequent degradation of mRNA. In Saccharomyces the CCR4-NOT complex comprises a core complex of 9 proteins (Ccr4p, Caf1p, Caf40p, Caf130p, Not1p, Not2p, Not3p, Not4p, and Not5p), Caf4p, Caf16p, and several less well characterized proteins.
proposed def
The Ccr4-Not complex is an eukaryotically   conserved deadenylase that can initiate cytoplasmic mRNA decay, and reduce translation by releasing poly(A)-binding protein (Pab1/PABPC1). Ccr4-Not contains seven core subunits, including two poly(A)-specific exonucleases, Ccr4/CNOT6/CNOT6L and Caf1/Pop2/CNOT7/CNOT8
(it seems more appropriate to focus on the  the "deadenylase activity")
ADD refs
Parker, R. (2012). RNA degradation in Saccharomyces cerevisae. Genetics 191, 671–702.
Tucker, M., Valencia-Sanchez, M.A., Staples, R.R., Chen, J., Denis, C.L., and Parker, R. (2001). The transcription factor associated Ccr4 and Caf1 proteins are components of the major cytoplasmic mRNA deadenylase in Saccharomyces cerevisiae. Cell 104, 377–386.
and
PMID:30601114
Title: | RNA-binding proteins distinguish between similar sequence motifs to promote targeted deadenylation by Ccr4-Not.
Authors: | Webster MW, Stowell JA, Passmore LA

Can the def be updated, then this can close?
Although I think @colinlog  was going to ask about  somebody about  CCR4-Not?

done


Creating tips app

@orIsrael
Where is this PR standing?

You need to change the title and fix all the commits

Once all conversations are resolved and bugs/code changes of course, please squash the commits since there is no need for several commits with the same title, one is enough

You are right it’s just the date update.
The migration file will be deleted after I fix the PR
השג את Outlook עבור iOShttps://aka.ms/o0ukef

מאת: rony yosef @.>
נשלח: Tuesday, April 12, 2022 8:34:58 AM
אל: redhat-beyond/Xpense @.>
עותק: Or Israel @.>; Mention @.>
נושא: Re: [redhat-beyond/Xpense] Creating tips app (PR #49)
@ronyyosef requested changes on this pull request.

In tips/models.pyhttps://github.com/redhat-beyond/Xpense/pull/49#discussion_r847983620:

@@ -0,0 +1,29 @@
+from django.db import models
+from django.utils import timezone
+from expenses.models import Expenses

+class Tip(models.Model):

author = models.CharField(max_length=32)
text = models.TextField(max_length=200)
date = models.DateTimeField(default=timezone.now)

category = models.CharField(

    max_length=32,

    choices=Expenses.Category.choices,

We can make it a shared models.TextChoices
Can you do it?

In tips/migrations/0002_alter_tip_author_alter_tip_text.pyhttps://github.com/redhat-beyond/Xpense/pull/49#discussion_r847985079:

@@ -0,0 +1,23 @@
+# Generated by Django 4.0.3 on 2022-04-11 19:18

Can you recreate the migration files so we will have only the init file?
To do it you need to ease the current tips migration file and the db
then run: python manage.py makemigrations and then migrate

In expenses/migrations/0003_alter_expenses_date.pyhttps://github.com/redhat-beyond/Xpense/pull/49#discussion_r847985974:

@@ -0,0 +1,20 @@
+# Generated by Django 4.0.3 on 2022-04-11 19:56

+import datetime
+from django.db import migrations, models
+from django.utils.timezone import utc
+
I don't see that you have changed something in the expenses model, why is there a new migration file?

In expenses/migrations/0002_alter_expenses_date.pyhttps://github.com/redhat-beyond/Xpense/pull/49#discussion_r847986053:

@@ -0,0 +1,20 @@
+# Generated by Django 4.0.3 on 2022-04-11 15:25

+import datetime
+from django.db import migrations, models
+from django.utils.timezone import utc
I don't see that you have changed something in the expenses model, why is there a new migration file?
—
Reply to this email directly, view it on GitHubhttps://github.com/redhat-beyond/Xpense/pull/49#pullrequestreview-938917413, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASKQVBKTEKWSWO6VPDIVCFTVEUDQFANCNFSM5SJRTP5Q.
You are receiving this because you were mentioned.Message ID: @.***>

@orIsrael you should resolve all the conversations, otherwise, I can't merge/review

@orIsrael please resolve conflict so I can merge

@orIsrael you have two commits when it should be only the relevant one.

@orIsrael Please fix the second commit and do rebase from main with git and not with the GitHub UI


During the setup phase, all the players think it is their turn.

This is a weird issue where all the players have a different version of the game state. I think it could be fixed by making instance variables static.

Fixed by changing the current player to a static variable.


update docs font to match abapgit

Before

After

This will only work for users on desktop who have installed SAP GUI or somehow manually installed the font?

i believe the fonts are specified in order of preference and the browser will try to do its best

I think the license would allow for redistribution?
https://experience.sap.com/fiori-design-web/downloads/#download-font-72

No, all text rendered in 72 and its intellectual property automatically belongs to SAP. But we don't pursue people for it because it would be too much work to read it all.
Anyone who uses open source software licensed under Apache 2.0 must include the following in their copy of the code, whether they have modified it or not:

The original copyright notice
A copy of the license itself
If applicable, a statement of any significant changes made to the original code
A copy of the NOTICE file with attribution notes (if the original library has one)

In all seriousness it should be fine, is there a difference between use and redistribution?

I don't know. Legally seems like a minefield so I'll just withdraw my statement ;)


[DOCS] Update codespell hook and word list

Did you read the Contributor Guide?

Yes, I have read the Contributor Rules and Contributor Development Guide

Is this PR related to a JIRA ticket?

No, this is a documentation update. The PR name follows the format [DOCS] my subject.

What changes were proposed in this PR?
Updated the codespell hook to increase spell checking coverage
How was this patch tested?
pre-commit run --all-files
Did this PR include necessary documentation updates?

No, this PR does not affect any public API so no need to change the documentation.

I don't think it is necessary to check notebooks for spelling as they contain lots of partial outputs

Before docs/usecases was binder and we were checking the binder folder for spelling.
So now we have less spell checking coverage.  No drama.
Close this PR if you wish.


Fix set HELPURLs correctly

Set HELPURL correctly for several multi-operation math and logic blocks.
Added HELPURL definition for min and max.
Fixes #1366

@SusanRatiLane File it as a separate issue and we can fix it after the other PRs are merged. I expect it isn't the only block and documentation have diverged. It might make sense to do an audit of all of the documentation to understand what's out of date with master.

Ooooh. I tested the convert radians/degrees block. I didn't realize there was another convert block.

OK, convert_number should be working now.


Support kernels from local directories, but just remote URLs

:+1: Oh my! Thank you :D

@maxfierke If you are looking to use this you must put your kernel tgz in a subdir of the git repo and then set COMPILED_KERNEL_URL in build.conf to the relative path.  This makes it work in the crazy docker build setup we have.  For example
Create ./assets/kernel.tar.gz
Set COMPILED_KERNEL_URL=./assets/kernel.tar.gz


vscode.TreeDataProvider regression in 1.85.x

Type: Bug
The following example works in versions < 1.85.0
Steps to reproduce:

Git clone:  https://github.com/GordonSmith/vscode-extension-samples
Checkout GH_201196 (See:  https://github.com/microsoft/vscode-extension-samples/pull/935/files for code changes)
Open the tree-view-sample folder in VS Code
Follow instructions in https://github.com/GordonSmith/vscode-extension-samples/blob/GH_201196/tree-view-sample/README.md to build and run
After the first "label updates" the tree will stop functioning and will not even respond to "refresh" requests.

It works as you would expect in VSCode < 1.85.0
VS Code version: Code 1.85.1 (0ee08df0cf4527e40edc9aa28f4b5bd38bbff2b2, 2023-12-13T09:49:37.021Z)
OS version: Windows_NT x64 10.0.23606
Modes:

System Info

Item
Value

CPUs
AMD Ryzen 9 5950X 16-Core Processor             (32 x 3400)

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)
63.91GB (36.18GB free)

Process Argv
--crash-reporter-id 207c1fc7-7e56-401a-9747-9b1568649f1b

Screen Reader
no

VM
0%

Extensions: none
A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vsdfh931:30280409
vshan820:30294714
vstes263:30335439
vscoreces:30445986
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
py29gd2263:30899288
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
azure-dev_surveyone:30548225
282f8724:30602487
f6dab269:30613381
showlangstatbar:30737416
0bi6i642:30917235
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
welcomedialogc:30910334
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
cp7184c:30925681

@benibenj FYI, looks like this was caused by the sticky scroll changes.

In case anyone else hits this, the work around is to ensure the "label" never changes after its been created (in my case I moved the my changing text into the description)

The real issue is that if you don't provide an id for each of your tree items, then we can't properly find them to modify or dispose them later. The easy fix to your issue in your extension would be to keep the id stable, then you can safely change the label to whatever you want.

Good to know - I thought I had tried to set the ID at one point, but obviously not with a stable id.


Waypoint marker

How can I hide the waypoint markers while showing the route between two locations?

Hi,
Just to confirm. You want to hide the waypoint markers but leave marker start and end, yes ?
Regards,

Yes,thats exactly what i need

Right now the library doesn't have such option which can be applied out of the box. I will put this on top of my todo list

Thank you polok,It will be great help


Query: clean-up expression printer when we can do breaking changes

We can make expression printer internal, since we expose Print method on ExpressionExtensions
Also, currently we repurposed 'generateUniqueParameterIds' to mean general optimized vs debug output. We should rename it to something more appropriate.

Note from triage: make sure not to break providers and not to force providers to use internal code when doing this.


[WIP] Add events

as part of ECOPROJECT-1662 Adding events for better visibility of SNR remediation process.

/lgtm
Giving others a chance to review as well, feel free to unhold
/hold

/unhold


Configurations/10-main.conf: Don't inherit assembler in Cygwin-common

The targets Cygwin-x86 and Cygwin-x86_64 are the ones that should do
this.
Fixes #8684

Confirmed to work in #8684. Final review, please?

(This is low hanging fruit)

Merged.
68ca1737ce58173001f2146b913388f872842f69 Configurations/10-main.conf: Don't inherit assembler in Cygwin-common


Problem with displaying emojis on Bitbucket

Report
What did you do?
Every time our pull request runs Danger on TeamCity by bundle exec danger.
What did you expect to happen?
I expect that emoji is displayed in some message in our repo on Bitbucket.
What happened instead?
Emojis aren't displayed. Instead it I see some text.

Your Environment

Which CI are you running on?
TeamCity + Fastlane

Are you running the latest version of Danger?
Yes

What is your Dangerfile?

swiftlint.lint_all_files = true
swiftlint.lint_files(fail_on_error: true, inline_mode: true)

xcode_summary.ignores_warnings = true
xcode_summary.report 'build/reports/errors.json'

Cool, you're welcome to give it a shot.

Does this issue still exist? Looks https://github.com/danger/danger/pull/1182 has solved the same problem :eyes:

I think this issue is similar to #1290 and it's been solved in https://github.com/danger/danger/pull/1326.
Let me close this one and feel free to reopen it if you have a trouble.


Lab4: Looking at 'real' data

Hi Caleb!
Just wanted to upload my lab in case you feel well enough and have time to look at it :) Hope you are recovering alright!
Erik

Hi, just a follow-up of our discussion during the lab.
I reviewed your code. What you do is simply add the height of each histogram from each image together, does it help a lot to draw background pdf?
Are there other ways to manipulate the pdf which makes the bkg pdf shape change that can help you separate bkg and signal?
Think about this :)
and don't worry about turning late :) I also hope Caleb will recover soon!

Thank you Haoran! I'll spend some time thinking about this.
Best,
Erik
-------- Original message --------
From: hrzhao76 @.>
Date: 11/4/21 2:15 PM (GMT-08:00)
To: SOLHAUG13/Phys434_Autumn2021 @.>
Cc: Erik Solhaug @.>, Author @.>
Subject: Re: [SOLHAUG13/Phys434_Autumn2021] Lab4: Looking at 'real' data (PR #9)
Hi, just a follow-up of our discussion during the lab.
I reviewed your code. What you do is simply add the height of each histogram from each image together, does it help a lot to draw background pdf?
Are there other ways to manipulate the pdf which makes the bkg pdf shape change that can help you separate bkg and signal?
Think about this :)
and don't worry about turning late :) I also hope Caleb will recover soon!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/SOLHAUG13/Phys434_Autumn2021/pull/9#issuecomment-961430047, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APHWKBZ5JRPO6PGH4LT7MT3UKMAZHANCNFSM5HHMWSNA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


  WasmSharedModulePtr shared_module_;
  ^~~~~~~~~~~~~~~~~~~
  WasmModulePtr
external/proxy_wasm_cpp_host/src/wamr/types.h:23:7: note: 'WasmModulePtr' declared here
using WasmModulePtr = common::CSmartPtr<wasm_module_t, wasm_module_delete>;
      ^
external/proxy_wasm_cpp_host/src/wamr/wamr.cc:138:20: error: use of undeclared identifier 'wasm_module_share'
  shared_module_ = wasm_module_share(module_.get());
INFO: 7378 processes: 3180 remote cache hit, 2963 internal, 1235 remote.
@envoy//test/exe:win32_scm_test                                 FAILED TO BUILD

Executed 0 out of 948 tests: 21 tests pass, 1 fails to build and 926 were skipped.

https://dev.azure.com/cncf/envoy/_build/results?buildId=131562&view=logs&j=e969334a-0e55-5c18-ac96-8b546753391e&t=b5329a9e-93c7-5412-9f4c-6f8d659e1b57&l=4312
not sure how essential the win32_scm_test is but we could possibly disable it
im guessing 1.22 will at least have a similar issue

(or possibly we just need to update some other deps im guessing)

yep re 1.22 https://dev.azure.com/cncf/envoy/_build/results?buildId=131563&view=logs&j=e969334a-0e55-5c18-ac96-8b546753391e&t=b5329a9e-93c7-5412-9f4c-6f8d659e1b57&l=1241

Looks like the problem is with com_github_wamr will give it  a try with this commit too https://github.com/envoyproxy/envoy/pull/24739

seems to still be failing https://dev.azure.com/cncf/envoy/_build/results?buildId=131596&view=logs&j=e969334a-0e55-5c18-ac96-8b546753391e&t=b5329a9e-93c7-5412-9f4c-6f8d659e1b57&l=9708
im guessing rules_rust


Importing images from S3

Hi,
I have created the project through Label Studio's python api. After creating task lists through the format,
{ data :
{ image : "s3://path_to_resource}
}
I am importing tasks through import_tasks() command. My question is,

Even after running import_tasks(), the cloud storage option shows 'Never Synced' option. Are these same, import and sync? Because if they are why is 'never synced' showing in cloud storage despite importing it.

Second question, is there a way to ensure the same tasks do not get imported(synced?) twice when using import_tasks? Can a check be introduced that only unique tasks get synced?

Any updates? Did you figure out how to do this?


Added tone() and noTone() stubs

Added simple stubs for tone methods

You are right!
I did not actively notice my auto formatting...
I changed this to your recommendation

Merged. Thank you.


Database missing error when copying an Agent Job with an ActiveX Scripting step

Bug Report

General Troubleshooting steps

[x] Verified running the latest release of dbatools?
[x] Verified errors are not related to permissions?
[x] Can duplicate in new/clean PowerShell session (clean = powershell -NoProfile)?

Version Information

PowerShell Version: 5.1.14409.1012
SQL Server (Edition|Version): Enterprise|12.0.5553.0

Steps to Reproduce

Create a SQL Agent job with a step of "ActiveX Script"
Run "copy-dbaagentjob" that will copy this job
Receive error:

WARNING: [Copy-DbaAgentJob][21:06:23] Database(s) VBScript doesn't exist on destination. Skipping job [JobName].

Relevant output from get-dbatoolslog:
3/19/2018 9:06:23 PM Copy-DbaAgentJob      Verbose                Working on job: PRISM Research
3/19/2018 9:06:23 PM Copy-DbaAgentJob      Debug                  ...
3/19/2018 9:06:23 PM Copy-DbaAgentJob      Warning                Database(s) VBScript doesn't exist on destination. Skipping job [PRISM Research].

[ ] Attach any screenshots (if possible/allowed)
[x] Attach logs generated by dbatools (Get-DbatoolsLog)
[x] Attach output from PowerShell console (if possible/allowed)

Additional information
If you check the dbo.sysjobsteps table this step has the script language in the database_name field.  I haven't tested to see what other job types fill in that information but there may be some more of these buggers in  there.

I've done some testing and it looks like this is the only one that, strictly speaking, isn't a database but the two SSAS subsystems have a server filled in so those may not be local databases.  I can do the code but does it sound like a good idea to exclude databases where the subsystem is ActiveScripting or where there's a server?


Can't chat with myself (for having a "saved messages" feature)

When I either search for my own username in New chat or if I click Chat on my own profile, an old conversation appears from when I sent a message to someone who were not yet on Keybase using their @facebook identity.
My log id: 2a342ab49330961501cd731c

keybase chat send junderwood,junderwood hey

replace junderwood with your name two times with a comma.
Once you send one message it will show up in your recent messages list

You need access to a terminal with the keybase command for first message though... :-(

Thanks for your example, but the same problem occurs

@balboah we'll take a look, sounds like a bug.

I hope this helps:
I don't know about the web interface or the CLI, but it is working fine with me using the windows client and the android client.


Table/comparison

Could a table of GUI frameworks/libraries be added (with filtering support) to areweguiyet? It should feature columns for different features like

the layout engine used
styling support
UI declaration model (like Slint DSL, HTML/JSX macro, SwiftUI/Flutter-like, Cushy, Xilem/Druid), platform support
whether it has a built-in component library
whether the component library can mimic native widgets
native widget support (mainly Android, iOS and web)
state (or mutation?) management paradigms like FRP, TEA and immediate mode.

Not too motivated to implement it myself, but I'll gladly review PRs implementing this!


No swagger output when running WebAPI self-hosted server as a Windows service

I have successfully used Swashbuckle with my C#/.NET/WebAPI based REST server running in a console window.  The output looks great and is just what I expected to see.  I see a title line and all my controllers, can expand them, try out calls, etc.  All works great.
However, using the exact same code running as a service (via ServiceBase.Run() in .NET), when I request the swagger UI output, I get the title to appear at the top of my swagger UI just as always, but now there's absolutely no content below that title.  Again, running as a console it works great.
While running as a Windows service, I can access any of my REST services successfully as well.  So there's no difference in addressing and server is responsive to normal REST operations, I just don't get any content in my swagger UI page.
Additionally, I have ensured that I run the service using the same user account as well.  I'm a bit stumped.  Any help greatly appreciated.

Are there any javascript errors on the page? Can you hit the generated json doc?

Schandlich, thanks for the response.  I just realized earlier this morning that I made a dumb mistake and didn't have the resources (CSS, etc.) accessible when deployed as a service.  I'll have to close this issue on my sillyness.  Thanks!

What do you mean saying "didn't have the resources (CSS, etc.) accessible"? I have a similar problem

can you please explain how you solved this issue?

Having the same issue, anybody found a resolution here? please help!


Error in find-all-references for 3 spread types

TypeScript Version:  2.1.1 / nightly (2.2.0-dev.20170124)
Code
(based on tests/cases/fourslash/findAlRefsObjectSpread.ts)
interface A1 { a: string };
interface A2 { a?: number };
interface A3 { a?: RegExp };
declare let a1: A1;
declare let a2: A2;
declare let a3: A3;
let a12 = { ...a1, ...a2, ...a3 };
a12.a;

Expected behavior:
Find-all-refs for a in a123.a matches the a in all of A1, A2, A3.
Actual behavior:
It only matches the a in A3. If one of the spread elements in the definition of a123 is removed, the it will correctly highlight both remaining spread elements.
CC @sandersn

getRootSymbols isn't called recursively, so retrieval of original symbols from a spread type needs to itself be recursive. It's a linear recursion so it would be better to implement it as a while/push loop in Javascript. The base case is a symbol that is (1) not transient (2) doesn't have a leftSpread property.

Will be fixed in #13288.

@sandersn Any way to fix this without #13288?


Only see label 'meets basic integrity'

Running A14 on Google Pixel 7A.
Should I see the label 'device integrity' or is it only shown when it meets the device integrity?

Hi there,
After selecting a profile with the script, or copying one, you'll hopefully see DEVICE_INTEGRITY in the Play Integrity Checker app you're using (or the one in Play Store). This may take a number of tries.
If you aren't passing DEVICE after 20+ attempts, you may need to change the ABI directory the script is taking profiles from. See the readme for more details on this.

Ran it a few more times, now it shows up.
thanks


cringe, remove systemd dependency or make option to build without systemd

iodined.c:61:11: fatal error: systemd/sd-daemon.h: No such file or directory
61 | # include <systemd/sd-daemon.h>
|           ^~~~~~~~~~~~~~~~~~~~~

Okay this is a problem in my distribution


AttributeError: 'NoneType' object has no attribute 'name'

When I encountered the above bug, I started debugging and found:
(Pdb) p self.builder_configs
{'sDCI_full': DCIConfig(name='sDCI_full', version=0.0.0, data_dir=None, data_files=None, description='sDCI full prompt set')}
(Pdb) p self.DEFAULT_CONFIG_NAME
'sDCI'
The source code is as follows. Its location is in anaconda3/envs/deepcom/lib/python3.12/site-packages/datasets/builder.py.
if config_name is None and self.BUILDER_CONFIGS: print("self.DEFAULT_CONFIG_NAME:", self.DEFAULT_CONFIG_NAME) import pdb pdb.set_trace() if self.DEFAULT_CONFIG_NAME is not None: builder_config = self.builder_configs.get(self.DEFAULT_CONFIG_NAME) [logger.info](https://logger.info/)(f"No config specified, defaulting to: {self.dataset_name}/{builder_config.name}") else:
Please tell me how I should modify it to make the code run through.

@Lenan22 ,
Thank you for finding this bug. A quick fix would be to change the Line 51 of DCI.py to
BUILDER_CONFIGS = [DCIConfig(name="sDCI", version=VERSION, description="sDCI full prompt set")]

We will fix this bug in the next release.

Thanks a lot @synxlin

Hello @Lenan22  and @synxlin ,
I tried this solution and now I found that when I download the DCI image using the url in
https://github.com/mit-han-lab/deepcompressor/blob/9d62c5c69fa862d09d9d5beeba6a51a5ba0cb54e/deepcompressor/app/diffusion/eval/benchmarks/DCI/DCI.py#L29
I encountered the error:
aiohttp.client_exceptions.ClientResponseError: 403, message='Forbidden', url='https://scontent.xx.fbcdn.net/m1/v/t6/An_zz_Te0EtVC_cHtUwnyNKODapWXuNNPeBgZn_3XY8yDFzwHrNb-zwN9mYCbAeWUKQooCI9mVbwvzZDZzDUlscRjYxLKsw
.tar?ccb=10-5&oh=00_AYBnKR-fSIir-E49Q7-qO2tjmY0BGJhCciHS__B5QyiBAg&oe=673FFA8A&_nc_sid=0fdd51'
Is this image url still working? Can you help me with this? Thank you!

Meta has invalidated this link. Please try the new link:
https://huggingface.co/datasets/mit-han-lab/svdquant-datasets/resolve/main/sDCI.gz
We will update it in the next release.

Got it, thanks! @lmxyy


Upgrade documentation

Looking for upgrade instructions for a minecraft server run on this image. It says that the container downloads the latest image, but I assume that if it sees an existing install it will not download even if the default stable version is higher. Is this due to minecraft not supporting upgrades or is there an upgrade path available to tell the install script to download the latest version again? Did I miss it in the readme?

That's a good point that the README doesn't explicitly state how to perform an upgrade. I'll write a specific section for that. Until then...
If VERSION is set to "LATEST" (the default) then it will identify the latest stable version and if a minecraft server jar is not present for that version specifically, it will download and use the new version. That logic works since the filename contains the version, such as minecraft_server.1.15.1.jar.


Present Modal From WYPopoverController

iam trying to present modal from Controller contained by WYPopovercontroller .
when dismissing modal the wypopovercontroller fit all screen size .
any help please.

Same problem here.

I solved the problem by changing the presentation style of the controller to be opened to "OverFullScreen".


Download helper causes horizontal scrollbar in fit-width mode

Combination of padding: 0px 0px 0px 8px; and width: 100%; styles on .download-link-container creates unneeded horizontal scrollbar in fit-width mode.
Also image becomes a little draggable horizontally in all modes (except full), even if image width is small.
Related commit: https://github.com/furyutei/twOpenOriginalImage/commit/5312cd60638ada84d8ff3952b67b0213d83fcf42

Thanks for the fix (https://github.com/furyutei/twOpenOriginalImage/commit/e7fe520a0a79fd1c21c7df268dbe6d9675f013c2).


Persisting the snapshots

Persisting the snapshots opens up the following use cases:

Offline browsing
Faster page loads across browser or mobile app sessions

However, I am not sure about the best way to design this and how well can this perform.

Should we use indexeddb with websql and localStorage fallbacks? localForage library is the standard for this behaviour, but that adds a dependency
Is it better to expose some sort of persist and hydrate callbacks so that the actual persistence doesn't have to be part of the library?

This adds a good amount of complexity, and to truly give offline powers, you need much more (like graceful error states etc). I think for now we're better off with the simpler model of letting the browser do prefetching.


Arcs with rx and ry being different are not considered valid

Inside the SVGParserArcToCurve.cpp on line 109 there is an SVG_ASSERT that ensures that rx and ry are not different. In the event that they are, an assertion is violated, thus the program stops. The comment there says untested and I think that means the code hasn't been tested for cases where they are different. The correct behavior is for such arcs to be rendered properly.
    SVG_ASSERT(angle == 0); // untested

    if (radiusX == 0 || radiusY == 0)
    {
        // this is actually a line
        path.LineTo(endX, endY);
        return;
    }

    SVG_ASSERT(radiusX == radiusY); // untested

    SVG_ASSERT(radiusX != 0 && radiusY != 0);

I did some testing and it seems that there are problems with Arcs that have a non-zero angle.
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 2000 1000" width="2000" height="1000">
  <path d="M 100,100 a 100 50 -45 0 0 100,0" stroke="red" stroke-width="10" />
  <path d="M 100,200 a 100 50 -45 1 0 100,0" stroke="red" stroke-width="10" />
  <path d="M 250,100 a 100 50 -45 0 1 100,0" stroke="red" stroke-width="10" />
  <path d="M 250,300 a 100 50 -45 1 1 100,0" stroke="red" stroke-width="10" />
  <path d="M 500,100 a 100 50 0 0 0 100,0" stroke="red" stroke-width="10" />
  <path d="M 500,200 a 100 50 0 1 0 100,0" stroke="red" stroke-width="10" />
  <path d="M 650,100 a 100 50 0 0 1 100,0" stroke="red" stroke-width="10" />
  <path d="M 750,300 a 100 50 0 1 1 100,0" stroke="red" stroke-width="10" />
  <path d="M 100,400 a 100 50 90 0 0 100,0" stroke="red" stroke-width="10" />
  <path d="M 100,550 a 100 50 90 1 0 100,0" stroke="red" stroke-width="10" />
  <path d="M 250,450 a 100 50 90 0 1 100,0" stroke="red" stroke-width="10" />
  <path d="M 250,600 a 100 50 90 1 1 100,0" stroke="red" stroke-width="10" />
  <path d="M 740,550 A 100 50 -45 0 0 550,500" stroke="red" stroke-width="10" />
</svg>

Rendering by Librsvg:

Rendering by SNV (Cairo):


Ai page

Thank you for your contribution to the IC Developer Portal. This repo contains the content for https://internetcomputer.org and the ICP Developer Documentation, https://internetcomputer.org/docs/.
If you are submitting a Pull Request for adding or changing content on the ICP Developer Documentation, please make sure that your contribution meets the following requirements:

[x] Follows the developer docs style guide.
[x] Follows the best practices and guidelines.
[x] New documentation pages include document tags.
[x] New documentation pages include SEO keywords.
[x] New documents are in the .mdx file format to support the previous two components.
[ ] New documents are registered in /sidebars.js, otherwise, it will not appear in the
side navigation bar.
[x] Make sure that the .github/CODEOWNERS file is
filled with new documents that you added. This way we can ensure that future Pull Requests are reviewed by the right people.

@reigj1 to make this easier to merge I would revert a few commits. Because I was asked to do the "on-chain" to "onchain" changes in its own commit:
3fedc789033b1b6f7ec22e6333b134b4ab0d7049
95c4fad3e509e7c621d84e9e81c5b6a0eb4027f9
7af7fce347485696e664daf579bbd8a8072aeeca
431441c51b317f3e80ccea744b9d811c493c07d3
2364b3b6f1338f7c8ed67c4ea51672f2d9480b26


#129 encrypt shared preferences

@tuancoltech This branch seems to have conflicts that must be resolved. Could you take a look at that?

@jo-elimuai Hi Jo, sure I'll look at it tonight


Sample extension rule with autocorrect?

The sample rules don't override withAutoCorrect and I can't tell from the documentation how it's intended to be used. I don't know what the block parameter is, whether or not I'm supposed to explicitly call withAutoCorrect when visiting a node, or what I'm capable of doing within this method. A simple example would help, or some further explanation in the docs.

I'm afraid withAutoCorrect should probably be deprecated or removed from the public API. Is actually never called in the codebase.
@arturbosch has way more context on this. So far autoCorrect in Detekt is used only for the formatting rules that are actually KtLint wrappers.

Okay, I see now. I was able to write a custom auto-correcting rule without ever needing withAutoCorrect, instead I just checked for the config's autoCorrect after reporting in the visit call. I still think it would be nice to see a sample that utilizes auto-correct, though, unless the whole feature is going to be deprecated?

autoCorrect is intended for ktlint's rules, which detekt wraps. Detekt's native rules don't support this feature.
Now, I'm curios. How do you autoCorrect with the Psi? Do you replace user code? @ahampe

There is this "hack" that you'll find in many ktlint rules if you look at the source:
if (autoCorrect) {
    (node as LeafPsiElement).rawReplaceWithText("...")
}

It works just as well from a detekt rule.

But I guess now that I know that it's intended only for ktlint rules, it would be kind of odd to provide an example of autoCorrect, wouldn't it?

Yes, your are right. That's why we don't include it for now. I'm sorry for the confusion.
However, nothing prevents you from writing autoCorrect rules with detekt. At some time in the past, there was a decision that detekt rules don't mess with user code.

Hey @ahampe, did you find a solution how to do this in detekt? I don't see how I can cast for example a KtNamedFunction or the bodyExpression to a LeafPsiElement as the cast will always fail.


Tx fee integration testing

[x] I have added integration tests/additional assertions around fees for incoming/outgoing transactions since it was not covered in the suite
[x] I have added fee expectations for joining/quitting pool such that it reflects behavior after ADP-2840 (i.e. fee is presented on both join and quit pool txs)

Comments
We used to show fee only for outgoing transactions and always show fee == 0 for incoming ones. That changed and now we show fee for all transactions. The exception is that in old transactions that were done in the Byron era the fee is still presented as 0.
As a follow up from the discussion https://github.com/input-output-hk/cardano-wallet/pull/3806#discussion_r1145026139 I'm adding some integration testing to cover the new behavior. The exception is not tested here because all integration tests transactions are done in post-Byron.
Issue Number
ADP-2840

bors r+


Vue draggable lib not working inside module during dev mode while it works on production build

Environment

Operating System: Darwin
Node Version:     v18.16.0
Nuxt Version:     3.11.2
CLI Version:      3.11.1
Nitro Version:    2.9.6
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      css, modules, components, vite, runtimeConfig, hooks, nitro
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Reproduction
create a simple nuxt module and installed the package
"@marshallswain/vuedraggable": "^4.2.0",

or
https://github.com/marshallswain/vue.draggable
According to documentation of this package you can use it either by
import draggable from '@marshallswain/vuedraggable'
  ...
  export default {
        components: {
            draggable,
        },

or
you can install it as a global component in a plugin, I did with this approach
import draggable from "@marshallswain/vuedraggable";
export default defineNuxtPlugin((app) => {
   app.vueApp.component('draggable', draggable)
})

Now on running it in dev mode it throws error in browser
Uncaught SyntaxError: The requested module '/_nuxt/node_modules/@marshallswain/vuedraggable/dist/vuedraggable.umd.js?v=11114f9e' does not provide an export named 'default' (at vuetify.mjs?v=11114f9e:2:8)

now on googling i tried by importing like
import * as draggable from '@marshallswain/vuedraggable'

the previous error will go away however it will give another error
that defineComponent is undefined in package's umd file
but if you run the code with
npm run build
npm run start

everything will work as expected.
Describe the bug
apparently nuxt is not able to run this component in dev mode
Additional context
No response
Logs
No response

This is likely an issue upstream with @marshallswain/vuedraggable and you can probably resolve with vite.optimizeDeps - see https://vitejs.dev/config/dep-optimization-options.html.

@danielroe  Thank you very much for your quick reply
after adding dependency to  module.ts of my nuxt module it works like a charm in dev mode
 nuxt.options.vite.optimizeDeps = defu(nuxt.options.vite.optimizeDeps, {
            needsInterop: ['@marshallswain/vuedraggable'],
            include: ['@marshallswain/vuedraggable']
        })


Use _.runInContext

For the node use branches you should use _.runInContext to create a pristine lodash instance and avoid augmenting the lodash package directly:
_ = require('lodash').runInContext()

Thanks! Will update.


 issue with text selection

🐛 Bug Report
Inside a <FocusScope contain> with a focused element inside, text selection becomes very hard and feels quite buggy.
As soon as there's an element focused.
🤔 Expected Behavior
Normal text selection.
😯 Current Behavior
On Firefox you loose control over what your currently trying to select, it selects a whole lot more.
On chrome the visual selection indicator does not even show up.
Firefox
https://user-images.githubusercontent.com/335467/108395631-4bb60880-7216-11eb-96c6-c2de7d801f7a.mp4
Chrome
https://user-images.githubusercontent.com/335467/108395671-5375ad00-7216-11eb-898b-a2dc9f25406f.mp4
💁 Possible Solution

Somehow related to #1479, I guess clicking outside the focused element should blur it.
🔦 Context

Building a dialog / modal component.

💻 Code Sample

https://codesandbox.io/s/react-aria-focusscope-doesnt-allow-blur-forked-5e2ls?file=/src/App.tsx
🌍 Your Environment

"Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:85.0) Gecko/20100101 Firefox/85.0"
"Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.146 Safari/537.36"
"Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/13.0 Safari/605.1.15 Epiphany/605.1.15"
🧢 Your Company/Team
Gandi
🕷 Tracking Issue (optional)

@pascalduez I believe this is working as intended and as @snowystinger mentioned here, allowing a state where nothing is focused within the FocusScope could be dangerous/confusing to screen readers. A fix to the above sandbox would be to perhaps add a wrapping div around the contents with tabIndex=-1 much like we do for our spectrum Modal/Dialogs. This would give focus a place to go when clicking away from the input field and allow you to perform text selection. What do you think?

Closing for now, feel free to reopen!


[WIP] Port to Scala Native

Opening this PR just to show progress, currently waiting for new release of scodec bits and Scala Native to fix remaining issues. Tests pass for fastparse/scalaparse/cssparse when using a local publish of scala native with https://github.com/scala-native/scala-native/pull/902.

Now blocked by https://github.com/lihaoyi/fastparse/pull/160 for the fastparseByte module.
Also hit linking errors for classparseNative/test
[info] Linking (1939 ms)
[error] cannot link: @java.lang.Class::getDeclaredFields_scala.scalanative.runtime.ObjectArray
[error] cannot link: @java.lang.Class::getMethod_java.lang.String_scala.scalanative.runtime.ObjectArray_java.lang.reflect.Method
[error] cannot link: @java.lang.Class::getMethods_scala.scalanative.runtime.ObjectArray
[error] cannot link: @java.lang.reflect.Field::getName_java.lang.String
[error] cannot link: @java.lang.reflect.Field::getType_java.lang.Class
[error] cannot link: @java.lang.reflect.Method
[error] cannot link: @java.lang.reflect.Method::getDeclaringClass_java.lang.Class
[error] cannot link: @java.lang.reflect.Method::getName_java.lang.String
[error] cannot link: @java.lang.reflect.Method::getParameterTypes_scala.scalanative.runtime.ObjectArray
[error] cannot link: @java.lang.reflect.Method::getReturnType_java.lang.Class
[error] cannot link: @java.lang.reflect.Method::invoke_java.lang.Object_scala.scalanative.runtime.ObjectArray_java.lang.Object
[error] unable to link

@olafurpg
https://github.com/lihaoyi/fastparse/blob/3da22d6480746b1582353b4b157c7dbba2686ff3/classparse/shared/src/test/scala/classparse/TestUtils.scala#L26 is using Java reflection to load bytecode, in order to feed it through the parser as a sanity check (here too https://github.com/lihaoyi/fastparse/blob/3da22d6480746b1582353b4b157c7dbba2686ff3/classparse/jvm/src/test/scala/classparse/ProjectTests.scala#L29)
We could probably get away with using direct filesystem access rather than going through classloaders and java-reflection, or even just disabling that entire suite of tests of Scala-Native. Apart from these bulk classloader-load-classfile-and-parse tests, that logic is all reasonable covered by unit tests in https://github.com/lihaoyi/fastparse/blob/3da22d6480746b1582353b4b157c7dbba2686ff3/classparse/shared/src/test/scala/classparse/ClassTests.scala so I'm not too afraid of it getting silently broken in Scala-Native

I linked the wrong issue above, I meant fastparseByte is blocked by https://github.com/scala-native/scala-native/issues/925 I disabled NativePlatform for fastparseByte and its dependents classparse and perftests.
The only remaining issue is that the following warning prints when loading sbt
Project defines platforms: js, jvm
project fastparse

  discards: native
Project defines platforms: js, jvm
project fastparse

  discards: native
project pythonparse

  discards: native
project scalaparse

  discards: native
project cssparse

  discards: native

even if native is enabled for those projects. I cancelled all the non-native builds on Travis to see if the CI can reproduce the warnings.

The CI failures seemt to be unrelated to this PR, https://travis-ci.org/lihaoyi/fastparse/jobs/263232670#L1528 twitter/scalading is failing in ProjectTests
We will have to wait until https://github.com/scala-native/scala-native/issues/925 is fixed to cross-build fastparseByte to native. utils/fastparse/scalaparse/pythonparse/cssparse are cross-built.
This PR is ready for review. Would be nice to merge #159 first. Only annoyance is the warnings on sbt load that are printed by sbt-crossproject https://github.com/scala-native/sbt-crossproject/issues/52 @densh mentioned the warning might be spurious in this case.

Looks fine to me. Feel free to take Scalding out of the test suite if it's misbehaving to make things green again

CI is green! 🎉 A sanity check in the logs shows native binaries are running https://travis-ci.org/lihaoyi/fastparse/jobs/264040997#L2488
I'll go ahead and merge this. Note that sbt still prints that annoying warning on startup, https://travis-ci.org/lihaoyi/fastparse/jobs/264040984#L865 I hope it doesn't bug you too much before it gets fixed in sbt-crossproject @lihaoyi 😜

Yeah I've never been very strict about warnings. You have my publishing credentials already IIRC, so feel free to publish when you're ready

fastparse 0.4.4 has been released with scala native support! changelog pending in #163


Pwa deployment in cloudflare

Features

Caches core assets for offline use
Shows offline page when network fails

Testing

Added Lighthouse PWA testing in CI
Requires perfect PWA score (1.0) to pass CI checks
Tests core PWA requirements including:

Service worker registration
Valid manifest
Required icons
Proper display settings

Potential Improvements

Can implement the Richer PWA UI which allows us to customize the PWA installation process
Demo (Ctrl + r to reload PWA app)

https://www.loom.com/share/042bfb4cdda84da0b4775f44cfd4ff05?sid=7f6a27a2-5de4-4d17-bb9f-d7984d4d0bd2
Closes #59

This PR is quite large. Anything we can split out?


Feature request: Go syntax deep printing

Often, I am writing units tests and have expected outputs from a function that are structs with many fields, or arrays of structs so there may be >100 fields. I'd like to get one deep print of the struct I am comparing against, get a Go syntax representation, and then edit field values as necessary to write good unit tests.
I currently use a combination ginkgo/gomega Expect output, which is close but not Go syntax, or fmt's %#v printing which is Go syntax but does not dereference pointers. This would be extremely useful. If this not on the roadmap at all, I would be happy to try and implement it in go-spew.

Hey @soroushjp,
See #6. I had the exact same desire 2 years ago, and that's what my go-goon fork tries to achieve.
If you do try it, let me know how it goes, and I'm interested in making it better. I use it myself very frequently.

Thanks @shurcooL, will check it out and let you know if it helps, appreciate the message!

:+1: on this feature request.  I'd find it immensely useful, editing the spew output by hand is not fun.

I considered this during initial development, but the main issue is that there are several cases where there is no clear Go syntax to recreate a given data structure.  For example, consider the following:
type Inner struct {
	Field1 string
	Field2 **int
}

In this case, there is no clear Go syntax such that you can initialize an instance of Inner due to the double pointer to an int.
The same issue arises in many scenarios such as double pointers to any type, single pointers to native types, interfaces, etc.
I would certainly accept an implementation, but I suspect you'll run into some situations that are unclear how the syntax should be generated.  For example, with the provided code above, it would require the use of local variables (then naming them something reasonable is another challenge).

Yeah, I saw your comment in the other thread after posting.  I agree it's a difficult issue, maybe another small tool could be created for doing that in the cases where it can actually be done sanely.

 However, IMO it's consistent and pretty readable, and only 1 line needs to be modified to make it valid Go. These situations come up less frequently, so it's still useful that the rest is dumped in Go syntax notation. It would've also been valid Go code if Field2 value were nil.

For the benefit of anyone interested, there's now a package for printing data as Go code: https://github.com/Mitranim/repr
It's made mostly for code generation, producing pretty code that you would write yourself (unlike the goon fork). 😄


Issue 04 : Detailed instruction is needed in -h command

Description: In -h command, it will be better, if there is more detailed instruction for the program.

on it


Ignore change item with gitObjectType == 'tree'

self.azure_devops_client.get_item in subsequent line will throw an error if the diff item is not a file (gitObjectType == 'blob')


fix: Patches the variable android_ndk_path

Patch to fix 'Undefined variable android_ndk_path in binding.gyp while trying to load binding.gyp' on Termux.

Checklist

[x] npm install && npm run lint && npm test passes
[x] tests are included
[x] documentation is changed or added
[x] commit message follows commit guidelines

Description of change
Specifying a default value for the 'android_ndk_path'' variable fixes this issue at 'https://github.com/termux/termux-app/issues/3858'

PR was closed because my implementation was 'unethichal', but since there isn't currently a way to define those variables it is being reopened.
I went through the docs at Google GYP and Node GYP

A more suitable solution for the problem has been discovered
https://github.com/termux/termux-app/issues/3858


Studio Versioning

This adds a little component in the Help Guide that shows the current version of the Studio. You can also click on the link to learn more about the changelog (changes between current version vs prior version).

Closing this PR since the target changes/fixes have already been added to both the production and staging branches. Thanks!


If someone just wants to extract matching trigger events, that should be possible (e.g., we should allow empty windows or absent windows structs.

Tagging @cpitsch for tracking

@justin13601 can this issue be closed now?

Yep, merged already


Kafka cluster health checking

kafka-manager could assist with health-checking of kafka clusters in one of two ways:

Pull-based: Expose an http api with health-checks in it (probably a list of checks and their results). Some other system would poll the health-check periodically and alert if checks are failing.
Push-based: Actively reach out when a problem is detected, through some kind of pluggable interface. Two useful out of the box implementations might be emails and just simple error logging.

I'm partial to the pull-based checks since I think they're probably easier to integrate with existing monitoring stuff. Either way, I think good stuff to monitor could be taken from the list here: https://kafka.apache.org/082/ops.html
Some of the most interesting stuff is:

Broker availability (are they in ZK and can we connect over JMX?)
Under-replicated partitions
Unavailable partitions

Hi Guys,
What is the state of this?? I want to plug some alert system and this will help me a lot.


Add apache drill provider

https://github.com/apache/airflow/pull/16884

https://github.com/conda-forge/staged-recipes/pull/16176
https://github.com/conda-forge/staged-recipes/pull/16178

Provider has been created: https://github.com/conda-forge/apache-airflow-providers-apache-drill-feedstock


Implemented swipe to remove chatBox

Automated message from Dropbox CLA bot
@kunall17, it looks like you've already signed the Dropbox CLA. Thanks!

nice!  works well afaict.  it's not the most discoverable functionality, but it definitely serves it's purpose. 👌

Agreed, but this will be needed as after narrowing it will focus the message edit text and then the chat box won't go automatically on scrolling, so then hopefully user will try to swipe to remove chatBox

💯  🌋


psexec replacement?

Sorry for a question as an issue but how do you think go-psexec can be a psexec replacement? The idea of psexec is, transfering a file through SMB and then running it using p/invoke. So you can run commands remotely. But in this case, if I am going to install 'server', then this is simply server/client software. Am I missing something? I didn't test it yet.
Thanks.

Hi @yusufozturk, good question. Back in the day this tool was developed, psexec was unreliable. It would only work sometimes and then timeout and fail other times, on the same machine.
You are right that this tool requires a "server" to be installed, this is no different from psexec though. Although you might not have to install a server for psexec, the server is still running and Windows still has to be "listening" on a tcp port to react on commands.
The thinking behind why we wrote this tool is so that you can install the server with native psexec (or group policies). At least then you only have to rely on native psexec once when you install the server, instead of every time you execute a command. The other reason was that this tool could also become cross platform, since it is written in golang. Native psexec is only Windows.


Version 6.0.0-beta.2 fails at render when used as a external dependency

Using the latest version as an external dependency (6.0.0-beta.2 at the time of this writing):
<script src="https://cdn.jsdelivr.net/npm/vue/dist/vue.js"></script>
<script src="https://unpkg.com/vue-tel-input"></script>
<link rel="stylesheet" href="https://unpkg.com/vue-tel-input/dist/vue-tel-input.css" />
...
<vue-tel-input v-model="value"></vue-tel-input>

Fails with:
[Vue warn]: Error in render: "TypeError: Object(...) is not a function"
...

Uncaught (in promise) TypeError: Cannot read property 'isCE' of null
at renderSlot (runtime-core.esm-bundler.js?6bd5:6123)
at Proxy.u (vue-tel-input.umd.min.js?e75b:1)
at renderComponentRoot (runtime-core.esm-bundler.js?5c40:1168)
at componentEffect (runtime-core.esm-bundler.js?5c40:5214)
at reactiveEffect (reactivity.esm-bundler.js?a1e9:42)
at effect (reactivity.esm-bundler.js?a1e9:17)
at setupRenderEffect (runtime-core.esm-bundler.js?5c40:5167)
at mountComponent (runtime-core.esm-bundler.js?5c40:5126)
at processComponent (runtime-core.esm-bundler.js?5c40:5084)
at patch (runtime-core.esm-bundler.js?5c40:4690)

I have 6.0.0-beta.4 and that still happens only in production.

I also got TypeError: Cannot read property 'isCE' of null. Upgrading Vue from 3.0.0 to 3.2.6 resolved it for me.

same as @matrei got a TypeError : Cannot read property 'isCE' of null
With Vue 3.0.5

I am also having this issue, using vue 3.2.20 and beta.6

For those who are still struggling
I solved it by referring to this.
vue/core issue

#4344(comment)
#2064(comment)

The code I modified in vue.config.js
const path = require('path')
module.exports = {
  ...
  // for VueTelInput
  configureWebpack: {
    resolve: {
      symlinks: false,
      alias: {
        vue: path.resolve('./node_modules/vue')
      }
    }
  }
}

For those who are still struggling I solved it by referring to this.
vue/core issue

#4344(comment)
#2064(comment)

The code I modified in vue.config.js
const path = require('path')
module.exports = {
  ...
  // for VueTelInput
  configureWebpack: {
    resolve: {
      symlinks: false,
      alias: {
        vue: path.resolve('./node_modules/vue')
      }
    }
  }
}

Thank you for this solution. its solve for me


2D Inverse Kinematics: Add a bool property to control the IK's bend orientation

The 2D IKs orient based on how you move bones in the chain. Often the limbs bend the wrong way and there's no control to fix that:

You generally want 2d IKs to bend only one way. in Spine2D, you have a bool property to define and toggle the bend's direction: http://esotericsoftware.com/spine-ik-constraints

Hadn't had the chance to check yet, but was wondering if the old IK feature was integrated into the new bones.
This screenshot accurately reflects our most recent results.  XD


Create a resource with Empty ID

Behaves really weird (to be investigated)
On the UI side, if you do:

KG response:

This is a Backend error. We should prevent that from the API.
From your side, you should simply handle errors accordingly.

:tada: :taco:


Packet IP mapping driver

Description
We need a dedicated virtual network driver for Packet, which assigns the NIC aliased public IP address to the Packet physical host (as part of pre action), and configures the S/DNAT translation between the VM public IP address and VM private IP address (as part of post action). This should leverage the new external NIC aliases functionality = public IP isn't configured inside the VM, but the traffic is only "routed" to the VM private IP.

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

Topic merged with #3042. Closing.


Escape control characters for DynamoDB source

Description
Escapes control characters for the DynamoDB source.
Issues Resolved
Resolves #5027
Check List

[x] New functionality includes testing.
[ ] New functionality has a documentation issue. Please link to it in this PR.

[ ] New functionality has javadoc added

[x] Commits are signed with a real name per the DCO

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

It looks like there is a flag for the Jackson parser that will simply allow it to process control characters. JFYI.


Use unique context param for ComprehensionScope

Fixes #1119

I wonder what makes ComprehensionScope different from all the other ScopeStatement which use a static.

Good wonder.
As understand it: both ClassDefinition and FunctionDefinition track their parent context, which is then used to create the local context (if needed). In case of the function, it goes though internal class GetParentContextFromFunctionExpression and eventually PythonOps. But ComprehensionScope doesn't do that and just uses Parent.LocalContext so in case of nested comprehensions there is a parameter name ambiguity. I guess it's because comprehensions were supposed to be light-weight?
I went here for a simple solution that works (I believe), but more elaborate solutions are possible too, of course. One would need to track the expression that represents the parent context of a comprehension.
Ideally, the whole ComprehensionScope could be redesigned and properly hooked up in the scope hierarchy.

Thanks the the explanation and the PR!

Maybe this is already known, but just for the sake of clearance, the problem with comprehension is that it's an expression, before that only statements can hold a scope, that's why the ScopeStatement is a subclass of statement, and when comprehension requires a scope, we have a problem. Now the ComprehensionScope hangs to the actual comprehension, instead of being part of the tree.


Less verbose output

Hi there, and first of all thanks for this awesome extension which considerably helps our build times.
I have however one question: is there a way to reduce the output of the library at each incremental build?
    ℹ ｢wdm｣: Compiling...
->  Type checking and linting in progress...
    ℹ ｢wdm｣: Time: 930ms
                                      Asset      Size   Chunks             Chunk Names
       807ce71d6f17985d91a1.hot-update.json  35 bytes           [emitted]
       507eb34ca7888537e9da.hot-update.json  44 bytes           [emitted]
                                 index.html  2.92 KiB           [emitted]
                                favicon.ico  14.7 KiB           [emitted]
        bundles/app.73cb96b356038f31eba9.js  3.94 MiB      app  [emitted]  app
    bundles/vendors.73cb96b356038f31eba9.js  30.3 MiB  vendors  [emitted]  vendors
     + 3 hidden assets
    ℹ ｢wdm｣: Compiled successfully.
    ERROR in /Users/.../index.tsx(280,5):
    no-console: Calls to 'console.log' are not allowed.
->  No type errors found
->  Version: typescript 3.5.3, tslint 5.18.0
->  Time: 1292ms

Type checking and linting in progress... is redundant with Compiling...
Version: typescript 3.5.3, tslint 5.18.0 is pretty useless information for me (i can understand how it can be helpful for you to debug reports though)
Time: 1292ms (also pretty unnecessary to me)

I tried setting silent: true, but it seems like is actually swallowing all output.

Some of that output is webpack; some is the plugin. If you'd really like fine grained control of what gets logged, you could use the logger option. This allows you to supply your own logger to the plugin (default is console).

If you'd really like fine grained control of what gets logged from the plugin, you could use the logger option. This allows you to supply your own logger to the plugin (default is console).

This looks like a great option, thanks for the answer!


Does graph coalesce return uncoalesced answers?

if we pass in 3 answers and 2 coalesce into 1, do we return the 3rd uncoalesced answer as well?  In other words, are we adding to the result set, or replacing the entire result set?

What is it doing?
What do we want it to do?
Make these the same.

It looked like when you ran it on the relay question, only coalesced answers were returned, so I'm assuming that we need to fix that here.

The deployed version of AC now only adds results.


Improve trace logging for pulls and processes (#22633)

Backport #22633
Our trace logging is far from perfect and is difficult to follow.
This PR:

Add trace logging for process manager add and remove.
Fixes an errant read file for git refs in getMergeCommit
Brings in the pullrequest String and ColorFormat methods introduced in #22568
Adds a lot more logging in to testPR etc.

Ref #22578

Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

LG-TM bot