Add the Ability to Specify an Alias in a Table Object

This PR adds a new meta-model object AliasableSqlTable that can contain an alias to be used when a select statement is rendered.
Resolves #413

Coverage remained the same at 100.0% when pulling 2a03903c40d86a282a4d2c7735a3e7ab472c47cd on jeffgbutler:new-alias-strategy into 8fa2e7667fad4741d3b547598edf1b3f5ea9caf9 on mybatis:master.

Hi, I got this in my project but looks like it's not working for EXISTS queries.
Here is an example
amlObjectMapper.selectDistinct(select -> select
                        .join(amlAttachment, "attachment", on(amlObject.objectid, equalTo(amlAttachment.attachmentid)), and(amlObject.objecttypeid, equalTo(amlAttachment.attachmenttypeid)))
                        .where(amlObject.objecttypeid, isEqualTo(AmlObjectType.ATTACHMENT.getValue()))
                        .and(exists(select(of("1"))
                                .from(taxonomyControlAttachment, "tca")
                                .where(taxonomyControlAttachment.attachmentcontrolid, isEqualTo(taxonomyControlIdVersion))
                                .and(taxonomyControlAttachment.attachmentid, isEqualTo(amlAttachment.attachmentid))
                                .and(taxonomyControlAttachment.attachmenttypeid, isEqualTo(amlAttachment.attachmenttypeid))
                                .applyWhere(limitToTypes != null && limitToTypes.length > 0
                                        ? where -> where.and(amlAttachment.attachmenttype, isIn(limitToTypes))
                                        : x -> {
                                })
                        ))

Resulting query

select distinct AML_Objects.ObjectId,
                AML_Objects.ObjectTypeId,
                AML_Objects.ObjectUpdatedBy,
                AML_Objects.ObjectCreatedBy,
                AML_Objects.ObjectUpdated,
                AML_Objects.ObjectCreated
from AML_Objects
         join AML_Attachments attachment on AML_Objects.ObjectId = attachment.AttachmentId and
                                            AML_Objects.ObjectTypeId = attachment.AttachmentTypeId
where AML_Objects.ObjectTypeId = ?
  and exists(select 1
             from TAXO_TaxonomyControlAttachments tca
             where tca.AttachmentControlId = ?
               and tca.AttachmentId = AttachmentId
               and tca.AttachmentTypeId = AttachmentTypeId)

Expected query (fragment)
               and tca.AttachmentId = attachment.AttachmentId
               and tca.AttachmentTypeId = attachment.AttachmentTypeId)

This is likely because the referenced object amlAttachment is aliased only in the outer query. As you can see, table TaxonomyControlAttachments is aliased tca.
I can confirm, however, this fixes the self-join alias problem. For the join queries, of course.

Please note this is not critical to our team as we could change the query in order to make it work. For now, we are not stuck

The aliases specified this way will not carry into to exists queries (or union queries either). You have a couple of options:

You could use a hard alias on the Attachment table aliasedAttachment = attachment.withAlias("attachment") and then use the aliasedAttachment table and its fields in the query. The aliasedAttachment object should do exactly what you want
You could explicitly qualify the columns in the exists query amlAttachment.attachmentid.qualifiedWith("attachment"). This is a bit wordier, but it will produce what you want without creating a separate table object

@djechelon I've created a new issue for aliases in exists queries - I should be able to solve this. You can track progress here: #437


Support for the poll controll cluster (TZ-948)

Is your feature request related to a problem?
The device i am working on require the Zigbee Poll Controll cluster (0x0020) but it seem that it is not supported yet by the zigbee stack (v1.3.2).
Describe the solution you'd like.
No response
Describe alternatives you've considered.
No response
Additional context.
Do you have a timeline for the support of the standard Zigbee clusters? Also, is there an alternative way to implement the unsupported clusters ?

@Vprime007 ，
We plan to support the poll control cluster in the future, so please be patient as we work on it.


Format current line when there is no selection

Is there any way to run the formatter on the current line? Right now, to do so, I have to select one or more characters on the line to run the formatter, when I'd like to just hit a shortcut key.

Is this solved by #12? It doesn't seem to work as indicated it should in that issue.

No, "Format Selection" is only available when something is actually selected.

Has been fixed upstream, cf. https://code.visualstudio.com/updates/v1_29#_format-selection


Allow working directory configuration in env of metadata plugins

The MACKEREL_PLUGIN_WORKDIR environment variable is handled by the plugin process for metrics and check plugins. So this value is often set in the env config within mackerel-agent.conf.
However, this environment variable is handled by the mackerel-agent process for metadata plugin. This is not intuitive when user want to configure the working directory for each metadata plugins.

Thank you.


using multi-remotes does not show correctly in/out elements

so, we introduced the possibility of have multiple remotes.
we also introduced the capability of define "default pulling/pushing remote" for the tool, different to the one that is defined as default (who is origin/branch usually)... the problem is that the browser does not show correctly this definitions hence introducing a lot of confusion.

I made a quick fix here: https://github.com/estebanlm/iceberg/tree/issue270
@npasserini can you validate this approach?

This should not be valid anymore in the latest version. There is a new UI to select the pull/push targets, and branches have gone back to have an upstream to be git compatible.


Channel withdraw and deposit

[ ] Implement channel withdraw and deposit transactions
[ ] Add withdrawals and deposits handling to Channel state machine
[ ] tests

closed by https://github.com/aeternity/elixir-node/pull/697


Issue from L

sdasd
DEV-60

check

Replying by mail
On Thu, 20 Apr 2023, 20:33 Alexey Zimarev, @.***> wrote:

sdasd
DEV-60 https://linear.app/eventstore/issue/DEV-60/issue-from-l
—
Reply to this email directly, view it on GitHub
https://github.com/EventStore/replicator-docs/issues/99, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACM4B2ZUPP2FQCBBPI45QMTXCF6P7ANCNFSM6AAAAAAXF4GCPA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

That worked 👆


Config option to disable the purse

It would be cool if you could disable the coin purse in your inventory.

Nevermind.
On Mon, Jun 26, 2023 at 8:20 AM glisco @.***> wrote:

I'm somewhat puzzled as to why exactly this would be something you want to
do - doing so would literally remove one of the core aspects of the mod and
make it, like, really hard to use?
If you do want the purse gone for whatever reason, nothing is stopping you
from just setting the coordinates in the config to a very large value to
push it off the screen.
Either way, could you offer us some more insight?
Cheers
—
Reply to this email directly, view it on GitHub
https://github.com/wisp-forest/numismatic-overhaul/issues/91#issuecomment-1607708676,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHYQTSMZ4YOFXB4JN54AFBTXNGSC7ANCNFSM6AAAAAAZRWKY7M
.
You are receiving this because you authored the thread.Message ID:
@.***>


📖 Add IP reuse documentation

What this PR does / why we need it:
Documents IP Reuse feature in https://github.com/metal3-io/cluster-api-provider-metal3/pull/656

@lentzi90 thanks for the review, this should be good now.

/test-ubuntu-integration-main


Identify all reaches upstream from a point

Find all stream reaches that drain to a user-selected point or stream reach.

Here's where we could investigate using a directed graph (which btw is what shape files get imported as by default)
https://en.wikipedia.org/wiki/Directed_graph
Here's how I see it going down:

Load the shp file using networkx. You should now have a directed network with all the segments having no direction set.
traverse the network and set the directions.
Use the "ancestry" algorithms  that networkx gives you to figure out what's upstream and what's downstream of where you are
Write the results.


Vendor GYP and libuv into the repo.

Instead of dynamically downloading these as needed during a build, this
checks in those two dependencies directly into the Wren repo. That's a
little lame because users of Wren who aren't building the CLI don't
actually need them, but they aren't too big, so it's not a huge deal.
It makes builds (particularly on Travis) more reliable, because they
don't have to pull down additional content over the network.

Built for me correctly on linux-x86_64.
While at it, speaking of libuv, I recently find and issue in wren code where libuv was using already free'd data. See the first patch of my change set:
https://github.com/mhermier/wren/commit/b5f74dcf186b291184664248260da7a5934c79c3 (libuv fixup.)
It seems libuv is taking care at freeing stdinStream, and will do it later, when hitting the event loop. So the extra free makes valgrind cries.

The Visual Studio solution will have to be updated, since it downloaded libuv into the project folder. I can submit a pr for that a bit later.

Argh, it seems like vcbuild.bat wants to build everything, tests included, even though we don't run them. There's only one msbuild incantation in there, and it's for the entire solution. 😕
Maybe there's a work-around, it would be a bit of a bummer to have to include all of libuv.

Hmm.. wouldn't a sub module make more sense instead of adding it to the wren repo itself? It will be counted in the LOC etc.

@paulsapps Because then it's still being cloned, therefore it still counts towards the bandwith limit (#437).

@paulsapps Because then it's still being cloned, therefore it still counts towards the bandwith limit (#437).

But only Gyp suffered from a bandwidth limited repository, not libuv.
I would suggest to only include Gyp in the repository.

A submodule on github will not have that problem @kirbyfan64  if it did then cloning the wren repo itself will too. By submodule I mean git submodule on github - a mirror.


9-a-lot-of-geometries-arent-implemented-in-parser

coverage: 100.0%. remained the same
when pulling 31393ae54688a727dc1d5d09a97e90bd4162c587 on 9-a-lot-of-geometries-arent-implemented-in-parser
into c2cf8eb6e7c2a41d63a508c55f76598e5f2e6313 on main.


Documentation for interaction workflow

Description
This PR adds preliminary documentation for the workflow for human-assistant interactions.
Note that this PR includes changes introduced in : https://github.com/facebookresearch/fairo/pull/916
Type of change
Please check the options that are relevant.

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] Proposes a change (non-breaking change that isn't necessarily a bug)
[ ] Refactor
[ ] New feature (non-breaking change that adds a new functionality)
[ ] Breaking change (fix or feature that would break some existing functionality downstream)
[ ] This is a unit test
[x] Documentation only change
[ ] Datasets Release
[ ] Models Release

Type of requested review

[ ] I want a thorough review of the implementation.
[x] I want a high level review.
[ ] I want a deep design review.

Checklist:

[ ] I have performed manual end-to-end testing of the feature in my environment.
[ ] I have added Docstrings and comments to the code.
[ ] I have made changes to existing documentation where needed.
[ ] I have added tests that show that the PR is functional.
[ ] New and existing unit tests pass locally with my changes.
[ ] I have added relevant collaborators to review the PR before merge.
[ ] [Polymetis only] I have verified that all scripts in tests/scripts runs on hardware.

Codecov Report

Merging #934 (8c9edc4) into main (9c0d326) will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #934      +/-   ##
==========================================
- Coverage   32.27%   32.19%   -0.09%
==========================================
  Files         346      345       -1
  Lines       34145    33901     -244
==========================================
- Hits        11020    10914     -106
+ Misses      23125    22987     -138

Impacted Files
Coverage Δ

agents/droidlet_agent.py


Create new version of homepage throw exception

carousel.products can be null. So, need have a null check.

So, it is possible that the productCodes is not in the carousel component data.

It works now on my local. Need test it on server.

Tested on an updated dev2.
Able to create a new version of homepage, with no errors.


Test case for paper

WIP.
Missing functionality

[x] running m file and test script
[x] fix loads: remove loads from KCL,
[x] add constant current harmonic load model
[x] add RMS voltage limit
[x] add THD voltage limit
[x] add RMS current limit & relax per harmonic current limits
[x] derive transformer losses
[x] add active filter model
[x] add RA limit
[x] add compensation objective for active filters
[ ] define total gen cost in terms of energy across all harmonics

Remaining mismatch between BASF data and current test case

[x] capacitor banks
[x] add in all bounds cross-harmonics (RMS, THD, RM)
[x] make branch r x b realistic
[x] make transformer params realistic

Discuss

active filters consume net 0 power -> but invidiual harmonics can still consume/generate active power

ready for review/merging.
I propose to have a chat on what to do next in terms of unit testing / refactoring


Seeded migrations with RowVersion produces invalid migrations

Migrations where you use HasData to seed the database with entities that use a RowVersion property produces future migrations where the call to DeleteData looks for an entity matching a seeded Id and a null rowversion. Delete then removes nothing from the database and the Insert operation fails.
Steps to reproduce
  migrationBuilder.DeleteData(
    table: "Members",
    keyColumns: new[] { "Id", "Version" },
    keyValues: new object[] { 1, null });

Further technical details
EF Core version: 2.1.2
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system:
IDE: 15.8.2

@AndriySvyryd to find dupe.

This is related to https://github.com/aspnet/EntityFrameworkCore/issues/11969

Thanks for the tracking info guys.


[ie/niconicochannelplus] Fix broken extractor

IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information

Fix VOD extraction and add support for generic URL like https://tenshi-nano.com/video/smLjQ59gotpcqFpBNgVb4vQq
Fixes #11651
Template

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Fix or improvement to an extractor (Make sure to add/update tests)
[ ] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

Hi @c-basalt ,
First great work, it successfully fixes the HTTP Error 400: Bad Request issue,  however, I've encountered a problem when trying to download paid videos. It seems as if the cookies aren't being parsed or used properly. Free content and videos with free parts work without any issues, but paid videos result in a 403 Forbidden error.
Here are the logs from my latest attempt:
[debug] Command-line config: ['--cookies', 'C:\\Users\\jerem\\cookies.txt', '--verbose', 'https://nicochannel.jp/ihara-rikka-officialfc/video/smo4DKnipKSrBACNu6SyddMg']
[debug] User config "C:\Users\jerem\AppData\Roaming\yt-dlp\config.txt": []
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [542166962] (source)
[debug] Lazy loading extractors is disabled
[debug] Git HEAD: 108532613
[debug] Python 3.10.9 (CPython AMD64 64bit) - Windows-10-10.0.22621-SP0 (OpenSSL 1.1.1q  5 Jul 2022)
[debug] exe versions: ffmpeg n7.1-11-g5c59d97e8a-20241012 (setts)
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.08.30, mutagen-1.46.0, requests-2.32.3, sqlite3-3.39.4, urllib3-2.2.3, websockets-14.1
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets
[debug] Loaded 1837 extractors
[NiconicoChannelPlus] Extracting URL: https://nicochannel.jp/ihara-rikka-officialfc/video/smo4DKnipKSrBACNu6SyddMg
[NiconicoChannelPlus] smo4DKnipKSrBACNu6SyddMg: Downloading site settings
[NiconicoChannelPlus] smo4DKnipKSrBACNu6SyddMg: Downloading video info
[debug] [NiconicoChannelPlus] smo4DKnipKSrBACNu6SyddMg: video_type=vod, live_status=not_live
[NiconicoChannelPlus] smo4DKnipKSrBACNu6SyddMg: Downloading video session
ERROR: [NiconicoChannelPlus] Unable to download JSON metadata: HTTP Error 403: Forbidden (caused by <HTTPError 403: Forbidden>)
  File "C:\Users\jerem\yt-dlp\yt_dlp\extractor\common.py", line 742, in extract
    ie_result = self._real_extract(url)
  File "C:\Users\jerem\yt-dlp\yt_dlp\extractor\niconicochannelplus.py", line 198, in _real_extract
    session_id = self._download_api_json(
  File "C:\Users\jerem\yt-dlp\yt_dlp\extractor\niconicochannelplus.py", line 53, in _download_api_json
    return self._download_json(f'{settings["api_base_url"]}{path}', video_id, headers=headers, **kwargs)
  File "C:\Users\jerem\yt-dlp\yt_dlp\extractor\common.py", line 1152, in download_content
    res = getattr(self, download_handle.__name__)(url_or_request, video_id, **kwargs)
  File "C:\Users\jerem\yt-dlp\yt_dlp\extractor\common.py", line 1112, in download_handle
    res = self._download_webpage_handle(
  File "C:\Users\jerem\yt-dlp\yt_dlp\extractor\common.py", line 962, in _download_webpage_handle
    urlh = self._request_webpage(url_or_request, video_id, note, errnote, fatal, data=data,
  File "C:\Users\jerem\yt-dlp\yt_dlp\extractor\common.py", line 911, in _request_webpage
    raise ExtractorError(errmsg, cause=err)

  File "C:\Users\jerem\yt-dlp\yt_dlp\extractor\common.py", line 898, in _request_webpage
    return self._downloader.urlopen(self._create_request(url_or_request, data, headers, query, extensions))
  File "C:\Users\jerem\yt-dlp\yt_dlp\YoutubeDL.py", line 4162, in urlopen
    return self._request_director.send(req)
  File "C:\Users\jerem\yt-dlp\yt_dlp\networking\common.py", line 117, in send
    response = handler.send(request)
  File "C:\Users\jerem\yt-dlp\yt_dlp\networking\_helper.py", line 208, in wrapper
    return func(self, *args, **kwargs)
  File "C:\Users\jerem\yt-dlp\yt_dlp\networking\common.py", line 340, in send
    return self._send(request)
  File "C:\Users\jerem\yt-dlp\yt_dlp\networking\_requests.py", line 365, in _send
    raise HTTPError(res, redirect_loop=max_redirects_exceeded)
yt_dlp.networking.exceptions.HTTPError: HTTP Error 403: Forbidden

It seems the extractor isn't fully handling the authentication flow for paid videos. Could this be related to a missing step or additional API requirement for premium content or is it a mistake on my side ?

Hi @Ryoshiin
Thanks for bringing this up. It turns out the site has dropped cookie auth in favor of localstorage-based JWT auth, so passing cookie no longer works for paid content.
I added support of login with username/password. In case you are using 3rd party login (nicovideo account), there is an option to directly passing JWT using --extractor-args (but keep in mind that token expires every 5min, so refresh the page before trying to pass JWT).

Hi @c-basalt
Thank you for adding these fixes,I truly appreciate your work. It’s a shame Nicovideo logins aren’t supported, as managing JWT tokens every 5 minutes is a hassle.
That said, I’ll likely create a NicoChannel account and get a subscription next month to make things easier. Thanks again for your hard work

I don't think there is currently a native way for yt-dlp to persist non-cookie credential after extraction (where new cookies can be persisted back to cookie file). That means I can still add an arg to take refresh_token which is less hassle, but it will invalidate browser login and is lost after single use.
One possibility is to persist the refresh_token as a cookie, so it can be managed by the cookiefile mechanism, or directly save it as a cache, but I am not sure if it would be acceptable practice.

The first solution wouldn’t fully solve the issue, and I can confirm this after trying it myself. The second option seems more viable, but I understand it might be a pain to implement and could raise concerns. Honestly, it’s probably easier for me to stop using third-party login altogether and switch to a native account instead.
Thanks for your work anyway!


ci: add danger file to ci

This will be to check for common issues folks have when creating PRs

hello @manh-vv! these are changes to CI, so the changes are reflected in the execution of the CI checks you see right here in this PR. as you can see, I'm adding the danger package script to the end of the ci package script we execute for the first CI github action, "lint and test".

if you would like to help work on the GraphiQL react app, we have an open PR for the rewrit and there are lists of things for folks to work on there!

@orta finally got to some relevant errors, it seems theres an issue where it cant find @typescript-eslint plugin although thats working fine for us for yarn lint. thoughts?

I've never used the plugin, so unsure - I generally recommend running the tools lint separately and reading the results JSON


Error 403 - The user isn't allowed to upload to project

Describe the bug

INFO     Response from https://upload.pypi.org/legacy/:
403 The user 'roqueds11' isn't allowed to upload to project 'Basic-Calculator'. See
https://pypi.org/help/#project-name for more information.
Expected behavior

To Reproduce

My Platform

Additional context

Hi, please see https://pypi.org/help/#project-name for more details.


[FEATURE] Customer behavior audit - forbid to upload local files

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I have searched in the issues and found no similar issues.

Describe the feature
We should forbid the customer to upload local files and throw exception if there is any extra local file involved in the engine.
Such as spark.files=/etc/security/keytabs/kyuubi.keytab.
Motivation
It is not security for kyuubi server.
Describe the solution
No response
Additional context
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

cc @yaooqinn @ulysses-you @pan3793

I'm fine with it as a configurable feature


Construct AbstractTask with a list of Action to execute

I want to declare the List<Action> that my custom task has to execute.
Context
I can't do this using the @TaskAction.
Doing so, Action can be reused and unit tested properly.
I would appreciate a sample code to do this.
Thanks.
Regards.

Most tasks should only have one action.  If you wanted to have several separate actions that a task does, you could have a single method that calls each of the other actions.  Those actions could be separate Action classes or methods or however you saw fit to divide them.
You can also add multiple actions with doFirst and doLast, but this is usually only done to existing task types that you can't modify.


Draft: Add verbose CLI flag to the writer::Basic (#192)

Resolves #192
Synopsis
For now there is no way to skip printing World on failed Steps.
Solution
Add skip-world CLI flag.
Checklist

Created PR:

[x] In draft mode
[x] Name contains Draft:  prefix
[x] Name contains issue reference
[x] Has assignee

[x] Documentation is updated (if required)
[x] Tests are updated (if required)
[x] Changes conform code style
[x] CHANGELOG entry is added (if required)
[ ] FCM (final commit message) is posted

[ ] and approved

[ ] Review is completed and changes are approved
Before merge:

[ ] Milestone is set
[ ] PR's name and description are correct and up-to-date
[ ] Draft:  prefix is removed
[ ] All temporary labels are removed

FCM
Add `verbose` CLI flag to the `writer::Basic` (#193, #192)

@tyranron It looks like GitHub Actions are struggling today. Some of the tests hang with log output, despite restarting 3 times
2021-12-27T08:39:08.8808297Z Found online and idle hosted runner in the current repository's organization account that matches the required labels: 'windows-latest'
2021-12-27T08:39:08.9420417Z Waiting for a Hosted runner in the 'organization' to pick this job...
2021-12-27T08:40:09.7555232Z Job was not picked up by the preferred runner, try requesting additional runners to run this job
2021-12-27T08:40:09.9047715Z Waiting for a self-hosted runner to pickup this job...
2021-12-27T08:40:44.9874642Z Job is waiting for a hosted runner to come online.
2021-12-27T08:40:48.3923630Z Job is about to start running on the hosted runner: GitHub Actions 5 (hosted)


Uncaught Error: Module "node:path" has been externalized for browser compatibility. Cannot access "node:path.join" in client code.

ideally node: protocol should work
quickfix: remove the node: protocol from imports
-import { join as joinPath } from 'node:path'
+import { join as joinPath } from 'path'

I also got into the same problem as you did. My site uses not Vite but rather Astro; however, I think they're not different from the rest since Astro is powered by Vite underneath.
Reproduction
import { readdirSync } from "node:fs"

const format = (text: string): string => {
    const capitalizedString = text.charAt(0).toUpperCase() + text.slice(1)
    const finalString = capitalizedString.replace(/\-/gm, ' ')

    return finalString
}

export const getFilesInDirectory = (path: string): { text: string, link: string }[] => {
    const dir = path.split('/').pop() as string
    const filesInDir =
        readdirSync(path, { withFileTypes: true })
            .filter((item) => !item.isDirectory())
            .map(({ name }) => name.split('.')[0])
    const files = filesInDir.map(file => {
        return {
            text: format(file),
            link: `${dir}/${file}`,
        }
    })

    return files
}

Expected behavior
Click the search input and be able to see the Algolia-integrated search box.
Actual behavior
Nothing happens, instead, I see the following error log in the devtool console:

I did install your plugin in order to get it fixed, but no magic happens.

i moved to https://github.com/sodatea/vite-plugin-node-stdlib-browser
... just works

@quoc1707 my other plugin does not take node: protocol imports into account, but I'll likely want to update it to allow those to be externalized. As for this package, I'm going to guess the underlying library I'm using doesn't recognize the protocol imports either. If you don't mind me asking, what's your use case for using the node: protocol if you plan to polyfill the imports anyway?

what's your use case for using the node: protocol if you plan to polyfill the imports anyway?

make node imports explicit

This has been fixed in v0.3.0.

@quoc1707 my other plugin does not take node: protocol imports into account, but I'll likely want to update it to allow those to be externalized. As for this package, I'm going to guess the underlying library I'm using doesn't recognize the protocol imports either. If you don't mind me asking, what's your use case for using the node: protocol if you plan to polyfill the imports anyway?

No, just nothing. I want to clarify that the import is a Node.js built-in module, since there're many alternative packages for these nowadays, like fs-extra or pathe.

@quoc1707 I figured that's what you meant. It definitely makes sense that you'd want to be able to polyfill the builtins, and I was able to get it working with node-stdlib-browser. I looked into sodatea/vite-plugin-node-stdlib-browser as suggested by @milahu, but it didn't correctly handle calls to process in the production build (dev server worked fine though). I think the newest version is a bit more robust, and I plan to introduce some configuration to allow you to toggle the node: behavior. I can imagine a scenario where you might want to polyfill regular imports (e.g. 'fs') and leave protocol imports alone (e.g. node:fs), and having a bit more control over the polyfills, in general, is something I will likely need in the near future.


OTP 18 compile error

vernemq/_build/default/lib/plumtree/src/hashtree.erl:497: erlang:now/0: Deprecated BIF. See the "Time and Time Correction in Erlang" chapter of the ERTS User's Guide for more information.

OTP 18 is not yet supported. (but will come)


compile issues with pyinstaller

Objective: compile an app (using pyinstaller) that has the smart_open module.
Issue: I think the "VERSION" file isn't importing correctly.  Once the app compiles, if you start it in the command line, the error will trigger.
Error: Here's the error message I'm seeing when running the app
Traceback (most recent call last):
  File "splitter_129.py", line 26, in <module>
    import smart_open
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 955, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 665, in _load_unlocked
  File "c:\python360564\lib\site-packages\PyInstaller\loader\pyimod03_importers.py", line 627, in exec_module
    exec(bytecode, module.__dict__)
  File "smart_open\__init__.py", line 39, in <module>
  File "smart_open\__init__.py", line 35, in _get_version
  File "smart_open\smart_open_lib.py", line 307, in open
  File "smart_open\smart_open_lib.py", line 512, in _shortcut_open
FileNotFoundError: [Errno 2] No such file or directory: 'C:\\Users\\domccorm\\AppData\\Local\\Temp\\_MEI118122\\smart_open\\VERSION'

Environment: Windows 10, Python360564
It looks like someone else ran into this issue and posted within SO: https://stackoverflow.com/questions/57482672/filenotfounderror-errno-2-when-running-pyinstaller-executable
I'm not sure if the suggestions/answers on SO work -- I've been struggling and testing (trial and error) to see if I can fix it
Thanks so much,
-Doug Jr.

Hi Doug, check out https://github.com/RaRe-Technologies/smart_open/pull/344 . Can you try that and report whether it solves your issue?

Hi Piskvorky,
Thank you for the suggestion -- your approach solved it.  It would be great if the main smart_open library used that method of converting the VERSION file into version.py -- I'm thrilled that it's working now.
Have a great day,
-Doug Jr.

I've been confronting with a same problem.
Could you tell me the commands you ran?


Fix hash value for correct comparison

Please answer these questions before submitting your pull requests. Thanks!

What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
Fixes #NNNN

Fill out the following pre-review checklist:

[ ] I am adding a new automated test(s) to verify correctness of my new code
[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency

Please describe how your code solves the related issue.
Please write a short description of how your code change solves the related issue.

recheck

I have read the CLA Document and I hereby sign the CLA


Add info about supported tags to props/attrs

For example, the current data for the label attribute is:
https://github.com/raquo/scala-dom-types/blob/6900584aea4e3ae85c5349e4e57ada925a849d4b/shared/src/main/scala/com/raquo/domtypes/defs/reflectedAttrs/ReflectedHtmlAttrDefs.scala#L609-L622
At least according to this page, it is supported only for <track>, <option>, <optgroup>, <menu>, and <menuitem> tags.
https://www.geeksforgeeks.org/html-label-attribute/
In fact, I'm confused—is the label for <track> and <option> really the same?
In any case, Calico would be able to take advantage of this information.

The reason I'm thinking about this now, is because I am looking at web components, and they often seem to be defining their own custom attributes that may be "clashing" with other HTML attributes.
For example, I'm stuck on label specifically because of its use in Material Button.
https://github.com/material-components/material-web/blob/e15c4b86d584cfda5dc850cb697bc9b9552e9536/docs/quick-start.md#usage
Basically we seem to have a "namespacing" issue, where "attributes" should be "namespaced" within their element, because they may have different implementations and types across elements (or not even be available).

Before you go down this road, keep in mind that an attribute's applicability is not just based on the element type, it can require all kinds of complex conditions that are very hard to express in types, e.g. see the example here: https://github.com/raquo/scala-dom-types#reasonably-precise-types
Also, I'm not sure how knowing which element types an attribute applies to will help you de-conflict two label attributes.

In fact, I'm confused—is the label for  and  really the same?

It has the same name and expects the same value type (string), they serve approximately the same purpose, that's about as "same" as it gets in the soup that is DOM. The native JS call to set the label attribute is the same in both cases.
If it's just for web components, then you can define special attribute types specific to that web component, without complicating the types of regular HTML attributes.
In Laminar the canonical approach is to scope the web component's attributes to the web component itself, and access them as properties, e.g. like so:
Button(
      _.id := "myButton",
      _.label <-- actionVar.signal,
      ...
)

Which is just syntax sugar for:
Button(
      Button.id := "myButton",
      Button.label <-- actionVar.signal,
      ...
)

(from https://laminar.dev/examples/web-components, but Antoine's Laminar SAP UI5 Bindings takes the same approach)
This syntax allows using both specific webcomponent attrs as shown above, as well as regular html properties (e.g. (_ => idAttr := "someId")) that you might not want to redundantly define on every web component.

Thanks!

Before you go down this road, keep in mind that an attribute's applicability is not just based on the element type, it can require all kinds of complex conditions that are very hard to express in types

Sure :) I did re-read that "reasonably precise" bit before opening this issue. FWIW I'm not on any particular mission to make it the API typed, I think what we have is good. It just happens that the way Calico is currently working, we can do this "for free". And also, the web component thing put this issues of clashes onto my mind.

If this is just for web components, then you can define special attribute types specific to that web component, without complicating the types of regular HTML attributes.

Yep, I studied the approach in the UI5 bindings. I'm not sure if this technique will work with how we are currently handling "modifiers" in Calico. And/or I didn't like it for other reasons 😂

Also just for the record an older issue about this is https://github.com/raquo/scala-dom-types/issues/13 but that was waaay before we had generators.
Now that we have generators, different UI libraries can pick and choose how much of the available data they want to use, so in principle I don't mind if someone added this data to SDT.

Also just for the record an older issue about this is #13 but that was waaay before we had generators.

Ah, my bad, I couldn't tell from the title :)

Also, I'm not sure how knowing which element types an attribute applies to will help you de-conflict two label attributes.

Good question. Here's a prototype of how I plan to do this in Calico. In particular, check out the fizzBuzzAttr which is a Boolean for fooTag and an Int for barTag.
//> using lib "org.typelevel::shapeless3-deriving:3.3.0"
//> using option "-Ykind-projector:underscores"
import shapeless3.deriving.K0

trait Modifier[E, M]

trait SetAttribute[A, V]

class Tag[E]:
  def apply[M <: Tuple](modifiers: M)(using
      K0.ProductInstances[Modifier[E, _], M]
  ): E = ???

class Attribute[A]:
  def :=[V](value: V): SetAttribute[A, V] = ???

// common to both
def sharedAttr: Attribute["shared"] = ???

// only on foo
def fooAttr: Attribute["foo"] = ???

// only on bar
def barAttr: Attribute["bar"] = ???

// on foo and bar, but with different types
def fizzBuzzAttr: Attribute["fizzbuzz"] = ???

trait Element
given sharedForElement[E <: Element]: Modifier[E, SetAttribute["shared", String]] = ???

trait FooElement extends Element
given fooForFooElement[E <: FooElement]: Modifier[E, SetAttribute["foo", String]] = ???
given fizzBuzzForFooElement[E <: FooElement]: Modifier[E, SetAttribute["fizzbuzz", Boolean]] = ???

trait BarElement extends Element
given barForBarElement[E <: BarElement]: Modifier[E, SetAttribute["bar", String]] = ???
given fizzBuzzForBarlement[E <: BarElement]: Modifier[E, SetAttribute["fizzbuzz", Int]] = ???

def fooTag: Tag[FooElement] = ???
def barTag: Tag[BarElement] = ???

def demo =
  fooTag(
    sharedAttr := "hello",
    fooAttr := "foo",
    fizzBuzzAttr := true
  )

  barTag(
    sharedAttr := "hello",
    barAttr := "bar",
    fizzBuzzAttr := 42
  )

Ah, I see. I figured it'd be something with implicits. Interesting pattern.
Note: I don't think that there's ever a case in the DOM where the same attribute / property has a different type based the element type. Like, the label attribute will always be string, for all elements that it's defined for. The only part I'm not sure about are HTML vs SVG attributes.

Ah, I see. I figured it'd be something with implicits. Interesting pattern.

Part of my motivation for this pattern was to avoid relying on implicit conversions, which seem to be discouraged in Scala 3.

Note: I don't think that there's ever a case in the DOM where the same attribute / property has a different type based the element type.

That may be true, but IIUC that seems down to luck or "good practice" rather than a hard rule.

The only part I'm not sure about are such incompatibilities between HTML and SVG attribute names.

Exactly, and web components creates new potential for such incompatibilities. As you point out, attributes for web components could be handled differently. But one of the big ideas of web components in HTML is that they can be used like vanilla elements, so I'm interested to see if we can preserve that experience.


Added RequiredPermissionWarnings to 4 different commandlets

Before creating a pull request, make sure that you have read the contribution file located at
https://github.com/pnp/powerShell/blob/dev/CONTRIBUTING.md
Type

[ ] Bug Fix
[ ] New Feature
[ ] Sample

Related Issues?
Fixes #X, partially fixes #4478, mentioned in #4472 and #4353 , etc.
What is in this Pull Request ?
Add warning to GetAvailableSensitivityLabel, GetDeletedFlow, GetListItem, GetSite commandlets about required permissions. Details below,
Get-PnPAvailableSensitivityLabel
PS C:\Research\PnPCommunity\powershell> Get-PnPAvailableSensitivityLabel
Get-PnPAvailableSensitivityLabel: Forbidden (403):
PS C:\Research\PnPCommunity\powershell>

PS C:\Research\PnPCommunity\powershell> Get-PnPAvailableSensitivityLabel
WARNING: Current access token lacks the following required delegated permission scope on the resource Microsoft Graph:
InformationProtectionPolicy.Read

Get-PnPAvailableSensitivityLabel: Forbidden (403):
PS C:\Research\PnPCommunity\powershell>

Get-PnPSite
PS C:\Research\PnPCommunity\powershell> Get-PnPSite
Get-PnPSite: The remote server returned an error: (401) Unauthorized.
PS C:\Research\PnPCommunity\powershell>

PS C:\Research\PnPCommunity\powershell> Get-PnPSite
WARNING: Current access token lacks one of the following required application permission scopes on the resource SharePoint Online:
Sites.Selected
 or
Sites.Read.All
 or
Sites.ReadWrite.All
 or
Sites.Manage.All
 or
Sites.FullControl.All

Get-PnPSite: The remote server returned an error: (401) Unauthorized.
PS C:\Research\PnPCommunity\powershell>

Get-PnPListItem
PS C:\Research\PnPCommunity\powershell> Get-PnPListItem -List "DynamicFormTest" -PageSize 1000
Get-PnPListItem: The remote server returned an error: (401) Unauthorized.
PS C:\Research\PnPCommunity\powershell>

PS C:\Research\PnPCommunity\powershell> Get-PnPListItem -List "DynamicFormTest" -PageSize 1000
WARNING: Current access token lacks one of the following required application permission scopes on the resource SharePoint Online:
Sites.Selected
 or
Sites.Read.All
 or
Sites.ReadWrite.All
 or
Sites.Manage.All
 or
Sites.FullControl.All

Get-PnPListItem: The remote server returned an error: (401) Unauthorized.
PS C:\Research\PnPCommunity\powershell>

Get-PnPSiteVersionPolicy

Set-PnPSiteVersionPolicy

Guidance

You can delete this section when you are submitting the pull request.*
Please update this PR information accordingly. We use this as part of our release notes in monthly communications.
Please target your PR to Dev branch. If you do not target the Dev branch we will not accept this PR.

Thanks @NishkalankBezawada !


Fix ODCS API call for modular koji tags

ODCS API param for modular koji tags is expected in the source object,
as per the API post method [1] and its documentation.
[1] https://pagure.io/odcs/blob/5bbe27a76518937beef76b30a33450f18c544028/f/server/odcs/server/views.py#_418

CLOUDBLD-1241

Signed-off-by: Athos Ribeiro<EMAIL_ADDRESS>Maintainers will complete the following section

[ ] Commit messages are descriptive enough
[ ] Code coverage from testing does not decrease and new code is covered
[ ] JSON/YAML configuration changes are updated in the relevant schema
[ ] Changes to metadata also update the documentation for the metadata
[ ] Pull request has a link to an osbs-docs PR for user documentation updates
[ ] New feature can be disabled from a configuration file

No RN added; Fix for enhancement introduced in this release cycle


trigger change

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 85.31%. Comparing base (2dc712b) to head (ead31d0).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@                    Coverage Diff                     @@
##           test-resuable-workflow      #47      +/-   ##
==========================================================
+ Coverage                   85.30%   85.31%   +0.01%
==========================================================
  Files                          28       28
  Lines                        1463     1464       +1
==========================================================
+ Hits                         1248     1249       +1
  Misses                        215      215


Swift version should be specified in the Podspec for PromisesSwift pod

Getting this error when updating to PromisesSwift 1.2.9 (cocoapods 1.9.3 and 1.10.0.beta.1):
[!] Unable to determine Swift version for the following pods:

- `PromisesSwift` does not specify a Swift version and none of the targets (`Dummy`) integrating it have the `SWIFT_VERSION` attribute set. Please contact the author or set the `SWIFT_VERSION` attribute in at least one of the targets that integrate this pod.

Hey @edudnyk are you able to propose a fix?

swift_version has been removed from the Podspec in this commit https://github.com/google/promises/commit/70c337e196c81f795eedb5d7a42cf966f07cdc3d. The commit message does not specify the reason for doing that. So, the proposed fix is to revert this commit.

@tristane0 do you remember the history behind removing the s.swift_version = '5' line from PromisesSwift.podspec.

I believe it was removed because we assumed that developers would set the Swift version in their xcodeproj target(s) that depend on PromisesSwift. Also, we wanted to ensure that we supported the most commonly used Swift versions and wanted to avoid having to update the Swift version(s) in the podspec each time a new version was released.
@edudnyk are you setting the Swift version for the target(s) in your xcodeproj that depend on PromisesSwift? If you are setting it, are you seeing this issue with a specific Swift version?

With build systems like buck and bazel, there is no xcodeproj initially, it gets generated by the build system.
I can confirm that if there is  xcodeproj file with the target that uses PromisesSwift, and that target's build settings have SWIFT_VERSION specified, the error does not get emitted.
But, it does get emitted when target does not specify it.
Let's say I have very old codebase that works with Swift 4, and you are using Swift 5 in your podspec. Why don't you want to enforce Swift 5 for your pod and, instead, inheriting Swift 4 in this situation?

For Bazel, you rely on the Swift version that's set by the current Xcode that you are using to compile your project with. You could override the "default" Swift version if you really wanted to by specifying the swift_version in the copts param of your swift_library target. But this is not really necessary anymore if compiling everything with Swift 5.1 (when module stability was introduced) or newer.
We are essentially saying the same thing with CocoaPods in that we are relying on the default Swift version for the current Xcode that our developers are using to build the PromisesSwift library. I don't think we have any limitations with the versions at the moment, so if a developer, for example, wanted to compile with Swift 4.2, I believe everything would still work.
@ykjchen and @shoumikhin may have additional thoughts.

@tristane0 Prior to defining swift_version in Buck or Bazel, the pod should be installed with cocoa pods. That's where the error is emitted.

@edudnyk I may be misunderstanding your current config for managing your dependencies. Are you using Bazel, Buck, or CocoaPods? It sounds like you may be using CocoaPods and Bazel or Buck. If yes, can you provide some additional details regarding your current setup for your dependencies?

I am using the cocoa pods to load dependencies, then build with buck. There is no initial xcodeproj to integrate the pods into. You can unzip this and run pod install inside of the unzipped folder to reproduce the issue
dummy-target-pods.zip
.

Thank you for the example project to repro. @shoumikhin may be able to provide more details about using Buck to avoid having to rely on Cocoapods.

Pods is dependency management system, there is no dependency management mechanisms in Buck, so I don't think that it is  feasible for me to switch from cocoa pods to something else.

Given that cocoa pods allow to specify :integrate_targets => false, it is inconsistent to rely that there is the parent project around when pod install occurs.

Just to clarify, are you using Buck to create the xcodeproj and relying on CP as the dependency manager? If yes, is there an issue with setting the swift version for the targets in your xcodeproj that depend on Promises rather than relying on CocoaPods to set the version?
To fix the issue you are seeing, which Swift version(s) are you proposing that we add to the PromisesSwift podspec?

are you using Buck to create the xcodeproj and relying on CP as the dependency manager

Yes

If yes, is there an issue with setting the swift version for the targets in your xcodeproj that depend on Promises rather than relying on CocoaPods to set the version?

No, there is no issue with Buck setup itself, if I have the source code of PromisesSwift locally, I will be able to build it with Buck. The issue is with the installation of the PromisesSwift pod v 1.2.9 via pod install (installation of PromisesSwift 1.2.8 works fine, because PromisesSwift.podspec v 1.2.8 has s.swift_version = '5').

To fix the issue you are seeing, which Swift version(s) are you proposing that we add to the PromisesSwift podspec?

I would define the supported version like min( xcode-non-beta-dev-tools-swift-version, promises-swift-highest-supported-swift-version).

Thank you for the clarification! I didn't realize that 1.2.8 contained the swift_version in the podspec.
Would you be able to put together a PR with your suggested fix so that we can discuss there? Can loop in @ykjchen as well.

@edudnyk are you using CocoaPods to fetch the source code and then add your custom BUCK file to it and build?
Can you just download the code from GithHub directly instead? Perhaps, you can leverage git submodules to manage the dependencies.

With submodules, there won't be a dependency management, so, as I said, it is not feasible.

@edudnyk your fix has been published in 1.2.10. Thanks for your contribution.


Modify axioms to fit OWL DL global constraints

I changed two axioms that used min 1 with a non-simple property to be some restrictions instead. This is logically equivalent (which is why HermiT never complained). The first commit in this PR resaves using the latest OWL API, resulting in a large number of formatting changes. The second commit is the fix.
The remaining non-DL issue is a problem with the imported ontology-metadata file:
OWL 2 DL Profile Report: Ontology and imports closure NOT in profile. The following violations are present:
Use of undeclared annotation property: <http://purl.org/dc/terms/license> in annotation [Annotation(<http://purl.org/dc/terms/license> "http://creativecommons.org/licenses/by/4.0/") in null]

I added a commit to update the version IRI. After merging we should create a tag cdao-rel-2019-06-26.

FYI, created release: https://github.com/evoinfo/cdao/releases/tag/cdao-rel-2019-06-26

Thanks!


Integer enum with value description

First of all, thank you for the great work.
Quick question: which annotations should be used to specify an integer enum with values description ?
components:
  schemas:
    RequestTypeEnum:
      type: integer
      enum:
        - 1
        - 2
      description: >
        Sort order:
          * `1` - Description of the requestType = 1
          * `2` - Description of the requestType = 2

Thank you,
Sebastien

Hey, thanks :)
Unfortunately, it's currently not possible to declare enums as ints. We're only supporting string values that are valid Java/Kotlin identifiers for name-based matching. We'd need to implement something like @IndexedEnum to enforce position based schema :thinking:


Add flag to allow/deny flying in claim.

Feature-request:
Add a feature wether a player can fly in a claim or not. This should not set the players fly-state to true if allowed (the player still has to get allowed to fly by another plugin), but should set the fly-state to false if denied.

dupe #267


Implement selective transitional styles

This PR adds a property, designVersion, to gds-theme, which can be used to selectively control the design version (2016 or 2023) for its descendants. This should also fix the broken design version switch in Storybook.

Lookin guuud! Have two thoughts:

Should it rather be token-version instead of design-version?
Should it be possible to manually register components as 2016 or 2023 so you can mix and match?

Lookin guuud! Have two thoughts:

Should it rather be token-version instead of design-version?

I think design-version makes sense, since this affects which stylesheets are used in components, and not just tokens.

Should it be possible to manually register components as 2016 or 2023 so you can mix and match?

Yes, this update makes this possible! The transitional styles can be loaded individually, or in bulk, just as before, but gds-theme can now be used to control where they get applied. So if you need to turn it on/off for an individual components, just wrap in a gds-theme and set the designVersion prop. I can demo later how this works.


[R20-1592] :sparkles: add app information api endpoint

Issue:
What I did

Adds api endpoint to return ripple version and dependency details useful for testers and debugging

How to test

Checklist

For all PR's

[ ] I've added relevant changes to the project Readme if needed.
[ ] I've updated the documentation site as needed.
[ ] I have added unit tests to cover my changes (if not applicable, please state why in a comment)

For new components only

[ ] I have added a story covering all variants
[ ] I have checked a11y tab in storybook passes
[ ] Any events are emitted on the event bus

@dylankelly Not entirely sure what caused it, but something exported through ripple-tide-api was throwing it - I isolated the one util that we needed and cypress is happy again


Impossible to accept Write command from nRF51822 chipset at RPi2 with Bleno running

I plan to make RPi2 function as a peripheral role with a customary service.
The service should maintain one 'Write' characteristic and nRF51822 chipset sends ATT command to change(write into) the value of the characteristic.
I edited "bleno/test.js" to rule out all the characteristics except one having write property and changed the service UUID and the characteristic UUID according to the sample "central" program for nRF51822.
The following is about the sample code used for nRF 51822.
https://devzone.nordicsemi.com/tutorials/21/
It seems like connection is somehow established between them, but write request is not handled at RPi2.
I also tested LightBlue app (in iPhone)  and it worked with no problem.
I ran "hcidump -t -X" and outputs are as follows.
withiPhone.txt
withnRF.txt
The kernel version is as follows.

I installed and tested bluez-5.39, 4.99, 4.101.
All of them showed similar results.

In case of connection with iPhone, response to write request showed up.
I also ran rpi-update to downgrade kernel version to 3.12.36+ according to following link.
http://stackoverflow.com/questions/34966858/connecting-ble-device-to-raspberry-fails
This is how i installed bluez. ( i installed D-BUS, and any of dependent packages via apt-get)
./configure --disable-systemd --enable-library
sudo make
sudo make install
I cannot see the write request from bleno.js side but i can see write request is coming from bluez side.
(both hcidump & btmon showed write ATT command had wrong handle value)
I suspect ATT information exchange (?) failed with nRF51822 chipset, however succeeded with iPhone.
I also read many related issues here and tried 'sudo service stop bluetooth'.
It showed no bluetoothd is launched? and i thought no GATT server is running at bluez side.
so i concluded there is no GATT server running to intercept ATT command message.(which im not so sure about)
I don't know what to do and where to start.
I keep eyes on this issue and can get back to any advice ASAP.
Please let me know any more information is required regarding this issue.
Thanks!

i forgot to mention this. LightBlue App can copy the service from RPi2 and mimic it. nRf 51822 succeeded to send write command to LightBlue ios app.

@vovojh it looks like your nRF51 app is trying to write to handle 0x0000:
    ATT: Write cmd (0x52)
      handle 0x0000 value  0x31 0x0a

Which is always an invalid handle. I'm closing this for now, doesn't seem to be anything related to bleno.


epic2-bw failed with KeyError: 'guess_bampe'

Hi, thanks for developing epic2. I'm not to successfully run the example data generated from epic2-bw --example. Here are the error message:
Traceback (most recent call last):
  File "/group/bioinfo/apps/apps/miniconda-4.8.2/envs/epic2-0.0.48/bin/epic2-bw", line 174, in <module>
    main(args)
  File "/group/bioinfo/apps/apps/miniconda-4.8.2/envs/epic2-0.0.48/lib/python3.9/site-packages/epic2/bigwig.py", line 100, in main
    treatment_ranges = files_to_coverage(args["treatment"], args)
  File "/group/bioinfo/apps/apps/miniconda-4.8.2/envs/epic2-0.0.48/lib/python3.9/site-packages/epic2/bigwig.py", line 75, in files_to_coverage
    ranges = file_to_ranges(f, args, "ChIP")
  File "/group/bioinfo/apps/apps/miniconda-4.8.2/envs/epic2-0.0.48/lib/python3.9/site-packages/epic2/bigwig.py", line 44, in file_to_binned_ranges
    bin_counts, _ = files_to_bin_counts([f], args, datatype)
  File "epic2/src/reads_to_bins.pyx", line 218, in epic2.src.reads_to_bins.files_to_bin_counts
  File "epic2/src/reads_to_bins.pyx", line 240, in epic2.src.reads_to_bins.files_to_bin_counts
KeyError: 'guess_bampe'

Could you please solve this issue? Thank you in advance!

Will look at. Thanks for notifying me

0.0.49 should be out soon. Mind checking if it works for you now? :)

Thank you for your quick response.
Python 3.7.4
pip uninstall epic2  ## Successfully installed epic2-0.0.49
~/.local/bin/epic2-bw -t ~/.local/lib/python3.7/site-packages/epic2/examples/test.bed.gz -c ~/.local/lib/python3.7/site-packages/epic2/examples/control.bed.gz -bw bigwigs/ -i2bw log2fc_bigwigs/ -cbw chip.bw -ibw input.bw -2bw log2_chip_input.bw

Error message:
Traceback (most recent call last):
  File "/home/xie186/.local/bin/epic2-bw", line 182, in <module>
    main(args)
  File "/home/xie186/.local/lib/python3.7/site-packages/epic2/bigwig.py", line 132, in main
    ranges.to_bigwig(bw_name, chromsizes, divide_by=control_sum)
TypeError: to_bigwig() got an unexpected keyword argument 'divide_by'

Thanks for your reply. I've tried fixing this in 0.0.50. But I have no way to test the code on my machine as I have not set up a proper Python env on my new M1 mac.
Please tell me if it works or not :)


Cannot read property 'use' of undefined

Storybook 6.2.9
ERR! TypeError: Cannot read property 'use' of undefined
ERR!     at Object.webpackFinal (...node_modules/storybook-css-modules-preset/index.js:21:24)

I may need a repo example, I think that maybe this plugin is conflicting with some other you have installed

I'm getting this, too, using Create React App.  I'll likely have an example later today, or this week.

I'll give it a shot!

FWIW, while there's an investigation: this solved the issue for me.
https://github.com/storybookjs/storybook/issues/2320#issuecomment-351804807

I may need a repo example, I think that maybe this plugin is conflicting with some other you have installed

I am also using the CRA and also the following addons   addons: [     "storybook-css-modules-preset",     "@storybook/addon-links",     "@storybook/addon-essentials",     "@storybook/preset-create-react-app",     "@storybook/addon-a11y",   ],
Nothing special else.

@Negan1911 I Sorry, I was not attentive. CSS modules are working with the storybook and RCA under the hood already because of RCA webpack configuration. No need of additional addons. You may close the issue


fix request-action issue [skip ci]

Signed-off-by: Peixin Li<EMAIL_ADDRESS>github action https://github.com/NVIDIA/spark-rapids/runs/1765236618?check_suite_focus=true
failed due to dependency's issue https://github.com/octokit/request-action/issues/71
tested in my forked repo.
This PR will be forced merged after approval w/o running CI, since the workflow is blocked by itself

build

build

the author just made a new release to resolve the issue. But let's still get this one into our repo for future usage

the author just made a new release to resolve the issue. But let's still get this one into our repo for future usage


[AMORO-2350] Fix native iceberg format optimize evaluation trigger condition

Why are the changes needed?
Close #2350.
Brief change log

KeyedTable && (ChangeSnapshotId || SnapshotId)
UnKeyedTable && SnapshotId

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before making a pull request

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Fix can't unset exported typed array element when the type is set to Node

Fixes #82034
The code failed at ERR_FAIL_COND(!_p->typed.validate(value, "set")) , NodePath can't be set to a TypedArray of type 'Object'.
I guess the easy way to fix this is what this pr did,  make _node_clear emit an empty variant instead of an empty node_path. But it might has unexpected impacts, I would appreciate any feedback or alternative suggestions.

Thanks!


OAuth login 404?

I'm pretty sure she's missing an email associated to her twitter account
supabase requires twitter accounts to have emails, otherwise it throws an error


Add support for Dark Mode

It would be nice if we can have dark mode support with this theme.

@ribice Do you have plans to support this?

Feel free to create a PR. My css skills aren't good enough (yet) to implement a dark theme.


Clean up in EntityMaterializerSource

https://github.com/aspnet/EntityFrameworkCore/blob/4236dfd7f2789782cbba26bf0af15b04dfaf2a0c/src/EFCore/Metadata/Internal/EntityMaterializerSource.cs#L29-L59
We marked old method obsolete to pass in property for value convertor work but now we don't use the property passed in.

I actually ended up reverting this in new query pipeline for now. Re-opening this and assigning to me to make sure we have correct change in place for final release.

Close; it is needed in the new pipeline, so nothing to do here.


separate loops for evaluating conditions and extracting group members

This patch was missing from ament/ament_package#69 and why it worked locally but not on the farm.

topological order now passes locally with these 2 PRs


style(@dpc-sdp/ripple-ui-core): fix primary nav alignment

Issue: https://github.com/dpc-sdp/ripple-framework/issues/1112
What I did

Fixes alignment on primary nav
Also fixes 'line height' for items that wrap

Checklist

For all PR's

[ ] I've added relevant changes to the project Readme if needed.
[ ] I've updated the documentation site as needed.
[ ] I have added unit tests to cover my changes (if not applicable, please state why in a comment)

For new components only

[ ] I have added a story covering all variants
[ ] I have checked a11y tab in storybook passes
[ ] Any events are emitted on the event bus

closes #1112


about  isMounted() cannot be used in es6

In Component Surface.js, here in a warning: "Surface: isMounted is deprecated. Instead, make sure to clean up subscriptions and pending requests in componentWillUnmount to prevent memory leaks."


ci: bump NUnit3TestAdapter from 4.5.0 to 4.6.0

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 73.30%. Comparing base (9affd59) to head (b307f67).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1090   +/-   ##
=======================================
  Coverage   73.30%   73.30%
=======================================
  Files         293      293
  Lines        3679     3679
  Branches      632      632
=======================================
  Hits         2697     2697
  Misses        812      812
  Partials      170      170

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Use certificate for domain and subdomain

I have multiple apps running in one docker environment, and use nginx to route the main example.com to App 1, and sub.example.com to App 2.
While this works so far, the letsencrypt certificate is just valid for sub.example.com.
Im using this configuration in my docker-compose file:
environment:
      - VIRTUAL_HOST=sub.example.com
      - LETSENCRYPT_HOST=sub.example.com
      -<EMAIL_ADDRESS>
and for App 2:
environment:
      - VIRTUAL_HOST=example.com
      - LETSENCRYPT_HOST=example.com
      -<EMAIL_ADDRESS>
How to use the proxy companion to properly generate certs for both hosts?

Ok, got it working now with the configuraton described above:
I had to manually inspect the docker volumes, and delete the whole certs there.
After restartign the containers, the certs are now generated properly.
But I think I'll still leave this open, as I don't consider this the proper solution.
When I add new containers to my docker environment, the lets encrypt companion should realize that certs are missing, instead of just using the ones already generated.

When I add new containers to my docker environment, the lets encrypt companion should realize that certs are missing

This is normally what it does. There was an issue in your case but I can't say what caused it with just the environment from the two proxyed apps.

I am having an issue where I am creating containers using the same env as above, but just for docker run. The result is that it doesn't always (at random) create the right certs for the sub domains and therefore flags them as being invalid.
In my docker run command I have the correct domain I want, but it is not the domain being used for the certs

Hi.
What version of the container are you using ? Can you provide the full docker run commands you are using to start your nginx-proxy stack and your proxyed containers, and if possible the log output of the letsencrypt container when it issues a certificate with the wrong domain set ?

It was a late night, but when I used the docker-compose rather than docker run commands it works. If it didn't work I could just recreate the docker-letsencrypt-nginx-proxy-companion.
Before I started on the docker-compose route, I did this:
docker run -d \
  --name nginx-proxy \
  -p 80:80 -p 443:443 \
  -v ${PWD}/certs:/etc/nginx/certs:ro \
  -v /etc/nginx/vhost.d \
  -v /usr/share/nginx/html \
  -v /var/run/docker.sock:/tmp/docker.sock:ro \
  --label com.github.jrcs.letsencrypt_nginx_proxy_companion.nginx_proxy \
  jwilder/nginx-proxy

docker run -d \
  --name nginx-letsencrypt \
  -v ${PWD}/certs:/etc/nginx/certs:rw \
  -v /var/run/docker.sock:/var/run/docker.sock:ro \
  --volumes-from nginx-proxy \
  jrcs/letsencrypt-nginx-proxy-companion

docker run -d \
  --name mainweb \
  --expose=80 --expose=443 \
  -p 80 -p 443 \
  -e "VIRTUAL_HOST=sub.my-domain" \
  -e "LETSENCRYPT_HOST=sub.my-domain" \
  -e<EMAIL_ADDRESS>\
  -v ${PWD}/mainweb:/usr/local/apache2/htdocs \
  httpd:2.2.34-alpine

This would work, but the problems started to come up when I started adding more containers, not always (and this was random) giving the right certificate.
The main container that was giving me issues where the combo of a mysql and phpmyadmin container.  If you try to reach http serving containers with directly published ports, that won't go through the proxy and the certificate won't be served.
The --link is a legacy argument dating back from when docker networks didn't exist, I know it's unfortunately still on a lot of containers docs but its usage should be avoided in favor of said docker networks.
To get the logs of the container, do docker logs container-name. If you'd prefer no to publicly post your domains and subdomains (they'll appear in the logs) you can send me the logs by mail.
Do you use the exact same config with docker compose ?

Yeah - I do so I will look into it, the ports thing makes sense.
I will also look into using networks, but as you said a lot of containers (and explanations on the web) still use them.
Sorry if this didn't answer the origin post.

Sorry if this didn't answer the origin post.

No problem 😃
Also, If you have a working setup with docker-compose don't spend too much time trying to find why your first attempt with docker run did not work as intended. I'll give it a try with the same configuration as yours but I don't see any obvious reason for the proxy to randomly serve the wrong certificate.

I seem to run into the exact same issue bcsjk11 is describing. I'm running a number of different subdomains with their own certificates, using largely the same setup bcsjk11 is using (except for using a separate nginx-proxy) network. All subdomains, such as jenkins.domain.com and api.domain.com work properly.
However, the base domain (domain.com) appears to try to use the certificate of one of the subdomains, I believe the most recently generated one.
For instance, these two URL's work properly: https://jenkins.carely.info, https://jira.carely.info/secure/Dashboard.jspa
However, https://carely.info/ is trying to use the certificate belonging to jenkins.carely.info.
docker-compose.yml of one of the subdomains:
version: '3'

services:
  jira:
    image: cptactionhank/atlassian-jira-software
    container_name: jira
    expose:
      - "8080"
    volumes:
      - /var/atlassian/jira:/var/atlassian/jira
    environment:
      - VIRTUAL_HOST=${HOSTNAME}.${DOMAINNAME}
      - LETSENCRYPT_HOST=${HOSTNAME}.${DOMAINNAME}
      - LETSENCRYPT_EMAIL=${LETSENCRYPT_EMAIL}
    restart: always

networks:
  default:
    external:
      name: nginx-proxy

docker-compose.yml of the main domain:
version: '3'

services:
  pts4-application:
    build: .
    container_name: pts4-application
    expose:
      - "80"
    environment:
      - VIRTUAL_HOST=${DOMAINNAME}
      - LETSENCRYPT_HOST=${DOMAINNAME}
      - LETSENCRYPT_EMAIL=${LETSENCRYPT_EMAIL}
    restart: always

networks:
  default:
    external:
      name: nginx-proxy

Where does your compose file pick those variables from ? Are you sure they are set correctly ? Have you tried with hardcoded values ? Do you have any logs from the letsencrypt container to share ?

The compose files pick the variables from a .env file in the same directory as the compose files. I've verified that the variables are set properly through docker inspect <container>. Unfortunately enough with hard-coded values it still doesn't seem to work.
I tried force renewing the certificates through docker exec nginx-letsencrypt /app/force_renew. The certificate for carely.info was generated properly, but then seems to use a different one instead.
The default.conf file is set as follows for the carely.info upstream, which seems to be incorrect:
# carely.info
upstream carely.info {
                ## Can be connect with "nginx-proxy" network
            # pts4-application
            server <IP_ADDRESS>:80;
}
server {
    server_name carely.info;
    listen 80 ;
    access_log /var/log/nginx/access.log vhost;
    include /etc/nginx/vhost.d/default;
    location / {
        proxy_pass http://carely.info;
    }
}

I can't believe I hadn't checked the LetsEncrypt logs yet. The core issue seems to be in here:
2018-03-06 11:29:09,735:DEBUG:certbot.main:certbot version: 0.21.1
2018-03-06 11:29:09,737:DEBUG:certbot.main:Arguments: ['-q']
2018-03-06 11:29:09,738:DEBUG:certbot.main:Discovered plugins: PluginsRegistry(PluginEntryPoint#manual,PluginEntryPoint#null,PluginEntryPoint#standalone,PluginEntryPoint#webroot)
2018-03-06 11:29:09,763:DEBUG:certbot.log:Root logging level set at 30
2018-03-06 11:29:09,764:INFO:certbot.log:Saving debug log to /var/log/letsencrypt/letsencrypt.log
2018-03-06 11:29:09,772:WARNING:certbot.renewal:expected /etc/letsencrypt/live/carely.info/cert.pem to be a symlink
2018-03-06 11:29:09,773:WARNING:certbot.renewal:Renewal configuration file /etc/letsencrypt/renewal/carely.info.conf is broken. Skipping.
2018-03-06 11:29:09,784:DEBUG:certbot.renewal:Traceback was:
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/certbot/renewal.py", line 60, in _reconstitute
    renewal_candidate = storage.RenewableCert(full_path, config)
  File "/usr/lib/python3/dist-packages/certbot/storage.py", line 434, in __init__
    self._check_symlinks()
  File "/usr/lib/python3/dist-packages/certbot/storage.py", line 493, in _check_symlinks
    "expected {0} to be a symlink".format(link))
certbot.errors.CertStorageError: expected /etc/letsencrypt/live/carely.info/cert.pem to be a symlink

2018-03-06 11:29:09,784:DEBUG:certbot.log:Exiting abnormally:
Traceback (most recent call last):
  File "/usr/bin/certbot", line 11, in <module>
    load_entry_point('certbot==0.21.1', 'console_scripts', 'certbot')()
  File "/usr/lib/python3/dist-packages/certbot/main.py", line 1240, in main
    return config.func(config, plugins)
  File "/usr/lib/python3/dist-packages/certbot/main.py", line 1142, in renew
    renewal.handle_renewal_request(config)
  File "/usr/lib/python3/dist-packages/certbot/renewal.py", line 443, in handle_renewal_request
    len(renew_failures), len(parse_failures)))
certbot.errors.Error: 0 renew failure(s), 1 parse failure(s)

Afterwards, I tried to fix the issue through this guide: https://github.com/certbot/certbot/issues/2550#issuecomment-197417732. Not sure if it's worked, as I've seem to hit the LetsEncrypt rate limit :P. Would that be the correct way of resolving this issue?

Wait, what ? Why and how do you get those log output from certbot ?
We aren't using certbot at all in this container but simp_le.

Where should I be retrieving the logs from in the actual container? I pulled these from /var/logs/letsencrypt from the host (which was strange anyway, because I don't have that folder mounted).
I did run a previous separate LetsEncrypt installation prior to using Docker which I uninstalled. Could that be causing issues? I don't recall ever using simp_le though.

Okay, looks like python-certbot was still installed on the host. I've completely removed it, but the issue still persists after restarting the containers. The base domain still uses an incorrect certificate.
Where would one find the LetsEncrypt logs in the container?

You can get them with docker logs name-or-id-of-the-container.

Ah, there we go. Looks like the symlink was indeed the issue, which in turn was caused by my pre-Docker install of the Certbot. Once the ratelimit passes I think the URL will work fine.
My sincerest apologies for wasting your time like this!

No apologies needed ! Could you detail how do you think your previous use of certbot led to a symlink issue ? That could be helpful to me on another issue (#337).

For some reason, when installing nginx-proxy-companion, I decided to set the certs directory to the existing letsencrypt live directory (/etc/letsencrypt/live), as I had assumed I uninstalled certbot properly and it's be easy to remember this directory on the host.
As it turns out, as I had previously already generated a certificate for the base domain, the carely.info folder already existed. This means that the symlinks could not be created by nginx-proxy-companion as it was pointing to an existing file.
As a result, the container failed to create a certificate for carely.info. And that leads to the issue you mentioned in #337: when the certificate generation fails, it uses the wrong certificate.
Disclaimer: This is still hypothetical, as I haven't been able to verify that removing certbot and the /etc/letsencrypt/live folder and having nginx-proxy-companion actually resolves the issue. This is because the certificate creation still fails due to rate limiting. But this seems like the most likely cause.

No activity for 90+ days, closing.


[RFE] Support Tofu/Terraform Workspaces

Various dartboard modules generate files (aka stateful artifacts) that are stored on the dartboard-runner machine. If using tofu/terraform workspaces, these files will be overwritten on a dartboard deploy or dartboard apply.
Suggestion: Ensure all modules that generate files place them in a directory prefixed with the terraform workspace name. ex: ${terraform.workspace}_config

[ ] Store all stateful artifacts in a directory named as ${terraform.workspace}_config
[ ] Should be able to set the terraform workspace that should be used in dart files

If the workspace doesn't exist, it should be created

At the moment all files should be created inside of the main/$backend directory, which kind of acts as a "workspace"
I am totally open to making that parametric so that it can point elsewhere

🤔 In my experience files are just stored in the "main" modules ${path.root}/config/ directory.

OK, so I misunderstood what "workspace" meant
reading https://opentofu.org/docs/cli/workspaces/ clarified the concept for me
I am totally OK with this change 👍


VOLUME should better support string array

restore that test once its fixed:
==================
volume command array variant
==================

foo:
  VOLUME ["/foo", "/bar"]

---

    (source_file
      (target
        name: (identifier)
        (block
          (volume_command
            (string_array
              (double_quoted_string)
              (double_quoted_string))))))

fixed


Fixing Fedora 35 for Power architectures

Attempt to fix compilation problems on Power for Fedora 35 (see: https://kojipkgs.fedoraproject.org//work/tasks/1457/62181457/build.log)

Thanks @hkaiser for changing this! I only pushed a clang-format fix, this was otherwise green.


getsockopt错误，无法连上

VPS报错如下：
2016/08/10 23:17:07 dial tcp <IP_ADDRESS>:8388: getsockopt: connection refused
2016/08/10 23:17:07 [ERR] yamux: Failed to read header: broken pipe

Hi:

Are you using the latest release?
Which OS do you use?
Which end for this issue related to, client or server?
Does your ISP or VPS blocked UDP?
Does your VPS firewall allows UDP pass through?
Are you using the same version for both client & server
Did you correctly set the -target value, -t value	target server address (default: "<IP_ADDRESS>:12948") on the server side?

Are you using the latest release?    Reply :  Yes,
Which OS do you use?  Reply : VPS: ubuntu 14.04, Client: WIN64
Which end for this issue related to, client or server? Reply :  Both for server and client
Does your ISP or VPS blocked UDP?   Reply : Not sure
Does your VPS firewall allows UDP pass through?  Reply :  no firewall
Are you using the same version for both client & server Reply :  Yes
Did you correctly set the -target value, -t value target server address (default: "<IP_ADDRESS>:12948") on the server side? Reply :  Yes, comand as follow:

./server_linux_amd64 -t "<IP_ADDRESS>:8388" -key "onlymeconnectA@a" -crypt "aes" -l ":968"
./client_windows_amd64 -r "VPS IP:968" -key "onlymeconnectA@a" -crypt "aes"-l ":8688"

#102

root@default:~# telnet <IP_ADDRESS>:8388
telnet: could not resolve <IP_ADDRESS>:8388/telnet: Name or service not known
root@default:~# telnet <IP_ADDRESS>:8388
telnet: could not resolve <IP_ADDRESS>:8388/telnet: Name or service not known

我用vultr, 也不能接。
Are you using the latest release? Yes,
Which OS do you use? VPS: ubuntu, Client: win7 64bit
Does your ISP or VPS blocked UDP? No
Does your VPS firewall allows UDP pass through?
sudo ufw status
Status: inactive
Are you using the same version for both client & server? Yes
Server:
~/kcptun# ./server_linux_amd64 -t "<IP_ADDRESS>:8388" -key "onlymeconnectA@a" -crypt "aes" -l ":968"
2017/07/29 02:43:01 version: 20170525
2017/07/29 02:43:01 listening on: [::]:968
2017/07/29 02:43:01 target: <IP_ADDRESS>:8388
2017/07/29 02:43:01 encryption: aes
2017/07/29 02:43:01 nodelay parameters: 0 30 2 1
2017/07/29 02:43:01 sndwnd: 1024 rcvwnd: 1024
2017/07/29 02:43:01 compression: true
2017/07/29 02:43:01 mtu: 1350
2017/07/29 02:43:01 datashard: 10 parityshard: 3
2017/07/29 02:43:01 acknodelay: false
2017/07/29 02:43:01 dscp: 0
2017/07/29 02:43:01 sockbuf: 4194304
2017/07/29 02:43:01 keepalive: 10
2017/07/29 02:43:01 snmplog:
2017/07/29 02:43:01 snmpperiod: 60
2017/07/29 02:43:01 pprof: false
2017/07/29 02:43:01 remote address: X.X.X.X:59964
2017/07/29 02:43:01 dial tcp <IP_ADDRESS>:8388: getsockopt: connection refused
2017/07/29 02:43:02 dial tcp <IP_ADDRESS>:8388: getsockopt: connection refused
2017/07/29 02:43:03 dial tcp <IP_ADDRESS>:8388: getsockopt: connection refused
Client:
D:\Program Files\Net\KcpTun>client_windows_amd64 -r "X.X.X.X:968" -key "onlymeconnectA@a" -cryp
t "aes" -l ":8688"
2017/07/29 10:43:01 version: 20170525
2017/07/29 10:43:01 listening on: [::]:8688
2017/07/29 10:43:01 encryption: aes
2017/07/29 10:43:01 nodelay parameters: 0 30 2 1
2017/07/29 10:43:01 remote address: X.X.X.X:968
2017/07/29 10:43:01 sndwnd: 128 rcvwnd: 512
2017/07/29 10:43:01 compression: true
2017/07/29 10:43:01 mtu: 1350
2017/07/29 10:43:01 datashard: 10 parityshard: 3
2017/07/29 10:43:01 acknodelay: false
2017/07/29 10:43:01 dscp: 0
2017/07/29 10:43:01 sockbuf: 4194304
2017/07/29 10:43:01 keepalive: 10
2017/07/29 10:43:01 conn: 1
2017/07/29 10:43:01 autoexpire: 0
2017/07/29 10:43:01 scavengettl: 600
2017/07/29 10:43:01 snmplog:
2017/07/29 10:43:01 snmpperiod: 60
2017/07/29 10:43:01 connection: <IP_ADDRESS>:59964 -> X.X.X.X:968
2017/07/29 10:43:04 stream opened
2017/07/29 10:43:06 stream closed
2017/07/29 10:43:07 stream opened
2017/07/29 10:43:07 stream closed
2017/07/29 10:43:08 stream opened
2017/07/29 10:43:08 stream closed

dial tcp <IP_ADDRESS>:8388: getsockopt: connection refused


TypeError: Error #1006: value is not a function

Upgraded to latest flex/flash/air libraries and game now fails with:
TypeError: Error #1006: value is not a function.
	at MethodInfo-1448()
	at org.osflash.signals::Slot/execute()
	at org.osflash.signals::OnceSignal/dispatch()

flex=4.15.0/flash=20.0
Any help appreciated.

Any chance of running that in a debug player (not even sure that exists?) to get some more info, it's really limited information and I no longer use flash, but could help if I had more.

Thanks @SimonRichardson - Was an issue in my listener code, all sorted now.


Check that padding, spacing, etc. work with inequal step over dimentions

[ ] math_core.py, grid_in_cube, spacing
[ ] math_core.py, grid_in_cube2, spacing
[ ] math_core.py, scale_xyz, scale
[ ] math_core.py, scale_xyz, scale
[ ] datagen.py, DataGenForSegmentation, cube_spacing
[ ] datagen.py, DataGenForShapeRegression, spacing
[ ] datagen.py, _scale_rotate_shift, scale
[ ] method_set.py, SamplingMethodSetNode, sampling_grid, spacing
[ ] sampling_grid_with_noise.py, SamplingMethodSetNode, sampling_grid, spacing
[ ] sampling_grid_with_noise.py, SDTMethodSetNode, surface_xyz2localsdt, spacing
[ ] method_set_train_task.py, TrainTaskSetNode, train_task_sdt2sdf, spacing
[ ] space.py, Space, preload, ps_padding, ns_padding
[ ] abstract.py, AbstractBBoxNode, plot
[ ] vizualize.py, save_3D_slices
[ ] vizualize.py, save_sdt_as_obj
[ ] vizualize.py, save_mesh
[ ] vizualize.py, save_3D_array

I'm going to test math_core.py


Renaming the project

New full name: Adaptive Filtering Wizard
Short name: AFWizard
Package name: afwizard
TODO:

[x] Text replacement adaptivefiltering -> AFwizard
[x] Python package replacement adaptivefiltering -> afwizard
[x] Github repository renaming
[x] Readthedocs integration fix
[x] Codecov integration fix
[x] PyPI new package, remove old one
[x] adaptivefiltering-library -> afwizard-library
[x] adaptivefiltering-test-data -> afwizard-test-data
[x] DockerHub repository

Renaming finalized.


Update geo

update to latest geo/geo-types version

Published as 0.9.0: https://crates.io/crates/osm_boundaries_utils/0.9.0


Retype gcp.storage.BucketObject::source as Asset

This is a fix for the bug seen in pulumi-gcp#87 - I suspect there will be fallout for examples and so forth, hence opening the PR early to check CI.

The build here should be fixed if rebased on #106 to pick up new pulumi-terraform changes.


fix for HHH-13651

I added support for null element in embedded collection snapshot equality calculation as it was before (version < 5.4.4 or version < 5.3.11).
This prevent to get a NPE while calculating the equality because the current code try to compute hashcode for null object.

thanks @nyouna ! I'd merge it but it needs some more steps:

[ ] use the Hibernate ORM code style
[ ] the JIRA includes a test; would be nice toadapt it and include it in our regression tests
[ ] if you could prefix commits with the JIRA code that would be great, like "HHH-13651
NPE on flushing when ElementCollection field contains null element" rather than "fix for .."

if you could do this, that would be great!

@Sanne Thanks for the comment, i'll fix the 3 points you mentioned ASAP and will update the PR.
About the test case, I wrote it (sorry for using 2 different users for jira (Google IDP) and for github.
I'll already launched it and verified it passes after the fix.
I'll include it in the regression tests. Do you think it is ok to add a test to org.hibernate.test.collection.bag.PersistentBagTest or do you prefer me to add it as a new test class?

@Sanne : remarks 1 and 3 were pushed. just 1 point (and this is the reason for now that I didn't squashed the 2 commits) applying the codestyle to the modified file (PersistentBag) updated a few other lines that are not related to my change. It seems it wasn't applied before. Is it ok for you?

I've merged this and included an adapted version of the test. Thanks!

@Sanne Thanks!


Video

Is it possible to work with video? It would be nice

Maybe. I will see what can be done. Interesting idea 👍

There is one small bug here, I'm downloading the video all right, I can play, flip and everything else, but only I press Cropp it turns it into a picture, it's possible to get some video, maybe there are extra parameters?

"press" means click?

When I execute this.myCroppa.generateDataUrl () I get png back, so I realized that I need another option to use, I need to use something else?

Yes, generateDataUrl won't give you a video. You need to use the technique introduced in these links:
https://developers.google.com/web/updates/2016/10/capture-stream
https://developers.google.com/web/updates/2016/01/mediarecorder

Hey!
I tried to use the option :video-enabled=true and I get a video object. Cool.
My problem is croppa do not get the first frame and draw it on the canvas. So right now I only see a transparent image and unfortunately there is no error.
Here a simple CodePen to reproduce the problem:
https://codepen.io/psyphant/pen/vYOQrZO


Rolled back PR #44, combined AppPool and WebSite state checks into 1 …

…resource that is compatible with new IIS Administration module and backwards compatible for anything with posh 2.0 and IIS 7+

Thanks, Jesse.


Running dockerfile hangs at Success SQLServer 2017. Remaining steps do not run.

Just trying to familiarize myself with windows containers I decided to attempt to build my own container using the dockerfile supplied with the Microsoft/mssql-docker project. When I run the build, it will get to the step where it installs sql express and it will hang at the Success...privacy policy details for sql server express 2017 (ver : 14.00.1000.169).
The host is a Windows 2016 server (ver : 10.0.14393) virtual machine (vbox) running with a docker pull microsoft/windowsservercore:latest (ver : 10.0.14393).
I can ctrl-c out of the build then do a docker run and it works however this is less than ideal as there are remaining tasks that need to run not to mention if I want to customize this dockerfile by adding additional tasks.
Thanks in advance.

@gforty
Which step you stuck in when building the dockerfile?
What version of docker are you using ?
Can you have a look in your Docker logs in AppData or ProgramData?
Please have a check following similar thread and see if that helps.
"docker build" immediately hangs indefinitely. Windows 10
Currently

@Pirlo-Zhang Thanks for the response.
It hangs at, what seems to be, the end of the sqlexpress install where it provides several "Success!" messages. It is almost as if it is waiting on a response from the user.
I am running docker version 17.06.2-ee-16 (client and server)
I forgot to mention I am attempting to do this offline. I did a docker pull for the windowsservercore online. I slightly modified the code removing the web-request and adding a COPY task for the sqlexpress executable from this (line 16-20 in original dockerfile):
`COPY start.ps1 /
WORKDIR /
RUN Invoke-WebRequest -Uri $env:sql_express_download_url -OutFile sqlexpress.exe ;
Start-Process -Wait -FilePath .\sqlexpress.exe -ArgumentList /qs, /x:setup ; `
To this:
`COPY SQLEXPR_x64_ENU.EXE /sqlexpress.exe
COPY start.ps1 /
WORKDIR /
#RUN Invoke-WebRequest -Uri $env:sql_express_download_url -OutFile sqlexpress.exe ;
RUN Start-Process -Wait -FilePath .\sqlexpress.exe -ArgumentList /qs, /x:setup ; `
I successfully built container using dockerfile with Invoke-WebRequest but not with my slight modification.
Thanks in advance!

OK. I apologize. If I wait long enough it eventually finishes. While I was attempting this build (several times yesterday), Windows update service had a hung process and was pegging the host VM's CPU so had I waited longer than an hour it eventually would have completed I believe. I freed up the CPU and reran with my modified Dockerfile and it ran successfully.


Fix build failures due to CocoaPods version

Recently the plugins CI started failing after a CocoaPods version change in the framework repo: flutter/flutter@c8bf7abeee7cc06b985aa719396603ff43821507.
This change brings the plugins CI in line with what the main Flutter repository is doing

Another case where https://github.com/flutter/flutter/issues/33440 would have saved us some trouble.
I wonder how this change affects our users, do they need to manually upgrade cocoapods when they get the new framework version?


CustomerUpdatedWebhook: use provided customer object from the event

It saves a network roundtrip, and might even be necessary if events are supposed to be processed in order.
TODO:

[x] tests?!

Updated branch, so please squash-merge.

@paltman
This is ready.


Reproducer for HHH-16049 Setting a property to its current value with bytecode enhancement enabled results in unnecessary SQL Update in some (many) cases

https://hibernate.atlassian.net/browse/HHH-16049
Draft: this is just a reproducer, not a fix.

Superseded by  #5967


Private Encrypted Channel Docs

Description of the pull request
Added docs to help people use private encrypted channels
...
Why is the change necessary?
Without docs, people won't know how to use them!
...

I have nothing to add to what Tom said :)

Thanks for the feedback Tom.
I've added a section in each of the installation instructions to let people know there's more if they're using encrypted channels and linked (correctly) to the docs. I'm not sure if there's a better way in markdown to make it more noticeable - like an info box or something.


Clean up MetricCollection

This converts up the current abstract MetricCollection into an interface, and uses a standard serialization method in fasterxml instead of the custom serializer (see removal of MetricCollectionSerialization).
This is in preparation of experimenting with additional Metric types, in which modifying MetricCollection would be a central change. Simplifying and streamlining this will improve the clarity of future pull requests.
While doing this, a few bugs to existing Metrics were detected, and tests introduced in BasicSerialization.

Codecov Report

Merging #334 into master will decrease coverage by 0.02%.
The diff coverage is 59.61%.

@@             Coverage Diff              @@
##             master     #334      +/-   ##
============================================
- Coverage     46.69%   46.66%   -0.03%
+ Complexity     2621     2620       -1
============================================
  Files           628      628
  Lines         16606    16641      +35
  Branches       1211     1217       +6
============================================
+ Hits           7754     7766      +12
- Misses         8401     8418      +17
- Partials        451      457       +6

Impacted Files
Coverage Δ
Complexity Δ

...ain/java/com/spotify/heroic/metric/MetricType.java
94.44% <ø> (ø)
6 <0> (ø)
:arrow_down:

...c/main/java/com/spotify/heroic/metric/Payload.java
50% <0%> (-19.24%)
4 <0> (-1)

...main/java/com/spotify/heroic/metric/FetchData.java
83.33% <100%> (ø)
13 <0> (ø)
:arrow_down:

...va/com/spotify/heroic/metric/MetricCollection.java
57.35% <60%> (-4.47%)
12 <9> (-1)

...m/spotify/heroic/metric/EmptyMetricCollection.java
60% <75%> (-6.67%)
3 <3> (+2)

...m/spotify/heroic/aggregation/simple/MaxBucket.java
42.85% <0%> (-9.53%)
4% <0%> (-1%)

...c/main/java/com/spotify/heroic/metric/Tracing.java
80% <0%> (-6.67%)
12% <0%> (ø)

...tatistics/semantic/MinMaxSlidingTimeReservoir.java
88% <0%> (-6%)
13% <0%> (-1%)

...otify/heroic/jetty/HttpJettyConnectionFactory.java
83.33% <0%> (-2.39%)
2% <0%> (ø)

...ify/heroic/jetty/Http2CJettyConnectionFactory.java
83.33% <0%> (-2.39%)
2% <0%> (ø)

... and 30 more

Continue to review full report at Codecov.

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


some fixes I made so that templates.js would not crash on the client side

Got this error on Chrome, and only on the disney branch:
Uncaught DataCloneError: An object could not be cloned.
Related:

http://www.nowherenearithaca.com/2013/07/solved-chrome-web-workers-and.html
https://github.com/csantanapr/dapp-examples/issues/31

So I fixed it :shipit:
As for wrapping callbacks[e.data.signature] in a conditional, I was getting undefined is not a function errors as well... there's probably an underlying issue for that, but this works as a bandaid.

I see why it errored out, we're passing click functions when we parse alerts. This fixes it thanks :+1:


Project workflow

We need to put together a workflow for working on any given project.

Leaving this open for now, but this is partially addressed in #7


Add support for localhost/loopback addresses as redirect URI for certain scenarios/Spring profiles

Expected Behavior
The acceptance of localhost/loopback address as a redirect URI should be configurable (yet still defaults to false).
Current Behavior
In OAuth2AuthorizationEndpointFilter.isValidRedirectUri(String, RegisteredClient), localhost/loopback address will always return false.
Context
While the backend developers are not really affected here, the developers of the frontend SPA which use localhost as the redirect URI is unable to test the features currently in development. Ideally, this should be configurable but defaults to the most secure configuration.
Any workaround besides downgrading to 0.1.0 will also be nice.

Hi @robbydf. Unfortunately, it looks as though this functionality is quite intentional. Here is the comment in that particular section of code:
		String requestedRedirectHost = requestedRedirect.getHost();
		if (requestedRedirectHost == null || requestedRedirectHost.equals("localhost")) {
			// As per https://tools.ietf.org/html/draft-ietf-oauth-v2-1-01#section-9.7.1
			// While redirect URIs using localhost (i.e.,
			// "http://localhost:{port}/{path}") function similarly to loopback IP
			// redirects described in Section 10.3.3, the use of "localhost" is NOT RECOMMENDED.
			return false;
		}

Any way you can use loopback IP or add an entry to your DNS host file for a domain you have set up with the provider?

Hello @sjohnr. Thanks but correct me If I'm wrong. I think loopback addresses are also invalid:

    		if (!LOOPBACK_ADDRESS_PATTERN.matcher(requestedRedirectHost).matches()) {
    			// As per https://tools.ietf.org/html/draft-ietf-oauth-v2-1-01#section-9.7
    			// When comparing client redirect URIs against pre-registered URIs
    			// authorization servers MUST utilize exact string matching.
    			return registeredClient.getRedirectUris().contains(requestedRedirectUri);
    		}

And for adding a DNS entry, I think making a whole team of front-end developers do that is a bit counter intuitive. My take is that this should be as friction-less as possible (if possible).

Hi @robbydf.  I re-read that section of the spec again just to be sure. I think it is actually allowed to use e.g. <IP_ADDRESS> in the redirect URI. Here's what it says:

Specifying a redirect URI with the loopback IP literal
rather than "localhost" avoids inadvertently listening on network
interfaces other than the loopback interface.  It is also less
susceptible to client-side firewalls and misconfigured host name
resolution on the user's device.

But I definitely see your point about friction. We can look into a configuration option. It is hard to say at this point when we might be able to get to it though. Any chance you might be interested in contributing something to make it configurable?

Hello @sjohnr. I retried using <IP_ADDRESS> and yes, it works. Thanks for pointing that out. I'll be closing this issue.
Regarding contribution, I'll certainly look in to it. Thanks again.


Define a view-scoped cache in memory, and have label_details use it

Turns out that fetching from the disk-based cache can be a bottleneck if we have to fetch the same large structure 16,000 times in a row.
That is what the label-list view currently does. It fetches the label-details structure twice per label (and there are 8000+ labels in production). For example, the code for getting any label's popularity (to show the colored bar on the label list) involves fetches the label-details structure from the disk-based cache. Those fetches apparently add up to incur 10-20 minutes of load time. It's still better that the popularities are cached at all, so we don't have to compute the popularities for every label on every page load, but it's still much slower than desired.
There were multiple ways of fixing this, but I opted to define another layer of caching: a "view-scoped cache" which is in-memory (thus, faster than the disk-based Django cache), created at the start of a web request, and cleared after handling the request. With this setup, the disk-based Django cache saves us from computing the popularities very often, and the view-scoped in-memory cache saves us from having to fetch the label details from disk more than once per view.
This might raise the question of why we don't use a single memory cache (Redis, Memcached) to try to get the best of both worlds. That might end up being a route I take later if there are more caching pains. But for now, I still find that a disk-based cache smooths things out on the testing, development, and maintenance side; for example, cached values that take a while to generate don't have to be re-generated when the server's restarted. I was also thinking that the view-scoped property of this new cache can be quite useful in other situations, such as caching the results of database queries.

Label list now takes 5 seconds to load instead of 10-20 minutes.