Index file nodes for rust crates [phab:D941]

Created by djrenren at 2016-06-22 23:14:13:
A barebones implementation of a lint-plugin-based indexer for rust.

djrenren wrote at 2016-06-22 23:30:48:

Remove wildcard from Cargo dependencies

zarko wrote at 2016-06-22 23:45:12:
How should I build and run the tests? bazel test //... didn't run anything Rust-related, then:
bazel build //kythe/rust/indexer:libindexer
INFO: Found 1 target...
ERROR: /usr/local/google/home/zarko/kythe-trunk/kythe/rust/indexer/BUILD:5:1: Executing genrule //kythe/rust/indexer:_libindexer-k8 failed: bash failed: error executing command /bin/bash -c ... (remaining 1 argument(s) skipped): com.google.devtools.build.lib.shell.BadExitStatusException: Process exited with status 1.
tools/build_rules/rust/run_cargo.sh: line 6: cd: kythe/rust/indexer/target/debug/: No such file or directory
Target //kythe/rust/indexer:libindexer failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 0.534s, Critical Path: 0.04s

djrenren wrote at 2016-06-22 23:49:03:
zarko: Tests can currently be run as:
bazel test //kythe/rust/indexer/testdata:basic

As to your issue I can't reproduce here it looks like cargo build is failing silently. Could you comment the result of using --verbose_failures

djrenren wrote at 2016-06-22 23:57:31:

Make run_cargo.sh cd properly

djrenren wrote at 2016-06-23 00:26:32:

Actually run the tests...

zarko commented inline (context lost during migration) at 2016-06-23 00:59:52:
This comment's cut off.

zarko commented inline (context lost during migration) at 2016-06-23 00:59:52:
Would it be better to define constants in a separate module instead of using inline string literals? See e.g. https://github.com/google/kythe/blob/master/kythe/go/util/schema/schema.go

zarko commented inline (context lost during migration) at 2016-06-23 00:59:52:
nit: add a period after complete sentences (here and elsewhere).

zarko commented inline (context lost during migration) at 2016-06-23 00:59:52:
nit: alphabetize imports?

zarko commented inline (context lost during migration) at 2016-06-23 00:59:52:
nit: avoid unnecessary linebreaks.

zarko commented inline (context lost during migration) at 2016-06-23 00:59:52:
Generally we've found it useful in indexers to limp forward rather than crash outright when something weird happens. Maybe you'll be able to output data for other files?

zarko commented inline (context lost during migration) at 2016-06-23 00:59:52:
This file needs a copyright header.

zarko commented inline (context lost during migration) at 2016-06-23 00:59:52:
Is this necessary (if you're defaulting to an empty corpus in your indexer)?

zarko commented inline (context lost during migration) at 2016-06-23 00:59:52:
I don't think CLANG_STANDARD is relevant here.

zarko commented inline (context lost during migration) at 2016-06-23 00:59:52:
This also needs updated.

zarko commented inline (context lost during migration) at 2016-06-23 00:59:52:
You might want to start out with this set to False.

zarko commented inline (context lost during migration) at 2016-06-23 00:59:52:
This is an interesting approach; how do you plan on handling links across corpora?

zarko commented inline (context lost during migration) at 2016-06-23 00:59:52:
base64 encoding is a requirement of the JSON wire format, not of the schema in general. Is this a layering violation?

zarko commented inline (context lost during migration) at 2016-06-23 00:59:52:
Edge kinds are strings (and must be utf8-encoded).

zarko commented inline (context lost during migration) at 2016-06-23 00:59:52:
Do you need to keep these around in memory? How do you feel about using a streaming output interface that eagerly writes Entrys? In that case you might not have to do quite so much copying.

zarko wrote at 2016-06-23 00:59:52:
Can the build rules be instructed to fail or print a warning if a nightly Rust isn't installed?
Regarding style mechanics: it seems like the Rust community standard is 99 columns and 4-space tabs. I don't see why we shouldn't stick to this (+ mandatory rust-fmt).
It would be useful to include a README.md that describes how the indexer should be invoked (and its various command-line and environment options).

djrenren commented inline (context lost during migration) at 2016-06-23 01:20:49:
haha whoops was supposed to be a continuation of the same sentence. Lost track of my grammar halfway through

djrenren commented inline (context lost during migration) at 2016-06-23 01:20:49:
Hadn't thought about that really... I suppose regardless of how I implement it, I'll need to define some way of noting and detecting corpora boundaries. I suppose I could encode that info into a CorporaConfig™ that encodes the base corpus name and then the required information to detect boundaries (presumably at crate and ffi boundaries?)

djrenren commented inline (context lost during migration) at 2016-06-23 01:20:49:
These actually are effectively streaming.  (see src/pass.rs). The interface of this function allows it to be separated from the driving logic. I'll have to look into the performance concerns here.

djrenren commented inline (context lost during migration) at 2016-06-23 01:20:49:
yes

djrenren commented inline (context lost during migration) at 2016-06-23 01:20:49:
Whoops, that's what I get for trying to commit code I'm not currently running. Yeah all strings in rust are UTF-8

djrenren commented inline (context lost during migration) at 2016-06-23 01:20:49:
The fact_value is a private member of the struct so once the object is created it's inaccessible. I'll have to check rustc, I may be able to store it as bytes on the node if rustc will base64 encode on the way out.

djrenren commented inline (context lost during migration) at 2016-06-23 01:20:49:
fair :(

djrenren commented inline (context lost during migration) at 2016-06-23 01:20:49:
so for crates and use statements I can totally do that. Modules not so much because order matters.

djrenren commented inline (context lost during migration) at 2016-06-23 01:20:49:
Sure thing. Just figured an inability to read the file was indicative of a big enough problem to abort. I'll bring it in line with everything else.

djrenren commented inline (context lost during migration) at 2016-06-23 01:20:49:
Huh didn't see it in our other build files, care to double check that?  The CXX indexer and indexer testdata BUILD files don't include one.

djrenren commented inline (context lost during migration) at 2016-06-23 01:20:49:
Because the tests are run locally, I believe it's possible an environment variable could leak in.

djrenren commented inline (context lost during migration) at 2016-06-23 01:20:49:
Copy pasta got me again.

djrenren wrote at 2016-06-23 01:20:49:
Will post an update tomorrow with the listed fixes.

shahms commented inline (context lost during migration) at 2016-06-23 20:28:24:
why? does order matter?

djrenren commented inline (context lost during migration) at 2016-06-24 18:46:07:
Followed the example of cxx_indexer_test where the first element is the entrypoint for the compiler.

djrenren wrote at 2016-06-24 18:48:39:

Address feedback. move constants into their own modules.

zarko commented inline (context lost during migration) at 2016-06-26 18:10:32:
That sounds reasonable (for a future diff). I also think that it's fine to assign corpora to crates (and not some finer unit of code).

zarko commented inline (context lost during migration) at 2016-06-26 18:10:32:
If you're only ever going to dump Entrys to output, wouldn't it make more sense to pass in an interface (possibly with node, edge, and ordinal-edge functions) that could do so without building intermediate data structures?

djrenren commented inline (context lost during migration) at 2016-06-27 17:21:49:
Yeah definitely could. Was avoiding this strategy at first because I wanted to minimize things I'd have to mock in tests. but yeah that'd probably be a solid performance gain. I can do that.  (Plus most of the testing will be done with the verifier anyway)

zarko accepted this revision at 2016-06-27 20:13:39

djrenren closed via commit d1aca5ad0603d723aab8f7913e3f6cca378954d0 at 2016-06-27 20:39:01


improves the display of text and java code

Plain text was getting merged with java code which has been fixed

@maverickmishra - who are you looking to review this?

sent a link to @stevengill

Cool - I assigned him.  Looks like the tests are failing because wikitude changed their docs url structure. I would just ignore them for now and I'll check in with Tim again on it.

What is the status of this? Are we merging and publishing?
We have a number of duplicates or let's call them re-affirmations :
https://github.com/phonegap/phonegap-intellij-plugin/issues/11
https://github.com/phonegap/phonegap-roadmap/issues/213


DOC: reconstruct SuperLU permutation matrices avoiding SparseEfficiencyWarning

The current documentation for scipy.sparse.linalg.SuperLU suggests creating an empty csc_matrix for each of the two permutation matrices and then populating it with ones in the right spots.  This raises

SparseEfficiencyWarning: Changing the sparsity structure of a csc_matrix is expensive. lil_matrix is more efficient.

However, it's just as easy to construct the populated matrices in a single step using the standard (data, (row_ind, col_ind)) constructor.

Thanks!

Without this patch the latest NumPy causes a refguide failure -- so labelling appropriately


Fix VBox VM Startup with 3D Acceleration Enabled

Description of changes
When 3D acceleration is enabled, VirtualBox needs libvulkan.so to start. This PR adds this to the existing wrapper.
In case, we can't wrap the binary this will not work unfortunately.
Fixes #313372.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This is confirmed to resolve the issue.

I will approve the PR once #303790 is merged

I feel like this fix is the same/similar as/to https://github.com/NixOS/nixpkgs/issues/282717.


Remove Plan-B Support

August 2021 is when it is being removed from Chrome so no point for us to continue supporting.
We also have some blocking issues for v3.1.0 around Plan-B. I would rather remove this code then fix something that is already on the way out.
My plan is to remove this in v4. I don't have a timeline for when that will be at this point.

Yes, it has a sense.


get_all should be able to accept an empty list

to reproduce:
test "get all should be able to accept an empty list" do
   result = table(@table_name) |> get_all([]) |> run
   assert result.data == []
end

in selection_test.exs

Fixed with 459a7cfe750957cf01a09718f71c709df8e59d8d


fix: always use session mode when connecting to pooler

What kind of change does this PR introduce?
Bug fix https://github.com/supabase/cli/issues/2077
What is the new behavior?
Since CLI commands are short lived, one off commands, there's no reason to use transaction mode.
Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
7 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.03%) to 58.981%

Files with Coverage Reduction
New Missed Lines
%

internal/utils/connect.go
2
96.72%

internal/gen/keys/keys.go
5
11.76%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
6262

Relevant Lines:
10617

💛  - Coveralls


Banner Redesign

A Banner/Logo Redesign needs to be done, I'll take care of this. Right now the current one is this. Originally part of #3

@Fleker Thoughts on the first draft/shot at it?

Looks a lot better than mine


Vesting timeoutconv can erase wrong object

it's possible to first move to the next object:
https://github.com/GolosChain/golos.contracts/blob/aaa75d7ced128ef99f331339f05d2760c1601836/golos.vesting/golos.vesting.cpp#L366
and then erase it instead current:
https://github.com/GolosChain/golos.contracts/blob/aaa75d7ced128ef99f331339f05d2760c1601836/golos.vesting/golos.vesting.cpp#L373
also code can modify record first and then erase it

PR #736


Get-OpenADComputer: LastLogonTimestamp blank if requested directly via -Property

Thanks for the bug report, it looks like I was use a case sensitive check when building the final AD object so that when it was getting the value LastLogonTimestamp as requested by -Property it failed to match lastLogonTimestamp that was returned by the server. The PR https://github.com/jborean93/PSOpenAD/pull/83 should solve this issue.


Any chance mergeIn method will be implemented?

+1, would also love to have a complimentary withoutIn method as well to delete nested keys.
ie.
Immutable({ a: { b: 1, c: 2 } }).withoutIn(['a'], 'b')
// -> Immutable({ a: { c: 2 } })

Not without a compelling motivating use case. :wink:

@rtfeldman I just found out the hard way that there isn't a mergeIn method, I just assumed it was there :)
I think that there are some very compelling use cases. A common pattern in redux is to have a object with all entities, and then use different set that just contains the ids that corresponds to those objects. e.g. I have this object
{
  entities: {
    '1': { ... },
    '2': { ... },
    '3': { ... },
    ...
  },
  sets: {
    active: [1, 6, 9, 31, ...]
    needAction: [1, 3, 5, 6, ...]
  }
}

To then add more entities is a simple merge on the entities obejct. But it's important that it's not a deep merge so that removed properties will get properly removed when updating the entities. A mergeIn function would be perfect now
fetch('/animals?needAction=true')
  .then(res => res.json())
  .then(Immutable)
  .then(animals => {
    return state.
      .setIn(['sets', 'needAction'], animals.map(animal => animal.id))
      .mergeIn(['entities'], animals.asObject(animal => [animal.id, animal]))
  }
}

@LinusU I would keep sets and entities in different state slices. And handle them with different reducers.

@LinusU That kind of data manipulation should occur only in reducers. I would keep sets and entities in different state slices and handle them with different reducers. And you won't need mergeIn for it.

Yes it should be done in a reducer, I just wanted to show some code that made more sense for people not coming from redux...
I'm not use that I agree on having them in different reducers though, since the two properties are tightly coupled I don't see why they couldn't be manipulated by the same reducer?

parent: { [parentId]: { children: { [id]: { ... } } } }
state.mergeIn(['parent', parentId, 'children', id], { merge me })
Is there anything that would be as clean as mergeIn above? It would be nice to have a completely flattened state, but this isn't always desirable.

It seems to be trivial operation.
For example, we have state shape:
const initialState = Immutable({
  someBusinessData: {},
  ui: {
    filters: {
      name: "",
      department: "",
    }
  }
})

And here is the reducer:
const reducer = (state = initialState, action) => {
    switch (action.type) {
      case "ui/change-filter": {
        return state.updateIn(["ui", "filter"], filter => filter.merge(action.payload));
      }
      default:
        return state;
    }
}

You can update a deeply nested parts of your state by mergeIn operation. It can be patches to existing models (especially when you have normalised store and getting access by some ID prop), reverting some optimistically updated values.
I think we should implement mergeIn method in seamless-immutable...


[ISSUE-626] E2e testing in GitHub Actions

Solved #626
Purpose
Automate e2e testing
To run the tests, you need to create an issue_comment with next structure:
/ci
operator_branch=branch1

and the tests will run automatically.
At the end, 2 artifacts will appear in your namespace: log.txt and report.html.
How its work you can found here: https://github.com/AnnDutova/csi-baremetal/pull/2
Situations:

Run test with default operator branch
 /ci
Answer:

Start CI
Parameters: operator_branch=''

operator_branch='' means that he will use default branch.

Run test with branch that doesn't exist

/ci
operator_branch=some_branch

Answer:
Branch doesnt exist
Parameters: operator_branch='some_branch'

When entering the required branch, we make a mistake in the keyword

/ci
operator_brnch=master

Answer
Keyword error
Parameters: operator_branch='keyword'

Right input

/ci
operator_branch=master

Answer
Start CI.
Parameters: operator_branch='master'

As default bot will sent information about result in any cases: CI tests failed/passed
PR checklist

[x] Add link to the issue
[x] Choose PR label
[ ] New unit tests added
[ ] Modified code has meaningful comments
[x] All TODOs are linked with the issues
[ ] All comments are resolved

@mishoyama Could you please take a look?

@AnnDutova @safronovD do we need to have the same flow in csi operator repo?

@AnnDutova @safronovD do we need to have the same flow in csi operator repo?

Yes, I can create task for it


refactor: remove custom ProjectionExec

We implemented our own custom projection ExecutionPlan that took a schema to project. With the latest DataFusion functions (particularly named_struct) we can now express this in terms of the built-in ProjectionExec. This reduces the maintenance burden on us and ensures the equivalents properties and optimizations rules from DataFusion work well without much effort on our part.

Codecov Report
Attention: Patch coverage is 90.90909% with 23 lines in your changes missing coverage. Please review.

Project coverage is 79.42%. Comparing base (e5e6fd5) to head (f2193ef).
Report is 1 commits behind head on main.

Files
Patch %
Lines

rust/lance/src/io/exec/projection.rs
92.00%
10 Missing and 8 partials :warning:

rust/lance/src/dataset/scanner.rs
84.21%
0 Missing and 3 partials :warning:

rust/lance/src/dataset/write/merge_insert.rs
75.00%
0 Missing and 1 partial :warning:

rust/lance/src/io/exec/knn.rs
80.00%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2608      +/-   ##
==========================================
+ Coverage   79.33%   79.42%   +0.08%
==========================================
  Files         213      213
  Lines       62538    62634      +96
  Branches    62538    62634      +96
==========================================
+ Hits        49615    49747     +132
+ Misses      10012     9980      -32
+ Partials     2911     2907       -4

Flag
Coverage Δ

unittests
79.42% <90.90%> (+0.08%)
:arrow_up:

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

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


The 'xstate-ninja' library may need to update its package.json or typings

working on a typescript project with vscode and vite, I got this vscode error :
Could not find a declaration file for module 'xstate-ninja'. '/project/node_modules/xstate-ninja/dist/xstate-ninja.js' implicitly has an 'any' type.
  There are types at '/project/node_modules/xstate-ninja/dist/index.d.ts', but this result could not be resolved when respecting package.json "exports". The 'xstate-ninja' library may need to update its package.json or typings.ts(7016)

what should I do to fixed this issue? or this library may need to update the package.json?
Here's I found similar issue on SO https://stackoverflow.com/a/76212193

Thanks for the report, I will make a PR momentarily.

Hi @rlaffers , thanks for updating.
however, the issue still exists in package @xstate-ninja/react
Could not find a declaration file for module<EMAIL_ADDRESS>
How to reproduce:
import { useMachine } from '@xstate-ninja/react'

@deden Version<EMAIL_ADDRESS>is out, should be fixed now.


Soft api client example

Codecov Report

Merging #36 (2527298) into master (307696f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #36   +/-   ##
=======================================
  Coverage   95.82%   95.82%
=======================================
  Files          29       29
  Lines         766      766
  Branches       96       96
=======================================
  Hits          734      734
  Misses         12       12
  Partials       20       20

Continue to review full report at Codecov.

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


Automated PR from brain-bican/models: ${ALL_CHANGED_FILES} have been modified.

This was a PR generated while testing new push_to_bkbit workflow in brain-bican/models repo. This PR does not need to be merged.


TimeoutError while checking bedrock status

I get the following error while checking bedrock status on a discord bot
Traceback (most recent call last):
  File "/app/.heroku/python/lib/python3.10/site-packages/disnake/ext/commands/slash_core.py", line 730, in invoke
   await call_param_func(self.callback, inter, self.cog, **kwargs)
 File "/app/.heroku/python/lib/python3.10/site-packages/disnake/ext/commands/params.py", line 1022, in call_param_func
   return await maybe_coroutine(safe_call, function, **kwargs)
 File "/app/.heroku/python/lib/python3.10/site-packages/disnake/utils.py", line 596, in maybe_coroutine
   return await value
 File "/app/cogs/minecraft.py", line 16, in check
   status = server.status()
 File "/app/.heroku/python/lib/python3.10/site-packages/mcstatus/utils.py", line 65, in sync_wrapper
   raise last_exc  # type: ignore # (This won't actually be unbound)
 File "/app/.heroku/python/lib/python3.10/site-packages/mcstatus/utils.py", line 61, in sync_wrapper
   return func(*args, **kwargs)
 File "/app/.heroku/python/lib/python3.10/site-packages/mcstatus/server.py", line 215, in status
   return BedrockServerStatus(self.address, self.timeout, **kwargs).read_status()
 File "/app/.heroku/python/lib/python3.10/site-packages/mcstatus/bedrock_status.py", line 55, in read_status
   data, _ = s.recvfrom(2048)
TimeoutError: timed out

I am getting this error on both versions v9.4.0 and v10.0.1, It used to work earlier but not working anymore, I didn't update anything in the code.

Can't reproduce with 10.0.0 (patch inside 10.0.1 fixed answer object, it can't do anything with this issue)
Python 3.10.8 (tags/v3.10.8:aaaf517, Oct 11 2022, 16:50:30) [MSC v.1933 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import mcstatus
>>> mcstatus.BedrockServer.lookup("play.histeria.fr").status()
<mcstatus.bedrock_status.BedrockStatusResponse object at 0x0000021AE5B5B820>

Could you give IP? Also, try to ping the server with https://mcsrvstat.us. If this worked before, when did it stop?

The bot is hosted on heroku. They use shared IP.
It was working properly 1 or 2 months ago, when I tried it today then it didn't work.

I mean server's IP. Bot's IP doesn't matter.

Minecraft serber's IP? it's ggnetworkk.aternos.me:34624

Still can't reproduce
>>> mcstatus.BedrockServer.lookup("ggnetworkk.aternos.me:34624").status()
<mcstatus.bedrock_status.BedrockStatusResponse object at 0x0000014A230EB820>

Send your code, please.

        server = BedrockServer.lookup('ggnetworkk.aternos.me:34624')
        status = server.status()
        if status.players_max == 1:
            its = "online"
        else:
            its = "offline"```

Did you try to ping your server from your computer? (not heroku)

Gives an error, it waits for few seconds & gives an error
Traceback (most recent call last):
  File "C:\test.py", line 4, in <module>
    status = server.status()
  File "C:\Users\New\AppData\Local\Programs\Python\Python39\lib\site-packages\mcstatus\utils.py", line 66, in sync_wrapper
    raise last_exc  # type: ignore # (This won't actually be unbound)
  File "C:\Users\New\AppData\Local\Programs\Python\Python39\lib\site-packages\mcstatus\utils.py", line 62, in sync_wrapper
    return func(*args, **kwargs)
  File "C:\Users\New\AppData\Local\Programs\Python\Python39\lib\site-packages\mcstatus\server.py", line 213, in status
    return BedrockServerStatus(self.address, self.timeout, **kwargs).read_status()
  File "C:\Users\New\AppData\Local\Programs\Python\Python39\lib\site-packages\mcstatus\bedrock_status.py", line 49, in read_status
    data = self._read_status()
  File "C:\Users\New\AppData\Local\Programs\Python\Python39\lib\site-packages\mcstatus\bedrock_status.py", line 58, in _read_status
    data, _ = s.recvfrom(2048)
socket.timeout: timed out

Exact code:
from mcstatus import BedrockServer
server = BedrockServer.lookup('ggnetworkk.aternos.me:34624')
status = server.status()
if status.players_max == 1:
      its = "offline"
else:
      its = "online"
print(its)

Works to me, @ItsDrike @kevinkjt2000 could you look at it?

Now suddenly it starts working locally but doesn't work on bot.

If it's working locally, you should write Heroku's support. This is their side problem.

I think issue is something else, sometimes it works but sometimes it doesn't.

Maybe atternos not always responses on pings? Anyway, this doesn't look like our side problem.

I asked their staff, it does respond to ping always.

Who? Bot? So issue can be closed?

it means aternos

Oh, it is expected. You should better ask Heroku.

As I said it does not work on my pc too sometimes, I client is not a problem, it's something else and doesn't seem like rate limits

This is what the site gives

Are you sure that server still active?

nvm, my mistake it wasn't

The code printed "offline" when I tried it. It would seem as if whatever machine the bot is running on is suffering from some resource issue or firewall. If this code works on multiple other machines, there's obviously no issue with mcstatus...


enable ssh

@cloudnautique I keep meaning to update the sshd_config.  Can you send a link for amazon's recommended settings.


Thank you for Flapi and request for guidance

Dear Flapi developers,
First and foremost, I want to express my sincere gratitude for developing Flapi. This library has the potential to greatly enhance workflow automation in FL Studio, and I'm excited about its possibilities.
I'm working on a project to automate the creation of tracks in FL Studio based on a JSON structure. However, I've encountered some difficulties as I'm not sure about the correct way to use Flapi for this purpose. My current approach seems to be incorrect, as I've noticed that Flapi doesn't have transport, channels, plugins, patterns, or playlist modules that I initially assumed it would have.
from flapi import transport, channels, plugins, patterns, playlist
import json

def create_track_from_json(json_data):
    data = json.loads(json_data)

    transport.set_bpm(data["tempo"])

Could you please provide some guidance on how to properly use Flapi to achieve the following tasks:

Setting the project tempo
Creating new channels
Adding instruments (plugins) to channels
Creating patterns and adding notes to them
Adding patterns to the playlist
Setting plugin parameters
Adding and configuring effects

Here's a snippet of the JSON structure I'm working with:
{
    "tempo": 140,
    "channels": [
        {
            "name": "Kick",
            "instrument": "FPC",
            "patterns": [
                {
                    "name": "Main Kick",
                    "notes": [
                        {"time": 0, "note": 36, "duration": 0.25, "velocity": 100},
                        {"time": 1, "note": 36, "duration": 0.25, "velocity": 100}
                    ]
                }
            ],
            "effects": [
                {"name": "Fruity Compressor", "settings": {"threshold": -6, "ratio": 4}}
            ]
        }
    ]
}

Any examples or explanations of the correct Flapi usage for these tasks would be immensely helpful. I'm eager to learn and make the most of this fantastic library.
Thank you again for your hard work on Flapi, and I look forward to your guidance.
PS and yes, LLM helped me in writing this)

Hey! Unfortunately, Flapi is currently undergoing a pretty significant rewrite to address some major stability issues on Windows, so for the time being it's not very reliable on anything other than MacOS.
In terms of your feature requests, Flapi is unfortunately limited by FL Studio's Python API capabilities. Here are the things you should be able to do:

Set the project tempo
Create patterns
Set plugin and FX parameters

The things that currently aren't possible are:

Creating new channels
Adding instruments to channels
Adding notes to patterns
Adding patterns to the playlist
Adding FX plugins to a mixer track

All of these features have been requested to Image-Line, so hopefully they will become possible in the future. For the time being, here is the documentation for FL Studio's Python API, which is what you would want to look at for how to do all of the above features.

As for the error that you're getting, my guess is this is to do with LLM hallucinations -- FL Studio's MIDI Controller Scripting API is comparatively new, so many LLMs don't know much about how to use it, and so hallucinate things.
If the modules are failing to import, that could be due to a recent issue with the FL Studio API Stubs library where the modules weren't being distributed correctly. Perhaps try upgrading that dependency (pip install --upgrade fl-studio-api-stubs) and see if it helps :)

I never did manage to get this to run on OSX, or scripting in general. I just assumed support was only catered to Windows.

Thank you very much for clarifying these points. If the API were more widely implemented, then my experience of creating music would have changed dramatically with the support of LLM. But apparently we’ll have to wait for major players and capital to get down to business


Setting initial positions of a kinematic chain is not possible

As a followup to #108, I just came up with another limitation: With using kinematic chains in the gazebo model, the solution of using SetPosition does not work:

(1653985691 636065512) [Dbg] [Link.cc:1724] we have a loop! cannot find nice subset of connected links, this link boom_cylinder_barrel connects back to parent moved_with_q1.
(1653985691 636103177) [Wrn] [Joint.cc:815] failed to find a clean set of connected links, i.e. this joint is inside a loop, cannot SetPosition kinematically.

The warnings are thrown here
https://github.com/osrf/gazebo/blob/gazebo11/gazebo/physics/Joint.cc#L813-L819
https://github.com/osrf/gazebo/blob/gazebo11/gazebo/physics/Link.cc#L1672-L1677
As far as I understand, SetPosition is simply not possible with kinematic chains, because the solution of the kinematics equations is not implemented within gazebo via position interface? Or does someone know a workaround for this?

I'm just bumping this during looking for solutions :D
https://robotics.stackexchange.com/questions/107157/how-to-control-a-closed-kinematic-chain-in-gazebo


feat(cli): add cli sort flag to subcommands

What type of PR is this?

Which issue does this PR fix:
What does this PR do / Why do we need it:
If an issue # is not available please add repro steps and logs showing the issue:
Closes #1625
Testing done on this change:

Automation added to e2e:

Will this break upgrades or downgrades?
Does this PR introduce any user-facing change?:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@laurentiuNiculae add examples in this PR conversation itself and cc: @mbshields

The sort-by flag is optional and can be used as such:
$ zli image list --sort-by alpha-asc --config zot-config
$ zli image cve --sort-by severity --config zot-config
$ zli search query --sort-by relevance --config zot-config
Each command has it's own list of sorting criteria:
list base     [ update-time, alpha-asc, alpha-dsc ]
list derived  [ update-time, alpha-asc, alpha-dsc ]
list digest   [ update-time, alpha-asc, alpha-dsc ]
list list     [ update-time, alpha-asc, alpha-dsc ]
list name     [ update-time, alpha-asc, alpha-dsc ]
list cve      [ severity   , alpha-asc, alpha-dsc ]

cve affected [ update-time, alpha-asc, alpha-dsc ]
cve fixed    [ update-time, alpha-asc, alpha-dsc ]
cve list     [ severity   , alpha-asc, alpha-dsc ]

search query   [ relevance, update-time, alpha-asc, alpha-dsc ]
search subject [ update-time, alpha-asc, alpha-dsc ]

repo list [ alpha-asc, alpha-dsc ]


Added elementary math fizz buzz

Used elementary math techniques for checking if divisible by 3 and 5
Added digits to check if it will be divisible by 3
checked last digit if 0 or 5 to be divisible by 5

Neat! Thanks :)


sdl: make include directories PUBLIC in fallback path

Because the library link is PUBLIC, when we have a new SDL2 with the targets defined, the include dirs also end up public. Change this in the fallback path for consistency.
This change is a result of me needing to call an SDL function in something as a workaround, which built fine locally... but blew up the actions build.

Oh, since I updated the actions to 22.04, the fallback code isn't getting tested. Hmm.


Checked "Field available in advanced search" causes error on composer autosave when included in a page type form

Steps to reproduce on 8.0.3:

Create a custom attribute that has "Field available in advanced search." checked.  I was able to cause this with a custom date attribute and a custom checkbox attribute.
Create a new page type and add the attribute to the form for it.
Create a new page in Composer using this page type.
Type some content into a field.
Move to another field and wait for the autosave message to appear.
Change another field and then focus a different field.
Wait for error to show (blank for me, although browser console shows 500 error)

I toggle the "Field available in advanced search." setting back and forth and it changes whether or not the error occurs.

This seems unlikely. Have you posted this to concrete5.org ?

Hmm. Didn't work on a brand new 8.0.3 install.  The site it's happening on is 8.0.3 upgraded from 8.0.0.  Site errors show blank (was troubleshooting that with Liquidweb and then Korvin on Slack, to no avail).  Didn't post to concrete5.org.

If I manually visit the page called via ajax I get:
An exception occurred while executing 'UPDATE CollectionSearchIndexAttributes SET ak_event_homepage = ? WHERE cID = ?' with params [null, "459"]: SQLSTATE[42S22]: Column not found: 1054 Unknown column 'ak_event_homepage' in 'field list'


How i Can Style This component currently it is full screen size . i want to make it 50% of screen

What version of ionic-selectable are you using?


No mention of where to get "psping.exe"

Step 2.4 says to run this command ".\psping.exe kms.core.windows.net:1688".  Command returns "The term 'psping.exe' is not recognized as the name of a cmdlet, function, script file, or operable program"  Upon further investigation, I found that psping.exe is part of the SysInternals tools.  This should be explained and a link to the tool provided.

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

ID: 87ea8024-c99f-0efe-f663-cc58348a3721
Version Independent ID: 85c8727b-915b-c920-76cd-6123844d20ad
Content: Troubleshoot Windows virtual machine activation problems in Azure
Content Source: articles/virtual-machines/troubleshooting/troubleshoot-activation-problems.md
Service: virtual-machines-windows
GitHub Login: @genlin
Microsoft Alias: genli

Hi @Dirk-Manderin,
The doc tells you to download and extract psping.exe in step 2.1. It is also hyperlinked to the relevant sysinternals download doc. Please see here . If you have no further issues I will close this issue now. Feel free to leave a comment below if you have anything else to add.


Need add get_json_object function to compatible with Hive Function

Feature request
Is your feature request related to a problem? Please describe.

we would meet a error as no such function when we use hive external function such as get_json_object，to compatible with get_json_object of hive, we need add get_json_object function and it would implement ability of get_json_string.
Describe the solution you'd like

Describe alternatives you've considered

Additional context

we would meet a error as no such function when we use hive external function such as get_json_object，to compatible with get_json_object of hive, we need add get_json_object function and it would implement ability of get_json_string.

@wangsimo0 PTAL

LGTM


Using milestones for minor and major version releases

** Problem Statement **
Currently, we are using labels to determine release version for this issue and pull request. This make it harder to track the progress for specific release and you can't assign release date for those labels. In addition, this could explode the list of labels in the future, since we have a release every ~6 weeks, that mean we will add ~8 additional labels every year.
** Proposal **
Using Milestones for tracking issues and pull requests for minor and major version releases and keep the use of Labels to sort issues by type, priority, severity, functional area, functional group.

+1 to using the GitHub milestones feature.
Just a couple initial questions:

Can anyone add an issue to a milestone?
Will milestones be time based (e.g., minor version milestone planned every N weeks)?
How do version labels cause explosion if maintainers are diligent in labeling and communicating with /asking questions of other contributors & community? Maybe combinatorial explosion of labels is a separate issue?
Do we open a separate issue to discuss logically organizing source "components" into separate GitHub teams (e.g., analytics, search, distribution) to make notifications easier? And maybe how maintainers join those teams?

Milestones can be used to track progress of feature development. Additionally, can we add the issues to the roadmap project - https://github.com/orgs/opensearch-project/projects/1

+1 to using the GitHub milestones feature.
Just a couple initial questions:

Can anyone add an issue to a milestone?
It require the same access level as Labels, any one with Triage access or can apply milestones
Labels

Milestones

Will milestones be time based (e.g., minor version milestone planned every N weeks)?
Yes, you can define due to date and track the progress based on the progress bar.

How do version labels cause explosion if maintainers are diligent in labeling and communicating with /asking questions of other contributors & community? Maybe combinatorial explosion of labels is a separate issue?
We will end adding ~8 labels every year to track minor versions.

Do we open a separate issue to discuss logically organizing source "components" into separate GitHub teams (e.g., analytics, search, distribution) to make notifications easier? And maybe how maintainers join those teams?
Thats sounds a good idea, will track it separately.

Milestones can be used to track progress of feature development. Additionally, can we add the issues to the roadmap project - https://github.com/orgs/opensearch-project/projects/1
Yes, nothing changed in the issue level, you can add issues to any project. Our proposal to tag issues for specific release using Milestones instead of labels.

I've not personally worked with milestones. @anasalkouz Do you have a few examples of projects using milestones well? I'd love to see how this plays out in practice.
Maybe folks in the community have strong opinions? Have we asked them?

@stockholmux, here is an example of github project using milestones.
https://github.com/apache/rocketmq/milestones?direction=asc&sort=due_date


Adds appveyor config to ensure build works on windows

https://ci.appveyor.com/project/adriancole/zipkin-azure though I suppose there's a way to get this under the project openzipkin? cc @eirslett @openzipkin/devops-tooling

green now but test results aren't showing.. maybe there's an integration step to do. https://ci.appveyor.com/project/adriancole/zipkin-azure/build/1.0.5/tests
at any rate this is progress as it shows maven will work on windows

once we get this together, we should try on zipkin itself as we've had requests about windows build working (especially zipkin-ui part)

merging as otherwise appveyor stains other pull requests. improvements welcome post-merge!


PushNotificationIOS.requestPermissions won't resolve if no event listeners are attached

Motivation
Resolves #13012
RCTPushNotificationManager uses startObserving to register for RCTRegisterUserNotificationSettings. According to the docs, the startObserving method won't be called until somebody subscribes to NotificationManagerIOS.
This means there is a scenario when the developer can call requestPermissions without subscribing to notifications first, but since RCTPushNotificationManager relies on NSNotificationCenter subscribtion, the result will never be returned.
Test Plan
When requesting permissions  the promise will resolve:
PushNotificationIOS.requestPermissions().then(console.log); without the need for calling PushNotificationIOS.addEventListener() first.

cc @frantic, does this fix make sense?

It's clean and awesome, thanks @peterp! My only suggestion is to not manually unsubscribe from notification center, because it might introduce some racing bugs.

Nice to see this issue get some attention! Cheers gents.

@facebook-github-bot shipit


Default originalSize and originalUnit to 0 to avoid warning message

If the regular expression preg_match('/([0-9]+)([a-z]+)\s+(solid|dashed|dotted|none)\s+(.+)?/ui', $ruleValue, $matches); does not match, the variable $matches does not get initialized, which results in a warning message being generated.
Avoid warning message by defaulting $matches[?] to 0, when not initialized.

Are you sure this is the correct code?
Maybe something like this:
		$originalSize = $matches[1] ?? 0;
		$originalUnit = $matches[2] ?? 0;


Caststatuschanged catch error: "Cannot read property 'newStatus' of undefined"

I catch error, when set addEventListener.

Init, and call like this question.
shaka.pl.js
export default class ShPlayer {

	async changeUrl(newUrl) {
		const video = document.getElementById('video');
		const videoContainer = document.getElementById('videoContainer');
		const player = new shaka.Player(video);

		const ui = new shaka.ui.Overlay(player, videoContainer, video);
		const controls = ui.getControls();

		controls.addEventListener('caststatuschanged', this.onCastStatusChanged());

		try {
			await player.load(newUrl);
		} catch (error) {
			console.error(error);
		}
	}

	onCastStatusChanged(event) {
		const newCastStatus = event['newStatus'];
		// Handle cast status change
		console.log('The new cast status is: ' + newCastStatus);
	}

	onPlayerErrorEvent(errorEvent) {
		// Extract the shaka.util.Error object from the event.
		this.onPlayerError(event.detail);
	}

	onPlayerError(error) {
		// Handle player error
		 console.error('Error code', error.code, 'object', error);
	}

	onUIErrorEvent(errorEvent) {
		// Extract the shaka.util.Error object from the event.
		this.onPlayerError(event.detail);
	}

	initFailed() {
		// Handle the failure to load
		console.error('Unable to load the UI library!');
	}
}

Player.vue
<template>

            <video data-shaka-player ref="video" id="video"
                autoplay controls></video>

</template>

index.html
<!DOCTYPE html>

    <meta charset="utf-8"/>
    <meta http-equiv="X-UA-Compatible" content="IE=edge"/>
    <meta name="viewport" content="width=device-width,initial-scale=1.0"/>
    <title>
        Player
    </title>
    <link href="https://fonts.googleapis.com/css?family=Rubik:300,400,500" rel="stylesheet"/>

    <link rel="stylesheet" href="./controls.css"/>

If remove line controls.addEventListener('caststatuschanged', this.onCastStatusChanged());
video play. Controls appear, but there are no cast buttons among them.

And incorrect time. (when i change url, i set url to live stream)
I did everything according to the example
If i set my url to shaka demo page . Always ok, i can select tv to cast, cast was working.
Why doesn’t the cast button appear?
What could I do wrong?

First, like your previous issue, you don't use a lambda for the event listener, so it calls the onCastStatusChanged immediately.  You should do:
controls.addEventListener('caststatuschanged', (e) => this.onCastStatusChanged(e));

Secondly, those are the native controls, so there is no cast support.  Try removing the controls attribute from the <video> element.

@TheModMaker Now there is no way to check the chrome cast, check a little later. As for the controls, I removed the controls from the video tag, nothing happened. The video is on, there are no controls.

As I mentioned in the other issue you filed, you should not recreate the player & UI every time the playback URL changes.  You're doing this:
	async changeUrl(newUrl) {
		const video = document.getElementById('video');
		const videoContainer = document.getElementById('videoContainer');
		const player = new shaka.Player(video);

		const ui = new shaka.ui.Overlay(player, videoContainer, video);
		const controls = ui.getControls();

Instead, that should happen in the constructor or initialization of the component.  Then, when the URL changes, just call player.load() on the existing player instance.

@TheModMaker I fixed my mistakes. Video when changing the stream, loaded correctly.
shaka.pl.js
export default class ShPlayer {

	constructor() {
		this.video = document.getElementById('video');
		this.videoContainer = document.getElementById('videoContainer');
		this.player = new shaka.Player(this.video);
		this.ui = new shaka.ui.Overlay(this.player, this.videoContainer, this.video);
		this.controls = this.ui.getControls();
		this.controls.addEventListener('caststatuschanged', (e) => this.onCastStatusChanged(e));
	}
	async changeUrl(newUrl) {
		try {
			await this.player.load(newUrl);
		} catch (error) {
			console.error(error);
		}
	}
	onCastStatusChanged(event) {
		const newCastStatus = event['newStatus'];
		// Handle cast status change
		console.log('The new cast status is: ' + newCastStatus);
	}

...............

}

Player.vue
<template>

            <video data-shaka-player ref="video" id="video"
                autoplay controls></video>

</template>

Glad we could help!  And thanks for posting again with the corrected code for others who may find this issue in a search in the future.

@joeyparrish  Oops, I mean that the video plays in the browser. But do not cast.
@TheModMaker I tested, removing the controls attribute from the <video> element. Nothing happened, there are no controls at all.

The cast button only appears if:

You include the Cast SDK
You configure a Cast receiver ID
You have a Chromecast device powered up and on the same network as your browser

You can see details on the Cast SDK in the UI tutorial: https://shaka-player-demo.appspot.com/docs/api/tutorial-ui.html
That tutorial also shows how to set the receiver ID through the DOM-based setup, but not through the API.  To configure that via the API, use the configure() method on this.ui, documented here: https://shaka-player-demo.appspot.com/docs/api/shaka.ui.Overlay.html
The format for the configuration is documented here (and linked from the Overlay docs): https://shaka-player-demo.appspot.com/docs/api/shaka.extern.html#.UIConfiguration
The field you want is the castReceiverAppId field.
Finally, make sure your Cast device is turned on and on the same network.  The button only shows up if there are devices available to cast to.
Does this help?

@joeyparrish i check castReceiverAppId  the field is empty.

I insert the code like this:

            <video data-shaka-player id="video"  autoplay></video>

The DOM-based setup won't work for you if you don't have the elements statically in the HTML.  If you're using the API to construct Overlay, you need to call configure() on it to set the receiver app ID.  Please see the links I sent.

@ivansteff Is this the same application as you reference above or a different one? :)
Using the code you posted above:
`
export default class ShPlayer {
constructor() {
	this.video = document.getElementById('video');
	this.videoContainer = document.getElementById('videoContainer');
	this.player = new shaka.Player(this.video);
	this.ui = new shaka.ui.Overlay(this.player, this.videoContainer, this.video);
            this.controls = this.ui.getControls();
            // Try this!
            this.ui.configure('castReceiverAppId', '7B25EC44');
	this.controls.addEventListener('caststatuschanged', (e) => this.onCastStatusChanged(e));
}
async changeUrl(newUrl) {
	try {
		await this.player.load(newUrl);
	} catch (error) {
		console.error(error);
	}
}
onCastStatusChanged(event) {
	const newCastStatus = event['newStatus'];
	// Handle cast status change
	console.log('The new cast status is: ' + newCastStatus);
}

}
`

@ismena Finally, I get it! Many thanks to your team for such a detailed explanation!


Don't echo invalid command responses for single + commands outside of #bot-cmd

Don't echo invalid command responses for single + commands
When mods make typos with macros, hotbot still replies with the name of the macro that people were trying to invoke after deleting the original message. That can be a little annoying/embarrassing when it's a mod trying to semi-anonymously push someone in the right direction.
Priority:
1
Suggested solution:
An if statement could be added to the section where commands are handled that checks whether the command is a single + or not before sending a failed command response.

Fixed as of 4270062f056336bebf88cd3a0bca5de30a89c7c7


Remove map background placeholder

This (temporary) background image flashes before the map kicks in. It should only be used on the mobile homepage where we don't have a map. Let's also replace it with the a better quality image while doing this.

Closing as this has been fixed a while ago.


Add HTTPS support for the operator services

Description:
Currently in k8s operator only support for the HTTP services. Need to add the HTTPS support for the services in the operator.
Suggested Labels:
Priority/High
Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

Fixed in - https://github.com/wso2/k8s-ei-operator/pull/15


Consider exposing language on FormLine and FormPage

Now that we added language parameter to the content methods, it might make sense to expose these properties.
Status: waiting for input from service team.

Decision was to not expose it in the SDK as service is considering removing it from their swagger.

Decision was to not expose it in the SDK as service is considering removing it from their swagger.


feat: Container Instances

Description
Based on this road map we are planning to add Container Instances cloud service at v1.5.0

#1

Cloud Service

Container Insatnces

Release Version

1.5.0

when Container Instances status is Pending, instance_view is None
Traceback (most recent call last):
File "/usr/local/lib/python3.8/site-packages/plugin_azure_cloud_services-1.4.0-py3.8.egg/spaceone/inventory/manager/container_instances/container_manager.py", line 59, in collect_cloud_service
_start_time = container['instance_view']['current_state']['start_time']
TypeError: 'NoneType' object is not subscriptable


How to generate TextureAtlas into a file directly ?

Hello
Is there a way to generate a TextureAtlas into a file ?
Because right now i can only generate TextureAtlas onto memory at runtime, but it takes more than 1 minutes because i load a lot of sprites :/

If you preprocess your TextureAtlas asset, the file that it writes out to disk will contain the final texture and all of the cell position data, which should load substantially faster. Is that what you mean?

I tried this, but it generate just  a atlas.uvc file, but when i load it then it is empty
App.ContentManager.Preprocess("sprites/atlas.xml",false);
SpritesAtlas = App.ContentManager.Load("sprites/atlas.uvc");

Oh no nvm it works :D :D thanks a lot :)


Content from adult channels is shown in Discover tab and in mod/home.php

From @zzottel on December 10, 2014 9:34
I wanted to try to fix that myself, but I found strange things and am not sure in which direction to go.
My plan was to set the currently unused item_flag ITEM_NSFW on every top-level post by a channel marked as adult and then filter using these items in home.php, in network.php (discover tab) maybe dependent on the current safe search setting.
What confuses me is that I don't see how a channel is marked as adult and what flag actually makes the difference. I thought it was the xchan_flags XCHAN_FLAG_SELFCENSORED and XCHAN_FLAG_CENSORED, but when I set the adult flag in the settings on a test channel, these flags did not get set. Instead, a channel_pageflag PAGE_ADULT is set which is never sent on to other servers/directories or used in any other way if I understand that correctly.
Safe search, OTOH, relies on the mentioned xchan_flags to be set.
Which is the flag to look at? Why doesn't making a channel an adult channel set the xchan_flags?
Copied from original issue: friendica/red#736

From @zzottel on December 11, 2014 11:22
Should be fixed with Mike's commit from last night.


Don't test PyPI publish in CI if secret unavailable

Adds the PyPI key as an environment variable on the step and then tests for its existence. Based on this solution.
Closes #369

Thank you for sorting this out! You're the man.


Check for old lambda run-times

AWS is constantly EOL'ing Lambda run-times.  Check for ones that have been EOL'd, or will be soon.

See the CloudFormation data for this at https://github.com/aws-cloudformation/cfn-python-lint/blob/master/src/cfnlint/data/AdditionalSpecs/LmbdRuntimeLifecycle.json


Passes props to performAction instead of this

Fixes an issue where this.props wasn't being correctly passed to spec.performAction, which is what the types are indicating was happening

:+1: Thanks!


Multiple distances in Cosmas Query

Multiple distance query needs a synchronization among the distances so that they refer to the same elements/terms.

For the cosmas query: das /+w1:3,s1:1 Buch
"das Buch" in the same sentence is returned as a match.
The real problem might not be the synchronization of same operands.

After a round of testing it seems the problem is not reproducible in recent versions of KorAP or the KorAP data. So I guess I will close this issue for now.

Fixed. There was not a problem with multiple distance queries or spans but with skipping in ElementDistanceSpan.


bug fix: fetch remote without revision, if error occurs reset revision

Pull Request Test Coverage Report for Build 1012

1 of 3 (33.33%)  changed or added relevant lines in 2 files are covered.
14 unchanged lines in 7 files lost coverage.
Overall coverage increased (+0.03%) to 62.289%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core/registry/servicecenter/cache.go
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

core/registry/healthz.go
1
47.01%

core/registry/servicecenter/cache.go
1
35.74%

core/config/sr_config.go
2
0.0%

eventlistener/loadbalance_event_listener.go
2
66.67%

core/rest_invoker.go
2
93.94%

core/loadbalancer/struct.go
3
66.67%

core/tracing/tracer_manager.go
3
79.07%

Totals

Change from base Build 1011:
0.03%

Covered Lines:
4980

Relevant Lines:
7995

💛  - Coveralls


Native View embedded inside React Native control height with Styles from React Native side

So I am trying to add a Native Component to React Native. I want to be able to control the size of the view from RN.
This is what I have so far:
<PublisherView style={{width: 500, height: 200, marginLeft: 50}}/>

Publisher.m
@interface RCT_EXTERN_MODULE(RCTOpenTokPublisher, RCTViewManager)

@end

PublisherManager.swift
import Foundation
import UIKit

@objc(RCTOpenTokPublisher)
class RCTOpenTokPublisherManager: RCTViewManager {
  override func view() -> UIView! {
    return RCTOpenTokPublisherView()
  }
}

PublisherView.swift
import Foundation

@objc(RCTOpenTokPublisherView)
class RCTOpenTokPublisherView : UIView {
  override init(frame: CGRect) {
    super.init(frame: frame)

    let publisherView = OpenTokService.shared.publisher?.view

    print("PUBVIEW", publisherView)
    print("BBB", self.bounds)
    print("BOUNDS", self.bounds.size)
    print("FRAME", self.frame)

//    publisherView?.frame = frame
//     publisherView?.frame = CGRect(x: 0, y: 0, width: self.bounds.size.width, height: self.bounds.size.height)
    publisherView?.frame = CGRect(x: 0, y: 0, width: 100, height: 100)
    self.addSubview(publisherView!)

//    let label = UILabel(frame: CGRect(x: 0, y: 0, width: 100, height: 15))
//    label.text = "This is Swift"
//    self.addSubview(label)
  }

  required init?(coder aDecoder: NSCoder) {
    fatalError("init(coder:) has not been implemented")
  }
}

I tried with self.bounds.size.width and that doesn't seem to work.
Is this possible to do?
Thanks.

Solved. For those that may be curious:
import Foundation

@objc(RCTOpenTokPublisherView)
class RCTOpenTokPublisherView : UIView {
  var publisherView : UIView? = nil
  override init(frame: CGRect) {
    super.init(frame: frame)

    guard let publisherView = OpenTokService.shared.publisher?.view else { return }

    publisherView.frame = self.bounds
    publisherView.autoresizingMask = [.flexibleHeight, .flexibleWidth]
    addSubview(publisherView)
    self.publisherView = publisherView
  }

  override func layoutSubviews() {
     super.layoutSubviews()
    self.publisherView?.frame = self.bounds
  }

  required init?(coder aDecoder: NSCoder) {
    fatalError("init(coder:) has not been implemented")
  }
}


Indicate open on versions table

Description
Adds an Open column to the versions table, and does corresponding renaming from Open Data to Open on the facets page.
Background on the renaming

Initially, we were considering an open data version to be a workflow with a test parameter file with publicly accessible test data
Now, we are considering an open version to be a version with publicly accessible test data, OR, a version that does not require any file inputs. The key point being that in both cases you don't need any additional permissions to run the version.

The versions table is getting crowded, although I think it might just be acceptable. Suggestions welcome.
Before:

After:

Facets
Before

After

Review Instructions
Verify that the Open column appears on the versions table in both the public and My workflows views. Verify that the facet looks good on the search page.
We only update public accessible when updating/creating a new version (it's not retroactive), so you'll need to push new versions to test this.
Issue
dockstore/dockstore#5340
Security
If there are any concerns that require extra attention from the security team, highlight them here.
Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that your code compiles by running npm run build
[x] Ensure that the PR targets the correct branch. Check the milestone or fix version of the ticket.
[x] If this is the first time you're submitting a PR or even if you just need a refresher, consider reviewing our style guide
[x] Do not bypass Angular sanitization (bypassSecurityTrustHtml, etc.), or justify why you need to do so
[x] If displaying markdown, use the markdown-wrapper component, which does extra sanitization
[x] Do not use cookies, although this may change in the future
[x] Run npm audit and ensure you are not introducing new vulnerabilities
[x] Do due diligence on new 3rd party libraries, checking for CVEs
[x] Don't allow user-uploaded images to be served from the Dockstore domain
[x] If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.
[x] Check whether this PR disables tests. If it legitimately needs to disable a test, create a new ticket to re-enable it in a specific milestone.

The versions table is getting crowded, although I think it might just be acceptable. Suggestions welcome.

Think it may be ok, at least without adding additional columns.
It seems like you could go two rows for the header since the "verified platforms" header is much bigger than its contents. The language versions column could also be a bit less redundant but with more work. (all different ticket though)

The versions table is getting crowded, although I think it might just be acceptable. Suggestions welcome.

Think it may be ok, at least without adding additional columns. It seems like you could go two rows for the header since the "verified platforms" header is much bigger than its contents. The language versions column could also be a bit less redundant but with more work. (all different ticket though)

dockstore/dockstore#5412

[ ]  If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.

Remember to create and link this, we're going to need to explain where we are (i.e. parameter files with only http/https linnks with no auth) vs where we want to go (s3 and gs links that are requester pays)


Critical issue with bower.json file

There is a comma missing in bower.json line: 26
Can this be fixed asap. Thanks

Should be fixed in 2.1.3. Let us know if you're still experiencing issues


Path of bower_component is wrong

After wiredep, the template would be:
<!-- bower:css -->
<link rel="stylesheet" href="bower_components/bootstrap/dist/css/bootstrap.css" />
<!-- endbower -->

But the full URL of bootstrap.css would be http://homestead.app/auth/bower_components/bootstrap/dist/css/bootstrap.css when I browse http://homestead.app/auth/login, it is wrong.
I found the pattern of ignorePath should be /(\..\/)*public/, not /(\..\/)*public\//, I fixed it in gulpfile.
Do I miss something?

I'm going to fix it, since this already happened to me, instead of injecting this:
<!-- bower:css -->
<link rel="stylesheet" href="bower_components/bootstrap/dist/css/bootstrap.css" />
<!-- endbower -->

That is relative to actual path, it will inject as a Laravel asset
<!-- bower:css -->
<link rel="stylesheet" href="{{ asset('bower_components/bootstrap/dist/css/bootstrap.css') }}" />
<!-- endbower -->

Thank you.


Add window position sensor

I´m not sure if this relates to PR kingsleyadam/local-abbfreeathome-hass#41 or if this is a completely different type of device.
I have a wireless window position sensor with the states open/closed/tilted which is mounted directly to the window handle.
It´s this one: Busch Jaeger 6222/1 AP-66-WL
Can this be integrated?
device config when closed:
{
  "00000000-0000-0000-0000-000000000000": {
    "devices": {
      "ABB7F59BE886": {
        "deviceReboots": "6",
        "floor": "02",
        "room": "05",
        "interface": "RF",
        "deviceId": "2042",
        "displayName": "WC Fenster",
        "unresponsive": false,
        "unresponsiveCounter": 0,
        "defect": false,
        "channels": {
          "ch0001": {
            "floor": "02",
            "room": "05",
            "displayName": "WC Fenster",
            "functionID": "64",
            "inputs": {},
            "outputs": {
              "odp0000": {
                "pairingID": 53,
                "value": "0"
              },
              "odp0001": {
                "pairingID": 41,
                "value": "0"
              }
            },
            "parameters": {}
          }
        },
        "parameters": {
          "par0065": "0"
        }
      }
    }
  }
}

interesting part when open:
            "outputs": {
              "odp0000": {
                "pairingID": 53,
                "value": "1"
              },
              "odp0001": {
                "pairingID": 41,
                "value": "100"
              }
            },

and tilted:
            "outputs": {
              "odp0000": {
                "pairingID": 53,
                "value": "1"
              },
              "odp0001": {
                "pairingID": 41,
                "value": "33"
              }
            },

@nrenner73 If you slightly modify the angel of the window handle, does the value of opd0001 changes or will it always be 0, 33, 100?

Just checked it, the values are only 0, 33 or 100.

I've transferred this issue to the PyPi package repo. Here's a PR to implement the additional features of this device.
https://github.com/kingsleyadam/local-abbfreeathome/pull/67


TrafficManager Linting validation, previous CR feedback

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Only Linter errors left are all discussed and agreed; need a way to suppress or ignore them:
ERROR: Operations API must be implemented for '/providers/Microsoft.Network/operations'.
Path: file:///C:/work/github/azure/azure-rest-api-specs/arm-trafficmanager/2017-05-01/swagger/trafficmanager.json#/paths
WARNING: Booleans are not descriptive and make them hard to use. Instead use string enums with allowed set of values defined.
Path: file:///C:/work/github/azure/azure-rest-api-specs/arm-trafficmanager/2017-05-01/swagger/trafficmanager.json#/definitions/DeleteOperationResult/properties/boolean
WARNING: Booleans are not descriptive and make them hard to use. Instead use string enums with allowed set of values defined.
Path: file:///C:/work/github/azure/azure-rest-api-specs/arm-trafficmanager/2017-05-01/swagger/trafficmanager.json#/definitions/TrafficManagerNameAvailability/properties/nameAvailable

I'm reviewing this today.

@vladca model validation is working now, please see the following validation failures for your examples
https://travis-ci.org/Azure/azure-rest-api-specs/jobs/238346518

I've addressed everything, bar the stack-size error. Please review one final time and merge

@vladca the name "Profiles_ListBySubscription" is correct.

 https://github.com/Azure/azure-sdk-for-python/pull/1221

 https://github.com/Azure/azure-sdk-for-ruby/pull/795

 https://github.com/Azure/azure-sdk-for-node/pull/2196


Facebook Authentication through Firebase

I would like to add Facebook Authentication With that project
Context
Facebook Authentication Though Firebase
Detailed Description
I am also a gssoc'21 participant... I would like to add Facebook authentication to it... If I can forward with that, Please assign me to here...Thank you

@himanshusharma89 @Mohitmadhav @SamarpanCoder2002 @SapinderSingh @Tushar-OP is this issue active ? and is someone assigned for this issue ??  if not , it will be grateful to work on this issue ?

This feature is still not finalised, we'll discuss this whenever the project channel is publically available on GSSOC Discord Server

Ok I am waiting for that......

If this issue applicable, Please assign me to here...I raised the issue at first

Yeah, if we proceed with this issue I'll assign it to you @SamarpanCoder2002

Yeah, if we proceed with this issue I'll assign it to you @SamarpanCoder2002

Thanks

Yeah, if we proceed with this issue I'll assign it to you @SamarpanCoder2002

@himanshusharma89 .... Can now I proceed with that....I am also a GSSOC'21 participant...I I can forward with that...Please assign me to here.....thanks

We don't require Facebook Authentication, so closing this issue.


when server dies - client socket state remains open - even after pings

I have an ESP8266 websocket server that can be simply unplugged, killing the server without any notification to the client.
In this case I would expect the client ping function to detect an error on sending the ping, and for the socket status to be marked as closed.
However when the server has died, and after enough time for several pings has passed we have
socket.isOpen() : true
socket.getSocket() : is not null
socket.getSocket().isConnected() : true
I would expect all of these to be false after a failed ping??
Also I would expect a callback to onError to the websocketlistener on a failed ping, but that doesn't happen.
One thing I noticed was the swallowed WebSocketException on line 124 in WritingThread.java, could that be the issue?
   try
    {
        // Send remaining frames, if any.
        sendFrames(true);
    }
    catch (WebSocketException e)
    {

line 124            // An I/O error occurred.
}

If the server is killed ungracefully without a WebSocket-level notification, waiting is performed in the TCP layer, not in the WebSocket layer. Waiting in the TCP layer takes more time than that in the WebSocket layer.

Sorry to reuse this issue. Is there an option to set ping timeouts? Seems to me like the websocket still doesn't get a ping. why it doesn't get it is irrelevant.

I solved this issue by manually setting a timeout on the underlying socket with ping interval + 10s. That way, it will timeout on the read() call, which then closes the connection.
Example:
WebSocket ws = mySocket;
ws.getSocket().setSoTimeout(ws.getPingInterval() + 10000);

Without setting a timeout the read() call will wait indefinitely, or at least too long for our use-case, I don't think this library uses the socket timeouts by default. I used the onConnected event to set this timeout, but I'm not sure if there might be a better place to set it.
I had a different issue where I used sendClose(1000) but the server never responded with a close frame, which caused the write thread to stop but the read thread to get stuck indefinitely. Calling disconnect(1000) instead seems to interrupt the read-thread and solves the issue in a different way but I want to avoid interrupting the thread since it also acquires a few locks.


Bad link identified in page

Hi team,
This document has a bad link, specifically under the installation section. "You can download and compile the solution locally, or download a pre-compiled binary, then run either:".
The "download a pre-compiled binary" link points to "https://aka.ms/csdmtool" which is dead.
Please update. thanks.

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

ID: 9edf1df2-588e-01ff-6d1c-234b8e51d550
Version Independent ID: 27873c08-b6cc-a52f-f277-ff9818a5e4d2
Content: Tutorial: Database migration tool for Azure Cosmos DB
Content Source: articles/cosmos-db/import-data.md
Service: cosmos-db
Sub-service: cosmosdb-sql
GitHub Login: @deborahc
Microsoft Alias: dech

@leescottt  Thanks for your feedback. We are reviewing the feedback and will update the documents appropriate.

@leescottt The binary was removed for some reason. We are figuring out the options to republish a new pre-compiled binary to a new location. In the meantime, you can locally compile it. I removed the doc link until we have a new location.

#please-close

Thank you
From: Sneha Gunda @.>
Sent: Tuesday, July 13, 2021 8:16 PM
To: MicrosoftDocs/azure-docs @.>
Cc: Lee Taylor @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Bad link identified in page (#78288)
@leescottthttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fleescottt&data=04|01|Lee.Taylor%40microsoft.com|7e23f54a1c0a4d1cf17208d9465c8693|72f988bf86f141af91ab2d7cd011db47|1|0|637618185485375445|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=SgkMifKkU8Ec1C5PKqVj1FcgbOBsAiMS4SHTAw%2Bfgyo%3D&reserved=0 The binary was removed for some reason. We are figuring out the options to republish a new pre-compiled binary to a new location. In the meantime, you can locally compile it. I removed the doc link until we have a new location.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F78288%23issuecomment-879486482&data=04|01|Lee.Taylor%40microsoft.com|7e23f54a1c0a4d1cf17208d9465c8693|72f988bf86f141af91ab2d7cd011db47|1|0|637618185485375445|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=ljgiY86g0fquI2IPX0ZEjSgjA6NUZVNSK7iVPgPqXWY%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAU2VVUMGCKKFRBGQWPZOLBTTXTJLDANCNFSM5AJNFLJA&data=04|01|Lee.Taylor%40microsoft.com|7e23f54a1c0a4d1cf17208d9465c8693|72f988bf86f141af91ab2d7cd011db47|1|0|637618185485385453|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=RY79YFJ1vG5SzRVVUVttdBgiVN2bQA%2B32%2Fpm1H%2F%2FVig%3D&reserved=0.


Adds support for Stella_en_v5 embedding model - 1.5B variant

Stella_en_1.5B_v5 is a top ranking model (top 5) in Retrieval and Reranking tasks as of 8th October 2024.
Model Card
This PR adds support for the model along with some examples.
Tasks:

[x] Check license: Model is licensed MIT
[x] Check numerical accuracy - Verified against source pytorch implementation. At f32 candle implementation yields mathematically equivalent up to a 3-decimal places rounded .. beyond which we observe some variations. Code used to verify numeric equivalence can be found here.
[x] Authors example from the model card added and reproduced.

Note:
Stella_400M seems to follow a significantly different implementation. Have not implemented it in this PR.
Closes #2525

Thanks!


Move the common db tests into a single testing directory

Codecov Report

Merging #112 into master will increase coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #112      +/-   ##
==========================================
+ Coverage    87.4%   87.54%   +0.13%
==========================================
  Files          21       21
  Lines        1429     1429
==========================================
+ Hits         1249     1251       +2
+ Misses        115      114       -1
+ Partials       65       64       -1

Flag
Coverage Δ

#dot
87.54% <ø> (+0.13%)
:arrow_up:

#dot_data
65.78% <ø> (ø)
:arrow_up:

#dot_sync
84.86% <ø> (ø)
:arrow_up:

#dot_undo
100% <ø> (ø)
:arrow_up:

#dot_x_db_mysql
83.95% <ø> (ø)
:arrow_up:

#dot_x_db_pg
84.52% <ø> (+2.38%)
:arrow_up:

Impacted Files
Coverage Δ

x/db/pg/pg.go
84.52% <0%> (+2.38%)
: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 9b6bf2f...ced1992. Read the comment docs.


Issues with Bloom models

Hi!
I'm trying to quantize a 3B Bloom Model (https://huggingface.co/bigscience/bloom-3b). But it seems like it's missing the alibi tensor when performing the forward pass of the model.
Could you help me troubleshoot?

Can you see if the bloomz (instead of the older bloom) model, from the same author, is producing the same result?
https://huggingface.co/bigscience/bloomz-3b

Hello, I have the same problem, even with the BLOOMZ 3 billion parameters model. Has a solution been found, or is there anything I can do to solve the problem?

It seems that the issue was that the tensor alibi given for the bloom model are not provided and need to be add via the init_quant file. I will open a patch branch.


Support Mistral 7B

Optionally pass credentials to tokenize (we need it to talk to textsynth)
Add mistral (disabled by default)
Tweaks a few model description to add context size
Adjust gpt3.5-turbo context size

Deploy order is core (the credentials are optional) and then front


treatment of f_ni and Psep in PLASMOD vs PROCESS

In GitLab by @efable on Apr 12, 2018, 13:02
@hlux @kellis
Presently, fvsrbnni and fvalue for psepbqarmax are possible iteration variables. However, that dont work in PLASMOD.
For the moment, we use fvsrbnni and psepbqarmax as inputs, without iteration associated, and also without Psep constraint.
To put back fvsrbnni and fvalue for psepbqarmax as possible iteration variables (which make sense on the long term), we have to think at how PLASMOD can use this without creating bifurcations which VMCON would not like (if Psep constraint is set up again).
For the fraction of n.i. current, most logical values are 0 or 1, without iteration variable, however, the iteration variable would make sense for intermediate scenarios with longer pulses (but not fully n.i.). At the moment, the solution to this has to be thought.
For Psep, again, psepbqarmax with fvalue cannot be used in iterations, however that would make sense when Kallenbach is used, because the non-linearities associated with Psep, SOL, and plasma behavior would lead to some room for omptimization with psepbqarmax*fvalue as iteration variables. But also this one has to be thought through before using it as iteration variable.

This lengthy discussion relates to Plasmod and can be closed.


Disable/fix spinbox drag functionality

Godot version:

3.1
Issue description:

Spinbox is responsible for
https://github.com/godotengine/godot/issues/30242
https://github.com/godotengine/godot/issues/11893
GraphEdits zoom spin box also has this problem.
To reiterate:
When moving drag data from get_drag_data over a spinbox node, the mouse cursor vanishes and locks. Sometimes it can be recovered though clicking, but that heavily depends on the circumstances.

get_drag_data() returns an Object In the linked example: https://github.com/godotengine/godot/files/3348065/TreeMouse.zip you're returning a boolean. According to the documentation, you should return null if nothing should be passed.
Modifying the code so that it returns null, instead of 'false' seems to work.

Another related issue: #42596
Last time I tried dragging it was quite buggy (at least in ColorPickers). You need to click in a specific spot to trigger it, otherwise you are selecting text or nothing happens.

Closing in favor of https://github.com/godotengine/godot/issues/43608, which has more information about the issue at hand.


Using spark with mongodb async driver

post("/save-link", (req, res) -> {
            Document newUser = new Document("username", "foo");
            userCollection.insertOne(newUser, (result, throwable) -> res.body("usersaved"));
            return null;
 });

Doing sth like the above is not working. What is the approach to handle async functions ?

AFAIK, async is not supported. Perhaps you could implement "by hand" something to the effect of 202 Accepted? See #641 for links and discussion.
Or if you're OK with having the client wait, and deploying Spark in non-embedded mode, make an asynchronous servlet?


Invalid Swift Support

Invalid Swift Support - The bundle contains an invalid implementation of Swift. The app may have been
built or signed with non-compliant or pre-release tools. Visit developer.apple.com for more information.

We have received this message from Apple after submitting a build to TestFlight created with xctool 0.2.3.
Created a build using xcodebuild and it just worked.
Note: The same process worked around 1pm EDT, started failing at 3pm EDT.

We are using xcodebuild to build and sign the builds. Most likely your xcode-select -p points to beta version of Xcode. Could you confirm that?
If you want to force a particular Xcode to be used run xctool.sh with DEVELOPER_DIR=<path to Xcode to be used> ./xctool.sh ...

@ExtremeMan, there are no beta versions of Xcode on our build server.
$ xcode-select -p
/Applications/Xcode.app/Contents/Developer

Hm. I don't know then. Could it be issues in iTunes Connect?

Yeah, I've seen a few irregularities with Apple dev centers and iTunes connect lately. A part of me thinks it might be Apple that flipped a switch on something but figured it wouldn't hurt to log it here either.

We ran into this issue recently.  The culprit was a missing SwiftSupport folder in the IPA.  Generating the IPA via the Xcode GUI creates this folder for you, but neither xcodebuild and xcrun does (it's a bug).  Shenzen works around this by creating the folder as part of the build script.  xctool needs to do the same thing.
I'm guessing Apple started enforcing this recently during validation, since it just started happening for us recently, although it could be something we changed on our side.

See this: https://techblog.badoo.com/blog/2015/02/09/code-signing-and-distributing-swift/
Basically is what @andyvb said

@DarthMike, @andyvb could any of you send a PR, please?

Sorry I actually don't use xctool, but came across this ticket during my investigation. I chimed in to reduce other people spinning their wheels on this issue. :smile:

@ExtremeMan I'm in the same situation as @andyvb. I don't use xctool. I might get to doing it later but I don't think I'm the best to person to solve it. See how it was solved in shenzen as a guidance: https://github.com/nomad/shenzhen/pull/192

I adapted Lorenzo's code from https://github.com/nomad/shenzhen/pull/192 to a standalone ruby script here: https://gist.github.com/ldiqual/31594faf581431423d7b. It will copy the appropriate swift frameworks from the Xcode toolchain to an already packaged IPA.
Usage:
$ add-swift-frameworks-to-ipa.rb app.xcarchive app.ipa

Support for building projects with xctool is deprecated and will not be updated to support future versions of Xcode. We suggest moving to xcodebuild (with xcpretty) for simple needs, or xcbuild for more involved requirements. xctool will continue to support testing.


Filtering problem on data table

Hello everyone,
I am facing problem on filtering. Both column-filter and table-filter does not work if I use template inside datatable as slot on that specific coloumn.
<template #category="{item}">

        <template v-if="item.category">{{item.category.category}}</template>
        <template v-else>
            <CBadge color="danger"> Category Not found</CBadge>
        </template>

</template>

My question is:

Is there any wrong the way I am doing things?
What I am missing?

Filter can detect string values, not objects.
The topic has been discussed in #62

@woothu thank you for your reply. I have solved this by computing items before sending data to table.
But I am facing another problem. Based on data I have to use badge. So I was doing the following way:
<template #cartoon="{item}">

        type {{n+1}}:{{cartoon.size}} pcs

        <CBadge v-else color="danger">Cartoon Not Found</CBadge>

</template>

How do I get around this?
Note: I tried to include badge based on data while computing the items before sending to the table but the problem is data table does not render the badge rather shows plain html.

You were doing it correct, why would you change it?

If I do this way my filter does not works.

Filters can work only on strings. This means that if you pass the object as cell content it couldn't be filtered.
slot modifications are without limits and have no connection to filtering.

In the above example I am passing cartoonn.size which is not a object. I did not quite get it when you mention about passing object to cell content. Because cartoon.size is a string.
Can I render html passing as data to the item? Data is being escaped by the dataTable.

What is the item object structure? I mean that string has to be directly on the column key.

Thank you @woothu I get it now. I was thinking table will filter the string that is generated inside td but it actually filters the data that is actually passed to the coloumn key.

I did a little work around and it fixed my problem.
Before sending data to the table I simply assign data  to the coloumn key that the user would search. Then I did slot manipulation as per my requirement. As table filters data under the coloumn key, slot manipulation would not cause any problem. The following example would clear more:
Before sending data to the table I am doing this:
IndexAllProducts() {
    this.$store.dispatch('System/Request/MakeRequest', {type: 'get', url: '/api/products'}).then((res) => {
         if (!res.http_status && res.http_status !== 200 && !res.data) return this.items = []
              this.items = res.data.map((item) => {
                  return {...item, category: (_.has(item, 'category.category')) ? item.category.category : false, cartoon:this.formatCartoon(item.cartoons)}
         })
    }).catch((error) => {
         console.log(error)
    })
},

formatCartoon(cartoons) {
    // var cartoons = item.cartoons
    if (_.isEmpty(cartoons)) return `Cartoon Not found`

    var a = "";
    cartoons.forEach((item, index) => {
         a = a + `type ${index + 1}: ${item.size} pcs `
    })

    return a
},

In the format cartoon I am formatting the cartoon coloumn key as type 1: value 1 type 2: value 2. When users search they would search value 1, value 2 and table would filter the data under the coloumn key.

React  core ui CDatatable tableFilter is not working and showing No filtering results


When moving to , LOCATION_CHANGE is called twice.

I am migrating a project that used react-router-redux to connected-react-router.
Looking through the console logs, I noticed that LOCATION_CHANGE was being called twice every time a page was navigated.
Here is the reproduction code.
https://codesandbox.io/s/connected-react-router-0tbdu
Is this the correct behavior?

I see the same behavior (twice LOCATION_CHANGE) even when dispatch the push action.
Relative issue: https://github.com/supasate/connected-react-router/issues/318

I found the cause. StrictMode is the cause.
After further searching, I found the following article:
https://reactjs.org/docs/strict-mode.html#detecting-unexpected-side-effects
It's probably caused by two renderings in development mode with StrictMode.

@joyfuI  you are right, thanks your useful advice

So should we just not use this library in conjunction with Strict Mode? Surely Strict Mode has a lot of benefits. What exactly is the issue with history as per #318?


WIP : Adding information about Rule keyword

Added a short paragraph to the reference.md file in content/gherkin.
Fixes issue #262

@mlvandijk, let me know if this is what you were looking for, or if I am completely off.

@mlvandijk decide which pull request to take

Oh no, multiple pull request... I appreciate both your effort!
@TomerPacific - as @jenisys PR is a little more elaborate, do you mind if we go with that one? (Maybe you can collaborate on it, if needed?)

@mlvandijk , I'm fine with whichever pull request you approve.

Abandoned in favor of #271 - thanks @TomerPacific and @jenisys both for working on this :)


Implement Manager role

[finishes #93738888]
Adds a manager role to the app.
They can create users in their own office.
They can list users in their office.
They have a dashboard that shows them searches
performed in their office.

There were the following issues with your Pull Request

Commit: 0b2e9532b1682256feb102dfe78678ee3a0b8394
Your subject line is longer than 50 characters

Guidelines are available at https://github.com/ministryofjustice/fr-staffapp

This message was auto-generated by https://gitcop.com

There were the following issues with your Pull Request

Commit: 0b2e9532b1682256feb102dfe78678ee3a0b8394
Your subject line is longer than 50 characters

Guidelines are available at https://github.com/ministryofjustice/fr-staffapp

This message was auto-generated by https://gitcop.com


Small changelog updates prior to next release.

Pull Request Checklist
Is this in reference to an existing issue?
Not quite. Follow up to discussion in https://github.com/sensu/sensu-go-chef/pull/66#issuecomment-543934603
General

[X] Update Changelog following the conventions laid out here

Purpose
Update the CHANGELOG with any important information we’ve missed before the next
release.

@majormoses You can check my work here in case I forgot something.


Error when upgrading to flutter

Steps to Reproduce

Unpack folder for flutter on linux
Run flutter upgrade
I get this error:

...nloading Dart SDK from Flutter engine 54ad777fd29b031b87c7a68a6637fb48c0932862
curl: (3) URL using bad/illegal format or missing URL

/dart-sdk-linux-x64.zipDart SDK from: https://storage.googleapis.com/flutter_infra/flutter/54ad777fd29b031b87c7a68a6637fb48c0932862
If you're located in China, please see this page:
  https://flutter.dev/community/china

Seems like sth is wrong with this link .
Also tried switching to another channel flutter channel dev

i was hoping this is just a netowork infrastructure issue. But It still didnt resolve itself.

This issue is actually realted to the line endings.
linux has issues with concatinating bash variables when they are loaded from a CRLF file.
[name@localhost bin]$ EV=$(cat /home/name/flutter/bin/test)
[name@localhost bin]$ echo $EV A
hi A

test is a LF file
[name@localhost bin]$ EV=$(cat /name/name/flutter/bin/internal/engine.version)
[name@localhost bin]$ echo "$EV" "a "
a f8bffb9620575a1c5e3026b595e7bfb68a85f3

engine.version somehow is a CRLF file (in a git repo... ugh ;) )
The result of that is, that the url with the ENGIEN_VERSION cannot be properly created beacuse the string is currupted when using the CRLF data from engine.version...
That is what caused my problem in the initial post.
Solution
changed engine.version CRLF -> LF

this issue can get closed but this root cause really needs to be addressed. (maybe converting the whole repo to LF?

I am also facing the same Issue
flutter doctor -v
...nloading Dart SDK from Flutter engine b6b54fd60631a3828c2e2c9b079b5d1d2d8c8c37
curl: (3) URL using bad/illegal format or missing URL

Failed to retrieve the Dart SDK from: https://storage.googleapis.com/flutter_infra/flutter/b6b54fd606/dart-sdk-darwin-x64.zip8c8c37
If you're located in China, please see this page:
  https://flutter.dev/community/china

$ flutter channel dev
...nloading Dart SDK from Flutter engine af51afceb8886cc11e25047523c4e0c7e1f5d408
curl: (3) URL using bad/illegal format or missing URL

/dart-sdk-darwin-x64.zipart SDK from: https://storage.googleapis.com/flutter_infra/flutter/af51afceb8886cc11e25047523c4e0c7e1f5d408
If you're located in China, please see this page:
  https://flutter.dev/community/china

i have the same error.. are there any fix for this?

same error... I can't do anything which is related to "flutter ~" command..

@toger5 Based on your comment here, I am going ahead and closing this issue as solved.
@cristiansoto7  @LeeLoHoon See the solution posted by the author and check if it resolves your issue.
If not, since this is an old issue, I'd suggest anyone facing this issue, file a new one by providing all required details.
If you disagree, write in comment and I'll reopen it.
Thanks.


Include domain-name support for the service regard-application

Note : This issue can be taken after completing the issue https://github.com/openBackhaul/ApplicationPattern/issues/350
In the initial CONFIGfile , there will not be any attributes related to domain-name.
In the server-side logic , the code needs to be enhanced so that if a domain-name attribute can be added via a service , then the server-side code should include the domain-name attribute along with the values and delete the ip-address attribute at the same time and vice versa.

This issue is fixed with changes in https://github.com/openBackhaul/OperationKeyManagement/pull/150


Strange warning on running (flycheck)

Warning: Unknown defun property `interactive-only' in flycheck-disable-checker
Warning: Unknown defun property `interactive-only' in flycheck-set-checker-executable

What this?

These properties work only with version 25


Can't connect to updates.jenkins.io from inside Docker image and during build

Issues and Contributing
All requests to updates.jenkins.io timeout. I started noticing this when the build of a Docker image that used the install-plugins.sh script failed. Then I ran some tests to see what was happening (see below).
It also fails with http://mirrors.jenkins-ci.org, but not with http://ftp-nyc.osuosl.org.
Docker commands that you execute
docker run -it appropriate/curl -o tmp.html https://updates.jenkins.io
Actual result
Timeout.
Expected outcome
jenkins $ docker run -it appropriate/curl -o tmp.html https://updates.jenkins.io
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  5961    0  5961    0     0  11597      0 --:--:-- --:--:-- --:--:-- 11574

Have you tried a non-dockerized Jenkins and get the expected outcome?
Yes, curl -o tmp.html https://updates.jenkins.io completes succesfully in a Bash shell.
Output of docker version
Client:
 Version:           18.09.3
 API version:       1.39
 Go version:        go1.10.8
 Git commit:        774a1f4
 Built:             Thu Feb 28 06:53:11 2019
 OS/Arch:           linux/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          18.09.3
  API version:      1.39 (minimum version 1.12)
  Go version:       go1.10.8
  Git commit:       774a1f4
  Built:            Thu Feb 28 05:59:55 2019
  OS/Arch:          linux/amd64
  Experimental:     false

Other relevant information

On the same machine, I can reach updates.jenkins.io in a bash shell, outside of a Docker container.
It only times out for this specific destination host, not for other destination hosts.

The machine on which the commands fail is a GCE VM created from an image, that was build using the following Packer template:
{
  "builders": [
    {
      "type": "googlecompute",
      "project_id": "wizardsofindustry",
      "source_image_family": "ubuntu-1804-lts",
      "source_image_project_id": "ubuntu-os-cloud",
      "zone": "europe-west4-a",
      "disk_size": "32",
      "image_name": "jenkins-agent-docker",
      "image_family": "docker-agent",
      "ssh_username": "ubuntu",
      "network": "<redacted>",
      "subnetwork" : "<redacted>",
      "use_internal_ip": true,
      "tags": ["nat"],
      "omit_external_ip": true
    }
  ],
  "provisioners": [
    {
      "type": "shell",
      "inline": [
        "sudo apt-get update",
        "sudo apt-get install -y default-jdk",
        "sudo apt-get install -y apt-transport-https",
        "sudo apt-get install -y ca-certificates",
        "sudo apt-get install -y curl",
        "sudo apt-get install -y software-properties-common",
        "curl -fsSL https://download.docker.com/linux/ubuntu/gpg | sudo apt-key add -",
        "sudo add-apt-repository 'deb [arch=amd64] https://download.docker.com/linux/ubuntu bionic stable'",
        "sudo apt-get update",
        "sudo apt-get install -y docker-ce",
        "sudo mkdir -p /home/jenkins",
        "sudo groupadd -g 10000 jenkins",
        "sudo useradd -d /home/jenkins -u 10000 -g 10000 -m -s /bin/bash jenkins",
        "sudo chown jenkins:jenkins /home/jenkins",
        "sudo usermod -aG docker jenkins"
      ]
    }
  ]
}

Hey,
Thanks for reporting this, it seems that you are having issues with Jenkins Infrastructure services and not the docker image itself, feel free to open an issue on issues.jenkins.io as explained here

Hey, Thanks for reporting this, it seems that you are having issues with Jenkins Infrastructure services and not the docker image itself, feel free to open an issue on issues.jenkins.io as explained here

olblak, why do you think this is an issue w/Jenkins infrastructure?  Reporter says it works outside of docker on the same VM.   timja, why closed ?

I am having this same issue on 2.350. Please reopen the ticket as its not working

jin-ahn, please verify that your MTU for the machine/VM is 1500 or whatever is appropriate for your setup.  If you have VMWare VCenter/VSphere, etc running under your VMs you can certainly have packet loss if MTUs are mismatched.  Updates in VMWare 6.5 to 6.7 started to be more stringent in networking rules around packet size allowance, etc.

Just figured I would leave my "ah!" comment on this because I was also running into this same issue. The MTU lead was a good one as I had forgotten that the host on which the docker container was running had all of its WAN traffic tunneled through a Wireguard VPN interface which ultimately meant that the container's WAN traffic did too.
And sure enough, the MTU on the Wireguard client and server was different from the one in the network the container was using. Aligning the values immediately fixed the issue and the plugins started downloading.
In case this is eye opening to anybody ending up on this issue...


Submit matike

Submit is maybe broken.

Coverage decreased (-0.5%) to 71.145% when pulling ba10ef7bce52920d8da81cfe966a456e61596890 on submit-matike into 99e00b0199329682bf5ab4d2aa3626515156a657 on master.

Changes Unknown when pulling b03a4e0f02f1740a6e370685b20d9c9839014406 on submit-matike into ** on master**.

Changes Unknown when pulling 732c51a21bf77f522b67ff4690ee6de807066005 on submit-matike into ** on master**.

LGTM.