Localization Support

Is your feature request related to a problem? Please describe.
Games sometimes require localization support. Currently, UIText and CCLabel do not have any support for this.
Describe the solution you'd like
First, cocos-studio already has a localization manager that is unused (No support in cocos studio to enable localization, although it is part of the flatbuffer schema, see options->isLocalized()). I was thinking that maybe we can edit the reader code to localize strings if some global flag is set. Also, localization should be done in such a way that it also enables to set the font on the UIText, because some languages like arabic might require a different font. Maybe something like this:
// TextReader.cpp
ILocalizationManager* lm = LocalizationHelper::getCurrentManager();
if (lm) // or something like if (g_FlagEnableLocalization)
{
    lm->setLocalizedString(label, text);
}
else
{
    label->setString(text);
}

// CustomLocalizationManager.cpp
void CustomLocalizationManager::setLocalizedString(Text* node, std::string_view text)
{
      auto font = this->getFontBasedOnLanguage(_currentLanguage);
      node->setFontName(font);
      node->setString(this->localizeString(text));
}

Describe alternatives you've considered
Another alternative is to just mark UIText::setString() and UIText::setFontName() as virtual so users can write their custom UIText nodes inherited from UIText and then they can override setString() and setFontName().
The TextReader in cocos studio can already be overriden by registering the type in ObjectFactory before the app loads, and there instead of doing Text::create() we can create our custom TextCustom::create() node.
And, one more approach could be to just add localization support engine wide, so that setString() function localizes the text before it sets it.
Please suggest a good approach so I can work on it and submit a PR!

I've elegantly written my own localization string and here is how it goes:
the Label or Text are inherited into a custom type called LocalizedLabel or LocalizedText and these classes that inherit the base just add one function, which is setStringLocalized() without passing an object reference or dealing with a singleton. one note setStringLocalized() can be added to the engine's code directly but I prefer not to modify the engine a lot as my changes could get a conflict. setStringLocalized() would go a localization manager singleton and check to look for a localized string in the files that have been loaded into that localization singleton, and if nothing is found (i.e file not loaded) then it would set the text to the same text as the localized key as to tell the developer that he/she forgot to load a localization file. also, the localization data could be loaded from JSON, plist, or XML files and for the font names it can be added as a second parameter or something.. that's the design I'd go for to minimize complexity!

That is fine, but the problem is with importing files from an editor, like
cocos studio or ax studio. We have to localize the strings while importing.
On Sat, Aug 6, 2022, 12:57 PM Turky Mohammed @.***>
wrote:

I've elegantly written my own localization string and here is how it goes:
the Label or Text are inherited into a custom type called LocalizedLabel
or LocalizedText and these classes that inherit the base just add one
function, which is setStringLocalized() without passing an object
reference or dealing with a singleton. one note setStringLocalized() can
be added to the engine's code directly but I prefer not to modify the
engine a lot as my changes could get a conflict. setStringLocalized()
would go a localization manager singleton and check to look for a localized
string in the files that have been loaded into that localization singleton,
and if nothing is found (i.e file not loaded) then it would set the text to
the same text as the localized key as to tell the developer that he/she
forgot to load a localization file. also, the localization data could be
loaded from JSON, plist, or XML files and for the font names it can be
added as a second parameter or something.. that's the design I'd go for to
minimize complexity!
—
Reply to this email directly, view it on GitHub
https://github.com/axis-project/axis/issues/775#issuecomment-1207166538,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWQGRVES7UCSJAS4UQO4PLVXYHXNANCNFSM55PPRK2A
.
You are receiving this because you authored the thread.Message ID:
@.***>

you mean as in localise them we store them with a specific language?

No... Lets say you create a scene in CocosStudio/X-studio. You add a Text node to it with this string - "EN_SETTING_FPS" which a localization key. When loading it in axis with the cocostudio extension there is currently no way to localize this.
Ideally, while loading itself, there should be a way to get the localized value for this string ("EN_SETTING_FPS" => "Frames Per Second") and set it on the text node via ->setString(localizedString)
You can read the code in TextReader.cpp, you will understand. There is already a localization manager there, but the key isLocalized() is not supported and is always false.

Oh! thanks for clearing that out I'll look into it!

Can you send a code portion of creating and adding text using TextReaders because I didn't find any examples of it

Hi,
You can do this -
Make a copy of TextReader.h and TextReader.cpp in your Classes/ folder somewhere. Rename the class to something else, e.g. CustomTextReader. Manually replace all the macros with their code, e.g. IMPLEMENT_CLASS_NODE_READER_INFO (This one is very important!)
The IMPLEMENT_CLASS_NODE_READER_INFO macros defines the type info TInfo for a type, in our case TextReader. Now we want the type to remain the same, that is "TextReader" but we want it to be parsed by CustomTextReader. When you replace the macro, change the TInfo declaration to this -
axis::ObjectFactory::TInfo CustomTextReader::__Type("TextReader", &CustomTextReader::createInstance);
Notice that in the string argument, I have used "TextReader". This is very important as this is the key of the reader that cocostudio uses to read UIText nodes.
Now the final step is to simply include your customtextreader file before any cocostudio files are included. This ensures that your type info will be registered first and when cocostudio textreader tries to register itself, it will fail because the map will already contain a key for "TextReader" (c++ maps do not allow duplicates, if a value exists, insertion is skipped).
If you still face issues, let me know and I will post my files.

Wouldn't it be easier and more convenient if we just stick the localization to the Label class directly? it feels much more organized than this mess

That would be the best!

cocostudio actually needs to be deprecated since it doesn't make sense and doesn't have documentation

would this be a viable solution to loading a localization file from json?
of course, you have to organize localization in a way so that you load a file called text_en.json and it retrieves the english keys automatically, and the same goes for other languages but that's just an example.
{
    "english": [
      "some english text",
      "english_font.ttf"
    ],

    "arabic": [
      "نصوص عربية",
      "arabic_font.ttf"
    ]
}

I think a better solution would be to leave the localization file loading and string replacement to the game itself. The UIText / Label should just call the callbacks of whatever localization solution the game has implemented. For example, the label class could have a callback member variable -
// CCLabel.h
typedef std::string_view (*LocalizerCallback)(cocos2d::Label*, std::string_view);
LocalizerCallback _localizeString;

// CCLabel.cpp
void Label::setString(std::string_view text)
{
    if (_localizeString)
    {
        text = _localizeString(this, text);
    }

    if (text.compare(_utf8Text))
/// ... Other code

The advantage of this approach is that the game can then load its localization files any way it wants! Be it json, binary, xml, etc. And the same goes for replacement, it doesn't have to rely on any pattern imposed by the engine, like you said - "ENGLISH_KEY_1": "Health: %s" ... it can follow any pattern like "ENGLISH_KEY_1": "Health: {1}" or "ENGLISH_KEY_1": "Health: {%HEALTH%}", whatever. The engine doesn't care. All it does, is that is calls the code that handles this replacement.

what about setting the font name and size? LocalizerCallback deals with it right?

Yeah, that's why the callback has the Label as the first parameter.

Actually you can't have a callback as pointer.. it would pollute the class

There you go

it got overwritten because my function only returns "WOW"

now each time you set a label to that localizer it will hold on to it until it dies then it decreases the ref's count by 1, I called retain to make sure it stays with me even if all labels are destroyed, that's great memory management!

Maybe this is helping too?

@theunwisewolf
Is it still an issue?
What is miss?


feat: add support for Postgres multi dimensional arrays in Go

This PR adds support for generating model fields from multi-dimensional postgresql array columns. The approach here is to pass around an additional column attribute array_bounds representing the dimensions of the postgres array column. When generating code, the array_bounds is checked along is_array to determine the dimensions of the resulting struct field. Would be happy to hear alternative approaches or solutions here as well. More info: Postgres Arrays
fixes issue: https://github.com/kyleconroy/sqlc/issues/1494
Replaces https://github.com/kyleconroy/sqlc/pull/1651 https://github.com/kyleconroy/sqlc/pull/2308

I don't have push rights to this branch @kyleconroy, so I updated the original branch and made a new PR https://github.com/kyleconroy/sqlc/pull/2338 with the fixes.


Anova rm addition

What's included and why?
This includes the addition of a repeated measures ANOVA to q2-longitudinal! I'm contributing this per @nbokulich's suggestion on this QIIME2 forum post.
A summary of changes:

Utilized statsmodels AnovaRM in _longitudinal.py to create a visualizer that performs a repeated measures ANOVA
Added a visualize_anova_rm function to _utilities, which creates a .qzv that contains an html table with the AnovaRM results
Registered this all as a visualizer in plugin_setup.py. I tried to keep description wording consistent with the LME, since this function has a flexible type for the input table containing the outcome variable (i.e. it could take a FeatureTable, or it could take something like first distances or an alpha diversity vector).
Added a citation for the AnovaRM method using least squares. This is referenced in the statsmodels documentation, but I wasn't sure what qualifies a source to be added to QIIME2 plugin citations. I'm happy to remove this, but I wanted to include it for now to err on the side of caution.
Imported numpy as np in _longitudinal.py so that aggregate functions can be called from the command line interface. I'm happy to remove this if it feels unnecessary, but without it I believe users won't be able to call any aggregate functions other than mean.

A note: this is my first contribution to the project, and walking through the developer docs (multiple times) has been a great learning experience. That being said, I'm sure there must be conventions that I've missed, and I'm more than happy to address those if you come across them! I haven't yet included tests yet, but I was hoping to get feedback on what I have in the meantime.
Thanks! :smile:

Thanks for this contribution @sterrettJD !
I have a few high-level comments first, and can take a more detailed review on the second pass:

would it be possible to make RM an option in the existing anova action in q2-longitudinal? Or at least make the API more consistent? (e.g., to use a proper formula structure instead of accepting a str of comma-delimited predictors). My preference would be to make RM an option within the existing action, and this would also limit code replication. On the other hand I see that some options are unique to each method so a distinct action might be appropriate, in which case both anova and anova_rm should call the same base function, and only the relevant options are exposed in each action. What do you think?
similarly, the visualizer does not need a distinct function, since it is mostly code replication. The visualize_anova function should be adjusted instead to enable/disable different features depending on the input (e.g., if residuals data are present or not).
The citation can be removed, the citations in QIIME 2 are generally more the "original publication" describing a method, as opposed to a textbook reference. Thanks for erring on the side of caution!
unit tests will be needed. You could just use example data from statsmodels as the test data here, but the idea is basically just to make sure that if anything shifts in Q2 it does not break the intended functionality!
the table input can be removed... metadata-transformable objects (including feature tables!) can be merged in Q2 to extract target data, e.g., if running anova with an ASV as the dependent variable. You were probably following the LME API as a guideline here, but that API pre-dates the addition of a feature table to metadata transformer, and should also be updated at some point. You can just drop this.

Let me know what you think and we can discuss next steps... Thanks!

would it be possible to make RM an option in the existing anova action in q2-longitudinal? Or at least make the API more consistent? (e.g., to use a proper formula structure instead of accepting a str of comma-delimited predictors). My preference would be to make RM an option within the existing action, and this would also limit code replication. On the other hand I see that some options are unique to each method so a distinct action might be appropriate, in which case both anova and anova_rm should call the same base function, and only the relevant options are exposed in each action. What do you think?

Sure thing! I think it would be feasible to expose it within the existing action, so I'll start with that. Once I make some progress, I may check back in to get your thoughts on if it seems more practical to split to a separate function.

similarly, the visualizer does not need a distinct function, since it is mostly code replication. The visualize_anova function should be adjusted instead to enable/disable different features depending on the input (e.g., if residuals data are present or not).

Makes sense!

metadata-transformable objects (including feature tables!) can be merged in Q2 to extract target data, e.g., if running anova with an ASV as the dependent variable. You were probably following the LME API as a guideline here, but that API pre-dates the addition of a feature table to metadata transformer, and should also be updated at some point. You can just drop this.

Cool! You're right, I was following the LME API, but that makes a lot of sense. I had been wondering why passing alpha diversity vectors to Metadata for LME worked but passing them as the input table did not.
Thanks for the helpful response and being willing to work with me on this, @nbokulich ! To let you know my timeline, I'm pretty booked through June 15, maybe later, but I'll try to get working on this by July and will check in if I have questions. I'll also address all of the lint issues!

Thanks @sterrettJD ! Just a quick comment on the timeline — that sounds totally fine from my end. The next QIIME 2 release is in August, so having this PR finalized by late July would be a good target if you want this in the next release.

Hi @nbokulich sorry about the delay - got wrapped up in summer projects and classes. I'll try to get back on this over the next 2 weeks! If it looks like that isn't reasonable I'll let you know. Thanks for following up!

Hi @sterrettJD ,
that timeline sounds fine, thanks! just checking in

Hi @nbokulich just wanted to touch base, I'm working on this now and will be doing more this week. Sorry about the delays!

Hi @nbokulich and @thermokarst , I'm working on implementing repeated measures as part of the base anova implementation. To keep the the API consistent (and allow users to provide a formula), I've implement a linear mixed effects model with random effects for the participant ID, which is passed to statsmodels' anova_lm (like the non-rm version, which is an OLS model passed to anova_lm).
However, anova_lm can't accept the mixed effects model results class, so I was wondering what your thoughts are regarding implementing this via statsmodels' AnovaRM function? The downside to AnovaRM is that it doesn't accept a formula, and users couldn't specify interactions.
I've looked into extracting model information to perform an ANOVA without statsmodels, but I worry about the robustness of this approach. Do you have any thoughts on what the best route forward might be? Would it be better to try to extract the model information to perform an ANOVA without statsmodels, or would using the existing AnovaRM function be better? Or would there be another option that I'm not considering? Thanks!

Hi @sterrettJD ,
Thanks for the update! My thoughts:

Keeping the API consistent will be ideal.
It sounds like AnovaRM is the correct way to do this in statsmodels.
to specify a formula: patsy could perhaps be used here to parse a formula input, and pass it to AnovaRM as a list of factors.
the documentation does not make this clear, but it looks like AnovaRM actually tests all interactions when multiple factors are passed in the list. This is not ideal either, but this also appears to be the case with the equivalent R function anova_test for repeated models. I suggest we start here, and any enhancements could be easier to layer on later.

I would avoid doing this without statsmodels — I agree, it does not sound like a robust approach.

Hi all, I've updated the anova function as discussed previously.

anova() now calls statsmodels AnovaRM if repeated_measures == True
Patsy is used to parse the formula for AnovaRM.
I've updated the anova function registration and added some basic unit tests for a one-way AnovaRM.

However, I don't yet have unit tests for a 2x2 repeated measures Anova. The function requires fully balanced groups, and the dataset I've used so far for unit tests doesn't have balanced groups for any 2x2 design, so I'll have to find/make one.
Additionally, I haven't implemented any post-hocs yet. Do you think that a paired samples t-test would be generalizable and appropriate? I wanted to check before implementing.
If you have any feedback on the implementation so far, I can update to address that. I'll also address the lint issues.

Hey @sterrettJD, thanks for all of your hard work on this! We are currently doing some PR triage and review - are you still in the process of going through the review comments from @nbokulich?

hey @ebolyen sure, that sounds like a good compromise. I have given this some testing this morning and this looks ready to merge.
@sterrettJD thanks for the hard work on this! Very pleased to have this in q2-longitudinal and I look forward to using it myself! If you find some time to further refine this action per our discussion above, please open a new PR.


Update login callback URL.

Description
This PR updates the login callback URL to work in the deployment environment. Currently, it has /contributions as the prefix, which will redirect to the Contributions BB endpoints. Before starting to review this PR, please let me know so that I can update the OAuth app's callback URL in GitHub. Thanks.
Fixes #803
Review Time Estimate
Please give your idea of how soon this pull request needs to be reviewed by selecting one of the options below. This can be based on the criticality of the issue at hand and/or other relevant factors.

[ ] Immediately
[x] Within a week
[ ] When possible

Type of changes
Please select a relevant option:

[ ] Bug fix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[x] Other (any another change that does not fall in one of the above categories.)

Checklist:
Please select all applicable options:

[ ] I have signed the Rokwire Contributor License Agreement (CLA). (Any contributor who is not an employee of the University of Illinois whose official duties include contributing to the Rokwire software, or who is not paid by the Rokwire project, needs to sign the CLA before their contribution can be accepted.)
[x] I have updated the CHANGELOG.
[x] I have read the Contributor Guidelines.
[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] My change requires updating the documentation.
[ ] I have made necessary changes to the documentation.
[ ] I have added tests related to my changes.
[ ] My changes generate no new warnings.
[ ] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

Merging to fix the login issue in the DEV instance.


[Hipcc bug] Wrong target code generation from a correct source

my_kernel.cu.txt
The attachment contains a very basic example, which is correctly executed on CUDA, but get wrong results on ROCM 2.10. Please check it.

@ghostplant, Currently hcc compiler assumes optimal block size is 256 and it does code optimization with the same assumption which may result in different output in some cases especially where the block size is greater than 256. Assuming a higher number (>256) will create a resource crunch which will impact performance hence it is suggested to the programmers to set launch_bounds(<max_block_size>) if (s)he knows that block size is going to be greater than 256.
E.g. In this case if you set I could see the expected output.
extern "C" global launch_bounds(1024) void template_op_kernel0( float* restrict output.......)
{
}
Closing the issue now feel free to open in case it does not help at your end.


[1.19.4] Documentation for The Aether mod.

Adds documentation for The Aether's CraftTweaker compatibility. This includes methods for manipulating the fuel items for the Altar, Freezer, and Incubator.

Updated documentation according to changing methods to be static.

Is there any reason for requiring an MCTag or an IItemStack as arguments instead of just IIngredient?

Why does removing an item require a user to know the previous burn time? How are they supposed to check it?

Updated documentation after changing methods to use IIngredient and removeFuel no longer needing a burn time parameter.

@bconlon1 could you please reply here when the update is released so we can go ahead and merge this, thanks!

The update with this compat has now been released, you are free to merge the PR.


Add foldx_gfp_lambo

To run GFP experiments we require:

[ ] a GFPWrapper
[ ] a GFPWrapperFactory
[ ] a hydra config for Lambo instantiation
[ ] problem registration
[ ] addition to objective_repository init
[ ] addition to the documentation

test iterations on lambo experiments

Hi Richard, you recently added this one, right?

Mixing GFP and Lambo is no longer a requirement, and somewhat a nonsense request on my end. Closing this one.


Projectiles un-randomizing mid run

As the title says, my friends and I were playing last night and a few stages in, a few of our character abilities had defaulted back to their normal state.
We've played a lot of this mod (30-40hrs) and it seems to be an issue since the update for hitscan projectile randomization.
Next time it happens I'll try and grab a video for you.

And by "next few days", I apparently meant "right now". This bug is fixed in version 0.7.2.


Unable to install Windows Subsystem for Linux (wsl)

I am running windows 10-18363 build....
Recently I tried to install WSL for windows but at every reboot updating windows reaches 35% and reboot to begain the installation and fail for the 2nd time. On the 3rd reboot it tries to uninstall the update and says {update unsuccessful} .
After opening ubuntu 20.4 it gives the following error
Installing, this may take a few minutes... WslRegisterDistribution failed with error: 0x8007019e The Windows Subsystem for Linux optional component is not enabled. Please enable it and try again. See https://aka.ms/wslinstall for details. Press any key to continue...
After checking Windows Features-Windows Subsystem for Linux is not ticked.
THANK YOU

Did you enable the optional component and reboot?

Yes...that's when it says update unsuccessful

Try to install it using PowerShell or Cmd:
Enable "Windows Subsystem for Linux"
dism.exe /online /enable-feature /featurename:Microsoft-Windows-Subsystem-Linux /all /norestart
Enable "Virtual Machine Platform"
dism.exe /online /enable-feature /featurename:VirtualMachinePlatform /all /norestart
restart system

Still the same issue

Still the same issues! since 4 days ago, after workaround don't got any idea more to solved this.

Please try to upgrade your Window 10 OS to build 19041 first, then try to install Windows Subsystem for Linux?

I have this same message but my situation is different.
My WSL2 and Ubuntu distro had already been working fine since I enabled and installed it a few months ago.
I booted my PC a few days ago and it said due to an error the BIOS setting were resetted to the default options, which also deactivated support for virtualisation.
Then Windows 10 booted up and WSL would not work anymore, saying that the windows features were not enabled.
So I first restarted and went into BIOS to reactivate the support for virtualisation.
When Windows 10 was loaded I still could not access WSL with the same error message as OP, however the Virtual Machine Platform, Windows Hypervisor Platform, and Windows Subsystem for Linux all appear to be activated according to the Windows Features settings.
I tried to deactivate the Virtual Machine Platform, Windows Hypervisor Platform, and Windows Subsystem for Linux in the Windows Features settings but it fails and I continue to receive error message 0x8000ffff.
So WSL is not available even though virtualisation is enabled in the BIOS, and the Virtual Machine Platform, Windows Hypervisor Platform, and Windows Subsystem for Linux all appear to be enabled in the Windows Features settings but I am unable to deactivate then reactivate them.
Does anyone know how to repair this ?????

It makes me carzy! Have you get any solution?


--baseline  returns no results

I'm running detect-secrets-hook from a container with the current directory mounted. What would be some reasons that when I add a baseline I get no results and no error at all, but when I remove it and leave out just the file name, such as: detect-secrets-hook something.json it reports correctly.
What could be the reasons for that?
I'm using version 0.12.2

Hi @lirantal,
Can you post a stack trace or example invocation so that we can better debug it?

@domanchi sure, how can I provide you with a stacktrace or anything useful to debug it?
So reminder that I installed detect-secrets in a container and I mounted the top level directory with .git as a volume there. When I run it without the baseline I get this kind of output:
root@b4fd6c41d694:/usr/src/app/server# detect-secrets-hook a.js
Potential secrets about to be committed to git repo! Please rectify or
explicitly ignore with an inline `pragma: whitelist secret` comment.

Secret Type: Secret Keyword
Location:    a.js:3

Secret Type: Secret Keyword
Location:    a.js:6

Possible mitigations:

  - For information about putting your secrets in a safer place,
    please ask in #security
  - Mark false positives with an inline `pragma: whitelist secret`
    comment
  - Commit with `--no-verify` if this is a one-time false positive

If a secret has already been committed, visit
https://help.github.com/articles/removing-sensitive-data-from-a-
repository

When with baseline I get nothing printed:
root@b4fd6c41d694:/usr/src/app/server# detect-secrets-hook --baseline .secrets-baseline a.js
root@b4fd6c41d694:/usr/src/app/server#

However, when I run the baseline command like above from the host machine (not inside a container) it works fine.

One difference that I now noticed is:

Locally on my host I have python 2.7 and detect-secrets 0.12.2 installed
On the docker container I have python 3.7 and detect-secrets 0.12.3

The output you provided helps!
My hypothesis is that this is expected behavior. The detect-secrets-hook is meant to be used as a pre-commit hook, and will only complain if things are found. However, if a baseline is provided, it will ignore the secrets provided in the baseline.
Methods to check whether my hypothesis is accurate:

Does your baseline contain the secrets found on a.js:3 and a.js:6?
If you manually edit your baseline, and remove one of those secrets, will the detect-secrets-hook complain?
If you print the error code, it should be 0.

root@b4fd6c41d694:/usr/src/app/server# detect-secrets-hook --baseline .secrets-baseline a.js
root@b4fd6c41d694:/usr/src/app/server# echo $?
0

I realized the issue is that I hadn't had git available in the docker image and that was the source of the silent failures. This is now working well and we can close the issue.
I have a docker image bundling this for easy compatibility:

GitHub repository: https://github.com/lirantal/docker-detect-secrets
Docker Hub: https://cloud.docker.com/u/lirantal/repository/docker/lirantal/detect-secrets

So you can now do something like:
docker run -it --rm --name detect-secrets --volume `pwd`:/usr/src/app lirantal/detect-secrets "src/index.js" "src/component.js"


Xorg X11 servers cannot be launched from Frecon, return to Chromium OS and use Crosh to launch X

Please paste the output of the following command here: sudo edit-chroot -all

Can't because of the error constantly terminating the connection to the chroot
Please describe your issue:
Sudo is disabled in the Crosh shell after about R119, fixed that by using the VT2 shell, auto tools has been removed, fixed that by using CROSH_BRANCH=silence, now the error described in the title shows up whenever I try sudo startgnome or sudo enter-chroot startgnome
If known, describe the steps to reproduce the issue:
Have a Lenovo IdeaPad Duet without the keyboard pack, with a USB-C dongle and a wired iMac keyboard and wireless mouse and the worst timing to try Crouton in the world.

It also happens whenever I try anything with -chroot at the end (edit-chroot, enter-chroot, etc.) but not when I use delete-chroot.
Tried all desktop environments and I am not skilled enough for not using them. Unity and Unity-Desktop do not work because I am trying to use Xenial and the last arm-based build of Unity was for Precise.

See https://github.com/dnschneid/crouton/discussions/5013. You can't open x11 servers from the vt2 shell.


Check notebooks in CI check workflow #76

ref #76
Add check for jupyter notebooks using treon
Update example notebook so it passes the check; use local data instead of remote.
Add Shakespeare and Company Project events dataset to use cases - both as an example and so we can reference it in the notebook.

This all looks great, the ReviewNB interface is slick. I haven't used treon before; that step basically makes sure you can execute all cells in the notebook in order without any errors?

Yes - if the imports don't work or syntax is wrong the check will fail.  (I think it can do more than that if you want to, but that is useful enough for my use cases).

@ColeDCrawford thanks for the comments. I was able to address the lxml weirdness by requiring a newer version of nbconvert, which seems somewhat better.
Also thanks for pushing me more to articulate the value and potential of undate with the S&co dataset. Would you mind taking a quick look at the paragraphs I added before I merge?


Refactor

Fixes #56 #55 #53

Coverage increased (+12.3%) to 61.027% when pulling 75adb2c191dbed5a74393640e082bd3fdc798ab9 on refactor into 37442000f2215801566729776275ba233ddcf5c8 on master.


Add external configurations to .codeclimate.yml

Add custom .codeclimate.yml which grabs externally hosted config files for PMD and checkstyle

Coverage remained the same at 98.0% when pulling b94bdbd719751d5bf24125579a6b277c33ef6eed on davehenton:master into 6ed3f5f8c9921dc3c5f0daa7df8d821474de7bad on thejohnfreeman:master.

Thanks for the contribution, Dave! Can you explain for me how CodeClimate knows which fetched files configure which engines?

Coverage remained the same at 98.0% when pulling<PHONE_NUMBER>44d9c2ae948451d188179e9591b297 on davehenton:master into 6ed3f5f8c9921dc3c5f0daa7df8d821474de7bad on thejohnfreeman:master.


VM handling of leading whitespace in multiline strings does not match the spec.

In section 16.5, the specification says:
If the first line of a multiline string consists solely of the whitespace characters defined by the production WHITESPACE 20.1, possibly prefixed by \, then that line is ignored, including the new line at its end.
The specification doesn't state this explicitly, but I believe that this means that leading whitespace should be removed before any other processing has been done, such as processing escape sequences (otherwise, the whitespace characters would no longer be prefixed by backslashes). The VM appears to be processing escape characters and even performing interpolation before removing leading whitespace.
I have a test case (language/multiline_strings_test) that I will commit after this issue is created (I need the issue number for the status files).

Where are we at with this? Is it still a useful issue to have open?

It still not following the spec.
The spec says:
If the first line of a multiline string consists solely of the whitespace characters defined by the production **WHITESPACE** possibly prefixed by `\`, then that line is ignored, including the line break at its end.

The "line" is ended by the first occurrence of CR+LF, CR (not followed by LF) or LF.
If the string literal is
var x = """\
x""";

(That is: quotes-backslash-linefeed), then all characters before the line end are not whitespace potentially prefixed by backslash, because there is a backslash which is not followed by a whitespace on that line. Both analyzer and VM accepts this.
We can then argue whether backslash escaping should happen before or after line splitting. On the "before' case, the backslash-linefeed reduces to linefeed, then you break before that linefeed, but the \n should also become a linefeed, and we won't break at that.
In the after case, there is a backslash on the first line, prior to the first line break, which is not before a space.
I'd still prefer to completely disallow backslashes on the first line - we ignore the first line of a multiline string if it contains nothing by whitespace characters. It's a tokenization issue to determine when the string contents start. Having to do string escape interpretation in order to figure out where the string starts leaves us with questions like the above, and we'd be better off just not worrying.

Everything in the comments here makes me believe this is a bug for back-ends (or maybe CFE). I'm not sure why it's tagged as analyzer. Tentatively tagging with VM.

Back-ends don't parse Dart sources. This looks like a parser / CFE issue.
/cc @johnniwinther

So @lrhn what do we actually want here?

Honestly: I want to change the spec.
This is such a gnarly and idiosyncratic corner of the syntax, that I doubt anyone ever uses it. This issue documents that it's gnarly and not even implemented according to spec. And nobody noticed.
So it's mainly supporting evidence that changing the spec shouldn't matter
It's not really worth spending much time on being spec compliant for something that is never used, so I'd mark it P3 or P4 and carry on, unless you really want to work on it.


bgpv2: pass types.Router in path and policy reconcilers

This change passes only required field to Policy and Path reconcilers. Instead of passing BGPInstance, we pass only the Router interface which is required by underlying implementation.

/test


Include "snowflake_failover_group_grant" resource

Is your feature request related to a problem? Please describe.
We are looking for the following grants: https://docs.snowflake.com/en/user-guide/security-access-control-privileges#failover-group-privileges
Since we frequently add failover groups, we would need to do the grant manually every time now. This is quite error-prone so we want to do that via Terraform.
Describe the solution you'd like
We need a corresponding grant resource for "snowflake_failover_group" resource, which is currently missing.
It would be similar to "snowflake_warehouse" and "snowflake_warehouse_grant".
Describe alternatives you've considered
Doing it manually by running the required SQL but it is error prone.
Additional context

Hey 👋
Closing the issue as it's referring to the deprecated resource and we're not supporting them. Please, try to use the latest provider version with the non-deprecated resources (e.g. snowflake_grant_privileges_to_account_role). If the issue persists on the non-deprecated resource, please create a new GitHub issue. The migration guide may help during migration to the latest resources. Here's a list of the latest, non-deprecated, grant resources:
Resources
snowflake_grant_privileges_to_database_role
snowflake_grant_privileges_to_account_role
snowflake_grant_account_role
snowflake_grant_database_role
snowflake_grant_application_role (coming soon)
snowflake_grant_privileges_to_share
snowflake_grant_ownership (coming soon)
Data sources
snowflake_grants


Bug - Foundations 03 - Grouping Selectors: Wrong font style used for desired outcome image

Complete the following REQUIRED checkboxes:

[x] I have thoroughly read and understand The Odin Project Contributing Guide
[x] The title of this issue follows the Bug - location of bug: brief description of bug format, e.g. Bug - Grid exercises: CSS not properly linked to HTML files

The following checkbox is OPTIONAL:

[x] I would like to be assigned this issue to work on it

1. Description of the Bug:
On the README, the image presented as "desired outcome" uses Times New Roman for the button texts, whereas the description of the exercise specifies Helvetica as the first font in the list, which has higher precedence and will in most cases be used.
2. How To Reproduce:
Open README.md, or check the desired-outcome.png
3. Expected Behavior:
The image should look like this:

4. Desktop/Device:
Should be reproducible under any setup.
5. Additional Information:
Simple mismatch between expectation and actual exercise outcome, nothing that stops the progress.

I can work on this issue assign this to me.

@Shubham1450 are you still working on this issue?
If so please open up a PR as this issue seems to come up quite often on discord


Get this library listed at https://android-arsenal.com/

OMG... IT ACTUALLY HAPPENED!!! Contacts, Reborn is now a part of the Android Arsenal!!!

Here is the link to it; https://android-arsenal.com/details/1/8393
TIME TO ADD THE BADGE IN THE README 🔥 🔥🔥🔥🔥🔥🔥


update packages

:x: Build TheIdServer 8.3.0-fix-dependencies.1+41.build.3943 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/bb0d1e6d42 by @invalid-email-address)

:x: Build TheIdServer 8.3.0-fix-dependencies.1+42.build.3944 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/e94afd1572 by @aguacongas)

:white_check_mark: Build TheIdServer 8.3.0-fix-dependencies.1+43.build.3945 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/a65ae5ceba by @aguacongas)


Decompiler assigns fields in single line rather than separately

Describe the bug
Assembly code:
MOV word ptr [soundPrimaryBufferFormat ],0x1
MOV word ptr [soundPrimaryBufferFormat.nChannels ], 0x2

Generated code:
soundPrimaryBufferFormat._0_4_ = 0x20001;

Defined structure:
typedef struct {
  WORD  wFormatTag;
  WORD  nChannels;
  DWORD nSamplesPerSec;
  DWORD nAvgBytesPerSec;
  WORD  nBlockAlign;
  WORD  wBitsPerSample;
  WORD  cbSize;
} WAVEFORMATEX;

Same thing happens for nBlockAlign and wBitsPerSample
Expected behavior
soundPrimaryBufferFormat.wFormatTag= 0x1
soundPrimaryBufferFormat.nChannels= 0x2;

Screenshots

Attachments
invalid_combine_fields_xml.txt
Environment (please complete the following information):

OS: Windows 10
Java Version: 11.0.6
Ghidra Version: 9.1.2

This particular 'bug' gets worse when using large registers e.g. xmm


⚠ Bofrost.be has degraded performance

In 838a9f0, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1273 ms

Resolved: Bofrost.be performance has improved in 498903a.


Footer design suggestion

Is your feature request related to a problem? Please describe.
Made a few changes to the footer. Will change font style after initial changes are approved
Describe the solution you'd like

Describe alternatives you've considered
Add shadow effect on hover maybe?
Are you working on this? (Yes/No)
For GCI students under good first issue.

I think rather than Black; dark grey colour similar to the navbar would make it look more good and uniform.

Actually I was thinking of changing navbar's colour to black

Oh, okay. Then that's a different story.

Footer has been redesigned, now this can be closed @tachyons


Block contacts by phone number, name, email

I want to block phone book contacts by phone number, name, email.
So I have two questions:
1- Does it possible to block contacts in Flutter?
2- Can I use this library for contact blocking?

How are you  able to run it.I am getting error of permission

I'm interested in this answer also

Any solution from this case? I interesting implement it.

Any progress on this? I am interested in this development.

I'm interested in this answer also.

I'm interested in this answer also.


Update chain-infos.ts to add IBCX

related with https://github.com/osmosis-labs/osmosis-frontend/pull/1162

@gnujoow is it ready for launch? LMK

I think it still needs to be added in ibc-assets, and defining a pool:spot in prices.ts would be ideal, if we can get a pool set up.

Preview looks good. No errors. Just not sure if this can be added now, or if this should be displayed with the banner and background (which need additional review, and thus could make it have to wait).


1180 es6 modules student profile 2

Who is this PR for?
Developers
What problem does this PR fix?
Makes our JS code more consistent
What does this PR do?
Move BarChartSparkline and ProfileChart to ES6 modules.
Screenshot (if adding a client-side feature)
Checklists
Javascript QA
If the PR touches Javascript, follow this checklist. If the PR doesn't touch Javascript, you can delete this section.

[x] Author checked latest in IE - Student Profile
[x] Reviewer checked latest in IE - Student Profile

Ready for review, tests need to be re-run at another time because of time zone bug.

@gabivoicu This looks great!  👍 https://github.com/studentinsights/studentinsights/pull/1563 fixes up the merge conflict so we can merge, I'm not sure how to git-fu making that change on here.

@kevinrobinson Sounds good 👍🏻


lowdb-api do not accept async adapters

index.js ~ line 61
const db = lowdb(new Adapter(file)); // db = Promise
db._ - undefined
error

any work around for this while it's fixed?


feat: Add du to images

I missed this today. Unfortunately, k8s doesn't easily let you see Volume used disk space, only the capacity.
I don't think du is a security risk. If so, please close this PR with a reason (so I'm educated!).

I think including du is warranted! We will also need to include it in the busybox config so it is built.
https://github.com/strangelove-ventures/heighliner/blob/68aa47e5da31f9bf9ea582a313f6e8679acf2fa8/busybox.min.config#L235

@agouin Hope I enabled it correctly!


Bugs When using OpenAI api with Celery/redit

Confirm this is an issue with the Python library and not an underlying OpenAI API

[X] This is an issue with the Python library

Describe the bug
Hello,
There seem to be a bug when using the Openai API with Celery/redit shared task. When running the function bellow.
I’m not sure exactly why. When In use langchain with Mistral api, Ollama or with any my own custom models it works really well, but it crashes when I use anything touching the openai api. I tried bypassing langchain to work directly with the openai api and I still do get this error  Does someone have any idea why ?
To Reproduce

pip install Django
pip install celery
brew install redit // for macs
create a random view function that will call the task at task.py

Code snippets
in views.py
def some_view_fucntion(request):
    for x in range(2):
         some_ai_function.delay()

in task.py
@shared_task
def some_ai_function():
    message = "Hello, write a message about testing "
    print(message)
    completion = openai.chat.completions.create(
        model=“gpt-3.5-turbo”,
        messages=[
            {
                 “role”: “user”,
                 “content”: message,
            },
        ],
    )
    print(completion.choices[0].message.content)

RETURNS :
ERROR/MainProcess] Process ‘ForkPoolWorker-2’ pid:86900 exited with ‘signal 5 (SIGTRAP)’
ERROR/MainProcess] Task handler raised error: WorkerLostError(‘Worker exited prematurely: signal 5 (SIGTRAP) Job: 0.’)

Traceback (most recent call last):
File" …
/python3.11/site-packages/billiard/pool.py", line 1264, in mark_as_worker_lost
raise WorkerLostError(
billiard.exceptions.WorkerLostError: Worker exited prematurely: signal 5 (SIGTRAP) Job: 0.

OS
macOS (m2)
Python version
Python 3.11.5
Library version
openai 1.8.0
### Tasks
- [ ] Create a work around get_architecture()  for celery tasks on MacOS

I got where the bug is. When you are using using a macOS with redis (without docker your application), Mac blocks Celery from accessing platform.architecture() which causes the function to fail. For few good reasons I did not yet dockerised my application and I encountered the problem. Found a workaround but it's not ideal. Is there a way that in the new version of the openai api _based_client.py throuwould be workaround of get_architecture()  ?
Best,

Ah, yes, I understand. We'll ensure those aspects can fail safely. Thank you for the report.

I have same BUG on my Mac M1X.
This has a great impact on my local testing, including chating with GPT through celery.
Is in urgent need of repair.

Ah, yes, I understand. We'll ensure those aspects can fail safely. Thank you for the report.啊，是的，我明白了。我们将确保这些方面能够安全地失败。感谢您的报告。

Hi @zyz954489346
Yes I can see they have change the code but it cannot fail safely with (try/except) due to the fact that get_achitecture() does not lead only to a python error.
Anyway a quick workaroud you can  do is go on your site-packages>openai>_base_client.py>get_achritecture()  and just set it to the following :
def get_architecture() -> Arch:
return "arm64"
comment the reste of the function:
# try:
#     python_bitness, _ = platform.architecture()
#     machine = platform.machine().lower()
# except : (etc...)
Best,
Jean FP


Update deprecated package System.Diagnostics.DiagnosticSource

List of NuGet packages and version that you are using: Microsoft.ApplicationInsights 2.22.0
Runtime version (e.g. net461, net48, netcoreapp2.1, netcoreapp3.1, etc. You can find this information from the *.csproj file): All currently supported
Hosting environment (e.g. Azure Web App, App Service on Linux, Windows, Ubuntu, etc.): Not relevant

Describe the bug
The Microsoft.ApplicationInsights NuGet package depends on the System.Diagnostics.DiagnosticSource package in version 5.0.0. That version is deprecated and should be updated.
Note that the current package version 8.0.1 doesn't support the target frameworks net452 and net46 and more. So the target framework of Microsoft.ApplicationInsights has to be updated to net462 (netstandard2.0 can stay).
To Reproduce
Check the deprecation status of the NuGet package System.Diagnostics.DiagnosticSource in version 5.0.0

This is something the .NET project recently encountered as it has a dependency on Application Insights.  Applications Insights is included in the .NET build dependency graph when "source building" for inclusion in Linux distros.  We could easily patch this dependency as we exclude all Net FX tfms in our build.  I wanted to upstream this upgrade but quickly discovered this was going to require a lot of changes to upgrade the net452 and net46 tfms.


[GOBBLIN-2120] Mark dagActionsConclude correctly

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[X] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-2120

Description

[X] Here are some details about my PR, including screenshots (if applicable):
Mark dagActionsConclude as failed if the delete failed

Tests

[X] My PR adds the following unit tests OR does not need testing for this extremely good reason:
N/A

Commits

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

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

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

Project coverage is 38.74%. Comparing base (2ea9622) to head (4dd4d51).

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #4010       +/-   ##
=============================================
- Coverage     55.20%   38.74%   -16.47%
+ Complexity     1650     1596       -54
=============================================
  Files           319      388       +69
  Lines         11808    15965     +4157
  Branches       1205     1580      +375
=============================================
- Hits           6519     6186      -333
- Misses         4710     9283     +4573
+ Partials        579      496       -83

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


merge documentation of self certs and update-java-ca-certificates

@samq-ws This is ready to merge


Allow render function to be sourced from story meta

Issue: https://github.com/storybookjs/testing-react/issues/104
What Changed
CSF3 allows for the render function to bubble up story to meta in the same way as args/parameters/etc, so this should be taken into account when composing stories.
Checklist
Check the ones applicable to your change:

[x] Tests are updated
[x] Documentation is updated

Change Type
Indicate the type of change your pull request is:

[ ] documentation
[x] patch
[ ] minor
[ ] major

Thank you so much for this contribution @TeaSeaLancs ! Great catch 🚀


Misc fixes and LDAPS support

The search operation would hang if the server returned just a SearchResultDone message, which can happen under various circumstances. This is now fixed.
Attribute lists can now be specified in searches.
LDAPS is supported, with the following caveats:

The server cert must be valid, the CA verifiable by the client, and hostname verification must pass. Self-signed certs are not supported. (Those are dictated by native-tls, which tokio-tls uses as the TLS library.)
For the above to work, a TLS connection (obtained by connect_ssl(), named so as to follow the ancient LDAP nomenclature) must be requested by hostname, not IP address.
The hostname is resolved synchronously by std::net::ToSocketAddrs.

With the above changes, this crate is now usable for a surprising number of real-world problems.
Now, my plans for the crate are more extensive/ambitious: I'm going to almost completely overhaul the code, for the reasons of RFC compliance and ergonomy (which is why I'd love to discuss the plans on IRC). However, that will take time, and I believe that meanwhile there should be a usable LDAP subset for those who need it.


HHH-18106 - @CheckHQL is reporting error when named query contains valid Java constant

See Jira issue HHH-18106
When named query in entity method annotated with @CheckHQL contains valid Java constant, processor should not report error

Are we done with this one? Shall we merge it?

@cigaly? Status?

Yes, you can merge that.

Sorry, @cigaly I now notice there are conflicts, can you resolve them please and then I will merge?

OK

@gavinking I've resolved conflict and pushed commit.

Excellent, thanks @cigaly!


Initial commit for Real Dev Squad Members section

Issue #21
Feature Details:
Built the initial RealDevSquad Members Section for members-site referencing the previous members section
Previous

New

Closed my previous PR and made this new one


Generate and register new storage backend

This is a feature request.
Right now, StartApiServer flow is highly opinionated about a few things like the storage backend to be used (enforces etcd) or the options passed onto the apiserver. While this allows the apiserver to work out-of-the box rather easily, it limits more advanced scenarios, like implementing your own storage layer and easily point to it.
I suggest implementing a generator for a new storage layer, i.e. apiserver-boot create storage --name=xyz that generates stub code for xyz in pkg/storage/xyz and some code in cmd/apiserver that points out how it can be configured and enabled.

@pires Would you be interested in implementing this?

Yes. I should be able to pick it up during the last couple weeks of August.

Any updates on this? Is this kind of related to this issue/question I posted: https://github.com/kubernetes-incubator/apiserver-builder/issues/226 ?

Yes it is. And I think I can spare some cycles in the next couple months for this.

Appreciate your response. If you don't mind, can you give me an overview of what you intend to do. Details would be good too. I'm trying to look around for any links that can provide me ideas how to implement but I can't seem to find any. If you have any of those, would appreciate if you can share it with me. I've been trying to look for this topic to justify the use of API-Server aggregation in our project as some people in the team are kind of reluctant with ETCD, so our plan is to try other persistence mechanism. So would appreciate if you can share your ideas about this. TIA.

As I'm reviewing the k8s.io/apiserver code, it looks like storage.Interface is the one used for implementing the storage. pkg/registry/generic/NewRawStorage() is the one that instantiates the actual storage. However from looking at factory.Create in NewRawStorage(), it looks like it only supports ETCD and ETCD3. Unless my understanding is wrong, etcd seems to be hardcoded into the apiserver code without an opportunity to pass a custom implementation of storage.Interface. So as a question, is there another way to do this, i.e. implement your own custom storage.Interface and be able use it in the aggregated api server? Would appreciate if you can share your thoughts if have ideas regarding this already. I'm really eager to get some answers to this question, so I can justify the use of the aggregated api-server to my team who wants to use another persistent storage instead of etcd. TIA.

Not really, @klustria. While, conceptually, supporting a different storage engine makes sense, the fact is kubernetes/apiserver library is very opinionated about using etcd.
But here's an idea. There's a related gdoc link in the same comment and it might worth a try.

It seems the metrics-server proves it can be done! In this case, metrics are stored in-memory and no etcd is involved.

If anyone is still interested in following-up, maybe inspiration can be taken from how k3s relies on kvsql to provide support for different storage backends.

/remove-lifecycle rotten
@pires interesting point, i opened #362 to track this feature

Documentation would at least be useful here. There's a single example of non-etcd but there's no examples of how to use it.


Cannot open problem page

🐛 Bug Report
When I open the leetcode extension tab and click on any problem, it does not open the problem page but shows : Command "node "/home/n349/.vscode-oss/extensions/leetcode.vscode-leetcode-0.18.1/node_modules/vsc-leetcode-cli/bin/leetcode",show,11,-x" failed with exit code "0".
To Reproduce
Steps to reproduce the behavior.
Expected behavior
Open the clicked problem description page
Extension Output
Command "node "/home/n349/.vscode-oss/extensions/leetcode.vscode-leetcode-0.18.1/node_modules/vsc-leetcode-cli/bin/leetcode",show,3,-x" failed with exit code "0".
Guidance: Press Ctrl+Shift+U, and toggle the channel to LeetCode.
Your Environment

os: fedora 39
extension settings: Default
nodejs version: v20.10.0
vscode version: 1.87.0
extension version: v0.18.1

Check here

Yeah, I am also facing the same problem. Can't load problems list.
Downloading problems
(node:7424) Warning: Accessing non-existent property 'padLevels' of module exports inside circular dependency
(Use node --trace-warnings ... to show where the warning was created)
[ERROR] session expired, please login again [code=-1]

We changed some API policies, now it worked,


Improve UI/UX of the website

update the color palette, update new SVGs, make the website UI/UX better.

Hi @gauravsinhaweb, could you assign this issue to me, please? I would love to work on this.

@parth-panchal sure, go for it.

@gauravsinhaweb can you assign me an issue to add a new section of faqs?

@TishaJhabak1014 Yes, you can do that. Create an issue in https://github.com/gauravsinhaweb/EduSmart/issues and start working on it. Let me know if you need any help :)

@TishaJhabak1014 Yes, you can do that. Create an issue in https://github.com/gauravsinhaweb/EduSmart/issues and start working on it. Let me know if you need any help :)

Issue created. Plz assign me and add the labels. Thank you.

Hi @gauravsinhaweb , I want to work on this could you please assign this to me?

Hi @rajputarjun2001 , Go for it 👍

Hi, @parth-panchal , let me know if you're still working in it.

Hi, @parth-panchal , let me know if you're still working in it.

Hi. Due to other engagements I couldn't find the time to work on this, that's why I unassigned myself from the issue, so that others who want to work can be assigned. So currently I am not working on it.

Hi, @parth-panchal , let me know if you're still working in it.

Hi. Due to other engagements I couldn't find the time to work on this, that's why I unassigned myself from the issue, so that others who want to work can be assigned. So currently I am not working on it.

Thanks for letting me know about it. Feel free to PR anytime you want.

Hey @gauravsinhaweb ,if this issue is still active...I would like to work on it

Hey @gauravsinhaweb ,if this issue is still active...I would like to work on it

assigned to you


ntropy

Please make sure that the boxes below are checked before you submit your issue. If your issue is an implementation question, please ask your question on StackOverflow or join the Keras Slack channel and ask there instead of filing a GitHub issue.
Thank you!

[ ] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/keras-team/keras.git --upgrade --no-deps

[ ] If running on TensorFlow, check that you are up-to-date with the latest version. The installation instructions can be found here.

[ ] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[ ] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

Closing this issue due to lack of information to work with. Please feel free to reopen this issue by describing  your problem in detail. Thanks!


Header/Footer support

It would be nice to have the possibility to show a header and/or footer with some info about the event, the talk, or some other usual things that get shown on this kind of presentations.
While basic Reveal.js doesn't offer anything specific, we could add it as simple static HTML elements.

This has been completed in #7


SharedKernel project is not a SharedKernel

From the original documentation:
Designate with an explicit boundary some subset	of the domain	model that the teams
agree to share. Keep this kernel small.

SharedKernel is about sharing a domain models across bounded context. Not about sharing infrastructure code. The shared kernel in this project is rather a Common/Core and has nothing to do with Domain Driven Design. DDD does not care about infrastructure.
I know people have many different opinions on this topic, thanks to the vagueness of the original source. Still, a dependency to MediatR and other infrastructure libraries in the SharedKernel project seems wrong.

It's true DDD shouldn't care about infrastructure, and you'll find there are no external dependencies (to out-of-process systems, etc) in the SharedKernel. You could argue that things like common base classes and utilities might be better off in a differently-named but similarly-consumed library, but I've never seen any value in doing so. In my experience the number of actual domain model types that should be shared between bounded contexts is vanishingly small - maybe a common User class for instance. So having just that 0-2 types in one library and then having a separate library with the base types, etc. seems like overkill. Hence, just the one library.

thanks for your opinion.


Sdklens

Correct minor typos

@VolodymyrBg thank you for your PR, but at this point in time we are in flux with a lot of changes and we are not able to take external contribution. Many of the issues will be sorted by the time we have a stable build.
After that we will have capacity to handle external PRs.
Closing this PR for now.


n0rad.zsh-theme

add n0rad.zsh-theme, a not overloaded prompt with git_completion support if found

There is a bug


SNlM0e does not seem to be present when being fetched

I was trying to use this package (well done btw), and it kept not working, so I decided to look into the code, and found that SNlM0e does not seem to be there (in the html data) when being fetched? It could just be something on my side, please look into it, many thanks.

I was trying to use this package (well done btw), and it kept not working, so I decided to look into the code, and found that SNlM0e does not seem to be there (in the html data) when being fetched? It could just be something on my side, please look into it, many thanks.

Thank you, please check your cookies, it will cause this issue


Create APIs for processing and forming SysEx 7 and SysEx 8 messages

Dropping because the open source MIDI libraries do a good job of this already.


Ctrl + Enter not running code

System information:

Operating system (distribution) and version: Windows 10 Enterprise
DBeaver version: 22.0.2
Additional extensions: none

Connection specification:

Database name and version: Netezza <IP_ADDRESS>
Driver name: JDBC
Do you use tunnels or proxies (SSH, SOCKS, etc)? No

Describe the problem you're observing:
When I run ctrl + enter on a script with multiple statements (separated by ;) or hit the execute SQL statement button ▶ it just runs ; and shows:

This started happening with the latest upgrade from 22.0.1 to 22.0.2 today. My settings have not changed and show:

Steps to reproduce:
These only run correctly if I highlight the text for the query I want to run before hitting ctrl + enter or the run SQL button ▶
select * from _v_table
;

select * from _v_session
;

Include any warning/errors/backtraces from the logs

Hello @rjake,
Thanks for the bug report.

I'm experiencing a similar issue running Version <IP_ADDRESS>204021640 on Cloudera Impala.  I can't run multi-line SQL commands in the SQL Editor.  If I put all the commands on a single line, it runs fine, but if there are multiple lines, the SQL command does not run and I get the same response that @rjake described above.  Help!

It happens when you have a semicolon on a separate line after a query. E.g.
select * from tableA
;

Workaround is to put semicolon in the end of query.
This bug was already fixed. Hotfix patch was applied to version 22.0.2, you can re-download it from dbeaver.io website or from GitHub.

verified

@serge-rider @ShadelessFox I am still having this issue. I tried installing from both
https://dbeaver.io/files/ea/  (Early access)
and
https://dbeaver.io/download/ (Prod)
I see version <IP_ADDRESS>2 04 04 1542 (windows 64) and Check for Updates says there are no new versions. Is there another place I should download this from or maybe the 🐛 is still here?

@serge-rider @ShadelessFox I am still having this issue. I tried installing from both https://dbeaver.io/files/ea/ (Early access) and https://dbeaver.io/download/ (Prod)
I see version <IP_ADDRESS>2 04 04 1542 (windows 64) and Check for Updates says there are no new versions. Is there another place I should download this from or maybe the 🐛 is still here?

same here.  I'm not getting a new version and the bug still exists.

Hello @rjake @joshcsmith13,
Can you please provide a sample of a query causing the bug to appear?
Additionaly, please post a screenshot of the About dialog.

@ShadelessFox  I have the same issue. as @rjake @joshcsmith13. Same code example as in previous posts apply. If the delimiter is on a new line or commented line it doesn't work. To be clear I get the same response as the original poster.
Both of these don't work:
select * from tableA
;

select * from tableB
--commented stuff ;

Is this because we're running a version from the future??  😄

My details are the same as @joshcsmith13

I see a new [duplicate] ticket attached to this one, but nothing new.
Has anybody heard anything?

Exactly. On mysql and postgresql I have no problems. On SAP HANA I have the problems (https://github.com/dbeaver/dbeaver/issues/16135). I use the latest version 22.0.2.

I'm hoping that latest status update from @dbeaver-pm  to remove the "wait for response" label means that somebody from the team will be looking into this.  It is confusing that we were originally told there was a hotfix, but nobody has had any luck with it.

Please check the "Script delimiter redefiner" connection setting in the "Advanced parameters".
Set it to empty if it's not.

The New DBeaver version is available.
https://dbeaver.io/download/
Please check your issue in the new version.

Please check the "Script delimiter redefiner" connection setting in the "Advanced parameters". Set it to empty if it's not. I hope it helps.

this solution worked for me using Cloudera Impala JDBC Driver 2.5.41.  thanks.

Just downloaded the version 22.0.3 and the problem still occurring with informix connection.
The solution mentioned above , about set delimiter redefiner on driver settings works for me too.
I have a ton of connections, but if set from "Database -> Driver Manager" , you set the configuration to all connection.
That was a acceptable solution....
annoying...but acceptable.

We are in the fixing process here https://github.com/dbeaver/dbeaver/issues/16212

This is caused by previous default values in mentioned above script delimeter redifiner. Should be fixed in 22.0.3 with #16218

Yay - it was fixed when I downloaded <IP_ADDRESS>205011839!


Authentication instance is not achievable from IsAuthenticatedUseCase

MOB 3164
Jira issue:
https://glia.atlassian.net/browse/MOB-xxx
What was solved?
Release notes:

[ ] Feature
[x] Ignore
[ ] Release notes (Is it clear from the description here?)
[ ] Migration guide (If changes are needed for integrator already using the SDK - what needs to be communicated? Add underneath please)

Additional info:

[ ] Tests updated? Added? Unit, acceptance, snapshots?
[ ] Logging for future troubleshooting of client issues added?

Screenshots:

@DavDo Could you please add the steps to reproduce so that we I could add it to the exploratory session scenarios?

Steps to reproduce:

Open testing app
Authenticate
Open the messaging welcome screen(start new messaging flow) screen

Expected result: A messaging welcome screen should be displayed.
Actual result: An error message is displayed.


Brazilian Portuguese translation of Basic Algorithm Scripting

[ ] Get Set for our Algorithm Challenges
[ ]  Reverse a String
[ ]  Factorialize a Number
[ ]  Check for Palindromes
[ ]  Find the Longest Word in a String
[ ]  Title Case a Sentence
[ ]  Return Largest Numbers in Arrays
[ ]  Confirm the Ending
[ ]  Repeat a string repeat a string
[ ]  Truncate a string
[ ]  Chunky Monkey
[ ]  Slasher Flick
[ ]  Mutations
[ ]  Falsy Bouncer
[ ]  Seek and Destroy
[ ]  Where do I belong
[ ] Caesar's Cipher

Working on it!

Closed in favor of https://github.com/FreeCodeCamp/TranslationExpansion/issues/51


Error building RNFBSDKExample on ios - issue with #import <FBSDKCoreKit/FBSDKCoreKit-swift.h>

🐛 Bug Report
Error building RNFBSDKExample on ios.  Got the error FBSDKApplicationDelegate not found.  By replacing
#import <FBSDKCoreKit/FBSDKCoreKit-swift.h> with
#import <FBSDKCoreKit/FBSDKCoreKit.h>
in ios/RNFBSDKExample/AppDelegate.mm it built and ran
To Reproduce
Build the example app for ios as per instructions, however I used Xcode to figure out what the problem was
Expected Behavior
It should have built.
Code Example
replace
#import <FBSDKCoreKit/FBSDKCoreKit-swift.h>
with
#import <FBSDKCoreKit/FBSDKCoreKit.h>
in ios/RNFBSDKExample/AppDelegate.mm
Environment
info Fetching system and libraries information...
System:
OS: macOS 13.2
CPU: (4) x64 Intel(R) Core(TM) i5-7500 CPU @ 3.40GHz
Memory: 32.29 MB / 16.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 18.14.2 - /var/folders/92/x0_tgnpn6vzdq5tr3lfqf4j40000gt/T/yarn--1677175610834-0.8988226430218316/node
Yarn: 1.22.19 - /var/folders/92/x0_tgnpn6vzdq5tr3lfqf4j40000gt/T/yarn--1677175610834-0.8988226430218316/yarn
npm: 8.12.1 - ~/react-native-fbsdk-next/node_modules/.bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.11.3 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.1, iOS 16.1, macOS 13.0, tvOS 16.1, watchOS 9.1
Android SDK:
API Levels: 29, 30, 31, 33
Build Tools: 29.0.2, 30.0.2, 30.0.3, 33.0.0, 34.0.0
System Images: android-30 | Google APIs Intel x86 Atom
Android NDK: Not Found
IDEs:
Android Studio: 2021.3 AI-213.7172.25.2113.9123335
Xcode: 14.1/14B47b - /usr/bin/xcodebuild
Languages:
Java: 17.0.6 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.1.0 => 18.1.0
react-native: ^0.68.2 => 0.68.2
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found

YES. I'm having the exact same issue. Everything was working fine when my versions where:
"react-native": "0.70.6",
"react-native-fbsdk-next": "^8.0.5",

However, once I updated react native to latest, I was forced to update my Facebook module too, so now I have:
"react-native": "0.71.3",
"react-native-fbsdk-next": "^11.1.0",

And I get the error that @sharived is getting
I also tried using v10 where I'd import an objective-c file instead of a swift file, and got the same error.

@mikeislearning did you try changing the include
#import <FBSDKCoreKit/FBSDKCoreKit.h>

@sharived
So I'm able to get it to build now with the swift file, due to one key change. I now import:
#import <FBSDKCoreKit/FBSDKCoreKit-Swift.h>

With Swift capitalized, then it builds without any errors.
However, it still seems to fail actually loading the app due to Firebase not loading correctly


Binary Search

Binary Search using Python

Linear Search using Python

@all-contributors please add  amanjhurani  for ideas

@all-contributors please add  #4PRITI24  for code

@all-contributors please add PRITI24  for code

@all-contributors please add  tusharkashyap63  for design

@all-contributors please add TastySalad for code


Improve documentation

https://github.com/spring-cloud-stream-app-starters/app-starters-release/issues/99.
Added a Test related to one of the examples.
Last 3 commits attempt to conform to original formatting. Still some issues. Please squash/fix during merge if necessary.

Merged as https://github.com/spring-cloud-stream-app-starters/httpclient/commit/0cc71c28ccb069e613d4175d4e2c18a992776e6e


Add animated rail example

please review the created example, the example was created using beamer widget so it can be used with other navigation libraries or default navigation

Now as I pushed the "location stack" feature, you can have just
onWillPop: () async {
  bool didPopLocation = _beamerKey.currentState.routerDelegate.beamBack();
  return Future.value(!didPopLocation);
},

and just check _beamerKey.currentState.routerDelegate.currentLocation is ... to see which menu item should be highlighted.

that is cool well done, I will try asap


Refactor View and ViewDescriptor.

Based on our ongoing discussion on API design, a few updates on View and ViewDescriptor:

Rename ViewDescriptor to View, rename View to ViewData
Rename ViewDescriptor.TagKeys to View.Dimensions

The diff looks too huge now, I'll try to break it down to individual commits.

Codecov Report

Merging #404 into master will not change coverage.
The diff coverage is 97.84%.

@@            Coverage Diff            @@
##             master     #404   +/-   ##
=========================================
  Coverage     90.02%   90.02%
  Complexity      583      583
=========================================
  Files            98       98
  Lines          2036     2036
  Branches        210      210
=========================================
  Hits           1833     1833
  Misses          150      150
  Partials         53       53

Impacted Files
Coverage Δ
Complexity Δ

...src/main/java/io/opencensus/stats/ViewManager.java
100% <ø> (ø)
1 <0> (ø)
:arrow_down:

...rc/main/java/io/opencensus/stats/StatsManager.java
100% <100%> (ø)
6 <5> (ø)
:arrow_down:

...ain/java/io/opencensus/stats/RpcViewConstants.java
100% <100%> (ø)
2 <0> (ø)
:arrow_down:

core/src/main/java/io/opencensus/stats/View.java
100% <100%> (ø)
2 <1> (+1)
:arrow_up:

...main/java/io/opencensus/stats/ViewManagerImpl.java
100% <100%> (ø)
4 <2> (ø)
:arrow_down:

...re/src/main/java/io/opencensus/stats/ViewData.java
100% <100%> (ø)
1 <1> (?)

...main/java/io/opencensus/stats/MutableViewData.java
89.28% <90%> (ø)
2 <1> (?)

...ain/java/io/opencensus/stats/MeasureToViewMap.java
92.68% <90.9%> (ø)
12 <8> (ø)
: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 39ac27f...c00759b. Read the comment docs.

Diff is too huge and difficult to review, will split this PR.


Add Google Analytics Event tracking

We should figure out which events to track in GA. ex. What services were selected

The first step is for Dan to decide on what should be captured. (Dan please don't close the ticket. We can use it to track the implementation after you're ready instead)

Things/Events to be captured

Language (English/French)

Filter Combinations clicked (so for a given session, which filters/values did the user click on)

it would be great if we could also send an event with the values of the number of resulting benefits, even better would be extra information as to which benefits were displayed)
Exit/Drop Off Event (did the user click on the final benefit tab)

Comprehension:
Adding buttons at the footer with a label "Did you find the information you're looking for: Y/N" to register Y/N

Added Feedback bar in #191 and we are tracking language changes already

Closed for now with #272


Coredata relationship not getting synced when un-publishing and publishing an entry with reference

In the contetful portal, when i un-publish an entry which has reference to other entries and then republish it again, the reference between them are re-instated. However, when syncing this does not happen.
In the iOS DB when entry is unpublished, the relationship is nullified according to delete rule, however when the entry is republished the relationship is not getting synced.
For example - suppose we have an 'Article' entry which references another 'Category' entry. If i unpublish the 'category' entry it is getting synced to the local DB and the relationship between them is nullified, however when i republish the 'Category' entry again, the entry itself is synced but the relationship between 'article' entry and 'category' entry is lost (in contentful portal this relationship exists as before). It is not getting synced.
In case of Android it is working without issues and the relationship is also getting synced.

@pravking please make sure you reset sync token and do a full sync again to verify whether republishing works for you. Let me know if everything is okay.

Thanks @tomkowz. I’ll try and let you know.
So for apps in production also we have to do a full re-sync by resetting sync token for this fix in the sdk update to work right?

Yes, I'd suggest to reset sync token on the prod and perform full resync. Not ideal, I know, but I'd recommend that.


Implement coverage and codecov

Codecov Report

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

:exclamation: Current head 9aa61c3 differs from pull request most recent head 87d92d8. Consider uploading reports for the commit 87d92d8 to get more accurate results

@@            Coverage Diff            @@
##             master      #24   +/-   ##
=========================================
  Coverage          ?   61.13%
=========================================
  Files             ?       12
  Lines             ?      193
  Branches          ?        0
=========================================
  Hits              ?      118
  Misses            ?       75
  Partials          ?        0

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 4fa29bd...87d92d8. Read the comment docs.


USB-C can’t hotplug after sleep

Sorry sir can I ask you some question. I had created EFI for my XPS9360 laptop base on your EFI but there is just one problem my laptop can't not hotplug USB-C. It just work after I turn on the machine and boot into MACOS BIGSUR. I hope you can hlelp me this case

Support on Big Sur is still under maintenance, please try the previous build.

Thanks for your advice.I have tried catalina but still the same issue. Can your laptop work nomally after sleep. ??

After sleep, USB-C is required to pre-plug to work normally

I also use TbtForcePower.efi driver to enable thunderbolt 3 and it works just before sleep. Have you try it???

After sleep the only thing I can hotplug is video. Anything else like data thunderbolt device cant not inject anymore

And the IO tree show the thunderbolt is RP01


Question, the output of global_pool are 1024-dimensional

Thank you for your contribution! As far as i know, the output of global_pool are 1024-dimensional. But in the paper it's 2048-dimensional. I wonder what the problem is?

2048 means two-stream feature, concatenate 1024 RGB feature and 1024 flow feature.

The models used by the author are not available online now.

Is there any update ?

Is there any update ?


chore: use Map for compiler memory cache

Summary
Maps are about twice as fast for larger data sets in Chrome 80, very much likely this applies to Node as well: https://stackoverflow.com/a/55711780/1850276
Test plan
Green CI
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
N.A.

Pull Request Test Coverage Report for Build 4586

7 of 8 (87.5%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.09%) to 92.794%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/compiler/instance.ts
1
2
50.0%

Totals

Change from base Build 4582:
-0.09%

Covered Lines:
1063

Relevant Lines:
1099

💛  - Coveralls


[Event Request] table 5901 "Service Item Line" - Event for trigger procedure UpdateResponseTimeHours()

Can we get the following event for the "Service Item Line" table: OnBeforeUpdateResponseTimeHours
procedure UpdateResponseTimeHours()
    var
        ///
        IsHandled: Boolean; ///NEW
    ///-
    begin
        ///-
        //OnBeforeUpdateResponseTimeHours(Rec);
        IsHandled := false; ///NEW
        OnBeforeUpdateResponseTimeHours(Rec, IsHandled, SkipResponseTimeHrsUpdate, xRec); ///NEW
        if IsHandled then ///NEW
            exit;
        ///-

        if not SkipResponseTimeHrsUpdate then begin
            if "Response Time (Hours)" <> xRec."Response Time (Hours)" then
                Validate("Response Time (Hours)", CalculateResponseTimeHours)
            else
                if "Response Date" = 0D then
                    Validate("Response Time (Hours)", CalculateResponseTimeHours)
                else
                    "Response Time (Hours)" := CalculateResponseTimeHours;
            SkipResponseTimeHrsUpdate := false
        end;
    end;

   [IntegrationEvent(false, false)]
    ///
    //local procedure OnBeforeUpdateResponseTimeHours(var ServiceItemLine: Record "Service Item Line")
    local procedure OnBeforeUpdateResponseTimeHours(var ServiceItemLine: Record "Service Item Line"; var IsHandled: Boolean; SkipResponseTimeHrsUpdate: Boolean; xRec: Record "Service Item Line") ///NEW
    ///-
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Allow dropping all existing database tables before import

Is your feature request related to a problem? Please describe.
In #810 the database used to be dropped automatically when importing a database. This functionality has changed. It's useful to drop all tables, especially when testing release updates and needing to revert to a previous database up over and over.
Describe the solution you'd like
Add a -d --drop flag to --import-db so that I can import a database from source and drop all tables.
ddev import-db --src --drop

Actually, I'd be of the opinion that drop should be the default, and no-drop an option. I was swayed too easily by that PR that changed it. And a flag would have been a better idea in the first place.

So change to drop by default, which reverts back to previous behavior (that most think might still be happening, unaware that it no longer drops) and adds a --no-drop flag to prevent dropping tables

Yes, I'd agree with that. Obviously we'd need to communicate it well, but I think that's the most user-friendly solution.


Validation pattern did not match (Regex)

I have a fairly complex nested blockchain schema where a block can be of multiple types with a variety of properties in each type.
Corvus.JsonSchema handles this very well in terms of type generation and validation (CompositionOneOfValidationHandler) however I am seeing some strange behaviour when validating one of the block type properties causing the block to appear as invalid due to properties validation pattern not matching. "Validation pattern - ada did not match  '^[0-9a-f]56$'", this could actually be the schema syntax where it has 'ada' as required but enforces the property names pattern at the same time. (^[0-9a-f]{56}$)
Higher up this then causes me an issue because the RollForward validate function is then returning false, when I check IsRollForward after parsing, despite this being a RollForward operation. (CorvusValidation.ObjectValidationHandler)
Schema (Refs inline for brevity)
"Value":
    { "title": "Value"
    , "type": "object"
    , "propertyNames": { "pattern": "^[0-9a-f]{56}$" }
    , "additionalProperties":
      { "type": "object"
      , "propertyNames": { "pattern": "^[0-9a-f]{0,64}$" }
      , "additionalProperties": { "$ref": "AssetQuantity" }
      }
    , "required": [ "ada" ]
    , "properties":
      { "ada":
        { "type": "object"
        , "additionalProperties": false
        , "required": [ "lovelace" ]
        , "properties":
          { "lovelace": { "type": "integer" }
          }
        }
      }
    }

Example Data
{{"ada":{"lovelace":2922180},"160a880d9fc45380737cb7e57ff859763230aab28b3ef6a84007bfcc":{"4d495241":500},"16fdd33c86af604e837ae57d79d5f0f1156406086db5f16afb3fcf51":{"44474f4c44":25000000},"5ad8deb64bfec21ad2d96e1270b5873d0c4d0f231b928b4c39eb2435":{"61646f736961":500000000},"5dac8536653edc12f6f5e1045d8164b9f59998d3bdc300fc92843489":{"4e4d4b52":80000000},"804f5544c1962a40546827cab750a88404dc7108c0f588b72964754f":{"56594649":500000},"a0028f350aaabe0545fdcb56b039bfb08e4bb4d8c4d7c3c7d481c235":{"484f534b59":2000000},"af2e27f580f7f08e93190a81f72462f153026d06450924726645891b":{"44524950":1350000000},"b6a7467ea1deb012808ef4e87b5ff371e85f7142d7b356a40d9b42a0":{"436f726e75636f70696173205b76696120436861696e506f72742e696f5d":5000000},"c0ee29a85b13209423b10447d3c2e6a50641a15c57770e27cb9d5073":{"57696e67526964657273":500000},"ea153b5d4864af15a1079a94a0e2486d6376fa28aafad272d15b243a":{"0014df10536861726473":1000000}}}
public bool IsRollForward
            {
                get
                {
                    return this.As<Generated.NextBlockResponse.ResultEntity.RollForward>().IsValid();
                }
            }

I was going to ask about the correct approach when using Corvus.JsonSchema for simply type generation from a specified schema, but also the possibility of opting to just reading data without performing additional validation, but still  ensuring the mapped data remains exactly as it was read.

So that looks like a problem with your schema definition.
You can't have a propertyNames constraint with that pattern on Value and then also have properties on Value that do not conform to that pattern - that will always be false.
You have correctly applied the pattern in your additionalProperties subschema, which should be fine.

Thank you for the quick reply. It helped me dig a bit deeper into the issue. Since then, I managed to work around it by manually extracting the direction property and using AsRollForward, and I did the same for the block type.
The main issue seemed to be with the autogenerated IsRollForward property, which has the IsValid() method chained at the end. At a glance I think this pattern appears for all types generated based on the oneOf schema options. The IsValid() method returns false because the schema is formatted incorrectly, which technically makes sense as it doesn't conform. However, I was relying on this property to determine which type to act upon.
            /// Gets the instance as a <see cref="Generated.Ogmios.NextBlockResponse.ResultEntity.RollForward" />.
            /// </summary>
            publicNextBlockResponse.ResultEntity.RollForward AsRollForward
            {
                get
                {
                    return this.As<NextBlockResponse.ResultEntity.RollForward>();
                }
            }

            /// <summary>
            /// Gets a value indicating whether the instance is a <see cref="NextBlockResponse.ResultEntity.RollForward" />.
            /// </summary>
            public bool IsRollForward
            {
                get
                {
                    return this.As<NextBlockResponse.ResultEntity.RollForward>().IsValid();
                }
            }

Thanks for all the help, much appreciated


Add option to disable synchronization for "Assigned Label" property

The synchronization is enabled by default to match original behavior.

Joel Johnson » ez-templates #17 SUCCESS
This pull request looks good
(what's this?)


Cache strategy using SoftReferences

I am curious to see how this performs in the real world. If it works well and later we move this portion of code to JavaParser we can replicate some of the Guava features. If not we can always go back to a simple HashMap. What do you think @matozoid ?

This may not be the best option. But, in my case, I had to completely remove cache then my memory don't overflows.
I'm parsing a large source graph with some thousand classes and even with 6GB I run in GC Limit Overhead. After disabling all caching, the entire process runs (slow) but using default memory limits (nothing setted).
The new culprit seems to be JavaParserFacade.cacheWithLambdasSolved and cacheWithoutLambdasSolved, but I don't know how to implement the softReference cache since cacheWithLambdasSolved needs two passes sometimes.

I personally like this as a simple (simplistic?) cache: https://stackoverflow.com/questions/27475797/use-linkedhashmap-to-implement-lru-cache

I would be inclined to accept this PR because it requires less code and it does not add new dependencies, with the possibility to move to the approach indicated by @matozoid if we merge into JavaParser. Would this be acceptable @matozoid ?

We have evaluated this commit by indexing a the sourcecode of archi with Sourcetrail. Before that we had the issue that once the heap was full the indexer would become very slow. This pull request solves that issue.
Moreover it may help to reduce the required memory for running the tests again.

@ftomassetti it's your project, of course it's acceptable :-)

It is more our project and anyway it is pointless without JavaParser, so it is better to agree on these things :D
Ok, I am going to merge it and if later we need to the migration I will do the implementation suggested by @matozoid
Thanks @bkmeneguello for your contribution: it is very appreciated!


Add .husky/* text eol=lf

Personally I would change everything to LF, I like the simplicity
I get that some windows-specific files can be crlf, like .bat files,
but I'm not sure why most files are not just defaulted to LF

Does Husky require LF?

Actually I saw something about sh scripts not working in cygwin if using crlf.
I think VonC on stack overflow recommends setting an explicit eol for all file types.
I'm curious what is the benefit of adding something like:
*.foo text
What's the benefit of not setting an explicit eol?

For me personally, I turn to this project to help me resolve eol issues. Say, for some odd reason, lots of files have the eol changed.
Maybe core.eol can effectively set all your text files to lf or crlf

So core eol can drive the default eol. Of course, users still have to set that git config somehow


Error when freeing MeshInstances built in another thread on the main thread.

Godot version:
Godot 3.2.3-stable
OS/device including version:
Linux Mint 20.1
Issue description:
I'm not sure whether this is a problem with Godot, but I'm pretty sure the code should be ok, hence my doubt - I'm trying to use a separate thread to generate mesh instances that have textures on them, and I'm sending the results to the main thread to add them to the scene tree. But some of these meshes I don't want anymore by the time they are built, so I free them in the main thread.
The problem somehow arises when I free the mesh instances in the main thread, as errors like the following get spammed out:
ERROR: get: Condition "!id_map.has(p_rid.get_data())" is true. Returned: __null
   At: ./core/rid.h:150.
ERROR: material_set_param: Condition "!material" is true.
   At: drivers/gles3/rasterizer_storage_gles3.cpp:2648.

ERROR: getornull: Condition "!id_map.has(p_rid.get_data())" is true. Returned: __null
   At: ./core/rid.h:160.
ERROR: mesh_clear: Condition "!mesh" is true.
   At: drivers/gles3/rasterizer_storage_gles3.cpp:4327.

This happens with both GLES2 and GLES3.
Steps to reproduce:

Create a script.
Start a thread that generates meshes with materials (that also include a texture) from scratch.
Send the meshes to the main thread with a mutex.
Free the meshes in the main thread.

Minimal reproduction project:
ThreadBugProject.zip

See https://github.com/godotengine/godot/issues/20964.

Ah, I followed the issue you mentioned, and switching the Rendering/Thread/thread_model setting to multi-threaded worked for the project I provided! I didn't know that setting existed before.
My personal project seems to straight-up crash with this fix though, but I'm guessing that's a separate issue.
Thank you!

@drwhut Good to hear it worked :slightly_smiling_face:  I'll close this as a duplicate of #20964 then
.


TCP/IP

Would TCP/IP support be difficult to add?

I think it's just a matter of putting in the effort and hours.
If you have the manuals handy, I don't see why it would be difficult to add support for TCP/IP.

Thanks!


Not compatible with scipy 1.0.0

Needed to downgrade to 0.19.1. Error below:
$ ulog_tools_pid_design --url http://review.px4.io/download?log=d3980cdd-c8ef-478b-a838-fba9956009c4 out.gains

Traceback (most recent call last):
  File "/home/tony/.local/bin/ulog_tools_pid_design", line 11, in <module>
    sys.exit(main())
  File "/home/tony/.local/lib/python3.5/site-packages/ulog_tools/control_opt.py", line 84, in main
    est_delay=est_delay
  File "/home/tony/.local/lib/python3.5/site-packages/ulog_tools/control_opt.py", line 29, in attitude_loop_design
    G=G_rate_ol, K_guess=[0.2, 0.2, 0], dcut_hz=dcut_hz)
  File "/home/tony/.local/lib/python3.5/site-packages/ulog_tools/_lqrofb.py", line 157, in pid_design
    K = lqr_ofb_design(K_guess, ss_open, verbose)
  File "/home/tony/.local/lib/python3.5/site-packages/ulog_tools/_lqrofb.py", line 106, in lqr_ofb_design
    constraints=constraints
  File "/home/tony/.local/lib/python3.5/site-packages/scipy/optimize/_minimize.py", line 495, in minimize
    constraints, callback=callback, **options)
  File "/home/tony/.local/lib/python3.5/site-packages/scipy/optimize/slsqp.py", line 378, in _minimize_slsqp
    fx = func(x)
  File "/home/tony/.local/lib/python3.5/site-packages/scipy/optimize/optimize.py", line 292, in function_wrapper
    return function(*(wrapper_args + args))
  File "/home/tony/.local/lib/python3.5/site-packages/ulog_tools/_lqrofb.py", line 55, in lqr_ofb_cost
    P = scipy.linalg.solve_lyapunov(A_c.T, -Q_c)
AttributeError: module 'scipy.linalg' has no attribute 'solve_lyapunov'

@lamping7 Can you please help with the terminal command for downgrading scipy.linalg?

@vmishra9 try pip install --user scipy==0.19.1 (assuming you didn't install the package with sudo)

Thanks. I get this comment:
Requirement already satisfied: scipy==0.19.1 in /anaconda3/lib/python3.6/site-packages
But I get the same error as you when I run "ulog_tools_pid_design"

I can't step you through troubleshooting this, but your python path is pointing to different places. It looks like you have multiple versions of scipy installed for different environments. I'm not sure of your env or how you installed the package.


Javascript Error

Duplicate: https://github.com/4gray/iptvnator/issues/257 and https://github.com/4gray/iptvnator/issues/248


fix safetensor saving

Waht does this PR do ?
This PR fixes the save_model function for safetensors. It was never tested so it had a couple of problems.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Thanks for implementing the suggestions, LGTM.


do not read books made by me, google doc, calibre, sigil and more - Android

As the title says, do not read books made by me, google doc, caliber, sigil and more
Nose not read correctly because the epub I do, even just edit text in a epub he read properly and then do not read the amendment.
Sigil use a partition with Windows 10, is configured to EPUB 3, I do not understand why not read my epub correctly.
please help!
https://youtu.be/c40JV2iBPoI

@pablo384 can you please send me crash report? Or if possible can you share that project on github?

@mahavir-jain of course, here the link: https://github.com/pablo384/ProEduka.git

@pablo384 fixed in 0.2.1 version.. please check..