No IBeacon detected

Hi Mmazzarolo, thank you, your plugin is good !
But i have a problem, when a launch my app, my ibeacon is detected one time  every x secondes ! Sometimes, no ibeacon, some time i can see my ibeacon... I have tried on iPhone, and it works great!
Do you have an idea?

Hey @Clowning, unfortunately I'm not able to maintain anymore this repo :( #31
Anyway I recently merged a PR #34 that might help you.
I have not published it to npm though, but you can try if from the master branch of this repository (let me know if it works for you, so I can push it to npm).

@mmazzarolo No you can't do that :o !! It's the only repo for ibeacons on Android :(( !
Ok so, i'ts definitively better, but not perfect ! Thank! I think you can push it to npm !

@Clowning I know, I tried to reach for help/maintainers but without success ;(
Thank you for the feedback though, I'll publish it to npm this evening!

Yes .. It's weird ! Lot of people like Java aha !
Thank you :-) !

@mmazzarolo But do you have an idea to keep my application alive when user kill it ? :/

If you mean when the app is removed even from the background... no.
The only way to do something like this is creating an Android Service (from the Java code), I'm sorry.  I started working on it in the past but had to give up because it was a time consuming task and I wasn't able to test it (no beacons available).

Arf !
Mhh but it's possible to run my javascript code from a Android Service ? I've never code that !
Send me your address i offer you a beacon :D aha

Sad news... Even iOS one seems like no more maintained.
I can take case of iOS version (I may since I need it) but android...

:(

I didn't even try bridging with android.
If it doesn't require to know android dev.

I'm at this 0% pure android experienced :smile:.

Hi @mmazzarolo,
As iOS version is no more maintained (react-native-ibeacon).
I've decided to handle it rather waiting a kind soul would evolve/fix it for us al (you may remember that at the end of the year 2016, RN introduced a bad breaking change in iOS, ex: #import RCTBrige is now #import <React/RCTBridge>.).
Here is my starting repository: react-native-ibeacons.

You did a great job on android so I wanna know if you are ok that I take your android version inside my repository?

I will try to merge iOS and Android versions so that javascript is the same (as much as possible).
In fact I hope I make it a complete RN module for iBeacons.
I don't have beacons but I have iPhones to create beacons, it is enough 👍
So I can deal beacons easily.

@MacKentoch of course man! Hope you'll bring back those repo to life!

Nice!
So I'll start Android now for real.

Glad to announce brand new: react-native-ibeacons.
it now (= v0.3.1) contains:

exact content as react-native-beacons-android for Android
exact content as react-native-ibeacon from iOS

It starts supports for

"react": ">=15.4.2",
"react-native": ">=0.41.2"

@Clowning I'm sorry I can't ensure support for previous version of React Native (I had to deal with both iOS and Android and I needed create it as fast as possible), but if it is ok feel free to give a feedback.
On Android, it fixes huge difficulties I encounter to range beacons with React Native 0.41.2.

@MacKentoch I use the last version of React-Native :-) ! I can try your package during this week! I hope your documentation is good ! :p !

@MacKentoch , I'm obviously going to link your repo in the README but first... may I give you a suggestion? The package name you're using, react-native-ibeacons, is a bit misleading in my opinion because you're targeting "beacons" and not "ibeacons" (which are the one implementing the Apple spec)... am I right?

@mmazzarolo yes I agree with you that is not the best name...
It works with nearly any beacons.
Names I wanted are already taken in npm.
The best for me would have been:

react-native-beacons
react-native-beacon

I had to fast fix and merge iOS and Android in a first time.
I wanted to avoid long lasting names. But something like

react-native-beacons-manager
react-native-beacons-api
may be finally better?

I'm opened to suggestions.

Both react-native-beacons-manager and react-native-beacons-api sound way nicer imo 🍹

here is the new one: react-native-beacons-manager.

@MacKentoch I linked your lib in the readme 👍

Dude, I am trying to use this package(react-native-beacons-manager).
But I can not make it run. There is no function like this.
"requestWhenInUseAuthorization"
I got error because of this.
Btw I am using react native 0.59.10
Also I used package for android. it has work but not find beacons.


Starting another bunch

I've going through the list and deleting companies who aren't looking for employees, only contributors (like Android Authority for blog articles) or similar.
Just ship this one when you're ready so I can get an employee of one of the companies to contribute.

That totally makes sense. Awesome, thank you as always @dougaitken :pray:


Remove INotebookModel for native notebooks

Too many work arounds
Lot of wrapping/unwrapping from ICell to nb
Right now this blocks multi language support (ICell always assumes things are Python)

Can add more work arounds yet again, but this needs to go as we don't need ICell for native notebooks. In the future we won't even need ICell (except for the ViewModel used in react layer)

This has already been resolved, hence closing this issue.

This has already been resolved, hence closing this issue.


[Feature] Build Requests Table Page

Notion ticket link

Request Table page

Implementation description

Front end for Request Table Page

Notes

populating table with real data and applying filters will be done as part of the related API hookup ticket

Checklist

[x] My PR name is descriptive, is in imperative tense and starts with one of the following: [Feature],[Improvement] or [Fix],
[x] I have run the appropriate linter(s)
[x] I have requested a review from the RCD team on GitHub, or specific people who are associated with this ticket

Just addressed your comments @EmilioMena !! Angela is going to reply to your question about useControllableState soon as well :)


Allow bulk overwriting application commands

Summary
Implemented method for bulk overwriting application commands and added a couple more builders in the context of interactions to facilitate that. The ApplicationCommandBuilder is a bit awkward because I didn't want to make any breaking changes to the existing interface of Bot#register_application_command and Bot#edit_application_command.

Added

Interactions::ApplicationCommandBuilder - needed something to encapsulate all data contained in an application command to facilitate the creation of the ApplicationCommandSetBuilder (see below).
Interactions::ApplicationCommandSetBuilder - builder that exposes simple DSL to define multiple application commands within a single block
Bot#bulk_overwrite_application_commands - public method that allows... well... bulk overwriting application commands. Works as shown in the YARD example.

bot.bulk_overwrite_application_commands(server_id: ENV["TEST_SERVER_ID"]) do |app|
  app.command(:ping, "Pong!")
  app.command(:echo, "Echo some text") do |cmd|
    cmd.string(:text, "Text to echo")
  end
end

Changed

API::Application.create_global_command
API::Application.edit_global_command
API::Application.create_guild_command
API::Application.edit_guild_command

All 4 of the above have been updated to take a hash of command parameters instead of separate arguments.
{
  name: :test,
  description: "some command",
  options: [],
  default_permission: true,
  type: 1
}

This could pose problems with .create_global_command and .create_guild_command as the name and description for those are no longer techincally required. Suggestions on handling this are welcome.

I think this looks good for the most part but I'm not a fan of removing the parameters in favor of a single hash for the API level methods. That's a blocker for getting this merged. If we can get that reverted and some eyes from another contributor on this we should be good to go

I know it's been a while since you've touched this (my apologies), I'm more than happy to make some changes myself to push this through if you've moved on from the feature 🙂

@swarley yeah it has been a minute 😅 I no longer need this myself but I don't mind looking into it again and making the changes. See what I can do tomorrow 😃

@swarley I've updated the PR to not touch existing code outside of a small fix and documentation.

Been cleaning up my PRs. This is too old and probably no longer relevant. Closing.


Alt +  Z

Whenever i press Alt + Z it just makes a sound. Nothing happens after that. What do I do?

you need GeForce experience or the Nvidia beta app


Incorrect unmatch API

It seems like the tinder_client.remove call expects a match ID not a user ID. This problem might exist in other areas of the API. I got stuck on this for a while. It would be good to rename it.
Another issue is that if the tinder_client.remove call returns {status: 400} because of passing in a user ID, the gem doesn't raise an error.
Another issue, but this is personal preference, is the naming of the method. Why not just call it unmatch?

Ah I see. Yeah, I've confirmed that remove works for unmatching by passing a match ID. I'll see if I can send a PR

So renaming it unmatch and passing a match_id seems ok to me :)


Allow empty when clause for case statements with an else branch

Lint/EmptyWhen reports an offence for the following code, but the empty when branch is there to prevent going into the else branch.  What would the "right" way be to express this?
case status
when STATUS_RUNNING
  puts 'Process already running.  Stopping.'
  exit 1
when STATUS_STALE
  puts 'Deleting stale PID file.'
  File.delete(PID_FILE)
when STATUS_STOPPED
  # This is fine.  Do nothing.
when STATUS_NOT_ROOT
  puts 'Permission denied.  Please run as root/administrator.'
  exit 6
else
  puts "Unknown status: #{status}.  Stopping"
  exit 2
end

Expected behavior
Do not report an offence.
Actual behavior
Reported offence:
W: Lint/EmptyWhen: Avoid when branches without a body.

RuboCop version
0.45.0 (using Parser <IP_ADDRESS>, running on ruby 2.3.1 x86_64-darwin14)

@donv: If you ask me, a node with a comment in it shouldn't be considered empty. (I think this is the convention we go with for other cops.) I'll see if I have time to amend this during this weekend. If someone else wants to pick it up, feel free to do so. 😀

started working on this

Awesome @hanumakanthvvn! Feel free to ping me if you have any questions. 😊

so this check should be at node level right?

@hanumakanthvvn: This works, but only if the comment is on the first line of the body. I think we should make it more robust than that, to cover the case where there's empty newlines:
when :bar

  # no-op
end

WDYT? 😀

@Drenmi Yes, but i thought of it violates the other cop rule of 'Extra blank line detected'. Do you want to skip the empty lines in this case?

@hanumakanthvvn: This is normally a judgement call, asking whether it ventures into the area of responsibility of the other cop. My example won't be caught by EmptyLines, because it checks for "two or more consecutive spaces."
In this case, I think we should check the entire body of the when node, since we otherwise produce a false positive if EmptyLines is not enabled.

@Drenmi Yes you are right the example you given is not catchable by any other cop. I will look into.
So here i need to handle if it is like
when :bar

  # no-op
end

but if it is like
when :bar
  # no-op

end

then it should be default warn like 'Extra blank line detected'
Am i on the right path?

@Drenmi WDYT about this fix. Please look once
def empty_when_body?(when_node)
   node_comment = range_with_surrounding_space(when_node.source_range).source_line.strip
   node_comment =~ /^\s*when/
end

@donv: If you ask me, a node with a comment in it shouldn't be considered empty. (I think this is the convention we go with for other cops.) I'll see if I have time to amend this during this weekend. If someone else wants to pick it up, feel free to do so. 😀

I'm not 100% percent this is the case - probably make cops ignore comments completely. Frankly I'm not a big fan of implicit nil returns, so this is not something I've considered to be an issue myself, but I understand your point of view. I just want people to always keep in mind there's not such thing as "do nothing" - it just a nil return.

I'm not 100% percent this is the case - probably make cops ignore comments completely. Frankly I'm not a big fan of implicit nil returns, so this is not something I've considered to be an issue myself, but I understand your point of view. I just want people to always keep in mind there's not such thing as "do nothing" - it just a nil return.

I actually personally agree with you, and I don't abide the use of unstructured comments. I remember working on some other cop, though, and there were complaints about this. I think (might be wrong on this) other cops consider nodes with comments "non-empty." If we want to change this and have "don't count comments" as a consistent rule, I'm not against it. 😀

@hanumakanthvvn How far did you get with making comments count as contents?

I just want people to always keep in mind there's not such thing as "do nothing" - it just a nil return.

Indeed, it is possible to make this offense go away by replacing the empty body with nil, and indeed RuboCop reports no offense at all in that case.
By the way, given that the value of the case statement is not used, you can also put 42, again without any offense reported. This is a little surprising (although it may be hard to detect unused case statement result values).

I don't buy the explicit nil argument as you may not care about the return value.

Can we re-open this issue? It takes me a few seconds to process that an explicit nil is just a placeholder for a no-op block, as opposed to anything semantically meaningful. I'd be happy to work on this if others agree that having an empty block with a comment is OK.

I too would like this reopened.  A comment explaining why the when-clause is empty is better than nil which does not explain anything.

This looks like it makes sense. I've opened PR #7907.


Releases

Can you please add a tag for the source that was distributed to NuGet and add a binary release to GitHub?

Added Release https://github.com/benaadams/System.Ben/releases/tag/1.1.0


Add training script

This PR adds an example training script for all classes. Classes with fewer than 100 positive examples in the golden dataset are commented in the script.

Right now it's made by hand. Might be a good idea to generate it going forward though.


Create a test contract for testing purposes

The goal of this issue is to have a test contract that allows interaction on the state in the tests that require it. Changes required:

Generate a simple compiled contract available for modules where interaction with the contract is required.

Currently these tests are ignored and marked with the comment:
// FIXME: Generate required test contract

We will take care of this in #337 since was necessary to test out the changes in the API anyway.


Segmentation Fault (SEGV_MAPERR) when destroying the view in Compose with 0.9.0

I need to display a 3D arrow in my Compose UI.
Here is the quite simple code that I am using for that:
private class ArrowSceneView(context: Context) : SceneView(context) {

    // Disable touch & drag
    override val cameraGestureDetector: CameraGestureDetector? = null

    val arrowNode: ModelNode

    init {
        cameraNode.transform = lookAt(
            eye = Position(x = 0.0f, y = 0.0f, z = 0.8f),
            target = Position(0f, 0f, 0f),
            up = Direction(y = 1f)
        )

        arrowNode = ModelNode(rotation = Rotation(x = -90f))
        addChild(arrowNode)

        isTranslucent = true
    }

    suspend fun load() {
        arrowNode.loadModel(
            context = context,
            lifecycle = lifecycle,
            glbFileLocation = "arrow_light_gray.glb",
            scaleToUnits = 1.0f,
            centerOrigin = Position(x = 0.0f, y = 0.0f, z = 0.0f)
        )
    }

    fun setArrowRotation(azimuth: Float, elevation: Float) {
        arrowNode.smoothTransform = Quaternion.fromEuler(Rotation(x = -90f + elevation, -azimuth, 0f)).toMatrix()
    }

}

@Composable
fun Arrow3d(azimuth: Float, elevation: Float, modifier: Modifier = Modifier) {
    val coroutineScope = rememberCoroutineScope()
    AndroidView(
        factory = { context ->
            ArrowSceneView(context).also { view ->
                coroutineScope.launch { view.load() }
                view.setArrowRotation(azimuth, elevation)
            }
        },
        update = { view ->
            view.setArrowRotation(azimuth, elevation)
        },
        modifier = modifier
    )
}

This works perfectly, until I load another view, and this SceneView has to be destroyed.
When that happens, I get the following crash:
2022-09-01 18:09:53.033 29427-29427/com.qorvo.uwb.android A/libc: Fatal signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x6d0 in tid 29427 (rvo.uwb.android), pid 29427 (rvo.uwb.android)

2022-09-01 18:09:53.331 29559-29559/?  A/DEBUG: Build fingerprint: 'google/raven/raven:13/TP1A.220624.021/8877034:user/release-keys'
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG: Revision: 'MP1.0'
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG: ABI: 'arm64'
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG: Timestamp: 2022-09-01 18:09:53.141892672+0200
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG: Process uptime: 12s
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG: Cmdline: com.qorvo.uwb.android
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG: pid: 29427, tid: 29427, name: rvo.uwb.android  >>> com.qorvo.uwb.android <<<
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG: uid: 10482
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG: tagged_addr_ctrl:<PHONE_NUMBER>000001 (PR_TAGGED_ADDR_ENABLE)
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG: signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x00000000000006d0
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG: Cause: null pointer dereference
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG:     x0<PHONE_NUMBER>0006c8  x1<PHONE_NUMBER>000000  x2<PHONE_NUMBER>000000  x3  00000074bd3e177c
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG:     x4<PHONE_NUMBER>88eb8c  x5<PHONE_NUMBER>000000  x6<PHONE_NUMBER>000051  x7  0000007fca1ad918
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG:     x8  02fb6cf1d647f1e1  x9  02fb6cf1d647f1e1  x10<PHONE_NUMBER>000007  x11<PHONE_NUMBER>000005
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG:     x12 0000007fca1ad5b8  x13 0000007fca1ad470  x14<PHONE_NUMBER>000002  x15 00000000ebad6a89
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG:     x16<PHONE_NUMBER>c747d0  x17 0000007fca1ae990  x18<PHONE_NUMBER>b62000  x19<PHONE_NUMBER>000000
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG:     x20<PHONE_NUMBER>000000  x21<PHONE_NUMBER>000000  x22<PHONE_NUMBER>3d2904  x23<PHONE_NUMBER>7e9c2c
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG:     x24 00000074bd200880  x25 0000007fca1aeaa0  x26 0000007fca1aeaac  x27 0000007fca1aeaa0
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG:     x28 0000007fca1ae9a0  x29 0000007fca1ae9a0
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG:     lr<PHONE_NUMBER>c6374c  sp  0000007fca1ae940  pc<PHONE_NUMBER>c63e2c  pst<PHONE_NUMBER>001000
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG: backtrace:
2022-09-01 18:09:53.331 29559-29559/? A/DEBUG:       #00 pc<PHONE_NUMBER>1b7e2c  /data/app/~~ho_nX2TnVkXRRpxQ2JNYhQ==/com.qorvo.uwb.android-gEJi4JJFKgNd2RCDrzQ6sA==/base.apk!libgltfio-jni.so (BuildId: c6c850b752925f24cdd0cecc117fd277bdf218c6)
2022-09-01 18:09:53.344 638-638/? E/tombstoned: Tombstone written to: tombstone_13

The crash happens with SceneView 0.9.0.
This code does not crash with SceneView 0.8.0.

@SalomonBrys this crash happens due to implementation of
// Model.kt
fun Model.destroy() {
    releaseSourceData() // <--- crash ---|
    Filament.assetLoader.destroyAsset(this)
}

Here we have a call to FilamentAsset.releaseSourceData(), which is used more than once, but should not.
First call is here:
// GLBLoader.kt
fun createModel(lifecycle: Lifecycle? = null, buffer: Buffer): Model? =
        assetLoader.createAssetFromBinary(buffer)?.also { asset ->
            resourceLoader.loadResources(asset)
            //TODO: Used by Filament ModelViewer, see if it's useful
            asset.renderableEntities.forEach {
                it.setScreenSpaceContactShadows(true)
            }
            asset.releaseSourceData() // <--- first call ---|

            lifecycle?.observe(onDestroy = {
                // Prevent double destroy in case of manually destroyed
                runCatching { asset.destroy() } // <--- second call in this implementation ---|
            })
        }

Removing it from Model.destroy() should fix the problem.

This should probably stay open until the next release is cut with a fix, whether #150 gets merged or the code moves away from lifecycle events (as indicated was the plan in said PR).
At the very least, I'd like it to remain open since it's how I'm keeping track as to if this is still an issue or not :)

What are the plans on getting this issue resolved? It's holding up a production version of my app using this library.


Does dnaapler change the sequences of the original input data at the reorientation region?

Hi, Could it happen that dnaapler slightly change the sequence of the original input data at the region of reorientation? I observed a few errors in these regions in some sequences.
Thanks in advance,

Hi @MostafaYA
Do you have an example or two? Dnaapler shouldn't be doing this.
George

Hi @gbouras13, thank you for your reply. Unfortunately, I have not kept the old files. I will check again. But you have clarified my question that dnaapler should not be the source of these changes. Thank you!


Ability to validate Module args, result and other types defined in wrapper schema

Is your feature request related to a problem? Please describe.
It'd be nice to have the functionality of validating input args before passing them to the client for invocation. This can be a feature to pre-validate the args before the actual wasm call happens.
Describe the solution you'd like
We can create a JSON schema from the defined GraphQL schema of a wrapper. We can use this schema to do input/output or type validation for any type/method from the GraphQL schema
Describe alternatives you've considered
We may be able to leverage GraphQL schema itself for validation.

Hey @Niraj-Kamdar, myself and @namesty have talked about this recently when working on WRAP ABI 0.2, and have documented our thoughts on this here: https://github.com/polywrap/wrap/tree/abi-0.2/abi#future-plans

@dOrgJelli @namesty  How would merkle root help here?

We were thinking it could be used to optimize the comparison of structured objects. Instead of iterating through each property of an object, and checking for type equivalency (recursively because prop might be struct), you could pre-compute a hash of the collection of properties.
Effectively this:
type Foo {
  prop1: String!
  prop2: Bar!
}

type Bar {
  prop1: String!
}

Could be computed as something like this:
hash(Foo) = multi(
  combine(hash("prop1"), hash(`String`), hash(`!`)),
  combine(hash("prop2"), multi(
    combine(hash("prop1"), hash(`String`), hash(`!`))
  )
)

The root hash(Foo) value that's computed could be used to compare two differnet ABIs that are being "linked" at runtime to see if they're equal. For example "Does the ABI the caller tells me exists in WrapperBar really exist within its ABI?"
Also could come in handy when doing version verification, seeing if a minor upgrade is actually a super-set of the previous version's ABI.

I am not talking about comparing 2 different ABIs, I am talking about comparing whether the objects or arguments are of valid types.
Ex:
type Foo {
  prop1: String!
  prop2: Bar!
}

type Bar {
  prop1: String!
}

would require the following object to be valid
{
  "prop1": "Hello",
  "prop2": {
     "prop1": "World"
  }
}

But this would be invalid
{
  "prop1": "Hey",
  "prop2": "Polywrap"
}


Where is ffmpegsumo.dll

I need to play h264 video in a electron.
but I can't find the ffmpegsumo.dll in Chome 45.0.2454's app directory.
I remember there was the ffmpegsumo.dll at Chrome 42.
I'm using Electron v0.34.1 (it use crhome 45.0.2454.85)

anybody know where is it?

oop's I'm sorry. electron supports h264 video by default.
my mistake was wrong url.


[C-1939] More than two emails after member enrichment make the emails column Inconsistent

Following member enrichment (see screenshot), the 'emails' column overflowed with multiple entries, causing UI inconsistencies. Instead, we could display '+2' or similar to represent additional emails, akin to our handling of excess tags.

From SyncLinear.com | C-1939

Hi @yeganathan18 , let me know on how you are working to tackling this issue!
The proposed solution is quite right to display +X which would maintain consistencies within the UI.
-which in my little knowledge could be targeted via printing on the first 3 or 2 emails, but I don't know how to tackle on the same!
And let me know if I could be of any help to you!
Thanks

Hey, appreciate your interest in the fix!
In the case of tags, when we add more than 3-4 for a member it shows up like tag1 tag2 tag3followed by a +2 and I'd love to see similar stuff for the emails column as well if it's more than two/three, where hovering over the number would reveal the leftover emails.
I'm also looping in our product designer @nunoeufrasio for a quick suggestion 😄

Yep, thanks for the warm regards;)
Let me know when we could start working on this enhancement feature and if I could be of any help to you!

@Bhardwaj-Himanshu @yeganathan18 hope this visual reference can help 🙂 when more than 3, apply the +N pattern. Let me know if you need anything else.

Hey @Bhardwaj-Himanshu!, I'm really stoked you're eager to learn! Unfortunately, I've got my plate full with other high-priority tasks at the moment and can't dive into this issue myself right now.
But, our team would be super happy to help you if you're interested in working on the issue 😄
By the way, don't forget to jump on our Discord server if you haven't already. (Insider tip: It's a great place to meet and chat with other contributors!)

I would love to hop onto this issue then @yeganathan18 . Overflowing your plate with my DM's too then;)

You can find the code for members table in: /frontend/src/modules/member/components/list/member-list-table.vue

Picking this up :)

@peoray Thanks for picking up the issue! I've assigned the issue to you 😄


Cleanup

@steveklabnik @jonathandturner I thought the first para of the community page was too direct with the superlatives (last commit).


fleetctl: {load|unload|start|stop} and get unit code cleanups

This patches set aims to make the code more consistent and robust. It does not change the current behaviour just improves the code, consolidate it, more debug logs and code comments.
The main goal is to let stop, start, load and unload to share the same code base and probably avoid any corner case or bug that may related to one of these commands.
Some logic can be easily shared later when we add stop to destroy command to solve #1000
There were some TODOs in the code base that I cleaned up with the last two patches. These patches will make it easy for us to implement later #1295
This new PR is to trigger semaphoreci.

It seems we can trigger semaphore in the original PR https://github.com/coreos/fleet/pull/1433 so closing this one.


fix(traffic_light_classifier): fix traffic light monitor warning (#8412)

Description
cherry-pick of https://github.com/autowarefoundation/autoware.universe/pull/8412
Related links
https://github.com/autowarefoundation/autoware.universe/pull/8412

How was this PR tested?
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

@zusizusi
Could you update the branch? It seems this branch is out-of-date with the base branch


Invalid or corrupt jarfile selenium-server-standalone-4.0.0-alpha-1.zip.jar

When i start the webdriver-manager i take this error, i did not change anything in my project that justified such an error, and I have been using it for more than a year, every day.
Error: Invalid or corrupt jarfile ./node_modules/protractor/node_modules/webdriver-manager/selenium/selenium-server-standalone-4.0.0-alpha-1.zip.jar

Node Version: 8.12
Protractor Version: 5.4.2
Browser(s): Chrome
Operating System and Version Ubuntu 16.04
Your protractor configuration file:

'use strict';

require('babel-core/register');

const myReporter = require('./Report/MyReporter');
const Jasmine2HtmlReporter = require('protractor-jasmine2-html-reporter');
const failFast = require('jasmine-fail-fast');

module.exports.config = {
    seleniumAddress: 'http://localhost:4444/wd/hub',
    framework: 'jasmine2',
    allScriptsTimeout: 600000,
    capabilities: {
        browserName: 'chrome',
        chromeOptions: {
            args: ['--test-type', 'no-sandbox'],
        },
    },
    onPrepare: () => {
        browser.manage().window().setSize(1200, 1024);
        jasmine.getEnv().addReporter(failFast.init());
        jasmine.getEnv().addReporter(myReporter);
        jasmine.getEnv().addReporter(new Jasmine2HtmlReporter({
            takeScreenshots: true,
            takeScreenshotsOnlyOnFailures: true,
            fixedScreenshotName: false,
            fileNameDateSuffix: true,
            cleanDestination: false,
            savePath: './Report/testReports/'
        }));
    }
};

A relevant example test

I'm having this same issue! :(

@sborland @sborland I solved this in my own way by changing the standalone-response.xml and update-config.json files. But this link can help you in a better way:
https://stackoverflow.com/a/55836347

@davidzaque thanks! This work around worked for me.

@davidzaque  I have the same issue. But I don't want to install selenium globally. My angularjs application has protractor in its node module and protractor has it's own dependencies where web driver gets updated   to latest. How to install it specifically to protractor and not globally?

Yup this is a bug. This appears to affect every version of the selenium server.
Here is the bug listed in webdriver-manager https://github.com/angular/webdriver-manager/issues/370
And here is the fix / discussion for the fix: https://github.com/angular/webdriver-manager/pull/371

I will get a new release out tomorrow.

as a temporary solution to this issue you can explicitly set the standalone version when running webdriver.
--versions.standalone=3.9.1

While running test-cases over protractor , this solution couldn't helping me out. How to use this workaround with protractor ?

i have only tested the solution with our current setup.
But when looking at the official docs
http://www.protractortest.org/#/
I would run

webdriver-manager update --versions.standalone=3.9.1
webdriver-manager start
protractor conf.js

I hope this helps you in any way.

@tomreinartz90  i had tried. Webdriver starts successfully over port 4444 but getting the error when running protractor conf.js
[15:28:48] I/hosted - Using the selenium server at http://localhost:4444/wd/hub
[15:28:50] E/launcher - Cannot define class using reflection
[15:28:50] E/launcher - WebDriverError: Cannot define class using reflection ```

@ZehraHN I found that when I used version 3.141.0, i'm no longer running into that issue.  Anything in the 3.8 or 3.9 range was causing me to get the reflection error.

Yes works now. Thankss @clymerrm

@davidzaque I have the same issue. But I don't want to install selenium globally. My angularjs application has protractor in its node module and protractor has it's own dependencies where web driver gets updated to latest. How to install it specifically to protractor and not globally?

@tejasbhosale i don't use protractor globally. I changed the file: node_modules/protractor/node_modules/webdriver-manager/selenium/standalone-response.xml
excluding references related to version 4.0.0-alpha and in the file:
/node_modules/protractor/node_modules/webdriver-manager/selenium/update-config.json
i made this change:
"standalone": {
        "last": "/vox/protractor-sigfacil/node_modules/protractor/node_modules/webdriver-manager/selenium/selenium-server-standalone-3.141.59.jar",
        "all": [
            "/vox/protractor-sigfacil/node_modules/protractor/node_modules/webdriver-manager/selenium/selenium-server-standalone-3.141.59.jar"
        ]
    },

After that, I kept these files out of the path node_modules, so they would not be updated, and added the following scripts to package.json:
"scripts": {
        "preinstall": "node_modules/protractor/node_modules/webdriver-manager/bin/webdriver-manager clean",
        "postinstall": "cp standalone-response.xml ./node_modules/protractor/node_modules/webdriver-manager/selenium/ & cp update-config.json ./node_modules/protractor/node_modules/webdriver-manager/selenium/ & node_modules/protractor/node_modules/webdriver-manager/bin/webdriver-manager update --versions.standalone=3.141.59",
        "start": "webdriver-manager start"
    },

I know, it was too much maneuvering to solve the problem, but it was how I managed to solve it before any other solution they had posted.

Faced the same issue as npm install is downloading a corrupt jar "selenium-server-standalone-4.0.0-alpha-1.zip.jar" .
Tried installing npm again.
It doesn't have selenium directory itself under webdriver-manager !
Files under webdriver-manager:
total 64
-rw-r--r-- 1 root root  1205 Oct 26  1985 gulpfile.js
-rw-r--r-- 1 root root   547 Oct 26  1985 config.json
-rw-r--r-- 1 root root  2455 Oct 26  1985 README.md
-rw-r--r-- 1 root root  1078 Oct 26  1985 LICENSE
-rw-r--r-- 1 root root  2699 Oct 26  1985 CONTRIBUTING.md
-rw-r--r-- 1 root root 21636 Oct 26  1985 CHANGELOG.md
drwxr-xr-x 2 root root  4096 Apr 25 15:08 bin
drwxr-xr-x 3 root root  4096 Apr 25 15:08 built
drwxr-xr-x 2 root root  4096 Apr 25 15:08 docs
drwxr-xr-x 5 root root  4096 Apr 25 15:08 node_modules
-rw-r--r-- 1 root root  3650 Apr 25 15:08 package.json

Done with https://github.com/cnishina/webdriver-manager/commit/7dc17ef36e93f71bc63475612e343ffb84efec0f
Also we are not downloading alpha versions since we were previously not downloading beta versions. Please update to webdriver-manager 12.1.1.

Done with cnishina/webdriver-manager@7dc17ef
Also we are not downloading alpha versions since we were previously not downloading beta versions. Please update to webdriver-manager 12.1.2. If you are using Protractor to download webdriver-manager, simply remove the node_modules directory and reinstall or do a npm install -f. Leaving this open for visibility.

Thanks for your prompt response on this issue. But my build at Jenkins still failing with same error. Is there anything I can do to?

@tejasbhosale Are you caching your node modules directory or workspace? You could clean up your Jenkins workspace and try again. I did a clean install and verified that you should get<EMAIL_ADDRESS>with Protractor 5.4.2 because of https://github.com/angular/protractor/blob/5.4.2/package.json#L29. Could you have a lock file that is set to 12.1.1?

@cnishina  Thanks for responding. I suspect similar but clearing workspace didn't work for me. I am going to research more in jenkins config. But good news is it worked on my local env so it must be jenkins. And I checked my project's pakcage.json it is using this version
"protractor": "^5.2.0"
Is that fine? is it still gonna get webdriver 12.1.2?

The below mentioned solution worked for me:

Uninstall/install Java
Setup the Environment variable if newer version of java installed

Navigate to :
Step 1:
C:\Users<Your User>\AppData\Roaming\npm\node_modules
Delete the below mentioned folders

protractor
webdriver-manager

Step 2:
Reinstall Protractor Globally
npm install -g protractor
Step 3:
Re run Webdriver-manager update
Step 4:
Webdriver-manager start will not throw any errors

I had same issue.

Ele é resolvido com a atualização do Protractor de 5.4.1 para 5.4.2.👍
# 5224 (comentário)

Thanks!!!!

For StackOverflow, I tried to find a post to answer. I ended up creating a question and answering it:
https://stackoverflow.com/questions/55869815/error-invalid-or-corrupt-jarfile-node-modules-protractor-node-modules-webdriv
I have a quick explanation about what webdriver-manager update is doing and why it happened. Could we upvote for visibility?

Maybe I don't understand how to define versions? I updated my package.json to include "protractor": "^5.4.2", but after removing node_modules and reinstalling I see node_modules/protractor/package.json has "webdriver-manager": "^12.0.6" (and the problem of downloading a zip.jar).

Maybe I don't understand how to define versions? I updated my package.json to include "protractor": "^5.4.2", but after removing node_modules and reinstalling I see node_modules/protractor/package.json has "webdriver-manager": "^12.0.6" (and the problem of downloading a zip.jar).

Do you have protractor installed globally as well?

No, I use ./node_modules/protractor/bin/protractor, and I see that which protractor turns up nothing.

when you run webdriver-manager version you get version 12.0.6 as well?
I mean the package.json wasn't changed for protractor 5.4.2, just the webdriver-manager is an updated version

I do not have it installed globally. I remember when I started this project it took me some time to discover it's the webdriver inside protractor that needs to be updated, with ./node_modules/protractor/bin/webdriver-manager update. When I check that version:
$ ./node_modules/protractor/bin/webdriver-manager version
[16:29:49] I/version - webdriver-manager 12.1.1

So you should be on the latest. The latest version is 12.1.4. Previous versions of 12 won't work because the file it is downloading changed. Please review the above for instructions to upgrade.

I did upgrade. Let me try explaining again, because it's very confusing: there are two different definitions of webdriver-manager in play.
The package.json in my project has "protractor": "^5.4.2". I have never explicitly had "webdriver-manager" included. When I have run the command to update, which is what pulls in the selenium package, I have run ./node_modules/protractor/bin/webdriver-manager update -- interacting with the webdriver build that is inside protractor's bin directory. This is kludgy, and requires doing the update after I've installed node_modules, but it worked, up until the breaking change.
Putting "webdriver-manager": "^12.1.4" in my package.json has no effect on what version protractor uses.  I can follow these steps:

npm install in my project directory
edit the file in node_modules/protractor/package.json to update webdriver-manager to ^12.1.4.
npm install in node_modules/protractor/
./node_modules/protractor/bin/webdriver-manager update

to prove that an updated protractor/bin/webdriver-manager will get the right version. But, I'm not looking forward to working "Now edit the post-install protractor package.json" into our CI pipeline. Is there a different way to force protractor to use the latest webdriver-manager?

Can you update your package.json and add webdriver-manager to your devDependencies?  That fixed the issue for me.

In the experiment above, both protractor and webdriver-manager (when added) were in devDependencies.

Have you verified that your version is 12.1.4 when you run ./node_modules/protractor/bin/webdriver-manager update?  There are two ways to do this:

Go to node_modules/protractor/node_modules/webdriver-manager/package.json and check the version number
Run node_modules/.bin/webdriver-manager version

Protractor (by default) should download the ^12.0.4 version which means it should get 12.1.4. If you are still getting a .zip.jar file that means you have an old version and you need to upgrade this.
I do not recommend adding it to your devDependencies. This is why:
Let's say if Protractor upgrades to version webdriver-manager@13 and you still have devDependencies set to webdriver-manager@^12. When you call ./node_modules/protractor/bin/webdriver-manager update, you are not guaranteed that you will run version 12 or 13. When Protractor calls for the files it will hopefully use version 13. If your files were downloaded with 12, then these files will not exist when Protractor requires them.

my-project/package.json: "protractor": "^5.4.2" in devDependencies , webdriver-manager nowhere
 $ rm -rf node_modules
npm install
cat node_modules/protractor/package.json|grep webdr

    "webdriver-manager": "bin/webdriver-manager"
    "@types/selenium-webdriver": "^3.0.0",
    "selenium-webdriver": "3.6.0",
    "webdriver-js-extender": "2.1.0",
    "webdriver-manager": "^12.0.6"

Protractor (by default) should download the ^12.0.4 version which means it should get 12.1.4.

Makes sense. And yet:
$  ./node_modules/protractor/bin/webdriver-manager version
`[11:19:58] I/version - webdriver-manager 12.1.1

Bewildering.

@mathpunk, is there a package-lock.json file at play here that may have it locked to 12.1.1?

We have an interesting scenario in our organization as we have our own automation-related npm package (on an internal registry) that abstracts automation from our SPAs and applications. That package uses protractor 5.4.2 and does not use a package-lock.json file as we have seen some issues with how npm has deduped dependencies of dependencies, which messed with some of our paths. Anyhow, installing a version of our automation package that has protractor as its dependencies in SPAs and applications that use package-lock.json files still results in webdriver-manager 12.1.1 being used. npm i would install that as a first-class citizen, which we don't want because 12.1.4 and 12.1.1 would still be present. npm update --depth 9999 webdriver-manager does not work as I would expect, which would be updating from 12.1.1 to 12.1.4. Short of manually changing package-lock.json in every consuming application of our automation package or regenerating the file fresh, which we don't want because it updates a lot of packages we want to keep where they are to prevent regression test work, I'm not sure what to do.
@cnishina, thanks for all the work you do on Protractor. Would you support a PR of a branch that is branched off 5.4.2, which would only update webdriver-manager to 12.1.4?

@simonua You cracked it! Removed package-lock.json, removed node_modules, installed, got webdriver-manager 12.1.4. I don't understand why versions are defined in two different places in this ecosystem, but hey, problem solved~

Glad that did it. Out of curiosity, if you were to revert your changes and run npm i<EMAIL_ADDRESS>with that explicit version you want instead, I suspect you would be in good shape.

Same issue here

Just download right file from selenium site https://selenium-release.storage.googleapis.com/index.html?path=4.0/
I took jar file and changed name from *.jar to *.zip.jar
worked ok


Using deepmedic with other images

Hello,
I know deepmedic was designed to work with brain images. Would it be appropriate to work with other tyoes of medical images (i.e. lung)?
Thanks.

Hello and thank you for your interest.
Yes, the method is made to be rather generic. Users have applied for a wide variety of problems, including organs outside the brain (heart/liver/lung/abdominal/bones etc etc). It should be very good for lungs.
Here is a publication where we have applied it for multi-organ segmentation 2 years ago out of the box:
Lavdas et al, Fully automatic, multiorgan segmentation in normal whole body magnetic resonance imaging (MRI), using classification forests (CFs), convolutional neural networks (CNNs), and a multi‐atlas (MA) approach, Journal of Medical Physics
https://aapm.onlinelibrary.wiley.com/doi/full/10.1002/mp.12492
K

Thanks a lot for your kind support, appreciate it!

Only interesting advice I can give is, I if you apply it for images that contain whole body, what I tend to do is, first, make a mask that does not contain the air (around the body). And provide it to deepmedic as roi-mask (in trainConfig). So that training is not wasted on samples from the air constantly.
This can be done easily eg by thresholding the air and filling the wholes. (cant remember if the mask changes performance much, but I always make one when I can).
K


Where should the 'viewport' object go?

Our proposal currently has the viewport object on document.documentElement. This may not be the ideal place for it.

@esprehn wrote on the blink-dev discussion:

"Any element can be the documentElement, that means you're extending the global interface to have a concept of viewport which doesn't seem right. Having this property on thousands of elements but only ever non-null on a single one per page (for the lifetime of the page) is very strange. Especially since if I remove the documentElement there still is a viewport."

Yah, the way that everything works today that is strange. As Rick mentioned, it's not clear that we'd ever want to pursue something like -ms-content-zooming but we don't want to close that avenue either.
Suppose we'd put this on window.viewport and later we wanted to give elements the ability to zoom independently (and thus have independent viewports), could we do something similar to how  documentElement.scrollTop delegates to window.scrollY or is that a wart that we'd rather not replicate? E.g. if we added Element.viewport we'd define it for the documentElement as delegating to window.viewport.

@rbyers

That's one option.  Another is to say that top-level viewport is NEVER controlled via an element, so even if we added Element.viewport in the future, documentElement.viewport would always be null.
Note that we probably want to avoid adding properties to window since that often conflicts with variables in use by apps.  I'd suggest instead navigator or document (perhaps depending on the precise behavior within iframes).

In that case, I think document makes the most sense sense since content embedded in an iframe shouldn't see the same visual viewport as the parent.
For example: if you use this to build position: device-fixed, like in the example I gave, having the viewport on document means an iframe has its own viewport which doesn't change when the page is scaled so you get the expected behavior.

@ymalik, @esprehn, does this sound ok?

Having viewport on document sgtm.

document.viewport seems fine.

I've changed to window.visualViewport. The feedback we've gotten from some web devs is that this seems more natural. The original objection was that this might conflict with existing variables named visualViewport since it's in the global scope but I did a search through httparchive and found no uses of visualViewport as a variable at all so I think this should be fine.

Even if some site does do var visualViewport = ... or window.visualViewport = ..., that'll continue to work fine (hiding our API) unless the same page is also trying to use our API elsewhere, right?  I agree (given the HTTPArchive data) that the risk should be very low.

Right, existing usages should be fine unless they're doing something like checking if visualViewport is defined but the risk should be negligible.


Training on CIFAR-10 takes a lot of GPU memory space

Hi @HobbitLong, thanks for the wonderful work. My issue might be a bit unusual.
I basically tried to use the Resnet-50 architecture provided in the Pytorch's torchvision package. The Resnet-50 network that you have used seems to differ from the one provided by "torchvision". Also, when I am trying to use your code "main_supcon.py" with "cifar-10" dataset, a batch size of 128 is occupying the entire GPU memory of 11 GB whereas, in my implementation where I have used "torchvision's" Resnet-50 architecture, a batch size of even 1024 is occupying only 5.5 GB.
And according to me, cifar-10 with batch size of just 128 should not occupy so 11 GB of memory since after all the images are of just (3, 32, 32) dimensions.
PS:

I have considered two views of each image, so effectively the batch size will be double of what is specified.
I am using NVIDIA GeForce 1080Ti.

Please let me know why your code is so much memory intensive.
Thanks!!!

Hi, @jainpulkit54,
Probably because of this line. It has drastically lowered the dimension (i.e., H and W) of your feature maps. For CIFAR-10 or 100, such large kernel is not suitable as the input image is just 32x32.

Hi, @jainpulkit54,
Probably because of this line. It has drastically lowered the dimension (i.e., H and W) of your feature maps. For CIFAR-10 or 100, such large kernel is not suitable as the input image is just 32x32.


Generate build status-dependent icons for the editor

Closes https://github.com/godotengine/godot-proposals/issues/541
Also auto-generates the icons for console executables under Windows. They are not pixel-exact to the previous ones, since they're composited dynamically, but they should be close enough.
Sample preview using some kind of web-based .ico previewer - might be a bit distorted.

Caveats:

Icon components are rendered to PNG first, then composited, because doing everything in SVG and then converting to PNG yielded bad results - some lines were thinner than others, also getting color-mixed with the neighbors.
I'm mainly a programmer, so SVGs present in the PR are not perfect - though I assume they are good enough placeholders. I've concocted them in GodSVG in conjunction with some kind of web-based SVG text path generator.
No linux icons because I couldn't get any response on how to tackle it in the proposal issue. :(

Also it made a bit of make_icons.sh obsolete, so perhaps it could supersede it further? I don't know how it's called though - make_icons.py is intended to generate files which are to be commited, whereas it looks like make_icons.sh is used for packaging?

The script for generating PNGs is pretty much all that can be done, Linux icon is not part of the executable or official distribution, so it's up to distro specific package maintainers how to use it.

That should be easily doable, but I need to know what kind of PNG sizes are needed.

That should be easily doable, but I need to know what kind of PNG sizes are needed.

No sure if there's a standard set of sizes. The following seems to be used 16x16, 22x22, 24x24, 32x32, 48x48, 64x64, 96x96, 128x128, 192x192, 256x256, but not every icon is using all of them. Some apps also use single SVG instead.

I've added a bunch of linuxbsd icons. Also generates the SVG variant. The problem is, generated icons don't pass through svgo, but since the pre-commit hook generates the optimized variant, and the script doesn't need to run that often, perhaps that's acceptable? If not then there's a problem - I see no better SVG libary than cairo and I'm not quite sure if text-based svg editing won't trigger the svgo anyway.

What's the status of this PR? I know it's probably of very low priority, but perhaps there are some concerns that I could investigate in the meantime.

@tehKaiN Can you provide Android icon_monochrome for different sizes too?
This is the svg file, you can resize it: https://godotengine.org/assets/press/icon_monochrome_dark.svg

I didn't know that they are required/nice to have since the original icon set didn't had those. Fixed!
Regarding using SVG, if the quality of current icons isn't good enough then I'll investigate going the SVG route, but that needs extra work to make the dev/beta/rc label svgs monochrome too.

Regarding using SVG, if the quality of current icons isn't good enough then I'll investigate going the SVG route, but that needs extra work to make the dev/beta/rc label svgs monochrome too.

I had suggested using the SVG as a base image, resizing it to your desired dimensions, and then exporting it as a PNG file.

A friendly reminder that another month passed by. I know it's super low priority issue, but I wonder - is there some kind of review queue where I could check how things progress? Or should I just wait for the reviews, or perhaps keep pinging here and there from time to time?


Update markdown to follow best practices

Description
Although the GitHub markdown parser is pretty forgiving, this follows best practices and should work with most markdown parsers.
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that validate the behaviour of the software
[ ] I validated that new and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Followed up with #57 but I put it in a separate PR because it might be more controversial.


Restore merge2

Fixes #
Changes proposed in this pull request:

File positions
Ouput

How to test this code:

[ ]
[ ]

Has been tested on minimum requirements:

[ ]  Powershell 3
[ ]  Windows 7
[ ]  SQL Server 2000

Has been tested on maximum requirements:

[ ]  SQL Server vNext
[ ]  Windows 10
[ ]  Azure Database

Tests for tester:

[ ] Working/useful help content, including link to command on dbatools web site
[ ] All examples work as advertised
[ ] Does not contain template content
[ ] Does not contain excessive/unnecessary amounts of comments
[ ] Works remotely
[ ] Works locally
[ ] Works on lower versions or throws error specifying version not supported
[ ] Works with named instances
[ ] Works with clustered instances
[ ] Handles offline/read only databases
[ ] Supports multiple servers (at the command line or piped from Get-SqlRegisteredServerName)
[ ] No un-handled errors which stop the command working with multiple servers

Thanks, bud! 🍖 🤘 🏅


Code Sign error using match and gym

I am running fast lane on a CI (bitrise.io) with match and gym used to sign and build. The output of match seems to be successful:
+----------------------+--------------------------------------+
|               Installed Provisioning Profile                |
+----------------------+--------------------------------------+
| App Identifier       | com.lovecrafts.lovecrafts            |
| Type                 | adhoc                                |
| UUID                 | eaa7baf9-2eeb-4776-a7f4-f659cffd69e4 |
| Environment Variable | sigh_com.lovecrafts.lovecrafts_adhoc |
+----------------------+--------------------------------------+
[03:05:57]: All required keys, certificates and provisioning profiles are installed 🙌
[03:05:57]: 🔒  Successfully encrypted certificates repo

Gym hits an error almost immediately though:
+----------------------+--------------------------+
|              Summary for gym 1.4.0              |
+----------------------+--------------------------+
| workspace            | ./Lovecrafts.xcworkspace |
| scheme               | Lovecrafts               |
| clean                | false                    |
| output_directory     | ./                       |
| output_name          | Lovecrafts               |
| silent               | false                    |
| include_symbols      | true                     |
| include_bitcode      | false                    |
| use_legacy_build_api | false                    |
| export_method        | ad-hoc                   |
| buildlog_path        | ~/Library/Logs/gym       |
| destination          | generic/platform=iOS     |
+----------------------+--------------------------+
[03:06:01]: $ set -o pipefail && xcodebuild -workspace './Lovecrafts.xcworkspace' -scheme 'Lovecrafts' -destination 'generic/platform=iOS' -archivePath '/Users/vagrant/Library/Developer/Xcode/Archives/2016-01-29/Lovecrafts 2016-01-29 03.06.01.xcarchive' archive | tee '/Users/vagrant/Library/Logs/gym/Lovecrafts-Lovecrafts.log' | xcpretty
[03:06:02]: ▸ Building Pods/Analytics [Release]
[03:06:02]: ▸ Check Dependencies
[03:06:02]: ▸ ❌  Code Sign error: No code signing identities found: No valid signing identities (i.e. certificate and private key pair) were found.

Running fastlane beta locally works fine (I know thats not particularly helpful), and I have followed the match Xcode setup guide:

Is there anything I am missing from this setup?

Does it work locally for you? Do you have multiple targets?

locally works fine (both by running fast lane and archiving via Xcode)
only one app target (+ unit and ui test targets)

Closing this since it is working now. I followed the instructions here: http://devcenter.bitrise.io/docs/provprofile-cert-export#section-troubleshooting


[Minor] Textbar text is not aligned properly

As shown in the image, the placeholder text is not aligned with the text in the regular text box, this is not the case on regular discord and is frankly triggering my ocd a lot.

I didn't really notice it until you pointed it out. I actually triggered me as well. Fixed.


Update README.md

Jérôme's domain got cybersquatted. Adding alternative link to the article as suggested by Jérôme himself.

Thanks!


Redis 6 Support

We seem to have a few failing tests with the introduction of Redis 6.0 into the pipeline. Unfortunately the current output is not very helpful:
https://github.com/aio-libs/aioredis/runs/1468097976?check_suite_focus=true#step:11:5000

Just to get this fixed quickly, some things to take note:

test_master__auth: add username as a keyword parameter next to password in conftest.py's start_server fixture. We may just need two different tests to make sure username and password are both checked.
test_auth: the message is just different for Redis 6.0. You may want to test both the message for password and the one for username like aforementioned.
test_failover_command : not entirely sure what's wrong
test_client_list : Redis 6.2 added laddr. I'm not able to see that for myself, strangely enough... Set to MOCK.Any
test_client_list__unixsocket: again, I couldn't see it, so I'm not sure what part of Redis instance could show this...
test_command_info: it's just a Redis 6.0 addition

I'm going to remove Redis 6 from the pipeline until we've patched this so we can have a green build again on master.

I'm going to remove Redis 6 from the pipeline until we've patched this so we can have a green build again on master.

The best you can do now

This issue should be resolved as of #891.


Bug ?!?

 //Discord emoji   if(message.content.startsWith(prefix+'d-emojiname')){     message.delete(1000)     var request = message.content.replace(prefix+'d-emojiname ', '')   DEmojiJS.emojiByName(request).then(Emoji => {     //try{     message.channel.send(${message.author} :)     message.channel.send(${Emoji.image} \n ``` ${Emoji.title} submitted by: ${Emoji.submitted_by}. ``` );     //}catch(err){message.channel.send("Couldn't find any emoji")}   }).catch(console.error)   }
This is the code I use, it's coming from the website, just ma de few tweaks but Nothing really changed.
It is returning TypeError: Cannot read property 'image' of undefined through the console…
Do you have any idea ???

//Discord emoji
if(message.content.startsWith(prefix+'d-emojiname')){
message.delete(1000)
var request = message.content.replace(prefix+'d-emojiname ', '')
DEmojiJS.emojiByName(request).then(Emoji => {
//try{
message.channel.send(${message.author} :)
message.channel.send(${Emoji.image} \n \`` ${Emoji.title} submitted by: ${Emoji.submitted_by}. ``` `);
//}catch(err){message.channel.send("Couldn't find any emoji")}
}).catch(console.error)
}

KappaYugi and Kappa are working, not the others on Discord Emoji website. Still testing things out rn.

Discord emoji is working, but wearycat, and every other emojis aren't working. wearycat is on the website's main page.

Ok, it looks like newest emotes cannot be used… ;-;

Emoji.image can't be used for emojis that are a bit far in popular, and all emoji beyond the limt. That's a big inconvenience.

Trying to fix this by not using the .image thing but it doesn't work

Ah, I was wondering when someone would eventually come across this.
This is unfortunately a problem originating from Discord Emoji themselves, their API data can be incredibly outdated, at times lacking as much as 3 months worth of data (hence DEmojiJS failing to return any sort of data regarding newer emotes). I'm not sure if they have any plans to sort this out or if it's part of an intentional caching plan or something but it's out of my hands and seeing as DEmojiJS fetches data from the official API alone and doesn't scrape anything from the website, I'm afraid you will have to come up with your own work-around.
Alternatively, you could try writing to them about this and hoping that public concern regarding this would motivate them to do something about it.

Yes, all emojis with the /ID_TITLE on their URL format are not useable… the API doesn't give you their IDs because they can't find the name of it, so you can't "automate" it by doing .../${Emoji.id}_${Emoji.title}
because of the API giving you an error. Only /TITLE can be used rn...

Huh? That's not right, my friend. This issue has nothing to do with the slug ID of the emote (id_title), it's to do with the fact that their API literally has no record of the emote you're requesting. I know this because I ran more than my fair share of tests prior to releasing this wrapper.
This is irrelevant either way because this (outdated, horribly written) version of DEmojiJS has no mechanism by which to fetch an emote using its slug (unless you manually modify it to do otherwise), so all you need to submit is the title, this is of course by design to make it more convenient for anyone using it.


Clause Error Persists in Logger when Deleted

Describe the bug
The clause error persists in the ErrorLogger component when deleted from the contract
To Reproduce
Steps to reproduce the behavior:

Run locally with cicero-ui linked through npm link
Add a clause template to the contract
Type a random letter into the clause, causing a parse error
Select the delete icon on the clause template, this will remove it
Notice that the error in the ErrorLogger still contains the parse error

Expected behavior
Deletion of the clause template should also remove its corresponding errors from the redux store
Screenshots

Additional context
Related issue: #111

I will work on it

@Andre-rsfc any progress on this? Opening back up in the meantime.

I would like to work on this as my first issue, however, I can't seem to find this specific issue. Upon deleting the contract, the error in the error logger at the bottom is removed and nothing is in the store. Am I looking in the right area for the error in the store? (Contract state -> clauses -> clauseId -> parseError)
I am, however, able to get a parse error by adding a contract, adding any character to the contract, adding a character below the contract, and upon adding a second character inside the contract the parse error for the original character in the contract appears. This was the only way I could get a parse error to show up on my end.

I would like to work on this as my first issue, however, I can't seem to find this specific issue. Upon deleting the contract, the error in the error logger at the bottom is removed and nothing is in the store. Am I looking in the right area for the error in the store? (Contract state -> clauses -> clauseId -> parseError)

I think this more refers to clauseTemplateState, where the template (which is used to parse the clause against to determine if there is an error) remains on state when the clause is deleted.

I am, however, able to get a parse error by adding a contract, adding any character to the contract, adding a character below the contract, and upon adding a second character inside the contract the parse error for the original character in the contract appears. This was the only way I could get a parse error to show up on my end.

This seems weird. So adding a clause and adding a character within the clause (not between quotation marks / inside variables) does not cause a parse error? You have to do this, then type outside the clause, then type in the clause again?

I went back and did some more testing, and the outcome is I'm able to replicate the error multiple ways now.
I have replicated it your way just a small handful of times, but the majority of the time is either the way I described earlier, or this: https://www.loom.com/share/27692afecb7b42e1acb152d82d751596
Recently, I haven't needed to type a third character for the error to throw, just the two characters as the video above shows.
Regardless I assume whatever is making this alternate is a completely separate issue and isn't related to the original issue of clearing the state? If that's the case, will clearing clauseTemplatesState upon deleting the template be the desired solution?

Yeah I believe this is a different issue. Quite bizarre, I've never seen this happen.
I think clearing the clauseTemplateState is the desired solution for this issue. I may be mistaken, but I believe this is added to the state only as a means to parse the clause. So if the clause is removed, the template is now dead weight and cannot be used if the same clause is added again - it is added with a uuid.

Clearing clauseTemplateState on deletion sounds like something I'd like to take a crack at. I'll update you as soon as I make some progress!

Yeah I believe this is a different issue. Quite bizarre, I've never seen this happen.

I have also not seen the issue where the parse error does not show up until you type outside the clause. Can you open that as a new issue please @ardelapaz with the browser you are using?

I think clearing the clauseTemplateState is the desired solution for this issue. I may be mistaken, but I believe this is added to the state only as a means to parse the clause. So if the clause is removed, the template is now dead weight and cannot be used if the same clause is added again - it is added with a uuid.

Yes, that sounds correct to me!

I've found that my main obstacle when trying to clear the clauseTemplatesState is obtaining the clauseTemplateRef from the clause that's being deleted, and passing that into a reducer to be able to remove that ID from the clauseTemplatesState. One workaround I found was to just set the clauseId as the key of each clauseTemplatesState item when added, since the clauseId easily accessible, but I'd rather not use that approach as it stores items in the clauseTemplatesState under the clauseId instead of it's own Id.
Is there a built in way that I'm not seeing to obtain the clauseTemplateRef (clauseTemplatesState item id or key) upon deletion of the clause?

I don't think so. I would be following how the clause template is added to the store when you add a clause, and try to replicate a similar behavior when deleting the clause to also delete its corresponding template reference.

Yeah I believe this is a different issue. Quite bizarre, I've never seen this happen.
I think clearing the clauseTemplateState is the desired solution for this issue. I may be mistaken, but I believe this is added to the state only as a means to parse the clause. So if the clause is removed, the template is now dead weight and cannot be used if the same clause is added again - it is added with a `

We need to be careful here. You can have multiple instances (clauses) of the same template.

We need to be careful here. You can have multiple instances (clauses) for the same template. Templates should be identified by their id. Clauses have an id assigned on creation, and you can get the id of their template.

@dselman, that is a good point. Additionally, even if all of the clauses using that template have been removed, the user may still wish to edit the template for later use? May need some UX input from @Michael-Grover on this one.

Templates should be identified by their id. Clauses have an id assigned on creation, and you can get the id of their template.

The issue that I was running into was being able to access that template's ID. The template's ID is only stored in the store under contractState -> clauses -> clasueTemplateRef but upon deletion of a clausetemplate the only thing that is passed through is the clauseId. See removeFromContract in contractSaga.js to see what I mean.

You can have multiple instances (clauses) for the same template.

I'm still not very comfortable with the project's flow, so I still don't fully understand what clauses you might have under the same template, but with my testing you can add multiple clauses and deleting a template on the contract resulted in deleting the correct clauseTemplatesState item from the state.
Also, none of the clause templates are loading in for me anymore. A recent commit or so must've changed something and isn't working as the netlify version is. It was working just last night before I sent the pull request, but after coming home from work today all I see is this:

hi @ardelapaz , update your branch with the latest master to get the templates to load. this was fixed in the lastest PR.

Fixed @DianaLease Thank you,

You can have multiple instances (clauses) for the same template.

This I don't fully understand. From what I understand, you can add templates from the right side, and the deletion button removes that template. What is meant by "Multiple clauses for the same template"? Isn't the point to delete the deleted clause template from the clauseTemplatesState upon deletion? I guess you could say I'm having trouble understanding what is a clause and what is a clauseTemplate.

Also just noticing, this delete functionality; isn't this issue #111 rather than #112 ? There isn't an error in clauseTempaltesState, only the templates themselves. So #112 and #111 are essentially asking for the same solution if I'm not mistaken?

Fixed @DianaLease Thank you,

You can have multiple instances (clauses) for the same template.

This I don't fully understand. From what I understand, you can add templates from the right side, and the deletion button removes that template. What is meant by "Multiple clauses for the same template"? Isn't the point to delete the deleted clause template from the clauseTemplatesState upon deletion? I guess you could say I'm having trouble understanding what is a clause and what is a clauseTemplate.

Please ensure that this works when you have multiple instances of the same clause added to a contract. Eg try adding 3 instances of acceptance-of-delivery. We want different errors for each, and deleting one of the clauses should not impact the errors for the others.

Added 3 instances of acceptance-of-delivery, caused a parse error in the middle (2nd) acceptance-of-delivery, and then deleted it. Clause with the error in contractState is removed from contractState in REMOVE_CLAUSE_FROM_CONTRACT and the corresponding clause template is removed from clauseTemplatesState upon deletion.
Adding multiple instances of the same template isn't a problem since each time a new clause is added, it gets a unique id. All I did was add the clause's id to the template item's state to ensure that the template deleted from the store has the clauseId that was just deleted. Since deleting a clause only passes the clauseId and not the template in which it lies, there was no other way to do it.
If this is the desired implementation I can recreate a pull request since I had to remake it due to the master branch update.

@irmerk @DianaLease What I wrote allows for multiple templates to be deleted correctly, even if they're the same template, which was the concern that closed the PR if I'm not mistaken? I'd like to know if I should open another PR or update something before continuing!

Based on https://github.com/accordproject/template-studio-v2/pull/144#issuecomment-550488134 from @Michael-Grover, I think #144 is the correct approach.


Daemon launches in mintty consoles don't disconnect from tty.

When the daemon is launched in the background using mutagen daemon in a mintty console, it has an event attached to the process that allows mintty (or Cygwin or someone) to track that it is still alive (event when we use CREATE_NEW_PROCESS_GROUP and DETACHED_PROCESS), and this can either prevent the mintty console from closing (it will just hang until the daemon is terminated) or reopening properly (it will say there are no available ttys until the daemon is terminated). I think the solution here is to manually trigger this event (it's either cygtty.input.avail.0 or cygtty.slave_alive.0).  There's a Windows API call for this called PulseEvent, but its documentation says it is unreliable.
In the mean time, there's a workaround: nohup mutagen daemon -r &. Or just launch the daemon from a cmd.exe console.
But anyway, if nohup can get around this problem then we probably can too, but whatever tty disconnection nohup is doing might rely on Cygwin C APIs that we can't get at from Go (or at least wouldn't want to), so we may have to hack this somehow.
An alternative, and perhaps better approach, would be simply developing a tray icon to host the daemon on Windows.

This is actually fixed in a very simple way: by wrapping the mutagen command inside the winpty command.
This workaround has been documented in fa06c07 and is thus fixed (at least as far as we can possibly fix it).


⚠ Bofrost.de has degraded performance

In ce59a64, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1848 ms

Resolved: Bofrost.de performance has improved in d955c89 after 22 minutes.


⚠ Bofrost.at has degraded performance

In a0d0443, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1340 ms

Resolved: Bofrost.at performance has improved in f8d0d50.


Issue with speed of clipper with custom PyTorch model

Dear Clipper admins,
I'm attempting to use Clipper to deploy a single model with approximately 1.3M parameters.
This model's forward pass works very quickly locally (0.1s), while the same model in Clipper generally takes 5 to 20 seconds (usually ~15) to do the forward pass.
I'm quite stumped at what the issue might be with this model, and while I would love to share a detailed example here, the code I am using is closed source.
Could you please help guide me in the right direction to solving my issue?
Many thanks in advance!

For comparison, I managed to implement the same model in Flask and get it doing predictions ~15 times faster, so my suspicion is that the issue is not (at least entirely) related to the containerisation.


HDDS-9886. hadoop fs -checksum fails with NoClassDefFoundError on Hadoop 2

What changes were proposed in this pull request?
hadoop fs -checksum (or hdfs dfs -checksum) fails with NoClassDefFoundError: CompositeCrcFileChecksum when using Hadoop 2.  CompositeCrcFileChecksum was added by HDFS-13056 only in Hadoop 3.2.0 (and backported to 3.1.1).  (Thus it may also fail with Hadoop 3.1.0 and Hadoop 3.0.x, but I haven't tried those.)
This PR fixes it by copying code from Hadoop.
https://issues.apache.org/jira/browse/HDDS-9886
How was this patch tested?
Added execution of hdfs dfs -checksum in Robot tests.
CI:
https://github.com/adoroszlai/ozone/actions/runs/7221000684/job/19675627539

Hadoop and Ozone have the same classpath.

We already have code with the same FQCN as the corresponding class in Hadoop, e.g. LeaseRecoverable (added in b97d943d5cee11933d4aed0665607c97c9be92ce).  I'm sure there are other examples as well.

Do you think we can do it but with a different classpath?

I can try.

Thanks @jojochuang, @szetszwo for the review.


Terminate nominations when members are not found

This is a continuation of #2323
Soon enough, we will be tracking nominations that have been sitting for more than 2 weeks.
When tracking these nominations, we can run into the case where a particular member is no longer part of our server, which leads to asking the question: what needs to happen then ?
I think that closing the nomination & archiving the thread (if it exists) would do the job, as there's no need to keep it open.
We could potentially track if there's an open issue for it in Github as well & close it.

We'll remove inactive members from the talent pool in https://github.com/python-discord/bot/issues/2302#issuecomment-1287221089. That, and ensuring we don't post vote threads for users not on the server should be enough IMO. This also avoids a user from being unnominated if they accidently left and rejoined or something similar.

I see, then i guess your other PR should take care of all of that since it'll be activity based, right?

Yeah, if a user isn't on the server then they can't send any messages of course, so it should have a similar effect.
For things like the github issue / active vote I think it's better for it to be handled manually if a user leaves so it doesn't just "disappear". We could send a message or something to let people know, although I don't think it's worth the added complexity for something so rare.
I'll close this, although thanks for thinking about this and making the issue!


Conflict Issues

@chukaibejih What service are you working on?

I haven't started yet but from the ER Diagram, I intend to start maybe lessons or module.
I see you have conflict issues, have "git pull" before pushing?

I have solved the conflicts. I have started modelling the course_service. I added an extra field "cover_image" which is not in the ER diagram for course.

I keep getting this error while migrating


Fix *.d parsing with gcc 9.3.0

Before this commit:

  $ rake
  ...
  $ rm build/host/src/gc.o
  $ rake
  rake aborted!
  Don't know how to build task '/home/foo/mruby/build/host/src/gc.o' (See the list of available tasks with `rake --tasks`)
  ...

After this commit:

  $ rake
  ...
  $ rm build/host/src/gc.o
  $ rake
  CC    src/gc.c -> build/host/src/gc.o
  ...

With gcc 9.3.0 on Ubuntu 20.04, build/host/src/array.d looks like:

  /build/host/src/array.o: /src/array.c \
   /include/mruby.h /include/mrbconf.h \
   /include/mruby/common.h \
   ...
   /include/mruby/range.h \
   /src/value_array.h

and it has been parsed to:

  [
                                           # /src/array.c missing
    "/include/mruby.h /include/mrbconf.h", # incorrectly parsed
    "/include/mruby/common.h",
    ...
    "/src/value_array.h",
  ]

After this change, *.d will be parsed correctly.

  [
    "/src/array.c",
    "/include/mruby.h",
    "/include/mrbconf.h",
    "/include/mruby/common.h",
    ...
    "/src/value_array.h",
  ]

Ref. https://github.com/mruby/mruby/pull/4571
cc @shuujii

Thank you for your report.
It doesn't depend on the version of GCC, but if the paths are short, it seems to be concatenated without line breaks (reproduced in Clang and older GCC).

ah thanks!
I generalized the comment on #5081.


Progress control

Q
A

Bug fix?
[ ]

New feature?
[x]

New sample?
[ ]

Related issues?
no

What's in this Pull Request?
Progress control shows progress of multiple SEQUENTIALLY executed actions.
The control's implementation is inspired by Site Designs run progress.

Codecov Report

Merging #230 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #230   +/-   ##
=======================================
  Coverage   72.71%   72.71%
=======================================
  Files          16       16
  Lines         744      744
  Branches      138      148   +10
=======================================
  Hits          541      541
  Misses        159      159
  Partials       44       44

Impacted Files
Coverage Δ

src/loc/en-us.ts
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

thanks @AJIXuMuK this was merged and will be available in version 1.13.0


chore(deps): update github-issues plugin version

Description
Update @backstage-community/plugin-github-issues version.
Which issue(s) does this PR fix

Fixes RHIDP-2727

PR acceptance criteria
Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer
Community plugins backstage version is: v1.30.2, so maybe this needs to wait till backstage update. Works locally fine:

/test e2e-tests

/test e2e-tests

/retest

/retest

/test e2e-tests

/retest

/test e2e-tests

Closing as github-issues version was updated in https://github.com/janus-idp/backstage-showcase/pull/1818


Add in-app barcode reader support

This may necessitate moving to react-native fully.

@robotoer I was researching how to just make the web app read from a handheld barcode scanner. It sounds like it just acts like a keyboard so it shouldn't be hard. Here are some Javascript examples.
I think this barcode scanner will require installing a driver on the computer that it's plugged into. So I still like the react native idea as a longer term solution that requires less setup. But I'm guessing that will require some tweaking to get accurate reading of linear barcodes, so in the short term is seems safer to rely on the barcode scanner that comes with the barcode label printer.

Thanks for doing that research! As far as I can tell, it shouldn't be too hard to get both working.
Regarding tweaking necessary to get the react native app version working, could you try out the test native app that's already in the project on some real labels once you have the printer in hand? My guess is that it should be able to read numbers from linear barcodes as-is!

I'll get started on the javascript reader this weekend!

Cool thanks! The other barcode-related thing (your call if it should be a separate ticket), is the ability to query other runs, by barcode. There is a 384-well plate barcode scan in pretty much every protocol, and later protocols need to grab info from earlier protocols to know what's in that plate, even if the earlier run was completed by a different person...

So upon further research it seems that the way these desktop barcode scanners seem to work is that upon scanning a barcode successfully, they "enter" the barcode's id/number/contents by simulating keypresses that match the id/number/contents. Thus, as long as the user has focused (read: clicked on) the input they want to scan a barcode label into, then it should just work as is.

Regarding the other data lookup UI, gotcha. Should be coming this week!

Marking this done since it works with a handheld barcode scanner.


Bug Report: Collection attributes are not saving

Describe the bug
with manage_collection permissions, we need to add in collection descriptions (and other attributes), but hitting save does not save the attribute-- the entry box disappears and no error message happens. The value clearly is not saving.
To Reproduce
Steps to reproduce the behavior:

Go to any collection attribute and try to add a description (only attribute I tested)
Click on save
Get nothing including no error

Clearly not the expected behavior! @RJSS8 noted this first

this bug is likely related https://github.com/ArctosDB/arctos/issues/8289
so I'll check out all functions once this is pushed to prod -thx


[ACR] Add record only flag to import and build-task tests

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[x] I adhere to the Command Guidelines.

@troydai /cc @northtyphoon @NatalieGong


Improve JS DOM interactions

A lot of JS code is using direct parent/chid DOM assumptions. It's bad because any change in the DOM structure will break it. Instead, it should use selectors as much as possible.

Relates to #133


ExpectationBuilder operator overhaul

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.181%

Totals

Change from base Build<PHONE_NUMBER>9:
0.0%

Covered Lines:
848

Relevant Lines:
855

💛  - Coveralls


Changed EBML DocType to match schema filenames/naming conventions

The DocType names in the schemata have been modified to either match or follow the conventions of the schema filenames. This is to allow (semi-) automatic schema detection when opening files, with a minimum of special-casing.
I'd rather have named them endaq_*, but half the filenames started mide_*, so that would be a greater change. Existing code doesn't use the DocType, only filenames.

DocType
schema filename

mide
mide_ide.xml (in idelib)

mide_flash_package
flash_package.xml

mide_command_response
command-response.xml

mide_config_ui
mide_config_ui.xml

mide_manifest
mide_manifest.xml


Fix QueryFilters sample program

In #2074, the name of the property "TenantId" was changed to "_tenantId", but the change was not propagated throughout the code, so the sample was not running anymore.

Thanks!


Enables WebAC by default.

Note:  this commit currently breaks the "clickability" of the one click.
See modifications to the README for new instructions. FWIW I tried a number of different
approaches including ensuring that the the filtered jetty-one-click.xml  ended up in the
jetty console jar as  WEB-INF/jetty.xml.  To no avail.
Resolves  https://jira.duraspace.org/browse/FCREPO-2631
What's new?
Basically this PR brings over webac the configuration from fcrepo-webapp-plus into the fcrepo-webapp module, ensuring that users who fire up fcrepo-webapp using the mvn jetty:run or the jetty console jar are able to see the webac framework in action.
How should this be tested?
Fire up the webapp as you normally would.  See README for the jetty console launch command.
You can login using fedoraAdmin:fedoraAdmin as your credentials.
Interested parties
Tag (@ mention) interested parties or, if unsure, @fcrepo4/committers

@awoods :  hold off on testing this until I work out a fix for the integration test failures.

@awoods :  okay this should be ready to go.  one click is once again one-clicky.  Other issues addressed.  <whew!>

@awoods :  okay - the auth has been pulled out for the one-click.

As of : https://github.com/fcrepo4/fcrepo4/pull/1246/commits/bc108bf7a14dfcf6c5d49061b242480edf91f9b5
When I run:
$mvn clean install -Pone-click
$java -jar fcrepo-webapp/target/fcrepo-webapp-5.0.0-SNAPSHOT-jetty-console.jar --headless
...I get a 403 with:
$curl -i localhost:8080/rest
$curl -i localhost:8080/rest -ufedoraAdmin:fedoraAdmin
$curl -i localhost:8080/rest -ufedoraAdmin:secret3
Should I be doing something differently?

@awoods:  my bad - I thought it was working, but my browser was caching my credentials.  I will fix.

or possibly I just looked at "/" rather than "/rest"

@awoods :  it is fixed now.


Geopolitical regions

Geopolitical/administrative regions should be available from something like GADM , FAO GAUL, or ISO19139 (e.g. ).
If no RDF is available, this may need to be created.
Note that ENVO has some of these, but may not be up-to-date. SDGIO could use these too.
@marieALaporte @elizabetharnaud @cmungall Do you think we should manage an OBO semantic version of this in ENVO and link to classes in, e.g., GAUL? Or should we go entirely with an external reference? GAUL looks pretty good as it maps to time stamped shapefiles that are relevant to the ag domain.
@brianjlowe FYI

GAUL licence seems to be very restrictive: http://www.fao.org/geonetwork/srv/en/metadata.show?id=12691

License: Formal permission to do something

maybe @jhpoelen knows

I've been using geonames - I believe they also have shapefiles associated with their administrative areas.

I second using geonames, they provide feature level containment which resolves a number of use cases such as "What country is this in".


Reduce Antrea binary sizes

@tnqn pointed out at the Antrea community meeting that the antctl binary which we include in the Antrea Docker image (as well as in release assets) is pretty large, almost 100MB, compared to binaries from other projects (for example, kubectl is only 50MB, yet packs more "features").
K8s uses https://github.com/kubernetes/kubernetes/blob/master/hack/lib/golang.sh to build production binaries, and it is probably a good reference.
A quick experiment revealed that building antctl with go build --ldflags "-s -w", which removes debug symbols, reduced the size of antctl-linux by around 30%.
before:
-rwxr-xr-x@ 1 abas  staff  94406832 Jan  4 07:20 antctl-darwin
-rwxr-xr-x@ 1 abas  staff  82919625 Jan  4 07:21 antctl-linux
-rwxr-xr-x@ 1 abas  staff  84231168 Jan  4 07:22 antctl-windows.exe

after:
-rwxr-xr-x@ 1 abas  staff  74835768 Jan 16 21:25 antctl-darwin
-rwxr-xr-x@ 1 abas  staff  57417728 Jan 16 21:25 antctl-linux
-rwxr-xr-x@ 1 abas  staff  59106304 Jan 16 21:26 antctl-windows.exe

This is a pretty significant gain. The obvious drawback is that if antctl panics "in the field", we won't have a useful traceback to use for troubleshooting (not accurate: we still get useful stack traces). However, it is very common to trim debug symbols from production binaries.
Removing debug symbols is a pretty low-hanging fruit. There may be other things we can do to further reduce binary sizes. Use this issue to document investigations.

[ ] Remove debug symbols from antctl release binaries (we already remove them from release assets, but not from antctl binaries included in container images). We could also remove debug symbols from other Antrea binaries (antrea-agent, antrea-controller), but that may impact our ability to troubleshoot issues in live clusters with no significant drawback (see below)..
[ ] Remove all file system paths by building release binaries with -trimpath. Unlikely to reduce the size much, but this is good practice. There should be no drawback.
[ ] Investigate whether some dependencies / imports are responsible for a significant size increase.

I was incorrect before: when using --ldflags "-s -w", we still preserve function names and line numbers. So stack traces for example still provide useful information. A lot of go tool pprof features are still accessible too (e.g. to inspect memory profiles), bunctionality that depends on access to the source code (e.g. the list command)  will require a local copy of the source code. Using these ldflags will also restrict the ability to use a debugger, which may not be as much as a concern (delve may also be able to use a local copy of the source code?).
On a separate not, I have noticed the following discrepancy: the antctl binaries included in release assets have debug symbols stripped while the antctl binaries included in Docker images do not.

Can we leverage the busybox way to combine all antrea executables into one binary?
We can implement a shared main package or just use some existing tools such as https://github.com/u-root/gobusybox/.
I did a quick test and confirmed with https://github.com/u-root/gobusybox/ that we can get a 64MB binary containing antctl, antrea-agent, antrea-cni, and antrea-controller.
makebb antrea.io/antrea/cmd/antrea-agent antrea.io/antrea/cmd/antctl antrea.io/antrea/cmd/antrea-controller
03:36:25 Disabling CGO for u-root...
03:36:25 Build environment: GOARCH=amd64 GOOS=linux GOPATH=/home/xliu2/go CGO_ENABLED=0 GO111MODULE= GOROOT=/usr/local/go PATH=/usr/local/go/bin
03:37:55 Successfully built "bb" (size 66994176 bytes -- 64 MiB).

~/antrea$ ls -lath bb
-rwxrwxr-x 1 xliu2 xliu2 64M Mar  1 03:43 bb

~/antrea$ ./bb
Failed to run command: command is not present in busybox: bb
Supported commands are:
 - antctl
 - antrea-agent
 - antrea-cni
 - antrea-controller

Can we leverage the busybox way to combine all antrea executables into one binary?

@xliuxu thank you for the proposal and the data. I can see the benefit to the image sizes, but it would also increase sizes of smaller binaries which needs to be distributed as individual executables, i.e. antrea-cni and antctl.

For antrea-cni, its size is 8MB, and needs to be copied to host's filesystem. If we have to copy a combined binary, we have to duplicate 64MB space which basically offsets the benefit, so it doesn't seem good to be included.
For antctl, its size is 45MB (after https://github.com/antrea-io/antrea/pull/6037), and we expect users to download it and use it locally, especially after it supports installation and upgrade. A combined binary will increase it by 20MB.
For antrea-agent and antrea-controller, they are delivered via two different images anyway, so building them into one isn't really helpful to the image sizes, and may even increase the image sizes a bit.
The only obviously beneficial combination is antctl + antrea-agent, and antctl + antrea-controller, which basically reduces the size of antctl. However, it also means antctl would be used in more than one way: the standalone antctl mode, the combined binary mode which needs a subcommand to use antctl. I'm not sure if we want to trade the inconstentcies with the saved size.

@antoninbas what's your opinion?

I feel the same way as @tnqn. That capability is great for programs that are meant to be shipped together most of the time, while preserving the ability to build them as separate binaries if needed. It may be better to put some additional effort into stripping unnecessary dependencies from antctl, antrea-agent and antrea-controller (I would say that antrea-cni is in a good place after the recent changes).

@antoninbas do you see any TODO for the item Investigate whether some dependencies / imports are responsible for a significant size increase.?


feat(nuxt): definePageMeta in "pages:extend"

🔗 Linked issue
resolve https://github.com/nuxt/nuxt/issues/19955

❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Hi :wave: , this PR moves the PageMeta plugin after module:done hook and use Pages instead of dirs to define wich files should be transformed

📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

I see... then should we just transform both dirs and pages ?
Dirs at initial module load and pages after modules:done ?

closing in favor of https://github.com/nuxt/nuxt/pull/20628


User defined features

Dear author, I've successfully run the colab, but is there any cell defines genre and instruments you mentioned in paper?
Or the colab just random selected it?

Our default pretained model uses all five instruments and without defining genre, so this colab instrument use all five instruments.

Sorry for other question, "In the first stage, the model predicts the type token by applying a linear projection on the output of the transformer. In the second stage, it uses type to pass through 6 feed-forward heads to predict the remaining 6 attributes at the same time".
I'd like to  ask, so first the model just predict type token, and then use the type token to continue predicting remaining 6 attributes, and then use inference stage to control, right?
Sincerely, William.

Yes, you can refer to forward_output (used in training) and forward_out_sampling (used in inference).


Update EIP-6381 interface

Description
The EIP-6381 interface was updated to include the latest version of the proposal.
There was a misidentification where IRMRKEmotes was labeled as IERC6381, where IRMRKEmoteTracker was the one that should have been renamed. This is now fixed, and the appropriate interfaces are assigned to the appropriate extensions.
Checklist

[x] Verified code additions
[x] Updated NatSpec comments (if applicable)
[x] Regenerated docs
[x] Ran prettier
[ ] Added tests to fully cover the changes

PR-Codex overview
This PR removes the IRMRKEmoteTracker interface and replaces it with IRMRKEmotable in the RMRKEmotable contract. It also adds new functions to the RMRKEmoteTrackerMock contract.
Detailed summary

Remove IRMRKEmoteTracker interface and replace with IRMRKEmotable in RMRKEmotable contract
Add new functions to RMRKEmoteTrackerMock contract:

bulkEmote
bulkEmoteCountOf
bulkPrepareMessagesToPresignEmote

The following files were skipped due to too many changes: docs/mocks/extensions/emotable/RMRKEmoteTrackerMock.md, contracts/RMRK/extension/emotable/IERC6381.sol, docs/RMRK/extension/emotable/RMRKEmoteTracker.md, contracts/RMRK/extension/emotable/RMRKEmoteTracker.sol, docs/RMRK/extension/emotable/IERC6381.md

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

This still needs the tests to be migrated from our proposal, but it serves its purpose for now. I need this as I want to start recalculation of the repository address.

We could modify the extension version to use the repository out of the gate (since the address is predetermined) 🤔

We could modify the extension version to use the repository out of the gate (since the address is predetermined) 🤔

What do we get from it? I don't see any value, just more code to maintain.


ActionBar ActionItem

Which platform(s) does your issue occur on?

iOS
emulator

For some reason it doesn't seem possible to have the popup attached to an ActionItem like below
<ActionBar title="Projects" class="action-bar">
  <StackLayout orientation="horizontal" ios:horizontalAlignment="center" android:horizontalAlignment="left">
    <Image src="res://logo" height="32"></Image>
  </StackLayout>
  <ActionItem id="menu" text="Options" ios.position="right" (tap)="showMenu()"></ActionItem>
</ActionBar>

However it seems like you can put a Label inside an ActionItem with an id and that will work.
<ActionBar title="Projects" class="action-bar">
  <StackLayout orientation="horizontal" ios:horizontalAlignment="center" android:horizontalAlignment="left">
    <Image src="res://logo" height="32"></Image>
  </StackLayout>
  <ActionItem ios.position="right" (tap)="showMenu()"><Label id="menu" text="Options"></Label></ActionItem>
</ActionBar>

Not sure if this is a bug or not, but at least this will help anyone else that comes across this problem.

Thank you! The label trick saved me!!  I wonder why it doesn't work directly on an ActionItem?


Please clarify your name.

Would you fill your name and year in license.
Copyright [yyyy] [name of copyright owner]

https://github.com/greenrobot/EventBus/blob/master/LICENSE

Would you tell me who has copyright of EventBus?
Please see this link.
https://github.com/JakeWharton/butterknife#license
As you know Jake Wharton is author of Butter Knife.
And I don't know who has copyright of EventBus.