chore: unpin nodejs version from 22.4

Motivation
Use the latest stable node release.
Description
Node version 22.5.1 fixes the regressions.
Steps to test or reproduce
Run all tests.

moving to draft as it seems we have some consensus to wait until 22 becomes active LTS

moving to draft as it seems we have some consensus to wait until 22 becomes active LTS

@wemeetagain  Previously we didn't had precedence to only use LTS versions, rather latest version. Even current codebase is still locked upto 22.4. So either we change it and completely strict 22 or use latest version of 22.
This PR was reverting an earlier PR because 22.5 introduced a regression, but 22.5.1 fixed it. Don't see any reason to keep restricting upto 22.4.

This PR was reverting an earlier PR because 22.5 introduced a regression, but 22.5.1 fixed it. Don't see any reason to keep restricting upto 22.4.

I personally don't have confidence in nodejs release process for current, apparently their CI had errors but still passed. The LTS release runs much more tests, does a whole range of ecosystem smoke tests, and is a whole less likely to break.
Having to debug strange node issue is not fun..and we have better things to do

Previously we didn't had precedence to only use LTS versions, rather latest version.

Yes, I think our experience the other day, where we had to pin the version, changed my mind a little bit.
It seems that the nodejs folks treat the new version with more leeway to be actually unstable and breaking before it becomes "current". Once it becomes "current", their standards for something breaking become much higher.
I tend to agree with Nico that we don't really care to have our daily progress possibly stopped again by a breaking version, and we can mitigate that by pinning a specific version until it becomes "current" LTS.

@wemeetagain Need to update the required check names to align with node version.

rerunning CI so the settings will remember the new jobs to enforce (settings only remember CI job names run in the last week)

:tada: This PR is included in v1.24.0 :tada:


Remove dependency on go-wire

iavl uses/vendors go-wire (which depends on ripemd160)
Make it either indpendendent from go-wire / amino, or update to amino.
Remove ripemd dependency from gilde file(s).

We need to make some decision here.
We clearly need to remove the go-wire dependency.
Based on how wire is invoked right now, it is only used to encode fixed length structures.
If we switch to amino, compatible implementations will need to have some awareness of amino. To me this looks like the typ3 table and varint encoding maybe but not thedisfix identifier.
Are we comfortable requiring this?
One use case for compatible implementations will implementing IBC in lotion and in our partners stacks.

I think this is semi relevant to
https://github.com/cosmos/cosmos-sdk/pull/454/files

Thanks for your input @zmanian. There is a related PR #53. I'm not sure yet if I found a bug here or if the way iavl uses amino needs to be updated.

Looks like IAVL never actually encodes a struct, so I dont think it needs to know the typ3 table - it only calls EncodeIntXxx and  EncodeByteSlice, which AFAIK are just purely proto3 encodings of ints and byte slices.
That sounds pretty ideal for this.

EncodeIntXxx is just big endian fixed size int encoding
EncodeByteSlice is varint LEB128 encoded length prefixed bytes

Int32 and Int64 are the fixed-size big-endian, but Int8 is apparently encoded as a Varint ...
In any case, there's nothing amino specific about these, right ?

Yeah 8 and 18 are varints

Yeah these aren't amino things. But LEB128 varints and bigendian fixed with ints are specifications the protobufs and amino incorporate

Closing this. The go-wire (golang package) dependency was removed with #53. I'll update the documentation to clarify encoding specific things.
linking to #56


add location params in media object

i notice sometimes a media will have a location.
However, it doesn't parse location params in Media class.
Hence, I basically follow thread-item.js and do same thing in parsing location

Sounds really great man! Thank you very much. One thing, can you make this.location to be top level property, like this? https://github.com/huttarichard/instagram-private-api/blob/master/client/v1/media.js#L41 and on get params refer to this top level property?

on that topic, https://github.com/huttarichard/instagram-private-api/blob/master/client/v1/thread-item.js should be done same way. Should be refactored. Make you changes big guy, and Im going to do new release with coverage for your functionality and for thread-item.js

ok, just one more question, I do not understand which props should be "top-level" and which should be "_params". Could you tell me the difference between them?
But anyway, I will do the refactor later.
Thanks for your fast reply.

@hiEven everything in _params should be properties which this repository don't have wrapper for and they don't require more functionality. So for example there is wrapper for Comment thus medium.comments should be array fullfiled with Comment instances. On the other hand medium. likeCount is a number which don't require more functionality so it can stay in _params. Top-level properties are for eaiser access. If you have medium you can write medium.comments.forEach(...) insteadof medium.params.comments, but medium.params.comments should be available too, in this case should be already serialized json.
Make sense?

@hiEven I will take a look on the tread-item.js and this tonight or tmorrow, so if you are busy don't worry I will merge your changes and make a new release.

That's ok. I am about to do it. It should not be a big problem I guess.
Thanks for your explanation. It totally makes sense to me

@huttarichard I just did the changes. If anything I can help, please tell me

@hiEven awesome man! Thanks a lot, going to merge this.

@hiEven ok should be published 0.4.9 :) well done


Improve import command with install/add functionality on a per-package basis

Description of the new feature / enhancement
I was surprised to see how light on features the import command is. I was about to switch my bespoke winget scripts over to use import instead of install, but instantly ran into some dealbreakers.
Notably, some of the packages I'm installing require the use of --custom, and I wish that I could also specify silent per-package.
I imagine some people would require --location and --architecture, and --override to genuinely make use of this feature as well. I can't imagine many people are using import given the current state?
Note this is distinctly different than  #3373 which I believe is wanting more functionality at the command level, rather than the per-package level.
Proposed technical implementation details
I haven't dug into winget-cli's code but I'm hopeful it isn't too difficult to add additional optional entries to packages.schema.2.0.json, for each of these features much the same way that Scope is in there.
Then from there, assuming import calls into the same path as winget install does, just construct the arguments from the imported json as necessary.
There is an open question on exporting some of these things given that the history of them may have been lost after install?

Related to:

https://github.com/microsoft/winget-cli/issues/3281

Reading into this more, it seems like WinGet Configuration is /actually/ what I am wanting in the long term and would have likely switched to that even if import had all of these improvements. Though I'll let the WinGet team close this issue as they see fit as it might highlight a couple pieces:

winget import remains rather lacking. You'll probably want to decide whether or not you want to keep supporting it as is or just switch to winget export producing a configuration.dsc.yaml, and swap winget import to
Allow tracking for metrics, I.E. user desire, etc.

We're planning on changing the default behavior for WinGet Export to generate a configuration file in the future. We still want to be able to "import" the existing packages.json format as well as a *.winget or *.dsc.yaml WinGet Configuration File. I think both places should end up working as people need, but the golden path in the long run is a WinGet Configuration file.

#4092


Fix unnecessary mut

I was trying to build master and I faced this compiler error:
   Compiling nix v0.26.1 (/home/jgomez/wasm-patched/nix)
error: variable does not need to be mutable
    --> src/sys/socket/mod.rs:1532:13
     |
1532 |         let mut p = &mut mmsghdr.msg_hdr;
     |             ----^
     |             |
     |             help: remove this `mut`
     |
note: the lint level is defined here
    --> src/lib.rs:48:9
     |
48   | #![deny(unused)]
     |         ^^^^^^
     = note: `#[deny(unused_mut)]` implied by `#[deny(unused)]`

error: variable does not need to be mutable
    --> src/sys/socket/mod.rs:1685:13
     |
1685 |         let mut p = &mut mmsghdr.msg_hdr;
     |             ----^
     |             |
     |             help: remove this `mut`
```
This has an easy fix, though :)

I'm using: `rustc 1.72.0 (5680fa18f 2023-08-23)` (just in case is relevant)

Ups!!! Wrong branch, sorry :)


Support proxy server

Hi - trying to get this to support proxy servers, as it's a requirement for my implementation.   I'm not seeing an easy way to pass thru options to HTTParty, but i think on HTTParty side should be easy to add the necessary options for proxy support if needed.
I'm wondering if maybe a bit of refactor around HOW options are passed back to HTTParty - maybe a hash instead of just a bunch of KV pairs?    Thoughts?   Help? :)

PR for this ended up doing a few more things - commented in PR thread.


TP2000-801 copy duties to next

TP2000-801 copy duties to next

Why

On the commodity codes / duties step of the create measure, duty sentences are often repeated across many rows of commodity codes. To speed up the process of creating pairs of commodity code and duties in this way, tariff managers have requested a capability that allows quickly copying a duty sentence from one row to the next.
What

On the comm code / duties step of the create measures wizard, provide a progressive enhancement that allows copying down the duties sentence from one form to the next.

Checklist

Requires migrations? No
Requires dependency updates? No

Codecov Report
Patch coverage has no change and project coverage change: -0.01 :warning:

Comparison is base (cf1f277) 92.21% compared to head (cac7a34) 92.21%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #853      +/-   ##
==========================================
- Coverage   92.21%   92.21%   -0.01%
==========================================
  Files         411      411
  Lines       26999    26998       -1
  Branches     2024     2024
==========================================
- Hits        24898    24897       -1
  Misses       1715     1715
  Partials      386      386

Impacted Files
Coverage Δ

measures/forms.py
97.75% <ø> (ø)

publishing/views.py
60.08% <ø> (-0.17%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Training fail

Hi
Training fail
raise ValueError('malformed node or string: ' + repr(node))
ValueError: malformed node or string: (192, 256)

try to pass image size argument as --image_size 192,256
this will fix the problem

Hi
@osama-ahmed
Still fails, throws error:
python DCGAN.py --data /data/images/*.png --epochs 100000 --output /data/output --image_size 192,256
File "DCGAN.py", line 193
print(f"{epoch} [D loss: {d_loss[0]} | D Accuracy: {100 * d_loss[1]}] [G loss: {g_loss}]")
^
SyntaxError: invalid syntax

f-strings are only supported in python 3.6+ versions
you have two options

upgrade your python version to 3.6+
or change that string
print(f"{epoch} [D loss: {d_loss[0]} | D Accuracy: {100 * d_loss[1]}] [G loss: {g_loss}]")
to another format.
https://realpython.com/python-string-formatting/

Hi
Will it support with python 3.6 or not?
On Wed, 14 Aug 2019, 21:04 Osama Ahmed<EMAIL_ADDRESS>wrote:

f-strings are only supported in python 3.6+ versions
you have two options

upgrade your python version to 3.6+
or change that string
print(f"{epoch} [D loss: {d_loss[0]} | D Accuracy: {100 * d_loss[1]}]
[G loss: {g_loss}]")
to another format.
https://realpython.com/python-string-formatting/

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/DataSnaek/DCGAN-Keras/issues/3?email_source=notifications&email_token=AFLNCW2AJHCN7JK3753QQATQEQQX3A5CNFSM4IIPXWNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4JGAWA#issuecomment-521298008,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFLNCW5SODWS4R74AISDPLDQEQQX3ANCNFSM4IIPXWNA
.


Fix vaccination history, include latest history endpoint in coverage

This completely adresses #237 and supersedes #238. It brings vaccination history back to status quo and includes the latest entry of the vaccination history in the coverage endpoint.
The benefit of doing this that it is possible to calculate the daily number of vaccinations (The deltas in the coverage table both include the J&J doses).

I forgot to tick the box @marlon360 . I pushed the code, hopefully it complies to the style policy now.


Routing engine

Adding routing engine.

I've reviewed the routing engine but either I seem to be doing wrong or something about the engine isn't going correctly.
I have enabled my Rewrite Module and set AllowOverride to All.
I have taken the liberty of doing some exploration inside the code and found that
when Router::dispatch () is called by the core after all files have been loaded in it always seems to return false.
It also always brings up the error "Couldn't find that route".
I hope you could clarify what I might not be doing correctly or whether something didn't go as planned.

Could you try to add:
"RewriteBase /"
Below "RewriteEngine on" in your .htaccess file.

hey joh
doe is niet
2015-11-26 10:29 GMT+01:00 jacted<EMAIL_ADDRESS>
Could you try to add:
"RewriteBase /"
Below "RewriteEngine on" in your .htaccess file.
—
Reply to this email directly or view it on GitHub
https://github.com/bobdesaunois/emmaphp/pull/85#issuecomment-159857209.

I have done this, my .htaccess file now looks like this:
RewriteEngine On
RewriteBase /

RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d

RewriteRule ^(.*)$ index.php?/$1 [L]

Can you try to dump this:
parse_url($_SERVER['REQUEST_URI'], PHP_URL_PATH);
and show results here.
Also what's the path for the framework?
Running it on: localhost/emmaphp/ ?

Stop met al die emails man

Comitted a new change that should fix subdir urls.
Can you try the new version.

After having tested the new version everything works accordingly and has been pulled. :) +1


[Bug] Strange memory usage

Description of the bug
Memory usage in Dashdot : 11GB
Memory usage in free: 10GB
Memory usage in top: 73MB (but actually around 10GB)
Memory usage in htop: 3.5GB
Free and top take into account the buffer and cache, and I guess Dashdot uses similar tools.
htop shows the actual memory being used (since buffered and cached memory are reserved by the OS and not really used, from what I understand at least).
I feel like it would be more useful to display the actual memory being used, like htop does (RES and not VIRT/SHR).
Here is an interesting article I've found, hope it helps in some way
https://peteris.rocks/blog/htop/

Thanks for the issue, but unfortunately I can't really reproduce it. I don't know what htop shows, but the system monitor in Linux shows the exact same result as dashdot.
Then the only thing "wrong" is, that I am using the 1024 base calculations, while writing "GB" instead of "GiB". This is not "really" wrong per-se, because "GB" (and "MB", "KB") gets used interchangeably, but it is technically wrong and confusion may arise. So I will switch to the unambiguous 1024 base measurements of "GiB", "MiB" and "KiB" to avoid any further confusion.


Make DecodeRaw accept ReadOnlyMemory<byte>

DecodeRaw is currently a wrapper for DecodeRawInternal which itself accepts ReadOnlyMemory<byte>.
There is no reason why the public function shouldn't also accept ReadOnlyMemory/ReadOnlySpan.

I think issue also applies to the whole library, it would be much better to use the Memory/Span types for public APIs


Install Requirements Fails, bad pbr version

Hello! I've been trying to get pygradle to work, but I'm having a rough time with a bad PBR version buried somewhere in the dependency chain. Here's the output I'm getting:
> Task :installTestRequirements FAILED
Error installing package using `[/mnt/c/Users/rniem/home/github.com/niemi-online/test/build/venv/bin/python, /mnt/c/Users/rniem/home/github.com/niemi-online/test/build/venv/bin/pip, install, --disable-pip-version-check, --no-deps, /home/ricky/.gradle/caches/modules-2/files-2.1/pypi/pbr/0.11.0/72801e9ffd52e7f438422a3e08b578e906cd270a/pbr-0.11.0.tar.gz]`
Processing /home/ricky/.gradle/caches/modules-2/files-2.1/pypi/pbr/0.11.0/72801e9ffd52e7f438422a3e08b578e906cd270a/pbr-0.11.0.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-gaxyldtn-build/setup.py", line 22, in <module>
        **util.cfg_to_args())
      File "/tmp/pip-gaxyldtn-build/pbr/util.py", line 261, in cfg_to_args
        wrap_commands(kwargs)
      File "/tmp/pip-gaxyldtn-build/pbr/util.py", line 482, in wrap_commands
        for cmd, _ in dist.get_command_list():
      File "/mnt/c/Users/rniem/home/github.com/niemi-online/test/build/venv/lib/python3.6/site-packages/setuptools/dist.py", line 496, in get_command_list
        cmdclass = ep.resolve()
      File "/mnt/c/Users/rniem/home/github.com/niemi-online/test/build/venv/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2297, in resolve
        module = __import__(self.module_name, fromlist=['__name__'], level=0)
      File "/tmp/pip-gaxyldtn-build/pbr/testr_command.py", line 47, in <module>
        from testrepository import commands
    ModuleNotFoundError: No module named 'testrepository'

Everything I can find online points to this version of pbr having a bug that leads to this. Here's a stack overflow post  w/ relevant excerpt: There is a bug with pbr 0.11.0, ... use pbr version 0.11.1 instead.
I'm not actually familiar enough w/ the python toolchain to know how to go track this down though (I looked through setuptools and couldn't find the explicit dependency). Could anyone here help track down what change is actually required here?
Thanks!

Hi, I got the same error,  testrepository have to be installed before pbr

This is not pygradle issue, but we know from experience that pbr can cause issues for people. For example we pin it in our pipeline. This is actually a well known issue with setup_requires being processed directly by setuptools and not by pip (or even Gradle in our case) because they are not recorded in either Python or Ivy metadata. See these parts of pip documentation for details:

https://pip.pypa.io/en/stable/user_guide/?highlight=setup_requires#installation-bundles -- see the warning box about setup_requires, and
https://pip.pypa.io/en/stable/reference/pip_install/?highlight=setup_requires#controlling-setup-requires

In pygradle we provide ways to customize both of these. The customization can be applied in build.gradle for a single project, or in a plugin that applies pygradle main python plugin and then adapts the settings for all your projects. That's what we do internally. For example, to use a pbr version that works for you, you can add this in your build.gradle:
dependencies {
    setupRequires 'pypi:pbr:1.8.0'
}

That will ensure that pbr is processed by Gradle and pip before the package that needs it is installed, so there will be no need for setuptools to go and fetch it during build.
To do the same in your plugin code for every project you could do:
pythonExtension.forceVersion('pypi', 'pbr', '1.8.0')
project.dependencies.add(CONFIGURATION_SETUP_REQS.getValue(), pythonExtension.forcedVersions['pbr'])

Notice that you can call forceVersion method inside python { ... } extension closure to force the dependencies that are transitively getting an unwanted version. However, remember that does not apply to setup_requires, so you'll have to add it to your dependencies anyway.
Similarly to configure how your setup_requires are handled by setuptools (other than pbr) you can define:
public static final String DISTUTILS_CFG = "..."  // your config
project.afterEvaluate {
    project.tasks.withType(InstallVirtualEnvironmentTask) { InstallVirtualEnvironmentTask task ->
        task.distutilsCfg = DISTUTILS_CFG
    }
}

or the equivalent setting in your build.gradle

This does not work for me. Could ypu help me?


Go Modules: update dependencies.

update zap version to 1.18.1
run go mod tidy

@kazeburo Can you review it?


added questions

Hey,
here are my questions.
I will keep an eye on this project, I will more than happy to help if needed.

3x easy JS
2x hard JS
2x easy React

Perfect.
Well done
Thanks


Run inside interpreter should change working directory

From<EMAIL_ADDRESS>on 2010-08-05T07:21:25Z
Hello,
I face the following problem. I have a file let's say c:\test\test.py which uses a dll stored in the same path c:\test\test.dll
My code would be something like
import ctypes
ctypes.cdll.LoadLibrary("test.dll") #dll in same directory as test.py
when I "run and interact" everything works fine, but on "run inside interpreter" I get an error like the module (test.dll) was not found as the current working directory is not c:\test
Honestly I haven't got the difference between "run and interact" and "run inside interpreter" yet. What is the difference of these two commands and is the behaviour I have as expected?
(I keep searching the possibility to mark an issue as defect/enhancement)
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=262

From pierre.raybaut on 2010-08-05T05:39:03Z
(a simple user can't mark an issue as an enhancement which is unfortunate as it would be easier for me as well)
Well, actually you've just pointed out the difference between these two 'run' commands:
[1] "run and interact": open a brand new Python interpreter, run the script and interact with it right after its execution (this is strictly equivalent to 'python -i script.py' in a terminal, so the current working directory is the script directory)
[2] "run inside interpreter": run the script inside an already existing interpreter which had its own life before executing this script (namespace, working directory, environment variables, ...) -- as a consequence, results are expected to be different from [1]. This is almost equivalent to the following actions performed in a terminal:  open a Python interpreter, do what you want inside of it (computings, ...) and run your script with the 'execfile' function.
In other words, [1] is to be used when you want to run your script as you would outside Spyder or any other IDE but you want to interact with it (for debugging purpose for example). [2] is to be used when you want to run one or more script(s) inside an existing interpreter in which you have already performed some computings and with data stored in the namespace that you don't want to loose between each script execution.
Status: WontFix


Should add content to "intro to pyhton"

As discussed, I like toadd issue regarding creating a content for the section of introduction to python please please consider this issue and assign it to me.

@Sriramkashyapks Yes You can work on it

@thecuriousteam i want to work on this, kindly assign this to me.


How to Restart keyword count? And other related questions

Hello,
I want a little help of you experienced guys.
I've created a couple of entries, just to test software's behavior LOL. Then I deleted some and seen that the count for keyword was starting in a letter, not in a number anymore. So I cleaned up the whole database, as I had no data. Well, didn't work.
So, question (1): How do I make it start from the 0 (or 1, IDK) again?
Another question (the first related) is: (2) why is keyword not unique/indexed in the database? Would it be any harmful if I make it unique?
And finally, last question (3): Is there any way to prevent, if I create a 'public page', that users create links with keywords lesser than 'N' characters, and restrict them for my own use?
Thanks for your attention.
Best Regards,
Bruno Moreira-Guedes

manually reset options.next_id in the database

I think it is

There's a way to do everything with a plugin. See https://github.com/YOURLS/YOURLS/wiki/Possible-With-A-Plugin


last uf2-build not working?

Hello,
today i set up a tracker and i tried to copy the last build (https://github.com/EngineerGuy314/pico-WSPRer/blob/main/build/pico-WSPRer.uf2) to the pico but it does not reboot. Is there an error in the build?
i tried some other uf2 examples and these file worked fine.
thank you very much and many thanks for the wonderful project
73 IN3IZL

the pico does not reboot after i copy the file. There is no LED activity and the COM-Port does not show up.

no way to debug without the serial port. email me the .uf2 that you downloaded and I will take a look.

this is the file i am using, i had to make a zip to upload it

ok problem solved, now i tried to download the file directly from github (raw).
First i was downloading it with "wget https://github.com/EngineerGuy314/pico-WSPRer/blob/main/build/pico-WSPRer.uf2" but after downloading it via the raw button it worked.
thank you very much


Optional Export Root Type Aliases

Changes
Adds the CLI option --root-types to optionally export all of the types from components for convenience.

A dependency on change-case was added given the need to account for various casing in the OpenAPI component schema name (e.g. 'some-type') not always being valid for TypeScript type identifier. (If this dependency can be avoided, I'm open to suggestions)
Relevant Issues:

https://github.com/openapi-ts/openapi-typescript/issues/1414

Previous PR's:

https://github.com/openapi-ts/openapi-typescript/pull/1260

How to Review
Keep in mind this is my first ever time working with the TypeScript Compiler API and AST so any recommendations for a way to write this in a cleaner way is more than welcome.
I've added a GitHub example called github-api-root-types.ts and the update:examples should generate this. Tried to stick to what seems to be standard practices here. If there are other preferred methods of testing/demonstrating this feature, please let me know!
Checklist

[x] Unit tests updated
[x] docs/ updated (if necessary)
[x] pnpm run update:examples run (only applicable for openapi-typescript)

Not sure why the openapi-react-query package is failing to build in CI as openapi-fetch is building properly

Not sure why the openapi-react-query package is failing to build in CI as openapi-fetch is building properly

GitHub Actions has had some minor timeouts/failures in the last week. Nothing major, but it seems a few PRs failed for no reason. Can’t complain about free CI though!

A dependency on change-case was added given the need to account for various casing in the OpenAPI component schema name (e.g. 'some-type') not always being valid for TypeScript type identifier. (If this dependency can be avoided, I'm open to suggestions).

Nope! This is fine. Any time a lightweight dependency is added that serves a need is OK. Better to have well-tested code handling a common usecase than write code off-the-shelf.

Tests looks great, and this meets all the requirements! 🎉 Thanks for adding; a lot of people will love using this feature.
Be sure to add a minor changeset (see comment) so we can merge & release this!

Added a minor changeset with a brief summary of the changes. Wanted this feature for a project and I've already been using this update locally. It cleans up the usage so much when trying to add types to my code! Glad to help everyone out 😄


fix: add warning regarding pattern values

I realized that it's easy to get "\" swallowed by the command line if not quoted, which messes up pattern values for regular expressions. I added a warning and and fixed the relevant examples in entites:configure, entities:convert and entities:create.

:tada: This PR is included in version 2.0.0-beta.2 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 3.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


tf.contrib.data.parallel_interleave with tf.data.Dataset.from_generator

I think this is a feature request (or maybe I miss something). When data is coming from multiple remote sources multiple tf.data.Dataset.from_generator should run in parallel. Therefore tf.contrib.data.parallel_interleave is already available, see this small code example.
def generator(n):
  # returns n-th generator function

def dataset(n):
  return tf.data.Dataset.from_generator(generator(n))

ds = tf.data.Dataset.range(N).apply(tf.contrib.data.parallel_interleave(dataset, cycle_lenght=N))

# where N is the number of generators you use

Describe the problem
This problem with the code above is that generator cannot look like this:
def generator(n):
  return consumers[n] # A python list with n items

Because then the error 'TypeError: list indices must be integers or slices, not Tensor' will occur. However it would be really nice to have support for multiple from_generator pipelines in parallel.
@mrry Do you have an idea whether this is indeed a feature request or that it is possible with the current codebase?

It's possible with some involved usage of tf.py_func(), but we've had the feature requested enough times that it's worth adding to the Dataset.from_generator() API. An implementation is currently out for review, and it will add an optional args argument to Dataset.from_generator(), allowing you to specify tensors that should be passed to the generator function as NumPy arrays (like the inp argument to tf.py_func()).
/cc @jsimsa FYI.

The implementation has been submitted and will become available in the next sync. To work around the problem using tf.py_func() in a Dataset.map(), you'd have to maintain a list of Python iterator objects, then write a Python function that takes an integer, looks up the corresponding Python iterator in the list, and then returns the result of calling next() on it. (This is essentially all that Dataset.from_generator() does, although there's some additional nuance to ensuring that the iterators are properly disposed of at the end of the computation.)


Getting cookies from HttpClientResponse throws

RangeError (index): Invalid value: Valid value range is empty: 0
I/flutter (26221): #1      _Cookie._validate (dart:_http/http_headers.dart:989:14)
I/flutter (26221): #2      _Cookie._parseSetCookieValue (dart:_http/http_headers.dart:933:5)
I/flutter (26221): #3      new _Cookie.fromSetCookieValue (dart:_http/http_headers.dart:849:5)
I/flutter (26221): #4      new Cookie.fromSetCookieValue (dart:_http:1112:16)
I/flutter (26221): #5      _HttpClientResponse.cookies.<anonymous closure> (dart:_http/http_impl.dart:317:26)
I/flutter (26221): #6      List.forEach (dart:core/runtime/libgrowable_array.dart:275:8)
I/flutter (26221): #7      _HttpClientResponse.cookies (dart:_http/http_impl.dart:316:14)

https://github.com/flutter/flutter/issues/20898

is there a way to speed this up? I'm stuck with this and can't upgrade to the newer version of flutter

+1
we are also block by this issue and it is critical for our application.

Invalid value: Valid value range is empty: 0#0
same error

+1

here is some example code
Future<String> ripWebPage(String url,
      {String postData, String postContentType}) async {
    String userAgent =
        'Mozilla/5.0 (compatible; MSIE 9.0; Windows NT 6.0; Trident/5.0)';

    if (url.contains("?")) {
      url = url.substring(0, url.indexOf('?'));
    }
    if (url.contains("#")) {
      url = url.substring(0, url.indexOf('#'));
    }
	List<Cookie> cookieJar;
    var client = new HttpClient();
    client.userAgent = userAgent;
    String retVal;
    var uri = Uri.parse(url);
    if (postData == null) {
      // this means it is a GET.
      retVal = await client
          .getUrl(Uri.parse(url))
          .then((HttpClientRequest request) {
        request.headers.set('Accept-Language', 'en-us');
        request.headers.set('Accept-Encoding', 'gzip, deflate');
        request.headers.set('User-Agent', userAgent);

        return request.close();
      }).then((HttpClientResponse response) {
        cookieJar = response.cookies;
        return response.transform(utf8.decoder).join();
      });
    }
}

Where you can pass in any URL that would have cookies returning, and cookieJar = response.cookies will return the error that the OP had posted.  However, I noticed if you put a breakpoint on that line and wait, response.cookies will return values.

Been looking into this with @gerryhigh and we've figured out the below. Was looking into where the exception is being raised dart:_http/http_headers.dart:989:14 code looks like this.
if (value[0] == '"' && value[value.length - 1] == '"') {
  value = value.substring(1, value.length - 1);
}

Seems its two things, first the length of value should be tested before you try to index -1, and second the check could indicate there is only 1 speech mark in which case you try to get substring(1, 0) which is invalid. I believe testing with the following headers should replicate the issue.
set-cookie: lang=en; path=/
set-cookie: _my_session=somereallylongstring; path=/; HttpOnly
set-cookie: basic_auth=; path=/; expires=Thu, 01-Jan-1970 00:00:00 GMT
set-cookie: basic_auth_checked=; path=/; expires=Thu, 01-Jan-1970 00:00:00 GMT
set-cookie: password=; path=/; expires=Thu, 01-Jan-1970 00:00:00 GMT
set-cookie: visitor_session_id=someid; domain=.mydomain.com; path=/; expires=Sat, 08 Sep 2018 16:54:31 GMT;

+1
is there any progress with this?

I wonder if this has been fixed in the latest version? We also came across this issue. Thanks.

Ran into this bug using the latest version of flutter:
$ dart --version
Dart VM version: 2.3.1 (Tue May 21 19:28:38 2019 +0200) on "windows_x64"

$ flutter --version
Flutter 1.5.4-hotfix.2 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 7a4c33425d (3 weeks ago) • 2019-04-29 11:05:24 -0700
Engine • revision 52c7a1e849
Tools • Dart 2.3.0 (build 2.3.0-dev.0.5 a1668566e5)

It's very concerning because it's been unresolved for almost a year and you can't guard against it, e.g. this still throws an error:
try {
  if (res.cookies != null && res.cookies.length > 0) {
  }
} on RangeError catch (e) {
  print("RangeError: " + e.toString());
  //ignore https://github.com/dart-lang/sdk/issues/34220
}

Hi, I fixed this bug (which is a duplicate of https://github.com/dart-lang/sdk/issues/35804) and the fix will be in the next stable release.


apply-setter kpt function may provide wrong results when using subpackages

we have 2 kpt packages
package A (main package)
package B (subpackage in package A)
package b defines a pattern for kpt-set `# kpt-set: prefix-${var1}-${var2}
if both vars are containing a substring of the kpt pattern, for example - in the given case, the apply-setter function cuts off the var2 in the result.
to reproduce the issue:

clone git@github.com:RolandOtta/kpt-issue-2.git
cd to directory A
kpt fn render . --output="/tmp/output" --truncate-output=false

the output already shows the invalid result
for package A/B it is still fine
[INFO] set field value to "prefix-foo-bar-foo2-bar2" in file "demo.yaml" in field "metadata.name"
package A then provides the wrong result
[INFO] set field value to "prefix-foo-bar-bar2" in file "demo.yaml" in field "metadata.name"
the problem seems to be caused by the regex pattern in the https://github.com/GoogleContainerTools/kpt-functions-catalog/blob/master/functions/go/apply-setters/applysetters/apply_setters.go#L271 function.
the following patch fixed my issue. as i am not really familiar with go i am not sure whether this patch could cause other issues.
@@ -207,7 +207,7 @@ func (as *ApplySetters) visitScalar(object *yaml.RNode, path string) error {
                return nil
        }

-       curPattern := setterPattern
+       //curPattern := setterPattern
        if !shouldSet(setterPattern, as.Setters) {
                // this means there is no intent from user to modify this setter tagged resources
                return nil
@@ -224,7 +224,7 @@ func (as *ApplySetters) visitScalar(object *yaml.RNode, path string) error {

        // replace the remaining setter names in comment pattern with values derived from current
        // field value, these values are not provided by user
-       currentSetterValues := currentSetterValues(curPattern, object.YNode().Value)
+       currentSetterValues := currentSetterValues(setterPattern, object.YNode().Value)
        for setterName, setterValue := range currentSetterValues {
                setterPattern = strings.ReplaceAll(
                        setterPattern,

@RolandOtta This is because the regex is ambiguous and the function in A/Kptfile is assuming that the current value of var1 is foo-bar-foo2 and is replacing it with new value foo-bar. Why don't you also declare the value of var2 in A/Kptfile ?

@phanimarupaka
we also experienced that issue in our use cases.
we often have the same kpt blueprint (e.g. a secretmanager secret) as a subpackage within one folder, which have overlapping setters with the root package.
in such situations it's not possible to copy var2 to A/Kptfile because then you'll overwrite the value in multiple subpackages.

The task here is
For value prefix-foo-bar-foo2-bar2
For setter pattern prefix-${var1}-${var2}
We should derive the values of var1 and var2
This is ambiguous pattern and has multiple possibilities, and can't have deterministic answer without additional data.
@unlimitedbits Can I know more about your actual use-case with an example ?
There are couple of ways to solve this
Avoid having var1 in A/Kptfile and push them to all sub-packages.
Use different setter name for var2(var2B)
But this can be solved in general using target selectors feature which will be released soon(by early october). Using this, you can select specific packages on which the setters should be applied.

Closing this issue as the alternatives are proposed and no response. Please reopen if you still have questions.


Carousel is breaking when auto play is applied

Description
Hi, why is it that carousel is breaking when I click one of the indicators while there is an autoplay function to slide the carousel items.
Repro Steps
Here's the living example of the bug http://bwsi.com.ph/. Try to click on the indicators until the carousel breaks.
Here's the js function to autoplay the carousel items :
window.setInterval(function() { 				$(".carousel").carousel('next') 			}, 4500);
The rest, I'm using the default config for carousel (fullwidth) and I've forced the height of the carousel to 500px.
Screenshots / Codepen

If you are doing autoplay, the correct way to implement something like this would be to refresh the interval whenever the user interacts with the carousel. So you might have to add an extra handlers to refresh your global interval when changing slides.

Oohh. I see. Thanks @acburst

Is there any plan of implementing an autoplay for carousel in the future?

Do you have an example of a handlers for it? I am using version 0.97 and it is difficult for me to update the whole project


Added capability to allow specifying an overwrite option in write image API

Looks good to me – thank you!


fix channel value race without threads

The error was due to changes in fakeFileSystem. registration server wasn't starting because it could not fine the socket file.
•E0308 15:42:36.193479    9680 uioPool.go:38] GetDeviceSpecs(): error getting vfio device file for device:
••••••••E0308 15:42:36.194735    9680 server.go:79] fakename device plugin unable to register with Kubelet : rpc error: code = Unavailable desc = all SubConns are in TransientFailure, latest connection error: connection error: desc = "transport: Error while dialing dial unix /tmp/sriov779172876/kubelet.sock: connect: no such file or directory"
•E0308 15:42:36.195561    9680 server.go:79] fakename device plugin unable to register with Kubelet : rpc error: code = Unknown desc = fake registering error
•••
•... Timeout [10.000 seconds]
Server
/root/go/src/github.com/intel/sriov-network-device-plugin/.gopath/src/github.com/intel/sriov-network-device-plugin/pkg/resources/server_test.go:18
resource server lifecycle
/root/go/src/github.com/intel/sriov-network-device-plugin/.gopath/src/github.com/intel/sriov-network-device-plugin/pkg/resources/server_test.go:85
starting, restarting and stopping the resource server
/root/go/src/github.com/intel/sriov-network-device-plugin/.gopath/src/github.com/intel/sriov-network-device-plugin/pkg/resources/server_test.go:106
should not fail and messages should be received on the channels [It]
/root/go/src/github.com/intel/sriov-network-device-plugin/.gopath/src/github.com/intel/sriov-network-device-plugin/pkg/resources/server_test.go:107
  Timed out

  /root/go/src/github.com/intel/sriov-network-device-plugin/.gopath/src/github.com/intel/sriov-network-device-plugin/pkg/resources/server_test.go:10

Locally it passes, but on travis it's failed 2 out of 3 times so far, but it is the second test (with watcher) that fails. I'll merge it and clean up a bit (as per comments below) and see if that works on Travis.
This message:
fakename device plugin unable to register with Kubelet : rpc error: code = Unavailable desc = all SubConns are in TransientFailure, latest connection error: connection error: desc = "transport: Error while dialing dial unix /tmp/sriov255653442/kubelet.sock: connect: no such file or directory"

comes from another test-case, which checks what happens if the plugin can't reach kubelet. There are dots •••• between this output and the stack trace meaning there were 4 tests in beetween that passed.
Setting up fake fs with /tmp dir will create something like this:
/tmp/sriov779172876/tmp. When defining files layout /tmp/sriovXYZ is always created first as a root dir and then inside that directory all defined files, symlinks etc. are being created.


When entering Screen ID, the check is too fast

When entering Screen ID when there is no screen.yml fail, the screen id check is too fast.
For example if the ID what i want to add is 7576, but there is also screen with ID 75, then the player adds screen 75 automatically when these two numbers are enterend. Cannot enter additional numbers 7 and 6.

now you shall press enter


UnicodeDecodeError when doing a test_package

I'm trying to adapt your conanfile.py for being able to have boost in conan for an isolated network. I also modified the conanfile.py in the test package. Both files are attached.
The differences between my version and yours are :

the sources are included in the recipe so I don't have to download boost and unzip it;
I have removed the python and bzip deps;

My configuration is the following : Win7, Visual Studio 10, Conan 1.6.0, Cmake 3.7.0
I get two differents errors depending on the use of short_paths = True or False.
The conan console with short_paths = True :

C:\Workspaces\Temp\boost_1_59_0>conan test_package
Exporting package recipe
boost/1.59.0@user/channel export: Copied 250 '' files
boost/1.59.0@user/channel export: Copied 7 '.graffle' files
[...]
boost/1.59.0@user/channel export: Copied 1 '.tst2' files: ifstream_line_iterator.tst2
boost/1.59.0@user/channel export: Copied 1 '.doxyfile' files: windows-paths-check.doxyfile
boost/1.59.0@user/channel: A new conanfile.py version was exported
boost/1.59.0@user/channel: Folder: C:\Conan\boost\1.59.0\user\channel\export
boost/1.59.0@user/channel: Package recipe modified in export, forcing source folder removal
boost/1.59.0@user/channel: Use the --keep-source, -k option to skip it
boost/1.59.0@user/channel: Removing 'source' folder, this can take a while for big packages
boost/1.59.0@user/channel: WARN: [Error 145] Le rpertoire nest pas vide: u'C:/.conan\xnm6jr\1'
WARN: config() has been deprecated. Use config_options and configure
Requirements
boost/1.59.0@user/channel from local
Packages
boost/1.59.0@user/channel:4c07c57fb6551a5dcaf87b7d4f715de1120ade0a
boost/1.59.0@user/channel: Installing package 4c07c57fb6551a5dcaf87b7d4f715de1120ade0a
boost/1.59.0@user/channel: WARN: Forced build from source
boost/1.59.0@user/channel: Building your package in C:/.conan\afinbr\1
boost/1.59.0@user/channel: WARN: Trying to remove dirty source folder
boost/1.59.0@user/channel: WARN: This can take a while for big packages
boost/1.59.0@user/channel: Configuring sources in C:/.conan\xnm6jr\1
Traceback (most recent call last):
File "", line 10, in
File "", line 6, in run
File "conan\conans\client\command.py", line 930, in main
File "conan\conans\client\command.py", line 832, in run
File "conan\conans\client\command.py", line 327, in test_package
File "conan\conans\client\manager.py", line 295, in install
File "conan\conans\client\installer.py", line 60, in install
File "conan\conans\client\installer.py", line 145, in _build
File "conan\conans\client\installer.py", line 198, in _build_node
File "conan\conans\client\installer.py", line 268, in _build_package
File "conan\conans\client\source.py", line 41, in config_source
File "shutil.py", line 189, in copytree
File "shutil.py", line 189, in copytree
File "shutil.py", line 189, in copytree
File "shutil.py", line 189, in copytree
File "shutil.py", line 183, in copytree
File "ntpath.py", line 85, in join
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe1 in position 12: ordinal not in range(128)
conan returned -1

With short_paths = False it goes a little further :

C:\Workspaces\Temp\boost_1_59_0>conan test_package
Exporting package recipe
boost/1.59.0@user/channel export: Copied 250 '' files
boost/1.59.0@user/channel export: Copied 7 '.graffle' files
[...]
boost/1.59.0@user/channel export: Copied 1 '.doxyfile' files: windows-paths-check.doxyfile
boost/1.59.0@user/channel: A new conanfile.py version was exported
boost/1.59.0@user/channel: Folder: C:\Conan\boost\1.59.0\user\channel\export
boost/1.59.0@user/channel: Package recipe modified in export, forcing source folder removal
boost/1.59.0@user/channel: Use the --keep-source, -k option to skip it
boost/1.59.0@user/channel: Removing 'source' folder, this can take a while for big packages
WARN: config() has been deprecated. Use config_options and configure
Requirements
boost/1.59.0@user/channel from local
Packages
boost/1.59.0@user/channel:85f780d0530411a64b0be4407b381706014b445d
boost/1.59.0@user/channel: Installing package 85f780d0530411a64b0be4407b381706014b445d
boost/1.59.0@user/channel: WARN: Package doesn't have a remote defined. Probably created locally and not uploaded
boost/1.59.0@user/channel: Building your package in C:\Conan\boost\1.59.0\user\channel\build\85f780d0530411a64b0be4407b381706014b445d
boost/1.59.0@user/channel: Configuring sources in C:\Conan\boost\1.59.0\user\channel\source
boost/1.59.0@user/channel: Copying sources to build folder
boost/1.59.0@user/channel: WARN: Filename too long, file excluded: C:\Conan\boost\1.59.0\user\channel\build\85f780d0530411a64b0be4407b381706014b445d\libs\geometry\doc\html\geometry\reference\spatial_indexes\boost__geometry__index__rtree\rtree_parameters_type_const____indexable_getter_const____value_equal_const____a
llocator_type_const___.html
boost/1.59.0@user/channel: Generated txt created conanbuildinfo.txt
Building Boost.Build engine
Bootstrapping is done. To build, run:
.\b2

To adjust configuration, edit 'project-config.jam'.
Further information:
- Command line help:
.\b2 --help

- Getting started guide:
http://boost.org/more/getting_started/windows.html

- Boost.Build documentation:
http://www.boost.org/build/doc/html/index.html

boost/1.59.0@user/channel: WARN: b2 toolset=msvc-10.0 link=static runtime-link=shared variant=release address-model=64  -j4
Performing configuration checks
- 32-bit                   : yes
- arm                      : no
- mips1                    : no
- power                    : no
- sparc                    : no
- x86                      : yes

Building the Boost C++ Libraries.
- symlinks supported       : no
- junctions supported      : yes
- hardlinks supported      : yes
- has_icu builds           : no

warning: Graph library does not contain MPI-based parallel components.
note: to enable them, add "using mpi ;" to your user-config.jam
- zlib                     : no
- iconv (libc)             : no
- iconv (separate)         : no
- icu                      : no
- icu (lib64)              : no
- compiler-supports-visibility : no
- message-compiler         : yes
- compiler-supports-ssse3  : yes
- compiler-supports-avx2   : no
- gcc visibility           : no
- long double support      : yes
warning: skipping optional Message Passing Interface (MPI) library.
note: to enable MPI support, add "using mpi ;" to user-config.jam.
note: to suppress this message, pass "--without-mpi" to bjam.
note: otherwise, you can safely ignore this message.
warning: No python installation configured and autoconfiguration
note: failed.  See http://www.boost.org/libs/python/doc/building.html
note: for configuration instructions or pass --without-python to
note: suppress this message and silently skip all Boost.Python targets
Component configuration:
- atomic                   : building
- chrono                   : building
- container                : building
- context                  : building
- coroutine                : building
- coroutine2               : building
- date_time                : building
- exception                : building
- filesystem               : building
- graph                    : building
- graph_parallel           : building
- iostreams                : building
- locale                   : building
- log                      : building
- math                     : building
- mpi                      : building
- program_options          : building
- python                   : building
- random                   : building
- regex                    : building
- serialization            : building
- signals                  : building
- system                   : building
- test                     : building
- thread                   : building
- timer                    : building
- wave                     : building

...patience...
...patience...
...patience...
...patience...
...found 11960 targets...
...updating 578 targets...
common.mkdir stage
[...]
msvc.archive bin.v2\libs\wave\build\msvc-10.0\release\address-model-64\link-static\threading-multi\libboost_wave-vc100-mt-1_59.lib
common.copy stage\lib\libboost_wave-vc100-mt-1_59.lib
bin.v2\libs\wave\build\msvc-10.0\release\address-model-64\link-static\threading-multi\libboost_wave-vc100-mt-1_59.lib
1 fichier(s) copié(s).
...updated 578 targets...
The Boost C++ Libraries were successfully built!
The following directory should be added to compiler include paths:
C:\Conan\boost\1.59.0\user\channel\build\85f780d0530411a64b0be4407b381706014b445d

The following directory should be added to linker library paths:
C:\Conan\boost\1.59.0\user\channel\build\85f780d0530411a64b0be4407b381706014b445d\stage\lib

boost/1.59.0@user/channel: Package '85f780d0530411a64b0be4407b381706014b445d' built
boost/1.59.0@user/channel: Build folder C:\Conan\boost\1.59.0\user\channel\build\85f780d0530411a64b0be4407b381706014b445d
Traceback (most recent call last):
File "", line 10, in
File "", line 6, in run
File "conan\conans\client\command.py", line 930, in main
File "conan\conans\client\command.py", line 832, in run
File "conan\conans\client\command.py", line 327, in test_package
File "conan\conans\client\manager.py", line 295, in install
File "conan\conans\client\installer.py", line 60, in install
File "conan\conans\client\installer.py", line 145, in _build
File "conan\conans\client\installer.py", line 201, in _build_node
File "conan\conans\model\manifest.py", line 28, in summary_hash
File "conan\conans\util\files.py", line 41, in md5
UnicodeEncodeError: 'ascii' codec can't encode character u'\xe1' in position 3040070: ordinal not in range(128)
conan returned -1

Any idea of what could cause the issue ? From what I understand some characters in paths are not well treated either for copying the sources or computing the hash.
If this has nothing to do with the conanfile.py itself I would gladly transfer this issue in the conan repo :)
conanfile.py.txt
conanfile_test.py.txt

Some more info :

When I do a conan build, it works fine
When I do a cd mypackage && conan package .. works fine too

I mixed up the conan version & the boost version :

I had two conan versions installed and the 1.6.0 was taking over the 1.6.1 (due to my %PATH%).
I used your conanfile.py for boost 1.60.0 whereas I was manipulating boost 1.59.0 and moreover keeping boost sources in a zip file ease the use as conan don't have to copy each file in the local store (had a problem with long path name when copying into the local store).

Everything works fine from my side now.
You can close this issue.

Thanks, yeah, the recipe from 1.59.0 is a little outdated and could have issues already fixed in 1.60
If you used 1.60 to create a 1.59 recipe just be careful with the library list declared in the package_info method. They change between those versions.


Add demo onboarding application

Signed-off-by: Moti Asayag<EMAIL_ADDRESS>rh-pre-commit.version: 2.2.0
rh-pre-commit.check-secrets: ENABLED

moved to workflow-examples instead.


TextField focus effect in FindNavigatorSearchBar

Description
Add TextField focus effect in Find Navigation Search Bar

Checklist

[x] I read and understood the contributing guide as well as the code of conduct
[x] My changes generate no new warnings
[x] My code builds and runs on my machine
[x] I documented my code
[x] Review requested

Screenshots
https://user-images.githubusercontent.com/20476002/165965037-96ec22d8-1268-419c-a5d0-891f068dacfb.mov

How does this look in dark mode? Can you post a video?

@austincondiff : Done.

@austincondiff : I know, this is the best I was able to do.
My branch is open so anyone who finds a better solution can make changes.

If we can achieve this effect, we need to make this a reusable component. So we can use it in other navigators in the sidebar.


Sending input with a + crashes

Maybe an url escaping issue? I didn't dig much deeper but wanted to report.
$ ollama run llama2:13b
>>> I cannot send input with a + in it
Error: Post "http://<IP_ADDRESS>:11434/api/generate": EOF

Hey @benr75, may I ask how much memory you have on your mac? One of the most common reasons for this error is due to out-of-memory.
I just tried using + works

64GB, all other queries worked. Let me know if there are some logs to look at or more info I can provide.

This is less likely an issue with + and more likely stability related. There are a lot of stability improvements release in 0.0.13 which should address this and other Post "http://<IP_ADDRESS>:11434/api/generate": EOF issues

@mxyng thx for looking into this. Testing on the new version is working with the provided prompt. Closing.


fate-material placeholder doesn't seem to work

Hi,
I've tried your fate-material and the placeholder doesn't seem to work there.
I've used it like that:
 <fate-material  [(ngModel)]="storyItem.data"
                           [placeholder]="'hello'" ></fate-material>

storyItem.data equals to '' (Empty string) on rendering.
After editor rendered it appears to be empty.
Am I doing something wrong?

I can't seem to reproduce this effect, which platform/browser do you see this occurring on? Would you be able to produce a reduced test case?

Hey,
I have the same issue. I use Mac OS and the placeholder doesn't work in Safari and Chrome as well.

@ivanlucky22 and @pichukov, do you experience this issue when visiting the project website (and clearing the content of the editors)? Or is it something that appears only when you use the editor in your own project?
If it’s on your own project maybe you cloud send me your package.json, cloud help to figure out if it’s an issue with the version of angular...

when I took a look at the fate-material selector template within the package, placeholder was not being set. Whereas for the the bootstrap template it was set. When I added [placeholder]="placeholder", it worked
            selector: 'fate-material',
            template: `<fate-input customClass="" [uiId]="uiId" [row]="row" [ngModel]="passthrough" (ngModelChange)="onChange($event)"></fate-input>

That's very strange @sandiphob, I normally material should take care of the placeholder, you shouldn't need to use the "native" one provided to the input element. Which version of material are you using?
I'll add the placeholder passthrough and make sure that it doesn't causes any other issues.

Hi @onaluf , I am using Material 7.0.1

Ok so I tried to use the placeholder attribute but it creates glitches when both itself and the "floating" version of material are both used.
I guess it may have to do with how MAT_LABEL_GLOBAL_OPTIONS is configured. I'll try to make both play nice together, maybe add the option to either user labels or placeholder there.

I wonder about something  @sandiphob, @pichukov and @ivanlucky22: do you wrap fate-material in a mat-form-field like so:
<mat-form-field>
   <fate-material placeholder="Some text"></fate-material>
</mat-form-field>

If not could you try to see if it solves your problem ?

Thank you @onaluf
I can confirm that placeholder works when wrapping mat-form-field around fate-material.


Fix label ownership apiversion bug

[x] Fix label ownership bug where if the apiversion differs it won't
change the controller owner.
[x] Add test case to ensure bug doesn't come back
[x] Change unit tests to be more easily expanded

@staebler I believe I've addressed your feedback. PTAL.

/hold

I tested that the new logic does indeed set the apiversion correctly on hive objects that we're reconciling.
It also was showing the correct behavior on objects that were already set correctly (no modifications).
/hold cancel


Fix broken links of Callback

keras/engine/training.py:2163:81: E501 line too long (95 > 80 characters)

Please fix

Fixed!

The issue appears to be fixed at head. Thanks!


Update loader-utils Package to Non-vulnerable Version

This PR is aimed at upgrading the loader-utils package to a version that is not vulnerable. As part of the PR, the storybook package has been updated to the latest version, and the MSW script has also been updated.

Thanks! For reference, this was the alert that is now fixed, and other should be as well (54-57).


Rebalancing issues with slow consumers

Versions
Please specify real version numbers or git SHAs, not just "Latest" since that changes fairly regularly.

Sarama
Kafka
Go

a20d26723487482b4f1af8b8537657a5ba72ae37 (1.25.0)
*
1.12

Problem Description
We've identified three issues that affect rebalancing when the consumer takes tens of seconds or minutes to process an event.
1: Consumers must deplete Messages() before they can be shut down
Sarama asks for the following:
// Please note, that once a rebalance is triggered, sessions must be completed within
// Config.Consumer.Group.Rebalance.Timeout. This means that ConsumeClaim() functions must exit
// as quickly as possible to allow time for Cleanup() and the final offset commit. If the timeout
// is exceeded, the consumer will be removed from the group by Kafka, which will cause offset
// commit failures.

This is a reasonable request since the Sarama waits for all ConsumeClaim()s to return before re-establishing a session and joining the new generation of the consumer group. However, it communicates this by close()ing the buffered Messages channel - not something the consumer can  act upon immediately, and even worse so without first reading all the messages in the channel.
If it takes 3 minutes to handle a single message (which is a viable scenario) the consumers have to set their ChannelBufferSize to 1 and their Rebalance.Timeout to 6 minutes + some slack (since the consumer will at worst have to handle the message it is handling and the one in the channel.
Proposed solution
Add:
// StopConsuming forces an immediate stop of consumption, in addition to closing the Messages() channel
StopConsuming() <-chan struct{}

To ConsumerGroupClaim. A consumer can then run their message handling in a background go routine and wait for completion of handling or StopConsuming. They can then decide whether to terminate processing immediately, wait a grace period or complete the current processing of the event (without having to deplete the Messages channel to understand that it should exit).
2: The first few fetches from Kafka may only fetch data from one or two partitions, starving the rest for a very long time (depending on message size / processing time)
Once a member joins the consumer groups and receives its partitions, they are fed into the "subscription manager" from different go routines. The subscription manager then performs batching and executes a fetch for all the partitions. However, it seems like the batching logic in subscriptionManager is faulty, perhaps assuming that case: order prioritizes which case should be handled when all are signaled which is not the case, according to the go docs (https://golang.org/ref/spec#Select_statements):
If one or more of the communications can proceed, a single one that can proceed is chosen via a uniform pseudo-random selection. Otherwise, if there is a default case, that case is chosen. If there is no default case, the "select" statement blocks until at least one of the communications can proceed.

For example - if you receive 64 partitions, each will be handled in their own go routine which sends the partition information to the bc.input channel. After an iteration there is a race between case event, ok := <-bc.input which will batch the request and case bc.newSubscriptions <- buffer which will trigger an immediate fetch of the 1 or 2 partitions that made it into the batch.
This issue only really affects slow consumers with short messages. If the condition happens with 1 partition being in the batch (even though 63 extra partitions have been claimed but didn't make it into the batch) the fetch will ask for 1MB (by default) of messages from that single partition. If the messages are only a few bytes long and processing time is minutes, you will not perform another fetch for hours.
Proposed solution
Replace the subscriptionManager logic with a delay based one here:
https://github.com/nuclio/nuclio/blob/1.1.x/vendor/github.com/Shopify/sarama/consumer.go#L741
3: Heartbeat timer is stopped once rebalance is detected, causing session to time out if tear down is long
The heartbeat loop exits immediately when a rebalance error is returned:
https://github.com/Shopify/sarama/blob/master/consumer_group.go#L760
If the consumers take a long time to tear down, the member will be ejected from the group due to session timeout.
Proposed solution
Stop sending heartbeats only AFTER the session has completed tearing down (I think this was the intent).
Summary
We can contribute fixes to these issues above if the solutions described here are agreed upon (we're currently running them through tests).

Regarding 1, I encountered a similar problem in a fairly common use case:
A Consumer doing some processing, then writing the results to a remote DB (AWS DynamoDB for example).
While most of the consumed items should be processed really fast, it is a fair assumption that some of them might fail due to throttling/network issues within the remote DB. In such a case, handling those failures using an exponential backoff retry is not possible as it might take more than Rebalance.Timeout to achieve a successful write.

I've come across this exact same problem. I'm building a consumer that will write records from a topic to a database, so the order in which messages are processed are really important. Since it depends on an external service, there is a function that retries failures with an exponential backoff.
But I was able to create scenario where, because of timeouts, the ordering guarantees were lost and the final state ended up inconsistent:
Scenario:

The topic has two partitions [0,1], with the messages [0a, 0b, 0c] and  [1a, 1b, 1c], respectively.
Worker W1 starts consuming, gets both partitions assigned, but for network reasons, processing takes longer than Session.Timeout
Worker W2 joins the group
Worker W1 receives ErrRebalanceInProgress, heartbeat stops
After waiting Session.Timeout, the broker assumes W1 is dead and removes it from the group
W2 receives both partitions and is able to process all the messages quickly
Database current state: [0 -> 0c]; [1-> 1c] (desired final state)
W1 recovers and writes old data to the database
Database current state: [0 -> 0a]; [1-> 1a] (Invalid state)

W1 eventually stops, since it lost its partitions, but because the heartbeat stopped, W2 processed the same data as W1, which led to an inconsistent sate.  If the heartbeat hadn't stopped, W2 wouldn't have been able to join until Rebalance.Timeout, which wouldn't cause this problem.
Of course, the first issue (Consumers must deplete Messages() before they can be shut down) is still problematic. Since there is no way of knowing when the consumer lost it's partitions, it is impossible to know when it should stop trying to insert the data in the database.

If anyone is in a dire state, we maintain a fork containing a branch with fixes to all the above here: https://github.com/iguazio/sarama/tree/slow-consumer (it's synced to 1.25.0). As the original issue states, we're still waiting for a dev to comment/approve before we invest the effort in a PR.

We've come across this exact same problem and your fork @pavius helps to get the problem solved for now.
The  <-claim.StopConsuming() read really helps to stop the processing by consumer handler in ConsumeClaim method.
Our scenario has messages that take about 1minute to be processed and committed. Without the StopConsuming chan we were getting many messages processed more than once, after the rebalance by scale the service from 1 instance to 10, for example.
The exact steps to reproduce our case were:

We have a topic called foo with 10 partitions;
start service with replicas=1 and consumer group = foo;
everything is processing ok;
we scale the service to 10 replicas for the same consumer group = foo;
the rebalance starts at server;
the first replica still processing some message;
after commit and finish the process in ConsumeClaim (range of claim.Messages()) the consumer does not stop because there is one goroutine for each partition;
the new service replicas start to process and some messages are duplicated (some times other rebalances happens and increase the number of duplicated messages).
Now with the @pavius fork we are able to use the StopConsuming chan to process or stop early if rebalance occurs.

Example using claim.StopConsuming chan:
func (c *consumerHandler) ConsumeClaim(session sarama.ConsumerGroupSession, claim sarama.ConsumerGroupClaim) error {
    fmt.Println("starting a consumeClaim for ", session.MemberID())
    defer fmt.Println("consumeClaim finish for ", session.MemberID())
    for {
        select {
        case msg, ok := <-claim.Messages():
            if !ok {
                return nil
            }
            wait := make(chan struct{}, 1)
            m := Message{
                session:     session,
                consumerMsg: msg,
                wait:        wait,
            }
            fmt.Println("received message", session.MemberID(), string(m.Key))
            select {
            case c.messages <- m: // we are processing outside, using a non buffered chan
                fmt.Println("processing...", session.MemberID(), string(m.Key))
                <-wait
                fmt.Println("finish msg", session.MemberID(), string(m.Key))
            case <-claim.StopConsuming():
                fmt.Println("stopping by stopConsuming - inside", session.MemberID(), string(m.Key))
                return nil
            }
            fmt.Println("end of process message", session.MemberID(), string(m.Key))
        case <-claim.StopConsuming():
            fmt.Println("stopping by stopConsuming - outside", session.MemberID())
            return nil
        }
    }
}

Note:
since our messages takes up to 1 minute to process we need to increase some timeout so everything is ok. I'll expose them here to help someone with similar issues:
func ConfigsByProcessingTime(maxProcessingTime time.Duration) *sarama.Config {
    conf := sarama.NewConfig()
    conf.Version = sarama.V2_6_0_0
    conf.Net.MaxOpenRequests = 5
    conf.Consumer.Return.Errors = true
    conf.Consumer.Offsets.Initial = sarama.OffsetOldest
    const partitions = 10
    conf.ChannelBufferSize = 0
    conf.Consumer.MaxProcessingTime = maxProcessingTime * partitions
    conf.Consumer.Group.Rebalance.Timeout = (maxProcessingTime * partitions) + 5*time.Second
    conf.Net.ReadTimeout = (maxProcessingTime * partitions) + 5*time.Second
    conf.Net.WriteTimeout = (maxProcessingTime * partitions) + 5*time.Second
    conf.Net.DialTimeout = (maxProcessingTime * partitions) + 5*time.Second
    return conf
}
``

Hey guys, I just ran into the same problem and I thought you can check sess.Context().Done() to known if you should stop the consumer
func (h *consumerGroupHandler) ConsumeClaim(sess sarama.ConsumerGroupSession, claim sarama.ConsumerGroupClaim) error {
	wait := make(chan error)
	for msg := range claim.Messages() {
		var m broker.Message
		if err := h.kopts.Codec.Unmarshal(msg.Value, &m); err != nil {
			continue
		}
		go func() {
			e := h.handler(&publication{
				m:    &m,
				t:    msg.Topic,
				km:   msg,
				cg:   h.cg,
				sess: sess,
			})
			wait <- e
		}()
		select {
			case <- sess.Context().Done():
				break
			case err := <- wait:
				if err == nil && h.subopts.AutoAck {
					sess.MarkMessage(msg, "")
				}
		}
	}
	return nil
}

https://github.com/Shopify/sarama/issues/1897#issuecomment-922495227

@pavius thanks for this excellent write up. For some reason I hadn't been aware of this issue until I stumbled upon it just now. I've been browsing over your changes via the old nuclio/nuclio vendor dir and your iguazio/sarama fork and I think they look reasonable. If you'd be willing to put together separate PRs for the three cases you identified that would be great. Particularly point 3. seems a quick and easy bugfix that we should get merged sooner rather than later, and your proposed change for 2. look reasonable to me, although I'd like to exercise it with some unittesting and sustained load somewhere myself to be sure.

Is there any new update for issue 1?

The suggested changes were merged into Sarama. Closing this issue as resolved


Add persistence for target.id property of the W3CTextAnnotation on parsing/serialization

Issue
According to W3C Anno spec, the target can have the id prop:
{
  "@context": "http://www.w3.org/ns/anno.jsonld",
  "id": "http://example.org/anno18",
  "type": "Annotation",
  "body": {},
  "target": {
    "id": "http://example.org/photo1",
    "type": "Image"
  }
}

This also matches the expectation of my application to assign specific IDs to the targets depending on which element the annotation was done.
Unfortunately, using the W3CTextFormat adapter I noticed that the assigned id is getting lost after parsing and won't be serialized back 👀

Possible Solution
I see the next ids connection between the internal and W3C models:

Internal

W3C

target.selector[X].id
⇔
target[X].id

So on parsing, we can assign the targets' id to selector entries and do the inverse process on serialization

Additionally, I think that not assigning the id to TextAnnotationTarget in the internal model is acceptable. Because they are already identified with the .annotation property due to the 1-1 connection

And... Maybe it's worth assigning the ids to the selectors by default in the rangeToSelector function... I'm on the fence about that idea, because those ids won't be used by the library itself 🤷🏻♂

Just briefly: IDs need to stay on the targets, because this otherwise breaks all our realtime scenarios.

But there are no ids assigned to the targets now in the system 🤔

Ah, wait - there are no ids on the target in the core model.

So is this a change that just affects the W3C adapter? (Sorry, don't have time to look into the details myself right now.)

Yep, only the adapter and the W3CAnnotationTarget type:
export interface W3CAnnotationTarget {

  id?: string;

  source: string;

  selector?: AbstractW3CSelector;

}


Support for Bfloat16 in A100s

A100 GPUs support bfloat16, can this be enabled on FasterTransformer?

Currently not, but we have such roadmap.
Besides, we still not observe accuracy regression when we use BF16 model on FT with FP16 directly.

We are supporting the bfloat16 on GPT model in latest v5.0 release.

Nice! Is it bfloat16 by default for gpt models?

nvm, found it

Set the "data_type" parameter of config.pbtxt to "bf16".


Scroll Window

The table is so large that I must zoom out to 33% on my browser to see the bottom. At this zoom I can not see the letters. My screen is 1920x1080. I don't know how I would be able to use this on my laptop.
There is no containing window or scroll bars. Having the ability to scroll would be a great benefit. Is there a way to contain it in a frame or limit the column sizes by the browser window?

@NLandisNeo Did you find any solutions for this issue? I'm facing the same problem where i can't scroll down and have to zoom out to 25% and still not everything is visible.


Responsive desktop version

The basic concept of responsive design was followed to develop the portfolio website for both mobile and desktop version.

The changes I have made after the review:

I have uploaded the backgrounds in the images folder for which the header and contact section didn't have background.
I have maintained the mobile first design procedure.
I didn't change the about myself section as in the Figma design the Languages are kept on the right side in the desktop version. I have attached a screenshot.

If needed I can change the about myself section as mentioned.


Release a CPU only wheel on PyPI

I am using XGBoost on CPU only and do not require the NCCL library
Until before XGBoost v2.1.0 I managed to installed the lib with pip and --no-binay option to compile the package without NCCL
Since v2.1.0, nvidia-nccl-cu12 is installed from PyPI and its huge 190MB weight makes my app not fit on my Heroku server anymore.
Is there another way to avoid the installation of this dependency than to edit the pyproject.toml file from XGBoost locally and install it locally ?

Perhaps pip install . --no-deps?

Not sure if toml project allows a different dependencies profile that overrides the default one. So far the optional dependency configuration only allows additional dependencies.

Alternatively, you can consider conda, which has greater flexibility but the version update is a bit behind.

Perhaps pip install . --no-deps?

Not ideal as I have other packages in my requirements file for which I want to keep the dependencies, unless I create a specific Heroku buildpack for XGBoost install which sounds a bit overkill.
I'd also have to check XGBoost's pyproject.toml anyway for any shenanigans every time I update its version
Why not moving "nvidia-nccl-cu12 ; platform_system == 'Linux' and platform_machine != 'aarch64'" down into [project.optional-dependencies] ?
Having NCCL as a required dependency really penalizes CPU users

It might be better to release a CPU-only package to PyPI, e.g. xgboost-cpu, for space-constrained environments.

I will leave it to @hcho to decide whether it's needed to maintain a fleet of packages (arm linux, x86 linux, windows, macos, arm macos, sdist) for a new pypi project.
It's a bit odd that pep517 has a way to turn on features in a package by having optional dependencies, but there's no way to turn off a feature.

@trivialfis For xgboost-cpu, let's just maintain x86 linux and Windows. No need to complicate things.

Thank you for looking into this @hcho3 . Please note that the feature has been requested for Python https://discuss.python.org/t/help-packaging-optional-application-features-using-extras/14074/18 before and there are lots of discussions. It's not surprising since packages all want to provide as many features as possible in the default build. If we were to make a new project, we should prepare for deprecating/archiving that project as well if pep517 were to support these types of optional dependencies in the future.

If we were to make a new project, we should prepare for deprecating/archiving that project as well if pep517 were to support these types of optional dependencies in the future.

I agree. We can archive xgboost-cpu once the Python ecosystem provides a way to optionally opt out of optional dependencies.

@rbeilvert As a workaround, you can add the following URL to your requirements.txt:
https://files.pythonhosted.org/packages/5e/30/b4b1c071964acefeda3faade5b86e1bf2f428c5713404b212941942e835d/xgboost-2.1.0-py3-none-manylinux2014_x86_64.whl

This will install a variant of XGBoost 2.1.0 that doesn't require NCCL. Our plan is to upload this package under the name xgboost-cpu so that it's easy to locate and install.

@trivialfis It turns out that I already did the necessary work to repackage XGBoost without GPU code in #10478 and #10483. So all it takes is the renaming of the package to xgboost-cpu.

It still needs to remove NCCL from pyproject right?

It still needs to remove NCCL from pyproject right?

This line removes NCCL from pyproject: https://github.com/dmlc/xgboost/blob/e9fbce9791469571481b59743bedf9d7d2a481a5/tests/buildkite/build-manylinux2014-x86_64.sh#L13

The xgboost-cpu package is now available on PyPI: https://pypi.org/project/xgboost-cpu/
You can install it with pip install xgboost-cpu.
@rbeilvert @agossard-gpfw @yassinezaim-cp @kmodry @jmarichez @NicolasLegeay @mathieu-luciani @gtramoy @CyprienDuv


showing squares instead of Hebrew characters

I tried opening a non english website does seem to open. do you have an eg which can be replicated.


[cmd] Add support for sarif export in parser cmd

Hello,
This PR allows to run CodeChecker parse -e sarif ./results -o sarif.json.
This PR only uses the convert function already present in Sarif Parser (https://github.com/Ericsson/codechecker/blob/master/tools/report-converter/codechecker_report_converter/report/parser/sarif.py#L297) and exposes it to the parser cmd.
This would close #4036 I think (I did not test the output on Github)

Using https://sarifweb.azurewebsites.net/Validation it seems that the SARIF output file have some issues that need to be fixed for Github integration.
Do you think another PR should be created to support Github integration ?

Linting issue are fixed. CI can run again.

@bruntib @vodorok ?


Support using GITHUB_TOKEN

Hey!
While I was working on #5 I got the following error a bunch of times:
Error fetching latest release: https://api.github.com/repos/orhun/menyoki/releases/tags/v1.0.0: status code 403

This is due to GitHub rate-limiting unauthorized requests, see: https://docs.github.com/en/rest/overview/resources-in-the-rest-api#rate-limiting
I solved this by generating a new personal access token and using basic HTTP auth for GET requests internally in dra. (i.e. I set a auth header for ureq::get calls)
So I'd like to propose a change about:

set "Authorization" header for GET requests if GITHUB_TOKEN is defined as an environment variable.

I also thought about adding a new argument such as --github-token but I'm not a big proponent of exposing secrets to the command line like that. Either way, let me know what you think and I'm happy to help implement this 🐻

Hi!
Yes. I was thinking about something like this, and currently working on it.
This would also allow to download assets from private repository which was my goal :)
I'll update this issue when it will be implemented

A quick update: I pushed the changes to main, and will be releasing a new version 0.2.2 in a couple of minutes :)