Feature Request: Disable Kanban Styling

Adding an option for disabling the styling for the Kanban plugin would be great. I am referring to the following lines of code:
https://github.com/colineckert/obsidian-things/blob/main/theme.css#L1020-L1076

Thanks @mudkipdev for creating the issue! I've added an option to the Style Settings plugin to disable the Kanban plugin styling. Lmk if you find any bugs. If not, please close this issue. thanks!

I'll let you know how the testing goes. Thanks.

Working fine, thanks!


How to install HDF on HDP

Hi
I have installed HDP using the Ansible playbooks. However, I would now like to install HDF as well. How can I do it?
Warm regards
Mayank

Hi @mayanksrivastav84
If you've already built your cluster, you won't be using these playbooks going forward. You'd have to follow: https://docs.hortonworks.com/HDPDocuments/HDF3/HDF-3.3.0/installing-hdf-on-hdp/content/hdf-upgrade-ambari-and-hdp.html
If you're building a new cluster, you have to simply add the NIFI_MASTER component in your blueprint_dynamic variable, to the node you want to run it, like here: https://github.com/hortonworks/ansible-hortonworks/blob/master/playbooks/group_vars/example-hdp3-singlenode-with-nifi#L37

Thank you !! That help.


feat(protocol-designer): minor copy change for aspirate delay tooltip

Overview
Closes #6448
Changelog
update tooltip
Review requests
Tooltip should match copy
Risk assessment
Low, just a tooltip

Looks good, should we do the same for dispense delay? @howisthisnamenottakenyet

On Slack she said nope, not until we do Dispense > Air Gap


Should we really use CC-BY 4.0 License?

Please consider the points mentioned on the Creative Commons website FAQ.
https://creativecommons.org/faq/

#Yes


Camera-orbit theta value rotates beyond 360 degrees

Description
When you rotate the model a lot in one direction, the theta goes over 360 degrees instead of wrapping back around. When setting the camera orbit theta back to 0deg, then the model overspins.
See here: https://nickel-longship.glitch.me/
This makes it hard to reset the camera back to the original position.
Browser Affected
All browsers
OS
All OS
Versions

model-viewer: 0.3.1
three.js: r101

Sounds good! We might be able to implement this by wrapping theta at 360 degrees. We should probably also fix interpolation so that it always moves across the shortest arc to the destination accounting for wrapping.

@elalish try the demo here: https://nickel-longship.glitch.me/


Reported errors don't agree with TypeScript IDE / CLI

Describe the bug

In the above you can see that the reported error is not reported in the IDE and I know it to be correct (e.g. this would pass in Jest before etc.).
Reproduction
Run tests on this branch https://github.com/jasonkuhrt/alge/tree/tests/try-vite-ts-plugin
System Info
Node 18, vitest 0.23

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

@jasonkuhrt This is the expected behaviour, expectType is doing a strict type equality comparison and in this case the input parameter in your create function is not a simple Record but an intersection of several Zod objects. In this case, if your goal is not to test the type equality but its assignability you should use expectAssignable.

@jasonkuhrt Maybe you want to test the tssert API which is a bit more flexible and less ambiguous.

@skarab42 Do you agree it is incorrect that the IDE and Vitest output disagree though?

@jasonkuhrt I don't see any errors in the IDE except the warning that says to avoid using unbound methods.

Sorry I misunderstood the question. It's normal not to have the error in the IDE nor with tsd. If you look at the implementation of the expectType function you see that it does strictly nothing and accepts all types/values. When the plugin encounters this function then it calls a custom version of the TS compiler to perform the type tests.

@skarab42 Ahhhh. Ok did not expect that. Do you prefer to work this way? Does it have to be this way? Is the idea to centralize all the "feedback" into one consistent place?
Not sure what I think yet.

@jasonkuhrt I really need to add more documentation to this project. Ideally I would like to use tsc directly but unfortunately I haven't found a way to do it and I don't think it's possible (but you never know until someone does). However, it is possible to make a plugin for vscode that could report the errors in the IDE (not in the near future). In any case, thank you for reporting.


Enforce consistent toString methods

Write a utility method in StringUtils that produces a consistent toString representation.
The method should have a signature something like this:
public static String makeToString(
    Class<?> type,
    Object... attributes
) { ... }

...where attributes must have a length that is a multiple of 2 and each even indexed item should be a string.
Idea of this being we can then use this like so:
public class Foo {
  private String bar;
  private String baz;

  ...

  @Override
  public String toString() {
    return makeToString(
        getClass(),
        "bar", bar,
        "baz", baz
    );
  }
}

...and expect output to be something like
Foo{bar="bar", baz="baz"}

A toStringBuilder may be best for this.
return new ToStringBuilder(this)
    .attribute("foo", this.foo)
    .attribute("bar", this.bar)
    .toString();


HBASE-23822:Fix typo in procedures.jsp

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  8s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  1s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
5m 58s
master passed

+1 :green_heart:
javadoc
0m 37s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m 34s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
javadoc
0m 36s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
90m 54s
hbase-server in the patch passed.

+1 :green_heart:
asflicense
0m 28s
The patch does not generate ASF License warnings.

106m 27s

Subsystem
Report/Notes

Docker
Client=19.03.5 Server=19.03.5 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1155/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1155

JIRA Issue
HBASE-23822

Optional Tests
dupname asflicense javac javadoc unit

uname
Linux 5392b960040d 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1155/out/precommit/personality/provided.sh

git revision
master / 24823ecfc9

Default Java
1.8.0_181

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1155/1/testReport/

Max. process+thread count
5789 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1155/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.


QueryProfilerWeight to extend FilterWeight

QueryProfilerWeight should override matches and delegate to the subQueryWeight. Another way to fix this issue is to make it extend ProfileWeight and override only methods that need to have a different behaviour than delegating to the sub weight.

thanks @iverase !


Dependency isn't available in the repo

[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	::          UNRESOLVED DEPENDENCIES         ::
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	:: com.bot4s#telegram-core_2.12;4.0.0-RC1: not found
[warn] 	:: com.bot4s#telegram-akka_2.12;4.0.0-RC1: not found
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::

I'll publish RC1 soon, staging is kind of convenient for me but not so for others.

Released RC1 to Maven, check the updated instructions. Please do report bugs or usability issues, I can also provide quick examples for some functionalities on demand.


Wrong variant names in Debug implementation for VecDiffType

In diff-struct 0.3.1 (latest released version) the Debug implementation uses Removed for all three (Removed, Altered, Inserted) variants of the VecDiffType enum.
impl<T: Diff> Debug for VecDiffType<T>
where
    T::Repr: Debug,
{
    fn fmt(&self, f: &mut Formatter<'_>) -> FmtResult {
        match self {
            VecDiffType::Removed { index, len } => f
                .debug_struct("Removed")
                .field("index", index)
                .field("len", len)
                .finish(),
            VecDiffType::Altered { index, changes } => f
                .debug_struct("Removed")
                .field("index", index)
                .field("changes", changes)
                .finish(),
            VecDiffType::Inserted { index, changes } => f
                .debug_struct("Removed")
                .field("index", index)
                .field("changes", changes)
                .finish(),
        }
    }
}

Hi, thanks for filing this issue! In the latest code this has been corrected, and I’m planning a new release soon; it’s just tricky with my schedule

Version 0.4.0 is out! The issue should be resolved in that version


Adding the required entities for our program

As the title says. We can modify the specifics of how they store data and what kind of data they store if needed.

making a new pull request, had to refactor import statements.


#157 - Added Maven Central keys and settings

@rultor merge

@rultor merge

@driver733 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@driver733 Done! FYI, the full log is here (took me 3min)


Failing FFI tests on CFE

The following ffi co19 tests are currently failing on the CFE (but not the analyzer).  The failures do not seem related to the class modifier feature, which is only used incidentally there.
co19/LibTest/ffi/Array/Array.multi_A01_t02
co19/LibTest/ffi/Array/Array.multi_A02_t01
co19/LibTest/ffi/Array/Array.multi_A02_t02
co19/LibTest/ffi/Array/Array.multi_A03_t02
co19/LibTest/ffi/Array/Array.multi_A03_t04
co19/LibTest/ffi/Array/Array_A02_t01
co19/LibTest/ffi/Array/Array_A02_t02
co19/LibTest/ffi/Array/Array_A03_t02
co19/LibTest/ffi/Array/Array_A03_t04
co19/LibTest/ffi/Struct/Struct_A02_t01
co19/LibTest/ffi/Union/Union_A03_t01
co19/LibTest/ffi/Union/Union_A03_t02
co19/LibTest/ffi/Union/Union_A03_t03
co19/LibTest/ffi/Union/Union_A04_t01

cc @dcharkes @johnniwinther

Looking at the first test it looks like it is only a matter where the error is reported. If so, the tests just need to be updated.
@dcharkes can you take a deeper look into this?

List of tests with logs: https://dart-ci.firebaseapp.com/current_results/#/filter=co19/LibTest/ffi

co19/LibTest/ffi/Array/Array.multi_A01_t02 -> error in different location, test needs to be updated
co19/LibTest/ffi/Array/Array.multi_A02_t01 -> https://dart-review.googlesource.com/c/sdk/+/293600 also error in different location
co19/LibTest/ffi/Array/Array.multi_A02_t02 -> error in different location
co19/LibTest/ffi/Array/Array.multi_A03_t02 -> https://dart-review.googlesource.com/c/sdk/+/293601
co19/LibTest/ffi/Array/Array.multi_A03_t04 -> https://dart-review.googlesource.com/c/sdk/+/293601
co19/LibTest/ffi/Array/Array_A02_t01 -> https://dart-review.googlesource.com/c/sdk/+/293600 also error in different location
co19/LibTest/ffi/Array/Array_A02_t02 -> error in different location
co19/LibTest/ffi/Array/Array_A03_t02 -> https://dart-review.googlesource.com/c/sdk/+/293601
co19/LibTest/ffi/Array/Array_A03_t04 -> https://dart-review.googlesource.com/c/sdk/+/293601
co19/LibTest/ffi/Struct/Struct_A02_t01 -> https://github.com/dart-lang/sdk/issues/46813
co19/LibTest/ffi/Union/Union_A03_t01 -> https://github.com/dart-lang/sdk/issues/46813
co19/LibTest/ffi/Union/Union_A03_t02 -> https://github.com/dart-lang/sdk/issues/46813
co19/LibTest/ffi/Union/Union_A03_t03 -> https://github.com/dart-lang/sdk/issues/46813
co19/LibTest/ffi/Union/Union_A04_t01 -> https://github.com/dart-lang/sdk/issues/46813

@dcharkes Have you filled co19 issues for the tests that need updates?


Create subsection "Nested workflow" in the docs

Moves the graph builder, if task and while task examples into this subsection. Also removed the graph_builder.ipynb since it is now generated. Solves issue #195

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

Project coverage is 79.63%. Comparing base (5937b88) to head (b4ed841).
Report is 56 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #277      +/-   ##
==========================================
+ Coverage   75.75%   79.63%   +3.88%
==========================================
  Files          70       65       -5
  Lines        4615     4906     +291
==========================================
+ Hits         3496     3907     +411
+ Misses       1119      999     -120

Flag
Coverage Δ

python-3.11
79.63% <ø> (+3.97%)
:arrow_up:

python-3.9
?

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

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

test_pause_task_after_submit seems a bit unstable it failed already two times in different jobs

I dont think the graph_builder example is really about the dynamic part as it just forwards inputs like any task does it. We could make another example that runs different tasks depending on the inputs. Something like
@task.calcfunction()
def add_one(x):
    return x.value+1

@task.calcfunction()
def modulo_five(x):
    return x % 5

@graph_builder(outputs = [...])
def my_modular(i: orm.Int):
    wg = WorkGraph()
    if i.value < 5:
        task = wg.add_task(add_one)
    else:
        task = wg.add_task(modulo_five)
    # need wg.selector to expose for linking?
    return wg

Then one could say that this does not preserve provenance and directly interlude to the If workgraph

Added an example for a dynamic use case of the graph_builder

Then one could say that this does not preserve provenance and directly interlude to the If workgraph

Why ti does not preserve provenance?

I would suggest not creating a dynamic-workflows section, but moving all three notebooks into the howto, so that user can see the dynamic, if and while immediately.

Okay, but the howto's starting to get messy and need some structure at some point, but I think for now one still can put everything there

Why ti does not preserve provenance?

I mean more that it does not store provenance as transparently as using the If WorkGraph, but looking at the provenance graph, It is not really capturing the if-then-else logic, but just only some additional information about the condition. But at least in the GUI the if-then-else is capture more transparently.

I merged now the nested and dynamic example because it seemed strange to see them separate


Send repository URL, not object

Current MsgFlo sends:
"repository": {
  "type": "git",
  "url": "..."
}

instead of just repository URL as specified in fbp protocol

@jonnor OK, found the problem. See ff4912b


Deployment: Areon Network Mainnet

Chain Name
Areon Network Mainnet
Chain ID
463
RPC URL
https://mainnet-rpc.areon.network
Block Explorer URL
https://areonscan.com
Deployment Funds
Yes

Done, thanks!

Hey, Can you please correct the chain id for Areon Network Mainnet from 464 to 463? It was entered incorrectly.

Done, thanks!

Hey, Can you please correct the chain id for Areon Network Mainnet from 464 to 463? It was entered incorrectly.

Oops, thanks for catching. Fixed in 49ac24b


[move-prover] Aborts-with check mode.

This introduces a new property for aborts_with to check whether an aborts specification is complete:
aborts_with [check] code1, code2, ...

... checks whether the given function aborts with the given codes, independent of any other aborts specifications. That is, if there is already an aborts_if P with code it will not influence the outcome of the aborts_with [check] codes condition.
By default, the check can only verify whether there are any missing abort codes. In order to check whether they are obsolete codes which can never occur, a negative check needs to be performed by passing --negative to the prover's command line. Negative checks are based on a feature we added as part of Kevin's internship (we expect a verification error and report an error if it does not occur). Using the --negative flag is still experimental and may not work in all cases as expected.
Details:

implements the aborts_with [check] functionality.
introducing checks whether pragmas and properties are used correctly. Previously, a typo in a pragma or property would have been silently ignored; this is not longer the case.
improvements on the negative verification infrastrucure.

Motivation
Support for transaction script documentation.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Added tests.
Related PRs
NA

/land


Parse and use platform.txt

INO manually assembled make recipes. Arduino now supplied ready made recipe templates in the platforms.txt file. Parse this and then replace most of the makefile logic in build.py to use these templates.

:1:
Sounds like a great idea for Arturo 2 refactoring

Yep. If you look at the develop branch you'll see I've already got this built. I'm currently working through library builds which is turning into a real pain.


chore: add pprof for stage

Adds the official pprof tool for the stage environment so the official Go utility can be used to attach to stage and investigate possible memory leaks. This is a temporary measurement, pprof will slow down stage a bit.
I was thinking maybe introducing a flag so we could turn it on, restart the app and have it, but I think I will be done after a week or two so we can remove this. Chances are that I will not be able find anything, the project is relatively big so pprof might not be too useful with too much data.
Example:
$ go tool pprof http://localhost:3000/debug/pprof/heap
Fetching profile over HTTP from http://localhost:3000/debug/pprof/heap
Saved profile in /home/lzap/pprof/pprof.app.alloc_objects.alloc_space.inuse_objects.inuse_space.001.pb.gz
File: app
Build ID: d805eb4a2bcc83604c2f0949f9d0e4d67ff21dea
Type: inuse_space
Time: May 20, 2024 at 7:03am (CEST)
Entering interactive mode (type "help" for commands, "o" for options)

(pprof) help
  Commands:
    callgrind        Outputs a graph in callgrind format
    comments         Output all profile comments
    disasm           Output assembly listings annotated with samples
    dot              Outputs a graph in DOT format
    eog              Visualize graph through eog
    evince           Visualize graph through evince
    gif              Outputs a graph image in GIF format
    gv               Visualize graph through gv
    kcachegrind      Visualize report in KCachegrind
    list             Output annotated source for functions matching regexp
    pdf              Outputs a graph in PDF format
    peek             Output callers/callees of functions matching regexp
    png              Outputs a graph image in PNG format
    proto            Outputs the profile in compressed protobuf format
    ps               Outputs a graph in PS format
    raw              Outputs a text representation of the raw profile
    svg              Outputs a graph in SVG format
    tags             Outputs all tags in the profile
    text             Outputs top entries in text form
    top              Outputs top entries in text form
    topproto         Outputs top entries in compressed protobuf format
    traces           Outputs all profile samples in text form
    tree             Outputs a text rendering of call graph
    web              Visualize graph through web browser
    weblist          Display annotated source in a web browser
    o/options        List options and their current values
    q/quit/exit/^D   Exit pprof

  Options:
    call_tree        Create a context-sensitive call tree
    compact_labels   Show minimal headers
    divide_by        Ratio to divide all samples before visualization
    drop_negative    Ignore negative differences
    edgefraction     Hide edges below <f>*total
    focus            Restricts to samples going through a node matching regexp
    hide             Skips nodes matching regexp
    ignore           Skips paths going through any nodes matching regexp
    intel_syntax     Show assembly in Intel syntax
    mean             Average sample value over first value (count)
    nodecount        Max number of nodes to show
    nodefraction     Hide nodes below <f>*total
    noinlines        Ignore inlines.
    normalize        Scales profile based on the base profile.
    output           Output filename for file-based outputs
    prune_from       Drops any functions below the matched frame.
    relative_percentages Show percentages relative to focused subgraph
    sample_index     Sample value to report (0-based index or name)
    show             Only show nodes matching regexp
    show_from        Drops functions above the highest matched frame.
    source_path      Search path for source files
    tagfocus         Restricts to samples with tags in range or matched by regexp
    taghide          Skip tags matching this regexp
    tagignore        Discard samples with tags in range or matched by regexp
    tagleaf          Adds pseudo stack frames for labels key/value pairs at the callstack leaf.
    tagroot          Adds pseudo stack frames for labels key/value pairs at the callstack root.
    tagshow          Only consider tags matching this regexp
    trim             Honor nodefraction/edgefraction/nodecount defaults
    trim_path        Path to trim from source paths before search
    unit             Measurement units to display

  Option groups (only set one per group):
    granularity
      functions        Aggregate at the function level.
      filefunctions    Aggregate at the function level.
      files            Aggregate at the file level.
      lines            Aggregate at the source code line level.
      addresses        Aggregate at the address level.
    sort
      cum              Sort entries based on cumulative weight
      flat             Sort entries based on own weight
  :   Clear focus/ignore/hide/tagfocus/tagignore

  type "help <cmd|option>" for more information

(pprof) text
Showing nodes accounting for 1024.12kB, 100% of 1024.12kB total
Showing top 10 nodes out of 11
      flat  flat%   sum%        cum   cum%
  512.07kB 50.00% 50.00%   512.07kB 50.00%  github.com/aws/aws-sdk-go/aws/endpoints.init
  512.05kB 50.00%   100%   512.05kB 50.00%  regexp/syntax.(*parser).newRegexp (inline)
         0     0%   100%   512.05kB 50.00%  github.com/go-openapi/strfmt.init
         0     0%   100%   512.05kB 50.00%  regexp.Compile (inline)
         0     0%   100%   512.05kB 50.00%  regexp.MustCompile
         0     0%   100%   512.05kB 50.00%  regexp.compile
         0     0%   100%   512.05kB 50.00%  regexp/syntax.(*parser).literal
         0     0%   100%   512.05kB 50.00%  regexp/syntax.Parse (inline)
         0     0%   100%   512.05kB 50.00%  regexp/syntax.parse
         0     0%   100%  1024.12kB   100%  runtime.doInit

I am also decreasing the parallel job run from 2 to 1 to ensure only one job runs at a time for easier debugging.

Ok here is how to get a dump:
curl --proxy http://xxxxxx.proxy.redhat.com:1234 -u "user:password" https://console.xxx.stage.redhat.com/api/edge/debug/pprof/heap -o heap.dump

go tool pprof heap.dump
> top

go tool pprof -http=:8082 heap.dump

See the official docs for more info.


Mark as Deleted Data Models are not visible to Administrators

Description
For Data Models that are created in sub-folders, when the Show Deleted Models filter is checked, Mark as Deleted data models do not appear in the Tree.
Steps to reproduce

Login as administrator
Find a data model that is in a sub-folder in the browse tree
Mark the data model as deleted
Use the filter in the Browse Tree to Include Deleted models or set user preferences to see Deleted data models (User>Preferences>Show Deleted Models)
Refresh page the Browse Tree

Expected behavior
The data model appears in the sub folder with Strikethrough text
Actual behaviour
The deleted data model does not appear in the sub folder
Environment
Desktop

OS: Windows 10
Browser: Chrome
Version: 103.0.5060.114

Additional context
The expected behaviour may have changed due to this update to mdm-core: https://github.com/MauroDataMapper/mdm-core/commit/21ec326adc1f8a720542bde452312b70cb08d0c1#diff-56322d760db078f3c89b053994f54fdff24ee21cf2a735596f92f1a0c74c50cf

This appears to be fixed in the 7.3.0 release.
With option to see deleted models (as an administrator):

With option turned off:

Closing this issue as a result.


When support caddy2?

When support caddy2?

I has not time for this job.
You can try replace "FROM" in caddy/Dockerfile to "caddy:2.0.0". I think its must work/


Emissions factsheet button not working

The 'year button' on the emissions factsheet is not working. The start year also shows the future scenario outcome instead of the current emissions. Example: https://pro.energytransitionmodel.com/regions/GM1586_oost_gelre?scenario=846488&time=present.

What exactly happens?
When I press the button using the link you sent, I see the values changing:

In this example the area has 366 KT emissions...

...whilst here is has 349 KT emissions. Both are numbers from 2018 and therefore should not diverge.

We thought this was due to a bug in the button?

Yes, I understand now, thanks!
I still can not reproduce this for other scenarios then this one. In this scenario it looks like 17kT of emissions have appeared in the built environment for the present day. Can this possibly be due to the electricity_interconnector_1_co2_emissions_present you uploaded? That's the only present-input I can see that was set.


Copy required jar file containing package "org.webrtc.voiceengine"

Thanks!

I think this pull request addresses issue #220


[GPoS] Enable Slashing for Staking Module

Fix #134

Compared to the init version, slashing module doesn't change much only except for this PR which is used for substrate's new weight mechanism. So I didn't modify this one.
Slashing is based on Stakers and Exposure, which we didn't change before. So actually it works well.
Tests about slashing are running just like they were in the substrate.

Looks great


React to Logging API changes

https://github.com/aspnet/Logging/issues/331

Hi @BrennanConroy, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Build is currently broken
https://github.com/aspnet/Security/issues/663

:shipit:


Issue with Importing positional_marked_npairs_3d from halotools.mock_observables.pair_counters

I've encountered an issue when trying to run my code which utilizes the halotools library. Specifically, I'm having trouble importing the positional_marked_npairs_3d function from the halotools.mock_observables.pair_counters module.
Here's the error traceback I received:
Traceback (most recent call last):
File "C:\Users\82103\Desktop\graphgan\GraphGAN\2d_alignment.py", line 12, in
from halotools_ia.correlation_functions import ed_3d, ee_3d, ed_3d_one_two_halo_decomp
File "C:\Users\82103\Desktop\graphgan\GraphGAN\halotools_ia\correlation_functions_init_.py", line 11, in
from .ed_3d import ed_3d
File "C:\Users\82103\Desktop\graphgan\GraphGAN\halotools_ia\correlation_functions\ed_3d.py", line 15, in
from halotools.mock_observables.pair_counters import positional_marked_npairs_3d, npairs_3d, marked_npairs_3d
ImportError: cannot import name 'positional_marked_npairs_3d' from 'halotools.mock_observables.pair_counters' (C:\Users\82103.conda\envs\galaxygan\lib\site-packages\halotools\mock_observables\pair_counters_init_.py)
I've tried using both version 0.8.0 and 0.8.2 of halotools, but encountered the same error in both versions. I've verified my halotools installation and even tried different versions, but the issue persists. I'm wondering if this function has been moved to a different module or if there might be some other underlying issue I'm not aware of.
Any guidance or assistance on this matter would be greatly appreciated.
Thank you for your time and consideration.

@mi-song I think I see what happened, please let me know if this helps.
We indeed did move some things around. Specifically, halotools_ia (from your import) no longer exists and had been merged into halotools itself.
The import for what you want should now be
from halotools.mock_observables.ia_correlations import ed_3d, ee_3d, ed_3d_one_two_halo_decomp
Please try this out and let me know if it fixes things. If not, ping me on your next reply and I'll keep looking into it

@mi-song did this resolve your issue?


Invalid or unexpected token

How to fix? TOKEN and CLIENTID wrote everything correctly

Nope

Go to src/player/index.js and in:
player.on("error", (error, queue) => {         client.logger.error(
change it with:
player.on("error", (error, queue) => {         console.log(error) client.logger.error(

Send full error

fulll

Bro I don't know what's wrong bc it works for me. Try to update ytdl-core by using npm i ytdl-core@latest and tell me if it worked

Thank you, it helped


Open Categories in RecyclerView

On clicking categories , the categories must open in a recycler view.

Working on it.


Display in aeromodeling club

Aeromodelling club has a bug in display of image

@evilstarksai  will you be a little more specific about this issue ....

@evilstarksai

Can you note down where you are getting a bug ...

@Shreyasi2002 it is in the background image.

@evilstarksai  I think the background image is not being rendered properly on your side. It is working fine on our side.


[PoC] db user translation layer

Sample Translation layer for your consideration:

The packages under internal/translayer contain per Altas resource translation layers for users and deployments.

Files in the controller package should not import the SDK library to call the API or use its types.

All translations and upgrades are confined to the translayer code.

Contract tests would use the translation layer directly.

This layout also enforces proper translations and comparisons, no more funky CMP or JSONCopies.

See commit 142314606ac35d3d5d5568c1f6acccdf10c87bc8 for a version when the translation layer is on an atlas.go file within the same package using the old Go API client

See commit  5d11f7cf28ca20b62e7e589356798a80fc4318de for the migration changes to the nee SDK alone.

All Submissions:

[X] Have you signed our CLA?

general question: how does the "contract" look like which the translation layer seems to be a prerequisite for?

The contract is not really fully defined until you have the tests with the validations that check precisely the contract behavior. Here we only have the shape of the interactions in the service methods and the translations we make.
This is NOT the contract test PoC, just the translation layer, which is probably more than half the code required.
What is left now is the helper code to be able to put the API side in the state required to be able to test the API calls in the service in the situations that define the contract with the Atlas. Those tests inform how and when the service calls defined in the translation layer should be called from the controller to achieve the expected results. Which would allow to test the reconciliation without the API with a great deal of confidence, other than if something changes on the API side, which the contract tests will detect.
Client Contract Tests do NOT need to cover all API behaviours, just the ones our client cares about.

general question: how does the "contract" look like which the translation layer seems to be a prerequisite for?

The contract is the service interface. On the other PoC I made that explicit, on this I will add the interface now.
The only requisite of the contract is that it only accepts and returns internal or Kubernetes types, it completely encapsulates Atlas SDK types as a lower protocol layer.

Will draft this until I rebase and add the unit tests and other suggested fixes

Lesson learned:
This PR is bigger and took way longer than I thought it would or should. Here is what I should instead have done in hindsight:

TDD the translation layer alone in isolation, do not connect it yet.
Have another branch on top experimenting with the usage from the rest of the code, keep it as draft only.
Post the translation layer PR alone in the internal package, unused but fully tested with both unit tests and contract tests.
Merge translation later.
Rebase the branch using it as a new PR including unit tests and post that.

Even though has more steps it would have been easier to debug and review, so the whole process would have been faster in comparison IMHO.


GH workflow triggered on release is failing

https://github.com/mtxr/vscode-sqltools/blob/6eaa1c266a33ef6916e5263081f781326d105d4d/.github/workflows/upload-release-assets.yml#L40
That file was deleted in this commit:
https://github.com/mtxr/vscode-sqltools/commit/c4b163886d245c680efb630c5ab8125de8ce5c0c

Log files of example failed run
logs_2486.zip

I need to revisit this guy. I believe things have changed here since the last release. I'll keep you posted


how to make a server softwere in python or c?

i want to make my server softwere becouse here are lot of but not alll are good enoth for me or the plugin coding uses languages that idk
is posible and basics haw i make a basic one so i earn and make a better one

You need to read through https://wiki.vg/Classic_Protocol and understand everything that is said on that page at least

and i use soket progrming for this ?
also how you conect to servers if you use the the classicube user and what are the Fallback IDs?

Yes, you need TCP sockets
I'm not sure what fallback IDs is referring to

Yes, you need TCP sockets
I'm not sure what fallback IDs is referring to

to what?

everywere are is like this 0x00  how i use them?

For client -> server packets:
packet_id = read_byte_from_socket();
if (packet_id == 0x00) handle_server_identification();
if (packet_id == 0x05) handle_set_block();
etc

For server -> client packets:
write_level_init() {
  char data[1];
  data[0] = 0x02;
  write_to_socket(data, 1);
}
etc


Add option to apply QoD to subnets

Problem description
The current API only allows for policies to be applied to a single IP address.  However, more homes are beginning to segment their networks and could have different WoD settings based on the subnet.
For instance, traffic on a home office subnet would be treated differently than a guest or IoT subnet.
Possible evolution
There are several approaches to solve this.
One is to replace the ipAddress property with networkCIDR
For IPv4, the would be a network address followed by a network mask.  <IP_ADDRESS>/24 or to specify a single IP <IP_ADDRESS>/128
To preserve backwards compatibility, the API calls could be changed to:
oneOf:
  - ipAddress
  - networkCIDR

Alternative solution

Additional context

@RandyLevensalor Thank you for your feedback. It is indeed interesting. But according to the OpenGateway procedure, new features should be handled this way:
GSMA OGW Product WS --> OGW Backlog Prioritization (Voting) --> CAMARA Backlog
If you want to add new features that affect the functionality of the service (not just technical changes to an existing one), it's preferable to follow this path, since GSMA is the one that manages priorities at the product level. Let us know if you need assistance in reaching this track. (CC @jgarciahospital)
In any case, from my perspective, the extension you suggest is probably more appropriate for an internal Operator application, where the user has a "control panel" or similar to manage their home network resources, devices and configurations (including subnets). But in the CAMARA context, where Operator exposes APIs to third party developers, the subnet information may not be known/accessible to the developer (exposed information is limited). I don't quite see the use case for an app (say Zoom app) running on my laptop (connected to my home network) requesting to apply QoD for a "work" subnet instead of the device IP corresponding to my laptop...

Closed as there were no further objections and the Open Gateway procedure for requesting new features was mentioned.


A way to set PriceScale width - currently there's no way to keep two charts' PriceScale width in sync one above the other

I have two charts (one for price, one for volume). I keep them in sync by usingthe various API observer functions. Works wonders.
Except the price scale width - I can get it using the priceScale('right').width() call, but I don't see any way to set it.
I'd like to be able to keep the width of both chart's price scales in sync (and keep them set to the max width of the two).
Thanks!

Bump!

Man, how did you display two panels on one chart?

I have two charts (one for price, one for volume). I keep them in sync by usingthe various API observer functions. Works wonders.
Except the price scale width - I can get it using the priceScale('right').width() call, but I don't see any way to set it. I'd like to be able to keep the width of both chart's price scales in sync (and keep them set to the max width of the two).
Thanks!

Man, how did you display two panels on one chart?

@matthewwwz It's two different charts...

Hi @quakemmo!
Have you got a jsfiddle or something else where we could reproduce the issue and check how to fix it if possible?

@romfrancois Hello, and thank you for the reply.
I don't think there's need for a jsfiddle - you can just grab any working LWC and the issue will apply to it - ie: you cannot manually adjust price scale width.
I just need a way to set the price scale width (so I can have several charts, find the widest price scale, and set all the charts' price scale to that width).

@romfrancois Bump <3
Do you need me to set a jsfiddle up?

Unfortunately that's not possible at the moment.

@romfrancois I see, thank you for your reply.
Is it a feature that you guys might consider adding?

Not in the near future no but we've tagged this ticket to keep an eye on it.

@romfrancois Thank you.
So right now, the recommended way to display a candlestick price graph along with a bar volume graph (for example), one below the other, is to just create ONE chart with two data series?
I took the approach of creating two different charts, and listening to their scrolling/zooming events to keep them in sync, did I go an unnecessary complicated route?
I'd like to achieve a result similar to what you can see here:
https://www.kucoin.com/trade/BTC-USDT
As you can see they seem to use two different charts that are kept in sync, yet somehow manage to keep the right price scale at the same width.

@quakemmo
It is not currently possible to exactly match what you can see on the link you share but Lightweight Charts does support volume series within the same chart so you don't need to manually synchronise anything.
Check out this example https://jsfiddle.net/TradingView/cnbamtuh/

Bump. I'd like to do this but not with volume, but arbitrary indicators, plotted below and synced.

Related: https://stackoverflow.com/a/73209722/1934064

Hi @quakemmo

I have two charts (one for price, one for volume). I keep them in sync by usingthe various API observer functions. Works wonders.
I took the approach of creating two different charts, and listening to their scrolling/zooming events to keep them in sync, did I go an unnecessary complicated route?

Do you mind to share more details about this? some code example maybe? I also need to keep the 2 separated charts in sync, and it seems that it is more complicated than I thought. Thank you very much!

Looking forwards for merge #1412
As a temporary solution I found effective adding extra symbols in price formatter.
For example knowing that you have large digit (i.e. bitcoin 30,000.00), you can kind of create min width for the priceScale panel
Of course not ideal and resulting string width highly depends on the font
			localization: {
				priceFormatter: (price: number) => {
					let priceStr = formatPrice(price);
					const distance = 9 - priceStr.length;

					if (distance > 0) {
						const whitespaces = [...new Array(distance)].map((_) => '_').join('');
						priceStr = `${priceStr}${whitespaces}`;
					}
					return priceStr;
				}
			}

@edew With the minimumWidth option this still doesn't address syncing the charts. I rather not set both charts with a fixed min width, and instead have them use the max of either.
To sync a 2nd chart with a 1st, I've come up with this:
setTimeout(() => {
chart2.priceScale('right').applyOptions({
    minimumWidth: chart1.priceScale('right').width(),
})
}, 0)

However, chart1.priceScale('right').width() is returning 0 for me, so I had to put it in a setTimeout to have it execute on the next loop. Should I open a new issue, or could this be re-opened?

priceScale('right').width() returning 0 before the chart has been rendered is expected. If you need to measure the width at startup then using setTimeout is the correct approach.

I rather not set both charts with a fixed min width, and instead have them use the max of either.

Setting the minimum width of the one chart to the width of the other (wider) chart does allow you to ensure that both price scales are the same width (maximum of the two).
You can use the subscribeSizeChange event on the timescale to assist with keeping all the price scales in sync.

I guess I was over thinking something like minWidthRef: otherChartRef. I've changed my code to adjust the minimumWidth on the subscribeSizeChange callback, and now no timeout tricks needed.
Is there a subscription for price scale change? I noticed if I scaled the price from 3 to 4 digits on the top chart then I need to adjust the minimumWidth again for the bottom chart.

You should be able to use the subscribeSizeChange for knowing when the price scale width changes since if the chart width is constant then the time scale width needs to change if the price scale width changes. Hope that is making sense.

I have 2 charts and the top chart I have disabled the timescale as I use the bottom charts timescale. For the top chart, I get a single callback with (width: 0, height: 0) when I have the option { timeScale: { visible: false } }, but I do get calls when it's set to true.

There is a paneSize method which you could use to get the width of the chart pane (which would be the same as the time scale if it was visible).
If you need a utility for visible price range then you could try this plugin: lwc-plugin-visible-price-range-util
A similar approach could be used to create a plugin which provides a subscriber for paneSize changes, or PriceScale width changes.

Thanks, lwc-plugin-visible-price-range-util works great!


Unify library into using consistent

Normally I don't mind that some of the other components are using different Vue styles that I am. However, I encountered a situation where because they are not uniform (and I'm using true TS syntax with defineProps) that I cannot make the component I'm building true dynamic with the other component because it does not use true TS itself. -- Meaning the component uses BLink but I cannot import those props as a type -- I think a slight rewrite to make them true TS script setup syntax could make the library work better as well as offering better typings to the end user. (Some components I have found are missing typings on emits for example, providing this would be beneficial to the user.). I could start building it, as it wouldn't take too long, however, I need the go ahead for a major change on other peoples components.
Edit: In addition to this, I think we could make it clear in the docs when one component props extends another.

I agree. I think it might make sense to target components that have tests with 'expected default structure`. This will minimize any initial conflicts/errors as the transition is made.

What do you mean? Can you give an example of one?

I agree. I think it might make sense to target components that have tests with 'expected default structure`. This will minimize any initial conflicts/errors as the transition is made.

@mariusa You seem to be very consistent with merging my requests. If I were to rewrite the application to full script setup syntax, with full(er) TS support, would it be merged? This would also include removing the need for some of the utility classes for omitting props and the likewise. I just would like to know if I put in the effort to enhance the library, that it would not be immediately rejected. As it would take a bit of time. Thanks

Yes, of course, that would be great! As long as the current functionality is kept (ie doesn't break existing components).
It would also help to send smaller PRs more frequent, rather than a big one once, if these changes can be split into some functional milestones. This way we also avoid conflicts with other small potential changes.
Thank you!

Yes, of course, that would be great! As long as the current functionality is kept (ie doesn't break existing components). It would also help to send smaller PRs more frequent, rather than a big one once, if these changes can be split into some functional milestones. This way we also avoid conflicts with other small potential changes. Thank you!

Okay, good. I only ask because a rewrite would take a bit of time and would be a shame for it to be wasted. I don't foresee any major issues.  Most of the code is retained and the only things to change is usually the way props and emits are defined. Some components are quite easy, some other edge cases, like some components built with render functions, or some components with nested components in them, tend to be harder. How do you think I should create these milestones?

@kwiksilver3441 several components will have test cases. Some have simple test cases that test the default initial render. I would target components with test cases first.

@kwiksilver3441 several components will have test cases. Some have simple test cases that test the default initial render. I would target components with test cases first.

I was planning on just converting alphabetically. To make things easy. I haven't reviewed any of the tests as they are somewhat rare.

SFC's that require complex interfaces -- i.e. those that extend other components props. May not yet be capable of being fully transformed into script setup syntax. I elaborated on this in my last PR. But, the workarounds I attempted to use resulted in errors when trying to import the default props object of the extended component, it results in an error of The 2nd argument of withDefaults must be an object literal.. Many of the components can be converted easily, however, the edge cases that require extensions are simply not able to be easily updated at this time. This will hopefully get fixed in a future update of Vue. But for now, I will avoid updating complex component cases.
In other news, those that are simple components, are growing well with some good TS enhancements.

Oh good , yeah I noticed the library is not building properly due to the several types of errors I am seeing. Is there a way to build the library circumventing the typescript errors at the moment?

Oh good , yeah I noticed the library is not building properly due to the several types of errors I am seeing. Is there a way to build the library circumventing the typescript errors at the moment?

Probably not. But j could focus on fixing them first if you'd like


IOS: Argument list too long: recursive header expansion failed

I am getting below error while running from Xcode.
Argument list too long: recursive header expansion failed at /node_modules/react-native-youtube/../../ios/Pods/boost-for-react-native/boost/numeric/odeint/iterator/detail.
My packages versions
"react": "16.3.1",
"react-native": "^0.55.4",
"react-native-youtube": "^1.0.1",

Same!
"react": "16.2.0",
"react-native": "0.53.0",
"react-native-youtube": "1.1.0",

Getting same issue. Any suggestion?

When this happens, I remove node_modules, ios/build, ios/Pods then yarn install, pod install.
It is pretty annoying indeed.

This one helps me resolve the problem
Duplicate of #286

I tried #286 solution and @paour one but didn't work for me, still getting the issue.

Also getting this issue. #286 did not work for me.

@marlon-wiprud what version of this library do you use?

Solution: remove .ios/build file and build the project again.


Font-family is set to verdana in style attribute

I've got the following code:
	var pie = new d3pie("my-chart", {
		"header": {
			"title": {
				"text": "My Overview",
				"fontSize": 22,
				"font": "helvetica"
			},
			"subtitle": {
				"text": "My Subtitle",
				"color": "#999999",
				"fontSize": 10,
				"font": "helvetica"
			},
			"titleSubtitlePadding": 12
		},
		"footer": {
			"color": "#999999",
			"fontSize": 11,
			"font": "helvetica",
			"location": "bottom-center"
		},
		"size": {
			"canvasHeight": 500,
			"canvasWidth": $("#my-chart").innerWidth(),
			"pieOuterRadius": "90%"
		},
		"data": {
			"content": data
		},
		"labels": {
			"outer": {
				"format": "label-value1",
				"pieDistance": 32
			},
			"inner": {
				"hideWhenLessThanPercentage": 2
			},
			"mainLabel": {
				"font": "helvetica"
			},
			"percentage": {
				"color": "#ebebeb",
				"font": "helvetica",
				"decimalPlaces": 0
			},
			"value": {
				"color": "#929292",
				"font": "helvetica"
			},
			"lines": {
				"enabled": true,
				"style": "straight",
				"color": "#cccccc"
			},
			"truncation": {
				"enabled": true
			}
		},
		"tooltips": {
			"enabled": true,
			"type": "placeholder",
			"string": "{label}: {value}, {percentage}%",
			"styles": {
				"fadeInSpeed": 260,
				"backgroundOpacity": 0.7
			}
		},
		"effects": {
			"pullOutSegmentOnClick": {
				"effect": "none",
				"speed": 400,
				"size": 8
			}
		}
	});

But when I look at devtools in Chrome the font is coming up as Verdana:

And I've spotted that it's got a font-family style attribute in the tag which is forcing verdana:

Do you know why this is happening, and how I can change it? The d3pie.js file doesn't mention verdana anywhere in it from a cursory Ctrl+F, so I'm not sure where it's coming from but I wonder if you could point me in the right direction?

Problem was between screen and chair - my JS was cached in the browser without me realising.


Update codecov.yml

trying to see if forcing 1.20.4 works

fixed upstream so now check-latest does pick up 1.20.4


🇷🇺 Russian translation

Hi, this took a little time but I finally managed to squeeze it out. Russian translation inbound!

Thanks @notbogdan I'm gonna wait some else that knows Russian language to review.

please help! 🦸♀

Someone "productionized" the translation here, this is a much better version which I will try and merge into this branch at some point. This one should be good to merge ^^

Someone "productionized" the translation here, this is a much better version which I will try and merge into this branch at some point. This one should be good to merge ^^

Oh sorry, I think you may have misunderstood - I meant that the translation I linked would've been good to merge, not as it is in its current form. No worries, I will open another PR.

Oh sorry, I think you may have misunderstood - I meant that the translation I linked would've been good to merge, not as it is in its current form. No worries, I will open another PR.

sorry @notbogdan =(

sorry @notbogdan =(


Add spacer block for a delayed first trigger

See #86

Proposed shortcode:
{{< cr-spacer >}}

Adds:
:::{.cr-spacer}
:::

And defined as:
.cr-spacer {
  visibility: hidden;
  padding-top: 20svh;
}

#103 ready for you, @andrewpbray!

Merged!


jspm install fails

tried to install via jspm and got following stdout:
 jspm install npm:skatejs-web-components
     Looking up npm:skatejs-web-components
     Updating registry cache...
     Looking up npm:document-register-element
     Looking up npm:skatejs-named-slots
     Downloading<EMAIL_ADDRESS>
warn Error on processPackageConfig
     Package.json dependency webcomponents.js set to github:webcomponents/webcomponentsjs#b77ca74, which is not a valid dependency format for npm.
     It's advisable to publish jspm-style packages to GitHub or another registry so conventions are clear.
     To skip npm compatibility install with jspm install<EMAIL_ADDRESS>"{jspmPackage: true}".

warn Error processing package config for npm:skatejs-named-slots.
ok   Installed<EMAIL_ADDRESS>(1.0.9)

err  Error processing package config for npm:skatejs-named-slots.

warn Installation changes not saved.

Hey @vedam I don't think this is an issue anymore since we're including document-register-element instead. Can you confirm?

confirmed. thx


on sideview by using GetCapabilities a traceback occured

Fatal error in MSS 8.3.1 on Linux-6.5.0-10008-tuxedo-x86_64-with-glibc2.35
Python 3.10.8 | packaged by conda-forge | (main, Nov 22 2022, 08:23:14) [GCC 10.4.0]
Please report bugs in MSS to https://github.com/Open-MSS/MSS
Information about the fatal error:
Traceback (most recent call last):
File "/home/reimar/PycharmProjects/MSS/mslib/msui/wms_control.py", line 596, in on_success
self.activate_wms(wms)
File "/home/reimar/PycharmProjects/MSS/mslib/msui/wms_control.py", line 783, in activate_wms
self.prefetch.disconnect(self.prefetcher.fetch_maps)
TypeError: disconnect() failed between 'prefetch' and 'fetch_maps'

This seems not to happen often, no user complaining yet,  I think we can shift it to 10.0

I've seen this kind of issue coming from wms_control.py when trying to close all widgets after each test. There seemed to be some kind of race condition in the object destruction. I think this issue is more pronounced when opening and closing the windows happens with little delay, as it happens in the test suite. So maybe this is less of an issue in the real world, as you say.
Also, I have no clue yet on how to fix this.


Damage counter

New ferumbras gets tracked as a player instead of creature  maybe its because how its wrote
14:23 xxxx loses 23 hitpoints due to an attack by Ascending Ferumbras.
14:26 You lose 370 hitpoints due to an attack by Ferumbras Mortal Shell.
and would be awsome to filter the damage ive received only or x or y player received

The reason for that is because the creatures Ascending Ferumbras and Ferumbras Mortal Shell are not known as creatures to Tibialyzer. Tibialyzer gets all the creature information from the tibia wikia, and apparently these two creatures are not present on tibia.wikia.com or have no image associated with them. If you want to fix this issue, you can do so by contributing to tibia.wikia.com and adding these creatures.

Creatures now have pages and placeholder images.

I have updated the database in commit 02ece8b71eb081eff8591fa65b929874e713752c to reflect the latest state of tibia wikia. So now Ascending Ferumbras and Ferumbras Mortal Shell are creatures, which should resolve this issue. I also updated the latest release, which you can download from here.


Can't compile on ubuntu 16.04

Hi,
I have the following error while running bootstrap.sh
Useless use of /d modifier in transliteration operator at /usr/share/automake-1.9/Automake/Wrap.pm line 60.
Makefile.am: C objects in subdir but AM_PROG_CC_C_O' not in configure.ac'
autoreconf: automake failed with exit status: 1

Did adding the AC prog line to configure.ac allow bootstrap to complete?

Well actually I had to install automake... :S
and now it works!


Looks like wttr.in is down

I believe wttr.in is down. https://wttr.in/ link is not working anymore and shows the following error

OK here? And https://www.isitdownrightnow.com/wttr.in.html

It is working now. Closing the issue.


Is there any way to get callback on if candlestick is not present on left (if moving left to right chart )

Here timescale is not present (which area i am taking about if timescale is not present in visible range at most left )

Sorry, I'm not sure what exactly the question is asking.
You might be interested in using the subscribeVisibleTimeRangeChange or subscribeVisibleLogicalRangeChange methods to be notified when the visible range is changed and then you can check if the left most bar is visible or not.


feat: Show newest version number

Show version number of new release in outdated warning.

Adds a new TypeScript object in src-vue/src/utils/FlightCoreVersion.d.ts that mirrors the Rust object. As discussed, uh somewhere, we wanna auto-generate TypeScript objects from Rust code directly but I haven't looked into that yet and I wouldn't block the PR because of it ^^


mix.extract() causing the resources processing to freeze with 95% emitting message

Laravel Mix Version: 1.0.3 (npm list --depth=0)
Node Version (node -v): 6.10.3
NPM Version (npm -v): 5.0.3
OS: Windows 8.1

npm list --depth=0

+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>`--<EMAIL_ADDRESS>
npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>
Description:
Hi artisans,
Can anybody confirm this (Especially on Windows) or am i the only one who have this issue !!
Steps To Reproduce:
Start with a new laravel installation
Edit the webpack.mix.js file like this
const { mix } = require('laravel-mix');

mix.sass('resources/assets/sass/front/app.scss', 'public/css')
   .sass('resources/assets/sass/back/admin.scss', 'public/css')
   .options({
       processCssUrls: false,
   });

mix.autoload({
    jquery: ['$', 'window.jQuery', 'jQuery']
});

mix.js('resources/assets/js/app.js', 'public/js');

mix.extract([
    'axios', 'vue', 'jquery', 'bootstrap-sass',
], '/assets/js/vendors.js');

After that run: npm run dev
λ npm run dev

> @ dev d:\...\test-mix
> npm run development

npm WARN invalid config loglevel="notice"

> @ development d:\...\test-mix
> cross-env NODE_ENV=development node_modules/webpack/bin/webpack.js --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js

 95% emitting

And if you comment the mix.extract([...], '/assets/js/vendors.js');, the build pass !!
Also, i want to point something weird with mix-manifest.json:
{
    "/d:/assets/js/vendors.js": "/d:/assets/js/vendors.js",
    "/js/app.js": "/js/app.js",
    "/css/app.css": "/css/app.css",
    "/css/admin.css": "/css/admin.css",
    "/js/manifest.js": "/js/manifest.js"
}

I think the first line "/d:/assets/js/vendors.js": "/d:/assets/js/vendors.js", is invalid because it starts with /d:/assets !!

I was getting the same thing. I removed my package-lock.json file and ran npm install again. It was fine. Try that otherwise repeat the same as above except in your package.json change laravel-mix version 0.*

Same issue here. Keeps freezing on Windows 10 with latest Laravel-mix version..

UPDATE: I've upgraded the laravel-mix to 1.0.7 and still have the same issue.
And it doesn't matter if you install the packages with npm or yarn.

Got the same issue but with mix.js() as well as with mix.extract() on win10. Not on my Linux though.

@arcanedev-maroc
'mix.setPublicPath('./');' solves this for me - can you confirm?

Hi @PascaleBeier,
Thanks for your suggestion but it doesn't work for me.

I use Laravel Mix also for non-Laravel projects where is no artisan file which, apparently, is being used by Laravel Mix to detect Laravel (in which case publicPath is being set to public). I was struggling with this issue after upgrading from version 0.8.8 to version 1.0.7 until I found out what was the cause of that 95% emitting message.
So, yes, @PascaleBeier is [almost] right, you should set publicPath like this:
mix
.options({
    publicPath: 'public'
})

Unfortunately, it doesn't solve my issue.
I'm using fresh laravel installation (v5.4) to test the new laravel-mix (v1.*).
@taai, i tried your solution but with no success :
const { mix } = require('laravel-mix');

mix.options({
    processCssUrls: false,
    publicPath: 'public'
});

mix.sass('resources/assets/sass/app.scss', 'public/css');

mix.autoload({
    jquery: ['$', 'window.jQuery', 'jQuery']
});

mix.js('resources/assets/js/app.js', 'public/js');

mix.extract([
    'axios', 'vue', 'jquery', 'bootstrap-sass',
], '/assets/js/vendors.js');

@arcanedev-maroc Easy! The cause of the problem is the same – wrong paths! In this case, you are starting the path with a slash ( / ), but you shouldn't, because this tool thinks that you want to define absolute path (/assets/js/vendor.js) and, I'm pretty sure, the Git bash is freaking out, because in case of disk D: the absolute path should be /d/assets/js/vendor.js... 😉
If you wanted to extract vendors.js file to public/assets/js/vendors.js, then this should do it:
mix.extract([
    'axios', 'vue', 'jquery', 'bootstrap-sass',
], 'public/assets/js/vendors.js');

@JeffreyWay , maybe you could make Laravel Mix to check if the disk drive exists? Because nobody can guess that 95% emitting means that the disk doesn't exist. But this is definately not a bug.

OMG @taai, you're a lifesaver 🙌
I've changed the code to this and it WORKS!!!
const { mix } = require('laravel-mix');

mix.options({
    processCssUrls: false,
    publicPath: 'public'
});

mix.sass('resources/assets/sass/app.scss', 'public/assets/css');

mix.autoload({
    jquery: ['$', 'window.jQuery', 'jQuery']
});

mix.js('resources/assets/js/app.js', 'public/assets/js');

mix.extract([
    'axios', 'vue', 'jquery', 'bootstrap-sass',
], 'public/assets/js/vendors.js');

@JeffreyWay, i don't know if it's a bug or an attendant behavior. But it's definitely a breaking change (Or only for Windows users ?) when you upgrade from laravel-mix 0.x to 1.x.

@arcanedev-maroc
Based on the documentation, the filenames are automatically done for you. Not sure if specifying a filename, which in this case, same filename does affect it or not.

Can someone please test this again? It is failing on my windows 10 system as well.

For me, without "mix.setPublicPath('public/assets');" it's stuck on 95% emitting.

@taai thank you for that, I was also using W10 and laravel-mix for a non Laravel project and
mix .options({     publicPath: 'public' })
works for me perfectly.

Putting

mix.options({
    publicPath: 'public'
})

in webpack.mix.js worked just fine
thanks.

I had the same issue and fixed by removing "/" from the beginning of the second arg
mix.js('resources/assets/static/js/app.c7b1338e8d5b4eb4714b.js', '/static/js/app.c7b1338e8d5b4eb4714b.js')
to
mix.js('resources/assets/static/js/app.c7b1338e8d5b4eb4714b.js', 'static/js/app.c7b1338e8d5b4eb4714b.js')


Support for WhatsApp Interactive Lists

From https://github.com/netflie/whatsapp-cloud-api/pull/30
Merged here @robertripoll


how to implement automation with multiple teams?

Hey Mattt! I'm currently using Jenkins for continuous integration and am writing a pre-build script that will refresh all of the provisioning profiles on the Jenkins device before trying to build the app in case they were invalidated due to an added device or something. I found that otherwise, I have to log into the box manually and refresh the accounts tab in Xcode.
Cupertino looks like the perfect tool to easily download all of our profiles and put them in the right place. My issue is that we use 2 teams (a Production one and an Enterprise one) and my script gets prompted on which team to choose rather than just downloading all of the profiles from both teams.
Do you have a recommendation on how to get around this? At first I was checking to see if there was an extra argument I could pass in to select the team using the first command (ios profiles:download:all) to avoid a second prompt but that doesn't appear to be the case. For the time being I am using Tcl expect with some success but it's not an ideal solution.
For what it's worth, here is the script I wrote which downloads all of the profiles from my first team:
#!/usr/bin/expect

set timeout 20

spawn ios profiles:download:all

expect "Select a team:" { send "1\r" }

interact

Any thoughts on this? Perhaps this would be something useful as a feature addition if there isn't a good way to do it currently.
edit: I'm seeing a --team switch in the internal help documentation but am unable to use it successfully. For instance: --team 1 doesn't seem to do anything.

This might be what you're looking for:
ios profiles:download:all --team $TEAMID --type distribution
via a previous comment here and the global options: https://github.com/nomad/cupertino/blob/master/lib/cupertino/provisioning_portal/commands.rb
Also, you'll find that Apple's TeamID will differ depending on the user that is logging in and their role in the dev portal.

Oh! looks like I was not using the correct parameter for teamid, using the actual ID from the dev center (rather than the number or the full string) seems to work!


OnClick recognizer on Scene (Compose)

This might be a little too much to ask, but it would be amazing to have a tap-recognizer on TOP of the existing interaction layer (which works great by the way!).
It would be highly appreciated if this was implemented, or if someone has a workaround,
Kind regards!

Resolved using:
sceneView.setOnClickListener {
    println("This is working!!!!")
}


Add alexa image_processing dependency

Breaking Change:
NA
Description:
Related issue (if applicable): fixes #29936
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[na] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[na] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

Thanks for your contribution but because of how a bunch of things are organised, this needs a more elaborate solution that we're working on in #29913


Publish image to docker hub

Can you please publish the latest release to docker hub?

The latest release is published here on github. What is it that you would like to see published in docker hub?

I would like to see the Dockerfile published here on GitHub in docker hub. The benefit is that instead of cloning the repository to build the image, I can just download it.

So available at https://hub.docker.com/repository/docker/openquantumsafe/python. Please check whether this is what you'd expect.
@vsoftco : Is this also OK for you? Shall we add a CI job to automate this build-and-push (along what's done in oqs-demos)? Also, we could add an OQS-enabled openssl via the python import ssl command: Sensible? Desirable?

@vsoftco : Could you please take a look at https://github.com/open-quantum-safe/liboqs-python/blob/mb-ossl/Dockerfile-openssl : This creates a liboqs-python enabled image also containing an oqs-enabled openssl - complete with an interop test against test.openquantumsafe.org: Would you want to have this in this repo? Auto-building for docker hub? Or shall we move this into oqs-demos?

@baentsch Thanks Michael, will take a look asap

@baentsch Thanks Michael, will take a look asap

Thanks! Will gladly do a PR if you'd like to host this here. Could also add CI creating image and pushing to docker hub -- unless you'd like to do that.


Group chats

Hi!
First of all thank you for this work!
I have a questiıon about my project.
When I install my app from playstore I need to get my group chats which is registered in the xmpp server.
I could not any idea about this in your package.
How can I do this?
Thank you very much again for your help!

@ismailgzn thanks for the feedbacks.
i don't see any such XEP defined for that and nither i can find any support with exteral plugins, May be you can do that with the Rest apis offererd by different servers.


Don't set unique_session_id if devise.skip_session_limitable is set

There is currently no way to disable the setting of the unique_session_id. This adds the skip_session_limitable option to set as well as fetch

Could you write some tests to support your work? This project really needs 'em!


Refactoring for Simbrief not working #1005

Fix a couple of issues -

The simbrief ID should only "transfer" over to the PIREP once the PIREP is filed... that prevents a cancellation/etc after a profile from discarding the briefing (in case it's used over).
The SimBrief HTTP resource uses the flight_id param with a lookup of the user_id + flight_id - but if that flight_id association is removed (like it is on prefile), then the URL generated as the response (on where to look up the briefing) is then errored out, with a blank being passed. Change the URL so that the briefing ID is passed, and then do a check on the user to make sure its their own briefing that they're using (a requirement from Simbrief)

Closes #1005
@FatihKoz Can you pull this branch down and test?

Will pull and test, earliest it will be 8 hours later though 'cause it is almost 3 am here. Will report back after testing.

Will pull and test, earliest it will be 8 hours later though 'cause it is almost 3 am here. Will report back after testing.

Take your time, thanks

Take your time, thanks

Just tested ...
phpvms7 : Generated a simbrief pack for a flight
acars : search flights , select the flight with simbrief pack , load , click start flight
No Errors.
Should I fly and finish the flight to test if it goes well till end ? ( with a short visual pattern maybe )

Just tested ...
phpvms7 : Generated a simbrief pack for a flight
acars : search flights , select the flight with simbrief pack , load , click start flight
No Errors.
Should I fly and finish the flight to test if it goes well till end ? ( with a short visual pattern maybe )

Yes, that'd be great, this should also fix the brief being attached to the pirep at the end of the flight

Yes, that'd be great, this should also fix the brief being attached to the pirep at the end of the flight

Well , tried to finish the flight with a short and low pattern at departure airport ... Probably because the latest checks for takeoff to enroute phase change acars did not allowed me to finish the flight after landing , thus cancelled the flight.
Now doing another pattern but will climb higher to trigger the enroute phase and will land back to origin

Well , tried to finish the flight with a short and low pattern at departure airport ... Probably because the latest checks for takeoff to enroute phase change acars did not allowed me to finish the flight after landing , thus cancelled the flight.
Now doing another pattern but will climb higher to trigger the enroute phase and will land back to origin

I can reduce that to 100 feet in the next build

I can reduce that to 100 feet in the next build

Second test flight ... Created a simbrief pack for the flight
Acars -> search -> load -> start flight ... No errors
Take off -> climb to trigger enroute phase -> return back and land to origin -> end flight ... No errors
Check database / simbrief table -> flight_id was null , pirep_id was populated
Check phpvms -> Simbrief was attached to the pirep as expected
Check phpvms -> Move self to origin again 'cause my diversion was not recognized -> Go to flights -> try to create simbrief -> No old briefings displayed as expected
Except the part that diversions are not recognized yet by acars, all went fine.

Second test flight ... Created a simbrief pack for the flight
Acars -> search -> load -> start flight ... No errors
Take off -> climb to trigger enroute phase -> return back and land to origin -> end flight ... No errors
Check database / simbrief table -> flight_id was null , pirep_id was populated
Check phpvms -> Simbrief was attached to the pirep as expected
Check phpvms -> Move self to origin again 'cause my diversion was not recognized -> Go to flights -> try to create simbrief -> No old briefings displayed as expected
Except the part that diversions are not recognized yet by acars, all went fine.

Diversions are a separate issue but the rest is working so that's great, thanks!

Diversions are a separate issue but the rest is working so that's great, thanks!

Diversions are a separate issue but the rest is working so that's great, thanks!

Sure , thanks for the quick fix

Diversions are a separate issue but the rest is working so that's great, thanks!

Sure , thanks for the quick fix


Exit PR: No need to ask for which repo in a multi-root setup

Extension version: 0.2.2
VSCode Version: 1.28
OS: macOS High Sierra

Steps to Reproduce:

Exit PR in a multi-root workspace.
It asks which repo do I want to change branch.
A PR is already associated with a specific workspace. For example, if I'm working on repo A/B/C and I'm exiting pr A//100, I wouldn't expect the Exit to ask me whether I want to switch branch for B or C.

Closing as I believe this was fixed as part of https://github.com/microsoft/vscode-pull-request-github/commit/b629b113f28050d5f311fae196b4297029eb7c89, I can no longer repro this


Shared buffer system memory tracking

There are merge conflicts. Can you rebase this?

Approach looked right, but I had a bunch of minor comments

Back to you for some changes

Had a couple minor comments, but otherwise this looked good. Let me know when those are fixed

Merged


2023 launch and config

I think I copied all of the files -- roslaunch controller_node 2023_compbot_combined.launch hw_or_sim:=sim works (aside from some CUDA errors since I don't have an nvidia gpu) -- but I wanted to confirm I didn't miss anything before merging.

Didn't end all files with a newline... closing (just kidding, looks good)

oh no, not sure how i overlooked that :)

i don't think merging causes any harm so here we go


How to compare Procname.java_type with Sil.typ

I am trying to build a Checker and I have the following pattern in my do_instr:
 | Sil.Call (_, Sil.Const (Sil.Cfun pn), ((_arg1_exp,arg1_typ)::_ as args), loc, _) -> (*...*)

In this particular checker I am only interested in method calls in which the class of the method is the same as the class of the first argument.
The closest I came up with was with the following expression:
((Sil.typ_to_string arg1_typ) = (Procname.java_get_class pn) )

This doesn't work. With the class Hello the expression above is equivalent to this:
"class Hello" = "Hello"

I can always remove the "class " prefix but I'm sure there is a better way of doing it...

In Patternmatch:
(** Get the class name of the type *)
val type_get_class_name : Sil.typ -> Mangled.t option

Then Mangled.to_string

Ok got it with:
let arg1_class =  match (PatternMatch.type_get_class_name arg1_typ) with
  | Some mangled_class_name -> Mangled.to_string mangled_class_name
  | _ -> ""


Using a USB camera?

Any suggestions on where to start to try to get this stuff running with just a USB camera?  Any hints would be appreciated.

Hi @bluewookie88,
If you check out the ofVideoGrabber example that should show you how to set it up.
Other than that remove the reference to the cam object:
Line 97
to Line 100.
Replace with ofVideoGrabber code
And Line 137
to
if(videoGrabber.isFrameNew()) {
       copy(videoGrabber,frame);
       // The rest of the openCv stuff
}

Best
David

Thank you! Very helpful.
Sent from my iPhone

On Feb 26, 2016, at 2:46 AM, David Haylock<EMAIL_ADDRESS>wrote:
Hi @bluewookie88,
If you check out the ofVideoGrabber example that should show you how to set it up.
Other than that remove the reference to the cam object:
Line 97
to Line 100.
Replace with ofVideoGrabber code
And Line 137
to
if(videoGrabber.isFrameNew()) {
copy(videoGrabber,frame);
// The rest of the openCv stuff
}
Best
David
—
Reply to this email directly or view it on GitHub.


Warning: Invalid regular expression message from 0.3.9 and next versions

Warning: Invalid regular expression from version 0.3.8 to 0.3.9 and next
The principal diference from 0.3.8 to 0.3.9 and next that we can see is
the use of "htmlprocessor": "0.2.0"
see commit 0beab4f6feb34e87b6ba3a884e066986ae0f0590
when we use 0.3.8 all work ok, but using 0.3.9 and next we obtain things like this:
Warning: Invalid regular expression: /

/: Nothing to repeat Use --force to continue.

thanks for contributing!, I'll be checking the issue and come back soon

i just publish a new version including<EMAIL_ADDRESS>with a possible fix https://github.com/dciccale/node-htmlprocessor/pull/22
let me know if the issue is still there

closing for lack of response. re-open if needed I will gladly help :)


'announce' room disapears from LHN when changing notification setting to 'Mute'

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Create a workspace
Navigate to 'announce' room
Click on announce room header
Click on 'Settings'
Click on 'Daily'
Select 'Mute'
Select another report from LHN

Expected Result:
announce room shouldn't disappear from LHN
Actual Result:
announce room disappears from LHN
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[x] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: 1.3.42-5
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Screencast from 2023-07-18 18-40-59.webm
https://github.com/Expensify/App/assets/43996225/f9c43261-5cc9-448a-9094-48e7be56a11a
Expensify/Expensify Issue URL:
Issue reported by: @natnael-guchima
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1689695228630369
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

We're discussing here on what the expected behavior is

We cleared up that this is actually the expected behavior, not a bug, closing
https://expensify.slack.com/archives/C03TQ48KC/p1689709321825069?thread_ts=1689623753.049399&cid=C03TQ48KC

@jasperhuangg it seems like there is another regression related this to issue. archived announce room is removed from LHN for participants too if admin set notification setting to mute before deleting the workspace. achrived announce room disappears if admin has set notification setting to 'Mute' before deleting workspace. I am guessing that we don't want to delete announce room reports when deleting workspace.

Hello @jasperhuangg, it seems like the PR that has caused this regression is reverted here (Revert "Hide muted chats from LHN"), and the test steps seem to be similar to that I have reported here. This is to check if I am eligible for reporting bonus.
Thanks.

@laurenreidexpensify I forgot, but let's get @Natnael-Guchima paid out for the reporting bonus

Hey, just want to clarify that we're re-reverting this PR because of miscommunication so hiding announce rooms from the LHN when they are muted is expected behavior. I am good with paying @Natnael-Guchima the reporting bonus as well though since we also thought the same thing but it turns out that this is not a bug.

Just added external to get Upwork job up, pls ignore @mollfpr

@Natnael-Guchima offer sent in Upwork

Accepted the offer. Thanks @laurenreidexpensify

Perfect - paid! closing


Support some way of inlining CSS into the HTML

Clear and concise description of the problem
I would like some way to have a CSS file compiled by Vite end up as an inline <style> tag in the resulting .html file. But without dumping everything into the html like vite-plugin-singlefile.
I have a lightweight splash/loading screen built into my html so the user doesn't see a blank white page while JS is loading. This needs a bit of CSS, however if this CSS is located in a separate CSS file there is an undesirable FOUC in between when the HTML for the loading screen's and when the separate delayed network request for the loading screen's CSS. So it's strongly desirable to include the loading screen's CSS inline with the loading screen HTML.
This however is not possible in Vite. In fact Vite actively does the opposite, any inline <style> styles in html get extracted out into an external stylesheet.
Suggested solution
We already have ?inline which has a similar behaviour in JS, i.e. return it raw instead of bundling it into a file. So one idea might be to make ?inline on an @import in <style> inline those styles into the <style> tag the import is present in.
<style>
@import url('./style.css?inline');
</style>

Alternative
I thought the easiest way to implement this as a plugin and tried implementing it in multiple ways. However this turned out to be absurdly difficult.

A pre transform plugin doesn't get CSS compiled by the user's postcss plugins and have to re-implement all of Vite's CSS handling.
A post transform plugin doesn't get the CSS at all, instead it gets a junk ES module so it can't extract it.
Vite has no way to append a postcss plugin without overriding the user's postcss config, so you can't add a PostCSS that'll extract something like @media inline { ... } either.

Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

I face a similar issue (can it be considered the same) where I’d like a tag <style> to remain in the end bundle, mainly because it’s embed in a SVG spritesheet, but Vite strips it from the build HTML and move it to the bundled CSS file.
Sample HTML:
<body>
    <svg style="display: none;">
        <symbol id="lock-path" viewBox="0 0 13 18" fill="currentColor">
            <!-- This 👇 is removed by Vite -->
            <style>
                .lock-path--unlocked { --locked-path: none; }
                .lock-locked-path { display: var(--locked-path, unset); }
            </style>
            <path d="M10.608 8.357h-9A1.625 1.625 0 0 0 .001 9.964s-.002 2.575 0 4.5C.003 16.42 1.611 17.998 3.537 18H8.68a3.553 3.553 0 0 0 3.535-3.536v-4.5a1.625 1.625 0 0 0-1.607-1.607Zm-3.536 4.702v2.048a.33.33 0 0 1-.321.321H5.465a.33.33 0 0 1-.321-.321v-2.048c-.699-.792-.193-2.099.964-2.136 1.143.037 1.663 1.344.964 2.136Z"/>
            <path d="M6.108 0C3.483.003 1.29 2.197 1.287 4.821v2.893h1.928V4.821c0-1.575 1.317-2.892 2.893-2.892 1.576 0 2.893 1.317 2.893 2.892 0 .17.15.322.321.322h1.286a.33.33 0 0 0 .322-.322C10.927 2.197 8.733.003 6.108 0Z"/>
            <path class="lock-locked-path" d="M9 4.8h1.93v2.91H9z"/>
        </symbol>
    </svg>
</body>

The <style> tag is moved and the end of the CSS, and I don’t want that:
.lock-path--unlocked{--locked-path:none}.lock-locked-path{display:var(--locked-path,unset)}

Maybe a solution could be to have a magic comment and/or a setting to keep inline CSS in the HTML and/or what @dantman is suggesting.

The issue is worse in 2.9 (in both 2.9.0 and 2.9.1): in addition to the problem previously mentioned,, now it’s broken during dev time, which means this:
<body>
    <svg style="display: none;">
        <symbol id="coin-smb-path" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 366 512" fill="none">
            <!-- This 👇 is extracted by Vite on build time -->
            <style>
                .coin { color: hsl(0, 0%, 0%); }
                .coin-color { fill: hsl(0, 56%, 52%) }
                .coin-bg { fill: hsl(28, 100%, 61%) }
            </style>
            <path d="M109 109H73v294h36v73h37v36h146v-36h37v-73l37-1V109h-37V37h-37V0H146v36h-37v73z" fill="currentColor"></path>
            <path d="M37 109H0v294h37v73h36v36h147v-36h37v-73l36-1V109h-36V37h-37V0H73v36H37v73z" class="coin-bg"></path>
            <rect x="110" y="403" width="73.3" height="36" fill="currentColor"></rect>
            <rect x="183" y="109" width="36" height="293.3" fill="currentColor"></rect>
            <rect x="74" y="109" width="36" height="293.3" class="coin-color"></rect>
            <rect x="110" y="73" width="73.3" height="36" class="coin-color"></rect>
        </symbol>
    </svg>
</body>

becomes this:
<body>
    <svg style="display: none;">
        <symbol id="coin-smb-path" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 366 512" fill="none">
            <!-- This 👇 appear during dev time -->
            <script type="module" src="/index.html?html-proxy&index=5.css"></script>
            <path d="M109 109H73v294h36v73h37v36h146v-36h37v-73l37-1V109h-37V37h-37V0H146v36h-37v73z" fill="currentColor"></path>
            <path d="M37 109H0v294h37v73h36v36h147v-36h37v-73l36-1V109h-36V37h-37V0H73v36H37v73z" class="coin-bg"></path>
            <rect x="110" y="403" width="73.3" height="36" fill="currentColor"></rect>
            <rect x="183" y="109" width="36" height="293.3" fill="currentColor"></rect>
            <rect x="74" y="109" width="36" height="293.3" class="coin-color"></rect>
            <rect x="110" y="73" width="73.3" height="36" class="coin-color"></rect>
        </symbol>
    </svg>
</body>

And as this line doesn’t load nor inject anything, this is what I see in Vite 2.9 during dev. time:

And this is what I see in Vite 2.8:

This comes from this project (using Vite 2.8) when I update it to 2.9 on my machine.

Maybe https://github.com/vitejs/vite/pull/7786 can fix this issues. If someone else has time to test, please report in #7786. I will fix it in time.

I tested the project mentioned in my previous comment with Vite 2.9.6, and the bug remains.

@meduzen do you happen to have a reproduction you can share?  Is the style tag still being extracted to a script?  I wonder, does it change anything if you move the <style> tag to the head rather than the body?

@IanVS I suppose you could see it, it’s the <style> tags, all in <svg>s (all in <body> in my case), but I guess the expectation is to have the ability to maintain <style> anywhere.

@meduzen sorry, but <style> tags must go in the <head>.  See https://developer.mozilla.org/en-US/docs/Web/HTML/Element/style.

The

Edit, my mistake, it's okay to have them in SVGs, as you said, but I missed that.  This seems like it is still a bug with vite, since it should not extract these, probably.

For me, Vite 2.9.7 solves the issue. Many thanks to the contributors. 🙇


Support for instruct/chat models

Hi, I wanted to know if the library has good in-built support for Instruct/Chat models, or if any changes are needed.
For example, I tested Llama-3.1-8b-Instruct with WISE, using hyperparameters provided in hparams/WISE/llama-7b.yaml (just changing the model name)
I tried: --data_type=ZsRE --ds_size=10 --sequential_edit
And got poor results, i.e. pre and post accuracies were both lower than llama-3.1-8b, i.e. the base model. This makes me wonder if chat templates are being applied, and whether methods are expected to work with them for methods like WISE (the paper doesn't report results on instruct-tuned models).
Any insights would be greatly appreciated!

Thank you for your attention. Currently, our framework does not support chat templates. While directly editing Llama-3.1-8b-Instruct with WISE shows some performance decline compared to Llama-3.1-8b, the results are still quite good. We conducted preliminary tests and found that using chat templates on Llama-3.1-8b-Instruct provides a modest performance boost, and it's still better than the base model. If conditions permit, we will consider gradually adding support for chat templates in the future. If you're interested, feel free to make modifications on your end—I’d welcome your feedback or a PR submission.

We have now added preliminary support for chat templates in the WISE method. To enable this, simply add use_chat_template: True in the configuration.

We provide an example using Llama-3.1-8b-Instruct here: Llama-3.1-8b-Instruct Example.
The specific chat template can be found here:

easyeditor/evaluate/evaluate_utils.py#L107
easyeditor/models/wise/utils.py#L84
Please ensure the chat templates are consistent across both files.

Feel free to reach out if you have any questions. We will continue improving this functionality over time.

And do you have any further questions?

Thanks for the fast response, this was quite useful!


add units for the staleReplicaTimeout

I can't find a single "spec" for the parameters, so I figure it's best to explain it in the examples and documentation. It took me a while to find the units in the code.
I'm still curious the mechanism of it- is this for a single replica before it gets replaced? What are the hazards of setting it too low or high?

force-pushed for the signoff.


Some JAX flags cannot be set by environment variables

Currently, the JAX_ENABLE_X64 flag can be set by environment variable:
$ export JAX_ENABLE_X64=True
$ python
>>> from jax.config import config
>>> config.FLAGS.jax_enable_x64
True

However, some flags cannot:
$ export JAX_XLA_BACKEND=tpu_driver
$ export JAX_BACKEND_TARGET="grpc://$COLAB_TPU_ADDR"
$ python
>>> from jax.config import config
>>> config.FLAGS.jax_xla_backend
''
>>> config.FLAGS.jax_backend_target
''

Thanks for raising this!
@skye since you were looking at this stuff recently, could you take a look at this issue?

@mattjj in which module should I look fot this?

in which module should I look for this?

@bharatnishant are you asking about these particular flags, or about configuration options in general?
In general, the 'good' config stuff is in jax/_src/config.py. That config stuff is good because it has a uniform way to set up environment variables, flags, and context managers. Some flags are defined elsewhere, and you can find those just by grepping for them.
The jax_xla_backend option is instead defined in xla_bridge.py, but as you can see it's deprecated in favor of jax_platforms, which does have an env var. @ayaka14732 I suggest using export JAX_PLATFORMS=tpu_driver instead of using JAX_XLA_BACKEND.
As for JAX_BACKEND_TARGET, it looks like that doesn't have an environment variable. Maybe we should add one there...
I don't think #9118 touches JAX_BACKEND_TARGET, so I'll add an environment variable there now (in an ad-hoc way, rather than using general jax/_src/config.py stuff). @skye lmk if that's mistaken.

@mattjj I wanted to know about the configuration options in general.
After you have made changes, is it needed to be changed in jax/_src/config.py as well for a proper fix? Should I add it there too?

Ah, I see!
I'm not sure if it's worth making jax_platforms or jax_backend_target use our general-purpose config machinery because AIUI those are likely to be set once per process (so we don't need the context manager stuff, handling state changes in a thread-safe and jit-sensitive way, or anything else). I could be wrong though.
@skye WDYT about making jax_platforms or jax_backend_target use the machinery in jax/_src/config.py?
@bharatnishant if you're looking for places to help, there may be other spots which are more fruitful.

Hi sorry, lost track of this! #9142 looks great.
@mattjj I agree with your analysis of jax_platforms and jax_backend_target. The only reason I can think of for using the standard config machinery is to have all flags/env vars in one place (and we can just throw away the context manager). I don't feel strongly either way.


http2 test suite from mod_h2 github repository

How to use:

run configure again after you checked this out
the following components need to be installed on your system:

python3, pytest
curl, nghttp, h2load

run the tests:

> make install
> cd test
> pytest

This starts the installed httpd on local ports 40001 + 40002, runs the test suite and tears it down again. To run
individual test cases, use
> cd test/modules/http2
> pytest -k test_004               # run all tests in test_004_post.py
> pytest -k test_004_07         # run test 07 in test_004_post.py

Merged in r1892476 in svn trunk.


TabView: Visual update

This is mostly a visual update based on design comps. Colors and margins changed slightly; the + button now looks like a mini tab. I also added TabItemsChanged events, which hooked up to ListView.Items.VectorChanged, and hid keyboard accelerator tooltips.

Can you share before/after screenshots of the changes?

Could you also share the design comp that you are working towards?

Could you also share the design comp that you are working towards?

Sure, here is the figma link:
https://www.figma.com/file/9OcpPKyRbLzCFIhNh2AWIi/Window-Tab-Exp

@teaP  the Figma files are internal only, so can't be viewed publically.

Here's a screenshot.

I'm not sure how to share the comps, then.

It looks a bit odd that the separator is cutting into the contour of the hovered tab imo 🤔 FWIW what Chrome does is fade out the separators around the tab when hovering it.

@stmoy Thanks, another question: why the TabItems property isn't just named Tabs or Items? TabView.TabItems makes it seem like there's another *Items property under TabView.


WIP: implements #150 interface vlan mapping

Fixes: #150

The dcim Interface model as three new fields. mode, native_vlan and tagged_vlan_members. Note that tagged_vlan_members is a ManyToMany relationship to ipam VLAN. This includes all associated migration, view, form, model, and api changes. Submitted as a WIP for review.

@lampwins Looking forward for when this can be merged!
Note, please take these comments lightly, I'm not in any way attacking the great work it appears you've accomplished.
While briefly browsing the code, I believe this includes setting an interface to "access" or "trunk" or "trunk all".  Is "trunk all" the same as (in Cisco terms) switchport mode trunk all?
Not being very familiar with other vendors, I'm wondering if that type is necessary?
However, it is highly likely that I missed something in the code that requires that type to be there, which if is the case, please ignore my comments entirely and please accept my appreciation.

Trunk means different things to different manufacturers. In Cisco it means multiple VLAN's on an interface and in HP for example, a trunk is link aggregation.

@bdlamprecht So the Trunk All mode is intended to mean all vlans are tagged. So yes, this would be cisco's switchport mode trunk all or set interfaces ge-0/0/0.0 family ethernet-switching vlan members all on junos. Whereas the Trunk mode allows the user to select specific vlans that should be tagged on the interface. Trunk all prohibits the user from selecting any tagged vlans from the list.
Remember that netbox is intended to model the concepts of networking, not any one vendor's implementation. That being said, I am open to renaming Trunk and Trunk all if there is a better name.

Yeah, I suppose that makes sense.
I'm just wondering why we would want to "prohibit the user from selecting any tagged vlans" at all? I haven't seen how this looks in the UI, but in my mind, we are adding unnecessary complexity when  allowing them to select all the vlans would accomplish the same thing.  Then, let the vendor translate that into however it wants in the configuration.
Perhaps if I could see how this looks in the UI, I might understand more where you're coming from.
Concerning this statement:

I am open to renaming Trunk and Trunk all if there is a better name.

If another name could be chosen to avoid confusion (for example in collin-clark's example above), I would be all for it, however, I can't seem to come up with one either.
Any other ideas for the name?

It is meant to say that all vlans are tagged so don't bother select anything. Each tagged vlan on an interface is a record in the database so we save the from having to manually tag potentially 4000 vlans.
Whats more, when it does come time to turn this into configuration, how would you know to use vlan members all instead of explicitly tagging all vlans?

Each tagged vlan on an interface is a record in the database so we save the from having to manually tag potentially 4000 vlans.

That is a good point I didn't even consider.
I'm sold on the current implementation (and you didn't even have to show me a screenshot of the UI :smile: )

Nice addition in the first place!
@bdlamprecht @lampwins

I am open to renaming Trunk and Trunk all if there is a better name.

Why don't you give the choice untagged vlan - tagged vlans - tagged all? I think thats the most generic way all network engineers will understand.
By the way if you have a trunk (in Cisco terms) it's possible to set your native (untagged) VLAN as well. How can you specify the switchport trunk native vlan 200 in here?

@nonsbe I can agree to that. The interface modes have been renames to Access, Tagged and Tagged All.

By the way if you have a trunk (in Cisco terms) it's possible to set your native (untagged) VLAN as well. How can you specify the switchport trunk native vlan 200 in here?

Yes this is supported. You can set an untagged_vlan as well as a set of tagged_vlans.