Don't use blueprints for this app

Blueprints are helpful on larger Flask apps, but they also make the source code more difficult to understand.
In retrospect, we should have built this app using Django. It's easier to manage larger projects with Django than with Flask.
For now, though, let's just remove the blueprints feature from this app. That should simplify your views.py and a few other files.

I'm using blueprints in this app in order to tests views properly(redirect, templates used), otherwise I can't import views in the context of tests without having conflicts.

Have you looked at the testing page in the Flask docs?
http://flask.pocoo.org/docs/0.10/testing/
This page describes how to test a Flask app (including views) without use blueprints. I noticed you're using Flask-Testing - it should work without blueprints too.
I really think blueprints make this app difficult for a new developer to understand. Can you please give it a try?
If it still doesn't work, I can live with removing some tests if that's what it takes to remove the blueprints.

Yes I really like a lot of times, in fact it kind of become an obsessions to achieve that(check my progression on the 4 flask tutorial triying to achieve that).  The thing is docs tells you how to test it, but when you put the hands on it you realize that going throw the test command and creating a test app, your views are not really loaded and then tests start failing because templates are not loaded, and you can't import there views without getting errors(in fact I started creating init methods for reusing the modules). The only way to do that would be to reuse the dev app and therefore environment db, which it's not a good practice. Take a time to think about what I'm saying, I may lose something on the way, but I really pushed to achieve that without blueprints. If it's ok to you to miss view tests at this point, I don't mind.

Okay - in that case let's remove the blueprints and also remove any tests that break. We won't worry about coverage for this app.
Thanks again for working on this.

Don't mention it. Done!


Underwrite Quote

The Actuary engages the client and agreed to the contract, further kyc (dummy data for MVP) will be securely managed by Oasis on IPFS.

Oasis is the leading privacy-enabled and scalable layer-1 blockchain network and combines high throughput and low gas fees with secure architecture to provide a next-generation foundation for Web3 and power DeFi, GameFi, NFTs, Metaverse, Data tokenization and Data DAOs.

IPFS is a peer-to-peer network and protocol designed to make the web faster, safer, and more open. IPFS upgrades the web to work peer to peer, addressing data by what it is instead of where it’s located on the network, or who is hosting it.
Filecoin - the world’s largest decentralized network, allows users to store, request, and transfer data via a verifiable marketplace. Filecoin is completely open-source, enabling people from all over the world to participate. The Filecoin protocol runs on an advanced and novel proof system — a combination of Proof of Spacetime (PoSt) and Proof of Replication (PoRep), to provide unmatched efficiency and reliability. Filecoin’s advanced technology provides a robust foundation to store the world’s most valuable data sets. An alternative to costly cloud storage, the Filecoin network offers efficiently priced and geographically decentralized storage, minimizing financial barriers, and allowing users to take advantage of its unmatched network capabilities.


main export in package.json is incorrect

I installed the package to my solution using npm:
npm i ejs4b
I got an error trying to import the module like an es6 module:
"Failed to resolve entry for package "ejs4b". The package may have incorrect main/module/exports specified in its package.json."
My import:
import ejs from "ejs4b";
I also tried the following but got an error:
import ejs from "ejs4b/ejs-es";
To fix the issue I had to change my import to the specific default ES6 file:
import ejs from "../../../../node_modules/ejs4b/ejs-es";

Can you give me a link to your repo, or a link to a minimal repo that
reproduces this?
On Wed, 28 Jun 2023 at 16:36, Glen Attwell @.***> wrote:

I installed the package to my solution using npm:
npm i ejs4b
I got an error trying to import the module like an es6 module:
"Failed to resolve entry for package "ejs4b". The package may have
incorrect main/module/exports specified in its package.json."
My import:
import ejs from "ejs4b";
I also tried the following but got an error:
import ejs from "ejs4b/ejs-es";
To fix the issue I had to change my import to the specific default ES6
file:
import ejs from "../../../../node_modules/ejs4b/ejs-es";
—
Reply to this email directly, view it on GitHub
https://github.com/mobily-enterprises/ejs4b/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQHWXVUVOETXASV32UM6KTXNPUJHANCNFSM6AAAAAAZWWB25U
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hey Tony, I can share a codesandbox with you
https://codesandbox.io/s/ejs4b-example-vysz7n
I've added two imports to a vanilla JS sandbox.

I will never apologise enough for the delay.
I am sorry. I  had archived this email by accident -- it never happens...
What I do is this:
import ejs from 'ejs4b/ejs-es.js'

Please reopen if it doesn't work.
Sorry again.


rm update in md

LGTM

LGTM, It would be nice to delete the corresponding code as well, but this can be done by the next PR.

Ok, I got it.


Add documentation and examples for layout, typography and colour

What does it do?
Adds documentation (in the style of GOV.UK elements) for Layout, Typography and Colour.
This PR doesn't include colour swatches, I'll open a new PR for those.
Screenshots (if appropriate):

What type of change is it?

New feature (non-breaking change which adds functionality)

Closing for now, this needs more work.


Need to provide TOPO_FILE arg in MOM_input otherwise MOM6 looks for bathymetry in topog.nc

I am currently using regional_mom6 version 0.5.1. I get the following error when I try to run the reanalysis example. Renaming a copy of the bathymetry.nc file in the inputdir as topog.nc resolved this. Just wanted to share this as a user experience.

thanks @AVEllepola! I just renamed the issue title to be descriptive of the situation.
cc @ashjbarnes

I'm wondering if the expectation for a bathymetry file called topog.nc comes from MOM6 itself or from the MOM6 input files that the regional-mom6 package provides. @ashjbarnes do you have an idea?

Here it seems that the "premade" input files are such that topography is expected to be read from file:
https://github.com/COSIMA/regional-mom6/blob/e16340de202cdf87d921d44a27253ccdc0af46e1/demos/premade_run_directories/common_files/MOM_input#L48-L51
But do we provide the TOPO_FILE variable?

seems to me that if we don't provide TOPO_FILE then MOM6 assumes that the filename is topog.nc
https://github.com/NOAA-GFDL/MOM6/blob/b98acd861d835c0bf5894f872516d2338a4791f4/src/initialization/MOM_shared_initialization.F90#L156-L158

@ashjbarnes, what's a good solution here you think?

Thanks @AVEllepola ! This should sort it out. I've tested with this branch and mom6 runs on the example demo again


[Hub Generated] Review request for Microsoft.KeyVault to add version 2019-00-00 - TEST

If you are a MSFT employee you can view your work branch via this link.
Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-js

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-node

Automation for azure-sdk-for-java

Automation for azure-sdk-for-go


Correctly use HCP account roles when creating HCP clusters

Using the incorrect roles is causing issues in CI https://github.com/openshift/release/pull/47894
OSD-20366

/lgtm
thanks for the fix!


feat: use course_names for operator dashboard filters

This change updates the operator dashboard to use the course_names datatset for course name and course run filters, which performs better than using the fact_enrollment_by_day dataset.
This should address the "Filters on Enrollments tab" item in #535

@SoryRawyer 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.


doriswriter support csv

Proposed changes
make doriswriter of DataX support format csv.  Format csv is more simple and faster than format json when data is simple(maybe, i think ...).
add property format: csv/json
add property column_separator: effect when format is csv, for example "\x01" , "^", etc...
Types of changes
What types of changes does your code introduce to Doris?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)
[ ] Code refactor (Modify the code structure, format the code, etc...)
[ ] Optimization. Including functional usability improvements and performance improvements.
[ ] Dependency. Such as changes related to third-party components.
[ ] Other.

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have created an issue on (Fix #ISSUE) and described the bug/feature there in detail
[x] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If these changes need document changes, I have updated the document
[ ] Any dependent changes have been merged

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

I don't think there's any need to do a conversion here, and the csv format can cause problems, such as empty strings, with the separator you specify in the data，The json format is the least error-prone

I don't think there's any need to do a conversion here, and the csv format can cause problems, such as empty strings, with the separator you specify in the data，The json format is the least error-prone

I think it ok to offer a option to user. And the default column separator can be multi-bytes separator to avoid some problems

When in stream load using format csv, does Doris recognize string '' ornull ?
My tests show only insert '' .
Can Doris insert null in this way ? Maybe a limitation when using format csv ?

When in stream load using format csv, does Doris recognize string '' or null ?
My tests show only insert '' .
Can Doris insert null in this way ? Maybe a limitation when using format csv ?

For null value, it should be "\N"

When in stream load using format csv, does Doris recognize string '' or null ?
My tests show only insert '' .
Can Doris insert null in this way ? Maybe a limitation when using format csv ?

For null value, it should be "\N"

thanks for your comment, it confused me a lot ╰(￣▽￣)╮


Override formatData.js

Hi,
how can I override formatData function? I was trying plugins extension from strapi docs https://docs.strapi.io/developer-docs/latest/development/plugins-extension.html#within-the-extensions-folder like this:
module.exports = (plugin) => {
  plugin.services.email.send = (ctx) => {
    console.log('!!email send!!')
  };
  return plugin;
};

Unfortunately it doesn't work.
I use strapi 4.3.6 and ezforms 0.1.0

Right now formatData isn't registered as a Strapi service so there wouldn't be a way to override it, its probably best to move it to the services folder, (probably move the entire utils folder there) I will get around to it soon or you're free to open a PR.

Hi,
This has been implemented in v0.1.1 please update and the plugin and you should be able to override the formatData function as its now a service

@yarikwest This has been added to docs https://ezforms.excl.dev/advanced/custom-formatting.html


[Feature]: Add the GitHub action workflow for greeting contributors

Description:
The current project lacks an automated action to greet contributors. GitHub Actions can significantly enhance project automation. I propose adding a GitHub Action workflow to greet new pull requests (PR) and issue creators, fostering a welcoming environment.
Proposed Changes:
Implement a GitHub Action workflow that triggers new PRs and issues. The workflow should include a greeting message to acknowledge and appreciate the contributors for their efforts.
Please assign me this issue under IWOC

@1010varun Please assign it to me


Turf Interactive Floorplan Button

Seems to be on all floor plan pages. Example: https://main--hubblehomes-com--aemsites.hlx.page/new-homes/idaho/boise-metro/caldwell/mason-creek/amethyst/19138-beechcraft-st/999
These assets will be going away when the previous development agency is offboarded.
Recommend removing both the grey buttons as the Floor Plan Handout is already available as a link amongst the other links.

@codevoyage please give this a look. we can update the links block to hide driving instructions. Or we can do this via import script.

@varunmitrasure, the import can handle removing both buttons.

@natemyersSHL
There is a link to the interactive floor plan in the links section on the right, which also needs to be removed along with the button, correct?
In my next import, I will remove both buttons as shown below, as well as the link to the interactive floor plan on the right from all the pages where this occurs, not just on Floor Plans but also from Inventory models, etc.

@codevoyage that's correct. All interactive media will be going away as it's created and hosted by the previous development agency.

I assume this also includes the interactive sitemap, like this: Interactive Site Plan, not just interactive floorplans.


Props not converted to camelCase by default

According to #330 json should be converted by default.
But if we use this example
@Html.React("ReactWorld", new { Name = "World" })
it will be rendered as
ReactDOM.hydrate(React.createElement(ReactWorld, {"Name":"World"}),...
Minimal repo to reproduce.

That's in .NET Core I believe.

You need to set up the camelCase behavior in the startup file. Something like
app.UseReact(config => {
        config
            .SetJsonSerializerSettings(new JsonSerializerSettings
            {
                StringEscapeHandling = StringEscapeHandling.EscapeHtml,
                ContractResolver = new CamelCasePropertyNamesContractResolver()

            });
});

Looks like this is intentional - we won't be able to change this in the source code without breaking existing clients.
https://github.com/reactjs/React.NET/blob/5f5f65d88c194d7c50e71ff8cb6d4a8e06b48aea/src/React.Core/ReactSiteConfiguration.cs#L43-L46
However, in ASP.NET core projects, that behavior is overridden:
https://github.com/reactjs/React.NET/blob/3c086e5454271d1030afa800df9dd123e2d300d6/src/React.AspNet.Middleware/ReactBuilderExtensions.cs#L52-L54.


Sorting Graphs By Dates

I am trying to produce a graph that displays how many statements a particular actor produces a day. Is there a way I can use the timestamp to sort my results by how many times a specific actor produces a certain verb a day?
right now I can only produce how many statements a particular has produced overall, which is far too many bars on the chart.
this is how my code is set up
dash.Mia = dash.createBarChart({
container: '#Mia.graph svg',
pre: function(data){return data.where('actor.name = "Mia Wallace" and verb.id = "http://activitystrea.ms/schema/1.0/access"')},
groupBy: 'timestamp',
aggregate: ADL.count(),
customize: function(chart){
chart.xAxis.rotateLabels(45);
chart.xAxis.tickFormat(function(d){ return /[^/]+$/.exec(d)[0]; });
}

Yes, you can sort by date by stripping off the time component of the timestamp. It'd look something like this:
dash.createBarChart({
    ...
    pre: function(data){
        return data.where(
            'actor.name = "Mia Wallace" and verb.id = "http://activitystrea.ms/schema/1.0/access"'
        ).math('date', '$[timestamp|0,10]');
    },
    groupBy: 'date'
    ...
});

Tell me if that works for you.

This is perfect.
Thank you Steven!
-Mike
On Wed, Jul 13, 2016 at 12:32 PM, Steven Vergenz<EMAIL_ADDRESS>wrote:

Closed #11 https://github.com/adlnet/xAPI-Dashboard/issues/11.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/adlnet/xAPI-Dashboard/issues/11#event-721830358, or mute
the thread
https://github.com/notifications/unsubscribe/AS_lUL8nLxiZRv7Slpqt_rtB0QyuLOOYks5qVRMjgaJpZM4JLk3g
.

Hey Steven,
I was wondering if it would be possible to filter by the hour as well or
the days of the week?
-Mike
On Wed, Jul 13, 2016 at 12:32 PM, Steven Vergenz<EMAIL_ADDRESS>wrote:

Yes, you can sort by date by stripping off the time component of the
timestamp. It'd look something like this:
dash.createBarChart({
...
pre: function(data){
return data.where(
'actor.name = "Mia Wallace" and verb.id = "http://activitystrea.ms/schema/1.0/access"'
).math('date', '$[timestamp|0,10]');
},
groupBy: 'date'
...
});
Tell me if that works for you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/adlnet/xAPI-Dashboard/issues/11#issuecomment-232411344,
or mute the thread
https://github.com/notifications/unsubscribe/AS_lUAdjwFc8rXXNEz_SVRQ01d_tDMtaks5qVRMegaJpZM4JLk3g
.

Yes you can, but it gets a little stranger because the resulting string is no longer a valid ISO time. I'd expect this to work:
dash.createBarChart({
    ...
    pre: function(data){
        return data.where(
            'actor.name = "Mia Wallace" and verb.id = "http://activitystrea.ms/schema/1.0/access"'
        ).math('hourstamp', '$[timestamp|0,13]');
    },
    groupBy: 'hourstamp'
    ...
});


Fix memory leaks ZipObject.cpp

CentralDirectoryFileHeader and LocalFileHeader uses MsDosDateAndTime.
MsDosDateAndTime is allocated (new) and not deallocated (delete).
MsDosDateAndTime doesn't need to be allocated in the heap.

I looked into it, and AOSP tests started failing early June.  I'm going to disable the test to unblock you.

/azp run

Thanks Ruben!


Make sqllogictest distinguish NaN from -NaN

Which issue does this PR close?
Closes #7402
Rationale for this change
DataFusion distinguishes NaN and -NaN so sqllogictest should distinguish them for precise test.
What changes are included in this PR?
Changed conversion.rs so that it returns "-NaN" if a value is a kind of NaN and its sign is negative.
Are these changes tested?
Modified existing test and added a simple test to select.rs.
Are there any user-facing changes?
No.

pyarrow error looks unrelated. Maybe re-trigger it later.

I believe CI will pass after merging up from main to get the change in  https://github.com/apache/arrow-datafusion/pull/7416

I merged up from main to get the CI test fix

We have encountered a problem related to this PR. The sign bit of the value representing NaN is platform-dependent. (As seen in this issue, I think the result of scalar functions on ARM is always NaN, not -NaN.). Can you suggest a way to solve this issue in a deterministic way? isnan() cannot solve the problem for columnar data.

Hi @berkaysynnada
Can we modify the query in sqllogictests using nanvl?

@berkaysynnada  I opened #7462 for the issue you hit.
sqllogictest works on your machine with that change?


bug: detects all bots as already blocked and not blocking

having first bot already blocked, any other bot if not blocked it's detected as already blocked

css selectors might more reliable, couldn't test it as cloudflare detects me know as suspicious traffic
with open('bots.txt', 'r') as file:
    for bot in file:
        bot = bot.strip()
        driver.get('https://9gag.com/u/'+bot)
        try:
            WebDriverWait(driver,5).until(EC.presence_of_element_located((By.CSS_SELECTOR, ".uikit-popup-menu a")))
            driver.find_element(By.CSS_SELECTOR, ".uikit-popup-menu a").click()

            WebDriverWait(driver,5).until(EC.presence_of_element_located((By.CSS_SELECTOR, ".uikit-popup-menu .menu ul li:nth-child(3) a")))
            blockButton = driver.find_element(By.CSS_SELECTOR, ".uikit-popup-menu .menu ul li:nth-child(3) a")
            print(blockButton.get_attribute('innerHTML'))
            if "Unblock" not in blockButton.get_attribute('innerHTML'):
                blockButton.click()

                WebDriverWait(driver,5).until(EC.presence_of_element_located((By.CSS_SELECTOR, ".ui-dialog__actions button.btn-color-danger")))
                driver.find_element(By.CSS_SELECTOR, ".ui-dialog__actions button.btn-color-danger").click()
                print("User "+bot+" successfully blocked")
            else:
                driver.get("https://9gag.com")
                print("User "+bot+" was already blocked. Skipping...")
        except:
            print("User "+bot+" was already blocked. Skipping...")


Fix more flag during block-wise transfer

If the payload was an exact multiple of the block size the index calculation was off by one.
This caused clients to make an additional redundant request at the end.
Also, GET requests would always have the more flag set, which was against the specs.

@JcBernack Thanks for your PR and could you please rebase the commit and squash the whitespace commit?

@JsonMa Ready to merge for me, it is right to fix bug without introducing "logic behaviour" changes ;-)


Gitea 1.12.4 always restart  and see 502 Bad Gateway

Gitea version (or commit ref):
Git version:1.12.4
Operating system:linux
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

2020/09/11 02:15:02 ...m.io/xorm/core/db.go:154:QueryContext() [I] [SQL] SELECT * FROM team INNER JOIN team_user ON team_user.team_id = team.id INNER JOIN team_repo ON team_repo.team_id = team.id WHERE (team.org_id = ?) AND (team_user.uid=?) AND (team_repo.repo_id=?) [3 5 124] - 364.565964ms
2020/09/11 02:15:02 ...er/issues/indexer.go:228:func3() [F] Issue Indexer Initialization timed-out after: 30s
2020/09/11 02:16:21 ...dules/setting/log.go:279:newLogService() [I] Gitea Log Mode: File(File:info)
2020/09/11 02:16:21 ...les/setting/cache.go:70:newCacheService() [I] Cache Service Enabled
2020/09/11 02:16:21 ...les/setting/cache.go:81:newCacheService() [I] Last Commit Cache Service Enabled
2020/09/11 02:16:21 ...s/setting/session.go:63:newSessionService() [I] Session Service Enabled
2020/09/11 02:16:21 routers/init.go:63:initDBEngine() [I] Beginning ORM engine initialization.
2020/09/11 02:16:21 routers/init.go:70:initDBEngine() [I] ORM engine initialization attempt #1/10...
2020/09/11 02:16:21 ...rm/session_schema.go:25:Ping() [I] PING DATABASE mysql
Description
gitea often restart  and see 502 Bad Gateway. see in log: er/issues/indexer.go:228:func3() [F] Issue Indexer Initialization timed-out after: 30s
[indexer]
REPO_INDEXER_ENABLED = false
Add this config, it did not work also.
Screenshots

It is said Issue Indexer Initialization timed-out after: 30s but not repository indexer.

Just to be clearer ISSUE indexer not repository.
It's likely that the data in data/indexers/issues.* is corrupt and you should just delete them.


[stable/cluster-overprovisioner] Issue-216: Honor Helm release namespace

Description
Addresses https://github.com/deliveryhero/helm-charts/issues/216
This change updates the cluster-overprovisioner Helm chart to honor the Helm release namespace. This is a common pattern implemented in most Helm charts and allows clients to install the chart into a namespace by passing the standard --namespace flag to Helm.
Checklist

[x] Title of the PR starts with chart name (e.g. [stable/mychartname])
[x] I have read the contribution instructions, bumped chart version and regenerated the docs
[x] Github actions are passing

@max-rocket-internet @mmingorance-dh Could someone please take a look at this PR and the referenced issue? Thanks.

Resolves https://github.com/deliveryhero/helm-charts/issues/216


Egghead.io is broken (SSL problem?)

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.02.11. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.02.11

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[ x Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
youtube-dl -v https://egghead.io/courses/how-to-contribute-to-an-open-source-project-on-github

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://egghead.io/courses/how-to-contribute-to-an-open-source-project-on-github']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.02.11
[debug] Python version 2.7.10 (CPython) - Darwin-16.7.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.4.1, ffprobe 3.4.1
[debug] Proxy map: {}
[egghead:course] how-to-contribute-to-an-open-source-project-on-github: Downloading course lessons JSON
ERROR: Unable to download JSON metadata: <urlopen error [SSL: SSLV3_ALERT_HANDSHAKE_FAILURE] sslv3 alert handshake failure (_ssl.c:590)> (caused by URLError(SSLError(1, u'[SSL: SSLV3_ALERT_HANDSHAKE_FAILURE] sslv3 alert handshake failure (_ssl.c:590)'),))
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 519, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2199, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 431, in open
    response = self._open(req, data)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 449, in _open
    '_open', req)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 409, in _call_chain
    result = func(*args)
  File "/usr/local/bin/youtube-dl/youtube_dl/utils.py", line 1089, in https_open
    req, **kwargs)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 1197, in do_open
    raise URLError(err)

Update python.

@dstftw I have the latest 2.7, which is supported by youtube-dl

Wrong.

The docs say that youtube-dl requires "Python 2.6, 2.7, or 3.2+"
Am I missing something here? 😕

Python version 2.7.10

2.7.10 != 2.7.14.

doh 😓
homebrew said it was the latest version 😕
Updated to Python 3.6.4 and it works, thanks! 🙂


java wrapper

Currently I am using rockdb jni in some of our indexing pipeline to index terabyte scale data .
I would like to try speeddb  integration to our indexing and serving backends to test out performance , cost , scalability.
Can you provide some pointers about if there is any jni bindings rimilar to maven rocksdbjni  that I can do some quick experiments with.
Also for better understanding , can you also porivde some pointers to kind of optimizations speeddb has done on top of rocksdb
What new parameters i would need to configure apart from typical rocksdb option file , any pointers would be very appreciated

Hi @patelprateek .
Speedb JNI is available through maven here - https://mvnrepository.com/artifact/io.github.speedb-io
For specific speedb features you can see them all in our documentation - https://docs.speedb.io/speedb-features
Not all of them are exposed in the java interface yet, we are working on it.
If you have a specific need, or if you want to to share your usecase and get assistance in configuration we are happy to do that on our discord channel - https://discord.com/invite/deMznf8aZZ
Closing this issue as there is no bug\feature - let's move the configuration part to discord


Add level trait to identified, expanded items.

Addressing issue #1207

would rather show it as not a trait tbh

I think as a tag is fine but showing it directly after rarity makes it look like a trait 🙃
Right above Price maybe?

Sample of the new version

Wrong way to use format. I'd recommend looking for examples of game.i18n.format() in the codebase


./scripts/setup fails for mac OSX

Installing node libraries fails with platform mismatch errors after spinning up the app frontend outside the VM

Was this issue already resolved?

I don't think so - I don't have one to test, but I would expect this to still fail due to inconsistencies between the host and guest that does not happen with Linux hosts.

I saw this issue in the node-sass repo, and running npm rebuild node-sass takes care of the issue for me. I'm not sure how this will play with our recent change to having yarn as a frontend package manager, though.


Heroku installation docs aligned with PR #40

Heroku installation docs aligned with the postgrest-heroku buildpack README.md

Check the CI errors, there are some misspellings(not included in our dict) and a syntax error.


Replace Bing by Google Option

Hi.
You did an amazing work.
Can you add an option to replace Bing by Google?
Thank you.

Thanks for your comments!
The problem with search capabilities is that APIs are not free to use.
Google search is about 5$/1000 req. The first 100 req/day are free.
I'll eventually propose Google as alternate search but I must think about how to protect myself against too heavy usage.

Thanks for the great mods Sven. I hope you figure out a way to offer an alternative. Preferably one without the extensive signup process that Microsoft currently requires.
Some people do no have a credit card or "non-VOIP" phone to give to Microsoft. Beyond that, many of us do not want to give Microsoft (or other data-mining companies) any information.
Please do not take that as criticism towards you, I understand your difficulties. This is just to express user viewpoints (that I'm guessing others share).
Either way, great work on the mod suite Sven!

I implemented the integration with Google Search API.
See : https://www.patreon.com/posts/image-search-72427626
However, it still requires an extensive signup process (but with Google Cloud).
Unfortunately, those APIs are not available without signup and API keys.
Openverse is the only exception but search results are limited to Creative Commons images.


upstream: Re-implement rust-in-macro for performance

Thanks @phillord
@brownjohnf can you test this please.
close #107

Yep, will do. Probably this weekend.

I merge this now, but it would be nice if you could post the differences with these changes.

This works amazingly well! Including a screenshot of the profiler now, showing basically no resource usage by this code. Thank you SO much to both of you for troubleshooting/making this fix so quickly and helping out an ignorant emacs newb like myself :-).
@brotzeit I'd be happy to make a PR to doom emacs (how I'm consuming it) bumping the version to current master if you think it's ready to go.

@brownjohnf Nice! And yeah that would be nice, I was planning to open a PR myself.

So after using this more, it definitely is better, but still not as snappy as I'd hope, or as fast as it seemed before the regression. I'm not sure what more to do about it though, and it'd be odd if these fixes make it fast enough in rust-mode but not rustic. Maybe I just have a slow machine.

Hmmm, do you know that it's faster in rust-mode ?

When I have the time, I will take a look at it, but I doubt I'm able to really improve it. I'm currently waiting for the native-comp branch in emacs to be more stable. I guess this will bring more performance improvements.


[doc]Update the doc of getting-started-provider-dev

Update the instruction "./cluster/kubectl.sh create -f cluster/addons/rbac/"
to avoid error arised.

What this PR does / why we need it:
Which issue this PR fixes(if applicable):
fixes #
Special notes for reviewers:

Release note:

NONE

/ok-to-test

/lgtm

/lgtm
/approve


[AutoPR sql/resource-manager] Adding VA support for manged instance

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3353

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/2435


Does this model take different sizes of images as i/p? Or does all the images have to be of the same WxH?

The input images can be of arbitrary size, we just resize them during the preprocessing process for better detection.

Thank you for clarifying!


feat(16216): Create a graph visualisation of the main elements of the Edge

See https://hivemq.kanbanize.com/ctrl_board/57/cards/details/
This PR is the first step of an epic dedicated to building a "workspace", where users will be able to observe and manipulate the topology of their Edge.
It creates a basic visual panel where the different elements of the Edge topology are identifiable, laid out and connected to each other according to their configuration.
Out-of-scope

The PR only offers a dynamic visualisation of the current topology. Adding or editing elements will be covered in subsequent tickets

Technical specs

The graph is built and rendered using the React Flow third-party library, see https://reactflow.dev/. It's a Typescript library with the edges and nodes of the graph being full react Component, allowing easy customisation for UI/UX building. It is also not bound to any particular layout algorithm, giving us all control on how to render the type of topology we will be facing within HiveMQ Edge.
The feature is currently behind a "feature flag" until all tickets of the epics are ready for general release. The flag is defined in the general configuration file as features.WORKSPACE_FLOW_PANEL but can be easily changed by modifying the environment variables. Create a .env.local file at the frontend's root and add the following key:

VITE_FLAG_WORKSPACE_FLOW_PANEL=true

Design
Different elements of the topology are identified and visualised on the "workspace":

the Edge itself,
the Bridge between the Edge and a MQTT Broker instance,
the different Protocol Adapter allowing industrial devices to communicate  with the Edge
the different Listeners and gateways to services associated with the edge

Each element in the visualisation has its own style and iconography. They might also contain various visual markers related to their configuration and status, e.g. :

the name and type of the element
the connection status of the element
the topics configured in the subscriptions of the elements

The graph visualisation library offers the following interaction with the topology:

Nodes can be dragged on the canvas and reorganised
Canvas can be zoomed in and out
Canvas can be translated in the viewport
Node (and multiple nodes) can be selected and manipulated by keyboard

Screenshots

I am unable to get the branch working. Having followed the procedure I get;

you need to do
pnpm install

to get all the dependencies imported (especially the new ones)

Just tested a very simple config and works fine.


Document Game.BlockingWaitSkipped

Game.BlockingWaitSkipped: returns the last reason of the last blocking action interruption: this is same as Wait* function return value, but also works for skipped blocking speech, and so on.

/// Gets the code which describes how was the last blocking state skipped by a user (or autotimer).
import static readonly attribute int BlockingWaitSkipped;

This has to be added to Game page
https://github.com/adventuregamestudio/ags-manual/wiki/Game

Added here:
https://github.com/adventuregamestudio/ags-manual/wiki/Game#gameblockingwaitskipped


Run integration tests with Netty leak detector

Given one of the core tasks of the proxy is to read and transform Kafka messages we are manipulating Netty ByteBuffers (either directly via the MemoryRecordsHelper or indirectly through the Kafka Message Encoder). Kroxylicious is a infrastructure service and thus expects to be available for prolonged periods of time without being restarted. Thus we want to ensure we are managing memory correctly and efficiently.
Describe the solution you'd like
We should ensure that all our integration tests run with the Netty Leak Detector set to PARANOID.
We should then fail the build if LEAK: appears in the test output (this will require us to ensure we enable logging during build runs).
It will be up to the people building distributions or deploying kroxylicious to determine what the appropriate level for production usage is.

603ff12f enabled the leak detection annotation on the FilterIT test.  Should we be doing that across all the ITs tests?

yeah we should @k-wall


How to open specific property pane page (cfr. Image Gallery webpart)

Category

[x] Question
[ ] Typo
[ ] Bug
[ ] Enhancement / Suggestion
[ ] Additional article idea

Expected or Desired Behavior
In the Image Gallery webpart, when editing an image you added to the page, the property pane opens with options to configure the selected image.
I can't find however how to open a specific property pane page from code, and also how to hide the "special" configuration pages from the webpart panel itself.

You can use this method to open the property pane, say on a button click:
this.context.propertypane.open();

@chakkaradeep thank you but that's not entirely what I meant.
In the Image Webpart provided in the o365 tenant, when clicking the edit button of an image, a specific pane opens to edit the selected image's properties.
Also, when editing the webpart itself, those "image property"-pages are not available (as in no paging controls at the bottom).

@NickSevens There is no magic there. Based on the context, relevant controls and properties are rendered in the property pane.

Ah, that's indeed less magic than I expected :)
Thanks for the help @chakkaradeep


Nothing happens!

When I try to run, nothing happens! PLZ FIX!!!
(Joke issue for those of you who actually thought I was serious)

that's not a bug but a feature

If nothing happens, then you did everything correctly

Why so serious?

Make a video lesson, please


Allow custom table column name even if translation exists

When a translation exists, the custom column name specified is ignored.
This commit uses the custom name in priority if it is specified.

Can you remerge your branch with master?, There was bug with Travis. I fixed it. I prefer  Travis to be green before merging your pull request. Thanks.

sorry for the delay, done


Text to Speech Feature

Hi,
you did absolutely great work! This is exactly what i was looking for.
The speech to text is working perfectly, so i wanted to ask you if you also planned to create a text to speech functionality?
This would make it complete and would come in really handy when creating AI voice chats, which is probably often requested from now on.
Cheers

Hey @levlau glad you like it :)
I plan to support TTS as Deepgram's new endpoints allow.
I was waiting to see if there was demand for it, so since there is I will update the pacakge soon with it !

I just added support for TTS in 2.1.0 :)
check deepgram.speakFromText !


TODO List Not Loading in 2.0

Factorio Version: 2.0.21
TODO List Version: 19.6.0
Thanks for working to update this mod for Space Age! Unfortunately, the new update doesn't seem to have worked. With all other mods disabled (including the official Space Age ones), TODO List doesn't show up on my screen, in my toolbar, or in my mod settings.

Hey! Yes, something has changed since factorio 2.0.0. We are looking into it. If you have ideas, don't hesitate to open a PR!


Having to assign both sides of a relationship - am I doing it wrong?

If this would be better submitted to StackOverflow, let me know and I'll move it there :)
I'm just looking for help understanding what's happening behind the scenes, with regards to Relationships.
When we have a one-many relationship, it seems we must set the relationship at both ends, (e.g. if we have a Town with many places, we must set place.town=town as well as [town.places addObject:place], whereas what I'd expect is that if I set place.town=town, then querying town.places would automatically contain that place.
It seems we must set it at both ends, i.e.:
Town *amsterdam = [Town new];
amsterdam.name = @"Amsterdam";

Place *brewery = [Place new];
brewery.name = @"Brouwerij t Ij";

brewery.town = amsterdam; // associate once
[amsterdam.places addObject:brewery]; // associate in the other direction, too

[realm beginWriteTransaction];
[realm addObject:amsterdam];
[realm addObject:brewery];
[realm commitWriteTransaction];

If we don't, those unset references will remain set to nil.
Is this a limitation of Realm, or am I missing something?
Cheers,
Ian

This is correct, you can also use backlinks so that you don't have to add them to both. https://realm.io/docs/swift/latest/#inverse-relationships


After installing an extension...

Can you see new or changed objects in the designer? I cant find new objects. Can you debug them in old debuger?
Thanks.

You cannot see the new objects in the object designer and there are no plans to support this scenario.. Debugging will be available directly from VS Code but we don't have an ETA yet.

I have found a way how to debug in old environment....its tricky
http://marko.decmanmolan.com/wp/2017/01/11/how-to-debugg/


Incorrect data in the providers

This document seems to be out of date with the Azure API schema.
For example you claim that
Microsoft.Sql/servers/databases/auditingPolicies/read
Microsoft.Sql/servers/databases/auditingPolicies/write
are settable permission scopes but when applying this, the Azure API says:
Status=400 Code="InvalidActionOrNotAction" Message="'Microsoft.Sql/servers/databases/auditingPolicies/write' does not match any of the actions supported by the providers."
Suggesting this document needs updating to clarify what is now no longer supported
You also get an error for Microsoft.Sql/servers/databases/connectionPolicies/*
Are these scops validated against the latest Azure API schema?

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

ID: 33219707-3b9f-cc6b-541e-a806dc82622a
Version Independent ID: b4e96391-bcf9-1f30-cb44-ed339469da4f
Content: Azure resource provider operations
Content Source: articles/role-based-access-control/resource-provider-operations.md
Service: role-based-access-control
GitHub Login: @rolyon
Microsoft Alias: rolyon

@ricohomewood Thank you for your insights . We are engaging the content authors for reviewing this document .


PerfCI Chaos Jenkinsfile Workflows

Type of change
Note: Fill x in []

[ ] bug
[x] enhancement
[ ] documentation
[ ] dependencies

Description

Add PerfCI Chaos Jenkinsfile Workflows under jenkins/PerfCI-Chaos

01-PerfCI-Chaos-OpenShift-Deployment
02-PerfCI-Chaos-Operators-Deployment
03-PerfCI-Choas-Windows-VMs-Deployment
04-PerfCI-Chaos-Upgrade-OpenShift-Deployment
05-PerfCI-Chaos-Verify-Windows-VMs-Deployment

Reformat all the folder names under jenkins/PerfCI

01-PerfCI-OpenShift-Deployment
02-PerfCI-Operators-Deployment
03-PerfCI-Workloads-Deployment
04-PerfCI-Grafana-Deployment
05-PerfCI-Backup-Report-Deployment

For security reasons, all pull requests need to be approved first before running any automated CI

@RobertKrawitz, any comments ?

@RobertKrawitz, You right, fix it with the following code:
# Check if the image exists
if [[ "$(sudo podman images -q ${QUAY_BENCHMARK_RUNNER_REPOSITORY} 2> /dev/null)" != "" ]]; then

    # Get containers using the image
    containers=$(sudo podman ps -a --filter ancestor=${QUAY_BENCHMARK_RUNNER_REPOSITORY} -q)

    if [[ "$containers" != "" ]]; then
        echo "Stopping and removing containers using the image..."
        sudo podman stop $containers
        sudo podman rm $containers
    fi

    # Now remove the image
    sudo podman rmi -f $(sudo podman images -q ${QUAY_BENCHMARK_RUNNER_REPOSITORY} 2> /dev/null)
fi

@RobertKrawitz, You right, fix it with the following code:
# Check if the image exists
if [[ "$(sudo podman images -q ${QUAY_BENCHMARK_RUNNER_REPOSITORY} 2> /dev/null)" != "" ]]; then

    # Get containers using the image
    containers=$(sudo podman ps -a --filter ancestor=${QUAY_BENCHMARK_RUNNER_REPOSITORY} -q)

    if [[ "$containers" != "" ]]; then
        echo "Stopping and removing containers using the image..."
        sudo podman stop $containers
        sudo podman rm $containers
    fi

    # Now remove the image
    sudo podman rmi -f $(sudo podman images -q ${QUAY_BENCHMARK_RUNNER_REPOSITORY} 2> /dev/null)
fi

Might as well do podman rm -f just to be certain.

I want to remove the image and not only the running container, thats the reason I must use rmi and not rm

I want to remove the image and not only the running container, thats the reason I must use rmi and not rm

Agreed, but when you do podman rm (if there are any containers), you should use podman rm -f to ensure that the containers are removed before you try to podman rmi.

@RobertKrawitz, got it. fixed.

@RobertKrawitz, do you have any further comments?


[webpack] revert change to output extension

didn't notice that this was breaking the app because the .js files were still being served from /dist.
plz review @georgeke

Coverage decreased (-0.03%) to 81.108% when pulling 61093d8f6742cdffd1d11fbb8242a4db84b586ca on ascott:alanna-fix-webpack-config into b295436bff6ad958fb75852d577f83c36667b4a0 on airbnb:master.

master is fixed, I don't think we need this at this point

hmm, looks like the master version of webpack.config.js is still using .jsx for the entry files: https://github.com/airbnb/caravel/blob/master/caravel/assets/webpack.config.js#L18

Oh you're right, though you need to change caravel/templates/caravel/dashboard.html to point to the .js as part of this PR.
The current master has standalone and probably other views pointing to .js that are broken (I just checked that). I think the the .jsx also makes it hard for the browser to find the .js.map for debugging properly.

Yep, once we change the entry.js to entry.jsx in dashboard.html this should be good.

Coverage decreased (-0.2%) to 81.002% when pulling 949de712a062aafdf2c7f607b394b8758007835e on ascott:alanna-fix-webpack-config into f034f2701e886d3b0e6b41203a65ad4c3fb1bea5 on airbnb:master.

lgtm


Peer to peer code API

Hi @Zafron047
Well done, it's great project
But
We have some suggestion for it

https://github.com/Zafron047/Leaderboard/blob/6d431e873e246328d7a6965868f0acdb1ba2cc45/src/Modules/Dom.js#L5
DOM element should have unique id
dist folder should be updated the updated

You can also merge your branches so you can deploy and add live demo link


Update grape to 0.15.0

@mod ping

Finished in 3 minutes 29.8 seconds (files took 7.8 seconds to load)
573 examples, 0 failures, 10 pending


[WIP] Augment get_route_for_model() to support REST API routes.

Closes: #DNE
What's Changed

This adds an api=True argument to nautobot.utilities.utils.get_route_for_model()
Replaced almost all bespoke implementations of manual route construction for plugins or API routes with calls to get_route_for_model()
Expanded the Development Best Practices guide with a section on "Getting URL Routes"

TODO

[ ] Explanation of Change(s)
[ ] Attached Screenshots, Payload Example
[ ] Unit, Integration Tests
[ ] Documentation Updates (when adding/changing features)
[ ] Example Plugin Updates (when adding/changing features)
[ ] Outline Remaining Work, Constraints from Design

@jathanism What's needed to move this out of draft status?


Enable react snippets extension to run on VS Code on the web

:wave: I'm Isidor, a PM from the VS Code team, and we recently announced VS Code for the Web at https://vscode.dev/, which provides a free, zero-install Microsoft Visual Studio Code experience running entirely in your browser.  You can learn more here.
We'd like to ensure react snippets can run in the web, and we have a guide for enabling extensions for the web here. As a couple of highlights:

In VS Code for the Web, both the UI and extension host run inside the browser.
A web extension is structured like a regular extension, but with a different main file: it's defined by the browser property
Access to workspace files needs to go through the VS Code file system API accessible at vscode.workspace.fs
There are currently three ways to test a web extension

Please let us know if you have questions or feedback on enabling your extension for the web - our team would love to learn from your experiences and collaborate. Thank you!

@isidorn thank you for this! merging and will release a new version this week.

@dsznajder awesome, great news! Thanks a lot 👏


[NodeJS]: readIPC from buffer fails with 'Arrow file does not contain correct header', while it works in ArrowJS

Using Node.JS
What version of polars are you using?
"nodejs-polars": "^0.2.0"
What operating system are you using polars on?
MacOS Big Sur 11.1
Describe your bug.
Reading in a buffer from an .ipc (ArrowStream) file using readIPC fails with Error: Arrow file does not contain correct header. At the same time the file is not corrupt since it can be loaded using apache-arrow's Table.from method
What are the steps to reproduce the behavior?
See code example below. I'll post both the .arrow file (works) and .ipc file (doesn't work) as attachment
const pl = require('nodejs-polars');
const { Table } = require('apache-arrow')
const { readFileSync } = require('fs');

const fromArrow = readFileSync('hits.arrow');
const fromIPC = readFileSync('hits.ipc');

// Read Arrow file by Arrow.js -> works
const df = Table.from([fromArrow])
console.log("df", df.count()) // 10

// Read Arrow file by polars -> works
const dfPolars = pl.readIPC(fromArrow)
console.log("dfPolars", dfPolars) // prints nice table with 10 entries

// Read IPC (ArrowStream) file by Arrow.js -> works
const dfIpc = Table.from([fromIPC])
console.log("dfIpc", dfIpc.count()) // 10

// Read IPC (ArrowStream) by polars -> Fails
const dfIpcPolars = pl.readIPC(fromIPC)
console.log("dfIpcPolars", dfIpcPolars) // Error: Arrow file does not contain correct header

Transfering this to the NodeJS repo as I have no way to reproduce this using Python/Rust.

Python Polars 0.19.2 throws the same error on this file: exceptions.ArrowErrorException: OutOfSpec("InvalidHeader")
df = pl.read_ipc('https://paste.c-net.org/ViperMoronic')
It seems that .ipc file needs to start and end with ARROW1 for Polars to work

@0xgeert Please try: pl.read_ipc_stream as described here. It works fine for me. Thx


why are u set this frame?

i know u are a good man after i saw this framework. so, i have a chance to provide you to do some amazing thing! Do u want to join us?

What team are you?
Where are you?


Fix typo

Proposed changes:

Fix a typo

Status (please check what you already did):

[x] made PR ready for code review
[ ] added some tests for the functionality
[x] updated the documentation
[ ] updated the changelog
[ ] reformat files using black (please check Readme for instructions)

@wochinge can you merge once this is built?


Toolbar visual bug with Atom Material UI

When I use the Atom Material UI theme, the toolbar has a visual bug. Not sure how to fix this.

What platform is this for? There are some small styling differences between the OSes.
You can open devtools in atom; you might try and see if there are any offending styles you can disable to get better behavior, or style additions to fix the issue.

This in particular is on Mac OS X 10.11.6 - I'll check that out and see what happens.

Looks like the styles from atom-toolbar are the cause.

Moved to https://github.com/dart-atom/atom-toolbar/issues/8


Import plugins from helm values

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
Plugins documentation and helm values are leading to confusion regarding how to enable a plugin.
Currently to enable some external plugin is required to add the following config:
experimental:
  plugins:
    enabled: true

It is possible as well to declare a list of plugins in helm values, eg:
experimental:
  plugins:
    enabled: true
    fail2ban:
      moduleName: "github.com/acouvreur/sablier"
      version: "v1.4.1-beta.3"

This is suggested in plugins page:

ref
Although experimental.plugins[index][pluginName] is not used by helm to pass as additional argument to traefik container.
I believe the solution would be:

Update plugin pages by add some note to avoid this confusion;
Check {{ .Values.experimental.plugins }} in _podtemplate.tpl and then pass the list as args to container eg:

args:
  - "--experimental.plugins.sablier.modulename=github.com/acouvreur/sablier"
  - "--experimental.plugins.sablier.version=v1.4.1-beta.3"

The second option is clearly a good idea.
Maybe it's also possible to avoid enabled and so be aligned with current plugins documentation.
Feel free to open a PR

@ndeitch, the v26.0.0 should resolve your issue. Feel free to reopen it if necessary ;-)


Update configuring-systems-for-high-accuracy.md

added a warning about STS

#assign: @robinharwood, @Xelu86
@robinharwood @Xelu86
Can you review the proposed changes?
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Fix static build to link agains zstd

It looks like upstream WasmEdge has zstd as a new dependency.
This is not currently captured in the build script, which results in a linking error when using the static feature.
This PR also changes wasmedge-sys minimum requirement from 1.7 to 1.7.4 since wasmedge-sdk is not compatible with wasmedge-sys 1.7.2 (due to the u64 -> Duration change in here)
The missing dependency should have been captured by the rust-static-lib CI workflow, but it seems that workflow has never been triggered:
https://github.com/WasmEdge/wasmedge-rust-sdk/actions/workflows/rust-static-lib.yml
This PR makes no attempt at resolving that.

Hello, I am a code review bot on flows.network.
It could take a few minutes for me to analyze this PR. Relax, grab a cup of coffee and check back later. Thanks!

@apepkuss
Let's tag a new version to ship this on crates.io

@apepkuss Let's tag a new version to ship this on crates.io

Yeah, I'll release a new version. Before that, I'll check the rust-static-lib workflow first as @jprendes mentioned it's not triggered. Thanks!

@jprendes wasmedge-sdk v0.13.2 and wasmedge-sys v0.17.5 have been released. Please check them out. Thanks!


xgboost example fails on raspberry pi3

I have installed xgboost successfully(I am able to import xgboost) on raspberry pi3 modelB(with raspbian buster which is 32bit OS) using the commands
git clone --recursive https://github.com/dmlc/xgboost
cd xboost
make -j4
cd python-package; sudo python3 setup.py install
However when I run the code from xgboost-tutorial it exists with error
/usr/lib/python3/dist-packages/sklearn/externals/joblib.py:1: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
import imp
terminate called after throwing an instance of 'std::length_error'
what():  vector::_M_default_append
Aborted
The same code runs successfully on my i5 laptop(with ubuntu 16.06, 64bit).
My question is how to avoid the std::length_error for the raspberry pi

Also faced this issue in a docker container running on rpi3b with the following code in a python script:
loaded_model = pickle.load(open("500-300.pickle.dat", "rb"))
predict = loaded_model.predict(X)

Tried rebuilding container with another base image and another python version, and pre-learned model loading through pickle and xgboost internal load_model(foo.model). Also checked if the model built in 32-bit python, or converting data to float32 will make any sense.
By the way, this code easily runs in a docker container in my Jetson Nano, powered by ARM64 OS.

Closing this, since XGBoost 2.0+ will not support 32-bit systems. See https://github.com/dmlc/xgboost/issues/9035#issuecomment-1578016136 for rationale.


Exception on wrapper cast

InvalidCastException: Specified cast is not valid.
UnityMvvmToolkit.Core.Internal.ObjectWrappers.PropertyWrapper`2[TSource,TValue].SetProperty (UnityMvvmToolkit.Core.Interfaces.IBaseProperty property) (at ./Library/PackageCache/com.chebanovdd.unitymvvmtoolkit@1.1.8/Runtime/Core/Internal/ObjectWrappers/PropertyWrapper.TSource.TValue.cs:56)
UnityMvvmToolkit.Core.Internal.ObjectHandlers.ObjectWrapperHandler.GetProperty[TProperty,TValueType] (UnityMvvmToolkit.Core.Interfaces.IBindingContext context, UnityMvvmToolkit.Core.BindingData bindingData, System.Reflection.MemberInfo memberInfo) (at ./Library/PackageCache/com.chebanovdd.unitymvvmtoolkit@1.1.8/Runtime/Core/Internal/ObjectHandlers/ObjectWrapperHandler.cs:123)
UnityMvvmToolkit.Core.BindingContextObjectProvider.GetProperty[TProperty,TValueType] (UnityMvvmToolkit.Core.Interfaces.IBindingContext context, UnityMvvmToolkit.Core.BindingData bindingData) (at ./Library/PackageCache/com.chebanovdd.unitymvvmtoolkit@1.1.8/Runtime/Core/BindingContextObjectProvider.cs:208)
UnityMvvmToolkit.Core.BindingContextObjectProvider.RentReadOnlyProperty[TValueType] (UnityMvvmToolkit.Core.Interfaces.IBindingContext context, UnityMvvmToolkit.Core.PropertyBindingData bindingData) (at ./Library/PackageCache/com.chebanovdd.unitymvvmtoolkit@1.1.8/Runtime/Core/BindingContextObjectProvider.cs:132)
MVVMExtensions.BindingContextObjectProviderWithCustomTemplates.RentReadOnlyProperty[TValueType] (UnityMvvmToolkit.Core.Interfaces.IBindingContext context, UnityMvvmToolkit.Core.PropertyBindingData bindingData) (at Assets/Scripts/MVVMExtensions/BindingContextObjectProviderWithCustomTemplates.cs:79)
UnityMvvmToolkit.UITK.BindableUIElements.BindableLabel.SetBindingContext (UnityMvvmToolkit.Core.Interfaces.IBindingContext context, UnityMvvmToolkit.Core.Interfaces.IObjectProvider objectProvider) (at ./Library/PackageCache/com.chebanovdd.unitymvvmtoolkit@1.1.8/Runtime/UITK/BindableUIElements/BindableLabel.cs:23)
ObjectWrapperHandler provides incorrect wrappers if normal and readonly properties are used

Imagine having 2 BindableLabels.
One of them is being binded to IReadOnlyProperty(label1), another one is being binded to IProperty (label2).
Bind label1, then label2 - everything works fine as two new property wrappers (ReadOnlyPropertyWrapper and PropertyWrapper) for int->str are created correctly.
Reset bindings for label2, then reset bindings for label1 - both property wrappers returned to pool and enqued in internal queue (first element in queue being PropertyWrapper).
In this state if you were to bind label1 again PropertyWrapper is dequeued (while label1 is being bound to IReadOnlyProperty) and when SetProperty of PropertyWrapper is executed following cast is failed
_property = (IProperty<TSource>) property;


bug(forge script): deploy progress is not shown

Component
Forge
Have you ensured that all of these are up to date?

[x] Foundry
[x] Foundryup

What version of Foundry are you on?
forge 0.2.0 (56dbd20 2024-07-05T00:18:40.438005158Z)
What command(s) is the bug in?
forge script
Operating System
None
Describe the bug
it seems disabled recently.
how can show deploy progress during forge script?

It could be related to https://github.com/foundry-rs/foundry/pull/8023 since which more script progress info is now being displayed in stderr which perhaps is not available in your envionment?

For future reference: this could be related to Docker specific and the --show-progress flag


C# Asembly Info standardization

Description of the changes:

Make sure that all project pull in an AssemblyInfo file.
Adding a VersionInfo file to create a common version across the C# assemblies.
Adding an editor config to standardize on the style we want VS to apply.

Before submitting a Pull Request:

[X] I reviewed CONTRIBUTING.md
[X] I built my changes locally
[X] I ran the unit tests
[X] I ran the functional tests with a device
[ ] I ran the performance tests with a device

I tested changes on:

[X] Windows
[ ] Linux

/azp help

/azp list

/azp run Microsoft.Azure-Kinect-Sensor-SDK

/azp run Microsoft.Azure-Kinect-Sensor-SDK


Boolean Flag Proposed In Kong PR Change #3703

Summary
This is in direct relation to my proposed PR here: https://github.com/Kong/kong/pull/3703
New flag hide_groups_header to turn off passing the API providers potentially extraneous growing X-Consumer-Groups header list.
Full changelog
*Edited /app/plugins/acl.md to include new flag field and related information about its use.
Issues resolved
Syncs with PR https://github.com/Kong/kong/pull/3703
Checklist:

[X] Ready to be merged(Yes since it points to the next release banch).

@jeremyjpj0916 Thank you!


MediaElement crashes with NullReferenceException from certain https locations and not others

Description
MediaElement crashes with certain videos from the internet.
Steps to Reproduce
I attached some xaml code that shows the MediaElement working in one scenario and failing in two others.  You need to uncomment the MediaElement control for each scenario to see it.
``

<ContentPage.Content>
    <ContentView>
        <Grid  BackgroundColor="Transparent">
                <Grid.RowDefinitions>
                    <RowDefinition Height="40" />
                    <RowDefinition Height="*" />
                    <RowDefinition Height="20" />
                </Grid.RowDefinitions>

            <Button Text="Test" Command="{Binding TestCommand}" />

            <!-- This one works fine -->
            <!--<MediaElement Grid.Row="1" Source="https://sec.ch9.ms/ch9/5d93/a1eab4bf-3288-4faf-81c4-294402a85d93/XamarinShow_mid.mp4"
          ShowsPlaybackControls="True" />-->

            <!-- Causes a System.NullReferenceException: 'Object reference not set to an instance of an object.' -->
            <!-- when viewed with a browser it will attempt to download.  The content-type is 	application/octet-stream  -->
            <MediaElement Grid.Row="1" Source="https://growjoey2020.blob.core.windows.net/media/2020/7/10/637299956869735561_637299956840476600.mp4"
          ShowsPlaybackControls="True" />

            <!-- Get a popup saying 'Can't play this video' and then it crashes with System.NullReferenceException: 'Object reference not set to an instance of an object.' -->
            <!-- this was saved on Azure with the content type is   video/mp4    and works fine if viewed from a browser-->
            <!--<MediaElement Grid.Row="1" Source="https://growjoey2020.blob.core.windows.net/media/hempliance-b2c.mp4"
          ShowsPlaybackControls="True"  />-->

        </Grid>
    </ContentView>
</ContentPage.Content>

Expected Behavior
To display the video crorrectly
Actual Behavior
Crashes with this exception:
System.NullReferenceException
Message=Object reference not set to an instance of an object.
Basic Information

Version with issue: Xamarin.Forms <IP_ADDRESS>1 (and earlier)
Last known good version: none
IDE: Visual Studio 2019
Platform Target Frameworks:

iOS:  I am unable to test it on ios at this time
Android: Target version is Android 9.0 but it did not work on previous versions either
UWP:  I have done no tests with UWP

Android Support Library Version:
Nuget Packages: Xamarin.Forms <IP_ADDRESS>1 (and earlier)
Affected Devices: Android devices, not able to compile ios devices at this time

I had similar issues when using the Octane video library on Android.  But on ios with the Octane library it was working fine.

We need to retest this to verify if this also happens with our code

@ashlar64 can you attach a reproduction sample?

@pictos test XAML is attached to the description)
What I noticed: The video that works fine (Xamarin Show) is starting to play in the browser when I copy/paste the link to search input in Google Chrome.
But the second one start the downloading process (downloads the video instead of playing)
It can be the reason...

@ashlar64 did you test on an emulator or real device?
https://stackoverflow.com/a/59898958/6255544


Convert media results from an object to an array

Problem

In the media store, the media results used to be stored in an array. Then, to simplify finding an item by ID, we converted it to an object with media ids as keys, and media details object as values. However, because we need arrays in most other places, we are constantly converting the object into array.
Description

Media store items should be an array of MediaDetail objects.
We get the items as an array, and then have a process of converting them into an object.
Here are places where we convert the object back into an array:
https://github.com/WordPress/openverse-frontend/blob/29bf0a0075548014a3d1280f0b56ae1f7e5f0b77/src/store/media.js#L166
https://github.com/WordPress/openverse-frontend/blob/29bf0a0075548014a3d1280f0b56ae1f7e5f0b77/src/pages/search/image.vue#L23
https://github.com/WordPress/openverse-frontend/blob/a2e7333e849b60c20f6cc55ea8fc914860c1490b/src/pages/search/audio.vue#L60
Additional context
This would be nice to do before converting the media store to Pinia.
Implementation

[ ] 🙋 I would be interested in implementing this feature.

Can we use store getters to obtain the results as an object or as an array depending on the needs?

Yes, we can, @dhruvkb ! In fact, I prefer to have a single source of truth in the store state, and convert it into any data structure that we need in the UI using getters.

Makes sense. If that's the case, it'll be better to have the results be a dictionary then? Building a dictionary from an array in a getter is much more expensive that calling Object.values(...).

I think the issue with using Object.values is that we'd be relying on insertion order (if we're lucky, sometimes insertion order is not the determining factor for Object key iteration order) to maintain the ordered results (which are presumably ordered significantly rather than randomly by the API).
Why not maintain both? When results are retrieved, use Object.assign({}, existingResults, Object.fromEntries(results.map(r => [r.identifier, r]))) to build the new object mapping. Then we amortize the cost of building the object by doing it very rarely compared to how often results are accessed.

I'm closing this issue because currently the API can give duplicate results, and that makes using an Array (without de-duplicating it) impractical.


Desktop flutter support

I really like the simplicity of this package, however, recently with the change from hive to shared preferences my app that has a management part that needs to run on the desktop has stopped working, because the plugin does not support it. Are there plans to re-offer desktop support?

I'm thinking of using a LocalStorageDelegate and so using any service like the same hive. I'll also look at how the implementation of SharedPreferences for the desktop is going

Hello, is there an update for this?

Use LocalStorageDelegate in the new version of hasura_connect. Example: HiveLocalStorage()


Rebase generic off monolithic

This commit rebases the generic code off the monolithic image
This allows the base image to be the more regularly accessed / edited image
removing the minor map changes directly in this generic image
Commits to lancachenet/monolithic:switcheroo will mimic this commit (https://github.com/lancachenet/monolithic/pull/133)
Also fixes lancachenet/generic#108

This is likely to break #109, #109, #94 and #93 as they will all now be relevant against lancachenet/monolithic instead

Build was actually succesful in https://app.circleci.com/pipelines/github/lancachenet/generic/95/workflows/d25085b5-e98d-48c1-8fb7-05ea697f21a1/jobs/211


ARC not used

Is there any reason this still uses manual release/retain instead of ARC?

The only reason is that I'm used to manual, so habit I guess.


Please correct misleading "Account expiration" section [requested 3 times]

[Enter feedback here]
Dear,
please put on the necessary effort and time to read the escalations and fix the phrasing which misleads the community.
This very same point was requested at 3 different times and yet, the article was not fixed
https://github.com/MicrosoftDocs/azure-docs/issues/76071
https://github.com/MicrosoftDocs/azure-docs/issues/60671
https://github.com/MicrosoftDocs/azure-docs/issues/56804
The issue is simple :

the current page phrasing states that to fix the limitation of "Account Expires" not being synced to Azure AD limitation,
a workflow should be put in place to disable the >>>Azure AD <<<< Account which is a wrong advise
as the Azure AD Connect server that syncs the onprem AD to Azure AD will undo it in the next export cycle / next 30 minutes

Please,
either clearly state that the account which should be disabled is the AD account (instead of AzureAD)
or please provide a better solution.
Best regards,
Thanks.

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

ID: c900f942-4456-5238-9f4c-a7a5bd685c04
Version Independent ID: a5d7d971-765f-0977-f531-232ea5924121
Content: Implement password hash synchronization with Azure AD Connect sync
Content Source: articles/active-directory/hybrid/how-to-connect-password-hash-synchronization.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

The following issue is being closed due to the age.  If this is still an issue, please re-submit this.  Thank you.
#please-close


Fix admissionregistration api version

With kubernetes/kubernetes#56687 merged, we don't need to explicitly enable the admissionregistration.k8s.io/v1beta1 group. This PR fixes the docs accordingly.

This change is

@tengqm Hmm, In swagger.json, I can't find InitializerConfiguration v1beta1. And I can't find ExternalAdmissionHookConfiguration at all. What am I missing?

@steveperry-53 Thanks for the question. Double checked the code. It looks like Intializer and IntializerConfiguration is still left in v1alpha1. The latest change was about adding something new (MutatingWebhookConfiguration, WebhookClientConfig, etc) resource types directly into v1beta1 API. That is weird. However, we should document what the code does, so I'm closing this PR.


Error in luksOpen $VOLUME

Hi
I am getting code 22: Invalid argument error while opening the LUKS volume. Looks like its complaining about $VOLUME name. Do you know what could be the reason?
core@node1 ~/coreos-encvol $ gpg -d /etc/opt/encvol/keys/$VOLUME.gpg | sudo cryptsetup -v --key-file - luksOpen $LOOPDEV $VOLUME
gpg: AES256 encrypted data
Command requires device and mapped name as arguments.
Command failed with code 22: Invalid argument
gpg: encrypted with 1 passphrase

Seems like you have to export the VOLUME variable with any name before executing any script.
$ export VOLUME=encryptedvol

Hi @lals1, and thank you for filling this issue. You are correct. I’ll add the missing instructions to the README.


refactor: improve function comments

See: https://tip.golang.org/doc/comment#cmd

/ok-to-test
/approve
/lgtm
/hold

/unhold


refactor: remove now unneeded NameSetter interface

Signed-off-by: Chris Laprun<EMAIL_ADDRESS>

This change actually cannot work without requiring a constructor taking the name as parameter for all DependentResource implementations.


Bluetooth Low-Energy Samples

Hey,
I saw that BTLE is now part of win32 and that's super exciting. From reading the documentation though, it seems like there isn't an example on how to subscribe to characteristics or equivalent.
Is this possible with the existing implementation, and if so, would you accept a PR with an example that shows how to subscribe to BTLE characteristic notifications? If so, I might need some rough notes on how this works so I can do a sample up.

Hi there! Thank you for the request. Yes -- you're very welcome to submit an example of this, but I have very little knowledge of implementing Bluetooth LE myself, so unfortunately I can't give you much in the way of pointers.
My focus with this project is projecting the underlying Win32 / COM APIs to Dart, but there are too many different subsystems for me to myself be an expert in all of them. You might look for a C/C++ example that you can port over, whether in the Microsoft documentation or elsewhere. If you submit something that is based on another example, please include a link to the original example in your pull request, and make sure that it's OSS-licensed, so that I can include the relevant acknowledgements in the readme.

Thanks for that Tim. I think I've found some doco here which I can consult: https://docs.microsoft.com/en-us/samples/microsoft/windows-universal-samples/bluetoothle/
Frustratingly, a lot of links on the page are returning "access denied". But there's still a sample I can reference, which I can try to make work. I think @Sunbreak is trying to implement this as well for his quick_blue package, so I'll keep an eye on that repository as well.

Thanks for that Tim. I think I've found some doco here which I can consult: https://docs.microsoft.com/en-us/samples/microsoft/windows-universal-samples/bluetoothle/

It is an example of UWP APIs of BluetoothLE, which quick_blue uses right now. Windows 1709+ acquired

I have done some examples on win32 APIs of BluetoothLE. But I wouldn't recommand using them
It is quite cumbersome to find nearby BluetoothLE devices via win32 APIs: https://stackoverflow.com/questions/54912411/detecting-ble-device-in-range-on-win32

If you're interesting in handling WM_DEVICECHANGE, take a look at https://github.com/woodemi/quick_device_watcher
Or you could reference Chromium project: https://chromium.googlesource.com/chromium/src/+/HEAD/device/base/device_monitor_win.cc

So, before this, I was creating a WPF app with BLE functionality but I much prefer using Flutter for the UI and other reasons. The reason why I have begun this journey is because I really want to use (and contribute) to a stable and reliable BLE solution for Flutter because I have a hobby app that uses BLE quite significantly.
So, to that end (and this might not be the best place for these questions, but I'll ask anyway...). These may be really stupid questions so I appreciate the patience. I'm really blazing the trail with Flutter on Windows here because I really want to create desktop apps with native functionality.

When we talk about UWP API's, I take it the are different to win32 API's? So, we can't leverage the BLE UWP API's from this project anyway, because they're not part of the win32 API?
Following on from above, we can technically use win32 for BLE, but it's much more complicated and involved than just using the UWP API's?
If Win32 API's are projectable via this project, does that mean that UWP API's are projectable as well? And, is that something that might happen in the future?

@Sunbreak  I currently use quick_blue in my project and it works well, but it crashes occasionally and when it does it doesn't produce usable errors. Android Studio just says "Lost connection to device" and nothing is logged that I could even create a GitHub issue with. I've cloned quick_blue and attached it to the running app but I've not been able to work out why my crash is happening or what to do about it.
Some disclosure: I'm a C#/.NET developer and I understand that C++ is a totally different beast. But I really want to understand it so I can contribute to BLE functionality with Flutter on Windows.

When we talk about UWP API's, I take it the are different to win32 API's? So, we can't leverage the BLE UWP API's from this project anyway, because they're not part of the win32 API?

UWP APIs are differrent from win32 APIs. And we could use most of UWP APIs via C++/WinRT: https://docs.microsoft.com/en-us/windows/apps/desktop/modernize/desktop-to-uwp-enhance

2. Following on from above, we can technically use win32 for BLE, but it's much more complicated and involved than just using the UWP API's?

Technically yes. But not worth. UWP APIs (or WinRT APIs) are much better

3. If Win32 API's are projectable via this project, does that mean that UWP API's are projectable as well? And, is that something that might happen in the future?

This project, i.e. win32 Dart package, is working on Dart projection on WinRT as well:

https://github.com/timsneath/win32/pull/259
https://github.com/timsneath/win32/pull/406
When projection on WinRT done, we could call WinRT API via dart:ffi

@Sunbreak I currently use quick_blue in my project and it works well, but it crashes occasionally and when it does it doesn't produce usable errors. Android Studio just says "Lost connection to device" and nothing is logged that I could even create a GitHub issue with. I've cloned quick_blue and attached it to the running app but I've not been able to work out why my crash is happening or what to do about it.

https://github.com/woodemi/quick_breakpad may help. We'd better open another issue on quick_blue repo


Is docker-compose correct?

I'm not absolutely sure, but it seems that the documentation from the official site is not enough. I've installed docker, docker-compose, cloned this repo and followed the instructions from the official site: https://cabotapp.com/dev/get-started.html
There a tons of errors, warnings, error traces, and fatals. Almost for every service. Please see the log I've attached: cabot.log.
I deleted all containers, removed all volumes, rebuiled images and started again. The same output.

@sergeyklay I had some issues with this and turned out when just wanted to start it I wasn't using the https://cabotapp.com/qs/quickstart.html right link to the repo which is bit different than in the one hack it yourself
git clone git@github.com:cabotapp/docker-cabot.git
Maybe try that one if you just want to start it?

Did you managed successfully start cabot using docker-compose project? I asking because I'm failed, at least a few weeks ago.

@sergeyklay yes I am running it locally for few weeks and recently also on our server :) so feel free to go on with that QuickStart link I sent you

Thank you


docs: example workflow - force-delete-pods.yaml

Motivation
Workaround for stuck pods described here. May be generally useful for non-responsive pod clean up.
Modifications
Force deletes all pods in the argo namespace except the argo-server, workflow-controller, and the pod running this workflow.
Verification
Tested against pods in various Running, Terminating, Completed, Error states.

Any help on the Codegen linting step would be appreciated


use Cloud resources to get simulation results quicker

Context
The simulation is very long. The simplest problem with 1000 iterations takes 30 min.
We have no idea of the algorithmic complexity of the rice simulation.
Solution proposed
Could we use Cloud resources to reduce the delay in getting a simulation outputs?
FYI @chrichri17 @Ethel2003

@abiolaTresor I will unassign you as you're are working on more urgent task.


outline of a new project description

first late night outline - we could work on that and merge at a later time? Feel free to delete / change every single sentence :)

Made some light edits putting things into a slightly more formal framework. Lots more to flesh out down the line obviously.

nice!


GUI system tray buttons won't get enabled after closing windows

Describe the problem
I have a problem with Netbird GUI (ver 0.28.6) with Ubuntu 23.10.

If you open some window items (like Advance settings or Routes) and then click on the system tray, then close the window item, the button of that item will be disabled until you restart the GUI app. (the Routes item will be re-enabled after disconnect/connect but Advanced settings item won't).

If you press the connect button, it will not get disabled, you can press it more than one time.(Idk if this produces a bug, but it mean you can send multiple up command to the daemon server).

3)If you mess around with system tray you'll get some bugs like one of the screenshot I'll send.
To Reproduce
Start to open the system tray menu while some windows are open, then close the windows and check the system tray menu.
Expected behavior
System tray menu buttons should be enabled/disabled correctly in the intended status.
Are you using NetBird Cloud?
Yes
NetBird version
0.28.6
ui: 0.28.6
NetBird status -d output:
Screenshots

This is the one if you mess around with GUI (connect & disconnect are enabled)

Network Routes button is disabled(window is closed) & I can't click on Advanced settings button
Additional context
I don't have the problem with Windows and I didn't test it with Mac.
I've found that the problem is not with the code logic but with the github.com/getlantern/systray package, I tested it with fyne.io/systray and found no problem(they both share the same API).

fixed in #2309


Add NOASSERTION back to the license picker

Humans should be able to choose NONE and NOASSERTION

@dabutvin so we are happy to partially revert the change that we made as part of #410?

Yes exactly.


[AutoPR adhybridhealthservice/resource-manager] typo: adhybridhealthservice/resource-manager/Microsoft.ADHybridHealthService

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5485

This PR has been merged into https://github.com/Azure/azure-sdk-for-java/pull/3002


ECR: policy is not set in some cases

Describe the bug
We've noticed that the ECR repo policy is not set upon repository creation if encryptionConfiguration, imageScanningConfiguration and imageTagMutability are present in the spec. If the spec doesn't have either of the mentioned attributes the policy is set normally. Also the policy is updated according to the spec after controller restart.
Steps to reproduce
apiVersion: ecr.services.k8s.aws/v1alpha1
kind: Repository
metadata:
  name: test-repo-01
  namespace: ecr-controller
spec:
  encryptionConfiguration:
    encryptionType: "AES256"
  imageScanningConfiguration:
    scanOnPush: true
  imageTagMutability: IMMUTABLE
  name: test-repo-01
  registryID: "1111111111111111"
  policy: |
    {
      "Version": "2012-10-17",
      "Statement": [
        {
          "Sid": "Test",
          "Effect": "Allow",
          "Principal": {
            "AWS": [
              "arn:aws:iam::000000000000:root"
            ]
          },
          "Action": [
            "ecr:*"
          ]
        }
      ]
    }

% aws ecr get-repository-policy --repository-name test-repo-01

An error occurred (RepositoryPolicyNotFoundException) when calling the GetRepositoryPolicy operation: Repository policy does not exist for the repository with name 'test-repo-01' in the registry with id '1111111111111111'

Expected outcome
The policy should be always set normally
Environment
ecr-controller:1.0.5

Kubernetes version

Client Version: version.Info{Major:"1", Minor:"25+", GitVersion:"v1.25.9-eks-0a21954"
Server Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.17"

Client version is irrelevant, we tried to use different clients

Using EKS (yes/no), if so version? No

AWS service targeted (S3, RDS, etc.): ECR

@jackivanov We just released ecr-controller v1.0.9 to address this issue. Can you give it a shot and share if it's working properly for you?


Split retention in 2 parts or multiple nflogs

We have a default of 120h retention.
While this default seems fine for the silences, it seems a lot too high for the nflog. Indeed, the nflog should ideally be kept only for ~110% of x=max( group_wait, group_interval, repeat_interval). When having a large number of alerts and a low x, alertmanager un-necessarily uses a lot of memory, because the state is broadcasted perpetually.
Here a heap of such a case: https://share.polarsignals.com/73d955e/
I see multiple ways forward:

Multiple nflogs, with exponential durations (e.g. 1m, 1h, 12h, 10h), based on the duration the notification should stay available in the log.
Allow setting a nflog-specific data retention policy.

Maybe we can write a new nflog where we remove the duplicates as well as part of some garbage collection process.


Do pull requests get reviewed?

I would like to contribute if pull requests gets reviewed

yes they are reviewed

🧢

caps


Feature: CUDA C++ continuity kernel implementation

NOTE: Branched from boundary condition and next kernel implementations, so those branches need to be merged first.

All these different PRs are getting a bit confusing now so I'm going to consolidate them into a single one.


add schools

please add my school.
BUSANDIGITALHIGHSCHOOL
Uploading busandigitalhighschool.txt…
thanks.

please add my school.
BUSANDIGITALHIGHSCHOOL
Uploading busandigitalhighschool.txt…
thanks.

There are several files in the request, along with file.zip. It must not be there. Please remove file.zip and make pull request again.  I have to close this request without merging. Thank you for your understanding.

There are several files in the request, along with file.zip. It must not be there. Please remove file.zip and make pull request again.  I have to close this request without merging. Thank you for your understanding.


autopep8 automatically adds a --diff argument that prevents file formatting

Environment data

VS Code version: 1.59.0
Extension version (available under the Extensions sidebar): v2021.8.1105858891
OS and version: Ubuntu 20.04
Python version (& distribution if applicable, e.g. Anaconda): 3.8.10
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): venv

Expected behaviour
Something like this:
import math, sys;

def example1():
    ####This is a long comment. This should be wrapped to fit within 72 characters.
    some_tuple=(   1,2, 3,'a'  );
    some_variable={'long':'Long code lines should be wrapped within 79 characters.',
    'other':[math.pi, 100,200,300,9876543210,'This is a long string that goes on'],
    'more':{'inner':'This whole logical line should be wrapped.',some_tuple:[1,
    20,300,40000,500000000,60000000000000000]}}
    return (some_tuple, some_variable)

Should be converted to:
import math
import sys

def example1():
    # This is a long comment. This should be wrapped to fit within 72 characters.
    some_tuple = (1, 2, 3, 'a')
    some_variable = {'long': 'Long code lines should be wrapped within 79 characters.',
                     'other': [math.pi, 100, 200, 300,<PHONE_NUMBER>, 'This is a long string that goes on'],
                     'more': {'inner': 'This whole logical line should be wrapped.', some_tuple: [1,
                                                                                20, 300, 40000, 500000000,<PHONE_NUMBER>0000000]}}
    return (some_tuple, some_variable)

Actual behaviour
Does not format.
Command seen in Output is:
~/Projects/PythonProjects/Misc/venv/bin/python -m autopep8 --in-place --diff ~/Projects/PythonProjects/Misc/Foo.py
However, my settings are:
  "python.formatting.provider": "autopep8",
  "python.formatting.autopep8Args": ["--in-place"],

With --diff being added in for no reason
Virtual env, paths, and everything are working, as confirmed when I run the command in the Terminal tab, just with --diff removed.

@MauricePasternak The extension uses --diff as a way to provide formatting as text edits that can be reversed with undo. In-place changes like with --in-place are not supported by the extension. If you remove --in-place does it not work as expected?

Thank you. That appears to have been the issue.

I am glad that it worked.


commands 'fsharp.scriptrunner.run' and 'fsi.SendFile' not found

Describe the bug**
Using the green icons give the following messages:

command 'fsi.SendFile' not found
command 'fsharp.scriptrunner.run' not found

Is there a way to manipulate these values?
Steps to reproduce
Steps to reproduce the behaviour:
-Load a fsharp script *.fsx
-Click on the green triangle icon OR click on the curved green arrow icon
--> error message:
command 'fsi.SendFile' not found or command 'fsharp.scriptrunner.run' not found
Related bugs
https://github.com/ionide/ionide-vscode-fsharp/issues/1086
Machine info

OS: (Arch) Linux
aur/dotnet-sdk-bin 5.0.9.sdk400-1 (+29 0.24) (Installed)
aur/dotnet-runtime-bin 5.0.9.sdk400-1 (+29 0.24) (Installed)
aur/dotnet-host-bin 5.0.9.sdk400-1 (+29 0.24) (Installed)
code: 1.45.0
Ionide version: 5.7.1

Got the same on a clean install

Both of those commands seem to be working fine for me on macOS

Still working for me on both macOS and Windows. Closing.


Model for Polish

Could you share a model for Polish?

I don't have a neural-based GEC model for Polish. As far as I know, there is no good quality parallel data for this language, only data sets that are available are created automatically, mostly from Wikipedia, and quite noisy.


Feature Request: Be able to get the ctx.message.clean_content, ie, the content of the message after the command, instead of having to split by the command name each time

Having an automatic clean_content parameter which gives this, for example by calling
clean_content = ctx.message.content.split(ctx.command.name)[1].strip() would be really useful

Added to the 2.0 project board

Hey, sorry for the slow reply. I meant to do it on the day, got distracted, did a bunch of stuff and then didn't remember to come back.
Yeah what you want can be done with twitchios command arguments:
@bot.command(name='hello', aliases=['t'])
async def hello_command(ctx, *, message):
    await ctx.send(message)

!hello whatever you want here

will result in
whatever you want here

As there seems to be a suitable feature that exists for your use case, I'll close this request for now.


Update Kaleidoscope to use the new burn format

These changes will be required once rgb-org/rgb#7 is merged.

ACK, please before merge update the reference in Cargo.toml to https://github.com/rgb-org/rgb to commit 1a626e1edd61bf5bc836b8fac5469e4e0d6c3426

Please wait to merge, we need more tests

Please wait to merge, we need more tests

Everything looks good now, ACK for me


RAML root level resource URI issue

Hi Kurtpa,
In my controller the class level URI is "/" and for method level URI I have proper REST URI.
After RAML generation of above controller. In generated RAML the root level resource URI is mapping with method level URI . As per my view the root level URI of RAML should be "/" .
The RAML which I have generated is right one or I had missed something ?

Hiya,
could you post some example URLs please because i'm not sure i'm understanding you


Samba server option

To avoid the problems with shared folders on Windows, I was trying to enable Samba server on the guest. Don't seem to be able to mount shared drives on the host however. What is the method for enabling Samba server on this configuration? Thanks

@edwinpopham - if you set the mount type for a synced folder to "smb", Vagrant does the rest for you automatically; no need to install anything separate in the guest. See: http://docs.drupalvm.com/en/latest/other/windows/#performance

Thanks for that but I was meaning using Samba server on the guest machine. So as not to have files on the host machine. I find it faster and more reliable this way on a windows machine. I have got it working by opening ports and installing Samba server on the vm, but is there an easier way to script this into the vm?

@edwinpopham - You could use pre-provision and/or post-provision scripts to do this. I also have a samba role for Ansible that does this too, but you couldn't run that along with the base provisioning unless you have a custom Vagrantfile.local that adds in another Ansible provisioner to run just that role.
See:

http://docs.drupalvm.com/en/latest/other/overriding-configurations/
http://docs.drupalvm.com/en/latest/extras/scripts/


Make some keys immutable

Is your feature request related to a problem? Please describe.
Currently it is possible for an authenticated client to make problematic updates. Currently we try to avoid client software doing these problematic things; however we need a mechanism to prevent these updates occurring.
Here are some of the keys whose values should not be permitted to be overwritten

public:publickey@alice
privatekey:at_pkam_publickey
shared_key.bob@alice

Describe the solution you'd like
For public:publickey@alice and privatekey:at_pkam_publickey we should prevent their value being modified by client updates (i.e. the update verb handler)
For shared_key.bob@alice we should also prevent the value being modified by client updates.
We should allow metadata to be updated. TBD whether there is some metadata which we should prevent being updated
Describe alternatives you've considered
As the concept of limiting updates to data after its initial creation may have broader applicability, we could introduce a new metadata field, 'immutable', which defaults to false
However we could also just start by putting guards around the three keys named above.
Additional context
No response

Marking for discussion at an architecture call before we move to implementation


docs/build-yocto-udooneo: Fix build script

bitbake m4-firmware before bitbaking the actual image.
Workaround for issue https://github.com/gmacario/easy-jenkins/issues/57
Signed-off-by: Gianpaolo Macario<EMAIL_ADDRESS>

LGTM, merging


Can't add a Python tool that uses from typing import List and def ip_info(ips: List[str]) -> dict:

Description
It is not possible to add a Python tool that contains.
from typing import List

def ip_info(ips: List[str]) -> dict:

I tried a version without a list, and this version was possible to add.
# from typing import List

def ip_info(ips: str) -> dict:

To Reproduce
Steps to reproduce the behavior:

Go to 'Tools'
Click on 'new Tool'
Insert the following example code for the tool definition:

import json
import requests
from typing import List

def ip_info(ips: List[str]) -> dict:
   """
   Get information about an IP address, such as location, company, and carrier name.
   : param ips: A list of IP addresses in the <IP_ADDRESS> format
   : return: Information about the IP address
   """
   ips_info = []
   for ip in ips:
      print(f"ip: {ip}")
      response = requests.get (f"https://ipinfo.io/{ip}/geo")

      print(f"response: {response}")
      response.raise_for_status()

      print(f"response: {response.json()}")
      ips_info.append(response.json())

   return json.dumps(ip_infos)

Press add
See error developer tools Bee UI and Bee API

Bee UI

Bee API

Expected behavior
The tool should be added.
Desktop (please complete the following information):

OS: macOS]
Browser: Version 131.0.6778.139 (Official Build) (arm64)
Version: Bee API, Bee UI, Stack only Infrastructure actual version from today 2024.12.24.
Container engine: Podman

Additional context
No additional images.

I am not familiar with Bee Stack, but I work for IPinfo. Do you have an adblocker running? Try disabling it and see if that works.

@abdullahdevrel @Tomas2D
This problem is related to saving the tool and not using the tool at the runtime. The tool can never be saved, if you have this kind of Python code from typing import List in your custom tool.
The same behavior is also for the other custom Python tool I tried; you can see it here https://suedbroecker.net/2024/12/19/create-custom-bee-agent-with-a-custom-python-weather-tool-a-step-by-step-guide/
Do you have an example of it working with such a definition where you could reproduce the problem I wrote in the To Reproduce  section?

Good catch @thomassuedbroecker. The problem lay in generating JSON Schema based on the function signature.
Fix: https://github.com/i-am-bee/bee-code-interpreter/pull/12
In addition, we should add proper error handling to the UI. A notification would say that the generated code is not valid. Unfortunately, we must propagate such information from the bee-code-interpreter.
cc @mmurad2, @geneknit, @JanPokorny


Shape Mismatch Error in VecNormalize during PPO Training

Steps to reproduce: I ran the following command as suggested in the readme:
python train_policy.py ../config/mujuco_BlockedHalfCheetah/train_ppo_HCWithPos-v0.yaml -n 1 -s 123
Beginning training
100%| 200000/200000 [00:00<00:00, 176379478.55it/s]
Traceback (most recent call last):
File "/home/username/Code/ICRL-benchmarks-public/interface/train_policy.py", line 311, in
train(args)
File "/home/username/Code/ICRL-benchmarks-public/interface/train_policy.py", line 211, in train
policy_agent.learn(total_timesteps=forward_timesteps,
File "/home/username/Code/ICRL-benchmarks-public/stable_baselines3/ppo/ppo.py", line 255, in learn
return super(PPO, self).learn(
File "/home/username/Code/ICRL-benchmarks-public/stable_baselines3/common/on_policy_algorithm.py", line 223, in learn
total_timesteps, callback = self._setup_learn(
File "/home/username/Code/ICRL-benchmarks-public/stable_baselines3/common/base_class.py", line 347, in _setup_learn
self._last_obs = self.env.reset()
File "/home/username/Code/ICRL-benchmarks-public/stable_baselines3/common/vec_env/vec_normalize.py", line 157, in reset
return self.normalize_obs(obs)
File "/home/username/Code/ICRL-benchmarks-public/stable_baselines3/common/vec_env/vec_normalize.py", line 113, in normalize_obs
obs = np.clip((obs - self.obs_rms.mean) / np.sqrt(self.obs_rms.var + self.epsilon), -self.clip_obs, self.clip_obs)
ValueError: operands could not be broadcast together with shapes (1,18) (17,)
Description: I encountered this error while following the tutorial to train a policy using PPO with train_policy.py. It seems like there's a mismatch between vector shapes during normalization. Specifically, the code is trying to operate on arrays with incompatible shapes: (1,18) and (17,).
It looks like the observation shapes aren't handled correctly when normalizing observations with VecNormalize.
Request: Could anyone help with this issue? It seems like a problem with the observation vector dimensions during PPO training. Any advice on how to resolve this shape mismatch would be appreciated.
Thank you!

Hi, this issue is primarily due to a version mismatch with the 'gym' library. Please ensure that you are using version 0.21.0 of 'gym'.

Thanks for the quick response!
I had initially installed a different version due to issues with Gym 0.21.0, but I managed to figure out how to install this version, and it indeed resolved the problem.
For anyone having trouble installing Gym 0.21.0, follow this link: https://github.com/openai/gym/issues/3202