CLEAN:  Remove some unnecessary types from S1

I think that S1 has types that are not needed, e.g., the datetime types.
As discussed here https://github.com/pandas-dev/pandas-stubs/pull/519#pullrequestreview-1271164311
we should see which ones we can safely remove without changing any tests or other parts
For things like Timestamp, Timedelta, Period, we may be able to get rid of all references to Series[Timestamp], Series[Timedelta] and Series[Period] because we have TimestampSeries, TimedeltaSeries and PeriodSeries, but we leave the references just in the class definition there from an inheritance perspective.
@ramvikrams to investigate

I think that S1 has types that are not needed, e.g., the datetime types.
As discussed here #519 (review)
we should see which ones we can safely remove without changing any tests or other parts
For things like Timestamp, Timedelta, Period, we may be able to get rid of all references to Series[Timestamp], Series[Timedelta] and Series[Period] because we have TimestampSeries, TimedeltaSeries and PeriodSeries, but we leave the references just in the class definition there from an inheritance perspective.
@ramvikrams to investigate

i'll look into it

1)For Series[Timestamp]
from test_scalars.py we have this TimestampSeries can't be used here
    c_series_dt64: pd.Series[pd.Timestamp] = pd.Series(
        [1, 2, 3], dtype="datetime64[ns]"
    )

We have instances like this, herealso we can't replace it
_EdgesTimestamp: TypeAlias = Union[
    Sequence[DatetimeLike],
    npt.NDArray[np.datetime64],
    pd.Series[pd.Timestamp],
    TimestampSeries,
    pd.DatetimeIndex,
]

we also have this I thik both these are req.
    def __eq__(self, other: TimestampSeries | Series[Timestamp]) -> Series[bool]: ...  # type: ignore[misc]

Series[Timedelta] can be changed with TimedeltaSeries at most places and similary with Series[Period]
we cannot also remove interval[*]  as they are referenced
we can remove datetime.*

1)For Series[Timestamp]
from test_scalars.py we have this TimestampSeries can't be used here
    c_series_dt64: pd.Series[pd.Timestamp] = pd.Series(
        [1, 2, 3], dtype="datetime64[ns]"
    )

OK, that's actually a bug in the tests and the types because the correct type should be Series[np.datetime64] but if you make that change, then the operators aren't properly defined and you'll get other errors, so for now, you can leave that test alone and it is something I can look to fix.  (It's non-trivial)

We have instances like this, here also we can't replace it
_EdgesTimestamp: TypeAlias = Union[
    Sequence[DatetimeLike],
    npt.NDArray[np.datetime64],
    pd.Series[pd.Timestamp],
    TimestampSeries,
    pd.DatetimeIndex,
]

Are you saying if you remove pd.Series[pd.Timestamp] from that declaration, a test will fail?

we also have this I think both these are req.
    def __eq__(self, other: TimestampSeries | Series[Timestamp]) -> Series[bool]: ...  # type: ignore[misc]

Yes, it's related to (1) above.  So you have to leave it in for now.

Series[Timedelta] can be replaced with TimedeltaSeries at most places and similary with Series[Period]

That's good.

we cannot also remove interval[*]  as they are referenced

Yes, I think that has to stay.

we can remove datetime.*

Good.

We have instances like this, here also we can't replace it
_EdgesTimestamp: TypeAlias = Union[
    Sequence[DatetimeLike],
    npt.NDArray[np.datetime64],
    pd.Series[pd.Timestamp],
    TimestampSeries,
    pd.DatetimeIndex,
]

Are you saying if you remove pd.Series[pd.Timestamp] from that declaration, a test will fail?

No, test do not fail but it is  _EdgeTimestamp is called in a func in that file, So will it be a good option to remove it

    def __eq__(self, other: TimestampSeries | Series[Timestamp]) -> Series[bool]: ...  # type: ignore[misc]

Yes, it's related to (1) above. So you have to leave it in for now.

I don't think it's related to 1 as we have more declarations like this in pandas-stubs\_libs\tslibs\timestamps.pyi they are used in Timestamp funcs

Are you saying if you remove pd.Series[pd.Timestamp] from that declaration, a test will fail?

No, test do not fail but _EdgeTimestamp is called in a func in that file, So will it be a good option to remove it

So just to clarify you can remove pd.Series[pd.Timestamp] in that TypeAlias

    def __eq__(self, other: TimestampSeries | Series[Timestamp]) -> Series[bool]: ...  # type: ignore[misc]

Yes, it's related to (1) above. So you have to leave it in for now.

I don't think it's related to 1 as we have more declarations like this in pandas-stubs\_libs\tslibs\timestamps.pyi they are used in Timestamp funcs

Try removing them all and see which tests fail, then put back the minimal number so the current tests pass.  It's related because I think if you take Series[Timestamp] out of the def __eq__(), a test will fail, but that's because of (1) above.

    def __eq__(self, other: TimestampSeries | Series[Timestamp]) -> Series[bool]: ...  # type: ignore[misc]

Yes, it's related to (1) above. So you have to leave it in for now.

I don't think it's related to 1 as we have more declarations like this in pandas-stubs\_libs\tslibs\timestamps.pyi they are used in Timestamp funcs

Try removing them all and see which tests fail, then put back the minimal number so the current tests pass. It's related because I think if you take Series[Timestamp] out of the def __eq__(), a test will fail, but that's because of (1) above.

Actually there are 5, from 3 of them we can remove Series[Timestamp] mypy,pyright pass the test but pytest shows 0 errors some time,1 the other time or 2 sometime. Don't know what is wrong with pytest

Actually there are 5(__eq__ , __ne__ , __gt__ , __ge__ , __le__), from 3(__gt__ , __ge__ , __le__) of them we can remove Series[Timestamp] mypy,pyright pass the test but pytest shows 0 errors some time,1 the other time or 2 sometime. Don't know what is wrong with pytest

It might relate to (1).  If you want, create a PR with what you have and I can take a look.

Actually there are 5(__eq__ , __ne__ , __gt__ , __ge__ , __le__), from 3(__gt__ , __ge__ , __le__) of them we can remove Series[Timestamp] mypy,pyright pass the test but pytest shows 0 errors some time,1 the other time or 2 sometime. Don't know what is wrong with pytest

It might relate to (1).  If you want, create a PR with what you have and I can take a look.

Yes I'll create a pr

@Dr-Irv Sir i'll make the PR tommorow, i am not feeling well today So taking rest.

@Dr-Irv Sir i'll make the PR tommorow, i am not feeling well today So taking rest.

No rush!


Remove performance tests

To not waste energy for (travis) builds, I'd like to have long-running performance tests not be run during ./gradlew test.
There are several ways to go about this

move performance tests to another package (e.g., benchmark package)
delete performance tests
exclude performance tests from ./gradlew test runs.

I don't have strong feelings which option to take.

(I'm aware only of the SignatureScheme tests that take a long time)

(I'm aware only of the SignatureScheme tests that take a long time)

I removed the performance benchmarking from the signature tests with #33 by enabling the debug pairing usage in the SPSEQ tests. Benchmarks should be part of the Benchmark project as you mentioned and be separated from unit tests.


Refactor the provenance visualization to use a right side nav-draw

Closes #163
Moves the provenance vis to a right side nav panel that can remain on the vis as the user is interacting with it. We could also reduce the size of the nodelink visualization area to just between the two panels when the provenance vis is rendering, but I'll check in with Alex to see his wishes for that.

Since we need to toggle the state of the provenance menu from 2 locations, I think this is simplest. If it were just one, we could use the v-model


Issue with using regular widgets.

When trying to save widgets either custom or OOTB that don't use UI components, they are unable to be inserted due to a javascript error using Adobe Commerce 2.4.6-p5
Uncaught ReferenceError: form is not defined
insertWidget http://localhost.test/static/version1715042648/adminhtml/Magento/spectrum/en_US/Grasch_AdminUi/js/wysiwyg/widget.js:535

@jeremiebarnes This has been fixed. Please update the module to version 1.0.4

Thank you @graschik !


Installing by pip

When I install subliminal by pip (pip install subliminal) version 0.7.4 is installed. I guess because version 0.8 is still not in stable release?
When am I able to install v 0.8 by pip?

Try latest stable 0.7.5


Update Mongo Java driver compile dependency

The Mongo Java driver used by https://mvnrepository.com/artifact/uk.co.panaxiom/play-jongo_2.12/2.1.0-jongo1.3 is https://mvnrepository.com/artifact/org.mongodb/mongo-java-driver/3.4.2. However, I have to use at least MongoDB 3.6, if not 4.0.
Looking at https://docs.mongodb.com/ecosystem/drivers/driver-compatibility-reference/#java-driver-compatibility, it seems I need a version of Play-jongo that compiles using at least Mongo Java driver 3.6, if not 3.8.
Are there plans for supporting higher versions of Mongo?

If you can get to Play 2.7.3, #74 should do the trick.

@arambau Is there a plan to push that PR? With that play-jongo would have a version compatible with Play 2.7, right?

@intelease My apologies, I never got a notification that you made a comment here. I don't have access to the main repository here, but if you clone mine, you can build the module and it will work with play 2.7. In fact, I'm actively using it in Play 2.8 as well.


Memory corruption when doing DB operations

Hello!,
We are trying to narrow out a memory corruption: "AccessViolationException" or "ExecutionEngineExceptions" in other computers.
We believe the problem is similar to the following:
https://github.com/ericsink/SQLitePCL.raw/issues/321
We first suspected of a threading issue while we were stress testing the issue to find a repro, but even with a global mutex the problem occurred, so after reading issue #321 we tried stopping the GC while in DB operations and that brought down the problem from always crashing during stress to never crashing again.
This is one of the crashes we get

... but we got others as well ... where the stack is busted, so hard to say.
Which other information can help to track down this problem?
We are not familiar with the code so not sure if we can pinpoint the problem directly.

What version are you using?  If this is #321, it was fixed in 2.0.4.

Hi Eric, we are getting this error in 2.0.4

#321 merge seems to be mentioning that it fixes the particular instance, but that it would be possible of other situations where a dangling pointer (or an uncounted reference to an object) make it out of a protected scope leading to the garbage collector to cause mayhem.
Given that the repro seems to be strongly tied to GC and the crashes materialize when trying to string a utf8 pointer, we have strong reasons to suspect that there might be other instances of the #321 bug. Unfortunately we are not smart enough to figure out in the codebase where it can be.

It is crashing here:
https://github.com/ericsink/SQLitePCL.raw/blob/fcddb4b28f0c9981f0617d4f9ae7e8acb58c0b82/src/SQLitePCLRaw.core/utf8z.cs#L172
Which indicates that the content of the sp member variable is dangling.
Looking on how this read only structure is built I'm unsure how the memory is protected, in multiple places:
How is the life range of the memory protected in this function?
https://github.com/ericsink/SQLitePCL.raw/blob/fcddb4b28f0c9981f0617d4f9ae7e8acb58c0b82/src/SQLitePCLRaw.core/utf8z.cs#L149-L159

I am looking more closely at this now.  I haven't worked on this issue in a while, so the details need to be swapped back into my brain's RAM.  :-)

"How is the life range of the memory protected in this function?"
Apparently it is not.  :-(
There are fixes I need to make here.  I seem to have gotten too fanatical about not copying strings.  Thanks for the investigative work.  I'll followup here when I make the fixes.

Cool, we are trying to build and debug to narrow down the problem.
We are sorry for not being of more help :(.

That's a very strange error.  Tell me again which version of .NET are you on?

We are on 5.0.2
By making the code to copy the strings where it comes from the native code the crashes are reduced a lot, but the app still crashes after an hour or so.
sqlite strings live range seems to be tied to the next Next() function call, I wonder if some enumeration build is going in parallel or something like that is affecting the memory.
Also I have read around that if you call GC.Collect too often it can triggers the exception, so not sure if some of the crashes we experience when stressing the app may not be the same crashes when we do not use the GC.Collect to accelerate the crash.
Unfortunately, when we see the crash the stack is already corrupted and not sure it happened even close to where the crash is pointed out.

One confusing thing here is that the docs say that exception is obsolete and no longer used:
https://docs.microsoft.com/en-us/dotnet/api/system.executionengineexception?view=net-5.0

We are puzzled as well.

We now believe that problem was not related to this code, SQLitePCL was just a victim of a memory corruption at a lower level in Dot Net 5. Sorry for the misreporting.
https://github.com/dotnet/runtime/issues/51350


refactor: improvements

Description
This PR ended up being a little more than I had intended. In short, it:

switches to vitest
adds support for shouldPrefetch
moves publicPath to a function (buildAssetsURL) that is responsible for returning the public path of an asset - currently on rendererContext but it could easily be moved to ssrContext if we want per-request capability
normalises a 'parsed resource' type that is cached on rendererContext
removes unused _dynamicEntrypoints cache
does not prefetch prefetches of dynamic dependencies
does not also add preload tags for styles that should be rendered as well
adds more attributes to prefetch links
adds support for rendering preload/prefetch to headers (renderResourceHeaders)

Closes https://github.com/nuxt-contrib/vue-bundle-renderer/pull/28, resolves https://github.com/nuxt-contrib/vue-bundle-renderer/issues/20, resolves https://github.com/nuxt-contrib/vue-bundle-renderer/issues/24.
Breaking changes

shouldPrefetch and shouldPreload now have a different function type: (resource: ParsedResource) => boolean

If we need to keep this unbreaking for now I can add new functions with the new signature, and keep the old one for now...
Recommended next steps

Remove bundle-runner dependency and integration.
Do not normalise manifest by default (instead users are recommended to normalise it themselves with normalizeClientManifest)
remove clientManifest and publicPath options from createRenderer

Hoping this PR goes out soon! 🤞🤞🤞 @danielroe  @pi0 i'll buy you both a beer 😝

Hi @danielroe @pi0, is there any plan to merge this PR on future? I'm facing the issue with prefetch https://github.com/nuxt/framework/issues/5260. Hope this PR can fix our problem 🌟

Is there any movement for this? We'd love to see that sooner - for our APP see all chunks preloaded is really a blocker.

@danielroe is working on final rewrite items to release as a major version 🚀


Saved tab set syncs only on chrome restart

I've been using Tabli on two machines which share a chrome profile, in part as a replacement for the now defunct TabRocket extension.   TabRocket's primary (really only) feature was to let you shuttle tabs back and forth between any number of machines, basically using the same technique Tabli uses to store window sets (Chrome's bookmark folders).   Each machine used a subfolder, and any new bookmarks that went into it would be loaded on the machine in question next time a sync happened, and then bookmark removed.
In Tabli, if I create a new window and save some tabs on one machine, chrome will happily sync the window in the Tabli Saved Windows bookmark folder, but when I open the Tabli window on the second machine, it does not show the new window.   Restarting Chrome causes the window to show up in both the pop up as expected.
Is there a way we could get a refresh button or, better, add a listener on the bookmark syncing using  chrome.bookmarks.onCreated  on the Tabli folder to detect when a sync has happened and auto-refresh the list of available windows?

This is a really important feature for me too. My main use case is switching sets of tabs/windows from my desktop to my laptop, and lack of this feature makes it quite cumbersome.

Same here, this feature/fix would improve the usefulness of this great addon.

I just pushed a new release of Tabli (0.9.9, you should be automatically updated) that should support this. It works by listening for updates to the bookmarks and bookmark folders that Tabli uses to store saved state. Once you're running the 0.9.9 version please add a comment here to let me know if this is working for you.

It updated already and I noticed it even before you commented here ;-) great, thanks for the fix.
Seems to work so far although I only tested it once by opening the laptop .... I will check things for a few days and report back.

Looks good to me so far. thanks again.

Perfect.  Now I can finally retire my hand-edited copy of TabRocket.

Excellent! Thank you all for the feature suggestion and your patience. Closing.


Implement new "dsl_gie_primary_segvisual_add/remove" services

/**
 * @brief Adds a named Segmentation Visualizer  to the named Primary GIE.
 * A Primary GIE can have at most one Segmentation Visualizer
 * Segmentation Visualization requires a corresponding configuration file, Industrial or Semantic.
 * @param[in] name unique name of the Primary GIE to update
 * @param[in] segvisual name of the Segmentation Visualizer to add
 * @return DSL_RESULT_SUCCESS on successful add, DSL_RESULT_GIE_RESULT otherwise
 */
DslReturnType dsl_gie_primary_segvisual_add(const wchar_t* name,
    const wchar_t*);

/**
 * @brief Removes a Segmentation Visualizer, previously added with dsl_gie_primary_segvisual_add,
 *  from the named Primary GIE.
 * @param[in] name unique name of the Primary GIE to update
 * @return DSL_RESULT_SUCCESS on successful update, DSL_RESULT_GIE_RESULT otherwise
 */
DslReturnType dsl_gie_primary_segvisual_remove(const wchar_t* name);

Going to implement the Segmentation Visualizer as a Component (Bintr) with it's own bin.  This means two extra ghost pads in the Pipeline for just the one element, but it greatly simplifies the implementation.


[CI] Refactor Jenkins CI pipeline + migrate all Linux tests to Jenkins

See #4234 for details.

Note that builds (compilation) are significantly faster because

They use C5 instances with faster CPU cores;
Build environment setup is cached using Docker containers, so we don't need to run package installation gazillion times

@CodingCat Can you look at this? In particular, I'd like to seek your feedback to the cross-version checks, where XGBoost4J JARs are tested against JDK 8, 11, and 12.

LGTM for jvm-packages part

@CodingCat FYI, this PR is adding a dummy project named xgboost4j-tester under jvm-packages. This is needed so that I can run code from xgboost4j-example

@CodingCat FYI, this PR is adding a dummy project named xgboost4j-tester under jvm-packages. This is needed so that I can run code from xgboost4j-example

that's fine, we can use it for more integration test in future

TODO: write a short doc explaining what tests/ci_build/ci_build.sh does

@trivialfis I found the fix for leak detector: google/sanitizers#764. I need to add PTRACE capability to docker runs.

Re-basing against the latest master to fix Windows tests

Running two Spark jobs in a row resulted in #4406, so I'm disabling the second Spark integration test for now

Will try to fix Mac CMake build issue in a follow-up PR

Merged. Thanks everyone!


Add Getting Started section to README

README Update
Please add a getting started section to the README so that folks know how to start with this project.

Added Getting Started section in README file.


Added spge-bg.txt

Sofia Vocational High School of Electronics "John Atanasoff", Bulgaria. We would love to have the chance to participate as well. =)

This appear to be a secondary school. This gem is for post-secondary schools ;)


Import Swagger generator

Problem: we need to apply the generators in a specific order so that the jsonParse methods come out first! Solution is to allow opting out of specific plugins per file.

Don't merge until I've changed from comma- to slash-delimited list in suppression.

Comma-delimited is fine now that we're excluding per plugin rather than per assembly.


How to delay firing of convo.say after the response from an API call

My convo ends prematurely due to an API call that takes too much time. What is the recommended way to queue the event convo.say('something') but delay its firing, so that the conversation does not end?

@devhaoy do not call convo.next() until your asynchronous request has completed.

Hi @benbrown  one issue with doing the heavy work in the response handler, before convo.next(), is that nothing will fire at all before all the work is processed. In my case I ask a user to send an image and then call an API: the whole process lasts 15 secs so i'd smooth it by sending a message to the user once the bot receives the image but before it calls the API ==> impossible if the API is called before convo.next(), in this case convo.say('I got your image')  won't fire until the API call goes through.
This is why I'm calling the API in the subsequent thread and before convo.ask('?' , function(response, convo) { . By doing this though, the conversation ends prematurely except if I artificially add 5 convo.say('something') to queue actions until the API call goes through.
Hence my problem: how can I queue a convo.say but not fire it until the API call is processed?

@devhaoy Aha, yes, this is tricky.
I am actually in the process of adding an additional hook for conversations so that you can trigger actions like this in between threads.  This would allow you to complete your thread 'I got your image' and then call gotoThread('completed'), but do the API call before the completed thread begins.
I am hoping to get this into the next version of Botkit!

@devhaoy take a look at this pull request -> https://github.com/howdyai/botkit/pull/754

Hi @benbrown ! excuse my noob, but could you add an example for beforeThread function? Do I have to use it along with gotoThread or does it work as standalone?
In my case I am calling the next thread as a simple function nextThread(response, convo) then convo.next(). Where do I include beforeThread in there?
Anyway, thanks for releasing a solution so quickly!

@devhaoy you can find it in the docs

@peterswimm that fix will work with threaded conversation only, i'm looking for anything that will produce the same result but within the default thread. Should repoen?

Could you explain why that is preferable behavior?

Hi @peterswimm  it's explained in my second message above. Basically I prompt user to send a picture then process it with several API, which last 15+ seconds in total. Sometime the upload of the pics itself will last 5-7 seconds if the user's connection is poor. I want to send a simple "Got your picture, please hold tight" message to the user once the upload is completed, so that he knows he should wait a bit (a not send another message for example, which could break the flow).
In order to do so I need to call the APIs after calling convo.next() but then the conversation ends prematurely. This is why I'm looking for any solution that will allow me to prevent the conversation from ending when I call the API beforea convo.ask

I have exactly the same problem.

Same here. I think we need something like convo.tick(), which will send the queued messages that have been added with convo.say, but won't move us to the next thread yet.

@TheDApact Maybe I did not understand at the end but, could you use transitionTo('Got your picture, please hold tight','api_succes') and beforeThread('api_succes', (convo,next) => {//api request}) ?


Self-preparing inner closures for second order

Codecov Report
Attention: Patch coverage is 17.24138% with 24 lines in your changes are missing coverage. Please review.

Project coverage is 5.31%. Comparing base (f80aab9) to head (59bf35e).

Files
Patch %
Lines

DifferentiationInterface/src/second_order/hvp.jl
20.00%
16 Missing :warning:

...ionInterface/src/second_order/second_derivative.jl
0.00%
8 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #291       +/-   ##
==========================================
- Coverage   96.66%   5.31%   -91.35%
==========================================
  Files          72      56       -16
  Lines        3812    1974     -1838
==========================================
- Hits         3685     105     -3580
- Misses        127    1869     +1742

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

Closing this because it violates a fundamental assumption of preparation: the function should not change its internal state


set rvmrc attributes for specific users

I've not been able to find a way in my json meta-data to get rvm-chef to set the some of the rvm settings:
I'd like to be able to set these for the deploy user.
rvm_project_rvmrc
rvm_gemset_create_on_use_flag
rvm_trust_rvmrcs_flag
Here's the full json meta-data file: https://github.com/concord-consortium/littlechef-servers/blob/ruby-lab-server/roles/lab-server.json
I'm specifying: "recipe[rvm::system]" in the json.
And here's just the rvm section:
"rvm": {
  "default_ruby": "ruby-1.9.3-p194",
  "rvmc": {
    "rvm_project_rvmrc":             1,
    "rvm_gemset_create_on_use_flag": 1,
    "rvm_trust_rvmrcs_flag":         1,
    "group_users": [ "deploy" ]
  },
  "user_installs": [
    {
      "user": "deploy",
      "rvm_project_rvmrc":             1,
      "rvm_gemset_create_on_use_flag": 1,
      "rvm_trust_rvmrcs_flag":         1
    }
  ]
},

Under the user_installs section I was hoping to be able to set those rvm variables
I AM seeing some variable written into ~/.rvmrc for the deploy user:
$ cat  /home/deploy/.rvmrc

# rvm configuration
#
# Generated by Chef for ip-10-32-174-243.ec2.internal
# Local modifications will be overwritten.
#
export rvm_path="/home/deploy/.rvm"

export rvm_gem_options="--no-rdoc --no-ri"

Is this a bug, or am I misunderstanding something, or is this feature not implemented?
Thanks

@stepheneb Can I close this issue?

Sure, I can't confirm it;s fixed but if I find I can still reproduce it later I'll re-open it. Thanks

Thanks @stepheneb


Test the jobs using japa or any other test runner.

Hey there,
I am here again ;).
How do I test the jobs using japa or any other test runner.

perform method could return a value to assert.
Also, you can fake & mock dependency serivces by app.container.swap.
This document might be useful: https://docs.adonisjs.com/guides/testing/mocks-and-fakes

Okay I was thinking that if we can manually call the perform  function in a test file and we can either assert or just check if the function ran successfully without any error.

Okay I was thinking that if we can manually call the perform function in a test file and we can either assert or just check if the function ran successfully without any error.

Sure, we can. Just be careful only if you enqueue another job in a job.
Maybe we should provide some mock or fake method in that case.


Add transaction's missing attributes

Missing attributes in transaction object:

authorized_at
captured_at
voided_at

They can be found using processout's REST API but not through Ruby's SDK.
This PR to add them.

Hello Processout,
i especially need 'authorized_at' in my code.
Thanks

Handled in #6
Will be published shortly, thanks!

Thx a lot :)


fix: error handling for tput

Hey @rb-determined-ai The error message about conda I jammed in there might be excessive, but we should at least fail with some more info when there is tput error.

Would you mind running make fmt? We don't have ci set up yet but if we did make check would fail.


added group norms

Hey Team, I have added the group norms into the collaboration Readme

After further research and team mate's guidance, I have opened the pull request in the right repository - my group project repository. So I will be closing the pull request here.


Sync

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 81.13208% with 20 lines in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

src/main/java/seedu/address/MainApp.java
0.00%
12 Missing :warning:

.../java/seedu/address/model/util/SampleDataUtil.java
0.00%
5 Missing :warning:

...rc/main/java/seedu/address/model/ModelManager.java
88.23%
2 Missing :warning:

...u/address/logic/commands/DeleteMeetingCommand.java
95.83%
0 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines
Coverage Δ
Complexity Δ

src/main/java/seedu/address/logic/Messages.java
95.12% <100.00%> (+0.83%)
7.00 <1.00> (+1.00)

.../seedu/address/logic/parser/AddressBookParser.java
100.00% <100.00%> (ø)
20.00 <0.00> (+1.00)

...ain/java/seedu/address/logic/parser/CliSyntax.java
91.66% <100.00%> (+1.66%)
1.00 <0.00> (ø)

...dress/logic/parser/DeleteMeetingCommandParser.java
100.00% <100.00%> (ø)
5.00 <5.00> (?)

...in/java/seedu/address/logic/parser/ParserUtil.java
98.68% <100.00%> (+0.19%)
30.00 <4.00> (+4.00)

src/main/java/seedu/address/model/Model.java
100.00% <100.00%> (ø)
2.00 <0.00> (ø)

src/main/java/seedu/address/model/UserPrefs.java
93.33% <100.00%> (+8.33%)
18.00 <2.00> (+4.00)

...ain/java/seedu/address/storage/StorageManager.java
97.91% <100.00%> (+0.54%)
24.00 <6.00> (+5.00)

...u/address/logic/commands/DeleteMeetingCommand.java
95.83% <95.83%> (ø)
12.00 <12.00> (?)

...rc/main/java/seedu/address/model/ModelManager.java
88.33% <88.23%> (-0.24%)
45.00 <5.00> (+4.00)
:arrow_down:

... and 2 more

... and 2 files with indirect coverage changes


Fix for invalid SSL certificate

added "verify=True" parameter in order to test sites with an invalid SSL certificate

requests.packages.urllib3.disable_warnings() this also solution

but first of all thanks bro


Facebook Pages DM

Expected Behavior

Current Behavior

Screenshots (if appropriate):
Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Franz Version used:
Operating System and version:

I cannot get the Facebook Pages DM to work?  When I log in it just takes me to my facebook timeline, rather than heading to the actual messages section of my page?


Update index.yml

removed the "*.md" since that caused 404-page not found in URLs.
like overview: text: Windows 10 on ARM , url: /windows/uwp/porting/apps-on-arm.md

Sorry, first time I try here on GitHub about documentation.
Seems like there are some error on new landing page: https://docs.microsoft.com/en-us/windows/arm/
The links ain't working correctly and needs to be fixed.
Overview: https://docs.microsoft.com/en-us/windows/uwp/porting/apps-on-arm.md
gives 404 error (seems like you are starting to migrate to a new landingpage yesterday tho.. )
but:
https://docs.microsoft.com/en-us/windows/uwp/porting/apps-on-arm
, works.
Don't know how to correctly contribute about this 404-error.
But I hope soon that you guys can have a good working Raspberry Pi4 Windows on ARM, easy  working. :)
Want to have a good working check in device that just runs one Edge webpage that are always logged in .

Email sent 3/24

We verified the links in a test PR in the private repo. Looks good; merging.


(housekeeping) Upgraded Xamarin.Forms to v3.5

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Updates Xamarin.Forms to version 3.5.x
What is the current behavior? (You can also link to an open issue here)
ReactiveUI targets Xamarin.Forms version 3.4.x
What is the new behavior (if this is a feature change)?
ReactiveUI targets Xamarin.Forms version 3.5.x
What might this PR break?
Events?
Please check if the PR fulfills these requirements

[X] Tests for the changes have been added (for bug fixes / features)
[X] Docs have been added / updated (for bug fixes / features)

How about in the event generator?

When I built locally the Tizen event generator failed.  But that is semi normal behavior on my windows box.  So I am letting the build server dictate if there is an issue generating the events.
I remember we had the question of if Xamarin.Forms 3.5 broke events.


Halt privilege dysfunctional

When granting the halt (and/or reboot) privilege to an otherwise unprivileged user, the power menu is not shown. When granting the halt privilege, it also breaks the login process. Steps to reproduce:

Create a new group
Edit group privileges to include "Diagnostics: Halt System"
Create a new user
Add user to the group
Log in with user

Expected result: user is logged in and able to halt the system
Actual result: user is redirected to https://ui/core/halt. When trying to access the page directly (https://<IP_ADDRESS>/ui/core/halt), the user is redirected, presumably because the permission is not granted and the system then tries to redirect the user to their default page.
If you additionally grant any other permissions, you can access those pages by navigating directly to the correct URL. In the menu, one can see that the Power menu is not shown. Presumably, this permission is completely broken and not only redirects incorrectly.
The reboot permission simply does nothing, but does not break the login/main page redirect.

@bleetsheep thanks, https://github.com/opnsense/core/commit/7ee1110d0f0e58a4d18246dbcbba276065243af1 should fix it.

When would this make it into a release, i.e. that it gets pulled in the update system for production?

you can use opnsense-patch 7ee1110 in the meantime, 19.1.2 will likely include the patch (within a few weeks)


object_recognition_ros_visualization: 0.3.6-0 in 'jade/distribution.yaml' [bloom]

Increasing version of package(s) in repository object_recognition_ros_visualization to 0.3.6-0:

upstream repository: https://github.com/wg-perception/object_recognition_ros_visualization.git
release repository: https://github.com/ros-gbp/object_recognition_ros_visualization-release.git
distro file: jade/distribution.yaml
bloom version: 0.5.20
previous version for package: null

object_recognition_ros_visualization
* Merge pull request #3 <https://github.com/wg-perception/object_recognition_ros_visualization/issues/3> from v4hn/colorful-tables
  Add color property to table display
* Add color property to table display
  Now tables can have different colors.
  This is helpful if you get tables from different sources...
* Merge pull request #2 <https://github.com/wg-perception/object_recognition_ros_visualization/issues/2> from v4hn/moc-boost-1.57
  Fix build with qt4's moc & boost 1.57
* Fix build with qt4's moc & boost 1.57
  This is a common workaround to make sure moc doesn't see
  preprocessorvariables it doesn't like in boost...
* Contributors: Michael Görner, Vincent Rabaud

I think this failed because you started the release before merging the previous one... You'll have to redo with -p.


[Discussion] High CPU load

I'm running Spleeter using spleeter-gpuand it causes extremely high CPU load when running 3 instances of Spleeter at once.
Would anyone care to explain why this is happening?
I'm sure it's not a bug or anything, I'm just curious as to why it's happening.

Definitively not a bug. Spleeter is a high computation process, and the GPU version will only performs Tensorflow GPU related computation into the GPU device, other operation are performed into CPU (such as audio loading, processing, model loading, etc, etc, etc), it also use multiprocessing for track writing to disk. No surprise it is burning a CPU is you run three instance of it.

Makes sense. Thanks for clarifying.
So basically, even though the GPU might be used, it's not necessarily where the actual splitting occurs, hence the high CPU load?

I see it max out my CPU just using one instance.  Granted its only for a few seconds.  But it takes out the audio while I'm livestreaming.
Windows 10
Processor	Intel(R) Core(TM) i7-10700F CPU @ 2.90GHz, 2904 Mhz, 8 Core(s), 16 Logical Processor(s)
GeForce RTX 3060 12GB RAM


Shader editor opening when clicking on node

Tested versions
4.4 dev5 and all godot 4 versions
System information
Shader editor opening when clicking on node
Issue description
Not sure if this is a bug or it's supposed to happen, but it can be annoying sometimes. When clicking on a mesh that has a shaderMaterial applied to it, and the material slot opened, clicking on that mesh will open the shader editor directly. This is unnecessary and is quite annoying if you're switching back and forth between editing shaders and working in the viewport.
video:
https://github.com/user-attachments/assets/7bdbedb5-ce51-40b9-85bb-d774e1fc062e
Steps to reproduce
Assign a shader material to any mesh
Minimal reproduction project (MRP)
any new project

Probably the same as #99772
Try to fold your shader property and see if stops:

Probably the same as #99772
Try to fold your shader property and see if stops:

Yes I knew that before. Is this intentional? There should be an option to turn off this behavior

This is a regression, after unfocus the node that should auto-fold but that's broken right now.

after unfocus the node that should auto-fold but that's broken right now.

It's not that simple. Making it fold brings back #81481
Relevant code:
https://github.com/godotengine/godot/blob/0f95e9f8e665cedaa5aace5f6f86de2b4b5a8c60/editor/editor_properties.cpp#L3387-L3393


Is there a way to NOT persist the occurrences? and just somewhat generate them based on a date range instead?

Trying to find a way to build out a calendar UI that isn't constantly creating records just so it can show them to the user.  Or am I missing something?  Or does it only persist them if you want them persisted?
Sorry!

Persisting occurrences is intended to come on top of the creation of the schedule. You can access all calculated occurrences as described in "Accessing IceCube" without the need of persisting stuff, but then have to take care of what to do with them by yourself. Meaning that currently you won't be able to list them up like you're used to do with database records.
This is something i always wanted to improve on. Ideally you could deal with the occurrences as if they were persisted. That would make everything far more easy to maintain and probably also to adapt to certain use cases. But i didn't manage to do it so far and - i'm so sorry - currently i'm so f*****g busy and it's really a pity that i cannot take care of this gem now. Hopefully this is going to change very soon.
Sorry! ;-)

Duuuuude don’t you worry about it! Thank you so much for your response.
I have this working without persisting but it’s sadly without your gem. Would love to pull it down for a bit and see if I could get it working with the gem but I’m kind of in the same boat as you of course!
Keep doing what you do!
Cheers mate!
On Mar 29, 2018, 5:38 AM -0400, Rafael Nowrotek<EMAIL_ADDRESS>wrote:

Persisting occurrences is intended to come on top of the creation of the schedule. You can access all calculated occurrences as described in "Accessing IceCube" without the need of persisting stuff, but then have to take care of what to do with them by yourself. Meaning that currently you won't be able to list them up like you're used to do with database records.
This is something i always wanted to improve on. Ideally you could deal with the occurrences as if they were persisted. That would make everything far more easy to maintain and probably also to adapt to certain use cases. But i didn't manage to do it so far and - i'm so sorry - currently i'm so f*****g busy and it's really a pity that i cannot take care of this gem now. Hopefully this is going to change very soon.
Sorry! ;-)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


How to get this project running

I express my gratitude for making this incredible AI project available to everyone. It's an outstanding contribution that benefits us all.
I'm highly interested in the project and would like to know how to get it up and running. Could you provide me with instructions on how to run the project?

The projet is already installed in my visual studio code workspace


Error fix for terminal based challenges

Fixes #225
Removing the use of the useMediaQuery hook from the terminalChallenge component.

tACK!

I have removed the https://github.com/saving-satoshi/saving-satoshi/blob/master/hooks/useMediaQuery.ts file as it is not being utilized and react-responsive dependency.


Follow up on seeing if running FTL tests is possible with our emulators

We are seeing a lot of flakes with the Firebase Test Lab emulator tests and it would be worthwhile to determine if we can support these tests natively within our own infra.
See: https://ci.chromium.org/ui/p/flutter/builders/try/Linux firebase_android_embedding_v2_smoke_test/62119/overview
There are currently two open issues that track temp disabling of flaky tests on two different API versions:
https://github.com/flutter/flutter/issues/123331
https://github.com/flutter/flutter/issues/135784

cc @reidbaker since we were discussing moving more emulator tests back to virtual devices in Firebase Test Lab, and how that's also flaky.


REST: Add Authorization to Vary header field when appropriate

When the resource defines is_authorized/2 we do not typically want to allow caching to be shared between users with different authorization header values. Therefore it may make sense to add "Authorization" to the "Vary" header by default.

I'm not sure this is necessary. rfc7234 section 3 says in part:
   A cache MUST NOT store a response to any request, unless:

   o  the Authorization header field (see Section 4.2 of [RFC7235]) does
      not appear in the request, if the cache is shared, unless the
      response explicitly allows it (see Section 3.2), and

It looks like the only way to possibly cache a response to an Authorization header is to use one of "public", "s-maxage", or "must-revalidate" for Cache-Control headers.
So Vary: Authorization wouldn't hurt, but it also seems like it isn't needed by default.

Yes you are of course correct. I saw this a few weeks after initially opening this ticket. There are other mentions about Authorization not being necessary in Vary in that RFC. I will close this. Thanks!

Great. I realized that section 7.1.4 is actually explicit on this point:

there is no need to send the Authorization field name in Vary because reuse across users is constrained by the field definition


⚠ ClamAV has degraded performance

In be01ed6, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 71 ms

Resolved: ClamAV performance has improved in 40e9319 after 12 minutes.


Swish payment method always triggers an error on mobile

Describe the bug
When an error occurs in drop-in, our Pronamic Pay plugin for WordPress redirects to a failed payment page. The payment status then gets updated accordingly and the error is logged.

https://github.com/pronamic/wp-pronamic-pay-adyen/blob/4.1.0/js/src/checkout-drop-in.js#L19-L38

For the Swish payment method, this seems to result in an unreasonable amount of failed payments (60% according to client) with the following error message:
swish App was not found

This error seems to be triggered by redirectToApp() of the QR loader:
https://github.com/Adyen/adyen-web/blob/68e1573e39ff925f3cbcb13799816269ea6c57a8/packages/lib/src/components/internal/QRLoader/QRLoader.tsx#L70-L77
We're thinking the really short timeout of 25 ms might be the culprit. In a test similar to redirectToApp(), it seems that even the simplest redirect will trigger the error, even though the redirect works perfectly fine:
<script>
setTimeout( function() {
	console.log( "Unable to redirect" );
}, 25 );

window.location.assign( 'https://www.adyen.com/' );
</script>

In the case of Swish, the URL being redirected to uses the custom protocol swish:// to open their app. This means that the browser might also asks for confirmation before opening the Swish app, which will take a little longer to confirm. It is impossible to confirm that this fast.
To Reproduce
Steps to reproduce the behavior:

on mobile, select Swish in drop-in
click the button to continue payment
the onError method of the checkout configuration is called with a AdyenCheckoutError

Expected behavior
The drop-in should not trigger an error if the Swish app can be opened. Also, if we would want to leave the QR code shown on mobile when the app is not installed (for example to use an other device to scan the QR code), there is no way to distinguish between a real error (like when the payment expires after 3 minutes in https://github.com/Adyen/adyen-web/blob/v5.23.0/packages/lib/src/components/internal/QRLoader/QRLoader.tsx#L94-L98) and the case that the app could not be opened. If the PayPal popup is cancelled for example, we do not redirect to our failed payment page:

https://github.com/pronamic/wp-pronamic-pay-adyen/blob/4.1.0/js/src/checkout-drop-in.js#L28-L30

Smartphone (please complete the following information):
I have tested on at least the following device, but assume it affects others too.

Device: iPhone
OS: iOS
Browser: Safari
Version: 15.6

@sponglord have you been able to check this issue?

I alerted Adyen to this issue at https://www.adyen.help/hc/en-us/requests/new?ticket_form_id=360000690879.

Hello Adyen team,
Can someone look at https://github.com/Adyen/adyen-web/issues/1712 and comment on that? Thanks in advance.
Remco Tolsma
Pronamic

Your request is logged with Ticket ID #3308212 and will be reviewed by our support staff. We typically respond to tickets within 24 business hours, however one of our agents will try to get back to you as soon as possible.

Hopefully we can send our joint customer an update on this issue soon.

Hey @rvdsteege ,
Thanks for the detailed description. We agree that indeed 25 ms seems too little for the mobile device to properly open the app, and unfortunately there is no exact science about the duration of the timeout to wait for the app to kicks in.
I created a fix increasing the timeout to 1 second, which should hopefully cover all the cases for different devices/operational systems (Android/iOS). This fix should be released early next week.
I will leave this Github issue open for now, so we can follow up on it and you can let us know if that fixed your problem!
Cheers

Thank you for the update, @ribeiroguilherme. However, I don't think slightly increasing the timeout in #1755 will make much difference: it will still trigger the checkout error when it shouldn't.
I made a quick screen recording to illustrate the issue. For demonstration purposes, the page in the video turns red when the error would normally be triggered (after 1s). Note that the error gets triggered even though I'm actually opening the Swish app — as requested by drop-in.
https://user-images.githubusercontent.com/10371164/191741678-3441794d-65c1-471f-a78c-ba509fe08871.mov

Based on https://stackoverflow.com/questions/24779312/simplest-cross-browser-check-if-protocol-handler-is-registered you may also have to conclude that there is no reliable way to determine whether an app could be found / opened.

Note that as at February 2020, there doesn't seem to be a working way to do this. Only considering Chrome, neither the solution from @kororo or @habsq work at the moment. The reason is that what we're trying to achieve by checking if a URL protocol handler exists is actually considered as being a security risk by some people.
So my advice is to not persevere in trying to get this to work.

https://www.fortinet.com/blog/threat-research/leaking-browser-url-protocol-handlers
It therefore does not seem correct to me to trigger this error after a certain time:
https://github.com/Adyen/adyen-web/blob/ff4c8a2ca08e4b87302f813bb07be8211df88ca0/packages/lib/src/components/internal/QRLoader/QRLoader.tsx#L70-L77
I don't think you can ever know for sure if the app was indeed not found. This whole implementation needs to be different/better?

Thanks for the input guys.
I spent some time investigating this issue and indeed seems like we can't know for sure if the app is available or not in the device. We need to rethink a bit about this current implementation, but it seems that triggering onError isn't ideal for this use case. I will align internally with the team and we will discuss how to move forward.
In the meantime, I suggest to ignore the error on yours onError callback implementation if the error message is swish App was not found - that should do the trick for now. I will keep this thread updated with the progress of this problem.

Hey all 👋
We also experience the Swish issues, and I have made a Pull Request that might solve it. Got any feedback? #1764

We've just deployed our app using the latest version and I can confirm that the Swish-trigger-error problem is solved 😄

Thanks for raising this issue guys. The fix is available on the 5.27.0 .

Thank you @DavidVujic and @ribeiroguilherme  — we'll update the Adyen integration in our plugin!


Updates SVG timeline:

Simplifies parent list;
Adds nested lists;
Adds aria-level to the heading;
Uses aria-hidden to hide visual content from screen readers;
Switches from aria-labelledby to aria-label on listitems (because screen readers need labels with more context).

Note: aria-labelledby was not working previously because the elements it
was applied to did not have explicit roles set.

Thank you!


List users of Tenants - should also include child tenant users with an option to choose filter them out

I'm submitting a ...

[ ] bug report
[ ] new feature / enhancement request
[X] improvement request (usability, performance, tech debt, etc.)
[ ] other

Traffic Control components affected ...

[ ] CDN in a Box
[ ] Documentation
[ ] Grove
[ ] Traffic Control Client
[ ] Traffic Monitor
[ ] Traffic Ops
[ ] Traffic Ops ORT
[X] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] unknown

Current behavior:

User Admin -> Tenants -> Users -> lists only users explicitly assigned to that tenant. doesn't include the users of child tenant.
Expected / new behavior:

The users of child tenant should be displayed
There should be an option to include child tenant users, check on - users of child tenant having current tenant as parent should be displayed along with current tenant users, check off - only user of current tenant should be displayed.
Minimal reproduction of the problem with instructions:

Anything else:

Assuming the tenant tree looks like:

root
-- tenant 1
--- tenant 1.1

I think it would be a good option to either:
a. see all the users explicitly assigned to tenant 1 (as it does now) or
b. have to option to see all users in the scope of tenant 1 (which would include tenant 1 and tenant 1.1 users)

Yes. That's exactly why this ticket was created.


Fix #2442 - should_retry_throttle isn't defined

Short Description:
Variables weren't correctly defined
Fixes:

#2442

Culprit:

#2407

Don't blame #2407 -- I only renamed your variable from #2357! :tongue:
But seriously, good catch.  Can't believe we both missed that. :-|

Why even bother having those variables? Makes the code more confusing IMO...
Just put the contents of the 'if' blocks inside the except clauses, since they both just continue anyways

Note that #2407 also has a typo: unexpected_reponse_retry (note reponse vs response)

Hi, can you please fix Line 120 of that file, there is a typo, should be unexpected_response_retry, currently we have unexpected_reponse_retry

please fix typo and rebase, I will pull when ready

@JSchwerberg there was no errors before your PR :)
The typo is fixed


关于训练tile_size的问题~

Excellent work！I have noticed that the tile_size used for model inference is 16 frames. The curious point is that the previous Mimic-Motion v1.1 model was able to infer at a tile_size of 72. I noticed in your paper that four 80G A100 were used for training, and it was also based on SVD. Do you have idea how to train/fine tune a model with tile_size of 72? Will your model consider supporting 72 frame inference in the future? Thank you!

Hi, training SVD requires significant computational resources, and our current GPU devices can only support training SVD with a 16-frame setting. I’ll work on optimizing the training pipeline to enable training with 72 frames in a single run in the future. Additionally, the training code and tutorial for StableAnimator will be released this Friday. Stay tuned!

Thanks~


Request: ability to "backup" a project

Continued from / related to https://github.com/zooniverse/Panoptes-Front-End/issues/1273
It would be great to have a way to export / import a text representation of a project and its workflows and subject sets (ignoring the actual subjects). That way people could "save" their project to their own computers.
Alternatively -- or in addition -- it would be nice to have a project versioning system so that people could (A) try alternative approaches in creating their projects; and (B) revert to an earlier draft of the project in case something went terribly wrong (e.g. a collaborator deleted all the workflows by mistake).
Discussion warranted, as the former puts more responsibility on the end-user, but should be fairly easy to implement. The latter is more complicated and puts the responsibility for "backing-up" on Zooniverse, but is more sophisticated and perhaps more user-friendly.

I wonder if the system @aliburchard and @brian-c have cooked up for Serengeti-style projects could help here - a csv file containing Workflow data.

On 30 Jul 2015, at 20:34, Margaret Kosmala<EMAIL_ADDRESS>wrote:
Continued from / related to zooniverse/Panoptes-Front-End#1273 (comment) https://github.com/zooniverse/Panoptes-Front-End/issues/1273#issuecomment-126315168
It would be great to have a way to export / import a text representation of a project and its workflows and subject sets (ignoring the actual subjects). That way people could "save" their project to their own computers.
Alternatively -- or in addition -- it would be nice to have a project versioning system so that people could (A) try alternative approaches in creating their projects; and (B) revert to an earlier draft of the project in case something went terribly wrong (e.g. a collaborator deleted all the workflows by mistake).
Discussion warranted, as the former puts more responsibility on the end-user, but should be fairly easy to implement. The latter is more complicated and puts the responsibility for "backing-up" on Zooniverse, but is more sophisticated and perhaps more user-friendly.
—
Reply to this email directly or view it on GitHub https://github.com/zooniverse/Panoptes/issues/1225.

It should be easy to import/export data (it's all JSON anyway), but nearly impossible to debug errors if someone changes the file manually (and they will).

What's the status of this?
We've been working on some customized aggregations using zooniverse/aggregation as a starting point, and right now that code queries Panoptes directly to get workflows. It would be much better to just read them from an exported workflow csv, but I don't think we have access to one.

@vrooje I'll add it to the future plans.
Are you actually manipulating stuff from the workflow? Or just want to save a request against Panoptes?

I just want to save it - no need to change anything.

While I think this is a good idea in principle, I can see how this might be better dealt as an API client feature, without needing special backend support for it. Closing it here, if anyone wants this feel free to work on it in the scope of panoptes-python-client or similar.


Converting that to
<svg width="89" height="26" ...

resolves the problem.

1exseems to be around 8px.

Great job! :+1:
One question, though: Wouldn't it be better to convert ex to em (of course only if Android browser displays em correctly)?
Maybe I'm a little paranoid, but I'm a little bit uneasy about converting a relative unit to an absolute unit. I checked the new version in Chromium and it seemed to rescale properly, but maybe there are some corner cases out there.

Changing <svg width="89" height="26" ... to <svg width="89em" height="26em" ... makes no difference on Android stock browser. However, Chrome on Android scales the SVG with em to enormously huge size:

Using a factor of 0.5 and the em unit leads to the correct size on Chrome but fails again on the Android Browser.
The Android Browser just ignores the unit for width and height. I don't see any other solution than stripping the unit.

Thanks for the investigation!
Ok, I see. Well, that's unfortunate. I just feel that we're planting a potential issue for the sake of supporting 10% of users. :cry:


Update the position information within note of original link before pasting the Archived link

Current behavior

When activating the archiving process (that takes a while) and switching to another not, the Archived links get pasted into the other note
Also, when moving the link within the same note, the Archived link gets pasted at the location where the original link was initially

Workaround
Manually move the Archived links to the correct position
Proposed solution

When triggering archiving process, save the note, the process was triggered for
When Archived link is ready for pasting into the note, search the note again for the location of the original link and position the Archived link accordingly

@ahue thanks for the report, in hindsight this is definitely something I should've considered haha :)
For now I think I'll add a note about not switching away until it completes, because I think fixing this might necessitate a larger rework of how the plugin works (interacting with files, instead of interacting with the editor)
I'm also realizing that I have the same issue if someone edits the file while the plugin is working.


only half taskbar vissible

When i close the taskbar configurator app i can see only half my taskbar vissible pls fix this problem as fast as you can

To be able to focus on major/current bugs I had to close all issues. The issues section has gotten very messy to a point that I couldn't follow it anymore. Most issues are duplicates, very old or are already fixed/solved. Feel free to reopen this issue if you still have this issue on the latest version of TaskbarX to get my attention. All feature requests have been put in a list here: 💡| Feature requests #870
I hope you understand ❤


TS2503: Cannot find namespace 'moment'

TypeScript Version:  2.0.10
I am not sure if it is a moment problem or a typescript behavior/bug.
I'm just trying to use moment within a d.ts file. Moment ships with a d.ts file and everything works as soon as I import something from there. If I try though to define my own d.ts file containing definitions from moment. The compiler complains
Code
interface SomethingWithMoment {
    moment(): moment.Moment;
}

Expected behavior:
Have a successful build
Actual behavior:
I get an TS2503: Cannot find namespace 'moment'
Other information that may be helpful:
Example project: https://github.com/valotas/ts-moment
Surprisingly when running tsc with --traceResolution it looks like it finds the right file within node_modules.
I tried both /// <reference types="moment" /> and /// <reference file="path/to/moment" />

@valotas Did you import from moment? You need to import the types before using them - they aren't global to TypeScript.

I am having the same issue when using a declare statement: declare const moment: moment.Moment;. I do not want to import the library because it already exists on the window. Any help is greatly appreciated.

As @blakeembrey correctly points out, moment does not expose a global declaration. You must import it.
@patrickhousley if you only use moment in type position, i.e as as a type annotation, TypeScript will not include import statement in emitted JavaScript.
You want to use
import { Moment } from 'moment';

That said, the following is likely not what you want
declare const moment: Moment;

because Moment is not the type of the moment.

@blakeembrey I thought that this should not be the case within d.ts files. Did that change? As soon as import {Moment} from 'moment', it works. Should it be that way? If yes please close this and ignore me :)

Yes, that's correct. It's possible you used DefinitelyTyped in the past, where modules usually explicitly pollute the global namespace. That style is slowly fading away.

But then, I should be importing my types.d.ts everywhere as it will be considered a module right?

@valotas correct, though I've sent out https://github.com/moment/moment/pull/3688 to address this.

@valotas correct, though I've sent out https://github.com/moment/moment/pull/3688 to address this.


[BEAM-5996] Append query name to temp location.

Append query name to temp location to prevent multiple jobs from writing to the same object.

Follow this checklist to help us incorporate your contribution quickly and easily:

[X] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[X] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

run java precommit

This should be updated past @kennknowles change

run java precommit

@kennknowles yes I ran Dataflow nexmark post commit yesterday from a test PR using the new Nexmark triggering feature and as it was successful I closed the Dataflow failure ticket.


Feature: Music Randomization

I think it would be a neat option to have, if it's possible, whether it's just shuffling the existing music or allowing custom music.

I chose not to include any form of music randomization since it usually ends up being a recipe for disaster, you just get musics that don't fit the theme of the current area and sometimes musics that were meant to be heard for 30 seconds end up in places where you spend a large amount of time. I know other Castlevania randomizers do it but it's never given a good result. As for custom musics this mod only has one so far, the Dead Lands theme for the K2C area (instead of the vanilla underground waterways theme).

I feel like that's kind of the point, having music that doesn't fit the situation. As for music that's too short, you could specifically exclude those, or put them in a separate pool to be randomized amongst themselves.


Add naturalcasecmp_to function to String

Functions as a complement to naturalnocasecmp_to
Also cleaned up some comments in the comparison functions
It compares the same way as that function but does not compare case-insensitively, meaning it can be used to sort things like files and still get the numbers be sorted in a human friendly way
Originally implemented for #76730, split off for possibility of being cherry-picked for 4.0 and possibly 3.x and in case the other PR doesn't get merged, and I feel this functionality is useful to get in

@mhilbrunner would you mind looking at the documentation changes here too as most of the ones from #76730 are really from here

Thanks!

Thank you!

Do we want to backport this for 3.x? I have a working version of the code for that


OptimalCombination pip import error

Hello, when I call pip install hierarchicalforecast and then try to import the OptimalCombination class I get an import error.
I was able to fix this issue but calling pip install git+https://github.com/Nixtla/hierarchicalforecast to import the newest version of the repo, so I think the pip is not up to date with the latest version of the repo.

Hey @dluuo!
Thanks for reporting the issue. We'll be releasing a new version of HierarchicalForecast to include OptimalCombination and another improvement in the next week. :)

Hey @dluuo! The new version of HierarchicalForecast is now released (https://pypi.org/project/hierarchicalforecast/)


Support Promise cancellation for all connectors

We now register a Promise cancellation handler so that the following code actually cleans up the underlying socket resource:
$promise = $connector->create('reactphp.org', 80);

$promise->cancel();

Closes #40

Updated to fix errors on legacy PHP versions, now ready for review :shipit:


Switch to a different clipboard hook for API key copy in prefs modal

Fixes #58
See if this works for you.  I switch the clipboard hook for this use case.

Weird that there are 2 ways to do this and one of them doesn't work.

I could make it work with the other Chakra hook, but to do it I have to always copy the key to the clipboard when the modal opens. This seems wrong to me.  Using this other one, I can do it only when needed.


Update static-pod.md

Add details about Ubuntu 16.04

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
For 1.11 Features: set Milestone to 1.11 and Base Branch to release-1.11
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
NOTE: After opening the PR, please un-check and re-check the "Allow edits from maintainers" box so that maintainers can work on your patch and speed up the review process. This is a temporary workaround to address a known issue with GitHub.>
Please delete this note before submitting the pull request.

@sb1975 we need you to sign the CLA before we can merge your PR. lmk if you have questions -- thanks!
/assign

👋 We've migrated the site's framework to Hugo. Please re-open this PR with changes to files in their post-migration location (/content/en/docs/...) and reference this PR in the new PR's description.


!! Not an issue !! Tv Shows

Hello! Can Tv shows and "emisiuni" from antena be showed only by year, not also month?

Unfortunately no, it is a limitation of their system, you can only filter
by month & year.
On Sat, 27 Mar 2021 at 11:18, the-poor-gamer-124 @.***>
wrote:

Hello! Can Tv shows and "emisiuni" from antena be showed only by year, not
also month?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/redmusicxd/iptv_ro/issues/12, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHAX5VG2G4R6KLA4XSYDCXDTFWPIFANCNFSM4Z4W6Z3Q
.

Ok. Can you help me add a few more elements to that html

Sure, what do you need?
On Sat, 27 Mar 2021 at 11:38, the-poor-gamer-124 @.***>
wrote:

Ok. Can you help me add a few more elements to that html
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/redmusicxd/iptv_ro/issues/12#issuecomment-808702706,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHAX5VGYNDC3YD4R7NYYVULTFWRSXANCNFSM4Z4W6Z3Q
.

Some css. Also, with digi, since my ipv4 is blocked, can I use my IPV6

For streams*y

No CSS as the scope of the html was just for testing purposes, in the
future will be in JSON format but I didn’t had the time or the mood to do
so, like all the other things for this project.
As for the ipv6, afaik their stream servers are only ipv4 so you can’t
access it from an ipv6 address.
On Sat, 27 Mar 2021 at 13:09, the-poor-gamer-124 @.***>
wrote:

For streams*y
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/redmusicxd/iptv_ro/issues/12#issuecomment-808714011,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHAX5VHYHWRDG5PJZHNDLVTTFW4GTANCNFSM4Z4W6Z3Q
.

ah, ok

I cannot wait for JSON format

Anwyays, until then, maybe you can make it easier to modify HTML?

what do you mean?

having an external html file ?
also do you use discord?

actually right now i'm working on replacing the HTML with JSON as it's been due for some time now and yes, i do use discord.

can you give me your @ and #
sâm., 27 mar. 2021, 16:17 Dan Negru @.***> a scris:

actually right now i'm working on replacing the HTML with JSON as it's
been due for some time now and yes, i do use discord.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/redmusicxd/iptv_ro/issues/12#issuecomment-808739999,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALAU4WFI6EHCX4GUASHPQATTFXSF5ANCNFSM4Z4W6Z3Q
.

rednblkx#6553


Screenshots are not being embedded to my cluecumber report

Screenshots are not being added to the report - I have added configurations I could see online but I am not seeing the button to show/hide the before/after hooks.

Hello! Without an example json file, I cannot see where your screenshot should be embedded. Are you sure screenshots are being taken correctly and attached to the scenario by your test framework?

hi! maybe the screemshot is embedded in your hooks. In that case click on Show scenario hooks.

If you want to keep the before and after hooks always open, just add this configuration parameter:
<expandBeforeAfterHooks>true</expandBeforeAfterHooks>

@niamhmccann , did this solve your problem?

Closed due to no reaction.


Replaced LHNOptionsList FlashList -> RN FlatList

Details

TBD: https://github.com/Expensify/App/issues/40048#issuecomment-2062193517.
Fixed Issues

$ https://github.com/Expensify/App/issues/40048
PROPOSAL: https://github.com/Expensify/App/issues/40048#issuecomment-2049738563
Tests

Launch the app and log in with an existing account with multiple chats.
Open several reports, tasks, and threads.
Everything should work as expected.

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

Offline tests

Launch the app and log in with an existing account with multiple chats.
Open several reports, tasks, and threads.
Everything should work as expected.

QA Steps

Launch the app and log in with an existing account with multiple chats.
Open several reports, tasks, and threads.
Everything should work as expected.

[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 either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[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(theme.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 UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or 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 and/or tagged @Expensify/design 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

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

Unfortunately still able to reproduce https://github.com/Expensify/App/issues/40048 and https://github.com/Expensify/App/issues/40360


multiple not working with ngf-drop

Hi Danial
I cannot upload multiple files using ngf-drop (see sample code here -> http://jsfiddle.net/huhjo9jm/705/ )
How I can do this?
Thanks in advance.

ngf-multiple="true" made it work!

@dieontime ok, it work! thanks a lot


Highlighting fails in language injections

Description
Using IJ language injections, it happens very often (always?) that highlighting fails.
One recent example:

Feels like this example should work out of the box because it's a self contained document.
Other cases are less clear. For an example, with an executable document:
{
  user {
    name
    unknownField
  }
}

SInce there's no schema to validate against, it'd be fine to show an error here but my preferred solution would be to only check the syntax, no the actual validation.
Version and Environment Details
Operation system: MacOS
IDE name and version: IntelliJ IDEA 2024.2.4
Plugin version: 242.21829.3

Do you have any GraphQL config files in this project?


Sample to collex pubsub

What and why?
How to test?
Trello
https://trello.com/c/L3m9uVhK/491-s36-replace-rabbit-with-pub-sub-sample-to-collection

/deploy sinhaa

/deploy baileyw


OpenAPI definition: Anonymizer list type in anonymize request does not reflect the underlying json data schema

Describe the bug
The OpenAPI definition of the anonymize request reads as follows:
anonymizers:
          description: "Object where the key is DEFAULT or the ENTITY_TYPE and the value is the anonymizer definition"
          type: object
          anyOf:
            - $ref: "#/components/schemas/Replace"
            - $ref: "#/components/schemas/Redact"
            - $ref: "#/components/schemas/Mask"
            - $ref: "#/components/schemas/Hash"
            - $ref: "#/components/schemas/Encrypt"

This translates to (and swagger-codegen agrees) as one of the Replace/Redact/etc. objects.
Actually, as also shown in the examples (e.g. here), this field is effectively a map from a string (entity name) to one of these objects.
To Reproduce
Generate a client from the Open API file to e.g. Golang using swagger-codegen
Expected behavior
The generated AnonymizeRequest.Anonymizers field should have a map type.
Instead, it is a pointer to a single struct that represents a single anonymizer.
Check it out here

Also valid for the deanoniymize endpoint


ci: move audit to poetry

Move pip-audit to poetry

Codecov Report
Patch and project coverage have no change.

Comparison is base (42b70b2) 95.44% compared to head (d39b4c8) 95.44%.

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

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2600   +/-   ##
=======================================
  Coverage   95.44%   95.44%
=======================================
  Files         975      975
  Lines       27221    27221
=======================================
  Hits        25981    25981
  Misses       1240     1240

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


Error to download model

Hi, thank you for your job.
I want to try your implementation but I get an ERROR 404 when I want to download the model :/
I also have a question, on which corpus the network was trained ?
Thanks

The script has been updated with the new address. Can you pull and try again?

@joonson wget http://www.robots.ox.ac.uk/~vgg/software/lipsync/data/face_detection_tf.zip 404 ERROR

It appears to be working fine -- maybe it was a temporary error.

I know the reason, you have to use new wget not the old one


Add levant

As we discussed on friday, levant might be very useful
https://github.com/hashicorp/levant

While working on this PR, I noticed that the terraform apply did not fail, although the mc(minio-client) nomad batch job failed to get deployed successfully.
module.minio.nomad_job.nomad-job-mc: Creating...
module.minio.nomad_job.nomad-job-mc: Creation complete after 0s [id=mc]

I went through nomad's github repo to find out if they have any such feature so as we can wait for the job to get successfully deployed  and i saw that they have an issue which is in Open state: Flag to wait till deployment actually succeeded
Alternatively, levant could be useful here to make such advanced checks:
https://github.com/hashicorp/levant/blob/master/README.md
Advanced Job Status Checking: Particularly for system and batch jobs, Levant ensures the job, evaluations and allocations all reach the desired state providing feedback at every stage.


Item Visibility representation?

In this issue, I would like to discuss possible representations of the visibility modifiers for items.

The reference defines 6 different Visibilities (including, defining none at all):

Visibility :
pub
| pub ( crate )
| pub ( self )
| pub ( super )
| pub ( in SimplePath )
Source: https://doc.rust-lang.org/reference/visibility-and-privacy.html

Rustc, switched from this representation to a simpler one, with only three values:
pub enum Visibility {
    Public,
    Restricted(DefId),
    Invisible,
}

Source: https://doc.rust-lang.org/nightly/nightly-rustc/rustc_middle/ty/enum.Visibility.html

The question is now, which representation works best for a stable API. We could also create an enum, with three values. Which will be stable and probably simpler to maintain, but is a bit further away from rustc's reference. Something like:
pub enum Visibility {
    Public,
    Restricted(Path),
    Invisible,
}

or
pub enum Visibility {
    Public,
    Restricted(ItemId),
    Invisible,
}

Another thing to consider, is that pub(crate) and pub(self) etc... might restrict the visibility to different modules, depending on the compilation, as the crate and module names can change based on conditional compilation and cargo file configuration. That's one thing that speaks for having separate representations.
Any thoughts on this? :upside_down_face:

You could do something along the lines of this:
pub struct Visibility {
    inner: Vis,
}

enum Vis {
	// whatever you want
}

impl Visibility {
    pub fn public() -> Self {}
    pub fn pub_crate() -> Self {}
    pub fn in_path(path: Path) -> Self {}

    pub fn is_public() -> bool {}
    pub fn is_pub_crate() -> bool {}
    pub fn is_in_path(path: Path) -> bool {}

    pub fn accessible_from_path(path: Path) -> bool {}

    // more methods
}

Doing this exposes nothing whatsoever about the internals, which can change freely without API breakage. This pattern can be generalized to anything, so long as the set of operations can be reasonably expressed using methods.

Well the is_public wouldn't be necessary because is_visible_to would already return true, but yeah, something along those lines. The compiler already has the concept of "accessible" and methods for checking it.

As I'm working on something extremely similar to visibility right now, I just thought of this. Is there any reason visibility couldn't be simplified even farther, having only "public" and "visible within this module"? The latter would subsume private, as the module of definition could be used.

For me, this highly depends on what information users want. I can imagine scenarios which say something like: "If the struct implements Trait, then it has to ". Having a struct with is_public and is_visible_from would cover almost every direct semantic information, but not how it was defined in code. pub(super) can't be distinguished from pub(crate::example) if the item is in a submodule of crate::example.
Maybe a hybrid might be good, where we have a struct with semantic information (is_public, is_visible_from) and a function to get the exact definition/written syntax :thinking:

That could work. It's easier to add methods than to take away (given the purpose is to be stable), so it would probably be best to start small and expand as necessary.


Invalid command prerelease

Problem

It's impossible for me to create a prerelease. The CLI help advertises a prerelease command but when I call changeset prerelease it it return an error saying

🦋  error Invalid command prerelease was provided

If I add the <tag> parameter like changeset prerelease beta the error is

🦋  error Too many arguments passed to changesets - we only accept the command name as an argument

Using version 2.10.3

This example usage was incorrect - probably something that got in unnoticed when iterating over pre mode feature. I've prepared a PR with the fix here: https://github.com/atlassian/changesets/pull/465


numbers are wrong

I added the badge to one repo, but the number that it shows is wrong: https://github.com/yegor256/xembly
I'm sure that the right number is what the hosted hoc service is showing: 16K+. Something is wrong with the calculation?

The action only call the hoc with arguments.
I saw that the numbers on the host and on Github are different.
Perhaps the reason is the different number of branches on host and Github?

@MikhailEpatko hoc command line tool doesn't work with branches, it only analyzes the master branch

@yegor256 when I start next command on the host:
hoc -d . -s 2000-01-01 -b 2024-03-07 -f int
it returns 412.
The same command in Github workflow on the same project returns 192.
When I added 1 line of code on the host hoc returned 413, in workflow - 193.
After I removed this line the numbers were 412 and 192 again.
The command
 hoc -d . -s 2000-01-01 -b 2024-03-07 -f int -e .github -e .idea -e .gitignore -e README.md -e LICENSE
returns 234 and 101.
The command ls -al returns for host and workflow the same:
total 40 drwxr-xr-x 5 runner docker 4096 Mar  7 06:58 . drwxr-xr-x 3 runner docker 4096 Mar  7 06:58 .. drwxr-xr-x 8 runner docker 4096 Mar  7 06:58 .git drwxr-xr-x 3 runner docker 4096 Mar  7 06:58 .github -rw-r--r-- 1 runner docker   11 Mar  7 06:58 .gitignore -rw-r--r-- 1 runner docker 1064 Mar  7 06:58 LICENSE -rw-r--r-- 1 runner docker 1414 Mar  7 06:58 README.md -rw-r--r-- 1 runner docker 1530 Mar  7 06:58 action.yml -rwxr-xr-x 1 runner docker [117](https://github.com/MikhailEpatko/hoc-badge-action/actions/runs/8184154830/job/22378148081#step:3:134)3 Mar  7 06:58 generate-badge.sh drwxr-xr-x 2 runner docker 4096 Mar  7 06:58 output-dir
May be hoc returns different numbers for host and Github workflow due to different environment?

@MikhailEpatko maybe this happens because the default value of fetch-depth of actions/checkout equals to 1?

@yegor256 I'll check it

@MikhailEpatko maybe this happens because the default value of fetch-depth of actions/checkout equals to 1?

Thanks! It was very helpful.

fixed


missing dependencies

the fix was working for about a week and around the time the game got updated it would no longer work I tried starting with a fresh install but nothing changed. I am not sure if its a melon loader problem or a game problem. I am not good with tech so I am not sure if this is the right thing to paste in here.
22:06:39.349] ------------------------------
[22:06:39.349] MelonLoader v0.5.4 Open-Beta
[22:06:39.350] OS: Windows 10
[22:06:39.350] Hash Code:<PHONE_NUMBER>4810154100579910097100505048995753975597975597101485351514856
[22:06:39.350] ------------------------------
[22:06:39.350] Game Type: MonoBleedingEdge
[22:06:39.351] Game Arch: x64
[22:06:39.351] ------------------------------
[22:06:39.351] Core::BasePath = C:\Program Files (x86)\Steam\steamapps\common\Syberia - The World Before
[22:06:39.352] Game::BasePath = C:\Program Files (x86)\Steam\steamapps\common\Syberia - The World Before
[22:06:39.352] Game::DataPath = C:\Program Files (x86)\Steam\steamapps\common\Syberia - The World Before\SyberiaTWB_Data
[22:06:39.352] Game::ApplicationPath = C:\Program Files (x86)\Steam\steamapps\common\Syberia - The World Before\SyberiaTWB.exe
[22:06:40.477] ------------------------------
[22:06:40.477] Game Name: SyberiaTWB
[22:06:40.478] Game Developer: Microids
[22:06:40.479] Unity Version: 2020.3.25f1
[22:06:40.479] Game Version: 1.0
[22:06:40.479] ------------------------------
[22:06:40.734] Preferences Loaded!
[22:06:40.748] Loading Plugins...
[22:06:40.751] ------------------------------
[22:06:40.752] No Plugins Loaded!
[22:06:40.752] ------------------------------
[22:06:42.132] Loading Mods...
[22:06:42.151] [WARNING] Some Melons are missing dependencies, which you may have to install.
If these are optional dependencies, mark them as optional using the MelonOptionalDependencies attribute.
This warning will turn into an error and Melons with missing dependencies will not be loaded in the next version of MelonLoader.

'Syberia: The World Before Fix' is missing the following dependencies:

'UnhollowerBaseLib' v<IP_ADDRESS>
'Il2Cppmscorlib' v<IP_ADDRESS>

[22:06:42.153] ------------------------------
[22:06:42.153] 1 Mod Loaded
[22:06:42.154] ------------------------------
[22:06:42.154] Syberia: The World Before Fix v1.0.2
[22:06:42.154] by Lyall
[22:06:42.156] SHA256 Hash: 3e4d7ab81a01f769212d765718272671c240785d48e8897a361c0c1bfc88351f
[22:06:42.156] ------------------------------
[22:06:42.158] [WARNING] Some Melons are missing dependencies, which you may have to install.
If these are optional dependencies, mark them as optional using the MelonOptionalDependencies attribute.
This warning will turn into an error and Melons with missing dependencies will not be loaded in the next version of MelonLoader.

'Syberia: The World Before Fix' is missing the following dependencies:

'UnhollowerBaseLib' v<IP_ADDRESS>
'Il2Cppmscorlib' v<IP_ADDRESS>
[22:06:42.209] [Syberia:_The_World_Before_Fix] Application started.
[22:06:42.221] MelonPreferences Loaded from Mods/SyberiaTWBFix.cfg
[22:06:42.225] [WARNING] Some Melons are missing dependencies, which you may have to install.
If these are optional dependencies, mark them as optional using the MelonOptionalDependencies attribute.
This warning will turn into an error and Melons with missing dependencies will not be loaded in the next version of MelonLoader.

'Syberia: The World Before Fix' is missing the following dependencies:

'UnhollowerBaseLib' v<IP_ADDRESS>
'Il2Cppmscorlib' v<IP_ADDRESS>
[22:06:48.412] [Syberia:_The_World_Before_Fix] [ERROR] System.IO.FileNotFoundException: Could not load file or assembly 'UnhollowerBaseLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' or one of its dependencies.
File name: 'UnhollowerBaseLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'
at MelonLoader.MelonHandler+<>c__DisplayClass46_0.b__0 (MelonLoader.MelonMod x) [0x0000c] in <0f6d2009dc5d4313a30db0346f3f685b>:0
at MelonLoader.MelonHandler.InvokeMelonMethod[T] (System.Collections.Generic.List1[T]& melons, MelonLoader.MelonHandler+InvokeMelonMethodDelegate1[T] method, System.Boolean remove_failed) [0x00032] in <0f6d2009dc5d4313a30db0346f3f685b>:0
[22:06:48.615] [Syberia:_The_World_Before_Fix] Skipped intro.
[22:06:48.648] [WARNING] Some Melons are missing dependencies, which you may have to install.
If these are optional dependencies, mark them as optional using the MelonOptionalDependencies attribute.
This warning will turn into an error and Melons with missing dependencies will not be loaded in the next version of MelonLoader.

'Syberia: The World Before Fix' is missing the following dependencies:

'UnhollowerBaseLib' v<IP_ADDRESS>
'Il2Cppmscorlib' v<IP_ADDRESS>
geforcenowdetect():GfnInitializeSdkFromPath(GfnRuntimeSdk.dll): -28.
geforcenowdetect():GfnInitializeSdkFromPath(Assets\Plugins\geforcenowdetect\GfnRuntimeSdk.dll): -28.
geforcenowdetect():GfnInitializeSdkFromPath(SyberiaTWB_Data\Plugins\x86_64\GfnRuntimeSdk.dll): 1.
geforcenowdetect():GfnIsRunningInCloud():false.
[22:06:50.476] [Syberia:_The_World_Before_Fix] [ERROR] System.IO.FileNotFoundException: Could not load file or assembly 'UnhollowerBaseLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' or one of its dependencies.
File name: 'UnhollowerBaseLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'
at MelonLoader.MelonHandler+<>c__DisplayClass46_0.b__0 (MelonLoader.MelonMod x) [0x0000c] in <0f6d2009dc5d4313a30db0346f3f685b>:0
at MelonLoader.MelonHandler.InvokeMelonMethod[T] (System.Collections.Generic.List1[T]& melons, MelonLoader.MelonHandler+InvokeMelonMethodDelegate1[T] method, System.Boolean remove_failed) [0x00032] in <0f6d2009dc5d4313a30db0346f3f685b>:0
Setting breakpad minidump AppID = 1410640
SteamInternal_SetMinidumpSteamID:  Caching Steam ID:<PHONE_NUMBER>0312723 [API loaded no]

Hey, so the problem is that in the last game update they switched from IL2CPP to Mono. It's kind of an odd decision, most games I've seen have gone from Mono to IL2CPP instead. Which basically means that I have to retarget the mod for the latest build of the game. This may take me a little while as I have to rewrite parts of the mod.

Looks like it actually wasn't that difficult. I've put up a new release that should work on the latest version of the game.
https://github.com/Lyall/SyberiaTWBFix/releases/tag/v1.0.3
Let me know if all is working for you and I'll close the issue.

it seems to be working perfectly thank you very much.

That's great to hear. I'll close the issue now.


Web3 Foundation Grant Program

if you could help with the docker file about V2, we could delivery it to Web3 Foundation following Grant Program.
Previous submit sample: https://github.com/Moehringen/Grant-Milestone-Delivery/blob/master/deliveries/Web3Go_milestone_1.md
This is the original application: https://github.com/w3f/Grants-Program/blob/master/applications/Web3Go.md
This is the guide on how to submit: https://github.com/w3f/Grant-Milestone-Delivery#mailbox-milestone-delivery-process

DONE