Stored Scripts are not serialized when requesting cluster state Metadata

NEST/Elasticsearch.Net version:
6.5.0
Elasticsearch version:
6.2.4
Describe the feature:
Currently requesting the cluster state metadata (client.ClusterState(s => s.Metric(ClusterStateMetric.Metadata))) does not serialize the stored_scripts portion of the response.
If I were to take a swing at implementing this, the sub field on the state response would provide something like a Dictionary<Id, IStoredScript>.  The (simplified) response json from the above request:
{
    "metadata": {
        "stored_scripts": {
            "my-script-id": {
                "lang":"painless",
                "source":"return 1;",
                "options":{}
            },
            "my-other-script-id": {
                "lang":"painless",
                "source":"return 0;",
                "options":{}
            },
        }
    }
}

I also notice the response includes an options that isn't currently being serialized. I'm not sure what this is useful for since we don't utilize it, but it is missing (maybe intentionally?)

Thanks for raising @cscorley - I have opened a PR to address this.

Thank you for the quick help!

Hi @cscorley - after speaking with some colleagues it is recommended that you don't rely on the cluster state metadata response - only that it is susceptible to breaking within minor versions. It is not versioned in the same manner as other endpoints.
Is there anyway you can use the script endpoints instead? https://www.elastic.co/guide/en/elasticsearch/reference/6.2/modules-scripting-using.html#modules-scripting-stored-scripts

This is now available in the 6.8.0 client recently pushed to nuget https://www.nuget.org/packages/NEST/6.8.0

Ah, yes, you are correct, there is no list scripts endpoint. This does seem like an oversight, let me open a ticket for you there.


Add documentation for the analytics library and packages

Code pull request: https://github.com/frontity/frontity/pull/472
Feature Discussions:

The analytics library
Comscore analytics package
Google Analytics package
Google Tag Manager package

This is for creating the documentation of all the analytics packages. The main package (@frontity/analytics) is not intented to be use as another package but extended by each specific analytics packages. I can explain that if you need so.

The analytics library details are only important for developers → info will go directly in the README.md
Each analytics package has a common interface that has to be documented in the docs

import Root from "./components";
import Analytics from "../types";

const analytics: Analytics = {
  roots: {
    analytics: Root,
  },
  state: {
    analytics: {
      pageviews: {},
      events: {},
    },
  },
  actions: {
    analytics: {
      // Get the functions for every analytics package
      // and run `pageview` for each one.
      pageview: ({ state, actions }) => (pageview) => {
        Object.entries(state.analytics.pageviews).forEach(
          ([namespace, shouldSend]) => {
            if (shouldSend) actions[namespace].pageview(pageview);
          }
        );
      },

      // Get the functions for every analytics package
      // and run `event` for each one.
      event: ({ state, actions }) => (event) => {
        Object.entries(state.analytics.events).forEach(
          ([namespace, shouldSend]) => {
            if (shouldSend) actions[namespace].event(event);
          }
        );
      },
    },
  },
};

export default analytics;

state.analytics.pageviews
state.analytics.events
actions.analytics.pageview
actions.analytics.event

state.analytics.pageviews
Map of namespaces with boolean values.
This object is used by actions.analytics.pageview to know which
analytics packages should send the pageview to their respective
services.
If you want to disable sending pageviews for a specific analytics
package, the respective namespace of that package should be set here to
false.
pageviews: {
  googleAnalytics: false,
  comscoreAnalytics: true
}

Closed as per https://github.com/frontity/frontity/pull/573 & https://github.com/frontity/docs/pull/226


fix(runner): Make exit code configurable when tests are failing

Fixes #1300

@johnjbarton Updated patch after you comments


Langchain dependency errors when installing with other libraries

The langchain version in pyproject.toml is fixed via a caret requirement (langchain = "^0.0.164") .
As langchain is updated almost daily, this makes fastapi-async-langchain basically unusable with other langchain-dependent modules, e.g.:
 and fastapi-async-langchain (0.5.2) depends on langchain (>=0.0.164,<0.0.165), jpllm (0.0.4)  is incompatible with fastapi-async-langchain (0.5.2).
`` ``

It would be great to change the langchain requirement to e.g. ">0.0.164"  or "~0.0.164"  to make the use more flexible.

@jpdus pushed a fix in 0.5.3 version. should be available soon!


Snippet won't load

On startup I get this error message.  It comes from having an asterisk in a SQL snippet.  Here is the error message from main scree alert it is followed by the snippet.
Tks
Jerry A
Failed to load snippets from '/home/jerry/.atom/snippets.cson'
/home/jerry/.atom/snippets.cson: unexpected *
SELECT All990s.* INTO BC990s
FROM All990s
INNER JOIN BCOrgs
ON All990s.EIN=BCOrgs.EIN;

Can you share your entire snippets.cson?

Maxs,
I've just started using atom so I only had the one snippet in there as
it wasn't making atom happy.  Since I wrote the report I've deleted the
snippet I sent you.  The remainder of the snippet file was the canned stuff
that come with the application.
Cheers
Jerry A
On Mon, Aug 24, 2015 at 12:03 PM, Max Brunsfeld<EMAIL_ADDRESS>wrote:

Can you share your entire snippets.cson?
—
Reply to this email directly or view it on GitHub
https://github.com/atom/atom/issues/8491#issuecomment-134300382.

@jlarget Ok, thanks. If you're no longer having a problem, I'm going to close this out. If you'd still like help making this snippet work, you can add it back, and post the contents of the file, and I'll help you troubleshoot.


stack-use-after-scope in ArenaAllocator.cpp (only Debug version)

The following code segment in the ArenaAllocator.cpp file has a stack-use-after-scope bug.
        void **policy = &this->freeList;
#if DBG
        if (needsDelayFreeList)
        {
            void *delayFreeList = reinterpret_cast<FreeObject **>(this->freeList) + (MaxSmallObjectSize >> ObjectAlignmentBitShift);
            policy = &delayFreeList;
        }
#endif
        *policy = TFreeListPolicy::Free(*policy, buffer, size);

@hikame Thanks for finding and filing this issue. It was fixed by https://github.com/Microsoft/ChakraCore/pull/4216.


Provide a way to step into with Carthage-integrated binaries

Currently it not possible to step into code located in our libraries. We should investigate if a simple solution exists. I think we could provide a configurable tool (maybe a script and / or an .lldbinit setup) which:

Checks out code for all dependencies in a directory.
Pulls all recent changes.
Switches to the commit has in the Cartfile.resolved directory.
Setup LLDB to point at those sources.

See this post for a possible LLDB strategy.
This setup should then be described in the documentation of all our libraries.

Version 6.0.0 introduces SPM support and drops Carthage entirely. This issue is therefore not relevant anymore.


[New Sheet] Fur & Forge

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: Fur & Forge  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: Hellashibe (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: Fur & Forge (i.e. Dungeons & Dragons, FATE)

[x] I have followed the Character Sheets Standards when building this sheet.

[x] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Character Sheet Info Roll20 Internal Use only.

Hey @colearkenach,
We need a publicly-available link to a marketplace listing (or just a public google doc of the rules if it's free) before this can be approved. I was able to find Hellashibe's site, but the rules don't seem to be up yet. To be clear; the game doesn't need to be finished, the rules just need to be published somewhere.
Nora

@NorWhal That's my bad! Dev sent me their copy of the game (Work in Progress, but the rules are available)
Here are the rules (and I'll edit the description above to include it): https://www.canva.com/design/DAF8ClaVE-s/nKtgZ2-cdmDmoaZjH0M_jw/edit?utm_content=DAF8ClaVE-s&utm_campaign=designshare&utm_medium=link2&utm_source=sharebutton

hi @colearkenach! Thank you for including the rules. As there is a git history of you removing another sheet, can you close this PR and open a new one with just the sheet for Fur and Forge? Once you do that, I will go ahead and merge.
Thank you!


gradle 7.3.3 can not generate R2

My project library can not found R2 When gradle version upgrade to 7.3.3. I guess that's namespace caused it,how can I fixed it ?

I have  the same  problem

butterknife and realm   all need to fix

is there no one to solve this problem ?  or  we need delete all butterknife ??

Any solution found yet for this issue?

这是来自QQ邮箱的自动回复邮件。

您好，现在无法回复您的邮件。我将尽快给您回复。

same ,please help us.

Delete all butterknife from your project. Butterknife is not maintained anymore. There's no way to add support for new Gradle version.

For me the problem is moving the package attribute from the manifest to gradle as namespace. Without it the generated R2 classes to not have package x.y.z at the top. Moving it back to the manifest solves the problem. Just a workaround that works for now.
Keep this in the AndroidManifest.xml.
<manifest
	xmlns:android="http://schemas.android.com/apk/res/android"
	package="x.y.z">

Instead of this in build.gradle
namespace 'x.y.z'


Supports PHP 7.2 and added missing doc comments

Hello @lkmadushan! Thanks a lot for your contribution 😄
Could you please rebase your branch before we merge it?

@curquiza done


Reference lists - Can we use setvalue for this

setValue action sets a specific value because an expression would always evaluate to a Value.
What is a Reference list and what are you trying to do?

By reference list, I mean static lists  - which may be used to populate  a drop down

createSheet action has been added as a new feature for this purpose. This allows you to create a data-sheet at run time using constants that are known at run time as well as fields that are known at run time.
This feature has been tested with the help of tutorial.createSheet.xml. I tested it for success as well as failure(meaning getting desired error messages) by interactively changing this file. Applicaiton.java has a main method that allows you to run any service in the IDE.
On an aside, it is better to use cached responses for hard-coded response like values for list service instead of hard-coding values in a service.


⚠ Jenkins Plugins Artifactory has degraded performance

In 971c11b, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1220 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 588e416 after 12 minutes.


Get Current Scene Key

I am currently running a tab bar.
I want to be able to control the android back button so if focus is on certain scenes it just returns to the home scene.  Then only exit the app if on the home scene.
Basically so it operates as a standard navigation stack.
Is there an API call to get the key so i can use it within backAndroidHandler?

My question is the same here.
Anyone has an answer ?

I really need this so may look at it this weekend and submit a pr if it can work. @toaa-media i havent looked at the src code yet but have you got any thoughts on how this should work?

Thank you @b8ne
I have found out a way to get current scene key. Something like this:

Create your own reducer to handle route navigation.
this.reducerCreate = params=> {  const defaultReducer = Reducer(params);  return (state, action)=> {  	console.log("ACTION:", action);  	if (action.type == ActionConst.FOCUS) {  		let scene = action.scene;  		this.currentScene = scene.sceneKey;                     ===>>> this is your current scene.  	}  	return defaultReducer(state, action);  } };

Add your reducer above to Router.
render() {      return (          <Router createReducer={this.reducerCreate}> ..................................

That's all. And it works fine now. :)

Awesome @toaa-media, thanks for sharing.  I havent had a chance to test anything yet but had a quick look at the source and may have another way given that all the data we need is already in the state.  Ill test both over the weekend and let you know how i go. Either way i feel we have a result here so ill close this issue.

Hey @toaa-media thats not working for me.  I already had a reducer setup by following the drawer setup docs. So just checked what that was outputting, and I am getting the scene key but then also the root key every time, so obviously the root will override everytime as its 2nd.  Have a feeling it may be to my scene hierarchy so ive included below.  Can you see anything that may fix it?
<Provider store={store}>
        <RouterWithRedux
          animationStyle={animationStyle}
          onExitApp={() => {
            console.log(store.getState());
            /*
            if (store.getState().currentScene.sceneKey == 'home') {
              return true;
            }
            Actions.home();
            return true;
          }}
          */
          return true;
        }}

        >
          <Scene key="root">
            <Scene key="drawer" component={NavigationDrawer} open={false}>
              <Scene key="main" tabs style={tabBarStyles.tabBarStyle} tabBarIconContainerStyle={tabBarStyles.tabContainer}
              pressOpacity={1}>

                <Scene key="home" initial title="HOME" navigationBarStyle={tabBarStyles.topBar} drawerImage={null}>
                  <Scene
                    key="home_1"
                    component={Home}
                    title="Home"
                    titleStyle={tabBarStyles.titles}
                  />
                </Scene>

                <Scene key="teams" title="TEAMS" icon={TabIcon} navigationBarStyle={tabBarStyles.topBar} style={tabBarStyles.iconContainer}>
                  <Scene
                    key="teams_1"
                    component={Teams}
                    title="Teams"
                    titleStyle={tabBarStyles.titles}
                    renderBackButton={() => <BackButton back={() => Actions.home()}/>}
                  />
                </Scene>

                <Scene key="news" title="NEWS" icon={TabIcon} navigationBarStyle={tabBarStyles.topBar} style={tabBarStyles.iconContainer}>
                  <Scene
                    key="news_1"
                    component={NewsList}
                    title="News"
                    titleStyle={tabBarStyles.titles}
                    renderBackButton={() => <BackButton back={() => Actions.home()}/>}
                  />
                  <Scene
                    key="news_2"
                    component={NewsArticle}
                    title="News"
                    titleStyle={tabBarStyles.titles}
                    renderBackButton={() => <BackButton back={() => Actions.pop()}/>}
                  />
                </Scene>

                <Scene key="sponsors" title="SPONSORS" icon={TabIcon} navigationBarStyle={tabBarStyles.topBar} style={tabBarStyles.iconContainer}>
                  <Scene
                    key="sponsors_1"
                    component={SponsorList}
                    title="Sponsors"
                    titleStyle={tabBarStyles.titles}
                    renderBackButton={() => <BackButton back={() => Actions.home()}/>}
                  />
                  <Scene
                    key="sponsors_2"
                    component={Sponsor}
                    title="Sponsor"
                    titleStyle={tabBarStyles.titles}
                    renderBackButton={() => <BackButton back={() => Actions.pop()}/>}
                  />
                </Scene>

                <Scene key="more" title="MORE" icon={TabIcon}
                  navigationBarStyle={tabBarStyles.topBar}
                  style={tabBarStyles.iconContainer} tabBarIconContainerStyle={tabBarStyles.moreContainer}
                  component={More}
                  onPress={() => {this.openDrawer()}}
                />

                <Scene key="club" title="CLUB" navigationBarStyle={tabBarStyles.topBar}>
                  <Scene
                    key="club_1"
                    component={ClubData}
                    title="Club"
                    titleStyle={tabBarStyles.titles}
                    renderBackButton={() => <BackButton back={() => Actions.home()}/>}
                  />
                </Scene>

                <Scene key="lifemember" title="LIFE MEMBERS" navigationBarStyle={tabBarStyles.topBar}>
                  <Scene
                    key="lifemember_1"
                    component={LifeMember}
                    title="Life Members"
                    titleStyle={tabBarStyles.titles}
                    renderBackButton={() => <BackButton back={() => Actions.home()}/>}
                  />
                </Scene>

                <Scene key="auth" title="LOGIN" navigationBarStyle={tabBarStyles.topBar}>
                  <Scene
                    key="auth_1"
                    component={Auth}
                    title="Login"
                    titleStyle={tabBarStyles.titles}
                    renderBackButton={() => <BackButton back={() => Actions.home()}/>}
                  />
                  <Scene
                    key="login_1"
                    component={Login}
                    title="Login"
                    titleStyle={tabBarStyles.titles}
                    renderBackButton={() => <BackButton back={() => Actions.pop()}/>}
                  />
                  <Scene
                    key="register_1"
                    component={Register}
                    title="Register"
                    titleStyle={tabBarStyles.titles}
                    renderBackButton={() => <BackButton back={() => Actions.pop()}/>}
                  />
                </Scene>

              </Scene>
            </Scene>
          </Scene>
        </RouterWithRedux>
      </Provider>

So the home scene is initial, but doesnt have a tab, the next 3 scenes are tabbed. the final tab opens the side drawer, which contains the rest of the scenes - if that makes sense.

@toaa-media what about Actions.pop? No key is provided with back action

is there now simple way to determine current scene name?

Yes. Actions.currentScene

@aksonov  Something like Actions.prevScene would be nice too.

@aksonov but it still gives the key 'root', isn't there a way to get the key of the innermost scene currently being rendered?

@aksonov work perfectly for me. Thank you

@vitorlsouza Hi, can you tell me more about how you're using Actions.currentScene?

@lukaszmalec6
Hi,
I am using to issue an alert when the user returns to the login scene.

Using in condition for example.
if(Actions.currentScene === 'formPrincipal') {
here Alert with 2 options, when user click
if user click YES return Actions.pop()
if user click NO return false
}


What is the timeout value in gpioSetISRFunc()?

For the API:  int gpioSetISRFunc(unsigned user_gpio, unsigned edge, int timeout, gpioISRFunc_t f), what is the function ofthe timeout?
//pin ButtonCenter is wired High with a resister network
//pin ButtonCenter when pressed is connected to ground

void displayInterrupt(int gpio, int level, uint32_t tick){
  printf("Interrupt %d\n", c);
  c++;
}

void main(){
...
  gpioInitialise();
  gpioSetMode(ButtonCenter, PI_INPUT);
  gpioSetPullUpDown(ButtonCenter, PI_PUD_UP);
  gpioSetISRFunc(ButtonCenter, FALLING_EDGE, 1, displayInterrupt);

  while(1){
  }
}

When I do the above, displayInterrupt() is called without the button being pressed.
When I changed to  gpioSetISRFunc(ButtonCenter, FALLING_EDGE, 0, displayInterrupt), then displayInterrupt() is called when the button is pressed.
Reading this "The level will be PI_TIMEOUT if the optional interrupt timeout expires." seems to tell me that the timeout value is used as a wait time before reading the pin and recording the level for the call back function.
If this is the case, setting it to 1 should not change the behavior when it is 0.

The timeout is used to get a response if the interrupt doesn't happen.  Say it is set to 1000 (milliseconds).  If the button isn't pressed for a second you would be called with a level of PI_TIMEOUT.  If the button was pressed or released within a second you would be called with a level of 0 or 1.  If there are regular (less than a second) button presses you would never be called with the timeout value.  The timeout is reset each time a level change or a timeout happens.


chore: import repository https://github.com/motasem2021/node-poc.git

this commit will trigger a pipeline to generate the CI/CD configuration which will create a second commit on this Pull Request before it auto merges

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/approve

approve

assign @motasem2021


NTLM Dummy record

Is your feature request related to a problem? Please describe.
https://github.com/dotnet/Kerberos.NET/issues/29
At this issue we having problem: if computer doesn't have any SPN assigned, client send NTLM broken token. For debug clarity, I guess, it would be ok to add information details about SPN.
Describe the solution you'd like
Add additional debug info if token starts with TlRMTVNTUAABAAAAl4II4gAAAAAAAAAAAAAAAAAAAAA

It should detect NTLM already and throw an error. Is it not doing that?

From: Zubastic<EMAIL_ADDRESS>Sent: Thursday, October 22, 2020 7:44:53 AM
To: dotnet/Kerberos.NET<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [dotnet/Kerberos.NET] NTLM Dummy record (#209)
Is your feature request related to a problem? Please describe.
#29https://github.com/dotnet/Kerberos.NET/issues/29
At this issue we having problem: if computer doesn't have any SPN assigned, client send NTLM broken token. For debug clarity, I guess, it would be ok to add information details about SPN.
Describe the solution you'd like
Add additional debug info if token starts with TlRMTVNTUAABAAAAl4II4gAAAAAAAAAAAAAAAAAAAAA
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/Kerberos.NET/issues/209, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAJHTYIRS5THHRFRFFUKVU3SMBAOLANCNFSM4S3JUN2A.

It should detect NTLM already and throw an error. Is it not doing that?
It throw generic error that NTLM not supported. But I think that it would be good if error with info about missing SPN would be thrown.

The issue is this library has no idea if it's because of a missing SPN or some other reason. Annoyingly it's also impossible to guess what the SPN should be because its dependent on what the service in question is and the library doesn't know what that is.
Couple solutions:

in this code change NtlmContextToken to throw (bold added): throw new NotSupportedException("NTLM is not supported. Is your service account missing an SPN registration?");
In your code catch NotSupportedException and check if it contains NTLM and indicate to caller its maybe because there's no SPN registered.

Hmmm, I guest, I could check if token starts with TlRMTVNTUAABAAAAl4II4gAAAAAAAAAAAAAAAAAAAAA and throw another error.

There's a helper method NtlmMessage.CanReadNtlmMessage(data) that might be of use to you. It'll parse it properly instead of doing a string comparison (there's no guarantee that string value wouldn't change in the future).

It'll parse it properly instead of doing a string comparison (there's no guarantee that string value wouldn't change in the future).

I want to check if it dummy string with empty login + domain. I create code to parse:
var b = Convert.FromBase64String("TlRMTVNTUAABAAAAl4II4gAAAAAAAAAAAAAAAAAAAAAGA4AlAAAADw==");
using (var ms = new MemoryStream(b))
{
    using (var br = new BinaryReader(ms))
    {
        string GetValue(BinaryReader reader)
        {
            var length = reader.ReadInt16();
            var alloc = reader.ReadInt16();
            if (alloc != length)
            {
                throw new ArgumentOutOfRangeException(nameof(alloc));
            }
            var offset = reader.ReadInt32();
            var currentOffset = reader.BaseStream.Position;
            reader.BaseStream.Position = offset;
            var data = reader.ReadBytes(length);
            var name = Encoding.UTF8.GetString(data);
            reader.BaseStream.Position = currentOffset;
            return name;
        }
        var header = Encoding.UTF8.GetString(br.ReadBytes(8));
        if (header.StartsWith("NTLMSSP"))
        {
            var ver = br.ReadInt32();
            switch (ver)
            {
                case 1:
                    var flags = br.ReadInt32();
                    var domain = GetValue(br);
                    var workstation = GetValue(br);
                    var major = br.ReadByte();
                    var minor = br.ReadByte();
                    var build = br.ReadInt16();
                    var unk = br.ReadInt16(); //always zero
                    var osVer = $"OS Ver: {major}.{minor} Build: {build}";
                    break;
                case 2:
                    break;
                case 3:
                    break;
                default:
                    throw new ArgumentOutOfRangeException(nameof(ver));
            }
        }
    }
}

That works too.


Unspecified error when displaying certain figures

Error on the PCA, UMAP, and TSNE tabs: "An error has occurred. Check your logs or contact the app author for clarification".
Dataset replicating this error can be accessed (internal link) here.
Error generated when running locally:

Warning: Error in mutate: Problem while computing Group = as.character(meta[unique(.data$SampleID)]).
✖ Group must be size 2113 or 1, not 0.
ℹ The error occurred in group 1: SampleID = 0.

This seems to be an issue with including "0" as a SampleID, but it needs more exploration.

This has to do with the way meta data IDs are accessed (e.g. line 36 of utils.R). We need to move away from expecting IDs to run from 1:n and use the ID directly.

This dataset also had a bonus bug in the form of some non-standard characters. I also added a quick check of the metadata file to guess an appropriate encoding to use when reading it in. Seems to work fine so far.

Issue 4 branch fixes this.


Update sbt-typelevel, sbt-typelevel-site to 0.5.3

About this PR
📦 Updates

org.typelevel:sbt-typelevel
org.typelevel:sbt-typelevel-site

from 0.5.0 to 0.5.3
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.typelevel" }
}]

labels: sbt-plugin-update, early-semver-minor, semver-spec-patch, version-scheme:early-semver, commit-count:n:2

Superseded by #40.


Problem on update if one of PK fields changed

Hi all,
I have a table which has a PK field as combination of three fields. When I wanted to update an entity in this table I saw it's not updating. Then I reversed some and found a problem in buildPkeyCriteria method.  When building PK key, criteria is generated using $this->fieldName attributes which return the new values of fields, so entity cannot be found in database.
Let me tell this with an example:
Original Row:
{
  "pk_field_1" 	: "a",
  "pk_field_2"	: "b",
  "pk_field_3"	: "c",
  "field_4"		: "d",
  "field_5"		: "e"
}

Updated Data:
{
  "pk_field_1" 	: "a",
  "pk_field_2"	: "bbbb",
  "pk_field_3"	: "c",
  "field_4"		: "ddddd",
  "field_5"		: "e"
}

Generated update query:
UPDATE `someTable`
SET `pk_field_2` = 'bbbb', `field_4` = 'ddddd'
WHERE `pk_field_1` = 'a' AND `pk_field_2`='bbbb' AND `pk_field_3` = 'c';

Thanks,
Can

Yes, this is a current limitation. Use a unique instead and a primary key which is auto-incremented. However, In the data-mapper branch this is supported.


chore: add CHANGELOG.md

Which problem is this PR solving?

Closes #557

Codecov Report

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

@@            Coverage Diff             @@
##           master     #574      +/-   ##
==========================================
+ Coverage    90.1%   90.75%   +0.65%
==========================================
  Files         149      128      -21
  Lines        7348     6016    -1332
  Branches      661      518     -143
==========================================
- Hits         6621     5460    -1161
+ Misses        727      556     -171

Impacted Files
Coverage Δ

...ges/opentelemetry-exporter-zipkin/src/transform.ts
100% <0%> (ø)
:arrow_up:

...telemetry-plugin-grpc/test/utils/assertionUtils.ts
100% <0%> (ø)
:arrow_up:

...entelemetry-exporter-jaeger/test/transform.test.ts
100% <0%> (ø)
:arrow_up:

.../opentelemetry-exporter-zipkin/test/zipkin.test.ts
100% <0%> (ø)
:arrow_up:

...pe-async-hooks/test/AsyncHooksScopeManager.test.ts
100% <0%> (ø)
:arrow_up:

.../opentelemetry-exporter-jaeger/test/jaeger.test.ts
100% <0%> (ø)
:arrow_up:

...entelemetry-exporter-zipkin/test/transform.test.ts
100% <0%> (ø)
:arrow_up:

packages/opentelemetry-web/test/WebTracer.test.ts

...-plugin-http/test/integrations/http-enable.test.ts

...lemetry-plugin-http/test/utils/DummyPropagation.ts

... and 22 more


Jenkins Gitea-Plugin: Cannot scan private repositories

Gitea version (or commit ref): 1.4.0
Git version: 2.17
Operating system: Alpine Linux 3.7.0 virt-hardened
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Description
Hi,
I have a problem with the Gitea-Jekins Plugin.
https://github.com/jenkinsci/gitea-plugin
It can only scan public projects of any organization.
I have a JenkinsCI user on my Gitea Instance which is Member of the Owner Group of the Organization.
In Jenkins i use the Credentials of the JenkinsCI user and the Scan runs successful - but reports "0 repositories found".
Without the credentials of the JenkinsCI user, i get HTTP 401 from Gitea - as expected.
But for some reason, i need to make a repository public to be visible to the repository scanning... which is bad for internal projects...
Is there any way to workaround this behaviour?
The GitHub Repo of the Plugin itself doesn't have Issues activated, sadly.
Gitea Organization Scan Log:
[Sun Apr 22 15:20:10 CEST 2018] Starting organization scan...
[Sun Apr 22 15:20:10 CEST 2018] Updating actions...
[Sun Apr 22 15:20:11 CEST 2018] Consulting Gitea Organization
Checking repositories...
0 repositories were processed
[Sun Apr 22 15:20:11 CEST 2018] Finished organization scan. Scan took 0.63 sec
Finished: SUCCESS

Feeling kinda ignored ._.
I don't know where else i could submit my issue, since the Plugin Author deactivated the ability and the Jenkins forums doesn't seem to provide their Issue Track for plugins anymore.
I followed some links of older issues related to the plugin and landed mostly on landing pages and 404 pages.

I expected quite a lot people using Jenkins.
Which one is the recommended / most popular CI that can integrate with Gitea?
I'm not sure if it may be an issue of Gitea itself, too - i don't know the API (yet).
Maybe it is the expected behavior to hide private repositories of organizations from API users? Or maybe there is a special "flag" that needs to be sent alongside the request to include private repositories?

Which one is the recommended / most popular CI that can integrate with Gitea?

Drone integration is quite good: http://docs.drone.io/install-for-gitea/

Maybe it is the expected behavior to hide private repositories of organizations from API users? Or maybe there is a special "flag" that needs to be sent alongside the request to include private repositories?

If anything, no access token is being passed so the repos aren't shown for that reason. But I don't know whether the plugin supports them :man_shrugging: Here is the documentation for the endpoint: https://try.gitea.io/api/swagger#/organization/orgListRepos

Thanks for the docs - reading them right now.
The access token should is probably not the problem, because if i don't configure the credentials of my JenkinsCI user in the CI Job Config, Gitea responds with 401.
Hm, i will take a look at the plugin's source (again) and try to find a hint why it behaves like this.

I have the same problem, but the workaround with admin user does not change anything

What does the Log of the Repository Scan report? (Jenkins' side)

I am also sure, this is a gitea bug. Here are the steps to reproduce:

Create a new Account on https://try.gitea.io/
Login and create a new Organisation
Change to this Organisation
Create a new Repository for this Organisation

Now I see at https://try.gitea.io/ under "my repositories" the new repository owned by the organisation.
But if I use https://try.gitea.io/api/swagger#/organization/orgListRepos (with authorize on top and change Schemes to HTTPS), After "try it out" - I doesn't see the new repository. The response body is empty.

I'm having the same issue here, too.
Jenkins: 2.121.1 (dockerized running on jenkins/jenkins:lts-alpine)
Gitea Plugin: 1.0.8
Gitea: f4b7b42 (dockerized running on gitea/gitea:latest)
For me, the admin workaround is fine at the moment ... unfortunatelly the admin account has access to all repositories - so everybody who has permissions to create jobs can possibly find out about private repositories that they should not see normally :(

I can confirm the issue with Gitea / Jenkins and also the workaround.
As mentioned by @rsiggi this is easy to reproduce on https://try.gitea.io/.
Using the API to get the repos of an org returns no results:
https://try.gitea.io/api/swagger#/organization/orgListRepos
Nevertheless, you can get details of an individual repo by providing the org and repo name:
https://try.gitea.io/api/swagger#/repository/repoGet
Hope that helps to narrow down the bug.

May I suggest to change the title of this issue, e.g.:

API/issues: orgListRepos does not return private repos without admin permissions / Jenkins Gitea-Plugin: Cannot scan private repositories

I had same issue.
The workaround is to promote gitea user to administrator.

This is present in Gitea Version: 1.11.0. Looks like the fix is only target at 1.6

I'm experiencing the same issue with the following version:
1.12.0+dev-276-g9789e0ad5 built with GNU Make 4.2.1, go1.13.7 : bindata, sqlite, sqlite_unlock_notify

Please reopen new issue with description on how to reproduce such issue

I'm experiencing the same issue with the following version:
1.11.3


File tab in Submissions

User Story
As a Program Manager or an applicant I would like to view previous submissions associated with the application, including current budget, cv in a file tab with each submission. Related to GH issue #2634
What happens now?
Previous submissions associated with an applicant are presented in the bottom right-hand corner. The budget, cv, and other documents uploaded by the applicant is toward the bottom of the submission.
What is the minimum the files tab in the Submissions tab should do?

[ ] make the files tab visible for the pm, applicant
[ ] display files that have been uploaded:

by applicants during the concept note stage
by applicants during the full proposal stage - e.g. cvs/bios/budgets/
previous concept note associated with the proposal

To be added later
Eventually the following should be added:

[ ] filter
[ ] sorting
[ ] search
[ ] include information about type of fund associated with the application

Priority

Medium

Affected roles

Staff
Applicants

Ideal deadline
Before

UI Designs
In the files tab view they see all the files that have been uploaded to Hypha that are related to the project:

Past submissions and current uploaded files are two different things. I do not see how we can add them in one table and make it useable.
I do not see how moving them to (hiding them in) another tab makes them easier to find, rather the opposite.


fix: 'AND' escaping for 'if(LLVM) ...' in 'src/shell'

Recall that CMake semantics are defined entirely
via string expansion. Without the string escaping, CMake tries to evaluate:
if(${STATIC} AND NOT ("${CMAKE_SYSTEM_NAME}" MATCHES "Darwin"))
→ if( AND NOT ("${CMAKE_SYSTEM_NAME}" MATCHES "Darwin"))
→ CMake Error at shell/CMakeLists.txt:19 (if):
    if given arguments:
      "AND" "(" "NOT" "(" "Linux" "MATCHES" "Darwin" ")" ")"

With the escaping, CMake evaluates as:
if("${STATIC}" AND NOT ("${CMAKE_SYSTEM_NAME}" MATCHES "Darwin"))
→ if("" AND NOT ("${CMAKE_SYSTEM_NAME}" MATCHES "Darwin"))
→ if(False AND NOT ("${CMAKE_SYSTEM_NAME}" MATCHES "Darwin"))
→ if(False)
→ ...

The key difference is '${STATIC}' evaluates to something like
'null', while '"${STATIC}"' evaluates to 'empty string/false'.

This was discovered when cleaning up #1687 to combine the LLVM and LLVM_BACKEND flags.

STATIC was removed in 5f4b1b1d44a55c542754a2e5e95cc8651b9c0ff2


test-branch/execute-job/wrong-commit-sha-on-pr/b534c99a-17eb-4646-a70c-2964761e1ce3

PR for testing f2429055882b36fc7f4c5e5c4ca25938e8bc362a

/canonical/self-hosted-runners/run-workflows 931ad21dd9ace68aaa0a7d341e94de96ad93fa6b


Handle subdependencies in Lockfile, and reenable pending specs in file_parser_spec

These specs have been disabled as file_parser doesn't yet handle the subdependencies in the Podfile.lock.

Fixed by #14


Made python -m lymph.cli.main work

:+1:

Better yet, define __main__.py inside lymph/cli and use "python -m lymph.cli ..."


For angular 9. * there is no version

For angular 9. * there is no version?
Thanks!

The current version of ngx-message-bus worked with my project as I migrated it to Angular 9, then Angular 10, and most recently Angular 11.


Add hgrid export excel samples

The XPLAT samples seem to not be loading all the data for WC and React. It seems that the hGrid has some problems when loading data from a class so we might want the data to be generated as a JSON for the hGrid samples.
This can easily be achieved by adding an XPLAT-CONFIG.json file in the data folder as in this commit.
Even with this change though, the WC excel-export sample does not load the child grids in the row islands for some reason.

Updated sample structure and added new data to align with Angular Hierarchical Grid Excel Export sample for consistency.

For some reason the WC samples continue to not show any data in the child grids. Can you try and figure out the reason, please? I suspect it has something to do with the WC template since I don't think there are any problems with the data structure anymore (as the other two platforms work great).
Additionally, I encountered two issues with the multi-column-headers-export samples:

in React the IgrExporterEventArgs class, which is used in the webGridExportEventMultiColumnHeaders template, is generated as IgrExporterEventEventArgs
no data file is being generated for the Blazor sample

I figured out why the WC samples were not showing any data in the child grids. The generated samples use igniteui-webcomponents version 4.7.1, while the hierarchical grid is added in 4.8.0 or so it seems. I changed the version and the child grids were fixed.

Yes, I actually use these versions: "@infragistics/igniteui-webcomponents-grids": "23.2.141", "@infragistics/igniteui-webcomponents-core": "23.2.141", "@infragistics/igniteui-webcomponents-layouts": "23.2.141", "@infragistics/igniteui-webcomponents-inputs": "23.2.141" and everything works as expected for me.

After investigating the problem of data file generation in the Blazor sample, I discovered that the root cause was null values within the dataset. To rectify this, I've conducted a thorough data refactor, replacing null values with appropriate alternatives. As a result, the data now generates successfully without any issues.


Update Circle CI config

Description

The change in the CI config addresses two problems:

External PRs are currently failing on CI. Running the NPM script npm run save-credentials on V2 now uses pwa-kit-dev tool. The command pwa-kit-dev save-credentials now validates the email before saving credentials.
https://github.com/SalesforceCommerceCloud/pwa-kit/blob/949cb775e0dfa31e5296c198c0dfad4f3845e479/packages/pwa-kit-dev/bin/pwa-kit-dev.js#L70
Externals PR doesn't have a valid e-mail.
Solution: Skip the save-credentials command on external PRs.

Generator jobs don't store the verdaccio.log file.
Solution: Store the verdacio.log file as a Circle CI artifact to allow further investigation when generator jobs fail on CI.

Types of Changes

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Documentation update
[ ] Breaking change (could cause existing functionality to not work as expected)
[ ] Other changes (non-breaking changes that does not fit any of the above)

Breaking changes include:

Removing a public function or component or prop
Adding a required argument to a function
Changing the data type of a function parameter or return value
Adding a new peer dependency to package.json

Changes

Skip the save-credentials command on external PRs and store the verdacio.log file as an artifact.

How to Test-Drive This PR

(step1)

Checklists

General

[ ] Changes are covered by test cases
[ ] CHANGELOG.md updated with a short description of changes (not required for documentation updates)

Accessibility Compliance
You must check off all items in one of the follow two lists:

[ ] There are no changes to UI

or...

[ ] Changes were tested with a Screen Reader (iOS VoiceOver or Android Talkback) and had no issues
[ ] Changes comply with WCAG 2.0 guidelines levels A and AA
[ ] Changes to common UI patterns and interactions comply with WAI-ARIA best practices

Localization

[ ] Changes include a UI text update in the Retail React App (which requires translation)

Why do we save credentials for internal PRs, but not external PRs?

What do we use the credentials for? Why doesn't it matter if they are missing?


chore: Update README.md

Nice! Can we use the mgprettier target to format the own README?

I did, I could actually add it as a target to run in our All target


Topbar added, only as a Label so far

@Arrinao  Pull before you continue working on this branch, I merged main into it

replaced by #34 and #36


🐛 [firebase_messaging] Firebase notification received twice in android in background mode

Notification receive twice in background mode.
I tried the code from firebase messaging sample code.
When showFlutterNotifcation is removed, notification is received one time but don't show headup notification.
What should I do?
@pragma('vm:entry-point')
Future<void> _firebaseMessagingBackgroundHandler(RemoteMessage message) async {
  await Firebase.initializeApp(options: DefaultFirebaseOptions.currentPlatform);
  await setupFlutterNotifications();
  showFlutterNotification(message);
}

for me also same issue, please respond if anyone have any solution

@GnuaAruht
Along with above comment / solution, check this and see if it helps to resolve the issue in your case.

Along with above comment / solution, check this and see if it helps to resolve the issue in your case.


remove All from Options All -Indexes

Options All -Indexes causes a 500 Internal Server Error. (Apache on hostpoint.ch / interxion).
Without the -All it works fine.
I'm not sure about all the effects, but I think the -All is already handled in the Apache directive and has not to be in .htaccess

What specifically is the Error 500 message?

It seems Options -Indexes depends on the context.
For example, without any + and - symbols:

<Directory "/web/docs">
  Options Indexes FollowSymLinks
</Directory>

<Directory "/web/docs/spec">
  Options Includes
</Directory>

then only Includes will be set for the /web/docs/spec directory. However if the second Options directive uses the + and - symbols:

<Directory "/web/docs">
  Options Indexes FollowSymLinks
</Directory>

<Directory "/web/docs/spec">
  Options +Includes -Indexes
</Directory>

then the options FollowSymLinks and Includes are set for the /web/docs/spec directory.

https://httpd.apache.org/docs/2.4/en/mod/core.html#options

What specifically is the Error 500 message?

[Wed Jan 11 17:44:25.050920 2023] [core:alert] [pid 90226:tid<PHONE_NUMBER>0] [client <IP_ADDRESS>:55274] /home/lindehof/www/rasen.lindehof.ch/public/.htaccess: Option All not allowed here

Can you check if your apache configuration file allows override of Options in the htaccess file?
It should be either:

AllowOverride All

or

AllowOverride Options

Can you check if your apache configuration file allows override of Options in the htaccess file?

Both are not allowed:
[Thu Jan 12 08:56:11.083171 2023] [core:alert] [pid 34223:tid<PHONE_NUMBER>2] [client 2a02:1210:3615:e400:c2e5:e96a:b60c:9f58:57092] /home/lindehof/www/rasen.lindehof.ch/public/.htaccess: AllowOverride not allowed here [Thu Jan 12 08:56:42.052244 2023] [core:alert] [pid 34121:tid<PHONE_NUMBER>2] [client 2a02:1210:3615:e400:c2e5:e96a:b60c:9f58:57098] /home/lindehof/www/rasen.lindehof.ch/public/.htaccess: AllowOverride not allowed here

No, I mean your actual configuration file httpd conf contains the AllowOverride lines.

I share my config apache2.conf
<Directory /home/http7_data>
        Options -Indexes
        AllowOverride All
        Order allow,deny
        allow from all
        Require all granted
</Directory>

With this my .htaccess running normal now, Can you sharing apache2.conf or log file?

i can't see my apache2.conf or other http conf files, it's on a shared hosting and unfortunately I don't have access to it...

The comment says "Disable directory browsing".
Then, this PR is no problem.
It seems we do not need to set All options.

i can't see my apache2.conf or other http conf files, it's on a shared hosting and unfortunately I don't have access to it...

In that case, we can assume your hosting provider does not allow override of all options. Anyway, this change seems harmless.

@sba Thank you!


failed to load locked problem! [0]

failed to load locked problem! [0]
when even I have signed in a premium account
was searching/click question:
Two Sum III - Data structure design

@belovedred1892 Will it help if you delete the cache? You can find the command in the command palette.

This problem still exist after delete cache.............

@belovedred1892
Found a root cause issue here: https://github.com/skygragon/leetcode-cli/issues/91
Looks like I need to update the cli version to resolve the problem.

Thanks for the investigation!

On vacation these days. Will prepare the new release when I get back. Thanks

Enjoy your vacation! Wait for your back

I also have a similar problem.
settings.json have the following definition, but it still doesn't work.
"leetcode.showLocked": true
Thanks @jdneo

Hi all,
I'm back from the vacation and will work on a new release this week. Sorry for the late reply...

I tested the<EMAIL_ADDRESS>But I cannot submit answer on Windows, though I can achieve that on Mac.
See the original issue here: https://github.com/skygragon/leetcode-cli/issues/134
Does anyone has the same issue as mine?

I haven't tried submitted answer directly from VS code

Can it work using cli directly with version 2.6.0?

Try a previous version in CLI? 2.5.4

Woooow~~  This problem is fixed now, Thanks so much for the fabulous work

The problem still exists on 13.1 in Centos VSCode 1.32.2

still there v0.17.0

Try this extension:


Refined GitHub is catching/logging unrelated errors

Description

The fix is to use stack.includes(chrome.runtime.getURL('')) to only match Refined GitHub errors.
How to replicate the issue + URL
Call this form the console and get an error that looks like it's coming from Refined GitHub
fetch("https://github.com/notifications/subscribe", {
    "body": "soup",
})

Initially reported in https://github.com/refined-github/refined-github/issues/7976 but I couldn't repro in Chrome/Firefox
Extension version
24.11.12
Browser(s) used
Safari
Token ⚠

[X] I set a token in the options page. I understand that some features require a token to work.

The fix is to use stack.includes(chrome.runtime.getURL('')) to only match Refined GitHub errors.

This might not work on Safari, I think the base URL is hidden in the stack


[network] Discovery module should utilize DirectSend protocol

🚀 Feature Request
Motivation
Today, the Discovery module utilizes raw substreams to execute the gossip protocol for p2p discovery. This is unnecessary, since we can utilize the DirectSend protocol for p2p gossip - this should remove some code duplication and move Discovery to right layer of abstraction.
We can take this idea further by making discovery a pluggable application module similar to consensus and mempool. This has 2 benefits:

ease the creation of a network library that is independent of the p2p nature of validators
allow different discovery module implementations for different parts of the libra ecosystem such as full nodes.

cc: @phlip9

Yeah, this definitely makes sense to me. The forthcoming on-chain discovery protocol feels like it has enough outside dependencies so as to not live inside network module.
I guess we would probably also include some new commands for application modules to connect/disconnect peers?
// network/src/interface/mod.rs

/// Requests [`NetworkProvider`] receives from the network interface.
#[derive(Debug)]
pub enum NetworkRequest {
    /// Send an RPC request to a remote peer.
    SendRpc(PeerId, OutboundRpcRequest),
    /// Fire-and-forget style message send to a remote peer.
    SendMessage(PeerId, Message),
    /// Update set of nodes eligible to join the network.
    UpdateEligibleNodes(HashMap<PeerId, NetworkPublicKeys>),
    /// +++ ADD +++
    DialPeer(PeerId, Multiaddr, oneshot::Sender<Result<(), PeerManagerError>>),
    /// +++ ADD +++
    DisconnectPeer(PeerId, oneshot::Sender<Result<(), PeerManagerError>>),

}

After further discussion offline, I believe we can rearchitect network a bit so that health_checker, connectivity_manager, and discovery are now pluggable application modules as opposed to built-in protocols. They can then use the provided direct_send and rpc interfaces that consensus, mempool, and state_sync use today.


PMID:21098122

increased duration of mitotic G2 DNA damage checkpoint during cellular response to ionizing radiation
normal histone H2A phosphorylation during cellular response to ionizing radiation

increased duration of mitotic G2 DNA damage checkpoint during cellular response to ionizing radiation FYPO:0005953
normal histone H2A phosphorylation during cellular response to ionizing radiation FYPO:0005954
edit file: b895d38c963f1ff31454347fe9ddd7e811a5ea3e
release: e4176e89bd1d1c1386ada6ef10a69fa585885360


needed?

Can't mixins already specify their own mixins array? I've never had a problem with this

Yes but they won't take advantage of the dependency management that is exposed with this project


API Response Codes link - Page Not Found

When navigating to the full list of API Response Codes  from any location the page cannot be found.
Example pages where this is happening:

Patient - https://nhsconnect.github.io/CareConnectAPI/api_entity_patient.html
Encounter - https://nhsconnect.github.io/CareConnectAPI/api_workflow_encounter.html#

Not aware of the status of this project.
Please not some of the content has moved https://simplifier.net/guide/nhsdigital/home
e.g. search parameters for Patient are listed here https://simplifier.net/guide/nhsdigital/NHSDigital-Patient#search
and will probably move to UKCore which is the new equivalent to CareConnect.


Invalid xsd reference

@Q-Tec90 commented on Tue Nov 26 2019
Hello,
the xsd reference in the example is invalid.
Could you please fix that.

@srvbpigh commented on Tue Nov 26 2019
Hello, @Q-Tec90
Thank you for your feedback.
We are actively reviewing your comments and will get back to you soon.
Kind regards,
Microsoft DOCS International Team

@CeciAc - what doc was this comment on?

Closing since there has been no response.


Allow underscores in slugs

It looks like in this PR we updated the slug function to remove anything that is not a character, number, or hyphen. I think we should also allow underscores in the slug.
The wiki article mentions that underscores or hyphens would be appropriate in a slug.
A number of users here currently use underscores in slugs, especially for the templated links in the time table viz (like /superset/dashboard/dashboard_{{metric }}) where the metric would have an underscore). So when they edit dashboards the underscores get removed and their links no longer work.
@mistercrunch @john-bodley

Fixed, thanks

LGTM


Removed unnecessary bucket parameter

Co-authored-by: @akash1810
Co-authored-by: @ioannakok
What does this change?

Removes the parameter pointing to the com-gu-frontend-artifacts bucket on the back of the silently failing deployments issue. After this change, this project will deploy to the correct value by default.
In terms of the application, there should be no noticeable impact to what's running.
How to test

Performing a deploy should continue to be successful.
How can we measure success?

Have we considered potential risks?

Images

Accessibility

[ ] Tested with screen reader
[ ] Navigable with keyboard
[ ] Colour contrast passed
[ ] The change doesn't use only colour to convey meaning

Deployed successfully in CODE in the correct bucket: https://riffraff.gutools.co.uk/deployment/view/f6f728a2-d5cc-4a54-93e3-6df3447df08a


Jaxb migration and upgrade

Migrated the dependency from old com.sun artifact to the reference implementation org.glassfish
https://mvnrepository.com/artifact/com.sun.xml.bind/jaxb-impl/2.3.3
https://mvnrepository.com/artifact/org.glassfish.jaxb/jaxb-runtime/2.3.3
Also, upgraded to the last 2.3.X -> 2.3.5
See issue #1514

Thanks sanmibuh! I've made some changes that won't allow me to merge it. However, I'll do it manually.

OK. Thank you :)


Fly should have a TRACE option

The Concourse ATC API is largely (entirely?) undocumented outside of code. One way to dramatically improve the explorability of the API and provide a lot of implicit API documentation coverage would be to have the fly CLI offer TRACE functionality similar to what the cf CLI for Cloud Foundry offers. With this feature enabled - by env var, configuration, flag, I don't care - HTTP requests and responses would be explicitly shown, inexpensively allowing broad visibility for starting places in interacting with the ATC API.

Moved to concourse/concourse#1094


Optimize SGD

Most use cases won't see any perf benefits, but in general we should always avoid having to do hash lookups when all we're doing is processing all entries.

good call! since this is basically the "sample code" for other optimizers it's good to have it using best practices


Tooltip day(s) pluralization is weird

no "s" on 21 - 31 - 41 - 51... days tooltip
idk whats going on here https://github.com/wakirin/Lightpick/blob/master/lightpick.js#L357 but I fixed it with a simple
return value > 1 ? arr[1] : arr[0]

That plural function for Russian, look at example:
` 1 день ... 3 дня ... 20 дней 21 день 22 дня ... `
I'll look for better solution, for multi-languages plural function.
Thanks for information.

Got idea from https://github.com/betsol/numerous
It's will be as function, example for Russian: https://gist.github.com/wakirin/8fdf443726f097326d927e0e85dbc5dd
default plural function will be:
...
tooltip: {
    one: 'day',
    other: 'days',
},
...
pluralize = function(i, locale)
{
    if (typeof i === "string") i = parseInt(i, 10);

    if (i === 1) return locale.one;
    if ('other' in locale) return locale.other;

    return '';
},
...

It's must work fine for most languages and if required another behaviour plural function, we can override it.
This changes will be include in 1.2.5 version.


furaffinity gallery error: 'Failed to load url'

Ripme version: 1.7.87
Java version:  1.8.0_31-b13
Operating system:  Win7 v6.1

Exact URL you were trying to rip when the problem occurred: e.g. http://www.furaffinity.net/gallery/silverbirch/
Please include any additional information about how to reproduce the problem:
Any FA url ending in /gallery/username or /scraps/username reproduces the problem

Expected Behavior
Ripping usergalleries on FA seemed to be working well until about two weeks ago(?).
Actual Behavior
Instant error upon trying to rip an FA gallery: 'Error: Failed to load [url] after 1 attempts'.  Have tried using the shared/default login, as well as putting my own credentials into rip.properties, doesn't seem to make a difference.  This happens alongside a rip complete message in the log, although nothing has been ripped: 'Rip complete, saved to [directory]'.
Tried at different times of day over the course of a week, does not seem to be a rate limit or anti-bot issue.  Though I can't know that for sure.

Hmm, as of this evening it appears now to be ripping fine.  Unsure what's changed, or what had changed and is now reverted (on FA side, my side, or etc).  Anyways, closed for now I suppose.


[Question] Set up android studio with viro react

I want to set up android studio with viro react for Hello World sample project. I have tried to modify my Android gradle and manifest files. However, I am facing two problems.

I cannot find "a/android/app/src/ovr/AndroidManifest.xml" in the Hello World project folder. I cannot add the code as shown in below to the file. How can I solve this problem?

I have tried to remove the red lines and add the green lines in my gradle file (Except for the code in question 1 above). When I tried to sync my project, it fails to sync the project and appears the messages below.

When I tried to upgrade the plugin to version 3.1.2, it appears the error as below.

Can somebody help pls?

Hi @HLneoh ,
For your 1st question - The file to change is at path android/app/src/ovr/AndroidManifest.xml. The a/ is added by git while creating the diff. It's a comparison between "a" and "b". Disregard them in the path when trying to find the file paths from that diff.
For your 2nd question - What version of Android Studio are you using? If you are seeing this error with Android Studio 3, can you try restarting your android studio? Also, what are the contents of your <your-project>/android/build.gradle ? They should be something on the lines of
allprojects {
    repositories {
        mavenLocal()
        maven {
            url 'https://maven.google.com/'
        }
        jcenter()
        maven {
            // All of React Native (JS, Obj-C sources, Android binaries) is installed from npm
            url "$rootDir/../node_modules/react-native/android"
        }
    }
}

The error that you are seeing Gradle DSL method not found: google() makes me believe you have something on the lines of
allprojects {
    repositories {
        ...
        google()
        ...
    }
}
Change the `google()` to `maven { url 'https://maven.google.com/'}` before updating your Android Studio and hit sync.

Let us know if that works for you,

Thanks,
-
Manish

Hi @HLneoh ,
Sorry for the delayed response here. Are you still running into this issue?
For the missing android/app/src/ovr/AndroidManifest.xml file, actually the the change in the gist means to add this file. In a git diff gist something like --- /dev/null +++ b/android/app/src/ovr/AndroidManifest.xml, /dev/null means there was nothing before, add the file mentioned in the +++ b. Easiest way to change this would be create a file at android/app/src/ovr/AndroidManifest.xml and paste the contents from the AndroidManifest.xml at the bottom of the gist file above. Additionally, you only need this change if you are trying to build your app for Samsung Gear VR platform. Otherwise you don't need this change.
For your second problem -> I just ran through our instructions for setting up Android Studio on Windows. We are missing adding the maven step I mentioned earlier in this thread. While we are updating the gist file, you can add it to your project as follows:
Open the build.gradle file located at ViroSample/android/build.gradle. You'll see an allprojects { ...} block in the file as
allprojects {
    repositories {
        mavenLocal()
        jcenter()
        maven {
            // All of React Native (JS, Obj-C sources, Android binaries) is installed from npm
            url "$rootDir/../node_modules/react-native/android"
        }
    }
}

Add the following line below the jcenter() line
maven { url 'https://maven.google.com' }

That should be the only change you'll need.
For your first error (in the complete log of the run) -> "Exit status 11, EventEmitter. error" This seems to be a react-native error rather than a Viro error. If you search online, you should find a lot of posts where other React-Native devs have run into the same issue like
https://github.com/react-community/create-react-native-app/issues/382
https://github.com/react-community/create-react-native-app/issues/533
Can you run through installing React Native on your Windows machine again by following instructions here -> https://facebook.github.io/react-native/docs/getting-started.html ? Make sure you select "Building Projects with Native Code" and development OS: "Windows" before following through the guide
Regarding your second error (your screenshot from Android Studio), that too seems to be a React Native Android Studio error folks have run into
https://github.com/react-community/react-native-maps/issues/1813
https://stackoverflow.com/questions/35674066/errorexecution-failed-for-task-appprocessdebugresources-java-io-ioexcept
Following solutions mentioned in these posts should help resolve this problem for you.
Good luck!


BCH Address Validator

I am not able to validate cash address. Only validating legacy bitcoin address

I see this issue too.


Streamline contributor experience / processes

Cirq has a growing community of contributors and it would be nice if the contributor experience and processes of cirq-on-iqm are similar to that of cirq. Some concrete suggestions for changes would be

Have a DEVELOPMENT.md highlighting the contribution process.
Run tests/coverage checks on PRs via github actions
Follow pattern of placing _test.py files next to .py

cc @balopat

Also add details about the release process, version naming and decide & document whether we want to make a pre-release version available to users via PyPi or not. Eg: Users can install the latest version of Cirq corresponding to the master branch on github via pip install --pre Cirq. Do we want a similar functionality for cirq-iqm ?

In fact coverage has been calculated for a while already. Here's an example of the report under 'Run tests' https://github.com/iqm-finland/cirq-on-iqm/runs/4062982391?check_suite_focus=true


update upgrade-provider generated commits to use conventional commit formatting

          the generated commits and PR description would benefit from conventional commit formatting

Originally posted by @displague in https://github.com/equinix/pulumi-equinix/issues/113#issuecomment-2234258100

Although it's not documented, I found a flag to add a prefix to the commit title: https://github.com/pulumi/upgrade-provider/blob/b625b8f79e1e57d625bbc31aadaefc380a917c16/main.go#L256

We can validate/enforce the title format with something like https://github.com/equinix/terraform-provider-equinix/blob/main/.github/workflows/validate_pr.yml This will help in automated releases

fixed in #116


Fix bug in import project function

This PR fixes a bug in the import project function and makes it a standalone function. The bug didn't affect the functionality of importing a project. The temporary directory was accidentally removed twice.

Thanks!


error when trying to fetch data use getMediaByUrl()

hi contributor and my friends.
i have one problem when call this method and then i got response
Message: Media with this code does not exist but the media url is also exist.

Same thing happens for me. Proof of concept:
<?php
use InstagramScraper\Instagram;
use GuzzleHttp\Client;

$instagram = Instagram::withCredentials( new Client(), '', '', null );
$instagram->loginWithSessionId( '<your-user-session-id-goes-here>' );

$media = $instagram->getMediaByUrl( 'https://www.instagram.com/p/BsOGulcndj-/' );
var_dump( $media );

Response from request to Instagram:
for (;;);{"__ar":1,"error":1357004,"errorSummary":"Sorry, something went wrong","errorDescription":"Please try closing and re-opening your browser window.","payload":null,"hsrp":{"hblp":{"consistency":{"rev":1005858897}}},"lid":"7121366591489386466"}'

Maybe related to #1067?

I've got a pull request up to fix this. See https://github.com/postaddictme/instagram-php-scraper/pull/1083

thanks for resolving thats problem @kingkool68


MAPREDUCE-7282. Move away from V2 commit algorithm

2349

 Quality gate: Failed
2 Issues

 2 Code issues

Details

 Quality gate: Failed
2 Issues

 2 Code issues

Details


118 modify loadfieldop and storefieldop to specify if field is static or instance field

good work Edu, just some minor changes I think


Use DI container to inject ContentFetchers class

Motivation

The DI container will solve passing the dependencies for “cross-cutting concerns” for us: Individual classes just have to request their dependencies (Cache, Logger) in a constructor and the container will pass them automatically upon instantiation.
Reduction of global state for making the testing easier: instead of formatters relying on $_SERVER superglobal and having to hustle with it in tests, they will just take RequestInfo object as a dependency and in tests, we will pass a custom one through the container.
The ability to customize e.g. CacheInterface implementation for library consumers: As simple as $dic->add(CacheInterface::class, fn () => new MyCustomCache());

What this PR does
This is in preparation to allow us injecting Cache instance, commits to be reviewed independently:

Implemented a very stupid dependency injection container that creates requested objects and passes them dependencies based on typehints of constructor method. Then I made BridgeFactory use it to create bridges.
Moved most of the stuff in lib/contents.php into a new ContentsFetcher class so that it can receive Cache.
Used a custom Rector rule to inject ContentsFetcher to all classes that rely on the moved helper functions (should be squashed with 2. once reviewed).

It's a good pr. Won't merge because of too much introduced complexity and enterprisey logic.

I removed the PSR 11 interface and constructors of individual bridges (they can get the ContentsFetcher from BridgeAbstract. I also made sure the individual commits still work (including the old functions as shims and removing them after the the commit automatically generated by Rector).

Sorry too much complexity. Don't want that container. If were to use a container it's not gonna do autowiring. Also I want to preserve getContents as a function not belonging to an object.

Can’t really say I like it but here it is without a container: https://github.com/RSS-Bridge/rss-bridge/pull/2939


Evaluate use of Otterize

          @qcaas-nhs-sjt @vvcb

I've been looking into approaches for managing network policies within the cluster and have come across Otterize, which supports a declarative way of expressing ingress and egress network policies between services in a K8s cluster.
It uses an intent-based access control (IBAC) approach, where a developer of a particular app/service (e.g. OHSDI Atlas) can declare an intent to communicate with other services (e.g. OHDSI WebAPI).
With the growing number of services we're looking to support and the associated complexity in understanding, defining and managing network policies going forward. Manually configuring NetworkPolicy/GlobalNetworkPolicy resources could become error-prone/time-consuming given the expected complexity of the SDE and the likelihood of different developers internally developing microservices that require access to other services to operate.
Otterize is open source and also has a 'free' cloud offering which provides certain features that might be useful to support development (i.e. real-time network traffic and policy mapping/vis https://github.com/otterize/network-mapper). But initially, we could use the intents operator (https://github.com/otterize/intents-operator) that listens for declared 'intents' resources and can automate the creation of L3/4 networking policies in the chosen CNI plugin for networking (i.e. calico). The 'intents operator' runs as a service in the cluster and can automate the creation of policies in calico. As calico can work within a bare metal and vendor k8s infrastructure and Otterize acts as a layer on top of calico, there wouldn't be compatibility issues.
The intents mapper tool looks pretty cool in that it can analyse existing traffic flows between services in the cluster and automatically infer/suggest the necessary intents needed to secure the cluster which we could initially use for development but in theory as new services are added/removed, it could act to notify us (or SecOps team?) of security issues and the need for additional networking policies in the cluster.
In terms of CICD and SecOps processes, we could establish a human-in-the-loop flow where intents are defined or suggested (using the mapper tool) in dev and the enforcement mode is used to apply the inferred 'access graph' as underlying network policies in the staging environment where a SecOps/IG team can verify and test before being applied in production.
Otterize also comes with some additional capabilities such as the credentials-operator which could be used for just-in-time management of IAM identities in Azure/GCP/AWS. Could be useful for managing L7 authentication to access managed services (e.g. Storage Accounts).
Thoughts?
Originally posted by @m1p1h in https://github.com/lsc-sde/lsc-sde/issues/38#issuecomment-2122053686

@m1p1h @vvcb
Have moved this to it's own ticket, so we don't flood the epic

So a few generalised questions about this:

What license is this offered on?
Are there other editions/add-ons which are paid for?
If there are paid-for offerings, which features are which?
Are we sharing any information with their services or is this entirely isolated?
If we are sharing information with their services, what is the nature of the information being shared?

In addition, I feel that there are two aspects to this we need to consider in isolation:
Network policies
We currently implement network policies via Calico and this gives us a fairly good set of tools that allow us to define our network policies and to restrict traffic to specific pods. The key thing with any such solution is that we need to understand which pods are talking to which and why? The network policies at present are permissive, we need to implement based on the principles of least privilege. Therefore the question is what does otterize do about this? What does it offer us above and beyond the features offered by Calico.
IBAC
I personally love the idea for this, however I have some questions surrounding the implementation of this in an enterprise environment:

How do these intents get implemented?
Does it create roles in Entra ID, and then apply these roles as role assignments in the Azure Subscription?
If so what level and scoping is this applied?
Does it apply each of these roles per pod using pod identity or does it utilise security groups or something else to group these together?
How does this fit in with the limitations for each of these services? Such as limits on the number or roles per tenant. or the limits on role assignments per subscription?
Does it  create app registrations? or managed identities? or something else?
What permissions does the controller need to function correctly?
How likely is it that the enterprise will give us that level of access?

My worry is that while it sounds like a wonderful idea, if this works as I would expect it to then it may create an unsustainable permission base that could run away from us.
An alternative approach is that we can fashion the roles manually and link to these only using the intents. We do currently have a number of custom roles we've fashioned to assist with this project, however these are not applied by us, they are applied by the network team and we are assigned the rights to assign these roles to other users/app registrations/identities/groups.


Generic Environment Sync action in Web UI

We would like a task that replaces drush sync in the web UI (but preferably isn't drush specific).

It will copy databases and assets/files between environments
For preference, it will be flexible about source and destination environments, but will only allow people to run it if they have permissions to the destination environment
For bonus points, if this can let you choose a sanitised database, that would be great, but I don't expect this.

HTH,

We're overhauling advanced tasks/custom tasks in Lagoon, and the goal will be to improve the documentation around this that will allow users to create these tasks themselves.
Lagoon probably won't ship with any built-in tasks in the future, because we plan to deprecate those standard drush based tasks eventually.

While I agree with deprecating the drush-based ones, I still think there should be a few that can be added just by adding something to .lagoon.yml or something.  I shall look forward to the revamped documentation :) .  Is there a ticket we can follow for the advanced tasks overhaul?

While I agree with deprecating the drush-based ones, I still think there should be a few that can be added just by adding something to .lagoon.yml or something. I shall look forward to the revamped documentation :) . Is there a ticket we can follow for the advanced tasks overhaul?

I will say now that tasks defined in the .lagoon.yml will probably not be a thing. The management of these becomes too difficult to know which environment, where it lives etc..
We will provide some example tasks that will perform some things like replacements for the more common tasks (file sync, db sync), but won't be ones that would include sanitising databases, etc. But from the examples, you would be able to copy and extend it to do whatever additional functionality you would want.
As for which ticket to follow, I guess this would be one of them, additionally:

#3400
#3477

We haven't road mapped anything related to this work specifically, so there is no ETA or planned/designed work for this. There is current documentation for creating custom tasks now, but this will eventually be deprecated once we decide on the new task API.

Agreed that general defining of tasks in .lagoon.yml shouldn't be a thing
I was thinking that in some specific cases (eg. this one, and probably 3-5 others) that it might be worthwhile having a special enabling thing that works just for those specific tasks
Agree that DB san doesn't belong as one of those special tasks, just the really common ones.

Good info on the tickets and status -- thanks!

Agreed that general defining of tasks in .lagoon.yml shouldn't be a thing
I was thinking that in some specific cases (eg. this one, and probably 3-5 others) that it might be worthwhile having a special enabling thing that works just for those specific tasks

The problem of managing things in the API via .lagoon.yml is a real bad one though. Builds are almost completely decoupled from the API by design (outside of any pre/post rollout tasks that users define), so introducing this mechanism into the build phase that would call back to the API to check or modify something in the API that could then effectively be overridden by a .lagoon.yml from another environment is why this is a bad idea.
The logic to work out if a task can or can't be run on a specific environment because the .lagoon.yml says so becomes super complex and difficult to maintain, so the simplest solution is to not do it it all via .lagoon.yml.
As we get closer to designing what custom tasks v2 looks like, tickets will get updated. But generally, it will be managed via the API/CLI only, not via builds.   Was just trying to make it more straightforward.  But if it's a CLI tool thingy, I can live with that.  Just means more scripting :) .

OK. Was just trying to make it more straightforward. But if it's a CLI tool thingy, I can live with that. Just means more scripting :) . Edit: Straightforward from a Dev-using-lagoon POV, not your POV.

Well since we haven't decided on the API, or designed anything yet, and we've only had discussions. There isn't really anything certain to compare it to yet. The solution we have discussed internally so far under would be far more configurable than the current custom tasks implementation.
Just for transparency, the initial discussions have been down this path, which is probably more than suitable.

being able to create custom tasks at the organization level
creating rule sets on the task/in the organization that would allow you to restrict
which projects or environments can run the task (regex/pattern matching etc.)
which group within the org can run the task (based on role still)
which environment types will be able to run the task (dev or prod envs)
which permission level can view/execute the task (guest, maintaner, etc..)

Since tasks are wrapped in permissions (ie who can execute, who can add them, which tasks are available to which project etc..), having some way to enable to disable tasks in the .lagoon.yml would not be able to override the permissions of a task, as that would potentially allow someone with git access to change the permission required to perform a task, or just enable a task randomly which maybe you don't want them to do. So you'd still need some way to limit which tasks could be requested/enabled/disabled in the .lagoon.yml still.

Tasks at the org level would definitely be a help.


add landing page

#3916

What is shown to the user while all the data is being sync'd and gathered. looks like they will just see a blank page.

Getting this on your branch. I cleared application data and reloaded. it happens after logging in.

What is shown to the user while all the data is being sync'd and gathered. looks like they will just see a blank page.

@bthaile are you seeing a blank page on featurepeek? It should show loading state like the below

Getting this on your branch. I cleared application data and reloaded. it happens after logging in.

actually I see it on master.

Also i don't see the above error - how are you triggering it?


Search found results but install could not find the package. "No package found matching input criteria."

Brief description of your issue
Winget search found the results but when we try to install the app it does not work.
winget search roblox

Although I am able to install it via Microsoft Store.

Steps to reproduce

First search your application

winget search roblox

Second install your application

winget install --accept-package-agreements --accept-source-agreements --exact --silent --id 9NBLGGGZM6WM

Expected behavior
Give appropriate error messages of why it could not find and install.
Actual behavior
Does not give a reason why it could not find a package to install. Even verbose logs does not give out any detail.
Environment
Windows Package Manager v1.2.10271
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19042.1645
Package: Microsoft.DesktopAppInstaller v1.17.10271.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Do you get the same results with winget install Roblox -s msstore?
Is your machine locale configured for the United States or another country?
There are some subtle differences in how the Windows Package Manager and the Microsoft Store UI behave. I'm trying to isolate what might be happening.

I can reproduce on my side and it might be related to https://github.com/microsoft/winget-cli/issues/1833.
Some of the affected games:

Halo Infinite (doesn't require money)
Candy Crush Saga (doesn't require money)
Roblox (doesn't require money)
Minecraft Launcher (doesn't require money)
Forza Horizon 4 Demo (doesn't require money)
Destiny 2 (doesn't require money)

These games show up in winget search but returns No package found matching input criteria. when doing a winget install.

winget search roblox doesnt even show roblox for me


Profile - Edit photo button is too big. Should be 28px instead of 40px.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

From @puneetlath: https://expensify.slack.com/archives/C01GTK53T8Q/p1618514242295200?thread_ts=1618513819.294300&cid=C01GTK53T8Q

edit photo button is too big. Should be 28px instead of 40px.

Also, which clicked, the dropdown opens over the green + instead of below the button. (This is being worked here https://github.com/Expensify/Expensify/issues/159168 and https://github.com/Expensify/Expensify.cash/issues/2090)

Expected Result:
Edit profile button has a tall size of 28px
Actual Result:
Edit profile button has a tall size of 40px.
Action Performed:

Login in staging.expensify.cash
Click on the profile icon
Click on profile

Workaround:
Visual issue, no need for workaround.
Platform:
Where is this issue occurring?
Web ✔
iOS
Android
Desktop App
Mobile Web
**Version Number:**1.0.23-1
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Expensify/Expensify Issue URL:

Triggered auto assignment to @greg-schroeder (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

@greg-schroeder adding the daily label, can you please triage and move the issue along?

cc @mountiny this might be a good first pick issue to get some Expensify.cash xp

PR merged. The updated template does not close the issue automatically.


Magento 2.4.4 setup upgrade error

I'm trying to install the module on Magento 2.4.4 (or 2.4.4-p1) but during
bin/magento setup:upgrade
I have this error:
call_user_func(): Argument #1 ($callback) must be a valid callback, class "PostFinanceCheckout\Payment\Setup\Patch\Data\AddData" not found#0
Has the module the compatibility with Magento 2.4.4? Is there a solution for this error?
Thank you

The same happens on Magento 2.3.6-p1

Dear bondazzi, kess78 this was I BUG, which should be fixed now.
Please delete our current plugin version completely from Magento 2 and reinstall the newest version.


Document new Tangram debug option

Follow-on to #279, we’ll need documentation for that option and an appropriate version bump.

I should rename option name from _debugTangram to debugTangram before documenting it

closed via 5d8e6a5e1d8d417d8a6e52536dab8cb3e35d58a6 , 975078a97a472b56db81e895c33ca7e5cfe7c19d


Support C# 6 features in Cake script

I thought Cake was based on Roslyn, so I was surprised to realize that C# 6 features are not supported (e.g. string interpolation, null conditional operator...)
Is this going to be supported?

Cake supports both released Roslyn versions (version 1.2.20906.2 and the latest bits 1.0.0-rc2). The old Roslyn (pre C# 6) is the default one.
To use the latest bits (with C# 6), pass the flag -experimental to Cake.exe.

Thanks, it works
Since C# 6 is now stable, is there a reason to keep this "experimental" ? Do you intend to switch to the latest stable Roslyn version ?

See: https://github.com/cake-build/cake/issues/293
While Roslyn is stable, it's scripting features were hidden and removed in the final version.

Well, the scripting bits are not out of beta yet and things have been changing before so we'll wait until after the 1.0 release to make it the default script engine.
Thanks for your feedback. I'll add some clarification in the documentation about this.

OK, thanks. I had not realized that the scripting part had not been released yet.


storage: should offer a way to set Content-Length for uploads

https://github.com/GoogleCloudPlatform/google-cloud-node/issues/601 is related
Without specifying Content-Length, Google Cloud Storage can’t tell whether a request successfully completed, it can only see that the client stopped sending data.
The omission of a Content-Length header contributes to a subtle data corruption bug in code which uses the following pattern:
func upload(…) error {
	var err error
	wc := client.Bucket(bucket).Object(object).NewWriter(ctx)
	defer wc.CloseWithError(err)
	_, err = io.Copy(wc, f)
	return err
}

In case the program panics (e.g. in the io.Reader f, or unrelatedly due to memory pressure), deferred statements will still be executed, so wc.CloseWithError(nil) (note the nil!) is called, which results in the underlying io.PipeWriter returning io.EOF, just like when a file is successfully read.
Upon panic, two things can happen:

The program recovers from the panic, and the uploaded file will be incomplete (!).

The program terminates. The upload probably times out, but I don’t know for sure (and can’t find any docs) how the server side handles that (especially without a Content-Length header).

I see a couple of things to improve:

Remove CloseWithError entirely. I don’t see how it differs from Close? Is there a real-world use-case for it?
Document that CloseWithError shouldn’t be called from defer statements (as a precaution).
Introduce the Content-Length header so that the server side can discard incomplete uploads.

cc @mpvl @merovius

The example code is buggy regardless; defer evaluates the arguments to the function call directly, so the given code will always pass nil, even in the absence of panics. The correctly wrong idiom (to illustrate the issue) would be
func upload(…) error {
	var err error
	wc := client.Bucket(bucket).Object(object).NewWriter(ctx)
	defer func() { wc.CloseWithError(err) }()
	_, err = io.Copy(wc, f)
	return err
}

and the correctly correct idiom would be
func upload(…) error {
	var err error
	wc := client.Bucket(bucket).Object(object).NewWriter(ctx)
	defer func() {
            if err != nil {
                wc.CloseWithError(err)
            }
       }()
	_, err = io.Copy(wc, f)
        if err != nil {
            return err
        }
	return wc.Close()
}

Not contending that the API makes it relatively easy to write the wrong code (as illustrated). But a) I see a good argument for having CloseWithError to explicitly roll back and clean up resources in the error-case, so I don't think 1 is an adequate solution and b) I believe CloseWithError needs to be called from a defer to ensure the correct cleanup of resources in case of a panic (if you care); just in the correct way, so 2 isn't an adequate solution either.
I'm not familiar with the implementation (and frankly too lazy right now to look it up :) ), but how is Close implemented? Does it literally just close the connection to the server and the server thus assumes the write was successful, or does it include some explicit commit-action? And if so, how does CloseWithError change that?
It seems 3 is needed/useful iff the protocol does not do an explicit commit. If it does, then it's irrelevant what the Content-Length is, as it won't commit anyway on short writes. If it doesn't, short writes could happen for any number of reasons (like cut fibers or power outages), so it's a bad idea to not guard against them and Content-Length seems like a good way.
All of that being said, how to correctly commit a write should definitely be documented, because evidently it's non-trivial and can contain subtle bugs :) I'd also argue that CloseWithError should probably rollback a write even if nil is passed (thus making it possible to elide the nil-check).

Thanks for the correction.
Close doesn’t include an explicit commit, and neither does CloseWithError do any explicit rollbacks. AFAICT, Google Cloud Storage’s API supports neither.
To expand on my suggestion ①: defer wc.Close() should be called instead of wc.CloseWithError, at which point CloseWithError could be removed.

We upload in chunks. Each chunk has a Content-Length. See https://github.com/google/google-api-go-client/blob/master/gensupport/resumable.go#L65.
If you really want to make sure your data landed OK, set MD5 or CRC32C on the Writer. If you use CRC32C, make sure you also set SendCRC32C.  http://godoc.org/cloud.google.com/go/storage#Writer. (Note that the first two fields are on Writer via the embedded ObjectAttrs.)

Thanks for the reply.
The ObjectAttrs.CRC32 field docstring states “This field is read-only.”, which seems to contradict setting it when uploading data…?
Could we extend the documentation to clarify that the effect of sending a CRC32 is that GCS can discard invalid uploads?
Also, could we add some sample code which demonstrates the use of CRC32?
I’m happy to send a PR for either.

https://code-review.googlesource.com/15852

Great, thank you very much!

Hi @jba We are facing a similar issue where the uploader is uploading 0 bytes.
Per your comment
https://github.com/GoogleCloudPlatform/google-cloud-go/issues/724#issuecomment-321873549
That goes through that path when the content being uploaded is greater than chunksize per
https://github.com/google/google-api-go-client/blob/master/storage/v1/storage-gen.go#L8751
And MediaBuffer is only created when content is greater than chunk size for resumable upload, per
https://github.com/google/google-api-go-client/blob/master/gensupport/media.go#L191
What happens on retries when it doesn't go through resumable upload ?

@dneralla I'll look into it.

@jba  Do you want me to file an other issue ?
         delta := rand.Int31n(3)       if delta < 2 {           return nil, &googleapi.Error{               Code:   503,               Header: res.Header,           }       }
I added something like the above in https://github.com/google/google-api-go-client/blob/master/storage/v1/storage-gen.go to simulate retries.
On the uploads greater than chunksize it was able to upload fine but on uploads less than chunksize it just uploads zerobyte files.


Update CryptoSJnet.toml - IP Changed

This is CryptoSJnet and i would like to change IP
PR from user  pr here
Description
If this is an UPDATE for a previous genesis validator
Checks are made against your net_address. If this has changed since the previous testnet, make sure you provide links of previous prs merged from your previous git username.
Thanks in advance!
Checklist before merging to draft

[x] Only one toml is added in this PR
[x] The file being added is indeed a .toml file
[x] The toml being added is to the correct folder, and only to the correct folder
[x] The eth_hot_key and eth_cold_key are present

checklist done


Git Branch is always HEAD on GitLab CI

New Issue Checklist

[ ] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
:git_branch always returns HEAD as the branch name when running on GitLab CI.
When running the lane locally, it returns the correct branch name.
It is the same issue as reported by this issue: https://github.com/fastlane/fastlane/issues/1033
Environment

**No Appfile found**
fastlane gems

Gem
Version
Update-Status

fastlane
2.63.0
🚫 Update available

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.0.0

bundler
1.16.0

io-console
0.4.5

CFPropertyList
2.3.5

i18n
0.8.6

minitest
5.10.3

thread_safe
0.3.6

tzinfo
1.2.3

activesupport
4.2.9

public_suffix
2.0.5

addressable
2.5.2

babosa
1.0.2

claide
1.0.2

clamp
0.6.5

fuzzy_match
2.0.4

nap
1.1.0

cocoapods-core
1.3.1

cocoapods-deintegrate
1.0.1

cocoapods-downloader
1.1.3

cocoapods-plugins
1.0.0

cocoapods-search
1.0.0

cocoapods-stats
1.0.0

netrc
0.7.8

cocoapods-trunk
1.2.0

cocoapods-try
1.1.0

colored2
3.1.2

escape
0.0.4

fourflusher
2.0.1

gh_inspector
1.0.3

molinillo
0.5.7

ruby-macho
1.1.0

nanaimo
0.2.3

xcodeproj
1.5.1

cocoapods
1.3.1

colored
1.2

highline
1.7.8

commander-fastlane
4.4.5

declarative
0.0.10

declarative-option
0.1.0

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20170404

dotenv
2.2.1

excon
0.59.0

multipart-post
2.0.0

faraday
0.13.1

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

faraday_middleware
0.12.2

fastimage
2.1.0

jwt
2.1.0

little-plugger
1.1.4

multi_json
1.12.2

logging
2.2.2

memoist
0.16.0

os
0.9.6

signet
0.8.1

googleauth
0.6.1

httpclient
2.8.3

mime-types-data
3.2016.0521

mime-types
3.1

uber
0.1.0

representable
3.0.4

retriable
3.1.1

google-api-client
0.13.6

json
2.1.0

mini_magick
4.5.1

multi_xml
0.6.0

plist
3.3.0

rubyzip
1.2.1

security
0.1.3

slack-notifier
1.5.1

terminal-notifier
1.8.0

unicode-display_width
1.3.0

terminal-table
1.8.0

tty-screen
0.5.1

word_wrap
1.0.0

rouge
2.0.7

xcpretty
0.2.8

xcpretty-travis-formatter
1.0.0

mini_portile2
2.1.0

nokogiri
<IP_ADDRESS>

slather
2.4.4

generated on: 2018-04-04

@micnguyen Yeah, you may have to do some manual overrides since what scan is reporting is "technically" correct 🙃 Can we close this issue for now since this is an issue more with the CI?

That's right, it is more of an issue with the CI as opposed to Fastlane. If anyone stumbles onto this, the quickest hack solution is to provide the $CI_COMMIT_REF_NAME variable from gitlab-ci.yml into Fastlane as a parameter and use that as necessary. :)
Closing.


Feature Request: folding by directory

Wouldn't it be awesome if bufexplorer had a possibility to expand/contract directories?
Something like
a/a/a.txt
a/a/b.txt
a/a/c.txt
a/b/a.txt
a/b/b.txt
a/b/c.txt

to
<a/a/...>
<a/b/...>

or
<a/...>

using vim folding e.g.

That does sounds interesting...


within_prior instead of log_prob for support check.

in the context of posterior.map() we are still using torch.isfinite(prior.log_prob(theta)) to check whether theta is in the support. This can result in value must be in support error in PyTorch.
This PR uses within_support instead to fix this.

I don't get it...there are no changes in posterior.map(). What am I missing?

Oh sorry, then the only connection to posterior.map() is that this error occurred for the first time when using posterior.map(). I guess because it involved log_prob evaluations with samples outside the prior.

I changed the test utils to pass the tests. the problem was that the prior was in [-2, 2] and the sample outside support was at 2, so actually in support.
Still good to go @michaeldeistler?


Change sbolstack-frontend to use the jar without dependencies

The sbolstack-frontend includes libSBOLj which causes conflicts with the libSBOLj jar file when they get out-of-sync.  Switch to using the sbolstack-frontend without dependencies and add the dependencies needed individually.
Note these dependencies are all available in the iBioSim gui/lib directory, since this is the version used by iBioSim for the same reasons.

close


addRequestSpecification thows error if baseUri is not set

If i want to merge a RequestSpecification to a RequestSpecBuilder it will throw an exeption java.net.MalformedURLException: no protocol: if the requestSpecification has no baseUri set.
Example:
        RequestSpecBuilder addBuilder = new RequestSpecBuilder();
        addBuilder.addHeader("test","test");
        RequestSpecification addSpec = addBuilder.build();

        RequestSpecBuilder builder = new RequestSpecBuilder();
        builder.addHeaders(headers);
        builder.addRequestSpecification(addSpec);
        builder.setBaseUri(uri);

If i set a baseUri to addBuilder
addBuilder.setBaseUri("https://localhost");

the addRequestSpecification(addSpec); works fine.

@detvetdu
Your example is incomplete and if I try to fix it, it works without exception:
RequestSpecBuilder addBuilder = new RequestSpecBuilder();
addBuilder.addHeader("test","test");
RequestSpecification addSpec = addBuilder.build();

RequestSpecBuilder builder = new RequestSpecBuilder();
builder.addHeaders(new HashMap<String, String>() {{ put("header", "headerValue"); }});
builder.addRequestSpecification(addSpec);
builder.setBaseUri("https://www.google.com");

given(builder.build()).get("/");

Maybe your URI is really malformed? What is the value of uri variable?

The uri is not set in addBuilder, i suppose it is null then.
In the RequestSpecBuilder builder it is not set before i call for addRequestSpecification.


fix: use configDir for save

On macOS, it still tries to read the configuration from ~/Library/Application Support/algia/config.json. #4 didn't fully cover the occurence of os.UserConfigDir in save function, so I've replaced it by configDir.

Thank you