Different database backends

Currently only mongodb is supported as a backend.
However sacred is able to use other databases as well. This could be supported in sacredboard as well.

This would be awesome, especially for the local backends like tinydb or file-storage

I think the first step would be to decide upon a web api. Ideally, I think this would somehow be codified in the form of an abstract python class which would then be subclassed by each database backend.
Any thoughts? I'm happy to take a look through the current implementation and propose an interface, but I think it might be more appropriate for main contributors to do this kind of "deep design" thinking.

My idea was to make either an abstract class that would the PyMongoDataAccess inherit from (if that makes any sense in a language like Python where we can do "duck-typing" - but perhaps it does for the sake of intelligibility).
The data format accepted and returned by PyMongoDataAccess would be then considered as a reference for other implementations - which is basically what @gideonite suggested.
I've tried to document the format of parameters requested by the MongoDB backend and I should do the same with return values.
Unfortunately, I've prioritised other features and aspects of the project (such as decoupling of the HTML view from its model to support its testability), so I've "postponed" this - not mentioning that new features like filtering (currently in the development branch) could be more difficult to implement with backends like file-storage.
Of course, any help would be appreciated. Basically, the important methods of the current interface are
def get_runs(self, sort_by=None, sort_direction=None,
                 start=0, limit=None, query={"type": "and", "filters": []}):
def get_run(self, run_id):
def connect(self): # not as important for other implementations - only referenced in bootstrap.py

The approach Sacredboard currently uses is to initialize the data gateway and put it into the current_app.config["data"] variable in bootstrap.py
It's definitely not the best solution, but might be enough for now.
If anyone creates a proof-of-concept for other backends using the here provided hints, I'm happy to consult it, but cannot start developing it on my own now because of my other duties.
I was also thinking whether we might not try replacing (or extending?) the current pseudo-API with something nicer, such as GraphQL. But would be just a nice-to-have.

I've implemented a basic interface called DataStorage and PyMongoDataAccess inherits from it. I've also started implementing a FileStorage class which simply does get_run and get_runs. I'm getting some weird DataTables errors though when I startup the web app.  I assume this is because either:
1 - Because the data format doesn't jive with the mongodb format, or
2 - I haven't implemented any filtering.
To create a run I simply load the run.json and add set run["config"] = load_config("config.json").
At pointers at this point? Obviously it would be nice to get it to run. I'm also a noob when it comes to writing pythonic code so will happily take code style comments.
Here's the commit on my fork: https://github.com/gideonite/sacredboard/commit/8cfac39ef97afbd9debf2e0242ab0a14c2ebe073

Thanks!
You can run sacredboard with debugging enabled (--debug) and navigate to <IP_ADDRESS>:5000/api/runs
There you can see that the returned object has to implement the "count" method to see the total number of records (DataTables uses this for pagination etc).
One of the solutions to achieve it is to add this to datastorage.py:
class DataIterator():
    def __init__(self, count, iterable):
        self.iterable = iterable
        self._count = count

    def count(self):
        "The number of items"
        return self._count

    def __iter__(self):
        return iter(self.iterable)

and to modify your get_runs method:
 def get_runs(self, sort_by=None, sort_direction=None,
                 start=0, limit=None, query={"type": "and", "filters": []}):
        all_run_ids = os.listdir(self.path_to_dir)
        blacklist = set(["_sources"])

        def run_iterator():
            for id in all_run_ids:
                if id in blacklist:
                    continue
                yield self.get_run(id)
        return DataIterator(len([id for id in all_run_ids if id not in blacklist]), run_iterator())

But even then there is a problem as the loaded JSON does not contain information about data types. Therefore, date and times, such as the experiment start time and last heartbeat time, are being loaded as strings and they need to be converted to appropriate Python representation using
    datetime.datetime.strptime('2017-05-28T12:18:33.091144', '%Y-%m-%dT%H:%M:%S.%f')

I hope this should be enough for making it work (without sorting, pagination and filtering).
Regarding coding style, if you have the tox package installed, you can run
tox -e flake8
It's quite pedantic, so there is still a lot of work to do even in my code. When everything is fine (line breaks, spaces etc) it starts checking the documentation strings.
What I personally found useful is to write tests together with the main code. You can find inspiration in test_mongodb.py (It uses a library that mocks MongoDB, so the test behaves almost as if it was talking to a real database).

Great feedback! Sorry for the delay in responding.
I've made some changes that (I think) addresses these issues. Namely, I created a FileStoreCursor object. This brings up another point which is that the object oriented programmer in me says to create another abstraction to match "DataStorage" called "DataCursor." I don't know if this is pythonic, but I don't know how else to guarantee that others that want to extend this code for their own datastore know what to implement.
I've had some troubles with tox because I use Anaconda instead of virtualenv. I couldn't find a developer wiki page and at this point it might be a good idea to create one. We can put some of the info that's been mentioned in this thread such as tox, --debug, etc. so that future developers can quickly see how to get started. I'm interested to know your development setup.
Here's the latest:
https://github.com/chovanecm/sacredboard/compare/develop...gideonite:filestore

Great job!
Don't worry about the delay - I'm currently busy with my university anyway.
I'll try to have a look at it tomorrow (I'm in Europe), so now only in brief:

The solution with FileStoreCursor basically corresponds to my DataIterator proposed above. If you look at the code, it does not depend on the FileStorage - it is general enough to be on the same level of abstraction as DataStorage because it's reusable also by the MongoDB layer and possibly by any other (let's name it DataIterator or DataCursor - I don't have any preference for that).
I had problems with Anaconda too. That's why I started using the Windows Subsystem for Linux (aka Bash for Windows) in Windows 10.  But as it's a beta, it sometimes simply stops working. So perhaps a Linux virtual machine (e.g. in VirtualBox) is better.
I am sorry about the missing documentation. In fact, Sacredboard as created as a part of my master's thesis so I spent more time writing the thesis than a proper documentation :-( Chapters 6 and 7 are shortly describing the development environment and commands (now I noticed I didn't mention the --debug option).
In two weeks, I am going to defend the thesis and to take the "final state examination", which is necessary to graduate here in the Czech Republic. I probably won't have much time by then because I need to basically go through almost all courses I took to prepare for the exam.
But I was thinking about either creating a wiki page or generating Sphinx documentation (it is a kind of standard for Python projects). That's why I started with completing code documentation docstrings because they would become a part of the generated doc.

But I'm glad you asked, it should be probably one of my priorities now.

I tried it now. Nice that the list actually shows something.
Now I see I should definitely add tests for testing interaction with the backend (#40) so that developers on the backend side always know whether the data they return are in the expected format.
Basically, the object you are building with _create_run should correspond to the object key in the example "/api/run/id" in #40.
So I think in order for even the Run detail view to work, you need to set run["_id"] and run["info"].
Again, thanks for doing this. It is helping not only with the functionality but also realising what's necessary to document and test.

Ah ok, now I understand a bit more about the context of the project. Makes sense.
I see your point now. Perhaps the more pythonic way of doing this, instead of heavy handedly forcing conformity via interfaces/abstract classes and the like, is to implement the required functionality in tests.
My only motivation for using the term "cursor" is to match the return type of the MongoDB interface.
Ok, I will make sure that the data type conforms to #40 and implement a test to make sure that the right keys are returned, etc. It will probably be some kind of integration test in which I create some experiments in /tmp and make sure the resulting json has the right keys.

It's OK to have abstract classes in Python where appropriate, such as the DataStorage class you created. It is good that it tells developers what methods they have to implement.
In the case of FileStoreCursor (aka DataIterator), I was just thinking the class could be put directly on the datastore level so that developers of other database backends can use it too.
When you are talking about integration tests, do you mean it would involve running sacred too? That would be amazing but if the test fails, it might not easy to tell whether the problem was caused by Sacred or Sacredboard. So I would see this as an additional level of testing.
I think testing of your module could be solved for instance by generating a few reports by Sacred, copying the directory e.g. to tests/data/fixtures/filestore and loading them from there.
You can perhaps also try to decouple loading the JSON files from building the Run objects as it looks like the latter part could be reused e.g. for the TinyDB backend.


Remove references to CommandLineOptions.Clo in SMTLib

This PR is the first in a series to remove the static variable CommandLineOptions.Clo. This variable has a few downsides:

When calling any function in Boogie, it's not clear from the function signature how it can be configured using CommandLineOptions.Clo and there's no easy way to figure this out. Relatedly, external callers of Boogie must initialise CommandLineOptions.Clo, but when they call a public Boogie API it's not obvious that they should.
Because CommandLineOptions.Clo is referenced from everywhere it has to live at the bottom of the project reference stack, in Core, but this means it can't refer to any interesting types defined higher in the stack and this leads to options typed object that have to be cast at a later time.

An example is the function CommandLineOptions.TheProverFactory.SpawnProver which returns object while in practice it returns a ProverInterface which is defined in the VCGeneration project that's not available from Core.

It's not possible to run Boogie in parallel with different command line options, since the options are shared between threads. Note this could also be solved by making CommandLineOptions.Clo thread-local.

This PR replaces references to the global variable CommandLineOptions.Clo that occur in the project SMTLib.

Here is my understanding of this PR:  It factors out a subset of the flags in CommandLineOptions into a separate interface called SMTLibOptions which is now implemented by CommandLineOptions.  These factored out options are presumably the ones that are referred to in the code that does the SMT encoding.  So far, so good.  How this PR contributes to the overall mission charted out in the PR summary remains a mystery to me.

These pull requests and changes are almost certainly going to have major effects on Corral (https://github.com/boogie-org/corral/) downstream.
Would it be possible to update Corral as well along the way (since you know the best what refactoring you are exactly doing)?

So far, so good. How this PR contributes to the overall mission charted out in the PR summary remains a mystery to me.

I'm not sure where the mystery comes from. The mission can be summarized as removing the static variable CommandLineOptions.Clo, for which we first need to remove all references to it. This PR removes all references to it that occur in the SMTLib project.
One step that this PR doesn't take is to place SMTLibOptions in SMTLib instead of in Core, which is required to remove downside (2) listed in the details. This change requires another refactoring that I'm leaving for another PR.

These pull requests and changes are almost certainly going to have major effects on Corral (https://github.com/boogie-org/corral/) downstream.

I don't think this PR has an effect on downstream projects, but indeed the stated goal of removing CommandLineOptions.Clo will, because downstream projects might refer to it. The migration fix that downstream projects can do is to declare their own DownStreamOptions.Clo and refer to that instead of CommandLineOptions.Clo. This is an easy change. After it downstream projects can decide whether to remove references to static variables as well or keep it the way they have it.
In the PR that actually removes CommandLineOptions.Clo, I can include an upgrade guide in a release notes file, but don't worry it'll be tiny. The alternative of only making fully backwards compatible changes in Boogie would I think be a major blocker in evolving Boogie.

Would it be possible to update Corral as well along the way (since you know the best what refactoring you are exactly doing)?

I can help if Corral upgrades to a Boogie version that requires it.

So far, so good. How this PR contributes to the overall mission charted out in the PR summary remains a mystery to me.

I'm not sure where the mystery comes from. The mission can be summarized as removing the static variable CommandLineOptions.Clo, for which we first need to remove all references to it. This PR removes all references to it that occur in the SMTLib project, so it's a step in that direction.
One step that this PR doesn't take is to place SMTLibOptions in SMTLib instead of in Core, which is required to remove downside (2) listed in the PR description. That change requires another refactoring that I'm leaving for another PR.

I am trying to understand what the final product after all the refactoring will look like.  It sounds like the class CommandLineOptions will remain and will implement a collection of different interfaces, each presumably for a different component in Boogie.  The static field clo in CommandLineOptions will go away and a separate instance of CommandLineOptions will be created to initialize an instance of some component.  Is my understanding correct?

The alternative of only making fully backwards compatible changes in Boogie would I think be a major blocker in evolving Boogie.

I agree.

I am trying to understand what the final product after all the refactoring will look like. It sounds like the class CommandLineOptions will remain and will implement a collection of different interfaces, each presumably for a different component in Boogie. The static field clo in CommandLineOptions will go away and a separate instance of CommandLineOptions will be created to initialize an instance of some component. Is my understanding correct?

I think so yes. The definition of CommandLineOptions will move to ExecutionEngine. The root calls into boogie like many of the methods in ExecutionEngine.cs will take an instance of CommandLineOptions.
Note that this PR has, IMO, a positive effect regardless of future PRs, since it makes it easier to see what options can be passed to SMTLib.


Qt main and Qt 5.15.2

I think that for now, we should keep this as an arch dependent package so we can test that compilation works for all packages

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin please rerender

Not sure why it isn't picking up the patched version. We can try again tomorrow.

@conda-forge/qt-main @conda-forge/qt this is ready for review.

5.15.3 will have to wait for: https://github.com/conda-forge/qt-webengine-feedstock/pull/5

@conda-forge/qt can we get a review here? We've been waiting a long time for qt=5.15, and several packages need it before they can be updated.

https://github.com/orgs/conda-forge/teams/qt can we get a review here? We've been waiting a long time for qt=5.15, and several packages need it before they can be updated.

We'll probably not use qt as the package just like we don't use matplotlib and just use matplotlib-base.
Please send a PR to the bot to add a migrator to change qt to qt-main in recipes.

We'll probably not use qt as the package just like we don't use matplotlib and just use matplotlib-base.
Please send a PR to the bot to add a migrator to change qt to qt-main in recipes.

I've never done this before, and it looks like name migrations are rare. Is there an example I can look at? Or maybe someone with more experience can handle this one.

Here is an example: https://github.com/regro/cf-scripts/blob/master/conda_forge_tick/migrators/matplotlib_base.py and some documentation is here: https://regro.github.io/cf-scripts/migrators.html#custom-migrations
But it looks like this one is more complicated than it should be because something went wrong during the migration and half of the file is about the fix of that rather than the actual migration ..

Here is an example: https://github.com/regro/cf-scripts/blob/master/conda_forge_tick/migrators/matplotlib_base.py and some documentation is here: https://regro.github.io/cf-scripts/migrators.html#custom-migrations
But it looks like this one is more complicated than it should be because something went wrong during the migration and half of the file is about the fix of that rather than the actual migration ..

Oh my, yeah, this looks like it needs attention from someone with more experience with migrations then I have.

Hi @beckermr - sorry to bother you, but do you have an idea how we can handle the migration from qt 5.12 to qt-main 5.15? It seems like it's going beyond the expertise of most of us, there doesn't seem to be an example that can be easily adapted (the matplotlib one I linked above does a rename but no version bump, and contains some code fixing a broken migration) and you seem to be very active in https://github.com/regro/cf-scripts/tree/master/conda_forge_tick/migrators so I thought I'd reach out.

You need to write a mini-migrator that changes qt to qt-main and add that as a piggy-back migrator in the version migration at https://github.com/regro/cf-scripts/blob/64c14692b14cf07655b65d01ec8fd75649e56ac5/conda_forge_tick/auto_tick.py#L661.
Then, you start a regular migration yaml like https://github.com/conda-forge/conda-forge-pinning-feedstock/blob/main/recipe/migrations/icu70.yaml. A sample mini-migrator is at https://github.com/regro/cf-scripts/blob/64c14692b14cf07655b65d01ec8fd75649e56ac5/conda_forge_tick/migrators/cross_compile.py#L37

Here is my attempt:
https://github.com/conda-forge/conda-forge-pinning-feedstock/pull/2929

Should we update this PR to use 5.15.3 instead, now that https://github.com/conda-forge/qt-webengine-feedstock/pull/5 is merged?

Having both versions wouldn't be a bad idea. we can deal with that later. Some stuff is still on ICU69. Seeing as this package is now trivial, it isn't a big deal.


feat: add checks for standard and proprietary header usage (#591)

Closes #591

@ddufourboivin-zalando I've incorporated (almost all) your comments and rebased the branch - please have another look

Codecov Report

Merging #760 into master will decrease coverage by 0.15%.
The diff coverage is 68.18%.

@@             Coverage Diff              @@
##             master     #760      +/-   ##
============================================
- Coverage     86.28%   86.13%   -0.16%
- Complexity      993     1013      +20
============================================
  Files           169      170       +1
  Lines          2633     2655      +22
  Branches        363      371       +8
============================================
+ Hits           2272     2287      +15
  Misses          186      186
- Partials        175      182       +7

Impacted Files
Coverage Δ
Complexity Δ

...lando/zally/rule/zalando/ProprietaryHeadersRule.kt
68.18% <68.18%> (ø)
20 <20> (?)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 06388fe...14628ac. Read the comment docs.

👍

:+1:


error: no instance of overloaded function "atomicAdd" matches the argument list

Hi, thank you for your wonderful work! I have a same error with #19 ,
to solve it I comment all places using atomicAdd in hashencoder.cu.
Will it have any effects?Thanks~

Hi, I think it don't have any effects because we don't use half precision (float16) during training.

Thanks~


Fix unfocused windows can't be dragged

Fixes #95577

The source of the problem was the call the ReleaseCapture in _process_activate_event and _set_mouse_mode_impl. ReleaseCapture causes the window movement to stop and receive a WM_EXITSIZEMOVE.
Before #93976 the _process_activate_event was not called in the situation where the activation and the move timer were active at the same time because the move timer replaced the activation timer.
I added the method _has_moving_window to check if a window is currently being moved and used it before calling ReleaseCapture. That fixed the problem. I also tested when having multiple opened windows in the editor (ex: Script editor).
The only problem I could think of is if ReleaseCapture had to be called on window activation while a window is being moved by the user. ReleaseCapture is not called anymore in _set_mouse_mode_impl. That's why I added a _set_mouse_mode_impl when receiving WM_EXITSIZEMOVE.

Tested, works great!


String Pattern solved and ready to be reviewed.

The string_pattern function checks whether a given pattern can be mapped to any part of a provided string. Each unique character in the pattern must correspond to a unique character in the string.
Functionality:
Iterates over substrings of the given string, each the same length as the pattern. Tries to map each character in the pattern to a character in the substring. Uses a dictionary for character mappings and a set to track already mapped characters. Ensures that each character in the pattern consistently maps to the same character in the string and that no two different characters in the pattern map to the same character in the string. Returns True if a valid mapping is found; otherwise, False.
Use Case:
This function is useful for scenarios that require identifying a specific sequence of unique characters in a string, important in string analysis and parsing tasks.
Overall, string_pattern is a specialized function for pattern-to-string matching, emphasizing the uniqueness and consistency of character mappings.

[ ] PR's title includes the challenge's name and language
[x] PR has short and clear description of the challenge
[x] PR has appropriate labels and milestones for easy identification
[ ] PR it is assigned to the owner
[x] reviewers are assigned
[x] the PR contributes only one focused change
[x] the branch is up to date with main/master
[x] the code works when pulled and run locally
[x] all conflicts are resolved (if any)
[x] It is linked to an issue in the appropriate column of the project board
[ ] feedback is addressed (if any, and if it is appropriate feedback.)

README Documentation

[x] The solution is documented in /src/README.md
[x] The markdown source is formatted
[x] Spelling and grammar is correct in all text
[x] The markdown looks correct when you preview the file
[ ] All links and images work
[ ] The README documents the solution's behavior, strategy(ies) and implementation(s)
[ ] There are use cases in code blocks to illustrate the function's behavior

Python Files

[ ] There is a module header
[x] There is a module docstring
[x] File names are in snake_case
[x] Test files are named test_<module_name>.py

Function Docstring

[x] Behavior description
[x] Parameter description
[x] Return value description
[x] Include any assumptions
[x] Include 3 or more (passing!) doctests
[x] Include 1-2 use cases (if necessary)

Function Implementation

[x] The solution is not is not copied from djeada
[x] The code is auto-formatted
[x] The code has no (reasonable) linting mistakes
[x] Variables are named with snake_case
[x] The function has a clear and helpful name
[x] The file's name matches the function name
[x] The code follows the strategy as simply as possible
[x] Variable names are clear and helpful
[x] Comments explain the strategy (if necessary)
[x] There are type annotations
[ ] (challenge) The code includes defensive assertions

Unit Test Suite

[x] The test class has a helpful name in PascalCase
[x] The test class has a docstring
Each unit test has

[x] A helpful name
[x] A clear docstring
[x] Only one assertion

[x] All tests pass
[x] There are tests for defensive assertions
[x] There are tests for boundary cases
[ ] Test suite includes black-box and glass-box tests

Guys, I did implement the suggested changes, if you could review it one more time and let me know what you think.
By the way, thanks:))

Guys, I did implement the suggested changes, if you could review it one more time and let me know what you think.
By the way, thanks:))

Good job! Keep going!


backport: e2e tests (#1193)

create e2e image and a makefile step to build

progress

e2e tests in ci

use root distroless image and correct volume path

remove chain b references

implement query balances

implement TestQueryBalances

trigger worflow

trigger

test-e2e Makefile step

fmt and sleep if service unavailable

README

restore branches

add changelog entry

exclude e2e from regular tests

-E flag for grep exclusion

grep

go mod tidy --compat=1.17

manually tidy go.mod

Closes: #XXX
What is the purpose of the change
*Add a description of the overall background and high level changes that this PR introduces *
(E.g.: This pull request improves documation of area A by adding ....
Brief change log
(for example:)

The metadata is stored in the blob store on job creation time as a persistent artifact
Deployments RPC transmits only the blob storage reference
Daemons retrieve the RPC data from the blob cache

Testing and Verifying
(Please pick one of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added unit test that validates ...
Added integration tests for end-to-end deployment with ...
Extended integration test for ...
Manually verified the change by ...

Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? (yes / no)
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? (yes / no)
How is the feature or change documented? (not applicable   /   specification (x/<module>/spec/)  /  Osmosis docs repo   /   not documented)

Codecov Report

Merging #1338 (fa92449) into v7.x (17afb0b) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head fa92449 differs from pull request most recent head 278ca45. Consider uploading reports for the commit 278ca45 to get more accurate results

@@           Coverage Diff           @@
##             v7.x    #1338   +/-   ##
=======================================
  Coverage   20.63%   20.63%
=======================================
  Files         199      199
  Lines       25803    25803
=======================================
  Hits         5325     5325
  Misses      19509    19509
  Partials      969      969

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 17afb0b...278ca45. Read the comment docs.


fava needs better responsive design

This is a follow-up to my comment on https://github.com/aumayr/fava/issues/39 and to tell everyone that I'm starting to think about this.
My response to the writing of (S)CSS by hand is having to solve the same problem, in a less general way, each time.  I prefer frameworks because people who have had to deal with these sorts of problems before have solved them more generally so the next time you think, "oh man, it'd be really cool if I could just ...", they probably already have a solution that fits within the framework.
I'll evaluate both cases though.  I don't really like the idea of having to tweak the layout every time there's a new screen/form factor/etc.

(Why do I have no idea how to use this interface? Can someone please assign this to me?)
A few things I've noticed just poking around:

The nav bar takes up too much space on a small screen.  That will need to be hideable.
Tables will need to be able to tell how many columns they have in them so that they can intelligently know when to switch from 50% width to 100%.  Right now if you have too many columns (like two currencies, plus other) it will usually extend outside the 50% size allotted.  This isn't cool.
Top bar extends to a funny length regardless of total width
Graphs don't always extend the right distance, but sometimes they do.

I'll evaluate both cases though. I don't really like the idea of having to tweak the layout every time there's a new screen/form factor/etc.

Please don't let you discourage from the "against-frameworks"-comment I made in #39 - you can introduce one if you like and it fits your needs for fixing this.

Why do I have no idea how to use this interface? Can someone please assign this to me?

Strangely I can't assign you as well :-/

@xentac For which devices/browsers/sizes are you looking to optimize/implement the responsive design?

My plan was to work through the Responsive resolutions offered by the chrome inspect window.  The smallest is 320x508.
My guess is the assignee has to be someone who owns/admins the project.  I wouldn't worry too much about it.  This isn't me trying to get admin access to this project or anything ;)

Please consider https://github.com/aumayr/fava/pull/321 as my first attempt.

Will close this, as #321 was merged (thanks @xentac!).


feat: expose snapshot types

Motivation
I found few did snapshots types that weren't exposed. I actually ultimatelly did not required those in my implementation but, it's cleaner anyway to expose those.
Changes

Expose few types for ic-mgmt in index.ts

Actually, I did not required those type because my bundler is friendly 😁
In my PR https://github.com/junobuild/juno-js/pull/220 it works because I resolve the type directly from the did file.
import type {
  list_canister_snapshots_result,
  take_canister_snapshot_result
} from '@dfinity/ic-management/dist/candid/ic-management';

So no blocker but, it will be nicer.


Bug: "UseView" in view-strategy.js is undefined, preventing getViewStrategy from returning relative path strings for view selection

This issue cropped up because I tried to dynamically select a view using a relative path string returned in my getViewStrategy() function. The erroneous code is here:
https://github.com/aurelia/templating/blob/master/src/view-strategy.js#L13
I suspect the code should actually read:
value = new UseViewStrategy(value);

Will get this fixed and released today.


Empty MGF file creation

Hello,
I have a problem when I creating new MGF files after the duplicate_filtering step. The MGF file found in the 0_neg_filtered folder is only 1 Kb in size and consists of a single spectrum.
I tried the example, and the same problem occurred.
Any suggestions what's going wrong?

Hi,
Sorry for the delay, I'm currently moving the project to a new repo it will have vastly improved performance and user accessibility so I will let you know once that is out.

Hi,
Thank you for your reply. I can't wait to discover your new repo. Do you have any idea when it will be released, a few weeks, 2-3 months or more than 6 months.


apply_min_dur possible bug

https://github.com/jianfch/stable-ts/blob/738fd98490584c492cf2f7873bdddaf7a0ec9d40/stable_whisper/result.py#L328
https://github.com/jianfch/stable-ts/blob/738fd98490584c492cf2f7873bdddaf7a0ec9d40/stable_whisper/result.py#L785
This probably doesn't affect usage, but the line condition seems to be always false for these lines. Is this a bug or am I missing something? Thanks

Yes, it's a bug. It was intended to be segment.words[i+1].duration < segment.words[i-1].duration. Thanks for pointing this out.

Since a fix is commited, I'm closing 👍


fix(php-fpm): fix in china, the aliyun source cannot fetch python problem

…blem.
Description

Motivation and Context

Types of Changes

[x] Bug fix (non-breaking change which fixes an issue).
[x] New feature (non-breaking change which adds functionality).
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected).

Definition of Done Checklist:

[x] I've read the Contribution Guide.
[x] I've updated the documentation. (refer to this for how to do so).


grype 0.75.0 :snowflake:

grype 0.75.0 :snowflake:
Destination channel: Snowflake
Links

PKG-4441
dev_url: https://github.com/anchore/grype/tree/v0.75.0

Explanation of changes:

new recipe from scratch (not on pypi, not on conda-forge)

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_f8oo40jxtl/clone` failed. (See above for error)
The following problems have been found:
===== ERRORS =====

clone/recipe/meta.yaml:17: should_use_compilers: The recipe requires a compiler directly
===== Final Report: =====
1 Error and 0 Warnings were found


SVG rendered differently between<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

Hi, thank you for building sharp. I want to report a possible bug:
SVG source:
<svg xmlns="http://www.w3.org/2000/svg" id="svg" width="512" height="512" viewport="0 0 512 512" version="1.1">
  <circle r="378" cx="256" cy="256" fill="transparent" stroke-dashoffset="0" stroke="#cccccc" stroke-width="300px"/>
  <circle r="378" cx="256" cy="256" fill="transparent" stroke-dasharray="565.48" stroke-dashoffset="0" stroke="#4f80ee" stroke-width="300px" style="stroke-dasharray: 2375; stroke-dashoffset: 1306.25; transform: rotate(-90deg);
    transform-origin: center;"/>
</svg>

<EMAIL_ADDRESS>output (reproducible source code):

<EMAIL_ADDRESS>output (reproducible source code):

Output by web browser preview:

Possible bug
Is this a possible bug in a feature of sharp, unrelated to installation?

[x] Running npm install sharp completes without error.
[x] Running node -e "require('sharp')" completes without error.

If you cannot confirm both of these, please open an installation issue instead.
Are you using the latest version of sharp?

[x] I am using the latest version of sharp as reported by npm view sharp dist-tags.latest.

If you cannot confirm this, please upgrade to the latest version and try again before opening an issue.
If you are using another package which depends on a version of sharp that is not the latest, please open an issue against that package instead.
What is the output of running npx envinfo --binaries --system --npmPackages=sharp --npmGlobalPackages=sharp?

  System:
    OS: macOS 12.3
    CPU: (10) arm64 Apple M1 Max
    Memory: 58.94 MB / 32.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.14.0 - ~/.nvm/versions/node/v16.14.0/bin/node
    Yarn: 1.22.10 - ~/.nvm/versions/node/v16.14.0/bin/yarn
    npm: 8.3.1 - ~/.nvm/versions/node/v16.14.0/bin/npm
  npmPackages:
    sharp: ^0.30.4 => 0.30.4

What are the steps to reproduce?

See the source code:

https://github.com/trungdq88/sharp-issue-29
https://github.com/trungdq88/sharp-issue-30

What is the expected behaviour?

Should render the same like the one shown in the browser
Please provide a minimal, standalone code sample, without other dependencies, that demonstrates this problem

const sharp = require('sharp');

sharp(
  Buffer.from(`
<svg xmlns="http://www.w3.org/2000/svg" id="svg" width="512" height="512" viewport="0 0 512 512" version="1.1">
  <circle r="378" cx="256" cy="256" fill="transparent" stroke-dashoffset="0" stroke="#cccccc" stroke-width="300px"/>
  <circle r="378" cx="256" cy="256" fill="transparent" stroke-dasharray="565.48" stroke-dashoffset="0" stroke="#4f80ee" stroke-width="300px" style="stroke-dasharray: 2375; stroke-dashoffset: 1306.25; transform: rotate(-90deg);
    transform-origin: center;"/>
</svg>
`)
)
  .png()
  .toFile('output.png');

Please provide sample image(s) that help explain this problem

<EMAIL_ADDRESS>output (reproducible source code):

<EMAIL_ADDRESS>output (reproducible source code):

This can be reproduced at the command line using the latest v2.54.1 of rsvg-convert.
docker run -it --rm -v "$PWD:/opt/tmp" alpine:edge /bin/sh -c "apk add librsvg && cd /opt/tmp && rsvg-convert input.svg >output.png"

Please can you report this upstream at https://gitlab.gnome.org/GNOME/librsvg/-/issues

This is due to the lack of transform-origin: center support. We can continue to track this at the existing #2884


Fix incorrect bind

Problem: When setting up elasticstore for the first time, I ran into the same issue as #41.
TypeError: Cannot read property 'request' of undefined at FirestoreCollectionHandler.existsApi

Debugging:

Added source-map-support so the crashing stacktrace has .ts line numbers instead of .js line numbers. Part of this PR.
Noticed error message mentions FirestoreCollectionHandler.existsApi. But the FirestoreCollectionHandler does not have a existsApi function. This indicated the function is being called on the wrong object.

Fix: The bind function on the elasticsearch client does not bind this correctly. this should be bound to the client object, not to the FirestoreCollectionHandler object. Same issue is also mentioned here.
Testing done: Works on my self-hosted deployment. Errors no longer occur in logs. Confirmed data appears in elastic search.
Connecting to ElasticSearch host elastic-search:9200
Connected to ElasticSearch host elastic-search:9200
Connecting to Firebase https://${DB}.firebaseio.com
Listening on: 'search' for queries (in: query -> out: result)
Cleanup will happen in 60000

        Begin listening to changes for collection: users
          documentId: ${ID}
          subcollection: ${SUB}
          include: [  ]
          exclude: [  ]

Added<EMAIL_ADDRESS>Added<EMAIL_ADDRESS>Added<EMAIL_ADDRESS>
Fixes #41

I don't understand the CI error. Would appreciate if you could take a look!

@nareddyt thanks so much for this PR! LGTM, so I'll go ahead and merge it. I'll fix the CI issue

@all-contributors please add @nareddyt for his coding!


Fix Resolve for Types with Generic Interfaces

Resolving types with generic interfaces failed. The given generic type was converted into it's generic definition before it was passed to the parent container. #126

Thanks for the PR!, We really need to get automated tests setup for this project.


Duplicates when uploading multiple files

for(int i = 0; i < count; i++){
                    Uri fileUri = data.getClipData().getItemAt(i).getUri();

                    multiFile.init(fileUri, new File(fileUri.getPath()).getAbsolutePath() , FileTypes.IMAGE_TYPE);

                    arrayList.add(multiFile);

                }

                //use this method...
     blazeFileUploader.startUploadingMultipleFiles(arrayList);

Fix kotlin:
The issue with your code is that you are reusing the same instance of multiFile object in each iteration of the loop. As a result, the arrayList will contain multiple references to the same object, resulting in duplicate items.
To fix this issue, you need to create a new instance of the multiFile object in each iteration of the loop
for (i in 0 until count) {
val fileUri: Uri = uriList[i]
val multiFile = MultiFile() // Create a new instance in each iteration
multiFile.init(fileUri, File(fileUri.path!!).name, FileTypes.IMAGE_TYPE)

arrayList.add(multiFile)

}

Thank you, the library is too old it's a mess, it could've been a much better one :D
Will try to work on it on my free time and rewrite the whole thing in Kotlin in a much better way


[$250] Web - Login container isn't in center @thesahindia

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:

Go to the login page
Decrease the screen size

Expected Result:
Login container should be in the center
Actual Result:
Login container isn't in center
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web

Desktop App

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers): any
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/174805956-32393d8e-d727-469e-94a7-1a283726d6c0.mov
Upwork job link: https://www.upwork.com/jobs/~01c152638d05db933a
Issue reported by: @thesahindia
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1654600244913639
View all open jobs on GitHub

Easily reproducible, making this a FirstPick

adding external so our contributors can take a look

Upwork job link - https://www.upwork.com/jobs/~01c152638d05db933a

Proposal
Cause
style information is not given when the application is at medium screen width (800-1024).
props.isSmallScreenWidth ? styles.signInPageNarrowContentMargin : {},
!props.isMediumScreenWidth ? styles.signInPageWideLeftContentMargin : {},
0-800 >> styles.signInPageNarrowContentMargin (marginTop: '40%')
800-1024 >> {}
1024 - ...  >> styles.signInPageWideLeftContentMargin (marginTop: '44.5%')
Solution
We can solve this problem by giving still information with small screen or big screen separation. So it will be in the same position for medium and large screens.
https://github.com/Expensify/App/blob/02fb4e9738fbe491c3a8e48dfb7abd141320baa6/src/pages/signin/SignInPageLayout/SignInPageContent.js#L56-L60
 behavior="position"
                contentContainerStyle={[
                    props.isSmallScreenWidth ? styles.signInPageNarrowContentMargin : styles.signInPageWideLeftContentMargin,
                    styles.mb3,
                    StyleUtils.getModalPaddingStyles({

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:

Go to the login page
Decrease the screen size

Expected Result:
Login container should be in the center
Actual Result:
Login container isn't in center
Workaround:
Unknown
Platform:
Where is this issue occurring?

Web
Desktop App

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers): any
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Screen.Recording.2022-06-02.at.11.51.58.PM.mov
Upwork job link: https://www.upwork.com/jobs/~01c152638d05db933a
Issue reported by: @thesahindia
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1654600244913639
View all open jobs on GitHub

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:

Go to the login page
Decrease the screen size

Expected Result:
Login container should be in the center
Actual Result:
Login container isn't in center
Workaround:
Unknown
Platform:
Where is this issue occurring?

Web
Desktop App

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers): any
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Screen.Recording.2022-06-02.at.11.51.58.PM.mov
Upwork job link: https://www.upwork.com/jobs/~01c152638d05db933a
Issue reported by: @thesahindia
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1654600244913639
View all open jobs on GitHub

Hii there is a simple solution to ride this issue, for centering the login container we fix it by using some styling according to the dimensions of the mobile by using this it's fixed and responsive also from all devices. Please apply this method I hope this will work.

@metehanozyurt Can re-write your update proposal's new comment.

Updated Proposal
Cause
style information is not given when the application is at medium screen width (800-1024). Its ok when height above the 854 (variables.minHeightToShowGraphics) because splits horizontally. But when screen height split vertically.
props.isSmallScreenWidth ? styles.signInPageNarrowContentMargin : {},
!props.isMediumScreenWidth ? styles.signInPageWideLeftContentMargin : {},
0-800 >> styles.signInPageNarrowContentMargin (marginTop: '40%')
800-1024 >> {}
1024 - ...  >> styles.signInPageWideLeftContentMargin (marginTop: '44.5%')
Solution
We can solve this situation by adding a second rule that checks the width and height information ((props.isMediumScreenWidth && props.windowHeight < variables.minHeightToShowGraphics)).
https://github.com/Expensify/App/blob/02fb4e9738fbe491c3a8e48dfb7abd141320baa6/src/pages/signin/SignInPageLayout/SignInPageContent.js#L55-L67
need to change like this
<LoginKeyboardAvoidingView
                behavior="position"
                contentContainerStyle={[
                    props.isSmallScreenWidth ? styles.signInPageNarrowContentMargin : {},
                    !props.isMediumScreenWidth || (props.isMediumScreenWidth && props.windowHeight < variables.minHeightToShowGraphics) ? styles.signInPageWideLeftContentMargin : {},
                    styles.mb3,
                    StyleUtils.getModalPaddingStyles({
                        shouldAddBottomSafeAreaPadding: true,
                        modalContainerStylePaddingBottom: 20,
                        safeAreaPaddingBottom: props.insets.bottom,
                    }),
                ]}
            >

https://user-images.githubusercontent.com/6093207/178557429-5527b264-32cb-4bf1-991d-9fb10c9991ab.mp4

@Santhosh-Sellavel are you happy with @metehanozyurt improved coment and solution?

@Santhosh-Sellavel are you happy with @metehanozyurt improved comment and solution?

Personally, I think it looks good

Sorry, @metehanozyurt  missed this one out earlier!
@metehanozyurt Proposal looks good!
🎀 👀 🎀  C+ Reviewed!
cc: @sketchydroide

@metehanozyurt @Santhosh-Sellavel @thesahindia you've been invited to the job on upwork.

Seems like the PR is close, this is coming along

deployed to staging, waiting now

We are still waiting on the deploy here, seems it's not ready yet.

Issue not reproducible during KI retests. (First week)

All paid.


Fix - Search filters - (you) disappears after selecting own user name

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/47712
PROPOSAL: https://github.com/Expensify/App/issues/47712#issuecomment-2299001034
Tests

Go to search > filters
Click From or To
Select your user name (current user)
Verify that (you) label doesn't disappear after you select it

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/user-attachments/assets/4da96785-6b4f-4cda-b8bb-573a491287b1

Android: mWeb Chrome

https://github.com/user-attachments/assets/f753c79d-d1fa-4df1-ad93-fd71570df1ac

iOS: Native

https://github.com/user-attachments/assets/b2a7cd67-b609-4c31-8c81-e889ed318ef1

iOS: mWeb Safari

https://github.com/user-attachments/assets/660592d5-605e-436c-a59d-d92493deedd0

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/f22b36bb-a552-4903-b3a4-49e94451c979

MacOS: Desktop

https://github.com/user-attachments/assets/7a8d4d43-d7cf-4d3e-a4cd-1500d2313517

This comment is not resolved yet - https://github.com/Expensify/App/pull/48600#discussion_r1771744371

@abdulrahuman5196 The conflicting PR has already implemented it exactly as my approach so now there is nothing to do in this PR.


[5.5] Fixed null remember token error on EloquentUserProvider

I added a check that $model->getRememberToken() isn't null before comparing it to $token using hash_equals so it doesn't throw an error.

Related ? https://github.com/laravel/framework/pull/21323/files

Well, in some way. Still - it doesn't guarantee that $model->getRememberToken() doesn't return as null. I got the error after this PR was merged.

@Naxon I am just referencing another PR.
I think issue with that file is spread across 3 releases already (just unfortunate file!).

And this PR (closed) is also related https://github.com/laravel/framework/pull/21322

Are we actually even testing what is happening here? Has anyone actually run this code? /cc @mcordingley ...
Or are we just throwing crap against the wall and hoping it sticks?

@taylorotwell  I ran the one I committed to check if it works, unfortunately no unit test.

Sorry. 😢 I ran the tests and everything passed just fine. I should have double-checked them to ensure that null would be handled, too.


Add tests for DeleteExamCommand and DeleteExamCommandParser classes

Codecov Report

Merging #268 (a245753) into master (cb143c7) will increase coverage by 0.70%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #268      +/-   ##
============================================
+ Coverage     59.96%   60.67%   +0.70%
- Complexity      858      869      +11
============================================
  Files           163      163
  Lines          3255     3255
  Branches        438      438
============================================
+ Hits           1952     1975      +23
+ Misses         1171     1148      -23
  Partials        132      132

Impacted Files
Coverage Δ

.../address/logic/parser/DeleteExamCommandParser.java
100.00% <100.00%> (+100.00%)
:arrow_up:

src/main/java/seedu/address/model/AddressBook.java
59.83% <0.00%> (+1.63%)
:arrow_up:

...rc/main/java/seedu/address/model/ModelManager.java
64.28% <0.00%> (+1.78%)
:arrow_up:

...eedu/address/logic/commands/DeleteExamCommand.java
100.00% <0.00%> (+100.00%)
:arrow_up:

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


"Failed to authenticate" recently started on working devices

We've also just recently had some power outages.
2 of 4 ACs (Toshiba) affected.
How does one verify the IP address (of the AC) that HASS is trying to use to communicate with it?
UPDATE:
The IP addresses have change on these ACs.
I will set them to static - but still need to know where to find the IP Addresses that HASS is expecting to use to communicate with them...

If you enable debug logging you should be able to see what IPs are being used.


XCTWaiter.subsystemQueue.sync is always non-null

No need to have the null check via "?"

This is optional because the value returned by the closure passed to this call to XCTWaiter.subsystemQueue.sync is optional. It's returning the value queue_didFulfillHandler, which is a property whose type is an optional closure. So I believe this is correct as written
I would expect this branch to not build successfully as a result… have you tried?


Allow for config to be erb processed

In some deployment environment setups (12factor) the application is expected to read its configuration information from the environment variables present on the system. By erb processing the config file in dry-component, it allows you to read the variables into application config (eg DATABASE_URL), and cleanly store constants (eg max_upload_size) in source control.

I'm not entirely sure this is the best idea, since Component::Config is designed to fetch config from matching ENV vars before falling back to the yml file – see https://github.com/dry-rb/dry-component/blob/master/lib/dry/component/config.rb#L17. Using ERB processing in the values from yml file means that the ENV- and yml-provided data will no longer match.
In your example, you mentioned 2 use cases:

Reading ENV vars
Cleanly storing constants

As far as I can tell, (1) is already taken care of by the existing Component::Config behaviour (the source code I linked to above).
Would you mind taking a bit more time to explain how you see (2) working?

I apologize for not making it clearer but one and two are two benefits produced by case one. Seeing the code you referenced now, it does accomplish a similar goal. I do feel though putting a reference to the environment variable in the yml is less confusing versus merging it in after the yml is loaded.

I'll close this one now since we've moved config over into dry-web and made it a lot more flexible, including the ability for you to provide your own entirely custom settings loader (see this and this for more).


Blue screen with empty username

there is a seriously issue with pGina-<IP_ADDRESS> and previously versions
on the login screen
when i write a username then clear it then press enter or the submit button it turns to blue screen with word connecting , i had to restart the machine , faced it several times
Please help ,

We'll look.


Finish transaction / federation details needed

like status

Is this related to a pending transaction? i.e. I just pegged in on-chain and am waiting for conf but the History doesn't indicate this. and the balance is not updated to reflect the status.

Probably need a way to indicate pending transactions. Those should require 10 confirmations.

Backend already has these statuses, front end just needs to handle them


导出的文件中输入0079自动把0都去掉了

导出的内容是字符串是正常的显示，但是一旦更改输入00几，就会自动去掉前面的0

Set a number format.  Demo https://jsfiddle.net/fobgtdyr/
var data = [["General", "0000"], [79, 79]];
var ws = XLSX.utils.aoa_to_sheet(data);
ws["B2"].z = "0000"; // set the number format to 0000 to force 4 digits
var wb = XLSX.utils.book_new(); XLSX.utils.book_append_sheet(wb, ws, "Sheet1");
XLSX.writeFile(wb, "issue2449.xlsx");

Checking for update after 10 am central today
On Fri, Nov 12, 2021 at 2:01 AM reviewher @.***> wrote:

Set a number format https://docs.sheetjs.com/#number-formats. Demo
https://jsfiddle.net/fobgtdyr/
var data = [["General", "0000"], [79, 79]];var ws = XLSX.utils.aoa_to_sheet(data);ws["B2"].z = "0000"; // set the number format to 0000 to force 4 digitsvar wb = XLSX.utils.book_new(); XLSX.utils.book_append_sheet(wb, ws, "Sheet1");XLSX.writeFile(wb, "issue2449.xlsx");
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/SheetJS/sheetjs/issues/2449#issuecomment-966899548,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVQWQIFSUG3LQWFCF6BGKQDULTCURANCNFSM5H4G2SBA
.

Thank you for your help.  I saw your demo and there was no problem when I converted it when EXPORTING. The main problem is that 00 will be automatically removed when I edit it again in Excel after exporting
I currently use a temporary solution
cell.z = "@"; .
To solve the problem of filling a multi-line empty string
Do you have a better solution?

00 has tradionaly been used for GO&S. When troubleshooting for 3G starting
@1G, over 20 ya go, spiking for 7-9G from Dragons Tail was no problem. I do
not wonder about my past, I know it. I was there. I lived it. What troubles
my thoughts, is the reason for 97…now. I am not lonely. I am alone. I have
always been alone in my travels through the spectrum documentation. 00 was
always asked of me, the older ones traveling felt the did not need it. I
ALWAYS traveled in a loop to cover for the ones who did not want to study
and guide for the group, worrying that they would miss something during
those times, they did. I asked not long after the 5th or 6th or 7th time to
00, what am I now…after I worked to to RS1 from RS10..Command & Control
said^you are Ecgo-3. Are you ok, with that?^…I thought for a moment in time
that I was special, then I realized, I was the termination expert for the
combining of RS to RF. I tire of monotonous repetitive natured questioned…I
am the only RSRF1-00…never forgetting that I am here to protect 00 from 80
all the way through every door to 00. I do not say if anymore, I say when
etc… it is something that only I know that NO ONE else could know…..Prime
is for the one who keep his heart shut, not their mouths..I have documented
the infrastructure of not only the fiber optics, instructing my favorite
phone company, AT&T plus Apple peripheral devices, to create a device for
5G to the residential house in the last 2 years, …all the while, keeping ,
not only my heart shut, as well as, my mouth…to treat me with the same
respect, honor, commitment, courage that I have given to them. If it is my
time to go. Send me an Angel to end my time alone. I will leave quietly
after kicking open the door, as I did so long ago, when we were working on
the Invisibility Cloak, ..asking when the door flew open after I kicked it-
Whos ass to have to kick to be a part of this team. Take everything from
me. Take my thoughts, my strength of character, my heart that beats for my
government, my soul that holds the Defense Department close to my chest,
take my tip point, my accuracy, my threading of a needle..if I am not
needing at the front gate nor the rear gate, then I am no longer needed…put
me out of my misery. I beg of you. Allow to help with the future
Warfighters in Electromagnetic Battle Management, or end me now.
Bryan Maurice Bellipanni
November 14, 2021
Non Desistas, Non Exieris
HUA
On Sun, Nov 14, 2021 at 20:45 bingyu929 @.***> wrote:

Thank you for your help. I saw your demo and there was no problem when I
converted it when EXPORTING. The main problem is that 00 will be
automatically removed when I edit it again in Excel after exporting
I currently use a temporary solution
cell.z = "@"; .
To solve the problem of filling a multi-line empty string
Do you have a better solution?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/SheetJS/sheetjs/issues/2449#issuecomment-968466467,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVQWQIE24GGUVIS6EEJ3BNDUMBX33ANCNFSM5H4G2SBA
.


add report_wire_length cmd

@eder-matheus Can I clone and build this branch to run the command?

@eder-matheus Can I clone and build this branch to run the command?

Yes, you can add my fork as a remote in your local repository and checkout to this branch.

@eder-matheus   I cloned and built your forked branch and tested with nangate45/gcd (default make in ORFS)  I opened the detailed route in gui  (make gui_5_route.def)

report_wire_length -net net36  -global_route   -- returns 0 wire length -- Once the design is detailed routed do we loose the global route info -- Matt was mentioning it is not saved in the DB
report_wire_length -net net36 -detailed_route -verbose -- core dumps
report_wire_length -net * -detailed_route -verbose -- prints out a bunch of nets and then coredumps when it reaches net VDD

Can we determine the state of the design  and default to appropriate mode (global or detailed route).

Once you quit and restart the guides go away.  Presumably you could use read_guides to get them back for this purpose or you could just run the command after global_routing.

@maliberty is correct. If you use the read_guides command the data will be restored into grt structs. But I think a check if there are guides in the structs should be added.
I forgot to skip power/ground nets. I'll fix it in the next commit.

@eder-matheus   I tried the latest commits and it fixes the issues I was seeing with the coredumps the state setting.

@maliberty @ravi-varadarajan Where should I add the documentation for this new command? I don't think it belongs to grt, since we have info from drt.

@maliberty @ravi-varadarajan Where should I add the documentation for this new command? I don't think it belongs to grt, since we have info from drt.

You could put it in grt with a note in drt.  Alternatively the whole command could move the the ord level since it is multi-component.


使用vllm部署qwen时可以配合embeddings model部署在一起都提供openai格式的api吗

是否已有关于该错误的issue或讨论？ | Is there an existing issue / discussion for this?

[X] 我已经搜索过已有的issues和讨论 | I have searched the existing issues / discussions

该问题是否在FAQ中有解答？ | Is there an existing answer for this in FAQ?

[X] 我已经搜索过FAQ | I have searched FAQ

当前行为 | Current Behavior
No response
期望行为 | Expected Behavior
No response
复现方法 | Steps To Reproduce
No response
运行环境 | Environment
No response
备注 | Anything else?
No response

您找个支持openai_api的embedding model应该就可以。Qwen属于生成类模型，不适合做embedding model。


UI for SEP-08 Send Asset flow

Screens and UI flow for the following:

[x] Start or Cancel SEND action
[x] SEND action
[x] Handle additional information required from Server (build client forms dynamically)
[x] Handle addition information response
[x] User review for revised transaction (Cancel or Send)
[x] Handle transaction send response (success or fail)

This should be complete by now. Please let me know if you think there's something missing.


Add substring support for FixedSizeBinaryArray

Which issue does this PR close?
Closes #1618.
What changes are included in this PR?

make the substring kernel support FixedSizeBinaryArray
add more test cases for binary and utf8
add benchmark for FixedSizeBinaryArray

Are there any user-facing changes?
Rename some benchmarks.
Bench Result
substring utf8 (start = 0, length = None)
                        time:   [19.811 ms 19.843 ms 19.883 ms]

substring utf8 (start = 1, length = str_len - 1)
                        time:   [21.270 ms 21.284 ms 21.299 ms]

substring fixed size binary array
                        time:   [19.717 ms 19.753 ms 19.798 ms]

Codecov Report

Merging #1633 (566e0ff) into master (7d00e3c) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1633      +/-   ##
==========================================
+ Coverage   83.02%   83.08%   +0.05%
==========================================
  Files         193      193
  Lines       55577    55772     +195
==========================================
+ Hits        46145    46338     +193
- Misses       9432     9434       +2

Impacted Files
Coverage Δ

arrow/src/compute/kernels/substring.rs
100.00% <100.00%> (ø)

arrow/src/array/transform/mod.rs
86.57% <0.00%> (-0.23%)
:arrow_down:

arrow/src/array/array_union.rs
90.63% <0.00%> (-0.09%)
:arrow_down:

parquet_derive/src/parquet_field.rs
66.21% <0.00%> (ø)

parquet/src/arrow/arrow_writer.rs
97.66% <0.00%> (+<0.01%)
:arrow_up:

parquet/src/encodings/encoding.rs
93.56% <0.00%> (+0.18%)
:arrow_up:

arrow/src/array/array_binary.rs
93.26% <0.00%> (+0.28%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7d00e3c...566e0ff. Read the comment docs.

I will merge this later today if no more comments.
cc @tustvold @alamb

Thank you @HaoYang670 @alamb ! 😄

Thanks a lot for your review @viirya @alamb !


RHBPMS-4106: GSS A ClassNotFound exception is shown while a …

…Service task is being executed

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

ok to test

Thanks @lazarotti. Would it be possible to include a test as well?
FYI: if you accept the invitation to droolsjbpm github org. unit, the PR builds will be automatically executed for your PRs.

Hi Petr - I really tried create an unit test for this but the bug depends on Classloader isolation and I was not able to create a test for this or even mock the scenario. I tried creating a custom classloader but I was stuck in isolating the class for the other parts of the scenario. If there is any test related to classloader issues let me know to take a look as basis.

About the Jenkins failure, I don't think it is related to this PR.

is this only 6.4.x branch related or it requires back ports?

Also forward port Maciej. Sorry, I add it as upstream to product 6.4.x and this is why the PR was just to 6.4.x. Should I open a new PR to other branches ?

if you could open these that would be great, thanks

@lazarotti I see that that jira does not have acks so I guess we need to wait with it before it gets all approvals.

The PRs for 6.5.x and master are:
https://github.com/droolsjbpm/jbpm/pull/516
https://github.com/droolsjbpm/jbpm/pull/517
Feel free to revert the 6.4.x commit to wait until the acks if necessary. Cheers

thanks @lazarotti closing these PRs as they are replaced with other that includes test case on top of your fix - https://github.com/droolsjbpm/jbpm/pull/521


New traffic release based on 4.9.0 [publish binary]

This is in https://github.com/Project-OSRM/node-osrm/pull/142 now


PING Edits for Introduction and How To Use

Updates to the security and privacy questionnaire from PING's review.  This week's work was focused on the Introduction and How to Use sections.

I like the included "concrete privacy concerns", though we're getting a significantly expanded document now ;-)
I also see the suggested way of requesting a review is addessed: (1) PING-contact recommended, (2) then TAG review.
I'm of course OK with that. My motivation was to have the questionnaire meaningfully filled when sent to the TAG, which is currently often not the case. What's the PING experience here actually? I believe this is currently also not the case. So the proposed workflow is now to:
(1) Optionally contact PING
(2) Mandatorily fill the questionnaire prior to requesting of TAG review?
In that case we might have a possible wording issue

"along with the description of changes or observations made
during the design and outcomes of your discussion with PING. "

Since the (1) is "recommended", and in (2) we have an assumption this has happened. Should we then
(1)  add "possibly along"
or
(2) remove the mentioning of a PING review when speaking of TAG review?
Not insisting on any particular - just observing. So not filling an edit.

Hi,
After a TAG call.
We're quite concerned with

" When requesting a Technical Architecture Group review, include the filled
questionnaire, along with the description of changes or observations made
during the design and outcomes of your discussion with PING. This allows
external reviewers understand the rationale, as well as the challenges and
evolution of the feature, with respect to security and privacy."

Specifically the bolded content here. The paragraph I suggested was meant to help us during TAG review, but we do not intend to have a firm requirements of having consulting with PING (although that is most welcome). The issue is that we're more or less formalizing a new model of SP design. But we do not think we should introduce a formal prerequisite like that. Let's either rephrase this particular sentence or remove it.
Previously today I included this change:

"during the design and outcomes of your discussion with PING, or other
external contributors or researchers. This allow

At this moment, it would probably be best to remove the sentence that assume a PING consultation as a prerequisite of asking for a TAG review (but SP questionnaire filled should be the prerequisite).
I'd suggest to change to:

" When requesting a Technical Architecture Group review, include the filled
questionnaire, along with the description of changes or observations made
during the design process. This allows
external reviewers understand the rationale, as well as the challenges and
evolution of the feature, with respect to security and privacy."

Additional comment: PING communication is recommended, but then the language uses word "should". So do you suggest it's an optional recommendation or a more mandatory step?

Hi @npdoty and thanks for your input. I agree on being unsure about RFC terminology. I guess we're not doing this.
I also agree that the more eyes on a feature, the better. That said, let's not mix PING and TAG reviews, unless we wand to establish a formal process for that. My desire was to simply make sure that TAG gets a SP-q filled meaningfully. Furthermore, TAG gets quite a lot of specs/features to review at an early stage (in the "explainer" format). Unrelated to the PING communication of course.
PING communication is most welcome, but we wouldn't view it as a formal prerequisite prior to a TAG review. We simply encourage considering of security/privacy. It is up to the authors to choose how they  want to go with that - and the preceding paragraph on PING is sufficiently indicative. In other case, let's merge what we have now and possibly see how the document look like when it's (hopefully) final.


Fixes #355 stack overflow with Pos based on the size of windows at startup. Added a OnResized action to set the Pos after the terminal are resized.

Thank you for tracking this down and the patch!


Sharing sprite between two layers?

I am trying to create a mesh that has layers of series of sprites both horizontal and vertical.

All the boxes are sprites. I have two layer L1 and L2. And there is one sprite shared between two layers.
But looks like the common sprite can only belong to either L1 or L2.
Is there a way to share sprite between two layers?
Thank you in advance! :)

That's a strange case. What will be behaviour of pixi-display in that case according to your idea? Suppose element has two parentLayers, how do you think it will be rendered?

Thinking this through it does seem a bit strange. More or less it looks like layers for fixed positions rather than specific sprites.
Will work on it to see if I find something.
Thank you!


Model graph not rendered

Just tried some basic stuff on Win 10 and everything worked nicely (tutorial 1), except that I can’t see any model graph.
No error message either (is there a log?), the pane just says: “No Graph”.
I’m on ver 0.9.5

The green status ist there, yes!
It says: Model built (1,199,882 parameters)

@jplasser Can you install TorchStudio 0.9.6, run it, build a model, and if it still doesn't display a graph, can you upload here the log files located in %USERPROFILE%\TorchStudio\logs ?


Command Line Tools install fails for Catalina

Describe the Bug
Due to changes in the output of softwareupdate —list,  Xcode Command Line Tools are not being installed for Catalina.
Error Message
- execute [“softwareupdate”, “—install”, “No Command Lines Tools available from Software Update Catalog!’]

Resolved by #213


Using MetricAffectSpans to get different typeface leads to wrong padding

Because of the missing Typeface-setting option, I tried a real clue:
Setting texts using SpannableStrings that include a different MetricAffectingSpan (see http://stackoverflow.com/a/15181195/273456)
But when doing that, the padding around the text gets really small, the toasts become extremely thin...
Could you help?

@Zordid I'm not sure if this is still an issue so I am closing this.
P.S.I have consolidated the managers into a single class called the Toaster in honor of your comment


ci: cache webview installer

https://github.com/firezone/firezone/actions/runs/9008970452/job/24752210427?pr=4928

https://github.com/firezone/firezone/actions/runs/9024515614/job/24798528039#step:4:57
Even though the cache worked, it seems like the installer just hangs sometimes. On Windows 2019 it finished in 1 minutes, on 2022 it's been going for 30 minutes.

Might not be our fault https://github.com/MicrosoftEdge/WebView2Feedback/issues/4443
I didn't notice this before, and now it's happening on any job that has setup-tauri, so I think it could be a regression in the WebView code, and I've only seen it on Windows 2022 so far. (static analysis and build-tauri)


PARQUET-805: Read Int96 into Arrow Timestamp(ns)

Also verified locally that this correctly reads the timestamps produced by Spark


Filtering with function in native traversal does not work in some cases

I wonder if this is a bug.
This works:
FOR v,e,p IN 1..100 OUTBOUND 'taxent/3010706975127' EXISTS_IN,PART_OF
LET col=PARSE_IDENTIFIER(p.edges[0]._id).collection
FILTER col=='EXISTS_IN'
RETURN p

But this does not work, albeit equivalent:
FOR v,e,p IN 1..100 OUTBOUND 'taxent/3010706975127' EXISTS_IN,PART_OF
FILTER PARSE_IDENTIFIER(p.edges[0]._id).collection=='EXISTS_IN'
RETURN p

But curiously this does work:
FOR v,e,p IN 1..100 OUTBOUND 'taxent/3010706975127' EXISTS_IN,PART_OF
FILTER (PARSE_IDENTIFIER(p.edges[0]._id).collection=='EXISTS_IN')==true
RETURN p

This seems to happen only in filtering traversals, but I did not test everything.

Hi can confirm that this is broken. seems to be true for all functions. I will look into it. Thank you very much for reporting it.

Thank you!
BTW, I think it doesn't make much sense that to know a collection to which a document belongs, we must call a function. That doesn't allow optimizations like in this case. The collection should be an intrinsic property of the documents readily accessible, thereby allowing query optimizations, IMHO.

hi @miguel-porto ,
i totally agree with you. This function was not really necessary unit the new Pattern Matching / Graph Traversal features and we are going to add it asap. The plan is that the query optimizer knows this special function for optimization.

Yes that's true, this became only important for the new native traversals... thanks, all the best!


Extension host terminated unexpectedly. Please reload the window to recover.

VSCode Version: Code 1.15.0 (8b95971d8cccd3afd86b35d4a0e098c189294ff2, 2017-08-09T20:16:18.034Z)
OS Version: Windows_NT x64 10.0.15063
Extensions:

Extension
Author (truncated)
Version

ng-template
Ang
0.1.4

path-intellisense
chr
1.4.2

bracket-pair-colorizer
Coe
0.10.8

vscode-eslint
dba
1.2.11

vscode-html-css
ecm
0.1.7

EditorConfig
Edi
0.9.4

tslint
eg2
0.17.0

php-intellisense
fel
1.5.0

beautify
Hoo
1.1.1

vscode-angular2-emmet
jak
2.0.3

csharpextensions
jch
1.3.0

angular-essentials
joh
0.2.0

Angular2
joh
2.4.0

pwa-tools
joh
0.1.0

vscode-JS-CSS-HTML-formatter
lon
0.2.3

code-beautifier
mic
1.7.2

easy-less
mrc
1.4.5

vscode-scss
mrm
0.6.2

csharp
ms-
1.11.0

debugger-for-chrome
msj
3.1.7

angular2-inline
nat
0.0.17

material-icon-theme
PKi
2.1.0

live-sass
rit
0.3.3

LiveServer
rit
1.6.8

sass-indented
rob
1.4.1

html-css-class-completion
Zig
1.8.0

Steps to Reproduce:

This seems to happen on save when saving.

Reproduces without extensions: No

If the problem doesn't occur if all extensions are disabled, then an extension seems to be the culprit and not VS Code. Please try to find the extension my manual bisect.


more hints about eliminating Data.Foldable.toList

Essentially a continuation of the already merged https://github.com/ndmitchell/hlint/pull/1569.
One thing I'm unsure about: Would these hints also catch/apply to Data.Set.toList calls (based on the Foldable instance for Set)? Or would it be necessary to duplicate all these hints (the ones from the previous issue and these ones here) for Data.Set.toList?

I have updated hints.md.
About this:

No - HLint does not know Data.Set.toList and Data.Foldable.toList are the same toList.

Would it then make sense to duplicate all the hints from https://github.com/ndmitchell/hlint/pull/1569 as well as the ones from this PR here, into versions for Data.Set.toList in place of Data.Foldable.toList?


[Feature] Custom order of issues

Hi, we're currently managing around 200 issues on a gitea repository and it's getting increasingly difficult to manage which issues are more important than others. We're currently using labels like Priority: High to give us some level of granularity, but it would be easier if we could sort them manually to have a priorized backlog.
If #11161 get's merged, we could do it in a project with only one column if the issues are shown one above the other.
Or maybe currently there is a way to do it that I'm unaware of.
Thanks

Isn't this issue a duplicate of #14377?

Isn't this issue a duplicate of #14377?

Indeed it is, sorry for the duplication

Indeed it is, sorry for the duplication


Caret in on-screen error output points to the wrong line

Describe the bug
When there is a LESS compilation error and the error appears in the browser, the caret points to the wrong line:

In the scenario above, the problem is on line 14 (display.less is missing). The error message at the top is correct, and the horizontal (column) location of the caret seems correct, but it's off by one line. It's pointing to line 13 instead of line 14. This issue is 100% reproducible with any kind of LESS compilation error.
Reproduction
N/A
Steps to reproduce
Induce a LESS compilation error (e.g. missing import).
System Info
System:
    OS: Windows 10 10.0.22621
    CPU: (8) x64 11th Gen Intel(R) Core(TM) i7-1185G7 @ 3.00GHz
    Memory: 4.77 GB / 15.71 GB
  Binaries:
    Node: 18.16.0 - C:\Program Files\Node.js\node.EXE
    npm: 9.8.0 - C:\Program Files\Node.js\npm.CMD
    pnpm: 8.6.10 - C:\Program Files\Node.js\pnpm.CMD
  Browsers:
    Edge: Spartan (44.22621.2215.0), Chromium (116.0.1938.69)
    Internet Explorer: 11.0.22621.1
  npmPackages:
    @vitejs/plugin-react: 4.0.4 => 4.0.4
    vite: 4.4.9 => 4.4.9

Used Package Manager
pnpm
Logs
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 reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Minimal repro: https://stackblitz.com/edit/vitejs-vite-kpntc1?file=styles.less

Testing the above repro, it was found that the 'less' error column needs +1.
use lessc

use less render API

The less has modified the toString method of LessError, resulting in column + 1.
https://github.com/less/less.js/blob/master/packages/less/src/less/less-error.js#L107C26-L107C26

Interesting, but isn't the problem with the line number, not the column? The caret is pointing to the wrong line, and that's off by one.


Add log collector

We need the ability to collect the logs (stdout + stderr) from every process into one or more files so that other processes can parse the log during lifetime and post-mortem.

All progress will be moving to https://github.com/dotnet/tye


Style property on elements

Is there a reason why there isn't a style function in Halogen.HTML.Properties?
Would like to do add some css to the element based on state.

style is defined in purescript-halogen-css... or would be if I had pushed anything to that repository! I'll fix it now.

It's there now, and I've made an initial release that is compatible with Halogen master.

Nice, thanks!


Get more information about "Test Run Aborted" failures

We see occasional cases where a test run fails with "Test Run Aborted" and no specific failing test. This is usually due to the test process crashing from a failed assertion or background thread exception. We get almost no data from these, so we need to look at how to get more data into the build log so we can identify the problem.

Closing in favor of https://github.com/aspnet/AspNetCore-Internal/issues/2079. As you mentioned, "Test Run Aborted" aborted is almost always the result of a test process crashing.
@JunTaoLuo let's add "vstest.console.exe" to the list of processes we monitor for crash dumps in https://github.com/aspnet/AspNetCore-Internal/issues/2079


Interop broken on Linux?

$ WASMTIME_BACKTRACE_DETAILS=1 go test -v ./trealla
=== RUN   TestInterop/custom_function
2023/07/19 18:29:51 query: interop_test(X).
2023/07/19 18:29:51 query: X is 1 + 1.
=== NAME  TestInterop
    interop_test.go:34: trealla: query error: error while executing at wasm backtrace:
            0: <unknown>!pl_query
            1: 0x1fe1db - fn_sys_host_call_2
                            at /Users/guregu/code/trealla/fork/src/predicates.c:7692:12
            2: 0x234bc6 - start
                            at /Users/guregu/code/trealla/fork/src/query.c:1611:14
            3: 0x23b115 - execute
                            at /Users/guregu/code/trealla/fork/src/query.c:1823:9
            4: 0xf8327 - run
                            at /Users/guregu/code/trealla/fork/src/parser.c:3630:3
            5: 0x223e56 - pl_query
                            at /Users/guregu/code/trealla/fork/src/prolog.c:162:12

        Caused by:
            wasm trap: call stack exhausted

Works ok on macOS+arm64 so probably something weird going on. This is running in a VM though.

This should take care of it: https://github.com/bytecodealliance/wasmtime-go/pull/180

Hmm... it looks like it's more complex than I thought.
wasmtime divides the stack space every time a guest -> host -> guest (reentrant?) call happens
Even with the larger stack, if there's more than 2 or so reentrant queries happening the stack just becomes too small.
This can be worked around by calling Clone() on the instance passed to the native predicate, but incurs an overhead.
It might be possible to avoid this with a more complex locking system but probably not worth it.


Create landing page on ices.dk for cetaceans portal

@NikkiTaylorJNCC can you find a few nice images of whales/dolphins (landscape format) and min. 400px resolution
link to JCDP hub JCDP hub

Attached a harbour porpoise image - the most applicable to the JCDP audience, and the JCDP logo if that could also feature. Can supply the Jpeg if better.

Attached a harbour porpoise image - the most applicable to the JCDP audience, and the JCDP logo if that could also feature. Can supply the Jpeg if better

thanks Nikki - these will work 🐬

initial landing page established https://www.ices.dk/data/data-portals/Pages/Cetaceans.aspx (visible if you have the URL but not linked to navigation)

Great! I'll let you know if we need any tweaks to the text (assuming we have free reign to adapt the text?)

Great! I'll let you know if we need any tweaks to the text (assuming we have free reign to adapt the text?)

please send any edits or additions 👍

@neil-ices-dk a few edits to the text on the landing page. Suspect I will need to run this by the SG as well in my next email update, so may be some suggestions from them also:
Cetaceans (whales, dolphins and porpoises) data to understand their abundance and distribution.
This portal assembles at-sea effort-related data collected via ship-based or aerial methods, collated under the Joint Cetacean Database Programme (JCDP).
The JCDP aims to streamline the process of accessing and utilising these data by collating the existing and forthcoming cetacean evidence base into a single resource. These data have a wide range of applications and have the potential to support analyses at a range of spatial and temporal scales. For more information, visit the JCDP hub (see links)
[Could we rename the link to ‘JCDP hub’]

@cmspinto when you select the map view, inventory or submit data on the JCDP landing page, it navigates you away from the landing page - is it possible to have a button added to those pages to return to the landing page somehow? Or open them up in a  new window..?

Maybe this could be a bit more visible, but that already exists :)
It's the link in the top that says Cetaceans.

Thanks for publishing the data overview dashboard!
@cmspinto @neil-ices-dk @niki
Couple of asks for the landing page:

Can we move the data overview dashboard link up to where the orange links are
Can we move the metadata catalogue also be moved up to the orange links under the porpoise image?

Can we rename 'data format' link to 'data template'

Could we also make sure all of them are either capitalised for all words, or all with just the first word? e.g Data Overview Dashboard v Data overview dashboard
Thanks!

@NikkiTaylorJNCC we can do that, although the web guidance is we would normally have 3-4 orange 'action' links. So if we move both of these up, we should demote one of the existing ones - which one should it be?

@neil-ices-dk Sophie's choice!
My suggestion is to demote data inventory and have the map as the main filter tool...then we can have the file submission log and data inventory together.

Landing page completed and published. Content may be reviewed in future as part of ongoing development and support.
Closed


🐛 [Bug]: tiny-engine-ui里bundle.json配置popeditor错了一个小小的地方

Environment
local
Version
node -v 18.20.3
Version
"@opentiny/vue": "~3.14.0"
Link to minimal reproduction

Step to reproduce
预览页会提醒报错原因，打开js一看发现命名不一致，需要改一下
What is expected
No response
What is actually happening
No response
What is your project name
any
Any additional comments (optional)
No response

可以直接给我们提 PR @sunny-Fung


Added aws infrastructure and updated docs

Added aws infrastructure and updated docs.

Hi, sorry for the delay on this. We've had a bit of an existential crisis with this repo, as when we initially published it we weren't anticipating that people would perceive it as a vector for contributing various ways to deploy Concourse with BOSH - it was initially just our toolchain for generating our single-VM VirtualBox image.
We're now embracing that and steering the documentation in this repo in that direction. I'll go ahead and merge this in now and tidy everything up. :) Thanks!


Add new-formatted retriever function

For refactoring purposes, we need to make a server application that will exist along with the current CDN application. Long in short, it will look like
* /
  * apt/
  * auth/
  * ...
  * server/
    * main.go
    * retriever.go
    * callbacks.go
  * ...

We need to add a retriever function.
The callback will build a query with a special method and pass this query to the retriever function. Retriever function will do its search using the search function and return a slice of files to the callback function.

Retriever function will take the following arguments:
fileID string
name string
owner string
token string
tags string
version string
limit int
offset int
repo string

Retrieve must return SearchResult structure that will have the following arguments:
1. fileID (MyBucket -> fileID)
2. owner (MyBucket -> fileID -> owner -> first key/value pair)
3. name (MyBucket -> fileID -> key:name)
4. repo (MyBucket -> fileID -> type -> buckets)
5. version (MyBucket -> fileID -> key:version)
6. scope (MyBucket -> fileID -> scope -> all key/value pairs)
7. md5 (MyBucket -> fileID -> hash -> key:md5)
8. sha256 (MyBucket -> fileID -> hash -> key:sha256)
9. size (MyBucket -> fileID -> size)

Done.


Adapt library for AndroidX

When building this library in modern AndroidX environment, there are dependency resolution errors, as such:
Duplicate class android.support.v4.app.INotificationSideChannel found in modules core-1.6.0-runtime (androidx.core:core:1.6.0) and support-compat-27.1.1-runtime (com.android.support:support-compat:27.1.1)
Duplicate class android.support.v4.app.INotificationSideChannel$Stub found in modules core-1.6.0-runtime (androidx.core:core:1.6.0) and support-compat-27.1.1-runtime (com.android.support:support-compat:27.1.1)
Duplicate class android.support.v4.app.INotificationSideChannel$Stub$Proxy found in modules core-1.6.0-runtime (androidx.core:core:1.6.0) and support-compat-27.1.1-runtime (com.android.support:support-compat:27.1.1)
Duplicate class android.support.v4.os.IResultReceiver found in modules core-1.6.0-runtime (androidx.core:core:1.6.0) and support-compat-27.1.1-runtime (com.android.support:support-compat:27.1.1)
Duplicate class android.support.v4.os.IResultReceiver$Stub found in modules core-1.6.0-runtime (androidx.core:core:1.6.0) and support-compat-27.1.1-runtime (com.android.support:support-compat:27.1.1)
Duplicate class android.support.v4.os.IResultReceiver$Stub$Proxy found in modules core-1.6.0-runtime (androidx.core:core:1.6.0) and support-compat-27.1.1-runtime (com.android.support:support-compat:27.1.1)
Duplicate class android.support.v4.os.ResultReceiver found in modules core-1.6.0-runtime (androidx.core:core:1.6.0) and support-compat-27.1.1-runtime (com.android.support:support-compat:27.1.1)
Duplicate class android.support.v4.os.ResultReceiver$1 found in modules core-1.6.0-runtime (androidx.core:core:1.6.0) and support-compat-27.1.1-runtime (com.android.support:support-compat:27.1.1)
Duplicate class android.support.v4.os.ResultReceiver$MyResultReceiver found in modules core-1.6.0-runtime (androidx.core:core:1.6.0) and support-compat-27.1.1-runtime (com.android.support:support-compat:27.1.1)
Duplicate class android.support.v4.os.ResultReceiver$MyRunnable found in modules core-1.6.0-runtime (androidx.core:core:1.6.0) and support-compat-27.1.1-runtime (com.android.support:support-compat:27.1.1)

There is a way to fix it by enabling jettifier. It would be much better if these issues were fixed at the library level. @gregcockroft, please consider adapting your very helpful library to AndroidX.

hi ,you can  add.  'android.enableJetifier=true'.    in.  gradle.properties

Added 'android.enableJetifier=true but same exception .


added hashtags to tweet. Also bugfixed the text which consist only of…

… the first part of a tweet until a hashtag appears.
CSV Separator is now pipe since it´s more stable.

By the way. Thanks for this nice package :)

Reminder :)

Reminder :)


DRAFT: Mythras v3 Complete Re-write

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.

[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).

[x] The pull request makes changes to files in only one sub-folder.

[ ] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)
It does but this was part of the mass i18n key renaming I did and I progammatically converted the key names in translations to match the new names while retaining the translation data.  I honestly think this is better than leaving a bunch of unused keys and having the community re-translate the pretty much everything.

[x] I have followed the Character Sheets Standards when building this sheet.

Changes / Description
The CHANGELOG.md file contains a more itemized list of changes.
This was a complete re-write of the sheet from the ground up.  I am the original and only contributor to the Mythras sheet so far so I'm not wiping out anyone else's work with this re-write.  This new version has been in the works for some time and the Mythras community have been given opportunities to test and provide feedback on the changes being made.
There are a lot of attribute consolidations and simplifications put into this version as I have been attempting to resolve some performance issues in the previous versions and make the sheet easier to maintain (Mythras is a toolkit system like GURPS in some ways and every setting that is released has its own tweaks).  The Django templates will eventually allow me to make more specialized versions of the sheet for systems which are published with the Mythras rule, but for now this Mythras sheet can be reconfigured to match any setting and/or optional toggle pieces for homebrew games.
Extensive work has been done to modernize the sheet including using the newer sheet syntax (legacy: false).  I have also re-worked the sheetworkers such that when a single input is updated it doesn't trigger a chain of triggers. Each edit should mass update all downstream changes as needed.
This v3.0 of the sheet is fully documented at https://wiki.roll20.net/Mythras_Sheet.
Please feel free to ask about anything.

Character Sheet Info Roll20 Internal Use only.


Default value props for date picker

Adds new props for date picker.

default-date-preset: Will apply the preset and dates accordingly, can accept both key and label.
default-start-date: Will apply the date as start date, also respects mode(range, single) .
default-end-date: If empty but start date has default, then same is applied to this one to avoid any glitch.

Note:

If default preset given then start and end date defaults are skipped.
Using default dates will automatically set preset to custom (if preset default not used).

Why are you adding this? Was there a request to add a default?

Was there a request to add a default?

Totally forgot about it 😅, yes this was requested from Kennan.