Option to pre-render the dynamic routes

Currently you can use a '*' as a wildcard in kit.prerender.pages to specify all non-dynamic routes. This adds a new token to do the dynamic routes, initially picked to be '**' but could be anything.
When pre-rendering a dynamic route, also creates the convention that params are filled in with the param name in square brackets, such that if you have a route like:
/article/[slug].svelte

Which would be concretely referenced by the user as say "https://mysite.com/article/cool-article", the resulting HTML artifact in the /build folder lives at:
/build/article/[slug]/index.html

This relates to https://github.com/sveltejs/rfcs/pull/52 and https://github.com/sveltejs/kit/issues/1561, and enables one to use adapter-static to generate artifacts which can be used in any type of server runtime, even if you have dynamic routes. Obviously in such a scenario, any parts of the page that depend on request-specific data would need to be client-rendered. This essentially enables a "static SSR / client-rendered" hybrid, where the shell of the page is static prerendered and some inner data-driven part is client-rendered.
Not very optimistic this will get approved, but taking a shot anyway. If this needs to go in a fork of SvelteKit, that's understandable.
Before submitting the PR, please make sure you do the following

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

How would someone visiting article/cool-article be served article/[slug]/index.html?

How would someone visiting article/cool-article be served article/[slug]/index.html?

So there's how I plan to do it, how someone could do it generally serverful, and maybe how it could be done serverless.
I'll end up having a Java (or maybe Kotlin) web server that loads all those HTML files into memory, maintaining their tree structure, and just splits the request path into parts and looks for each part, falling back to a part name involving square brackets if that exists. Failing that it would serve the 400 page.
The same idea could work for serverful setups involving Java, Python, NodeJS, whatever you like. I worked at a company one time that used Django (this was offerup.com), and I can imagine wanting to iteratively migrate pages over from Django to Svelte without the added architectural complexity of putting a reverse proxy in front. One could simply have Django serve the static files, even when the paths have wildcards.
Happy to make an example reference implementation of a "serve the static assets" in whatever language you like, or pseudocode.
For serverless, one could copy all the /build folder up to say AWS S3 and serve it via Cloudfront. In this case you'll need some path rewriting, which actually you'd need anyway to serve the index.html files. So you transform a request for / to serve the file /index.html off disk. For these parameterized routes, you'd have to add some path rewriting that says for example /article/* => serve /build/article/[slug]/index.html. All this rewriting can be done in AWS with Lambda@Edge, simple Node functions to rewrite the URL.

I don't think we'd want to add something that requires the user to do so much work to use. Anything we added should probably be supported by at least one of the SvelteKit adapters.

Fair enough. Not having this isn't really blocking anybody, it just means they'll need to duplicate the route information as determined by the folder structure, in the svelte config. Perhaps we can revisit this if other people end up wanting to do the same thing, that is using static pre-rendering for the general case, which will often include parameterized paths.
I'll close, but glad this is on record and people can refer to it if they want. For what I plan to use SvelteKit for in the enterprise environment, we probably wouldn't even be productionizing until a year from now, so it's not urgent anyhow.


Cache expires value always undefined

hi and thanks for store2.
trying to set cache expire value, min (ie. fourth parameter) value is always undefined when debug the _.set function in store.cache.js
I should not understood something but what ? Here is my code
var ustorage = store.namespace(uniqId);
ustorage.set('mykey', {firstdata:1, seconddata: 1}, 60);
60 seconds not retrieved in cache !
Same if i do not use namespace.
Thanks

Are you sure you have store.cache.js loaded in that window?
https://raw.githubusercontent.com/nbubna/store/master/src/store.cache.js
Cache won't work without the cache extension.

@nbubna thanks for quick answer. Yes store.cache.js is loaded. When i console.log(area, key, string, min) in the _.set function in store.cache.js, result is ok for area, key, string but min is undefined.
After test, i found that the stringify function inside store2 (2.14.0) does not return the cache integer value but only the second parameter. I will test again for this stringify function.
Stefan

Sure i do no not understand how works your stringify function, as for me, never return third parameter. So cache value is undefined and ignored. Perhaps miss understood the _replacer param?
Thanks

No, it's not you. There's a bug. When i added replacer support, i broke store.cache.js. I should only be passing "overwrite" to stringify as a replacer if it is a function, otherwise it needs to be passed on to _.set so extensions can use it. I'll try to fix this Thurs. Sorry, no time before then, i think.

The 2.14.1 release is out, it fixes what was broken.


stackItems attempt 2

Issue Number
#397 replacement because I'm terrible at using git
Overview of PR
Removing the API updates that were accidentally merged in to #397 turned out to be an ordeal. This PR should be cleaner.
Don't forget to update the CHANGELOG.md file with any changes that are in this PR

Thank you for reviewing.
Most of calendar.js that I change I don't fully understand the implementation, I'm just doing copy paste job on functions and changing where required.
I would prefer to open a new issue for coverage and testing. Code coverage of this component is around 23% at the moment. There is very little testing for the user interaction, which is what this component is built around. I currently have no way to test #401

@acemac  I found this line
https://github.com/namespace-ee/react-calendar-timeline/blob/master/src/lib/utility/calendar.js#L444
any idea on what is isOverlay?

@acemac this code in stackItems is like a boom. I will get to unit test it after custom headers hopefully

@acemac I've been unit testing and refactoring stackItems and stackAll utils from this PR. It is still work in progress but I would appreciate your feedback
https://github.com/acemac/react-calendar-timeline/pull/2


Multi project setup: line count returns first project's line count only

This opens line counter for the first project, instead of the targeted project:

+1

+1

+1


Create LED methods and access them by their colors.

This PR introduces the led_set and led_toggle methods, as well as an enum list of available colors for selection.
The changes are fully documented and supported by examples.

Can we add a method to read the leds state?

Can we add a method to read the leds state?

Please take a look, added new commit that implements this method.


Batch report - Add Time points

Add optional column with time points in Table 1.

Re-opening. Time points are available from Pinery. This will be a temporary solution until time points are carried over to File Provenance Report.


Error report does not show module name anymore

The previous version displayed the module name in which there was an error. Now this has been replaced with a button to jump to that module. Would it be possible to continue to show the module name as well?

I just came here to open this issue too -- would be great to have back.  Without it there's some unnecessarily processing where I have to study the code a bit and figure out which file it's in.  I think it used to be up where the module jumping button now is:

➕ to include the file name.
Not sure what is the behavior of the "Jumpo to problem" button (maybe open in the default editor?) but it does nothing on my setup.

The latest release (v1.3.5) has added back in the relative filepath, and the button styling has been removed to make it easier to select the text, too!

What's the best way to update this with elm-land? Is it built in, e.g. going to require a new elm-land version?

Looks like elm-land pins the vite-plugin-elm-watch version exactly: https://github.com/elm-land/elm-land/blob/31beffca1e5ed3ebd93acc34d1563f566e23a617/projects/cli/package.json#L47
So, yes, it needs a new elm-land version.
I think you can add something like this to your package.json as a workaround (I’ve used overrrides before, but I always have to fiddle with it and sometimes remove package-lock.json, haven’t tested this right now):
"overrides": {
  "vite-plugin-elm-watch": "1.3.5"
}


Fixed iOS 8 issue with arrow direction

Currently, the calculation for the available view size for the popover compensates for orientation in a way that is unnecessary on iOS 8, since the view has already been rotated according to device orientation.
This fixes that.

This looks good. However, the library supports iOS 7 too. Suggest introducing this inside a check if there are any regressions.

For what it's worth, the app that we're developing also supports iOS 7 and this calculation appears to function correctly for us on both 7 and 8. It should be doing the exact same thing as before on iOS 7, since the _ignoreOrientation flag is only YES on iOS 8.
Without this fix, it's a little buggy on iOS 8 in that it can choose arrow directions that force the popover to be compressed beyond the preferredContentSize, even if there's plenty of room in another direction.

:+1:


Unable to compile Realm when targetting watchOS

How frequently does the bug occur?
All the time
Description
Am using Realm integrated in a pure SPM library project that targets iOS, tvOS macOS and watchOS. With latest update of packages can no longer build for watchOS. The error is:
.../simulated_failure.cpp:174:1: error: thread-local storage is not supported for the current target
thread_local bool (*s_mmap_predicate)(size_t);
^
1 error generated.

Stacktrace & log output
No response
Can you reproduce the bug?
Yes, always
Reproduction Steps
Sample project is available here
clone project
from project root directory
$ open Package.swift
In xcode select watchOS as run target
Build
Version
10.21.1
What SDK flavour are you using?
Local Database only
Are you using encryption?
Yes, using encryption
Platform OS and version(s)
watchOS 7.4. up
Build environment
Xcode version: 13.2.1
Dependency manager and version: Xcode SPM (swift 5.5.2)

Hi @mwoollard. I still cannot reproduce your issue even with the sample project. Are you using macOS Monterey?

Yes, using Monterey, am also seeing this both on my dev laptop (M1) within Xcode and CI scripted builds (Intel).

Additional, FWIW, if I change the compile time condition @ line 171 to
#if (REALM_ARCHITECTURE_X86_32 && REALM_IOS) || REALM_WATCHOS

Code compiles

Thanks for the suggestion, I created a PR for it on core. You can keep track of the issue here and on the PR https://github.com/realm/realm-core/pull/5188.

This has been fixed on core and will be available shortly.

This is still happening using SPM and Realm 10.24.0. Can we reopen this please?

Are you still getting the same error?

yes

I think this is now only issue when targeting M1 simulator. I created new issue in realm-core here. Could you confirm you are using M1 Mac?

I am not using a M1 Mac.

This is what I get when I try to add Realm to an watch extension using SPM

The same issue for my project with the latest Realm 10.24.2 on my Intel Mac.
But the problem is only happened on the simulators, not when testing on actual device.

@dianaafanador3 Facing the same issue, I can confirm that your branch fixed the issue for me (on WatchOS). Any plans to merge these changes?


Uncaught Error: Cannot read property 'object' of undefinedTypeError: Cannot read property 'objec...

[Enter steps to reproduce below:]

...
...

Atom Version: 1.10.2
Electron Version: 0.37.8
System: Mac OS X 10.11.6
Thrown From: Atom Core
Stack Trace
Uncaught Error: Cannot read property 'object' of undefined
TypeError: Cannot read property 'object' of undefined
at ObjectsRegistry.get (/Applications/Atom.app/Contents/Resources/electron.asar/browser/objects-registry.js:44:28)
at EventEmitter. (/Applications/Atom.app/Contents/Resources/electron.asar/browser/rpc-server.js:309:31)
at emitMany (events.js:121:13)
at EventEmitter.emit (events.js:195:7)
at EventEmitter. (/Applications/Atom.app/Contents/Resources/electron.asar/browser/api/web-contents.js:141:25)
at emitTwo (events.js:100:13)
at EventEmitter.emit (events.js:185:7)
At /Applications/Atom.app/Contents/Resources/electron.asar/renderer/api/remote.js:178

Error: Cannot read property 'object' of undefined
TypeError: Cannot read property 'object' of undefined
    at ObjectsRegistry.get (/Applications/Atom.app/Contents/Resources/electron.asar/browser/objects-registry.js:44:28)
    at EventEmitter.<anonymous> (/Applications/Atom.app/Contents/Resources/electron.asar/browser/rpc-server.js:309:31)
    at emitMany (events.js:121:13)
    at EventEmitter.emit (events.js:195:7)
    at EventEmitter.<anonymous> (/Applications/Atom.app/Contents/Resources/electron.asar/browser/api/web-contents.js:141:25)
    at emitTwo (events.js:100:13)
    at EventEmitter.emit (events.js:185:7)
    at metaToValue (/Applications/Atom.app/Contents/Resources/electron.asar/renderer/api/remote.js:178:13)
    at undefined.remoteMemberFunction (/Applications/Atom.app/Contents/Resources/electron.asar/renderer/api/remote.js:112:18)
    at AtomEnvironment.module.exports.AtomEnvironment.getWindowDimensions (/Applications/Atom.app/Contents/Resources/app.asar/src/atom-environment.js:636:29)
    at AtomEnvironment.module.exports.AtomEnvironment.storeWindowDimensions (/Applications/Atom.app/Contents/Resources/app.asar/src/atom-environment.js:670:36)
    at WindowEventHandler.module.exports.WindowEventHandler.handleWindowBlur (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:182:35)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:3:61

Commands
     -1:04.8.0 core:paste (atom-text-editor.editor.is-focused)
  2x -1:04.3.0 core:save (atom-text-editor.editor.is-focused)
     -0:20 core:paste (atom-text-editor.editor.is-focused)
  3x -0:19.6.0 core:save (atom-text-editor.editor.is-focused)
     -0:02.4.0 core:paste (atom-text-editor.editor.is-focused)
  3x -0:01.9.0 core:save (atom-text-editor.editor.is-focused)

Config
{
  "editor": {}
}

Installed Packages
# User
atom-beautify, v0.29.13 (active)
autocomplete-ruby, v0.1.0 (active)
rails-i18n, v0.1.3 (active)
rails-snippets, v2.8.0 (inactive)
ruby-slim, v0.2.0 (active)
atom-dark-syntax, v0.27.0 (inactive)
atom-dark-ui, v0.52.0 (inactive)
atom-light-syntax, v0.28.0 (inactive)
atom-light-ui, v0.44.0 (inactive)
base16-tomorrow-dark-theme, v1.1.0 (inactive)
base16-tomorrow-light-theme, v1.1.1 (inactive)
one-dark-ui, v1.5.0 (active)
one-light-ui, v1.5.0 (inactive)
one-dark-syntax, v1.3.0 (active)
one-light-syntax, v1.3.0 (inactive)
solarized-dark-syntax, v1.0.2 (inactive)
solarized-light-syntax, v1.0.2 (inactive)
about, v1.5.3 (active)
archive-view, v0.61.1 (active)
autocomplete-atom-api, v0.10.0 (active)
autocomplete-css, v0.11.2 (active)
autocomplete-html, v0.7.2 (active)
autocomplete-plus, v2.31.1 (active)
autocomplete-snippets, v1.11.0 (active)
autoflow, v0.27.0 (inactive)
autosave, v0.23.1 (active)
background-tips, v0.26.1 (active)
bookmarks, v0.41.0 (active)
bracket-matcher, v0.82.1 (active)
command-palette, v0.38.0 (inactive)
deprecation-cop, v0.54.1 (active)
dev-live-reload, v0.47.0 (active)
encoding-selector, v0.22.0 (active)
exception-reporting, v0.39.0 (active)
find-and-replace, v0.201.0 (inactive)
fuzzy-finder, v1.3.0 (active)
git-diff, v1.1.0 (active)
go-to-line, v0.31.0 (inactive)
grammar-selector, v0.48.1 (active)
image-view, v0.58.2 (active)
incompatible-packages, v0.26.1 (active)
keybinding-resolver, v0.35.0 (active)
line-ending-selector, v0.5.0 (active)
link, v0.31.1 (inactive)
markdown-preview, v0.158.0 (active)
metrics, v0.53.1 (active)
notifications, v0.65.0 (active)
open-on-github, v1.2.0 (inactive)
package-generator, v1.0.0 (inactive)
settings-view, v0.242.2 (active)
snippets, v1.0.2 (active)
spell-check, v0.67.1 (active)
status-bar, v1.4.1 (active)
styleguide, v0.47.0 (active)
symbols-view, v0.113.0 (inactive)
tabs, v0.100.2 (active)
timecop, v0.33.2 (active)
tree-view, v0.208.2 (active)
update-package-dependencies, v0.10.0 (active)
welcome, v0.34.0 (active)
whitespace, v0.33.0 (active)
wrap-guide, v0.38.1 (active)
language-c, v0.52.1 (active)
language-clojure, v0.21.0 (active)
language-coffee-script, v0.47.2 (active)
language-csharp, v0.12.1 (active)
language-css, v0.37.1 (active)
language-gfm, v0.88.0 (active)
language-git, v0.15.0 (active)
language-go, v0.42.1 (active)
language-html, v0.45.1 (active)
language-hyperlink, v0.16.0 (active)
language-java, v0.23.0 (active)
language-javascript, v0.119.0 (active)
language-json, v0.18.2 (active)
language-less, v0.29.5 (active)
language-make, v0.22.2 (active)
language-mustache, v0.13.0 (active)
language-objective-c, v0.15.1 (active)
language-perl, v0.35.0 (active)
language-php, v0.37.2 (active)
language-property-list, v0.8.0 (active)
language-python, v0.45.0 (active)
language-ruby, v0.69.0 (active)
language-ruby-on-rails, v0.25.0 (active)
language-sass, v0.56.0 (active)
language-shellscript, v0.22.4 (active)
language-source, v0.9.0 (active)
language-sql, v0.23.0 (active)
language-text, v0.7.1 (active)
language-todo, v0.28.0 (active)
language-toml, v0.18.0 (active)
language-xml, v0.34.9 (active)
language-yaml, v0.26.0 (active)

# Dev
No dev packages

This is a duplicate of #16922 - feel free to subscribe there for updates.

@50Wliu thanks


Add support for Java CompletableFuture

CompletableFuture is a sizable API with a non-trivial implementation, that we do not need in the core. So we won't integrate that in the core repo. This is why I'm closing this issue.
You can provide support for it in a separate library without depending on us, for example see https://github.com/scala-js/scala-js-java-logging as a small example, and https://github.com/cquiroz/scala-java-locales for another externally provided part of the JDK (not administered by the scala-js organization).

Thanks for that information.


remove highcharts from Whitelist in favor of @types/highcharts

https://github.com/DefinitelyTyped/DefinitelyTyped/pull/39599

is react-highcharts the only package that depends on highcharts? I don't understand why it's safe to remove highcharts from the whitelist.

I guess highcharts was unused in the whitelist.

react-highcharts, and highcharts-ng


Repo sync for protected CLA branch

The pull request is created from master637671493688197119 to master to fix git push error for protected CLA branch

@huypub : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Updating dependency versions

Description of changes:
Bumping cloudstack to latest and changing the linuxkit golang version.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm
/approve


Provide stable tags

Hi !
Nice image which does what is advertised ! Thanks.
It would be nice to provide stable tags (that do not change over time) such as (for the current Dockerfile) :

py3.7.3-gdal3.0.0

Currently, the stable tag will likely change when GDAL or python version is bumped.
Thanks,
Olivier

Hello, Olivier!
Thx for your interest.
I really wanted to do it at least when I updated GDAL to 3.0.0
So, what about to keep Dockerfiles with different versions of Python and
GDAL and separate directories? Or branches would be preferred?
On Tue, Jun 18, 2019 at 3:43 PM olivierdalang<EMAIL_ADDRESS>wrote:

Hi !
Nice image which does what is advertised ! Thanks.
It would be nice to provide stable tags (that do not change over time)
such as (for the current Dockerfile) :

py3.7.3-gdal3.0.0

Currently, the stable tag will likely change when GDAL or python version
is bumped.
Thanks,
Olivier
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/andrejreznik/docker-python-gdal/issues/10?email_source=notifications&email_token=ADUEGGQ3S2IPIBWLEB3K363P3DJ7BA5CNFSM4HY7ULCKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G2EIK4Q,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADUEGGU4MUXRP2CEYKAYQA3P3DJ7BANCNFSM4HY7ULCA
.

Hello, Olivier!
Thx for your interest.
I really wanted to do it at least when I updated GDAL to 3.0.0
So, now I decide what would be preferred to keep Dockerfiles with different versions of Python and GDAL in separate directories or use different branches.

I'd use branches for versions (as in the long term there may be plenty of them)


Documentation and cleanup of the Kyber avx2 code.

I expect that all modules will now use "intrinsics" instead of directly using core::arch::x86_64?

Yup.


Fix broken namespace logic in TopologySpreadConstraint

This logic is broken, when passing in any list of excluded namespaces it will skip all namespaces since it just looks at the size of include+exclude together. This PR breaks them out into separate checks for sanity
/kind bug
/priority critical-urgent

This looks good to me. I added a hold just in case @ingvagabund or someone else wants to review before merging. Feel free to remove the hold when you are ready for merge.
/lgtm
/hold

This looks good to me. I added a hold just in case @ingvagabund or someone else wants to review before merging. Feel free to remove the hold when you are ready for merge.
/lgtm
/hold

Jan is starting PTO today so may not see this. But since we were working on this earlier I'm comfortable removing the hold and letting this merge
/hold cancel


Tencent-Hunyuan-Large-MoE-389B-A52B

https://huggingface.co/tencent/Tencent-Hunyuan-Large
Model Introduction
With the rapid development of artificial intelligence technology, large language models (LLMs) have made significant progress in fields such as natural language processing, computer vision, and scientific tasks. However, as the scale of these models increases, optimizing resource consumption while maintaining high performance has become a key challenge. To address this challenge, we have explored Mixture of Experts (MoE) models. The currently unveiled Hunyuan-Large (Hunyuan-MoE-A52B) model is the largest open-source Transformer-based MoE model in the industry, featuring a total of 389 billion parameters and 52 billion active parameters. This is currently the largest open-source Transformer-based MoE model in the industry, featuring a total of 389 billion parameters and 52 billion active parameters.
By open-sourcing the Hunyuan-Large model and revealing related technical details, we hope to inspire more researchers with innovative ideas and collectively advance the progress and application of AI technology. We welcome you to join our open-source community to explore and optimize future AI models together!
Introduction to Model Technical Advantages
Model

High-Quality Synthetic Data: By enhancing training with synthetic data, Hunyuan-Large can learn richer representations, handle long-context inputs, and generalize better to unseen data.

KV Cache Compression: Utilizes Grouped Query Attention (GQA) and Cross-Layer Attention (CLA) strategies to significantly reduce memory usage and computational overhead of KV caches, improving inference throughput.

Expert-Specific Learning Rate Scaling: Sets different learning rates for different experts to ensure each sub-model effectively learns from the data and contributes to overall performance.

Long-Context Processing Capability: The pre-trained model supports text sequences up to 256K, and the Instruct model supports up to 128K, significantly enhancing the ability to handle long-context tasks.

Extensive Benchmarking: Conducts extensive experiments across various languages and tasks to validate the practical effectiveness and safety of Hunyuan-Large.

Benchmark Evaluation
Hunyuan-Large pre-trained model achieves the best overall performance compared to both Dense and MoE based
competitors having similar activated parameter sizes.  For aggregated benchmarks such as MMLU, MMLU-Pro, and CMMLU,
Hunyuan-Large consistently achieves the best performance, confirming its comprehensive abilities on aggregated tasks.
Hunyuan-Large also shows superior performance in commonsense understanding and reasoning, and classical NLP tasks
such as QA and reading comprehension tasks (e.g., CommonsenseQA, PIQA and TriviaQA).
For the mathematics capability, Hunyuan-Large outperforms all baselines in math datasets of GSM8K and MATH,
and also gains the best results on CMATH in Chinese.We also observe that Hunyuan-Large achieves the overall
best performance in all Chinese tasks (e.g., CMMLU, C-Eval).

Model
LLama3.1-405B
LLama3.1-70B
Mixtral-8x22B
DeepSeek-V2
Hunyuan-Large

MMLU
85.2
79.3
77.8
78.5
88.4

MMLU-Pro
61.6
53.8
49.5
-
60.2

BBH
85.9
81.6
78.9
78.9
86.3

HellaSwag
-
-
88.7
87.8
86.8

CommonsenseQA
85.8
84.1
82.4
-
92.9

WinoGrande
86.7
85.3
85.0
84.9
88.7

PIQA
-
-
83.6
83.7
88.3

NaturalQuestions
-
-
39.6
38.7
52.8

DROP
84.8
79.6
80.4
80.1
88.9

ARC-C
96.1
92.9
91.2
92.4
95.0

TriviaQA
-
-
82.1
79.9
89.2

CMMLU
-
-
60.0
84.0
90.2

C-Eval
-
-
59.6
81.7
91.9

C3
-
-
71.4
77.4
82.3

GSM8K
89.0
83.7
83.7
79.2
92.8

MATH
53.8
41.4
42.5
43.6
69.8

CMATH
-
-
72.3
78.7
91.3

HumanEval
61.0
58.5
53.1
48.8
71.4

MBPP
73.4
68.6
64.2
66.6
72.6

Hunyuan-Large-Instruct achieves consistent improvements on most types of tasks compared to LLMs having similar
activated parameters, indicating the effectiveness of our post-training.    Delving into the model performance
in different categories of benchmarks, we find that our instruct model achieves the best performance on MMLU and MATH dataset.
Notably, on the MMLU dataset, our model demonstrates a significant improvement, outperforming the LLama3.1-405B model by 2.6%.
This enhancement is not just marginal but indicative of the Hunyuan-Large-Instruct’s superior understanding and reasoning
capabilities across a wide array of language understanding tasks. The model’s prowess is further underscored in its performance
on the MATH dataset, where it surpasses the LLama3.1-405B by a notable margin of 3.6%.
Remarkably, this leap in accuracy is achieved with only 52 billion activated parameters, underscoring the efficiency of our model.

Model
LLama3.1 405B Inst.
LLama3.1 70B Inst.
Mixtral 8x22B Inst.
DeepSeekV2.5 Chat
Hunyuan-Large Inst.

MMLU
87.3
83.6
77.8
80.4
89.9

CMMLU
-
-
61.0
-
90.4

C-Eval
-
-
60.0
-
88.6

BBH
-
-
78.4
84.3
89.5

HellaSwag
-
-
86.0
90.3
88.5

ARC-C
96.9
94.8
90.0
-
94.6

GPQA_diamond
51.1
46.7
-
-
42.4

MATH
73.8
68.0
49.8
74.7
77.4

HumanEval
89.0
80.5
75.0
89.0
90.0

AlignBench
6.0
5.9
6.2
8.0
8.3

MT-Bench
9.1
8.8
8.1
9.0
9.4

IFEval strict-prompt
86.0
83.6
71.2
-
85.0

Arena-Hard
69.3
55.7
-
76.2
81.8

AlpacaEval-2.0
39.3
34.3
30.9
50.5
51.8

Citation
If you find our work helpful, feel free to give us a cite.
@misc{sun2024hunyuanlargeopensourcemoemodel,
      title={Hunyuan-Large: An Open-Source MoE Model with 52 Billion Activated Parameters by Tencent},
      author={Xingwu Sun and Yanfeng Chen and Yiqing Huang and Ruobing Xie and Jiaqi Zhu and Kai Zhang and Shuaipeng Li and Zhen Yang and Jonny Han and Xiaobo Shu and Jiahao Bu and Zhongzhi Chen and Xuemeng Huang and Fengzong Lian and Saiyong Yang and Jianfeng Yan and Yuyuan Zeng and Xiaoqin Ren and Chao Yu and Lulu Wu and Yue Mao and Tao Yang and Suncong Zheng and Kan Wu and Dian Jiao and Jinbao Xue and Xipeng Zhang and Decheng Wu and Kai Liu and Dengpeng Wu and Guanghui Xu and Shaohua Chen and Shuang Chen and Xiao Feng and Yigeng Hong and Junqiang Zheng and Chengcheng Xu and Zongwei Li and Xiong Kuang and Jianglu Hu and Yiqi Chen and Yuchi Deng and Guiyang Li and Ao Liu and Chenchen Zhang and Shihui Hu and Zilong Zhao and Zifan Wu and Yao Ding and Weichao Wang and Han Liu and Roberts Wang and Hao Fei and Peijie She and Ze Zhao and Xun Cao and Hai Wang and Fusheng Xiang and Mengyuan Huang and Zhiyuan Xiong and Bin Hu and Xuebin Hou and Lei Jiang and Jiajia Wu and Yaping Deng and Yi Shen and Qian Wang and Weijie Liu and Jie Liu and Meng Chen and Liang Dong and Weiwen Jia and Hu Chen and Feifei Liu and Rui Yuan and Huilin Xu and Zhenxiang Yan and Tengfei Cao and Zhichao Hu and Xinhua Feng and Dong Du and Tinghao She and Yangyu Tao and Feng Zhang and Jianchen Zhu and Chengzhong Xu and Xirui Li and Chong Zha and Wen Ouyang and Yinben Xia and Xiang Li and Zekun He and Rongpeng Chen and Jiawei Song and Ruibin Chen and Fan Jiang and Chongqing Zhao and Bo Wang and Hao Gong and Rong Gan and Winston Hu and Zhanhui Kang and Yong Yang and Yuhong Liu and Di Wang and Jie Jiang},
      year={2024},
      eprint={2411.02265},
      archivePrefix={arXiv},
      primaryClass={cs.CL},
      url={https://arxiv.org/abs/2411.02265},
}

just skeptical about why some benchmarks are missing from LLAMA, and qwen2.5 72b is omitted from leadboard...
hope distilled versions are released with DPO.
regards.


DD-1227 make dansBagValidatorClient part of validateDansBag configura…

Description of changes

dansBagValidatorClient is renamed to httpClient and it is now part of validateDansBag  block in config.yml file.
Code changes to get and set httpClient as  dansBagValidatorClient

How to test
Suggestion: Run the service with check option
===> INFO  [2023-01-26 11:41:01,751] [main] io.dropwizard.cli.CheckCommand: Configuration is OK
Related PRs
(Add links)

Notify
@DANS-KNAW/dataversedans

@aliassheikh: Looks good. One small comment about unnecessary @JsonProperty, but I´ll merge it as is and make that change myself in the interest of speed.


Fix / upd: properties declared on parent after inheritance takes place are now passed to children

I'm not sure if this is a bug or intended behavior, but currently: if an ActiveNode class A is inherited by B, and then an ActiveNode property is subsequently declared on A, B will not have that property (i.e. properties are only be transferred at inheritance).
This PR makes it so that every property declared on a parent is also present on the children (unless undef_property was explicitly called on a child).
In achieving this, I plucked ActiveSupport/core_ext/class/subclasses and added the subclasses method to Class.
This pull introduces/changes:

properties declared on parent classes after inheritance has taken place are properly inherited by subclasses
all classes now have subclasses method

Example
class Person
  include Neo4j::ActiveNode

  class Superhero < Person
  end

  property :name
end

hero = Person::Superhero.new
hero.name = 'John'
hero.name
#=> 'John'

Pings:
@cheerfulstoic
@subvertallchris

This PR has a minor conflict with #1419

Codecov Report

Merging #1428 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1428      +/-   ##
==========================================
+ Coverage   96.87%   96.88%   +<.01%
==========================================
  Files         205      205
  Lines       12457    12485      +28
==========================================
+ Hits        12068    12096      +28
  Misses        389      389

Impacted Files
Coverage Δ

lib/neo4j.rb
97.95% <100%> (+0.02%)
:arrow_up:

lib/neo4j/shared/property.rb
97.54% <100%> (+0.06%)
:arrow_up:

lib/neo4j/shared/enum.rb
98.46% <100%> (+0.04%)
:arrow_up:

spec/e2e/inheritance_spec.rb
100% <100%> (ø)
:arrow_up:

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 4e4f5af...6884bf5. Read the comment docs.

I suppose this update has the capacity to break things... If it is something you're interested in accepting, might be a good idea to release it along with #1419

Thanks so much for this!  I would say this falls into the bugfix camp, but I'll release it in 9.0.0 just to make sure.  I resolved the conflict and now waiting for Travis to finish before merging (though I might not get back to it this evening)

awesome

9.0.0 released

Whoo hoo


AWK processor

I found googles mtail when doing some research and it reminded me how similar it is to the Benthos AWK processor.
https://github.com/Jeffail/benthos/blob/master/docs/processors/awk_functions.md
https://github.com/google/mtail/blob/master/docs/Language.md

Interesting, they must have their own implementation of AWK in there somewhere.


Window resize fixes layout - bottom of the page has initially big white blank space

Well, the topic pretty much covers the issue. Initially my page has white blank space on the bottom of screen, but if I resize the window it fixes itself. I pretty much copy pasted the starter page, dunno why the starter page is working just fine tho...

https://github.com/almasaeed2010/AdminLTE/issues/1033
Excactly the same issue here, but I'm just using Angular 1.5. Any suggestions?

Thanks for sharing your solution. I am sure this will be beneficial to others.

I'm not using Angular, just plain AdminLTE - getting very same problem and solution above does not work at all.

Using Angular 1.5, the solution provided doesn't work all the times. In fact, the first time I load the webpage is working okai, but if i refresh it stops working.
Any idea?
Thanks!

..anyway, if you can point me to the JS methods od AdminLTE, I can give more tries.. but I'm struggling to find them.


Enable strict typing mode

A few typescript-related improvements we could make:

enable "strict" mode and fix all resulting typing issues
collect and strongly type all of the basic generator bits, like InstallTarget and Install, etc

I've fixed a few bugs now that basically came down to types not being respected, so I think this would be good for the codebase, and it would make refactoring a lot easier.

Another one: currently typescript is not inferring optional types on interfaces since we don't have strict null checks enabled, which is causing it to think there's dead code in resolve.ts, for instance.


can you create a test script to test the trained model on new hinglish text?

There is a generalized notebook that you may find useful for training with your own dataset https://github.com/cozek/trac2020_submission/blob/master/notebooks/Transformer%2BAttentionHead.ipynb
My code is a little too long which you may find complex.
Feel free to just take the AttentionModel class and use your own training code.

ok Thanks @cozek for the quick response


fix: pass product in CatalogGrid's onItemClick handler

Resolves #251
Impact: minor
Type: bugfix
Component
This PR simply passes product in CatalogGrid's onItemClick handler. This will allow the tracking decorator in starterkit access to the product.
Screenshots
N/A
Breaking changes
None
Testing

Catalog Grid examples
Click first example
Confirm alert appears with product's title

🚀 Preview deployed
Built with commit 4a28e7b8ac106dcae34ab5d597232dcbbeca71ef
https://deploy-preview-252--stoic-hodgkin-c0179e.netlify.com

:tada: This PR is included in version 0.30.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add token request extractor to TCC

Refactor TCC that the tokenRequest extract logic can be injected. This
allows to use alternativ implmentations of the extracting logic other
than the default transient-based logic. A use case for this is, the
FPC-TCC. Currently, FPC does not support transient data, so the
tokenRequest must be provided via function args.
Signed-off-by: Marcus Brandenburger<EMAIL_ADDRESS>

An example of an alternative extraction method can be found in the FPC-TCC sample.
https://github.com/mbrandenburger/fabric-secure-chaincode/blob/another-example/samples/chaincode/token-validation/main.go#L77


Don't send spam mail

You're sending out spam mail to (probably?) crawled mail addresses from GitHub profiles. This is not okay. I've reported you to MailChimp and on GitHub.

I only send 100 mail to most efficient profiles (top 100) how like GoodbyDPI to invite them to test my app and help to fix bugs. this is not a spam man :/

It is spam if it's advertising for which I didn't sign up :P


Add support for Proxy Autoconfig

This is tricky. Have to do DNS stuff, download and execute JS, etc. Some platforms may do this for us.

For Windows in a corportate environment the AutoConfig URL is stored in the registry under the following path
HKEY_USERS\{USER SID}\Software\Microsoft\Windows\CurrentVersion\Internet Settings\AutoConfigURL
User SID can be obtained calling
wmic useraccount where name='%username%' get sid
This takes some time on my machine so if theres a quicker method that would be more suitable.

Below is a link to a C library which parses PAC files. It may be possible to build and use it via ffi or use the code as reference for implementing a rust version.
https://github.com/pacparser/pacparser

Thanks for the link. It looks like there isn't all that much code there. I think it wouldn't be too much work to build the same sort of thing on top of https://github.com/dflemstr/v8-rs or https://github.com/servo/rust-mozjs/.

I guess it is just running a JS function at the end of the day so yeah either of those may be good.


Consolidate the libraries

The entire purpose of splitting the libraries was to create a seam for people to introduce their own implementations for these components.  However, the current implementation does require that the client download three separate libraries.
This change will merge the four existing libraries into one, yet leave the seam available for anyone who cares to use their own implementation.

Maybe consider making a metapackage like Manatee.Trello.All which only references the base packages needed to quickly get started? The other libraries will then be pulled into target projects automatically.

That's one possibility, but consolidation has the advantage of out-of-the-box functionality.  It would do all of the configuration internally allowing you to download and run with it.


[ALLUXIO-2441] Netty data server handlers unittests

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13023/
Test PASSed.

@gpang  PTAL. Thanks

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13032/
Test PASSed.

Merged build finished. Test PASSed.

FYI, I have a PR that introduces options for the waitFor method ... if my PR merges first, you should incorporate it into your PR

@peisun1115 the waitFor PR (https://github.com/Alluxio/alluxio/pull/4616) has been merged, could you please update the newly introduce waitFor method to use the WaitForOption? thanks

@gpang  PTAL
@jsimsa  I've merged with your change

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13051/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13052/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13075/
Test PASSed.

Merged build finished. Test PASSed.

@calvinjia I've addressed your comments. PTAL

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13078/
Test PASSed.


Add matchLabels to template options selector

Description of problem
The matchFields selector is not ideal for dealing with labels, so this adds a matchLabels selector.
Proposed solution
Implement matchLabels and matchExpressions as per this RFC
Example
  resources:
    - name: source-provider
      templateRef:
        kind: ClusterSourceTemplate
        options:
        - name: imgpkg-bundle-template
          selector:
            matchLabels:                               # <========= add this
              app.tanzu.vmware.com/source-type: imgpkg-bundle

Use metav1.LabelSelector


MULTIARCH-3677: Add pagination for listing service instances

Fixes: https://github.com/openshift/machine-api-provider-powervs/issues/14

/hold
Need to test on cluster

/hold cancel
I have tested the changes on cluster and works as expected and ready for review.


the timestamp is delayed by 9 hours.

Prerequisites

[x] Did you read FAQ section?
[ ] Did you test with the latest releases or commit?
not yet.
I'm using 20180918.
I'll try after download is completed.

Description
the timestamp of downloaded file is delayed by 9 hours.
I'm in Japan (UTC+9) and I think that's the reason.
Steps to Reproduce
using PixivUtil2 with --startaction=4

It is stored in utc.
for example, https://www.pixiv.net/ajax/illust/69925083
https://github.com/Nandaka/PixivUtil2/blob/master/PixivModelWhiteCube.py#L234
I'm not sure if I should convert to local pc time? maybe can use something like https://stackoverflow.com/a/10854983

oh ok.
no problem if it's the intended behavior.
thanks.

I think in the web ui, it will be adjusted to the time zone based on your location in the profile settings?
For example:
https://www.pixiv.net/member_illust.php?mode=medium&illust_id=70995384
Web shows October 3, 2018 6:09 AM

API shows 2018-10-02T22:09:47+00:00 (9h differences)

What is the date shown in your side (assuming your location is not set to tokyo/japan)?

the date shown in my side is
https://www.pixiv.net/ajax/illust/70850475
"createDate":"2018-09-24T09:36:51+00:00"
https://www.pixiv.net/member_illust.php?mode=medium&illust_id=70850475
September 24, 2018 18:36
downloaded file
20180924 09:36:51

i forgot this.
70850475.txt
Date          = 09/24/18 09:36

I assume, you also setting the location to tokyo/japan? if not, then meaning the timestamp in the web page is always use japan as the time zone offset.

Yes. the location is set to Japan. and the system locale is also Japan.
but the timestamp of downloaded file is in UTC+00:00 time zone.

Ok, I checked, the web ui will follow based on the location you set for the time adjustment, but I cannot get the configuration.
I think I can set it based on local pc timezone if you want?

yes please.


[BUG] operator set_progress error

Describe the problem
The source comments indicate that set_progress should accept a float & string...
https://github.com/voxel51/fiftyone/blob/a9d02f2d3898e87a57a3c57ce1925c451cb85259/fiftyone/operators/executor.py#L726-L727
...but later attempts to access progress.progress
self.log(f"Progress: {progress.progress} - {progress.label}")
Proposed fix:
self.log(f"Progress: {progress} - {label}")
Code to reproduce issue
class SetProgressError(foo.Operator):
    @property
    def config(self):
        return foo.OperatorConfig(
            name="set_progress_error",
            label="Set Progress Error"
        )

    def execute(self, ctx):
        ctx.set_progress(progress=0.5,label="half way")

System information

OS Platform and Distribution (e.g., Linux Ubuntu 22.04): Ubuntu 20.04.6 LTS
Python version (python --version): Python 3.10.13
FiftyOne version (fiftyone --version): v0.23.7
FiftyOne installed from (pip or source): pip

Other info/logs
  File "/usr/local/lib/python3.10/dist-packages/fiftyone/operators/executor.py", line 728, in set_progress
    self.log(f"Progress: {progress.progress} - {progress.label}")
AttributeError: 'float' object has no attribute 'progress'

Willingness to contribute
The FiftyOne Community encourages bug fix contributions. Would you or another
member of your organization be willing to contribute a fix for this bug to the
FiftyOne codebase?

[ ] Yes. I can contribute a fix for this bug independently
[ ] Yes. I would be willing to contribute a fix for this bug with guidance
from the FiftyOne community
[x] No. I cannot contribute a bug fix at this time

@yrrah thanks for catching this 😅
Fixed in https://github.com/voxel51/fiftyone/pull/4417 and will be released in fiftyone==0.24.0 (coming next week).


Importing inline functions in MinGW

What are the steps to reproduce this issue?

Using MINGW64 to build the library

What does happen?
I managed to build the library. However, when I want to run the example, I get the following error: "'dllimport' attribute ignored on inline function"
What were you expecting to happen?
I guess there is a problem in mingw to link inline functions
Any logs, error output, etc?
(If it’s long, please paste to https://ghostbin.com/ and insert the link here.)
Any other comments?
…
What versions of fesapi are you using?
<IP_ADDRESS>

Hi Behzad,
We have no experience at all with Mingw64 and FESAPI. So, we have no direct answer.
If it is possible for you to use VS (community edition for instance), it would help but, I guess you cannot.
At a first glance, your "error" message" looks like a "warning", not like an error. Ignoring this attribute may be totally correct.
How do you know that the example fails? Is there another error message? Cannot you see anything? Is there a seg fault?

Hi Philippe,
I removed DLL_IMPORT_OR_EXPORT at the beginning of all functions and made a few changes in the CMake files, then rebuilt the library. Now, everything works and the examples run as they should. I don't know why DLL_IMPORT_OR_EXPORT causes the issue in MinGW.


Fixes for broken Agricraft Botania seed mutations

A few of the botania mutations were missing the "vanilla" tag on the parent plants, and so weren't registered.

Sweet, thanks

forgot to add these, I'll do it now


Add private browsing button (widget)

Sorry about that. Made another pull request by accident.

Sorry about that. Made another pull request by accident.


MIP102c2-SP18 Amendment PR

Originating post: https://forum.makerdao.com/t/mip102c-sp18-mip-amendment-subproposals/22027
Do not merge unless RP passes

Vote did not pass


Add custom ledger app name support

Currently LaunchpadLedger forces users to use Cosmos ledger app. However there are cosmos-sdk based chains uses their own ledger app as they have different coin type (eg, Band, Terra, Kava, Desmos)
This PR solves the problem by allowing users to input an option ledgerAppName to pass the ledger app name check

Thanks a lot for your contribution and upstreaming the changes you need instead of forking the codebase.
This work was merged as part of #908.


chore(remix-dev): replace xdm with @mdx-js/mdx v2

This PR addresses issue #1802 - xdm is now deprecated, and all functionality has been ported to @mdx-js/mdx.
Closes: #1802

[x] Docs
[x] Tests

Testing Strategy:
All existing tests pass with yarn test. Additionally, I created a playground and verified locally that mdx files still build as expected.

Thanks,  deasync is very problematic on my side and even more with vercel and node 18.
Would be very nice to include in the next release 🙏

@jacobwgillespie could you please rebase this on latest dev and fix conflicts? I'll try to get eyes on this PR.

@machour yep, rebased, thank you!

Hi @jacobwgillespie!
Are you still interested in getting this one merged?
If so, please rebase onto latest dev & resolve conflicts

@MichaelDeBoey Thanks for the ping, I have rebased onto the latest dev 👍

I'm not sure why the tests are failing, it's upset about packages/remix-dev/modules.ts which I don't think is affected by this PR.

Oh I see it, it's the new *.mdx types, let me take a look


invalid BooleanBuilder orAllOf

using BooleanBuilder orAllOf method like
e1.orAllof(e2, e3)
expected result is e1 or (e2 and e3) but actual result is e1 or e2 and e3
and this where cluase result occur diffrent result in mySql.
This is a fatal flaw in the BooleanBuilder. anyone know about this issue?

Are you sure?
and should have higher precedence than or, so parentheses aren't necessary.

@Shredder121
sorry my bad :( close issue


Python 3 dict inheritance

For whatever reason, the __dict__ property does not get inherited, so it needs to be re-implemented in a few classes.

It might be better to fix this by defining __slots__ = () for these classes (instead of defining the __dict__ property.)

I'm not sure that would do anything as I think that's already true:
In [1]: from collections import namedtuple
In [2]: Foo = namedtuple('Foo', 'foo bar')
In [3]: class Bar(Foo): __eq__ = lambda self, other: True

In [4]: f = Foo(1, 2)
In [5]: f.__slots__
Out[5]: ()

In [6]: b = Bar(1, 2)
In [7]: b.__slots__
Out[7]: ()

I'm not sure that would do anything as I think that's already true:

Without getting in to the subtleties of it all (I'd have to spend a while researching them!):
>>> class Wibble(Foo): pass
>>> Wibble(1, 2).__dict__
{}

>>> class Wobble(Foo): __slots__ = ()
>>> Wobble(1, 2).__dict__
OrderedDict([('foo', 1), ('bar', 2)])

Oh, of course, I tested it by setting __slots__ after class creation and not during.

OK, switched to __slots__, but only where necessary. Does it make sense to add it on all the namedtuple derivatives?

Does it make sense to add it on all the namedtuple derivatives?

Not sure... probably yes. It appeals from a consistency point of view, and the derivatives were all intended to be immutable.
Out of interest, how did you spot the need for this PR? (I'm trying to recreate the relevant test failures so I can better understand the possible impact.)

Out of interest, how did you spot the need for this PR? (I'm trying to recreate the relevant test failures so I can better understand the possible impact.)

I think I've tracked this down to things like _CoordMetaData.__eq__ failing because its calls to self._asdict() and other._asdict() return {} on Python 3 without this PR.

Does it make sense to add it on all the namedtuple derivatives?

Not sure... probably yes

Enough vagueness! If I have to choose an option, then :+1: for slots. :wink:

I added it to everything except ArrayStructure, where it cannot be added because the __new__ sets the self.size attribute.

Tests currently broken by ContinuumIO/anaconda-issues#445.

I added it to everything except ArrayStructure, where it cannot be added because the new sets the self.size attribute.

We could define size as a property instead?

Yep, seems to work alright.

Looks good. :+1:


Removed timed exceptions that made code unclear.

As requested, timed exceptions for firing before and after timespan were removed.
Now PetriNet.fire returns an error code specified in enum PetriNetFireOutcome.

This change is

Review status: 0 of 7 files reviewed at latest revision, 3 unresolved discussions.

src/org/unc/lac/javapetriconcurrencymonitor/monitor/PetriMonitor.java, line 464 at r1 (raw file):

fired
tried to fire

test/org/lac/javapetriconcurrencymonitor/test/cases/PetriNetTestSuite.java, line 669 at r1 (raw file):

	 * <li> And p2 has no tokens </li>
	 * <li> When I fire t2 </li>
	 * <li> Then the fire returns false </li>

The fire return PetriNetOutcome.NOT_ENABLED

test/org/lac/javapetriconcurrencymonitor/test/cases/PetriNetTestSuite.java, line 708 at r1 (raw file):

	 * <li> And p2 has 1 token </li>
	 * <li> When I fire t2 </li>
	 * <li> Then the fire returns true </li>

The fire return PetriNetOutcome.SUCCESS

Comments from Reviewable

Review status: 0 of 7 files reviewed at latest revision, 3 unresolved discussions.

src/org/unc/lac/javapetriconcurrencymonitor/monitor/PetriMonitor.java, line 464 at r1 (raw file):
Previously, mauriludemann (Mauri) wrote…

fired
tried to fire

I tried to write "fires". Tries to fire is ok

test/org/lac/javapetriconcurrencymonitor/test/cases/PetriNetTestSuite.java, line 669 at r1 (raw file):
Previously, mauriludemann (Mauri) wrote…
The fire return PetriNetOutcome.NOT_ENABLED

Done.

test/org/lac/javapetriconcurrencymonitor/test/cases/PetriNetTestSuite.java, line 708 at r1 (raw file):
Previously, mauriludemann (Mauri) wrote…
The fire return PetriNetOutcome.SUCCESS

Done.

Comments from Reviewable

Review status: 0 of 9 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 5 of 7 files at r1, 4 of 4 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


I met this mistake.

Another exception was thrown: HttpException: Connection closed before full header was received

same to me

ReProduce: ______?

me too


persistently substitute invalid job attributes with default values - not only in add_job

Commit 6d14c1b now allows the add_job to be successfully submitted for jobs with invalid (non-utf8) job-name or requesting-user-name, by susbstituting the invalid strings with default ones ("Untitled" for job-name, "anonymous" for requesting-user-name).
However the send_document job is still aborted during validate_user() - we get:
Returning HTTP Unauthorized for Send-Document
It looks like send_document() - >validate_user() -> cupsdIsAuthorized() is picking up the original "bad" non-utf8 requesting-user-name, although this was replaced with "anonymous" in add_job with upstream fix 6d14c1b. So I assume subsequent jobs after add_job still get the original requesting-user-name, correct?
If validate_user is commented out in send_document(), or if cupsdIsAuthorized() does the same check/replacement of requesting-user-name as commit 6d14c1b does in add_job, the problem is solved. However the first is a hack and the second code duplication.
What is the preferred way to fix the requesting user name for all jobs? Is code duplication (substituting requesting-user-name with "anonymous" in cupsdIsAuthorized() enough? Or should the attribute checks be in a separate function that will somehow be used by jobs that need it?

[master 092db1e29] Additional changes for the scheduler to substitute default values for invalid job attributes when running in "relaxed conformance" mode (Issue #5229)

Thank you @michaelrsweet it works for me as well with this commit.


Update cli

Update the cli to use --port instead --label=@port that was too complex.
Discover the path of executables and stop requiring explicit path.

Codecov Report

Merging #143 into master will not impact coverage.

@@           Coverage Diff           @@
##           master     #143   +/-   ##
=======================================
  Coverage   57.27%   57.27%
=======================================
  Files          30       30
  Lines        3773     3773
=======================================
  Hits         2161     2161
  Misses       1467     1467
  Partials      145      145

Impacted Files
Coverage Δ

trireme.go
50.63% <ø> (ø)
:white_check_mark:

monitor/rpcmonitor/rpc.go
82.5% <ø> (ø)
:white_check_mark:

supervisor/iptablesctrl/acls.go
86.52% <ø> (ø)
:white_check_mark:

processmon/processmon.go
62.91% <42.85%> (ø)
:white_check_mark:

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 a8c4a52...b668718. Read the comment docs.


Add More Gentoo Deps

I created the dev-python/pydocstyle ebuild personally, and it now exists in the ROS Overlay. Also, the ros-pkg-resources module is just a part of setuptools on Gentoo.

The only odd change here is the opensplice addition. This is needed for superflore to resolve the key correctly.
If preferred, I could hard-code the resolution, but I would really rather not.

If opensplice is available upstream in an appropriate version we might as well leverage that.

The only odd change here is the opensplice addition. This is needed for superflore to resolve the key correctly.

This is only odd because I messed up and forgot to upstream the addition of an opensplice key from the should-be-defunct ros2 rosdep tree https://github.com/ros2/rosdistro/commit/1871b9e565b9c9e6d3a846a671ee6c64f712b623
I think the reason I hesitated is that the package is in the OSRF and ROS 2 repositories rather than upstream. Should we add the ubuntu key as well?

Should we add the ubuntu key as well?

I don't think it's unreasonable to add it. While it's not safe to assume that rosdep being installed => they have set up the OSRF / ROS 2 repositories, I don't think we're necessarily responsible for maintaining the other installation methods. That's the approach I've been going with for the Gentoo installs, at least.

In any case I think we can resolve that issue separate from getting Gentoo operational. I can PR that separately for discussion.

I think the reason I hesitated is that the package is in the OSRF and ROS 2 repositories rather than upstream. Should we add the ubuntu key as well?

If we expect people to be able to create / release packages that depend on it if should either:

be available upstream
be imported in the ROS repo and distributed by us

Without this it cannot be added to the rosdep database

If opensplice is available upstream in an appropriate version we might as well leverage that.

@allenh1 the gentoo packages website is still down and I can't find this package. Can you provide a link and a version number?

Can you provide a link and a version number?

That's a good point. Ardent targets opensplice 6.7.

the gentoo packages website is still down and I can't find this package. Can you provide a link and a version number?

@mikaelarguedas Sorry for the delay in response, had to take an exam.
There is no upstream Opensplice for Gentoo, only the one I wrote in the overlay. It's for 6.7 as well.


[Question]: images and tables

Hi
My questions may sound very specific as I am working on a very interesting use case on the top of ragflow.
Is there a way to extract the images from the pdf so if I ask in the chat to see the images I get the images only (now it says there are images in the pdf)? Even as I manually added images (uploaded them) to files and linked them to the knowledge base, the chat still didn't recognize that. (It will also makes more sense to be able to automate bulk action to link uploaded files to knowledge base rather than one by one.
And it will be more powerful to have the tables in the document also extracted as separate chunks.
Is my impression right that when you increase response length in the setting that impact the response quality? And how to solve that if you are pulling the answer using api calls?

Getting the images alone in chat is very interesting.
For chat API, you can get image id in response by which you can get the image by 'GET' /v1/document/get/

@KevinHuSh I want to work on a POC like you find in the attached video (if this work I believe you will like the applied case solution we'll build on the top of ragflow). I will need help with the communication with ragflow from there (Whether deployed on my server or on yours). Where can you guys help with this (if there is a reasonable fee, I am fine with that as well)?
What I am doing is:

I upload a pdf in the form (I mention another alternative below)
I add many other fields in the form. And in each field settings I add a prompt that posts to the api a request to get answered based on the data in pdf data, for example: ' get the address of the place ', then I add another field with prompt:  ' Find the name of the owner ' and so on.
I add a button generate and when i hit that it sends all fields prompts to open ai and get back data to fields.
save that data or print it out as a report,

I am assuming there will be a need to upload the pdfs as knowledge bases before posting prompts. I am fine with that as well but then:
Can I create knowledge bases to send prompts to via apis? And if yes, will I get back knowledge_base name as it is created so I send that name in the prompts to populate the fields? If not, how to prepare the knowledge base in the rag directly but then call it from the form builder?
https://github.com/infiniflow/ragflow/assets/108686600/72dfad25-a7f6-4a88-aaf7-3a9acfce202f

Yeah I'm also working on something similar. I'm planning to extract an image from a pdf file in the knowledge base, and then hopefully feed that image to the llm for analysis (instead of using OCR cuz it's somewhat lossy for images, screenshots, etc.). Is that doable with ragflow? Especially the extracting image part.

The images have already been recognized and extracted. The chunk list of a doc show them all.


.png resolution for export

Feature name
png resolution
Is your feature request related to a problem? Please describe.
A description of what the problem is.
When I export to a .png I am not able to read any of the resources as they are all blurry.
Describe the feature you'd like to see implemented
A description of what you would like to see.
Similar to draw.io be able to choose the dpi etc. to export as.
Describe the value this feature will add to AWS Perspective
Tell us how this feature might improve AWS Perspective.
Wouldn't have to export to draw.io and then export to .png
Describe alternatives you've considered
A description of any alternative solutions or features you've considered.
Export to draw.io and then export from there as a .png etc. where you can choose the quality.
Additional context
Add any context or screenshots about the feature request here.

We've evaluated this request and added it to our backlog but it is not prioritised in the immediate roadmap for the solution.


fix: add secrets check based on scm source

I changed the secrets config to use secrets_group value to correctly assign secrets based on the atlantis_gitlab_user, atlantis_github_user or atlantis_bitbucket_user values that are passed to the module.
Fixes https://github.com/terraform-aws-modules/terraform-aws-atlantis/issues/67

@antonbabenko can you review this pr ?
It is a blocker for us and we need this to be merged.
Thanks

hi @bmihaescu - could you please merge master/rebase to update your branch and change your title to something like fix: add secrets check based on scm source - thank you!

Hi @bmihaescu , @bryantbiggs and @antonbabenko ,
I hope all is well.
Would it be possible to merge/rebase master again and get this merged?
Thank you all for the contribution and maintenance, it is greatly appreciated!

@Szasza I didn't have time to look into this issue myself, but earlier today there was a new release v2.24.0 of this module which touched the same variables as this one (#151).
Regarding this PR, I am not sure how adding another set of conditions is making it good. Can it be simplified instead of adding more cases after rebase?
@bryantbiggs Please let us know what do you think? Hopefully, you use this more than me lately :)

I'll try to take a look this week - the conditional checks have been getting a bit unwieldy!

@antonbabenko @bryantbiggs awesome, thanks to both of you, it is greatly appreciated!


invalid notification number with skype

I have the skype icon showing 2 notifications; upon reload of the plugin / Franz, still 2 notifications. Not a single notification in the official skype client.

Are you using the latest version skype web?
I think you are probably refereeing a known issue on the previous version of skype web. Dupe #513

It's a different issue; I have the latest version. I forgot in my original post: I'm on Mac.
In my case, Franz is always showing me 2 notifications; the '2' doesn't go away while there are unread messages, news, etc as confirmed by the official client.


Only a ReactOwner can have refs.

Error: addComponentAsRefTo(...): Only a ReactOwner can have refs. You might be adding a ref to a component that was not created inside a component's `render` method, or you have multiple copies of React loaded

A component has following code:
<NavItem onClick={...}>
  <i className="..."/>
  <NotificationBadge
    count={this.props.count}
    effect={Effect.SCALE}
  />
</NavItem>

All works fine when I remove this NotificationBadge

I can't reproduce it on example.
http://georgeosddev.github.io/react-notification-badge/example/
Your log say,
you have multiple copies of React loaded
Please check your app's react version.


Add Armenian Language Localization

Hi @aramvr
Thanks for this! I will just have to look up how to create an alphabetical keyboard for this before merging, since a keyboard locale requires those parts as well to function with KeyboardKit Pro.
Until then, you can use your localized strings without having the keyboard locale, and create a native locale with the locale identifier. That should at least translate the keys correctly.

No problem, thanks for that!

Hi @aramvr
I just wanted to let you know that I'm still intending to add an Armenian language, but there are a bunch of things that I'd need to do with the layout provider model to make this happen.
I'll implement it as soon as everything is in place.

Hi Daniel,
Thanks for the update and let me know if I can assist in any way as an
armenian language speaker.
On Fri, Oct 28, 2022 at 10:46 Daniel Saidi @.***> wrote:

Hi @aramvr https://github.com/aramvr
I just wanted to let you know that I'm still intending to add an Armenian
language, but there are a bunch of things that I'd need to do with the
layout provider model to make this happen.
I'll implement it as soon as everything is in place.
—
Reply to this email directly, view it on GitHub
https://github.com/KeyboardKit/KeyboardKit/pull/427#issuecomment-1294535600,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEFKO2UAMC6MIFR7XN7GDIDWFNZE5ANCNFSM5RXEIDLQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Aram Vardanyan
Front End Software Engineer
@.***
+374 44595955 <+37444595955>

Hi @aramvr
I have added support for Armenian now, which will be available in the upcoming 6.8 version.
Unfortunately, it's not possible to fully test it for others than me until it's live, but I'd be very happy if you could just take a look at the localized strings and keyboard locale information in the master branch.
For instance, is hy sufficient as locale identifier, or must it be hy_AM?  I also noticed differences when comparing your localized strings in this PR with the ones I saw in the system keyboard.
I'd love your feedback before I merge 👍

Hi @danielsaidi
I've looked into your changes and added some feedback.
Overall it looks great! thanks for doing that!


Mat Tool Bar Issue

Reproduction
Use StackBlitz to reproduce your issue: https://stackblitz.com/edit/angular-tjjqpn
Steps to reproduce:

Go to the stackblitz URL above, check if gets a scroll to the page, if no reduce the height of the window
After it runs in the browser, scroll to bottom of the screen
Click on the pick one button, it will open a dialog
Observe the top header it disappears

Expected Behavior
Ideally the top header section should not disappear or change its position when it is fixed
Actual Behavior
the top header disappear(even if Its tool bar from material components it disappears)
Environment

Angular:
CDK/Material: Material 8.1.3
Browser(s): Chrome
Operating System (e.g. Windows):

Hello There,
I found this weird thing, can you guys please help
Thanks!

I don't think its material issue,just by adding top:0px to header style fixes your issue.
https://stackblitz.com/edit/angular-tjjqpn-ei5n5c?file=index.html

yes this is what had to do to fix this
but I wanted to know why I have to put this
as I checked further I see that when modal opens it changes the position of the fixed header to something say -355px
this should not be happening by default.. no..?

(revisiting older issues as part of a triaging fix-it)
Please keep GitHub issues for bug reports / feature requests. Better avenues for troubleshooting / questions are stack overflow, gitter, mailing list, etc.
(this is a canned response)


Java indexer: implement GRPC CompilationAnalyzer server [phab:D162]

Created by schroederc at 2015-04-02 22:21:16:
Depends on #372

fromberger accepted this revision at 2015-04-09 13:30:43

schroederc closed via commit 0cd44b5906dd6855b1da1ce2bb3a9ec0c82463b2 at 2015-04-13 20:44:19


Update how-to-create-a-data-driven-unit-test.md

TestMethodAttribute is preferred over DataTestMethodAttribute.

#sign-off Thanks, @NN--- !


JSON Values Not Rendered When RedisJSON Module Installed

Describe the bug
redis-gui does not handle JSON values when their type is reported by the cluster to be ReJSON-RL (raw). The type is accurately displayed in the UI, hovering over it does properly switch the mouse cursor to the ？ icon, and clicking it does still display the Redis Data Types documentation, but the UI never actually loads the selected key's value - it displays the Loading ↻ animation ad infinitum.
This behavior only seems to happen when the connected cluster has the RedisJSON module installed / loaded.
To Reproduce
Steps to reproduce the behavior:

Enable the RedisJSON module
Set some JSON-valued key: redis-cli -h {REDIS HOST} -p 6379 JSON.SET some-json-key '$' '{"some":{"json":"value"}}'
Attempt to load the some-json-key value in redis-gui

Expected behavior
The UI should load the values as JSON values.

Having the same issue.

RedisJSON module support added in v2.3.0

@ekvedaras You're incredible. Thank you!


Path parameter not present in the path : No error

I changed the bug goal because I thought the parameters was not parsed but I wasn't watching the correct object.
I tried to parse this small swagger spec because the "bar" parameter is specified as a path parameter but does not appear in the path :
string swagger = @"
openapi: 3.0.1
info:
  title: Sample API
  version: v1
servers:
  - url: 'api.example.com'
paths:
  '/test/{foo}':
    get:
      parameters:
        - name: foo
          in: path
          required: true
          schema:
            type: string
        - name: bar
          in: path
          required: true
          schema:
            type: string
      responses:
        '200':
          description: Ok
          content:
            application/json:
              schema:
                type: string

            ";
            var openApiDocument = new OpenApiStringReader();
            var temp = openApiDocument.Read(swagger, out var diagnostic);

But when I check the "diagnostic" variable I see there is no error.
Microsoft.OpenApi : 1.2.3
Microsoft.OpenApi.Readers : 1.2.3
Thanks :-)

Thank you for the PR.  I will get this change incorporated into the 1.2.4 release.


android:exported needs to be explicitly specified for element <receiver#org.infobip.mobile.messaging.geo.BootReceiver>. Apps targeting Android 12 and higher are required to specify an explicit value for android:exported when the corresponding component has an intent filter defined.

Hi,
I am facing an issue after installing the package in my Ionic project. When I try to build my Android app, I receive the following error message:
'android:exported needs to be explicitly specified for element <receiver#org.infobip.mobile.messaging.geo.BootReceiver>. Apps targeting Android 12 and higher are required to specify an explicit value for android:exported when the corresponding component has an intent filter defined.'
This issue is preventing me from building and deploying my app. I have checked my AndroidManifest.xml file, but I am unable to resolve this issue.
Can you please help me resolve this issue and provide any guidance or fixes that may be necessary? Thank you for your help.

Hi @TK2011-TH are you using this version https://github.com/infobip/mobile-messaging-cordova-plugin/releases/tag/3.0.0-rc4 ?
We have android 12 supported only for that, please check migration guide https://github.com/infobip/mobile-messaging-cordova-plugin/wiki/Migration-to-version-3.x.x

Hi riskpp, still get the same errors about 'Manifest merger failed with multiple errors, see logs'. Seems like some how the package still merge unsupported receivers.

Hi riskpp,
Thank you so much, it works now.
Sorry, but I have encountered another issue, and I am not sure if I should open a new issue for it.
Before this, my app was already working with Firebase by using this package: https://github.com/chemerisuk/cordova-plugin-firebase-messaging. We already have our backend to send push notifications to the app.
After solving the Android 12 issue, even Im able to build the app successfully, but it kept crashing constantly, not sure is it possible to make it works together. If not, and I need to use only the Infobip package, will I still be able to send messages from my own backend (which requires the Firebase device token), or do I need to move to Infobip completely?

Hi @TK2011-TH please close this issue and create new one, also provide crashlog, in general it's possible to use infobip native sdk together with other push provider, we have this articles you may check
https://github.com/infobip/mobile-messaging-sdk-android/wiki/How-can-I-achieve-handling-by-Infobip's-Firebase-and-my-Firebase-messaging-service%3F
https://github.com/infobip/mobile-messaging-sdk-android/wiki/Android-Manifest-components
But for cordova might be tricky, you may need to perform some native coding for it.
Regarding sending messages from your backend, you may do it but they won't be handled properly by infobip sdk.


Bump fastapi dependencies

Purpose
Bumps FastAPI and FastAPI's dependencies to allow installing fideslib.
There are 2 failing tests, test_generate_field and test_field_collect_matching after updating the dependencies. The dependency causing an issue is Pydantic. Both tests are failing for the same reason. They are comparing two objects with nested objects for equality. In older versions of Pydantic, the nested objects were created with pointers so the equality test would pass because the objects were at the same memory address. Now Pydantic creates the objects by creating new instances of the objects. Because of the, even though the values contained in the objects are the same, the tests fail because the test object id's are no longer the same. Any suggestions on getting around this?
Changes

FastAPI bumped and dependencies installed with fastapi[all] removed from requirements.txt

Checklist

[ ] Update CHANGELOG.md file

[ ] Merge in main so the most recent CHANGELOG.md file is being appended to
[ ] Add description within the Unreleased section in an appropriate category. Add a new category from the list at the top of the file if the needed one isn't already there.
[ ] Add a link to this PR at the end of the description with the PR number as the text. example: #1

[ ] Applicable documentation updated (guides, quickstart, postman collections, tutorial, fidesdemo, database diagram.
If docs updated (select one):

[ ] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[ ] documentation issue created (tag docs-team to complete issue separately)

[ ] Good unit test/integration test coverage
[ ] This PR contains a DB migration. If checked, the reviewer should confirm with the author that the down_revision correctly references the previous migration before merging
[ ] The Run Unsafe PR Checks label has been applied, and checks have passed, if this PR touches any external services

Ticket
Fixes #

even though the values contained in the objects are the same, the tests fail because the test object id's are no longer the same. Any suggestions on getting around this?

I believe we'll have to implement a "deep equality" check, like:
class Something:
    def __eq__(self, other):
        return self.__dict__ == other.__dict__

The version of black that was pinned turned out to be incompatible with the newer version of Click so I'm bumping that also.

Thanks @PSalant726, I tried this initially and it didn't work, but I only did it at the last level and not the nested classes also. I'm thinking now maybe it will work if I take it all the way down the chain.


Onboarding Journey Documentation

Story
As a partner hospital or lab, in order to better understand the onboarding process, I need a visual representation of how the onboarding process works and where I'm at within it.
Pre-conditions

Post MVP partner onboarding will rely on the same mechanisms as used by initial partners (i.e. newly discovered partner requirements need not be considered in scope of this story)

Acceptance Criteria

[ ] We have two separate workflow diagrams that outline the user journey, one for HCOs and one for PHLs, for onboarding to RS ETOR.
[ ] We've created a process for capturing onboarding activities within the backlog (which aligns to the visual representation in the AC above).

Tasks

[ ] share out miro board with team
[ ] collab session
[ ] other tasks?

Decisions

Optional: Any decisions we've made while working on this story

Notes

Optional: Any reference material or thoughts we may need for later reference

I will pick this back up this sprint


unban request

Did you follow the steps outline in the README?

[x] Yes I did
[ ] No, I found them confusing

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

My discord handle: mirakoya
I was unable to submit an appeal through form so will try to do it here
Why do you think you were you banned: i'm not sure as i didn't break any rules
Why do you feel you should be unbanned: i have 20+ score of my gitcoin passport and would like to get discord role
What will you do to avoid being banned in the future: Do whatever it takes to avoid getting banned and do whatever it takes to make sure your bot doesn't mistake me for a bot.
Thank tou in advance and have a good day Mirako

You might as well close the request, because ypu wont get any support here.
This is not affiliated with zkSync in any shape or form


Add the contracts generating tool as a package

This will allow us to reuse it across projects.
Also added abi generation function so this can be used when interacting
with the contracts.
Signed-off-by: Krasi Georgiev<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build 548649983

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 38.71%

Totals

Change from base Build 548648251:
0.0%

Covered Lines:
1524

Relevant Lines:
3937

💛  - Coveralls

opened by mistake


PHP closing tag no longer syntax colored

VSCode Version: Code - Insiders 1.13.0-insider (afe47dc84467b487ce0079ff72f55e7ef4f7ebca, 2017-05-30T10:34:18.253Z)
OS Version: Darwin x64 16.6.0
Extensions:

Extension
Author
Version

ng-template
Angular
0.1.3

code-settings-sync
Shan
2.8.1

sort-lines
Tyriar
1.3.0

Bookmarks
alefragnani
0.15.1

project-manager
alefragnani
0.18.0

vscode-eslint
dbaeumer
1.2.11

vscode-html-css
ecmel
0.1.5

tslint
eg2
0.15.0

Angular2
johnpapa
2.3.2

vscode-icon-theme
jtlowe
1.5.0

karyishlight
smlombardi
1.0.0

theme-karyfoundation-themes
karyfoundation
14.0.0

HTMLHint
mkaufman
0.3.3

vscode-stylefmt
mrmlnc
2.4.1

angular2-inline
natewallace
0.0.17

typescript-hero
rbbit
0.12.0

stylelint
shinnn
0.24.0

darcula-extended
smlombardi
3.3.2

slime
smlombardi
2.6.2

theme-tesla
smlombardi
6.0.0

change-case
wmaurer
1.0.0

Steps to Reproduce:

Take a typical one-line php code block, e.g. <?php echo "foo"; ?>
The closing ? is not colored.  Inspecting it with the theme tool shows a discrepancy between the scope at the top and the cascade at the bottom.  See image:

I guess it shouldn't have source.php, is that the discrepancy you're talking about? It's colored correctly in the out of the box themes though, so you may want to open an issue on the theme you're using.


Increase Slack Id Lengths

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


Update Xcode 8 recommendation project settings

Coverage remained the same at 73.272% when pulling d012c5c384b9ad9eb13bf90365d686b674c55cd7 on cleanup into 5a758c4906e76c2ba38e3a494ebfa7b0c08ac27a on master.


Epic: Aux file store v2

Motivation
To store aux file efficiently, we use one key for each of the aux file. To workaround the fixed-size key constraint, we hash the file name into the key. As the chance of hash collision is low, it is likely that we can get one aux file stored in one key.
DoD
Implementation ideas
Tasks

PoC PR https://github.com/neondatabase/neon/pull/7375
[x] missing key error improve https://github.com/neondatabase/neon/pull/7393
[x] key encoding https://github.com/neondatabase/neon/pull/7401 https://github.com/neondatabase/neon/pull/7502
[x] feature flag https://github.com/neondatabase/neon/pull/7424 https://github.com/neondatabase/neon/pull/7505
[x] read path, using vectored get to do range scan - https://github.com/neondatabase/neon/pull/7503 https://github.com/neondatabase/neon/pull/7468
[ ] write path + basebackup read path https://github.com/neondatabase/neon/pull/7491 https://github.com/neondatabase/neon/pull/7517
[x] grab user data and see how the new approach can improve the bloat situation

^--- likely done in the week of Apr 22 - Apr 26

[ ] merge PRs from previous weeks (1 left)
[ ] separate scan interface to avoid maintaining untouched keyspaces
[ ] decide migration method and write migration code
[ ] sparse keyspace support when creating image layers https://github.com/neondatabase/neon/pull/7567
[ ] handle deletion https://github.com/neondatabase/neon/pull/7567

^--- likely done in the week of Apr 29 - May 3

[ ] testing and benchmark
[ ] guide on enable
[ ] ensure scan does not go pass image layers
[ ] measure memory consumption of scanning and warn if it gets too large
[ ] metrics https://github.com/neondatabase/neon/issues/7443
[ ] https://github.com/neondatabase/neon/issues/7563

^--- likely done in the week of May 6 - May 10

[ ] remove the aux file v2 flag and migrate everyone to v2
[ ] prod?

Other related tasks and Epics
The parent epic: https://github.com/neondatabase/neon/issues/7290. We will discuss further tasks like storing pg_stats and storing logical size in the new metadata key space in that epic issue.

This week: after https://github.com/neondatabase/neon/pull/7517 is merged, testing & benchmarking on staging.
Improve on known perf bottlenecks:

memory consumption for scans is currently not limited

fundamental limitation with using vectored scan

All major works are done and closing this epic issue for now. I have created issues for all follow-up tasks. Might need one extra day at some point to implement the migration path when we decide how to roll this out to all users.


Make the tracer thread-safe

Problem
This gem raises DiverDown::Trace::CallStack::StackEmptyError while investigating a Rails application.
I assumed the cause is thread-unsafe code, and I made a small code to reproduce it.
This code raises the StackEmptyError (it is not 100% occurred as with most thread-unsafe problems).
require 'diver_down-trace'

class M
  def f
    N.new.g
  end
end

class N
  def g
  end
end

defn = DiverDown::Trace::Tracer.new(module_set: { modules: [M] }).trace do
  10.times.map do
    Thread.new { 10_000.times { M.new.f } }
  end.each(&:join)
end

pp defn.to_h

$ ruby -Ilib/ tmp/test.rb
#<Thread:0x000000011ff92f48 tmp/test.rb:16 run> terminated with exception (report_on_exception is true):
/Users/kuwabara.masataka/ghq/github.com/alpaca-tc/diver_down/lib/diver_down/trace/call_stack.rb:52:in 'DiverDown::Trace::CallStack#pop': DiverDown::Trace::CallStack::StackEmptyError (DiverDown::Trace::CallStack::StackEmptyError)
        from /Users/kuwabara.masataka/ghq/github.com/alpaca-tc/diver_down/lib/diver_down/trace/session.rb:52:in 'block in DiverDown::Trace::Session#build_trace_point'
        from tmp/test.rb:16:in 'block (3 levels) in <main>'
/Users/kuwabara.masataka/ghq/github.com/alpaca-tc/diver_down/lib/diver_down/trace/call_stack.rb:52:in 'DiverDown::Trace::CallStack#pop': DiverDown::Trace::CallStack::StackEmptyError (DiverDown::Trace::CallStack::StackEmptyError)
        from /Users/kuwabara.masataka/ghq/github.com/alpaca-tc/diver_down/lib/diver_down/trace/session.rb:52:in 'block in DiverDown::Trace::Session#build_trace_point'
        from tmp/test.rb:16:in 'block (3 levels) in <main>'

Solution
This is because several threads update the same DiverDown::Trace::CallStack object.
So I separated the call stack object for each thread.

Thank you for the great PR!


[edge] missing loadNuxtConfig export

Version
v2.2.0
Reproduction link
https://github.com/nuxt/nuxt.js
Steps to reproduce
Import loadNuxtConfig via const { loadNuxtConfig } = require("nuxt-edge/bin/common/utils") in a custom startup script
What is expected ?
const { loadNuxtConfig } = require("@nuxt/cli-edge") to return a function
What is actually happening?
TypeError: loadNuxtConfig is not a function
Additional comments?
We're using loadNuxtConfig to setup and boot our express app with nuxt.render middleware in dev/production mode, in order to register our own shutdown handlers and process.on(...) handlers for the app

This bug report is available on Nuxt community (#c8021)

version is actually dev-branch/2.3.0 but it is not available on cmty yet :)

Will be landed with #4160 :)

thanks! :)

Now on Edge 👏


Question: How to set the confidence score when training the face mesh model

Hello, in "mediapipe\mediapipe\modules\face_landmark\face_landmark.tflite", it can output the face confidence score :

I want to know when training this model, how to set this face confidence score, is it always set to 1, or is it comes from the output score of face detection model (in this way, it might be smaller than 1) ?

Hi @wwdok ,
Minimum confidence value ([0.0, 1.0]) from the landmark-tracking model for the face landmarks to be considered tracked successfully, or otherwise face detection will be invoked automatically on the next input image. Setting it to a higher value can increase robustness of the solution, at the expense of a higher latency. Ignored if static_image_mode is true, where face detection simply runs on every image. Default to 0.5.

Hi, @sureshdagooglecom Thanks for your explaination, i know what you said. Sorry that i didn't describe my question clearly. Actually, i am mimickiing face_landmark.tflite to build and train a face landmark model, but i meet some questions about the traning data, one of them is how to provide the confidence score to supervise the training, so it is about training phase, not inference phase, i am thinking is it always set to 1 for all training images, or is it comes from the output score of face detection model for each training image individually?

I would love to hear more about your progress and setup here. Are you going to share the training setup once you are done?

Hi @wwdok ,
We always set this to 1.

@sureshdagooglecom Got it, thanks you ~ @opchronatron Due to company's restriction, I may not open source it once i have done, very sorry ... I think writing the training code is not the most difficult part, but the dataset, now i am planning to use 3DMM to generate the landmark annotations.

Hi @wwdok,
Could you please show us your created model which is same as mediapipe. Can you tell us how did you create model? I used  link to create the model and train with WFLW (98 keypoints) dataset. Model was generalised well but results was not so great especially for close eyes and pupils. I used WingLoss function to train the model.


NuclioUI Url change from localhost to real url

We deploy your chart to eks cluster, we try to deploy function, and deployment is working but, when we go to mlrun-ui and click on the function which deployed this redirect to nuclio-ui  but with localhost address, once i replace the localhost with my nuclio url. is working. how i can replace nuclio ui url. when deploy the chart and is impact to mlrun ui.
The following image is mlrun dashboard, look on the bottom we can look the localhost address.

Thanks.

Hi @IgorKurylo - Thank you for raising this issue, can you please let us know what is the helm command you have used to spin up MLRun CE on your cloud?
make sure you provide it with either --set global.externalHostAddress=<external-ip-address> or --set mlrun.nuclio.uiURL=<ingress> if you have ingresses setup.

tl;dr information about creating functions with clusterIP / ingresses.
You can set MLRun to deploy Nuclio functions with ClusterIP instead of NodePort.
Nnote that those functions would not be invokable externally - but only within the cluster, unless you add ingresses as well.
Setting MLRun to work with ClusterIP as Kubernetes service type:
--set mlrun.api.extraEnvKeyValue.MLRUN_HTTPDB__NUCLIO__DEFAULT_SERVICE_TYPE=ClusterIP

If you want MLRun to create the Nuclio function with an ingress as well, then make sure you add below as well:
--set mlrun.api.extraEnvKeyValue.MLRUN_HTTPDB__NUCLIO__ADD_TEMPLATED_INGRESS_HOST_MODE=onClusterIP

If you wonder about the ingress host it will generate - then, you might need to tweak a bit the values.yaml and extend nuclio's platform configuration
...
nuclio
  platform:
    ...
    kube:
      defaultHTTPIngressHostTemplate: "{{ .ResourceName }}-{{ .ProjectName }}.{{ .Namespace }}.your-domain.com"
      defaultServiceType: clusterIP

If you need ingresses to be created with pre-defined annotations:
add
defaultHTTPIngressAnnotations:
  key: value

under nuclio.platform.kube as well (see above.)

OK, but the ALB Ingress controller works only when the service is on NodePort and not in ClusterIP. so if I change the value to cluster IP I can't set up ingress to function. What is the correct way to deploy functions and expose them via an ingress controller in your ecosystem, I would like to deploy functions and bind them to one ingress which creates dynamic rules.
You can please explain what the best process is for our use cace?

Well, depend on your case and your limitations. if you need to adhere your cloud conventions, then, to create NodePorts:
--set mlrun.api.extraEnvKeyValue.MLRUN_HTTPDB__NUCLIO__DEFAULT_SERVICE_TYPE=NodePort

and
--set mlrun.api.extraEnvKeyValue.MLRUN_HTTPDB__NUCLIO__ADD_TEMPLATED_INGRESS_HOST_MODE=always

same goes for Nuclio's platform configuration
...
nuclio
  platform:
    ...
    kube:
      defaultHTTPIngressHostTemplate: "{{ .ResourceName }}-{{ .ProjectName }}.{{ .Namespace }}.your-domain.com"
      defaultServiceType: NodePort

That means, your function would be deployed with NodePort by default and will ensure an ingress would be created as well.

OK thanks, I will try it.

@liranbg any suggestion ?

under nuclio's section

Ok, so i need to pull the values.yaml and add
kube:       defaultHTTPIngressHostTemplate: "{{ .ResourceName }}-{{ .ProjectName }}.{{ .Namespace }}.your-domain.com"       defaultServiceType: NodePort
or I can put this with --set in helm chart.

hmm you can kubectl -n <namespace> edit cm nuclio-platform-config and add it there by yourself after, or need to provide it within a values file. it is a bit tricky or perhps Id say advanced usage to define it but all lego pieces are there.
note that nuclio-platform-config is a configmap that represent nuclio's configuration. once you edit it, you will need to restart both nuclio controller and dashboard deplyoments.

OK, do you have the option to change ingressClassName?

hmm, good question. perhaps @TomerShor will know. is nginx IngressClassName configurable for nuclio functions?

@liranbg @IgorKurylo nginx IngressClassName is not configurable in nuclio functions.
You can configure an Ingress' host, paths, path type and TLS configuration.

@IgorKurylo  - let us know if there's anything else we can help with.

Thanks, guys.


Update docs toctree and add instructions for contributing

What current issue(s) does this address, or what feature is it adding?
Updates the toctree so all info is displayed on the "readthedocs" website
Also added instructions for making contributions to neo-python/docs
How did you solve this problem?
@localhuman prompted me to run make docs then Google to figure out how to update a toctree
How did you make sure your solution works?
Personal Testing
Are there any special changes in the code that we should be aware of?
No
Please check the following, if applicable:

[ ] Did you add any tests?
[ ] Did you run make lint?
[ ] Did you run make test?
[x] Are you making a PR to a feature branch or development rather than master?
[x] Did you add an entry to CHANGELOG.rst? (if not, please do)

I'm getting a couple of warnings when building
/Users/erik/Documents/code/neo-python/docs/source/changelog.rst:120: WARNING: Unexpected indentation.
/Users/erik/Documents/code/neo-python/docs/source/changelog.rst:124: WARNING: Block quote ends without a blank line; unexpected unindent.
/Users/erik/Documents/code/neo-python/docs/source/changelog.rst:250: WARNING: duplicate label [0.4.3] 2017-12-21, other instance in /Users/erik/Documents/code/neo-python/docs/source/changelog.rst
/Users/erik/Documents/code/neo-python/docs/source/install.rst:129: WARNING: Title underline too short.

I'm aware the first 3 are likely not caused by your PR, but it would be great if you can resolve those such that your instructions about no warnings holds and people start from a valid state :)
Other observations;
you seem to have started your branch from a really old commit (May 18, 2018, 0a47d011a9d7f0e51ed0adda0a300d56a1caee0d). As a result the documentation generated show a version identifier (above the left menu, above the search bar) of 0.7.0-dev and the changelog is also outdated. I'm going to assume it will work out fine if we merge, but in the future it would be nice if you could try to branch from the most recent commit at the time (or even merge development before committing). Other than that, useful additions. Thanks!

I figured out merging CoZ development into my development branch. #githubnoobproblems


cemu-ti: Fetch patch from upstream instead of using a modified version in-tree

Yesterday or today, I learned that fetchpatch has the relative attribute. That would have been really helpful when making it, but oh well.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[x] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 354358

x86_64-linux

:white_check_mark: 1 package built:

cemu-ti


Fee Payer drop-down in settings highlights two options at one time

Description
Reference this issue:https://github.com/hashgraph/hedera-transaction-tool-demo/issues/117
Steps to reproduce
see above
Additional context
No response
Hedera network
testnet
Version
0.4.0-alpha.2
Operating system
No response

JavaFX has two similar controls: ComboBox and ChoiceBox. They are both dropdowns. The fee payer is a ComboBox, because it needs to be editable (in case the user adds an account that is not in the dropdown)
If I want to change the highlight behavior I need to use a ChoiceBox, but that is not editable, which would then require much less elegant solutions for adding a fee payer.


Configure deployment_path more conveniently

Currently if you want to configure the deployment_path variable, you need to set something like:
set :deployment_path, proc { Dir.pwd.gsub('\n', '') + '/folder' }

I suggest in to hide that logic and configure it only with folder

I like that idea


TargetPlatformMinVersion should be set to some default for win8 targeting

Being able to specify win8 as a target as this project allows is great, but it's broken by default without adding this line:
https://github.com/AArnott/pinvoke/blob/6b7928725039c62ec145bc863ab8738a5d4e78d2/src/Directory.Build.props#L12
Otherwise this build error results:

C:\Program Files (x86)\Microsoft Visual Studio\IntPreview\MSBuild\Microsoft\VisualStudio\v15.0\AppxPackage\Microsoft.AppXPackage.Targets(794,25): error MSB4086: A numeric comparison was attempted on "$(TargetPlatformMinVersion)" that evaluates to "" instead of a number, in condition "'$(TargetPlatformMinVersion)' >= '10.0.17200.0'". [D:\temp\bug2517\win8lib\win8lib.csproj]

Shouldn't this be defined by default?

😕 Why is it that pasting permalinks in github issues sometimes inserts the text of the permalink'd code and sometimes just leaves the link?