[1/7] lnwire: add new Gossip 1.75 messages

This is the first step towards the Gossip 1.75 epic
In this PR, the definitions for the new gossip messages defined in the
taproot-gossip proposal are implemented. This includes:

announcement_signatures_2
channel_announcement_2
node_announcement_2
channel_update_2

I expect there to still be a few updates to these messages as the proposal progresses
The flow of the PR:

Commits 1-4: A rename is done for each of the messages which will now have a _2 counterpart. This is done so that we can also have an interface to describe both the _1 and _2 messages. For example: ChannelAnnouncement is renamed to ChannelAnnouncement1 so that we can add ChannelAnnouncement2 later along with a ChannelAnnouncement interface.
Commits 5-9: Small preparation commits to make certain lnwire types more easily reusable & to prepare various util functions.
Commit 10: add new nonce fields to channel_ready
Commit 11: add new block range fields to GossipTimestampRange
Commits 12 - 14: Add the interfaces for the messages (for example, add the ChannelAnnouncement interface and ensure that it is implemented by ChannelAnnouncement1.
Commit 15: add a MsgHash helper
Commit 16-20: Add the new message types.

Question:
Note that this initial set of Gossip PRs does not make use of NodeAnnouncement2 yet. So if preferred, I could remove it from this PR.

@ellemouton, remember to re-request review from reviewers when ready

@ellemouton, remember to re-request review from reviewers when ready

@yyforyongyu: review reminder
@ellemouton, remember to re-request review from reviewers when ready

@ellemouton, remember to re-request review from reviewers when ready

@yyforyongyu: review reminder
@ellemouton, remember to re-request review from reviewers when ready

@yyforyongyu: review reminder
@ellemouton, remember to re-request review from reviewers when ready

@ellemouton, remember to re-request review from reviewers when ready

@ellemouton, remember to re-request review from reviewers when ready

@yyforyongyu: review reminder
@roasbeef: review reminder
@ellemouton, remember to re-request review from reviewers when ready

@roasbeef: review reminder
@ellemouton, remember to re-request review from reviewers when ready

@roasbeef: review reminder
@ellemouton, remember to re-request review from reviewers when ready

@roasbeef: review reminder
@ellemouton, remember to re-request review from reviewers when ready

@roasbeef: review reminder
@ellemouton, remember to re-request review from reviewers when ready

@roasbeef: review reminder
@ellemouton, remember to re-request review from reviewers when ready

@roasbeef: review reminder
@ellemouton, remember to re-request review from reviewers when ready

@roasbeef: review reminder
@ellemouton, remember to re-request review from reviewers when ready

@roasbeef: review reminder
@ellemouton, remember to re-request review from reviewers when ready

@roasbeef: review reminder
@ellemouton, remember to re-request review from reviewers when ready

@roasbeef: review reminder
@ellemouton, remember to re-request review from reviewers when ready

@roasbeef: review reminder
@ellemouton, remember to re-request review from reviewers when ready

@roasbeef: review reminder
@ellemouton, remember to re-request review from reviewers when ready

@roasbeef: review reminder
@ellemouton, remember to re-request review from reviewers when ready

@roasbeef: review reminder
@ellemouton, remember to re-request review from reviewers when ready

@roasbeef: review reminder
@ellemouton, remember to re-request review from reviewers when ready

sorry about the lag here - it is on the list. Just need to get things through with the RB stuff first. Then focus is back on this

@roasbeef: review reminder
@ellemouton, remember to re-request review from reviewers when ready

@roasbeef: review reminder
@ellemouton, remember to re-request review from reviewers when ready

!lightninglabs-deploy mute

I agree with both comments so im wondering what you guys think is the best solution here?

Perhaps the solution is moving the validation interface into the netann package?

Thanks for the review @Roasbeef  & @yyforyongyu  🔈


onAuthError return rejected promise

@bortevik @brancusi this PR shouldn't have been merged. This breaks the user experience when someone enter the wrong password and then enters correct one. onAuthError should return resolve() or unresolved promise.
Have a test run in your apps and see what I mean.


Default Windows download is for all users, then prompts to install user install

Since it prompts the user to download the user install, shouldn't that be the default download?

This is handled by https://github.com/Microsoft/vscode-internalbacklog/issues/254


in ios, audio stops after returning to home page (flutter: Fatal Error: Callback lookup failed!)

I'm trying to figure out how to keep my audio playing after the app is in the background in ios. I've been looking at the 'example/lib/player_widget.dart' and 'example/lib/main.dart'. It seems that my problems is that I am getting the following message on my terminal when I return to the home page, however, when I'm running the example app, I do not get the following message.
flutter: Fatal Error: Callback lookup failed!
The following is my code, does anyone know what I am missing in order to keep the audio alive when I go back to home page?
The following is my code
import 'package:flutter/material.dart';
import 'package:audioplayers/audioplayers.dart';
import 'package:audioplayers/audio_cache.dart';
import 'dart:io';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);
  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  var playMusicFlag = true;
  AudioCache audioCache = AudioCache();
  AudioPlayer advancedPlayer = AudioPlayer();
  bool seekDone = true;

  @override
  initState() {
    super.initState();
    AudioPlayer(mode: PlayerMode.LOW_LATENCY);
    if (Platform.isIOS) {
      if (audioCache.fixedPlayer != null) {
        audioCache.fixedPlayer.startHeadlessService();
      }

      advancedPlayer.startHeadlessService();
    }
  }

  Future loadMusic() async {
    advancedPlayer = await audioCache.loop("music/clocktick.mp3",
        mode: PlayerMode.LOW_LATENCY);
  }

  Future pauseMusic() async {
    await advancedPlayer.pause();
  }

  @override
  void dispose() {
    super.dispose();
    advancedPlayer.stop();
  }

  @override
  Widget build(BuildContext context) {
    // This method is rerun every time setState is called, for instance as done
    // by the _incrementCounter method above.
    //
    // The Flutter framework has been optimized to make rerunning build methods
    // fast, so that you can just rebuild anything that needs updating rather
    // than having to individually change instances of widgets.
    return Scaffold(
      appBar: AppBar(
        // Here we take the value from the MyHomePage object that was created by
        // the App.build method, and use it to set our appbar title.
        title: Text(widget.title),
      ),
      body: Center(
        child: RaisedButton(
          color: Colors.blue,
          textColor: Colors.white,
          child: Text("play"),
          onPressed: playbtn,
        ),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    );
  }

  playbtn() {
    if (playMusicFlag == true) {
      print("play music ${playMusicFlag}");
      loadMusic();
      playMusicFlag = false;
    } else {
      print("stop music ${playMusicFlag}");
      pauseMusic();
      playMusicFlag = true;
    }
  }
}

Have you tried to run without LOW_LATENCY mode enabled?

I added this in the plist file then it worked.
	<key>UIBackgroundModes</key>
    	<array>
    		<string>audio</string>
    		<string>remote-notification</string>
    	</array>

Would you be interested in sharing a bit more information if you found more info about this? If so, it would be great to add it in the README.md and create a PR


Unable to edit transactions on Chrome

Whenever i click edit I get the attached error message and need to refresh ynab

.

same here!

Same issue here. subscribing to this issue and disabling the toolkit until it's resolved
Chrome Version 103.0.5060.134 (Official Build) (64-bit)

Same issue -- I saw another thread reporting the same thing too

I'm having the same problem in Chrome!!!  Very frustrating to find that this isn't a "new" issue but has been going on now for SEVERAL days!!!

I'm having the same problem in Chrome!!! Very frustrating to find that this isn't a "new" issue but has been going on now for SEVERAL days!!!

Calm down, this is a free extension and you can easily work around it or disable the toolkit until the developer(s) have had time to fix the issue.

I agree, These guys are INCREDIBLE at resolving issues and applying fixes. Just disable the extension for some time until it's resolved.

Same Issue here, this has been ongoing for far too long. What gives?

Same Issue here.

FWIW, It's always good to take a look around at other issues already filed and pull requests. Looks like it's been reported on several issues and there's an open pull request that looks like the fix. Looks like one could build the extension applying that fix, if waiting for it to be merged and released isn't tenable.

samsies ...
macOS Monterey v12.4
Brave Version 1.41.100 Chromium: 103.0.5060.134 (Official Build) (arm64)

Hey @annamonomaniac the fix for this should be in our current release! If you notice the issue still exists and you're definitely running the newest version of the toolkit, please reopen! Thanks!
Please note, all browser can take up to an hour to provide updates to all users. You can force an update if it's available for you by going to chrome://extensions and clicking the "Update" button. Firefox users can go to about:addons, click the setting cog, and click "Check for updates...".


feature request - optionally mention corepack in the pnpm installation instructions

Thank you for creating this package, I found it suggested here: https://github.com/npm/cli/issues/2660#issuecomment-1764148955
Request
The current output does not mention the corepack alternative mode of installation:

If you don't have pnpm, install it via "npm i -g pnpm".

Would you be open to providing the option to suggest the corepack installation method instead?
https://nodejs.org/api/corepack.html
Context
With corepack, running yarn immediately results in this error:

Usage Error: This project is configured to use pnpm

But npm install still just works. ( I am not sure whether this is a deliberate corepack design decision. )
This is of course where this package comes in.
However, we now run the risk of developers installing pnpm globally, while they are expected to enable corepack and setup pnpm that way.

Hi,
I understand your concerns. Actually the output message was copied from the original only-allow package, and I wasn't aware of that.
Since there are many more methods to install pnpm (npm, corepack, Homebrew, Scoop, etc.) and different teams may have different conventions on how to configure package managers, maybe it would be better to provide a link to the official installation guide instead?
Use "pnpm install" for installation in this project.

If you don't have pnpm, go to https://pnpm.io/installation
and find installation method that suits your environment.

Thank you for your consideration and I think you make a good point and suggestion.

<EMAIL_ADDRESS>was released with the new message mentioned above. Thank you! ❤


chore: store wallet stuff inside a sub-dir

That's a bit cleaner in the directory

Not worth the hassle

Not worth the hassle

Actually no hassle at all, there is nothing to do. Worst case we have the wallet db twice if we don't clean up.

bors r+


doesnt work on tvOS

tested on tvOS 13 and 14. just shows a black screen. i fixed the bug where the sample video isnt included in tvos sample but it still just sits there displaying a blank view.

i take that back, there was another bug where the player was never getting initialized in the sample app.

i take that back, there was another bug where the player was never getting initialized in the sample app.

How u solved this??


JBQA-13277 - Jetty HTTP2 client proposal capable of storing pushed

resources URLs.

I've tried the code with Undertow and it fails for me with
 - Execution interrupted: null
org.eclipse.jetty.util.FuturePromise.get(FuturePromise.java:130)
jmeter.plugins.http2.sampler.JettyHttp2Client.sendRequest(JettyHttp2Client.java:60)
jmeter.plugins.http2.sampler.HTTP2Sampler.sample(HTTP2Sampler.java:295)
jmeter.plugins.http2.sampler.HTTP2Sampler.sample(HTTP2Sampler.java:189)
org.apache.jmeter.threads.JMeterThread.process_sampler(JMeterThread.java:434)
org.apache.jmeter.threads.JMeterThread.run(JMeterThread.java:261)
java.lang.Thread.run(Thread.java:745)

and less often with
- Execution interrupted: org.eclipse.jetty.io.RuntimeIOException: javax.net.ssl.SSLHandshakeException: Unrecognized SSL message, plaintext connection?
org.eclipse.jetty.util.FuturePromise.get(FuturePromise.java:138)
jmeter.plugins.http2.sampler.JettyHttp2Client.sendRequest(JettyHttp2Client.java:60)
jmeter.plugins.http2.sampler.HTTP2Sampler.sample(HTTP2Sampler.java:295)
jmeter.plugins.http2.sampler.HTTP2Sampler.sample(HTTP2Sampler.java:189)
org.apache.jmeter.threads.JMeterThread.process_sampler(JMeterThread.java:434)
org.apache.jmeter.threads.JMeterThread.run(JMeterThread.java:261)
java.lang.Thread.run(Thread.java:745)

Please ignore my last comment which I have deleted. It was caused by running the jmeter by mistake against 8080 (plain http)

I've noticed one strange behavior, when doing request http://localhost:8443/undertow-perf-test-app/dynamic.jsp, notice usage of http with port requiring ssl, then the plugin considers this successful samples. On the other hand when I use e.g. curl, I get curl: (52) Empty reply from server.
By looking at the jmeter plugin code and jmeter logs, I can see that the response in jmeter.plugins.http2.sampler.JettyHttp2Client.OpenStreamListener#onHeaders is 200. Maybe some jetty special behavior or bug? At least I would expect the request to fail in such case.

Actually it's a good point. I think the important question is how to handle response code in various cases. It looks like headers were received correctly in the case you described above but there was not data response (this case is currently not taken into account by the client)?Is it possible?
I need to investigate this. Thanks

Should be fixed now.


User request: blog auto save

https://lichess.org/forum/lichess-feedback/can-we-please-have-a-auto-save-feature-for-lichess-blogs
Since a blog can be saved as unpublished, I wonder whether a simple mechanism could be made to save the blog thus while writing.

Implemented in v2.3.79


sgx-tools: use VERSION file as the package version when running make package

Refer to the implementation for shim: https://github.com/alibaba/inclavare-containers/pull/1338

Has been fixed in pr.


AttributeError: 'datetime.date' object has no attribute 'tzinfo'

Hi, I get the exception below from inside 'radical' during parsing a ical file. A DTSTAMP value is parsed as datetime.date, but later vobject accesses the tzinfo attribute. tzinfo is only available for datetime-objects with a time component.
File ".local/lib/python3.4/site-packages/vobject/base.py", line 1125, in readComponents
     component.transformChildrenToNative()
File ".local/lib/python3.4/site-packages/vobject/base.py", line 674, in transformChildrenToNative
     child.transformChildrenToNative()
File ".local/lib/python3.4/site-packages/vobject/base.py", line 673, in transformChildrenToNative
     child = child.transformToNative()
File ".local/lib/python3.4/site-packages/vobject/base.py", line 198, in transformToNative
     raise ParseError(msg, lineNumber)
 vobject.base.ParseError: At line 8: In transformToNative, unhandled exception on line 8: <class 'AttributeError'>: 'datetime.date' object has no attribute 'tzinfo' (<DTSTAMP{'VALUE': ['DATE']}20170916>)```

This is actually because the spec says that the value of "DTSTAMP" MUST be a date-time value, that is, from section 3.3.5 of RFC 5545:

Format Definition:  This value type is defined by the following
notation:
  date-time  = date "T" time ;As specified in the DATE and TIME
                             ;value definitions


MI - INSTALL and SETUP section (Hazelcast for deployment)

Need to validate whether the hazelcast-related deployment instructions in [1] and [2] is applicable to EI 7 Micro Integrator.
[1] https://docs.wso2.com/display/EI650/Clustering+the+ESB+Profile#ClusteringtheESBProfile-Optional:ConfiguringHazelcastproperties
[2] https://docs.wso2.com/display/ADMIN44x/Troubleshooting+in+Production+Environments#TroubleshootinginProductionEnvironments-Checkingthehealthofacluster

On hold until Toml configs are finalized.

Done.
Clustering is not currently available for the MI.


Watermark Offsets

I am having troubles with the two methods in the Producer: GetWatermarkOffsets and QueryWatermarkOffsets.
From initial testing:
QueryWatermarkOffsets Returns

WatermarkOffsets

High

Value = Current Max Offset

Low

Value = Lowest Offset Available

GetWatermarkOffsets Returns

WatermarkOffsets

High

Value = Invalid[-1001]

Low

Value = Invalid[-1001]

I am just curious if I am not implementing GetWatermarkOffsets correctly, as well as if these methods are supposedly doing the same thing.

While QueryWatermarkOffsets() queries the broker for the requested offsets, GetWatermarkOffsets provides an optimized alternative that uses the latest cached known values for the High and Low watermarks, but this only works if the consumer is actively consuming the requested partitions.
If there is no cached watermark it will return the invalid/unset offset which is -1001.

I have to say that this is an awesome! Now that I understand the difference that is. Thank you for giving me an efficient feature like this!

@edenhill By latest cached values, do you mean the consumer lag ?

The high watermark offset will be the last stored offset on the topic/partition (not the current consumer position).
I'm going to mark this method as unstable in the upcoming release because I find it a bit confusing that the call doesn't depend on the specific consumer (and people, quite logically, often expect that it does). At some point we will likely have an Admin client (in addition to Producer and Consumer), and this functionality might be better restricted to being there.

@Dexu5 Each time the consumer fetches messages for a partition it will also receive the current high watermark of that partition - i.e., the offset of the last/newest message in the partition + 1.
The low watermark is the oldest message in the partition and there is no automatic functionality to get updated on when this changes so it is polled at every statistics.interval.ms instead - but since old messages are only removed when log segments are rolled over and retention policy kicks in the low watermark changes very infrequently (compared to the high watermark).
The consumer lag is the difference between the high watermark and the consumer's current position. So the main use of the cached GetWatermarkOffsets API is to allow a consumer application  to calculate its consumer lag in an efficient manner.
The non-cached QueryWatermarkOffsets API will ask the broker for the low and high watermark offsets and is thus much slower, the typical use-case is typically something more one-off:ish, like initiating a bsearch or similar.

@edenhill @mhowlett Thank you so much for clearing the doubts. This surely got me leads on further processing on my project.


My slide

created a branch, created a file and made a commit, and opened a pull request

created a branch, created a file and made a commit, and opened a pull reques

created a branch, created a file and made a commit, and opened a pull reques


Additional fix for #783

This fixes the issue #783  reported by @gregorypratt
In this case, parentage-values like '20%' should be kept as is.
@keyframes fade {
  20% {
    opacity: 1;
  }
  85% {
    opacity: 0;
  }
}

This is an additional PR to #785.

Coverage remained the same at 88.86% when pulling b79c64234ee4d8cdd46a62cdba7c4a1fd680b00a on cognitom:fix-css-keyframs2 into 9aa1f01796d1806f953f6b87018bede19e0b0f0e on muut:dev.

thanks


Bitcoingold support? Its been an year BTG

We are using trust wallet for a while.
But we are waiting for bitcoingold to be added in trustwallet as member of 200K community are waiting for it.
People find trustwallet great and easy way to store their asset we are waiting for bitcoingold support in it.
As we want to withdraw our coins their in it.
Your kind action in this matter will be highly appreciated.
Its been a while looking forward for it,
Kindly refer to someone else as telegram send to pull request
kindly send or assign to someone who take the responsibility to add and support asset
Thanks

Duplicate, #1909


Add sync/async responses for remote control status

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

New feature to add response codes for remote control status. Addresses #15

What is the current behavior? (You can also link to an open issue here)

Status can be set, but querying status does not pass back the response code, and there is no async support when notify: remote is enabled.

What is the new behavior (if this is a feature change)?

Adds a getter and a setter to the remote command. Adds an async handler.

Other information:

Codecov Report
Base: 53.73% // Head: 53.69% // Decreases project coverage by -0.03% :warning:

Coverage data is based on head (45ee0fc) compared to base (c76d0a1).
Patch coverage: 54.54% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
- Coverage   53.73%   53.69%   -0.04%
==========================================
  Files          36       37       +1
  Lines         804      825      +21
  Branches      133      133
==========================================
+ Hits          432      443      +11
- Misses        321      331      +10
  Partials       51       51

Impacted Files
Coverage Δ

src/commands/remote.ts
26.31% <36.36%> (+4.09%)
:arrow_up:

src/asyncHandlers/remoteInfo.ts
62.50% <62.50%> (ø)

src/asyncHandlers/index.ts
100.00% <100.00%> (ø)

src/codes.ts
98.14% <100.00%> (+0.07%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Not sure why the dependencies test is failing? Error log doesn't give a lot of clues and I've the PR doesn't change the dependancies.

It's just a dev dependency that needs to be updated, something for the maintainers to look at (me). I'll try to release this immediately.


Optional string might not work

Describe the bug
mo_string.OptionalString is not working as command flag
To Reproduce

dev test recipe -recipe "recipe name"

Expected behavior
Run a recipe test that given argument
Screenshots, or log snippets
n/a
Desktop (please complete the following information):

OS: macos
toolbox Version 66.dev

Additional context

Add tests on rc_value package


Issue 470: reportback quantity validations

What's this PR do?
Adds a library which filters a text string for numbers and spelled numbers, and converts those numbers to digits.
This library is added to the reportback flow, where the user-submitted response for the "quantity done" question is then filtered for digits before being submitted to the reportback API.
If a response with no numbers is submitted, the user receives an error message, which has been added to the reportback config do.
Where should the reviewer start?
npm test validates whether the user receives the message_quantity_sent_invalid message after submitting an invalid message.
The library contains its own tests, and its validity can be tested independently.
This functionality can also be tested with this postman collection.
How should this be manually tested?
npm test.
Any background context you want to provide?
When receiving a reportback submission through its API, the Drupal DoSomething app doesn't accept anything other than digits--"1", "2"--for the quantity of things done.
Previously, if a user submitted a non-digit response during the "quantity" question of the SMS reportback flow--i.e., if a user submitted "one" instead of "1"--the DoSomething app would interpret that as "0".
This became problematic during campaigns like "I Heart Dad". When users were asked "how many dads did you measure the blood pressure", a large percentage of users responded with "the only one I have" or "one".
What are the relevant tickets?
Closes #470.

:+1:


Upgrade testing for SQLite

How it works:

Download release version of Wakapi
Run tests against release version
Run current build of wakapi
If the service can come up, it means that migrations succeeded

This PR also changes run_api_tests.sh to have two optional arguments database type and upgrade testing, which defaults to sqlite and no upgrade testing respectively.
Part of #407

Great contribution! :pray: A few remarks:

[ ] Can you please add a wait_for_wakapi statement before the first test run?
[ ] Could you add another 1-2 seconds sleep to the wait function? I noticed that even when the instance is up, the first one or two test cases still fail due to connection reset
[ ] If one of the tests fails, the Wakapi process is not properly terminated, but remains running and thus blocks the port
[ ] The second positional argument "y" looks a bit non-intuitive for an outside user. Could you make it --migrations or so?
[ ] Minor one: add logging before the release downloading
[ ] Minor one: lower-case variable names (for consistency)

Awesome contribution, thanks you so much!


CI test to check if documentation is generated correctly

It would be nice to have a CI test that checks if the generated readthedocs documentation breaks.

See the new test here:
https://dev.azure.com/jpivarski/Scikit-HEP/_build?definitionId=5
The YAML for this test doesn't actually run the documentation workflow; you'll have to add that. And you might want to remove the compilation, too (now that I think about it), because the documentation needs to be buildable without compilation (so that it works on ReadTheDocs). That will take the running time from 2 minutes down to almost nothing.


.populate() creating empty associations

Account.find().populate('lastAssignment')
When I use populate to get the last assignment on the model below, it will also will also attach an empty array of assignments. If I populate assignments Account.find().populate('assignments') all is right with the world.
module.exports = {
  migrate: 'safe',
  tableName: 'accounts',
  autoPK: false,
  attributes: {
		id: {type: 'INTEGER', primaryKey: true},
		companyId: 'INTEGER',
		name: 'STRING',

		// Associations

		company: {
			model: 'company',
			columnName: 'companyId'
		},

		assignments: {
			collection: 'assignment',
			via: 'account'
		},

		lastAssignment:{
			model: 'assignment',
			columnName: 'lastAssignmentId'
		}

	}
};

Thanks for posting, @randallmeeker.  I'm a repo bot-- nice to meet you!
It has been 60 days since there have been any updates or new comments on this page.  If this issue has been resolved, feel free to disregard the rest of this message.  On the other hand, if you are still waiting on a patch, please:

review our contribution guide to make sure this submission meets our criteria (only verified bugs with documented features, please;  no questions, commentary, or bug reports about undocumented features or unofficial plugins)
create a new issue with the latest information, including updated version details with error messages, failing tests, and a link back to the original issue.  This allows GitHub to automatically create a back-reference for future visitors arriving from search engines.

Thanks so much for your help!


[r] Let SOMADataFrame.write accept Arrow RecordBatch in addition to Arrow Table

Context: #839

Fixed by #963


handle namespaces

passes through namespace config as specified in https://github.com/opencontainers/runtime-spec/blob/v1.0.0-rc4/config-linux.md#namespaces

adds a test for namespace sharing

fixes tests to clean up created containers

Awesome, thanks.


Add subword movement support

I will be fleshing out some ideas for #377 in this PR. Comments and feedback are welcome. When all of the main changes are made and you agree and are happy with them, I can clean up the commit history and add the necessary documentation etc.
TODO:

[ ] w movement

[ ] revert or re-fix https://github.com/atom/atom/commit/ba3ab41

[ ] b movement
[ ] e movement
[ ] w text object

atom/atom@ba3ab41 from atom/atom#7612 makes the subword regex match line endings if there's any trailing whitespace and line beginnings if it's followed by any whitespace. I currently worked around it, but I think the regex should be changed.

Can someone help me figure out why some of the tests fail when I use alt as a modifier. They seem to work just fine when actually used. I tried messing around with the way the spec-helper dispatches keyboard events, but to no avail.

This is a guess: it seems to me that CI is complaining about the presence of focused specs. Please try changing fdescribe to describe.

I like the procedurally generated tests with itMovesByWord and itMovesBySubWord, by the way!

I'd kept the focused specs because I hadn't really gotten them to pass on my own machine yet and wasn't worrying too much about the CI build. I added a commit to change that now, however, so that the CI logs would show the actual problem I'm facing.

Not working on this anymore


 Specifically check if CGO is on, and boringcrypto experiment is on, and in that case generate runtime dependencies on openssl libraries, fips config, and fips provider.
This will reduce number of by-hand declared runtime dependencies on the go-fips compiled binaries.
Functional Changes

[x] This change can build all of Wolfi without errors (describe results in notes)

It reduces manually specified runtime dependencies in many golang based -fips packages.
Notes:
SCA Changes

[x] Examining several representative APKs show no regression / the desired effect (details in notes)

The openssl-config-fipshardened is Chainguard-specific, however in practice it means "anything that will get one correctly configured OpenSSL fips provider" with no restriction as to which OpenSSL it is, or which fips provider build it is. Non-chainguard APK distributions can call that thing whatever they want, and add a virtual providers for it.
If this is deemed too specific, maybe we should add a more generic provides name; and use that in SCA; and have that match in Chainguard i.e. something fake like so:openssl-fips-provider and then use that everywhere.
W.r.t. detecting correct toolchain this should work for many go-fips builds from Chainguard, Microsoft, RHEL, Canonical. Thus it is universall-ish enough.
Notes:
Linter

[x] The new check is clean across Wolfi
[x] The new check is opt-in or a warning

Notes:
Test case added for new behavior using a binary that mimics a fips go binary close enough.
Developed on x86_64, should work on arm64 too.

TODO: somebody on a Mac, please try make test and let me know if it works correctly.


Display freelook speed and zoom in units in the 3D editor viewport

Now that the 3D editor grid is infinite and adjusts its step automatically, this helps people get a better sense of scale when moving around in the 3D viewport.
Since the infinite grid PR was merged, I've had trouble figuring out a scene's scale unless I had View Information enabled (which takes quite a bit of space on small displays). This should hopefully alleviate one of the downsides of using an automatically-adjusting grid.
This also fixes the indicator bar drawing on hiDPI displays.
Preview
The "u" stands for "units".
Freelook speed

Zoom center distance

Another style issue: https://travis-ci.org/github/godotengine/godot/jobs/698418563
I'd suggest setting up the clang-format pre-commit hook.

Another style issue: https://travis-ci.org/github/godotengine/godot/jobs/698418563

Bump.

Thanks!

Cherry-picked for 3.2.2.


The program (vectorAdd) hangs when page cache is oversubscribed

I did the test for page cache oversubscription with very simple setting:
Environment:
Linux 5.4.0-131-generic #147-Ubuntu SMP
CPU: Intel(R) Xeon(R) Gold 6226R CPU @ 2.90GHz
GPU: Nvidia A100 @ 1410 MHz
Driver version: 515.65.01
CUDA version: CUDA 11.7

benchmark/vectoradd/settings.h:
numThreads = 256;
maxPageCacheSize = 2097152;
n_elems = 131072;

The program would hang because
https://github.com/ZaidQureshi/bam/blob/ca34b6e3c8e62afb40a026a26411381e63abdfb8/include/page_cache.h#L1836
cnt never decreased to zero so the page could not be reused.
I also encountered this issue sometimes when I used the large size (like heisenbug...):
benchmark/vectoradd/settings.h:
numThreads = 256;
maxPageCacheSize =<PHONE_NUMBER>8;
n_elems =<PHONE_NUMBER>;

Is this still an issue, if yes, please reopen. Else I am closing this.


Streamline Xcode Debug

I recently updated our guide to reflect recent changes in Detox and be more all-around complete.
However, there are steps there that should be the default and shouldn't require setup. This should ease the process a lot.

Guide was updated a while ago and is now as easy as it can be. The rest is out of scope.


Do not reorder the tabs

The position of the tabs is jumping around.
A better solution would be to override the keyboard shortcut and manage the stack of tabs.

Closing this, as it's by design. In fact the planned direction was to do something very similar to enhanced-tabs. Reordering the tabs was way simpler, though.


logger callbacks

commit 761bb030fcb998dfe9915601a538f6087fb6d978
selftest: add log-callbacks

commit 56803f4389f7557a47eba7e7e78015cddefcf424
logger: introduce logger callbacks

loggerCallback() calls callbacks, log() and []logFnFilters(),
which can be set by the libbpfgo consumer via SetLoggerCbs().

This moves output filtering from C libbpf_print_fn() to
Go filterOutput() which can be set as one of the slice of filter funcs.

This also introduces LogWarnLevel, LogInfoLevel and LogDebugLevel
constants.

Fixes: #276
Fixes: #6
Fixes: #55
Fixex: https://github.com/aquasecurity/tracee/issues/2417

https://github.com/aquasecurity/tracee/issues/2417
#276
#6
#55

Related fix during the work:

#285

Tests
libbpfgo/selftest/log-callbacks on  2417-logger-callback [$!?]
❯ make
make -C /home/gg/code/libbpfgo libbpfgo-static
make[1]: Entering directory '/home/gg/code/libbpfgo'
CC=clang \
        CGO_CFLAGS="-I/home/gg/code/libbpfgo/output" \
        CGO_LDFLAGS="-lelf -lz /home/gg/code/libbpfgo/output/libbpf.a" \
        GOOS=linux GOARCH=amd64 \
        go build \
        -tags netgo -ldflags '-w -extldflags "-static"' \
        .
...

libbpfgo/selftest/log-callbacks on  2417-logger-callback [$!?]
❯ sudo ./main-static

libbpfgo/selftest/log-callbacks on  2417-logger-callback [$!?]
❯ echo $?
0

Changing one filter callback for tests purpose only (== instead of !=):
func(libLevel int, msg string) bool {
    return libLevel == C.LIBBPF_WARN
},

tracee on  main [$!?⇡] via 🐹 v1.19.5 via ⍱ v2.3.4 took 3s
❯ sudo ./dist/tracee -t uid=1000 -t comm=who --log warn 2>&1 | head -20
{"level":"warn","ts":1675175312.2060382,"msg":"libbpf: loading object 'embedded-core' from buffer"}
{"level":"warn","ts":1675175312.2061195,"msg":"libbpf: elf: section(3) raw_tracepoint/sys_enter, size 248, link 0, flags 6, type=1"}
{"level":"warn","ts":1675175312.2061718,"msg":"libbpf: sec 'raw_tracepoint/sys_enter': found program 'tracepoint__raw_syscalls__sys_enter' at insn offset 0 (0 bytes), code size 31 insns (248 bytes)"}
{"level":"warn","ts":1675175312.2062,"msg":"libbpf: elf: section(4) .relraw_tracepoint/sys_enter, size 32, link 223, flags 40, type=9"}
{"level":"warn","ts":1675175312.2062263,"msg":"libbpf: elf: section(5) raw_tracepoint/sys_enter_init, size 2368, link 0, flags 6, type=1"}
{"level":"warn","ts":1675175312.2062707,"msg":"libbpf: sec 'raw_tracepoint/sys_enter_init': found program 'sys_enter_init' at insn offset 0 (0 bytes), code size 296 insns (2368 bytes)"}
{"level":"warn","ts":1675175312.206299,"msg":"libbpf: elf: section(6) .relraw_tracepoint/sys_enter_init, size 160, link 223, flags 40, type=9"}
{"level":"warn","ts":1675175312.206323,"msg":"libbpf: elf: section(7) raw_tracepoint/sys_enter_submit, size 24904, link 0, flags 6, type=1"}
{"level":"warn","ts":1675175312.2063725,"msg":"libbpf: sec 'raw_tracepoint/sys_enter_submit': found program 'sys_enter_submit' at insn offset 0 (0 bytes), code size 3113 insns (24904 bytes)"}
{"level":"warn","ts":1675175312.2064219,"msg":"libbpf: elf: section(8) .relraw_tracepoint/sys_enter_submit, size 416, link 223, flags 40, type=9"}
{"level":"warn","ts":1675175312.2064548,"msg":"libbpf: elf: section(9) raw_tracepoint/sys_exit, size 312, link 0, flags 6, type=1"}
{"level":"warn","ts":1675175312.2065027,"msg":"libbpf: sec 'raw_tracepoint/sys_exit': found program 'tracepoint__raw_syscalls__sys_exit' at insn offset 0 (0 bytes), code size 39 insns (312 bytes)"}
{"level":"warn","ts":1675175312.2065287,"msg":"libbpf: elf: section(10) .relraw_tracepoint/sys_exit, size 32, link 223, flags 40, type=9"}
{"level":"warn","ts":1675175312.2065518,"msg":"libbpf: elf: section(11) raw_tracepoint/sys_exit_init, size 1200, link 0, flags 6, type=1"}
{"level":"warn","ts":1675175312.2065969,"msg":"libbpf: sec 'raw_tracepoint/sys_exit_init': found program 'sys_exit_init' at insn offset 0 (0 bytes), code size 150 insns (1200 bytes)"}
{"level":"warn","ts":1675175312.2066414,"msg":"libbpf: elf: section(12) .relraw_tracepoint/sys_exit_init, size 144, link 223, flags 40, type=9"}
{"level":"warn","ts":1675175312.2066822,"msg":"libbpf: elf: section(13) raw_tracepoint/sys_exit_submit, size 18976, link 0, flags 6, type=1"}
{"level":"warn","ts":1675175312.2067318,"msg":"libbpf: sec 'raw_tracepoint/sys_exit_submit': found program 'sys_exit_submit' at insn offset 0 (0 bytes), code size 2372 insns (18976 bytes)"}
{"level":"warn","ts":1675175312.2067814,"msg":"libbpf: elf: section(14) .relraw_tracepoint/sys_exit_submit, size 416, link 223, flags 40, type=9"}
{"level":"warn","ts":1675175312.206826,"msg":"libbpf: elf: section(15) raw_tracepoint/trace_sys_enter, size 8120, link 0, flags 6, type=1"}

Context

https://github.com/aquasecurity/tracee/pull/2600

LGTM after the small changes I requested (if you agree).

@rafaeldtinoco thanks for reviewing this.

As aquasecurity/tracee#2600 (review) concluded, we'll tackle this creating a libbpfgo.Logger interface.

Returing to this soon.

Despite having passed all the tests, I need to define the callbacks on them to make them silent.

Despite having passed all the tests, I need to define the callbacks on them to make them silent.

Solved. Ready for review.

@geyslan minor nits and then we merge. LGTM.

After merging this and https://github.com/aquasecurity/tracee/pull/2663, I'll open an issue in both projects to tackle the hard-coded filters move.

Feel free to undraft tracee's change and move on there. Thanks!


Modifications to support simple CGRA RTL->Layout.

Modifications to support the CGRA-Flow generated RTL to Layout.

Hi @yyan7223, let's update this and submit?

Sorry I forget last week, and I have updated the code as you required.


Support overriding the SCM type and server URL

Changes
When using the git resolver it was only possible to use one single provider across cluster.
If the user wanted to mix different providers to fetch the task from they would not have been able to do so.
With this change we now support overriding the SCM type and server URL as a parameters in the PipelineRun for the user to pass.
Fixed bug #7448
/kind feature
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[X] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[X] Has Tests included if any functionality added or changed
[X] Follows the commit message standard
[X] Meets the Tekton contributor standards (including functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
[x] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
User are now able to override the global server URL when using the git resolver to allow fetching from multiple git providers.

/lgtm


Update Dependencies

Here's an updated package.
package.json

thanks for that ... i'll take a look at it and test it out over the weekend... I'll try to get a version up by next week.


Attempted to call an undefined method named "GetSubObjects" of class "MangoPay\PayInWebExtendedView".

public function GetExtendedCardView($payInId) { return $this->GetObject('get_extended_card_view', $payInId, '\MangoPay\PayInWebExtendedView'); }

Hi @gaelcancre
This method seems outdated, can you reload the latest version of the SDK. Let us know of any issues.
Regards,
Max


Frontend for the Member Portal Django CMS

Need to create a basic template for the Member Portal Django CMS Templates.

I finished the projects template for cms

(2 hours)

Planned Hours: 5 Hours
End Date: July 12th, 2021

This task is not ready yet, i have some enviroment problems i cant even pull or push the code, i can't login into member portal, i can't create superuser, i talked with Peter today but we didn't find any solutions yet.

Toni will be trying to fix his computer tomorrow.


[AddressLowering] Map-into-context type for yield.

Previously, the type for the storage into which a loadable value which was yielded via an indirect convention was obtained from the SILFunctionConventions and the SILYieldIfo but not mapped into the context of the current function.  Here, that's fixed.

@swift-ci please test


Fix Notebook JSON format

Fix error "Notebook does not appear to be JSON".

Thanks @albertvillanova. I'm really curious how this happened.... I'm going to look around a bit then merge this.

Ah I see, from the last marge I did on this. Some how it got in there without the new line. Thanks!

Welcome! ;)


Enterprise deprecated

The enterprise is deprecated in the DSL and should be replaced with groups.
Could someone hint to all the places which might need changes for it to work?

Thanks for raising this! A good ol' search and replace should work pretty well here given that Enterprise is used mostly as metadata. Roughly the DSL has to be added (similar to this), then the group expression needs to be added (that the new group DSL would instantiate, similar to this), the ModelizeDesign function in the mdl package needs to be updated to take this new field into consideration. We should also probably test to make sure that the design serializes correctly for the structurizr service.

@beffge nudge: any chance you might be able to take a look?

Hey, sorry for responding so slow on your comment. I can't promise anything here timeline wise but it is till on my radar.

I just thought I'll look into this again. But I got a bit confused.
To me it looks more like the DeploymentEnv i.e. in the resulting JSON just a string and during modelling as a function. Difference is just, it can be at different locations as well as it can be nested.
With regards to being nested, it requires
        properties {
            "structurizr.groupSeparator" "/"
        }

for which I didn't find anything similar. How would you approach this?

OK, I tried to add it but got lost 😓 It seems like, since group can be part of Design, SoftwareSystem and Container as well as nested, it's more way more than just the existing Enterprise. I started adding it, but I was just assuming what I had to do, since I don't use anything but the mdl package. I'm pretty sure any PR I'd create would require a lot of discussion and rework i.e. it will be faster and more efficient if someone which uses the "other" parts implements it.

Fair enough! I'll keep the issue opened so one of us can take a look at some point.


use static loggers for principal attribute repository config

Closes #1736
use static loggers for principal attribute repository config

Also fix the other compile errors plz.

Great! thanks.

Will port fwd to master.

you're welcome


chore: update hermetic build docker volume usage

Follow up of https://github.com/googleapis/sdk-platform-java/pull/2615
The way we use the docker image changed with that pull request. Here we adapt to those changes.

Confirmed locally that generation/hermetic_library_generation.sh worked:
Library postprocessing finished successfully
Regenerating root pom.xml
Regenerating gapic-libraries-bom
Configuration diff:
28c28
<     personnel. - test2
---
>     personnel. -test2
[update-library-generation-docker-volume-usage e01a95db362] chore: generate libraries at Wed May  1 05:54:59 PM UTC 2024
 2 files changed, 3 insertions(+), 3 deletions(-)
Enumerating objects: 9, done.
Counting objects: 100% (9/9), done.
Delta compression using up to 24 threads
Compressing objects: 100% (5/5), done.
Writing objects: 100% (5/5), 486 bytes | 486.00 KiB/s, done.
Total 5 (delta 4), reused 0 (delta 0), pack-reused 0 (from 0)
remote: Resolving deltas: 100% (4/4), completed with 4 local objects.
To https://github.com/googleapis/google-cloud-java
   1d3caed6359..e01a95db362  update-library-generation-docker-volume-usage -> update-library-generation-docker-volume-usage

From https://github.com/googleapis/google-cloud-java/actions/runs/8913106957/job/24477945575?pr=10770
docker: Error response from daemon: failed to create task for container: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: exec: "--baseline-generation-config-path=/workspace/baseline_generation_config.yaml": stat --baseline-generation-config-path=/workspace/baseline_generation_config.yaml: no such file or directory: unknown.

This is because 2.39.0 has a different interface. We need to wait until 2.40 is released to perform this change

Included in https://github.com/googleapis/google-cloud-java/pull/10758


feat: added layout on pages with Outlet component from react router

added layout on pages with Outlet component from react router

Thanks @Nischaymrzn


Header

I recommend including the logo, which in the header, in the side navigation, like this layout:

@ShiranHi I recommend using the primary navigation pattern.
@bennypowers Can @ShiranHi use our new footer, but in Hebrew?


solved issue #47

Added a scroll to top button. please review the or and merged accordingly under gssoc-extended and hacktober fest accepted levels.
Uploading Recording 2024-10-06 095704 (1).mp4…

@Devanshu1603 what is your issue no?

issue no #47


cpp_utils/BLEAdvertising ctor missing peer_addr_type for esp-idf v3.2

I was getting errors using the BLE classes after updating to esp-idf 3.2 ("Invalid advertisting peer address type parameters").
it's solved by adding the following line in BLEAdvertising::BLEAdvertising():
m_advParams.peer_addr_type = BLE_ADDR_TYPE_PUBLIC;

Thanks for info, looks like its been implemented as per request:
https://github.com/espressif/esp-idf/issues/2714

Should be fixed in last commit.

I think you missed BLEScan.h from the commit. I'm missing definitions for BLEScan::getResults() and BLEScan::clearResults()

@rkone Fixed, thanks.

Thank you for the updates! Now to figure out what else got broken migrating to 3.2...


SimpleOrders action scheme on the Github version works unexpected

I am using the github version: pip install git+https://github.com/tensortrade-org/tensortrade.git. This addresses the plotly keep showing problem. But, a new problem emerges as the new training results are vastly different from previous well-trained results (old results are using the package from pip install tensortrade). It seems that the reason is the action scheme doesn't use any sell actions in training. The following two screenshots are training results of the exact same code with the same parameters. The first is the new training result, the second is the old training result. The action scheme I used is action =SimpleOrders(trade_sizes=10)
New result (unexpected): https://flic.kr/p/2kHaYY2
Old result (expected): https://flic.kr/p/2kHeABo

I think that the problem exists in get_orders function.
The size of quote should be changed to the base standard.

Thanks. Has this been fixed?

Yes. agent can use sell action.

That's great. Thanks!


Create config.json

The FIWARE Foundation need a simple JSON file available in a standard location in each repo to be able to continue to make FIWARE Releases and improve the degree of cross-product integration testing that can occur. This change has been agreed by the TSC and has been added to the contribution requirements.
Please suggest or amend values where known.

Superseded.


How to get a filedescriptor？

If you use the native epoll/kqueue transport you can use channel.fd() to get the file descriptor. Otherwise its impossible as there is not public API in the JDK


Banker and a Sloth Kata

A coding kata for translating stakeholder definitions into software requirements.

The instructions at the top and the conversation at the bottom don't really seem to align. And I can't see what this has to do with any of it? http://sunnytree.org/openup/

Ok, thank you for the feedback.


Update cli argument parser to warn about ignored/invalid arguments instead of exit

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [x] feature request

Command (mark with an x)
- [ ] new
- [x] build
- [x] serve
- [x] test
- [x] e2e
- [ ] generate
- [ ] add
- [ ] update
- [x] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Versions

node -v; npm -v; ng v
v10.14.1
6.4.1

     _                      _                 ____ _     ___
    / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
   / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
  / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
 /_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
                |___/

Angular CLI: 7.1.0
Node: 10.14.1
OS: win32 x64
Angular:
...

Package                      Version
------------------------------------------------------
@angular-devkit/architect    0.11.0
@angular-devkit/core         7.1.0
@angular-devkit/schematics   7.1.0
@schematics/angular          7.1.0
@schematics/update           0.11.0
rxjs                         6.3.3
typescript                   3.1.6

Repro steps

Run any cli command with an argument that command should not be able to understand, even when that command builder was changed.

Using @angular-builders/jest:
ng test my-app my-component.ts

The log given by the failure

Unknown option: my-component.ts

Desired functionality

I would like this to be parsed by the builder, as the builder would/should understand that argument.
I understand why the change was introduced, as if you pass an invalid argument to a builder command, or a tester command it's most likely an error, but I really think you should take in consideration the number of issues opened by this stricter parser. (#12305, #13078, Sure there are several others).
I am NOT asking to support passing this arguments to the default builders, as #11408 or #12299.
Mention any other details that might be useful

Hi there,
The actual parser does support positional arguments like ng test my-app my-component.ts If they are declared in schema.json.
Example;
"name": {
	"type": "string",
	"$default": {
	"$source": "argv",
	   "index": 0
	}
},
"file": {
	"type": "string",
	"$default": {
	"$source": "argv",
	   "index": 1
	}
}


The test and training accuracy keeps on changing with the same collected data.

I have made sure that the training images and the training data are the ones from my recent collection(not the old ones). I got the training accuracy of 89% and test of 60 percent. When I re-ran the mlp_training.py, the accuracies dropped significantly to 60 and 43, respectively. Now, when I ran the code for the third time, the accuracy was again dropped to 76 and 36. What might be the reason for this?

When you train the model, parameters (weights) are randomly initialized, and the data is shuffled into train and validation set. You probably need more data, collect more images and give it a try.


Add support for searching published bindings by controller

Not the prettiest UI, but it works.

Many thanks for this! I think the UI is perfectly usable. First make it work, then make it pretty.
I manually rebased to amend tests and docs before merging.


AUTHENTICATION_USERS does not work

hey,
i have find your Application by YouTube-DB Tech and i have test it.
Sorry but the funktion AUTHENTICATION_USERS does not work.
my docker compose file is:
services:
enclosed:
image: corentinth/enclosed:1.6.3
container_name: enclosed
ports:
- 8787:8787
volumes:
- /mnt/enclosed/data:/app/.data
environment:
- SERVER_CORS_ORIGINS=https://xxx.yy
- AUTHENTICATION_USERS=/User Authentication Key Generator/
- AUTHENTICATION_JWT_SECRET=change_me
restart: unless-stopped
I have no logon.

Hello @thokich, you have to set the env variable PUBLIC_IS_AUTHENTICATION_REQUIRED to true to enable authentication
See here https://docs.enclosed.cc/self-hosting/configuration

Hy, thanks for your fast help.
sometimes you can't see the forest for the trees.
It's a german saying.
I will see what nice features you will add in feature.
Thx from Germany


Citation information is needed

I would like to refer to your project in my research. How could I cite your work?

Check out the following feature which may be helpful:
https://twitter.com/natfriedman/status/1420122675813441540?t=1znNpKkPvr7KUM8q4m4ffw&s=08


Feature request: Indentation

Would it be possible to make Admonition invokable with indents rather than backticks?

Actually, maybe. Let me look into it. But I would recommend using the MSDoc syntax.

Here's the thing though: There's a lot of non-coders who would like to use indents for formatting rather than code blocks. So ideally I'd like the default admonition to be invoked with only an indent, and multiple indents to create nesting.

This can now be done with the Microsoft Document Syntax.

I'm not sure how this works. Plain indents are still code blocks, not Admonition blocks.

This is how you do it:
    [!quote]
    This is an admonition!

Please read through the section I linked to in this comment for the syntax.

Thanks, but that’s not quite the same thing as the default admonition being able to be invoked with only an indent.


Additional quote symbol is added to rule when saving

I created a new rule copied from existing rule for replacement:
function(plugin){     var n = plugin.replaceMonomer(org.helm.webeditor.HELM.BASE, 'P','sP');return n>0; }
You can see it on the screenshot:

after being saved, you can see the following code with additional quote around the monomer codes:

When trying the apply that rule, these new quote symbol cause an error message:

this issue does not exist with the other replacement script using single quote. I also tried editing the code but the new quote symbol are still added.

I believe this is an issue on the server side.
Markus, could you check it out?

I believe the quote issue has been resolved. Yohann, please can you check?

This issue is fixed


In DIAL XL users can not use application from DIAL Marketplace

EPAM AI DIAL chat version
0.22.0
How to reproduce
Steps:

open [DIAL XL]
open chat in overlay
click DIAL Marketplace
click qg 2.0 card
click Use application

Actual result
Actual result: "User application" button is not clickable
also Bookmark button and x-button are not clickable, but looks like enabled
Expected result
DIAL XL users should be able to select model/application though DIAL Marketplace
Confidential information

[X] I confirm that do not share any confidential information

verified on staging successfully


autograd.numpy.round() gives always 0 as gradient

def foo(x):
  return autograd.numpy.round(x**2,decimals=12)

grad_foo = autograd.grad(foo)
print(grad_foo(2.)) # gives 0.0 but should be 4.0

Maybe I'm missing something, but why do you think it should give a non-zero value? The function is flat almost everywhere.

It's a quadratic function, the gradient is linear (2*x)

Hm, I originally missed the decimal=12 part, and was going to agree with you, but I'm not sure now. Why do you need to round to the 12th decimal point? Even with this condition, the function is still flat AE. I'm guessing the decimal=12 is to test what happens in the limit, as decimal -> inf => round should like linear?

The round() was in a existing code I applied to autograd. Once I identified round() as the issue I got rid. The background was an expression to find the angle between two vectors by cos_alpha = v1.dot(v2) / (norm(v1) * norm(v2)) which numerically exceed slightly [-1,1] and then arccos(cos_alpha) failed. Clearly rounding was not the best way to fix it (but this was the code).
Now I see what you mean with flat - round(f) is a piecewise constant function an yes, having the gradient 0 is the correct answer. Thanks for pointing it out - I missed it, as finite differences did what I wanted.
So it's not a bug but a correct implementation :) I'll close the issue.

Was not a bug

For constraining a value to be between [-1, 1], take a look at np.clip!


Investigate using alternatives to mapbox

what are the main issues/goals for this?

Ideally, we would have the ability to swap out map packages and keep everything working. This may not be reasonably possible or there may be restrictions, like staying within the deck GL framework.

This ticket is more investigative as to if it is reasonably possible and if we want to add support for multiple mapping packages.

At first glance, it seems that the easiest alternatives, staying within DeckGL, would be ArcGIS, Carto, Google 3D Tiles, and Google Maps


Add simple API to run migrations

Originally posted by @benjaminpkane in https://github.com/voxel51/fiftyone/issues/824#issuecomment-769216125:

The only problem is that the down migration will not run for users.
The only way to get down migrations to run for users is to introduce logic in the migrations runner that downloads "future" revisions from the web at runtime. A v0_7_2.py revision will never exist in a 0.7.1 install...

I haven't thought deeply about this, but what about introducing an API that lets the user run a quick command to upgrade or downgrade their DB to any desired package version. Then they install the appropriate version of the tool:
# I'd like to downgrade
fiftyone --version
# 0.7.2

# Let me prep my DB for this
fiftyone migrate --version 0.7.0

# And now I can downgrade
pip install fiftyone==0.7.0

This is manageable. Shouldn't take more than a few hours to stand something up.

This is manageable. Shouldn't take more than a few hours to stand something up.


feat(react-query): expose "rootKey"

Description
Addresses #18
Type of change

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Environment:

OS: mac Monterey 12.4
NodeJS: 16.17.0

Checklist:

[x] I have followed the CONTRIBUTING doc and the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Further comments
This PR introduces and exposes a .rootKey value on the hook, to allow flexibility in accessing the static query keys of the generated functions. See discussions in #18
For example, you can now easily invalidate all the queries of your hook with:
queryClient.invalidateQueries({ queryKey: [useMyQuery.rootKey] })

Ah I need to also change the tests in dev-test/githunt/types.react-query.ts.

Hi @bitttttten,
yes, for this, you need to run the following command locally and commit the diff:
yarn generate:examples

Thanks!

any update to this? I don't mind to take over this PR if this is an abandoned child

@kambing86 it's abandoned for now indeed. So you're more than welcome to take it over.

@kambing86 Do you think you'll take this over? It would be a useful feature

Hi, is it ok to wait for the merge now? Or does he still have issues to be resolved?

I created a PR https://github.com/dotansimha/graphql-code-generator-community/pull/327 to replace this PR

#326 was merged


Create banner v2

Ultimately I would like to work with Stockard on a nicer banner.

https://sagebionetworks.jira.com/browse/DESIGN-377

Done


add DefaultDataSubCategoryType

Related Issues

T-13720 Schema classifier service
T-14003 Default data subcategories
T-13996 Enumerate categories and test them
closes T-14018

Security Implications
[none]
System Availability
[none]

Given this is a public repo, do we want to have links to Height in the TODOs? These TODO links aren't sensitive but maybe some others will be, not sure

I wasn't sure about that either. @michaelfarrell76 or @anotherminh , what do you think? Should I also remove those links from the Related Issues section above?

I'm ok with leaving them honestly. If we're serious about making these open-source then i think the associated issues/tasks should be open-sourced too (i know they're not right now bc Height requires auth but the spirit of it is that it should be OK for the public to discover i think)

@MadDataScience @anotherminh @LuGary good q. i dont have a strong opinion. i agree that it would be nice if the tickets were viewable, but i also feel like its much easier to track in height and itll get done faster if we track it there alongside everything else.


Update README.md

Resolve #1  issue
Solve
Added screenshot of the website in Readme file.
Screenshot

Looks good... 🔥
Thanks for contributing!


crypto/tls: Failed handshake should not send any data to client

What does 'go version' print?

go version go1.3 windows/amd64

What steps reproduce the problem?

Snippet: http://play.golang.org/p/eu_6CStY3k

1. Run the snippet above
2. Browse to http://localhost:8181

What happened?

Depending on platform or browser different things might happen.
If the browser thinks the data looks binary, it usually starts a download.
If the data looks plain-text'ish, it outputs bytes to the browser.

What should have happened instead?

The connection should have been closed without any data sent, since it doesn't get past
the TLS handshake due to protocol mismatch.

Or alternatively, a feature specific to net/http using TLS could downgrade the protocol
to HTTP and deal with it in some way.

Please provide any additional information below.

The culprit is the following line:
https://code.google.com/p/go/source/browse/src/crypto/tls/conn.go#582

If commented out, it produces the expected "No data received" error on Chrome.

Fixed by 2a8c81f amongst others.


OpenTelemetry support + fix for headers

note this is a breaking change
correlation id can be set by the client application, however opentelemetry is preferred.

#1 - should be supported by this


Update 1.4.4g appears to cause game to get stuck on first loading screen with mod

Owlcat pushed an update this morning. Game is now getting stuck at the loading screen. Did some experimenting and it looks like it's TabletopTweaks-Base that's causing it.
With mod disabled

With mod enabled

Thank you for all your amazing work Vek!

i have the same issue

You can disable Bloodlines.BloodlineRestrictions and it will work for now. Patch will be out when I get off work in a few hours.


Tests: Add Run Tests

Add smoke tests that run an example.

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Addressed with https://github.com/conda-forge/openpmd-api-feedstock/pull/76

@conda-forge-admin, please rerender


I'm a schoolboy,  learning NFC

it's my ETH address: 0x72C886493f8E716635dDD164D5a655FdABb6f18b , please reward, thanks

Please never visit Github again.
Thanks.


Remove the default policy server.

Description
Removes the default Policy Server installed in the post-install hook. This is necessary because the Kubewarden controller should be running when Helm try to install the Policy Server. However, sometimes the controller is not fast enough, causing an error in the installation. Furthermore, installing the default Policy Server in a post-install hook prevent the user from upgrading the resource using Helm
Fix #52

I think it should work if we just remove the post-install hook and leave the default PolicyServer. We needed the post-install hook when we had just one helm chart as sometimes it tried to create the default PolicyServer before the CRD, then it failed.
That should not happen anymore because there is a separate chart for CRDs. I don't see a problem if the default PolicyServer is created before the controller is running

I have some doubts about removing the default PolicyServer. I fear this can degrade the user experience.
This changes if:

this is the only way to ensure upgrades via helm chart
the race condition described above happens pretty often

Maybe we should reach out to other helm experts we have at hand, like Matt

This is necessary because the Kubewarden controller should be running when Helm try to install the Policy Server. However, sometimes the controller is not fast enough, causing an error in the installation

It is fine if the default PolicyServer is created before the controller. Controller will see it once it starts. Unless I'm missing something? This was a problem before when we had CRDs in the same helm chart.

I have some doubts about removing the default PolicyServer. I fear this can degrade the user experience.

I'd leave the default PolicyServer. Removing it would add an extra step in the installation process.

It is fine if the default PolicyServer is created before the controller. Controller will see it once it starts. Unless I'm missing something? This was a problem before when we had CRDs in the same helm chart.

No, it's not. Our Helm chart has a web hook for the PolicyServer kind. Which means that every time you deploy a Policy Server you need the controller running and able to respond. That's the problem. When we are installing the chart, we try to install a default policy server. That's means that the controller must be running and ready to receive request from kube-api. There are some situations like described in https://github.com/kubewarden/helm-charts/issues/52 where the controller is not receiving requests for any reason. Which cause our Helm installation to fail.

Maybe we should reach out to other helm experts we have at hand, like Matt

Okay, I'll do that.

No, it's not. Our Helm chart has a web hook for the PolicyServer kind. Which means that every time you deploy a Policy Server you need the controller running and able to respond. That's the problem. When we are installing the chart, we try to install a default policy server. That's means that the controller must be running and ready to receive request from kube-api. There are some situations like described in #52 where the controller is not receiving requests for any reason. Which cause our Helm installation to fail.

 I missed the webhook. Sorry about the noise

what if we add a pre-install hook to the default PolicyServer? then it would be created before the webhooks

I think the name kubewarden-policy-server for the chart might be a bit misleading. Some users might think it installs the PolicyServer CRD instead of the default PolicyServer. Maybe something like kubewarden-default-policy-server or kubewarden-default-resources if we plan to add some default policies in the future. What do you think?

I'm glad that you propose the name. I agree with you and I was originally trying to find a better name. But I did not find a better one. I like the kuberwarden-default-resources name. I'll update the chart name asap.

How about kubewarden-defaults as a name, for it to be shorter?

I added some default policies to be installed when the user enabled a configuration. Please, review it again.
Do not merge the PR yet. I need to check a possible bug when installing ClusterAdmissionPolicy and AdmissionPolicy together.

I've looked at how the policies are then defined and I'm not convinced about the current approach.
I think we should go with only one of these two approaches:
* define cluster wide policies and have an "exclusion list" with some default values.

* define namespaced policies and have a list of namespaces where the policies have to be created.

I think that having both is just too confusing and also requires more business logic inside of the helm chart (which is always hard to write and maintain due to "limitations" of helm).

I see. I would say to use namespaced policies then. This way we ensure that the user must thinks and defines namespaces where the policies are applied. Furthermore, it avoid causing issues in the whole cluster by accident.

Thanks @jvanz!
Requesting changes to remove the -namespace suffix from policies.
My two cents: given we don't know how many namespaces a user might have when installing Kubewarden, and given that it's less handy having to list them, so they can install Kubewarden pointing to a list of namespaces in policies.
Also given that this would imply that they would have to do something in order to activate policies on newly created namespaces, I think is a better approach for a first installation to go with Clusterwide + namespace reject list (e.g. maybe initially kube-system + rancher specific namespaces)

Sorry guys, I totally forgot the namespaceSelector field in the  ClusterAdmissionPolicy . Which allows us to apply policies in the whole cluster skipping some namespaces. My bad.
Should I update the PR to use the "reject list" approach?

I would go ahead with this solution.

Done

I added some default policies to be installed when the user enabled a configuration. Please, review it again.
Do not merge the PR yet. I need to check a possible bug when installing ClusterAdmissionPolicy and AdmissionPolicy together.

This PR can be merged. The problem was in the yaml creation.


[Question] How can I define custom EventType for a sub class of goog.ui.Component?

I've defined custom EventType for a sub class of goog.ui.Component like following:
goog.provide('a.b.c.SomeClass');
goog.provide('a.b.c.SomeClass.EventType');

goog.require('goog.ui.Component');

a.b.c.SomeClass = class extends goog.ui.Component {
  /* some codes... */
}

/**
 * @enum {string}
 */
a.b.c.SomeClass.EventType = {
  CUSTOM_EVENT_A: '...',
  CUSTOM_EVENT_B: '...',
};

But, when I updated Closure Libray and Compiler to v20190709, the above code becomes causing JSC_HIDDEN_PROTOTYPAL_SUPERTYPE_PROPERTY_MISMATCH and JSC_HIDDEN_SUPERCLASS_PROPERTY errors.
How can I define custom EvenType for a sub class of goog.ui.Component with ES2015 class inheritance?

The short answer is you can't.  This is actually a known issue that's blocking us migrating many of the goog.ui classes to ES6.
The underlying issue is that ES6 classes have class-side inheritance, and Closure Compiler (and TypeScript as well, FWIW) seem to want to enforce that static properties on a subclass are assignable to static properties on the base class.  In my opinion, this is incorrect, since subclasses are not generally substitutable for base classes (see https://github.com/microsoft/TypeScript/issues/4628#issuecomment-696436047 for a familiar example).  But that's beside the point, it's unlikely to change any time soon.
Instead, the approach we recommend is to make this a module with "named exports".  That is,
goog.module('a.b.c.someClass');

exports.SomeClass = class extends goog.ui.Component { ... }
/** @enum {string} */
exports.EventType = { ... }

This avoids needing to nest the enum on the class, which our internal TypeScript infrastructure doesn't even really support anyway (so there's another reason we're moving away from that pattern).
More broadly, I'd also recommend getting off of goog.ui entirely.  It's been effectively unmaintained for years, and we continue to get more and more hesitant to touch anything about it as time passes.

@shicks Thank you for answering!


Adding TITLE attribute when inserting external link breaks the link

When inserting an external link, if the URL and link text fields are supplied, that is fine, but if the title field is also supplied, the output for the link is incorrect, resulting in double-quotes in the wrong place and resulting in no 'title' attribute as would be expected.

I think this issue belongs in prose/prose.
@ctxlken can you please open the issue there and be very specific about 1. what text input you're using, 2. what you expected it to produce, and 3. what actually came out.


Angular user sevice cache user details in browser

Description:
Angular user sevice cache user details in browser (and not shot to backend every time we need to know something about user).
Current behaviour:
There is no cached user details available.
Expected behaviour:
User details are available in browser somehere in user service object.

Implemented in #600


refactor: Apply webpack-defaults

[x] Apply webpack-defaults tooling updates
[x] Apply webpack-defaults grammar updates
[x] Apply webpack-defaults style updates
[x] Convert existing tests to Jest from Mocha

The desired effect here is to update style & grammar without changing the way etwp works.
Note, this is being merged into #540 making this refactoring etwp & webpack v3 only
Closes #477

Thanks Tobias, I'll get everything updated this evening.


Npm scripts

Add npm scripts so don't have to mess around with aliases.
2.0 is looking 👍

@shinnn nice!! Updated with shorthand paths.
Looks like karma is slightly different and binary is under ./node_modules/karma/bin/karma

hmmm... not a bad idea... not bad. I feel a little weird though, having a project called "gulp-starter" but not using the gulp command. I'll have to think on this.

Good call. Addressed in #164 Thanks!


correct fixed width counter

solves issue #560 correctly
previous PR didnt fix the correct element.
Also we had to counter elements, modified to have only one

@LiavGut please check this PR which has the latest counter fixed width
https://deploy-preview-617--good-dapp.netlify.com/

@LiavGut
currently its set on 20px width per character

@LiavGut
currently its set on 20px width per character

@sirpy
This looks great. thanks.
So just need to fix the font issue of the button.

@liavgut What font issue? the counter is roboto slab. is the text not the
right font?

@LiavGut What font issue? the counter is roboto slab. is the text not the right font?

The font on the light blue button (the last part of it - the G$ and the number) are in Roboto Slab

@LiavGut


cert-manager loop if an ingress with two hosts but twice the same secret is provided

Describe the bug:
cert-manager hang if an ingress with two hosts but twice the same secret is provided.
Example ingress:
spec:
  tls:
  - hosts:
    - endpoint1.test.com
    secretName: both-use-the-same
  - hosts:
    - endpoint2.test.com
    secretName: both-use-the-same

This will block the cert-manager instance handling this ingress with an infinite loop of:
I0920 14:00:15.759615       1 controller.go:168] ingress-shim controller: syncing item 'example/example-ingress'
I0920 14:00:15.759640       1 sync.go:140] Certificate "both-use-the-same" for ingress "example-ingress" already exists
I0920 14:00:15.759664       1 sync.go:140] Certificate "both-use-the-same" for ingress "example-ingress" already exists
I0920 14:00:15.759678       1 sync.go:143] Certificate "both-use-the-same" for ingress "example-ingress" is up to date
I0920 14:00:15.953745       1 controller.go:182] ingress-shim controller: Finished processing work item "example/example-ingress"

Expected behaviour:
One of the following:

Cert-manager refuses the ingress as it cannot handle it
Cert-manager creates a SAN Certificate instead and maybe a warning as this doesn't seems like a good practice (I guess it would fail again on several ingress with the same secret anyway so this might not be a good idea)
Cert-manager retries as it currently does but does not block the whole instance
Cert-manager fails just like it currently does but the problem is documented and it is a wontfix

Steps to reproduce the bug:
Push an ingress with this spec for tls:
spec:
  tls:
  - hosts:
    - endpoint1.test.com
    secretName: both-use-the-same
  - hosts:
    - endpoint2.test.com
    secretName: both-use-the-same

Anything else we need to know?:
I tried to find any issue with the same problem / documentation stating that this does not work but couldn't find anything.
Environment details::

Kubernetes version (e.g. v1.10.2): v1.15.3
Cloud-provider/provisioner (e.g. GKE, kops AWS, etc): ---
cert-manager version (e.g. v0.4.0): quay.io/jetstack/cert-manager-controller:v0.5.2
Install method (e.g. helm or static manifests): static manifests (via Kubespray addons)

/kind bug

Hi there!
v0.5.2 is a very old version of cert-manager now - nearly a year. Would you be able to upgrade to the v0.10 and come back?
You can follow the docs here.

Please re-open/open a new issue once you've tested this with 0.10 - we've had some changes recently that prevent this sort of thing causing an infinite loop 😄


Cannot write text with '

i trying to translate the file in french but i have a erreur with the word contening this caractere ---> '
exemple: &7Permet d'ouvrir une Mini Box'

solved. remplace ' in start and end to "


Lead Conversion Setting is not honoring naming convention with multiple contacts

Changes

We fixed an issue that caused Lead-converted Account naming not to work as expected when you selectan existing Household Account during conversion. Now, if you select an existing Household Account when converting a Lead, the Account name changes according to EDA Settings.

Issues Closed
[W-9229603]
Testing Notes

QA passes. Confirmed by both Raven and myself.


Incompatibilities with LinkML

Test cases
Linting
$ linkml lint ancient.yml
/home/james/git/mixs-minas/extension-ancient/src/mixs/schema/ancient.yml
  warning  Enum 'DamageTreatmentEnum' does not have recommended slot 'description'  (recommended)
  warning  Enum 'LibTypeEnum' does not have recommended slot 'description'  (recommended)
  warning  Enum 'BioCulturalLabel' does not have recommended slot 'description'  (recommended)
  warning  Enum 'LibStrand' does not have recommended slot 'description'  (recommended)
  warning  Subset 'combination_classes' does not have recommended slot 'description'  (recommended)
  warning  Subset 'sequencing' does not have recommended slot 'description'  (recommended)
  warning  Subset 'environment' does not have recommended slot 'description'  (recommended)
  warning  Subset 'nucleic acid sequence source' does not have recommended slot 'description'  (recommended)
  warning  Subset 'investigation' does not have recommended slot 'description'  (recommended)
  warning  Subset 'ethics' does not have recommended slot 'description'  (recommended)
  warning  Permissible value of Enum 'DamageTreatmentEnum' has name 'partial-udg'  (standard_naming)
  warning  Permissible value of Enum 'DamageTreatmentEnum' has name 'full-udg'  (standard_naming)
  warning  Permissible value of Enum 'BioCulturalLabel' has name 'BC P'  (standard_naming)
  warning  Permissible value of Enum 'BioCulturalLabel' has name 'BC MC'  (standard_naming)
  warning  Permissible value of Enum 'BioCulturalLabel' has name 'BC CL'  (standard_naming)
  warning  Permissible value of Enum 'BioCulturalLabel' has name 'BC CV'  (standard_naming)
  warning  Permissible value of Enum 'BioCulturalLabel' has name 'BC CNV'  (standard_naming)
  warning  Permissible value of Enum 'BioCulturalLabel' has name 'BC R'  (standard_naming)
  warning  Permissible value of Enum 'BioCulturalLabel' has name 'BC CB'  (standard_naming)
  warning  Permissible value of Enum 'BioCulturalLabel' has name 'BC OC'  (standard_naming)
  warning  Permissible value of Enum 'BioCulturalLabel' has name 'BC O'  (standard_naming)
  warning  Permissible value of Enum 'BioCulturalLabel' has name 'BC NC'  (standard_naming)

✖ Found 22 problems in 1 schema

Validating against test data
$ linkml validate -s /home/james/git/mixs-minas/extension-ancient/src/mixs/schema/ancient.yml minas-ancient-test.csv  --target-class Ancient
[ERROR] [minas-ancient-test.csv/0] Additional properties are not allowed ('sample_id' was unexpected) in /
[ERROR] [minas-ancient-test.csv/0] 'Natural History Museum London 2' is not of type 'array', 'null' in /collection_context_name
[ERROR] [minas-ancient-test.csv/0] 'Non-udg' is not of type 'array' in /damage_treatment
[ERROR] [minas-ancient-test.csv/0] '2024-07-01' is not a 'date-time' in /dna_extraction_date
[ERROR] [minas-ancient-test.csv/0] 'Museum of Mongolia' is not of type 'array', 'null' in /ethical_authority
[ERROR] [minas-ancient-test.csv/0] '2024-01-01' is not a 'date-time' in /ethical_date
[ERROR] [minas-ancient-test.csv/0] 'GN12112322' is not of type 'array', 'null' in /ethical_id
[ERROR] [minas-ancient-test.csv/0] 'XYZ' is not of type 'array', 'null' in /experimental_procedures
[ERROR] [minas-ancient-test.csv/0] 12 is not of type 'array', 'null' in /lib_concentration
[ERROR] [minas-ancient-test.csv/0] '10.10023/1203981' is not of type 'array', 'null' in /lib_preparation_protocol
[ERROR] [minas-ancient-test.csv/0] 'FALSE' is not of type 'boolean', 'null' in /neg_cont_status
[ERROR] [minas-ancient-test.csv/0] 12 is not of type 'array', 'null' in /num_capture_amp_cycles
[ERROR] [minas-ancient-test.csv/0] 'formalin' is not of type 'array', 'null' in /preservational_treatment
[ERROR] [minas-ancient-test.csv/0] 1920 is not of type 'string', 'null' in /recovery_date
[ERROR] [minas-ancient-test.csv/0] 'acetone' is not of type 'array', 'null' in /samp_decontam_pretreat
[ERROR] [minas-ancient-test.csv/0] 10900 is not of type 'string' in /sample_age
[ERROR] [minas-ancient-test.csv/0] 'C14' is not of type 'array', 'null' in /sample_age_inference_methods
[ERROR] [minas-ancient-test.csv/0] 'BC MC' is not of type 'array', 'null' in /biocultural_label
[ERROR] [minas-ancient-test.csv/0] 'Microbes only' is not of type 'array', 'null' in /ethics_perm_scope
[ERROR] [minas-ancient-test.csv/0] 'single' is not of type 'array', 'null' in /lib_strandedness
[ERROR] [minas-ancient-test.csv/0] 'Y. pestis' is not of type 'array', 'null' in /probe_set
[ERROR] [minas-ancient-test.csv/0] 'UDI configurs' is not of type 'array', 'null' in /mid_config
[ERROR] [minas-ancient-test.csv/1] Additional properties are not allowed ('sample_id' was unexpected) in /
[ERROR] [minas-ancient-test.csv/1] 'cultural_era' is a required property in /
[ERROR] [minas-ancient-test.csv/1] 'damage_treatment' is a required property in /
[ERROR] [minas-ancient-test.csv/1] 'sample_age' is a required property in /
[ERROR] [minas-ancient-test.csv/2] Additional properties are not allowed ('sample_id' was unexpected) in /
[ERROR] [minas-ancient-test.csv/2] 'cultural_era' is a required property in /
[ERROR] [minas-ancient-test.csv/2] 'damage_treatment' is a required property in /
[ERROR] [minas-ancient-test.csv/2] 'sample_age' is a required property in /

minas-ancient-test.csv

For validation:

multivalued doesn't seem to mean what we think it does

Question to TWG: how closely to linkml do they plan to conform?


Offline Support

Feature request
I am building an application for a user group that spends a majority of their time ouside of a network connection. Having an offline sync or cache would simplify development.
Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Either a recommened approach to caching or storing data offline or providing some magic behind the scenes to sync offline data.
Describe alternatives you've considered
Manually creating a local database copy with Core Data, Swift Data, or Realm Swift.
Additional context

I'm not really sure this library should handle this. (A recommended approach and/or more documentation would be great though.) There could be so many ways to implement a local state, as you mentioned SwiftData, Realm, or others. I am currently building an app that uses SwiftData and Supabase, and while it might be nice to have these wrapped up together, its probably better for the choice to be up to the individual app or team since each team may have different priorities.
I'm curious to discuss it further though, this feels like it would be way down the list on things to tackle and require a lot of thought beforehand to do it right.

Firebase has a feature which you can disable cache on initialization of the sdk. This way if the developer is not interested they can easily turn it off

PowerSync is one option.
https://docs.powersync.com/client-sdk-references/swift

For my app at least, managing the local state while the user is not connected to a network, then eventually syncing with the server — this isn't something I want the Supabase SDK to handle "automatically". It's not easy to assume exactly how each apps needs to handle this scenario.

I've written a few offline sync engines on various projects and I recently wrote a full, offline-first, two-way sync system with this library and GRDB, leveraging the observation tools of each library to also maintain realtime features when a connection is available.
This was all highly specific to the end application's requirements and wouldn't be able to cover every usecase an end user might need from this feature. It would also enforce a certain regitity that supabase currently leaves very fleixble in its data modelling.
If we're instead talking about a simple offline "cache" like what Firebase provides (Firebase doesn't provide offline writes as far as I recall, but I haven't used it in years) then you could argue this library could provide something there but that is a fairly trivial feature to implement.
I'd vote for keeping this out of this library and keeping it laser-focused on what it's currently doing really well.


Added Action and Func Delegate

Code submission for evaluation article "Action and Func delegate in C#"

retest this please

retest this please

retest this please


Improve visualization

Rasa Core version:
0.7.4
Python version:
2.7.14
Operating system (windows, osx, ...):
Debian buster
Issue:
Improve visualization capabilites

add support for newly created slot types
create searchable pdfs (worth if you have really large documents)

Content of domain file (if used & relevant):

What do you mean with add support for newly created slot types?

It doesn't seem to work for custom created slot types. I got errors like this one for example, and from a brief glimpse, I don't think this error is based on the definition of the new class. At least it looks similar to the example in the documentation. Probably should test with a new class, similar to the example in the rasa core docs.

Traceback (most recent call last):
File "/home/michael/.anaconda3/envs/allianz/lib/python2.7/runpy.py", line 174, in _run_module_as_main
"main", fname, loader, pkg_name)
File "/home/michael/.anaconda3/envs/allianz/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "/home/michael/.anaconda3/envs/allianz/lib/python2.7/site-packages/rasa_core/visualize.py", line 48, in
domain = TemplateDomain.load(args.domain)
File "/home/michael/.anaconda3/envs/allianz/lib/python2.7/site-packages/rasa_core/domain.py", line 425, in load
slots = cls.collect_slots(data.get("slots", {}))
File "/home/michael/.anaconda3/envs/allianz/lib/python2.7/site-packages/rasa_core/domain.py", line 465, in collect_slots
slot_class = Slot.resolve_by_type(slot_dict[slot_name].get("type"))
File "/home/michael/.anaconda3/envs/allianz/lib/python2.7/site-packages/rasa_core/slots.py", line 61, in resolve_by_type
"module path is correct: {}.".format(type_name)
ValueError: Failed to find slot type. Neither a known type nor. If you are creating your own slot type, make sure its module path is correct: driver_birthdays.
Makefile:49: recipe for target 'visualize' failed


xen: patch with XSA-464

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

464 is easy to get in without problems but we still have to pull 463 in, as our builder is searching for https://xenbits.xen.org/xsa/xsa${id}.patch but xsa463.patch does not exist that way.
See https://xenbits.xen.org/xsa/advisory-463.html

i'll deal with 463

Sorry, maybe should have merged this while waiting on 463.

@SigmaSquadron Can we make the XSA fetching stuff more normal? It’s one thing to get bikeshedded into all those bells and whistles when they don’t harm anyone, but in this case the mismatch between the expectations of the fancy stuff and the reality of how the patches were released has actively impeded deploying a security patch. (Though apparently the series might not apply cleanly anyway…)

I briefly discussed that concern with Alyssa a while back. Personally I'm okay with just a normal, in-line fetchpatch. It's not like there are too many XSAs applied at any one time anyway.

Also, I was meaning to drop Qubes' build reproducibility patches. They don't work as expected.

Even just fetchurl should be okay, since they’re released canonically as patch files rather than that being a generated forge thing that could change format. (A point Alyssa has made to me.)


Adjust integration tests to wait until Bridge is fully removed

Signed-off-by: Karel Suta<EMAIL_ADDRESS>Insert JIRA Issue number and update link
Please make sure that your PR meets the following requirements:

[ ] Your code is properly formatted according to this configuration
[ ] Your commit messages are clear and reference the JIRA issue e.g: "[MGDOBR-1] - $clear_explanation_of_what_you_did"
[ ] All new functionality is tested
[ ] Pull Request title is properly formatted: MGDOBR-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

How to trigger pipelines and use the bots:

Run the end to end pipeline
Annotate the pull request with the label: safe to test. If you want to run the pipeline again, remove and add it again.

Rebase the pull request
Comment with: /rebase.

Looks like it doesn't help, closing


fix css containing box collision with filament imageEditor

Fix collision with filament imageEditor.
Change how the Backdrop-filter is applied to not create a new containing box.
Before:

After fix:

Thanks for the PR! I have some deadlines now, but will look into it soon!

Released in 1.1.3!


Fix non-delimited library names

These were accepted in https://github.com/alexcrichton/gcc-rs/pull/212, but never actually worked and weren't being tested. This fixes the logic errors that prevented the new style from working, and also updates the tests to use the new style.

Thanks!


allow publishing of text/html emails

This allows publishing of emails with Content-Type: text/html type, by quickly converting them to plaintext. I ran into this issue, when i tried to point my traccar instance's email notifications towards my ntfy instance.
Container image to test ghcr.io/tamcore/ntfy@sha256:caa2c16b9eef9fd6a3d32e9b4621bb32218198fb5931481f3b96ce72819f22b0

While that would be a nice feature to have, I am not willing to pull in a dependency for this, especially not one that only implements 300 lines of code and has 78 stars. But don't bother searching for a more prestigious project because I really don't think bringing in a lib for this is is something I want to do.
Aside from the library issue, there is another one: this now prefers html content if it comes first, which it usually does. So you'd have to do something with ordering.
All in all, unless you find a stdlib way to so this, i don't think I am willing to accept this PR.
Feel free to create a ticket, and if a few people like it, i may reconsider.

Understood. I guess a bit of basic regex'ing to strip the HTML code could be an option? Could also move the text/html condition bellow multipart/, which then should fix the ordering issue.

Re regex: I am not sure how I feel about the regex-based HTML stripping. I suppose since we're not ever evaluating the HTML it's fine, but I don't know what cases it covers. I'm more of a "do it right, or don't do it at all" kinda guy, so not sure.
I think if you can prove to me with some real HTML emails (check your inbox) that your logic will produce good-enough results (in a unit test), I'm gonna accept the PR. Otherwise I think I'd rather not have it. Nobody has asked for this before, so I'm hesitant.
Re ordering: I don't think the re-ordering of the if-statements will achieve the text/plain-affinity that I'd like. If we have a multipart message like this:
<multipart header>
<text/html part>
<text/plain part>

Then we'll still pick the text/html part first.


[cirque] Improvements of chip device ctrl test

Problem
ot-ctl factoryreset returns immediately, but ot-agent takes time to finish factory reset, the test will fail if we started the test too early.
The output of catchsegv is not readable enough.

Summary of Changes
Add a check and wait for thread factoryreset finish
Use gdb to wrap the program.

Fixes #<<<<<FILL ME IN  - issue number(s). If no issue, please create one>>>>>

Is it possible for the thread team to fix ot-ctl ?

@erjiaqing Can you fix the conflicts here? thanks!

Close this PR since the base is reverted, will resubmit a PR later.


add ability to have different image file extensions in a single collection

Add ability to have multiple different file extensions for images in a single collection.
Also moved amount calculation inside the condition in /methods/bundlr.rs to prevent panic, as well it should only need to be calculated when lamports_fee > balance. Was added inside the condition in #420 but then moved out in #421
Also needed to remove (comment) #![allow(clippy::uninlined_format_args)] due to a clippy error. Added in #421.

We should have some checks for what type of extensions are allowed, rather than allowing every extension type. We can have a defined list of what extensions are accepted like {png, jpg, gif, html}. If any one of the extensions does not match with the lsit of extensions then we should not proceed.

100% agree, I was just unaware of what extensions metaplex might want to allow, so left it open, would be pretty easy to add that implementation if they would like.


Allow for multiple sessions per user

Currently user login sessions are stored in the DB in the user itself.  Do JWT instead.

This was fixed by #13.

This was fixed by #13.


Handle error returned by Reset()

Pull Request Test Coverage Report for Build 193

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

Totals

Change from base Build 189:
0.0%

Covered Lines:
494

Relevant Lines:
583

💛  - Coveralls


Nested relationships GenericRelatedObjectManager object is not JSON serializable

I have a nested relationship defined as here: http://www.django-rest-framework.org/api-guide/relations/#nested-relationships
When i use MultipleModelAPIView i'm getting TypeError: GenericRelatedObjectManager object is not JSON serializable, but the same serializers works fine in the generics.ListAPIView, and viewsets.ModelViewSet
Any idea?

Hey @awolfmann: I noticed you closed this pretty soon after opening it.  I just wanted to check to make sure you actually got your problem sorted out!

Hey @MattBroach thanks for writing! Yes, after just some hours i have opened the issue, i fixed the problem, it was my mistake with some of the serializers i used on the view.
Thanks again!

Cool!  Glad you got it sorted out.