View scroll under keyboard after first key press on iOS

As the video shown, the view scroll under the keyboard after the first key pressed on iOS. Could someone suggest a solution? Thanks in advance.
App run on iPhone 13 simulator with iOS 15.2
react-native: 0.66.1
react-native-keyboard-aware-scroll-view: 0.9.5

https://user-images.githubusercontent.com/50413696/171209683-031099b2-138c-4aff-bd27-54af251cfb70.mp4

having same issue
I'm seeing that the issue happens after i add text to input
and resets itself when focusing
so if i only focus from one text input to another, the scroll is correct
if i change text in one text input, the scroll is wrong
if i focus back and forth, even after the first change, the scroll is correct again

adding worked for me
keyboardOpeningTime={Number.MAX_SAFE_INTEGER}

mentioned in #418

adding worked for me
keyboardOpeningTime={Number.MAX_SAFE_INTEGER}

mentioned in #418

this just makes the view do nothing for me. it doesn't scroll to avoid the keyboard which... is the point of the library?

adding worked for me
keyboardOpeningTime={Number.MAX_SAFE_INTEGER}

mentioned in #418

This worked for me, thx!
RN: 0.68
"react-native-keyboard-aware-scroll-view": "0.9.5"


Refactor readTableFile & writeTableFile

Current Implementation

Currently, these functions are implemented as DataKit.importTableFile and writeTableAndHeader respectively and have differing feature sets.
Suggested Enhancement

[ ] Rename the functions to readTableFile and writeTableFile.
[ ] Generalize the translation of data types to format strings as defined for textscan.

Expected Benefits

One stop functions for reading and writing fully defined MATLAB tables without losing metadata defined as tbl.Properties.[...] as well as the definition of the column data types.
Is this something you're interested in working on

Yes.

These functions should probably also be moved to the Utilities repo.

This is now implemented in the Utilities repo.

[ ] Implement it here.


option to make dupe chapters have different color or add filter to make only dupes appear

Describe your suggested feature

It would be used to spot duplicated chapters in between various ones.
That way, we would be able to choose which chapter to filter out, without having to look through a lot of chapters.
OR
add a filter to only show the duplicated chapters, for a faster way to look for them without having the possibility to miss one or another in an entry that has a lot of chapters.

the second option would be the best in my opinion, but any of them would be appreciated.
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an official extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to version 1.11.2.
[X] I will fill out all of the requested information in this form.

something like this ?
tachiyomi#8804


Update Default OpenAI LLM to "gpt-4-1106-preview" with Parallel Functionality

This pull request updates the GlobalSettings configuration to use the "gpt-4-1106-preview" model of OpenAI's language learning models (LLMs). This change incorporates the latest developer release from OpenAI dated 11/6/23, which introduces support for parallel function calling. This enhancement promises to significantly improve performance by allowing concurrent operations, making our application more efficient and responsive.

Hey, thanks for the PR, but...
we cant just swap default GPT-3.5 model for GPT4
there would be a significant increase in costs, and take into account that a some ppl use this also for simple functions like name extraction etc.
What I though of doing here is to swap the default GPT-4 model for the new, preview version
Than you can use GPT4 wherever you want either by using prompt_type argument, as shown here Using predefined prompt types
or by choosing llm_selector_rule (will automatically select the GPT4 model by estimating the context window and prediction size) as shown here: #automatic-llm-selection
If you want to use GPT4 as a default for your whole repo, that is certainly possible by defining global settings somewhere in your code, as described here:
https://github.com/ju-bezdek/langchain-decorators#defining-custom-settings

Ok, so new version 0.3.0 has been published.. gpt-4-preview is currently the default gpt-4 model
but unless you specify it by setting prompt type, or llm_selector_key="gpt-4", the lastest gpt-3.5 with 16k window will be used


Put No evaluators for function symbol warning in debug info

It is very common to have constant functions that have no evaluators in the spec. It is very annoying to see a list of warning No evaluators for function symbol: ... in the command line.

@yzhang90 If the warning is not displayed by default, then there is no point in having it at all. How do you feel about these alternatives?

A symbol attribute that turns off the warning for the associated symbol.
Turn off the warning (always) for constant functions.

I think an attribute like no-evaluator will be more general.

another solution is to print 'No evaluators' warnings once at the beginning and do not print any `'No evaluators' warnings again during the execution.


There are still locking problems.


Write the "The Tuple Type" concept

Official docs:
https://book.cairo-lang.org/ch02-02-data-types.html#the-tuple-type
The idea is to have a short and to-the-point article about the concept. Avoid copy/pasting everything from The Cairo Book; the Book should be used as a more in-depth learning resource for students after they finish the concept on Exercism (or they can use it in parallel).

Exercism Concepts docs:
https://exercism.org/docs/building/tracks/concepts
Use the following Rust's concept as inspiration for the format:
https://github.com/exercism/rust/blob/main/concepts/structs/about.md

@misicnenad Please assign it to me.

@rajeebkm need any help?
Let me know if you can't work on this at the moment, so we can free this issue for other contributors

Hi @misicnenad I have raised a PR for tuple type. It seems some merge conflicts are there.
https://github.com/exercism/cairo/pull/106
Please review.

Thanks


[Competitive programming]: K- Concatenation Problem codechef

Description 📜
Added solution to K-concatenation problem from codechef using C++ language
Fixes #5052

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[√] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[√] Competitive #Programming

Checklist ✅

[√] I follow Contributing Guidelines & Code of conduct of this project.
[√] I have performed a self-review of my own code or work.
[√] I have commented my code, particularly in hard-to-understand areas.
[√] My changes generates no new warnings.
[√] I'm GWOC'21 contributor

Bold replaced with heading tags, removed unnecessary spaces except the ones in input and output test cases because they come in a single line if not left an extra line @ravireddy07


[WIP]: feat: support opensumi extension contributes code snippets in packageJson

Types

[x] 🎉 New Features

Background or solution

Changelog
support extension contributes code snippets in packageJson

wonderful!


add clinics_to_users model

Pull Request Test Coverage Report for Build 568016085

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.296%

Totals

Change from base Build 545998003:
0.0%

Covered Lines:
182

Relevant Lines:
189

💛  - Coveralls


lesson 2

viewer.ndisplay doesn't do anything, not needed, isn't part of napari API

the previous view_image contains ndisplay=3 anyway. you could have viewer.dims.ndisplay=3 in case someone has changed it though in mean time

yeah viewer.dims.ndisplay = 3 was the goal, I figured that out later in the game 😂 .


CNDB-10190: Fix CQLSH tests

Left two small comments, the rest LGTM. We first need to merge ccm and the pipeline changes, then probably PR gate here.
I verified in the cqlshlib run:
we start them with the correct JDK (this was something we silently didn't do right in Apache Cassandra):
INFO  [main] 2024-07-23 13:22:07,209 CassandraDaemon.java:620 - JVM vendor/version: OpenJDK 64-Bit Server VM/11.0.23
We need to ensure the right tag/branch is picked up in requirements.txt. Currently it points to a test branch in pylib/requirements.txt


Cast vote on open proposal

Acceptance Criteria

Implements voting button options according to Figma design
Cast vote button creates a transaction for casting the vote


atom/mouse_opacity needs to be implemented

http://www.byond.com/docs/ref/#/atom/var/mouse_opacity

Added in 6b2b93bb5e8b3785108517b831e56da114563be2.


variable dlon should be changed to deltalon

variable dlon should be changed to deltalon

Already fixed by https://github.com/grpc/grpc/pull/15216, but thanks anyway!


WriterAtPath not working

I need to create a text file inside epub zip before opening it. But i am not able to get writer using WriterAtPath method, it throws

mkstemp failed: invalid argument

this is what i tried
#if ENABLE_ZIP_ARCHIVE_WRITER
    unique_ptr<ArchiveWriter> w = _archive->WriterAtPath(filePath);
#endif //ENABLE_ZIP_ARCHIVE_WRITER

please guide me, what am i doing wrong here?

To my knowledge, nobody tested the "write" API of ReadiumSDK except maybe the original developer), I am not sure it ever really worked.

@danielweck so, what do you suggest i should try? Because the file contains some key information which the epub will not work without.
Thanks.

@danielweck I see that readium sdk is using an old version of libzip which contains many deprecated methods, including zip_add and zip_replace. May be the error is because of this?

I would personally use ReadiumSDK as a read-only lib to consume a ready-made EPUB (not to modify publication packages). I would write platform-specific code (e.g. Android Java or Kotlin) to alter existing EPUBs, if this was indeed needed.
By the way, why do you need to change EPUB contents?

@danielweck thanks for your suggestion. i will think on it.
The epubs that i have are not standard epubs and contains manifest/spine info in a .json file instead of a .xml file. I am trying to generate and write a respective xml file based on json data. So that it can be read by readium.

I am afraid consuming non-EPUB files is out of scope for ReadiumSDK. I am closing this issue now, feel free to re-open or continue to discuss if you have updates to share. Thanks!


[draft] Allow Inventory Manager to list, pull, and push to player Curios slots

closes #479

One thing I noticed is, that you don't check if curios is loaded since it's a optional dependency
So if curios is not loaded, the game would crash when someone runs these functions

And ignore the failed github action, these currently do not work for pull requests

I can suggest reusing peripheral plugin logic as environmental sensor do.

One thing I noticed is, that you don't check if curios is loaded since it's a optional dependency

Would implementing this as a plugin handle that? I'll give it a try

Making sure I understand correctly: I should move the Curios calls to a new class in common/addons/curios/, create a PERIPHERAL_PLUGINS like in the Environmental Sensor, and define and use InventoryManagerPeripheral.addIntegrationPlugin ?

Just encountered this: https://github.com/TheIllusiveC4/Curios/issues/238
so this might not even be possible unless there's a workaround

Just encountered this: TheIllusiveC4/Curios#238 so this might not even be possible unless there's a workaround

Oh indeed, that's a issue
A workaround would be that we check the items before adding them

Edit: wait, BasePeripheral has addPlugin - should I use that, or only call addPlugin at initialization like the Environmental Sensor? For now, I'll follow what the Environmental Sensor does as recommended by SirEdvin

Yeah, do it like edvin recommended it
That's the right way

Any progress? I don't want to put you under pressure or give you time pressure
I'm just asking because I'll release an update soon and want to know if I'll include your changes or not

I don't think I'll have it ready soon - academics kicked back up. I have an
idea on how to filter it, but making everything stable and all will take
time
On Fri, Jul 7, 2023, 16:11 Srendi @.***> wrote:

Any progress? I don't want to put you under pressure or give you time
pressure
I'm just asking because I'll release an update soon and want to know if
I'll include your changes or not
—
Reply to this email directly, view it on GitHub
https://github.com/SirEndii/AdvancedPeripherals/pull/480#issuecomment-1626015216,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AASLGEJ4JA3R7LLY5GNMMKLXPBUP7ANCNFSM6AAAAAAZ3TP4OU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Alright, no problem
And thank you for the quick response

Greetings @Kurtoid
Do you have any plans to work on this?

Whoops, I took a break from minecraft and forgot about this. It looks like Curios now has the support needed to do this without workarounds. I'm not sure if I can do it in a timely manner, though

Greetings
I will close this for now
If anyone wants to implement this, feel free to just leave a message or create a new PR


[BP-1.13][FLINK-23418][e2e] Increase the timeout to make kubernetes application ha test more stable

Backport #16602 to release-1.13.

Manually merged via 4efc2c193bc002776346cc2b2d3a71377291c79c.


Picture are saved as base64

Seems it was working previously - but now uploaded pictures are saved as base64 instead of jpg.
Any idea ? @woodpig07

fix deployed

Closing


Simulator crashes with invalid init action

If the init action refers to vars, the simulator breaks. Instead, we should properly report/explain the error.
MWE:
module test {
  var x: int

  action init = x' = x
  action step = x' = x
}

Results in:
$ quint run test.qnt
quint run <input>

Simulate a Quint specification and check invariants

Options:
  --help         Show help                                             [boolean]
  --version      Show version number                                   [boolean]
  --out          output file (suppresses all console output)            [string]
  --main         name of the main module (by default, computed from filename)
                                                                        [string]
  --out-itf      output the trace in the Informal Trace Format to file
                 (suppresses all console output)                        [string]
  --max-samples  the maximum on the number of traces to try
                                                       [number] [default: 10000]
  --max-steps    the maximum on the number of steps in every trace
                                                          [number] [default: 20]
  --init         name of the initializer action       [string] [default: "init"]
  --step         name of the step action              [string] [default: "step"]
  --invariant    invariant to check: a definition name or an expression
                                                    [string] [default: ["true"]]
  --seed         random seed to use for non-deterministic choice        [string]
  --verbosity    control how much output is produced (0 to 5)
                                                           [number] [default: 2]

AssertionError [ERR_ASSERTION]: invalid simulation failed to produce a result
    at compileAndRun (/home/gabriela/projects/quint/quint/dist/src/simulation.js:80:26)
    at runSimulator (/home/gabriela/projects/quint/quint/dist/src/cliCommands.js:407:51)
    at EitherConstructor.asyncChain (/home/gabriela/projects/quint/quint/node_modules/@sweet-monads/either/cjs/index.js:125:16)
    at /home/gabriela/projects/quint/quint/dist/src/cli.js:58:39 {
  generatedMessage: false,
  code: 'ERR_ASSERTION',
  actual: false,
  expected: true,
  operator: '=='
}

This seems to be also happening when calling setBy with a non-existent key

Also about setBy: sometimes it properly reports an error, but it seems like it finishes the entire simulation to only then report it. So you wait for nothing.

Changing the trace.result/outcome from Maybe to Either should help identify the problem(s) here.

A similar problem: the simulator crashes when running with an invariant that does not exist (ie, mistyping the name)


Is it possible to redirect downlod() to stdout?

Subj.
Thanks in advance.

You can create a custom function and pass it to on_progress_callback.
See an example from the documentation:
from pytubefix import YouTube
from pytubefix.cli import on_progress

url = input("URL >")

yt = YouTube(url, on_progress_callback = on_progress)
print(yt.title)

ys = yt.streams.get_highest_resolution()
ys.download()

Here a custom function from Pytubefix itself was used.

It seems like he wants to redirect the download, for example directly to the console or a data stream (stream) without saving to disk, what I understood was this.

@mezantrop I'll try to do it, I've already opened a branch to do it

@JuanBindez, yes, this would be a nice feature, I'd use it to stream into ASCII converter to use it later as a source for the terminal screensaver.

Looks good! Thank you!


NodeJS attempt: ENOENT, no such file or directory

I am using jest-cli 0.4.15 and jasmine-reporters 1.0.2 and am getting the following error:
Warning: writing junit report failed for 'output', 'TEST-CheckboxWithLabel.xml'. Reasons:
  Rhino attempt: java is not defined
  PhantomJs attempt: __phantom_writeFile is not defined
  NodeJS attempt: ENOENT, no such file or directory 'D:\Projects\git\carmax.com\output\TEST-CheckboxWithLabel.xml'

Using the following configuration

jasmine.VERBOSE = true;

require('jasmine-reporters');

var reporter = new jasmine.JUnitXmlReporter("output");
jasmine.getEnv().addReporter(reporter);

If I manually create the output directory it works fine. Looks like there is some bug related to directory creation.

Did you ever figure this out? I agree this looks like an issue creating directories. jasmine-reporters uses node-mkdirp to create directories. Likely the problem is either in this library or with the permissions being used by the process executing jasmine-reporters.
If you haven't already, one thing you could try is updating the version of mkdirp (latest is 0.5.1) -- if such solves your problem, I am happy to update the dependency version. If you have otherwise figured the problem out, I would love to close out this issue.

@bloveridge I will give it a try and let you know.

FWIW, I am having this same problem, and I can confirm that it still happens after manually upgrading the mkdirp dependency to version 0.5.1.  This is just running on my local machine by calling jest in my project's root directory.  The errors don't occur if I pre-create the test results output directory.

I hit this same issue today. A fix for this would be great. I'll have to commit an output file to persist the directory in git so that I can get my test scripts to run in Jenkins, just so that it can then parse the JUnit output. It would be nice to have the test output folder be transient instead of permanently in our repo.

Actually, in 2.2.0 this is working great, which includes the mkdirp update. mkdirp needed to be updated to remove references to octal literals (and it was in 0.5.1 there, and 2.2.0 here.)
After that update, you need to run Jest with sudo so that it can create the directory.
Here is my setup in case anyone else is curious:
setupTestFrameworkScriptFile:
var reporters = require('jasmine-reporters');
var junitReporter = new reporters.JUnitXmlReporter({
    savePath: __dirname +'/output/',
    consolidateAll: true,
    filePrefix: 'xmloutput',
});

var env = jasmine.getEnv();
env.addReporter(junitReporter);

Jest package.json configuration:
"jest": {
    "globals": {
      "__DEV__": true
    },
    "setupTestFrameworkScriptFile": "<rootDir>/jestSupport/setup-jasmine-env.js",
    "testPathIgnorePatterns": [
      "/node_modules/",
      "packager/react-packager/src/Activity/"
    ],
    "collectCoverage": true,
    "testFileExtensions": [
      "js"
    ],
    "unmockedModulePathPatterns": [
      "promise",
      "source-map",
      "react",
      "jasmine-reporters"
    ]
  }

devDependencies:
    "jasmine-reporters": "^2.2.0",
    "jest-cli": "^0.9.0",

After having this working for a few days, having to use sudo does seem like it's going to be problematic. So while you can use the above as a workaround, I don't think it's a good enough solution to close this issue. However, I'm also not sure whether this a mkdirp issue or a jasmine-reporters issue.

It sounds like a permissions issue with how node is executed in general. If your filesystem doesn't allow the user node is running as to create the target directory (which using sudo bypasses), node isn't going to be able to create the directory whether you try via mkdirp or any other mechanism.
You could look into changing the permissions of the parent directory to be writable by the user node runs as. You could also try any other mechanism to create the directory (such as using the native fs module, or trying fs-extra which has its own mkdirp equivalent). It sounds like a permissions issue that probably won't be satisfied by these other mechanisms, but if you find one that works I would be happy to change jasmine-reporters to use that new mechanism instead of mkdirp.

@bloveridge I think you are correct. I think that the Jenkins user was unable to modify the files as they were created manually by running the tests myself on my build machine first.
I added a step to my build script to manually delete the existing output scripts before running the tests each time, and it's all working for me now!


add x

Made a few mistakes pushing and pulling at some point but now it is clean

Thanks for your feedback, should be good now?


Set max_version for some upgrade tests

These tests are only designed to be run on 2.1 at the highest, and for
2.2 and later they fail due to Java 8 compilation errors.

I think you should be able to set "2.2" as the max_version, instead of "2.1.99". "2.2.0" > "2.2" returns true, so that should exclude any 2.2.X version appropriately.

I checked the @since code and it's using LooseVersion for comparison instead of strings, so "2.2" wouldn't work.

Can you use @since(max_version='2.1.x') the way auth_test does then? It looks a lot nicer than 2.1.99. Otherwise, +1, feel free to merge.

Sure, I didn't know that LooseVersion would handle that correctly, thanks.  Done.


Keyboard shortcuts in Azure Portal for blade operations

Can we have keyboard shortcuts for tasks such as:

Close a blade.
Traverse (move back and forth) through the blades.

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

ID: 18f82699-cc31-072f-94a2-0dbebdf8b592
Version Independent ID: b9f0c581-b671-abdb-5e44-7f0b2f3599bc
Content: Azure portal keyboard shortcuts
Content Source: articles/azure-portal/azure-portal-keyboard-shortcuts.md
Service: azure-portal
GitHub Login: @adamabmsft
Microsoft Alias: kfollis

@alenjalex Thanks for the feedback! Appreciate  if you could leave this feedback in our feedback forum to get some traction from our product group and up votes from the community.


When adding a reason, if you add quotes it breaks tag-release

Should be addressed in: https://github.com/LeanKit-Labs/tag-release/pull/121


Bad lock file

Describe the bug
The lock file is invalid.  It looks like it may have been generated on an old version of Node.
To Reproduce
Steps to reproduce the behavior: Refer to the CI test log: https://github.com/cytoscape/enrichment-map-webapp/runs/8078381146?check_suite_focus=true
If npm ci can't run on Node 16, then the server can't be deployed.
Expected behavior
npm ci should install dependencies properly.

Now it's green


add nginx template

Going to add unit tests.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #128 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #128   +/-   ##
=======================================
  Coverage   96.11%   96.11%
=======================================
  Files          18       18
  Lines         721      721
  Branches       57       57
=======================================
  Hits          693      693
  Misses         18       18
  Partials       10       10

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 0242116...bb80d69. Read the comment docs.

appears to be partial duplicate of #131

#131 is for container support, the older version, which currently our TensorFlow containers rely on.


Getting issues with jQuery.noConflict call

Since https://github.com/yourlabs/django-autocomplete-light/pull/1042 I am getting this error, and the widgets are not loading properly.
jquery.init.js:9 Uncaught TypeError: Cannot read property 'noConflict' of undefined
    at jquery.init.js:9
(anonymous) @ jquery.init.js:9
11:54:28.601 autocomplete.init.js:8 Uncaught TypeError: Cannot read property 'fn' of undefined
    at autocomplete.init.js:8
    at autocomplete.init.js:123
(anonymous) @ autocomplete.init.js:8
(anonymous) @ autocomplete.init.js:123
11:54:28.610 select2.js:5 Uncaught TypeError: $ is not a function
    at select2.js:5
    at select2.js:116

All the files referenced (jquery.init.js, autocomplete.init.js, select2.js) are the one provided by django-autocomplete-light, i.e.:

autocomplete_light/jquery.init.js
autocomplete_light/autocomplete.init.js
autocomplete_light/select2.js

I believe this is caused by the fact that the jQuery variable used should be django.jQuery
https://docs.djangoproject.com/en/2.1/ref/contrib/admin/#contrib-admin-jquery
Note that admin/js/jquery.init.js is actually being called before autocomplete_light/jquery.init.js in my case, which could be the crux of the issue here (according to the comments)

Ah, it looks like I may have missed an interaction between DAL's jquery.init.js and some other usage of django's jquery.init.js. Could you please describe in what way you're using DAL that's causing this error message to appear? For instance, are using DAL on a Django Admin form, or a custom form, or perhaps a CMS form (e.g. Wagtail)? And, if possible, could you check out the source of the page that's throwing this error, and find out which version(s) of jQuery are being loaded?

In standard admin forms, but I'm using other libraries as well.
The version of jquery is the default provided by django 2.0.8 (I believe that was jquery 2.2.3)
I will attempt at sending a patch once I get to it (in the coming days hopefully)
What I don't understand with https://github.com/coredumperror/django-autocomplete-light/blob/d9e552adc83bac8dbef8b4fd3e461b10713517a8/src/dal/static/autocomplete_light/jquery.init.js content is the purpose of doing all this
Wouldn't yl.jQuery = django.jQuery; be enough?
By re-introducing the dependency on admin/js/jquery.init.js to be loaded before autocomplete_light/jquery.init.js in my opinion that would be enough. But I'm probably missing something here?

Shoot, there was something nibbling at the back of my mind about that fix I wrote, and now you've exposed exactly what I did wrong. The change I made worked for me in the Wagtail admin, but will not work in the django admin, because it uses a different set of javascript includes. I had thought that things would work out, but clearly I was wrong.
I'm going to need to spend some time debugging this to find a universal solution. But I don't have think I'll have time today. For now, I'd suggest downgrading to DAL v3.3.0 until I figure out what the best path forward is.

OK, I'm as confident as I can be that the newest commit on master fixes this issue. It's a real doozey of complicated change, though. No wonder the original code had a comment about wishing there was a better way to do jquery.init.js.

Let's release a version that works with django admin again rather, you can stick to that one for wagtail.
I will make a release that passes our tox tests asap.

master is working fine for me 👍 feel free to close this issue

Also, tests in the admin are actually working fine for me, except one for the GFK but let someone fix it. It's a pain that travis has degraded free service so much it systematically fails running our tests for a while.


Synched Charts Updating Data

Hi
https://codepen.io/vhphan/pen/MMddve
I am trying to update the synchronous charts with new data.
However anytime I update one of the charts, all of the charts are updated with the same data.
In the codepen, I just modified the code from the documentation for synchronous charts.
Is there anything I am missing?

I think I'm seeing the same issue. Eventually multiple charts render the same data for me, too.

It looks like there are additional args that can be specified. By passing in updateSyncedCharts: false you can achieve the desired behavior:
https://github.com/apexcharts/apexcharts.js/blob/4d0cad839caa579a88d5b03ca1aef678f295930a/src/apexcharts.js#L409


252 remove engines from package.json

This Pull Request should close the issue #252
Description
Consumers of the npm package just get the dist files so they aren't affected by this AFAICT. What Milligram uses to build and develop itself does shouldn't affect consumers. So having a Node.js version could be considered wrong.
This will allow Milligram to use whatever Node.js version we want to use without having to support unsupported Node.js versions anymore.
Maybe in the future make sense Milligram to use only the up to date versions of the Node.js (>=lts) but for now, the engines property should be removed from package.json
Hightlight

remove the engines property from package.json (8c80c8f)

@westtrade @davidroeca please, can you review this pull request

👍 looks good to me!

Is it possible to release a new version with the fix of node engine?

@GreatWizard I will prepare the new version ASAP #254

@westtrade @davidroeca @GreatWizard
Milligram v1.4.1 https://twitter.com/cjpatoilo/status/1273691342720372738

Thank you!


Add support for analyzing androidTest source sets.

Relates to https://github.com/autonomousapps/dependency-analysis-android-gradle-plugin/issues/411.

Build failed https://gradle.com/s/fwbzokbnxfat2

Build failed https://gradle.com/s/cjqpll2hqcclc

Build failed https://gradle.com/s/u4kb26ndirifs

Build failed https://gradle.com/s/a4vm533zqgspq

Build failed https://gradle.com/s/xo4sdqgw3qfh4


Complete QA

cancel proposal
fund/defund
withdraw

The use case scenarios have been done, the work almost correctly let aside for a few bugs that I listed in the following issues
#49 #50 #51


Backend for listing instantiations of primitives

This PR contains a new backend that lists all of the unique instantiations of primitives within a program into a JSON file. This is necessary for my Calyx-FIRRTL efforts because FIRRTL does not support parameterization of modules, and therefore I have to generate individual FIRRTL modules that each correspond to a specific primitive instantiation in Calyx. This backend is the first step, and the produced JSON file will be processed by a script that I will write next. I'll also add fud2 support for this backend soon.
As a side note, this backend would also be similarly useful for the BTOR2-Cider efforts as described in #1845 under Generation Scaffolding.
This backend borrowed a lot of code from the Resource Estimation backend, which I'm very thankful for :slightly_smiling_face:
Please let me know if there's any points I can improve here!! Also, I'm personally not a fan of the namings I used (primitive-inst, primtives.rs, etc) but didn't have any better ideas, so I'd really appreciate any suggestions on that front :slightly_smiling_face:
Usage
You can run the backend by using the -b primitive-inst flag, and the optional -o <OUTPUT_FILE> flag to write the JSON to a file.
ex) Running with language-tutorial-mem:
cargo run -- examples/tutorial/language-tutorial-mem.futil -b primitive-inst -o p.json

we get in p.json...
[
  {
    "name": "std_mem_d1",
    "params": [
      {
        "param_name": "WIDTH",
        "param_value": 32
      },
      {
        "param_name": "SIZE",
        "param_value": 1
      },
      {
        "param_name": "IDX_SIZE",
        "param_value": 1
      }
    ]
  }
]

Wonderful; looks great! Merge at will, I say. :smiley:


Pop All But Shiny Wrinklers button

would help in my dream of amassing 12 shinies

hmm, I can see why one would want to do that, lol.

Perhaps this should be implemented where if there is a shiny wrinkler present then the "pop all" button says "pop all non-shiny wrinklers" and next to it there is a checkbox that can be checked to change the button to be "pop ALL wrinklers." So the default would be to preserve your shiny wrinklers, and there's a very conscious step needed to pop the shiny ones. This might help people who don't know what shiny wrinklers realize they've got something special on their hands, too.

With how rare a shiny is, I think it's safe to say you can always manually pop the shiny and not have the pop all ever pop a shiny.

Fair enough

Added to Version 2.0106.1

Thanks!
On Tue, Apr 24, 2018, 5:40 PM Aktanusa<EMAIL_ADDRESS>wrote:

Closed #140 https://github.com/Aktanusa/CookieMonster/issues/140.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Aktanusa/CookieMonster/issues/140#event-1592924534,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD4gMjLo6eRl5vClUtmjqpHRuEY0yta2ks5tr8YNgaJpZM4O6_WG
.


Scenes call list

Is there any list of called scenes? I would like to make a back button that returns to the scene called before

No. The backend only stores the current scene name, position (scene step) and context. For nested scenes, it also stores the call stack (name, step and context for every upstream/containing scene).


feat: Added Smart substitution support for S3

Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

JUnit test case
Manual testing

/ok-to-test=abf30e9

/ok-to-test sha=cd0d5b0

/ok-to-test sha=12144a8

/ok-to-test sha=b3592e6

/ok-to-test sha=b3592e6


[Bug Report] el-switch change event not working expected

Element UI version
2.13.1
OS/Browsers version
macOS Chome 80.0.3987.163 (Official Build) (64-bit)
Vue version
3.0.0-beta.2
Reproduction Link
https://jsfiddle.net/afnsbxhp/
Steps to reproduce
Open console, run jsfiddle and check the console. You will see the changed log and mounted log.
What is Expected?
When I click el-switch the change event should be run.
What is actually happening?
The change event working when data changed so when my page loaded the change event working automatically but I didn't click el-switch. How can I run an event when el-switch updated by click?

I defined outer div for el-switch and set click event for outer div. I solved the problem this way.


Swift 4.0 new

Include updates from master for swift 4.0.

I messed up. @tonyarnold what caused the need for index/section labels in the previous 4.0 PR?


SMOE or XMOE Network how to "evaluate" and "save and resume"

Hello there,
i'm interested in using the XMOE network and I have some questions regarding how to evaluate its performance on a validation set, how to save checkpoints, and how to resume training from a saved checkpoint.
Evaluation on Validation Set:
Could you please provide some guidance on how to evaluate the XMOE network on a validation set?  Also, I'm using the Distributed Data Parallel (DDP) mode, and I'm wondering whether I need to evaluate the XMOE network on all devices or only one device?
Saving Checkpoints:
How can I save the XMOE model's checkpoints during training? What's the recommended way of doing this? Given that each GPU has its own experts and shared parameters, should I save all the parameters on each device or is there an API that can centralize the parameters and save them to avoid redundancy?
Resuming Training from a Saved Checkpoint:
How can I resume training the XMOE model from a saved checkpoint? What's the recommended way of doing this? Is there any specific API or command I should use?
Thank you in advance for your help. I'm looking forward to using the XMOE network in my projects.

I'm also interested in reproducing the fine-tuned results in the X-MoE paper. If you can release some related scripts and pre-trained models, that would be perfect.

Hello there,
i'm interested in using the XMOE network and I have some questions regarding how to evaluate its performance on a validation set, how to save checkpoints, and how to resume training from a saved checkpoint.
Evaluation on Validation Set: Could you please provide some guidance on how to evaluate the XMOE network on a validation set? Also, I'm using the Distributed Data Parallel (DDP) mode, and I'm wondering whether I need to evaluate the XMOE network on all devices or only one device?
Saving Checkpoints: How can I save the XMOE model's checkpoints during training? What's the recommended way of doing this? Given that each GPU has its own experts and shared parameters, should I save all the parameters on each device or is there an API that can centralize the parameters and save them to avoid redundancy?
Resuming Training from a Saved Checkpoint: How can I resume training the XMOE model from a saved checkpoint? What's the recommended way of doing this? Is there any specific API or command I should use?
Thank you in advance for your help. I'm looking forward to using the XMOE network in my projects.

Evaluation:
You can check the code for more details regarding the evaluation/generation of MoE models. It should be feasible to evaluate on a single device as long as the GPU memory is enough.
Checkpoint:
Here is an example of saving and loading MoE checkpoints. The dense part and the expert parts are stored separately to avoid redundancy.


Help!does jdbc-importer support ES template?

My ES version is 1.4.4,jdbc-importer version is 1.7.3
My problem is:
I do not want to define "index_settings" or "type_mapping" in JDBC importer definition file,I want to define them in ES template.does jdbc-importer 1.7.3 support it?

This feature is outside of JDBC importer. Create index template in ES, and use it from JDBC importer by
select myindexname as _index, ... from table ...
where myindexname is the index name that matches the pattern configured in the index template.

Thanks for your help.
1、where should I define the command "select myindexname as _index, ... from table ..."?in the JDBC importer definition file? What I want to do is import data of table "aaa" from Oracle to ES, Do you mean I should change "select * from aaa " in JDBC importer definition file into " select abc-index as _index from （select * from aaa)"?. I'm confused.
2、Could you give me an example?
Thanks for your patience again!

Of course you must define the SQL statement of the JDBC importer.
select * from aaa is a naive example SQL which works, but it does not address Elasticsearch index, type, or id. It is just a simple test. The asterisk * is not suitable for Elasticsearch field names either.
To create Elasticsearch documents, you need something like select myindexname as _index, mytypename as _type, mydocumentid as _id, myvalue1 as fieldname, myvalue2 ... from aaa where ...
The pseudo column names beginning with _ are documented at https://github.com/jprante/elasticsearch-jdbc
You can freely choose myindexname, mytypename, etc. as you like,  as literals or values from DB.


Adding routers to high level concepts

Description
This PR includes the changes requested in [Documentation]: adding Routers to High-Level Concepts #7884
The routers concept was added to High-Level Concepts.
Also fixing a typo in the file (and and → and).
Fixes #7884
Type of Change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] I stared at the code and made sure it makes sense

Suggested Checklist:

[ ] I have performed a self-review of my own code
[ ] My changes generate no new warnings

Hello, I am open to any feedback or suggestions.

Thanks!

Happy to help.


SetSlug should be called after Post_Insert Actions

Initially reported by: @chrismeller as #TRAC1455
Post::setguid() is called first in Post::insert() (and setguid() calls setslug()). This should not be so, as it prevents the default slug-uniqueness logic from working if you want to change the slug.

I just tested and apparently this has been fixed.


Change in lookup method for credentials

I had a bug (ok, two bugs) in the previous PR I made for compatibility tweaks, and my intent regarding the role of organization has changed as well.

Coverage decreased (-100.0%) to 0.0% when pulling 63e9e152458d1004f145e24575cdd410ae08bf80 on AlanCoding:organization_changes into d7b255e8bfcef59a7a2cd25ab968439ae5577539 on ansible:master.


_config.yml variables from asciidoctor.attributes can't also be used as 'site.asciidoctor.attributes.foo' in jekyll templates

The jekyll docs note that...

All the variables set via the command line and your _config.yml are available through the site variable. For example, if you have url: http://mysite.com in your configuration file, then in your Posts and Pages it will be stored in site.url

...and nested structures such as this onebelow work fine and allow for variables like {{ site.my_stuff.attributes.foo }} in liquid templates to be resolved to bar...
my_stuff:
  attributes:
    foo: "bar"

However, something about the way the jekyll-asciidoctor plugin is initialized seems to be "eating" it's config and preventing the asciidoctor attribute declarations to be re-used as jekyll variables in liquid templates.  So for example with  this _config.yml snippet...
my_stuff:
  attributes:
    foo: "bar"

asciidoctor:
  attributes:
    foo: "yak"

...the attribute {foo} in an index.adoc file will be expanded to yak but if the liquid template contains the following:
foo1:  {{ site.my_stuff.attributes.foo }}
foo2:  {{ site.asciidoctor.attributes.foo }}

the output will be...
foo1:  bar
foo2:

That's correct. That's not how this part of the configuration is consumed. It's handled specially. If you know of a way to make these settings available to liquid, I'm open to considering it. But it doesn't work as you describe because it was not a requirement for the design of this plugin.
I think the SiteDrop would need to be enhanced somehow to allow access to it, but liquid in general is very strange in what objects it permits to be navigated (very different from Slim and Haml).

That's correct. That's not how this part of the configuration is consumed. It's handled specially.  If you know of a way to make these settings available to liquid, I'm open to considering it. But it doesn't work as you describe because it was not a requirement for the design of this plugin.

I am in no way a jekyll/liquid expert, so if you're saying this is expected behavior then so be it -- feel free to close.
I opened the issue only because:

I wanted to re-use my asciidoctor attributes as jekyll variables in my liquid templates
the jekyll docs seemed pretty clear that any variable in _config.yml should be usable as a site.* variable
when i tried doing that it didn't work
the behavior of asciidoctor's attributes in this regard seems inconsistent with everything else in _config.yml (ie: seemed like an implementation problem, not a documentation problem)

For example, my _config.yml includes the following...
gems: [jekyll-asciidoc]

...and if i drop {{ site.gems[0] }} into my liquid templates it resolves as expected to jekyll-asciidoc even though jekyll is clearly/probably using that gems config option for important things long before my templates are evaluated -- but the variable is still available for use.
Random experiments with other variables (even deeply nested variables) in _config.yml produced similar results.  It was only when I tried using site.asciidoctor.attributes.foo that I ran into problems -- Suggesting that the jekyll-asciidoc(tor) plugin was doing "something" with those configuration variables during initialization that was inconsistent with how jekyll expected them to be used.

the jekyll docs seemed pretty clear that any variable in _config.yml should be usable as a site.* variable

That's extremely misleading of those docs because it does not take into account plugin configuration. It should say that any configuration that isn't part of a plugin configuration is likely (but not always) to be available to Liquid. So it's a generalization.

Liquid templates can access simple structures, but they cannot access complex structures like a Hash with symbol keys. It requires special handling. There are various reasons why we have to use a Hash, and, for that matter, modify the raw site configuration. It has to do with how Jekyll is implemented and the lifecycle it exposes to plugins. We are limited by its design. I'd argue this is just the way this plugin works.
Having said all that, there is a way to access the attributes you define in _config.yml another way. You can use a YAML reference to define the attributes at the top level, then pass that to the configuration in the asciidoctor block.
asciidoc_attributes: &attrs
  foo: bar

asciidoctor:
  attributes: *attrs

Then you can reference:
{{ site.asciidoc_attrs.foo }}

Keep in mind, these will be the unprocessed entries.

That's extremely misleading of those docs because it does not take into account plugin configuration.

understood.

You can use a YAML reference to define the attributes at the top level, then pass that to the configuration in the asciidoctor block. ...

Ahhh... excellent! thank you for the suggestion/pointer to YAML reference - those do solve the problem nicely.

:+1:

I documented this in the README. See https://github.com/asciidoctor/jekyll-asciidoc#using-asciidoc-attributes-in-a-liquid-template.


[SPARTA-530] field name too much descriptive

GIVEN a running Sparta instance
WHEN you create a kafka output
THEN you can set the property "requiered ACKs" (before known as "Specify whether kafka producer waits for an acknowledgement from the broker (1), or not (0)") using checkbox (true/false) values instead of 0/1 values.
The documentation is already updated. Please check https://stratio.atlassian.net/wiki/display/SPARKTA/7.4+Outputs#id-7.4Outputs-8.4.9Kafka

+1

:shipit:

Comment accepted. Will treat it ASAP

Squashing and checks in progress.


[SPARK-30883][TEST] Cancel the tests when setReadable and setExecutable don't work

What changes were proposed in this pull request?
The java api setWritable,setReadable and setExecutable dosen't work well because root can read / write or execute every files. Maybe, we could cancel these tests or fast failure when the mvn test is starting.
Why are the changes needed?
When we run the test in root role, the test will fail.
SPARK-3697: ignore files that cannot be read. *** FAILED ***
  2 was not equal to 1 (FsHistoryProviderSuite.scala:187)

temporary shuffle/local file should be able to handle disk failures *** FAILED ***
  tempShuffleFile3.exists() was true All disks are broken, so there should be no temp shuffle file created (DiskBlockManagerSuite.scala:124)

Does this PR introduce any user-facing change?
No
How was this patch tested?
UT

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@srowen  Sure, running tests as root is not a good idea. I run my tests in gitlab ci/cd. Root is its default user. Maybe, it's not bad to cancel these tests. Or just disable root to run all tests?

Yeah, don't run as root. This is a good example of why - it defeats the purpose of a few of the tests.

Agree with Sean, you should avoid running test cases as the root user.

I agree. Is it necessary to add some code to prevent running as root? @jiangxb1987

Do you have an easy way to do it? I don't think it's worth to add more complexity to ensure that.


fix up Wconversion issues in float_math/double_math

We are fixing all Wconversion as part of issue (TODO), however fixing the ones in float_math/double_math is much more risky, and these functions pass exhaustive tests as is, so I have disabled Wconversion files for these for now

Wconversion as part of issue (TODO)

I believe that was #182


Adding a MANIFEST.in to include the license and README files.

Coverage remained the same at 90.576% when pulling c77fd76a9658e996996753e53a50fd4a95a4a439 on add-manifest into e95866d2dc680418a41522649dfcf237492de162 on master.

Coverage remained the same at 90.576% when pulling c77fd76a9658e996996753e53a50fd4a95a4a439 on add-manifest into e95866d2dc680418a41522649dfcf237492de162 on master.


NFC doesn't turn on (xz3)

from settings nfc can be enabled but it doesn't seem to work, not that can do much in sfos in general.
nfc logs app shows:
2021-07-20 07:21:39.812 Parsing /proc/4421/status
2021-07-20 07:21:39.813 NFC enabled
2021-07-20 07:21:39.813 dbus-neard: Enabled: true
2021-07-20 07:21:39.823 Wrote /var/lib/nfcd/settings
2021-07-20 07:21:54.853 Name ':1.162' timed out
2021-07-20 07:22:09.854 Name ':1.51' timed out
and dmesg
[ 2731.080071] dsi-ctrl:[dsi_ctrl_handle_error_st
atus] tx timeout error: 0x40

Haven't checked it out at AOSP10, unfortunately.
In README, NFC support was carried over from earlier versions and it did use to work at some point. Maybe already at later SFOS versions with AOSP9 we somehow lost it - I remember there were reports about it. Never checked it further as not much can be done with NFC on SFOS anyway.
Should drop NFC support from README and restore that if it will be available and shown to be working.
As for figuring out what's wrong - would be good to do so. Just priority is low(ish) due to lack of clear functional advantage.

Pity NFC is almost unusable in SFOS, remember when J1 was launched TOH was based on NFC.
In earlier versions of this AOSP9 port NFC used to work, it's a low priority issue anyway, good to be listed tho.

On AOSP9 -at some point at least- when i was putting my debit card near the phone it used to vibrate. After some point it stopped but have no idea if it was something that changed in SFOS or AOSP. Same behavior with AOSP10. Doesn't seem to work.
AFAI understand there is only one app that uses NFC on SFOS?

NFC is still not working on my XZ2c. unsurprising if not in AOSP10, of course, but figured id check anyway

Pity NFC is almost unusable in SFOS
what do you mean? i use NFC daily on Xperia X Compact.
(maybe you mean that only a handful of native apps use it? i actually have no idea what other people use NFC on their phones for, aside from 2FA)

it's a low priority issue anyway
:'( personally, its my last issue stopping me switching to XZ2c as my daily driver

I don't know if NFC works on AOSP10 itself. I hope it does, but someone needs to establish it. @teleshoes - want to test it?
If it does work on AOSP10 then it is a matter of checking out whether we are missing something in SFOS side. Would be good to start and check if any AOSP10-based SFOS port has NFC working. But that would make sense after we know about AOSP10 itself

@rinigus i tried to flash this:
aosp-10.0-20200818_aosp_h8314.zip
https://androidfilehost.com/?w=files&flid=299805
and i got:
> fastboot flash system_a system.img
Invalid sparse file format at header magic
Sending sparse 'system_a' 1/2 (768005 KB)          OKAY [ 23.230s]
Writing 'system_a'                                 OKAY [  0.001s]
Sending sparse 'system_a' 2/2 (470299 KB)          FAILED (remote: 'Error: Last flash failed : Volume Full')
fastboot: error: Command failed

is there another image i can use? or am i supposed to resize partitions or something after flashing SFOS?
(ive never used or flashed android before, only maemo/meego/sfos)

Sorry, I am not up to date with those images. In principle, you can build one using AOSP guides. Otherwise it could be difficult to find AOSP10 image these days.
You could also ask at corresponding thread. Maybe image is buggy? Or download failed?


Messy Installation with PIP

When installing things with PIP, e.g.
pip install deepl --user

things get unnecessarily messy in the package target directory afterwards:
~/Library/Python/3.7/lib/python/site-packages $ find . -depth 1 -newermt '10 min ago'
…
./CHANGELOG.md
./deepl
./deepl-1.13.0.dist-info
./LICENSE
./SECURITY.md
…

The files CHANGELOG.md, LICENSE, and SECURITY.md should be placed inside one of the deepl directories, not into the main package directory. If I was to look for information for deepl, I would check inside the directories, not inside random files where it's not clear instantly where they do belong to. Also if all packages would do it like that only the most recently installed package would have a SECURITY.md, f.ex.

Thanks for reporting this! This will be fixed in an upcoming version.

This has been fixed, installing the current version via pip no longer pollutes the site-packages directory


Filtering Underlying Data

It looks like in the Querying Data section Summary Data will be available to be retrieved on a per column basis (very happy about that!)
Will this also be available for Underlying data?  It would likely help resolve this use case

Yes actually and thank you for bringing it to our attention that that's not in the migration guide.
In general, any new functionality we've added to the Extensions API will make it to the Embedding API where appropriate. As you can see in the Extensions API reference, we have a columnsToIncludeById option that you can pass in to getUnderlyingDataAsync

In general, any new functionality we've added to the Extensions API will make it to the Embedding API where appropriate

Music to my ears 👏


chart: improve testing for data correctness

Signed-off-by: Paweł Krupa (paulfantom<EMAIL_ADDRESS>

What this PR does / why we need it
Building on top of #564 to add more tests based on promql queries.
This change is supposed to:

make metrics test script more generic
add support for testing for various .*_build_info metrics to ensure components are up and running
remove unnecessary Role and RoleBinding objects

Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #571

Special notes for your reviewer
Checklist

[x] Chart Version bumped
[x] CLA signed

While debugging CI errors, I noticed it was quite hard to reproduce and tests our tests. Due to this problem, I rewrote all tests so they are NOT modified directly by helm and their parametrization is done by environment variables. This in turn allows to extract scripts from kubernetes objects and simplify their local testing.


starship: 1.1.1 -> 1.2.1

Motivation for this change
https://github.com/starship/starship/releases/tag/v1.2.0
https://github.com/starship/starship/releases/tag/v1.2.1
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

1 package built:

starship


Bug: Generic System link validation fails with unknown values

genericSystemLinkSetValidator.ValidateSet() doesn't check whether the configuration is known (it may depend on some computed values) leading to error:
│ Error: Value Conversion Error
│
│   with apstra_datacenter_generic_system.all,
│ An unexpected error was encountered trying to build a value. This is always an error in the provider. Please report the following to the provider
│ developer:
│
│ Received unknown value, however the target type cannot handle unknown values. Use the corresponding `types` package type or a custom type that handles
│ unknown values.
│
│ Path:
│ Target Type: []blueprint.DatacenterGenericSystemLink
│ Suggested Type: basetypes.SetValue

fixed by 172


Use match_attributes to filter on device level attributes also

This PR extends the match_attributes section to allow filtering on any attribute, not just interface level ones.

Thanks! This is tricky code with getting SetAttrDrop() right. But the tests are pretty extensive so lets ship it.


GitLens conflicts with the merge conflict code lens

Originally reported here: https://github.com/eamodio/vscode-gitlens/issues/319
Basically the issue as reported is that when GitLens adds its git code lens, and there are merge conflicts, that the merge conflicts code lens doesn't show up. This issue has been reported numerous times on the GitLens repo, and it seems to be only affecting vscode on linux.
I have only seen this once (ever), but I'm also not using linux. But in theory, GitLens shouldn't be able to conflict with any other code lens provider.
Any ideas?

@alexandrudima Any thoughts on editor view zones overriding others?

The editor allows for multiple view zones after the same line number and they will "accumulate".
@jrieken What do we do in case of multiple code lens providers for a file?

@jrieken What do we do in case of multiple code lens providers for a file?

They are merged and concatenated, e.g Lens 1 | Lens 2.

Basically the issue as reported is that when GitLens adds its git code lens, and there are merge conflicts, that the merge conflicts code lens doesn't show up

In a sense of they get removed when gitlens kicks in?

I can reproduce with this snippet
    vscode.languages.registerCodeLensProvider('*', {
        provideCodeLenses(document) {
            if (document.lineAt(0).text === 'Lens') {
                return [new vscode.CodeLens(
                    document.lineAt(0).range,
                    { command: 'extension.sayHello', title: 'This is a long lens from another provider ' }
                )];
            }
            return undefined;
        }
    })

    vscode.languages.registerCodeLensProvider('*', {
        provideCodeLenses(document) {
            if (document.lineAt(0).text === 'Lens') {
                return [new vscode.CodeLens(
                    document.lineAt(0).range,
                    // { command: 'extension.sayHello', title: 'This is a long lens from another provider ' }
                )];
            }
            return undefined;
        },
        resolveCodeLens() {
            throw new Error('lens fail');
        }
    })


⚠ Tukarcoid has degraded performance

In dff127f, Tukarcoid (https://tukar.co.id) experienced degraded performance:

HTTP code: 200
Response time: 2627 ms

Resolved: Tukarcoid performance has improved in c5e92fc after 8 minutes.


form update request unique

failed to export postman when form request has params. in my case form update request unique, email, {USER_ID}

Hi @teoprayoga, could you provide steps to reproduce this issue, please?

Closing until reproducible steps are included.


Future: define a default ExecutorService for subsequent operations

See this comment.
Future.of(fooExecutor, this::computation) // executes the async computation using fooExecutor
    .on(barExecutor) // this line is optional, it will execute all supsequent operations using barExecutor
    .map(...) // this will execute the map operation on the same task the future currently runs on
    .map(bazExecutor, ...) // this will execute (only) this map operation on a task created by bazExecutor
    ...

Baking this feature into Future does not scale well. We are already able to do it using the new Future.join() operation.
Example revisited:
// instead of...
Future.of(fooExecutor, this::computation)
    .on(barExecutor)
    .map(...)
    .map(bazExecutor, ...)
    ...

// ...we write
Future.join(bazExecutor, tryCompleteBaz ->
    Future.join(barExecutor, tryCompleteBar ->
        Future.of(fooExecutor, some::computation).onComplete(tryCompleteBar::test)
    ).map(...).onComplete(tryCompleteBaz::test)
).map(...)
...

// ...or
Future.of(fooExecutor, some::computation)
    .flatMap(bar -> Future.successful(barExecutor, bar).map(...))
    .flatMap(baz -> Future.successful(bazExecutor, baz).map(...))

Note: The Future.successful() calls immediately complete the future without creating a new thread.
It is nice to see that the existing API composes well. It is much nicer than introducing an additional state variable to Future (which eats extra mem and extra implementation logic) and introducing copies of all transformational methods that take an ExecutorService. That would blow up the API roughly by factor 2.


feature: increases coverage for AzureBlob

I achieved good coverage with most generic methods. Surely there are more situations we can test, but this is a initial harness that will enable us to more confidently make changes to this class.
Also made some necessary fixes to AzureBlob itself, and small refactors. I tried to keep then to a minimal to avoid breaking anything before tests were available.
Oh, I also added a new construtor to enable injecting the configuration. We'll need to deal with that as LightConfigurator is too rigid (check discussion at #68).

The build will fail for now, since I'm relying on Azure Storage Emulator do make tests. I'll try to figure out how to make it run within GitHub actions or I'll just move to Azure DevOps.

Oh, I've already got an answer: https://stackoverflow.com/questions/59413096/can-i-run-a-docker-container-with-github-actions/59415658#59415658
Now I need to use that to run Storage Emulator on a docker, should be easy. Also will probably need to configure it externally because we wouldn't want to do that locally (I use the emulator from the SDK, ppl should be able to configure what they want to use).

I ended up needing to use Azurite because MSFT emulators provide no Linux support :(


Release/1.6.0

Release Checklist:

Issue resolutions:

[ ] Closes #
[ ] Closes #

Unit tests:

[ ] Pass
[ ] Updates current tests

Documentation:

[ ] Adds tutorial notebook

[ ] Updates CHANGELOG.md

📚 Documentation preview 📚: https://stitchee--250.org.readthedocs.build/en/250/

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

Project coverage is 83.64%. Comparing base (140f356) to head (d4aea53).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #250      +/-   ##
==========================================
- Coverage   86.11%   83.64%   -2.47%
==========================================
  Files          11       11
  Lines         648      648
==========================================
- Hits          558      542      -16
- Misses         90      106      +16

Flag
Coverage Δ

integration
?

unittests
83.64% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Add Atlas Server to product menu

Closes #47.

Do you like it in "More"? Or did you have another place in mind?
@HeyStenson for review, please. We shouldn't merge too eagerly, though, since we don't want this out before the Atlas Server docs page.
cc @mapbox/atlas

@davidtheclark I think "More" is a great place for it - @alulsh, do you agree?
Also, Alex let me know this morning that the product name is now going to be Atlas GL.

@HeyStenson @davidtheclark let's go with Atlas instead of Atlas Server.
Agree with leaving it in "More" (for now at least 😄).

@davidtheclark @colleenmcginnis Changed the name! Timing-wise, when does it make sense to merge this?

I caught this branch up to master so I could add an entry to the new changelog.
You can merge it whenever @HeyStenson! We'll need to do a release and bump the version of dr-ui we're using again in the Atlas docs for the changes to appear. Luckily Dan walked me through the release process earlier this week so I should be able to do it myself this time 🎉  (unless there are special permissions needed for such things cc @davidtheclark).


Add wheel scrolling intervention article.

Target Live Date: 2019-02-07

[x] This has been reviewed and approved by (sahel@)
[x] I have run npm test locally and all tests pass.
[x] I have added the appropriate type-something label.
[ ] I've staged the site and manually verified that my content displays correctly.

CC: @petele

Whoops!
There were 2 critical errors that broke the build and prevented it from being automatically deployed.
There were 1 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
ERRORS
src/content/en/updates/2019/02/scrolling-intervention.md#L19 - Do not hard code developers.google.com in links.
src/content/en/updates/2019/02/scrolling-intervention.md#L30 - Do not hard code developers.google.com in links.
WARNINGS
src/content/en/updates/2019/02/scrolling-intervention.md#L75 - Consider adding a "was this page helpful?" widget to your page: https://developers.google.com/web/resources/widgets#helpful

Whoops!
There were 2 critical errors that broke the build and prevented it from being automatically deployed.
There were 1 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
ERRORS
src/content/en/updates/2019/02/scrolling-intervention.md#L19 - Do not hard code developers.google.com in links.
src/content/en/updates/2019/02/scrolling-intervention.md#L30 - Do not hard code developers.google.com in links.
WARNINGS
src/content/en/updates/2019/02/scrolling-intervention.md#L75 - Consider adding a "was this page helpful?" widget to your page: https://developers.google.com/web/resources/widgets#helpful

:+1:

:+1:


Batch mode tries to query "where in (Null)"

models :
type User struct {
kallax.Model table:"users"
ID           int64 pk:"autoincr"
Username     string
Email        string
Password     string
CreatedAt    time.Time
Cards        []*Card       fk:"user_id"
CreditCards  []*CreditCard fk:"user_id"
}
type Card struct {
kallax.Model table:"cards"
ID           int64 pk:"autoincr"
User         *User fk:"user_id,inverse"
Number       string
}
type CreditCard struct {
kallax.Model table:"credit_cards"
ID           int64 pk:"autoincr"
User         *User fk:"user_id,inverse"
Number       string
}
Query :
q: = models.NewUserQuery().WithCards(nil).WithCreditCards(nil)
users, err :=  store.Debug().FindAll(q)
Debug output:
2017/07/09 14:39:52 kallax: Query: SELECT __user.id, __user.username, __user.email, __user.password, __user.created_at FROM users __user LIMIT 50 OFFSET 0, args: []
2017/07/09 14:39:52 kallax: Query: SELECT __card_Cards.id, __card_Cards.user_id, __card_Cards.number FROM cards __card_Cards WHERE __card_Cards.user_id IN ($1,$2,$3,$4,$5,$6,$7,$8,$9,$10,$11,$12,$13,$14,$15), args: [3 40 35 41 36 42 37 43 1 44 38 45 2 46 39]
2017/07/09 14:39:52 kallax: Query: SELECT __creditcard_CreditCards.id, __creditcard_CreditCards.user_id, __creditcard_CreditCards.number FROM credit_cards __creditcard_CreditCards WHERE __creditcard_CreditCards.user_id IN ($1,$2,$3,$4,$5,$6,$7,$8,$9,$10,$11,$12,$13,$14,$15), args: [3 40 35 41 36 42 37 43 1 44 38 45 2 46 39]
2017/07/09 14:39:52 kallax: Query: SELECT __user.id, __user.username, __user.email, __user.password, __user.created_at FROM users __user LIMIT 50 OFFSET 15, args: []
2017/07/09 14:39:52 kallax: Query: SELECT __card_Cards.id, __card_Cards.user_id, __card_Cards.number FROM cards __card_Cards WHERE __card_Cards.user_id IN (NULL), args: []
2017/07/09 14:39:52 kallax: Query: SELECT __creditcard_CreditCards.id, __creditcard_CreditCards.user_id, __creditcard_CreditCards.number FROM credit_cards __creditcard_CreditCards WHERE __creditcard_CreditCards.user_id IN (NULL), args: []
With first query kallax returns just 15 records from table users (with limit 50), so why kallax queries users table again with 15 offset and then queries creditcards, cards with "WHERE user_id IN (NULL) "?

Hi, thanks for the report! We'll try to fix this as soon as possible.
The second query with the offset 15 is expected behaviour, if it encounters 0, then the batching stops as it does not know the total amount of results. The NULL thing is definitely a bug, it should not have made such query after receiving 0 rows.

Does kallax really need to continue quering users table again if amount of rows from last query is less than limit?

Actually, it wouldn't need to. Will start a separate issue to track that enhancement

@Venommmm Can you confirm that this is fixed in v1.2.14?


ADD: Fisher-Yates Shuffle in C , C++ ,Python

Is your feature request related to a problem? Please describe.
The Fisher-Yates shuffle, also known as the Knuth shuffle, is an algorithm used to randomly shuffle or permute the elements of an array or a list. It provides an unbiased and efficient way to generate a random permutation of the elements.
Describe the solution you'd like
The Fisher-Yates shuffle ensures that every possible permutation of the elements is equally likely to occur. It has a time complexity of O(n), where n is the number of elements in the array or list, making it efficient for shuffling large collections. i can code this in c,c++, python
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
please assign me this issue @Kumar-laxmi

Assigned!
@shravanii2308 : C, C++, Java & Python


Fix flake8 config issue

Codecov Report

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

@@           Coverage Diff            @@
##             devel     #192   +/-   ##
========================================
  Coverage         ?   89.97%
========================================
  Files            ?       10
  Lines            ?     2603
  Branches         ?      573
========================================
  Hits             ?     2342
  Misses           ?      169
  Partials         ?       92

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 3b5d85f...bfbe12d. Read the comment docs.


Subquery with another table in BatchUpdate

I have 2 tables: Assets and MaximoAssets, linked by MaximoAssetID column in Assets table.
I want to set MaximoAssetID for each record based on custom criteria:
asset.SerialNumber = maximoAsset.AssetNum
So the actual SQL query can look like this:
UPDATE A
SET A.[MaximoAssetID] = B.ResourceID
FROM [Assets] AS A
INNER JOIN maximo.Assets AS B
ON A.[SerialNumber] = B.[AssetNum]

I have done many attempts with EF+ and got mainly invalid SQL and at the least erroneous LINQ query was
context.Assets.Update(x => new Asset()
    {
        MaximoAssetId = context.MaximoAssets
            .Where(ma => ma.AssetNum == x.FinanceNumber)
            .Select(ma => ma.ResourceId)
            .SingleOrDefault()
    }
);

which produced this SQL query (some columns from JOIN are removed):
UPDATE A
SET A.[MaximoAssetID] = B.[SerialNumber]
FROM [Assets] AS A
INNER JOIN (
    SELECT [a].[AssetID], [a].[FinanceNumber], [a].[MaximoAssetID], [a].[PlateNumber], [a].[SerialNumber]
    FROM [Assets] AS [a]
) AS B ON A.[AssetID] = B.[AssetID]

As you see there is no appearance of MaximoAssets table, but SerialNumber value used in == is assigned. When I tried other columns (e.g. FinanceNumber) in ==, they appeared in Update statement.
As far as I learned from LINQ syntax and generated SQL, BatchUpdate can be used to update columns based only on current table values. Is it possible in general to use other tables data?
For instance, this is LINQ query which EF Core translates to single valid SQL squery:
context.MaximoPM
    .Where(x => context.Assets.Any(a => a.SerialNumber == x.AssetNum))
    .ToList();

You see, inside WHERE clause I am able to access any table.

Hello @Hauptsatana ,
I don't think your scenario is possible. You are using the context inside the Update method and that's something we do not support.
Inside the Update, you should use your x with Navigation Properties:
context.Assets.Update(x => new Asset()
    {
        MaximoAssetId = x.MaximoAssets
            .Where(ma => ma.AssetNum == x.FinanceNumber)
            .Select(ma => ma.ResourceId)
            .SingleOrDefault()
    }
);

I'm not sure if it will work either if you have a navigation property but it worth a try.
If you can provide us a test project with this scenario, I will be happy to look a little to it to determine if there is something we can do.
Best Regards,
Jonathan

Hello @Hauptsatana ,
This issue will be closed since we haven't heard back from you.
Feel free to reopen it if you can provide the information we asked.
Best Regards,
Jonathan

Using a navigation property does not appear to work:
var foos = context.Foos.Where(f => f.DeletedAt == null);
foos.Update(f => new Foo { Sum = f.Bars.Sum(b => b.Value) });

Produces:
System.Data.SqlClient.SqlException : Incorrect syntax near 'Id'.
   at System.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at System.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)
   at System.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
   at System.Data.SqlClient.SqlCommand.RunExecuteNonQueryTds(String methodName, Boolean async, Int32 timeout, Boolean asyncWrite)
   at System.Data.SqlClient.SqlCommand.InternalExecuteNonQuery(TaskCompletionSource`1 completion, Boolean sendToPipe, Int32 timeout, Boolean asyncWrite, String methodName)
   at System.Data.SqlClient.SqlCommand.ExecuteNonQuery()
   at Z.EntityFramework.Plus.BatchUpdate.Execute[T](IQueryable`1 query, Expression`1 updateFactory)
...

Hello @psyren89 ,
Could you open a new thread about this issue?
Best Regards,
Jonathan

Done: #416


Convert apply-tier-4-visa to ERB

Nice! Have you regenerated HTML artefacts for regression tests? I'm pleasantly surprised not even whitespace changes were introduced.
:+1:

Have you regenerated HTML artefacts for regression tests? I'm pleasantly surprised not even whitespace changes were introduced.

Yup, and me too :-) I think the template could be made slightly nicer with a little more whitespace but for now I'm mostly interested in keeping the output identical.

The "Update apply-tier-4-visa-files.yml" commit looks as if it's at the wrong point in the git history i.e. I would've expected it to be last. Otherwise this looks good to me.

The "Update apply-tier-4-visa-files.yml" commit looks as if it's at the wrong point in the git history i.e. I would've expected it to be last.

I reordered the commits and merged this branch to master in f0081a48c31345397619304d6ce9fa8f724567e2.


Unconditionally require Meson subdir fix

Fixes #170

Notet that I didn't update the runtime requirement from Meson 0.62.0. I'll leave that to you.

I agree with Ralf, we should update project.dependencies too.


cfn cluster is getting created and after few mins its getting roll back

cfn cluster is getting created and after 10 mins its getting roll back.
Received 1 FAILURE signal(s) out of 2. Unable to satisfy 100% MinSuccessfulInstancesPercent requirement

Specify the --norollback flag when you do a create. Then once the stack is in CREATE_COMPLETE or CREATE_FAILED, grab the ip address of the master instance from the console and ssh in. Start with the master instance and look at the /var/log/cfn-init.log and /var/log/cfn-init-cmd.log logs.
Looks like the problem is with one of the compute nodes. You're going to want to ssh into those nodes and look at the same logs. Grab the private ip from ec2 console.
This is either a problem with our bootstrap cookbook or your pre/post install script. Post your config (with aws creds removed) so we have more context.


RcmAdmin: Bootstrap Viewport

Add a “Bootstrap Viewport” switcher to modify the layouts for the different views (xs, sm, lg, ect).

Please try to add new js to RCM Admin, try not to change existing (it is fragile).
I had a bit of JS that was made to handle this in RcmAdmin, not sure if it is still there

Please try to add new js to RCM Admin, try not to change existing (it is fragile).


Russian text in inline block with --listing

Create md-doc and placed in any space follow text:
Text Русский текст other text
Try to run converting md to pdf (on windows system)...
Command to starting converting:
pandoc "t.md" -o t.pdf -t latex --pdf-engine=xelatex -f markdown --template ".\settings\eisvogel.tex" --listing
And result:
Error producing PDF. ! Undefined control sequence. \lst@arg ->╨а             ╤Г╤Б╤Б╨║╨╕╨╣ ╤В╨╡╨║╤Б╤В l.386 Text \passthrough{\lstinline!╨а╤Г╤Б╤Б╨║╨╕╨╣ ╤В╨╡╨║╤Б╤В!}
If disable option '--listing', then pdf build successful

To quote the listingsutf8 documentation:

Package listings does not support files with multi-byte encodings such as UTF-8.

This means that typesetting Russian text in listings is not possible without extensive work. It may be possible to redefine all the listings commands to use listingsutf8 instead but I don't have the time to experiment with this (PRs are welcome though).
As you already noticed, a possible workaround is running pandoc without --listings so that it typesets code using the verbatim environment.


Some emissive materials not shown correctly

I have some emissive materials that are not shown correctly.
It seems that the RayTracingObject BaseColor needs to be set manually
On my emissive materials the baseColor is black and only the emissive color is set:

Seems that the value then not gets applied correctly:

On first creation/if told to link, RayTracingObjects will inherit the properties of their associated material with respect to the links defined in the material linking xml, but it will only happen when the object is built


Refined Genetic Attributes Structure

Review and refine the current genetic attributes structure to accommodate a broader range of traits and interactions, ensuring coherence and consistency in creature generation.

[x] Analyze the existing genetic attributes structure.
[x] Identify areas for improvement or expansion to support additional traits and interactions.
[x] Propose and implement modifications to the structure to enhance flexibility and coherence.

Covered by #68 SIF


Open suggestions even if char is not a letter

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.89.1
OS Version: Windows 10

Suggestion menu seems to be automatically shown only when typing letters.
Other characters require a manual opening of the suggestions menu.
https://github.com/microsoft/vscode/assets/5413308/042b7c47-ab6e-44f2-b692-fe3e6b9971ec
(I would like my snippets to appear automatically in all cases, even if they are not triggered by letters.)

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


View Syncmatic button not clickable.

The button is simply just black.
Others on server can use the mod.
Note: i am using multiconnect to connect to 1.17.1 from 1.19.1

Hmm Multiconnect might block syncmatica then.


Add a Gitter chat badge to Readme.md

okhosting/OKHOSTING.Data now has a Chat Room on Gitter
@okhosting has just created a chat room. You can visit it here: https://gitter.im/okhosting/OKHOSTING.Data.
This pull-request adds this badge to your Readme.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

badge


Reorganization

So, despite the confusing name, amd64 is x86_64.
So I suppose realistically we should only include one of x86 and amd64, unless we plan on supporting 32-bit machines (which may be reasonable, if there's legacy hardware in clusters).

So, despite the confusing name, amd64 is x86_64.
So I suppose realistically we should only include one of x86 and amd64, unless we plan on supporting 32-bit machines (which may be reasonable, if there's legacy hardware in clusters). In which case we should clear that up.

Please provide a reference.

So, despite the confusing name, amd64 is x86_64.
So I suppose realistically we should only include one of x86 and amd64, unless we plan on supporting 32-bit machines (which may be reasonable, if there's legacy hardware in clusters). In which case we should clear that up.

Please provide a reference.

https://en.wikipedia.org/wiki/X86-64
Or just any google search. It's relatively commonly known. But I put in x86 mostly by mistake cause I couldn't actually test it. x86_64 is known as AMD64 because AMD were the ones that invented it. We may still have a 32-bit machine which is just x86 and not AMD64. It's not to be confused with e.g. ARM


Remove Geyser Version Dependency

Ideally one should be able to drop GeyserReversion into any version of Geyser and it should just "work". However presently some changes in Geyser will break GeyserReversion as seen in https://github.com/GeyserMC/Geyser/pull/1640 with a simple fix in #24
A fix is:

Create submodules for each breaking change from Geyser based upon its commit hash
Abstract away from the current way of interfacing with Geyser and instead pass to the appropriate submodule as required

Not sure if this has anything to do with this but i'll put it here anyway
https://github.com/Hellohi3654/Geyser/issues/233

I believe this is an issue because geyserreversion needs to support bedrock v422

This was resolved through reflection but I'll have a look at the link tonight to see what the issue may be.

I might leave this open as even today I had to make a breaking change with GeyserCommand changing syntax. I'm not sure how important it is that this supports multiple versions of Geyser but I can see future issues when people are running multiple forks of Geyser where some forks have future features OR when GeyserReversion does not yet support the latest Geyser.
May need to think about how to abstract this out in a module system but since Geyser doesn't use versions we'll need reflection to autodetect when to do what.

I might leave this open as even today I had to make a breaking change with GeyserCommand changing syntax. I'm not sure how important it is that this supports multiple versions of Geyser but I can see future issues when people are running multiple forks of Geyser where some forks have future features OR when GeyserReversion does not yet support the latest Geyser.
May need to think about how to abstract this out in a module system but since Geyser doesn't use versions we'll need reflection to autodetect when to do what.

Wouldn't it be easier and more universally useful to write to the protocol version and not to a Geyser commit?  Geyser is undergoing rapid development, but the common goal is support for a (or a few) protocol versions.
I see the ViaVersion and ProtocolSupport are plugins as well so there must be a benefit to that design, but it really would be nice to have a protocol proxy for Bedrock since many (but not all) clients are updated very quickly and it seems the servers are always playing catch up.  Just supporting a few recent protocol version would help with client/server compatibility.

Closing this but just a note that I am indeed going to start a new project just dealing with protocol conversion likely as a proxy that sits in front of Geyser (or indeed bedrock itself).


msg : string [51] Credential type 'alexa-home-conf' is not registered

Hi.
I registered and can log in on https://alexa-node-red.bm.hardill.me.uk/devices (I tried it using copy/paste).
Adding a Alexa home Node, filling in the data and deploying I always get this message.
I'm clueless what the error is.

Hmm. Could be an old installation, currently it is 0.15.x.

Moving to 0.16.2 solves the problem.
Hint: execute update-nodejs-and-nodered


Add support to Symfony 3

deptrac requires all Symfony components in ~2.7. It can be interesting to add Symfony 3 components support.
Also, this library is incompatible with sensiolabs-de/deprecation-detector: it requires nikic/php-parser ~2.0, but deprecation-detector requires nikic/php-parser ^1.3.

can you provide more information about what exactly is missing and why it would be useful?
i use deptrac in some symfony 3 projects :)

@slde-flash My bad, I updated the description :)

This issue is just about installing this library. Binary file (as phar) works in Symfony 3 project

ah got it :) i would recommend using the phar file. because you don't want deptracs dependencies in your project.
i'll open an issue to clarify that you should use the phar file instead on using composer to require it.
i am going to update all composer dependencies on a regular basis.
right now we don't have a release cycle (which would include updating all dependencies).

I usually install sensiolabs-de dependencies in my project (like deprecation-detector) to have binary directly in my project. This tool changes a bit, that's why


Intermittent build failures due to a Spark driver issue

The Spark driver used by bulkdata webapp causes the build to intermittently fail due to the following error:
java.net.BindException: Cannot assign requested address: Service ‘sparkDriver’ failed after 16 retries (starting from 38000)! Consider explicitly setting the appropriate port for the service ‘sparkDriver’ (for example spark.ui.port for SparkUI) to an available port or increasing spark.port.maxRetries

We originally thought that there was a port conflict between the Sparq driver and Berkeley DB Java Edition (used by Janusgraph). This prompted the following change:
        spark = SparkSession.builder()
            .appName("parquetWriter")
            .master("local[*]")
            .config("spark.ui.enabled", false)
            .config("spark.driver.port", "38000")
            .getOrCreate();

setting the (starting) port to 38000 thinking this would avoid the conflict. Unfortunately the problem persists. @prb112 suggested that we change the maxRetries value to 1000.

We've done quite a few builds since this was merged and I havn't seen that issue again yet...lets consider it fixed and we can reopen if it pops up again.


Remove initial line feed plugin + php component = bad line numbering

The "remove initial line feed" plugin gets triggered on "before-highlight" and changes env.code (removes the redundant initial line feed).
The php component has a hook on "before-insert", which restores env.code to env.backupCode.
The line numbering plugin gets triggered on "complete" and uses env.code to figure out how many lines there are so it counts an extra line.
Possible fix (works fine for me): env.backupCode = env.code; in the Remove Initial Line Feed plugin.

@Golmote @zeitgeist87 Why do we have a remove initial line feed plugin? Isn't this covered by the whitespace normalizer?

@Golmote @zeitgeist87 Why do we have a remove initial line feed plugin? Isn't this covered by the whitespace normalizer?

The remove initial line feed plugin is deprecated. It says so in all caps on its page. We could remove it in the next release.


Canary Causes Owo or Minecells to crash on server

Mod Name
Canary
Mod Homepage
https://www.curseforge.com/minecraft/mc-mods/canary
Describe the bug
Crashes on Server
Steps to reproduce
When they are together in a server it causes a halt of the server continuing not even crashing.
[06.09 17:47:19] [Server] [main/ERROR] [minecraft/Main]: Failed to start the minecraft server
[06.09 17:47:19] [Server] java.lang.RuntimeExceptionMod fluids must override getFluidType.
[06.09 17:47:19] [Server]     at net.minecraftforge.common.ForgeHooks.getVanillaFluidType(ForgeHooks.java:902) ~[forge-1.20.1-47.1.3-universal.jar%23661!/:?]
Logs
https://pastebin.com/B0g6quz1
Additional context
No response

Isnt much of a big deal, just wanted to make it known for other people in the future

I see what I did here, whoops. Fluid type registration is skipped when there is no renderer present, which there can never be on a server anyway.


Fixed AutoToot not Tooting on 1.0895B

Not working on previous version. Also haven't fixed the lerp easing not working. I believe its because _lastNoteEndTime and _lastNoteEndY values are never set anywhere so they are always worth 0??

lgtm - tested and works as expected on my machine. Thanks for the PR! :)


Change the file logger directory to use tfm

Old Path: Logging\artifacts\logs\Microsoft.Extensions.Logging.Testing.Tests\NET Framework 4.7.2650.0\AssemblyTestLogTests\ForAssembly_ReturnsSameInstanceForSameAssembly.log
New Path: Logging\artifacts\logs\Microsoft.Extensions.Logging.Testing.Tests\net461\AssemblyTestLogTests\ForAssembly_ReturnsSameInstanceForSameAssembly.log

Originally I wanted to make a separate PR for setting the default file logging location but since I opted for the assembly attribute method of setting the TFM, I might as well include this change as well.
The default location is $(RepositoryRoot)/artifacts/logs. This can be overridden by specifying a different location via ASPNETCORE_TEST_LOG_DIR.

Also added fix to close file handles


qualifying Microsoft 365 subscription

If your organization has a qualifying Microsoft 365 subscription and five (5) or more users, you can change the storage space to more than 5 TB. To discuss your needs, contact Microsoft support. You must assign at least one license to a user before you can increase the default OneDrive storage space.. "
What are the qualifying plan please elaborate ?

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

ID: 0252c106-56e3-96be-5935-ad8aa488e9bd
Version Independent ID: 901c0c99-c360-3db1-42bc-35f4c70da8b3
Content: Set the default storage space for OneDrive users - OneDrive
Content Source: OneDrive/set-default-storage-space.md
Service: one-drive
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

Hello @robinpoulose1 how are you? I am looking into this

Hello @robinpoulose1
You can see here which plans have eligible unlimited OneDrive storage
https://www.microsoftpartnercommunity.com/atvwr79957/attachments/atvwr79957/NLMW/96/3/Modern Work User Subscription Plan Comparison for Enterprise.pdf
Hope this information helps you

Hello @robinpoulose1
You can see here which plans have eligible >5TB OneDrive storage
https://www.microsoftpartnercommunity.com/atvwr79957/attachments/atvwr79957/NLMW/96/3/Modern Work User Subscription Plan Comparison for Enterprise.pdf
Hope this information helps you

Thanks ! it  is clarified now.


Get selenium tests working with Brave

Works on Windows -- I can't get it to work on Linux so far.

working on mac; good enough for now


RuntimeError: expected scalar type Half but found BFloat16

Is this about me using zluda , used your workflow from reddit and only changed the clip from cuda to cpu

solution?


[datasets] Allow detection task for built-in datasets

This PR:

makes it possible to use built-in datasets also to plug in the detection training script
extend tests
update eval scripts - no it works with different args out of the box

Any feedback is welcome

@odulcy-mindee I tried to handle it with kwargs in the _AbstractDataset but to be honest I didn't like to forward both args only to raise the error ..because in this case it feels like both (detection_task and recognition_task) would do anything in the "background" 😅
Do you know what i mean ?
Edit: double checked - we would need to init _AbstractDataset with detection_task and recognition_task to handle this well at one point for each dataset ... but from the logic which is handled by the base class this doesn't fit
class _AbstractDataset:
    data: List[Any] = []
    _pre_transforms: Optional[Callable[[Any, Any], Tuple[Any, Any]]] = None

    def __init__(
        self,
        root: Union[str, Path],
        img_transforms: Optional[Callable[[Any], Any]] = None,
        sample_transforms: Optional[Callable[[Any, Any], Tuple[Any, Any]]] = None,
        pre_transforms: Optional[Callable[[Any, Any], Tuple[Any, Any]]] = None,
    ) -> None:
        if not Path(root).is_dir():
            raise ValueError(f"expected a path to a reachable folder: {root}")

        self.root = root
        self.img_transforms = img_transforms
        self.sample_transforms = sample_transforms
        self._pre_transforms = pre_transforms
        self._get_img_shape = get_img_shape

    def __len__(self) -> int:
        return len(self.data)

    def _read_sample(self, index: int) -> Tuple[Any, Any]:
        raise NotImplementedError

    def __getitem__(self, index: int) -> Tuple[Any, Any]:
        # Read image
        img, target = self._read_sample(index)
        # Pre-transforms (format conversion at run-time etc.)
        if self._pre_transforms is not None:
            img, target = self._pre_transforms(img, target)

        if self.img_transforms is not None:
            # typing issue cf. https://github.com/python/mypy/issues/5485
            img = self.img_transforms(img)

        if self.sample_transforms is not None:
            # Conditions to assess it is detection model with multiple classes and avoid confusion with other tasks.
            if (
                isinstance(target, dict)
                and all(isinstance(item, np.ndarray) for item in target.values())
                and set(target.keys()) != {"boxes", "labels"}  # avoid confusion with obj detection target
            ):
                img_transformed = _copy_tensor(img)
                for class_name, bboxes in target.items():
                    img_transformed, target[class_name] = self.sample_transforms(img, bboxes)
                img = img_transformed
            else:
                img, target = self.sample_transforms(img, target)

        return img, target

    def extra_repr(self) -> str:
        return ""

    def __repr__(self) -> str:
        return f"{self.__class__.__name__}({self.extra_repr()})"

wdyt ?

Here, I see what you mean. It lacks a variable is_recognition_dataset and is_detection_dataset in my opinion, but that's for another PR

I think too for the moment we can stay with it as is, but this would be an possible way to clean up the code a bit :+1:


Usage of deprecated methods on the helper

Describe the bug 🐛
Among other things I haven't found yet, helper.toggleRefinement -> helper.toggleFacetRefinement
To Reproduce 🔍
In lots of places, one is in connectMenu for example, but it's best to do them all at once, also change the tests etc.
Expected behavior 💭
Replace all usage of deprecated helper functions, so we can remove them

(for Hacktoberfest, you can done one by one and it's still useful!)

I take it :)