Merge 1.1 into master (redux)

Alternative to #253 after getting tests to pass on master
Fixes #259

Todo

[ ] Get tests to pass, there are some logger things to figure out, probably screwed up in the merge

:100: Chad, this is looking great :green_heart:
indeed, it's been a while since 1.1 was merged to master (mostly due always being in a rush when working on yet another 1.1.x minor release). I was hoping to do some merging and cleanup work on master before 1.2.0 but than the week only has 7 days :sweat:. think we degraded the "production" quality expectations with the 1.2.x releases, at least for now.
anyways, I am very glad to see the work you did here. maybe some of it is redundant but there are a lot of important fixes that are otherwise missing from master, thus :+1: from me...

Thanks @kares, nice to see you!
I think the main thing I could use some input on was the intent of the earlier changes to the way the loggers work across the branches, and whether they are still relevant these days. I wasn't really sure what the main philosophy was for 1.2 so was really just trying to "infer", which is probably a bit tough for me, as not a Rack expert.
For example, in https://github.com/jruby/jruby-rack/pull/262/commits/6abde22131da858660d97515546bba51436c5a0a I made the tests pass, but I'm not sure if I am merging something which should only apply to 1.1 and has no role in 1.2 and why I needed to do this. 😅
Given the divergence of the branches, there is a risk of (accidentally) reinstating some historical complexity from 1.1 that was intended to be removed/ignored on 1.2.
Other than that, my intention before un-drafting the PR was

to actually sanity test it with at least my "real" application
to play around with the examples and appraisals stuff (which I suspect do not work/pass anymore) to understand what confidence they give us and whether it's important to extend that to newer rails versions etc.

I note that the old travis setup seemed to run the appraisals, so at some point they passed, but right now they are not included in the GHA CI which makes me a little nervous. Even if many of those rails versions do not now need to be supported/appraised, it'd probably be good to run at least Rails 6.1 I imagine, while noting that at least in some use cases (#244) Rails 7.0 doesn't currently work right.
But ya, I also got a bit busy in personal life. Will have some more time/space back soon.

@kares @headius so now that we got approval for this what is needed now?  Land and release?  Should we have some smoketest set up?  I feel like this project has languished due to lack of knowing if it is safe to release.
A question I would like answered is how do we QA this?  At this point even if we don't answer this I feel like we should release just to get feedback on what else is still broken (forward progress even if wrong probably is right for now).  That said, I feel like the lack of confidence is paralyzing our ability to move forward.

Give me a week or so to sanity check this? I'm not blocked here, just been very busy moving house so haven't had time to make progress 🙏
I'll have a go at the appraisals but since they are not running on master right now either this doesn't make things "worse" for 1.2 so probably can be decoupled 😅

Should we have some smoketest set up? I feel like this project has languished due to lack of knowing if it is safe to release.

I usually had a dummy sample Rails (as well as Sinatra) app that I would deploy to Tomcat as a .war and see how it's doing. Unless of course there were fixes which targeted other servers than I would deploy to multiple ones (e.g. if there was a Jetty related bug report) as needed...

I usually had a dummy sample Rails (as well as Sinatra) app that I would deploy to Tomcat as a .war and see how it's doing. Unless of course there were fixes which targeted other servers than I would deploy to multiple ones (e.g. if there was a Jetty related bug report) as needed...

@kares I don't doubt there could be a lot of deploys to test all the things supported but even a golden path deploy I think would help improve confidence.  At one point I think @mkristian had a bunch of CI tests on jruby/jruby doing some tests with web containers.

I finally dug back to this in my inbox and could help land it... what needs to be done at this point?

I finally dug back to this in my inbox and could help land it... what needs to be done at this point?

@headius At the moment I think it's on me to sanity check this in a real environment, plus look at the appraisals.
I did an initial sanity test with GoCD and #259 is definitely fixed. However right now I am having a weird problem with expected session cookies not being set by browser on some Rack-managed routes leading to duplicate Jetty session creation on some routes (which then causes Jetty failures) which I am trying to solve. I need to verify it is related to changes made on 1.2 branch or something else that's gone wroing  in my env (perhaps incorrect Set-Cookie from server).

Ok give me a shout if there's any way I can help!

The problem above with my sanity test is unrelated to this change - there is some Firefox Total Cookie Protection change causing an issue with a same site iframe on http://localhost that doesn't happen in a real environment. Sigh :)
All seems to be working fine with jruby-rack so far with GoCD locally.
Will move on to see what I can do with the appraisals.

Ok a month later, where do we stand? @kares @jlahtinen @chadlwilson I'd like to figure out the path forward.

I try to find time to test our apps with this version in next 6 hours and report back.
Not sure if you need any other input from my side.

diff --git a/src/spec/ruby/rack/servlet/response_capture_spec.rb b/src/spec/ruby/rack/servlet/response_capture_spec.rb
index 8ef3e38..b106b9b 100644
--- a/src/spec/ruby/rack/servlet/response_capture_spec.rb
+++ b/src/spec/ruby/rack/servlet/response_capture_spec.rb
@@ -103,11 +103,4 @@ describe org.jruby.rack.servlet.ResponseCapture do
     expect( response_capture.isHandled ).to be true
     expect( response_capture.getStatus ).to eql 200
   end
-
-  private
-
-  def servlet_30?
-    Java::JavaClass.for_name('javax.servlet.AsyncContext') rescue nil
-  end
-
 end
diff --git a/src/spec/ruby/spec_helper.rb b/src/spec/ruby/spec_helper.rb
index e586731..8906a9b 100644
--- a/src/spec/ruby/spec_helper.rb
+++ b/src/spec/ruby/spec_helper.rb
@@ -61,7 +61,7 @@ module SharedHelpers

   def servlet_30?
     return @@servlet_30 unless @@servlet_30.nil?
-    @@servlet_30 = !! ( Java::JavaClass.for_name('javax.servlet.AsyncContext') rescue nil )
+    @@servlet_30 = !! ( Java::javax.servlet.AsyncContext rescue nil )
   end
   private :servlet_30?

With these changes rspec specs passed and rake gem command generates a .gem file while using java 11.

@jlahtinen Thanks for trying this out! Does the resulting gem also work as you expect?
@chadlwilson Seems like we could merge in @jlahtinen's patch and get this merged?

Yes, those changes do seem to help getting the specs to pass with bundle exec rake specs on both Java 8 and Java 11 - as well as via mvn test which was working earlier - thanks! Not sure I understand why, but I'll not ask too many questions :D
I've made this change, and made some subsequent tweaks to get the CI running on all LTS Java versions as well as some minor dependency bumps. Strangely Java 17 is failing a single test on GHA where it passes locally though. Hmm.

@headius I wasn't able to fully test resulting gem yet. I start that work now once again.

about https://github.com/jruby/jruby-rack/actions/runs/12028436967/job/33531563307?pr=262
I saw same problem happening sometimes and sometimes not ...
I didn't try to understand what could be the reason but maybe I (we) should ...

I can always drop the tests for Java 17 and/or 21 if they are mysteriously unstable. Only added them in as were working locally and I didn't think it'd be an issue.
I can sometimes get a failure locally but haven't figured out a pattern yet.

I am quite sure (not 100%) that I used Java 11.
And I have to continue testing this with our apps later. Now about 20h pause.

Yes, the existing tests are stable under Java 8 and 11. I use jruby-rack in production with both Java 17 and Java 21 without issue so it's likely an issue specific to the tests - perhaps the mocking.

From my perspective, this is ready to go now (or as good as it's productive to get for now).
I'll create a separate draft PR for the appraisals (basically that runs tests alongside Rails versions as the old Travis build did) and see if anyone can help me work on it/fix them :-)

@chadlwilson This is great, thank you for your help! I think it's probably fine to merge now and we'll deal with the appraisals separately.
@kares What do you think? Shall we go ahead and merge? I know folks have been waiting on another release with recent patches and 1.1 stuff merged forward.

I was not able to deploy 2 rails app to same tomcat9 with java 11 with this version of jruby-rack
Caused by: java.lang.LinkageError: loader constraint violation: when resolving method 'int org.jruby.RubyString.cat19(org.jruby.util.ByteList, int)' the class loader 'bootstrap' of the current class, java/lang/Object, and the class loader org.apache.catalina.loader.ParallelWebappClassLoader @53811376 for the method's defining class, org/jruby/RubyString, have different Class objects for the type org/jruby/util/ByteList used in the signature (java.lang.Object is in module java.base of loader 'bootstrap'; org.jruby.RubyString is in unnamed module of loader org.apache.catalina.loader.ParallelWebappClassLoader @53811376, parent loader java.net.URLClassLoader @22d8cfe0)
at java.base/java.lang.invoke.MethodHandleNatives.resolve(Native Method)
at java.base/java.lang.invoke.MemberName$Factory.resolve(MemberName.java:1070)
at java.base/java.lang.invoke.MemberName$Factory.resolveOrFail(MemberName.java:1098)
... 55 more
We are currently running version https://github.com/jruby/jruby-rack/compare/master...jlahtinen:jruby-rack:kvp5 that allows to deploy more than 1 rails app to same tomcat9

@jlahtinen While its hard to tell for sure, I'm not aware of anything changed in these changes from 1.1.x over the years that would materially change whether your hack in https://github.com/jruby/jruby-rack/commit/fdbcf6a0fe1d8c957bad7cb545ad2850b961d41e would work before/after these changes?
That error looks like there are perhaps two different underlying JRuby versions being loaded somehow, given it is complaining about org/jruby/RubyString between the bootstrap Tomcat classloader and the ParallelWebApp classloader - both Tomcat classloaders - and given that jruby-rack defines JRuby as provided scope, which requires you to supply an appropriate JRuby version in the right place yourself?
...unless it's something to do with the loggers.... it's been quite a while since I tangled with Tomcat classloaders.

Maybe that error ok to have after merging this.
And lets resolve this multiple rails apps later?

LinkageError

@jlahtinen I concur with @chadlwilson... that error seems like you have two copies of JRuby loading. Perhaps there's a JRuby jar in the bootstrap jars for Tomcat? For multi-app deployments, there should really only be the copies in each app and not a global one.
That method (RubyString.cat19) exists but is deprecated in 9.4 (replacement is catWithCodeRange added in <IP_ADDRESS>). It will be removed in JRuby 11 (or whatever big release comes after 10.0). We should update jruby-rack to use the replacement method at some point, but it has been around for a long time.

I think we do not have a global one there.
Can you spot anything else that could be the reason?
[root@f69b8a4c8384 tomcat9]# pwd
/usr/share/tomcat9
[root@f69b8a4c8384 tomcat9]# find -name '*jruby*'
./foo_apps/ROOT/WEB-INF/gems/gems/concurrent-ruby-1.3.4/lib/concurrent-ruby/concurrent/synchronization/jruby_lockable_object.rb
./foo_apps/ROOT/WEB-INF/gems/gems/jruby-rack-1.2.2.pre1
./foo_apps/ROOT/WEB-INF/gems/gems/jruby-rack-1.2.2.pre1/lib/jruby
./foo_apps/ROOT/WEB-INF/gems/gems/jruby-rack-1.2.2.pre1/lib/jruby-rack-1.2.2-pre1.jar
./foo_apps/ROOT/WEB-INF/gems/gems/jruby-rack-1.2.2.pre1/lib/jruby-rack.rb
./foo_apps/ROOT/WEB-INF/gems/gems/nokogiri-1.16.7-java/lib/nokogiri/jruby
./foo_apps/ROOT/WEB-INF/gems/gems/racc-1.8.1-java/lib/java/racc/cparse-jruby.jar
./foo_apps/ROOT/WEB-INF/gems/specifications/jruby-rack-1.2.2.pre1.gemspec
./foo_apps/ROOT/WEB-INF/lib/jruby-core-<IP_ADDRESS>-complete.jar
./foo_apps/ROOT/WEB-INF/lib/jruby-rack-1.2.2-pre1.jar
./foo_apps/ROOT/WEB-INF/lib/jruby-stdlib-<IP_ADDRESS>.jar
./bar_apps/ROOT/WEB-INF/gems/gems/concurrent-ruby-1.3.4/lib/concurrent-ruby/concurrent/synchronization/jruby_lockable_object.rb
./bar_apps/ROOT/WEB-INF/gems/gems/jruby-rack-1.2.2.pre1
./bar_apps/ROOT/WEB-INF/gems/gems/jruby-rack-1.2.2.pre1/lib/jruby
./bar_apps/ROOT/WEB-INF/gems/gems/jruby-rack-1.2.2.pre1/lib/jruby-rack-1.2.2-pre1.jar
./bar_apps/ROOT/WEB-INF/gems/gems/jruby-rack-1.2.2.pre1/lib/jruby-rack.rb
./bar_apps/ROOT/WEB-INF/gems/gems/nokogiri-1.16.7-java/lib/nokogiri/jruby
./bar_apps/ROOT/WEB-INF/gems/gems/racc-1.8.1-java/lib/java/racc/cparse-jruby.jar
./bar_apps/ROOT/WEB-INF/gems/specifications/jruby-rack-1.2.2.pre1.gemspec
./bar_apps/ROOT/WEB-INF/lib/jruby-core-<IP_ADDRESS>-complete.jar
./bar_apps/ROOT/WEB-INF/lib/jruby-rack-1.2.2-pre1.jar
./bar_apps/ROOT/WEB-INF/lib/jruby-stdlib-<IP_ADDRESS>.jar

Another app works and another fails to
Caused by: java.lang.LinkageError: loader constraint violation: when resolving method 'int org.jruby.RubyString.cat19(org.jruby.util.ByteList, int)' the class loader 'bootstrap' of the current class, java/lang/Object, and the class loader org.apache.catalina.loader.ParallelWebappClassLoader @5f4098bd for the method's defining class, org/jruby/RubyString, have different Class objects for the type org/jruby/util/ByteList used in the signature (java.lang.Object is in module java.base of loader 'bootstrap'; org.jruby.RubyString is in unnamed module of loader org.apache.catalina.loader.ParallelWebappClassLoader @5f4098bd, parent loader java.net.URLClassLoader @22d8cfe0)
        at java.base/java.lang.invoke.MethodHandleNatives.resolve(Native Method)
        at java.base/java.lang.invoke.MemberName$Factory.resolve(MemberName.java:1070)
        at java.base/java.lang.invoke.MemberName$Factory.resolveOrFail(MemberName.java:1098)

Chad has done a great job here, the actual production affecting changes are all from a stable 1.1 release, thus I am in favor of merging and cutting a new release.
The LinkageError is unclear, does not seem like changes in this PR would cause that and should potentially be a separate ticket.

@headius I created https://github.com/jruby/jruby-rack/issues/268 didn't have too much time to think how to create a good report, sorry.


port to newer versions and add anchor aura

We're not planning to port to newer versions as rn:

A lot of stuff to do to upgrade
Why??

For anchor aura we can think about doing it


[Bug] Destroyer - lost blocks

If the Destroyer is in the process of breaking a block, and the world is unloaded, the block it is breaking is lost.

I can't seem to replicate this.
Is the destroyer linked to any other node? If not, the item would still be inside the destroyer.
If it is linked, are you sure that the item isn't inside another node?
If you can't find the item anywhere:
Singleplayer or Multiplayer?
What block were you breaking?
What other node is destroyer connected to?
Thanks.

Singleplayer.
Various, but I remember granite being one of them.
The destroyer is linked to an inserter, which is inputting into a chest so I can count the items.

Is this a consistent thing or only happening some of the time?

I've been able to reproduce it by exiting the world in single player. I'll test it again tonight to make sure I'm not miscounting.

Sorry for the late follow up. I've been unable to reproduce the supposedly disappearing blocks. I was probably miscounting.
Anyway, I'm really liking what you've made so far, and have yet to break it, but I'll keep trying! Once 1.12 forge is stable, Anima Mundi is a strong candidate for my modded SSP instance.


Suggestion: add a way to sync pic/photo to contact

Hello,
You made an awesome work with this project.
Unfortunately, I really would like to add a picture to Elmer Fudd, but I have totally no idea on how to do that.
Is that easily feasible?
Thank a lot for your help and sharing your work!

I've added an image to the sample contact for future reference

btw, in reference to the code you pasted, you can just read the file's bytes directly into an array, instead of decoding and then re-encoding the jpeg.  I used BitmapFactory in my own sample just because I was referencing a Drawable instead of an image file.


Sources of the pump'n'dump detector

Hi there. Your paper "Pump and Dumps in the Bitcoin Era: Real Time Detection of Cryptocurrency Market Manipulations" mentions a tool to detect pump'n'dump activity in 25seconds. Are sources available somewhere?

Hi @dgellow, unfortunately, there is no public version available online of our source code. However, in the paper, there are all the details to replicate our tool.

Hi @dgellow, unfortunately, there is no public version available online of our source code. However, in the paper, there are all the details to replicate our tool.

Thanks

Thanks


Add curatory groups to organisations and platforms

Curatory groups should be added to organisations and platforms as well in the same way they are added to packages.

is already realised for platforms
OAI API and ES Index should also give out the curatory groups

Added in c615fa97be5aec9eeddedc412b5ccba60efb8645


Can not read file because of URI.open method

Back to use open method
Change default sort to recently updated

Moved to https://github.com/ThanhKhoaIT/rails_execution/pull/7


replace parts override bug when multiple custom element instances are placed in view

replaceable parts work fine, but the issue is that if you put multiple instances of the custom element in one view the replaced template of the last  custom element instance overrides the previous one, while the correct behavior is that each one of them should be isolated.
for example
<template>
  <require from="./custom-elements/compose-collection"></require>
  <require from="./custom-elements/form-field"></require>

      <compose-collection data.bind="items">
        <template replace-part="item-template">
          Replacement: ${item.name} - ${item.src}
          <button click.delegate="$parent.$parent.Test()">Test</button>
        </template>
      </compose-collection>
      <button click.delegate="AddDynamicItemsData()">Add Dynamic Data</button>

      <compose-collection data.bind="formFields" test.bind="test">
        <template replace-part="item-template">
          <form-field label.bind="item.label" value.bind="item.value"></form-field>
        </template>
      </compose-collection>
      <button click.delegate="AddDynamicFormFieldsData()">Add Dynamic Data</button>

</template>

so what happens is that replace-part of the second instance of compose-collection overrides the first one

Glad to tell you that this is already fixed in master. Release going out in the next day. Sorry for the delay!

you rock :)


5T8G

Key
Value

View
https://database.shabados.com/sources/2/page/158/line/11

Source
Sri Dasam Granth

Page
158

ID
5T8G

Line
iqn kr ifMB n iksU idKwXo ]

Depends on https://github.com/GurbaniNow/gurmukhi-fonts/issues/9


add centos 8 support

Hi,
the changes in my commit add support for CentOS 8.
Works like a charm on my installations.

+1


Hovering over operators causes view to jump around in infinite loop

Firstly, I love the direction this extension is going. One issue I've run into is on-hover changes the size of various components, and because this moves the box you're hovering away from the mouse it bounces back and forth (see gif below). This happens in all 3 tabs. Would be good to fix by finding a way to avoid resizing the outer boxes or something similar.

hmmm, interesting. obviously doesn't happen on my machine because that's pretty aggressive. I'll get on it and try and reproduce. Any chance you could paste your settings?

Here's my settings. This is on macOS, my laptop monitor, ~1,047 width by 700 height.

Azure Data Studio version: azuredatastudio 1.5.0 (9ca6200018fc206d67a47229f991901a8a453781, 2017-12-15T12:00:00.000Z)
OS version: Darwin x64 18.2.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz (8 x 2200)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (0.05GB free)

Process Argv
/Users/kevcunnane/projects/azuredatastudio/.build/electron/Azure Data Studio.app/Contents/MacOS/Electron . --remote-debugging-port=9222 --no-first-run --no-default-browser-check --inspect=5875

Screen Reader
no

VM
0%

Extensions (4)

Extension
Author (truncated)
Version

server-report
Mic
0.1.4

sql-vnext
Mic
0.10.2

PowerShell
ms-
1.11.0

queryplan-show
phi
0.0.1

Fixed it in latest, thanks!

@kevcunnane I think I might have actually finally found the right combination of query plan and browser size to reproduce. Looks like it came down to when the selected operator had a longer set of details than the selected one (or vice versa). On hover of the item the size of the operator display was changing and vue was getting cute trying to remember the scroll position. This was shifting things around causing a mouseout and returning to the previous size. Which caused the size of the operator to change again which caused vue to get cute trying to remember the scroll position and another adjustment leading to another mouseout.
I have a small little workaround of changing the min-size of the display to a bit larger than the view window to hopefully minimize the opportunity for this to happen that I'll publish in the next release.


[QA][CI] Have tests running with various deployment setup in CI

Currently, we run tests against the ocis server started from a single binary. We want to have a CI setup where we can start multiple instances of a service (as a load balancing) so that we can test against various deployment strategies.
Tasks

[ ] one ocis & 2 storage user services. try to spin it up in drone-ci (without Kubernetes)
[ ] try to run e2e tests on that setup
[ ] check the logs if both services receive requests

Taken from https://confluence.owncloud.com/pages/viewpage.action?pageId=58296733

@saw-jan This issue needs to be discussed whether to invest time for doing and running such things in drone as it seems to be whole lot of task.

[ ] unskip smoke e2e test suite (see comment)

https://github.com/owncloud/ocis/pull/10482#discussion_r1846402263

It's only the problem with drone setup. I can run the upload test and it passes on local docker compose setup (multi-service)

I can reproduce the large upload failure with a docker-compose setup.
When running the upload e2e test from the host machine, the test works fine. But if I run the same test from a docker container then the large upload test fails.

Now I can reproduce it even when running from host machine.
If I use this compose, the large upload test passes. Note OCIS_URL
ocis:
    image: owncloud/ocis-rolling:master
    ports:
      - 9200:9200
    environment:
      OCIS_URL: https://localhost:9200
      ...

Run test with:
BASE_URL_OCIS=localhost:9200 \
HEADLESS=true \
pnpm test:e2e:cucumber tests/e2e/cucumber/features/smoke/uploadResumable.feature

But I use do the following then I cannot upload file larger than 9MB
ocis:
    image: owncloud/ocis-rolling:master
    ports:
      - 9200:9200
    environment:
      OCIS_URL: https://host.docker.internal:9200
      ...
    extra_hosts:
      - host.docker.internal:host-gateway

Run test with:
BASE_URL_OCIS=host.docker.internal:9200 \
HEADLESS=true \
pnpm test:e2e:cucumber tests/e2e/cucumber/features/smoke/uploadResumable.feature

I see these logs:
storageusers-2-1  | {"level":"info","service":"storage-users","pkg":"rhttp","traceid":"47503678543d4903986757df053c350c","time":"2024-11-19T10:25:42Z","line":"github.com/cs3org/reva/v2@v2.26.0/internal/http/interceptors/auth/auth.go:195","message":"skipping auth check for: /data/tus/2da25800-d76b-4257-82e4-18b92c086bb9"}
storageusers-2-1  | {"level":"warn","service":"storage-users","pkg":"rhttp","traceid":"47503678543d4903986757df053c350c","time":"2024-11-19T10:25:42Z","line":"github.com/cs3org/reva/v2@v2.26.0/internal/http/interceptors/auth/auth.go:248","message":"core access token not set"}

And using this cors config made the test pass:
OCIS_CORS_ALLOW_ORIGINS: https://host.docker.internal:9200,https://host.docker.internal:9201


Add New Input

New input please, i really like it in some other engine builds that arent kade
(ive never done a pull request so sorry if i get something wrong)

First, what do you mean by "New Input". Second, that whole PR is a mess with many unrelated commits being pulled along with it so it's an instant no.
I recommend you read a bit about how Pull Requests work, branches and all these concepts before you start contributing first because PR's like these will never make it through.

First, what do you mean by "New Input". Second, that whole PR is a mess with many unrelated commits being pulled along with it so it's an instant no.
I recommend you read a bit about how Pull Requests work, branches and all these concepts before you start contributing first because PR's like these will never make it through.

i think he put this on the wrong repo, cuz he said "other engine builds that arent kade" and this is kade-engine so like.. yeah

seems u PRed to the wrong repo


es6 shim causing ts compilation error

Getting this error as a conflict between the es6 shim and core-js.
\node_modules@types\core-js\index.d.ts(21,14): error TS2300: Build:Duplicate identifier 'PropertyKey'.
\wwwroot\modules\angular-2-local-storage\typings\globals\es6-shim\index.d.ts(3,14): error TS2300: Build:Duplicate identifier 'PropertyKey'.
I've tried putting all manner of things in my ts.config exclude section but can't seem to get past this.  Any ideas?  Should the shim really be in there anyway?

I have pushed a new release that removes the use of ES6-shim completely (using the TypeScript lib compiler option instead).
Try the new version @paladinu, and if you are still having issues feel free to reopen this issue!

Thanks!  I'll give it a shot.


When will add quantization feature?

As title discribe.

Quantization is higher priority in our plan, we will release it as soon. You can star mmrazor  so as to be notified as early.

Look forward next release version.


compileSdkVersion is not specified. Please add it to build.gradle

Description
Android will fail to compile
Steps to Reproduce

try and build android on a react native app with latest installed

Expected Behavior
I expect the app to build
Relevant Logs / Console output
> Configure project :newrelic-react-native-agent
:newrelic-react-native-agent package.json found at /Users/hillary.snyder/Documents/Projects/movespring/ms-mobile/node_modules/newrelic-react-native-agent/package.json
:newrelic-react-native-agent:newrelic.android using default value: 7.0.0
:newrelic-react-native-agent:newrelic.ndk using default value: 1.0.3
:newrelic-react-native-agent package.json found at /Users/hillary.snyder/Documents/Projects/movespring/ms-mobile/node_modules/newrelic-react-native-agent/package.json
:newrelic-react-native-agent:version set from package.json: 1.3.2 (1,3,2 - 1003002)
5 actionable tasks: 5 up-to-date

FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring project ':newrelic-react-native-agent'.
> compileSdkVersion is not specified. Please add it to build.gradle

Your Environment

ex: Output of npx react-native info

    OS: macOS 13.5.1
    CPU: (12) arm64 Apple M2 Max
    Memory: 53.45 MB / 64.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.5.1 - /usr/local/bin/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 9.8.0 - /usr/local/bin/npm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Managers:
    CocoaPods: 1.12.1 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 23.0, iOS 17.0, macOS 14.0, tvOS 17.0, watchOS 10.0
    Android SDK: Not Found
  IDEs:
    Android Studio: 2022.3 AI-223.8836.35.2231.10671973
    Xcode: 15.0/15A240d - /usr/bin/xcodebuild
  Languages:
    Java: 11.0.20 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 18.2.0 => 18.2.0
    react-native: 0.71.13 => 0.71.13
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

ex: Which platform is the issue happening on? (Android, iOS, tvOS, etc.)

        buildToolsVersion = "33.0.0"
        minSdkVersion = 21
        compileSdkVersion = 33
        targetSdkVersion = 33

I can also see compileSdkVersion was removed in this commit from last week: https://github.com/newrelic/newrelic-react-native-agent/commit/c9630db747a9a942a120f95430c4887dccfbd402

@HilSny which version of android gradle  plugin are you using? I will add it for next release.

@ndesai-newrelic
classpath("com.android.tools.build:gradle:7.4.2")
Android doesn't build with this at all because to my understanding its a requirement to have this field included. Do you know why it was removed?
We vetting new relic as a potential provider for us but right now we can't test.

i will do a hotfix release to fix this issue, can you manually add it to our gradle file for your testing?

For sure! Thank you so much for your super quick response :)

Adding manually to build.gradle works fine
/node_modules/newrelic-react-native-agent/android/build.gradle
 defaultConfig {
        minSdkVersion 21
        targetSdkVersion 33
        compileSdkVersion 33
        versionCode 1
        versionName version
    }

Add    compileSdkVersion 33
Also for me I had to change my project's minSdk to 24 and it works fine

@HilSny @AkhtarZaman7 can you check it in our latest release? it is fixed now.

@ndesai-newrelic same in 1.3.2

@LFMAKER it is already released yesterday.


Documentation lies regarding z-index ordering of setLegend

Describe the bug
A user reported that the documentation in the help panel states the following under setLegend(bitmaps):

however this link shows a game where it can easily be seen that this ordering is not respected.
To Reproduce
Steps to reproduce the behavior:

Open above link
Modify legend order
Run game

Expected behavior
draw order should respect setLegend documentation
Actual behavior
draw order does not respect setLegend documentation

@leomcelroy stated he believed this is likely a regression.  I will investigate this tomorrow by doing a git bisect and attempt to find the specific commit that caused this apparent regression.

This is a minimal example that does seem to demonstrate that the z-ordering is respected.
https://sprig.hackclub.com/share/3kovFmLi0HQaMQNvOMQd
Try swapping the order of the sprites.
What the bug should be in the provided example is that setLegend should clear the entire sprite table in the movement code.
onInput("w", () => {
  getFirst(player).y -= 1
  setLegend([ player, playerMaps[0]])
})
onInput("a", () => {
  getFirst(player).x -= 1
  setLegend([ player, playerMaps[1]])
})
onInput("s", () => {
  getFirst(player).y += 1
  setLegend([ player, playerMaps[3]])
})
onInput("d", () => {
  getFirst(player).x += 1
  setLegend([ player, playerMaps[2]])
})

The error may be here:
https://github.com/hackclub/sprig-engine/blob/main/src/web/index.ts#L94


Update 03-prompt-engineering.md -Minor fix

This seems to be incorrect or a typing mistake with Python code. After removing the "await" keyword, it is working though, this is a minor change.
Module: 00
Lab/Demo: 00
Fixes # .
Changes proposed in this pull request:

something wrong, taking deeper look

something wrong, taking deeper look

looks like something wrong with async / await in python code.


The item in your hand is not compatible when use "/nmc set spoiled"

Describe the bug
With some foods, using the command "/nmc set spoiled" tells me that the object is not supported.
To Reproduce
In my case I have used beef food, but it happens with others.
Expected behavior
You should set the food as expired just like with cooked beef or other foods that do work.
Screenshots
With a food that doesn't work:

With one that does work:

Minecraft Version:

1.20.4

Server Version:

Paper 1.20.4

Plugin Version

NoMoreCooked-0.0.9c-RELEASE

Problem solved in version 0.0.96 of NoMoreCooked available on Spigot.
If you encounter other problems, don't hesitate to reopen another issue or reply to this one if it concerns this problem.
Have a nice day !


Improve playwright ci test

This is a fix for #210

Summary by CodeRabbit

New Features

Introduced test cases for funding and membership pages, including donate, membership, members, and past members pages.
Added fixture classes for CommunityPage, FundingPage, and DownloadPage to interact with specific page elements and enhance test coverage.

Refactor

Restructured tests for community, funding, and download pages to improve code readability and maintainability.

Much better now! Small refactoring advice: the pages can be inherited. There should be a base page with methods and selectors that are used everywhere, so that other pages could inherit from it. For example, the "goto" function was separately implemented on each page. But it's OK for the current scope


fix: don't rotate blocks with a facing=up/down

@mworzala is it ok for this to return null/empty string if the input is null/empty string? Or maybe other code was relying that this would output north to prevent some issues in that case. If so I'll need to change it to have cases for up/down instead of using the default to just return what was input.
The code currently forces north if a block is not facing north/east/south so if it is facing up or down, it will break its state.

I believe this code is fine because it checks if the facing property != null before trying to rotate


Refactor directory structure.

Main reason for this refactoring was to separate test definitions/logic from the controllers & utils logic.
With current directory/package structure it was hard for me to understand which files are test files and which are not.
Hope this PR make our testing structure more clear to newcomers.

@rhopp WDYT about moving tests folder to pkg/framework?

@flacatus My thinking was

pkg/framework

framework logic. Any wrappers for containers/describes, any "baked in" beforeall etc. methods, loggers, .. And any other "frameworky" stuff :-D

pkg/utils

Any utility packages/files/logic can go here

pkg/tests

Any "specfiles" (even thouh they are not specfiles :-D) - files containing the test logic - containing Describes Its etc.
WDYT?


Add Requires PHP and Requires at least

See: https://make.wordpress.org/core/2020/02/26/miscellaneous-developer-focused-changes-in-wordpress-5-4/

We can remove this functionality from functions.php now.
https://github.com/roots/sage/blob/10.0.0-dev/functions.php#L21-L33

If we remove https://github.com/roots/sage/blob/7a343ad8e6cdd861820879415e00dcbfe372f7f2/functions.php#L21-L33, then we need to bump WordPress requirement to v5.4.
Is that okay?

We've never shied from bumping version requirements in Sage since it's a starter theme, so it's anticipated that devs are working with a new installation. I'm fine with bumping to WP 5.4, but we can wait to see what @retlehs thinks.

fine with me :+1:

Note that the target date for release of WordPress 5.4 is 31st March 2020.
See: https://make.wordpress.org/core/5-4/


Reduce BoilerCode?

Question
Hello,
I'm using the openapi-generator with Vapor, and I found that I ended generating a lot of code that is identical, because all the responses, no matter the error, are almost identical. I created a discussion in the forum of swift (here: https://forums.swift.org/t/trying-to-reduce-boilercode/76278/17 ) but seems that there is nothing appart of what I did with some macro, that allow you to reduce a huge bunch of this stuff.
Maybe I'm doing it wrong. There is some more elegant way to reduce this?
Thanks,
Jaume.

Hi @Dracks,
this tool generates code for the OpenAPI document. If the OpenAPI document contains a lot of duplicated types, the generated Swift code will contain duplicated types. There isn't much the tool can do.
Could you post your OpenAPI doc? Maybe there are ways to optimize that

I see, that makes sense. The timing is fortunate here, we recently landed an improved error handling proposal, which would allow you to write convenience functions that validate your input and throw specific errors that get translated to the appropriate response status codes. So you wouldn't need to repeat as much of the error handling in every handler - it's designed to enable the writing of reusable utility functions, which still result in specific error codes.
Can you give that a try and see if it helps?

your proposal here is to simply throw an error that implements that protocol, and then catch it on the middleware?

Yup, exactly.

that will means, that if in the open-api, you add any extra field in the error types, this will not be detected. right?

I meant that you'd hand-write an error type in your project and conform that to the protocol. I didn't mean that you'd use the generated schema as an error type.

Yeah, I get it.

Okay, closing for now, please reopen if the new error handling features didn't help.


[Applied Visual Design - FCC(Beta)]: Challenge describes wrong error

Challenge create-movement-using-css-animation has an issue.
User Agent is: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36 OPR/43.0.2442.1144.
Please describe how to reproduce this issue, and include links to screenshots if possible.

console result:

The @keyframes rule for 0% should use the left offset of 0px.
The @keyframes rule for 50% should use the left offset of 25px.
The @keyframes rule for 100% should use the left offset of -25px.

Sure enough, the left properties are all there. He/she is missing the top property as shown in the css code. These errors do not reflect on what the user has done wrong.
suggestion result:

The @keyframes rule for 0% should use the top offset of 0px.
The @keyframes rule for 50% should use the top offset of 50px.
The @keyframes rule for 100% should use the top offset of 0px.

^^^ This will get these people thinking ^^^
or
warn the user that the challenge asks that the user should add the left offset to the @keyframes rule, not replace the top offset with the left offset

@Brekmister: The challenge code actually includes the top property at all 3 keyframe stages, therefore the tests are right in checking only for the inclusion of the left property at the keyframe stages. Even with the suggestive empty space below the top property to place the left property, and the instruction text:

Add a horizontal motion to the div animation. Using the left offset property, add to the @keyframes rule so rainbow starts at 0 pixels at 0%, moves to 25 pixels at 50%, and ends at -25 pixels at 100%.

maybe it would be wise to place a note/comment to strictly add a left property and not replace the top property, or any other code for that matter, as a reminder. A simple comment in the challenge code would suffice. The tests should not have to be rewritten or modified for this challenge.

@cdrainxv I completely agree with you on that. I would personally just change the text of the error output on possibilities on what the user possibly did wrong instead of giving the output it already is. Assuming we are saying that the tests return true for passing and false for failing.
If it is such the case of true and false, why are they telling the users what is wrong in the code when the website only knows whether the code passed or not? That actually doesn't sit that well with me although that may be the way that it has to be.

@Brekmister: There's no way to account for all the possibilities of wrong answers a user could have typed in. The current regex test works just fine because it is only testing for the presence of the left property with the  correct values according to the instructions provided. The browser doesn't need to know if your code is correct or not, it will either compile what is in the editor if the syntax is correct and display it on the iframe, or it will not, but only the test can confirm if the code you have written is correct.
Take a look at how the challenges are tested to get the overall idea of how they work and the reason for their specificity.
You can see that the test messages must reflect the assert test. The message only highlights what you did wrong according to the constraints of the test itself. As the top property was not meant to be changed, then only the left property must be the source of error (according to how the message is written). Assert tests only return booleans (true or false), and the message is to inform you what the reason could be for the failure or serve as a starting point for debugging your code.
With your route of providing several possibilities of what a camper could have done incorrectly, that could lead down a rabbit hole where a camper might not emerge from, as they can't be too sure which one of the (countless) possible errors they could have included in their code -- unless  specific ones have been pointed out for them, as in the test messages. If the error doesn't coincide with their code, then there's always the help rooms available to seek help or guidance.
With the whole instruction setup and the example provided, the test messages seem appropriate for this challenge. I hope that kind of clears up the reasoning behind how the tests are written and how the messages are worded. 😄

@cdrainxv  That does clear things up. Since that is the case scenario then you have the best proposal

it would be wise to place a note/comment to strictly add a left property and not replace the top property, or any other code for that matter, as a reminder. A simple comment in the challenge code would suffice.

I hope you can agree with me that this is a very common mistake and perhaps people like me are going to get tired real quick of people making that mistake

@Brekmister thanks for raising this issue, and @cdrainxv that was an great explanation of the tests and clarification of the challenge. We can add a sentence at the end of the instructions to remind folks not to remove anything.
Suggested instructions:

Add a horizontal motion to the div animation. Using the left offset property, add to the @keyframes rule so rainbow starts at 0 pixels at 0%, moves to 25 pixels at 50%, and ends at -25 pixels at 100%. Don't replace the top property in the editor - the animation should have both vertical and horizontal motion.

For anyone who would like to make this change, please review the Contributor's Guide and feel free to ask any questions in the Contributor's chat room. You can find the code to make this change here.  This is my code and ive done the right things im supposed to do, but it still says im wrong?????????????????????????????????????


Use GitHub Actions for CI

Summary
Use GitHub Actions for CI and publish related test reports.
Acceptance criteria

[X] Unit tests of repo projects (oscm, oscm-app, oscm-rest-api, etc.) are running with GitHub Action
[X] Test failures are indicated with respective code places in a user-friendly manner
[X] Test reports for the master trunk are published in build failure as well as in success case
[X] Comprehensive code coverage reports for the master trunk are published in build failure as well as in success case

Additional context
N/A

Completed with pull requests:
https://github.com/servicecatalog/oscm/pull/1270
https://github.com/servicecatalog/oscm/pull/1267
https://github.com/servicecatalog/oscm/pull/1269
https://github.com/servicecatalog/oscm-app/pull/127
https://github.com/servicecatalog/oscm-app/pull/129
https://github.com/servicecatalog/oscm-app/pull/130
https://github.com/servicecatalog/oscm-app/pull/131
https://github.com/servicecatalog/oscm-identity/pull/101
https://github.com/servicecatalog/oscm-identity/pull/102
https://github.com/servicecatalog/oscm-approval/pull/68
https://github.com/servicecatalog/oscm-mail/pull/10
https://github.com/servicecatalog/oscm-mail/pull/12
https://github.com/servicecatalog/oscm-mail/pull/13
https://github.com/servicecatalog/oscm-commons/pull/51
https://github.com/servicecatalog/oscm-rest-api/pull/256
https://github.com/servicecatalog/oscm-rest-api/pull/257
https://github.com/servicecatalog/oscm-rest-api/pull/258
https://github.com/servicecatalog/oscm-rest-api/pull/259
https://github.com/servicecatalog/oscm-rest-api/pull/260
https://github.com/servicecatalog/oscm-rest-api/pull/262
https://github.com/servicecatalog/oscm-rest-api/pull/266
https://github.com/servicecatalog/oscm-rest-api/pull/267

Done.


tests: dma: chan_blen_transfer: test breaks if DMA cannot access flash

Describe the bug
In tests/drivers/dma/chan_blen_transfer, the tx_data buffer is declared const:
https://github.com/zephyrproject-rtos/zephyr/blob/b82b5b0734c30490368e21627423f10036487343/tests/drivers/dma/chan_blen_transfer/src/test_dma.c#L25
As a result, this buffer is placed in .rodata and ends up in FLASH region. This causes the test to fail on SoCs equipped with a DMA controller that cannot access flash memory, such as STM32WB09 MCUs.
Expected behavior
Test should succeed on SoCs with a DMA as long as it can perform memory -> memory (i.e., RAM -> RAM) transfers, regardless of flash -> memory transfer capabilities.
A possible fix is to remove the const qualifier on tx_data - without the qualifier, the buffer seems to be placed in RAM, and the test succeeds.
Logs and console output
Test log:
Running TESTSUITE dma_m2m
===================================================================
START - test_tst_dma0_m2m_chan0_burst16
Preparing DMA Controller: Name=dmamux@48800000, Chan_ID=7, BURST_LEN=2
Starting the transfer

    Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/dma/chan_blen_transfer/src/test_dma.c:137: dma_m2m_test_tst_dma0_m2m_chan0_burst16: (test_task(dma, CONFIG_DMA_TRANSFER_CHANNEL_NR_0, 16) == TC_PASS) is false

 FAIL - test_tst_dma0_m2m_chan0_burst16 in 2.028 seconds
===================================================================
START - test_tst_dma0_m2m_chan0_burst8
Preparing DMA Controller: Name=dmamux@48800000, Chan_ID=7, BURST_LEN=1
Starting the transfer
E: dma stream 7 is busy.
E: cannot configure the dmamux.
ERROR: transfer

    Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/dma/chan_blen_transfer/src/test_dma.c:137: dma_m2m_test_tst_dma0_m2m_chan0_burst8: (test_task(dma, CONFIG_DMA_TRANSFER_CHANNEL_NR_0, 8) == TC_PASS) is false

 FAIL - test_tst_dma0_m2m_chan0_burst8 in 0.034 seconds
===================================================================
START - test_tst_dma0_m2m_chan1_burst16
Preparing DMA Controller: Name=dmamux@48800000, Chan_ID=6, BURST_LEN=2
Starting the transfer

    Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/dma/chan_blen_transfer/src/test_dma.c:137: dma_m2m_test_tst_dma0_m2m_chan1_burst16: (test_task(dma, CONFIG_DMA_TRANSFER_CHANNEL_NR_1, 16) == TC_PASS) is false

 FAIL - test_tst_dma0_m2m_chan1_burst16 in 2.028 seconds
===================================================================
START - test_tst_dma0_m2m_chan1_burst8
Preparing DMA Controller: Name=dmamux@48800000, Chan_ID=6, BURST_LEN=1
Starting the transfer
E: dma stream 6 is busy.
E: cannot configure the dmamux.
ERROR: transfer

    Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/dma/chan_blen_transfer/src/test_dma.c:137: dma_m2m_test_tst_dma0_m2m_chan1_burst8: (test_task(dma, CONFIG_DMA_TRANSFER_CHANNEL_NR_1, 8) == TC_PASS) is false

 FAIL - test_tst_dma0_m2m_chan1_burst8 in 0.034 seconds
===================================================================
TESTSUITE dma_m2m failed.

------ TESTSUITE SUMMARY START ------

SUITE FAIL -   0.00% [dma_m2m]: pass = 0, fail = 4, skip = 0, total = 4 duration = 4.124 seconds
 - FAIL - [dma_m2m.test_tst_dma0_m2m_chan0_burst16] duration = 2.028 seconds
 - FAIL - [dma_m2m.test_tst_dma0_m2m_chan0_burst8] duration = 0.034 seconds
 - FAIL - [dma_m2m.test_tst_dma0_m2m_chan1_burst16] duration = 2.028 seconds
 - FAIL - [dma_m2m.test_tst_dma0_m2m_chan1_burst8] duration = 0.034 seconds

------ TESTSUITE SUMMARY END ------

===================================================================
PROJECT EXECUTION FAILED

Test log, after removing the const storage qualifier of tx_data:
Running TESTSUITE dma_m2m
===================================================================
START - test_tst_dma0_m2m_chan0_burst16
Preparing DMA Controller: Name=dmamux@48800000, Chan_ID=7, BURST_LEN=2
Starting the transfer
DMA transfer done
It is harder to be kind than to be wise........
 PASS - test_tst_dma0_m2m_chan0_burst16 in 2.015 seconds
===================================================================
START - test_tst_dma0_m2m_chan0_burst8
Preparing DMA Controller: Name=dmamux@48800000, Chan_ID=7, BURST_LEN=1
Starting the transfer
DMA transfer done
It is harder to be kind than to be wise........
 PASS - test_tst_dma0_m2m_chan0_burst8 in 2.015 seconds
===================================================================
START - test_tst_dma0_m2m_chan1_burst16
Preparing DMA Controller: Name=dmamux@48800000, Chan_ID=6, BURST_LEN=2
Starting the transfer
DMA transfer done
It is harder to be kind than to be wise........
 PASS - test_tst_dma0_m2m_chan1_burst16 in 2.015 seconds
===================================================================
START - test_tst_dma0_m2m_chan1_burst8
Preparing DMA Controller: Name=dmamux@48800000, Chan_ID=6, BURST_LEN=1
Starting the transfer
DMA transfer done
It is harder to be kind than to be wise........
 PASS - test_tst_dma0_m2m_chan1_burst8 in 2.015 seconds
===================================================================
TESTSUITE dma_m2m succeeded

------ TESTSUITE SUMMARY START ------

SUITE PASS - 100.00% [dma_m2m]: pass = 4, fail = 0, skip = 0, total = 4 duration = 8.060 seconds
 - PASS - [dma_m2m.test_tst_dma0_m2m_chan0_burst16] duration = 2.015 seconds
 - PASS - [dma_m2m.test_tst_dma0_m2m_chan0_burst8] duration = 2.015 seconds
 - PASS - [dma_m2m.test_tst_dma0_m2m_chan1_burst16] duration = 2.015 seconds
 - PASS - [dma_m2m.test_tst_dma0_m2m_chan1_burst8] duration = 2.015 seconds

------ TESTSUITE SUMMARY END ------

===================================================================
PROJECT EXECUTION SUCCESSFUL

Please send a patch with the const qualifier removed since you've already done the work and tested it.

PR open: #75245


Design a screen for users to choose the DID-method

Description
Design a screen for users to choose the DID-method when creating and importing identities.
Acceptance Criteria

[ ] Design a simple screen for users to select a DID-method. Here's the info we have to display for each: name, description, homepage and a icon

Later on #64, we may improve this screen explaining the user why this is important. Alternatively, we may merge both tasks. Something to discuss.

@satazor I believe we had this screen before, it was created with placeholders to other DID methods. The latest decision was hiding it from the design comps since it was only supporting did:ipid (not configurable, read only), but we can get back to this, definitely something to think later on.

@andreforsousa you are right, but for #249 user story to land, we need to get back to this screen. We will need to discuss how to make this as easy and intuitive as possible.


source2il: remove Asgn lowering

Summary
Simplify source2il by removing the in-place assignment lowering for
complex types.
Details
Assignments for complex types are supported in the L10, so the
lowering is no longer necessary.
The special-casing of complex types is removed for assignments. In
addition, wrapping an lvalue expression in a Copy operation is now
handled at where the expression is embedded (via the new genUse)
instead of where it's produced.

Notes For Reviewers

made possible by #42

Removing the Addr operations from the L10 output also improves the quality of the final bytecode, since taking the address of a local unconditionally disables the SSA form for it, which prevented some optimizations.
It's not yet possible to create code eligible for such optimization with the source language, so there's no effective difference at the moment.


Update to UCD 15

Since UCD 15 has been released (and Unifont 15.0.01), let's do it.

I've merged it. For future UCD updates, I'd like to have the underlying architecture changed. that is, the idea is to not have the UCD tables pre-parsed and converted to C++ inline, but to be able to runtime load UCD .txt files as much as possible. Not quite possible for all enum class types I think, but they should hopefully rarely change.


[Suggestion] Clean Done and Create History file

First, thank you for this awesome simple, yet very useful program.
To further improve usability of this program, I  want to propose 2  new features:

Batch remove done task. It can become a chore to remove done task 1-by-1. Perhaps an API such as please clean.
Any deleted should be moved to another file such as history.json. Then, introduce new API such as please history to display all done and "cleaned" task. This should allow the user to keep task history, while keeping the main task clean from bloat.

I understand the main concern is speed. Both of above suggestion should not impact the performance as it was outside of default please command.

Hey! Thank you for the suggestion! I have added the a "clean" command in the latest commit, just as you suggested.
I am not very sure about the history feature though, We will think about what we should do with that.

Thanks! History stuff is more like good to have, not essential. But clean, that's a huge QoL improvement!

This feature will be in v0.2

v0.2 is out :)


docs: Fix release checklist link in releases doc

The link to the release checklist in the Releases document was linking
to the parent document, not the separate checklist document.
Fixes #73.
Signed-off-by: James O. D. Hunt<EMAIL_ADDRESS>

lgtm


Include the code that we need

cin and cout require iostream.
QT_VERSION requires QtGlobal.
QSocketNotifier requires QSocketNotifier.
Unrelated to includes: let Qt figure out the spec.

Found by Jeremy Evans on behalf of the OpenBSD project.

@jferris (or anyone on a Mac) -- could you test this out? We removed the -spec argument from qmake as it does not support additional platforms without modification. Also, qmake seems to be able to figure out the spec on its own, and on OS X correctly detects clang instead of GCC.
I had tested this previously (on the breakpad branch) but never merged it in. I would expect things to be fine but I do not know the history of why the spec was set manually. It seems to have stuck around from when it was originally added to the Rakefile, and later moved to CapybaraWebkitBuilder.

(or anyone on a Mac)

I'm not on a Mac anymore.

I do not know the history of why the spec was set manually.

When I first started the project, it didn't compile correctly on OS X without the spec. I forget what the exact error was. This may have been improved since the early Qt versions we started on (maybe 4.6).

Hey @mhoran. If you can tell me the steps of what I need to do, I can test it out on my Mac laptop.

Thanks @nritholtz! Check out the include-things branch. All that needs to be checked is whether this branch still compiles on a Mac. rake build should be sufficient.

@mhoran  Here is gist with output after checking out include-things branch and running rake build.
Please let me know if there is anything I can do to help.

It looks like it was successful, despite the warning. Not sure where that's coming from. If you try to run the specs (bundle && rake), do they pass? (I know you have a separate issue for the compatibility specs).
Thanks!

Here's the bundle && rake results.
The only spec I see that failed is the selenium one, which I know is failing due to me having a newer version of Firefox. I have a PR, https://github.com/thoughtbot/capybara-webkit/pull/740, which bumps the version to support latest Firefox versions, which should have this spec pass.

Awesome, thanks!

Merged as 0506f24a0a254957047588d9315f5c927a9091a5.


Use min_specialization for annotations and type erasure

Enable the nightly min_specialization feature.
Provide a default implementatiion of Annotate for all T where T: serde::Serialize.  The default implementatiion provides no comments or formatting annotations.
Provide a form of type erasure (which does not use erased-serde) to allow for serializing of any dyn Annotate trait objects with comments and formatting annotations (because of the default implementation, all serde::Serialize objects implement Annotate).

Signed-off-by: Chris Frantz<EMAIL_ADDRESS>

As discussed in chat, I've cleaned up this PR:

Remove the prior type identification scheme.
Update the bazel rules: serde-annotate now requires nightly.
Re-run cargo-raze to eliminate no-longer-needed dependencies.


Show user information in event workflow details

Currently, if you go to the workflows tab of the event details modal and check which user started a workflow, you will see that users id. If you are using auth services like e.g. Shibboleth, the user id is likely to be a random assortment of letters and numbers that are not helpful in determining who started the workflow. This PR changes the "submitter" value returned from the backend, so that instead of the user id the user name and email are displayed.
How it currently looks:

With the PR:

This PR presumes that no one actually has a use for the user id in the workflow tab. If you think otherwise, please let me know. We can work something out, e.g. keeping the user id and creating an extra column for the user name and email.
Your pull request should…

[x] have a concise title
~[ ] close an accompanying issue if one exists~
[x] be against the correct branch
~[ ] include migration scripts and documentation, if appropriate~
[x] pass automated tests
[x] have a clean commit history
[x] have proper commit messages (title and body) for all commits

Why not show the username in brackets?

We have students and staff separated into different users, but they share the same email address if one person is a student and staff member at the same time. After this PR we would no longer be able to differentiate between them. The username would enable us to tell students and start members apart.

Sure, if the user ids are useful to people then I'll put them back. Did you mean like this?:

Yes, exactly!

Thanks for this improvement. I have successfully tested the change in our environment.


Bug: Some tours still not working

Related to issue #556
Not working in 1.5.0:

Lightsource Tracking
The Lost Citadel
Importing Monsters

Cause:
Compendiums have a consistent ID but folders not to. Also, folders structure changes are not maintained through updates. 3 of the tours require navigating the folder structure to open a compendium. A new approach is required to make tours work regardless of folder structure.

Best solution I could see was to ignore folder structure and simply render the compendium.


Need SubsamplingScaleImageView.visibleFileRect(Rect fRect) analog

I need to get the visible area Rect of the source bitmap to save it into another one (android wallpapers app).
It seems I need something like davemorissey SubsamplingScaleImageView.visibleFileRect(Rect fRect) in ZoomableState.
I tried to look at contentTransformation & transformedContentBounds, but did not succeded in getting the exact rect which is shown.
It would be nice if you add such method.

This sounds similar to https://github.com/saket/telephoto/issues/64#issuecomment-2086222965 maybe the solution works for you as well?

Thanks, but this is different. Composable-to-bitmap api writes to a bitmap a piece of the screen and I need a piece of the original bitmap.

Hmm, I would have expected transformedContentBounds to give you the bounds you're looking for.
https://github.com/saket/telephoto/blob/7015d6eed1c1ff876844682728912bcfb94773f9/zoomable/src/commonMain/kotlin/me/saket/telephoto/zoomable/RealZoomableState.kt#L179-L184
Are its numbers significantly off?

I made some experiments.
I have  bitmap 1920 x 1080 (aspect ratio = 1,777777777777778) and screensize = 2000 x 1200 (aspect ratio = 1,666666666666667). I show it full size with ContentScale.Crop and want to get the visible area from the bitmap.
If I pan to top left corner, then visible bitmap crop area will be
left=0, top=0, right=1800, bottom=1080
According to logs I have
contentSize=Size(1920.0, 1080.0) offset=Offset(0.0, 0.0) scaleX=1.1111112 scaleY=1.1111112 transformedContentBounds=Rect.fromLTRB(0.0, 0.0, 2133.3, 1200.0)
If I pan to bottom right corner , then visible bitmap crop area will be
left=120, top=0, right=1920, bottom=1080
According to logs I have
contentSize=Size(1920.0, 1080.0) offset=Offset(-133.3, 0.0) scaleX=1.1111112 scaleY=1.1111112 transformedContentBounds=Rect.fromLTRB(-133.3, 0.0, 2000.0, 1200.0)
So I finnaly succedded to get this area correctly with the following code
`getCroppedRectFromBitmap(
sourceBitmap,
contentWidth = zoomableState.contentTransformation.contentSize.width,
contentHeight = zoomableState.contentTransformation.contentSize.height,
scaledRectWidth = imageContentSizeInDp.width.value * density.density,
scaledRectHeight = imageContentSizeInDp.height.value * density.density,
scaledOffsetX = zoomableState.contentTransformation.offset.x,
scaledOffsetY = zoomableState.contentTransformation.offset.y,
scaleX = zoomableState.contentTransformation.scale.scaleX,
scaleY = zoomableState.contentTransformation.scale.scaleY,
)
fun getCroppedRectFromBitmap(
sourceBitmap:Bitmap,
scaledRectWidth: Float,
scaledRectHeight: Float,
scaledOffsetX: Float,
scaledOffsetY: Float,
scaleX: Float,
scaleY: Float,
): Bitmap? {
val scale = max(scaleX, scaleY)
val left = abs(scaledOffsetX / scale).toInt()
val top = abs(scaledOffsetY / scale).toInt()
var width = (scaledRectWidth / scale).toInt()
width = min(width, sourceBitmap.width - left)
        var height = (scaledRectHeight / scale).toInt()
        height = min(height, sourceBitmap.height - top)
        return Bitmap.createBitmap(sourceBitmap, left, top, width, height)
}

`


Pull Calculator

Calculates number of required pulls if given desired percentage of at least one (default 50%).
OR
Calculates percentage of at least one if given number of pulls.

Given number of pulls can instead be a combination of orundum, originate prime and/or max $USD willing to spend

Deadline: Before 2020-05-27 10:00 (UTC-7)


SO-4640: Change Date to String in create version API

Change effectiveTime property to String type to avoid possible timezone issues on Java API level.
This is an improvement on the existing functionality, everything should behave the same as before.

Codecov Report

Merging #749 (ff90668) into 7.x (55d421f) will increase coverage by 0.01%.
The diff coverage is 57.77%.

@@             Coverage Diff              @@
##                7.x     #749      +/-   ##
============================================
+ Coverage     58.78%   58.79%   +0.01%
+ Complexity    12748    12746       -2
============================================
  Files          1772     1771       -1
  Lines         65308    65290      -18
  Branches       6429     6427       -2
============================================
- Hits          38390    38387       -3
+ Misses        23910    23891      -19
- Partials       3008     3012       +4

Impacted Files
Coverage Δ
Complexity Δ

...snowowl/core/rest/admin/RepositoryRestService.java
11.32% <0.00%> (ø)
3.00 <0.00> (ø)

...nal/snowowl/core/rest/codesystem/VersionInput.java
100.00% <ø> (ø)
7.00 <0.00> (ø)

...tional/snowowl/core/branch/BranchMergeRequest.java
84.61% <0.00%> (ø)
4.00 <0.00> (ø)

...nal/snowowl/core/codesystem/CodeSystemVersion.java
56.41% <ø> (ø)
15.00 <0.00> (ø)

...version/CodeSystemVersionCreateRequestBuilder.java
100.00% <ø> (ø)
6.00 <0.00> (ø)

...c/com/b2international/snowowl/core/date/Dates.java
88.88% <ø> (+18.30%)
12.00 <0.00> (ø)

...wowl/core/locks/DatastoreOperationLockManager.java
79.42% <ø> (ø)
49.00 <0.00> (ø)

...nal/snowowl/core/locks/OperationLockException.java
31.57% <18.75%> (+6.57%)
2.00 <1.00> (+1.00)

...re/codesystem/version/VersioningConfiguration.java
70.58% <50.00%> (+5.88%)
6.00 <1.00> (ø)

.../core/repository/RepositoryTransactionContext.java
62.91% <66.66%> (ø)
39.00 <0.00> (ø)

... and 15 more

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 55d421f...ff90668. Read the comment docs.

Codecov Report

Merging #749 (ff90668) into 7.x (55d421f) will increase coverage by 0.01%.
The diff coverage is 57.77%.

@@             Coverage Diff              @@
##                7.x     #749      +/-   ##
============================================
+ Coverage     58.78%   58.79%   +0.01%
+ Complexity    12748    12746       -2
============================================
  Files          1772     1771       -1
  Lines         65308    65290      -18
  Branches       6429     6427       -2
============================================
- Hits          38390    38387       -3
+ Misses        23910    23891      -19
- Partials       3008     3012       +4

Impacted Files
Coverage Δ
Complexity Δ

...snowowl/core/rest/admin/RepositoryRestService.java
11.32% <0.00%> (ø)
3.00 <0.00> (ø)

...nal/snowowl/core/rest/codesystem/VersionInput.java
100.00% <ø> (ø)
7.00 <0.00> (ø)

...tional/snowowl/core/branch/BranchMergeRequest.java
84.61% <0.00%> (ø)
4.00 <0.00> (ø)

...nal/snowowl/core/codesystem/CodeSystemVersion.java
56.41% <ø> (ø)
15.00 <0.00> (ø)

...version/CodeSystemVersionCreateRequestBuilder.java
100.00% <ø> (ø)
6.00 <0.00> (ø)

...c/com/b2international/snowowl/core/date/Dates.java
88.88% <ø> (+18.30%)
12.00 <0.00> (ø)

...wowl/core/locks/DatastoreOperationLockManager.java
79.42% <ø> (ø)
49.00 <0.00> (ø)

...nal/snowowl/core/locks/OperationLockException.java
31.57% <18.75%> (+6.57%)
2.00 <1.00> (+1.00)

...re/codesystem/version/VersioningConfiguration.java
70.58% <50.00%> (+5.88%)
6.00 <1.00> (ø)

.../core/repository/RepositoryTransactionContext.java
62.91% <66.66%> (ø)
39.00 <0.00> (ø)

... and 15 more

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 55d421f...ff90668. Read the comment docs.


feat: batch array changes

Previously only property changes were batched; now array changes in
record arrays are also batched.  Note that this is not the case for
tracked arrays, but this feature should be added there as well.
As part of this change, remove ArrayProxy as the base class of
RecordArray.  Although our arrays are array proxies, the base class is
not adding value and it's easier to implement all of the logic
ourselves.

Testing in-app for additional context.
Also this is a breaking change.  Although I don't expect it to actually impact apps.


Can't download finalbosses.com's videos.

Hello.
Example: $ youtube-dl -v http://finalbosses.com/playing-with-sacks-golden-axe/
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://finalbosses.com/playing-with-sacks-golden-axe/']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.08.28
[debug] Python version 2.7.3 - Linux-3.2.0-4-amd64-x86_64-with-debian-7.9
[debug] exe versions: avconv 1.2.12, avprobe 1.2.12, ffmpeg 1.2.12, ffprobe 1.2.12, rtmpdump 2.4
[debug] Proxy map: {}
[generic] playing-with-sacks-golden-axe: Requesting header
WARNING: Falling back on generic information extractor.
[generic] playing-with-sacks-golden-axe: Downloading webpage
[generic] playing-with-sacks-golden-axe: Extracting information
ERROR: Unsupported URL: http://finalbosses.com/playing-with-sacks-golden-axe/
Traceback (most recent call last):
File "/home/ant/bin/youtube-dl/youtube_dl/extractor/generic.py", line 1211, in _real_extract
doc = parse_xml(webpage)
File "/home/ant/bin/youtube-dl/youtube_dl/utils.py", line 1594, in parse_xml
tree = xml.etree.ElementTree.XML(s.encode('utf-8'), **kwargs)
File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 1301, in XML
parser.feed(text)
File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 1643, in feed
self._raiseerror(v)
File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 1507, in _raiseerror
raise err
ParseError: undefined entity: line 15, column 0
Traceback (most recent call last):
File "/home/ant/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 655, in extract_info
ie_result = ie.extract(url)
File "/home/ant/bin/youtube-dl/youtube_dl/extractor/common.py", line 287, in extract
return self._real_extract(url)
File "/home/ant/bin/youtube-dl/youtube_dl/extractor/generic.py", line 1804, in _real_extract
raise UnsupportedError(url)
UnsupportedError: Unsupported URL: http://finalbosses.com/playing-with-sacks-golden-axe/
Thank you in advance. :)

Works fine with latest version.

dstftw: Yep, it works now. Thanks. :)


discussion: allow client to receive arbitrary data

In developing a custom server I'm finding the need to send more data to the client than just the url, id, etc.
From this chunk of code it's clear the data is strictly controlled to 5 keys.
https://github.com/localtunnel/localtunnel/blob/master/lib/Tunnel.js#L62
Is there a specific reason the client would want to prohibit extra data that 99.9999% of the time wouldn't exist? Possible security implications is the best guess I can provide, but cannot actually think of a specific issue. If that's the case possibly hide it behind an option so only module authors consuming localtunnel can enable it.
Happy to make a PR or two demoing different approaches if this is even reasonable.

Nope, there is no reason other than there was no extra data since the client implementation evolved with the server. If you have a specific use case or need for more data I would be interested to hear it.

So it turns out this didn't actually suit my needs as I want data to be returned after the tunnel is opened, but I'll share the use case for reference.
The project I'm working on doesn't require a public facing tunnel but rather exposes the tunnel to an internal testing services. The tunnel server and testing services reside on the network, after the tunnel is established the server dispatches the test server to test on the generated domain.
The desire was to send the results back over that original HTTP request, but as the server doesn't establish the tunnel until after the request ends I ended up just using a second request to fetch the results.


No compatible SDK found

Dear supporters,
Problem encountered on https://dotnet.microsoft.com/en-us/learn/dotnet/hello-world-tutorial/create
Operating System: windows 10
I am following a course on Test Driven software development on PluralSight. I was asked to install MS Visual Studion Code and .NET Core, which I have done. The verification for .NET Core installation (typing dotnet in cmd) passed.
Since stuff rarely works, I usually try to get some background understanding, so I decided to follow the .NET Core tutorial.
The command
dotnet new console -o MyApp -f net6.0
yields
Could not execute because the application was not found or a compatible .NET SDK is not installed.
Possible reasons for this include:

You intended to execute a .NET program:
The application 'new' does not exist.
You intended to execute a .NET SDK command:
It was not possible to find any installed .NET SDKs.
Install a .NET SDK from:
https://aka.ms/dotnet-download

The MyApp folder was not created.
Indeed. As far as know, I haven’t installed any .NET SDKs and I was never asked to.

Thanks for your response.
Sorry, I was mistaken. I have actually installed SDK 6.0. (I apparently thought Core was not SDK.)
Typing in the command prompt
dotnet --info
yields
Version: 6.0.3
Commit:  c24d9a9c91
.NET SDKs installed:
No SDKs were found.
.NET runtimes installed:
Microsoft.AspNetCore.App 6.0.3 [C:\Program Files (x86)\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 6.0.3 [C:\Program Files (x86)\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 6.0.3 [C:\Program Files (x86)\dotnet\shared\Microsoft.WindowsDesktop.App]
To install additional .NET runtimes or SDKs:
https://aka.ms/dotnet-download
I dotnet-sdk-6.0.400-win-x64.exe again. (Version 6.0.4 while info mentions version 6.0.3)
In the Microsoft .NET DSK 6.0.400 installer pick Repair. It was allegedly successful.
Dotnet --info still yields the same.
dotnet new console -o MyApp -f net6.0 again yields the error.
I go to aka.ms/dotnet-download,but it is unclear what I am supposed to download.
So in stead I uninstall it with  Microsoft .NET DSK 6.0.400 installer.
dotnet –info still yields the same result.
So I uninstall Microsoft .NET Runtime 6.0.3 (x64) and (x86). (There are 2 more .NET applications I leave.)
Then I install  Microsoft .NET DSK 6.0.400 again.
dotnet --info again yields the same (version 6.0.3).
The command
dotnet new console -o MyApp -f net6.0
again yields what looks like the same response.

This might be an issue with PATH.
Check the output of the where.exe dotnet command and if your system is x64 and the x86 path returns first, please follow the steps at https://github.com/dotnet/core/issues/5962#issuecomment-780084535 to fix the issue.

I have an x64 system.
where.exe dotnet
yields
C:\Program Files (x86)\dotnet\dotnet.exe
C:\Program Files\dotnet\dotnet.exe
My PATH System Variable is (no, I don’t get the fancy table) :
%JAVA_HOME%\bin;%MAVEN_HOME%\bin;TRICENTIS_SEARCH%;%SystemRoot%\system32;%SystemRoot%;%SystemRoot%\System32\Wbem;%SYSTEMROOT%\System32\WindowsPowerShell\v1.0;%SYSTEMROOT%\System32\OpenSSH;C:\Program Files (x86)\dotnet;C:\Program Files\dotnet;
Thus, the pertinent paths are in there, but I copy
C:\Program Files\dotnet
to before
C:\Program Files (x86)\dotnet\
where.ex dotnet still lists the x86 one first.
dotnet --info still yields the same, as does dotnet new console -o MyApp -f net6.0.

So, certainly it's the issue with the path that you're running into.
Make sure the path is updated with C:\Program Files\dotnet coming up first.

I restart the computer.
I instruct
where.exe dotnet
Response :
C:\Program Files\dotnet\dotnet.exe
C:\Program Files (x86)\dotnet\dotnet.exe
dotnet –info
yields
.NET SDK (reflecting any global.json):
Version:   6.0.400
Commit:    7771abd614
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.19044
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\6.0.400\
global.json file:
Not found
Host:
Version:      6.0.8
Architecture: x64
Commit:       55fb7ef977
.NET SDKs installed:
6.0.400 [C:\Program Files\dotnet\sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 6.0.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 6.0.8 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 6.0.3 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.8 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 6.0.3 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 6.0.8 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Download .NET:
https://aka.ms/dotnet-download
That is different than before.
I instruct
dotnet new console -o MyApp -f net6.0
That yields much info than the animation shows in the command prompt and a folder in the wrong place. I moved it to a different location.
The rest of the tutorial worked. Thanks for your help.

This is a new issue, but it is also about some version not being found.
The teacher gave some C# code in Visual Studio and used dotnet watch test in the command prompt. Then good stuff happened.
Wen I instruct
C:\Users\Knarf\Documents\School\Informatica\Software Testing\TDD - The Big Picture\Code examples\TDD-TheBigPicture-master\FizzBuzz-Start\FizzBuzz.Tests>dotnet watch test
I get
[stuff about framework no longer being supported and some other stuff]
Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
Testhost process exited with error: You must install or update .NET to run this application.
App: C:\Users\Knarf.nuget\packages\microsoft.testplatform.testhost\15.0.0\lib\netstandard1.5\testhost.dll
Architecture: x64
Framework: 'Microsoft.NETCore.App', version '1.1.2' (x64)
.NET location: C:\Program Files\dotnet
The following frameworks were found:
6.0.3 at [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
6.0.8 at [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Learn about framework resolution:
https://aka.ms/dotnet/app-launch-failed
To install missing framework, download:
https://aka.ms/dotnet-core-applaunch?framework=Microsoft.NETCore.App&framework_version=1.1.2&arch=x64&rid=win10-x64
. Please check the diagnostic logs for more information.
Test Run Aborted.
dotnet watch ❌ Exited with error code 1
I tried some stuff on docs.microsoft.com/nl-nl/dotnet/core/runtime-discovery/troubleshoot-app-launch?pivots=os-windows to resolve the issue.  (en-us i.s.o. nl-nl looks like the English version of the page.)
The link leads to a download of version 1.1.13, which is allegedly outdated. Should I install it anyway ?
I didn’t try ‘Binaire bestanden downloaden’ because it told me to use a file (dotnet-sdk-6.0.400-win-x64.zip) that looks identical to what I already have.
I also didn’t try forward-rolling because of the warning that that could change the behaviour of the application.
How can I use dotnet watch test on the exercise project ?


Deleting an answer option when creating a question only works when pressing '+' afterwards

Works on fix/small_bugs


add version in Mapzen documentation

It is quite difficult to identify to which version the documentation on Mapzen's website refers.
https://mapzen.com/documentation/tangram/Javascript-API/
Since most of the users of tangram.js are used to Leaflet, maybe a similar pattern should be used:
http://leafletjs.com/reference-1.0.3.html
Also, it is puzzling for an outsider why one should go Mapzen's site for the documentation but be on an independent Github account for the issues ;)

Hi @katossky – you're right, we have a little notice on https://mapzen.com/documentation/tangram/Tangram-Overview/ stating that the documentation always refers to the latest version of the library, but I've added another to the docs homepage to make it more prominent. In the future we may version our documentation, but so far we simply make a note in the docs when a method or property is deprecated.
And thanks for the feedback on the documentation! We're in the process of reorganizing and redesigning the website, so hopefully this will be more clear in the future.


Airplay doesn't show on Apple devices on initial boot up.

Describe the problem
When booting my rp3b+ airplay service starts and runs fine, but doesn't show up on any devices on the local wifi. A restart of the airplay service corrects the issue. Happens on both my balena-sound Rpi devices.
To Reproduce
Steps to reproduce the behavior:

Turn on pi.
Wait for it to boot and be 'ready'
Open another apple device (iphone, ect) and look for device listing on airplay - not there.
open balena sound app, and restart airplay service, then device shows up and works at expected.

Expected behavior
Airplay should be available upon initial startup.
Device information

Board (hardware info)  Raspberry Pi 3B+
Device type (as seen in balenaCloud): [e.g. Raspberry Pi 3 (using 64bit OS)]
balenaOS version: balenaOS 2.80.3+rev1

Additional context
Add any other context about the problem here.

I have the same issue in a RP3 model B with a fresh installation. After rebooting the airplay service the device appeared, but not at first.

I have the same problem, I must restart airplay service for see rpi on apple device.
airplay  fatal error: pa context is not good -- the error message "Connection terminated"

Fixed in v3.6.11. For details see https://github.com/balenalabs/balena-sound/pull/508

Same happens on clean install on RPi3B with 3.9.6


Implement MyPy compatibility

I'm hitting "Overall lines covered not found" (https://github.com/irongut/CodeCoverageSummary/blob/f138489/src/CodeCoverageSummary/Program.cs#L172C41-L172C74) when trying to display coverage data produced by the most used Python type checker — MyPy.
It's generated using the --cobertura-xml-report CLI flag per https://mypy.readthedocs.io/en/stable/command_line.html#cmdoption-mypy-cobertura-xml-report.
To grab an example, go to https://app.codecov.io/github/aio-libs/yarl/commit/1fe1e02357537b5cf348b2088bcedb30847fbbf9, find an upload with the MyPy flag in the widget on the right, click Download. You'll get the payload sent to the Codecov service. The beginning on the text is just a list of files, so scroll down to the <<<<<< network delimiter and you'll see the XML under it.

I also filed an issue over @ MyPy, as it seems to be violating the spec: https://github.com/python/mypy/issues/17689.

Looks like this should be fixed in v1.12.0. Just needs to be tested.


Only authorized users logged in with CAS can access the site

The ability to view ScannedBooks (or the Mainfest that represent it), etc. should be subject to  the access controls set when creating the resource.


Hound add ssh key to docker image

When running hound in docker it is not possible to add ssh keys there to be able to index private github repositories.
If it possible then it is not documented anywhere.

Try mount your folder ssh
docker run -d -p 6080:6080 --name hound -v $(pwd):/data  -v ~/.ssh:/root/.ssh etsy/hound

Thanks that helped!
Closing this issue.

@mbernabeu run the command givens
panic: invalid character '"' after object key:value pair

goroutine 1 [running]:
main.main()
	/go/src/github.com/etsy/hound/cmds/houndd/main.go:129 +0x806

Do you have any ideas?

@mbernabeu never minds, my bug.


Please use - and not _

In npm packages we would usually use - to separate words.
Please use the same for the name of the repository.

i do not have the rights to rename the repo anymore.

i think you did it


Add double dashes to README.md

I tried to use the CMD-line options mentioned in the README file, but as I expected, they missed a dash and thus resulted in an error if used. I've added a second dash to make them work, in all occurrences in the README.

Thanks!


Stimulated peripheral blood mononuclear cells (Ucar)

Primary Wrangler:
Enrique
Secondary Wrangler:
Ray, Marion

Associated files:
StimulatedPBMCs-issue83
Google Drive:
https://drive.google.com/drive/folders/1tec_2-dgyH-z8Ak2taBhnJdPsMLLpANB
Key Events

[x] Send initial instructions about contributing, including questionnaire and T&Cs
[x] Receive project questionnaire
[x] Assign the Primary Wrangler
[x] Receive T&C form filled out by at least 1 representative of group/project
[x] Send custom metadata template spreadsheet, how-to guide and directions for submitting data files
[x] Receive spreadsheet (to a sufficient standard to submit dataset)
[x] Receive data (to a sufficient standard to submit dataset)
[x] Curate metadata spreadsheet with ontologies
[x] Upload sheet to validate metadata
[x] Run graph validator
[x] Check linking
[x] Validate data files
[x] Ask the Secondary Wrangler for an end-to-end review of the project. Ask the Expertise Wrangler to review specific tabs if needed
[x] Get final approval of submission including metadata spreadsheet and data files
[x] Submit to production and inform ingest-devs the project is ready for archiving
[ ] Email UX to inform that they're ready to send survey

Please track the below as well as the key events:

Track date first spreadsheet received and final spreadsheet sent by editing ticket to include
date next to event.
Track spreadsheet iterations by placing asterisks next to receive spreadsheet event.
Track any metadata issues/tickets made for dataset with a bulleted list of links under
received spreadsheet event. Links should be to the ticket in the metadata repo.

Received HCA Dataset Questionnaire response.
Pre-filling HCA spreadsheet to be sent.

There are responses in questionnaire that I need to follow up before sending a reply:
* Were the donors/mice alive at the time of sample collection?  Response: Alive
* What type of consent do you have to release the data and metadata for your dataset?   [Donor level metadata]	Response: not sure

Sent spreadsheet and a reminder about T&C form.

Sent email with data upload instructions and asked if they need upload area credentials (Ray has already created an upload area for their project)

Sent them an email with upload area uuid and aws credentials
Gave them an estimate of within 2 weeks to get accessions to them

Still awaiting reply from Nathan with:

filled in metadata spreadsheet
confirmation that they have uploaded their data to the upload area I sent them

We got a spreadsheet from Nathan, sent an email with the spreadsheet back (curated) for them to review and make sure we fully captured their experiment.
Decisions with this dataset are as following:

Archive CITE-Seq (with a little bit of manual metadata tweaking)
Archive VCF files in EVA

Currently detected an error (Tried to redefine a protected keyword in VCF) in one of the files. Checking all the files to make sure it is the only error before coming back to them

DON'T EXPORT FOR NOW: Before exporting, we need to figure out how to change the metadata in ingest with minimal impact.

The latest vcf files are uploaded and have been validated - They are valid.
Filling the EVA metadata. Will need to wait until we have the BioSamples/ENA accessions

Received latest spreadsheet on friday, saved it in the drive folder

Updated the spreadsheet with a couple of corrections and additions:

Added Ray, Marion and myself as wranglers
Added an enrichment protocol column in cell suspension to link the enrichment properly.
Added ontologies. There is no ontology for Illumina Novaseq S4, so I need to request the term.

@rays22 will secondary wrangle this dataset.

Spreadsheet version hca_spreadsheet-2020-Lawlor_ES_no_vcf_v5_ontologies.xlsx has failed the following graph validator tests:
20-09-07 14:34:51 [ingest_graph_validator.actions.test_action] - ERROR: test [no_orphans.adoc] failed: non-empty result.

20-09-07 14:34:52 [ingest_graph_validator.actions.test_action] - ERROR: test [min_experimental_design_length_5.adoc] failed: non-empty result.

20-09-07 14:34:52 [ingest_graph_validator.actions.test_action] - ERROR: test [ensure_seq_libr_prep_protocols.adoc] failed: non-empty result.

The problems might stem from the way the pooled cell suspensions are modelled in the metadata spreadsheet.
There are 10 distinct cell suspensions in the spreadsheet. Shouldn't there be only a single pooled cell suspension, linked to 10 distinct specimen, linked to 10 donors?

The metadata in the spreadsheet and the experimental design graph look correct to me for spreadsheet version hca_spreadsheet-2020-Lawlor_no_vcf_v6_ontologies_Ray_review.xlsx.

CITE-Seq data is archived under ENA (Accessions under latest spreadsheet)

VCF data is currently being archived

Full experimental design graph from hca_spreadsheet-2020-Lawlor_no_vcf_v6_ontologies_Ray_review.xlsx.

Project label: CITEseqPBMCProject
Dissecting the protein and transcriptional responses of human immune cells to T cell and monocyte specific activation
Submission Sep 8, 2020, 10:32:12 AM; dfb86fe3-ef1f-42ed-b559-d74f4b3b3a11 has been archived in BioStudies and BioSamples.
BioStudies accession: S-SUBS15
INSDC project accession: ERP124005

@ESapenaVentura :  is there anything else left to archive (e.g. VCFs)?

If I remember well, the EVA submission gets added to the normal ENA submission so we didn't get any accession from EVA
Might need to double check, though

@ESapenaVentura is this archived already? are the next asks export (when we start that again) and send to SCEA?

This is, for the moment, not suitable for SCEA
But yes, it is already archived

This dataset has not been made public yet, we are waiting until they publish the manuscript.
Another problem with this dataset is the multiplexing, which might prove difficult as the pipelines currently don't support that.

This dataset has been exported to the Terra staging area.

Has this one already gone to SCEA or do they not accept CITE-seq

There is now a paper for this dataset https://www.frontiersin.org/articles/10.3389/fimmu.2021.636720/full that the authors would like included

I can confirm that the update was successfully exported.

New metadata file with the publication update: project_UUID/metadata/project/projectUUID_2021-05-12T14:05:50.850000Z.json

    "publications": [
        {
            "doi": "10.3389/fimmu.2021.636720",
            "url": "https://doi.org/10.3389/fimmu.2021.636720",
            "authors": [
                "Lawlor N",
                "Nehar-Belaid D",
                "Grassmann JDS",
                "Stoeckius M",
                "Smibert P",
                "Stitzel ML",
                "Pascual V",
                "Banchereau J",
                "Williams A",
                "Ucar D"
            ],
            "title": "Single Cell Analysis of Blood Mononuclear Cells Stimulated Through Either LPS or Anti-CD3 and
Anti-CD28",
            "pmid": 33815388
        }
    ],

[ ]  Notified the HCA production data import team about the export.

There was an issue with importing the update into the DCP.

Hi ray, the staging area located at  gs://broad-dsp-monster-hca-prod-ebi-storage/prod/efea6426-510a-4b60-9a19-277e52bfa81 is failing json validation on the collection_time property for several of the specimen_from_organism metadata files. For example:
gs://broad-dsp-monster-hca-prod-ebi-storage/prod/efea6426-510a-4b60-9a19-277e52bfa815/metadata/specimen_from_organism/14636efe-85cd-4c22-8ca6-e119e7600714_2020-09-08T09:32:16.090000Z.json fails the oneOf validation against the specimen_from_organism schema for the value 2018-07. I can provide more details if needed (cc: Claire Rye)
...
To clarify, the dataset is fine and will be included in the next release (7) without any changes from our side or need for re-export. Jeff (he/him) do you want us to complete another import form?
...
Jeff Korte  6 days ago
Claire Rye, yes please.  Please indicate it is for Release 7.

I have completed a new import form for release 7.


[Improvement][Seatunnel-web] Add support to execute seatunnel-web REST API e2e

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
Write base classes for writing integration test cases.
The base test class must initiate the SeaTunnel-Web server using an in-memory database.
It should also launch the SeaTunnel-Engine to perform end-to-end functionality verification.
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

+1

Raised PR https://github.com/apache/seatunnel-web/pull/180

PR is merged https://github.com/apache/seatunnel-web/commit/7205efe14cf42683ffb9a22113d6d4fe32e6f857


Needs loose description Luhn algorithm and potential applications for readme

Firstly this sounds pretty cool! I was thinking we could yoink some text from wikipedia with an acknowledgement. This is a bit wordy, but I think the relevant parts are, "This is a few places where you might need it" "The algorithm is public domain" "It is not designed for cryptographic security but rather to prevent accidental entry".
Thanks for your contribution to humanity, every little bit adds up.

From wikipedia
The Luhn algorithm is used in a variety of systems, including:
Credit card numbers
IMEI numbers
National Provider Identifier numbers in the United States
Canadian social insurance numbers
Israeli ID numbers
South African ID numbers
South African Tax reference numbers
Swedish national identification numbers
Swedish Corporate Identity Numbers (OrgNr)
Greek Social Security Numbers (ΑΜΚΑ)
ICCID of SIM cards
European patent application numbers
Survey codes appearing on McDonald's, Taco Bell, and Tractor Supply Co. receipts
United States Postal Service package tracking numbers use a modified Luhn algorithm[4]
Italian VAT numbers (Partita Iva)[5]
The Luhn algorithm or Luhn formula, also known as the "modulus 10" or "mod 10" algorithm, named after its creator, IBM scientist Hans Peter Luhn, is a simple check digit formula used to validate a variety of identification numbers. It is described in US patent 2950048A, granted on 23 August 1960.[1].
The algorithm is in the public domain and is in wide use today. It is specified in ISO/IEC 7812-1.[2] It is not intended to be a cryptographically secure hash function; it was designed to protect against accidental errors, not malicious attacks. Most credit cards and many government identification numbers use the algorithm as a simple method of distinguishing valid numbers from mistyped or otherwise incorrect numbers.

Thank you for the suggestion! This was mostly an exercise to improve my F# skills, so it is still in its infancy, but I will definitely consider adding more documentation as I clean things up.

Yeah I figured even if people don't leverage it yet, it would still be good for teaching :).


added-display-captions-option

Added an option to display instagram post captions as overlay on the posts on hovering.  display_caption is true by default. A screenshot of the caption visible on hovering is attached:

Looks great! I made some small changes:

Changed display_captions default value to false to avoid changing the current behavior for users that use the CDN.
Replaced template string for a normal string (scaping line breaks) as this is not compatible in any version of IE.
Added a condition to avoid adding any caption styles if display_captions is disabled.
Changed max-width:100% for width:100% to avoid cases like this:

->

Thank you!

Closes #14

This was my first ever pull request. Thank you for letting me contribute, and for explaining the tweaks you made. I learnt a lot from this experience 😊

Noo, thank you for your work! Really appreciated


Image upload button

Any ideas?

You could try to hack a solution with the javascript part. But why not just use the extension already out there. medium-editor-insert-plugin


Add support for GEMM e2e Test For CUDA backend on F16 input

This pull request adds support for testing e2e GEMM on a CUDA backend for F16 input and F16 accumulation.

Functional testing of F16 GEMMs requires setting the right tolerance value to ensure correctness checks.
Tolerance-based testing is fragile and finding tolerances is hard.
Instead we fill the buffers as small integers centered at zero and test for equivalence.

This pull requests adds e2e_matmul_direct_f16_gpu_large_LLVMGPUMatmulTensorCore_cuda_cuda more e2e GEMM test to CUDA backend. We have not total 4 e2e GEMM tests on CUDA backend.
manigupta@manigupta-gpu-a100 ~/cpu_machine_workspace/repos/iree/iree_tree_1/iree-build-debug $ ctest -j96 -R e2e_matmul.*cuda

1/4 Test #41: iree/tests/e2e/matmul/e2e_matmul_direct_f32_gpu_large_LLVMGPUMatmulTensorCore_cuda_cuda ...   Passed    2.83 sec
2/4 Test #42: iree/tests/e2e/matmul/e2e_matmul_direct_f16_gpu_large_LLVMGPUMatmulTensorCore_cuda_cuda ...   Passed    3.09 sec
3/4 Test #40: iree/tests/e2e/matmul/e2e_matmul_direct_f32_gpu_large_LLVMGPUMatmulSimt_cuda_cuda .........   Passed    3.38 sec
4/4 Test #43: iree/tests/e2e/matmul/e2e_matmul_direct_f32_large_split_k_cuda_cuda .......................   Passed    4.43 sec

(sorry if some of these things have been discussed in prior comments - I hadn't been following this PR and am just looking at what the latest commits show - but the vm value changes will have to be dropped and the e2e matmul test will need its own copy of those types if you go the generic route - but I strongly suggest reworking that to something simpler)


AUTH SDK :Missing client_id in options.cookies in authorize endpoint

https://github.com/sst/ion/blob/72fab0f5805e09c673cf781a4e5e2dc24fbd4750/sdk/js/src/auth/handler.ts#L283
https://github.com/sst/ion/blob/72fab0f5805e09c67jjjj3cf781a4e5e2dc24fbd4750/sdk/js/src/auth/handler.ts#L247
client_id is not added to the options.cookies which means its not included in the payload that is checked in the /token endpoint

PR is here #310


fix(open-scd): import locales from relative URL

Imports locales from the right place when a bundled version of open-scd-core is being loaded from a remote host.

This is needed for the same scenario as #82 .


[DOC] Added Table of Contents to ReadMe

Added a Table of Contents to ReadMe to make navigation easier.

A welcome change, thank you! :)


Unable to update to latest version

Describe the bug
Unable to update to latest version, it looks like you restricted the package to Laravel 10 only, it worked just fine in the past.
Steps To Reproduce

Show the code of your field.
What changes have you made to your global config?
Version of:
This package 1.3.2
Filament 3.2.92
PHP 8.3.8
Laravel 11.11.1
Anything else that you think might be relevant

Expected behavior and actual behavior
Since version 1.3.2 worked on Laravel 11, i would expect 1.4 to work also
Screenshots

Same here =(

Ah, sorry. This was an oversight on my part. New version coming up asap.

This is now fixed in version 1.4.1