Help with build-librubberband.sh

Is your feature request related to a problem? Please describe.
Adding audio stretching library rubberband.
Describe the solution you'd like
rubberband filter enabled and available in runtime
Describe alternatives you've considered
atempo is available but produced audio quality is lower
Additional context
I am following how other libs were added (x264, opus etc), however rubberband seems to have different structure. I am using https://github.com/breakfastquay/rubberband/blob/default/otherbuilds/Makefile.linux but fail to compile ffmpeg.
Happy to make a PR once this is working.
build-librubberband.sh:
#!/bin/bash

set -euo pipefail
source $(dirname $0)/var.sh

# clean
rm $BUILD_DIR/lib/librubberband.a -f

LIB_PATH=third_party/librubberband
CONF_FLAGS=(
  --prefix=$BUILD_DIR                                 # install library in a build directory for FFmpeg to include
  --host=i686-gnu                                     # use i686 linux
  --enable-shared=no                                  # not to build shared library
  --disable-asm                                       # not to use asm
  --disable-rtcd                                      # not to detect cpu capabilities
  --disable-doc                                       # not to build docs
  --disable-extra-programs                            # not to build demo and tests
  --disable-stack-protector
)
echo "CONF_FLAGS=${CONF_FLAGS[@]}"

LIB_PATH1=third_party/libsamplerate
(cd $LIB_PATH1 && \
  emconfigure ./autogen.sh && \
  CFLAGS=$CFLAGS emconfigure ./configure "${CONF_FLAGS[@]}")
emmake make -C $LIB_PATH1 clean
emmake make -C $LIB_PATH1 install

LIB_PATH2=third_party/libsndfile
(cd $LIB_PATH2 && \
  emconfigure ./autogen.sh && \
  CFLAGS=$CFLAGS emconfigure ./configure "${CONF_FLAGS[@]}")
emmake make -C $LIB_PATH2 clean
emmake make -C $LIB_PATH2 install

export CPATH="${BUILD_DIR}/include"
emmake make -C $LIB_PATH -f otherbuilds/Makefile.linux clean
emmake make -C $LIB_PATH -f otherbuilds/Makefile.linux

cp $LIB_PATH/lib/librubberband.a $BUILD_DIR/lib
cp $LIB_PATH/rubberband.pc.in $EM_PKG_CONFIG_PATH/rubberband.pc
sed -i -e 's,%PREFIX%,'"$BUILD_DIR"',g' $EM_PKG_CONFIG_PATH/rubberband.pc
echo "Requires: samplerate" >> $EM_PKG_CONFIG_PATH/rubberband.pc

cp $LIB_PATH/rubberband/RubberBandStretcher.h $BUILD_DIR/include
cp $LIB_PATH/rubberband/rubberband-c.h $BUILD_DIR/include

I can see rubberband in build dir, however running configure-ffmpeg.sh with --enable-librubberband fails with
ERROR: rubberband >= 1.8.1 not found using pkg-config

which is confisung because I can see rubberband in emconfigure pkg-config --list-all
rubberband.pc
prefix=/home/jozefchutka/ffmpeg.wasm-core/build
exec_prefix=${prefix}
libdir=${exec_prefix}/lib
includedir=${prefix}/include

Name: rubberband
Version: 1.8.2
Description:
Libs: -L${libdir} -lrubberband
Cflags: -I${includedir}
Requires: samplerate

ended up overriding makefile see https://github.com/jozefchutka/rubberband/blob/yscene/otherbuilds/Makefile.em

ended up overriding makefile see https://github.com/jozefchutka/rubberband/blob/yscene/otherbuilds/Makefile.em

Hi, this is no longer available, could you please update the link here?
Also, is rubberband available at this time in the latest stable build or do I have to do something extra with the build for that?
Thanks for the answer.


Improve SEO

@barrasso please review

@barrasso cool. Updating and submitting

@kfichter @barrasso just updated. Please review. If you need me to take the SEO to the next level. Please let me know.


Jurassic Unity3D Asset Store

Hi,
I have a question.
I noticed that the last released version at Unity Asset Store was Nov 28, 2011.
What do you advises me? Is this version at Unity Asset Store reliable or I should build a new .dll using the master at paulbartrum/jurassic Github?
Thank yo so much. 👍

I honestly didn't even know it was on the Unity Asset Store. There have definitely been a lot of bug fixes since 2011, but on the other hand the author of the unity package may have made changes to make it work better on Unity. So... hard to say.


Implement UISizeDependent protocol

Description:

Implement UISizeDependent protocol
Make Numeric type conform UISizeDependent protocol
Add necessary conversion logic to CGFloat

Why do we need develop and main branches instead of just main?

Removed develop branch


OpenAI-DotNet 6.5.3

Added missing ConfigureAwait to await calls

p.s thanks for making this library - its awesome.

@SJMakin did you see any other places we're not using configure await calls?

Apologies for the build issue. Push a change, but looks like I just missed the boat!

BTW there are some redundant awaits in your overloads in the endpoint classes.

Thanks again for this lib x


Failed to load the latex package

[Enter steps to reproduce below:]

Open atom
CMD+O to open a folder. I have a reactjs project that was working as intended yesterday, and with no code changes atom now crashes. Very odd.

Atom Version: 1.1.0
System: Mac OS X 10.10.3
Thrown From: Atom Core
Stack Trace
Failed to load the latex package
At Cannot read property 'prototype' of undefined

TypeError: Cannot read property 'prototype' of undefined
    at __extends (<anonymous>:1:1)
    at <anonymous>:5:3
    at Object.<anonymous> (<anonymous>:5:3)
    at Object.<anonymous> (<anonymous>:1:1)
    at Module._compile (module.js:428:26)
    at Object.defineProperty.value [as .coffee] (/Applications/Atom.app/Contents/Resources/app.asar/src/compile-cache.js:169:21)
    at Module.load (module.js:353:32)
    at Function.Module._load (module.js:308:12)
    at Module.require (module.js:363:17)
    at require (module.js:382:17)

Config
{
  "core": {},
  "editor": {
    "tabLength": 4,
    "softWrap": true,
    "invisibles": {}
  }
}

Installed Packages
# User
fonts, v0.6.0
language-latex, v0.4.1
language-opencl, v0.1.1
latex, v0.14.0
merge-conflicts, v1.2.8

# Dev
No dev packages

Please update the latex package - the current version is 0.28.0 and you're using 0.14.0 :wink:.


add service-control action and fixed issue #715 - [closed]

In GitLab by @liyanq528 on Dec 7, 2017, 07:51
Merges 20171205 -> dev

In GitLab by @wknet123 on Nov 13, 2019, 10:15
the build is FAILED console log

In GitLab by @wknet123 on Nov 13, 2019, 10:15
the build is FAILED console log

In GitLab by @wknet123 on Nov 13, 2019, 10:15
the coverage is  7.7% check console log

In GitLab by @wknet123 on Nov 13, 2019, 10:15
closed


GamePixSDK

Description
Integrate GamePix SDK into your game!
How to use the extension
Integrate GamePix SDK into your game!
Checklist

[X] I've followed all of the best practices.
[X] I confirm that this extension can be integrated to this GitHub repository, distributed and MIT licensed.
[X] I am aware that the extension may be updated by anyone, and do not need my explicit consent to do so.

What tier of review do you aim for your extension?
Community (Unreviewed)
Example file
GamePixSDK Example.zip
Extension file
GamePixSDK.zip

@Luis06YT this looks very interesting!  Let us know if you need any assistance or guidance.

@Luis06YT Thank you for your extension!
This extension is interesting and opens new possibilities on GamePix, we will make a maxiumum to have it integrated in GDevelop as soon as possible to provide the GamePix SDK to all GDevelop users.
Here my review, there is small things to adjust but the overall is functional and well made.

The minified code of the SDK in onFirstSceneLoaded is unreadable, to ensure clarity and transparency of source code we'd ask for a human friendly version of the SDK.

Like the bot commented above, some descriptions in fucntions miss a dot at the end of the sentence. e.g:

The SDK is loaded in the global scope of the document, we strongly suggest to move it in the gdjs scope, like how it's done for the Poki extension to ensure the viability and access by the engine to the SDK.
See how is plugged the Poki's SDK:

gdjs._pokiGamesSDKHtmlExtension = {
    isSdkReady: false,
    isPlayingCommercialBreak: false,
    commercialBreakJustFinishedPlaying: false,
    isPlayingRewardedBreak: false,
    rewardedBreakJustFinishedPlaying: false,
    shouldRewardPlayer: false,
};

const logger = new gdjs.Logger("Poki games SDK HTML");
function addScript(src) {
    return new Promise((resolve, reject) => {
        const scriptElement = document.createElement('script');

        scriptElement.setAttribute('src', src);
        scriptElement.addEventListener('load', resolve);
        scriptElement.addEventListener('error', reject);

        document.body.appendChild(scriptElement);
    });
}
addScript('https://game-cdn.poki.com/scripts/v2/poki-sdk.js').then(() => {
    PokiSDK.init().then(() => {
        gdjs._pokiGamesSDKHtmlExtension.isSdkReady = true;
        logger.log("Poki SDK successfully initialized.");
    }).catch(() => {
        logger.log("Initialized, but the user likely has adblock.");
    });
})

Expression function name should start by a verb when possible, for instance:
verifyRAd => IsRewardedAdWatched
Functions name should have a meaning name related to the description.
verifyAd => IsAdEnded
intertitialAd => ShowIntertitialAd
etc

I'll let someone else from the @GDevelopApp/extensions-team check behind me if I didn't missed some points.


Another approach with multiple workflows.

I thought about the original way, which read git commit message from CLI and then use it in workflow logic conditions, but it seems impossible, since:

The logic conditions is processed in compilation time, not run time.
git log wont work without checkout the source.

In this workaround,

I created a setup workflow which used to get the commit message and then pass it to the build workflow.
In the build workflow, we check if the git commit message and then decide whether to proceed the job.

https://app.circleci.com/pipelines/circleci/qHXqJBvUMYDAjmxWMtFb8/2a96639b-8816-46f8-bf48-f77dba6fbeb1?branch=release-3

OK, so that actually works. Thanks for being so persistent on this. I will do some more tests and work it into Hugo's setup early this week.


Unable to determine asterisk version, assuming Asterisk 13... you should expect problems to follow.

core show version:
Asterisk certified/16.8-cert2 built by...
In your code:
private static final String VERSION_PATTERN_16 = "^\sAsterisk (GIT-)?16[-. ].";
Should be:
private static final String VERSION_PATTERN_16 = "^\sAsterisk (GIT-)?(certified/)?16[-. ].";
OR:
private static final String VERSION_PATTERN_CERTIFIED_16 = "^\sAsterisk certified/(GIT-)?16[-. ].";

Could you push a pr with the change.
On Wed, 1 Jul 2020, 10:48 pm AhmadHamdanMontyMobile, <
<EMAIL_ADDRESS>wrote:

core show version:
Asterisk certified/16.8-cert2 built by...
In your code:
private static final String VERSION_PATTERN_16 = "^\sAsterisk
(GIT-)?16[-. ].";
Should be:
private static final String VERSION_PATTERN_16 = "^\sAsterisk
(GIT-)?(certified/)?16[-. ].";
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/asterisk-java/asterisk-java/issues/334, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAG32OGK3VGAZFW7TOZMJZ3RZMWBDANCNFSM4ONMDVFQ
.


Added extra dependencies

Implemented extra dependencies as discussed in #1766.
This can be tested locally via the command pip install -e .[extras] by replacing extras with:

nospam: requests dependencies for custom requests and testing

requests_cache, requests_ratelimiter

repair: for price repairing functionality

scipy

Maybe also add to #1084 that nospam is required to run tests and link the Installation instructions in the readme.

Regarding the naming, I think nospam makes sense concerning development because it's only needed for testing. However, since users could also install via the pip cmd to make custom requests (as in the readme), should we change it to reflect its use case?

should we change it to reflect its use case?

Isn't the use case same? Reduce spam on Yahoo.

should we change it to reflect its use case?

Isn't the use case same? Reduce spam on Yahoo.

The way I interpreted the readme 'Smarter scraping' section was that the use case of requests_cache is a custom way to make requests via modifying User-agent. I guess there's also caching and the uses of requests_ratelimiter include the rate-limiting to avoid triggering Yahoo's blocker, so nospam makes sense.

The way I interpreted the readme 'Smarter scraping' section was that the use case of requests_cache is a custom way to make requests via modifying User-agent.

That's a README problem so my fault. To keep it concise I combined 2 examples:

custom User-agent on a session
replace requests with requests-cache

Maybe User-agent example should be removed.

Maybe User-agent example should be removed.

Should we change/remove it in the readme for this PR or just keep as is for now?

Can lower scipy minimum to 1.6.3.

Should we change/remove it in the readme for this PR or just keep as is for now?

I don't mind if in this PR. As you had confusion, probably you're best to decide whether/how to change.

Can lower scipy minimum to 1.6.3.
I don't mind if in this PR. As you had confusion, probably you're best to decide whether/how to change.

Added these changes in my latest commit.

I've made some 'personal preference' tweaks to README. If no complaint, then I'll merge.

I've made some 'personal preference' tweaks to README. If no complaint, then I'll merge.

Looks good to me!


Attester Slashing 5116

https://medalla.beaconcha.in/validator/5116
Eth1 address: 0xf689d779d9108FAa0Ba447399Fb2477687923278
No proposed blocks, so no graffiti

This guy (Lamboshi aka Nolan#0383) is also a mod at the ethstaker discord server.

I will DM him

Whoa super sleuths nice find. I was just rushing to set up all 5 clients last night and was not careful about copy pasting keys between clients. Definitely will not be doing that on mainnet.


notifications: add request action notifications

actions: send notification on request accept action
:heart: Thank you for your contribution!
Description
closes #339
Send notification when executing a request action.
Checklist
Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

[ ] I'm aware of the code of conduct.
[ ] I've created logical separate commits and followed the commit message format.
[ ] I've added relevant test cases.
[ ] I've added relevant documentation.
[ ] I've marked translation strings (for relevant code).
[ ] I've followed the CSS/JS and React guidelines (for relevant code).
[ ] I've followed the web accessibility guidelines (for relevant code).
[ ] I've followed the user interface guidelines (for relevant code).
[ ] I've identified the copyright holder(s) and updated copyright headers for touched files (>15 lines contributions).
[ ] I've NOT included third-party code (copy/pasted source code or new dependencies).

Third-party code
If you've added third-party code (copy/pasted or new dependencies), please reach out to an architect.
Reminder
By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

You license your contribution under the same terms as the current repository’s license.
You agree that you have the right to license your contribution under the current repository’s license.

First implementation for the accept action. Open for input (especially on the template).
Idea is to reuse the RequestActionNotificationBuilder for other actions and only change the type, in order to fetch a different template and make it easily customizable.
Tests will be added once the implementation details are specified :handshake:

closing in favor of https://github.com/inveniosoftware/invenio-rdm-records/pull/1447


Should ILDG read require checksum?

Giovanni has this issue that an ILDG reader of his will not be read by our code. This is because that configuration has no checksum, which our code looks for.
Looking again at the ILDG specification, I am surprised to see that a checksum is not mandatory!
My intuition is that we should try to rewrite the reader to compare checksums only if there is one upon reading. Otherwise collaborations like Giovanni's, who have configurations that technically follow the ILDG standard, won't be able to use our code easily. What do you all think?

[ ]  Allow it to read without checksum, have it print a warning to screen that there's no checksum.

addressed with 63928f1


MRS100 hardware 2.0.0

Hi,
since I have some "old" device with old hardware version, even if I update firmware through meross app, the latest version I can download is 2.1.4, and, I think cause of hardware version, native position cannot be implemented on it. On older meross_lan version, with signal close and open, all worked fine.
The problem is that on current meross_lan version, the signal open/close cannot be set anymore, so is set to 50s as default.
Can it be enabled again, or maybe detects hardware version, and don't use native position on them but signal close/open?
EDIT: from logs, seems like the configuration is not saved to device correctly, it returns always the default 50s.
mrs100-1695595965.csv

Update: I tried also using cloud configuration, and even with that the configuration of signalOpen and signalClose doesn't work, if made from HA, works only if made from official meross app. At this point I suppose they add some kind of authorization to update device config, or they save it on the cloud so on get device config, the getack returns data from the cloud. I think the only way to make it working on local mqtt configuration, is to save device config locally, and don't let getack config data overwrite it.
I can upload logs for local or cloud test if need any.

In the next days I will try to sniff app's http payload (since that is the only one working when trying to update device config) with albertogeniola's sniffing tool, to see if something changed. I'll update this post if I'll found something.


Fun

I feel like there could be ways to make the game more engaging to the players

Can you identify a couple of ways?

First, I suggest the introduction of Variability Cards, such as Wildcard and Challenge Cards, to inject a thrilling element of unpredictability and active participation into the gameplay. These cards would require players to navigate dynamic changes or tackle specific scenarios and tasks, fostering a more immersive experience.
Additionally, recognizing the importance of staying current, I recommend updating our prompts and responses to reflect contemporary and relatable AI scenarios. This update would not only make our game more pertinent but also encourage deeper, more meaningful discussions among players about the real-world implications of AI technologies => I proposed some extra lines that I added in the prompts.csv and responses.csv files and I think inviting players to contribute new content by editing these files can keep the game fresh and evolving.
To add a competitive edge and urgency, implementing timed rounds could also significantly enhance the game's pace, compelling players to think and act quickly. Moreover, introducing role-playing elements, where players assume various personas or roles, could further deepen engagement and offer diverse perspectives on the scenarios presented.
I also edited the README.md file to add the new features and propositions in the pull request.


fix: hide focus pointer when the associated element is out of view in welcome tour

Description
We hide the onboarding indicator when the ui element being indicated is scrolled out of view.
Fixes #10749
Fixes #10925
Fixes #10752
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Manually
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:red_circle: Total coverage has decreased
// Code coverage diff between base branch:release and head branch: fix/focus-pointer

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
total
56.71 (0)
38.66 (0)
36.22 (-0.01)
56.96 (0)

:red_circle:
app/client/src/pages/Editor/GuidedTour/utils.ts
13.59 (-1.97)
0 (0)
12.5 (-0.83)
14.58 (-1.51)

:green_circle:
app/client/src/selectors/commentsSelectors.ts
85.25 (1.64)
64.71 (2.95)
73.33 (0)
90.59 (2.35)

:red_circle:
app/client/src/utils/WorkerUtil.ts
89.76 (0)
70.59 (-1.96)
100 (0)
93.33 (0)

:green_circle:
app/client/src/utils/autocomplete/TernServer.ts
52.94 (0.23)
41.67 (0.84)
36.21 (0)
56.99 (0.25)

/ok-to-test sha=15e5a8a

/ok-to-test sha=c50da3d


Azure Bot Service Tag IP Restriction for Web App

Looks like from Azure portal, it shows much more service tags can be configured for Web App Inbound Access Restrictions.
Can you please confirm if can use "Azure Bot" service tag to limit the incoming traffic from bot service only?

[Enter feedback here]

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

ID: d90bbf62-44f9-30a6-d6ea-bbf776f6efc5
Version Independent ID: b3961eac-6c70-b345-f48c-e4c0ec67059c
Content: Azure App Service access restrictions - Azure App Service
Content Source: articles/app-service/app-service-ip-restrictions.md
Service: app-service
GitHub Login: @ccompy
Microsoft Alias: ccompy

@RonnieQuan Thanks for the question. We're investigating this and will get back to you shortly.

@RonnieQuan Yes. As mentioned in the docs, by default, service tags reflect the ranges for the entire cloud. Some service tags also allow more granular control by restricting the corresponding IP ranges to a specified region.
So, the service tag AzureBotService allows you to filter inbound traffic from Azure Bot Service for the entire cloud, but AzureBotService.EastUS narrows the range to only the storage IP address ranges from the EastUS region. Use the tag that fits your requirement appropriately.
Hope that addresses your query. We will now proceed to close this thread as resolved, but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


Difference betweem the quality on reader player me hub and the talkinghead output

Hello @met4citizen  , i remark the difference of quality between the model created on reader player me hub itself (preview)  and the one talking head is fetching .
From rpm hub :

With talking head example code :

Please How to fix that ?
This is the model id : 66f55316e9a27f067c0d0e04

I don't believe there are differences in model quality. The Ready Player Me Hub just uses different lighting and other settings.
Here is a screenshot of your model from the TalkingHead test app, with a dark background and some added lighting.

If you'd like to use TalkingHead's built-in lights (like I did here) you can modify the lighting using the following options: lightAmbientColor, lightAmbientIntensity, lightDirectColor, lightDirectIntensity, lightDirectPhi, lightDirectTheta, lightSpotColor, lightSpotIntensity, lightSpotPhi, lightSpotTheta, and lightSpotDispersion. You can find more details about these options in the project's Readme.

Got it mika , thanks


Issue 175 indefinite integral

Description
Implements the indefinite integral for the cases which we require (i.e. integrate current to get potential).
Fixes #175
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Key checklist:

[x] No style issues: $ flake8
[x] All tests pass: $ python run-tests.py --unit
[x] The documentation builds: $ cd docs and then $ make clean; make html

You can run all three at once, using $ python run-tests.py --quick.
Further checks:

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works
[x] Any dependent changes have been merged and published in downstream modules

Codecov Report

Merging #265 into master will decrease coverage by 0.2%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master     #265      +/-   ##
==========================================
- Coverage   97.96%   97.75%   -0.21%
==========================================
  Files          52       52
  Lines        2950     2991      +41
==========================================
+ Hits         2890     2924      +34
- Misses         60       67       +7

Impacted Files
Coverage Δ

pybamm/discretisations/discretisation.py
100% <100%> (ø)
:arrow_up:

pybamm/spatial_methods/finite_volume.py
97.1% <100%> (+0.23%)
:arrow_up:

pybamm/spatial_methods/spatial_method.py
91.3% <66.66%> (-8.7%)
:arrow_down:

pybamm/expression_tree/unary_operators.py
96.03% <70.58%> (-3.97%)
: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 9720cf0...e593067. Read the comment docs.


Could not load services and user information

Describe the bug
Error message "Could not load services and user information" at the lower end of the Ferdi UI
To Reproduce
Steps to reproduce the behavior:

Start Ferdi
After a while the message appears
Open View > Developer Tools, Network tab
See a lot of HTTP 500 status errors going to the following resource

/opt/Ferdi/resources/app.asar/api/utils/auth.js:1

When I click on the Application tab I see the following query:

select * from `users` where `id` = 1 limit 1 - SQLITE_ERROR: no such table: users

In the Console tab there are more details of the error:

<IP_ADDRESS>:45569/v1/me:1
Failed to load resource: the server responded with a status of 500 (Internal Server Error)

/opt/Ferdi/resources/app.asar/stores/lib/CachedRequest.js:1
Uncaught (in promise)
Response
body:ReadableStream
bodyUsed: true
headers: Headers {}
ok: false
redirected: false
status: 500
statusText: "Internal Server Error"
type: "cors"
url: "http://<IP_ADDRESS>:45569/v1/me"

Expected behavior
No Error 500, message should go away.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Ubuntu GNU/Linux 18.04.3 LTS
Browser Chromium; Firefox
Version 5.4.0-beta.4

Looks like there is a problem with the database of the internal server.
The file is located inside the application userdata folder (~/.config/server.sqlite on Linux). Could you please check:

if the file if readable and writable
that the file is >=41KB in size (to make sure it isn't empty)

Could you also please try renaming the database file (so the internal server will create a new one) and restart Ferdi? Just to check if the problem is in the database file.

Had the same problem. After update Franz server was replaced with Ferdi server.
@bittner check if you have https://api.meetfranz.com or https://api.getferdi.com in your settings.

The ~/.config/Ferdi/server.sqlite file is 57344 bytes of size.
The content of that file is:
>>> c.execute("SELECT name FROM sqlite_master WHERE type='table' ORDER BY name;").fetchall()
[('adonis_schema',), ('recipes',), ('services',), ('sqlite_sequence',), ('workspaces',)]

As for the settings, I had https://api.getferdi.com, I removed that and tried again and now I'm locked out as it seems (I'd have to start over with the accounts). The import at http://localhost:45569/import is also not available. :worried: Setting the value to https://api.meetfranz.com doesn't help.

I managed to us the Franz server again by:

setting https://api.meetfranz.com as Server in the settings
log out (restarting alone doesn't help)
log in using the Franz server credentials

This may seem logical, but it's not 100% intuitive. -- Well, I'm an idiot: It's even written in the user interface, just below the text field. :roll_eyes:
Maybe we can help idiots like me by either asking whether to log out immediately (popup?) or by highlighting the hint below the text field after the text field content was changed.

Anyway, the original problem from above still persists (e.g. when I choose "Use Ferdi without an Account" on the login screen).
I noticed, the import is now available from a public URL via a link on the login screen. nice!

Somehow the problems I face I somewhat related to workspaces. Or at least that feature is severely affected. I originally added another Slack team and wanted to add that to a specific workspace. That didn't work and the above mentioned messages started coming up (or maybe they were there before, I don't exactly remember). Now I figured the entire Slack team was gone; I added it again and tried to add it to the workspace; result: some error message; then I saw all the workspaces were gone. :face_with_head_bandage:

I looked through the changes between beta.2 and beta.4 again and I think I found your issue:
In order to implement the user settings feature, we had to add the table users to the local database.
New users would have this table in their database but as you already used the "Use Ferdi without an Account" feature on beta.2, your local database wan't updated. We'll have to look into building a migration assistant for the internal server that will automatically update the local database as needed.
For now, if you know how to execute SQL commands in an SQLite database you should be able to simply run this command to add the missing table:
CREATE TABLE `users` (`id` integer not null primary key autoincrement, `settings` text, `created_at` datetime, `updated_at` datetime)

I was on beta2 and upgraded and had this. When I checked there was a users table and even after I created one per the above comments, I still get this error.

@markwalkom Can you try renaming the database file just to make sure the problem is in the database migration and not the code.

That "worked", but it removed all my accounts :(

Of course, it was just for testing.
To you mind sharing your original database with me for testing? The database contains your services but no service content like your messages. Otherwise I'll try to find the error together with you over GitHub.

Yeah I can do that, I am https://keybase.io/markwalkom if that helps?

@markwalkom Can you please revert the move of your old database again (so Ferdi is using your original database) and execute this SQL command in the database:
INSERT INTO  "users" ("id") VALUES ('1');

and see it it works again?

That worked @vantezzen! :D

Ferdi is not only expecting the table "users" to be in the database but also a row with the user ID of 1 to be existent - this row will be used to save the settings.
For this update you'll have to run these two commands yourself, for future updates we'll have to create a migration assistant when making changes to the database structure.

Ferdi internal-server will now ship with a database migration wizard that should prevent this error from happening in future versions.

Just updated from 5.4.0 to 5.4.1-beta.1 and lost all my services. I've tried logging out, restarting, no dice. If I view my account on https://api.getferdi.com/user/data, all my services are there and intact. So, I'm not quite sure what's going on or how to fix it. Any ideas?


Support for 2D systems

I'm not sure at the moment the effort involved to create corresponding System2D, Link2D, Motor2D etc. It would be nice to support this to allow for rapid prototyping on simple toy examples. I will do a quick check to see the effort, but if you have any insights or guidance here, let me know.
I'm still keeping an eye on DRAKE because I think it might be perfect for more advanced systems later on. But I think this library will still be useful from a learning perspective for other students to dip their toe into trajectory optimisation, before having to become familiar with a full fledged robotics toolbox.
What you think?

So I'm looking at implementing the bare minimum for 2D support - that would be Link2D, Motor2D, System2D. However, there is nothing 3D specific to System3D besides the plotting. I'm thinking that we have a base class SystemBase which both System2D and System3D inherit so that we minimise code duplication. That said the Link2D is also pretty similar to the Link3D and again plotting is the main difference (as well as the construction). Could think about a LinkBase too. I haven't looked at Motor2D yet but I'm thinking that should just be a simple torque variable about the one possible axis of rotation.

Awesome, I just skimmed some of the commits and they look good! I feel bad every time I see a "fix bug" relating to code I put there :face_with_spiral_eyes: hope the library isn't being frustrating to work with
Hah, I didn't know the cost function for my entire msc too. What you're doing sounds really cool though! I don't have a real reason why I didn't add 2D versions of everything. I guess I thought it would be a lot of work, and I wasn't planning on using it myself.
It would be awesome if you could make a model in 2D, get it working, rapidly figure out the cost function/useful constraints/etc, then change a few lines and have it working in 3D (with the same/similar cost). Nice work!

Awesome, will let you know if I do!


update travis

@bradjasper Your comment on #115 made me think. The 1.8a1 will probably be removed from pypi as soon as the final version is released.

I honestly have no idea why the django 1.8 installation fails on python 2.7 and 3.2 but works on 3.3 and 3.4.
It works fine on my local machine with python 2.7.

Thanks @yannik-ammann — I've noticed some random failures related to pip downloading before. Not sure if that's related, but appreciate the contribution!


Limit maximum width of editor settings popup

This is a quick "fix" to https://github.com/godotengine/godot/issues/6437
Settings popups were incredibly wide since they were simply opened with a ratio relative to the editor windows size. I kept that ratio, but limited the width to 800 editor units (it felt like a nice trade off between lengthy property values and trackability of them back to their names).
Applies to project and editor settings popups.
Before:

After:

IMO, it's too small now. A bit wider would be ok.

@neikeq Maybe add a setting for this until popups get resizable?

@RayKoopa Settings, settings, settings everywhere, would be cool if there are less, since people start getting lost there :laughing:

Sidenote: I don't really see the use of a smaller editorsettings/projectsettings popup -- both are important enough to be big.

@bojidar-bg At the width of the first screenshot it gets almost impossible to track the property name to the value. The screen is too big to see what value belongs to what line. I had several positive opinions about my change in Discord so I gave it a try :D Of course it would be better to have resizable popups / column widths, but we don't have that at the moment.

2¢, IMHO, workarounds are useless, as they have to be reverted later on anyway. Of course it would be better to have resizable popups and columns and so on, but (still IMHO) we shouldn't monkeypatch it constantly.

Yeah, I understand. I better edit settings in a text editor because at the moment it's just eye straining. Maybe I can look into the popups :3

Usability states, "too much white space makes your eyes get lost."

Yeah, but too little, and it feels cluttered.

Is 800px too little?
Your eye should not get lost when going from one side to the other. That's all.
Please suggest a better solution :)

A better solution is suggested in the linked issue.

What about alternating coloring of rows? That would allow to keep the wide window while improving the trackability.
Maybe such a thing could be implemented as a property for certain widgets.

Good idea. This issue is kinda outdated anyway as I implemented resizable dialogs. Just need to get merged so I make the editor remember your window pos / sizes.

I think alternative row colors will be a nice to have ... also might be a
newbie task thing?
On 5 February 2017 at 20:52, Ray Koopa<EMAIL_ADDRESS>wrote:

Good idea. This issue is kinda outdated anyway as I implemented resizable
dialogs. Just need to get merged so I make the editor remember your window
pos / sizes.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/pull/7543#issuecomment-277540029,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AT59_VziaYhpQEpVTn0MCUiw-Rv7h_g_ks5rZhpagaJpZM4Lj-cv
.

@ClikCode Yeah, open seperate issue, and we will tag it as such :smiley:

Superseded by #7929.


Add TopologySpreadConstraints in PodTemplate

Changes
Resolves #5137.
This PR aims to add TopologySpreadConstraints in PodTemplate to enable spread Pods across cluster among topology domains.
/kind feature
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Docs included if any changes are user facing
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
Added TopologySpreadConstraints in PodTemplate to enable spread Pods across clusters among topology domains.

/kind feature

/retest

/ok-to-test

/test check-pr-has-kind-label

/test pull-tekton-pipeline-alpha-integration-tests

Thank you @seongpyoHong
/approve

/lgtm

/retest

/retest

/retest


Android: Focusing a Text Input does not open Android on screen keyboard

Tried to use the demo on an Android phone, but the keyboard is not shown when focusing a Text Input.

Yea, that one is a bit annoying. I'm not actually sure how to resolve that one. I think it'll be a combination between the lib and some js in the html page hosting the app.
If you have any suggestions on how to resolve this I'd appreciate it!

Just a heads up. I've been playing with this one. I have a wip up https://ebitenui.github.io/textinput/

@scratch85 I wrote some code that opens a hidden text box via Javascript and then copies the inputs from that textbox into the ui lib. It's still a little "shaky" but seems to work.
https://github.com/ebitenui/ebitenui/blob/master/internal/jsUtil/util_js.go

I believe this is working now. I went ahead and cleaned up the change so you can now select the type of keyboard you want to open on  a mobile device! Please open a new issue if you find problems with this change =)  I'll be kicking out a new numbered release soonish but you can pull it by the latest commit. Thanks!


Guid values are not supported

Originally reported https://github.com/microsoft/kiota/issues/4335
https://github.com/std-uritemplate/std-uritemplate/blob/95b7aceaddf3c0ccc02d0e8cf4d7c2a90b423098/csharp/Std.UriTemplate.cs#L470
https://github.com/std-uritemplate/std-uritemplate/blob/95b7aceaddf3c0ccc02d0e8cf4d7c2a90b423098/java/src/main/java/io/github/stduritemplate/StdUriTemplate.java#L403
https://github.com/std-uritemplate/std-uritemplate/blob/95b7aceaddf3c0ccc02d0e8cf4d7c2a90b423098/typescript/src/index.ts#L398
etc...
Are not converting a guid/uuid value to string. While we could implement that conversion in kiota abstractions, it'd be more consistent to do it here.
Happy to implement that myself, or to hand it over to somebody else if there are any volunteers.

No super strong opinion on where we should do it, I'm ok with doing it here if anyone volunteers to do the job.
Probably we should have a "list of supported native types" in the Readme though ...

I think there should be if with calue == Guid and that's it

I think there should be if with calue == Guid and that's it

Correct, but:

for all the languages
tested as we test the native dates

yep, exactly what Andrea said.
Are you planning on taking this on @illunix ?
Thanks for the help!

For the date tests, here is what we're referring to

@baywet I wanted to have a grasp of the issue and drafted #144
I think we should re-evaluate the situation:

There are only 4 languages with a "Native UUID/GUID" type in the standard library (C#, Java, Python, Swift)
There is no Guid native type in Typescript as far as I understand it (e.g. this won't fix the originally reported issue)

At this point, I tend to think that we should fix this in the abstractions libraries, wdyt?

Go : the vast majority of applications are using this implementation https://github.com/google/uuid
TypeScript: the community is a bit more spread out, if we don't want to make an opiniated choice, we could just test for the present of "toString()" and call it when available.
dart: I don't know anything about dart, sorry
PHP: kiota is using this https://github.com/ramsey/uuid not sure how widely adopted it is but nobody has complained so far.
Ruby: kiota is using this https://rubygems.org/gems/uuidtools not sure how widely adopted it is
Swift: type is native

I believe we could chose to implement it here based on that analysis. The only languages (besides dart) where I could see people complaining about the choice potentially are PHP and Ruby.
But then again, if you disagree, I won't push too hard on this one, and defer it to kiota abstractions.

I'm usually in the camp of "break hard so it's easy to tell something is wrong", but I've received the feedback multiple times this approach can be a little drastic. So I'm open to have my mind changed on the topic.
I'll go ahead and open bugs in the kiota abstractions repositories so they can be fixed as time allows.
Once I'm done, I'll close this one as not-planned.
Thanks for the input here.

break hard so it's easy to tell something is wrong

Agreed, let's keep going with the current approach unless we have strong reasons to change our minds 👍

@illunix thank you for your patience while I bounce you around the repositories here as we're trying to identify what the right course of actions is.
The next issue you'll want to engage on is this one https://github.com/microsoft/kiota-typescript/issues/1113
Please reach out there, I'll try to provide you with guidance so you can PR it and you have a fixed version quickly.


Is this project still alive?

Hi Ivan,
just wondering if this is still alive and if it has got anywhere close to ready for use?
Ta
Peter.

Hi @pnunn ,
well the original app I got it to make some simple tests, and I didn't take into account get it work for some bigger projects. But currently I'm working on coding a base library to get it work for other projects.
Are you needing some directions about the protocol or what do you have in mind?
Greetings,
Ivan

@IvanGaravito I have a requirement where I need to read about 500 values, once in a minute. Can you help me out?

Hi @dushyantbangal,
by this time you'll have to wait the first stable version. Currently the library is still under development, so please wait.
Greetings,
Ivan

Hi,
I would like to know if the DNP3 library is stable and ready to use.
Thank you,
Eyad


style bug, on mobile app

online Game page:
when opening the screen keyboard on the mobile phone, the height of the page is changing and it doesn't look like how it should be

fix bug!
Commit:
https://github.com/dorfingerjonas/TicTacToe/commit/4fc7342755012e9465e450a65993608c6d8904fb


[WIP] Decouple DifferenceEquations dependency

The simulation should be downstream.

The downstream simulation and gradients should pass now with https://github.com/SciML/DifferenceEquations.jl/pull/17. But don't merge yet!

After a review (and possibly merge later) of SciML/DifferenceEquations.jl#17, I'll add the tests back here and we can merge this one afterward.


Confirm Wraith Brawn

This currently isn't used in Valkyrie, but the value should be confirmed.

Confirmed!


Add changelog

Would be great to see what has changed between versions, I'm unsure if my app will break while updating to 0.10.0
Thanks

Here's the changelog: https://github.com/vitalets/checklist-model/releases


2011 Districts Shapefile seems to have an island inside Odisha

See https://groups.google.com/d/msg/datameet/AZivLFyc-LQ/unNQ8gxoAgAJ

The "island" in this case is Kandhamal district, for which a shape is present in the 2011 districts shapefile. I think this issue can be safely closed.


Clipboard positioning fix

@chris-huggins Another one for you to review

@discodavey I'm getting 500 error when trying to open an article on https://pr-357--epp.elifesciences.org/

I'll update the branch and see if this fixes it @chris-huggins

@scottaubrey Getting a 500 server error on this. Please can you investigate.


Location of setsocketopt

I am by far not an expect, but I am confused why socket options are set here: https://github.com/johnmehr/gitup/blob/2547cfcaf9ddf20b4ebb9b6aac6b7b1c0fc7d56d/gitup.c#L853-L871 in setup_ssl() instead of in connect_server(). TLS is set up when the connection has already been established.

This was a bad copy/paste on my part.  How does it look now?

LGTM.


Fix Subsquare OpenGov API

Only request 1-- entries
Only call Subsquare API once when screen is opened
Do not wait for Subsquare  to return results and display referenda without metadata untill Subsquare results are loaded (to match iOS implementation)
#8694988u4

Task linked: CU-8694988u4 Android. Subsquare API fix


Fix the global access to the overridden font

Sorry. The previous bugfix does not seem to fix the original issue, as Moonscript has restrictions on global vars definitions. Since I had struggles to compile and run the project on Windows 11 (extremely awkward installation process for Luarocks and dependencies of the correct versions), eventually I had to get Linux and compile there. The fix is probably not the best example of behaving with globals, but it does its job. Please feel free to make it prettier.

Also, the latest Alfons 5.0.1 seems to have changed its bundled packages with updated APIs, so the build file looks as follows:
diff --git a/Alfons.moon b/Alfons.moon
index 00383c5..a760780 100644
--- a/Alfons.moon
+++ b/Alfons.moon
@@ -7,13 +7,13 @@ tasks:
 		sh "convert icons/icon.svg -resize 48x48 icons/icon.png"
 		sh "convert icons/icon.svg -resize 256x256 icons/icon.jpg"
 	clean: =>
-		fs.delete "vnds/" if fs.exists "vnds/"
+		delete "vnds/" if fs.exists "vnds/"
 	compile: =>
 		tasks.clean!
-		fs.copy "src/", "vnds/"
+		copy "src/", "vnds/"
 		for file in wildcard "vnds/**.moon"
-			shfail "moonc #{file}"
-			fs.delete file
+            sh "moonc #{file}"
+			delete file
 	run: =>
 		tasks.compile!
 		shfail "love vnds"

Thanks for the PR! As you can probably tell, I haven't had much time to invest in VNDS-LOVE recently so it has become a bit out of date with all the dependencies. Merging this in for now to fix the bug, but I might end up refactoring it a bit in the future.


refactor(emoji): remove didUnmount

Description
React community has a misconception about the warning for setState. So, we can safely remove the extra useRef.
I thought Raycast doesn't warn it, but after creating this PR, I noticed it's probably wrong.
If Raycast depends on react-reconciler, and if React doesn't release the new version of it yet, then we can't merge this PR yet.
Let me open this PR as draft to show best practice in the future.
Type of change

Bug fix / improvement for somebody else's extension. Extension author: @FezVrasta

Screencast

Checklist

[ ] I read the extension guidelines
[ ] I read the documentation about publishing
[ ] I checked that extension's icon has size of 512x512

Thanks for the PR! I wasn't aware of this

I changed the fix with FIXME, this should be mergeable.


Import from CSV a tick box to not auto create field lengths (best guess), set "Target type" as varchar(1024)

Is your feature request related to a problem? Please describe.
The problem is the guess gets it wrong at times. Some data is lot longer and import fails. Not sure which field(s) failed.
Describe the solution you'd like
Allow a test option by making a wide column to browse or even extra scan for widest.. find the cultprit data.
Describe alternatives you've considered
A present have to manually relook at data and trial and error this could be the field.. the data issue.
Additional context
From early use of this program it has improved. But last 2 our 4 CSV from Excel, took me good amount time to work it out.

We have an option to increase the amount of sampe rows that would be checked for maximum column lenght during import

Ok.. will check this out...   shown sampling defaults 100 records..  I could change it to say 12000..
Quick look at another file it auto changes to 1200..  got it..

@SherlockSoftware so the issue is solved for you?

It's been a long time since no update on this issue, so I suppose it's solved.


Updating Config Context Fails

I have a somewhat complicated config_context of a series of nested-dicts/lists. Updating this fails, even when updating it to the same values.
Example:
Existing Config Context JSON
{
    "bgp_overlay": {
        "neighbors": [
            {
                "description": "Switch1",
                "ip": "<IP_ADDRESS>"
            },
            {
                "description": "Switch2",
                "ip": "<IP_ADDRESS>"
            }
        ]
    },
    "bgp_overlay_rr": {
        "neighbors": [
            {
                "description": "Switch3",
                "ip": "<IP_ADDRESS>"
            },
            {
                "description": "Switch4",
                "ip": "<IP_ADDRESS>"
            }
        ]
    }
}

when attempting to update and save I get the following:
>>> test = nb.extras.config_contexts.get(name='test_config')
>>> test.data['bgp_overlay']['neighbors'].append({'ip': '<IP_ADDRESS>', 'description': 'new_switch'})
>>> test.save()
False

even writing to the same value fails:
>>> test = nb.extras.config_contexts.get(name='test_config')
>>> holder = test.data
>>> test.data = holder
>>> test.save()
False

I believe I have found out the root issue. Pynetbox interprets "bgp_overlay" and "bgp_overlay_rr" as duplicate keys. Changing either of those keys allows a save. This does appear to be a bug in pynetbox, as if .save() returns False then it never sends a PATCH to the netbox server.

Think that was fixed in https://github.com/digitalocean/pynetbox/pull/266. What version of pynetbox are you using?

pynetbox==5.3.1

I think the issue is that #266 has an update for "local_config_data" and does not include "config_context" as well.

Ah gotcha, will look into it.


Casdoor DB requires DROP access to be granted to my DB user

The user created for my database doesn't have access to DROP or ALTER OR DELETE for security reasons in my organization. At max we can add drop or alter or delete access for few tables but not for the entire server. Is there a way to bypass this error. If dropping the table is the only option what are the other tables to which drop access should be given ?
panic: Error 1142: DROP command denied to user '###############' for table 'casbin_api_rule' .
Need some help here. Thanks.

@praveenkumar1798 Could you provide more detailed steps to trigger the bug? For example, what operations were performed on what data?


Support/Allow Typescript 5 as a Peer Dependency

Great work on this tool! I was looking for something simpler than tsup and came across your simple yet powerful solution!
I have been playing around a bit, and it seems that Typescript 5 works just fine, so could we add that to the peer dependencies (like ^4.1 || ^5.0) so that we would not get warnings about that?
I ran into this specifically with PNPM, but I assume the same warning happens in other tools.

Thank you! And for sure, happy to accept a PR!

Closed via https://github.com/privatenumber/pkgroll/pull/35


Use common OIDC token env vars for identity live tests

Support oidc az cli login in post deploy scripts for multi-cloud. Requires https://github.com/Azure/azure-sdk-tools/pull/9099

/azp run net - identity - tests

/azp run net - identity - tests

/azp run net - identity - tests

API change check
API changes are not detected in this pull request.

@christothes I've been running the weekly in the background. Trying to find the right VM sku that will have capacity in our TME sub right now (my latest change to v5 didn't work).


porch should support cloning an abstract package even when fn render operation fails.

Sharing more background here:
Ghost application package has a validation function that helps user discover required inputs (hostname in this case) for the package. So cloning ghost package in deployment repo leads to an incomplete package with fn render failing reporting an expected failure.
One can reach this state while editing a package as well (imagine a mis-configuration).
So the expected behavior here is porch should allow saving (and cloning) the package state even in these mis-configuration scenario.

/cc @ChristopherFry

As we discussed earlier, it probably should block Proposing - but allow the cloning and saving. So it has to stay a Draft until it passes all validators.

As we discussed earlier, it probably should block Proposing - but allow the cloning and saving. So it has to stay a Draft until it passes all validators.

+1.


Support for x-amazon extensions for AWS API Gateway

We are looking to be able to generate an AWS API Gateway compatible swagger.json file which could be used to automate the setup of an API Gateway as part of our deploy process.
Curious to know if there is any plan to support the inclusion of the AWS API Gateway Extensions into the swagger file that the hapi-swagger extension outputs? As defined here: http://docs.aws.amazon.com/apigateway/latest/developerguide/api-gateway-swagger-extensions.html

@rossanthony this still needed?

Closing. Please reopen if necessary.


Playing as 'formable nation'

Hi,
Im not sure if this is already implemented but is there a way for the presence to update showing a formable nation such as Austria-Hungary? Currently, atleast for me anyway, when forming such countries the presence still says Playing as Kingdom of Hungary (or whicever nation formed). I thought this might have something to do with the name of the save created but no.
Another question is the flag of Austria-Hungary. The tag of AH is whichever country formed it eg. Hungary which will obviously cause an issue as HUN tag will provide the Hungary flag.
Many thanks.

Hello there!
Thank you for opening an issue. Let me address your points:

I'm not sure if this is already implemented, but is there a way for the presence to update, showing a formable nation such as Austria-Hungary? Currently, at least for me anyway, when forming such countries, the presence still says "Playing as Kingdom of Hungary" (or whichever nation formed it).

This feature is not yet implemented! In Hearts of Iron IV (HOI4), the game continues to display the country as HUN, even though its properties (like the name or focus tree) might change after forming a new nation.
It's a great suggestion, and I’ll pin your issue for further investigation. While I can't promise immediate implementation, I'll definitely look into it!

Another question is the flag of Austria-Hungary. The tag of AH is whichever country formed it, e.g., Hungary, which will obviously cause an issue as the HUN tag will provide the Hungary flag.

This is a similar issue. Since HOI4 treats Austria-Hungary as HUN in the game files, it keeps displaying the Hungary flag (associated with the HUN tag) instead of updating to the Austria-Hungary flag.

I thought this might have something to do with the name of the save created but no.

It's not related to the save name but rather the internal game data stored in the save file. Unfortunately, our project doesn’t currently handle this level of detail. If you're curious about how it works, feel free to check our wiki—though this specific information isn’t covered there yet.
Sowwyy about that!
Thanks again for your feedback and for bringing this to our attention! 😊


more than one texture per surface?

@jstoter gave the noise modelling example with different scenarios/params having diff results which become textures of the walls of buildings.

@kenohori @clausnagel
okay my proposal is as follows: a CityObject can have different "themes", either with materials or textures. The "material" is a JSON object with key-object pairs, the keys being the themes, eg:
"material": {
          "irradiation": {
            "values": [[0, 0, null, null, 1, 1]]
          },
          "irradiation-2": {
            "values": [[2, 2, null, null, 1, 1]]
          }
        }

and the values are just the references to the materials in the "appearance". "texture" has exactly the same behaviour with key-object pairs.
        "texture": {
          "winter-textures": {
            "values": [
              [ [[0, 10, 23, 22, 21]], [[null]], [[null]], [[0, 1, 2, 6, 5]], [[0, 2, 3, 7, 6]], [[0, 3, 0, 4, 7]] ],
              [ [[null]], [[null]], [[0, 0, 1, 5, 4]], [[0, 1, 2, 6, 5]] ]
            ]
          },
          "summer-textures": {
            "values": [
              [ [[1, 7, 3, 2, 1]], [[null]], [[null]], [[1, 1, 2, 6, 5]], [[1, 2, 3, 7, 6]], [[1, 3, 0, 4, 7]] ],
              [ [[null]], [[null]], [[1, 0, 1, 5, 4]], [[1, 1, 2, 6, 5]] ]
            ]
          }
        }

And to ensure that a viewer picks the right theme when >1, one can define the following in the "appearance":
  "appearance": {
    "default-theme-texture": "summer-textures",
    "default-theme-material": "irradiation",
    "vertices-texture": [

If not define then the viewer just picks whatever it wants.

✅ 4e61127d85874ed16af0193fa1444ea7c650b290
@jstoter

Sorry for re-opening. The null values for non-textured surfaces are not intuitive. And at least my Java Gson lib refuses to generate them automatically.
My suggestion is to simply use null instead of [[null]] in your example.
            "values": [
              [ [[0, 10, 23, 22, 21]], null, null, [[0, 1, 2, 6, 5]], [[0, 2, 3, 7, 6]], [[0, 3, 0, 4, 7]] ],
              [ null, null, [[0, 0, 1, 5, 4]], [[0, 1, 2, 6, 5]] ]
            ]

I have been watching these discussion roll by, and I am no expert in the development of such formats. However, maybe I can just pitch in here. I would assume that models will often have many surfaces with no texture. For example, with irradiation, you may only want these textures on the glazed areas. In such cases, endless lists of null values might not be desirable.
An alternative may be to explicitly list the surface indices for each set of values rather than relying on the array position.

I don't think [null] or [[null]] make much sense. Since those arrays don't have the number of vertices as their corresponding ones in the geometry object, a developer cannot assume that they have the same structure. So, I'd second @clausnagel's suggestion. Going back to a discussion with @hugoledoux some time ago, maybe we can take it one step further and say that arrays consisting solely of null objects (e.g. [null, null, null] or [[null], [null]]) can always be flattened to null.

Using null instead of `[[null]]`` is indeed better. As Ken points out, that means that we have a "hierarchical representation" in a way: the null propagates to the other primitive "below".
This is one idea Ken and I discussed a few months ago, but we left it aside because the implementation is more complex. But perhaps time to bring it back now. It would solve most of the issues I reckon (@phtj: I dislike have new IDs to refer to the primitives... sorry) and have more compact files.
So the idea is that for "material" and "semantics" one can put a value at the level-higher in the hierarchy of arrays and that value propagates below. Imagine for instance that one Solid has all the same material "blue, which is the 7th in the array of appearance, then instead of enumerating for each surface the value 6, it can be put once:
"material": {
          "mycolours": {
            "values": 6
          }
  }

The same applies to null and for semantics with Semantic objects.
I've built in c++ a quick demo with a snippet showing how to extract the material/texture/semantics for a single geometry: https://github.com/tudelft3d/cityjson/blob/hierarchical-arrays/hierarchy-tests/main.cpp
Everything working. @kenohori tried for null with Ruby and it's also super simple.
What do you think?

If I get your hierarchical idea right, then wouldn't it de-facto just make sense for "material"?
For example, isn't it rather unlikely that all surfaces of a "MultiSurface" or "[Composite|Multi]Solid" of a building share the same semantics? And if all values of a "texture" property are null, then wouldn't it be more straight forward to completely omit "texture" instead of assigning one null value?
For "material", I see the benefit (similar to CityGML, where you can assign an app:X3DMaterial to an aggregate surface such as gml:MultiSurface). But I agree with your concern that the implementation becomes more complex. I prefer using Json binding libs (like Gson) over having to manually parse the Json. In the binding world, you would have to deal with the fact that "values" can be both an integer and an array. Since this is not possible (well, at least in my Java world), you have to inject some manual parsing code. Doable, but more complex.
So if my understanding is correct, then perhaps adding one additional "value" property to "material" might be an alternative. If there is only one material for all surfaces, one can pick "value", and otherwise "values".

Btw, I support the proposal of @kenohori to flatten arrays containing solely null values to a single null.

I'm also in favour of the null flattening, but it breaks the schema since the depth of the arrays is prescribed for the different Primitives... One solution is to modify the schema just so that it's only a JSON object, and write code to to further validate (cityjson-valschema does this to some extent already). Or perhaps there's another way to define this in a schema, but I am not aware and would have to investigate this further.
Also, a few points from the discussions above:

semantics could also be a lot of the same values sometimes I reckon, think of semantics for WaterBody for example, where the river is triangulated... but agreed it'll be rather rare.
gson doesn't have a function that gives you the type you have at runtime? like json["geometry"].is_array() or similar?
if the hierarchical arrays proposal is too complex (I'm ambivalent on this one I must say, it is more complex but if the 2 functions I wrote are translated in a few languages then it's done, no need to reinvent them), then I like the "value" for "material"" very much.

So yeah, let's do a poll I guess!

No opinions so far? Well, here is my two cents: I was not aware that using null instead of [[null]] would break the schema. So personally I rate schema validation higher than a flattened array representation.
I wonder about the proposed change to the "values" property of "material". Is it possible to define "values" in the schema such that it can either take an integer or an array as value? If not, then I would also vote against this change :-)
Regarding Gson: The idea is that you define a Java class which represents a Json object and whose fields represent the properties of this object. The serialization and deserialization is then done automatically, so you do not have to fiddle around with Json elements. But this requries unique data types. As mentioned above, you can also write custom code in Gson like in your example in order to handle non-default cases.
Btw, I have completed a first CityGML-to-CityJSON support in citygml4j (against the current v0.3 schema). It should (hopefully) work for all sorts of building representations in CityGML. So I am happy to produce example CityJSON files which might be helpful in studying the layout of CityJSON datasets - just let me know. The changes are not yet on GitHub. I first have to implement a clean API and also want to work on CityJSON-to-CityGML.

We were busy thinking about and testing what flattening means in terms of implementation in different programming languages. Putting implementation first might be a philosophical difference with respect to CityGML 😉.
 There are conditional evaluations that can be called even if something is null (?. and &. respectively). With flattened values, we do need a series of ifs at every level, but the code is relatively easy in all languages we tested. In theory, any language with type introspection (or a library that provides it) should work.
Hugo and I also looked into how to redefine the schema to allow for arrays of varying depth. I think it should work by defining a series of custom types at every level of the hierarchy. I'm in favour of this, but it does make the schema more complex...
Regarding the Gson serialisation, I reckon you could implement a custom (de)serialiser for the array types. Something like this:
https://stackoverflow.com/questions/19588020/gson-serialize-a-list-of-polymorphic-objects

@kenohori Thanks for your response.  In fact, I have already implemented the flattened array approach in my CityJSON support in parallel to the v0.3 schema. Your link to the custom Json adapters is exactly what I meant with custom code in my previous post.

It seems that we decide here, and for me either way is good. It seems for you too Claus, and Ken too. If we're ambivalent, I'd then go for the flattened array approach because:

better to introduce this early in the process. Otherwise if we wait and then at v2.0 (coming soon btw!) then it's too big of a change
only deserialisation need to be supported in a way, writing CityJSON can be with an arrays of the same depth
we all agree that for null values it is needed and good. Thus the schema need to be modified anyway. The current schema that defines the depth of each 3D Primitive doesn't work, but we can modify with a few tricks. The issue is that, just looking at the schema, perhaps it'll less expressive then the text of the specs. But the validator "cityjson-valschema" will have a proper validation. The question is: what is the CityJSON specs, the text or the schema? With JSON I reckon the text has to have more weight...

If you approve this (with a thumbs up) then I'll figure something out for the schema (or @kenohori will, I recall you promising :)).
And it's fantastic if citygml4j works, I do want some files yes, mine are all rather hacked.

with v0.5 the Semantic Surfaces are handled "better", and there is no need for hierarchical arrays. And "texture" can't have it either, so that leaves only "material"... I have decided, after an unanimous vote with myself, that the hierarchical arrays should be skipped. However, the idea of @clausnagel of having "value": 2 for the "material" if the the whole geometry has the same material has been implemented in v0.5.


Use default file

Some tracks lessen the barrier for users by providing an empty implementation file, which prevents the user from having to create the file themselves. An example of such a track is the Elm track, but the Scala track is also discussing this. How do you all feel about this? I think it would indeed lower the barrier, especially if we will also be providing exercise-specifc .csproj files.

I think it would help, I thought this was how they all worked as I've otherwise done python, elixir, and elm.
I was just coming to report on this, and to volunteer to generally make this easier for linux users such as myself. I completed the hello world exercise using dotnet core, and added nunit to a project file I created such that 'dotnet test' ran the tests. I'd like to investigate having that work for others, but I need to ramp up on both csharp and exercism a bit more first.

@ckhrysze I am currently waiting on the .NET Core CLI to be finalized. Once that is released, I will be converting all exercises to run on .NET Core, including a project for each exercise that contains an empty implementation file. Thanks for the feedback!

@ErikSchierboom when you say that you're waiting for the .NET Core CLI to be "finalized" what do you mean? I thought 1.0 has already been released.

@robkeim Well, the .NET Core CLI has been released, but they are still preview releases (not even beta releases thus). Alongside the CLI work is the MSBuild work, which will replace the current .NET Core project.json format. This work is also not done. Once both of these are done, we will be able to do some very cool stuff such as:

Create very minimal project (.csproj) files for each exercises, which makes it easier for users to get started.
Have the exercises run on .NET Core, which makes the cross-platform support much better and allows us to have better instructions.
.NET Core allows us to have people build and test the projects on the command-line.

Adding a default file to those project files (this issue) is already technically possible, but I think this would fit in better with the above rewrite.

Ok that makes sense, thanks for the explanation!

This was the post I was looking for (I added this comment to #183) before finding this one.
@ErikSchierboom It looks like the CLI tooling is no longer in preview... finally :)
https://www.visualstudio.com/en-us/news/releasenotes/vs2017-relnotes

Updated .NET Core and ASP.NET Core - No longer a preview workload. Fixed several bugs and improved usability of .NET Core and ASP.NET Core tooling. Project migration from project.json/xproj to csproj is much more reliable. The “Configure Continuous Delivery...

.NET Core tooling is now automatically included in “ASP.NET and web development” workload and an optional component in “.NET desktop development” workload in the installer. It is also available as a workload called “.NET Core cross-platform development” under “Other toolsets” section in the installer. We have fixed several bugs and usability issues for .NET Core and ASP.NET Core tooling. The csproj project format has been simplified and project migration from project.json/xproj to csproj is much more reliable.

@robkeim I'm busy porting the C# exercises to Xunit (on a branch of mine). I'll then work next on creating the new .csproj files.

Sounds good @ErikSchierboom!

I'm working on this in the .NET Core branch.

While working on this, a question arose. I am creating a minimal skeleton file for each exercise. That means the user can focus on the actual implementation, instead of the boring and repetitive creating of files. For simple exercises, I think this is a huge win. Consider the following default file for the acronym exercise:
using System;

public static class Acronym
{
    public static string Abbreviate(string phrase)
    {
        throw new NotImplementedException("Please implement this function");
    }
}

Looks clean to me, and it will let the user focus on the implementation. However, do we also want to create stubs for any required constructors? I'm leaning towards yes, but what do you feel? @balazsbotond @robkeim @jwood803 @bressain?
Would you like this:
using System;

public class Anagram
{
    public string[] Match (string[] potentialMatches)
    {
        throw new NotImplementedException("You need to implement this function.");
    }
}

or this:
using System;

public class Anagram
{
    public Anagram (string baseWord)
    {
        throw new NotImplementedException("You need to implement this function.");
    }

    public string[] Match (string[] potentialMatches)
    {
        throw new NotImplementedException("You need to implement this function.");
    }
}

Note that the tests require that specific constructor, so we could supply it instead of having users use ReSharper or Visual Studio to generate it for them.

Note that for some exercises, it might make sense to do require the user to specify the constructors. An example of this is the dot-dsl exercise, for which I would create the following stub:
public class Node
{
}

public class Edge
{
}

public class Attr
{
}

public class Graph
{
}

You can check my progress here: https://github.com/ErikSchierboom/xcsharp/tree/default-file

@ErikSchierboom I'm kind of in the "no stubs" camp. I think it's fine to include a stubbed out file for the first few exercises but after that, I feel there is value in the user creating files from scratch. Yes it's repetitive but repetition is how you learn the language and I wouldn't want to take that away from the user.
That said, I understand that decision may have already been made which is fine since I've been mostly MIA. My opinion however, is if we're going all in on stubs, let's keep it to a minimum so that we're not being too prescriptive.

@bressain Thanks for chipping in! We had the same sort of discussion on the F# track. We can leave this issue open, as we can easily remove stubs if more people agree with your sentiment :)

To @bressain's point, I think that we definitely need stubs for the first few exercises in order to reduce friction and on-boarding to the track.
Given the fact that we're providing the tests we're already being fairly prescriptive in terms of function composition, naming, etc so I don't feel the stubs should take out much of the "meat" of the exercises as opposed to just reducing a few lines of boilerplate code.
(Disclaimer: I'm just getting back from vacation and catching up on everything, so this thinking hasn't taken everything that's happened in the past few weeks into consideration)

I understand @bressain's point, but it kinda depends on who you think is your target audience. For people completely new to C#, it might make sense to have them create the files at some point. However, for people already having some C# that want to hone their C# skills, it doesn't really bring anything to their table I feel. Also, when you get to the later exercises, you must already have some C# skills otherwise you wouldn't get there. For those people, which thus are relatively proficient, creating the stubs wouldn't be a problem, but they also wouldn't learn anything from it.
I've also done worked on several other language tracks, with some tracks providing stubs (Haskell), some providing stubs for some exercises (Java) and some not providing any stubs (F#). It is probably no surprise, but I really like the Haskell track where a stub was provided for me. The Java track at some point didn't provide me with a stub, but all I did was to use my IDE to generate the stub for me. That did not teach me anything useful, expect how to use my IDE.
So I do get the point @bressain is making, and it is a good one, only I feel that the previous points give the stubs approach the edge.

With #199 merged, this issue can now be closed! 🎉


Is there any pre-trained model to use for?

hello, sir. Maybe need bpe.model & model.pt to run decode.py, right? May u provide a demo file for testing?

sorry, i missed the HF repo,
https://huggingface.co/frankyoujian/Edge-Punct-Casing/tree/main

https://k2-fsa.github.io/sherpa/onnx/punctuation/pretrained_models.html#sherpa-onnx-online-punct-en-2024-08-06-english-only
Please also have a look at the above link.

@XhrLeokk I have integrated this model to K2 repo, you can refer to PR to sherpa-onnx for deployment.


Specify the libraries after source files

otherwise, if the linker links the libraries "--as-needed", they won't be linked at all.

Thanks!


[Bug report] Can't get another bean instance from bean factory, despite of the docs

Describe the bug
Documentation says:

By the way, you can even lookup other beans within the factory:

di.RegisterBeanFactory("beanID", Singleton, func(context.Context) (interface{}, error) {
	return di.GetInstance("someOtherBeanID"), nil
})

But it fails to initialize a container.
panic: container is not initialized: can't lookup instances of beans yet

goroutine 1 [running]:
github.com/goioc/di.GetInstance(...)
	/tmp/gopath719623559/pkg/mod/github.com/goioc/di@v1.7.1/di.go:499
main.main.func1({0x5a0440?, 0xc000016b40?})
	/tmp/sandbox1767667263/prog.go:16 +0xc7
github.com/goioc/di.createSingletonInstances()
	/tmp/gopath719623559/pkg/mod/github.com/goioc/di@v1.7.1/di.go:414 +0x8cc
github.com/goioc/di.InitializeContainer()
	/tmp/gopath719623559/pkg/mod/github.com/goioc/di@v1.7.1/di.go:104 +0xa5
main.main()
	/tmp/sandbox1767667263/prog.go:22 +0xb1

Program exited.

See example:
https://go.dev/play/p/K-dlscM8hvi
package main

import (
	"context"

	"github.com/goioc/di"
	"github.com/samber/lo"
)

func main() {
	bean1 := "value1"
	_ = lo.Must(di.RegisterBeanInstance("bean1", &bean1))
	_ = lo.Must(
		di.RegisterBeanFactory(
			"bean2", di.Singleton, func(ctx context.Context) (interface{}, error) {
				innerBean1 := di.GetInstance("bean1").(*string)
				bean2 := *innerBean1 + "!"
				return &bean2, nil
			},
		),
	)
	lo.Must0(di.InitializeContainer())
}

Good catch! Indeed, looking up another bean works only if the scope for the bean factory is Prototype. It would fail with Singletone (which makes sense). I've updated the readme 👍

Ok, @dtitov, thanks for the information.
Is there a way to build a bean factory for a bean that has an external dependency?
For example, there's a bean Config with credentials and other configuration, and I want to make a bean like this:
type MyServiceClient struct {
    api *MyServiceApi
}

where MyServiceApi is made like this:
api = NewMyServiceApi(config.Url, config.Token)

So I have a bean of type Config, and want to build a bean of type MyServiceClient with BeanFactory, where I need to use Config bean.
How to do that?

I see only two options here:

Either don't make your config a bean - just have it as a plain Go struct (global var) and use it wherever you need.
Or create your MyServiceApi not with a bean factory, but as a regular singleton bean, and then use PostConstruct() to initialize it by applying your configuration from the injected singleton Config bean.

Unfortunately, singleton beans produced by the bean factory can't look up other beans during initialization (as you already discovered), so there's not much that can help here...

Yeah, I see now. That's unfortunate, I got used to using Spring's bean factories that have such an ability.
Probably, it's worth building a graph of dependencies and create some singletons first, so that other factories are able to lookup them during initialization later. Pretty much like Spring does that 🙂

Yes, this is a good idea/suggestion. Unfortunately, I have very little time now to dedicated to this project, but if you like - your PR on this will be well appreciated :) For now, I'll close this issue, but feel free to reopen it if you get any new input.


Add an interactive console for users as well

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I see several green ci/circleci builds in the "All checks have passed" section of my PR (connect CircleCI to GitHub if not)
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.
[x] I've added or updated relevant unit tests.

Motivation and Context
Sometimes I wish to send code for users to test directly into their setup. This would allow to do it.
Description
I've used the similar code in our u3d project here for years https://github.com/DragonBox/u3d/blob/8c156f401f2b8443936e3873b8566a965c783165/lib/u3d/commands.rb#L68-L80
Testing Steps

I'm wondering if we should name this command repl instead. Because pedantically speaking the term console does not imply interactivity but more reading output (think macOS's Console.app), while things like pry and irb are called REPL (Read-Execute-Print Loop).
That being said since you added bin/console with such a name too we should keep the names consistent.

I wouldn't change bin/console as it follows the bundler convention. Console there probably refer to the original system consoles. The consistency is I think also a strong argument to keep the command identical.
Note that Mac's Console is probably one of the few IT system named console that doesn't do input. I'll blame Apple for the confusion.

Nice! 🌟 this is awesome and quite intuitive IMO.
I think this deserves to be documented somewhere, for discoverability! Idk where exactly, though. It might be in this repo, or in https://github.com/fastlane/docs, what do you think? Where should this live?

yes, in this repo and in the docs would be nice, together with a nice screenshot or even an animated gif.
Example:

I hope to use it to help troubleshoot issues like #21765 :)

@lacostej you probably need to reopen this manually 👀

yes, but I don't seem to be able in the same PR. Something went wrong.

now it lets me

nope, it is stuck. I'll reopen a new one.


FileNotFoundError: [Errno 2] No such file or directory: './checkpoint/Segmentation/Seg_model.pth'

Can you provide Seg_ Model. pth file, thank you.

The segmentation model is downloaded from:https://github.com/Linfeng-Tang/SeAFusion/blob/main/model/Fusion/model_final.pth.


Adding Web UI instead of integration setup menu

Hi,
I would like to start by saying a really love this integration.
I do find the integration menu a bit complicated and not very intuitive.
I would like to propose a new feature to have it all in a web UI.
This will make it possible to expand the configurations and customization.
Again I really love this and thank you

Thanks for the sweet comments! This is my first integration so I'm stumbling with getting the integrations menu right. It does looks cramped and cumbersome and hopefully on next major version release we'll have that sorted out.
Can you elaborate on the web ui idea?

Hi, can you try out the code on the 3.0.0-beta branch? (Or wait until the weekend for the next release). The Integration flow was totally rewritten and shall be much more intuitive. Also now we have a wiki!


Add support for Kubernetes 1.12.2

What this PR does / why we need it:
See https://github.com/kubernetes/kubernetes/releases/tag/v1.12.2
TODO:

[x] upload Windows binary

If applicable:

[ ] documentation
[ ] unit tests
[ ] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:
Add support for Kubernetes 1.12.2

/lgtm after rebase


feature: Lazy load on file name

Did you check the docs?

[X] I have read all the lazy.nvim docs

Is your feature request related to a problem? Please describe.
Multiple different languages use .toml files for project configuration. crates.nvim is a useful tool for managing rust project configurations, but setting ft = "toml" unnecessarily loads crates.nvim on pyproject.toml files in python.
Describe the solution you'd like
Setting filename = "cargo.toml" or filename = "pyproject.toml" would allow these packages to only load when necessary.
Describe alternatives you've considered
No alternative
Additional context
No response

{
lazy=true
setup=function() vim.api.nvim_create_autocmd("BufEnter",{pattern="cargo.toml", callback=function() require"lazy".load({plugins="crates.nvim"}) end) end
}

not needed to add this feature imo since it isn't used very often and would just make things more complicated and it's really easy to do it yourself

The docs of crates.nvim also mention another solution, under "For lazy loading".
I modified it slightly, as the docs still use packers use:
{
    'saecki/crates.nvim',
    event = { "BufRead Cargo.toml" },
    dependencies = { 'nvim-lua/plenary.nvim' },
    config = true
}
``

oh I forgot that you can create an event like this. Under the hood this does exactly the same as my snippet and should be the way to load something on filename


How to schedule job to run bi-monthly.

How to make job run every haft month at the 16th day and the 1st day of the moth?

25 10 1,16 * *
run every 1th and 16th at 10:25

you can try: https://crontab.guru


Remove Gemnasium reference from README

They were aquired by Gitlab and the service is no-longer available as a
standalone service.
https://docs.gitlab.com/ee/user/project/import/gemnasium.html

/reviewme @rainforestapp/devs


Add a Blocks Piece to the Math Spec

This should go through and combine all the policies, mechanisms, boundary actions, etc. Will want to also add assertions that no name is repeated


hackerrank vowels and consonants

Closing Issue/Referencing Issue
issue reference (#)
Changes Made In Pull Request

Hi @ows-ali Please review the PR


deps: Run yarn upgrade

Tested and the app runs on my iPhone and on the office Android device.

Thanks for taking care of this! Looks good — merging.


DropdownMenu has secret focusable regions for each item in the menu

STEPS TO REPRODUCE

Run the app below.
Tap the "tab" key 7 times.

EXPECTED RESULTS
The focus should cycle through Before 1, Before 2, Before 3, the text field, the drop-down arrow, After, and back to Before 1.
ACTUAL RESULTS
When the focus would enter the text field, it instead goes to the values that are in the dropdown. You can see this by pressing the "enter" key, it suddenly changes the text field value to "3".
CODE FOR TEST PROGRAM
import 'package:flutter/material.dart';

void main() {
  runApp(
    MaterialApp(
      home: Material(
        child: Row(
          children: [
            Expanded(
              child: FilledButton(
                child: const Text('Before 1'),
                onPressed: () {},
              ),
            ),
            Expanded(
              child: FilledButton(
                child: const Text('Before 2'),
                onPressed: () {},
              ),
            ),
            Expanded(
              child: FilledButton(
                child: const Text('Before 3'),
                onPressed: () {},
              ),
            ),
            DropdownMenu<int>(
              dropdownMenuEntries: List.generate(
                5,
                growable: false,
                (int index) => DropdownMenuEntry<int>(value: index, label: '$index'),
              ),
            ),
            Expanded(
              child: FilledButton(
                child: const Text('After'),
                onPressed: () {},
              ),
            ),
          ],
        ),
      ),
    ),
  );
}

Ref: [DropdownMenu] Focus Traversal does not work between multiple fields

I just tested adding more entries in the DropdownMenu and concluded that more the number of entries, more will be the number of focus traversal to get to the DropdownMenu field.
So, if the DropdownMenu field that is going to be focused has 6 entries, it will take 6+1 focus traversal (eg: [TAB] key presses) before the field finally gets focus. This happens even if the menu is not opened.
Looks like focus goes through the entries first and then finally to the input field which is just the reverse of what should have been the correct way to focus. i.e. Focus the input field first & then the entries.

Yeah, this is a dupe of that bug.


text wrapping overlap

When text wraps in the console, it winds up printing on top of whatever is on the next line with very bizarre looking results:

I'm having this problem as well with version 0.9.4, atom version 0.176.0.

I got the same issue,  but I found it only happens when I use left or right panel for the console. atom  0.181.0,  atom term2 0.9.4

Do you have softWrap enabled? If so, try setting softWrap: false under term2 in your config file.

I'm experiencing the same problem.

I tried setting softWrap: false in my term2 config, but I still get the same problem.

We need to resize more often, we should probably listen to more events.


Example client

A very basic example client showing how to interface with wsgateway connections cleanly

Completed in resources/nodejs_demo/demo_echo.js


[Docs] Update tool docs

更新 tool 相关的文档

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

Comparison is base (f04d8a8) 70.70% compared to head (94c0e42) 70.70%.
Report is 4 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #254   +/-   ##
========================================
  Coverage    70.70%   70.70%
========================================
  Files           62       62
  Lines         3106     3106
========================================
  Hits          2196     2196
  Misses         910      910

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


Update Composer dependencies (2022-05-09-00-06)

Visual regression test passed!
View the visual regression test report


ruby-ci-build: add internal rubybuild tool

Curious external contributors: This is still a rough draft of an idea, not a promise we're moving in x or y direction. Don't read too much into this or the quality of it (please :wink:).

Maybe this should copy odie from brew.sh and use it instead of panic so it could include messages indicating what went wrong? The individual steps of the build aren't echoed, so it may not be clear at what point it failed. (Unless you turn on -x for bash tracing. Which might not be bad here, since it's not calling additional shell functions, so you wouldn't get excessive spew; just echoing of each build step, like a verbose make invocation.)

Forgot about this. Killing it given the move to Ruby 2.0 should be close, fingers crossed.


Create monotonic impl for STM32L0

Unfortunately STM32L0 (Cortex M0) only has 16 bit timers, which makes things a bit tricky. This implementation uses the 16 bit TIM6 timer and a prescaler of 2048.
Running the example results in a fizzbuzz counter being printed via UART at a frequency of 2 Hz.

I hope everything is correct :slightly_smiling_face: A review would be appreciated.

Awesome example and thank for sharing!


ios 1079

This change is

I missed that ios_1079 was merged into iOS so this PR contains already reviewed #1359 and #1378

Test (Success) 90147336 build is completed (9726ba5)
See https://github.com/ThaliTester/TestResults/tree/901473369726ba5_ios_1079_baydet/ for the logs

Test<PHONE_NUMBER>26ba5(9726ba5) has failed
See https://github.com/ThaliTester/TestResults/tree/901473369726ba5_ios_1079_baydet/ for the fail logs


Latest lerna v7 does not use useWorkspaces flag

I noticed the eslint-plugin-import started failing after I upgraded to lerna 7 which removes the useWorkspaces flag
This package I think detects the useWorkspaces flag to find packages though
I can keep the useWorkspaces flag in my lerna.json but note that lerna has a "lerna.json fixer script" that removes this flag

Actually cannot even keep the useWorkspaces flag without getting some error, lerna errors out with message
$ lerna run useDist --scope "@jbrowse/plugin*" --scope @jbrowse/text-indexing
lerna notice cli v7.1.1
lerna ERR! ECONFIGWORKSPACES The "useWorkspaces" option has been removed. By default lerna will resolve your packages using your package manager's workspaces configuration. Alternatively, you can manually provide a list of package globs to be used instead via the "packages" option in lerna.json.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Error: Process completed with exit code 1.


Do not manage to be automatically redirected to original URL

I'm using the following middleware for the oauth/authorize route:
app.get('/oauth/authorize',
  login.ensureLoggedIn(),
  server.authorize(function(clientId, redirectURI, done) {

  Client.findOne({clientId: clientId}, function(err, client) {
    if (err) { return done(err); }
    if (!client) { return done(null, false); }
    if (client.redirectURI != redirectURI) { return done(null, false); }
    return done(null, client, client.redirectURI);
  });
}),
server.errorHandler(),
function(req, res) {
  res.render('dialog', { transactionID: req.oauth2.transactionID,
                         user: req.user,
                         client: req.oauth2.client
  });
}
);

The connection URL is the following one:
http://localhost:1337/oauth/authorize?client_id=O3UTGRFNI1&response_type=code&redirect_uri=http://localhost:1339&scope=http://localhost:1337
When the user is not logged, he is redirected towards the login page but when he logs he is redirected towards the index page (instead of being redirected towards the above URL). Any idea why this is happening ? I've certainly missed something but I cannot figure out what (probably linked to returnTo).
I've also set up a complete example in the following question:
http://stackoverflow.com/questions/25833533/secure-nodejs-app-with-oauth2-not-automatically-redirected-to-allow-deny-page

Did you ever resolve this issue? I'm having the same problem.

For anyone else running into this issue, it was indeed something very simple and is clearly in the docs:
app.post('/login', passport.authenticate('local', { successReturnToOrRedirect: '/', failureRedirect: '/login' }));
You need to give your passport route the successReturnToOrRedirect parameter so that it knows what to do after you login.

Thanks a lot, I didn't got this.

@nodeit thanks

@nodeit thanks, that answer is gold.
Although the option is documented, it's not intuitive at all. successReturnToOrRedirect: '/' might indicate a redirect to root rather than the original url.

I did this and noticed the user is logged in only after the third attempt... I removed the package and it worked well... How do i get around this? github.com/miracle0403/newsitemlm.git


dh_cms_set_peerkey() -- allow only absent parameters in RFC

dh_cms_set_peerkey() -- Fix the incorrect condition -- Only absent parameters allowed in RFC XXXX
Fixes #25824

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

@t8m  -- can you mark approval:done on this?

@openssl/committers -- pinging for review.

@t8m -- can you mark approval:done on this?

No I cannot as this does not have two reviews yet.

@t8m -- can you mark approval:done on this?

No I cannot as this does not have two committer reviews yet.

Okay, Could you help getting 2nd one? I'm not sure how to request.

Okay, Could you help getting 2nd one? I'm not sure how to request.

Unfortunately it is sometimes hard to get the attention of committers. Your ping should be sufficient, but apparently it was not.

This pull request is ready to merge


No information on how to assign RBAC at image definition or image version scope

There is no information on how to assign RBAC at image definition or image version scope.
Also - would be nice to know how to grant RBAC for creating new image versions without being able to change the image definitions.

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

ID: 8dc9d89d-c72c-3e3f-c825-21e75c0508b1
Version Independent ID: 8b588298-622f-b27d-5724-b1fc9a566d74
Content: Share resources in an Azure Compute Gallery - Azure Virtual Machines
Content Source: articles/virtual-machines/share-gallery.md
Service: virtual-machines
Sub-service: gallery
GitHub Login: @sandeepraichura
Microsoft Alias: saraic

Thank you for reaching out. At this time we are reviewing the ask and will provide an update as appropriate

Thanks for reporting this. I have assigned the issue to the content author to investigate further and update the document as appropriate.

@mark-strasser-nnl - Assigning RBAC at image definition or image version is same as assigning RBAC at gallery level.  we recently updated public docs on how to use RBAC sharing for compute gallery, please refer and let me know if you have additional questions: https://learn.microsoft.com/en-us/azure/virtual-machines/share-gallery?tabs=portal

#please-close


Keep link color when visited

Many THX for the suggested fixes @mabulgu

@gunnarmorling decided that links shouldn't open in a new tab per default which is why I removed this change.

I slightly modified your suggested color-related changes because for me, the link coloring didn't work correctly after testing it with 3 browsers FF,Chrome,Safari on Mac OS.

Hi, @hpgrahsl thanks for the merge and appreciate your updates!
@gunnarmorling Is there any special reason for not having the opened pages in new tab? I thought staying on the same page and not losing the main page by opening the resources in a new tab would be useful as it is a common marketing practice. Was there a concern or sth? (just wondering)