React36: Manage-Order of events (part 2)

I want to make sure I understand the order of events when this code runs. Here's what I think - please tell me how wrong this is. Also, my terminology may be a little strange - I'm not sure what to call things.
When the button is pressed, the message property of the state object of Controller is modified and, as a side effect, this triggers a new rendering of all 4 elements in the render() method of Controller. In particular, this perhaps causes something like the function call:
React.createElement(Dialog,{message: "second"},null);
(but probably not this particular function???). Whatever the actual function is, it does not instantiate a new Dialog object (because I tested that the constructor is not called). Rather, this creates a brand new javascript object in memory representing Dialog (but not a new Dialog instance), which has a "props" that is a different object from the previous one (ie, at a new address in memory).  Then this new props is passed to the (already existing) Dialog instance , where it first shows up as nextProps and then as props. So this chain of events does not modify the old props, but replaces it.
The reason I'm going through all this is that I understand that props is supposed to be treated by the program as immutable, and therefore the previous props (in the "old" Dialog javascript object) must be replaced with the new one, and this new one is passed to the (already existing) Dialog object and becomes nextProps and then props.
Also, presumably this is why, in a further challenge, you compare nextProps props using (!==) and not checking the property value. That may be confusing.
I hope this is clear.
I'm not sure what's going on on cmd-enter (I use a mac). It seems to run the code twice, calling both constructors both times. There is something strange happening there, because under the button is "second".
Thank you.

So, this is really digging into how React actually works. For that, you should consult the documentation fro a deeper understanding because I am definitely not an expert on the inner-workings of React. However, in general, my understanding is that React keeps a virtual DOM in memory of your current app structure. When a change occurs (a state update occurs), React will compute the next representation of the DOM as a result of this change in memory. It will then create a diff of these two representations to determine exactly what actually needs to be changed, and only then will it go into the actual DOM and make these changes. After a component is initially mounted to the page (that is when the constructor would be called), all further changes will occur in the way I just described. One of React's advantages is that all of that logic is abstracted away from the user.
I'm not sure what you mean about nextProps and !== in a future challenge. If you think there is something confusing there let me know which challenge you are talking about.
Also, cmd-enter is running the tests on your code. If you're looking at your console a lot to try to understand what is going on, I would recommend that because of the way we are testing the code you may be seeing output there that isn't necessarily a result of your code. If you are trying to really inspect how React works in that way I would suggest just writing some React code in a different environment.

In React 37 you have (note !==)
If you wrote return nextProps !== this.props the component would only update when it receives new props that are not equal to the current props.
Is there a challenge where the application, on a button click say, changes the number of elements in a component?  For example, a list where the number of list elements can vary.
Thanks for your responses
Steve
On Jan 27, 2017, at 3:36 PM, Sean Smith<EMAIL_ADDRESS>wrote:
So, this is really digging into how React actually works. For that, you should consult the documentation fro a deeper understanding because I am definitely not an expert on the inner-workings of React. However, in general, my understanding is that React keeps a virtual DOM in memory of your current app structure. When a change occurs (a state update occurs), React will compute the next representation of the DOM as a result of this change in memory. It will then create a diff of these two representations to determine exactly what actually needs to be changed, and only then will it go into the actual DOM and make these changes. After a component is initially mounted to the page (that is when the constructor would be called), all further changes will occur in the way I just described. One of React's advantages is that all of that logic is abstracted away from the user.
I'm not sure what you mean about nextProps and !== in a future challenge. If you think there is something confusing there let me know which challenge you are talking about.
Also, cmd-enter is running the tests on your code. If you're looking at your console a lot to try to understand what is going on, I would recommend that because of the way we are testing the code you may be seeing output there that isn't necessarily a result of your code. If you are trying to really inspect how React works in that way I would suggest just writing some React code in a different environment.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/bonham000/fcc-react-tests-module/issues/168#issuecomment-275767974, or mute the thread https://github.com/notifications/unsubscribe-auth/AWkV27mc4whZqyx6xVQuoapj_f3DyTG_ks5rWlU6gaJpZM4LwLO-.

Yes that's exactly right. If the props is a string or number then that is what will happen. If it was an object or array it would not be as simply as just comparing them with !== but the notion of 'If the next props are the same as the current props, don't re-render' is the core idea. I think I will update how that reads so it's more clear.
I don't think there are any challenges at the moment where the number of items in a list changes, but the challenges where you map items in a list out would be easy to add a button that removes elements from the list to achieve that.

In thinking about this, you might spend a little more time in the beginning on the origin of props and its structure in relation to the inline attributes.
On Jan 27, 2017, at 3:36 PM, Sean Smith<EMAIL_ADDRESS>wrote:
So, this is really digging into how React actually works. For that, you should consult the documentation fro a deeper understanding because I am definitely not an expert on the inner-workings of React. However, in general, my understanding is that React keeps a virtual DOM in memory of your current app structure. When a change occurs (a state update occurs), React will compute the next representation of the DOM as a result of this change in memory. It will then create a diff of these two representations to determine exactly what actually needs to be changed, and only then will it go into the actual DOM and make these changes. After a component is initially mounted to the page (that is when the constructor would be called), all further changes will occur in the way I just described. One of React's advantages is that all of that logic is abstracted away from the user.
I'm not sure what you mean about nextProps and !== in a future challenge. If you think there is something confusing there let me know which challenge you are talking about.
Also, cmd-enter is running the tests on your code. If you're looking at your console a lot to try to understand what is going on, I would recommend that because of the way we are testing the code you may be seeing output there that isn't necessarily a result of your code. If you are trying to really inspect how React works in that way I would suggest just writing some React code in a different environment.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/bonham000/fcc-react-tests-module/issues/168#issuecomment-275767974, or mute the thread https://github.com/notifications/unsubscribe-auth/AWkV27mc4whZqyx6xVQuoapj_f3DyTG_ks5rWlU6gaJpZM4LwLO-.

Thanks, I’ll look for that.
On Jan 27, 2017, at 4:18 PM, Sean Smith<EMAIL_ADDRESS>wrote:
Yes that's exactly right. If the props is a string or number then that is what will happen. If it was an object or array it would not be as simply as just comparing them with !== but the notion of 'If the next props are the same as the current props, don't re-render' is the core idea. I think I will update how that reads so it's more clear.
I don't think there are any challenges at the moment where the number of items in a list changes, but the challenges where you map items in a list out would be easy to add a button that removes elements from the list to achieve that.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/bonham000/fcc-react-tests-module/issues/168#issuecomment-275777164, or mute the thread https://github.com/notifications/unsubscribe-auth/AWkV228zc3qPNRnLt2nBRKA0IE-tQDRbks5rWl9CgaJpZM4LwLO-.


Changes in #92 took no effect

Hello again! Thx for replying to #93 and merging #92.
Unfortunately these changes took no effect because PR was merged after new lodash imports were created in code:(
i've prepared new PR #95 to fix this again.
I've also tried to add some linting rules like http://eslint.org/docs/rules/no-restricted-imports, to automatically prevent creating such imports in the future. But it looks like that eslint currently in broken state:( And it is not a part of the project build

merged


As a marketing analyst, I want to know the population of people living in cities versus those living in rural areas for each region so that I can decide on either urban-centric or rural-centric approach.

Identify the relevant tables and columns from the world database.
Write an SQL query to retrieve population data for both urban (cities) and rural areas across each region.
Ensure the query distinguishes between urban and rural populations for each region and provides the total population for both categories.
Analyze and verify the data.


feat!: separate fungible and non-fungible assets

Context

Closes #4087, related to #4782

Solution

Describe the approach taken to achieve the objective / resolve the issue.

Migration Guide (optional)

If this PR contains a breaking change relative to the main branch, provide an instruction on how affected parties might need to adapt to the change.

Review notes (optional)

For complex PRs, try to provide some information on how to approach the review more effectively.
For example, is there a natural order in which the affected files should be reviewed?

Checklist

[x] Remove asset registering and unregistering
[x] Remove Store assets
[ ] Reintroduce Store assets as proper non-fungible tokens
[ ] Make the overall API easier to use & more explicit
[ ] (optional) I've written unit tests for the code changes.
[ ] All review comments have been resolved.
[ ] All CI checks pass.

I need help with a Kura-related error in one of the custom executor tests (executor_custom_instructions_complex):
  2024-09-24T12:31:24.525768Z ERROR iroha_core::kura: Failed to store block, error: IO(Os { code: 2, kind: NotFound, message: "No such file or directory" }, "/var/folders/fw/cqq4z0z576g9fh9b29ghfly80000gn/T/.tmpqDUcqp/blocks.data")
    at crates/iroha_core/src/kura.rs:261
    in iroha_core::kura::kura_receive_blocks_loop

Kura has encountered a fatal IO error.
thread 'kura' panicked at crates/iroha_core/src/kura.rs:262:21:
Kura has encountered a fatal IO error.


remove buildtest show schema feature.

Fix #215.
Refactor code, remove regtest and update docs where reference
to schema. Removed the example docs generated by docgen script

@vsoch I am going to close this PR. I realize you added two new options in #219 in buildtest show schema which conflicts with this PR.


Primary rule should accept this keyword

Fixes #5702.
Allows this to appear as primary, for example in String(this == "x").

How to replicate CI configuration locally?

Build Chain tool does "simple" maven build(s), the builds are just Maven commands, but because the repositories relates and depends on each other and any change in API or class method could affect several of those repositories there is a need to use build-chain tool to handle cross repository builds and be sure that we always use latest version of the code for each repository.
build-chain tool is a build tool which can be used on command line locally or in Github Actions workflow(s), in case you need to change multiple repositories and send multiple dependent pull requests related with a change you can easily reproduce the same build by executing it on Github hosted environment or locally in your development environment. See local execution details to get more information about it.

How to retest this PR or trigger a specific build:

for pull request and downstream checks

Push a new commit to the PR. An empty commit would be enough.

for a full downstream build

for github actions job: add the label run_fdb

for Jenkins PR check only

If you are an ASF committer for KIE podling, login to Jenkins (https://ci-builds.apache.org/job/KIE/job/drools/), go to the specific PR job, and click on Build Now button.

@tkobayas please review.

Fixing progress: There were 410 failed tests (-18 approximately).


MacOS build script & Github Action for Trelis 17.1 and Cubit 2020.2

This builds on top of #84 , review after merge.
This also relies on svalinn/mcnp2cad#73
This includes:

build script to build the plugin on MacOS
GItHub Action to build the plugin on MacOS for Trelis17.1 and Cubit 2020.2

this works<EMAIL_ADDRESS>added a new PR depedency: svalinn/dagmc#728
we could avoid svalinn/dagmc#728 and limit hdf5 version to 1.10.7

getting the following error when trying to load Cubit2020.2 plugin built with the GitHub action:
ERROR: <stdin>, line 0. Error loading /Applications/Coreform-Cubit-2020.2.app/Contents/MacOS/plugins/libsvalinn_plugin.so:
ERROR: <stdin>, line 0.
Finished Command: undo on

installing brew install eigen hdf5 using brew solves it.
I guess we have to ship some of those within the package

it was actually slightly more complicated than that:
one of the hdf5 deps was required not hdf5 itself brew install szip

I fixed it adding /libsz.2.dylib in the plugin and running :
sudo install_name_tool -change /usr/local/opt/szip/lib/libsz.2.dylib @rpath//libsz.2.dylib libsvalinn_plugin.so

inclusion of szlib fixed it.

I plan to review this after we merge #84 to focus on the Mac parts...

Reworking this so it can match the linux design,
It'll be broken for a while, please ignore this until I ping you :)

@gonuke @pshriwise I reshaped the MacOS build to leverage the linux_share_build.sh (now named unix_shared_build.sh)
I have no way right now to test the build_plugin_macos.sh.
I didn't re-tried the linux docker build against those new change as well.

our PyNE amalgamation broke this...

Still curious about making the mac.yml file a little more similar to the linux.yml file, but that might be best as a conversation

I PR'ed some changes to address

I think this is ready to go @pshriwise

as a reminder this depends on svalinn/mcnp2cad#73

@pshriwise @gonuke I replaced the cmake files with a .diff files and apply a patch now instead of replacing those 2 files...
I think this is ready to go in
(as we said the merge between Mac.yml and linux.yml should go in a later PR)


enhanced the extension

Issue: #7

Visual Feedback in Editor: Matching columns in the CSV file are now highlighted directly within the Visual Studio Code editor. This provides users with immediate visual feedback on which columns match the columns defined in the selected Prisma schema.

Interactive Interface for Prisma Schema Selection: Users can now select the Prisma schema file using Visual Studio Code's built-in file picker dialog. This simplifies the process of specifying the Prisma schema file path, enhancing the user experience.

Informative Messages: When matching columns are found, the extension displays informative messages to indicate which model the columns match. This helps users quickly understand the relationship between the CSV columns and the Prisma schema models.

These are the features added.

https://github.com/SamagraX-Stencil/stencil/assets/50093149/5c31d7b8-953f-4e70-93cf-1054e4307343

@techsavvyash please review the PR

vscode.mp4

Why do we have to select the schema.prisma manually?

vscode.mp4

Why do we have to select the schema.prisma manually?

because if we declare it in the extension file itself, and we deploy the extension the user can't modify the path as because he doesn't have the source code of the extension.

What new have you done in this? This functionality is already present in the current state of the VS Code extension, seems more like a downgrade where we are picking the schema.prisma by hand and we have to right click after selecting data to validate and not just run it like a command from the VSCode command pallate.

What new have you done in this? This functionality is already present in the current state of the VS Code extension, seems more like a downgrade where we are picking the schema.prisma by hand and we have to right click after selecting data to validate and not just run it like a command from the VSCode command pallate.

no, we can use the command pallate to

sir when we deploy the extension, how the user can access the source code? It is not possible right, for that only I've added option to select the file.

vscode.mp4

Why do we have to select the schema.prisma manually?

because if we declare it in the extension file itself, and we deploy the extension the user can't modify the path as because he doesn't have the source code of the extension. @techsavvyash

We should provide an option to configure the path to schema file using the extension settings if the user absolutely needs it, but in majority cases the schema.prisma of concern would always be present in the root level prisma folder.

sir when we deploy the extension, how the user can access the source code? It is not possible right, for that only I've added option to select the file.

why hard code the path? use environment variables/config parameters with a default value.

so, is the pr okay? or any other features should be implemented sir?

no, the PR is not okay, we are discussing the problems only.

sir, what is the solution you suggest for these problems?

please read the conversation again.

yeah sir, so we can fetch it from env

also please suggest me new features that can be implemented, I'm running out of ideas

i am still not seeing what is the new thing you have implemented till now, the current functionlaity you are giving is already present in the current state of the extension, what is the point of this PR?

It is to make some enhancement to the existing extension sir

What is the enhacement?

highlighting the matched columns, and configure path to prisma file instead of hard coding it in the extension itself

highlighting the matched columns, and configure path to prisma file instead of hard coding it in the extension itself

is the second part implemented?

which one? option to configure the path to prisma file

or the highlighting?

also, it's better to highligh the unmatched columns (errors) rather than the matched columns.

which one? option to configure the path to prisma file?

this one

which one? option to configure the path to prisma file?

this one

that's the selecting prisma file from file explorer that pops up

also, it's better to highligh the unmatched columns (errors) rather than the matched columns.

alright will do that

which one? option to configure the path to prisma file?

this one

that's the selecting prisma file from file explorer that pops up

I don't think picking files from the file picker counts as giving the path via a config. Stencil is built with the idea to "enhance" DX not downgrade it.

alright sir, will close this pr, and will implement some other feature.
can you please suggest some new feature that i can work on?


RAT-290: upgrade animal-sniffer-maven-plugin to 1.21

@XenoAmess thanks for your PR - can you provide a bit more of explanation what this new profile does and how it may change the release process of RAT? By only seeing the changes I'd like to know more about any implications of the change.

well it should be able to pass ci but we don't have enough money for docker.

@XenoAmess thanks for your PR - can you provide a bit more of explanation what this new profile does and how it may change the release process of RAT? By only seeing the changes I'd like to know more about any implications of the change.

@ottlinger
Fine.

javac added a parameter as --release, at jdk9+.
In short it describe which version of jdk would the produced jar used with, and openjdk guys will make sure it can run under it.
If not then fire bug report to them.
https://docs.oracle.com/javase/9/tools/javac.htm#JSWOR627
https://www.baeldung.com/java-compiler-release-option

maven-compiler-plugin can read maven.compiler.release property. If there be such a property, it will pass it to the javac.

--release is useful for jdk9+, but notice that if we pass this param to jdk8-, then it will fail error.
So we must find a way to only pass it when jdk version >= 9
That is why we use a auto-activate profile with condition.
Once the condition [9,) meat, means jdk version >= 9, then maven auto activate this profile.
When activate this profile, this profile set a property <maven.compiler.release>8</maven.compiler.release>
Then maven-compiler-plugin read the property <maven.compiler.release>8</maven.compiler.release> and pass it to javac.
Then the magics openjdk guys leaves in javac would help the compiler correctly compile jdk8-usable jar, which means do not choose jdk-8 unexist functions and other trickey things, that should not cared by we non-jdk repo.

@XenoAmess you ran into the same problem as I did when I just updated the reference in the pom.xml alone:
https://app.travis-ci.com/github/apache/creadur-rat/jobs/558978072

@XenoAmess can you be so kind to rebase your PR as the build configuration changed due to RAT-296 ;) Did you already file a contributor agreement? https://www.apache.org/licenses/contributor-agreements.html
Thanks again for your help

@XenoAmess let's see if all pipelines run through .... I'll merge afterwards.
Thanks again for your help :) I'll add an entry to the changelog and do the red-tape later.


In single server mode, the player can't see the other players spawned before him

Try to fix SingleChannelDataView. The SpatialChannelDataView doesn't have this issue.

Can't repro


Support the masquerade statement

Allow users to define masquerade configuration.
Ref: https://www.mankier.com/5/libnftables-json#Statements-Network_Address_Translation

Implemented with #30


Bug - Admin Area: Can't add or move subcategories using the 'Sort Categories' tool

Issue
When signed in as an admin user, the 'Sort Categories' tool doesn't let you add a subcategory to a category that doesn't already have any subcategories.
Fix
Tweak the sorting functionality so that it will be able to drag a category into another and make it a subcategory.

Approved.

Tested. Will get users to UAT.

Passed UAT on staging 3/11/2020.
Tested successfully on live 6/11/2020.


Update capture-browser-trace.md to be more explicit about concerns when submitting browser traces.

@JnHs
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@DavidAnson looks good, thank you!
#sign-off

Merging now.


fix: fix storageFrom.jsx rename failure [skip actions]

Signed-off-by: moweiwei<EMAIL_ADDRESS>
What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #19
Special notes for reviewers:

Does this PR introduced a user-facing change?

Additional documentation, usage docs, etc.:

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@39c962f). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #20   +/-   ##
=========================================
  Coverage          ?   48.33%
=========================================
  Files             ?        6
  Lines             ?      542
  Branches          ?      150
=========================================
  Hits              ?      262
  Misses            ?      219
  Partials          ?       61

skip-checks: true

/lgtm
/approve


Minor fix in UAVCAN driver startup logic

UAVCAN driver silently ignores repeated start commands without error. This allows to avoid failures when UAVCAN driver is started from extras script before default initialization sequence is executed.

Thanks, fixed!


Fixed typo that was preventing videoWasViewedVungle from working on iOS.

I was getting a crash when checking if a Vungle video had been watched. Turns out there was a typo in IOSHelper.m.

Thanks for fixing this bug.

You're welcome.
Thank you for putting together this great package!! It's saved me a lot of time :-)

No problem.
We are looking to create an official team for the Cocos Helper to provide constant updates/features, is this something you would be interested being apart of.

Sorry for the late reply! I've just sent you a PM on the Cocos2D-X forum.


Add support for scaling transform

Users specified scene scaling is not currently supported (only translation is available). Though in 3.0.0.beta we are only required (contractually) to support 2D data, the contour and barb renderer can support 3D variables. Moreover, even for 2D renderers the lack of scaling means that displacing a plane (e.g. 2DData, Image) by a height field is effectively a no-op for most ESS data sets

Stas, let's talk about what needs to be done here.

Add entry to transform table for "origin" about which to do scaling and rotation.
This needs to be done in GeometryWidget as well as Navigation/Viewpoint tab.
Need to modify

transform paramsbase class
Transform Widget (in vaporgui/applications0
OpenGL render code

Scaling from the Navigation->Viewpoint tab seems to perform a different operation than scaling individual renderers.  I believe that the origin is not being set to the same values.
To reproduce:

Spin up Vapor with KatrinaMaster.nc
Create a TwoDData instance, apply height variable HGT
In the Geometry tab, scale the Z axis by 100.  Confirm that this is the desired effect.
In the Navigation->Viewpoint tab, scale the scene by 100.  The renderer will drop down outside of the domain box.  Note that the origin is ~10km here, while it's ~0 for the TwoDData renderer.

This is working. Picking the default origin is tricky. Each renderer picks one based on the default variable selected. For twoD this is a 2D variable, so the component of the origin is 0. The data set transforms select the origin by looking at all active variables, which includes 3D variables. Hence, here the default of the Z component is not 0.


Submit shufflenet implementation

Fei, thanks for putting shufflenet into its own directory. In addition, can we put it one level lower into the framework name... the convention we are following is as follows:
E.g.:

inference/edge/object_classification/mobilenets/tflite/


Refactor tags to support links and non-links | Make iregs search use tags macro

iRegs search has tags in the search results, but unlike elsewhere they are not links. https://github.com/cfpb/consumerfinance.gov/pull/7831 assumed topic tags would be links, so provided a hover state, which is erroneously shown on iregs topic tags. This PR refactors the tags macro to support both links and not for tags.
Changes

Refactor tags macro to support tag links or not.
Update iregs search topic tags to use tags macro.

How to test this PR

Visit the following pages and hover the topic tags and resize to mobile and hover:

http://localhost:8000/rules-policy/regulations/search-regulations/results/?q=mortgage&regs=1002&regs=1003&results=25&order=relevance
http://localhost:8000/about-us/blog/we-are-extending-the-deadline-for-comments-about-data-brokers/
http://localhost:8000/about-us/blog/?filter1_topics=bureau-milestones
http://localhost:8000/rules-policy/regulations/search-regulations/results/?regs=1002&q=mortgage
http://localhost:8000/about-us/events/cfpb-finex-webinar-identity-theft-scams-older-adults/
http://localhost:8000/about-us/blog/?filter1_topics=know-before-you-owe
http://localhost:8000/about-us/blog/whats-ahead-for-wells-fargo-and-its-customers-ar/

Works as advertised! It is a little odd that the touch target "rails" appear on mobile width for the non-clickable topics, but I know that's consistent with what we had before.

Yeah I noticed that too. We should probably have a stand-alone tag (well, actually we kinda do), and then that is the non-link thing, and then when you aggregate them together in a list they use the tags macro and are only links. But yeah, for now since the rails are there, that wouldn't work.


SQLAlchemy: Add support for 'autogenerate_uuid' in column creation

Solves #533
Where by setting the column:
Column('x', sa.Text, primary_key=True, crate_autogenerate_uuid=True)

will generate:
x STRING DEFAULT gen_random_text_uuid() NOT NULL

Checklist

[x] CLA is signed
[x] Docs
[x] Tests

Circling back to this, I'm effectively checking against
Column(..., server_default='value')

with
if default is not None:
    raise sa.exc.CompileError(
            "Can only have one default value but server_default"
            " and crate_generate_uuid are set"
               )

But It'd probably also clash with Column(..., default=lambda:str(uuid.uuid4()))
I will have a deeper look at this.

Hi @surister,
thanks for your patch. It looks good to me so far.
Regarding your question about option clashes: I don't know if and how SQLAlchemy prevents to use both default and server_default, or if it doesn't do at all.
Depending on how SQLAlchemy handles that, we should not necessarily add anything on top - only if it makes sense, and doesn't overcomplicate the code.
With kind regards, Andreas.

Using default + crate_autogenerate_uuid doesn't really 'clash', it seems that CrateDB just ignores the DEFAULT directive and puts in whatever value it gets on the INSERT

More importantly when using it as:
Column(..., crate_autogenerate_uuid=True, primary_key=True)

We get the following warning:
SAWarning: Column 'aaaa3.x' is marked as a member of the primary key for table 'aaaa3', but has no Python-side or server-side default generator indicated, nor does it indicate 'autoincrement=True' or 'nullable=True', and no explicit value is passed.  Primary key columns typically may not store NULL.

Because technically, there is no autoincrement_column and server_default nor default is set, so SQLAlchemy cannot assure that the value will not be Null (and nullable in this case is set to False by default)
SQLAlchemy warning source

Dear @surister,

So, the outcome for this topic could be essentially to cover that detail about func.gen_random_text_uuid() within the documentation at, for example, docs/sqlalchemy.rst.

I've just submitted GH-585, so I am closing this. Apologies again that I was giving wrong guidance on this matter through GH-533.
With kind regards,
Andreas.


[Bug] right clicking on profile while in a voice chat crashed the client (since  5d13037)

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
this is the error in the console:

if u give me 30 minutes i'll open a PR with a bug fix

ah, I didn't expect that there might be empty, I will push it with the next commit with other small fixes


Number Input component

Description
It would be great if there is a number input with buttons to increase and decrease value.
Props would be: min, max, step, autofocus, disabled, etc
for reference: https://github.com/marcocesarato/react-native-input-spinner
Additional context
No response

You can use the UInput component and set the type to number and use all the other props, it will work.

The increase/decrease button of <UInput type="number" /> is tiny, and is not possible to click in mobile screen.
I also tried to make a component for number input with props and emit, but it it not beautiful as other components of Nuxt UI.

If you want a custom component like in your reference, you can use the ButtonGroup and put a button with a - icon, an input with the .number modifier and another button with the + icon.

I wrote a custom NumberInput component with increase/decrease buttons for Nuxt UI.
It might imperfect but it works and satisfy me.
Here is the code, someone might need it.
Cheers,
<template>
  <UButtonGroup :size="size" orientation="horizontal">

    <UButton @click="decrement" icon="i-heroicons-minus-20-solid" color="gray"/>
    <input type="number"
      :value="value"
      @input="handleInput"
      :min="min"
      :max="max"
      :step="step"
      class="w-20 text-center item-center border-solid border-2 border-gray-300"
      />
    <UButton @click="increment" icon="i-heroicons-plus-20-solid" color="gray"/>
  </UButtonGroup>

</template>

<script setup lang="ts">
import { ref, watch } from 'vue';

// Define props
const props = defineProps({
  modelValue: {
    type: Number,
    default: 0
  },
  min: {
    type: Number,
    default: -Infinity
  },
  max: {
    type: Number,
    default: Infinity
  },
  step: {
    type: Number,
    default: 1
  },
  size: {
    type: String as PropType<"md" | "sm" | "xs" | "2xs" | "lg" | "xl" | undefined>,
    default: 'md'
  }
});

// Reactive value reference
const value = ref(props.modelValue);

// Watch for external modelValue changes
watch(() => props.modelValue, (newVal) => {
  value.value = newVal;
});

// Emit function
const emit = defineEmits(['update:modelValue']);

// Increment function
const increment = () => {
  const newValue = Math.min(value.value + props.step, props.max);
  value.value = newValue;
  emit('update:modelValue', newValue);
};

// Decrement function
const decrement = () => {
  const newValue = Math.max(value.value - props.step, props.min);
  value.value = newValue;
  emit('update:modelValue', newValue);
};

// Handle input change
const handleInput = (event: Event) => {
  const target = event.target as HTMLInputElement;
  let newValue = parseFloat(target.value);
  newValue = Math.max(Math.min(newValue, props.max), props.min);
  value.value = newValue;
  emit('update:modelValue', newValue);
};
</script>

<style scoped>
  /* Hide Arrows From Input Number */
  /* Chrome, Safari, Edge, Opera */
  input::-webkit-outer-spin-button,
  input::-webkit-inner-spin-button {
    -webkit-appearance: none;
    margin: 0;
  }

  /* Firefox */
  input[type=number] {
    -moz-appearance: textfield;
  }
</style>

The code above has a bug. The prop "size" does not work.
For instance, if we call <NumberInput v-model="val" :size="xs" /> , the console will show a warning: Vue warn]: Property "xs" was accessed during render but is not defined on instance.


Add formatting to messages

@devinmarieb I was able to get the message component working and also added some formatting.  Take a look and let me know what you think!

Looks good. Excited for you to explain what you did. Without merging first and being able to play with it, I'm curious what the hourly stuff is doing. I'm guessing time stamp on the message?


check driver.Valuer response, and skip the column if nil

Problem:
Some sort of column are updated by nil always, when type for the column implements driver.Valuer interface and it's Value() method return nil. It should check requiredField.
Solution:
Check the response value of driver.Valuer.Value(), if it is nil and not requiredField, skip the column.

Codecov Report

Merging #1167 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #1167      +/-   ##
=========================================
- Coverage    54.7%   54.7%   -0.01%
=========================================
  Files          42      42
  Lines        7646    7648       +2
=========================================
+ Hits         4183    4184       +1
  Misses       2935    2935
- Partials      528     529       +1

Impacted Files
Coverage Δ

statement.go
66.96% <100%> (+0.08%)
:arrow_up:

xorm.go
64.61% <0%> (-1.54%)
:arrow_down:

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 fd49db5...fa40f1e. Read the comment docs.

Codecov Report

Merging #1167 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #1167      +/-   ##
=========================================
- Coverage    54.7%   54.7%   -0.01%
=========================================
  Files          42      42
  Lines        7646    7648       +2
=========================================
+ Hits         4183    4184       +1
  Misses       2935    2935
- Partials      528     529       +1

Impacted Files
Coverage Δ

statement.go
66.96% <100%> (+0.08%)
:arrow_up:

xorm.go
64.61% <0%> (-1.54%)
:arrow_down:

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 fd49db5...fa40f1e. Read the comment docs.


Update pms link

https://github.com/ambientsound/pms now points to https://github.com/kimtore/pms

Thank you :)


Desiginig and Code of the top bar of the Main Activity

Adding the top bar to the homepage of the app.(Main Activity)
Assign this to me.

I am working on the complete design of the main activity ....Will add more pull requests.

@Qomfortzone assigned

Done Under #81


specifying weblogic deployment  upload directory

When we use deployApps.sh, the ear/ear files are uploaded to wlsdeploy/applications/ directory under the <DOMAIN_HOME>. Is it possible to provide an alternate location to where the files should be uploaded ? like uploads directory under <DOMAIN_HOME>/servers/<Admin_server_name>/upload or other directories external to <DOMAIN_HOME>

The WDT tools (create, deployApps, ...) use the wlsdeploy folder for several reasons.  The primary reason being safety.  With the known folder approach, WDT is able to reduce the chance of failures, greatly simplify file operations, and prevent accidental damage to an existing domain.  The cost of stepping out of this "safe zone" is not something we are interested in taking on at this time.
It is possible to have applications outside the wlsdeploy folder.  For example, you can pre-place the file in a location outside the domain home, and then model the full path of that file in the WDT yaml.

Thanks Derek. I will use the suggested method.


Out of inode in the case of large table/partition numbers

Enhancement
Currently, TiFlash would need to create lots of files when there are many tables or partitions.
For each table or partition, we need to create at least one DTFile. And a DTFile contains all column files, along with other meta data files. For example, here are the files in DTFile for create table A (a int, b int);:
%2D1.dat
%2D1.idx
%2D1.mrk
%2D1024.dat
%2D1024.idx
%2D1024.mrk
%2D1025.dat
%2D1025.idx
%2D1025.mrk
1.dat
1.idx
1.mrk
1.null.dat
1.null.mrk
2.dat
2.idx
2.mrk
2.null.dat
2.null.mrk
config
meta.txt
pack
property

And those files could cause "Out of inode" issues.

Also, TiFlash will create empty DTFile even if there is no data in the table or partition. Which is totally unnecessary and should be optimized.


Create importable bltool-sources.jar

When importing bltool.jar into Eclipse, there's no available source attachment that can be used for code debugging. This along with the lack of an available JavaDoc makes knowing the callable functions for bltool confusing.

I've never used Eclipse, but all of the source code is in the jar:
$ unzip -l bltool-0.2.2-standalone.jar | grep bltool | grep clj
   1021  12-12-2015 13:19   META-INF/leiningen/bltool/bltool/project.clj
  12327  02-10-2015 10:04   bltool/data/backloggery.clj
   1374  02-10-2015 10:03   bltool/data/default.clj
    373  05-25-2014 21:14   bltool/data/edn.clj
      0  11-07-2013 21:53   bltool/data/html.clj
   1677  02-10-2015 10:04   bltool/data/steam.clj
    678  02-10-2015 09:58   bltool/data/text.clj
   3210  02-10-2015 10:04   bltool/data/xboxlive.clj
    420  12-12-2015 13:08   bltool/data/json.clj
    413  02-10-2015 10:03   bltool/flags.clj
    247  12-12-2015 13:08   bltool/data.clj
   4812  12-12-2015 13:08   bltool/core.clj

Does Eclipse need some sort of additional plugins for Clojure code or something? Or do I need to add something to META-INF so that it can find the source?

I'm not exactly sure how source attachments would would work for Clojure projects, as I'm more familiar with Java libraries.
I'm personally looking to use bltool for a game tracking program built in Java. Unfortunately, I'm unfamiliar with Clojure, and haven't been able to figure out how to properly call bltool's functions from a Java application

I've never really worked on calling Clojure from Java, but this section of the Clojure/Java Interop manual may be helpful -- it looks like you can just load the jar, then get an IFn representing a clojure function and call it from Java.
It's possible to make a Clojure library expose more "java-like" interfaces with normal Java classes and suchlike, but this requires extra work on the Clojure side and it's something I haven't done with bltool.


swift 5 Xcode 10.2 support

Would appreciate an update to the latest release

support

I'm on 4.2, maybe I should wait until this.


add markdownlint to pre-commit

we don't need a GitHub superliner. That's too complex. I was wrong about the thing that markdownlint cant be individually added.

Nice, thanks!


How do I create those oauth credentials?

From the usage doc:
    --client-id       OAuth2 Client ID
    --client-secret   OAuth2 Client Secret
    --refresh-token   OAuth2 Refresh Token

Is this the right URL? https://developer.chrome.com/webstore/using_webstore_api
That's leading me to https://console.developers.google.com/apis/credentials which doesn't feel right. The OAuth flow described there is one where a user consents in-browser. 😱

Yeah, the official Google docs on that topic are pretty dreadful. I burned through an hour or so one night trying to figure it out (but apparently never documented it 😛).
I'll get some updated docs in the readme later today.

Apologies for the delay. Here are the steps:

Visit https://console.developers.google.com/apis/api/chromewebstore/overview
Click the button to create a new project (confusing, but totally separate from the project for your extension)
When redirected back to the Overview page, click Enable for the API
Click the Credentials tab on the left-hand side of the page
Click Create credentials >> OAuth Client ID

Go through the Configure consent screen prompt (you just need to enter a project name)

You should then be given a client_id and client_secret
Now, go back to this page, and start at step 9. Make sure you go through the OAuth acceptance page logged in under the account that owns the extension you're planning to publish.
Once you've jumped through all the hoops outlined in that document, you should have your client_id, client_secret, and refresh_token.

Pretty ridiculous, right?
Let me know if those steps work for you, and (if you don't mind) please point out any places my steps are lacking along the way. Once you confirm the steps above, I'll update the readme with the info.

@DrewML did all that and there is no refresh token there

If you manually go through the Google web page to get the token, does that mean you no longer need the ClientId and ClientSecret? I think this app should make it so you only need the ClientId and Client Secret. Going through the webpage gives you an access token, but that is not the refresh token is it?

@thecotne did you figure it out? It is terribly manually, but step 9 will give you an "authorization code", which you then use in curl to retrieve something like this:
{
"access_token" : "aaa",
"token_type" : "Bearer",
"expires_in" : 3600,
"refresh_token" : "zzzz"
}
after you get the refresh_token you can use this at any time to make a new access_token to do the deeds

did you figure it out?

could not
but i found another script https://github.com/pastak/chrome-webstore-manager
does all same things ...

that script looks more thorough (handling the authentication and access code) - cool thanks for the lead.

step 1. use your clientId and webbrowser to obtain an authorization code
https://accounts.google.com/o/oauth2/auth?response_type=code&scope=https://www.googleapis.com/auth/chromewebstore&client_id=$CLIENT_ID&redirect_uri=urn:ietf:wg:oauth:2.0:oob
step 2. use curl, your client id, your client secret and the authorization to get the access token and refresh token

curl "https://accounts.google.com/o/oauth2/token" -d
"client_id=$CLIENT_ID&client_secret=$CLIENT_SECRET&code=$CODE&grant_type=authorization_code&redirect_uri=urn:ietf:wg:oauth:2.0:oob"

not sure why you couldn't programmatically obtain these, maybe it is too hard to programmatically push the button?

sorry i missed step 8 from DrewML guide
because it says "go back to this page" i have not clicked it and ...
i was thinking back to google developer console

There are some step missing where I'm not quite sure what to do.

When redirected back to the Overview page, click Enable for the API
Ok but but should we create the API key or not from there?

Step 5 to 7
What Application type should we select? (If I select Chrome App it doesn't give me any client_secret)

@micky2be guide is actually OK

Ok but but should we create the API key or not from there?

on step 5 you create Click "OAuth Client ID" kinda like API key

What Application type should we select? (If I select Chrome App it doesn't give me any client_secret)

it's not chrome app it's for terminal use
and don't miss step 8

Hmmm, I believe I did everything correctly.
Using a REST client it seems to work with https://www.googleapis.com/oauth2/v3/token instead of https://accounts.google.com/o/oauth2/token

The confusion here is that CWU asks for a refreshToken but we actually need the accessToken
I wrote an extended guide in the main repo: 🎉 https://github.com/DrewML/chrome-webstore-upload/pull/5
I'm hosting the guide's images in this comment.


Codestyle ☝

In this PR was added code alignment and code style fixes
All generated code should satisfy swiftlint with our yml file. If it's not, please create an issue.

Now generated code can be debugged easily, love it 🏄♂❤


[feature] Kirki and selecting multiple images for gallery or slideshow

Sorry if this is not the place to post this, but I am not sure where else to ask for help or give suggestions for kirki. If there is a more appropriate place, let me know.
Is there a way to select multiple images say, if I want to create a slideshow? I have read some people possibly use the repeater, but wouldn't that mean you have to set a new image multiple times instead of using WP's built-in multi-select?
I know customizer has a gallery option, is something like that built into Kirki (or going to be)? Thanks!

This will be implemented in version 3.1 👍

Hello,
I am using the customizer and wondering if image gallery feature implemented or not.
It is set to 3.1 milestone and we have 3.0.26 on GIT yet.
Any update on this?
The feature was: Allow multiple images selection on the image type.
Thanks,
Aman

Hey there!
No, this has not yet been implemented.
I'll try to squeeze it in before 3.1, but definitely not 3.0.26

Hey there,
I want to know that multiple image select feature is now implemented or not? btw awesome work, Love Kirki.
Thank You

Duplicate of https://github.com/aristath/kirki/issues/905


Adding Notes

Description
Great feature - pointing out a small observation:
"To add a new note, press the  Note button on the toolbar above the map, or press the shortcut key N"
The Note button is not automatically available in the menu button on the toolbar above the map for Google Chrome/Windows. Is it supposed to be automatically available at the top? I can only enabled it using 'shift + N'. 'CapsLock + N' or just 'n' will not enable it.
Screenshots

Version
2.4.0-pre.0
What browser are you seeing the problem on? What version are you running?
Chrome v128.0
The OS you're using
win
Steps to reproduce

Select shift + N to start a new note which is slightly different from instructions

The browser URL at the time you encountered the bug
https://rapideditor.org/canary#map=19.00/33.76099/-84.38890&background=EsriWayback_2024-08-15&datasets=fbRoads,msBuildings&disable_features=boundaries
The auto-detected useragent string for your browser (leave blank if you're manually filling this form out)
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36

Yes good catch - we should probably update the docs here.

Looks like this now:


Implement better type inference in components.

Based off on work in #68, #69, and #60, we'll want to ensure our components have the same level of type safety as our hooks now do. This gets trickiest with Subscription, whose return data type will differ based on whether a handler is supplied or not. We may consider doing what we did w/ the hooks implementation where we have separate Subscription and SubscriptionWithHandler components to handle this polymorphism.

Roadmap

[ ] Query
[ ] Query Examples
[ ] Mutation
[ ] Mutation Examples
[ ] Subscription
[ ] Subscription Examples


LIBUSB_ERROR_IO when writing to a reconnected device

Bug Description:
Getting LIBUSB_ERROR_IO upon trying to write to and endpoint of an open device which was previously connected, disconnect and reconnected.

Steps to Reproduce:

Connect, device.open(), claim interface and call transfer write to a USB device. (works fine)
Detach the USB device.
Re-attach same USB device.
Open the re-connected device, claim interface and call transfer write to a USB device:

const device = findByIds(vid, pid);
device.open();
const interface = device.interfaces[0];
interface.claim();
interface.endpoints[0].transfer(data, (error)=> console.error(error?.message))

Transfer fails with callback error LIBUSB_ERROR_IO

Additional Information

Operating System: Windows 10
Runtime Node 20, Node 18
Node USB Version 2.11.0, 2.13.0

I wonder if the claimed interface is retained across the plugging?

I tried calling .release() on it but there is an error in the callback.

This feels like a libusb error, but I lack time to investigate :(

I updated the test suite to run the open-claim-transfer tests both before and after a disconnection cycle:
https://github.com/node-usb/node-usb/tree/test-reconnected
It uses the same sequence you outline above.
I tried these tests on macos and windows 10 and saw no issue with my test device.
Do you have any other information?


Configure location of generated files

Cont. discussion from https://github.com/zpqrtbnk/Zbu.ModelsBuilder/issues/73

Currently we put all our models in ~/Models/DocumentTypes, so that would be within the web.
But of course this is for dev mode, since the point of doing this is having something like "Domain Entities" and MVC Models we can unit-test and do logic with, and obviously always compile manually. Having them in App_Code kind of kills all of the more "hardcore" developer solution setups.
But I've had requests to support outside web paths for codegen, so I suppose it's something people would want to do. For instance putting the classes in a "common assembly" with less dependencies. (I guess Umbraco.Core must be referenced in any case)

Supporting a ~/Models/DocumentTypes of some sort should be easy. The whole point of generating them in ~/App_Data/Models was for the same reason: avoid the app restart that happens when generating in App_Code. Will look into how we can make it an option.
Going outside the web paths... I guess it would work provided that the site has permissions to write in the target directory. So maybe there would be a second option "EnableWritingOutsideOfWebPathsAndYesIUnderstandTheRisks" of some sort.

Hehe, yeah, sounds good. Might iterate a bit on the name.

Is solved in #94 btw. :)

Release in 3.0.5!

\o/
All that's left for me to be a very happy dev is opening up the writer. ;)

Also love the hidden unsafe external dir. option. Will have to test soon.


Relicense under MIT?

@Gerschtli
Let's branch off the discussion here.
Las year, in a different repo altogether, I've started to bring Nix to Android. Back in the days, it was a Nix installation script heavily inspired by the official one, which I presumed to be authored by @edolstra and published under LGPL 2.1. Or at least I hope so, because it doesn't really come with a license header.
I thought that one day it could ship as part of a Nix, as a script to install it under Termux, so I kept the license.
Then I rewrote everything, going self-contained and Termux-free, installing through zipballs instead, but kept the license. Then you rewrote it, and kept the license.
If I were asked to trace it back to the original Nix script, I'd say that only these these two lines are what's lifted from the original install script and made it through the rewrites:
USER=${config.user.userName} ${prootCommand} "$PKG_NIX/bin/nix-store" --init
USER=${config.user.userName} ${prootCommand} "$PKG_NIX/bin/nix-store" --load-db < .reginfo

What we're heading towards today, for better or worse, is home-manager, nixos or nix-darwin territory. I would argue that this is small enough to be uncopyrightable, and those three projects already bear more similarity to nix-on-droid.
That said, how do you feel about relicensing the whole contents of this repo to MIT?

I am completely okay with relicensing it. As I said before, I am not that into the license thing: What are the consequences of this step. Will it simplify the reuse/modification of home-manager, nixos or nix-darwin code?

As I said before, I am not that into the license thing

Heh, it doesn't work that way, really =) It's like saying "I'm not into this law thing" - the laws, lawmakers and law enforcement don't really care about your opinions, you're subject to that no matter what. But yeah, it's a quite a hindrance at times.

Will it simplify the reuse/modification of home-manager, nixos or nix-darwin code?

Yes, we won't have to do neither mixed licensing nor that 'permissive with GPL modifications tainting it' thing, we'll only need to preserve the author names, basically. But more importantly, those would be able to reuse our changes back, which would be a nightmare in any other scenario.
If you agree, we need to replace the license text and update the license headers, and then land your latest PRs. I can do it myself, but I'm just about to start my workday, so it'll have to wait for quite some time.

Heh, it doesn't work that way, really =)

Yeah.. I know, I should have a deeper look into that. :D

If you agree, we need to replace the license text and update the license headers, and then land your latest PRs. I can do it myself, but I'm just about to start my workday, so it'll have to wait for quite some time.

Sounds like a plan ;)

I think I've found an acceptable balance between brevity and compliance (and it's definitely more than the others are doing).
Please rebase your PRs to ad73458 (sorry for a force-push, I've added the header to the config template again).
When using MIT-licensed code from other projects, add notices in the following format:
# Copyright (c) 2019-2020, see AUTHORS. Licensed under MIT License, see LICENSE.

# <Inspired by|Based off|Uses XYZ from|...>
# <url-of-the-file>
# (<original copyright notice taken from LICENSE of the project you're borrowing from>,
#  licensed under MIT License as well)

See example in modules/environment/etc/make-etc.sh.
For other files, just the first line should suffice.

That sounds good, but maybe you should remove this header from the nix-on-droid.nix.default :)
Will rebase my PRs tomorrow

Yes, definitely, see above.

Ah, sorry, my bad ;)


Move ApiController under ManageIQ::API namespace

Purpose or Intent
This is the first major step of moving from having a monolithic ApiController into many object-oriented controllers. The planned scheme is as follows:
app/controllers/manageiq/api/base_controller.rb # => Existing common API code
app/controllers/manageiq/api/vms_controller.rb # => Specialized Vms controller that inherits the base controller's default behavior
# etc....

This first step of putting everyhing under the API namespace is being made to avoid clashes with the proposed more generic sounding BaseController, VmsController, etc...
This was a fairly unsatisfying change to make (no wins yet) but since it is so large I want to submit it on its own. Apologies to future backporters and history-spelunkers!
@miq-bot add-label api, refactoring
@miq-bot assign @abellotti

just curious, I thought the rubocop max LineLength we check for is 120, yet I see the 80 length warning all over above.

@imtayadeway can you fix the align hash warnings in the logger.rb, these are new with the introduction of the encompassing Module/Class. Thanks

LGTM!!  moving to OO one step at a time 👍


Remove didReceiveResponse hook

The naming of this hook is deceiving; if this hook is overridden it becomes
responsible for returning the parsed body and handling errors if they occur. It
was originally introduced in
https://github.com/apollographql/apollo-server/issues/1324, where the author
implemented it due to lack of access to the complete response (headers) in the
fetch methods (get, post, ...). This approach isn't a type safe way to
accomplish this and places the burden of body parsing and error handling on the
user.
Removing this hook is a prerequisite to a subsequent change that will introduce
the ability to fetch a complete response (headers included) aside from the
provided fetch methods which only return a body. This change will reinstate the
functionality that the author of this hook had originally intended in a more
direct manner.
We can consider reinstating this hook or something similar in the future with well-documented and use-case-driven semantics. Without knowing other reasons for why this hook should exist (aside from mentioned issue, which will be addressed with an alternative approach), trying to guess what this hook should do or mean is a bit pointless.

@glasser I opened https://github.com/apollographql/datasource-rest/issues/109


Set default AppVer for main branch

I propose to always have a valid AppVer in code base, and we should bump it for every major release.
For example, in this PR:

https://github.com/go-gitea/gitea/pull/18124

The gitea-sdk expects to get a valid AppVer from server response, and the gitea-sdk also have some version-depended behaviors.

please see discussion in regards to this line here: https://github.com/go-gitea/gitea/pull/9916

please see discussion in regards to this line here: https://github.com/go-gitea/gitea/pull/9916

please see discussion in regards to this line here: #9916

How to resolve the problem described in #18124  (https://github.com/go-gitea/gitea/pull/18124#pullrequestreview-842261656) ? If we do not set AppVer globally, then it would appear in many unit tests.

please see discussion in regards to this line here: #9916

How to resolve the problem described in #18124  (https://github.com/go-gitea/gitea/pull/18124#pullrequestreview-842261656) ? If we do not set AppVer globally, then it would appear in many unit tests.

I'll "fix" the sdk ... please not this again ...

https://gitea.com/gitea/go-sdk/pulls/560


Display Legends in Pie graph using react chartjs1

Im using "react-chartjs": "^0.7.3", and I want to display my legends using Pie Graph and my code looks like this. The pieConfig the configuration that I want to display, and the this.state.dataPie is my default
pieConfig:{
   //Boolean - Whether we should show a stroke on each segment
   segmentShowStroke: true,

   //String - The colour of each segment stroke
   segmentStrokeColor: "#fff",

   //Number - The width of each segment stroke
   segmentStrokeWidth: 2,

   //Number - The percentage of the chart that we cut out of the middle
   percentageInnerCutout: 0, // This is 0 for Pie charts

   //Number - Amount of animation steps
   animationSteps: 0,

   //String - Animation easing effect
   animationEasing: "easeOutBounce",

   //Boolean - Whether we animate the rotation of the Doughnut
   animateRotate: false,

   //Boolean - Whether we animate scaling the Doughnut from the centre
   animateScale: false,

   fullWidth: true,

   //String - A legend template
   legendTemplate: "<ul class=\"<%=name.toLowerCase()%>-legend\"><% for (var i=0; i<datasets.length; i++){%>
<span style=\"background-color:<%=datasets[i].lineColor%>\"><%if(datasets[i].label){%><%=datasets[i].label%><%}%>
<%}%>
",

   legend: {
      display: true,
    },
 }
}

    this.state = {
       dataPie:[{
         value: 300,
         color: "#F7464A",
         highlight: "#FF5A5E",
         label: "Red"
       }, {
         value: 50,
         color: "#46BFBD",
         highlight: "#5AD3D1",
         label: "Green"
       }, {
         value: 100,
         color: "#FDB45C",
         highlight: "#FFC870",
         label: "Yellow"
       }]
}

<Col xs={12}>
         <Paper style={{padding: '30px 20px', marginBottom: 35}} zDepth={2} rounded={false}>
           <Row style={{margin: '40px 40px 40px 40px'}}>
             <Col xs={12}>

                 <Pie
                   data={this.state.dataPie}
                   options={this.props.pieConfig}
                   width="1000"
                   height="556"
                 />

             </Col>
           </Row>
         </Paper>
       </Col>

```I hope someone who could help me to display the legends in react-chartjs 1

Hi @michaelroevievictoria can you provide fiddle? I would love to provide fix for this.

Hi @joshi-mehul thanks for the help Pal! 👍  I try to figure it out how to do it.. sorry if I don't have jsfiddle code: but here's some code snippets (react-js, babel, ES6)
pieConfig:{
//Boolean - Whether we should show a stroke on each segment
segmentShowStroke: true,
   //String - The colour of each segment stroke
   segmentStrokeColor: "#FFF",

   //Number - The width of each segment stroke
   segmentStrokeWidth: 1,

   //Number - Amount of animation steps
   animationSteps: 0,

   //String - Animation easing effect
   animationEasing: "easeOutBounce",

   //Boolean - Whether we animate the rotation of the Doughnut
   animateRotate: true,

   //Boolean - Whether we animate scaling the Doughnut from the centre
   animateScale: false,

   fullWidth: true,

   responsive: true,

   showInLegend: true,

    maintainAspectRatio: true,

    legendTemplate: "<ul style=\" display: inline;\" className=\"<%=name.toLowerCase()%>-legend\"><% for (var i=0; i<segments.length; i++){%>
<div className=\"indicator_box\" style=\"background-color:<%=segments[i].fillColor%>; padding: 5px ; margin: 8px 10px 5px 10px; display: block; float: left; \">
<%if(segments[i].label){%><%=segments[i].label%><%}%>
<%}%>
",
 }

}

@michaelroevievictoria great, happy that you've fixed the issue.

@michaelroevievictoria please close the issue now.


TODO add ESC key support

The dropdown should close if the ESC key is pressed. If focus was inside of the list, it should return to the button trigger.

Resolved with commit https://github.com/MelSumner/ember-a11y-dropdown/commit/563360239f4e6a284c54869794c18fce9876d985


Is it possible to change caption using youtubePlayerController?

No it's not possible as the Iframe Player API do not provide such functionality.


[Bug] [seatunnel.engine.server.rest] The submit task rest API cannot adapt to changes in the master

Search before asking

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

What happened

当使用 /hazelcast/rest/maps/submit-job 提交任务时服务端抛出空指针，因为org.apache.seatunnel.engine.server.rest.RestHttpPostCommandProcessor#handleSubmitJob 111 行 jobId是必须得，与官方文档不符合。

/hazelcast/rest/maps/submit-job 接口无法自适应，master节点的变动，接口使用者无法感知到。会导致接口调用返回异常“"Please don't get coordinator service from an inactive master node”

SeaTunnel Version
2.3.3
SeaTunnel Config
seatunnel:
  engine:
    history-job-expire-minutes: 1440
    backup-count: 1
    queue-type: blockingqueue
    print-execution-info-interval: 60
    print-job-metrics-info-interval: 60
    slot-service:
      dynamic-slot: true
    checkpoint:
      interval: 10000
      timeout: 60000
      storage:
        type: hdfs
        max-retained: 3
        plugin-config:
          storage.type: hdfs
          fs.defaultFS: hdfs://cdh1:8020 # Ensure that the directory                                                    has written permission
map:
    engine*:
       map-store:
         enabled: true
         initial-mode: EAGER
         factory-class-name: org.apache.seatunnel.engine.server.persi                                                   stence.FileMapStoreFactory
         properties:
           type: hdfs
           namespace: /tmp/seatunnel/imap
           clusterName: seatunnel-cluster
           storage.type: hdfs
           fs.defaultFS: file:///

Running Command
http://192.xxx.xxx.201:5801/hazelcast/rest/maps/submit-job/?jobId=111111112

{"env":{"execution.parallelism":"2","job.mode":"BATCH","checkpoint.interval":"10000"},"source":[{"plugin_name":"Jdbc","url":"jdbc:mysql://192.xxx.xxx.202:3306/bigdata?serverTimezone=GMT%2b8&useUnicode=true&characterEncoding=UTF-8&rewriteBatchedStatements=true","driver":"com.mysql.cj.jdbc.Driver","connection_check_timeout_sec":"100","user":"xxx","password":"xxx","query":"select * from user"}],"sink":[{"plugin_name":"HdfsFile","fs.defaultFS":"hdfs://192.xx.xx.xxx:8020","path":"/srv/data_universe/data_center/data_pool/DW_USER_DAY/2023-12-04/batch_xxxxx_from_mysql_1.txt","tmp_path":"/srv/data_universe/cluster_worker/tmp/seatunnel","file_format_type":"text","field_delimiter":"|"}]}

Error Exception
Please don't get coordinator service from an inactive master node

Zeta or Flink or Spark Version
No response
Java or Scala Version
JDK 1.8
Screenshots
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

Hi @letusgoH
the job id issue has fixed at https://github.com/apache/seatunnel/pull/5702
and submit in non master node is in this pr https://github.com/apache/seatunnel/pull/5950


How to add products via phpmyadmin manually/ How to access admin/vendor pages

How can I add products/blogs/categories

manually through phpmyadmin? And also, how can I access the admin/vendor page? Is there other ways to integrate other templates included in your project? I'm a newbie by the way. THANK YOU SO MUCH

@ruey-agcang
All you can add from /admin with username and password "admin". Admin vendor page must be enabled from /admin -> settings , and you will see login/register fileds at the top of your template.


Crash on Win32/Debug

Hi,
i get a crash when i start the app with win32/debug and close it without doing anything.
Call stack:

OK, Access violation occours only with TFrameStand from GetIt, using the GitHub component works fine.
Is GetIt component out of date?
Best regards
Dirk

The version on GetIt is a couple of commit behind. I will ask for an update soon (I'd like to add a new feature at least before asking for an update).
Thanks


Good code but problem with the add tokens to my account

same thing with me

yeah can someone fix it

paste the code from the .js file in to this website:
https://skalman.github.io/UglifyJS-online/
it will minify the code. Paste this code into a bookmark with the prefix "javascript:" (ex. javascript:minified_code_goes_here)
click on the bookmark when you want to use it and it should work
Don't use any of the global hacks that add things to your account since you can be banned for it, all other hacks are fine


Rename VarcharsTest to match Surefire naming pattern

Thanks, fixed

@electrum there are quite a few classes named XxxTest (especially in product tests world). You may want to update them too.

@findepi The only instances I could find were product tests. The root POM changes the Surefire configuration to only check for Test*, but I don't know if that also applies to product tests. If it does, that means those classes aren't being run today (which is the case for VarcharsTest).

We could

register Test* and *Test in surefire
have a check at runtime (should be easy with a testng listener loaded via service loader) that enforces test naming convention

This way, we would close the loop and make it highly unlikely to have tests which are not run.


Adding channel-based CEL validation

What type of PR is this?
/kind cleanup
/kind test
What this PR does / why we need it:
This adds the ability to specify per-channel CEL validation rules. At the same time, it adds a missing rule to Routes that ensures their parentRefs are unique.
This also takes the opportunity to move things around a bit:

hack/cel-validation -> pkg/test/cel
hack/verify-examples-kind.sh -> hack/verify-crds-kind.sh

Does this PR introduce a user-facing change?:
CEL validation now ensures the uniqueness of Route ParentRefs.

/cc @gauravkghildiyal @shaneutt

Wow this is really great! Thanks a lot
LGTM

/lgtm


Cleanup vestigial commented out commands in Makefile.

well this is vague, but there are some commented out lines within rules that should be cleaned up.


load org.apache.rocketmq.client.trace.TraceBean class throw Can not get local ip

TraceBean has a constant field LOCAL_ADDRESS , initialize by static method UtilAll.getIP()。The static method’s implementation ignore some ip address  (like:  start with 254 ) by private method UtilAll.ipCheck()  。Finally ，when ip address started with 254 will lead to Can not get local ip Exception.
java.lang.ExceptionInInitializerError: null
	at org.apache.rocketmq.client.trace.hook.ConsumeMessageTraceHookImpl.consumeMessageBefore(ConsumeMessageTraceHookImpl.java:58)
	at org.apache.rocketmq.client.impl.consumer.DefaultMQPushConsumerImpl.executeHookBefore(DefaultMQPushConsumerImpl.java:140)
	at org.apache.rocketmq.client.impl.consumer.ConsumeMessageConcurrentlyService$ConsumeRequest.run(ConsumeMessageConcurrentlyService.java:400)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
Caused by: java.lang.RuntimeException: Can not get local ip
	at org.apache.rocketmq.common.UtilAll.getIP(UtilAll.java:562)
	at org.apache.rocketmq.client.trace.TraceBean.<clinit>(TraceBean.java:23)
	... 8 common frames omitted
Caused by: java.lang.RuntimeException: Can not get local ip
	at org.apache.rocketmq.common.UtilAll.getIP(UtilAll.java:559)
	... 9 common frames omitted

It seems the ipCheck method has some problems. @geektao would you like to fix the error? @RongtongJin

I have no idea to fix ipCheck method. I just replace LOCAL_ADDRESS value  with the RemotingUtil.getLocalAddress() method. @vongosling

https://github.com/apache/rocketmq/pull/2376
I have submitted a PR to fix this issue, please help to review it. @vongosling


Automated cherry pick of #5600

Cherry pick of #5600 on release-5.24.

#5600: Affix comment input to bottom in

/cc  @bradjcoughlin

This has already been cherry-picked. In 5.24 the linting issues don't apply because the linting rules were changed upstream.


test/aborttest from 1.1.1-pre3 make test tickles Problem Reporting GUI

UI Problem Reporting tools are triggered by the test/aborttest, leading to GUI noise in what is otherwise a detached, stdio-redirected make test invocation.
Not sure if there is a workaround that can be automated. Observed on Fedora 27.

Not planning to fix this for 1.1.1. Not sure if there is a fix! Move to post 1.1.1 milestone for now.

I do not think there is a way to fix this. We would have to drop the test and I do not think we want to. Closing.


Microsoft Authenticator

It would be great if the "Google Authenticator" text can be updated to neutralize branding and include both Microsoft and Google Authenticator, as Microsoft Authenticator works the same;
ios: https://itunes.apple.com/sg/app/microsoft-authenticator/id983156458?mt=8
android: https://play.google.com/store/apps/details?id=com.azure.authenticator&hl=en

This is a good point!

@cssquirrel can we make this happen as your working on it at the minute?


[AIRFLOW-1115] fix github oauth api URL

Make sure you have checked all steps below.
JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-1115
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a JIRA issue.

Description

[x] Here are some details about my PR, including screenshots of any UI changes:
This PR fixes the oauth flow when github_enterprise host is "github.com" itself. In this case the API hostname is not github.com but api.github.com (https://developer.github.com/v3/#schema)
For all other cases, the existing behavior is correct (https://developer.github.com/enterprise/2.13/v3/#schema) and it is not affected by this PR.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
There doesn't seem to be any test for the auth backends and it is a small change. However please let me know if I need to add a test.

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[x] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Given that this is a github enterprise auth backend, it's a bit confusing to have code implicitly convert github.com to https://api.github.com which is GH, not GHE. Why not just set host=https://api.github.com in the config file to use this backend for non-enterprise GH auth.

@jgao54 Github uses github.com for the oauth calls and  api.github.com for the API calls (like fetching user info). This means that if I set host=https://api.github.com as you suggest, the oauth requests would fail.
It may be possible to create a new configuration variable like api_host or something similar but it seemed a little overkill just for this use case. Please let me know what you think.

@renzofrigato  understood now. thanks for clarifying, this lgtm !


mLabs sunsetting products -- what does this mean for the heroku add-on

What are we trying to do?
MLab is becoming part of MongoDB and as such its products will eventually need to be migrated to MongoDB Atlas.
MLabs announcement blog post here:  https://blog.mlab.com/2018/10/mlab-is-becoming-a-part-of-mongodb-inc/, relevant excerpt below:

What This Means For You, Our Customer

You will be given plenty of time to migrate, and nothing will be required of you for at least 4 months. We expect to have all customers migrated to Atlas within the next 12 months.
Once migrated, your Atlas database will be hosted on similar hardware and cost the same or less than your original plan on mLab.
We will provide tools that allow you to migrate with either minimal downtime or no downtime to your application.
All of this applies to users with mLab Heroku add-ons as well.

What feature or behavior is this required for?
required for database and basic app functionality
How could we solve this issue? (Not knowing is okay!)
Will eventually need to use a different add-on for the database. For now, can monitor the
situation and await guidance from Mongo/Atlas -- perhaps they'll provide an alternative add-on? Otherwise, would need to use a different add-on.

looked at this to see what was going on and here's what this came back with: https://docs.mlab.com/mlab-to-atlas/ probably nothing to do here yet.

I am in the process of actioning this migration. The sandbox instance has already been migrated to atlas, and the other instances will be migrated later this week. I will close this ticket once that happens.

It is done!


When the backend changes the action server UI doesn't reflect the changes

What happens is the following:
The user starts the action server with action-server start and then goes on to the builtin UI. At that point he changes the action and restarts the action server: the builtin UI will still be bound to the old settings (so, for instance if the api key is now needed or the arguments are different, the UI won't work properly).

This issue has been moved here, under Sema4AI/actions repository.


Removes the function body from the source info in case ByteBlock allocation hits OOM.

@atulkatti,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@Microsoft/chakra-developers
This has already been reviewed in codeflow.


MvxCachedImageView  binding issue

I am building a solution with Xamarin.Android. It uses MvxCachedImageView in MvxRecycler. While it is running on emulator with small pictures from emulator camera, it looks fine. Once I tested it on real phone with some real pictures, the app crashed after a few pages loading in the recycler.
When I check the debug log, I found that binding for DownSampleWidth is not working. This is the binding in the axml and the message in debug log.
    <ffimageloading.cross.MvxCachedImageView
        android:layout_width="@dimen/listitem_route_picture_image_size"
        android:layout_height="@dimen/listitem_route_picture_image_size"
      local:MvxBind="LoadingPlaceholderImagePath 'res:ic_image_grey_700_48dp.png'; ErrorPlaceholderImagePath 'res:ic_error_outline_grey_700_48dp.png'; Transformations Transformations; DownSampleWidth DownSampleWidth; ImagePath PicturePath" />

[0:] MvxBind:Warning: 31.15 Failed to create target binding for binding DownSampleWidth for DownSampleWidth
I thought it was an issue with my project at first. However, I found that the DownSampleWidth error also happens in the GitHub sample project. Scrolling down the recycler would also cause loading errors.
The crashing and loading error are most likely due to pictures are not downsampled.
My questions:

Would that be something missing in my environment that caused this binding issue?
Is it possible to specify TransparencyChannel in axml?
Is it possible to specify DownSampleWidth with resource dimension (@dimen/listitem_route_picture_image_size) instead of hardcoded width in pixel in the class object?

Thank you very much.

Hi @Nick1611,
What is the DownSampleWidth property type in your view model?

[0:] MvxBind:Warning: 31.15 Failed to create target binding for binding DownSampleWidth for DownSampleWidth

IMO it's rather an issue with MvvmCross than FFImageLoading.

Is it possible to specify TransparencyChannel in axml?

This property is depreciated, you should use BitmapOptimizations property instead. By default FFImageLoading disables transparency channel if it's not needed, but if the image has transparency channel, it's enabled.

Is it possible to specify DownSampleWidth with resource dimension () instead of hardcoded width in pixel in the class object?

Yes, just set DownsampleUseDipUnits = true, then density independent points are used

it is the same as in the github sample:
    public double DownSampleWidth => 200d;

Initially I tried to resolve it from the mvx issue direction too. All other bindings are good except this one. Later on, I found the same error in the sample project.

@Nick1611 Does it work when you disable linking?

Hi Daniel,
Linking was disabled!
I got a clue from your last answer and looked into the implementation of MvxCachedImageView. The field is defined as DownsampleWidth in MvxCachedImageView, not DownSampleWidth. It would be good to update the sample axml.
Thank you very much for your great support again. Can't get the same level even with paid service.

Nice catch! Thanks :)


python312Packages.google-cloud-bigquery-storage: 2.24.0 -> 2.25.0

Changelog: https://github.com/googleapis/python-bigquery-storage/blob/v2.25.0/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 311710 run on x86_64-linux 1

12 packages built:

python311Packages.dbt-bigquery
python311Packages.dbt-bigquery.dist
python311Packages.google-cloud-bigquery
python311Packages.google-cloud-bigquery-storage
python311Packages.google-cloud-bigquery-storage.dist
python311Packages.google-cloud-bigquery.dist
python312Packages.dbt-bigquery
python312Packages.dbt-bigquery.dist
python312Packages.google-cloud-bigquery
python312Packages.google-cloud-bigquery-storage
python312Packages.google-cloud-bigquery-storage.dist
python312Packages.google-cloud-bigquery.dist

Result of nixpkgs-review pr 311710 run on x86_64-linux 1

12 packages built:

python311Packages.dbt-bigquery
python311Packages.dbt-bigquery.dist
python311Packages.google-cloud-bigquery
python311Packages.google-cloud-bigquery-storage
python311Packages.google-cloud-bigquery-storage.dist
python311Packages.google-cloud-bigquery.dist
python312Packages.dbt-bigquery
python312Packages.dbt-bigquery.dist
python312Packages.google-cloud-bigquery
python312Packages.google-cloud-bigquery-storage
python312Packages.google-cloud-bigquery-storage.dist
python312Packages.google-cloud-bigquery.dist


feature request: multi-publish

Hi,
this looks like a great tool to do multi-crate publishing. What I mean: I have a project where a lot of crates live within the same repository (imag). All of these crates depend on each other (well, it is a tree-ish structure, not a circle of course). I released my first version (0.2.0) some days ago. I did it with this script:
while true; do
   for dir in *; pushd "$dir"; cargo publish; popd; done
done

(not exactly that script, but you get the gist of it)
Another possibility would be to manually find the dependency tree and do it by hand. Or, and that's what I'm suggesting: using a tool for it which internally gets the dependency tree of all crates to work on and cargo publish them in the right order.
TL;DR: allow me to cargo multi publish a set of crates that might depend on eachother and optimize this process to publish them in the right order (so, crates at the root of the dependency tree first, others later).

Interesting idea. It means that multi publish will need to be special case. Nonetheless, it worth exploring


When using a + for a format, a completely different error appears.

So when I use ['-f', 'bestvideo'] or ['-f', 'best'], everything works fine. But when using ['-f', 'bestvideo+bestaudio'] when the + symbol is used, this completely unrelated error occurs:
url.js:154
    throw new ERR_INVALID_ARG_TYPE('url', 'string', url);
    ^

TypeError [ERR_INVALID_ARG_TYPE]: The "url" argument must be of type string. Received type undefined
    at Url.parse (url.js:154:11)
    at Object.urlParse [as parse] (url.js:148:13)
    at processData (C:\Users\Neeron Bhatta\Documents\Coding\Hobul Downloader\Server\test\node_modules\@microlink\youtube-dl\lib\youtube-dl.js:67:17)
    at getInfo (C:\Users\Neeron Bhatta\Documents\Coding\Hobul Downloader\Server\test\node_modules\@microlink\youtube-dl\lib\youtube-dl.js:120:46)
    at done (C:\Users\Neeron Bhatta\Documents\Coding\Hobul Downloader\Server\test\node_modules\@microlink\youtube-dl\lib\youtube-dl.js:273:12)
    at done (C:\Users\Neeron Bhatta\Documents\Coding\Hobul Downloader\Server\test\node_modules\@microlink\youtube-dl\lib\youtube-dl.js:50:12)
    at ChildProcess.exithandler (child_process.js:285:7)
    at ChildProcess.emit (events.js:198:13)
    at maybeClose (internal/child_process.js:982:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:259:5)

Umm, help?

Same problem here :(

Anyone know a work-around?

I believe it's not possible currently with this library. The reason being, that the original CLI creates 3 files, where as the JS library pipes the first one, then gets confused as to why there's more. Someone will need to build that feature directly into this library.
If you need a workaround, you could download Best Video and Best Audio, and then combine them with ffmpeg. A good example of this is https://github.com/fent/node-ytdl-core/blob/master/example/ffmpeg.js

It seems to be a problem when use something like bestvideo+bestaudio the youtube-dl should download 2 files, video and audio separated, but the lib only works with one single file for download.
The error occurs on processData method.
const headers = Object.assign(
    { Host: url.parse(item.url).hostname },
    data.http_headers
  )

When using the format with + the item.url is undefined.
I also tried to download it with youtubedl.exec, the download works but the callback is called before the conversion finishes with error in stderror:
ERROR: unable to rename file: [Errno 2] No such file or directory

And the error message receive was:
Error: Command failed with exit code 1: /...project_path.../node_modules/youtube-dl/bin/youtube-dl --print-json --recode-video=mp4 http://www.youtube.com/watch?v=s4UhQ4hZzRM

What am I trying to do?
I was trying to download this video: http://www.youtube.com/watch?v=s4UhQ4hZzRM in 1080p mp4 format.
When I use format=best[height<=1080] it returns the video in 720p resolution even it has a 1080p resolution.
When I use --recode-video=mp4 --format=bestvideo[height<=1080]+bestaudio with youtube-dl on CLI, it works with the video in 1080p and MP4 format. The youtube-dl download an webm video and a m4a audio and merges into a .mkv file, then finally converts it to mp4.


Access-Control-Allow-Origin should have a valid Origin

Currently, Access-Control-Allow-Origin in the response has the origin with the trailing slash, which is not a valid origin format, and may never match the one the browser put in the request, and thus arise a blocked by cors policy in the browser.
Ref: https://fetch.spec.whatwg.org/#http-access-control-allow-origin

Fixed by #127


ColoredConsoleTarget - Fix bug in handling of newlines without word-highlight

Resolves #3917
Bug was introduced with #3018, because Ansi-coloring required extra handling for newlines.

thanks!


M3: Remote Attestation Registry Module

The ra-registry module serves the purpose of maintaining a register of remote attested substraTEE-worker enclaves.
register(signing_pubkey, call_encryption_pubkey, ra_quote_from_IAS)
This function is called by the worker when the enclave is set up or after each SW update modifying the MRENCLAVE
The module has to parse the ra_quote and make sure it is valid by

validating IAS signature
validating quote: acceptance should be configurable. In the end we only allow fully patched SGX running in release mode, but for debuging we'll allow debug mode as well and maybe even unpatched SGX
dispatches an Event `WorkerRegistered(signing_key, encryption_key,
counterpart supplying the quote: https://github.com/scs/substraTEE-worker/issues/18

revoke(signing_key)

TBD

registration lifetime
RA quotes are valid for a configurable amount of blocks. Cleaning up however is a task that demands running through the entire registry, which isn't something we should do in runtime for performance reasons. We might want to use a substrate-offchain-worker for this. Because such a worker may take as long as it takes to return moreover it can access chain state directly. Another option is to use a substaTEE-worker for this, but accessing chain state is more difficult.
open questions

should we also register information about where to reach a worker (IP or Kademlia addr).
Answer: Probably we can let workers promote their services themselves. users then just verify their keys using the ra_registry
how can we convince the user that ONLY the correct wasm is running on that worker enclave?
Answer This is up to the enclave TCB to ensure: The user has to supply the hash of the wasm code with every call
how do we add/revoke RA services keys (adding new manufacturers/attestation services keys) -> srml-democracy?

Hints:

check out how sawtooth verifies IAS quotes: https://github.com/hyperledger-labs/private-data-objects/blob/fce68acd5e51043b54ce2fdf7509e568f5402552/sawtooth/common/pdo_signing.py#L401
it might be difficult to parse an IAS quote in a runtime module, as no string tools are included in rstd. We might need to implement our own host call for this (similar to blake2_256, which is not executed in wasm but a host-provided call)

Additional function:
list_enclaves()
which returns a list of all registered enclaves, incl. their address

We will actually need the URL for the worker service to be in the registry as well.

A requester looks up the enclave registry where he finds the worker enclave's signing pubkey along with the workers URL.
The requester then requests the shielding key from the worker via WS (currently in M2 this key is hard-coded). This request must include a nonce to avoid replay.
The worker returns the shielding pubkey and the nonce, signed by the enclaves signing pubkey
the requester validates the signature and can then encrypt his request tx payload with the newly obtained shielding key
After invocation, the client verifies the counter has been updated by requesting the counter state from the worker (depends on https://github.com/scs/substraTEE-worker/issues/19)

Implemented partially with M4 (https://github.com/scs/substraTEE-node/releases/tag/M4).
The exchange of the shielding key does not implement a nonce and the keys are exchanged over a TLS connection without additional encryption.


my compiler cann't compile code in Point.cxx:28

In point.cxx file, there are invalid operands。

the error log is followed
./src/Point.cxx:27: error: invalid operands to binary expression ('std::stringstream' (aka 'basic_stringstream<char, char_traits, allocator >') and 'std::string_view' (aka 'basic_string_view'))
After I used name.data() to handle it, the porgram throw a unhandled exception from ucrtbased.dll

I don't know if it's because I don't have a local database

Could you please describe your env? OS, compiler, etc? I need to reproduce it.

Could you please describe your env? OS, compiler, etc? I need to reproduce it.

thank you, I compiled this on the Windows 10 platform, the compiler is MSVC 2017 x64，C++ 17
And when I set up my local database Settings,no longer throws exceptions


Clarify the flags for compiling local versus production

Currently, when you compile using the compile.py script, you must pass in a --target flag. However, this is confusing as target commonly refers to an output directory. In this case, it refers to the relative path set for links within the site so that a user can debug the site locally and still have it work correctly when published to GitHub pages.
While the behavior of what the switch does shouldn't change, the flag to switch it should. Instead of -t/--target, a user should pass no flag to compile for local and a --prod flag to compile for GitHub pages.

Taking a look at this.
Hopefully commenting on this is enough to assign it to me.


Lita should be able to connect to multiple chat services at the same time

I'd like to be able to run the same robot on both IRC and Campfire (and ideally any number of other services). The robot should share persistence across both services.
For example, using lita-karma:

Campfire-me> something~~
Campfire-Lita> something: 0
IRC-me> something++
Campfire-me> something~~
Campfire-Lita> something: 1

@jimmycuadra
I want to implement bot that communicate few users between each other. But each user can use different adapter. For example one user uses Telegram, next one slack, but at the same moment they should have the same context (karma extantion, as example). How I can do it by the best way?


Allow merging multiple XMLs

This question was asked on stackoverflow but I feel it would be a good addition to the cli.
Post:  https://stackoverflow.com/questions/54901871/merge-nunit3-xmls-into-one-html-for-publishing-to-jenkins

You can track this enhancement here


InvocationTargetException when running unusedCode

When running unusedCode from sbt shell in our codebase we always get:
[error] (Global / unusedCode) java.lang.reflect.InvocationTargetException

with no additional output, for every sbt module. Is it some known issue/limitation of the current version of the plugin? If not, do you know if there is some nice way to track down the real reason behind it? I've tried sbt --verbose --debug but it doesn't help much in this case. I don't have any minimal yet, but I can try to create some

please try set ThisBuild / scalafixOnCompile := false or remove scalafixOnCompile := true

Thanks. I've tried set ThisBuild / scalafixOnCompile := false (we don't use scalafixOnCompile := true), but the issue still remains, unfortunately. Also, we've got a bunch of plugins in project/plugins.sbt - I've disabled them all, except sbt-native-packager, but it didn't help either

I'm seeing the exact same thing on the latest version 0.2.0

I made a minimal example to reproduce this issue. https://github.com/tkawachi/testunused

I made a minimal example to reproduce this issue. https://github.com/tkawachi/testunused

https://github.com/xuwei-k/testunused/runs/7384036712
Could not reproduce.

When I ran unusedCode from IntelliJ IDEA's sbt shell on Windows, the problem occurred, but not when I ran it from WSL's sbt. Hmmm. I wonder what it is?

I thought IntelliJ IDEA sbt shell was suspicious, so I tried IntelliJ IDEA sbt shell on Linux as another environment, but the problem didn't occur.


support importing .ts videos

Sweet! Thanks @gitnew2018! 🎉
Thanks for keeping them alphabetical! 😄


ADC: Add support for Microchip MCP3204/MCP3208

These commits adds support for the MCP3204/MCP3208 12-bit ADCs from Microchip.

DT stuff looks good to me.


How to increase the toolbar font size and validation not working in firefox browser.

I'm submitting a
Bug report
Description

For validation added  [required]="true" but initially br tag added in firefox browser because br tag prevents validation.
How can i increase my toolbar font size issue.

Browser
Firefox - x.x.x

Any solutions you got?

Not yet...can you give fix for that?
Thanks & Regards,
Rabin Jude G

From: S A R<EMAIL_ADDRESS>Sent: Wednesday, July 25, 2018 2:17:52 PM
To: Sibiraj-S/ngx-editor
Cc: George Rabin Jude; Author
Subject: Re: [Sibiraj-S/ngx-editor] How to increase the toolbar font size and validation not working in firefox browser. (#125)
Any solutions you got?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Sibiraj-S/ngx-editor/issues/125#issuecomment-407681519, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Aa_bDsXVXWBTP1ipyqzvXnFH5RitVwh2ks5uKDC4gaJpZM4T67q1.
NOTICE: This e-mail message and any attachment to this e-mail message contains confidential information that may be legally privileged. If you are not the intended recipient, you must not review, re-transmit, convert to hard copy, copy, use or disseminate this e-mail or any attachments to it.  If you have received this e-mail in error, please immediately notify us by return e-mail and delete this message.  Please note that if this e-mail message contains a forwarded message or is a reply to a prior message, some or all of the contents of this message or any attachments may not have been produced by the sender.  This notice is automatically appended to each e-mail message leaving the senders e-mail domain. Thank you.

Use ngx-editor2 which is a fix for this

.fa {
font-size: 1.4rem !important;
}
Add this style to change the toolbar size


Initial release of sync script for all platforms

Two versions; one for use within Task Scheduler, one for use on an ad-hoc basis with manual entry of api key, credentials and API URL

Merging pull request in

Thanks @mikenelsonjr