Input features of imdb_fasttext example neither BOW nor bag-of-ngrams

I have read the original paper and I believe it proposes that the input features to the model are in the form of bag-of-words/bag-of-ngrams. On the other hand the imdb_fasttext implementation appears to be using sequences of word tokens.
Am I missing something here?

I believe you are correct.
Fasttext uses character-level n-grams, which means that, for instance, for the word "where", the resulting vector will be the average of the following vectors:
[<wh, whe, her, ere, re>, ]
This is not what is being done in https://github.com/keras-team/keras/blob/master/examples/imdb_fasttext.py
Instead, in the example, the n-grams are being computed at word-level.
The example should either be fixed or removed, as it is wrong.

Well I guess you can always tokenise at character level and compute the new n-grams.
To my understanding and since the data are not processed sequentially, the output of the embedding layer is equivalent to one that takes as input bags-of-ngrams. Correct me if I am wrong.

I guess that's correct, but the IMDB dataset that is used in the example has each comment defined as a list of integers, each of which mapping to a word - this can be seen here: https://github.com/keras-team/keras/blob/master/keras/datasets/imdb.py
Given this, I don't think it's really possible to take that dataset as it's defined and make the example work, since the step of computing the n-grams for each word would have had to be done on top of the "raw" dataset.
I might be misunderstanding something, however, and please correct me if I'm wrong as well.

Yes you are right, since the IMBD dataset provided with keras is already tokenised. The model works ok at a word level but it is not the same as the original fasttext version.


cli: add "manual invocation is unsupported" to internal commands

containerd publish is implementation-detail and we never support
compatibility of this command across releases.
So users should not be aware of this subcommand.
containerd --help no longer shows the help of this subcommand but
containerd publish --help can still be printed.
EDIT: I updated PR to just add "manual invocation is unsupported" to --help of internal commands
Signed-off-by: Akihiro Suda<EMAIL_ADDRESS>

Codecov Report

Merging #1780 into master will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #1780      +/-   ##
=========================================
+ Coverage   49.14%   49.2%   +0.05%
=========================================
  Files          77      77
  Lines        7705    7705
=========================================
+ Hits         3787    3791       +4
+ Misses       3275    3271       -4
  Partials      643     643

Flag
Coverage Δ

#linux
49.22% <ø> (ø)
:arrow_up:

#windows
45.01% <ø> (+0.06%)
:arrow_up:

Impacted Files
Coverage Δ

metadata/content.go
32% <0%> (+0.94%)
: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 9cb91f8...7d3b5d4. Read the comment docs.

I don't think we need to do this.  Other plugins and such can use this to publish events.  That is why the events API has a public publish endpoint

Plugins can still execute the command.
My intention is to hide this internal stuff from human users.
If we should not hide, I would propose rename this to containerd internal-util publish or split to containerd-shim-util so as to make sure this is not a daemon feature.

We discussed the other week and we would rather have subcommands on containerd than, reexec or separate binaries to keep the containerd package smaller but more explicit than re-exec

We need the same type of functionality for the windows work

If we are adding more internal subcommands, can we create a single containerd internal-util and move all internal subcommands under that? (containerd internal-util publish).
(If internal-util is too long, util or internal would be fine)
Also, if we are not going to hide these internal subcommands from human users, at least we should add a help string so that human users won't get confused, and we can freely change the command without considering compatibility:
diff --git a/cmd/containerd/publish.go b/cmd/containerd/publish.go
index b975c075..ef281138 100644
--- a/cmd/containerd/publish.go
+++ b/cmd/containerd/publish.go
@@ -20,7 +20,7 @@ import (

 var publishCommand = cli.Command{
        Name:  "publish",
-       Usage: "binary to publish events to containerd",
+       Usage: "binary to publish events to containerd (internal command; do not use)",
        Flags: []cli.Flag{
                cli.StringFlag{
                        Name:  "namespace",

But they can use it if they want, we don't have to hold ppls hands.

But they can use it if they want, we don't have to hold ppls hands.

Yes, they can use, but they should not blame us if we break the CLI.
So I updated PR to just mention "manual invocation is unsupported". PTAL.

AFAIK, the CLI isn't covered by our compatibility warranty. Maybe that should be made explicit in the documentation.
I'm not a use fan of the added message, don't want people to be scared away and not play around :)

Plus we have to support it anyways because we are depending on it, it has to be stable.  I don't think we need this change.

AFAIK, the CLI isn't covered by our compatibility warranty. Maybe that should be made explicit in the documentation.

Aren't we going to keep compatibility of the daemon mode of containerd (i.e. without  any subcommand)?
Without keeping compatibility of containerd, it is impossible for users to update containerd, because their init script will fail.

Aren't we going to keep compatibility of the daemon mode of containerd (i.e. without any subcommand)?
Without keeping compatibility of containerd, it is impossible for users to update containerd, because their init script will fail.

The init script comes with the package, and package manager should update it as needed upon version change

The init script comes with the package, and package manager should update it as needed upon version change

Users can supply custom flags, and it should be kept and functional after apt/dnf update

Those flags are usually in /etc/defaults/ and package manager usually have something to take care of those cases. Also, changelogs :)

Stabilized components are listed in RELEASES.md. There is no warranty, guarantee or support provided, as per the license, which supersedes the releases document. We only take measures to prevent breakage between versions on the stabilized components.
According to release document, the containerd command is out of scope for stabilization. However, in order to provide smooth updates, the behavior of this command will be preserved for internal purposes in the 1.0 release train, and likely longer. If users of events would like to ensure stability across versions, they should use the GRPC API directly. We may also add an equivalent functionality to ctr in a future release.
Given that, I think plastering "unsupported" all over the CLI sends an odd and confusing message.


Create CookieScrapper.py

Description: Cookie Extraction and Integration with BardCookies
This pull request introduces a significant enhancement to the BARD-API project by enabling the extraction of cookies from a user's web browser. These extracted cookies can then be passed through BardCookies for further processing and integration into the application.
Motivation and Context:
Cookies are essential for maintaining user sessions and storing user-specific data on web applications. The ability to extract and utilize these cookies within the BARD-API project opens up new possibilities for authentication, personalization, and improved user experiences.
This change addresses the following key objectives:
Cookie Extraction: Implement a mechanism to extract cookies from the user's browser, ensuring the secure and efficient retrieval of session-related data.
Integration with BardCookies: Enable the seamless passing of extracted cookies through BardCookies, allowing for easy integration with existing user sessions and authentication mechanisms.
Enhanced Testing: While test mocking remains a desirable future improvement, this pull request aims to lay the groundwork for testing the cookie extraction and integration functionality as thoroughly as possible. Detailed test cases and scenarios have been added to validate the changes.
Related Issue:
This pull request is related to Issue #XXX, where the need for cookie extraction and integration was discussed and identified as a valuable addition to the project.
How Has This Been Tested:
Comprehensive testing has been conducted to ensure the correctness and robustness of the cookie extraction and integration features. The testing environment includes various browsers and platforms to validate cross-browser compatibility.
Test Cases:
Cookie Extraction: Ensure that cookies are extracted correctly from the user's browser when making requests.
Integration with BardCookies: Verify that extracted cookies can be passed through BardCookies without loss or alteration.
Authentication and Session Handling: Test the impact of extracted cookies on user authentication and session management.
Edge Cases and Error Handling: Test scenarios where cookie extraction or integration might fail and implement appropriate error handling.
This description provides a more detailed overview of the changes, their motivation, and the testing approach. You can fill in the specific details based on your actual code changes and testing results.

If there are no further comments, I will proceed to close this Pull Request. Feel free to enhance the content or suggest new ideas at any time. Even minor contributions, such as typos, docstrings, or formatting, are always welcome.
Great effort and idea, but we've decided not to merge it due to some concerns.
Thank you.


Model stopped oscillating

The commit  071ea752c85678c4dc394dbf683cd78eda693839 stopped the model from oscillating.  In this commit, "More literature fixes", the value for k0 was changed from 50 to 4.82, which is a surprisingly large change.  I suspect that 48.2 might have been intended.  Changing the value of k0 to 4.8 restores the oscilation, as shown below.

Now I will try to do a commit to fix this.

Correct!  You even guessed the correct fix.


Support virtual list/scrollable list

Is there any way the comboboxfield shows a scrollable list or virtual list?

Unfortunately no, not at this time. I'd recommend using downshift to implement it.


[ENH] Patch translations

I've made this work for Sphinx-proof, and the patch should also apply here.
This is related to #13
Currently, this patch only contains french translations.

Thanks @vincentxavier. I am about to merge #37 which is a pretty extensive rewrite of this package (with the aim to make it more maintainable over the long term) and use more sphinx internals. It is passing all test cases with some minor changes to the title styles. Would love to work with you to fix up #42 once this is merged.

Hello,
I've seen this extensibe PR. I'll watch when you merge it, and I'll try te
rebase my work on the new main branch.
Thanks
Le 06 déc. à 03:15 mmcky a écrit

Thanks @vincentxavier. I am about to merge #37 which is a pretty extensive rewrite of this package (with the aim to make it more maintainable over the long term) and use more sphinx internals. It is passing all test cases with some minor changes to the title styles. Would love to work with you to fix up #42 once this is merged.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/executablebooks/sphinx-exercise/pull/42#issuecomment-986377145

--
Vincent-Xavier JUMEL Id: 0xBC8C2BAB14ABB3F2 https://blog.thetys-retz.net
Société Libre, Logiciel Libre http://www.april.org/adherer
Parinux, logiciel libre à Paris : http://www.parinux.org

@vincentxavier I saw that you had made some commits so I have enabled the CI to run for you.


Wasm tiny_keccak improved by 10% while call heavy benchmarks and in particular host function calls have regressed significantly in both native and Wasm.


[bug]: RPC ERROR: Disk Management Error Device wipe error, offset 0. Cannot wipe header on device /dev/EMBASSY_**/main. Restart Embassy to try again.

Prerequisites

[X] I have searched for existing issues that already report this problem.

embassyOS Version
0.3.3
Device
Laptop/Desktop
Device OS
MacOS
Device OS Version
13.2
Browser
Safari
Browser Version
16.3
Current Behavior
Trying to setup Start9/Embassy but I keep getting disk errors:

[bug]: RPC ERROR: Disk Management Error Device wipe error, offset 0. Cannot wipe header on device /dev/EMBASSY_**/main. Restart Embassy to try again.

The problem only occurs with my MX500 1TB SSD (M3CR033). I tested it what an BX500 120GB and the problem does not occur with that drive. The SSD is working just fine. Tested the drive with the software of Crucial. It has not errors and the firmware is also up-to-date.
I tried two controllers. A jMicron and Sabrent. In both cases the BX500 works, but the MX500 doesn't. I formatted the drive multiple times. Even re-partitioned the drive as well. No luck. Al formatting and repartitioning is done on MacOS. The drive check is done on Windows.
Also with Raspiblitz, Umbrel and Sentinel the drive works fine and does its job.
My hardware:
Raspberry Pi 4 + 4GB memory
Official Raspberry Pi Powersupply
SSD: Crucial MX500 1TB
Expected Behavior
I would expect the drive to be formatted correctly and work like it does with other software too.
Steps to Reproduce
Install Embassy OS on an SD card
Add a Crucial MX500 1TB ssd to the USB 3 port of the pi
Use the official power supply of a raspberry pi
Try to do the setup of Embassy OS. After selecting the drive and trying to format the drive, the system gives errors.
Anything else?
I do notice that it takes a really long time (couple of minutes) for the drive to show up as the data storage drive during setup. I don't know if this is normal behavior.

I recommend using a larger power supply. The official pi power supply is insufficient for full speed use of an external drive.

Not reported in a while, likely Pi power supply-related


Add "Новый подкаст (2)_после правок.final.doc" podcast

What does this PR do?
Add Resource(s) | Remove Resource(s) | Add info | Improve Repo
For resources
Description
Why is this valuable (or not)
How do we know it's really free?
For book lists, is it a book?
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

Thanks!


Language support

Does it support spanish?

Not yet. It supports only English. In the future, I will look to provide support for multiple languages.


nixos/fluentd: add support for config reload

We need config reload, because we generate some aspects of fluentd config automatically.


How to use in LInux

how could one use shary on Linux?
I've tried:
cargo build
but it failed with Error:
Some errors occurred: E0432, E0658.
Here is a pastebin with full error

@vk0xOrg Hi. This program depends on the latest Rust stable release (1.41). Please install rustc using rustup instead of the one provided by the distro.

Works, thank you.


DMM Drivers - NPLC and Min/Max/Avg support

Add ability to configure NPLC on a driver level in both the Fluke and Keithley DMM drivers
Added a function to have the drivers sample x times and then return the minimum, maximum and average of the samples

@clint-lawrence @jcollins1983 Are you two happy to proceed?

Yep, my earlier comment have all been address.

Update Release Notes please!


Feedback for 06/04/17

Recurring task
Change date for recurring task, the next date created should not change after editing the date of the task.
Should let user change that instance or all instance.
Should perhaps consider implementing abstraction occurrence pattern design.
Test bugs
Try find bugs and fix them when ever possible.
Test scripts
Come out with test scripts.

Acknowledged

Addressed


CP-SAT Occasional Segmentation Fault

What version of OR-Tools and what language are you using?
Version: 9.11.4210
Language: Python
Which solver are you using (e.g. CP-SAT, Routing Solver, GLOP, BOP, Gurobi)
CP-SAT
What operating system (Linux, Windows, ...) and version?
Linux; Ubuntu 22.04
What did you do?
Run the basic python script to load and run the model
What did you expect to see
Model runs without issue; will converge after a few minutes to an objective around 30000. Certificate of optimality takes rather a lot longer on this model
What did you see instead?
Segmentation fault within the first ~20 seconds after presolve. Happens about 10% of the time.
Make sure you include information that can help us debug (full error message, model Proto).
user$ python run_model.py cp_sat_model.proto
Model loaded successfully from cp_sat_model.proto

Starting CP-SAT solver v9.11.4210
Parameters: log_search_progress: true
Setting number of workers to 32
.
.
.
#Bound   9.43s best:inf   next:[19048,10000000] fs_random
#Bound   9.54s best:inf   next:[19088,10000000] fs_random
#Bound   9.66s best:inf   next:[19116,10000000] fs_random
#Bound  11.17s best:inf   next:[19130,10000000] fs_random
#Model  11.99s var:2265/2267 constraints:17457/17458
#Model  13.12s var:2263/2267 constraints:17456/17458
#Bound  13.74s best:inf   next:[20369,10000000] objective_lb_search
#1      14.15s best:55582 next:[20369,55581] rens_lp_lns (d=0.97 s=772 t=0.10 p=1.00 stall=0 h=auto_l0)
Segmentation fault (core dumped)

crashing_model.zip
Anything else we should know about your project / environment
13th Gen Intel® Core™ i9-13900K × 32

I think it may be easier to repro if you set solver.parameters.use_lns_only = True

I ran it for 3 days on main. No crash. I will close it.

I ran it manually around 100 times. It never crashed.
And I know we fixed a few crashes post 9.11.
I will continue trying.
Laurent Perron | Operations Research | @.*** | (33) 1 42 68 53
00
Le mar. 8 oct. 2024 à 18:04, Will Floyd-Jones @.***> a
écrit :

@lperron https://github.com/lperron my apologies I should have been
more specific in the ticket. The issue will either happen in the first 20
seconds or it won't happen at all. When you ran it for 3 days, was it
constantly restarting or was it just running?
Here's the stacktrace I get when I manage to repro:
Thread 1 "sat_runner" received signal SIGSEGV, Segmentation fault.
_int_free (av=0x7fff40000030, p=0x7fff414429e0, have_lock=) at ./malloc/malloc.c:4590
4590	./malloc/malloc.c: No such file or directory.
(gdb) bt
#0  _int_free (av=0x7fff40000030, p=0x7fff414429e0, have_lock=) at ./malloc/malloc.c:4590
#1  0x00007ffff52a5453 in __GI___libc_free (mem=) at ./malloc/malloc.c:3391
#2  0x00007ffff6777326 in operations_research::sat::FeasibilityJumpSolver::~FeasibilityJumpSolver() () from /home/f-j/Downloads/or-tools/build/bin/../lib/libortools.so.9
#3  0x00007ffff6777609 in operations_research::sat::FeasibilityJumpSolver::~FeasibilityJumpSolver() () from /home/f-j/Downloads/or-tools/build/bin/../lib/libortools.so.9
#4  0x00007ffff68e939f in operations_research::sat::(anonymous namespace)::ClearSubsolversThatAreDone(std::vector<int, std::allocator > const&, std::vector<std::unique_ptr<operations_research::sat::SubSolver, std::default_delete<operations_research::sat::SubSolver> >, std::allocator<std::unique_ptr<operations_research::sat::SubSolver, std::default_delete<operations_research::sat::SubSolver> > > >&) () from /home/f-j/Downloads/or-tools/build/bin/../lib/libortools.so.9
#5  0x00007ffff68ea3f5 in operations_research::sat::NonDeterministicLoop(std::vector<std::unique_ptr<operations_research::sat::SubSolver, std::default_delete<operations_research::sat::SubSolver> >, std::allocator<std::unique_ptr<operations_research::sat::SubSolver, std::default_delete<operations_research::sat::SubSolver> > > >&, int) () from /home/f-j/Downloads/or-tools/build/bin/../lib/libortools.so.9
#6  0x00007ffff66c06a2 in operations_research::sat::(anonymous namespace)::LaunchSubsolvers(operations_research::sat::SatParameters const&, operations_research::sat::SharedClasses*, std::vector<std::unique_ptr<operations_research::sat::SubSolver, std::default_delete<operations_research::sat::SubSolver> >, std::allocator<std::unique_ptr<operations_research::sat::SubSolver, std::default_delete<operations_research::sat::SubSolver> > > >&, absl::lts_20240722::Span<std::__cxx11::basic_string<char, std::char_traits, std::allocator > const>) () from /home/f-j/Downloads/or-tools/build/bin/../lib/libortools.so.9
#7  0x00007ffff66be9a5 in operations_research::sat::(anonymous namespace)::SolveCpModelParallel(operations_research::sat::SharedClasses*, operations_research::sat::Model*) () from /home/f-j/Downloads/or-tools/build/bin/../lib/libortools.so.9
#8  0x00007ffff66b8a12 in operations_research::sat::SolveCpModel(operations_research::sat::CpModelProto const&, operations_research::sat::Model*) () from /home/f-j/Downloads/or-tools/build/bin/../lib/libortools.so.9
#9  0x000055555556e5c4 in main ()
(gdb)
—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/4400#issuecomment-2400249759,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3NZXE7W2EILXL5UXKDZ2P675AVCNFSM6AAAAABPM7TWCCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBQGI2DSNZVHE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Desktop support

Do you plan to add support for Windows/Linux/Mac to Isar? This seems like a good fit for a project I'm planning but it would need to have desktop support. You mention that it will be multiplatform in the README, but only mention Android, iOS and Web.

@davystrong yes definitely. There are already binaries for Isar Core and they just need to be copied in the correct directories. I didn't have time to test Isar on desktop yet so I didn't include the binaries by default.

@davystrong yes definitely. There are already binaries for Isar Core and they just need to be copied in the correct directories. I didn't have time to test Isar on desktop yet so I didn't include the binaries by default.

Ok, great! Hadn't noticed those. Thanks for the quick reply!

Ok, great! Hadn't noticed those. Thanks for the quick reply!

All desktop platforms are supported now

And it works in my case. Thanks!


Mixed GAS/Intel syntax in disassembly view and unavailable registers

Type: Bug
Problem description
When I set the Intel assembly syntax, I get the GNU syntax of the disassembly code for some stack frames, in others I get the correct Intel syntax. Also, many of the CPU registers cannot be inspected.
How to reproduce
Test project:
bug_var_unavail.zip
I set Intel syntax:

Open File / Preferences / Settings
on the left list panel, look for Extensions / CodeLLDB / Launch configuration defaults
on the right panel, Lldb>>Launch: Init Commands, click Add Item
enter settings set target.x86-disassembly-flavor intel

I debug the provided test program:

new configuration
adding 7 as program parameter ("args": ["7"] in the launch.json file
breakpoint on line 23 (first line in function test_it)
F5

if I double-click on main in the call stack
=> I get the correct Intel syntax

If I double-click on static bool bug_var_unavail::Test::test_it() or most of the other lines in that stack frame (or if I launch the command palette LLDB: Toggle Disassembly)
=> I get the GNU syntax

Many registers cannot be inspected by hovering them or typing register read in the debug console

In the debug console

reg read

General Purpose Registers:
rbx = 0x00000252AE4E6060
rdi = 0x00000252AE4EB150
rsi = 0x0000000000000000
rbp = 0x0000000000000000
rsp = 0x0000009EA876FD40
r12 = 0x0000000000000000
r13 = 0x0000000000000000
r14 = 0x0000000000000000
r15 = 0x0000000000000000
rip = 0x00007FF694AA47AC  bug_var_unavail.exe`main + 28
eax = 0xA876FA00
ebx = 0xA876F830
bx = 0xF830
57 registers were unavailable.

Further information
Doing a Step Over after launching in debug mode, on the first line of test_it(), performs a step-into the calc() method. If from there I try a Step Out it does nothing...
So it seems that this version of LLDB may not be entirely stable for Windows, and perhaps it fails to receive the correct syntax configuration in most of the cases. Or did I configure something wrong?
Environment
LLDB version, from the debug console:

lldb version 15.0.0-custom
rust-enabled

Rust active toolchain:

stable-x86_64-pc-windows-msvc (default)
rustc 1.65.0 (897e37553 2022-11-02)

Visual Studio Build Tools version: 2022 (fresh install)
Extension version: 1.8.1
VS Code version: Code 1.73.0 (8fa188b2b301d36553cbc9ce1b0a146ccb93351f, 2022-11-01T15:34:06.111Z)
OS version: Windows_NT x64 10.0.19044
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-10700K CPU @ 3.80GHz (16 x 3792)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
15.82GB (9.28GB free)

Process Argv
--crash-reporter-id 4b4f32f7-a3d7-41c2-b88f-c873c90fe509

Screen Reader
no

VM
0%

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
azure-dev_surveyone:30548225
pyindex848cf:30577861
nodejswelcome1:30587005
fc301958:30595537
3biah626:30602489
gswce2:30605431
3d0df643:30608728
dbltrim-noruby:30604474
89544117:30607850

Fixed in 1.9.0


fix(Makefile): Call bootstrap before we run generate

closes #1274
We call out to glide to get a list of directories when we run generate. This fails if the person doesnt have glide. So lets just go ahead and call bootstrap as part of that process

@jchauncey,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@sgoings why not? calling bootstrap before generate should install glide if it isnt available. unless im missing something.

nevermind i figured out the issue.

Also https://github.com/Azure/acs-engine/pull/1276/files#diff-b67911656ef5d18c4ae36cb6741b7965R10 will cause problems

ah good catch not sure how my find/replace didnt notice that

Pulled this PR and ran locally without glide installed. No warns/errors/glide complaints. 👍


prefix key with # in where clause also get quoted

I'm not sure why stick to quoting value even if # if prefixed. Sometime we just need to use a raw expression, for example, 32bit php don't support 64bit integer values, but mysql supports bigint. Using raw expression do helps.

Check out the new feature raw object https://medoo.in/api/raw.


Custom configuration for JCache

Hi, currently the caches available hardcodes the configuration object. Is it possible to customize this during initialization? This is required for stats / metrics into jcache for example. Thanks!

You can extend SerializableEntityCache and then override configure which receives the configuration object


Fix issue in China region "cn-north-1 is not a valid region name" issue in lambda code

Fix issue in China region "cn-north-1 is not a valid region name" issue in lambda code
Fix code:
instance-scheduler/schedulers/instance_scheduler.py
        # valid regions for service
        runtime_region = os.environ['AWS_REGION']
        if runtime_region == 'cn-north-1' or runtime_region == 'cn-northwest-1':
            self._valid_regions = boto3.Session().get_available_regions(service.service_name,partition_name='aws-cn')
        else:
            self._valid_regions = boto3.Session().get_available_regions(service.service_name)

The fixed code also included in https://github.com/liangruibupt/aws-instance-scheduler


Deprecate pipeline_configurations with extension in pipeline.yaml

Is your feature request related to a problem? Please describe.
The naming pipeline_configurations within pipeline.yaml can cause confusion with actual pipeline definition.
pipeline_configurations:
    aws:
        secrets:
            qchea-test-secret:
                secret_id: "kafka-secrets"
                region: "us-west-2"
                sts_role_arn: "arn:aws:iam::xxxxxxx:role/os-os-test-role"
            schema-secret:
                secret_id: "self-managed-kafka-schema"
                region: "us-west-2"
                sts_role_arn: "arn:aws:iam::xxxxx:role/os-os-test-role"
kafka-pipeline:
  source:
    kafka:
      encryption:
        type: "none"
      topics:
      - name: "quickstart-events"
        group_id: "groupdID1"
      bootstrap_servers:
      - "<IP_ADDRESS>:9092"
      authentication:
        sasl:
          plaintext:
            username: "${{aws_secrets:qchea-test-secret:username}}"
            password: "${{aws_secrets:qchea-test-secret:password}}"
      schema:
        type: confluent
        registry_url: https://psrc-m5k9x.us-west-2.aws.confluent.cloud
        version: 1
        schema_registry_api_key: "${{aws_secrets:schema-secret:schema_registry_api_key}}"
        schema_registry_api_secret: "${{aws_secrets:schema-secret:schema_registry_api_secret}}"
        basic_auth_credentials_source: USER_INFO

Describe the solution you'd like
rename it into extension
extension:
    aws:
        secrets:
            qchea-test-secret:
                secret_id: "kafka-secrets"
                region: "us-west-2"
                sts_role_arn: "arn:aws:iam::xxxxxxxxx:role/os-os-test-role"
            schema-secret:
                secret_id: "self-managed-kafka-schema"
                region: "us-west-2"
                sts_role_arn: "arn:aws:iam::xxxxxxxxx:role/os-os-test-role"
kafka-pipeline:
  source:
    kafka:
      encryption:
        type: "none"
      topics:
      - name: "quickstart-events"
        group_id: "groupdID1"
      bootstrap_servers:
      - "<IP_ADDRESS>:9092"
      authentication:
        sasl:
          plaintext:
            username: "${{aws_secrets:qchea-test-secret:username}}"
            password: "${{aws_secrets:qchea-test-secret:password}}"
      schema:
        type: confluent
        registry_url: https://psrc-m5k9x.us-west-2.aws.confluent.cloud
        version: 1
        schema_registry_api_key: "${{aws_secrets:schema-secret:schema_registry_api_key}}"
        schema_registry_api_secret: "${{aws_secrets:schema-secret:schema_registry_api_secret}}"
        basic_auth_credentials_source: USER_INFO

Describe alternatives you've considered (Optional)
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

I like the proposal. It is very close to the OpenTelemetry Collector approach: https://opentelemetry.io/docs/collector/configuration/. OTel uses the plural "extensions" though. Plural was also used in the original label "pipeline_configurations".


Version 1.4.0 not available on npm

Hi,
I'm trying to use gentelella with npm and webpack but when I installed it and then tried to compile it, I had problems with the file custom.scss which contains references to images which are not in the repository.
It looks like the version 1.4.0 fixed that problems. But I can't install it with npm.
When I force the gentelella version to 1.4.0 in my package.json, npm says this version doesn't exist.
Could you add this version so that I can install it with npm ?
Thanks,
Paul

Indeed, this would be great. Much obliged!

Hi, you can also directly fetch v1.4.0 from github via npm by specifying the release commit:
npm install --save puikinsh/gentelella#3448fdfdf16a41707d6e2a8154d592a88d85c41c


Batocera ES fork support

Would it be possible to looking into supporting Batocera ES fork?
https://batocera.org/
https://github.com/batocera-linux/batocera-emulationstation
https://discord.com/invite/JXhfRTr
Thanks!

Batocera ES seems to have some issue on  tag with video.
I just update the theme and tested on windows batocera ES. Should work on other platform.


make pip installable

Thank you! @JackLangerman

Thank you! @JackLangerman

Thank you for open-sourcing this helpful code :-)


Add fetch context

Add support for fetch context as described here: https://developers.cloudflare.com/workers/runtime-apis/fetch-event#syntax-module-worker
This enables a key use case that is using waitUntil to manipulate cache asynchronously.

I tested with my owner worker app and it's working correctly. Not sure how to add proper tests for it in the worker-sandbox app.

@dbw9580 - thank you! I'll get this tested ASAP and will reach out here if I have any questions.

Any updates?

@dbw9580 - sorry for the delay. I'm onboarding a new team member this week who will be working through current PRs with me (including this one). It should be included as part of the next release, likely at the end of the week. Thank you!

I think it depends on where #67 is.. if we're close to done, I agree let's wait. If not, maybe it can borrow some of this Context code.. @caass, can you chime in here on progress?

I see this and the cache api as two separate things, although the key use case for fetch context is to manipulate cache with waitUntil.  Can we test this without involving cache? Say, set up a counter in a durable object, and increment that counter in waitUntil, then after a period of time, check if the counter has been updated.

So, let's leave #67 out of consideration for this PR. We may just need to start the Cache API work separately.
@zebp - let's chat about if this should be the final PR to include in the 0.0.8 release tomorrow 👍


think and test in pipelines?

https://bitfieldconsulting.com/golang/scripting
could write/test in pipelines like:
run: mutate().report().exit(), returns nothing
mutate: files().mutateFile(), returns a stream of report output
files: lists go files from a root directory down
mutateFile: patterns().mutatePattern(), returns a stream of report output
patterns: lists mutation patterns
mutatePattern: candidates().mutateCandidate().test().restore(), returns a stream of report output
candidates: lists candidates for a pattern
mutateCandidate: mutates the candidate
test: runs all the tests
restore: restores the code to pre-mutation/test state
report: reports the results as streamed from mutate
exit: exits based on final success state of the stream

initial idea: each pipeline starts with a source, and ends with a sync, and data flows through it serially (though the parts of the pipeline will execute concurrently).

could simplify with pipeline parts that expand/shrink...
run: mutate().report().exit(), returns nothing
mutate: files().patterns().candidates().mutateEach(), returns a stream of report output
files: lists go files from a root directory down
patterns: lists mutation patterns for each candidate
candidates: lists candidates for a pattern
mutateEach: mutateCandidate().test().restore(), returns a stream of report output
mutateCandidate: mutates the candidate
test: runs all the tests
restore: restores the code to pre-mutation/test state
report: reports the results as streamed from mutate
exit: exits based on final success state of the stream

interesting mostly side notes:

how about splitting data streams by case? (no candidates found vs candidates found in file)
how about duplicating data streams for simultaneous handling?
how about recombining data streams? (overall file summarization)
how does elvish handle these cases? they have this kind of structured data piping...


Update refined, refined-scalacheck_1.13 to 0.9.8

Updates

eu.timepit:refined
eu.timepit:refined-scalacheck_1.13

from 0.9.3 to 0.9.8.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a nice day!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [{ groupId = "eu.timepit" }]

labels: semver-patch

This has been done as part of #150 .


enrollments panel progress

Progress on #146,
Adds ability to edit enrollments in any current semester to the coordinator page.
Users also get relevant grade information on their profile page.
Things missing:

Allow users to enroll in semesters
Self service enrollment deletion
JSON/CSV exportation
Enrollment deletion

Also fixes some terribly ugly code I wrote earlier.

I went ahead and rebuilt a lot of the form to be more semantic (using checkboxes for true and false) with bootstrap classes that correctly style it. If you approve of these changes then I can go ahead and merge it.


flutter run accepts empty --local-engine-host argument.

Updated Issue
EDIT: It turns out that in the issue below, the problem is the flutter run line. I used:
cd dev/manual_tests
flutter run -d linux --local-engine-host --local-engine=host_debug_unopt  lib/hardware_keyboard.dart

When I should have used:
cd dev/manual_tests
flutter run -d linux --local-engine-host=host_debug_unopt --local-engine=host_debug_unopt  lib/hardware_keyboard.dart

(notice the argument to --local-engine-host)
I think this is just a bug in the option parsing where it interprets the --local-engine=host_debug_unopt not as an argument in itself, but as the argument to --local-engine-host. It should be checking the argument it receives for '--local-engine-host' (and probably '--local-engine' too, in case the situation is reversed) to make sure it a) doesn't start with "--" and b) exists as a directory. Alternatively, if it's just allowing an empty argument, it shouldn't allow that.
Original issue filed
I tried adding a new API to the Dart code in my local engine build, built the engine, and wanted to test those changes with updated flutter code from the flutter repo, but I am unable to successfully run an application using the local engine with the updated dart changes.
I built the host_debug_unopt engine with:
flutter/tools/gn --runtime-mode=debug  --unoptimized  --no-goma  --full-dart-sdk
autoninja -C out/host_debug_unopt

and when (in the flutter repo) I run:
cd dev/manual_tests
flutter run -d linux --local-engine-host --local-engine=host_debug_unopt  lib/hardware_keyboard.dart

This fails to compile because the local version of sky_engine doesn't include the Dart changes I've made in the engine, so the newly added API doesn't exist.
I then tried adding something like this to the  pubspec.yaml of the app I am running:
dependency_overrides:
  sky_engine:
    path: /usr/local/home/gspencer/code/engine/src/out/host_debug_unopt/gen/dart-pkg/sky_engine

Which doesn't work, with the same issue: the Dart code doesn't include the new APIs somehow.
I tried removing the sky_engine directory in the cache directory and replacing that with a symbolic link to the built sky_engine in the engine build, which fixes things for the IDE, but still won't run the app (with the same errors).
I tried also adding the dependency_overrides to the packages/flutter/pubspec.yaml, but that doesn't help.
I filed https://github.com/flutter/flutter/issues/137268 because I was running into this problem, but as I tried to fix it to run in a number of different ways, I realized that it actually was also just broken.

flutter doctor -v
[!] Flutter (Channel [user-branch], 3.16.0-17.0.pre.32, on Debian GNU/Linux rodete 6.5.3-1rodete1-amd64, locale en_US.UTF-8)
    ! Flutter version 3.16.0-17.0.pre.32 on channel [user-branch] at /usr/local/google/home/gspencer/code/flutter
      Currently on an unknown channel. Run `flutter channel` to switch to an official channel.
      If that doesn't fix the issue, reinstall Flutter by following instructions at https://flutter.dev/docs/get-started/install.
    ! Upstream repository git@github.com:gspencergoog/flutter.git is not a standard remote.
      Set environment variable "FLUTTER_GIT_URL" to git@github.com:gspencergoog/flutter.git to dismiss this error.
    • Framework revision e73bce7a7f (2 hours ago), 2023-10-25 08:46:40 -0700
    • Engine revision 6e09ee14e2
    • Dart version 3.3.0 (build 3.3.0-56.0.dev)
    • DevTools version 2.28.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform
      update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /usr/local/google/home/gspencer/Android/Sdk
    • Platform android-33, build-tools 32.1.0-rc1
    • ANDROID_HOME = /usr/local/google/home/gspencer/Android/Sdk
    • Java binary at: /usr/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.8+7-Debian-1)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • Debian clang version 14.0.6
    • cmake version 3.26.4
    • ninja version 1.11.1
    • pkg-config version 1.8.1

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/linux#android-setup for detailed instructions).

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Debian GNU/Linux rodete 6.5.3-1rodete1-amd64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 118.0.5993.117

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

Thanks for digging into this. I think the action item here is to:

It should be checking the argument it receives for '--local-engine-host' (and probably '--local-engine' too, in case the situation is reversed) to make sure it a) doesn't start with "--" and b) exists as a directory


Question: how to aggregate multiple prompts.txt

I have a few fundamentally different conversations I want to test, let's say 3 for simplicity.  So thus I have three different prompts.txt files.  Each prompts.txt file has its own unique set of variables and tests.
How can one either:

Have one promptfooconfig that is able to have multiple (prompts.txt, set of vars, set of asserts)
Aggregate the results of 2+ promptfooconfig from different subdirectories?

So I see for the promptfoo-action, it's looking in prompts/.  What does the directory structure within prompts/ look like?  Are there like subfolders with many pairs of prompts.txt/promptfooconfig.yamls?
prompts
├── can-fly
│   ├── promptfooconfig.yaml
│   └── prompts.txt
└── can-swim
    ├── promptfooconfig.yaml
    └── prompts.txt

Is the desired result to have all the vars and tests combined into a single view?  In this case, would it only make sense if there was some overlap between the vars/prompts/models being tested?

I think it's combining the results of 2+ configs (with separate vars/prompts/tests) into:

Main ask: aggregated statistics
Nice to have: one promptfoo view

So lets say can-fly will have 10 passes and 2 failures, and can-swim will have 6 passes and 4 failures.  How can those be combined into a summary statistic (and possibly one promptfoo view), with just one promptfoo invocation?
In other words, it's like having promptfoo run on an entire directory of configs, and report an aggregated result

Got it.  I think https://github.com/promptfoo/promptfoo/pull/285 should solve this use case, both aggregated stats and aggregated view.  The web view might get a bit unwieldy if the evals don't have prompt/provider/var overlap.

Oh nice I like that, sweet! Can you add the ability to crawl directories?  Something like promptfoo eval -d prompts

Added :)  Will land the PR shortly
On Sun, Nov 19, 2023 at 2:57 PM James Braza @.***>
wrote:

Oh nice I like that, sweet! Can you add the ability to crawl directories?
Something like promptfoo eval -d prompts
—
Reply to this email directly, view it on GitHub
https://github.com/promptfoo/promptfoo/issues/170#issuecomment-1818005252,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACLYJVWZAWSNRV5PQRTHS3YFKFDVAVCNFSM6AAAAAA5AUIAVGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJYGAYDKMRVGI
.
You are receiving this because you commented.Message ID:
@.***>

Looks like the PR landed, closing this out, thank you again!


Allow Ignore For Child Dependencies

Fix for bug #927. Ignore package if name matches specified in bower.json.
Permit fetch if package is explicitly declared in dependencies or devDependencies.
Designed for users who need tighter control of versions due to corporate liscence restrictions or other reasons.

Just a note, we've been using this for several months now and it works quite well for us with no unexpected side effects.

@rosskevin How are you using this patch? "Manually" using a patched version of bower? or is there an automated way to grab this version?

@cfoellmann
npm uninstall -g bower
npm install -g https://github.com/SignalDancer/bower/tarball/Bower-Ignore-Dependencies

Looking forward to seeing this feature!

waiting for this feature :dancer:

Me too!

Very useful!

Does anyone know when the 1.4 release is scheduled?
I am really looking forward to this.

This just fixed one of my biggest headaches.  I'm looking forward to it getting into the main line.

That doesn't fully work though.
For example I have
"dependencies": {
  "jquery": "^2.1.3",
  "lodash": "^3.1.0",
  "backbone": "~1.1.2"
},
"ignoredDependencies": [
  "underscore"
]

When I run bower install the first time it works as expected
bower install
bower install
bower jquery#^2.1.3             cached git://github.com/jquery/jquery.git#2.1.3
bower jquery#^2.1.3           validate 2.1.3 against git://github.com/jquery/jquery.git#^2.1.3
bower lodash#^3.1.0             cached git://github.com/lodash/lodash.git#3.1.0
bower lodash#^3.1.0           validate 3.1.0 against git://github.com/lodash/lodash.git#^3.1.0
bower backbone#~1.1.2           cached git://github.com/jashkenas/backbone.git#1.1.2
bower backbone#~1.1.2         validate 1.1.2 against git://github.com/jashkenas/backbone.git#~1.1.2
bower bootstrap-sass#~3.3.3     cached git://github.com/twbs/bootstrap-sass.git#3.3.3
bower bootstrap-sass#~3.3.3   validate 3.3.3 against git://github.com/twbs/bootstrap-sass.git#~3.3.3
bower jquery#>= 1.9.0           cached git://github.com/jquery/jquery.git#2.1.3
bower jquery#>= 1.9.0         validate 2.1.3 against git://github.com/jquery/jquery.git#>= 1.9.0
bower backbone#~1.1.2          skipped underscore
bower lodash#^3.1.0            install lodash#3.1.0
bower bootstrap-sass#~3.3.3    install bootstrap-sass#3.3.3
bower jquery#^2.1.3            install jquery#2.1.3
bower backbone#~1.1.2          install backbone#1.1.2

If I run it again, though:
bower install
bower underscore#>=1.5.0    not-cached git://github.com/jashkenas/underscore.git#>=1.5.0
bower underscore#>=1.5.0       resolve git://github.com/jashkenas/underscore.git#>=1.5.0
bower underscore#>=1.5.0      download https://github.com/jashkenas/underscore/archive/1.7.0.tar.gz
git bower underscore#>=1.5.0       extract archive.tar.gz
bower underscore#>=1.5.0      resolved git://github.com/jashkenas/underscore.git#1.7.0
bower underscore#>=1.5.0       install underscore#1.7.0

This Sounds bad and I really need to check this out- finally.

So... will this ever be merged?

waiting for this merge to happen too!!!

@rosskevin thanks using it like you suggested works as expected, waiting for this merge

Guys - it's been almost a year. Can we get a merge here?

I would love to see this feature but it seems to be not ready for merge.
I would love to help but I can not help with JS programming

Fix to mentioned behavior + integration tests in test/commands/install.js would be nice.

@sheerun I will attempt to find time to work on this sometime soon. That said, why is the TravisCI build passing if integration tests are failing? Are they not covered by this as priorly that is what I had been using as the "flag" for whether it was merge-safe with all the tests.

Looking forward to see this feature!

+1

+1

+1

+1

+1

@sheerun maybe you should change the milestone.?

+1

+1

+1

+1

@SignalDancer would you mind adding some tests to this PR?
If anyone else would like this feature ( @Arkh1 , @frankthelen , @shea256 , @cfoellmann , @leafymind , @azaslavsky [and others] ) the big blocker is simply adding tests, if anyone would like to help and see this feature land, feel free to fork this commit and reopen a new commit into master with added tests.
Thanks everyone for the interest.

We are working with a Chinese team who can't access some dependencies with bower (due to national internet restriction). Allowing to ignore some of them in order to provide them manually would be very useful.

For the love of god, accept this pull request.

Any reason for not accepting this pull request? It's kind of old topic now but still it's not working.

+1

@krzysztofantczak: see https://github.com/bower/bower/pull/1394#issuecomment-97456155

@samccone Some high level tests added in PR #1828 , is that enough/acceptable to get this feature included?

What is the status of this?

+1

+1

+1

+1
It's quite frustrating to use Bower without such a basic functionality.

Re-posting question from earlier:
" @samccone Some high level tests added in PR #1828 , is that enough/acceptable to get this feature included? "

+1

+1

@sheerun - @twalpole added the requested tests to PR #1828.  Is there anything else required to get this merged?

+1

＋1

+1

Tests are in. What's the hold up?

:+1:

+1

+1

I would really love to see this merged back into the trunk and released.

+1

+1 indeed!

+1

A lot of legitimate use cases for this feature expressed above.
+1

+1

Hey :) I see all +1, but we still needs tests for this feature to accept (see my comment from March). For example we need to be sure that dependency is not ignored if one package ignores it the other is not.
fizfuz
foobar   - depends on fizfuz
package1 - depends on foobar
package2 - depends on foobar, ignores fizfuz
application - depends on package1, package2

In such case fizfuz should be installed, even it's ignored by package2.
If ignoreDependencies can be defined only in non-dependency (in the case above, application), it must be defined in .bowerrc, not bower.json (in my opinion it should be the case). Even then, we need tests.
If someone wants to volunteer to implement this, please comment :)

Tests were put in on another pull request, mentioned on Jun 22?
" @samccone Some high level tests added in PR #1828 , is that enough/acceptable to get this feature included?"

@sheerun see my comment from June 22 where I added some tests and asked if it was enough and got zero response.  I completely understand wanting tests but then when people do what you ask a little feedback would be appreciated

@sheerun https://github.com/bower/bower/pull/1394#issuecomment-132378107

Hi, can we get an answer to this question? Are the tests on the other pull request sufficient?

The tests in PR don't test behaviour when the dependencies of project define ignoreDependencies. Please read my comment why it can be complicated: https://github.com/bower/bower/pull/1394#issuecomment-142087421
If module I made ignores jquery, it doesn't mean jquery should not be installed in projects my module is dependency of.

And because Bower flattens dependencies, it probably makes little sense to allow ignoreDependencies in published modules, because always some other module may include it..
ignoreDependencies could be .bowerrc property instead (bower ignores .bowerrc files of dependencies). If done this way, the tests you've created should be probably enough.

Thank you for clarifying that :)

I don't know if the OP is still involved in this request given the age of this thread. @SignalDancer - you still alive over there to do a reroll of this feature given that input?

@sheerun   Ok -- PR #1828 has been updated to have ignoredDependencies be a property from .bowerrc  -- Are there any other changes required for this to be accepted?

:disappointed: :clock1:  sooo based on reading this thread here, as well as my own use cases, seems like this is a feature that a lot of bower users have been in need of for some time now, i see there were concerns in the past,(quite understandable, with a product with such a varied base of regular users to continually placate) but what is the hold up now  :grey_question: :exclamation: :grey_question: :exclamation: :grey_question: :exclamation: lets do this :+1:

So was this ever merged... ?

@ciel nope, the world seems to still be on hold for this, @sheerun  ?

Now that the requested change is in - I'd really not like to see this wait another year. Code is ready to roll - can we have a merge?

Hey all, remember this is OSS @sheerun is generously donating his time to support bower, with little to no support, instead of being critical why not offer to help in whatever way you can.
cherry-pick this change and test it out, review some of the open issues, review some of the open PRs, anything is super helpful. We are all in this together :sparkling_heart:

that sort of feedback is all we are looking for, i personally am willing to help wherever i can, (for something big like this for sure) but when a projects developers dont respond to countless requests, we cant just go and assume what still needs to be done on their end, all we (all of us with 0 push access to the repo) can do is wait for responses before we can try to continue to work on things, we all have lives just like you guys do, i mean from this side of things everything looks alright. all i can say, is from all of the comments in this thread its very clear that this is a feature that the community is really looking for, seems odd to me that the developers of such awesome software let their users wonder about when and if a feature will be implemented for over a year, but i will go ahead and try to clean up this PR if that will get this moving.

I think it would be easier to be patient if we had any idea how long the wait might be. We really don't know if being quiet will still get this done is the thing - this sat with the test request fulfilled for 3 months before we heard back on anything, so it seems like being loud about it was the only way to get feedback.  Even if it were as simple as "I'll check on this on the 15th and see if the issues are resolved -" that'd be better than not knowing, y'know? We'd know what to expect is all.

Everyone, I merged this PR into PR #1828 , implemented the tests there and made the requested change so the setting is now read from .bowerrc rather than bower.json -- It would probably be best if we move any further comments/change requests over to that PR since thats where @sheerun or whoever else can merge it in really needs to be looking

Awesome! Thanks :)

@sheerun this can be closed now since you merged it in through the other PR

Today I've spent more than 8 hours to merge it and other requested features to bower (I still need to release it today/tomorrow). Such review and merging takes a lot of my spare time. If someone would like to help developing bower, please ping me on<EMAIL_ADDRESS>

Thanks @sheerun! There are a lot of us who really appreciate and benefit from the work you put into developing and maintaining the project.

Thanks @sheerun - it's a lot of extra work, but it is appreciated :)

Big win and big thank you @sheerun

Thousands of us thank you!
--
Benjamin Allison
Designer

On Sep 27, 2015, at 11:17 AM, Remy Schrader<EMAIL_ADDRESS>wrote:
Big win and big thank you @sheerun
—
Reply to this email directly or view it on GitHub.

Awesome, Thanks @sheerun

big thanks to @sheerun

Many tks @sheerun!!

So, grateful congregation, how to use it?

@OnkelTem in .bowerrc add an array of packages to ignore
ignoredDepedencies: ['package1', 'package2',...]

Bower 1.6.2 is now released with ignoredDependencies in .bowerrc support. Thank you!


Permit to disable/enable toggling on click event

This PR doesn't correspond  exactly to what  is mentioned in #53 but it permits to keep the default toggling when needed.
As an example, I'd like to keep this behaviour and be able to change the style of the main button depending on what I've clicked.
Thanks for your work !

@akofman When you say "keep the default toggling", what do you mean?

With the actual version, if we add an onClick callback on the main button then the toggle open/closed is not triggered anymore. So I propose to keep the same behaviour whether it exists an onClick callback. And if we really want to stop the toggle behaviour we have the possibility to add clickToggle to false.

@dericgw is it clear enough for you or do you want me to change something else ?

@dericgw Sorry for the delay. If I understand what you're saying, I feel like this should be default behavior and not need an extra prop. If there is a case where an extra prop is really needed, could you explain that scenario, please? If there is a scenario in which another prop is needed, what would be the percentage of consumers needing to do that?


Mekanism gas tank autocrafting doesn't work

Issue description: If I try to autocraft the ultimate gas tank (Mekanism)  the crafting preview says that i'm missing an Elite gas tank, but i already have that pattern.
Note: I put the patterns in a Crafter.
What happens: Refined Storage does not autocraft the Ultimate Gas Tank, it keeps saying "Missing: 1 Elite Gas Tank", but I already have that pattern.
What you expected to happen:
Autocrafting the Ultimate Gas Tank
Steps to reproduce:
1.Make all the patterns in order to autocraft the "Ultimate Gas Tank" from Mekanism
2.Open the crafting grid and select it to autocraft it
...
Version:

Minecraft: 1.12.2
Forge: <IP_ADDRESS>38
Refined Storage: 1.6.15

Does this issue occur on a server?
no

Duplicate of #2064
Manually replace the tank in the pattern with a tank that you have crafted.

Duplicate of #2064
Manually replace the tank in the pattern with a tank that you have crafted.

Still doesn't work :/

Other things that may add more NBT tags and cause this to not work include:

having the tank in your inventory
placing the tank down
inserting fluid into the tank

The only way to check these things is by looking at the actual NBT data.

Might you explain me what "looking a the NBT data" means? I don't know what is it. If I need to craft the tank my self without putting it in my inventory, how could I replace that in the pattern? Thank you for your help.

Might you explain me what "looking a the NBT data" means? I don't know what is it.
There are some mods that allow looking at the NBT data (the data that is on the item) depending on your pack there are different ways of doing so.

If I need to craft the tank my self without putting it in my inventory, how could I replace that in the pattern?
The item only goes into your inventory once you put it into one of the 36 slots of the inventory. If you keep the tank inside RS system or only on your cursor the changes are not applied.

Oh and make sure you do not have Oredict enabled on the pattern.


small fix to optimization code

add nesterov to SGD
add weight decay to gamma of batchnorm

any update on this? we can consider merge it in if the optimizer is backward compatible with original SGD when nestrov is false

I am going to merge this in 24 hours. Please let me know if there is any concerns

@mavenlin can you act upon the comments and change it to subclassing? thanks!

@tqchen @piiswrong it is fixed now

Thanks! this is now merged


java 8 or java 7?

The pom specifies java8-parent as parent pom, but explicitly sets java7 in the compiler plugin configuration.
Either works for me, I'm just asking for clarity :)

It should be 1.8 in the pom now.

It should be 1.8 in the pom now.

It's still 1.7 in master branch's pom.xml (in the compiler plugin).

It's still 1.7 in master branch's pom.xml (in the compiler plugin).


Correct spelling, change icochrone to isochrone.

Docs Build status updates of commit 45bac0e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

BingMaps/index.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Update badges and documentation

Add badges from shields.io for Cookbook and Build status
Add badge for pull reminders
Add documentation folder for all resources

This change is

          1 Warning

:warning:
This is a big Pull Request.

  Generated by :no_entry_sign: Danger

Thanks for updating the links :)
Fat fingered them!

Going to skip the changlog this time as there are no feature updates. Yolo

Didn't know you could do that in .md.  Very nice.


Non optimal header location and inclusion

Hey, first of all thanks for your library.
I've noticed that your directory structure could possibly causes inclusion issues because
there is no toplevel (include) directory which seperates the inclusion path from the rest of the repository.
A better (and more established) way would be to move all your public headers into a directory like include/toml such that no one has to pass the whole project directory to the compiler as inclusion directory. This could possibly mess up other includes otherwise (and also it exposes the whole repo to the compiler).
You could also then reference your own headers through an absolut path:
#include <algorithm>
#include <toml/from.hpp>
#include <toml/result.hpp>
#include <toml/value.hpp>

Additionally the google C++ style guide recommends to include standard headers before the own ones in alphabethical order to profit from a possible inclusion order optimization: https://google.github.io/styleguide/cppguide.html#Names_and_Order_of_Includes

I see, I understand. You are right.
However, some projects use this repository via git submodule, and changing the include path would be a breaking change. So I'll solve this when I release version 4.
The inclusion order can be improved without any changes, so I will do that soon.

Theoretically you could copy the headers into an include directory while keeping the old files.
The content of the old files then could be replaced by a #include <toml/CURRENT_FILE.hpp while producing a deprecated inclusion warning. Through that you would keep the semantic versioning while also giving your users a notice for migration and to make it possible for everybody to use the new inclusion right away.

That sounds nice. When we see the prospect of releasing version 4, I will add it.
I have several tasks for the next release, like std::u8string support and improvement of options while serialization, so it would take a while.

I have released v4.0.0 and moved all the include files into include/.


Old maptrees unable to open

Previously, we used integers, rather than strings to refer to the bins. They were labeled 'id' rather than 'name' as we have now. I can manually overcome this label switch, however due to the type difference, a fascinating error message occurs.
This was while running the example to extract an ROI 3 degrees around the crab, from the example in hal already (but for Mrk 421).
Traceback (most recent call last):   File "make_crab_set.py", line 18, in <module>     m.write("crab_maptree.hd5")            File "/Users/chad/hawc/chad_hal/hawc_hal/maptree/map_tree.py", line 171, in write     'Bin name inconsistency: {} != {}'.format(bin_id, analysis_bin.name) AssertionError: Bin name inconsistency: 0 != 0

Additionally, it may be a good idea for backward compatibility to add logic to account for older maps to avoid this kind of problem
Traceback (most recent call last):   File "./fitModel.py", line 513, in <module>     like = HAL("HAWC", map_tree, det_res, roi, options.pixelwidth)   File "/Users/chad/hawc/chad_hal/hawc_hal/HAL.py", line 57, in __init__     self._maptree = map_tree_factory(maptree, roi=roi)   File "/Users/chad/hawc/chad_hal/hawc_hal/maptree/map_tree.py", line 22, in map_tree_factory     return MapTree.from_root_file(map_tree_file, roi)   File "/Users/chad/hawc/chad_hal/hawc_hal/maptree/map_tree.py", line 53, in from_root_file     data_analysis_bins = from_root_file(map_tree_file, roi)   File "/Users/chad/hawc/chad_hal/hawc_hal/maptree/from_root_file.py", line 70, in from_root_file     data_bins_labels = list(root_numpy.tree2array(f.Get("BinInfo"), "id"))


Migrate to Spark 2.0

This migrates spark-notebook to Spark 2.0 dropping support from earlier Spark versions .
for #708
some testing to be done

[x] using deprecated HiveContext works OK
[x] check main widgets: PivotChart
[x] check DownloadCsv (API changed,  consolidation into 1 file & rename broken) #724

just Ignore merge commit mess for now.

@andypetrella so what's the plan for future? do we drop support of spark 1.6 and leave spark 2.0 only?
cc @astrauka

this is basically good enough now.
a bit more testing and we can 🚢  it.
@andypetrella @astrauka

:clap:

Forgot to merge the commits, so squashed in #727  (looks like github is amazingly good at squashing that horrible merge commit noise)


Missing AYS service template: blockdevice.raid

When executing the blueprint on nastest4:
https://moehaha.barcelona.aydo.com/cockpit/repo?repo=nastest4
In general, what should I do in case a template is missing?

I don't think it's bundled by default:
https://github.com/Jumpscale/ays_g8_nas
You need to have it in your repository. you need to have those servicetemplates " https://github.com/Jumpscale/ays_g8_nas/tree/master/servicetemplates " in your servicetemplates directory

thanks @xmonader
So that means that I'd need the broken functionality as reported here #132?
(for demo purposes I do not want add the template at the CLI)
Is this something you can fix and/or do you need the help of @zaibon?

It's a design decision. of course I can add it to ays_jumpscale repo, but that decision is taken by @rkhamis or @zaibon
Maybe I can add it locally to your system?

This is indeed by design. We try now to have different repository for different service template so the templates can have there own live and we can create bug and versioning on them.
If you want to use it in your cockpit, you need to make sure you added this repository to the list of template repository first.

Adding this template to my repository requires CLI?
We also need a way to add templates using the Cockpit Portal I guess...

fix in https://github.com/Jumpscale/jscockpit/commit/c5e56fd5eb650fe74ab801af95c9b21e2d399ab3


Node 7 CI tests fail due to oracledb install failure.

Node 7 CI tests fail due to oracledb install failure

Related Issues: No issues, but PR #1710 and PR #1712 were affected by this failure

Introduction
Every Travis-CI build fails against Node 7 due to an issue with oracledb installation.
Issue Description
oracle/node-oracledb#811 documents that there are no prebuilt binaries for Node 7 on oracledb. Travis-CI runs Bookshelf builds against Node 4, 6, and 7. Bookshelf's .travis.yml tries to install oracledb in the before_install section.
https://github.com/bookshelf/bookshelf/blob/ed0d72692a15ef189dc056e551853393c3b0e143/.travis.yml#L19-L23
The Node 7 build fails for each PR and merge as a result of the install issue with oracledb.
Steps to reproduce issue

Fork Bookshelf project
Cut new branch on forked project
Commit change
Submit PR against bookshelf/master branch
Travis-CI webhook starts testing PR
Node 4, 6 install correctly and run Bookshelf tests
Node 7 fails during oracledb install

Expected behaviour
I expect oracledb to install correctly in Travis CI's Node 7 build. After successfully setting up the environment, PR changes should run against Bookshelf tests.
Actual behaviour
Oracledb fails to install in Travis CI's Node 7 build. The PR or merge fails without ever running against Bookshelf tests.

Solution could make use of #1609, which states: "OracleDB tests will be loaded and executed only when oracledb is installed". However, this hinges on OracleDB failing gracefully during install and not exiting CI with an error.

I'll just update Travis config file to test on Node 8.x instead.


Various masquerading fixes

This change is

test-me-please

Coverage increased (+0.05%) to 41.928% when pulling 405d583e342ebf1776a30f72199445a2ef6f5879 on pr/snat-improvements into f063e7e4b22a075f49c0f19477942cfbb9902dbe on master.

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

fyi, from last two Jenkins runs:

https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/11667/
https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/11664/


How can I get FFPLAY to build?

Thank you very much for this amazing script. I need a bit of help. I have looked at the build.sh script in the ffmpeg folder and it does NOT contain --disabled-ffplay, yet ffplay is not being built. What do I need to do to get it to build? FFMPEG and FFPROBE are building just fine. Thanks!

Also wanted to mention that adding "--enable-ffplay" as mentioned here does not cause ffplay to be built for me. However, when I go into the "sources" directory and do make clean, ./configure, make. It creates ffmpeg, ffprobe, and ffplay, but they aren't for android, they are for linux. I can't figure out how to get them to build for android that way.

Hello @petermg
According to FFmpeg's configure script:

ffplay_deps="avcodec avformat avfilter swscale swresample sdl2"

I believe it expects sdl2 library to be provided (prebuilt for Android).

Does this mean I have to enable sdl2 in the build script or ?

When you simply do clean, ./configure, make, then you compile FFmpeg for your host machine, not for Android. This project tries to make the cross-compilation easier (building for Android specifically).
I personally successfully used ffmpeg binary in Android's terminal. But as for ffplay, it supposed to open a platform-dependant UI (a window). On Android this isn't the way software runs - you have to properly package an Android app and manually provide a ANativeWindow to FFmpeg. I don't have much details on the process, I'm afraid.

When you simply do clean, ./configure, make, then you compile FFmpeg for your host machine, not for Android. This project tries to make the cross-compilation easier (building for Android specifically).
I personally successfully used ffmpeg binary in Android's terminal. But as for ffplay, it supposed to open a platform-dependant UI (a window). On Android this isn't the way software runs - you have to properly package an Android app and manually provide a ANativeWindow to FFmpeg. I don't have much details on the process, I'm afraid.

That makes sense. Thank you!! And thanks so much again for this phenomenally useful script!!!

Does this mean I have to enable sdl2 in the build script or ?

I think so, yes. The SDL2 has to be accessible (and assessible) by FFmpeg's configure script during its execution. You can either manually build it from sources like the rest of libraries here, or somehow provide the prebuilt version of it to the build.
By the way, currently ffmpeg-android-maker builds everything from sources. Providing something in a prebuilt form is a good thing to think about. Thanks.


add sh and zsh, add Linux's pwsh, always quote default shell

This PR introduces a breaking change to always quote Command passed to a custom shell.
Comments and discussion welcome.

Add a test for non-windows systems in ShellCommandExecuteTests.fs for SH

ZSH is native on macos, isn't it? => Then add a test for it.

Sure but I will have to "blindly" add Macos stuff as I do not have any Apple gear.

Add SH & ZSH to the provided Fli.Shells in the README.md

Yup.

had to also update a PWSH->string test case for Linux

That looks good. 🙂
Appreciate your work.  I will create a release later this day.
If you want you could develop that feature with stdout/stderr as output (#63)?


Need to disable module compress

Hello Kaiwan. Great book!
Just found a minor inconsistency which I'd like to bring to your attention.
Just realized modern kernels (> 5.13 apparently) for x86_64 enable module compression by default which is why when we install modules, the compressed version of it gets installed. On the stock version of 6.1.25, compression mode was set to ZSTD in my own case. The book however expects us to find *.ko files at /lib/modules/$(kernel-name)/kernel.
It would be great if you could update the errata/addendum list to indicate that we need to set module compression to None at:
Enable loadable module support -->Module compression mode (None)
That is all!

@redbilledpanda thanks v much! Have added this to the Known Errata section


Mobs do not spawn

Well i put this mod on my mod list and once i started the world, hostile mobs didn't spawn, after a few minutes i remove the mod and start again the world and mobs start to spawn.
Im not sure, but is like this mod disable the spawn of hostile mobs
Edit: Solved

Hello, what was your issue? Was the issue from another mod or was it an incompatibility?

Hello, what was your issue? Was the issue from another mod or was it an incompatibility?

idk what it was, but for say in other words i was touching the config settings. I wanted to disable the option to make hostile mobs to attack villagers (which option i never found) and i touch other config that one disable the spawn of mobs. So i was sending this tread wrongly because i fixed putting in true again.
{
"general": {
"mobsLessFear": true,
"merchantHostility": true
}
the meerchanthostility if i put on false disable every hostile mob from the game to prevent it spawn. But... since u send me a message, could you help me?, which option on the config i have to turn in false to disable the option to make hostile mobs to attack villagers?.

That's funny, I just replied to you on CurseForge.
There is no option in config files that should disable the spawn of mobs. If I understand correctly, when you put this to false "merchantHostility": false, hostile mobs don't spawn anymore? If yes, this is a bug. I will do some testing.
But you were right, this is the option you should disable if you don't want hostile mobs to attack villagers. Note that it disables only the one I added this behavior to, but not the ones that had it already, such as zombies.

That's funny, I just replied to you on CurseForge.
There is no option in config files that should disable the spawn of mobs. If I understand correctly, when you put this to false "merchantHostility": false, hostile mobs don't spawn anymore? If yes, this is a bug. I will do some testing. But you were right, this is the option you should disable if you don't want hostile mobs to attack villagers. Note that it disables only the one I added this behavior to, but not the ones that had it already, such as zombies.
[EDIT] After small testing, disabling it disables the behavior without breaking mob spawn. Feel free to give me more info about it

oh, alright. Im gonna check again this night. Maybe was just a coincidence but honestly im not sure, i was walking in the night far away from my house and mobs never spawned when i turn in false the merchanhostility. But when i turn on again mobs started to spawn like normally.
Im gonna check it and i tell you news about it c:
BTW, I forget, the mod version i was playing it was the latest one for 1.20.1.


Reduce the scope of multiple utility classes in core module

This PR reduces the scope of some classes in the core module, namely:

MapSequence
SkipSequence
TakeSequence
WhereSequence
StringByteSequence
StringCodePointSequence
MapKeySequence
MapValueSequence

These classes do not appear in the documenation, so if code relies on their presence, it's some kind of undefined behavior. As a counter-example, MapEntry was not removed as its name explicitely appears in the documentation.
Some pros:

The names do not appear anymore in the core module and can be used safely in user code
The number of core symbols is reduced
The creation of classes is paid only when needed, i.e. when the function is called. As a consequence, it reduces the runtime of core loading and hence the startup time.
If the classes have to be changed in the future, it will be easier as they are not part of the core interface anymore

Some cons:

These classes have to be created at each call of the functions. But the functions create a whole sequence so the cost can be balanced with the cost of the loop (that does not change).

Very interesting.  Someone is going to ask for performance numbers I'm sure. :-)

The names do not appear anymore in the core module and can be used safely in user code
The number of core symbols is reduced

Curious, which of these was the primary motivator for this PR?

it reduces the runtime of core loading and hence the startup time.

I'd say for most use cases this feels backwards... I'd rather have the runtime performance and let it start up a few milliseconds slower.  I've been doing a lot of test suite work lately and we're already so much faster than so many things at startup.

If the classes have to be changed in the future, it will be easier as they are not part of the core interface anymore

Perhaps they should simply be named TakeSequence_ as another way of solving this concern.

No need to ask for performance numbers. There is a certain issue in the compiler/vm, that makes it not really suitable to have embed class inside function. Each time you call the class you patch each method, and the performance are far beyond one would expect since we recreate a class on each calls.
In simple terms, this is not the way to go, even if it works.

Thanks for the PR!
Every one of these is used extensively in regular wren code, it would have a large impact on general performance if you pay the cost for every single call site. I don't think the trade off it the right one, the start up time is negligible and the runtime issue is far more important.
I also don't think this is an issue to solve, I know it was discussed in other issues but sometimes these discussions are solutions in search of a problem.
The effort is appreciated though.

and can be used safely in user code

@ruby0x1 Is this perhaps a small valid point?  If we use symbol_ elsewhere to indicate "private", should these classes be marked the same - or are they intended to be part of the public API even if undocumented?

@ruby0x1 OK, no problem.
Just to mention a small point though. Three of these classes appear in tests and, IMO, they should not appear as they are not part of the interface.

in test/core/string/bytes.wren, StringByteSequence appears but the documentation says that bytes should return a Sequence
in test/language/module/implicitly_imports_core/module.wren, MapKeySequence and MapValueSequence appear but again, the documentation does not specify the actual type of keys and values (just Sequence)

Do you want a PR just for this?

they should not appear as they are not part of the interface.

I'm not sure I see a problem with parts of our test suite testing the internals... that sounds like more of a policy decision?  I was more concerned with the naming and whether we want people just making their own new instances of StringByteSequence for whatever reason or whether we should flag it as private API.

StringByteSequence appears but the documentation says that bytes should return a Sequence

I think StringByteSequence is just an implementation detail here.  It's a Sequence by all intents and purposes because it inherits from Sequence... I'm guessing the fact that it's a subclass isn't relevant to the API usage.


fix(cli.rs): don't enforce a rustfmt.toml

What kind of change does this PR introduce? (check at least one)

[X] Bugfix
[ ] Feature
[ ] Docs
[ ] New Binding Issue #___
[X] Code style update
[X] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[X] Yes. Issue #___
[ ] No

The PR fulfills these requirements:

[X] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)
[X] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:
We got a lot of complaints about enforcing a rustfmt in the generated tauri project.

Should I also remove it from the plugin templates ?

yes, probably remove it. Our format should be used by our codebase only.
Devs should have freedom to choose theirs


Cback support

Hi,
We've got a forum that is running for 11 years. But we need something else.
The forum runs on cback. (www.cback.de) I'm trying to move to vbulletin or so. But no one has support for a cback conversion.
I can, if needed, give you a clean cback database dump with 2 users or so.

What do you want to import? Just users? Or more than users?

HI Zachery,
Thanks you for your answer!
I want to import the posts/topics, Users + signatures, and forums.

Can you get me a SQL with some example data in it?

Yes, I can. How can I send it to you?

Sorry for the very late reply, I don't have time to develop a new importer. But good luck, if you did find any one who developed this and they want to release it back here I'm happy to merge it.


hf token?

Hi - testing this out, but after my second test i'm getting some issues with HF access.... I've tried adding HF_TOKEN=... to the .env file, but that didn't seem to do anything = not sure what the answer is?
here's what i'm getting after running
"python src/main.py -f llc.txt" on some text regarding llc formation
Extracting BPMN data...
Error when extracting BPMN data: Rate limit reached. Please log in or use a HF access token
Generating graph...
Traceback (most recent call last):
File "~/process-visualizer/src/main.py", line 60, in
generate_graph_pdf(output, notebook)
File "~/process-visualizer/src/process_bpmn_data.py", line 1148, in generate_graph_pdf
bpmn.generate_graph()
File "~/process-visualizer/src/graph_generator.py", line 552, in generate_graph
for global_index, element in enumerate(self.data):
^^^^^^^^^^^^^^^^^^^^
TypeError: 'NoneType' object is not iterable

nvm - used
huggingface-cli login
and then entered a token from the site


Error creating wallet using illegal characters

Creating a new wallet with an illegal character (e.g. "testWallet*") on Windows, leads to the following error:

Thanks for the bug report. Fixed in ed2914f. The wallet name dialog will now replace illegal characters with underscores.


review screen problems

Michelle and I are having problems with getting ALIncome things to work in the review screen. Specifically I want to display the word "None" under categories where they did not select anything (try saying none of the above to everything). Can't get that to happen. I also want the user to be able to Add an item in the review screen. I'm not having luck. (Try saying you have a piece of real_estate and a bank_asset then try to add. For real_estate you get Error There appears to be an infinite loop. Variables in stack are x.source. For bank_assets they disappear from the review screen when you add one.) Do you know of any fixes or workarounds to get this to work? Here's my repository and I commented out the user info so I can jump into the incomes, expenses and assets quickly in the interview.
Michelle B she/her (Guest)
Just to add to the report: I believe the object we were troubleshooting was an ALExpenseList. Even though we input that the user had no expenses, the ALExpenseList had two items. The items had very few properties. For example, no value. We couldn't find where those items are being added. Even trying to put expenses.revisit under Edit was enough to cause an error that wouldn't show the section on the review screen. We can get the "variables and values". Other objects were also having problems, but we didn't look into those.

The main problem is that the ALIncome things are not being gathered when "None of the Above" is selected.  A list that isn't gathered is the same as a variable that is undefined in a review screen - it just won't show up.  To fix this, we need to make sure the lists are gathered before the review screen.


try jdk 11

Codecov Report

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

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   85.64%
  Complexity        ?      543
=========================================
  Files             ?      119
  Lines             ?     1609
  Branches          ?      167
=========================================
  Hits              ?     1378
  Misses            ?      210
  Partials          ?       21

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 cc5c6fa...f1c3686. Read the comment docs.


Multi-tenant authentication does not work with a client secret credentials

Package Name: @azure/identity
Package Version: 3.0.0
Operating system: macOS 12.6
[X] nodejs

version: v16.16.0

[X] typescript

version: 4.8.3

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
Trying to authenticate with a tenant id that is different then the tenant id in the client secret, does not work, even when passing the options "additionallyAllowedTenants" correctly (with '*' or with both the tenant ids)
To Reproduce
Steps to reproduce the behavior:

Register an app in Azure AD and create a client secret credentials for it
Give permissions to the app to other tenant
Run the following code:

import { ClientSecretCredential } from '@azure/identity';
import { Client } from '@microsoft/microsoft-graph-client';
import { TokenCredentialAuthenticationProvider } from '@microsoft/microsoft-graph-client/authProviders/azureTokenCredentials';

const tokenCreds = new ClientSecretCredential(
  '<first-tenant-id>',
  '<client-id>',
  '<client-secret>',
  { additionallyAllowedTenants: ['*'] },
);

const authProvider = new TokenCredentialAuthenticationProvider(tokenCreds, {
  scopes: ['https://graph.microsoft.com/.default'],
  getTokenoptions: { tenantId: '<second-tenant-id>' },
});

const client = Client.initWithMiddleware({
  authProvider,
});

client.api('/organization').get().then((res) => console.log(res));

the code fails with the following error (on the last line):

The current credential is not configured to acquire tokens for tenant <first-tenant-id>. To enable acquiring tokens for this tenant add it to the AdditionallyAllowedTenants on the credential options, or add "*" to AdditionallyAllowedTenants to allow acquiring tokens for any tenant.

Expected behavior
The code should work
Additional context
The issue seems to be here:
https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/identity/identity/src/msal/nodeFlows/msalNodeCommon.ts#L306
It looks like this line does not take into account the "additionallyAllowedTenants" option and doesn't pass it to the "processMultiTenantRequest", which uses a default empty array:
https://github.com/Azure/azure-sdk-for-js/blob/0ea9f1462fb598a0d95503f351a46e47321e3233/sdk/identity/identity/src/util/processMultiTenantRequest.ts#L19

Fixed as per #23704 and #23734


Custom vif setup script not working with mirage-net-xen 1.4.1

For jitsu we use a faster vif setup script (available here and in C here) instead of the default in /etc/xen/scripts/vif-bridge. With mirage-net-xen version 1.4.1 they stopped working. Downgrading to 1.4.0 fixes the problem.
The unikernel hangs after "MAC: ..." :
Netif: add resume hook
Netif.connect 0
Netfront.create: id=0 domid=0
MAC: 00:16:3e:00:9f:c4

This is in xenstore:
 vif = ""
  0 = ""
   backend = "/local/domain/0/backend/vif/2/0"
   backend-id = "0"
   state = "4"
   handle = "0"
   mac = "00:16:3e:00:9f:c4"
   tx-ring-ref = "9"
   rx-ring-ref = "8"
   event-channel = "4"
   request-rx-copy = "1"
   feature-rx-notify = "1"
   feature-sg = "1"

As the default script works I guess the problem is in the custom scripts, but I'm not sure what should be added to them to fix it. Any suggestions? /cc @djs55

The only change in 1.4.1 was to wait for the backend to enter state=4. I'd
be interested to know what was in the xenstore backend directory
(/local/domain/0/backend/vif/2/0 in this case). There might be a log
message from netback in the domain 0 log.
The standard vif script might be doing a bit more than ours? In the past it
would write a special 'hotplug-status=connected' key, I'm not sure if
that's still needed.
Do you know if libxl is running the script itself or via udev? IIRC it's a
configuration option which is written somewhere in xenstore. Perhaps the
hotplug-status=connected key was only needed for the running via udev case.
(sorry for the brain dump)
On Fri, May 1, 2015 at 3:42 PM, Magnus Skjegstad<EMAIL_ADDRESS>wrote:

For jitsu we use a faster vif setup script (available here
https://github.com/mirage/jitsu/blob/master/scripts/fast-vif-bridge and
in C here https://github.com/mirage/jitsu/blob/master/scripts/add_vif.c)
instead of the default in /etc/xen/scripts/vif-bridge. With mirage-net-xen
version 1.4.1 they stopped working. Downgrading to 1.4.0 fixes the problem.
The unikernel hangs after "MAC: ..." :
Netif: add resume hook
Netif.connect 0
Netfront.create: id=0 domid=0
MAC: 00:16:3e:00:9f:c4
This is in xenstore:
vif = ""
0 = ""
backend = "/local/domain/0/backend/vif/2/0"
backend-id = "0"
state = "4"
handle = "0"
mac = "00:16:3e:00:9f:c4"
tx-ring-ref = "9"
rx-ring-ref = "8"
event-channel = "4"
request-rx-copy = "1"
feature-rx-notify = "1"
feature-sg = "1"
As the default script works I guess the problem is in the custom scripts,
but I'm not sure what should be added to them to fix it. Any suggestions?
/cc @djs55 https://github.com/djs55
—
Reply to this email directly or view it on GitHub
https://github.com/mirage/jitsu/issues/14.

--
Dave Scott

Thanks! Writing hotplug-status=connected to xenstore seems to fix the problem with fast-vif-bridge at least.
/local/domain/0/libxl/disable_udev is set to "1" in Xenstore

Hm looks like it didn't fix the problem completely... maybe I was just lucky and won the race a few times.


rules validation object

This passes the object currently being evaluated into the schema rules.  #315

Excellent, thanks for your quick reply!

On the readme, should I enumerate all the things available to this in the rules?

Thanks @Neobii . Released in 1.6.1


[functionality] Implement completion() methods

Most of the current inference providers only implement the chat_completion() method. The completion() method raises a NotImplementedError. We should implement this method for all the inference providers:

meta-reference
fireworks
together
ollama
bedrock

Would love to help. Will check the missing implementations.

Am currently working on the issue will update in hour

Would love to help. Will check the missing implementations.

am working on the issue too, should i stop?

Oh no, please continue! Are you going to implement all missing implementations? Maybe I can help with some of those.

Oh no, please continue! Are you going to implement all missing implementations? Maybe I can help with some of those.

i am will try to do some but i may need help with the other thanks!

@ashwinb are the kvstore_impl and EmbeddingIndex implementations being handled by the team internally?


fix: nav and search on mobile

🚀 Deployed on https://66fc30670c21d78ab54dea90--cozy-fox-0defec.netlify.app


ENH: added example notebook for xsvs ( skxray.core.spackle module fun…

…ctions)

@ericdill added an example notebook for xsvs analysis(skxray.core.speckle)

@ericdill I updated this after merging your PR

ready for review, only one top-level heading

Would you post an nbviewer link, please?

@danielballan here is the nbviewer link for xsvs analysis http://nbviewer.ipython.org/github/sameera2004/scikit-xray-examples/blob/xsvs/demos/speckle/X-ray_Speckle_Visibility_Spectroscopy.ipynb

At the end of this notebook can you add
import skxray
print(skxray.__version__)

Similar to your other PR, can you use https://github.com/tacaswell/xray-vision/blob/master/xray_vision/mpl_plotting.py#L90 for the ROI plotting?

@ericdill sure I will add version and use the X-ray_vision/mpl_plotting

Fixed this PR (added skxray version now using X-ray_vision/mpl_plotting)
Reday for review

Looks great!
Only comment is to use a linear color map in cell 5

@tacaswell sure thanks, which colormap you suggest? I am trying different ones didn't find a good one!

'viridis', failing that 'hot' or 'gray'

@tacaswell ready for review changed the colormap to viridias, ready for review
new nbviwer link
http://nbviewer.ipython.org/github/sameera2004/scikit-xray-examples/blob/xsvs/demos/speckle/X-ray_Speckle_Visibility_Spectroscopy.ipynb


Fixing path to aws_accounts.json

Should fix #672.
@AlexCline - If you can confirm that this fixes the issue, I'll push a fix to master as well.

Coverage remained the same at 60.323% when pulling 8eb8d3e5dcef7db9db11dc798c3aa234e8d586ad on issue_672_aws_accounts_path into 622b5afc0e0f31cd84ffb9645f12271da7312d76 on develop.

Coverage remained the same at 60.323% when pulling 128b559184277a391ce2b9093e0376cf5eef7d1c on issue_672_aws_accounts_path into 622b5afc0e0f31cd84ffb9645f12271da7312d76 on develop.

Hmmm, that only kind of works.
When I apply the patch to /usr/local/src/security_monkey/security_monkey/manage.py and run python security_monkey/manage.py, I don't get the error message.
When I run monkey, it's running the packaged version from /usr/local/lib/python2.7/dist-packages/security_monkey-0.9.0-py2.7.egg/security_monkey/manage.py.  If I apply the patch to that manage.py, it still looks in the package location for the file, and I get the same error message I reported.

ahh you're right.  I'll try another fix tomorrow or monday.


CSS

CSS
User Story
As a user,
I want the page to be formatted and aesthetic,
So that the page is pleasant and easy to read.
Acceptance Criteria
Given a landing page,
When I visit the page,
I will see formatted text and images that guide my experience through the page.

Added CSS