Docker-ce Installation issue

After successfully install to OL8 from Centos, i tried to install Docker-ce but faces many issues of dependencies.
dnf install docker-ce
Last metadata expiration check: 0:30:46 ago on Wed May 12 12:23:18 2021.
Error:
Problem: package docker-ce-3:20.10.6-3.el8.x86_64 requires container-selinux >= 2:2.74, but none of the providers can be installed

cannot install the best candidate for the job
package container-selinux-2:2.107-2.module+el8.1.0+5440+994fc847.noarch is filtered out by modular filtering
package container-selinux-2:2.123.0-2.module+el8.1.0+5460+5d763c32.noarch is filtered out by modular filtering
package container-selinux-2:2.124.0-1.gitf958d0c.module+el8.2.0+7621+b33f33e5.noarch is filtered out by modular filtering
package container-selinux-2:2.124.0-1.module+el8.1.1+5502+fbec5cc6.noarch is filtered out by modular filtering
package container-selinux-2:2.124.0-1.module+el8.2.0+5584+b2b2d3fb.noarch is filtered out by modular filtering
package container-selinux-2:2.124.0-1.module+el8.2.0+5585+88100286.noarch is filtered out by modular filtering
package container-selinux-2:2.124.0-1.module+el8.2.0+7615+180dc822.noarch is filtered out by modular filtering
package container-selinux-2:2.124.0-1.module+el8.2.0+7618+3a616245.noarch is filtered out by modular filtering
package container-selinux-2:2.135.0-1.module+el8.2.1+7658+86e51d52.noarch is filtered out by modular filtering
package container-selinux-2:2.75-1.git99e2cfd.module+el8+5190+20ee9d65.noarch is filtered out by modular filtering
package container-selinux-2:2.75-1.git99e2cfd.module+el8+5201+6423ecab.noarch is filtered out by modular filtering
package container-selinux-2:2.94-1.git1e99f1d.module+el8.0.0+5214+b7869c3a.noarch is filtered out by modular filtering
package container-selinux-2:2.94-1.git1e99f1d.module+el8.0.0+5215+77f672ad.noarch is filtered out by modular filtering
package container-selinux-2:2.94-1.git1e99f1d.module+el8.0.0+5336+de35ffea.noarch is filtered out by modular filtering
package container-selinux-2:2.94-1.git1e99f1d.module+el8.0.0+5337+91623acb.noarch is filtered out by modular filtering
package container-selinux-2:2.94-1.git1e99f1d.module+el8.0.0.z+5239+bd2c200f.noarch is filtered out by modular filtering
package container-selinux-2:2.94-1.git1e99f1d.module+el8.1.0+5440+42cffa37.noarch is filtered out by modular filtering
package container-selinux-2:2.144.0-1.module+el8.3.0+7866+f387f528.noarch is filtered out by modular filtering
package container-selinux-2:2.124.0-1.gitf958d0c.module+el8.3.0+7864+d0e2eddf.noarch is filtered out by modular filtering
package container-selinux-2:2.130.0-1.module+el8.3.0+7863+c7f8774f.noarch is filtered out by modular filtering
package container-selinux-2:2.130.0-1.module+el8.3.0+9619+9eeb6ed6.noarch is filtered out by modular filtering
package container-selinux-2:2.155.0-1.module+el8.3.1+9659+c1901784.noarch is filtered out by modular filtering
package container-selinux-2:2.130.0-1.module+el8.3.0+9660+5fcec09e.noarch is filtered out by modular filtering
package container-selinux-2:2.124.0-1.gitf958d0c.module+el8.3.0+9668+293abd4d.noarch is filtered out by modular filtering
package container-selinux-2:2.130.0-1.module+el8.3.0+9670+b9fad87d.noarch is filtered out by modular filtering
package container-selinux-2:2.107-3.el7.noarch is filtered out by modular filtering
package container-selinux-2:2.77-5.el7.noarch is filtered out by modular filtering
(try to add '--skip-broken' to skip uninstallable packages or '--nobest' to use not only best candidate packages)

---- I tried everything i could from "checking repo" | "update yum" to "add new repo" | reinstall serveral dependencies but no luck

Podman is a drop-in replacement for Docker CE. Can you use Podman in Oracle Linux 8: https://docs.oracle.com/en/operating-systems/oracle-linux/podman/

I second what @honglinsu said.  If you also install the podman-docker package, this that effectively aliases the docker command to podman. Installation instructions are here:  https://docs.oracle.com/en/operating-systems/oracle-linux/podman/podman-install.html

As for why Podman was referred to here: Docker CE is not supported on OL8; the OS family that includes Oracle Linux (this includes RHEL and CentOS) has switched to podman as its standard container technology.
I know it's a change, but it's one worth understanding and using if you're going to use this OS family. Give it a try :)

Did you try adding --nobest as per the message? That may get you the container-selinux package from Docker. If not, you can try adding module_hotfixes=True to the Docker CE repo definition to see if that bypasses the modular filtering.

Did you try adding --nobest as per the message? That may get you the container-selinux package from Docker. If not, you can try adding module_hotfixes=True to the Docker CE repo definition to see if that bypasses the modular filtering.

Hey Thanks, So basically i did add module_hotfixes=True to Docker-ce.repo to bypass Module but still no luck same package error
For what i understand - It's being filtered out by something with yum, it's being blacklisted/filtered from being installed.

@Djelibeybi
There were blaclisted ( container-selinux ) package which not letting docker-ce to install but i did search module filtering from dnf docs - https://dnf.readthedocs.io/en/latest/command_ref.html#package-filtering so basically its been filtered out by yum ¯_(ツ)_/¯.
So for bypass add module_hotfixes=True to Docker-ce.repo and then i checked list -
dnf module list
and then found Container tools -
dnf module reset container-tools - Reset those modules
and then install - dnf module install container-tools - (because somehow in OL8 @tvierling "podman as its standard container technology"
It resolves the modular filtering of Container-selinux ---
"Coffee Break"
dnf install -y docker-ce --nobest  - ( https://oracle-base.com/articles/linux/docker-install-docker-on-oracle-linux-ol8 follow these steps wonder why " dnf remove -y runc " to remove buildah cockpit-podman libvarlink podman i guess same @tvierling   "podman as its standard container technology" ¯_(ツ)_/¯
at last - dnf install -y docker-ce --nobest --nogpgcheck
Worked and installed Docker-ce.
Thanks Everyone.

Thanks for reporting back, @johna77.


formula_installer: fix build/test reqs not being pruned correctly

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

This check doesn't have anything to do with whether the requirement itself is a build/test requirement so split it from that logic.
Summer project: write tests for all this.
Fixes #7618.

Summer project: write tests for all this.

Yeh, that'd be great. Another thing I was mulling: try to consolidate this logic a bit more (which may also make it easier to test); there's similar logic in formula.rb, build.rb, dependencies.rb and formula_installer.rb. Could warrant a new class (or be in dependencies.rb) with no internal state.

Yeah, that's also an idea. Though there's several intentional small differences between the implementations which make it very awkward (e.g. we don't want the satisfied check for brew deps).
I'd still like to eventually find a way to merge dependencies and requirement pruning a bit more as that would significantly simplify that (remove the need to do the reverse checking stuff the requirements need to do - like the one in this pull request).

Though there's several intentional small differences between the implementations which make it very awkward (e.g. we don't want the satisfied check for brew deps).

Aye, I'm thinking that's the type of thing that could be passed as a keyword argument and checked as a conditional.

I don't want to do something big like that without proper tests however.

Agreed. I think given the lack of tests at the moment splitting it out and writing some tests may go well hand-in-hand.


Refactor CLI

CLI violates the SRP.
Divide CLI code into Application and CLI.

Cleaned it up. Good enough for now.


SOLR-16415: ZK DistributedMap should reject slash in IDs.

asyncId should not have forward slashes in it
SizeLimitedDistributedMap should be resilient to directories (from before)

https://issues.apache.org/jira/browse/SOLR-16415
From what I see, this will work for both Overseer & Distributed Admin API modes.  The latter is processed by DistributedApiAsyncTracker and it uses DistributedMap and that's what this PR touches.

@patsonluk you may be interested based on your previous modifications

I'll merge this Monday if I hear no further feedback.  We've been using this in production and it's been peer reviewed.


The password input element doesn't get fully highlighted when using auto-fill - Reported by @thesahindia

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open app
Enter valid user name and manually enter password
Reopen app
Enter valid username with saver password
Select to pre-fill password

Expected Result:
Display should be consistent
Actual Result:
Color is inconsistent in the background
The password input element doesn't get fully highlighted when using auto-fill
Context
"To fix this, I guess we’d need to make the input stretch full width of the container with a lot of right padding, and then absolutely position the toggles/controls on top of it with a transparent background"
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: 1.1.24-0
Reproducible in staging?: Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: @thesahindia
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1640198971467800
View all open jobs on GitHub

Triggered auto assignment to @NikkiWines (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Proposal
to fix this we just need to absolutely position the eye icon.
This is how it will look -

for password manager this is how it will look like  -

and this is how other websites have handled it -
I think we can leave it as it is.
Wordpress

LinkedIn

Looks good for upwork, applying External label

Triggered auto assignment to @jboniface (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

job posted: https://www.upwork.com/jobs/~016e11e755669a7e5c

Triggered auto assignment to Contributor-plus team member for initial proposal review - @rushatgabhane (Exported)

Triggered auto assignment to @Luke9389 (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@thesahindia I just wanna confirm if your proposal works on mobile too?

The images from this proposal with overlapping symbols look a little janky right? Not sure if I'm down for this route.

This isn't a serious proposal, as it is platform specific and may have undesirable performance tradeoffs, but to work around the password manager space conflict issue described in @thesahindia's proposal, I found that updating BaseTextInput to  poll for the :-webkit-autofill selector worked well in my test environment for webkit browsers specifically. I have yet to get npm run ios to work on my computer so I couldn't tell you if it works on the iPhone.
Given the hacky nature of the code below, I'd recommend either of the following: (1) accept the password manager icon overlap/space conflict (as described/recommended above), or (2) simply live with the original bug until there's some other better, natively supported solution found, perhaps involving more CSS control of the :autofill pseudo-element and fancy use of the :has pseudo-class.
Anyhow, I'm copying the investigative work I did here so posterity sees that another approach was attempted even if it's not the ideal solution.
componentDidMount() {
    // In order to maintain a consistent background color when autofill is
    // active, we poll for the -webkit-autofill selector and update the
    // component's background accordingly.
    //
    // See https://github.com/Expensify/App/issues/7016
    this.pollingForAutofill = true;
    const autofillCheck = () => {
        if (!this.pollingForAutofill) {
            return;
        }
        if (this.input.matches(':-webkit-autofill')) {
            const bg = getComputedStyle(this.input, '-webkit-autofill')['background-color'];
            this.setState({autofillBackgroundOverride: bg});
        } else {
            this.setState({autofillBackgroundOverride: null});
        }
        requestAnimationFrame(autofillCheck);
    };
    requestAnimationFrame(autofillCheck);

    // We are manually managing focus to prevent this issue: https://github.com/Expensify/App/issues/4514
    if (!this.props.autoFocus || !this.input) {
        return;
    }
    this.input.focus();
}

// [...]

componentWillUnmount() {
    this.pollingForAutofill = false;
}

// [...]

/* style for TextInput's view wrapper */
style={[
    styles.textInputContainer,
    this.state.isFocused && styles.borderColorFocus,
    (this.props.hasError || this.props.errorText) && styles.borderColorDanger,
    ...(
        this.state.autofillBackgroundOverride
            ? [{'backgroundColor': this.state.autofillBackgroundOverride}]
            : []
    ),
]}

There's a great discussion about the password manager / eye overlap issue in this WordPress issue: https://core.trac.wordpress.org/ticket/48222
A workaround proposed in that thread was to place the eye icon underneath the input, right aligned, along with the text "Show password" or "Hide password" depending on state.

Thanks for your input @textninja, I'm not sure what to do here. Discussing on slack

Ignore this bug or
Eye icon under the input.

The images from this proposal with overlapping symbols look a little janky right? Not sure if I'm down for this route.

I agree, overlapping kinda looks janky. Let's not do that.

How about tweaking the design a little:

@shawnborton

I'm a fan of this design change @anthony-hull, very clever.

thanks @Luke9389!
I propose for this approach to add left padding to the Pressable so that the touchable area increases with the visual area:
https://github.com/Expensify/App/blob/7a4354b3879b5ca12e2d2394b2404074358f3389/src/components/TextInput/BaseTextInput.js#L196-L207
And adding the border radius overrides and right border to TextInputWithName when secureTextEntry is true:
https://github.com/Expensify/App/blob/7a4354b3879b5ca12e2d2394b2404074358f3389/src/components/TextInput/BaseTextInput.js#L176-L195
I would likely extract the styles conditional secureTextEntry styles for TextInputWithName into a function as the style array is getting quite long and the cognitive load of reading it is increasing.

@anthony-hull Awesome
Increase left padding. Add border when secureTextEntry is true. Sounds good.

I would likely extract the styles conditional secureTextEntry styles for TextInputWithName into a function as the style array is getting quite long and the cognitive load of reading it is increasing.

I don't think that's necessary. The function will be used only once. You could new line them I guess.

🎀👀🎀 C+ reviewed.
@Luke9389
Visual issue and @anthony-hull's design is very clean. Let's go with that.

Sorry for the late feedback - I don't think I like the small border. Can we go back to where we had it just floating in the input before? This way it will look consistent with other inputs like dropdowns or calendar pickers. Thanks!

There are downsides to it floating in the input. If you scroll up, that implementation will cause overlapping icons from password managers.
I just looked at TextNingas link and that thread has a great spread of the different options and how they might look:

There's a great discussion about the password manager / eye overlap issue in this WordPress issue: https://core.trac.wordpress.org/ticket/48222

Got it, then I'd say you could keep your current implementation but just remove the border?

Ok, you've got the 🟢 green light 🟢 @anthony-hull. Thanks for the proposal, and thanks to @rushatgabhane for being the acting Contributor+ on this issue. I'll see you both on the PR 👍

This is without the border

Should have the linked PR finished tomorrow

Thanks!

it's not overdue

Not overdue

You can use the reviewing label to keep Melvin from applying Overdue (as long as reviews are actually happening 😄 )

@jboniface Applied! Let's close the issue


v2022x refresh2, Cliking on the Dynamic Session in the VE Tree does not result in a transition

When a dynamic session is created in Cameo V2022’s MDK, it appears on the Tree on the left side of the View editor, but clicking on it does not navigate to the page.
Environment:
MDK version: 6.1.0
VE version: 5.0.0
MMS Version: 4.0.20
MagicDraw or Cameo Systems Modeler Version: v2022x refresh2
Operating system: [e.g., Windows 10, macOS Mojave] Windows 11

Hello,
I have found a solution regarding this issue and would like to share it.
It appears that the method of obtaining the ID during page transitions was different.
I made the following modifications, built it myself, and confirmed that it works. However, I have not been able to check all the impacted areas, so could you please review it?
Modified file:
src/ve-app/pane-left/left-pane.component.ts
Line 437
before modification:
const viewId = branch.type !== 'view' ? branch.viewId : branch.data.id;
after modification:
let viewId = '';
if (branch.type === 'view' || branch.type === 'section') {
viewId = branch.data.id;
} else {
viewId = branch.view.id;
}

Thank you for the correction.


Environment
Docker
Version
0.10.7
Describe the problem
All labels, tooltips, headings, descriptions, etc... are missing.
This is my first time installing it, I used the exact docker-compose from: "https://homarr.dev/docs/introduction/installation" in portainer.
Below are screenshots of what it looks like.

Additional info
No response
Please tick the boxes

[X] You've read the docs
[X] You've checked for duplicate issues
[X] You've tried to debug yourself

Hi @zwollner ,
I'm sorry to hear that you're having issues with Homarr.
Did you mount the exact same paths as in the installation guide?

yes I did.
I'm not sure what happened, I completely destroyed and recreated it twice with the same result.
but I just tried again a 3rd time and now it seems fine, so I'm not sure what was going on.

No worries! We've had this happen to quite a few users and it seems to occur randomly.
I hope you enjoy Homarr otherwise :)


Handling Spaces and Line Breakers in HTML input

When I pass the following HTML:

Explore the Wonders of the Ocean

The ocean is a vast and mysterious place, teeming with life and beauty. From the smallest plankton to the largest whales, the marine ecosystem is incredibly diverse.

It converts it to an invalid Contentful rich text and creates blocks with node type text as root because of \n characters. it return error when you pass it to Contentful management API.

I solved it by removing all the spaces and line breakers between HTML tags, at the start and end of my input and I believe library should have an option to choose between ignoring spaces and line breakers(by minifying HTML) or converting them to valid
 tags behaviour.
Even if it's not needed to have an extra option, at least solve it by implementing one of options expressed above and then pass it to converter.
Steps to Reproduce:

Pass the above HTML to the converter.
Observe the resulting Contentful rich text structure.
Expected Behaviour: The HTML should be converted to a valid Contentful rich text structure without creating blocks with node type text as root. when I pass this to Contentful API, it returns the error which is invalid block type because of text nodeType.

Actual Behavior: The HTML is converted to an invalid Contentful rich text structure with blocks having node type text as root due to \n characters. when I pass it Contentful management API, it return error said the block with node type of text is invalid.
Suggested Solution: Add an option to ignore or convert spaces and line breakers to something works or minify the HTML before converting it to the Contentful rich text data structure.

Good catch, and thanks for the detailed issue, @amirhbakan!
If I'm catching the issue correctly here the problem is just the text-nodes at the root of the document. Which means any text or whitespace at root of the original HTML-input would result in the same problem? E.g.:

Heading

Text outside an element

Text inside an element

My understading is that we can't turn all \n characters into
, because newlines are a valid part of text inside an HTML-document, and does not necesseraly serve the same purpose as
. If that behavior is intended by a user of the library, it should be possible to do it with a custom converter function.
Since the problem we're dealing with is text and whitespace that is at the root of an HTML-document (outside any HTML-tag), we're actually dealing with invalid HTML? But since we're dealing with what might be partial HTML-snippets, we'll have to handle it anyway.
It would be possible to just add an option to ingoreWhitespaceAtDocumentRoot or something like that. But it seems like a very specific solution. Could also just do that by default without an option, since the resulting document will be invalid anyway. That would however not solve the problem with other text nodes at the document root.
I'll have to think about a solution a bit, thanks for the input!

Ok, since we're specifically dealing with Contentful here, maybe we could just wrap all text nodes at the root level of the document in a paragraph block?
If we reverse the situation, the Contentful Rich Text editor would never let you write text-nodes at the root level, they'll always be wrapped in a paragraph-node.

Hello, @amirhbakan!
It has taken some time to get back to this. I've implemented a few changes that I hope will solve this problem, they are currently available in #242. Do you think those changes will solve your problems?

:tada: This issue has been resolved in version 1.6.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Bookmarks and folders directly in 'Other bookmarks' are not being sorted despite 'Sort Other Bookmarks' setting

Hi Dev,
I just recently came across a following issue that bookmarks and folders located directly in 'Other bookmarks' are not being sorted despite 'Sort Other Bookmarks' setting is turned on. However, 'Sort Other Bookmarks Sub Folders' setting works still.

Observed on Chrome version 76.0.3809.132 (Official Build) (64-bit). Kindly let me know if you need any additional information.
Best regards,
Kaedin

Hi Kaedin,
Thanks for the screenshots. That definitely looks like something is wrong.
Can you tell me what OS and version of OS you are running on?
I'm using the same Chrome version on macOS Mojave and when I drag a root level bookmark in my "Other Bookmarks" folder it closes the bookmark popup and then when I open "Other Bookmarks" again, everything is sorted as expected. I'm guessing if you do the same procedure by moving a bookmark under "Other Bookmarks", that bookmark just stays wherever you last left it?

Hi @nightmode,
Actually there is no issue after all! Maybe I was in a need of my morning coffee to get my head working...
Somehow, I had placed all the folders and bookmarks, which usually resided in 'Other bookmarks', actually into the 'Boomarks bar' and thought I was browsing 'Other bookmarks' instead through the expansion button.
Sorry for confusion and false alarm here!

A happy ending so no worries here. ^_^
❤ ☕


Pandas support for Delta Lake

Hi Delta Lake community, I had asked the Pandas team to investigate adding Delta Lake support since I bounce between Pandas and Spark, and they said they wouldn't support it since it looks like Delta Lake seems tied to Spark for its APIs (see pandas issue 35017). Is this true?

The only stable public APIs, currently provided by Delta Lake, are through the DataFrameReader/Writer (i.e. spark.read, df.write, spark.readStream and df.writeStream). Options to these APIs will remain stable within a major release of Delta Lake (e.g., 1.x.x).
All other interfaces in this library are considered internal, and they are subject to change across minor/patch releases.

I personally would love to start using Delta Lake more extensively in my machine learning projects, but I'd be nervous if I had to give up on being able to read it in to my laptop for exploratory data analysis. Thanks for considering!

@jackwellsxyz Can you use koalas? https://koalas.readthedocs.io/en/latest/reference/api/databricks.koalas.read_delta.html

Thanks @brkyvz for the feedback! Ultimately, I'm afraid that if I store data in Delta Lake format, then I tie myself to only analyzing it using Spark, as koalas only runs on Spark if I'm not mistaken. It would be nice to be able to open the data in pandas in a Jupyter notebook on my local laptop as well. Are there any architectural limitations of the Delta Lake file format that would prevent it from being opened on a local machine with pandas? For example, parquet and json formats are both well supported in pandas via the Apache Arrow and fastparquet libraries.

I agree. It makes no sense to bind Delta to Spark. Delta is a storage format while Spark is an execution engine. Modern data management tend to separate storage from compute (unlike to old days of Hadoop). Exposing Delta to Pandas will open the doors to countless use cases.

I agree with @manucohenyashar! In the company I work for, we need to use the delta formats on the gold data layer to enable us keep versioning and continue improving our pipeline.

Is there an update on this?

For those who need access to delta lake natively outside of JVM, i recommend giving https://github.com/delta-io/delta-rs a try. Its python binding already supports reading delta tables using pandas, see examples at https://github.com/delta-io/delta-rs/tree/main/python#usage.
I think it's best to leave the official reference implementation depend on spark because that's the only way to process large scale delta tables.

Pandas support already parquet. Pandas contributors would implement delta lake if databricks creates the same pyarrow connector as for parquet. Can databricks help?
Below is pandas contributor reply:
https://github.com/pandas-dev/pandas/issues/40573

@Sutyke see https://github.com/delta-io/delta-rs/tree/main/python#usage, we already support processing delta tables with native pandas.

The idea is not only to read delta tables to a data frame but also to
persist a data frame to a delta table.
I don't see that supported today.
In short, it should be possible to work with delta tables without using
Spark anywhere in the pipeline.
On Thu, Mar 25, 2021 at 2:34 AM QP Hou @.***> wrote:

@Sutyke https://github.com/Sutyke see
https://github.com/delta-io/delta-rs/tree/main/python#usage, we already
support processing delta tables with native pandas.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/delta-io/delta/issues/467#issuecomment-806401006, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABULBRY7IM64N7UINXD7ADDTFLKPFANCNFSM4OK2ZJAA
.

@manucohenyashar that's our medium term goal, we are will be using delta-rs in https://github.com/delta-io/kafka-delta-ingest to perform delta table write for streaming input, then process them in spark downstream. Delta-rs currently already have append delta table write working in a PoC, we are working on making it production ready.

any update on this,  is python now able to create Delta table without spark ?

@djouallah yes, see https://delta-io.github.io/delta-rs/python/

The question is if it is possible to *CREATE *a new delta table *without *using
spark in python?
Manu
On Thu, May 27, 2021 at 11:35 PM QP Hou @.***> wrote:

@djouallah https://github.com/djouallah yes, see
https://delta-io.github.io/delta-rs/python/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/delta-io/delta/issues/467#issuecomment-850086537, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABULBR75IOINAAHRTO2P2JLTP4FPLANCNFSM4OK2ZJAA
.

@MrPowers thanks, I think the issue is obviously still not addressed, need to write delta without spark using only python.

@houqp is this the file you envision the API exposure being handled in? Or should a new file be created for this purpose? https://github.com/delta-io/delta/blob/master/python/delta/tables.py

Closing this. https://github.com/delta-io/delta-rs/pull/294 has added an API to load a Delta table as a pandas dataframe.

@zsxwing  sorry the feature request is how to write a delta table from Python  without Spark? is this supported

@djouallah this is being tracked in https://github.com/delta-io/delta-rs/issues/542 and @wjones127 is driving it.


path flag of generator not working

Describe the bug
whatever i write into the path flag for the jet generator has no influence on the destination path. When using the example path ./gen, the resulting files still just get generated into my workspace root with this structure: "DBName/schemaName/..."
there is no .gen folder created. When i create that first or try a different naming, still it is not copyied inside there nor a folder is created. it just cares about the database name and schema name for the path of generation.
Environment (please complete the following information):

OS: Windows 11
Database: postgres

What's the full command you ran?

Correction: it seems on windows/vscode terminal the "./" infront of the .gen folder is the problem. when using . path the entire path is not used.
jet -dsn=postgresql://postgres:postgres@localhost:5432/webDev?sslmode=disable -schema=public -path=./test //broken
jet -dsn=postgresql://postgres:postgres@localhost:5432/webDev?sslmode=disable -schema=public -path=test //working

That's interesting, the relative path with ./ should work. I'll check on a windows machine.

Yes, tested with powershell, also omited relative path. The debug output of the generator also completly omits the relative path:
'Destination directory: webDev/public' -> missing the "test" folder when provided as relative path.
having non relative path is interpreted as relative path though, meaning the ./ is unnecessary

The debug output of the generator also completly omits the relative path

That's when the path is ./test  right?

having non relative path is interpreted as relative path though, meaning the ./ is unnecessary

I agree, but then again path handling in Windows is a bit different.
Can you please try with this command and let me know?
jet -dsn=postgresql://postgres:postgres@localhost:5432/webDev?sslmode=disable -schema=public -path=.\test

That's when the path is ./test right?
yes, if the path is relative it seems to get completly ignored and omited.

Can you please try with this command and let me know?
jet -dsn=postgresql://postgres:postgres@localhost:5432/webDev?sslmode=disable -schema=public -path=.\test

same behavior, path omited

Interesting, .\test on Windows should have worked per my understanding.
I'll look into it.

Confirmed relative names starting with ./ and .\ don't work.

This is related to how PowerShell parses arguments.
The value of the path flag is empty in Jet when it's passed like this,
jet.exe -source=sqlite -dsn="main.db" -path=.\dest\dir

See this SO question for more context.
How to pass arguments:
When using PowerShell, the arguments should be quoted like so,
jet.exe -source="sqlite" -dsn="main.db" -path=".\dest\dir"

It'll work as expected then.

cmd doesn't have any problems either way, but in Windows, I'd err on the side of caution and use quoted strings as arguments anyway.


applying pre k8s 1.2.1 bundles to 1.21+ clusters

When applying CRDs from unpacked bundles, rukpak doesn't convert the CRD version to match apiVersion supported by the underlying cluster.
If a bundle image has a CRD with {apiVersion: [apiextensions.k8s.io/v1beta1], kind: CustomResourceDefinition} and the cluster supports the apiVersion {apiVersion: [apiextensions.k8s.io/v1], kind: CustomResourceDefinition}, rukpak tries to apply the CRDs directly, failing with no matches for kind \"CustomResourceDefinition\" in version \"[apiextensions.k8s.io/v1beta1]\"\nensure CRDs are installed first
some affected bundles:
"quay.io/openshift-community-operators/kiali@sha256:cb01816226cc731c8c35f6d0643447468dfa3db4bb6132320e9dc2bba1ce5c66"
"quay.io/openshift-community-operators/prometheus@sha256:6fbd3eaa123054c5023323d1f9ab7cbea178087fcb7cb4f3e83872c6a88d39a1"
"quay.io/openshift-community-operators/redis-operator@sha256:b8ab4fcdbef94394b6126857e40eecbadf700da0b123ced37af5902471ecae8e"

I might be able to be persuaded, but I think my stance is that Rukpak, in general, should not do any "magic" like this. If a bundle contains an object that is not supported by the API server, I'd expect a failed installation and a message in the BD status that says what object/gvk could not be applied due to api not existing on the cluster.


List active sessions using the API

Hello! I'm filling this issue as per my conversation with Mark Mandel on Discord.
Is your feature request related to a problem? Please describe.
I am hosting a game that gets exposed to the Internet by Quilkin using the standard sidecar pattern on a generic k8s, with no Agones involved (yet?).
I happen to need to know the real client IP specific players use to connect to the game at a given moment.
This is far from easy at the moment since the game server only sees Quilkin's own IP. If I'm not mistaken, this IP is always <IP_ADDRESS> (due to the sidecar) for each players, but client port differs for each session.
Describe the solution you'd like
A /sessions API endpoint of sorts that would return currently active sessions with i.e.:

The real Client IP & Client Port, which in my case would be the player's to Quilkin
The client port (and possibly client IP?) Quilkin allocates to that session and uses to connect to the endpoint (game in my case)
would let me map the latter to the former directly from the game.
The endpoint's IP and port Quilkin is proxying the client to (I wouldn't have much use for this myself having only one endpoint configured, but I suppose this could be useful otherwise)

I am not quite sure which format to use to return data would be optimal, but a simple JSON would likely work perfectly for me.
That endpoint may also be useful to troubleshoot specific sessions, and maybe provide additional session-specific data (i.e. lifetime, time since last packet transited or other metrics Quilkin might collect in the background).
Describe alternatives you've considered
N/A, though I am very much open to other solutions!
Additional context
Just in case, my current Quilkin config is as follow:
apiVersion: v1
kind: ConfigMap
metadata:
  name: quilkin-sidecar-config
  labels:
    app: game
data:
  quilkin.yaml: |
    version: v1alpha1
    admin:
      address: "[::]:9091"
    id: game-proxy
    port: 8888
    clusters:
      default:
        localities:
          - endpoints:
              - address: <IP_ADDRESS>:7777

And the container declaration in my kube manifest, alongside the game's container:
[...]
      - name: quilkin
        image: us-docker.pkg.dev/quilkin/release/quilkin:0.5.0
        args: ["proxy"]
        ports:
        - name: proxy
          containerPort: 8888
          hostPort: 7777
          protocol: UDP
        livenessProbe:
          httpGet:
            path: /live
            port: 9091
          initialDelaySeconds: 3
          periodSeconds: 2
        resources:
          requests:
            memory: "128M"
            cpu: "0.25"
          limits:
            memory: "512M"
            cpu: "1"
        volumeMounts:
          - mountPath: /etc/quilkin/quilkin.yaml
            subPath: quilkin.yaml
            name: quilkin-config
            readOnly: true
[...]

Some more context is available in the Discord conversation linked at the top, but I am of course ready to provide more details if necessary :)

Thank you for your issue!
This where I scope creep this feature a little bit and ask, instead of having dedicated endpoints for each kind of data users are interested in querying, what if we instead had a GraphQL endpoint, and we merged this with /config? That would also be more useful for your use-case where you could subscribe to sessions, and then you'd receive session data as it becomes available in Quilkin, rather than needing to repeatedly poll and search for new information yourself.
What do you think @markmandel ?

I knew I was going to have to learn GraphQL one day 😄  runs off to read docs
Yeah, this makes a lot of sense, and it's expandable over time. We can put config info in here, session info, and potentially more -- and it can change depending on mode, since the endpoint has built in discover ability. Yeah, count me as sold.
I do wonder if /config  is the right endpoint name (/graphql ?), but that's minor detail.
@XAMPPRocky got a favourite Rust + GraphQL library?


fix(gtfs-lib): bump gtfs-lib for various bug fixes

Checklist

[x] Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
[x] Any modified or new methods or classes have helpful JavaDoc and code is thoroughly commented
[x] The description lists all applicable issues this PR seeks to resolve
[x] The description lists any configuration setting(s) that differ from the default settings
[ ] All tests and CI builds passing

Description
This PR bumps gtfs-lib to 4.2.5 for various bug fixes (described in its release notes).

Codecov Report

Merging #177 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##               dev    #177   +/-   ##
=======================================
  Coverage     5.15%   5.15%
  Complexity      94      94
=======================================
  Files          133     133
  Lines         6929    6929
  Branches       907     907
=======================================
  Hits           357     357
  Misses        6544    6544
  Partials        28      28

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 32c2f9d...dd6e905. Read the comment docs.

:tada: This PR is included in version 3.3.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Would you provider a demo by use js?

Or provider a ui-router-ng2.js match angular2 js version.

Are you looking for an ES5 version of the quickstart?


Include ioggstream's proposals into the new draft

@ioggstream made a few proposals for the docs, we should include them in the new draft both here and in the cookie-cutter repo: https://github.com/teamdigitale/dati-semantic-guida-ndc-docs/pull/15#pullrequestreview-1752392203

@AgostinoPurificato could you check if the proposals are in the new guide?

@Clou-dia @bfabio
The suggested changes no longer apply to the cookie-cutter repository since with https://github.com/teamdigitale/dati-semantic-cookiecutter/pull/19 we removed files that explained things already described in the new guide, so we replaced some of the files with links to the guide.
In any case, I will incorporate the suggestions where applicable in the new guide.


JS cleanup

Remove unused variable
Replace one-liner if statements
Use const where possible
Output module usage to console in more cases

This change is

Too much redundancy with #7


firebase-admin does not work

Main problem 😿
the deployment occurs without problems, however I can not connect to my database. I did an example with a node script and I was able to connect to firestore
I do not know why 'now' has problems with that module.

How to reproduce the error 🐛 ?
Step 1

Open the Firebase  console and create a new project

In the Database section, click on Test Firestore

Click on enable

Click on Generate new private key

Step 2

Create new project mkdir example-test

Create an index.js file

yarn init -y

Add the following dependency to your project: yarn add firebase-admin

Step 3
Place the following code in the file index.js

const admin = require('firebase-admin');

const serviceAccount = require('./serviceAccountKey.json');

admin.initializeApp({
  credential: admin.credential.cert(serviceAccount)
});

const db = admin.firestore();

const data = {
  name: 'Mark',
  lastName: 'Drew'
  age: 21
}

db.collection('person').doc('Mark').set(data);

run the following command: DEBUG=*fun* ~/Downloads/now-macos dev --debug --port 8080
package.json
{
  "name": "db-serverless",
  "version": "1.0.0",
  "main": "index.js",
  "license": "MIT",
  "dependencies": {
    "firebase-admin": "^7.0.0"
  }
}

@MAPESO We need a little more information to determine if this is a bug in ncc.

What error do you get specifically?
Can you provide the code from index.js?
Have you tried changing the firewall settings in your database to allow connections from any IP address? It's possible firebase is blocking Now deployments.

Hi, @styfle I just added detailed instructions to reproduce this problem ^-^

Have you tried changing the firewall settings in your database to allow connections from any IP address? It's possible firebase is blocking Now deployments.

I have not seen any option similar to that :/

I just went ahead and set up all the Firebase details here and ran the following index.js with npm install -g ncc && ncc run index.js:
const admin = require('firebase-admin');

const serviceAccount = require('./serviceAccountKey.json');

admin.initializeApp({
  credential: admin.credential.cert(serviceAccount)
});

const db = admin.firestore();

const data = {
  name: 'Mark',
  lastName: 'Drew'
  age: 21
}

db.collection('person').doc('Mark').set(data);

I then get the following output:
ncc: Module directory "/home/guybedford/Projects/ncc/node_modules/hash-stream-validation" attempted to require "fast-crc32c" but could not be resolved, assuming external.
ncc: Module directory "/home/guybedford/Projects/ncc/node_modules/bytebuffer/dist" attempted to require "memcpy" but could not be resolved, assuming external.
    1kB  protos/google/api/annotations.proto
    1kB  src/load.d.ts
    2kB  protos/update.sh
    2kB  src/index.js.map
    2kB  src/load.js.map
    2kB  src/index.d.ts
    2kB  protos/google/protobuf/empty.proto
    3kB  protos/google/firestore/v1beta1/common.proto
    3kB  protos/google/type/latlng.proto
    4kB  protos/google/rpc/status.proto
    4kB  protos/google/protobuf/struct.proto
    4kB  protos/google/protobuf/wrappers.proto
    5kB  protos/google/firestore/v1beta1/document.proto
    6kB  protos/google/protobuf/timestamp.proto
    6kB  protos/google/protobuf/any.proto
    6kB  protos/google/firestore/v1beta1/query.proto
    8kB  protos/google/firestore/v1beta1/write.proto
   15kB  protos/google/api/http.proto
   27kB  protos/google/firestore/v1beta1/firestore.proto
  121kB  sourcemap-register.js
  128kB  protos/firestore_proto_api.d.ts
  260kB  roots.pem
 7959kB  index.js
 8976kB  index.js.map
12109kB  src/node/extension_binary/node-v64-linux-x64-glibc/grpc_node.node
20680kB  [7579ms] - ncc 0.15.2

The behavior for Date objects stored in Firestore is going to change
AND YOUR APP MAY BREAK.
To hide this warning and ensure your app does not break, you need to add the
following code to your app before calling any other Cloud Firestore methods:

  const firestore = new Firestore();
  const settings = {/* your settings... */ timestampsInSnapshots: true};
  firestore.settings(settings);

With this change, timestamps stored in Cloud Firestore will be read back as
Firebase Timestamp objects instead of as system Date objects. So you will also
need to update code expecting a Date to instead expect a Timestamp. For example:

  // Old:
  const date = snapshot.get('created_at');
  // New:
  const timestamp = snapshot.get('created_at');
  const date = timestamp.toDate();

Please audit all existing usages of Date when you enable the new behavior. In a
future release, the behavior will change to the new behavior, so if you do not
follow these steps, YOUR APP MAY BREAK.
(node:5964) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
(node:5964) UnhandledPromiseRejectionWarning: Error [ERR_UNHANDLED_ERROR]: Unhandled error. (/tmp/google/api)
    at EventEmitter.emit (events.js:178:17)
    at EventEmitter.<anonymous> (/node_modules/walkdir/walkdir.js:217:1)
    at Object.onceWrapper (events.js:277:13)
    at EventEmitter.emit (events.js:189:13)
    at fn (/node_modules/walkdir/walkdir.js:71:1)
    at statter (/node_modules/walkdir/walkdir.js:109:1)
    at walkdir (/node_modules/walkdir/walkdir.js:221:1)
    at Function.module.exports.walkdir.sync (/node_modules/walkdir/walkdir.js:15:1)
    at module.exports.COMMON_PROTO_DIRS.map.dir (/node_modules/google-gax/build/src/grpc.js:71:1)
    at Array.map (<anonymous>)
(node:5964) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:5964) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

The walkdir error above is definitely an ncc error, but odd that this seems different to the error previously posted.

@MAPESO I would suggest using npm install -g ncc and then ncc run index.js for testing further to ensure we are directly testing ncc bugs.

@guybedford  This is very strange, my output looks different from yours 😕
ncc: Module directory "/Users/mark/db-serverless/node_modules/hash-stream-validation" attempted to require "fast-crc32c" but could not be resolved, assuming external.
ncc: Module directory "/Users/mark/db-serverless/node_modules/node-fetch/lib" attempted to require "encoding" but could not be resolved, assuming external.
ncc: Module directory "/Users/mark/db-serverless/node_modules/write-file-atomic" attempted to require "worker_threads" but could not be resolved, assuming external.
ncc: Module directory "/Users/mark/db-serverless/node_modules/debug/src" attempted to require "supports-color" but could not be resolved, assuming external.
ncc: Module directory "/Users/mark/db-serverless/node_modules/bytebuffer/dist" attempted to require "memcpy" but could not be resolved, assuming external.
   1kB  dist/src/load.d.ts
   1kB  dist/protos/google/api/annotations.proto
   2kB  dist/src/load.js.map
   2kB  dist/src/index.d.ts
   2kB  dist/src/index.js.map
   2kB  dist/protos/update.sh
   2kB  dist/protos/google/protobuf/empty.proto
   3kB  dist/protos/google/firestore/v1/common.proto
   3kB  dist/protos/google/firestore/v1beta1/common.proto
   3kB  dist/protos/google/type/latlng.proto
   4kB  dist/protos/google/rpc/status.proto
   4kB  dist/protos/google/protobuf/struct.proto
   4kB  dist/protos/google/protobuf/wrappers.proto
   5kB  dist/protos/google/firestore/v1beta1/document.proto
   5kB  dist/protos/google/firestore/v1/document.proto
   6kB  dist/protos/google/firestore/v1/query.proto
   6kB  dist/protos/google/protobuf/timestamp.proto
   6kB  dist/protos/google/firestore/v1beta1/query.proto
   6kB  dist/protos/google/protobuf/any.proto
  10kB  dist/protos/google/firestore/v1beta1/write.proto
  10kB  dist/protos/google/firestore/v1/write.proto
  15kB  dist/protos/google/api/http.proto
  26kB  dist/protos/google/firestore/v1/firestore.proto
  27kB  dist/protos/google/firestore/v1beta1/firestore.proto
 127kB  dist/protos/firestore_proto_api.d.ts
 272kB  dist/roots.pem
7871kB  dist/index.js
8425kB  [52044ms] - ncc 0.15.2
(node:9061) UnhandledPromiseRejectionWarning: Error: db-serverless package.json is not node-pre-gyp ready:
package.json must declare these properties:
binary.module_name
binary.module_path
binary.host
    at Object.validate_config (/Users/mark/db-serverless/dist/index.js:147687:15)
    at Object.module.exports.exports.find (/Users/mark/db-serverless/dist/index.js:44259:15)
    at Object.module.exports.module.exports.interfaces.google.firestore.v1beta1.Firestore.retry_codes.idempotent (/Users/mark/db-serverless/dist/index.js:24492:12)
    at __webpack_require__ (/Users/mark/db-serverless/dist/index.js:22:30)
    at Object.module.exports.Object.setPrototypeOf.__proto__ (/Users/mark/db-serverless/dist/index.js:139493:12)
    at __webpack_require__ (/Users/mark/db-serverless/dist/index.js:22:30)
    at Object.module.exports.Object.defineProperty.value (/Users/mark/db-serverless/dist/index.js:174202:27)
    at __webpack_require__ (/Users/mark/db-serverless/dist/index.js:22:30)
    at Object.module.exports.module.exports.out (/Users/mark/db-serverless/dist/index.js:128527:14)
    at __webpack_require__ (/Users/mark/db-serverless/dist/index.js:22:30)
(node:9061) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:9061) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

btw The result of the code line 99153 is a firebase function
    function FirebaseNamespace() {
        // Hack to prevent Babel from modifying the object returned as the default admin namespace.
        /* tslint:disable:variable-name */
        this.__esModule = true;
        /* tslint:enable:variable-name */
        this.credential = firebaseCredential;
        this.SDK_VERSION = '7.0.0';
        /* tslint:disable */
        // TODO(jwenger): Database is the only consumer of firebase.Promise. We should update it to use
        // use the native Promise and then remove this.
        this.Promise = Promise;
        this.INTERNAL = new FirebaseNamespaceInternals(this);
    }

@guybedford

To hide this warning and ensure your app does not break, you need to add the
following code to your app before calling any other Cloud Firestore methods:

there must be some way in which ncc can solve that without us reaching that alternative. It is not practical for an average user to have to do that configuration before using firestore in now

@MAPESO thanks for providing the new details here. Unfortunately that's not quite the right source line as the debug output there has the corresponding line of the parent call to find() as db-serverless/dist/index.js:24492, not line 99153. Can you share that code here rather? You should be able to see an explicit .find() style call in that section and that is what we want to work out.
It is quite odd that we are getting different errors, I'm not sure why that is the case. If you have any ideas where our workflows might be differing here let me know too. And thanks for your help tracking this down!

there must be some way in which ncc can solve that without us reaching that alternative. It is not practical for an average user to have to do that configuration before using firestore in now

Note I get this error both with and without ncc - that is, it seems this is a Firestore-specific warning about the workflow and nothing to do with ncc as far as I can tell.

@MAPESO are you running on MacOS?

are you running on MacOS ?

Yes : ) these are the specifications of my computer

Note I get this error both with and without ncc - that is, it seems this is a Firestore-specific warning about the workflow and nothing to do with ncc as far as I can tell.

But firebase example work directly with node and express etc 🤔
Result of the 24492 code line
    binary.find(path.resolve(path.join(__dirname, '../package.json')));

If you have any ideas where our workflows might be differing here let me know too.

This is a clone of my repository, I hope that can help. Remember to use your firebase keys
repository

@guybedford we moved this error to #317  I offer a better explanation : )


tr_shader: add r_lightStyles cvar to disable light styles

#750 must be merged first, the target of this branch has to be changed to master once #750 is merged.
This allows to set r_lightStyles 0 to disable light styles.
Light styles are special effects that blends multiple light maps to create fake dynamic lights, like lights of blinking neons.
Every extra light map is a different material stage and can't be collapsed, meaning every extra light map require an extra render pass.
When extra light maps tiles are stored in a light map file not shared with any static light map tile, the option would even prevent the loading of those extra light map files from file system and not upload them to GPU, reducing both load time and memory usage.
The purpose of this option is to be enabled by default but disabled in low graphics presets.

Seems reasonable. LGTM


Could not find method classpath() for arguments

I added
buildscript {
    repositories {
        jcenter()
    }
    dependencies {
        classpath 'com.jfrog.bintray.gradle:gradle-bintray-plugin:1.0'
    }
}

to my build.gradle and when I ./gradlew build, I get
vtomole@vtomole:~/jQuil$ ./gradlew build

FAILURE: Build failed with an exception.

* Where:
Build file '/home/vtomole/jQuil/build.gradle' line: 23

* What went wrong:
A problem occurred evaluating root project 'jQuil'.
> Could not find method classpath() for arguments [com.jfrog.bintray.gradle:gradle-bintray-plugin:1.+] on object of type org.gradle.api.internal.artifacts.dsl.dependencies.DefaultDependencyHandler.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 0s
vtomole@vtomole:~/jQuil$ ./gradlew build

FAILURE: Build failed with an exception.

* Where:
Build file '/home/vtomole/jQuil/build.gradle' line: 23

* What went wrong:
A problem occurred evaluating root project 'jQuil'.
> Could not find method classpath() for arguments [com.jfrog.bintray.gradle:gradle-bintray-plugin:1.+] on object of type org.gradle.api.internal.artifacts.dsl.dependencies.DefaultDependencyHandler.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 1s

How do I fix this?
Thank you.

Instead of implementing my own, I just copied from here: https://github.com/bintray/bintray-examples


Installation issue with Ansys Python

pip install visualization is not working with Ansys Python because psutil cannot be compiled.
One workaround is to pass --system-site-packages while creating the virtual environment.

Hi, I have tried with --system-site-packages and got below error:
D:\Users\Consulting\2022\PyFluent\23R1_env>"C:\Program Files\ANSYS Inc\v231\commonfiles\CPython\3_7\winx64\Release\python\python.exe" -m venv --system-site-packages env
Access is denied.

Seeking more information on the nature of the dependency.


Bref\Console\Deployer not found

I can run bref init
but when running brief deploy:
PHP Fatal error:  Uncaught Error: Class 'Bref\Console\Deployer' not found in /home/ubuntu/test/demo/vendor/bin/bref:114
Any clues?
thanks in advance,

Hi, I'm sorry I have no idea, this is really surprising behavior. Did you install Bref in an empty directory?
Can you check that the class/file exists in the vendor directory?

thanks for replying, i started all over, and created it in a blank dir.
it works. thanks!
a few more questions,  i have a laravel application that can be created using composer,
is it better to:
a) composer create-project myown-laravel-app
then add cd into the new directory and do:
composer require mnapoli/bref
vendor/bin/bref init
or start blank, get bref working and then try to import the laravel app?

I would advise starting with a working Laravel app (and follow the documentation about Bref and Laravel). However it's always good to first deploy a simple lambda, without Laravel, just to get it working at first. Having small successes is important! :)


better configuration options for API base URLs

Current version of BinanceApiConfig does not allow customization of API base URL but only base domain which does not make sense. It is not possible to configure testnet API with that (see https://testnet.binance.vision). This pull request improves configuration options to fully customizable base URLs. I removed two methods (setBaseDomain, getBaseDomain) so it is not backwards compatible but fix should be very easy for every dev.

Love it!
Thanks!


Get champion basic info

Is your feature request related to a problem? Please describe.
The feature is not related to a problem but is anew addition to the project.
Describe the solution you'd like
when a request is made to /championName or /championId we should be able to return the champion win rate, pick rate ban rate and number of total games in ranked. hint: ranked games on SR have queueId= 420 in the database. those stats should also be position dependent
Describe alternatives you've considered
N/A
Additional context
N/A

I'll work on this.


Implement the Relayer v0 concurrency architecture

Summary
Implement ADR-005 (#326)

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

closed via #364


Default command for indexer subsystem

The default command for the indexer should

Take a single BooleanSupplier shouldLoad for if the intake has a note and the elevator is down
Stop when the sensor sees a note
Stop when shouldLoad is false

Closed with PR #39


Helmet Detection Scenario edge pod CrashLoopBackOff

images：docker pull kubeedge/sedna-example-joint-inference-helmet-detection-little:v0.3.0
the container of edge can not  running ,and there is no logs
env: k8s； 1.22.6
kubeedge：1.21.1

Hi,
One of the reasons could be the architecture that you are using is different from the architecture of the image built for.
kubeedge/sedna-example-joint-inference-helmet-detection-little:v0.3.0
This image is built for linux/amd64.
There is another image built for arm64. I suggest you try this if you are using devices with arm architecture.
poorunga/sedna-example-joint-inference-helmet-detection-little-arm64:v0.3.0


feat: format json

Not sure what happened to the Docker CD
cc @nschonni

@juzhiyuan I think that CI job is just generally broken right now, not to do with your change


Cycle protection

How about adding some cycle protection? Now effector with cycle blocks browser until max callstack exceeded: https://share.effector.dev/s1OCDGzu
For example, restrict max count of node calculation per tick. Smth like this:
graphite.frameCalculations++;
if (graphite.frameCalculations > MAX_FRAME_CALCULATIONS) {
    throw new Error("Max node calculations per frame exceeded. Probably, you have cyclic depenency. Please, investigate error callstack to find it.");
}
if (!graphite.resetTImeout) {
   graphite.resetTImeout = setTimeout(() => {
         graphite.frameCalculations = 0;
         clearTimeout(graphite.resetTImeout);
    });
}

Repl indeed has cycle protection, in case when your code stuck in infinite loop, after reloading page it will insert an error to beginning of the code

The main issue in your example is synchronous console rendering, which consumes a lot of resources, without console.info this example executed in milliseconds
https://share.effector.dev/9ZXy7FDz


Add transitive reference to PlatformAbstractions

Hi @ryanbrandenburg, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Why is this needed now?

This project references Microsoft.DotNet.ProjectModel, which recently lost PlatformAbstractions as a transitive dependency, but still needs it at build time.
@NTaylorMullen is that explanation any good?

Where is it used?

Actually, looks like this project doesn't use platform abstractions. Don't have to do this project :+1:

Ok then. Good catch @davidfowl.


Cannot control WB-800-IPVM-12

Hello!
I have a WB-800-IPVM-12 and I can't seem to be able to control it using this companion module. Since this device is not in the list, I have used "Custom" and set "Telnet" as the protocol and input the number of outlets (12). When I test a button, the log gives an error that implies it is still trying to use HTTP.
outlets: 12
error: Error executing action: Cannot read properties of undefined (reading 'get')

For what it's worth, I am able to send commands to this device using telnet otherwise. Was hoping to streamline my approach using this module! Thanks.

As a follow up, I have already created 24 buttons, each for on/off on the 12 outlets. If I change the connection type to WB-250-IPW-2 I am able to control all of the outlets as expected:
outlets: 12
setting protocol to telnet
outlets: 2
log in successful
debug: Sending: ?Firmware
debug: Sending: ?Model
debug: Sending: ?ServiceTag
debug: Sending: ?Hostname
debug: Sending: !OutletSet=1,ON
debug: Sending: !OutletSet=1,OFF
debug: Sending: !OutletSet=2,ON
debug: Sending: !OutletSet=2,OFF
debug: Sending: !OutletSet=3,ON
debug: Sending: !OutletSet=3,OFF
debug: Sending: !OutletSet=3,ON
debug: Sending: !OutletSet=12,ON
debug: Sending: !OutletSet=12,OFF

Though, I am unable to create new buttons from this point forward because a new action dropdown only shows two outlets.


[EN DateTimeV2] Support [duration]+starting+[datetime] pattern as date range or date time range

Support duration starting date time pattern as date range or date time range.

2 weeks starting date/datetime
2 days starting date/datetime

Please look into open comments next ?

Which comments?


Make CLI for this app

Hey,
I am developing my own CLI for image processing right now and I thing it can be good idea to make one CLI for this project too. I have basic knowledge of click which is library for creating theese. It can be good if user want to know information quickly in cmd.

It's not clear to me what you're proposing. Please provide more details of how this CLI would be used.
The script already does have command line options, as documented here:
https://github.com/per1234/inoliblist#script-command-line-options

I mean addigg it to pip and add some kind of search option

I mean adding it to pip an
add some kind of search option

I mean adding it to pip

I really don't see the value in doing that. I don't foresee the script being of general use to people to the extent that they would want to be able to install it via pip. I am doubtful that anyone other than myself has ever run this script. I see the generated list I publish here as being the component that's useful to others (less so since I haven't updated it in a while). For the very few people who would want to run the script, I'm sure they wouldn't mind just downloading it from GitHub.
I'm open to being convinced otherwise if you can provide some strong arguments of why this is needed.

add some kind of search option

It's not clear to me what you're proposing. Please provide a detailed explanation of how this would work.
The front end of the list already does have a search feature (though it's certainly not ideal (see https://github.com/per1234/inoliblist/issues/1).

Okay I saw the issue now and I can look at that. My goal was to have a command line command which will take name of the library and search for that in the list from command line. There can be some colors too

It can have option to upliad code to github repo and automatically open issue for adding the library so end user can only work on cli and don't change to something else.

My goal was to have a command line command which will take name of the library and search for that in the list from command line.

The purpose of inoliblist.py is to generate the list. I wouldn't want any code added to that file that is not related to that purpose. That means it would need to be a separate search script. My initial thought is that if someone was going to the trouble to run the search script, they might as well just open the list in a spreadsheet program and use its search feature. Do you think the script would provide a benefit over that? If so, how?

It can have option to upliad code to github repo and automatically open issue for adding the library

That's not how this list works. It generates a list of Arduino libraries automatically by using the GitHub search API, as well as the list of libraries in Arduino Library Manager index. There is no mechanism for manually adding libraries to the list, nor would that be useful. I have documented the sources for the list:
https://github.com/per1234/inoliblist#list-sources
If someone wants their library on the list, they only need to be sure that it will be found in one of those sources.
Of course, they would also need to wait for me to get around to updating the list. My original plan was to generate the list as part of the continuous integration system, which would automatically be run weekly. I knew that the duration of the list generation greatly exceeded Travis CI's maximum job duration of 50 minutes. Circle CI has no maximum job duration, though it does have a maximum build duration of 5 hours. So I set up the periodic list generation using Circle CI:
https://github.com/per1234/inoliblist/commit/88ff89e98edc4d9cf9836470561652757ee08f27
However, as the script evolved, I found that Circle CI's 5 hour maximum build duration was not sufficient and so I could no longer use it automatically update the list:
https://github.com/per1234/inoliblist/commit/d3a1790e4a08465e305d6fb6af8a08f6176a2338
After that, I didn't pursue the automated list updates any further and fell back on updating them myself as I had time. I haven't had much indication that the list was getting a lot of use by other people, so I have only updated the list as I needed it for my own use, which is not very often. If I got the sense that people were using the list, I would make more of an effort to keep it updated regularly.

Ok. Thanks for response. Write me back when you will need some help.

I'd welcome help with any of the issues listed in the issue tracker:
https://github.com/per1234/inoliblist/issues
In addition, I would welcome suggestions for how I could set up automatic periodic updates of the list. You can see the necessary commands in the Circle CI build configuration here:
https://github.com/per1234/inoliblist/commit/88ff89e98edc4d9cf9836470561652757ee08f27


Use unique_ptr to manage visited_list_pool_

Using raw pointers is error-prone. For example, previous implementation would leak pool in case it's created and loadIndex is invoked.

Hi @ttsugriy, it looks like the CI is not passing. Can you please look into that?

Sure thing, @yurymalkov, it's due to a difference between macOS and Linux headers. Should be fixed now.

Thanks!


fix: fix legacy bolt get

Changes

#1193  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @MasterPtato and the rest of your teammates on  Graphite


Bind DirectionalLight buffer to uniform slot even when not used

This avoids a WEBGL error causes PanoramaSkys to fail to render. This is essentially the same fix as we used in https://github.com/godotengine/godot/pull/64341.
Fixes: https://github.com/godotengine/godot/issues/68164

Thanks!


LUA Error on some Vendors like "Danky" in my case

Actual Behavior
see LUA Error
Steps to Reproduce the Problem

find Danky
Open Vendor Window
Error

LUA Error:
13x !Vendorer\core.lua:1877: Usage: GetItemQualityColor(index)
[string "=[C]"]: in function `GetItemQualityColor'
[string "@!Vendorer\core.lua"]:1877: in function `UpdateMerchantInfo'
[string "@!Vendorer\core.lua"]:1801: in function <!Vendorer\core.lua:1801>
[string "=[C]"]: in function `MerchantFrame_UpdateMerchantInfo'
[string "@FrameXML\MerchantFrame.lua"]:138: in function <FrameXML\MerchantFrame.lua:131>
[string "=[C]"]: in function `MerchantFrame_Update'
[string "@!Vendorer\vendorfilter.lua"]:566: in function `UpdateMerchantItems'
[string "@!Vendorer\vendorfilter.lua"]:612: in function `RefreshFilter'
[string "@!Vendorer\vendorfilter.lua"]:572: in function `ResetFilter'
[string "@!Vendorer\core.lua"]:1728: in function `?'
[string "@Ace3\CallbackHandler-1.0\CallbackHandler-1.0-7.lua"]:119: in function <...Ons\Ace3\CallbackHandler-1.0\CallbackHandler-1.0.lua:119>
[string "=[C]"]: ?
[string "@Ace3\CallbackHandler-1.0\CallbackHandler-1.0-7.lua"]:29: in function <...Ons\Ace3\CallbackHandler-1.0\CallbackHandler-1.0.lua:25>
[string "@Ace3\CallbackHandler-1.0\CallbackHandler-1.0-7.lua"]:64: in function `Fire'
[string "@Ace3\AceEvent-3.0\AceEvent-3.0-4.lua"]:120: in function <Ace3\AceEvent-3.0\AceEvent-3.0.lua:119>

Locals:
(*temporary) = nil

already did the "proper" fix in my pull request but cause reasons you changed it and it throws an error again :D
here we go again, change:
return tonumber(currencyID), info;

to
return tonumber(currencyID), info.name, info.quantity, info.iconFileID, info.quantityEarnedThisWeek, info.maxWeeklyQuantity, info.maxQuantity, info.discovered, info.quality;

OR (if you don't like my fix) change ALL the functions where you call Addon:GetCurrencyInfo because the return value does not match the needed values!
for example:
C_CurrencyInfo.GetCurrencyInfo(515) while 515 = Darkmoon Prize Ticket), returns this:
[1]={
  canEarnPerWeek=false,
  quantityEarnedThisWeek=0,
  isHeaderExpanded=false,
  isTradeable=false,
  maxQuantity=0,
  iconFileID=134481,
  maxWeeklyQuantity=0,
  isHeader=false,
  name="Darkmoon Prize Ticket",
  isTypeUnused=false,
  isShowInBackpack=false,
  discovered=true,
  quantity=5,
  quality=1
}

which are 14 values
but your function MUST return this and MUST be this order:
name
quantity
iconFileID
quantityEarnedThisWeek
maxWeeklyQuantity
maxQuantity
discovered
quality

it's just 8 values  AND it is in a different order then C_CurrencyInfo.GetCurrencyInfo return!
so you can not just return info ^^
greetings, fuba

Hello, I created a PR addressing all these issues for DragonFlight! From what I tested, the addon is completely functional.
https://github.com/Sonaza/Vendorer/pull/32.
Feel free to use the code from my fork that is compatible with DF, making the addon functional again, until the author can approve it!
https://github.com/sebid/Vendorer/archive/refs/heads/master.zip


Can i get all polygons?

Hi.
I have a list of polygon descriptions and am painting them with shape directive. Something like this:
// in controller
vm.polys = [{
            "id": 40571,
            "name": "Via Garibaldi, Genoa, Italy",
            "path": [
              [44.412482, 8.930734], [44.410795, 8.929901], [44.409432, 8.935265],
              [44.410795, 8.935989], [44.412482, 8.930734]
            ],
            "area": 0.0000092841999999853514
          }, ...];

// in template
<shape
  ng-repeat="poly in ctrlGeo.polys track by poly.id"
  name="polygon"
  paths="{{poly.path}}"
  stroke-color="#FF0000"
  stroke-opacity="0.8"
  stroke-weight="2"
  fill-color="#FF0000"
  editable="{{poly.editable || false}}"
  on-click="ctrlGeo.onPolyClick(poly)"
  fill-opacity="0.35"></shape>

Thing is that some of polygons are overlapping and it's impossible to select some of them. For example, one big polygon overlaps some smaller ones and i can't select them.
I'm thinking to bind click event on map (instead of shape's on-click) and check how many polygons were hit (google.maps.geometry.poly.containsLocation will help). But thing is that i need to iterate over all polygons in the map (no more them 200) to check if each of them was hit. And i can't find the way to get this list of polygons.
Any ideas?

To click a polygon possible, I think zIndex options is for that.
https://developers.google.com/maps/documentation/javascript/3.exp/reference#RectangleOptions.

That helped. Thanks.


Not working in safari 12.1.1

i m getting this error while using this code.
error :- navigator.getUserMedia error:Error: Invalid constraint

Thanks for the heads-up. Which example are you getting this error for?

example url - https://simpl.info/mediarecorder/

Sadly, MediaRecorder is not supported by Safari on macOS or iOS:
https://caniuse.com/#search=mediarecorder
https://www.chromestatus.com/features/5929649028726784

do u know any other way to record video in safari

The WebKit bug for MediaRecorder is here, and you may want to take a look at the article here.
Note that MediaRecorder is now an Experimental Feature in Safari in both iOS 12.4 and iOS 13 public beta 4.
There are polyfills such as this one for audio, but I don't know of anything for video (see the WebKit bug for reasons why this is difficult).

Closing this issue, but reopen if you need to.


simplify build

[x] simply use tsc
[x] rm husky and lint-staged
[x] switch to just using NPM

Closed via https://github.com/rpearce/hex/pull/19


"days ago" shouldn't only consider the newest version

let's say we have a nuget package which has frequent releases e.g.:
1.0.0      12 days ago
1.0.1      10 days ago
1.0.2      7 days ago
1.0.3      5 day ago
1.0.4      3 day ago
1.0.5      2 day ago
1.0.5      1 day ago
...

our project has a reference to 1.0.0 and our nukeeper config has a 7 day threshold. the package will actually never get updated because the newest version will never be over the threshold. so with a 7 day threshold i would actually prefer if nukeeper would create a PR with version 1.0.2 so that we still move forward.

@OneCyrus We have solved this before by having different pipelines for packages with different update cadences. It's possible to use the include/exclude arguments with different age and change arguments.
Third party packages we would use something like minor 7d, but for internal packages perhaps major 0d.


HIVE-24481: Skipped compaction can cause data corruption with streaming

What changes were proposed in this pull request?
See the details in  HIVE-24481
Why are the changes needed?
Fix the data corruption issue.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Unit test

@deniskuzZ @klcopp can I ask for a review


Issues with CMOR checks for marine biogeochemistry

Hi,
I've encountered a few issues with CMOR checks while trying to set up a metric to evaluate Marine Biogeochemistry in CMIP5 models.
My current test case is looking at chlorophyll in the CanESM2 CMIP5 data.
/badc/cmip5/data/cmip5/output1/CCCma/CanESM2/historical/yr/ocnBgchem/Oyr/r1i1p1/latest/chl
The first issue is that BGC CMOR_tables don't accept the allvars short_name in fix_data or fix_metadata. The allvars short_name is used to apply a fix to data files before they're loaded by iris in the preprocessor. However, at the moment, the CMOR check fails because allvars doesn't appear in the CMOR table for CMIP5.
The second issue that I've encountered occurs when looking at ocean data with a depth component (ie 3D data). The CMOR check isn't able to find the generic_level in the cmip5 input data files. Could this be related with the absence of a the ocean annual mean field type? There is nothing like TO2Ys or TO3Y in the tab_fld_typ table on in the users guide.
Similarly, I'm struggling to find documentation on how to apply the extract_levels processor on ocean data. However, I think may be simpler to resolve once the generic_level issue is addressed.
Thanks for all your help with this, and I apologise if there are obvious solutions available.
Lee

I've figured out that the cause of the second issue listed above was that the CMIP5 model data all use depth as the name of the vertical dimension of the model, however the CMOR standard is olevel.
A temporary solution is to edit for CMIP5_Oyr dictionary in two places, by adding depth to the generic_levels at line 65.
As I'm looking at chlorophyll, I also need to change the olevel to depth in the dimensions section of the chl variable entry.
However, I must reiterate that changing the CMOR tables probably is not a permanent solution. (Hence why I hesitate to commit these changes.)

We should not modify the CMOR tables, if possible.
@jvegasbsc is there a more elegant solution?

The third issue I describe here was indeed easy to address once the generic_levels issue was resolved.
However, it did highlight that there was no way to extract a vertical level outside the min-max depth range.
In CMIP5 models, the ocean starts at the surface at a depth of 0m. However, the first grid cell centre is the start of the possible extract_levels range. The first grid cell is typically around 1m thick, so this means that it's not possible to extract the surface layer.
Fortunately, there was an easy fix, I added a new interpolation scheme  nearest_extrap where the extrapolation scheme used by stratify was set to 'nearest'. The changes are here:
https://github.com/ledm/ESMValTool/commit/69b4fa74219d42b72c55763d77cd9b81327f601c#diff-ac696e627fac6e38dec3934ffdd38010

Hi
First of all, sorry for the delay.

The first issue is that BGC CMOR_tables don't accept the allvars short_name in fix_data or fix_metadata. The allvars short_name is used to apply a fix to data files before they're loaded by iris in the preprocessor. However, at the moment, the CMOR check fails because allvars doesn't appear in the CMOR table for CMIP5.

The allvars is just a code for a fix that should be applied to all files in a given model. By the description of the error, I think you are putting allvars in the namelist and it's failing as it should. You just need to put your desired var and it will applly first the fixes for allvars and then any specific fixes if found

The second issue that I've encountered occurs when looking at ocean data with a depth component (ie 3D data). The CMOR check isn't able to find the generic_level in the cmip5 input data files. Could this be related with the absence of a the ocean annual mean field type? There is nothing like TO2Ys or TO3Y in the tab_fld_typ table on in the users guide.

I've figured out that the cause of the second issue listed above was that the CMIP5 model data all use depth as the name of the vertical dimension of the model, however the CMOR standard is olevel

A temporary solution is to edit for CMIP5_Oyr dictionary in two places, by adding depth to the generic_levels at line 65.

The only thing that we check in a generic level coordinate is that it has axis='Z'. You may need to add a fix for the affected variables to add this attribute to the depth coordinate. There is no need to touch the tables for that

Hi @jvegasbsc,
thanks for the reply. The parts about the allvars are very helpful. However, I'm finding the generic_level problem to be somewhat intractable.

The only thing that we check in a generic level coordinate is that it has axis='Z'. You may need to add a fix for the affected variables to add this attribute to the depth coordinate. There is no need to touch the tables for that

Unfortunately, I'm not finding this to be an adequate solution.
At your suggestion, I've added the following lines to the allvars call in the fix_metadata function:
        cube.coord('depth').attributes['Axis'] = 'Z'

        cube.coord('depth').attributes['generic_level'] = True

        cube.coord('depth').var_name = 'olevel'

I have also verified that these lines are active and are being called by the preproccessor, but I'm still seeing the same error message:
esmvaltool.cmor.check.CMORCheckError: There were errors in variable thetao:

generic_level: does not exist in olevel

This is the same problem that I had before. This error message occurs in line 237 of the cmor/check.py module: https://github.com/ESMValGroup/ESMValTool/blob/version2_development/esmvaltool/cmor/check.py
What happens is that the check.py tries to load the CMOR compliant 'olevel' field from the CMIP5 cube, using
        if self._cube.coords(axis, dim_coords=True):

            ....

where axis is olevel, from the CMOR table. This is statement evaluates to false, because there's not a olevel coordinate in the cmip5 cube. Iris is hardwired to load the Z component in the ocean under the CF-compliant standard_name depth, so the CMOR compliant standard_name olevel can't be found anywhere.
At first, I had thought that we could use fix_metadata to change the standard_name depth to olevel  using:
cube.coord('depth').standard_name = 'olevel'

But, this results in the error:
ValueError: 'olevel' is not a valid standard_name

Basically, we're hitting a point where the CF and CMOR standards are not compatible in iris. Any ideas?
Thanks for the help,
Lee

Can you try substituing this
        cube.coord('depth').attributes['Axis'] = 'Z'
        cube.coord('depth').attributes['generic_level'] = True

with this
        cube.coord('depth').axis = 'Z'

Unfortunately, this does not resolve the problem described above.

Can you try with version2_development branch? This should be fixed now

@ledm when you back from the beach, man, can you confirm this one works now pls?

I am guessing this has been solved since eons ago? @ledm

I think so, yeah.


Evaluate integration of fusty-flow inside flowjs to simplify shipping and reduce projects overhead

Given the semplicity of the fusty-flow mock against flow.js i would like to discuss the integration of fusty-flow directly inside flow.js.
The reason are the following:

the codebase of fusty-flow.js is really base with respect to the overhead of build/release for the developers / setup for the user that it introduces;
fusty-flow.js is currently missing any unit test; nor on travisCI nor on saucelabs.com; duplicating all this infrastructure would simply add overhead to the project; if merged inside the other libraries this overhead would be reduced.

What i'm proposing?

to merge https://github.com/flowjs/fusty-flow.js/blob/master/src/fusty-flow.js inside flow.js project;
to merge https://github.com/flowjs/fusty-flow.js/blob/master/src/fusty-flow-factory.js inside ng-flow project.

@AidasK if the plan sound fine for you i can take the time to do it.

For myself, I am not using fusty-flow anymore, because most of the browsers supports flow.js.
If we take statistics from my google analytics, only 7% of internet explorer users are using IE9,IE8,IE7... Other 93% are using IE10,IE11... And this number is rising.
Do you think it is worth supporting them? I can't event test if fusty-flow works on IE9.

I vote to ditch them :)


accuracy of test is very low

@zhulf0804 Hi, I trained according to your code, and the obtained weight file has a very good effect on the eval set, but the effect on the test set is very poor. Do you know the reason? thanks for your answer

Hello @Bennyliiiiii , I have not submitted test result on the official server before.
I tested several samples from test set. Some detection results are good, while others are with few false positive detection boxes.
Could you please post your result on test set here ?


Add support for older OCaml versions

The code uses syntax like let (let*) = Lwt.bind in that is supported OCaml 4.08.1 onward. Luckily dune has preprosessing support to enable this syntax with older compilers and we should use it so that we can support also older compiler versions.
More info: https://dune.readthedocs.io/en/stable/concepts.html?highlight=future_Syntax#future-syntax

This adds a dependency on https://github.com/ocaml-ppx/ocaml-syntax-shims though. Maybe you could get rid of let* in this package?

This package is small enough that we could get rid of the let* syntax. On the other hand, it improves readability of the code so it would be nice if we can keep it. What is the drawback of having the added dependency?

The added dependency is really small, it has no transitive dependencies. I am more concerned with the added compile step and the complexity it adds and the case when something doesn't work or doesn't work as expected.
The current use of let* could be easily replaced with Lwt.map without giving up too much of readability.

Note that colombe uses the syntax extension too - but for an other purpose than lwt. So, by transitivity, letters requires ocaml-syntax-shims.


Invalid Russian translation /ru/docs/Web/HTTP/Заголовки/If-Modified-Since

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details

The phrase

When used in combination with If-None-Match, it is ignored, unless the server doesn't support If-None-Match.

is translated into Russian as

При использовании в сочетании с If-None-Match заголовок If-Modified-Since игнорируется, если сервер не поддерживает If-None-Match.

which is incorrect and has the opposite meaning of what is written in English.
Suggested translation is

При использовании в сочетании с If-None-Match заголовок If-Modified-Since игнорируется, ~если~ кроме тех случаев, когда сервер не поддерживает If-None-Match.

@vladogol thanks for reporting this. Are you able and willing to log into MDN and fix Russian language issues yourself?
We don't have any dedicated Russian team members to handle such work, and I would happy if you just fixed such problems, rather than needing to report them here.

I've made the update


Add GitHubMergeRef, GitHubRepo, GitHubIssue, GitHubCommit, GitHubRevi…

Summary of Pull Request:
I ported more of the danger-js DSL to danger-swift.
I'm trying to keep the APIs similar but also suitable for Swift.
Added:

GitHubMergeRef
GitHubRepo
GitHubIssue
GitHubIssueLabel
GitHubCommit
GitHubReview
GitCommit
GitCommitAuthor

Renamed:

Results -> DangerResults

The only thing that's awkward right now is GitCommit doesn't have a sha or parents value when listed under the array of GitHubCommit objects. Therefore, both of these properties are optional and that's not ideal in the long run IMO.

It would also be useful if you had any more danger-js JSON from a PR with labels and reviews.

Messages

:book:

Just verifying this continues to work.

  Generated by :no_entry_sign: dangerJS

You can use the url https://peril-staging.herokuapp.com/api/v1/pr/dsl?owner=artsy&repo=eidolon&number=609 to generate any PR DSL you'd like :

The CI break is my fault - https://github.com/danger/danger-js/issues/357

Thank you for fixing that 💯


Lima upload

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

not in scope


phantomjs network monitoring response body undefined always

1. Which version of PhantomJS are you using? Tip: run phantomjs --version.
2.1.1
2. What steps will reproduce the problem?
1. monitoring network of website
3. Which operating system are you using?
mac el capitan && windows 10
4. Did you use binary PhantomJS or did you compile it from source?
binary

i use this code
var url = "http://google.com"
var page = require('webpage').create();
page.onResourceReceived = function(response) {
console.log('Receive ' + response.body);
};
page.open(url);
and its always response undefiend

onResourceReceived does not contain the body in the response:
http://phantomjs.org/api/webpage/handler/on-resource-received.html

@thoop how can i get then the data

You can't currently.
See:
https://github.com/ariya/phantomjs/issues/10158
https://github.com/ariya/phantomjs/pull/13922
https://github.com/ariya/phantomjs/issues/13908

sadly to see it's empty in response body in onResourceReceived and couldn't change it

this issue has been raised in 2011 https://github.com/ariya/phantomjs/issues/10158 and as the related feature has never worked, I am guessing it never will.


code.createBlueprint is not a function

Hello, I'm trying to work through the examples at https://polkadot.js.org/docs/api-contract/start/code/ and am getting the error "code.createBlueprint is not a function"
I'm using  "@polkadot/api-contract": "^4.13.1"
I'm using ink! and attempting to work with the incrementer example contract.
I am using an unmodified substrate canvas chain.
I can execute  const code = new CodePromise(api, abi, wasm);  without any error.
I'm using var abi = HERE DOC about 300 lines of JSON text from incrementer's metadata.json file and this appears to parse correctly.
I'm using var wasm = "0x... about 36K of hex" from incrementer.contract and this also appears to be working correctly.
If I search the source code for createBlueprint I see about 10 references to createBluePrintTx, but if I try to use this method instead, I get the same error.
Any clues greatly appreciated.  Thanks!

Docs need updating (I'm guessing here), the createBlueprint and other items have been deprecated for a very long time.

Closing, the deprecations tracked in #172


Thank you for your contribution

Hey,
Sorry to bother you, I am doing a research project that is highly related to graph dynamics systems. I found this repository is really useful for my project. If you need help writing the doc or translating it into another language, I will gladly help.
Also, I am kind of familiar with 3D models, and I can write some 3D interfaces to connect with this library.
Best
Lin

Hi Lin,
Glad you found this project useful!
Incidentally, it was work I also did during (undergraduate) research, mainly for deriving fluid-like dynamics on graphs. Before I had to switch gears to other work, my hope was to generalize the interface to allow easy expressibility of dynamics on graphs using Hodge theory (see this paper) and some machinery of the discrete exterior calculus (see M. Desbrun's paper).
Some of these "discrete" calculus analogs require extensions for graphs, since viewed as a simplicial complex, they are quite trivial -- for example, the Hodge Laplacian for incompressible vector fields on a graph without triangles vanishes. I started developing some of the formalism e.g. via the cycle basis, but more development & analysis would be needed there (see the advective derivative); non-uniqueness of graph decompositions is potentially an interesting question I was thinking about at the time.
On the interface side, I would gladly welcome any contributions, especially those improving performance, interactivity, and overall cleanliness of the code. I hacked the current "interface" together in Bokeh mostly to allow debugging & interacting with models in real-time, but I ran into limitations of the browser quickly (a PyQT/pyqtgraph app, as an example, would be so much more responsive.)
If you'd like to contribute on the interfacing, documentation, API, or formal aspects, I would be glad to collaborate! Feel free to post a gist/PR/issue and we can think about the best way to integrate it.
Cheers.
Anand


Delete Mode Broken State Glitch

The application can enter a state where delete cannot be used. Currently unknown how to reproduce. Copy still works.
https://github.com/RAIRLab/Peirce-My-Heart/assets/43485956/00fb3c10-df2f-42a8-8263-1362f9074a84

Discovered how to reproduce on master:
Draw something
Move in global move mode
can not delete


pylint to precommit

@kattni I think the new hook for pylint_tests is set up now. However it did flag several items in the existing test scripts.


Fix broken links & badges

Also clarified which nuget the version was mentioning in badge by adding an explicit label

Thanks!


Add random fuzzing of inputs

ideally abstract over the backend so we can use both of the below:
https://github.com/AltSysrq/proptest
https://github.com/BurntSushi/quickcheck
instead using just the function selector as calldata, detect if the function has args and generate random inputs for N iterations
https://github.com/gakonst/dapptools-rs/blob/master/dapp/src/runner.rs#L104

should take some pointers from the echidna fuzzing engine which keeps a malicious input db and has what they call "constant mining" (recycles state vars and return data as input)

oh and shrinking would be rly helpful too

I think PropTest already has shrinking https://github.com/AltSysrq/proptest/search?q=shrink, as implemented in #44

Can we keep open for adding advanced features like mutation or constant mining? Or should I crate a new issue

Yeah let's get more specific issues open for that, along with tags.


Automated cherry pick of #3824: fix resize disk

Cherry pick of #3824 on release/2.11.
#3824: fix resize disk

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@ccd628b). Click here to learn what that means.
The diff coverage is 52.63%.

@@              Coverage Diff               @@
##             release/2.11   #3827   +/-   ##
==============================================
  Coverage                ?   7.13%
==============================================
  Files                   ?     480
  Lines                   ?   77996
  Branches                ?       0
==============================================
  Hits                    ?    5567
  Misses                  ?   71962
  Partials                ?     467

Flag
Coverage Δ

#aFlag
7.13% <52.63%> (?)

Impacted Files
Coverage Δ

pkg/compute/guestdrivers/kvm.go
3.53% <0%> (ø)

pkg/compute/models/guests.go
0.63% <0%> (ø)

pkg/hostman/diskutils/nbd/nbdman.go
57.5% <83.33%> (ø)

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 ccd628b...215bfeb. Read the comment docs.


Fix DynamicResourcePool IllegalArgumentException

What changes are proposed in this pull request?

BlockWorkerClientPool supports closing multi times
DynamicResourcePool support releasing multiple times for the same resource

Why are the changes needed?
Fix https://github.com/Alluxio/alluxio/issues/15556
Does this PR introduce any user facing changes?
No

alluxio-bot, check this please

Can you try #15579 and see if it solves the error you were seeing?

Hi @secfree, thanks for the PR, but a resource is not supposed to be released more than once. When this IllegalArgumentException is thrown, it suggests there is a bug somewhere else.
In your case, can you provide more details on why you are seeing the error in #15556 ? What were you trying to do with Alluxio before that error occured?

Thanks for your reply.
The location is here: https://github.com/Alluxio/alluxio/blob/master/core/client/fs/src/main/java/alluxio/client/file/FileSystemContext.java#L574
    if (poolMap.containsKey(key)) {
      poolMap.get(key).release(client);

After the resource is GC, this poolMap still contains the key because GC does not remove the key from poolMap.

but a resource is not supposed to be released more than once

As BlockWorkerClient is Closeable, and a Closeable instance supports closing multiple times - "If the stream is already closed then invoking this method has no effect." Each close will trigger release one time. The code is located at
https://github.com/Alluxio/alluxio/blob/master/core/client/fs/src/main/java/alluxio/client/file/FileSystemContext.java#L534
If we want to avoid calling release multiple times, we may need to add a new field isClosed into BlockWorkerClient.  What do you think? @dbw9580

Can you try #15579 and see if it solves the error you were seeing?

I will try this and update you.

@secfree
If the proposed fix in #15579 does not work, then it is indeed the case that the closable resource was closed twice, not that the wrong key was computed when closing the resource.
Perhaps you can help us understand why the ClosableResource wrapping BlockWorkerClient was closed twice? To silence an error like this is easy, but to understand why there was an error in the first place is more important.

Perhaps you can help us understand why the ClosableResource wrapping BlockWorkerClient was closed twice?

Only one business team was testing and they failed to reproduce the issue when we request them for full log.
For ClosableResource, as it support closing multiple times from the definition, so we should fix this part, right?
public interface Closeable extends AutoCloseable {

    /**
     * Closes this stream and releases any system resources associated
     * with it. If the stream is already closed then invoking this
     * method has no effect.
     * ...
     */
    public void close() throws IOException;
}

@dbw9580 any updates here?

@dbw9580 any updates here?

As I mentioned in https://github.com/Alluxio/alluxio/pull/15562#issuecomment-1131321183, I think we need to understand first what is causing the error @secfree was seeing in #15556. I proposed a fix in https://github.com/Alluxio/alluxio/pull/15579 and @secfree may be able to provide more feedback.

For ClosableResource, as it support closing multiple times from the definition, so we should fix this part, right?

This statement is correct, however, I think ClosableResource is supposed to implement AutoCloseable instead of Closeable. Many instances of this class are used to wrap a resource acquired from a resource pool, and the resource pool does not support releasing a resource more than once. We should change ClosableResource to implement AutoClosable instead.

As #15579 works but this not, so I will close this.


feat(config): split components to chunks for independent load

Prod dist are now slit to chunks.
Description

Now you can load all components: import "@momentum-ui/web-components";
Or individual components: import "@momentum-ui/web-components/comp/md-button";
As addition fix to namespaces added.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Documentation update
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

Checklist:

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

Having hard times to merge 3 files that is left
web-components/src/components/timepicker/TimePicker.ts
web-components/src/components/datepicker/DatePicker.ts
web-components/src/components/tooltip/Tooltip.ts

thx
I think it is ready to merge


cant run minecraft with this mod

Before it worked normally, but now i downloaded it again and minecraft cant run. How can I fix it? I attached latest log.
latest.log

Make sure to install the correct jar. Also, that's not the complete logs.