Suggested clarification to documentation to fix "Error downloading org.clulab:glove-840b-300d-10f" issue

Hi,
Cool project! I am interested in Odinson but don't have much experience with sbt. I had some trouble with the build instructions on the docs page that I was eventually able to fix. I am posting this here in case others run into the same issue. I think updating the docs to describe this issue would save others time and headaches in the future.
Backstory
I tried to build by following the sbt build instructions here but got an error "Error downloading org.clulab:glove-840b-300d-10f".
I did some research, including looking at the processors library and eventually realized I needed to add a resolver to the build file. I thought that was all I needed but I kept hitting more errors. From this stackoverflow post I realized I needed to add withAllowInsecureProtocol(true) when adding the resolver.
Improvements requested:
If you could show how to add an http resolver by adding the following line to the demo build.sbt I think it would be very helpful for those unfamiliar with sbt.
resolvers += ("Artifactory" at "http://artifactory.cs.arizona.edu:8081/artifactory/sbt-release").withAllowInsecureProtocol(true)

libraryDependencies ++= {
  val odinsonVer = "0.5.0"
  Seq(
      "ai.lum"        %% "odinson-core"             % odinsonVer,
      "ai.lum"        %% "odinson-extra"            % odinsonVer
  )
}

Thanks, @AbeHandler .
Just to confirm: things are working for you after you added that resolver?
(That one relates to a dependency of odinson-extra).

@myedibleenso yes things are working in the sense that $sbt compile runs successfully and without error with the change I outlined above

Hi, @AbeHandler , after adding  resolvers += ("Artifactory" at "http://artifactory.cs.arizona.edu:8081/artifactory/sbt-release").withAllowInsecureProtocol(true)   I got some error messges.   Things do not work for me.
K:\software\odinson-0.5.0>sbt compile
[info] Loading settings for project odinson-0-5-0-build from plugins.sbt ...
[info] Loading project definition from K:\software\odinson-0.5.0\project
K:\software\odinson-0.5.0\build.sbt:178: error: value withAllowInsecureProtocol is not a member of sbt.librarymanagement.MavenRepository
resolvers += ("Artifactory" at "http://artifactory.cs.arizona.edu:8081/artifactory/sbt-release").withAllowInsecureProtocol(true)
                                                                                ^
[error] Type error in expression
Project loading failed: (r)etry, (q)uit, (l)ast, or (i)gnore? r
[info] Loading settings for project odinson-0-5-0-build from plugins.sbt ...
[info] Loading project definition from K:\software\odinson-0.5.0\project
K:\software\odinson-0.5.0\build.sbt:178: error: value withAllowInsecureProtocol is not a member of sbt.librarymanagement.MavenRepository
resolvers += ("Artifactory" at "http://artifactory.cs.arizona.edu:8081/artifactory/sbt-release").withAllowInsecureProtocol(true)
                                                                                ^
[error] Type error in expression
Project loading failed: (r)etry, (q)uit, (l)ast, or (i)gnore?

Whether or not .withAllowInsecureProtocol(true) is needed or allowed is dependent on sbt version.  It looks like it isn't being allowed, so please try without.


Update github pages index

Fixed some links, updated game version, etc.

I think it's v1.14.0-f9 since yesterday 🤔

updated


Error: This Suspense boundary received an update before it finished hydrating.

Bug report
Description / Observed Behavior
When using next/dynamic in combination with SWR 2.0.3 We get an error:
Error: This Suspense boundary received an update before it finished hydrating. This caused the boundary to switch to client rendering. The usual way to fix this is to wrap the original update in startTransition

Expected Behavior
SWR 2.0.2 does not throw an error
Repro Steps / Code Example
https://github.com/iljamulders/nextjs-dynamic-swr
$ yarn install

$ yarn dev

const Books = dynamic(() => import('@/components/Books'))

const fetcher = (url: URL) => fetch(url).then(res => res.json())

export default function Home() {

  const { data: books } = useSWR('https://63d8db905a330a6ae16f31da.mockapi.io/Books', fetcher)

  return (
    <Books books={books} />
  )
}

Additional Context
SWR version: 2.0.3

This is actually a react bug which has alrealy been fixed react@next react-dom@next
Here is the reproduction
https://codesandbox.io/p/sandbox/priceless-darwin-frq81n?file=%2Fpages%2Findex.tsx&selection=[{"endColumn"%3A1%2C"endLineNumber"%3A1%2C"startColumn"%3A1%2C"startLineNumber"%3A1}]
The current workaroud for this issuse const Books = dynamic(() => import('@/components/Books'), { ssr: false })

Thanks @promer94
This bug is indeed fixed in react@next react-dom@next
Closing this issue


Add Pancake Swap V3 Dependencies

@pancakeswap/v3-core
@pancakeswap/v3-periphery

Hey, thanks for the suggestion, added!
https://soldeer.xyz/project/@pancakeswap-v3-core
https://soldeer.xyz/project/@pancakeswap-v3-periphery


Previously consumed OTP is considered valid after drift period has passed

RFC 6238 says that once an OTP is used, it cannot ever be used again.
The tests here are not properly testing the scenario where an OTP is consumed, then the user waits an amount between the drift period and twice the drift period, then tries entering the OTP again. The expected behavior is that the OTP should not be valid, but it is.
The problem with the tests is that they are freezing Time.current, but because current_otp_timestep depends on the current time, it must not be frozen. Instead Timecop.travel must be used to advance in time.
Here is a failing spec:
describe '#validate_and_consume_otp!' do
  let(:otp_secret) { '2z6hxkdwi3uvrnpn' }

  before :each do
    subject.otp_secret = otp_secret
  end

  after :each do
    Timecop.return
  end

  context 'with a stored consumed_timestep' do
    context 'given a previously correct OTP' do
      it 'fails to validate' do
        consumed_otp = ROTP::TOTP.new(otp_secret).at(Time.now)
        subject.validate_and_consume_otp!(consumed_otp)

        amount_to_travel = (subject.class.otp_allowed_drift).seconds
        Timecop.travel(amount_to_travel)

        expect(subject.validate_and_consume_otp!(consumed_otp)).to be false
      end
    end
  end
end

I updated my comment. I had pasted in the wrong code initially.
If you play around with the time_to_travel value, you'll see that when the amount of time to travel is equal to the drift period or between the drift period and twice the drift period, the test fails, but if you advance 60 seconds, then the test passes.

Here is an interesting discussion on this subject: https://github.com/mdp/rotp/issues/44

The ROTP gem has now implemented this via the verify_with_drift_and_prior  method. See: https://github.com/mdp/rotp#preventing-reuse-of-time-based-otps
Would it make sense for this gem to use that functionality as opposed to your own implementation?

I think that'd be a worthwhile change to make. Thanks for pointing it out!
If you were to open a pull request, I'd be happy to review it.


received a naive datetime while time zone support is active.

Deployment Method

[ ] Installer
[ ] Docker Development
[X] Docker Production
[ ] Bare metal Development
[ ] Bare metal Production
[ ] Digital Ocean image
[ ] Other (please provide a link Steps to Reproduce

Describe the problem*
I get an error in my log when starting the container. Not sure how to correct it.
Steps to Reproduce

Start the container.

Relevant log output
ython version 3.10.13 - /usr/local/bin/python3
/root/.local/lib/python3.10/site-packages/django/db/models/fields/__init__.py:1358: RuntimeWarning: DateTimeField PartPricing.updated received a naive datetime (2023-11-25 00:00:00) while time zone support is active.
  warnings.warn("DateTimeField %s.%s received a naive datetime "
/root/.local/lib/python3.10/site-packages/django/db/models/fields/__init__.py:1416: RuntimeWarning: DateTimeField NotesImage.date received a naive datetime (2023-09-26 06:55:06.646529) while time zone support is active.

sounds like #5223; hard to say without a full bug report/version stump

Probably a similar fix. If you can work out which API request it correlates to that would be very helpful.


Add support for indexmap

It would be nice if rsn supported indexmap (through a feature flag) so it can be used for serializing when field order matters.

Is there anything in particular you feel like this crate needs to do? indexmap supports all serde serialization formats by enabling the optional serde  feature in its crate. The Value type in this crate already treats maps as ordered-preserved, key-value pairs by returning them in a Vec.

Apparently not. I'm so used to having to enable the indexmap feature in serde-json and toml, that I simply overlooked the fact that it already works out of the box. Sorry for the inconvenience.


Fixed Rotation of Arrow Icon in a Row

I think I am to blame on this one, heh

After all the hours of looking into timings I have finally decided on getting it fixed by myself. 👍


log_dict method breaks on MultitaskWrapper + MetricCollection combination

🐛 Bug
I am trying to log metrics with the MultitaskWrapper and MetricCollection combination, which  analogous to the test case that can be found in integrations/test_lightning.
Logging this configuration of metrics with Lightning's built in log_dict method throws a ValueError (see in the reproduction snippet), which is confusing as logging either MultitaskWrapper or MetricCollection individually works with the log_dict method and this exact logging approach is presented in the integration tests.
To Reproduce
Steps to reproduce the behavior...

Code sample
The multitask configuration from integrations/test_lightning::test_task_wrapper_lightning_logging:
self.multitask_collection = MultitaskWrapper({
    "classification": MetricCollection([BinaryAccuracy(), BinaryAveragePrecision()]),
    "regression": MetricCollection([MeanSquaredError(), MeanAbsoluteError()]),
})

The log_dict method splits the MultitaskWrapper into the individual tasks, however it tries to call the log method on the MetricCollection, which breaks with the following ValueError:
E       ValueError: `self.log(classification, MetricCollection(
E         (BinaryAccuracy): BinaryAccuracy()
E         (BinaryAveragePrecision): BinaryAveragePrecision()
E       ))` was called, but `MetricCollection` values cannot be logged

This ValueError is not caught when running the test, as it seems to be silenced by the no_warning_call.

Expected behavior
log_dict working for the MultitaskWrapper + MetricCollection combination or some warning that it is not possible, as this way the corresponding testcase is misleading.
Environment
Environment

CentOS
Python 3.11
torchmetrics == 1.3.0.post0 (with poetry)
torch == 2.2.0.post0 (with poetry)

Hi @ponderbb, thanks for raising this issue and sorry for my slow reply.
It would seem to me that this was actually fixed in PR #2349 which is part of the v1.3.1 release of torchmetrics. It should therefore work if you update to the newest torchmetrics.
Closing issue.

That is great news @SkafteNicki, I will try with the new release! :)


Removing User-collection from Litmus DB, adding backend communication to auth server

Remove User-collection from Litmus DB and add backend communication to auth server to get user-related details, this will prevent DB inconsistencies in case of some queries/operations.

@montatip16


MathJax rendering breaks find in page highlight support

Issue Summary
Essentially find-in-page (using CTRL+F) can successfully find text in CHTML-rendered MathJax, however this text isn't highlighted in any way as would be expected for regular text.
Steps to Reproduce:
Try this sample in the demo:
<math>
  <mi>test</mi>
</math>

Use the browser's CTRL+F find in page feature to search for "test", notice MathJax text is findable, but isn't highlighted.
Technical details:

MathJax Version: Latest
Client OS: ChromeOS 99
Browser: Chrome 99

Supporting information:
Notice find in page can find the text, but no highlighting is applied:

Currently expected behavior but about to change, see https://github.com/mathjax/MathJax-src/pull/734

Actually, the browser is not finding the typeset output, but rather the assistive MathML that is inserted in the page for those using screen readers and other assistive technology.  This MathML is visually hidden, but available to screen readers.  It is actually the "test" that is in <mi> in that hidden MathML that the browser is finding, but because it is not visible in the display, you don't see it being highlighted.  You can verify that this is true by turning off the assistive MathML in the MathJax contextual menu (under the accessibility submenu); if you do that, the search should only find the on in the input text area.
The current CHTML output is created using CSS content specifiers for ::before selectors on certain elements, and such content is neither selectable nor searchable.  As Peter points out, that is going to change in the next release, and it looks like browsers will be able to find the "test" in the output (and show it highlighted), but it will still also find the assistive MathML output, which is not be visible.  I don't see any way to have the assistive MathML available to screen readers while not allowing it to be found by the browser search.  Sighted users can, of course, turn off assistive MathML to avoid the issue.

I don't see any way to have the assistive MathML available to screen readers while not allowing it to be found by the browser search.

Well at least Chrome is finally implementing MathML, so in future people probably should just emit native MathML rendering where supported, in which case the accessibility tree and render tree will be able to share the same nodes. Having the extra invisible in just the CHTML output won't matter too much as hopefully people will just use the CHTML output for supporting older browsers without native rendering support.


Reduce code complexity in flake file

Reduce code complexity to improve code quality. It's currently set to 15 in flake8 config file.

If reduced to 10, the following errors need to be fixed :
.\lisa\runner.py:235:5: C901 'RootRunner._start_loop' is too complex (15)
.\lisa\search_space.py:97:5: C901 'IntRange.check' is too complex (12)
.\lisa\search_space.py:299:1: C901 'check_countspace' is too complex (13)
.\lisa\runners\legacy_runner.py:629:1: C901 '_track_progress' is too complex (13)
.\lisa\sut_orchestrator\azure\platform_.py:601:5: C901 'AzurePlatform._get_environment_information' is too complex (15)
.\lisa\sut_orchestrator\azure\platform_.py:728:5: C901 'AzurePlatform._get_location_info' is too complex (11)
.\lisa\sut_orchestrator\azure\platform_.py:809:5: C901 'AzurePlatform._create_deployment_parameters' is too complex (15)
.\lisa\sut_orchestrator\azure\platform_.py:1183:5: C901 'AzurePlatform._resource_sku_to_capability' is too complex (11)
.\lisa\tools\reboot.py:50:5: C901 'Reboot.reboot' is too complex (12)
.\lisa\util\process.py:63:5: C901 'Process.start' is too complex (12)
.\lisa\util\process.py:144:5: C901 'Process.wait_result' is too complex (13)


where is rest apis docs?

like  'https://confluence.atlassian.com/display/BITBUCKET/Use+the+Bitbucket+REST+APIs'

gitbucket exposes some github API v3 if I am not mistaking (https://developer.github.com/v3/).
For the moment I do not know a doc page on that, you can perhaps start one. Look at the controller classes in the source code, some must implement those github endpoints.

I write it https://github.com/takezoe/gitbucket/wiki/API-WebHook

@nazoking Good job! Thanks a lot.

@pinghe good job


Extension is broken since the modrinth notification update

Mod name got replaced by mod id and
Notifications don't get cleared on the Extension
Broken link (Fixed in #7)
Scooped tokens should be recommended and instruction to get them should be updated
Maybe something else I didn't yet notice

Thanks, I merged the PR, the mod ID is fixed and I'll make proper clearing of the new notification type next (it will actually remove the notification).

Notification clearing now works properly, API token information got updated, I think that's everything that broke since the update. I didn't notice any more bugs while testing the new features so I'll close this issue 👍


Throttling errors when discovering resources not supported by advanced query

This issue was found by a customer, I am creating this issue on their behalf to track the fix.
Describe the bug
When the discovery process runs we see an error There were 3 errors when importing resource types unsupported by advanced query in the logs. Running the discovery process with logging set to DEBUG, gives the follow error logs:
{
  "level": "error",
  "message": "There were 3 errors when importing resource types unsupported by advanced query.",
  "timestamp": "2022-11-25T10:11:26.232Z"
}
{
  "errors": [
    {
      "item": "AWS::ECS::Cluster",
      "message": "Rate exceeded",
      "name": "PromisePoolError",
      "raw": {
        "$fault": "client",
        "$metadata": {
          "attempts": 3,
          "httpStatusCode": 400,
          "requestId": "50f3ee6b-76ca-420d-8f37-13f47b1fbcb7",
          "totalRetryDelay": 1271
        },
        "__type": "ThrottlingException",
        "message": "Rate exceeded",
        "name": "ThrottlingException"
      }
    }
    // ...
   ]
 }

This has been fixed in v2.0.2 that was released on Friday.


encryption for RF69_COMPAT

rf12_encrypt(KEY) is working fine when using rf12 modules (without RF69_COMPAT). When using rfm69cw modules with RF69_COMPAT sending and receiving is working fine but the encryption is just ignored (seems to be not implemented). Any ideas?

Ah, that's probably a bug - with the RF12 driver, an XTEA-based software implementation is used, whereas the RF69 driver uses built-in h/w AES for native mode. I suspect that the compat mode has never been used or implemented.

Is it possible to use this library in native mode - without the RF69_COMPAT flag (when every module in my network is using the rfm69cw) and enable encryption? As I cannot find any examples for that.

Whoops, had a better look - you're right, the Arduino/JeeLib version of RF69 does not support encryption in native mode either.
I can not fix this right now - if you're willing to wait until end of next week, I can have a go at either bringing the RF12 encryption logic into the RF69 compat mode driver, or enabling the AES mode in the native version (I'm not sure at this point which will be easier).

Sure :-) Thank you.

Any news?

Currently having the same problem. Any chance you will invest time into this in the near future?

I'm afraid that won't happen any day soon (at least not by me), despite my earlier promise. It's been quite a while since I worked on the RF12 and RF69 compatibility code in JeeLib.

It might be interesting to remove line 32 from RF69_compat.h in jeelib.
#define rf12_encrypt        rf69_encrypt
Not tested it and no guarantees!

On second thoughts that would work on the wrong buffer and thereby not achieve the objective.


Spring HandsOn-1

Spring HandsOn-1

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
6

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


getPurchaseHistory and getAvailablePurchases consume purchase without calling finishTransactionIOS

Version of react-native-iap
3.4.0
Version of react-native
0.60.5
Platforms you faced the error (IOS or Android or both?)
IOS ( __DEV__ mode only so far)
Expected behavior
getPurchaseHistory and getAvailablePurchases do not affect purchases, they just report
Actual behavior
Calling getPurchaseHistory and getAvailablePurchases will consume previously unconsumed consumable purchases
Tested environment (Emulator? Real Device?)
Real device DEV mode
Steps to reproduce the behavior

Make an in app purchase of a consumable.
Do not call finishTransactionIOS.
Call getPurchaseHistory or getAvailablePurchases, log response, see that the response includes the purchase.
Call getPurchaseHistory or getAvailablePurchases again, log response, see that the response NO LONGER includes the purchase.

@defualt Would you like to try it removing these lines?

Removing those lines, calling getAvailablePurchases and getPurchaseHistory results in empty array.  However the purchases do not get consumed because when I reset the app, purchaseUpdatedListener logs the recent unconsumed purchase.

deleting just this line has the desired effect but I don't know if this breaks something else.
https://github.com/dooboolab/react-native-iap/blob/e33d06f6ac44f6bf80de7a32d7ace2863f4e4cfe/ios/RNIapIos.m#L429

I'm experiencing the same issue.
In fact there are a lot of points in the code where [...finishTransaction:..] is called even if the app never calls .finishTransaction().
IMHO this is a serious bug that has to be fixed as soon as possible.
What if my customer makes a purchase but do not consumes it immediately (...there are zillions of reason why this may happen in a real application..)? Today, the net result is that any pending transaction is immediately finished even the purchase has not been consumed, which may imply (and actually does) money loss for the customer.
Please fix it ASAP.

Moreover... this is a signal of another bug: even if the transaction has been finished without request, getPurchaseHistory() should return it but it doesn't...
So, we have no way to work around the problem.

any news about this?

Stale? not at all, we are still waiting for a fix.

Yes, this is a real issue I'm experiencing as well. Seems to be a critical bug if so.

I agree, this is a significant issue. Once you verify the purchase once it disappears and cannot verify it again. This is very problematic for lifetime purchases.

@defualt Would you like to try removing these lines?

FYI - Removing these lines seemed to fix the issue.

Please fix it!!...

@defualt Would you like to try removing these lines?

FYI - Removing these lines seemed to fix the issue.

@emilioicai Could you kindly tell the effect of removing these lines?
I'll try to remove this and deploy new version if there isn't any answer.

FYI - Removing the lines worked for me a while back, and I am using it in production. I just need to remember to make the change again for any future upgrades unless it is updated in the next release.

FYI - Removing the lines worked for me a while back, and I am using it in production. I just need to remember to make the change again for any future upgrades unless it is updated in the next release.

Thanks for your confirm. I'll try to manage to remove the lines in next release when I do not get the answer from @emilioicai who've added the code.

Any update on this?

This is still an open issue. I still have a patch commenting out this line.

This line is still in the production version of the code.
[[SKPaymentQueue defaultQueue] finishTransaction:transaction];
I still have it manually removed in my production applications. Without it, I was not able to receive the purchase history on the device after the purchase was made.

Still an issue +1

We should be able to get purchase history without consuming

@hyochan it looks like @emilioicai has only made a few contributions to github in the past few months, and most of them are to react-native-lottie. Is there an assistance anyone else can provide to help get this patched?

finishTransactionIOS has been removed since version 7.2.0 and replaced by a unified finishTransaction that works on both platforms. Please try on version 7.3.0 or newer and let us know. Closing this ticket


GetAllTopLevelWindows() does not work with VB.Net

Describe the bug
Using UIA2, .Net 6, app.GetAllTopLevelWidnows() does not detect the window created by Show(), ShowDialog(), Show(Me) and ShowDialog(Me).
Code snippets
I created 2 forms. 1 has a button to launch the other one.

I am using xUnit as well to do the testing.

string ExePath = Directory.GetCurrentDirectory() + "\\TestProjectVB.exe";

      Application app = Application.Launch(ExePath);

      UIA2Automation automation = new ();

      Window main = app.GetMainWindow(automation);

      Button btn = main.FindFirstChild("Button1").AsButton();

      btn.Click();

      Window[] ws = app.GetAllTopLevelWindows(automation);

     // error occurs here
      Window? w = ws.FirstOrDefault(x => x.Name == "Form2");

      Assert.NotNull(w);```

After further testing, this is not an issue.  My apology.


Help page 'Boilerplate Language for DMPs' link is broken

The link goes directly to a download link for
https://scholarsphere-stage.libraries.psu.edu/downloads/cv43nw822
The item still exists in the system, but is just present at a different location:
https://scholarsphere-stage.libraries.psu.edu/downloads/51c18dg40v

Thought I fixed it, but it wasn't, it is fixed now.


what is the directive flow when customer ask alexa to play music in AVS Device SDK

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:

I ask alexa to play music, AVS Device SDK receive a speak directive to SpeechSynthesizer with a audio attachment, then SpeechSynthesizer finish the speech, AVS Device SDK receive a play directive to AudioPlayer with a url, then receive a RenderPlayerInfoCard when music playback is started. So music playback in AVS Device SDK seems pretty slow.
But in echo show , when I ask it to play music, the behavior is different, I can see that echo show start caching PlayerInfocard UI data when SpeechSynthesizer is playing speech, I guess it is caching music audio data at the same time. So music playback in echo show is very fast.
Here is my question, in AVS Device SDK, does the play directive does not send unitl AVS receive SpeechSynthesizer PlaybackFinished event? Does the RenderPlayerInfo directive does not send unitl AVS receive AudioPlayer PlaybackStared event?
Here is my log：
log.txt
What is the expected behavior?

For best customer experience, when SpeechSynthesizer  is handling speak directive, play and RenderPlayerInfo directive should be received by AVS Device SDK at the same time, So the AVS Device SDK can start caching audio data and gui data.
What behavior are you observing?

speak, play and RenderPlayerInfo directive is received one by one unitl last direcive is handled.
Provide the steps to reproduce the issue, if applicable:

ask  AVS Device SDK to play music , then observe alexa behavior and see debug log.
Tell us about your environment:

What version of the AVS Device SDK are you using?
  <1.9.0>

Tell us what hardware you're using:

[ ] Desktop / Laptop
[ ] Raspberry Pi
[* ] Other - tell us more:  STB

Tell us about your OS (Type & version):

[ *] Linux
[ ] MacOS
[ ] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

@aumatsui
thanks for your answer，now the best way I can find to reduce the time It consume before starting play music is buffering music data when the SpeechSynthesizer is speaking, now I have three questions

you say "at this point this is a known limitation and we are already tracking this issue", can you provide more information about the limitation?
since there is limitation, why echo show can cache music data when the SpeechSynthesizer is speaking?
can I modify the avs sdk to buffering music data when the SpeechSynthesizer is speaking?
looking forward to your reply，thank you.

Hi @yanggewei, thanks for your follow up questions.  The limitation mentioned above is an artifact of the AVS protocol - the AudioPlayer.Play Directive is sent once Alexa's speech has finished, so there is not much opportunity to buffer audio ahead of time.  I expect that Amazon's devices, which don't use AVS internally, may have different protocols at this time.  However, it is on our roadmap to converge the behavior.  At this time I'm not sure there is a way to optimize the SDK's behavior, since it depends on the sequence in which AVS sends the Directives.

I'll close this ticket for now, as I don't think there is more to add from the SDK perspective.


Review: greedyjit

New SS option "greedyjit", if nonzero, will remember all created shader groups and optimize/JIT them all up front, concurrently, without locking, the first time any one executes.
This may end up optimizing/JITing shader groups that never need to execute, and also may worsen "time to first pixel."  But if you tend to JIT most of the shaders you specified anyway, this will make sure it's done with perfect parallelism, no locking.
Even if a renderer doesn't want this enabled generally, turning it on selectively is also great for debugging problems with OSL compilation, since it forces all shaders to optimize/jit very early in the render.  It sucks when there's a bug in OSL's optimizer, showing up only in a shader that's first used 7 hours into a render.  Not that I have any experience with that, no sir.
N.B. The app itself doesn't need to be aware of the multithreading; this spawns a bunch of new threads for the compilation, which will subsequently be terminated when all the shaders have been compiled.  It will make as many threads as there are cores, which, if there are several renders on the box simultaneously, may briefly be "over-threading".  But I don't see much harm in that, it should be for only a tiny fraction of the time of a render and doesn't retain any more memory than it would have originally (just the JITed bytes).

Just checked in a revision that exposes to ShadingSystem interface an optimize_all_groups() method, allowing explicit renderer control of when this should happen.  And if the renderer doesn't do it, it will happen at first shader execution, and I also put some new safeguards to make sure that only one thread will launch a bunch of threads, thus limiting the tendency to overthread.


Bundler clean --force after system upgrade.

I want to develop locally using jekyll then push it to github pages. I did it successfuly when my system was Ubuntu 14.04 LTS, but after upgrading to 16.04 today, my jekyll environment stop working.
I ran the command /usr/local/bin/bundle clean --force (with sudo command was executed successfuly)
I expected Bundler to remove all gems, but gives:
Error details
Errno::EACCES: Permission denied @ unlink_internal - /usr/local/lib/ruby/gems/2.1.0/cache/jekyll-watch-1.3.1.gem
  /usr/local/lib/ruby/2.1.0/fileutils.rb:1454:in `unlink'
  /usr/local/lib/ruby/2.1.0/fileutils.rb:1454:in `block in remove_file'
  /usr/local/lib/ruby/2.1.0/fileutils.rb:1459:in `platform_support'
  /usr/local/lib/ruby/2.1.0/fileutils.rb:1453:in `remove_file'
  /usr/local/lib/ruby/2.1.0/fileutils.rb:791:in `remove_file'
  /usr/local/lib/ruby/2.1.0/fileutils.rb:569:in `block in rm'
  /usr/local/lib/ruby/2.1.0/fileutils.rb:568:in `each'
  /usr/local/lib/ruby/2.1.0/fileutils.rb:568:in `rm'
  /usr/local/lib/ruby/gems/2.1.0/gems/bundler-1.11.2/lib/bundler/runtime.rb:192:in `block in clean'
  /usr/local/lib/ruby/gems/2.1.0/gems/bundler-1.11.2/lib/bundler/runtime.rb:192:in `each'
  /usr/local/lib/ruby/gems/2.1.0/gems/bundler-1.11.2/lib/bundler/runtime.rb:192:in `clean'
  /usr/local/lib/ruby/gems/2.1.0/gems/bundler-1.11.2/lib/bundler/cli/clean.rb:11:in `run'
  /usr/local/lib/ruby/gems/2.1.0/gems/bundler-1.11.2/lib/bundler/cli.rb:384:in `clean'
  /usr/local/lib/ruby/gems/2.1.0/gems/bundler-1.11.2/lib/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
  /usr/local/lib/ruby/gems/2.1.0/gems/bundler-1.11.2/lib/bundler/vendor/thor/lib/thor/invocation.rb:126:in `invoke_command'
  /usr/local/lib/ruby/gems/2.1.0/gems/bundler-1.11.2/lib/bundler/vendor/thor/lib/thor.rb:359:in `dispatch'
  /usr/local/lib/ruby/gems/2.1.0/gems/bundler-1.11.2/lib/bundler/vendor/thor/lib/thor/base.rb:440:in `start'
  /usr/local/lib/ruby/gems/2.1.0/gems/bundler-1.11.2/lib/bundler/cli.rb:10:in `start'
  /usr/local/lib/ruby/gems/2.1.0/gems/bundler-1.11.2/exe/bundle:19:in `block in <top (required)>'
  /usr/local/lib/ruby/gems/2.1.0/gems/bundler-1.11.2/lib/bundler/friendly_errors.rb:7:in `with_friendly_errors'
  /usr/local/lib/ruby/gems/2.1.0/gems/bundler-1.11.2/exe/bundle:17:in `<top (required)>'
  /usr/local/bin/bundle:23:in `load'
  /usr/local/bin/bundle:23:in `<main>'

Environment
Bundler   1.11.2
Rubygems  2.2.2
Ruby      2.1.2p95 (2014-05-08 revision 45877) [x86_64-linux]
GEM_HOME  /usr/local/lib/ruby/gems/2.1.0
GEM_PATH  /usr/local/lib/ruby/gems/2.1.0:/home/alisson/.gem/ruby/2.1.0
Git       2.7.4

I believe this is fixed by #4173; try upgrading to the latest Bundler.


Generate static binaries with musl

I downloaded the latest release and got GLIBC errors on an old ubuntu box:
❯ spacer
spacer: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by spacer)
spacer: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by spacer)
spacer: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by spacer)

So I cloned the repo & built from source using musl - which worked! 🎉

Thank you for your contribution! :)


Empty dropdown showing while async search

Version

Vue version: 3

Description
I use layout that have separated dropdown block (margin-top).
So, if I search something with closeOnSearch="true" I have ugly empty div.
Is it possible to hide it while search is going?
On default multiselect styles (even tailwind) that div is showing too - it have ~2px, but I can see it.
By the way - version 2.1.0 and lower does not have this problem.

Jsfiddle:
I made practical example with big border-width to show you that problem (start typing in search box).
https://jsfiddle.net/rom4i/evLky132/

Thanks for reporting. Fixed in 2.2.0.


Avoid appending to url in a loop

If multiple tokens are use we need to start with the original api url
rather than appending to the previously modified one.

have to clarify that with the company. vacation season ...

got the ok, cla accepted

So, did this get deployed yet?

My apologies @lnussel. Yes, this was deployed on August 28th.


major updates to tomography functionality

first pass as making python3 compatible (tomopy dependents seems to conflict with other dependents in python3 - will need some time to work out)

file by file major updates:

plugins/diFFit/XRD1Dviewer.py
--> define set of default background options (with reset button)
plugins/diFFit/XRD2Dviewer.py
--> fixed clickability on images (shows correlating lines correctly)
--> fixed image loading for 2DXRD mapfiles (tells error is no 2DXRD data)
--> fixed units for sending data properly from 2DXRD viewer to 1DXRD viewer
plugins/tomo/tomography.py
--> combine algorithm parameters into single list
--> fixed orientation assignment in reshape_sinogram
plugins/wx/mapimageframe.py
--> removed superfluous comments/variables
plugins/wx/mapviewer.py
--> removed superfluous comments/variables
--> made dt-correction and hot-columns global options in drop down menu
--> added option for viewing XRD data with selected areas in drop down menu
--> added saving of tomography data to hdf5file
--> made area stats show upon opening file
--> fixed erase flag when prompted yes/no to delete area (previously always True)
--> lasso works to show spectra for reconstructed tomograph areas
plugins/wx/tomoimageframe.py
--> brand new frame create for viewing sinogram and tomography in same window
--> lasso works to show spectra for reconstructed tomograph areas
plugins/xrd/xrd.py
--> reads xrd files of various types (netcdf, tiff, ...) for creating xrd1D dataset
plugins/xrd/xrd_pyFAI.py
--> subtract dark file from images when building dataset
plugins/xrmmap/xrm_mapfile.py
--> added saving of tomography data to hdf5file
--> reworked how detector lists and roi lists are generated
--> update how xrd,xrf flags are read/defined
--> utilized new version_ge function
--> added functionality for retrieving tomography roi/data

Great!  let's go for it and see what might break...


SilkUpdateRequestProcessor

As part of the index chain it would be useful to have an UpdateRequestProcessor that manage Silk [1]  and provides interlinking services to the incoming data

[1] http://silk-framework.com/

Renamed in "SilkRequestHandler" as it seems it doesn't make sense to run the interlinking process for each triple. That process should be executed on top of a ready dataset and not while indexing. So I think a good way to embed this functionality would be a dedicated RequestHandler that triggers the interlinking process, after some data have been properly indexed.


Incorrect behaviour of horizontal scroll bar in results grid

System information:

Win 7 Enterprise SP1
DBeaver 5.2.5 + 6.0.1
Bundled Java version
No additional extensions

Connection specification:

Oracle 12
OJDBC
Do you use tunnels or proxies (SSH, SOCKS, etc)? No.

Describe the problem you're observing:
I am trying to scroll horizontally through a result list. I want to do this a 'page' at a time rather than one column at a time. This does not seem to work, as the horizontal scrollbar seems not to adhere to standard behaviour.
Steps to reproduce, if exist:
When clicking in the white space in the horizontal area, to either side of the scroll bar, the results grid shifts only one column to the left or right. The same behaviour, in other words, as when clicking on either arrow at the end of the scroll bar.
What I think should happen is that the results grid should shift by as many columns as are visible to the user.
Include any warning/errors/backtraces from the logs

I agree. It is unexpected behavior!

I would like to add that on macOS horizontal scrolling seems broken too.

It also seems as if you cannot scroll partially through a column. This is very annoying for columns with very long values as I can't see the rest of the value if it's wider than the screen because it scrolls completely past the column to the next one.

DBeaver 7.2.1 - via software centre snap
Ubuntu 20.04
I'm noticing a similar problem, not sure if it's the same. Essentially, Shift + horizontal scroll either moves one column from it's current view, or it allows normal scrolling for a partial view of the results e.g. it will scroll fine between 20% and 85% of the columns. If I repeatedly attempt to scroll back and forth, this "good" scroll region changes (reduces, but not evenly) and eventually doesn't work at all.
This seems to work a little better if the application window is smaller, but snapped to the left half of my screen it's almost useless.

will be implemented in #10184


Send email via Sendgrid on win

I think the next think I'm going to do is update it to send an email on a win. I think the easiest way to do to this is to use send grid. Anyone can sign up for a free account, so I can allow setting the  SENDGRID_API_KEY in the .env file, and if it exists, use sendgrid to shoot off an email to the email address used to log in when an entry wins.
I know I could use nodemailer, but then I'd have to ask for user's to put a bunch of stuff in .env, seems like a pain all around.
I really want to make it easier to not miss wins, so this will be my next priority.
Originally posted by @jpchip in https://github.com/jpchip/giveaway-grabber/issues/32#issuecomment-478196456

As requested, allow specify additional emails to be cc'ed.

@famewolf Latest release supports this, and supports a cc email.

I guess now I need to ask if the send email feature can handle if the internet is down when it first attempts to send the email?  Will it retry after x minutes upon failure?


google-api-python-client is in maintenance only mode

The old Google client libraries (google-api-python-client) are now in maintenance only mode. They are superseded by google-cloud-python.
Most of the notebooks need to be updated to be kept up-to-date.

I just went through and updated notebooks/Creating TCGA cohorts -- part 1.ipynb myself and can share some pointers that might be helpful to you. Unfortunately, it was not as straightforward as I had hoped.
My updated version of that notebook is here and includes all 3 types of changes described below.
There are basically 3 things that need to be updated:

As mentioned, the new client libraries are under the umbrella of google-cloud-python and versions past v0.28 differ in a variety of ways from their forebears. The latest release notes for google-cloud-bigquery v0.30 indicate that it is a v1.0 release candidate, so I suspect it is reasonable to develop against the API in v0.30, though waiting for v1.0 is also a reasonable thing to do. In any case, there is a good client migration guide that summarizes the changes.
There is a new version of the Datalab utilities package which provide the %sql and %bigquery magic commands. Those commands are now deprecated and superseded by the %bq magic command. There is a migration guide for this as well, and I found the new documentation to be a helpful reference.
Additionally, the notebooks currently use a legacy SQL dialect. Newer BigQuery clients now default to a new dialect called "Standard SQL" (though you can still override this and use legacy SQL). There is also a migration guide for Standard SQL, though I suspect Legacy SQL will be supported for some time if you chose to keep it.

Happy to answer specific questions about the updated notebook as well (esp. since notebooks are hard to diff).

thank you @pumpikano!  after a long delay we are finally getting back to using Datalab and will be updating our notebooks over the next couple of weeks


[1.16.4] Culling sounds in sndctrl-client.toml disables them entirely, rather than reducing the repetition.

Mod Version:
1.16.4-<IP_ADDRESS>
Forge Version:
35.1.36
Link to client log:
https://gist.github.com/Vaelzan/5f288c7c0fe7556ad8ef0d14ed4b4ff3
Link to crash log:
N/A, not a crash.
Description:
Enabling culling for any sound disables it entirely (the subtitle still shows, however), while the description of sound culling in the 1.14/1.15 version indicates that it should cut down on how often the sounds play rather than disable them entirely (and there is no description of sound culling at all in the 1.16.4 version). From my understanding it should be the "blocking" setting that does this, not "culling".
This is in Valhelsia 3 (my development version, but the problem occurs in 3.1.5), and you can see the configs that I'm testing with in our GitHub repository here: https://github.com/ValhelsiaTeam/Valhelsia/tree/1.16/config/dsurround - the configs have several audio effects disabled from mobeffects-client and the fog from environs-client,
You can see that I recently disabled culling there since people complained about animals no longer making noises at all, and I was able to verify that. During testing I was turning culling on and off using the Ctrl-S config menu to verify that the sounds weren't playing at all.
If required I'm happy to turn on debug logging and do further testing, but hopefully you're able to reproduce the issue and diagnose the problem yourself.
As an aside, and unrelated entirely to this issue, the reason we disabled some of the audio is due to the audio being quite literally painful to a person with an auditory processing disorder due to some of the high-pitched frequencies used. We would gladly enable these settings again if that was resolved somehow (notably the footsteps with armour rustling). It is also a touch unintuitive that the footsteps don't play at the same volume as vanilla Minecraft footsteps and instead have their own volume slider (instead of using the existing ones), since they were initially "headphone warning required" loud in my experience with default configs.

I was suspecting something like this.

I was suspecting something like this.

Pushed <IP_ADDRESS> to CurseForge with a fix for volume calcs.

Pushed <IP_ADDRESS> to CurseForge with a fix for volume calcs.


refactor: misc refactors and test updates

Introduction :pencil2:
Misc refactors to address comments on review here which tracks merge of branch main-stbr-tfm-5 into main.

Codecov Report
Attention: 206 lines in your changes are missing coverage. Please review.

Comparison is base (91637c1) 17.31% compared to head (ef768fa) 13.46%.
Report is 225 commits behind head on main-stbr-tfm-5.

Files
Patch %
Lines

...ervice/put-utilisation-report-status.controller.ts
0.00%
38 Missing and 3 partials :warning:

dtfs-central-api/src/drivers/db-client.ts
37.50%
11 Missing and 4 partials :warning:

...al-api/src/v1/routes/utilisation-reports-routes.js
0.00%
14 Missing :warning:

...rs/tfm/amendments/tfm-get-amendments.controller.js
0.00%
10 Missing :warning:

...pi/src/v1/controllers/bank/get-banks.controller.js
0.00%
7 Missing :warning:

...c/v1/controllers/tfm/users/tfm-users.controller.js
0.00%
7 Missing :warning:

dtfs-central-api/src/generateApp.js
0.00%
5 Missing :warning:

...c/v1/controllers/tfm/users/tfm-teams.controller.js
0.00%
5 Missing :warning:

...ports-reconciliation-summary.controller/helpers.ts
91.37%
5 Missing :warning:

...src/services/repositories/utilisation-data-repo.ts
75.00%
2 Missing and 2 partials :warning:

... and 43 more

Additional details and impacted files
@@                 Coverage Diff                 @@
##           main-stbr-tfm-5    #2599      +/-   ##
===================================================
- Coverage            17.31%   13.46%   -3.86%
===================================================
  Files                 1120      504     -616
  Lines                25591    10480   -15111
  Branches              4101     2116    -1985
===================================================
- Hits                  4432     1411    -3021
+ Misses               18423     7506   -10917
+ Partials              2736     1563    -1173

Flag
Coverage Δ

api-test
0.48% <ø> (ø)

ui-test
?

unit-test
15.55% <52.86%> (+7.59%)
:arrow_up:

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

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


[SPARK-28655] [patch] Support to cut the event log, and solve the history server was too slow when event log is too large.

…slow when open the application and event log is too large, and add event log split with the config spark.eventLog.cuttype
What changes were proposed in this pull request?
Support to cut the event log, and solve the history server was too slow when event log is too large. When the cuttype is configed, the EventLoggingListener will check the logpath, if the logpath is changed, the eventlog writer will be created when all jobs are end. And there is a write/read lock to ensure the multiprocess safe.
Why are the changes needed?
When the eventlog is very large, the history server will be oom.
How was this patch tested?
manual tests

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Hi, @shaohk . Thank you for making PRs again. But, SPARK-28655 is an improvement PR and Apache Spark don't allow new feature backporting. So, an improvement PR to non-master branch is invalid from the beginning. Like your previous PR, I need to close this PR, https://github.com/apache/spark/pull/25387 .
You can open this to master branch. BTW, the new PR description should answer @srowen 's question when you reopen this to master.

This isn't opened vs master.  Why is this needed?

thank u very much

Hi, @shaohk . Thank you for making PRs again. But, SPARK-28655 is an improvement PR and Apache Spark don't allow new feature backporting. So, an improvement PR to non-master branch is invalid from the beginning. Like your previous PR, I need to close this PR, #25387 .
You can open this to master branch. BTW, the new PR description should answer @srowen 's question when you reopen this to master.

This isn't opened vs master.  Why is this needed?

thank u very much


BigQuery Anonymous Authentication

Description
Adds the option to authenticate to BigQuery using anonymous credentials. Useful for public datasets and unit testing. Will help solve for https://github.com/dbt-labs/dbt-bigquery/issues/358

Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

Thanks for opening the PR, @kpolley. The change looks good. Do you think we could add a small unit test - in either conftest or test_bigquery_adapter?

Added a unit test in test_bigquery_adapter. Let me know if there's anything else and feel free to modify the PR as well.


[Spark] Minor Refactor Spark Commands

Purpose

Linked issue: close #xxx

Tests

API and Format

Documentation

@Zouxxyy can you take a look?

+1


Defining column_name in YAML for generic data tests is only for advanced use cases

Contributions

[X] I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates
https://docs.getdbt.com/reference/resource-properties/data-tests#test-an-expression
What part(s) of the page would you like to see updated?
It looks to me like the code example here https://github.com/dbt-labs/dbt-core/issues/10966#issuecomment-2461275071 lead to some confusion.
I don't know off the top of my head how we could/should refactor it, but we should give it a look if we can.
Currently, this code example seems to be trying to cover two things at the same time:

Using a SQL expression within a data test
Apply the data test to a model instead of a column

So maybe things will be less confusing if those two things are split into their own separate code examples?
It might be helpful to use a different example for SQL expressions. One idea is expression_is_true from dbt-utils:
Test an expression
models:
  - name: model_name
    tests:
      - dbt_utils.expression_is_true:
          expression: "col_a + col_b = total"

Another example:
models:
  - name: model_name
    tests:
      - dbt_utils.expression_is_true:
          expression: "delivery_date >= order_date"

Something to keep in mind is that this looks like it originally pointed to this example:
models:
  - name: orders
    tests:
      - unique:
          column_name: "country_code || '-' || order_id"

Additional information
No response

+1 for the improvement here


Too much macro dependence?

I like Turing.jl, I really do. But one thing that seems to get in the way all of the time is the @model macro. DSLs are usually constructed as a way to constrain the possible inputs so that a compilation process can be written on a simplified form. Example: Stan wrote its own derivatives for every term for its AD, so it's constrained to an interface where you can use only the terms it defined. For JuMP, it needs to know what terms are linear, quadratic, integer, etc. in order to specialize, so it has a DSL that specifically captures that information.
Using DSLs isn't that great in many circumstances because, well, you might not have a full programming language available to you. But even more importantly, in many cases you have to write into a language, and thus a script needs to be built at compile time. Again, Stan you build a whole program as a string.
Turing.jl sits in an odd location of design space because it has a DSL-based interface... but it doesn't need to. Turing.jl is built on things like ForwardDiff.jl and Tracker.jl which are language-wide AD systems, so in theory any Julia code could work. In practice, things that are definable in the macro are what is allowed. @model is very good with allowing arbitrary Julia functions, but it still has the issue that, as a macro, it is evaluated at compile time. So like Stan, if you want to programmatically create models, you have to interpolate into a compile time script and then run the compiler. This goes unnoticed in a lot of Turing.jl usage because a lot of users are writing models in the global scope, in which case there's implicitly an eval happening after each command.
However, this gets tricky when defining a Turing model in a function because... functions run at runtime and not at compile time, so the macro is expanded before the function values are known. So for example, let's look at the DiffEqBayes.jl integration with the ODE solvers. Let's say we had a list of variables that we want to be named syms in the output. In theory we could do
syms = [:a,:b]
function (syms,priors,....)
    @model bif(x) = begin
    for i in 1:length(priors)
      syms[i] ~ priors[i]
    end
    ...

but that would make a bunch of variables named syms, not a variable named the symbol value that is in syms[i], so this is distinctly different from writing a ~ priors[1]. If you want the output to have named chains like [:a,:b], you could construct an expression for the @model, interpolate
$(syms[i]) ~ priors[i]

and then eval the expression, but I think it's clear that is showing that the model isn't truly a macro.
At its core, the issue is that a model isn't actually a macro because it doesn't actually have to use compile-time information: the model is actually a function, and the macro is just a nice way to construct it. The simplest solution of course is to document the internals, like in https://turing.ml/docs/advanced/ , but I find it interesting that there are no test cases and tutorials that show how to use this, given that this is what you'd need to do if you don't want the whole structure defined at compile time.
With https://github.com/TuringLang/Turing.jl/pull/819 and reconstructing the output chain https://github.com/TuringLang/MCMCChains.jl#parameter-names with new names I can probably hack my way around to get the output I want without resorting to eval and using what's documented/tested/expected to be used, but I think this is something to think about in future tutorials.

Two quick comments before I go to sleep:

Most of the changes in #81 were changes in normal functions, not the macro.
Having the variable symbols be runtime information provided by the user maybe possible but I will need to think about it some more.

@ChrisRackauckas We'll try to reduce macro dependency in v0.8, where we'll redesign the DSL implementation.

@ChrisRackauckas I am working now on #965 which uses much less macro magic and enables the use of user-defined variable names using the syntax:
x ~ NamedDist(dist, name)

where name could be a string but even better (for performance) a Turing.VarName constructed using Turing.VarName{:y}("[1]") for example. In case of a vector of variables, the Vector{<:VarName} can be built before defining the model and just used inside the @model body to enable type stable sampling.

I believe this issue was solved by #965. The variable names are now not baked into the model and NamedDist can be used to define custom random variable names. The model macro was also significantly simplified in #965. Please re-open if you feel this issue wasn't properly addressed in #965.


Subscribe Provider to Eigenstate rather than using it as a store

The current model assumes that React is synchronous, which it is not. This could lead to subtle inconsistencies if React's batching causes it to behave unpredictably with respect to state.

Still no explicit subscription, but this is effectively now the case.


Improve performance of COMBINE API for BiGG models

[x] Speed up JSON encoding. Done in #4288.
[x] Extract specifications of manifest from manifest file (in S3) rather than COMBINE archive file: ~0.3 s. Done in #4285
[ ] Speed up extraction of metadata: ~0.5s for iAF1260

Read from individual S3 files rather than COMBINE archive; requires passing manifest or path to manifest

[ ] Speed up extraction of SED-ML specifications: 2-5s (mostly content download)

Read from individual S3 files rather than COMBINE archive; requires passing manifest or path to manifest. At least for modest size archives, downloading the archive appears to be a small part of the overall duraton.

[ ] Allocate more memory CPU to COMBINE API

I ran SED-ML extraction for all 92 BiGG models. The average (using https://combine.api.biosimulations.dev) was 3.3 s, ranging from 0.2-8.0 s. This includes the time to transfer the result to my machine. It should be less from the COMBINE API to the dispatch service. For localhost to localhost I get 0.1-1.4 s (avg 0.9).
The core SED-ML extraction is only around 0.2 s on my machine (max ~0.4 s). Half of that is libSED-ML.

@bilalshaikh42 I looked into the performance of the COMBINE API SED-ML extraction. There doesn't seem to be anything obvious to improve with the code. One other thought is to increase the CPU assigned to the COMBINE API (e.g., the deployment is much slower than my local machine).
https://github.com/biosimulations/deployment/blob/aa2ba78a1484e06728c97b40654ab06cf8968b77/base/apps/combine-api.yaml#L24

I am not seeing the cpu usage for the combine-api come anywhere near  the limit, so that won't make much difference. The only other reason that I can think of is that it has something to do with the gunicorn wrapper

My recollection is that you mentioned that the combine api is going down. It seems that the CPU time and load are fine.
I could take another look at the gunicorn settings. However, I'll be grasping at straws. The settings aren't particularly clear.
Is there any log information that is illuminating?


use base64 instead of iso-8859-1

gzip make byte array, and transfer it to string with base64 compress method. ISO-8859-1 string is right but not good.

Instead of relying on the non-portable sun.misc package, I suggest using guava's BaseEncoding.
You may have to upgrade guava's version in subprojects.gradle .

thank you , I upgrade guava version and use the base64 method instead of sun version.


Correct link to Subscriptions API

The old link was pointing to the 4.1 version which apparently doesn't actually support the Subscriptions API. If you clicked on that link, you were automatically redirected to the 5.0 page (with a message "The requested page is not available for vsts-rest-4.1. You have been redirected to the newest product version this page is available for.")

@flcdrg : Thanks for your contribution! The author, @elbatk, has been notified to review your proposed change.


Fixes - Delete of folders which are empty

Carry forward changes made to sub fork.

@ltouati
can you please review PR?

@ltouati are you interested in considering any of the PRs that have been raised recently?  Or should we consider this repository to be abandoned, and devolve our ongoing improvements entirely from it?

Hi Oliver,
Sorry for the delay. Yes I'll merge your PR next week
L.
Le ven. 5 juin 2020 à 09:44, Oliver Lockwood<EMAIL_ADDRESS>a
écrit :

@ltouati https://github.com/ltouati are you interested in considering
any of the PRs that have been raised recently? Or should we consider this
repository to be abandoned, and devolve our ongoing improvements entirely
from it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ltouati/vfs-gcs/pull/14#issuecomment-639315646, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAENB4HEGK4REYPU7CX55ADRVCO55ANCNFSM4JYYUR4A
.


Pagination issue for getUserPlaylists()

Spotify, by default, limits the length of the playlist array to 20. Using getUserPlaylists() function only returns 20 playlists.

Welp, figured it out 5 minutes after typing the above.
Do something like this.
spotifyApi.getUserPlaylists({ limit: 50, offset: 0 })

Then you can write logic to increase the offset by 50 everytime until you reach the end of the playlists. The limit max is 50 tho.

Ah ok. This is really helpful! Thanks a lot!

@simondebbarma thanks!


Added slack settings OSSK-382

Describe changes
Closes OSSK-382
Pre-requisites
Please ensure you have done the following:

[ ] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] I have based my new branch on develop and the open PR is targeting develop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.
[ ] If my changes require changes to the dashboard, these changes are communicated/requested.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

@coderabbit review

@coderabbitai review

@avishniakov @strickvl I had to revert the multi-inheritenance of the Config because of some linting issues, so I used slack_channel_id in settings and default_slack_channel_id in the config. I know its a bit weird but I think it somehow makes sense in this case. Let me know if you disagree


Generate ECS/TCR comparisons

cal-cmip6-abrupt0p5xCO2-ex.ipynb with the abrupt-0p5xCO2 experiment is missing the ECS/TCR comparisons to various ESMs, GSMs, AOGCMs CMIP6 tests results using Gregory Method, among possible others.

Added #8, Added CMIP6 files from UoMelbourne, created cal-cmip6-abrubt notebook…
to graph CMIP abrupt output.
Changes made to: https://github.com/hsbay/cdrmex/commits/main/cal-cmip6-abrupt0p5xCO2-ex.ipynb


BUG: WinLeave/BufLeave are not fired on closure of Neo-tree buffer

Did you check docs and existing issues?

[X] I have read all the docs.
[X] I have searched the existing issues.
[X] I have searched the existing discussions.

Neovim Version (nvim -v)
NVIM v0.10.0-dev-ebe489d
Operating System / Version
Arch Linux
Describe the Bug
When I close the Neo-tree window (with :bd), none of the Window/Buffer closure events are fired. I suspect this has something to do with the focus stealer but I am not really sure.
Screenshots, Traceback
No response
Steps to Reproduce

Load minimal configuration (nvim --clean -u repro.lua)
Execute :Neotree command
Execute :bd command
See nothing print out on closure of Neo-tree buffer

Expected Behavior
WinLeave/BufLeave events are properly fired so they can be captured in other plugins
Your Configuration
-- Minimal configuration
-- mini.lua
-- Use with the --clean -u flags. EG nvim --clean -u mini.lua
-- This config will create a temp directory and will blow away that temp directory
-- everytime this configuration is loaded. Great for simulating a new installation
-- of a plugin

-- Setting some basic vim options
-- Some junk because I am sick of formatting tables in print
local _print = _G.print
local clean_string = function(...)
    local args = { n = select("#", ...), ... }
    local formatted_args = {}
    for i=1, args.n do
        local item = select(i, ...)
        if not item then item = 'nil' end
        local t_type = type(item)
        if t_type == 'table' or t_type == 'function' or t_type == 'userdata' then
            item = vim.inspect(item)
        end
        table.insert(formatted_args, item)
    end
    return table.concat(formatted_args, ' ')
end
_G.print = function(...)
    _print(clean_string(...))
end

vim.opt.mouse = 'a'
vim.opt.termguicolors = true
-- If you want to play around with this, you can set the do_clean
-- variable to false. This will allow changes made to
-- underlying plugins to persist between sessions, while
-- still keeping everything in its own directory so
-- as to not affect your existing neovim installation.
--
-- Setting this to true will result in a fresh clone of
-- all modules
local root = vim.fn.fnamemodify("./.repro", ":p")
-- DO NOT change the paths and don't remove the colorscheme

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
    vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
    vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
    "folke/tokyonight.nvim",
    -- add any other plugins here
}

local neotree_config = {
    "nvim-neo-tree/neo-tree.nvim",
    dependencies = { "MunifTanjim/nui.nvim", "nvim-tree/nvim-web-devicons", "nvim-lua/plenary.nvim" },
    cmd = { "Neotree" },
    keys = {
        { "<Leader>e", "<Cmd>Neotree<CR>" }, -- change or remove this line if relevant.
    },
    opts = {
        -- Your config here
        -- ...
    },
}

vim.api.nvim_create_autocmd({'WinEnter', 'BufWinEnter', 'BufEnter'}, {
    callback = function(details)
        print(string.format("%s on buffer %s", details.event, details.buf))
    end
})

table.insert(plugins, neotree_config)
require("lazy").setup(plugins, {
    root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

Related:

#571

Its an older issue so instead of necroing it, I created a reproducible configuration and created a new issue

I am also facing this issue. Are there any workarounds or fixes for this?

I don't know how to fix this. Let me dig in deeper.

It might have missing nested = true flag to allow other events to get triggered when callback from this one is being executed.

I don't know how to fix this. Let me dig in deeper.

It might have missing nested = true flag to allow other events to get triggered when callback from this one is being executed.

Bingo! Thanks @echasnovski
Looks like we both forgot nested so it was hard to debug lol

Merged upstream.
@echasnovski Could you check out the latest nui and test if the issue with mini.statusline is fixed?

Yes, it does work with current main branch of 'mini.nvim'. There were also some recent changes (two days ago, it seems) on 'mini.statusline' end that made it all work.

Glad to hear that. Thanks for having your time @echasnovski !!
Great work BTW ;)


Add support for cookie parameter in OpenAPI 3.0

Resolves https://github.com/kiwicom/schemathesis/issues/21

Codecov Report

Merging #67 into master will decrease coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #67      +/-   ##
==========================================
- Coverage   99.66%   99.63%   -0.04%
==========================================
  Files          10        9       -1
  Lines         299      273      -26
  Branches       41       37       -4
==========================================
- Hits          298      272      -26
  Partials        1        1

Flag
Coverage Δ

#unittests
99.63% <100%> (-0.04%)
:arrow_down:

Impacted Files
Coverage Δ

src/schemathesis/types.py
100% <100%> (ø)
:arrow_up:

src/schemathesis/models.py
100% <100%> (ø)
:arrow_up:

src/schemathesis/_hypothesis.py
100% <100%> (ø)
:arrow_up:

src/schemathesis/schemas.py
99.14% <100%> (+0.03%)
:arrow_up:

src/schemathesis/filters.py
100% <0%> (ø)
:arrow_up:

src/schemathesis/loaders.py
100% <0%> (ø)
:arrow_up:

src/schemathesis/lazy.py

Continue to review full report at Codecov.

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

Great job! Thank you! :) Could you, please, add an entry to the changelog.rst file?

Great job! Thank you! :) Could you, please, add an entry to the changelog.rst file?

Should have read contributing guide properly. Fixed.

Thank you very much!


Bug: USER_CANCELLED response on Android and IOS

Capacitor version:

Run npx cap doctor:
npx cap doctor

   Capacitor Doctor

Latest Dependencies:

  @capacitor/cli: 3.2.4
  @capacitor/core: 3.2.4
  @capacitor/android: 3.2.4
  @capacitor/ios: 3.2.4

Installed Dependencies:

  @capacitor/android: 3.2.0
  @capacitor/core: 3.2.0
  @capacitor/cli: 3.2.0
  @capacitor/ios: 3.2.0

[success] Android looking great! �
[error] Xcode is not installed
npm notice
npm notice New major version of npm available! 7.22.0 -> 8.0.0
npm notice Changelog: https://github.com/npm/cli/releases/tag/v8.0.0
npm notice Run npm install -g<EMAIL_ADDRESS>to update!
npm notice

Library version:

3.0.1

OAuth Provider:

Azure AD

Your Plugin Configuration

{
	appId: 'xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx',
	authorizationBaseUrl: 'https://login.microsoftonline.com/xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx/oauth2/v2.0/authorize',
	accessTokenEndpoint: 'https://login.microsoftonline.com/xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx/oauth2/v2.0/token',
	responseType: 'code',
	pkceEnabled: true,
	logsEnabled: true,
	scope: 'https://xxxx.onmicrosoft.com/xxxxxx/tasks.read https://xxxx.onmicrosoft.com/xxxxxx/tasks.write',
	web: {
		redirectUrl: 'http://localhost:8100/',
		windowOptions: 'width=642,height=586,location=off,scrollbars=on'
	},
	android: {
		redirectUrl: 'msauth://id.package.my/xxxxxxxxxxxxxxxxxxxxxxxxxxxx' // URL encoded signature hash
	},
	ios: {
		pkceEnabled: true, // workaround for bug #111
    	redirectUrl: 'msauth.id.package.my://auth'
	}
}

Affected Platform(s):

Android
Android version: 8.0.0. Device model: SM-G935F

Current Behavior

This is an Ionic 5 app.
On Windows/Web (Chrome browser), we are able to log in as expected.
On Android we get plugin response USER_CANCELLED.
What is different on Android compared to Web is that on the login.microsoftonline.com browser login page we are shown a second screen (after logging in) saying:

Are you trying to sign in to XXXXX?

Where XXXXX is the name of our SPA/Android app registration in Azure. Beneath are two buttons: Cancel and Continue. Selecting Continue (or Cancel) does the same thing, returns to the app and displays an error to the app/webview console:
Error: USER_CANCELLED at Object.cap.fromNative (http://localhost/:432.32) at <anonymous>:1:18.
The plugin doesn't seem to log any information like I usually see on the web version (i.e. info prefixed with I/Capacitor/OAuth2ClientPlugin:)
When I check the Azure AD logs, I see an authentication Failure followed by an immediate Success. The failure details includes:
Sign-in error code
50199
Failure reason
For security reasons, user confirmation is required for this request. Please repeat the request allowing user interaction.
Additional Details
MFA requirement satisfied by claim in the token

My understanding is that the Failure/Success happens whilst on the Azure login pages. The Failure triggers the display of the Are you trying to sign in to XXXXX? message, which then, after selecting Continue causes the Success message. But I may be wrong on this.
The error code details for the above 50199 error can be found here. The details being:

AADSTS50199 | CmsiInterrupt - For security reasons, user confirmation is required for this request. Because this is an "interaction_required" error, the client should do interactive auth. This occurs because a system webview has been used to request a token for a native application - the user must be prompted to ask if this was actually the app they meant to sign into. To avoid this prompt, the redirect URI should be part of the following safe list:http://https://msauth://(iOS only)msauthv2://(iOS only)chrome-extension:// (desktop Chrome browser only)

My understanding is that msauth Android is being called by a webview, and because it doesn't trust the webview, it requests the extra verification from the user that the app they are logging into is the one they intend to log into. (related comment).
In addition to the above configuration, I have tried each permutation of handleResultOnNewIntent and handleResultOnActivityResult. All yield the same result, except for:
handleResultOnNewIntent: true,
handleResultOnActivityResult: true

Which displays to the console:
Error: ERR_GENERAL at Object.cap.fromNative (http://localhost:432:32) at <anonymous>:1:18. Again, there is no other info logged to the console.
I've tried every possible fix as specified in the plugin documentation as well as fixes discussed in the issues repo section. Nothing has worked.
Expected Behavior
After logging in on Android, I expect the tokens to be returned to the app/webview so that the app can call the API using Bearer auth.
I noticed in one of the issues that someone else got this plugin working with Ionic, so I'm hoping Ionic isn't a show stopper and that we can resolve this issue.
Thanks!

By way of an update. I have set up and tested the Apple/IOS version of the Ionic app. I get the same set of screens described above. The only differences are:

After clicking "Continue" on the Azure auth popup, we get a popup message saying: "Safari cannot open the page because the address is invalid."

The error message we see is: Error: USER_CANCELLED: CapacitorException@capacitor://localhost/vendor.js:94505:14 user-script:2:424:49
I would greatly appreciate a response. Even if it's to say that you do not have time to look at this.
Many thanks!

Hi there. I can see that this was assigned to Android, but the issue is also happening on IOS. Could you please give an indication when you might be able to look at this? Many thanks.

I can confirm I'm getting the same issue on Android. However, iOS appears to be working.
I cannot find a work around for Anddroid. This is a bit of a show stopper!

Same here

I have the same issue with Azure provider. Any insights?

I have the same issue with the 3.0.1 version. Any solution or workaround for this issue as temp solution?
OAuth rejected Error: USER_CANCELLED
Thanks in advance

So it turns out the USER_CANCELLED response was caused by a different configuration issue on each environment, which coincidentally caused the same response to be returned. I had mistakenly thought the issue was in the plugin itself, i.e. that it wasn't able to handle Azure AD's CmsiInterrupt flow, but it turns out that was incorrect.
The Android issue was that a second Activity element ("net.openid.appauth.RedirectUriReceiverActivity") was required in AndroidManifest.xml file. We only had a single Activity element ("com.company.project.MainActivity"), and we were missing the android:host parameter too. The readme documentation has been updated to explain it here: https://github.com/moberwasserlechner/capacitor-oauth2#android-1.
The IOS issue was that the CFBundleURLSchemes string was incorrectly set to com.yourcompany.yourproject rather than msauth.com.yourcompany.yourproject (i.e. it was missing the msauth. prefix). The readme documentation explains it here: https://github.com/moberwasserlechner/capacitor-oauth2#ios.
One other thing to note was that we did not set the resourceUrl as our Azure SPA app registration referenced a second Azure API app registration because it needed to obtain additional/custom scopes. The readme documentation has been updated to explain it here: https://github.com/moberwasserlechner/capacitor-oauth2#custom-scopes.
Thanks to the plugin developer @moberwasserlechner for his support in identifying the configuration issues and for updating the Azure documentation section (https://github.com/moberwasserlechner/capacitor-oauth2#azure-b2c) to help users configure their apps correctly in the future.


iOS sounds and the ringer/mute switch

When the ringer/mute switch on a side of an iPhone is enabled, all sounds/music inside a game become silent. There are other games that continue to play sounds in this state.
As I understand, these lines set such behaviour:
https://github.com/libgdx/libgdx/blob/master/backends/gdx-backend-robovm/src/com/badlogic/gdx/backends/iosrobovm/IOSAudio.java#L36
https://github.com/libgdx/libgdx/blob/master/backends/gdx-backend-moe/src/com/badlogic/gdx/backends/iosmoe/IOSAudio.java#L36
Is there any way to make this behaviour configurable?
Please select the affected platforms

[X] iOS (robovm)
[X] iOS (MOE)

Apple guidelines are that you only override the switch if you are playing media, not a game. The whole purpose of the switch is to instantly silence everything. As an iPhone user, I would be very annoyed if a game did this.

Thank you, Mario!


Where can I find out what the number returned means?

I have error 547.  What did I do wrong?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6493c898-6f12-f44f-26df-28864300c101
Version Independent ID: 9c3a6e72-7f63-7e73-722c-433e43715770
Content: ERROR_NUMBER (Transact-SQL) - SQL Server
Content Source: docs/t-sql/functions/error-number-transact-sql.md
Product: sql
Technology: t-sql
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

Closing this now, please comment below if there are further issues.


Crash at compile time

When compiling my program with the bevy_skybox_cubemap dependency, it fails as cargo has a conflict with selecting a version for 'syn'. Note that the package 'goldberg' is used for obfuscating code when compiled and this failure persists without the package installed. This issue is present when compiling on MacOS and Windows.
Here is the crash output:
error: failed to select a version for `syn`.
    ... required by package `goldberg v0.1.0`
    ... which satisfies dependency `goldberg = "^0.1.0"` of package `client v0.1.0 (C:\Users\NullUsxr\source\repos\NullUsxr\project\client)`
versions that meet the requirements `^1.0.91` are: 1.0.100, 1.0.101, 1.0.99, 1.0.98, 1.0.97, 1.0.96, 1.0.95, 1.0.94, 1.0.93, 1.0.92, 1.0.91

all possible versions conflict with previously selected packages.

  previously selected package `syn v1.0.65`
    ... which satisfies dependency `syn = "=1.0.65"` of package `bevy v0.5.0`
    ... which satisfies dependency `bevy = "^0.5.0"` of package `bevy_skybox_cubemap v0.1.0`
    ... which satisfies dependency `bevy_skybox_cubemap = "^0.1.0"` of package `client v0.1.0 (C:\Users\NullUsxr\source\repos\NullUsxr\project\client)`

failed to select a version for `syn` which could resolve this conflict

And here is the crash output without the goldberg package:
error: failed to select a version for `syn`.
    ... required by package `darling_core v0.13.0`
    ... which satisfies dependency `darling_core = "=0.13.0"` of package `darling v0.13.0`
    ... which satisfies dependency `darling = "^0.13"` of package `ndk-macro v0.3.0`
    ... which satisfies dependency `ndk-macro = "^0.3.0"` of package `ndk-glue v0.5.0`
    ... which satisfies dependency `ndk-glue = "^0.5"` of package `bevy_internal v0.8.0`
    ... which satisfies dependency `bevy_internal = "^0.8.0"` of package `bevy v0.8.1`
    ... which satisfies dependency `bevy = "^0.8.1"` of package `client v0.1.0 (C:\Users\NullUsxr\source\repos\NullUsxr\project\client)`
versions that meet the requirements `^1.0.69` are: 1.0.100, 1.0.101, 1.0.99, 1.0.98, 1.0.97, 1.0.96, 1.0.95, 1.0.94, 1.0.93, 1.0.92, 1.0.91, 1.0.90, 1.0.89, 1.0.88, 1.0.87, 1.0.86, 1.0.85, 1.0.84, 1.0.83, 1.0.82, 1.0.81, 1.0.80, 1.0.79, 1.0.78, 1.0.77, 1.0.76, 1.0.75, 1.0.74, 1.0.73, 1.0.72, 1.0.71, 1.0.70, 1.0.69

all possible versions conflict with previously selected packages.

  previously selected package `syn v1.0.65`
    ... which satisfies dependency `syn = "=1.0.65"` of package `bevy v0.5.0`
    ... which satisfies dependency `bevy = "^0.5.0"` of package `bevy_skybox_cubemap v0.1.0`
    ... which satisfies dependency `bevy_skybox_cubemap = "^0.1.0"` of package `client v0.1.0 (C:\Users\NullUsxr\source\repos\NullUsxr\project\client)`

failed to select a version for `syn` which could resolve this conflict

Thanks and have a great day.
~NullUsxr

This library was build for Bevy 0.5.0 and hasn't been updated.
There were significant changes between 0.5.0 and 0.6.0 to the render pipeline.
I spent several hours today trying to update this project to work on Bevy 0.8.1 but I can't figure out how to make the new render pipeline things work. Everything I do just crashes at runtime for mysterious reasons, and the folks on the Bevy discord were not very helpful.
I'm really sorry this won't work for you.
I recommend you look into the Bevy Skybox example which just uses a really big cube with a cubemap texture on it.

As of Bevy 0.11 skyboxes are provided natively in Bevy and I have marked this crate as deprecated in the readme.


Cannot parse @deprecated JSDoc on file with line ending \r\n

The code in ExportMap explicitly requires the line ending to be \n, but it's not always true. On Windows, we have default line ending \r\n.
https://github.com/benmosher/eslint-plugin-import/blob/798eed7e559adab2eac07bf1b3e3518d4b7a4296/src/ExportMap.js#L233
I can send fix for this.

Thanks, that'd be great.


SARGAM, a music streaming website

## Describe the project
SARGAM is a front-end developed music streaming website with different Log In, Sign Up page and all the navigation bar links are functional.
Issue Type:

[ ] New Project

Screenshots

@TusharKesarwani @zugzwang03 Please assign me this project under SSOC 2.0.


[Bug]: --ignore-package <pathOrName> option is not working

related with https://github.com/TanStack/query/pull/6701
with @TkDodo, We just want use --ignore-package <pathOrName> option of sherif that be added by https://github.com/QuiiBz/sherif/issues/41

But we met this error (https://github.com/TanStack/query/pull/6701#issuecomment-1891055388). @QuiiBz Could you check this issue please?

Here's the explanation of the issue, and I've updated the documentation to make it work with any shell: https://github.com/TanStack/query/pull/6701#issuecomment-1903834659


TGRSIDetector::AddFragment

Here are my ideal solutions to handle this.
When one makes a TGRSIDetectorHit, they can make a call to a constructor with a fragment such as
TGriffinHit hit(*frag);

We would then delegate the constructors
TGriffinHit::TGriffinHit(const TFragment &frag) : TGRSIDetectorHit(frag){

    //stuff, and you can even overwrite what the base hit is doing if you really want
}
TGRSIDetectorHit::TGRSIDetectorHit(const TFragment &frag) : fAddress(frag.ChannelAddress) ... etc{
    //stuff
}

The other solution would be to make a call to a TGRSIDetectorHit::SetFragment(const TFragment&) function. And then have something that looks like this:
TGriffin::AddFragment(TFragment *frag){
   TGriffinHit hit;
   hit.SetFragment(frag);
   //Do griffin specific stuff
}

The constructor way seems a lot cleaner to me. Anyone have any issues with this?

The only problem I see with that is that one fragment can contain multiple hits so the constructor would somehow have to create multiple hits and return them?
Adding a SetFragment method seems a better way to go, and I think some of the Tigress detectors already have a SetVariables method that takes a TFragment* as argument.


[Feature Request] Provide a non-react version of Link for Javascript via npm

Hi folks,
I don't use React, but I want to use the client-side "Link" UI (with pure Javascript), however I don't want to import this directly into the page with a <SCRIPT> tag.
I prefer to npm install the library as Javascript module, then use a bundler like webpack (or similar) to package a single JS file that includes my app and any dependencies.
Allowing only the <SCRIPT> tag makes it difficult (impossible?) to write unit tests for client side javascript code
Any plans to do this?

this, and/or an esm version that can be imported via a cdn


Allow SFTP connection with SSH key

Allow SFTP connection with SSH key
Motivation
We do not use password to connect to SFTP server, instead we use SSH Key.
paramiko provides the possibility to set the key_filename in the connect kwargs argument.
However this is not used due to the fact that the ssh._connect() forcefully set the password argument.

Merged. Congratulations on your first contribution to smart_open! 🥇


Refs #35102 -- Optimized Expression.identity and friends.

Per the improvement discussed on ticket-35102 to address the performance regression reported by @tonybaloney.

@felixxm ended up not including

diff --git a/django/db/models/sql/query.py b/django/db/models/sql/query.py
index a79d66eb21..911d0c7848 100644

Index: django/db/models/sql/query.py
===================================================================
--- a/django/db/models/sql/query.py
+++ b/django/db/models/sql/query.py
@@ -422,6 +422,8 @@ class Query(BaseExpression):
         return obj

     def relabeled_clone(self, change_map):
+        if not change_map:
+            return self
         clone = self.clone()
         clone.change_aliases(change_map)
         return clone

As it seemed that relabeled_clone was only called with an empty change_map from query combinator tests and it originated from these two methods which seemed like a better injection point?
diff --git a/django/db/models/sql/query.py b/django/db/models/sql/query.py
index a79d66eb21..a52ef4ab76 100644
--- a/django/db/models/sql/query.py
+++ b/django/db/models/sql/query.py
@@ -972,6 +972,9 @@ def change_aliases(self, change_map):
         relabelling any references to them in select columns and the where
         clause.
         """
+        if not change_map:
+            return self
+
         # If keys and values of change_map were to intersect, an alias might be
         # updated twice (e.g. T4 -> T5, T5 -> T6, so also T4 -> T6) depending
         # on their order in change_map.
diff --git a/django/db/models/sql/where.py b/django/db/models/sql/where.py
index ae7a2a1224..00bfa1653b 100644
--- a/django/db/models/sql/where.py
+++ b/django/db/models/sql/where.py
@@ -204,6 +204,9 @@ def relabel_aliases(self, change_map):
         Relabel the alias values of any children. 'change_map' is a dictionary
         mapping old (current) alias values to the new values.
         """
+        if not change_map:
+            return self
+
         for pos, child in enumerate(self.children):
             if hasattr(child, "relabel_aliases"):
                 # For example another WhereNode

@charettes Thanks :+1: I think it's ready :star:

As it seemed that relabeled_clone was only called with an empty change_map from query combinator tests and it originated from these two methods which seemed like a better injection point?

Agreed, added :+1:

Thanks for merging @felixxm, it'll be interesting to see if AVS catches it this time.
If it doesn't then I suspect we'll need to tweaks things up slightly in how it performs benchmarks as that would mean something is off.

There seems to be caching / a race condition in the django-asv publish pipeline so the results built are not immediately part of the gh-page artifact so we'll need to wait tomorrow to see them displayed but looking at the produced artifacts it appears that we did see an improvement of about 40%

django/django@02eaee12 0.0013847883125013283
django/django@4fec1d2c 0.00099774222727236


Wrong Path On Windows

After I tried to run the command aspri --wp-plugin-build-check I got the error:
2024/04/04 16:26:07 open C:\Users\isonn\Local Sites\fab\app\public\wp-content\plugins \floating-awesome-button/C:\Users\isonn\Local Sites\fab\app\public\wp-content\plugins\floating-awesome-button.php: The filename, directory name, or volume label syntax is incorrect.

It seems aspri can't detect the path correctly, especially on windows. The correct path should be C:\Users\isonn\Local Sites\fab\app\public\wp-content\plugins \floating-awesome-button\floating-awesome-button.php

close the issue, PR been merged


Selected Extension is unreadable

text is gray on orange which is hard to read

Thanks for letting me know. I'll work on an update.

Sorry mentioned this issue by mistake. But I don't face this problem


Github Theme

Is your feature request related to a problem? Please describe.
GitHub Theme.
Describe the solution you'd like.
.
Describe alternatives you've considered.
.
Add any other context or screenshots about the feature request here.
No response

what does this issue do? please elaborate?

/assign

please don't do such things @DragonUncaged , I have said this earlier as well. Don't just try to get the scores by putting spammy issues :(


bar-x not fetching value from $scope

I have a function that fetches a json using $http.get. On success, I execute this code:

$scope.ocupacao = data;
$scope.xaxis = 'mes';
$scope.yaxis = ['txo'];
$scope.label = ['Taxa de Permanência'];

Where mes and txo are the only keys of the json.
When I call, from the html, {{xaxis}} and {{yaxis}} they show mes and txo correctly, but when I use:

bar-x = 'xaxis'
bar-y = 'yaxis'

Only the second resolves to the correct value and the bar-chart shows undefined on the X axis. But if I, manualy, write:

bar-x = 'mes'

The bar-chart shows the value properly.

hi @filipeaguiar
i didnt understand your issue.
can you share a fiddle with your use-case ?

It's basically this:
When I use bar-x="xaxis" the labels at the bottom of the chart appears as undefined
When I use bar-x="mes", they appear normal, showing month names
Even when {{xaxis}} returns mes within the same controller.
I'm not very familiar with jsfiddle, but I can give it a try if you didn't understand this time.

hi @filipeaguiar can you fork and reproduce by this fiddle ?

I'll try and post the results later. BTW, I'm using Angular 1.5.0, could this be the problem?

I dont think so. the fiddle above uses 1.5.0-rc.1

Ok, I did the fiddle, here it is:
https://jsfiddle.net/68L1xp4u/

@filipeaguiar do not see anything new, can you double check your fiddle? are you applying your changes?
We try to map all morris options. If there is something missing, please create a new issue.

Sorry, as I told you, I'm not familiar with jsfiddle. But this is the updated code:
bar-x attribute problem

@filipeaguiar got it
in this case you have to pass the var with brackets {{}}
eg.: https://jsfiddle.net/stewones/0qqs6rhb/

Thank you, it worked. But this should be listed in the documentation.

@filipeaguiar np :beers:
in fact this is a ng 1.x issue rather than our lib. anyway pr´s are always welcome


Level-1 buttons only work in the center

You should have to move the buttons to the center of the screen to use them

Extending this issue, the buttons can be clicked on the upclick after a drag. this is technically a bug and this issue will be relabeled as such as I believe that fixing this will have the same effect as the previous issue.


Call to undefined method App\Models\ChangeRequest::changeable_id()

Bug report
What I did
I am trying to open a model (setupShowOperation).
What I expected to happen
See model fields.
What happened
I get an error "Call to undefined method App\Models\ChangeRequest::changeable_id()"
Model App\Models\ChangeRequest
    use CrudTrait, Moderatable;

    public function changeable(): MorphTo
    {
        return $this->morphTo();
    }

    public function user(): BelongsTo
    {
        return $this->belongsTo(User::class);
    }

Migration
        Schema::create('change_requests', function (Blueprint $table) {
            $table->id();
            $table->foreignIdFor(User::class)->constrained()->cascadeOnDelete();
            $table->morphs('changeable');
            $table->string('new');
            $table->string('old');
            $table->string('reason');
            $table->string('field');
            $table->foreignIdFor(Status::class)->constrained()->cascadeOnDelete();
            $table->dateTime('mod_at')->nullable();
            $table->timestamps();
        });

Method setupShowOperation
    protected function setupShowOperation()
    {
        CRUD::field('changeable_type');
        CRUD::field('changeable_id');
        CRUD::field('old');
        CRUD::field('new');
        CRUD::field('reason');

        $this->crud
            ->column('status.name')
            ->type('relationship')
            ->label('Status')
            ->type('string');

        CRUD::column('created_at');
        CRUD::field('field');
    }

If I remove changeable fields
    protected function setupShowOperation()
    {
        CRUD::field('old');
        CRUD::field('new');
        CRUD::field('reason');
        CRUD::column('created_at');
        CRUD::field('field');
    }

I don't see any fields other than created at. There aren't any other fields ('old', 'new', 'reason', 'field' - missing).
Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
PHP VERSION:
PHP 8.1.10 (cli) (built: Aug 30 2022 18:05:49) (ZTS Visual C++ 2019 x64)
Copyright (c) The PHP Group
Zend Engine v4.1.10, Copyright (c) Zend Technologies
LARAVEL VERSION:
v9.52.4@9239128cfb4d22afefb64060dfecf53e82987267
BACKPACK PACKAGE VERSIONS:
backpack/crud: 5.5.2
backpack/generators: 3.3.16

Hi @AlexKomMsk,
In Backpack, the fields which you want to show are known as columns and editable fields are known as fields.

Columns are used in show and list operation to display the value; CRUD::column()
Fields are used in create and update operation; CRUD::field()

That's why you they were not visible to you.

Yes, that's right. Thank you.